Open ClassyTeddy opened 3 years ago
This is great @ClassyTeddy and @nailsonlandim, thanks for putting it together!
See Sarah's email that I've forwarded to the group with her comments from testing. Was going to cut and paste here, but it looks like the formatting is lost.
Default cleat location in drawer stacks/hampers should be "Below" in certain situations #287 All tests appeared to show this working, eventually. But when I brought in a counter after drawers, it did not update automatically. It took me bringing in a toe kick for the update to happen.
Shelf stack numbers do not update immediately when you change the number of shelves in the prompt. But, as I continue to work in the file they update. I’m not sure if there is a pattern as to when the updates occur, but in one instance I increased the depth of the partitions. When I did that the shelf stack number increased at the same time.
These two issues from @sarahhightower's email seem to be relatively the same issue. For whatever reason, it appears like her instance that she was working in didn't update the scene right away when changes were made/when things were added. She had to change another thing for it to actually update the scene. I am unsure as to why as I cannot replicate this issue.
Fillers: Option to edge bottom edge #272: Did not work for me .
This is working as intended for me, both visually and in the export. When @sarahhightower does return, I will have to see what went wrong with what she was working with.
I received error messages when I attempted to generate 2D views so I could not test any of those Github issues. Tyler, I posted in the group chat about that so I’m hoping to get help and be able to test this Friday, as I’m in training all day tomorrow.
I did see this error message as well, and pushed a fix for what is hopefully the only problem with 2D views generation.
I do have to ask @JJanneh if he added the lines of code of 3041-3099 in the data_closet_carcass.py
file, as I do believe the code that was causing the error may be something you copied when adding filler setback, without realizing the intended purpose of what you copied. Let me know if the code looks familiar when you can.
I'm seeing the same issue with changing the shelf stack number of shelves; number only changes in scene after changing something else afterward.
Also seeing that the hutch back is extending out to the ends of fillers instead of nestling between the outer partitions.
I'm also not seeing the option to edge bottom of fillers, and the capping fillers are not edge banded in my screen.
I get this error when trying to load snap defaults now. I don't think it did so this morning when I loaded:
@ClassyTeddy Thanks for all the help. amount changed to .value thanks for update on the code in the data_closet_carcass.py file line 3052 x_placement = self.product.obj_bp.location.x + l_thickness.value() + left_filler_setback_amount() changed----> x_placement = self.product.obj_bp.location.x + l_thickness.value() + left_filler_setback_amount.value()
@JJanneh I already fixed the issue. I told you that. The line in question has nothing to do with the prompts menu. Please tell me why you copied lines 3041-3099.
@ClassyTeddy
J that has to do with a problem with what you put in the drivers. That is about 2000 lines up in a different class from what I am talking about.
@ClassyTeddy This was a sample from the left_filler.value() #LEFT FILLER above lines 2980-3028
So then yeah, it should be removed. You copied over code that needed to look at the width for the plan view, and wrote extra code that instead of using the value of the width of the filler, it used the filler's setback value.
I get this error when trying to load snap defaults now. I don't think it did so this morning when I loaded:
I just pushed a CCItems change that should hopefully solve this issue. I am unsure if this was causing other issues as well.
Just tested Fillers: Option to edge bottom edge #272: issue. The options show up and are working
Also seeing that the hutch back is extending out to the ends of fillers instead of nestling between the outer partitions.
A fix for this is ready for testing on the 1.2.5-Merge-Testing
branch.
After Testing Update L/R Capping filler, L/R EdgeBottom, Add Hutch Backing, Hamper and Drawer Stacks and Drawer and TopShelf Image below shows working.
I can't select other colors besides Oxford white for 1mm edge, drawer face, or general materials. You can see the slim black box trying to open, but there's nothing there. The countertop, stain color selections, etc look to be there, however.
when I click to render selected scenes in 2D view walls disappear and an error comes up.
Here's what I see after I save the 2D view and the pdf pops up. THe PDFs aren't the design I made (they're somewhat close, but have extra openings and items running of in various directions. I've added the PDF here as well. I don't think I can manipulate the design anymore at the point. The walls turn orange when selected and I can't get the full room/other walls to show up.
I can't select other colors besides Oxford white for 1mm edge, drawer face, or general materials. You can see the slim black box trying to open, but there's nothing there. The countertop, stain color selections, etc look to be there, however.
It is an issue with CCItems.csv
. I have a fix for this on 2.0.
~This is likely causing the other issues you are seeing.~ Let me apply the fix to the 1.2.5-Merge-Testing
branch.
edit: Still seeing the 2D view issues above after fixing csv import.
I am seeing this error on startup.
Are you seeing the same if you open the console window?
I guess I'll chime in on this one....I'm pretty sure the new csv file has ALL 1mm Tape removed from it now. This is a change that is happing at our production side, where everything that was label "1mm" is now "1 mm Dolce". Brock can verify all of this, but I do know that those "1mm" entries have all been removed.
Fix for importing CCItems.csv
is available on branch: 1.2.5-Merge-Testing
.
Hi @TylerStandage, can you share the file involved in the 2D View/PDF Issue?
Another thing to note here:
Whenever there is a change to CCItems.csv
, the "Load SNaP Defaults" command needs to run to rebuild the database and reflect those changes in the UI.
This happens automatically for a user when an auto-update happens. So this is nothing a user should have to do.
Because devs/testers are effectively using Git in place of the auto-updater this does not happen automatically.
Hi @TylerStandage
Just fixed the errors for 2D views. remind to clean the existing 2D views and generate them again.
OK, thanks for all the work and updates today! I've just completed another round of testing and most of these are in place as far as I can tell. The issues that I have not passed off as of yet (Nailson, I think the errors I was getting earlier are user-error):
Thanks and talk to you all tomorrow!
I'm able to remove L/R panels and make top or bottom shelves non-KD
I will look into why the L/R panels are removable. I tried to make the top/bottom KD forced as locked, but I couldn't find a way. But, I just had an idea for it that I'm going to try as well.
@ClassyTeddy can you confirm the drilling is correct?
As far as I can tell from measurements and the debug view, it is correct.
324: I believe that the image below shows a situation where the height of the pard on the right should be moved. @nailsonlandim let me know if I'm missing something on this:
IM Errors
You're correct @TylerStandage, I've pushed fixes for that and for the errors you mentioned to me on IM.
Question on issue #304 pertaining to drawer box depths for lateral file rails. The drawer prompt/options works well for this. In my 2D there is no indication that the drawer box should be 17" with the 19" depth partitions. Not sure if that's just an export notation or if this should be on the 2D as well?
A fix for issue #310 is ready for testing on the 1.2.5-Merge-Testing
branch. Let me know if this solution will work @TylerStandage
I just pushed a fix for #233 to the 1.2.5-Merge-Testing branch. The double swing door icon should be drawing correctly now.
Hi @TylerStandage, @sarahhightower, I have now pushed a fix for #267 to the 1.2.5-Merge-Testing branch. The extraneous ceiling height dimensions should no longer appear.
Great thanks Antonio! Have a great weekend
On Fri, Apr 16, 2021 at 4:57 PM AntonioGloria @.***> wrote:
Hi @TylerStandage https://github.com/TylerStandage, @sarahhightower https://github.com/sarahhightower, I have now pushed a fix for #267 https://github.com/CCDevNelson/Snap-Updates/issues/267 to the 1.2.5-Merge-Testing branch. The extraneous ceiling height dimensions should no longer appear.
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/CCDevNelson/Snap-Updates/issues/330#issuecomment-821728145, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASMXVVUKS7PSLU7XRH6WET3TJDFHXANCNFSM4257M35Q .
On #267, I'll let Sarah test and comment on that one. Also, I'm seeing that #262 is in place as well. Sarah, do you agree we can check that one off?
@nailsonlandim @AntonioGloria @ClassyTeddy, let me know what you see as still needing to finish up before we can get this version out the door. There are a few boxes not checked yet, and I know Nailson has the accordion views to add in as well (without dimensions included).
Hi,
Added Drawer box depths for lateral file rails on 2D views just like the example below.
If you have any other questions about it just let me know.
I found a bug involving the Veneer Material Type in the Closet Materials
tab. Selecting the veneer material caused nothing else to be drawn in the tab due to an error occurring where no material was found for veneer. I found that the issue was because there were name differences between our .blend files and the display names in the database (there was an extra space). I solved the issue and it is ready for testing on the 1.2.5-Merge-Testing
branch.
I just pushed a fix for the dashed lines showing in the drawers. This was related to the labeling changes made in #295. This fix should also make the text of some dimensions look black again instead of like an outline.
233 looks good to me too, Antonio. I see double doors swings now. @sarahhightower is that what you see/expected?
Yes, looks great!
On #267, I'll let Sarah test and comment on that one.
I don't notice this issue in my testing. Looks good!
Also, I'm seeing that #262 is in place as well. Sarah, do you agree we can check that one off?
Almost. The only thing I still see missing from this one is the labeling of exposed ends. The prompt has the option to note when there are exposed ends and it should function just like the top shelf does when there are exposed ends. This is with a dash to the letters "exp". Here's what it looks like now:
Hi @TylerStandage, @sarahhightower, I have pushed the fix for the capping bottom exposed edge labels.
Top Shelf should now once again automatically cover filler, and the filler should now disappear and have it's values set to 0 when unselected.
Added Accordion Views.
I've added a checkbox on Closet 2D Setup, so the user can choose if they will be generated, and the PDF options are changed. by default the checkbox is unmarked.
Hi, @TylerStandage and @sarahhightower,
Just pushed a fix for toe kicks
So if there is a Corner Shef / L-Shelf it gets the correct measure, and if there is more than one TK, it will place it with an offset, without repeating them.
A fix that removed the Shelves Only
library item is available and ready for testing on the 1.2.5-Merge-Testing
branch.
I have pushed a fix for the bolding effect appearing in elevations to the 1.2.5-Merge-Testing
branch.
For situations where the additional pards appears it can be solved by cleaning orphan data from the blender file. to do it follow these steps:
Go to the corner with the three angled lines, click and drag it
It will appear another canvas, go to the small select box and choose the Outliner option
Go to the select box that says 'All Scenes' and choose 'Orphan Data'
And click Purge Data
It will show a confirmation message, click it and wait a while
After it you can close the outliner by joining the area, do it with the mouse right button at the line between the 3d canvas and the outliner, click join area
Now an arrow will appear, move the mouse right and click it, now the outline will disappear
Thanks @nailsonlandim, we'll keep this in our back pocket for an answer to questions that come up in this regard.
EVERYONE, thank you so much for all the testing, updates, new coding, trouble shooting, brain storming, and re-testing/testing one more time that you have all put in over the past couple of weeks! From what I can tell it looks like this is ready to be sent over to Ryan to get ready for release later this evening. I'll give another 30 minutes for anyone to make a comment, find a bug, raise an issue, etc... .If we don't have any objections by 2:05pm AZ time, @ClassyTeddy let's get the pull request prepared and sent over to Ryan!
This issue thread will be a place for everyone to test and share any findings for all the commits made since release 1.2.4. The branch to be testing on will be
1.2.5-Merge-Testing
The following are a list of issues that should have fixes available for them on this branch.
2D Views Changes
Corner and L Shelf Changes
Filler Changes
Misc. Changes
If I could get some thorough testing on the
1.2.5-Merge-Testing
branch from @TylerStandage, @sarahhightower, @kategodfrey, and @SeanRadvan when you guys have the time that would be great. Feel free to check the box next to issues you think are good to go.Also, if anyone notices that I missed an issue that should be in this list please let me know. I will make sure that it is merged and add it to this list.