Skip to content
This repository has been archived by the owner on Mar 1, 2024. It is now read-only.

Loader for Macrometa GDN #484

Merged
merged 16 commits into from
Sep 29, 2023
Merged

Loader for Macrometa GDN #484

merged 16 commits into from
Sep 29, 2023

Conversation

dain-macrometa
Copy link
Contributor

This is a PR requesting merge to include my loader for access to the Macrometa GDN.

https://www.macrometa.com/

Copy link
Collaborator

@jerryjliu jerryjliu 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. could we not check in poetry.lock?

llama_hub/library.json Outdated Show resolved Hide resolved
llama_hub/macrometa_gdn/base.py Outdated Show resolved Hide resolved
llama_hub/library.json Outdated Show resolved Hide resolved
@jerryjliu
Copy link
Collaborator

@dain-macrometa quick bump on this! would love to get this merged

@dain-macrometa
Copy link
Contributor Author

Sorry about the late reply! Is there anything else I'm missing? @jerryjliu

Copy link
Collaborator

@EmanuelCampos EmanuelCampos left a comment

Choose a reason for hiding this comment

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

@dain-macrometa I did one comment, and make sure to remove poetry.lock please

llama_hub/macrometa_gdn/base.py Outdated Show resolved Hide resolved
Copy link
Collaborator

@EmanuelCampos EmanuelCampos left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution

@EmanuelCampos EmanuelCampos merged commit a207c69 into run-llama:main Sep 29, 2023
3 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants