search

vverma022 / model-merge

Model-Merge is online platform which combines the power of multiple AI models in one.
6 stars 3 forks source link

Node Modules in Backend #7

Closed vverma022 closed 1 month ago

vverma022 commented 1 month ago 
          Sorry, I wasn't clear about my problem.

The issue that I am facing right now concerns with node_modules not being ignored despite adding node_modules to backend/.gitignore file. As the new files in node_modules are being tracked by git, I can't make a sensible commit.

Originally posted by @NatsuDrag9 in https://github.com/vverma022/model-merge/issues/6#issuecomment-2229964559

NatsuDrag9 commented 1 month ago

Assign this issue to me. I'll resolve it.

NatsuDrag9 commented 1 month ago

A few queries:

Image 1 image

Image 2 image

  1. Is it Groq or Grow in the image 1?
  2. The case of the api keys mentioned in readme (refer image 2) do not match the case of environment variables used in routes/ai_model_name.js. For example, the readme mentions Cohere_API_Key while cohere.js uses COHERE_API_KEY. I've fixed this bug locally on my pc but haven't pushed it yet. If you're ok, I'll push it in the same branch that addresses this issue on node-modules?
  3. groq.js uses LLAMA_API_KEY but the readme file mentions Groq_API_Key. What to do about this?
  4. Do I need to create api keys for other routes that are not mentioned in the readme file?
vverma022 commented 1 month ago

The Readme needs to be edited. Please use the variable names in the code base. I have not updated the readme.

NatsuDrag9 commented 1 month ago

The Readme needs to be edited. Please use the variable names in the code base. I have not updated the readme.

This answers the first three questions. What about question 4? Also, can you share the link to generate the api key for the env variable AI_ML_API_KEY?

vverma022 commented 1 month ago

The Readme needs to be edited. Please use the variable names in the code base. I have not updated the readme.

This answers the first three questions. What about question 4? Also, can you share the link to generate the api key for the env variable AI_ML_API_KEY?

Not integrated in the application as of now. But for reference https://aimlapi.com/app/sign-up/

NatsuDrag9 commented 1 month ago

The Readme needs to be edited. Please use the variable names in the code base. I have not updated the readme.

This answers the first three questions. What about question 4? Also, can you share the link to generate the api key for the env variable AI_ML_API_KEY?

Not integrated in the application as of now. But for reference https://aimlapi.com/app/sign-up/

Understood.

NatsuDrag9 commented 1 month ago

Just to be sure, is this the link to llama api?

vverma022 commented 1 month ago

Just to be sure, is this the link to llama api?

https://groq.com

NatsuDrag9 commented 1 month ago

The package.json in frontend/ does not have the start script. How do I run the frontend of the application?

vverma022 commented 1 month ago

The package.json in frontend/ does not have the start script. How do I run the frontend of the application?

Npm i to install the dependencies And then you can run npm run dev.

NatsuDrag9 commented 1 month ago

I've updated the readme on my branch. Review it

NatsuDrag9 commented 1 month ago

By the way, I fixed this backend/node_modules issue. I just tested the application on my local pc and it is working fine. Shall I merge it into main on my fork and then create a pull request?

vverma022 commented 1 month ago

Please create a pull request.

NatsuDrag9 commented 1 month ago

I created a pull request. Review and let me know regarding any changes.

NatsuDrag9 commented 1 month ago

Since you merged the pull request, I think this issue can be closed now.