Skip to content
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

zenodo-badge-sync #269

Merged
merged 3 commits into from Oct 8, 2022
Merged

Conversation

p-mishra1
Copy link
Contributor

#268 Added the new badge and changed the font of required line in README to bold.

@p-mishra1
Copy link
Contributor Author

@Saransh-cpp
have a look

Copy link
Collaborator

@Saransh-cpp Saransh-cpp left a comment

Choose a reason for hiding this comment

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

Thanks, @p-mishra1! Looks good! Two minor changes below -

README.md Outdated
@@ -24,7 +24,7 @@ Main features of Vector:
- Pure Python with NumPy as its only dependency. This makes it easier to install.
- Vectors may be represented in a variety of coordinate systems: Cartesian, cylindrical, pseudorapidity, and any combination of these with time or proper time for Lorentz vectors. In all, there are 12 coordinate systems: {_x_-_y_ vs _ρ_-_φ_ in the azimuthal plane} × {_z_ vs _θ_ vs _η_ longitudinally} × {_t_ vs _τ_ temporally}.
- Uses names and conventions set by [ROOT](https://root.cern/)'s [TLorentzVector](https://root.cern.ch/doc/master/classTLorentzVector.html) and [Math::LorentzVector](https://root.cern.ch/doc/master/classROOT_1_1Math_1_1LorentzVector.html), as well as [scikit-hep/math](https://github.com/scikit-hep/scikit-hep/tree/master/skhep/math), [uproot-methods TLorentzVector](https://github.com/scikit-hep/uproot3-methods/blob/master/uproot3_methods/classes/TLorentzVector.py), [henryiii/hepvector](https://github.com/henryiii/hepvector), and [coffea.nanoevents.methods.vector](https://coffeateam.github.io/coffea/modules/coffea.nanoevents.methods.vector.html).
- Implemented on a variety of backends:
- **Implemented on a variety of backends:**
Copy link
Collaborator

Choose a reason for hiding this comment

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

I meant un-bold-ing this on the website, like it is in the README.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

removed bold font in README.md in index.rst the issue was because of previous line "_" changed it.

@@ -133,8 +133,8 @@ The API reference details the functionality of each ``class`` and ``function`` p
:target: https://pypi.org/project/vector/
.. |Conda latest releasetatus| image:: https://img.shields.io/conda/vn/conda-forge/vector.svg
:target: https://github.com/conda-forge/vector-feedstock
.. |DOI| image:: https://zenodo.org/badge/DOI/10.5281/zenodo.5942083.svg
:target: https://doi.org/10.5281/zenodo.5942082
.. |DOI| image:: https://zenodo.org/badge/DOI/10.5281/zenodo.7054478.svg
Copy link
Collaborator

Choose a reason for hiding this comment

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

The badges on the website should be split into 2 lines, just as they are in the README!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Did the splitting.

@codecov-commenter
Copy link

Codecov Report

Base: 81.04% // Head: 81.04% // No change to project coverage 👍

Coverage data is based on head (0da47d8) compared to base (000741c).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #269   +/-   ##
=======================================
  Coverage   81.04%   81.04%           
=======================================
  Files          96       96           
  Lines       10624    10624           
=======================================
  Hits         8610     8610           
  Misses       2014     2014           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@Saransh-cpp Saransh-cpp merged commit 69545c1 into scikit-hep:main Oct 8, 2022
@Saransh-cpp Saransh-cpp linked an issue Oct 10, 2022 that may be closed by this pull request
@Saransh-cpp Saransh-cpp added this to the v1.0.0 milestone Oct 18, 2022
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.

Update the zenodo badge + sync the badges on website and GitHub
3 participants