Skip to content

Fixes for crashes in the app #243

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
May 27, 2025
Merged

Fixes for crashes in the app #243

merged 2 commits into from
May 27, 2025

Conversation

rozyczko
Copy link
Member

@rozyczko rozyczko commented May 16, 2025 •
edited
Loading

  1. Fixed setting Model Index - no layer update is needed. Calling this here caused circular loop of signals layer->model->layer->model->....
  2. Fixed adding new layer. Adding new Material instance was done the "old" way which no longer is implemnted. This was also causing crashes.
  3. Fixed MacOS installer

@rozyczko
fixed adding a new Material
2feabbd 
fixed setting Model Index
@rozyczko rozyczko added [scope] bug Bug report or fix (major.minor.PATCH) [priority] highest Urgent. Needs attention ASAP labels May 16, 2025
@rozyczko
Macos installer fix (#244)
b43e960 
* test with older charset_normalizer

* add other platforms
@rozyczko rozyczko merged commit 522138a into develop May 27, 2025
20 checks passed
rozyczko added a commit that referenced this pull request May 28, 2025
@rozyczko
v1.1.1 release (#245)
28bd546 
* copyright date update
* Fixes for crashes in the app (#243)
* fixed adding a new Material
* fixed setting Model Index

* Macos installer fix (#244)

* Version number update (#246)
@rozyczko rozyczko mentioned this pull request Jun 26, 2025
Merged
rozyczko added a commit that referenced this pull request Jun 26, 2025
@rozyczko
v1.1.1 release (#245) (#251)
a31c1ed 
* copyright date update
* Fixes for crashes in the app (#243)
* fixed adding a new Material
* fixed setting Model Index

* Macos installer fix (#244)

* Version number update (#246)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
[priority] highest Urgent. Needs attention ASAP [scope] bug Bug report or fix (major.minor.PATCH)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@rozyczko