Skip to content

Conversation

@krober10nd
Copy link
Collaborator

  • if new mesh doesn't contain blank fort.13 nodal attributes, map_mesh_properties won't map it back over
  • this enables all old attributes to be mapped regardless if the new mesh has blank default values or not
  • probably should do this also for the fort.24?

* if new mesh doesn't contain blank fort.13 nodal attributes, map_mesh_properties won't map it back over
* this enables all old attributes to be mapped regardless if the new mesh has blank default values or not
@WPringle
Copy link
Collaborator

I don't quite understand this.
What is a blank nodal attribute?
The numofnodes may be different between old and new mesh right?

@krober10nd
Copy link
Collaborator Author

krober10nd commented Feb 28, 2021 via email

@WPringle
Copy link
Collaborator

I see, but numofnodes should be updated to length(ind) still.

And in case of you pass 'ind' then new mesh is equal to the old mesh right?

@WPringle
Copy link
Collaborator

obj.f13 = m_old.f13; 
obj.f13.NumOfNodes = m_old.f13.NumOfNodes; 
obj.f13.NumOfNodes = m_old.f13.nAttr; 

should be something like this:

 obj.f13 = m_old.f13; 
 obj.f13.NumOfNodes = length(ind);  

@krober10nd
Copy link
Collaborator Author

I didn’t have any problems with this. Will have to see later when I have time.

I rather have a function to set default attributes for a nodal attribute.

And then leave the migration the way it was

reflecting comments by william.
@krober10nd krober10nd merged commit e5f121c into Projection Feb 28, 2021
@krober10nd krober10nd deleted the krober10nd-patch-3 branch February 28, 2021 23:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants