-
Notifications
You must be signed in to change notification settings - Fork 9
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
Combo expand #127
base: main
Are you sure you want to change the base?
Combo expand #127
Conversation
The output of changes with the above logic is followed.
|
These drugs are categorized as Stimulants, and show why it is not possible to use that category to apply Amphetamine combo info. EG: Cannabis should not have amphetamine combos.
|
Moon and I had a conversation about this last night and I wanted to forward some discussion points we talked about:
Changing over wildcard and category drugs to specific drugs and giving them default interactions for said wildcard/category.
|
After talking with @TimeX4 they had a suggestion that I thought was fantastic. In regards to the interactions, we could mark every interaction that comes up for Opioids with a warning at the bottom saying something on the lines of "Mixing opioids with other downers is very dangerous!.. etc etc" This would allow us to avoid the issues that may come about from applying all interactions to all drugs in certain classes. (such as amphetamines interactions not being true for all drugs that are considered an amphetamine) I also suggested when we publish the v3 of the chart we can add a option to fill out a form to give any suggestions/changes. Making it very easy for anyone to give suggestions. |
This PR is an example of the changes that would be made to the drugs.json if we were to convert the wildcard and category drugs in combo.json into proper drugs.
There are a couple different ways we do this logic:
Actually, we need to look into the 'nbomh' drugs, as they may match nbome or 2c-b
The following drugs are ignored in this process since they already map to an existing drugs.json entry:
alcohol, amt, caffeine, cannabis, cocaine, dextromethorphan, diphenhydramine, dmt, ketamine, lithium, lsd, mdma, mephedrone, mescaline, mushrooms, mxe, nitrous, pcp, tramadol