Skip to content
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

Transfers automated conflict detection search choking existing model runtime #444

Open
GiudGiud opened this issue Oct 7, 2024 · 0 comments
Open

Transfers automated conflict detection search choking existing model runtime #444

GiudGiud opened this issue Oct 7, 2024 · 0 comments
Assignees
@GiudGiud
Labels
model_maintenance Removing deprecation, adapting to new techniques for performance or usability

Comments

@GiudGiud
Copy link
Collaborator

GiudGiud commented Oct 7, 2024

Reason

The automated value conflict detection helps find mistakes in models, but it scales like N^2 and it really drives up the runtime of some models. We should use it with parcimony on the vtb as external users might not know to turn it off

Design

As search_value_conflicts = false on transfers, along with the documented data that no conflicts were found

Impact

Faster multiphysics models
@GiudGiud GiudGiud added the model_maintenance Removing deprecation, adapting to new techniques for performance or usability label Oct 7, 2024
@GiudGiud GiudGiud self-assigned this Oct 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@GiudGiud GiudGiud

Labels
model_maintenance Removing deprecation, adapting to new techniques for performance or usability
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@GiudGiud