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

Added additional filament vendors. #4232

Closed
wants to merge 3 commits into from

Conversation

anselor
Copy link
Contributor

@anselor anselor commented Feb 25, 2024

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.

Added Anker printers to filament presets dialog.
Added helper to generate formatted strings for dialog from profile JSON files.
@anselor anselor force-pushed the filament_presets_updates branch from 3dbc64a to 10004a8 Compare February 27, 2024 15:17
Copy link
Owner

@SoftFever SoftFever left a 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?

@anselor
Copy link
Contributor Author

anselor commented Mar 1, 2024

Do you mind to add some details to this PR?

Sure, just let me know what details.

@thijstriemstra
Copy link

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.

@anselor
Copy link
Contributor Author

anselor commented Mar 3, 2024

@thijstriemstra

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?

@anselor
Copy link
Contributor Author

anselor commented Mar 7, 2024

@SoftFever What details did you want added for this? Thanks.

@SoftFever
Copy link
Owner

@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.

@SoftFever SoftFever added the wait for response waiting for response from PR creator label Apr 3, 2024
@SoftFever
Copy link
Owner

It seems the PR has gone stale.
I'm closing it for now.
Feel free to reopen it in the future.

@SoftFever SoftFever closed this Apr 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wait for response waiting for response from PR creator
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants