Skip to content
This repository has been archived by the owner on Oct 27, 2021. It is now read-only.

added fmuResourceLocation to Instance Structure #9

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Jiter
Copy link

@Jiter Jiter commented May 5, 2021

To work with the resources folder in the model the temporary resource location is needed.
I added it to the modelInstance Struct and copy it during initialization.

Copy link
Contributor

@adriantirea adriantirea left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pay attention to fmuResourceFile encoded string. It will be received as encoded URI. To use it you need to decode it. It will be the responsibility of the FMU to decode it

You need more code to be consistent. Follow the example of instanceName which is a string exactly like fmuResourceFile.

  • check that allocation succeeded. See line 172.
  • you need to free the memory in fmi2FreeInstance.

Jiter and others added 2 commits June 8, 2021 14:25
- checked that allocation succeeded
- free memory when freeing instance
@Jiter
Copy link
Author

Jiter commented Jun 8, 2021

Thank you.
I incorporated the things you mentioned.
Hopefully it will be helpful for you.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants