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

Project page #84

Merged
merged 4 commits into from
Sep 17, 2024
Merged

Project page #84

merged 4 commits into from
Sep 17, 2024

Conversation

amirrr
Copy link
Collaborator

@amirrr amirrr commented Aug 10, 2024

Upload to S3 bucket and save result in DB

The code changes include adding the `bunnet` dependency to the `requirements.txt` file and updating the `openai` dependency to version 1.40.2. These changes are necessary to support the new functionality introduced in the code.

Recent user commits:
- popup on upload
- multi file upload
- Add a celery task queue to backend (#82)
- Update README.md
- Merge pull request #81 from Watts-Lab/markwhiting-patch-1
- Update README.md
- assign docker command to task definition
- clean up docker compose for development env
- Login window (#76)
- Update README.md
- upload button

Recent repository commits:
- popup on upload
- multi file upload
- Add a celery task queue to backend (#82)
- Update README.md
- Merge pull request #81 from Watts-Lab/markwhiting-patch-1
- Update README.md
- assign docker command to task definition
- clean up docker compose for development env
- Login window (#76)
- Update README.md
- upload button
@amirrr amirrr linked an issue Aug 10, 2024 that may be closed by this pull request
4 tasks
The code changes rename the `Papers` model to `Paper` in the `papers.py` file. This change improves code clarity and follows the singular naming convention for models.
@amirrr amirrr marked this pull request as ready for review September 17, 2024 21:26
Copy link
Member

@markwhiting markwhiting left a comment

Choose a reason for hiding this comment

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

This looks good. I think we will have a lot of tweaks once we play with it more.

@amirrr amirrr merged commit ca88c83 into main Sep 17, 2024
1 check passed
@amirrr amirrr deleted the 79-set-up-the-projects-concept branch September 17, 2024 21:47
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.

Set up the Projects concept
2 participants