A variety of fixes and features #795
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The GridFinity Refined base adds various features to GridFinity, but remains compatible with GridFinity. Notable features: - dovetail tabs to join panel - screw holes to affix bins to a panel - holes for magnets in the base
…e them for cutting on standard plywood panels
The photoframe module sets up a logger (this is okay) but then proceeds to set the loggers level and output handler. This causes photoframe to emit logging statements to the console (i.e. visible when boxes_main is run with --examples) and makes it difficult for upstream code to turn off undesired logging.
The original boxes_main would handle args via sys.argv, which limits functionality (for example adding a --debug flag that enables the logging module at the debug level). Full backwards compatibility is maintained using argparse.REMAINDER. This also then supports the convention of using '--' to separate boxes_main args and generator args when there is ambiguity
…lities The current test setup can only verify that generators default output doesn't change; this change uses a YAML file to automate generation of examples and supports making custom examples with custom arguments. This is also reusable to automate creation of multiple boxes. For example, you are generating a set of boxes for a silverware drawer and you can write a single YAML file and then share that file with others to generate all the boxes.
A common use case (at least for me) is that I want to make multiple cuts that won't fit on a single standard 12"x12" panel. I don't like having to manually arrange pieces to fit on these panels, so this script automates that action.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR has a mixure of fixes and features - and I appreciate it will be a lot to review. Each commit should be relatively isolated, so I'm happy to extract them into standalone PRs if desired so that easy merges can happen sooner.
The commit messages have more deatils, but the notable changes are:
examples.ymlwhich allows generating examples that don't use default args with the--examplesargumenttest_svg.pyalso usesexamples.ymlso it can test generators without default arguments--multi-generatorto generate multiple boxes in a repeatable and shareable fashion. A few of the YAML files I used for my projects are attached as examples.gridfinity_refined_bases.txt
silverware_drawer.txt