Skip to content

Quick_Start_Examples #13

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Quick_Start_Examples #13

wants to merge 8 commits into from

Conversation

thomascfd
Copy link
Contributor

@thomascfd thomascfd commented Aug 17, 2023

Quick start examples that are relevant to python (no WebUI quick starts). The examples are updated based on recent Flow360Scripts guidelines.

Mesh files same as on documentation page. (apart from XV15 as volume zones still don't work in new python client)

Options for case submission are both from Mesh ID (added to examples in admin portal, but not visible in new python client. To use this, log-in to examples account directly) and upload from S3

XV15 adaptive CFL contains two JSON files as "volume zones" still doesn't work in new python client. We will remove legacy json and replace mesh URL link, once it does

Some pylint issues still exist due to capitalization.


with open("case_name_list.dat", "w", encoding="utf-8") as f:
for line in case_name_list:
f.write(f"{line}\n")
Copy link

@rundaji rundaji Jan 10, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[Optional][TBD] Perhaps we can dump both case name and case id into the dat file? Just in case the new user who's unfamiliar with Flow360 submit multiples cases with the same case name.

I see the logic in download_data.py, I think it's OK to keep the script as it is

However, it seems all 4 quick starts are using the case name to identify the case we'd like to download. Perhaps we can pick one of the quick starts. Dump both case name and case id into the *.dat file and fetch the case according to the case id?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure I like this idea. I would like to keep the case names for folders within a directory and plotting, but I like the idea of still fetching the case using case ID.

axes = []


def plot_sectional_forces(data, plot_name):
Copy link

@rundaji rundaji Jan 15, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would be nice if we can display the sectional loading on a single blade.
Or at least set the xlim = [root,tip]. Currently between [0, root] we're showing the sectional loading from other blades --> kind of confusing for new users

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants