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

[MRG] Updating citation and codemeta #837

Open
wants to merge 11 commits into
base: latest
Choose a base branch
from
Open

Conversation

luizirber
Copy link
Member

@luizirber luizirber commented Jan 9, 2020

A mix of running

and merging the results.

TODO

Checklist

  • Is it mergeable?
  • make test Did it pass the tests?
  • make coverage Is the new code covered?
  • Did it change the command-line interface? Only additions are allowed
    without a major version increment. Changing file formats also requires a
    major version number increment.
  • Was a spellchecker run on the source code and documentation after
    changes were made?

Related PRs and issues:

@codecov
Copy link

codecov bot commented Jan 9, 2020

Codecov Report

Merging #837 (5624174) into latest (9712009) will increase coverage by 4.99%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           latest     #837      +/-   ##
==========================================
+ Coverage   90.20%   95.20%   +4.99%     
==========================================
  Files         126       99      -27     
  Lines       21192    17492    -3700     
  Branches     1594     1594              
==========================================
- Hits        19117    16653    -2464     
+ Misses       1844      608    -1236     
  Partials      231      231              
Flag Coverage Δ
python 95.20% <ø> (ø)
rust ?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/core/src/index/mod.rs
src/core/src/ffi/nodegraph.rs
src/core/src/ffi/cmd/compute.rs
src/core/tests/minhash.rs
src/core/src/sketch/nodegraph.rs
src/core/src/index/bigsi.rs
src/core/src/index/sbt/mhbt.rs
src/core/src/sketch/minhash.rs
src/core/tests/test.rs
src/core/src/index/sbt/mod.rs
... and 17 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9712009...5624174. Read the comment docs.

@luizirber
Copy link
Member Author

luizirber commented Jan 11, 2020

Running git shortlog --summary --numbered (from #622)

$ git shortlog --summary --numbered
   770  C. Titus Brown
   130  Luiz Irber
    34  Laurent Gautier
    17  Tim Head
     7  Connor Tiffany
     7  Olga Botvinnik
     5  Pranathi Vemuri
     5  Taylor Reiter
     3  S. Joshua Swamidass
     3  brooksph
     2  Camille Scott
     2  Daniel Standage
     2  Harriet Alexander
     2  Titus Brown
     1  Andreas Sjödin
     1  Arfon Smith
     1  Connor T. Skennerton
     1  Daniel Dsouza
     1  Katrin Leinweber
     1  NapsterInBlue
     1  Phillip Brooks
     1  Ricky Lim
     1  Tessa Pierce
     1  ljcohen
     1  pyup.io bot
     1  swamidass

@luizirber
Copy link
Member Author

I used the codemeta generator from the Software Heritage project for some cleanups.

@luizirber
Copy link
Member Author

Another interesting tool: sourcecred takes into account all interactions in the repo (issues, comments, PRs...), and there is a GitHub Action in development for running it automatically.

I ran it manually, here's an example screenshot:
image

@luizirber luizirber changed the base branch from master to latest February 15, 2021 22:52
@ctb
Copy link
Contributor

ctb commented May 6, 2021

hello! I decided to try taking over this PR, preparatory to the JOSS paper #1361. Any objections to merging this (after appropriate review) and punting the remaining stuff to a new issue, @luizirber?

I'm not sure about keeping the new github action. Guidance welcome.

@ctb ctb changed the title Updating citation and codemeta [MRG] Updating citation and codemeta May 15, 2021
@SidharajYadav

This comment was marked as spam.

@mr-eyes
Copy link
Member

mr-eyes commented Apr 24, 2022

This PR was left behind for a while, I am just bringing it up for an update or review.

@ctb ctb mentioned this pull request Jan 3, 2023
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4.0 issues to address for a 4.0 release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants