-
Notifications
You must be signed in to change notification settings - Fork 8
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 get_network_actioned_tasks
and get_task_actioned_networks
to AlchemicaleClient
#214
Conversation
* Implemented methods: get_taskhub_actioned_tasks and get_task_actioned_networks * Added tests: test_get_taskhub_actioned_tasks and test_get_task_actioned_networks
* Added get_network_actioned_tasks to client * Added get_network_actioned_tasks endpoint to API * Added tests for Alchemiscale.get_network_actioned_tasks
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good so far @ianmkenney! Thanks for working on this!
alchemiscale/interface/client.py
Outdated
task | ||
ScopedKey of a task which may or may | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For symmetry with get_network_actioned_tasks
, does it make sense to also add a task_weights
kwarg?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
added
get_network_actioned_tasks
and get_task_actioned_networks
to AlchemicaleClient
get_network_actioned_tasks
and get_task_actioned_networks
to AlchemicaleClient
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #214 +/- ##
==========================================
- Coverage 81.96% 81.69% -0.28%
==========================================
Files 22 22
Lines 2828 2873 +45
==========================================
+ Hits 2318 2347 +29
- Misses 510 526 +16 ☔ View full report in Codecov by Sentry. |
…ve weights This shortcuts a lot of unnecessary chatty communication between the API service and the database.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work @ianmkenney! I made some major simplifications to the AlchemiscaleAPI
methods by adding weight
s output to the Neo4jStore
methods they call. Let me know if you are in favor of these changes; if so, we'll merge.
fixes #173
This PR implements methods at the
statestore
,interface/api
, andinterface/client
levels to give users access to Tasks andAlchemicalNetworks
based on their relationship with one another.