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

Cognition difficulty calculation rework: Low AR part #28110

Open
wants to merge 109 commits into
base: master
Choose a base branch
from

Conversation

Givikap120
Copy link
Contributor

@bdach
Copy link
Collaborator

bdach commented May 6, 2024

What is going on with these? Why is there this, and then #28112 and then also #28113? It's not like any of these are marked as dependencies of the other? Is this three versions of the same rework or nothing? Why is none of this detailed in the OP? I only see a vague google doc that mentions these three parts apparently but doesn't expound what they have to do with one another?

@Givikap120
Copy link
Contributor Author

What is going on with these? Why is there this, and then #28112 and then also #28113? It's not like any of these are marked as dependencies of the other? Is this three versions of the same rework or nothing? Why is none of this detailed in the OP? I only see a vague google doc that mentions these three parts apparently but doesn't expound what they have to do with one another?

pp committee asked me to split my rework into 3 logical parts, so it will be easier to understand how it works
you (and peppy) can ignore these PRs for now

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

Successfully merging this pull request may close these issues.

None yet

4 participants