Project improvements and VERTEX code migration#2
Merged
Conversation
sr-murthy
force-pushed
the
project-improvements
branch
2 times, most recently
from
0db6e16 to
e110dff
Compare
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
sr-murthy
force-pushed
the
project-improvements
branch
2 times, most recently
from
e618c05 to
e8331cd
Compare
Welcome to Codecov 🎉Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests. ℹ️ You can also turn on project coverage checks and project coverage reporting on Pull Request comment Thanks for integrating Codecov - We've got you covered ☂️ |
sr-murthy
force-pushed
the
project-improvements
branch
from
e8331cd to
6619703
Compare
sr-murthy
force-pushed
the
project-improvements
branch
11 times, most recently
from
e101159 to
2faf023
Compare
![github-code-quality[bot]](https://avatars.githubusercontent.com/u/9919?s=60&v=4){kind=small}
sr-murthy
force-pushed
the
project-improvements
branch
from
a8aceb5 to
8659f5f
Compare
sr-murthy
force-pushed
the
project-improvements
branch
from
8659f5f to
a8f57a4
Compare
sr-murthy
changed the title
sr-murthy
requested review from
EstebanGarciaG,
ISARICData and
alasdairwilson
and removed request for
ISARICData
sr-murthy
force-pushed
the
project-improvements
branch
2 times, most recently
from
f3e0e96 to
eec3e98
Compare
Member
alasdairwilson
left a comment
alasdairwilson
left a comment
There was a problem hiding this comment.
I think we should keep the structure::
If i was the one that was trying to redistribute the functions from the veretx IA/ID files then I would rather have categories to insert into, this means you can do it in a much more iterative way without reverting early changes. rather than have to try and group the functions together adhoc as you go, having to rework early categorisations as you make later iterative changes. To me, useful work was done and it is just gonna need done again.
I am not really invested either way though, so you can keep the change if you want.
sr-murthy
force-pushed
the
project-improvements
branch
2 times, most recently
from
9dc6141 to
bae65f0
Compare
sr-murthy
force-pushed
the
project-improvements
branch
2 times, most recently
from
9cb3dc8 to
3e0eb5a
Compare
chore: update README with status badges test: add Codecov YML chore: clean up deps in project TOML docs: Set up Sphinx docs style: lint docs: clean up conf ci: clean up tests step in test workflow to use PDM + remove the install package step chore: add security policy chore: add keywords to project TOML refactor: copy analytics and visualisation libraries (and the tests) from VERTEX + update project TOML with Plotly dep chore: remove commented out code from analytics and draw chore: delete all unusued libraries (including tests) and assets from the repo chore(deps): add pylock TOML Various project improvements chore: add missing `getREDCapData` lib. + `logging` subpackage from VERTEX VERTEX code migration for REDCap data retrieval, analytics and visualisation libraries chore(deps): add `requests` to deps Project improvements and VERTEX code migration chore: remove commented out code in `getREDCapData` Project improvements and VERTEX code migration chore: add license badge to README Project improvements and VERTEX code migration
sr-murthy
force-pushed
the
project-improvements
branch
from
3e0eb5a to
5be28a1
Compare
sr-murthy
changed the title
EstebanGarciaG
approved these changes
Apr 10, 2026
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
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.
vertex.logging) REDCap data retrievel (vertex.getREDCapData.py), analytics (vertex.IsaricAnalytics.py) and visualisation (vertex.IsaricDraw.py) libraries into the repo - ✅