Sketchup wants to break my model

I have a 2 models that appear OK using Solid Inspector & SU solid full 2.2.
Every time I open them in SU it asks me if I want SU to “fix” them.
If I tell it to go ahead, then chk it again with the above there r problems with them.
So why is SU wanting to “fix” them but breaking them. these r the 2 files.

SU Make 15.3.331 x64

https://drive.google.com/open?id=0BxMubV-sPLfWdW5MT2xZU3lkd2c
https://drive.google.com/open?id=0BxMubV-sPLfWTzk4WTUyV3lfQk0

You need to provide permission for anyone with the link to be able to open the file. You can’t get help unless this is corrected first.

1 Like

So you think SU has it in for you?
Check and see how you have check for model problems in windows, preferences, general set.
Just a guess

1 Like

Well, OK, that keeps SU from checking the model.
Since it wants to break it when it does, that is good!
The question still remain, why does it want to break a perfectly good model?

The more pressing question is how does anybody get to see this “broken” model to which you refer? You brought this up, but have failed to demonstrate the condition. As I indicated in my first reply to you, we can’t begin to offer any help if you don’t allow access to the models. You apparently have uploaded two different files to Google Drive, but you do not give permission to download them.

The things that are checked to decide whether a Group or Component is a solid are not the same as the things that are checked to decide whether a model needs to be “fixed”. It is quite possible to build a model in which the solid testers are happy but there are other issues that SketchUp’s test will think should be fixed. It is also quite possible that SketchUp’s automated fix will damage some aspect of your model, such as causing a Group to fail the solid test. Usually this means it wasn’t drawn well in the first place.

As @jvleearchitects has noted twice, we can’t tell you what is the specific problem unless you grant permission to download your models.

1 Like

I was gmailed permission and had a look at one of the files…
Bill,
It was already a solid, but when I ran ‘Fix Problems’ it returned this list…

Results of Validity Check.

CLoop (46039) for CFace (46038) has zero area
Loop 0 of CFace (46038) is an invalid outer loop
Entity CFace (46038) should be erased - done
CLoop (46127) for CFace (46126) has zero area
Loop 0 of CFace (46126) is an invalid outer loop
Entity CFace (46126) should be erased - done
CLoop (46482) for CFace (46481) has zero area
Loop 0 of CFace (46481) is an invalid outer loop
Entity CFace (46481) should be erased - done
CLoop (49332) for CFace (49331) has zero area
Loop 0 of CFace (49331) is an invalid outer loop
Entity CFace (49331) should be erased - done
CLoop (49433) for CFace (49432) has zero area
Loop 0 of CFace (49432) is an invalid outer loop
Entity CFace (49432) should be erased - done
CLoop (56845) for CFace (56844) has zero area
Loop 0 of CFace (56844) is an invalid outer loop
Entity CFace (56844) should be erased - done
CLoop (57514) for CFace (57513) has zero area
Loop 0 of CFace (57513) is an invalid outer loop
Entity CFace (57513) should be erased - done
CLoop (59644) for CFace (59643) has zero area
Loop 0 of CFace (59643) is an invalid outer loop
Entity CFace (59643) should be erased - done
CLoop (59893) for CFace (59892) has zero area
Loop 0 of CFace (59892) is an invalid outer loop
Entity CFace (59892) should be erased - done
CLoop (59925) for CFace (59924) has zero area
Loop 0 of CFace (59924) is an invalid outer loop
Entity CFace (59924) should be erased - done
CLoop (59992) for CFace (59991) has zero area
Loop 0 of CFace (59991) is an invalid outer loop
Entity CFace (59991) should be erased - done
CLoop (60096) for CFace (60095) has zero area
Loop 0 of CFace (60095) is an invalid outer loop
Entity CFace (60095) should be erased - done
CLoop (60498) for CFace (60497) has zero area
Loop 0 of CFace (60497) is an invalid outer loop
Entity CFace (60497) should be erased - done
CLoop (66859) for CFace (66858) has zero area
Loop 0 of CFace (66858) is an invalid outer loop
Entity CFace (66858) should be erased - done
CLoop (68928) for CFace (68927) has zero area
Loop 0 of CFace (68927) is an invalid outer loop
Entity CFace (68927) should be erased - done
CLoop (69223) for CFace (69222) has zero area
Loop 0 of CFace (69222) is an invalid outer loop
Entity CFace (69222) should be erased - done
CLoop (69528) for CFace (69527) has zero area
Loop 0 of CFace (69527) is an invalid outer loop
Entity CFace (69527) should be erased - done
CLoop (69563) for CFace (69562) has zero area
Loop 0 of CFace (69562) is an invalid outer loop
Entity CFace (69562) should be erased - done
CLoop (69573) for CFace (69572) has zero area
Loop 0 of CFace (69572) is an invalid outer loop
Entity CFace (69572) should be erased - done
CLoop (69620) for CFace (69619) has zero area
Loop 0 of CFace (69619) is an invalid outer loop
Entity CFace (69619) should be erased - done
CLoop (69670) for CFace (69669) has zero area
Loop 0 of CFace (69669) is an invalid outer loop
Entity CFace (69669) should be erased - done
CLoop (69705) for CFace (69704) has zero area
Loop 0 of CFace (69704) is an invalid outer loop
Entity CFace (69704) should be erased - done
CLoop (69720) for CFace (69719) has zero area
Loop 0 of CFace (69719) is an invalid outer loop
Entity CFace (69719) should be erased - done
CLoop (69730) for CFace (69729) has zero area
Loop 0 of CFace (69729) is an invalid outer loop
Entity CFace (69729) should be erased - done
CLoop (69736) for CFace (69735) has zero area
Loop 0 of CFace (69735) is an invalid outer loop
Entity CFace (69735) should be erased - done
CLoop (69741) for CFace (69740) has zero area
Loop 0 of CFace (69740) is an invalid outer loop
Entity CFace (69740) should be erased - done
CLoop (69790) for CFace (69789) has zero area
Loop 0 of CFace (69789) is an invalid outer loop
Entity CFace (69789) should be erased - done
CLoop (78042) for CFace (78041) has zero area
Loop 0 of CFace (78041) is an invalid outer loop
Entity CFace (78041) should be erased - done
CLoop (81003) for CFace (81002) has zero area
Loop 0 of CFace (81002) is an invalid outer loop
Entity CFace (81002) should be erased - done
CLoop (85850) for CFace (85849) has zero area
Loop 0 of CFace (85849) is an invalid outer loop
Entity CFace (85849) should be erased - done
CLoop (85938) for CFace (85937) has zero area
Loop 0 of CFace (85937) is an invalid outer loop
Entity CFace (85937) should be erased - done
CLoop (86485) for CFace (86484) has zero area
Loop 0 of CFace (86484) is an invalid outer loop
Entity CFace (86484) should be erased - done
CLoop (90182) for CFace (90181) has zero area
Loop 0 of CFace (90181) is an invalid outer loop
Entity CFace (90181) should be erased - done
CLoop (90369) for CFace (90368) has zero area
Loop 0 of CFace (90368) is an invalid outer loop
Entity CFace (90368) should be erased - done
CLoop (93237) for CFace (93236) has zero area
Loop 0 of CFace (93236) is an invalid outer loop
Entity CFace (93236) should be erased - done
CLoop (102300) for CFace (102299) has zero area
Loop 0 of CFace (102299) is an invalid outer loop
Entity CFace (102299) should be erased - done
CLoop (104079) for CFace (104078) has zero area
Loop 0 of CFace (104078) is an invalid outer loop
Entity CFace (104078) should be erased - done
CLoop (115233) for CFace (115232) has zero area
Loop 0 of CFace (115232) is an invalid outer loop
Entity CFace (115232) should be erased - done
CLoop (115241) for CFace (115240) has zero area
Loop 0 of CFace (115240) is an invalid outer loop
Entity CFace (115240) should be erased - done
CLoop (123928) for CFace (123927) has zero area
Loop 0 of CFace (123927) is an invalid outer loop
Entity CFace (123927) should be erased - done
CLoop (124131) for CFace (124130) has zero area
Loop 0 of CFace (124130) is an invalid outer loop
Entity CFace (124130) should be erased - done
CLoop (124213) for CFace (124212) has zero area
Loop 0 of CFace (124212) is an invalid outer loop
Entity CFace (124212) should be erased - done
CLoop (129074) for CFace (129073) has zero area
Loop 0 of CFace (129073) is an invalid outer loop
Entity CFace (129073) should be erased - done
CLoop (129980) for CFace (129979) has zero area
Loop 0 of CFace (129979) is an invalid outer loop
Entity CFace (129979) should be erased - done

it appears one of your repair plugins is adding a lot edge loops then not completely removing them…
I doubt it’s TT’s one…

john…

@john_drivenupthewall, how large is the model? The long list of faces with no area suggests to me that the model contains tiny features that are running afoul of SketchUp’s lower size tolerance.

