-
Notifications
You must be signed in to change notification settings - Fork 13
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
ODA Upgrade #698
Comments
Edit: 2nd Edit: |
I actually managed to compile it with the original Boost pre-builts, 142 toolchain, and forced C++ 14. Still need to try this with the new Teigha version, but we might just get this working without throwing too much over. |
@FJThiel 😢 I have up to VS2022 installed and I think we're all using the same prebuilt libraries, so as long as you can get a version of VC to work with the whole thing and redistribute any libraries that has been updated, it's all good. We don't use windows in production so it's mainly just an issue between you me and @sebjf That's assuming we are not running into similar issues with linux... 😆 |
Okay, in that case I would like to go all the way up to VS22's toolchain (VC17, msvc 143) so we don't need to do this again for a few more years. |
…they crash the compiler of the new toolchain. Will be removed anyway by Sebastian in the near future.
Changes noticed so far:
|
When comparing models in the viewer with their representation in Revit, with some models I see colours being off and/or transparency and textures getting lost. Is this just how things are with some files or something that I should dig deeper into? |
Do you mean comparing to uploading a previous revit version or comparing to opening the model in Revit If it's the former probably should have a quick sanity check, if it's the latter then as long as it's consistent with the previous revit version upload then it's probably out of scope |
For the moment I was just comparing them with Revit. They are the Revit sample files because I figured if something has the new 2025 features in it that could cause problems, it would be them. The downside would be that I have no earlier upload to compare to. Or so I thought. Just saw that Autodesk offers earlier versions of some on their webpage, so I am going to use them for comparison now! |
Comparison with 2024 samples did not show any visual difference for the Revit files. Only thing of note: Both, old and new bouncer throw a warning for some models: |
Also leaving this here in the unlikely case that it becomes an issue in the future: For the new version, the test with the sampleHouse.rvt failed because instead of 1341 mesh nodes, there are only 1339 returned. Close inspection of the mesh nodes produced by new and old bouncer revealed that these missing nodes belong to a group of four mesh nodes with the group name "Pad 2_490319". The new bouncer produced only two of them, but these two are larger. Summed together, the number of faces is identical and the number of vertices is down by 12. It was also found that these were not the only nodes affected. All in all, 44 groups were found where the number of faces has been altered. Given that there is no visual apparent change, this is most likely just a minor change how the ODA libraries process the mesh before handing it to us. |
Because of the behaviour described in the previous comment, a test in the AssetGenerator project needs to be altered. The change is part of #5 of that project and will need to merged in with the PR of this issue. |
No visual difference with Navisworks files either. |
DWG and DGN look the same as well. |
Description
The bouncer is currently running on Teigha_2024.8, which prevents files from the 2025 versions of the autodesk tools to be imported. This necessitates an update of the ODA version. It is expected that some things might break in the process that will need to be fixed.
Goals
Tasks
Related Resources
Product Ticket: #384
The text was updated successfully, but these errors were encountered: