-
-
Notifications
You must be signed in to change notification settings - Fork 939
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
Added additional filament vendors. #4232
Conversation
Added Anker printers to filament presets dialog. Added helper to generate formatted strings for dialog from profile JSON files.
…rs.xyz and major retailer offerings.
3dbc64a
to
10004a8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you mind to add some details to this PR?
Sure, just let me know what details. |
Would be great to use https://docs.python.org/3/library/argparse.html#module-argparse for the script, providing a standarized way to create commandline scripts. |
I would have were there parameters to gather. In this case the script is just a helper to generate the list of names alphabetized and formatted to more easily update the c++ code in the dialog. The python script has the names formatted for easy manual update and generates the list of strings that can be pasted into the dialog without interrupting the code style currently in that file. Was there a specific thing you wanted to have parameterized? |
@SoftFever What details did you want added for this? Thanks. |
Sorry for the late response. A little bit of context for the PR would be helpful, such as details about what problem the PR is aiming to solve or enhance. Screenshots would also be extremely useful. It's not immediately clear to me what a PR is trying to achieve if little background information is provided. |
It seems the PR has gone stale. |
Added Anker printers to filament presets dialog.
Added helper to generate formatted strings for dialog from profile JSON files.
Added PHA material type
Changed to allow all material types when creating custom filaments.