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

Add first release of graphite database #40

Merged
merged 6 commits into from
Jul 11, 2024
Merged

Add first release of graphite database #40

merged 6 commits into from
Jul 11, 2024

Conversation

trappitsch
Copy link
Collaborator

This PR does not just add the first graphite database on Zenodo, but also extends all the tooling in order to accept graphite databases in teh future!

  • update bib file with graphite references
  • extend reference appending capability and add graphite refs to database
  • refractor and update techniques files by appending from graphite db excel
  • extend db addition to graphites and add first graphite release to database
  • update database with graphites and fix tests

@codecov-commenter
Copy link

codecov-commenter commented Jul 11, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.37%. Comparing base (d67db3b) to head (7a5870b).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #40      +/-   ##
==========================================
+ Coverage   83.36%   84.37%   +1.01%     
==========================================
  Files          11       11              
  Lines         571      608      +37     
==========================================
+ Hits          476      513      +37     
  Misses         95       95              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@pep8speaks
Copy link

Hello @trappitsch! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 9:80: E501 line too long (82 > 79 characters)

Line 31:80: E501 line too long (81 > 79 characters)
Line 32:80: E501 line too long (86 > 79 characters)
Line 49:80: E501 line too long (87 > 79 characters)
Line 153:80: E501 line too long (82 > 79 characters)
Line 155:80: E501 line too long (80 > 79 characters)
Line 158:80: E501 line too long (86 > 79 characters)
Line 202:80: E501 line too long (83 > 79 characters)
Line 204:80: E501 line too long (80 > 79 characters)
Line 208:80: E501 line too long (81 > 79 characters)
Line 246:80: E501 line too long (84 > 79 characters)
Line 251:80: E501 line too long (80 > 79 characters)
Line 262:80: E501 line too long (84 > 79 characters)
Line 267:80: E501 line too long (82 > 79 characters)

Line 91:80: E501 line too long (81 > 79 characters)
Line 116:80: E501 line too long (82 > 79 characters)
Line 175:80: E501 line too long (84 > 79 characters)

@trappitsch trappitsch merged commit f5fcff0 into main Jul 11, 2024
6 checks passed
@trappitsch trappitsch deleted the graphite_db branch July 11, 2024 09:36
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.

3 participants