-
Notifications
You must be signed in to change notification settings - Fork 25
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
Removes blocking behavior when trident is not configured #142
Open
chummels
wants to merge
25
commits into
trident-project:main
Choose a base branch
from
chummels:install
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
25 commits
Select commit
Hold shift + click to select a range
faa031f
Updating front page of docs.
chummels 0ed785c
Changing install docs.
chummels d76daf0
Rewriting parse_config to prevent it from expecting user input.
chummels 018e9e2
Removing special behavior for READTHEDOCS since code will no longer b…
chummels d51d4bd
Changing create_conf to configure
chummels a395687
Parsing config only when necessary for function to run for ion_balance.
chummels bf6c619
Parsing config only when necessary for function to run for ray_genera…
chummels 68a8a63
Parsing config only when necessary for function to run for spectrum_g…
chummels fa4dff3
Removing check_import() because evidently the astropy bug that made t…
chummels 6beb171
Making parse_config give verbose error when first time checked.
chummels 6b1512f
Changing subsequent errors to use mylog.
chummels 5b20325
Changing configure to be auto_config function.
chummels e257928
Adding trident exceptions
chummels 0fb29bf
Using exceptions instead of mylog for errors.
chummels b5906ca
Removing ionization_table kwarg from ray_generation because not actua…
chummels b274247
Removing unused import
chummels cb239df
Updating use of exceptions
chummels da7d699
Removing excess whitespace for flake
chummels df49d60
Merge branch 'master' into install
chummels e0170cc
Adding some more text during auto_config.
chummels b7db7aa
Adding text to auto_config.
chummels 47b5365
Fixing test failure.
chummels faaa977
Updating installation docs to address PR comments. Removing demesh refs.
chummels ca58836
Updating install instructions to address new auto_config()
chummels 95d3365
deprecating keyword.
chummels File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Since yt is a hard dependency for trident, it should be possible to just to
pip install trident
to get stable releases of both codes. It might be useful to add a suggestion to try that before installing yt and trident separately.