-
Notifications
You must be signed in to change notification settings - Fork 0
Add requested 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
Conversation
examples/data_selector/view.py
Outdated
| # circumstances. | ||
| NeutronDataSelector( | ||
| v_model="data.selected_files", | ||
| base_paths=["/HFIR", "/SNS"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would specify something that you know will exist, such as $HOME and /tmp.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wait... is this an example of DataSelector or NeutronDataSelector? You should have one of each.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll create one just for the DataSelector and set some paths as you mentioned.
README.md
Outdated
| 4. [Conditionally disabling elements](examples/conditional_disabling) | ||
| 5. [Changing Pydantic rules based on user input](examples/dynamic_pydantic_rules) | ||
| 6. [Complex Pydantic rules](examples/complex_pydantic_rules) | ||
| 7. [Using datafiles from the Analysis Cluster](examples/data_selector) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we move this to a separate ORNL Only section?, ditto ONCat.
README.md
Outdated
| 6. [Complex Pydantic rules](examples/complex_pydantic_rules) | ||
| 7. [Using datafiles from the Analysis Cluster](examples/data_selector) | ||
| 8. [Selecting datafiles from ONCat](examples/oncat) | ||
| 9. [Running an NDIP tool](examples/run_ndip_tool) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please change NDIP to Galaxy
| with vuetify.VCard(classes="mx-auto my-4 pa-1", max_width=600): | ||
| with HBoxLayout(classes="mb-2", gap="0.5em", valign="center"): | ||
| InputField(v_model="data.input_path") | ||
| with vuetify.VBtn( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why VBtn?
|
@jarrah42 I've updated the PR. Created an ORNL-specific subdirectory for examples and added an example for the plain DataSelector. Let me know if you want to see any other changes! |
jarrah42
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thanks.
Summary of Changes
Adds the requested examples from https://code.ornl.gov/groups/ndip/-/epics/252.
Checklist
Documentation Updates
Additional Notes
Closes #7
Closes #8