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.

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

Architecture reversal: read TLM values through the TLM plugin #21

Closed
Vectorial1024 opened this issue Jul 30, 2022 · 2 comments
Closed
Labels
enhancement New feature or request
Milestone

Comments

@Vectorial1024
Copy link
Owner

Because of the nature of C# and their dll, as long as the main mod is updated with a new vesion number (my style), the plugins must also be updated by simply rebuilding it. If you do not rebuild the plugins then the plugins will simply not work because the required assembly could not be found.

With #20, it is becoming more feasible to just integrate the TLM plug-in back into the main mod. However, it is not trivial because of the complexity of the TLM code involved.

@Vectorial1024 Vectorial1024 added enhancement New feature or request focus A focus for the short term: is WIP or being planned and removed focus A focus for the short term: is WIP or being planned labels Jul 30, 2022
@Vectorial1024
Copy link
Owner Author

It suddenly occured to me, that I can reverse the architecture and achieve something very similar to what I plan to achieve.

Instead of having a TLM plugin that modifies the behavior of the main mod, I can tell the main mod to check the status of the TLM mod through the TLM plugin. This will allow simpler reflection than the monster reflection needed for the TLM plugin to check the TLM mod itself if I keep going in the same direction, and this also allows me to not bump-update the TLM plugin every now and then.

@Vectorial1024 Vectorial1024 changed the title Integrate TLM Plug-In into the main mod Architecture reversal: read TLM values through the TLM plugin Aug 6, 2022
@Vectorial1024 Vectorial1024 added this to the 202208-01 milestone Aug 7, 2022
@Vectorial1024
Copy link
Owner Author

This is working fine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant