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

API Zoo Data Generator #229

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

Amokhalad
Copy link
Contributor

This project provides a web application for fetching API calls from a API documentation urls into a specified format (Option 1), to easily generate training points to train a fine-tuned LLM model in performing API calls.

Copy link
Owner

@ShishirPatil ShishirPatil 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 this PR @Amokhalad ! We were long due on enabling this. One question - does this deserve it's own repository or should it be a sub-directory here? Thoughts?

react-data-converter/README.md Outdated Show resolved Hide resolved
@@ -0,0 +1,13 @@
# Client Ignores
Copy link
Owner

Choose a reason for hiding this comment

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

I was wondering if we want to move this into a sub-directory inside API-Zoo? Thoughts?

@@ -0,0 +1,13 @@
# Client Ignores
Copy link
Owner

Choose a reason for hiding this comment

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

Why do we have a .gitignore inside the global .gitignore? Can we migrate this into the global .gitignore?

@@ -0,0 +1,5990 @@
{
Copy link
Owner

Choose a reason for hiding this comment

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

Do we need package.json as well ?

Copy link
Contributor Author

@Amokhalad Amokhalad Mar 15, 2024

Choose a reason for hiding this comment

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

I think both package.json and package-lock.json are needed.

  • package.json holds what dependencies to be installed. Also, currently defines a proxy for the backend, and a custom script to run the webpage on port 3000.
  • package-lock.json holds the versions of each package, so no dependency issues occur if another person wants to contribute to the webpage

react-data-converter/client/public/next.svg Outdated Show resolved Hide resolved
react-data-converter/client/public/vercel.svg Outdated Show resolved Hide resolved
@Amokhalad
Copy link
Contributor Author

Thanks for this PR @Amokhalad ! We were long due on enabling this. One question - does this deserve it's own repository or should it be a sub-directory here? Thoughts?

What about adding this as a sub-directory within the Gorilla website's codebase, and making the webpage accessible through a navbar link on the API-Zoo-Index page? like this

Screenshot 2024-03-15 at 11 29 20 AM

@eitanturok
Copy link
Contributor

Hello.

I was just curious what the status of this PR is.

Thanks!

@Amokhalad
Copy link
Contributor Author

Hello.

I was just curious what the status of this PR is.

Thanks!

Hey I made a new PR here: link

Currently it's under review and hopefully will be merged to gh-pages branch soon

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants