forked from tardis-sn/tardis
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Oldphysics docs #3
Closed
Closed
Conversation
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
…multi-line # to be docstrings where appropriate
Fixed the last file for tardis-sn#680
Fixed two random indentation irregularities found upon review
* fixing issue tardis-sn#1177 * fixing notebook headers so they don't show up in toc * fixing toctree for docs/running/index * adding notebooks to toctree * fixing notebook headings to not show up in toctree
* Inside of tardis/docs/pphysics/plasma/plasma_plots updated two files multi-line # to be docstrings where appropriate * Some more documentation updates * Finished issue tardis-sn#680 * Fixed the last file for tardis-sn#680 * Fixed two random indentation irregularities found upon review * Update base.py * Update base.py * Update packet_source.py Going through and writing documentation guidelines, noticed that the carriage returns at line 450 and 456 were missing somehow, most likely from me missing them. I added those two carriage returns to make them consistent with the rest of the docstrings.
fixes for ci docs
Old physics reorganization
Modified grammar, fixed a type, and made the docstring example comply with the guidelines stated below.
Editted the grammar
Adding the typo changes that I forgot to in the first commit
Added possessiveness
…ion guidelines in it Restructured development workflow and added code quality & documentation guidelines in it
* add docstr-coverage workflow minor changes run on commited files only run on commited files only (2) run on commited files only (3) run on commited files only (4) run on commited files only (5) [skip ci] fail if coverage is lower than previous one fix typo make job conditional make job conditional (2) test update job test update job (2) [skip-ci] test update job (3) [skip-ci] test update job (4) [skip-ci] test update job (5) [skip-ci] test update job (6) [skip-ci] test update job (7) [skip-ci] round value store commit sha [skip-ci] store commit sha (2) [skip-ci] try simpler approach [skip-ci] try simpler approach (2) [skip-ci] try simpler approach (3) [skip-ci] try simpler approach (4) [skip-ci] try simpler approach (5) [skip-ci] try simpler approach (6) [skip-ci] use docstr-coverage latest release * update .docstr.yaml * remove line (workflow will fail) * blame files on pr
…rdis-sn#1424) * Renamed Kromer Plot to SED Plot throughout the module * Rename kromer_plot file name to sed_plot * Changed SED to SEDec * Added subpackage docstring
* Update documentation-build.yml * Removed random sn * Update documentation-build.yml * Update CHANGELOG.md [skip ci] * Update CHANGELOG.md [skip ci] * Update CHANGELOG.md [skip ci] * Hotfix to remove bad tick marks * Undo edits from master * Delete CHANGELOG.md * Fixing changelog differences * Fixing changelog differences from tardis * Trying to undo changelog updates Co-authored-by: TARDIS bot <[email protected]>
* initial commit * change upstream url * remove GIT_LFS_SKIP_SMUDGE=1 * update docs * variables in yaml * fix typos * fix typos
…n#1441) * Renamed SEDec to SDEC throughout the module * Changed module name and in init file
…#1442) * Added lum_to_flux divison in element hist calculations * Raise error if distance <= 0 * changed unit order for the flux axis to be erg/s/cm2/AA * Instructed how to use luminosity case in error message
Co-authored-by: antreev-brar <[email protected]>
…1438) * Restructured gui index to incoprorte viz tools * Added notebook to demo SEDec plot * Added notebook demonstrating SEDec plot * Renamed SEDec to SDEC * Added distance option & additional options usage * Changed ref link name in index * Added gif to demo notebook * Increase matplotlib plot size * Fixed inconsistency in docstring * Made grammatical edits in notebook & hid warnings * Fixed top level heading from ## to #
* Fixed italicised Units in sdec_plot * Fixed Italics Luminosity & Flux Lambda representation * Reverted back to old notation
Reproduced SDEC plots in demo notebook
… gui to visualization (tardis-sn#1458) * Renamed gui to visualization * Added functionality to create redirects in sphinx conf.py * Fixed errors and documented code
* reorganizing files * fixing a few warnings * moving some files around * merging models/examples/examples, models/examples/index, and models/examples/modelconfig into models/index * deleting models/csy/csv_test.ipynb * changing header
Now returns the proper pandas series that are required for the cython formal integral wrapper (tardis-sn#1464)
…within it (tardis-sn#1462) * Reorganized widgets & tools modules * Changed import paths as per new subpackage structure * Updated imports in demo notebooks * Added EOF newline to init files
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Description
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: