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

Query Engine Refactor #165

Open
7 of 16 tasks
JabobKrauskopf opened this issue Aug 14, 2024 · 0 comments · Fixed by #231
Open
7 of 16 tasks

Query Engine Refactor #165

JabobKrauskopf opened this issue Aug 14, 2024 · 0 comments · Fixed by #231
Assignees
Labels
epic Tracks multiple related issues as a high-level task p-high Priority: High python Related to the python medmodels refactor Related to refactoring rust Related to rust medmodels

Comments

@JabobKrauskopf
Copy link
Member

JabobKrauskopf commented Aug 14, 2024

Completely remove the old query engine and write a new one from scratch with added funcitonality.


After the above issues the first draft of the query engine will be merged.


code example 1

a = edge.attribute("amount").min()
b = a.add(5)

After this, a should represent e.g. 5 and b should then represent 10

@JabobKrauskopf JabobKrauskopf added rust Related to rust medmodels python Related to the python medmodels p-high Priority: High refactor Related to refactoring epic Tracks multiple related issues as a high-level task labels Aug 14, 2024
@JabobKrauskopf JabobKrauskopf self-assigned this Aug 14, 2024
@JabobKrauskopf JabobKrauskopf linked a pull request Oct 11, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
epic Tracks multiple related issues as a high-level task p-high Priority: High python Related to the python medmodels refactor Related to refactoring rust Related to rust medmodels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant