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

PR Batch 5 (WIP) #1162

Closed
1 task done
nponeccop opened this issue Apr 13, 2023 · 13 comments
Closed
1 task done

PR Batch 5 (WIP) #1162

nponeccop opened this issue Apr 13, 2023 · 13 comments

Comments

@nponeccop
Copy link
Contributor

nponeccop commented Apr 13, 2023

Duplicates

  • I have searched the existing issues

Summary 💡

So here is another FIFO batch.

Examples 🌈

Broken by the module introduction but fixed next day

Abandoned by 23:00 UTC

Abandoned next day

Conflicts from Earlier Batches

Motivation 🔦

I changed the way I compile PR lists, so while the issue is empty the list of issues in the batch is kept elsewhere. I will publish it shortly before the batch is frozen so that we can see the remaining ones.

@nponeccop nponeccop mentioned this issue Apr 13, 2023
1 task
@Artemonim
Copy link
Contributor

Guys, be careful, please, when you review and merge PRs.
You merged #1397, but didn't check the necessary changes that are now waiting for separate PRs: #1420, #1419, #1405.
Many people in our Discord (including me) don't now how to fix this now.

@Dr33dM3
Copy link
Contributor

Dr33dM3 commented Apr 15, 2023

@Artemonim you are supposed to download the stable release the master branch is usually broken and for development purposes only.

@Slowly-Grokking
Copy link
Contributor

#992 is data_ingestion.py is broken now

File "D:\Auto-GPT\autogpt\data_ingestion.py", line 4, in
from autogpt.config import Config
ModuleNotFoundError: No module named 'autogpt'

@bszollosinagy
Copy link
Contributor

@Artemonim i try to fix most of it in PR #1534 Take a look if you want.

@dhruv-programmes
Copy link

Wdym by "Broken by the module introduction" ?

@nponeccop
Copy link
Contributor Author

nponeccop commented Apr 15, 2023

A major refactoring by the core team created new conflicts in several PRs

@nponeccop
Copy link
Contributor Author

@Slowly-Grokking File a separate issue

@Enigma94
Copy link

@nponeccop Thanks for taking a look and trying to organize in this project, however there are obvious issues with the core teams approach to this open-source project, and until issues are resolved i dont see any point in contributing

@don4of4
Copy link

don4of4 commented Apr 16, 2023

@Enigma94 Perhaps instead of (or in addition to) engaging in such a negative comment you could suggest what you think could be done to better manage the project. Obviously, this repo has blown up beyond anyone's reasonable expectations.

@nponeccop How can we help the core team with this huge backlog?

@nponeccop
Copy link
Contributor Author

@don4of4 Let's discuss the help at #1870

@ForestLinSen
Copy link

Hi @nponeccop , I've resolved the conflicts and reopened pull request #1170 could you please change the status of that pull request?

@nponeccop
Copy link
Contributor Author

@Enigma94 I understand that you have concerns about the core team's approach to this open-source project, and we appreciate your feedback. It would be really helpful if you could share a list of specific issues you've encountered, so that we can work together to address them and improve the project. Constructive criticism and open dialogue are essential for a thriving open-source community, and we value your input.

@ntindle
Copy link
Member

ntindle commented Apr 23, 2023

Closing as out of date

@ntindle ntindle closed this as completed Apr 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests