forked from PlasmaPy/PlasmaPy
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
update master from PlasmaPy master #2
Merged
Merged
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
…e parameter hints of decorated functions
…eck_relativistic` (#619) Presevering function hints when decorated by `check_quantity` and `check_relativistic`
* Add ExB drift * Add force drift * Cleanup code, fixup documentation * Add examples * Pep8 fixes * @namurphy review fixes
* use pprint * Improve plot legends * pep8 fixes I also need to trigger travis
The roadmap has been out of date for a while, and we decided that it would be better to pull it out of the main repository so that it could be updated and versioned separately.
Delete development roadmap
Add file with metadata for Zenodo upload
* Remove obsolete top-level INSTALL.md file We originally kept our installation directions in this file, but have since moved the information to our docs directory. We kept this file in case it was linked to from anywhere else on the World Wide Web, but it has been a long enough time that we can safely remove it. * Update setup.cfg * Update citation information A recommended practice for scientific reproducibility is to cite the specific version of software used for a project, and an established practice is to cite a paper or informational resource that describes it. This commit updates CITATION.rst and the near-identical docs/about/citation.rst to include both of these references. We previously decided to keep both of these files despite the duplication to make it extra apparent how to cite PlasmaPy. I changed plasmapy.__citation__ to be a list of DOI links, including the standard reference from 2018 and version 0.2.0. The metadata for our releases on Zenodo could potentially change (for example, adding the name of a contributor that wasn't known before), so a DOI is the most robust way to go about it. I reserved the DOI for version 0.2.0 on Zenodo, so the links to that will not work yet.
* README * Improved readme * add feedback form link * Add communication info to documentation
Update change log for release of version 0.2.0
* post v0.2.0 cleanup * Update version in docs
Hello @rocco8773! Thanks for submitting your pull request.
PlasmaPy follows the PEP8 style guide, which may be checked locally using pycodestyle |
rocco8773
pushed a commit
that referenced
this pull request
Apr 12, 2024
* Add Z as a kwarg for CustomParticle ...whilst deprecating 'charge' being provided a real number that represents the charge number. * Add CustomParticle.charge_number * Rename test * Remove obsolete comment * Update CustomParticle serialization * Add changelog entry #1 * Add changelog entry #2 * Add changelog entry #3 * Add a deprecation warning * Update type hints * Minor updates * Fix test * Apply suggestions from code review Co-authored-by: Dominik Stańczak-Marikin <stanczakdominik@gmail.com> * Test CustomParticle setters * Test CustomParticle setter for symbol Co-authored-by: Dominik Stańczak-Marikin <stanczakdominik@gmail.com>
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.
No description provided.