@slbaumgartner,
Steve,
I scaled it up, saved before I ran that check, it was solid before and after…

it’s only 3.6MB [now] and very cleanly modelled…

if @ruggb [ Bill ] attached it directly it would be easier to discuss…

john

I don’t know how closely you watch postings but it’ s not unusual for ThomThom’s solid inspector, Tig’s solid solver to not be able to fix models and if the program has the option to attempt repair it may break the model. Many times when trying to help with SU problems I will run separate different programs ( repair filters) attempting fixes and it is not unusual to find unreferenced vertices, zero area faces, self intersecting faces in a SU. If one attempts to fix those the sequence repairs are attempted can influence the success of repair or not.
The fact SU’s own check tool is flagging problems should raise red flags and give you incentive make models available. Also you should also indicate where the models came from. All sites are not equal. :smiling_imp:

I am sorry – I just have not figured this GDrive sharing thing out yet.
try these links

https://drive.google.com/file/d/0BxMubV-sPLfWTzk4WTUyV3lfQk0/view?usp=sharing
https://drive.google.com/file/d/0BxMubV-sPLfWdW5MT2xZU3lkd2c/view?usp=sharing

I imported the model from a dnld .stl file at meters. Then scaled it back down and saved the parts in individual files.
@slbaumgartner thought may be correct based on that.
Many of the .stl files I dnld r so complex that I must import at meters to avoid being none manifold.
But if the file is manifold is SU’s issue with small pieces something that can be fixed or must I just work around it. The workaround is just not to let SU touch it if the other tests say it is good.
I can put up the original file if that would help - but it contains 4 parts

I couldn’t directly attach those files as they r just past the upload size limit. I tried!

Actually, I guess if that could be fixed I wouldn’t have to import at meters. It must be connected to the same issue. which by past questions leads me to believe it is not a fixable issue.
It was just aggravating till I found out how to turn it off.

Bill,
do a fresh import in a new model using meters, then do the standard ‘Fix Problems’ and see if it reports the same as I got…

if it does it may be the importer settings…

john

The shroud1 model is typical of ones you can obtain form sites that have different working criteria than what is expected of clen su models, two many polys , what appear to be lots of auto fold edges, what I would call a dirty model. No wonder the problems you are having.
After removing 18,026 duplicate vertices I found no unreferenced vertices, no null faces , no self intersecting faces and no none manifold edges. Running the solid inspector shows a number of stray lines which were removed. The model still has areas it cannot fix but running SU native fix problems tool no longer reports any issues.
My recommendation: you need to be more selective in the models you use if you do not what to spent time fixing them. :smiling_imp:
Here is shroud_mac1bb.skp (2.6 MB)

Do you have the file name and url where located??

I went back to the site. RE downloaded the file for the 6th time. Opened Su and imported it at meters with merge coplanar faces checked, Checked them - all good. Selected each piece and scaled them back to mm. Checked them again. turned on the SU check and saved them. The only thing it asked me is if I wanted to save them with a non-default scale (no clue what it is talking about).

No errors this time!
I went back and did it again without merge coplanar faces and got no errors that way either.

I hate ghost problems.

I have no idea what I could have done differently the last 5 times that cause the problem.
sorry to bother u.
I hope I won’t be back for this issue again…

the only other variable is that there r 2 copies of this file on the site. I can’t remember which one I have been downloading, I think I will try the other now since I remember which one I got this time.

OK - I am going to consider that the answer—I dnld the other file and it has issues. Now to tell the author to remove that file.
thanks for ur help guys

The second model you posted in the above is not much better than the first one , after removing 62,576 duplicate vertices from it it has 2 more vertices and faces vs the first one. The model was decimated to get the poly counts down with out making it too lumpy. solid inspector then reported one area problem which after repair makes mode solid.
I still have to complain about your lack of cooperation of supplying info when trying to help you. It would have made the process mnuch quicker. Here is solid model shroud2_mac1cc.skp (591.2 KB)

Could you provide a link to the .stl file, or attach the .stl file itself?

on this page

look for fan shroud-40mm.stl
I informed the author that the other file was bad and he removed it.
This one is the last one I used and it seems to have no issues
— except needing to import in meters. But to avoid errors I now import everything in meters.
thx

Down loaded stl and imported in SU. Noted: left side of screen shows repairing at import and then cleaning. What is conclusion.? => Original authors model is modified before you even open in SU??
Then stl opens in SU all four components, 2 bell cranks and two shrouds each show solid , but the vol. reports crank to crank and shroud to shroud are not same but relative small difference.