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.

Sign up for GitHub

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

Jump to bottom

add the tools tl modules to API agent __init__.py #228

Closed
wants to merge 8 commits into from
Closed

add the tools tl modules to API agent __init__.py #228

wants to merge 8 commits into from

Conversation

bastienchassagnol
Copy link

Branch for building the tools API agent to retrieve scanpy functionalities.

@bastienchassagnol bastienchassagnol added the enhancement New feature or request label Dec 10, 2024
@bastienchassagnol bastienchassagnol self-assigned this Dec 10, 2024
@bastienchassagnol bastienchassagnol changed the base branch from main to biohackathon3 December 10, 2024 12:58
slobentanzer
slobentanzer reviewed Dec 11, 2024
View reviewed changes
Copy link
Contributor

@slobentanzer slobentanzer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good for a start, but doesn't contain any code in scanty_tl.py; let me know once there is an update to review :)

@bastienchassagnol bastienchassagnol marked this pull request as draft December 11, 2024 08:39
@bastienchassagnol bastienchassagnol linked an issue Dec 11, 2024 that may be closed by this pull request
Create API function calling module for scanpy tl #222
Open
@bastienchassagnol bastienchassagnol mentioned this pull request Dec 11, 2024
Open
@bastienchassagnol
Change pymilvus dependency in the pyproject.toml from the fixed versi…
9044563 
…on 2.2.8 to superior or equal to 2.2.8. Indeed, it appears that the grpcio 1.53.0 external dependency of pymilvus version 2.2.8 is not compatible with Windows OS 11 and Python version 2.12.3, whatever it is the wheel or source version. Running pytest does not yield any errors, beyond raising deprecated warnings
@mengerj
api agent for scnapy tl using the generate_pydantic_class_from_module
a46df7a 
method. Currently scanpy is imported when ScanpyTLQueryBuilder.parametrise_query is called.
@mengerj
generic method to generate pydantic classes for functions in a module.
d4f3184 
Only includes functions which dont start with "_"
@bastienchassagnol bastienchassagnol deleted the dev/tl-bastien branch December 11, 2024 12:50
@bastienchassagnol bastienchassagnol restored the dev/tl-bastien branch December 11, 2024 12:52
@bastienchassagnol bastienchassagnol deleted the dev/tl-bastien branch December 11, 2024 12:53
@bastienchassagnol bastienchassagnol restored the dev/tl-bastien branch December 11, 2024 12:53
@bastienchassagnol bastienchassagnol deleted the dev/tl-bastien branch December 11, 2024 12:56
@bastienchassagnol bastienchassagnol removed a link to an issue Dec 11, 2024
Create API function calling module for scanpy tl #222
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@slobentanzer slobentanzer slobentanzer left review comments

Assignees

@bastienchassagnol bastienchassagnol

@mengerj mengerj

@kvitoslava-yarish kvitoslava-yarish

Labels
enhancement New feature or request
Projects
BioCypher Development
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@bastienchassagnol @slobentanzer @mengerj @kvitoslava-yarish