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
Added list collections api #154
Conversation
…ethods in base class for the same, updated deepdoctection library and modified dockerfile for pytorch
@@ -201,6 +201,10 @@ def get_collections(self) -> List[Collection]: | |||
collection = Collection.parse_obj(self._get_entity_from_run(run=ml_run)) | |||
collections.append(self._polulate_collection(collection)) | |||
logger.debug(f"[Metadata Store] Listed {len(collections)} collections") | |||
end = time.time() | |||
print( |
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.
should we remove this now?
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.
Done.
"type": run_params.get("data_source_fqn").split("::")[0], | ||
"uri": run_params.get("data_source_fqn").split("::")[1], | ||
"fqn": run_params.get("data_source_fqn"), | ||
"metadata": run_params.get("metadata", {}), |
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.
this metadata field will never be presend, so you can remove that
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.
done.
@@ -9,13 +9,11 @@ | |||
|
|||
|
|||
@router.get("/") | |||
def list_data_source(): | |||
async def list_data_source(): |
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.
you will need complete details api too
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.
So, better expose this as new api
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.
Done.
backend/server/routers/collection.py
Outdated
@@ -35,6 +35,16 @@ def get_collections(): | |||
raise HTTPException(status_code=500, detail=str(exp)) | |||
|
|||
|
|||
@router.get("/list-collections") |
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.
@router.get("/list-collections") | |
@router.get("/list") |
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.
done.
No description provided.