Skip to content

adds citation button#169

Merged
jonasteuwen merged 1 commit intoNKI-AI:mainfrom
wdika:add-citation-button
Jan 10, 2022
Merged

adds citation button#169
jonasteuwen merged 1 commit intoNKI-AI:mainfrom
wdika:add-citation-button

Conversation

@wdika
Copy link
Copy Markdown
Contributor

@wdika wdika commented Jan 10, 2022

This PR is a feature request, adding the 'Cite this repository' GitHub button through the CITATION.cff file.

@codecov
Copy link
Copy Markdown

codecov bot commented Jan 10, 2022

Codecov Report

Merging #169 (9170f6c) into main (811b0d7) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #169   +/-   ##
=======================================
  Coverage   74.39%   74.39%           
=======================================
  Files          70       70           
  Lines        6300     6300           
=======================================
  Hits         4687     4687           
  Misses       1613     1613           

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 811b0d7...9170f6c. Read the comment docs.

@wdika wdika marked this pull request as ready for review January 10, 2022 11:53
@jonasteuwen jonasteuwen merged commit 43543b5 into NKI-AI:main Jan 10, 2022
@wdika wdika deleted the add-citation-button branch January 10, 2022 12:14
jonasteuwen added a commit that referenced this pull request Feb 10, 2022
* Improve code quality (#158, #166)
* Reduce pylint warnings
* Reduce pyflakes warnings
* Add codacy code quality badge
* Improving code quality by renaming variables
* Bump version: 1.0.0 → 1.0.1-dev0
* Adds citation button (#169)

Co-authored-by: wdika <dimitriskarkalousos@gmail.com>

* Update model_zoo.rst

* Fix linter errors (#170)

* Fix some linter warnings
* Add .prospector.yml
* Update docs/index.rst

* Delete README.md

* Update README.rst

* Merged MRI models engine to MRIModelEngine

* Minor fix

* Func to get size of objiect

* - Fix typo
- Refactor to have functions not class members

* - Remove process_output from base class

* First attempt to rewriting validation process (will not pass CI)

* Rewrite validation process (will not pass CI)

* Improve validation

* Improve validation

* Log metric output

* Improve metric log output

* Replace ISMRMD with xml parser

* Make sure the metric is cloned

* Refactoring

* Refactoring

* Refactoring

* Fix roll test

* Remove `is_validation_process`

* Move ismrmrd dependency to dev dependencies

* Install extras for tox

* Make compatible tests for new updates

* Minor fix

* Fix pytest warning

* Fix pylint errors

* Update black.yml

* Update black.yml

* Update black.yml

* Black 22.1.0 fixes

* Black 22.1.0 fixes

* Updated docs, fixed `write_output_to_h5` to work with new predict function

* Update `utils`

* Add test

* Update rim_engine.py

* Minor fix

* Disable tensorboard warnings

* Remove whitespace

* Update train.py

* Update varnet_engine.py

* Test engine.train

* Test engine.train

* Refine test

* Fix typo and unused import

* Add crop option for inference

* Codacy pseudonumber complains

* Update setup.py

`h5py 3.6.0` caused memory leak

* Minor documentation fix

* Update writers.py

* Update mri_models.py

* Edit `set_all_seeds`

* Update subsample.py

* Minor fix

* Fix/supress codacy warnings

* Fix/supress codacy warnings

* Fix/supress codacy warnings

* Lazy formatting

* Lazy formatting

Co-authored-by: Jonas Teuwen <2347927+jonasteuwen@users.noreply.github.com>
Co-authored-by: Dimitris Karkalousos <d.karkalousos@amsterdamumc.nl>
Co-authored-by: wdika <dimitriskarkalousos@gmail.com>
Co-authored-by: Jonas Teuwen <j.teuwen@nki.nl>
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.

2 participants