-
Notifications
You must be signed in to change notification settings - Fork 24
Integrate simulation in other projects #612
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
base: future
Are you sure you want to change the base?
Conversation
@@ -7,7 +7,7 @@ | |||
using System; | |||
using System.Collections.Generic; | |||
|
|||
namespace Moryx.Simulation |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would you be so kind as to put an entry into the migration guide for this package change? 😬
using Moryx.Simulation.Examples; | ||
using Moryx.Resources.Benchmarking; | ||
|
||
namespace Moryx.Simulation.Tests |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you want to merge the test projects as well? 🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you intend to move this to Moryx.Resources.Benchmarking
? It is duplicated now if I'm not mistaken 😅
using Moryx.ControlSystem.Cells; | ||
using Moryx.Resources.Benchmarking.Messages; | ||
|
||
namespace Moryx.Resources.Benchmarking |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you add this from scratch? Maybe you could extend the PR message just a bit, cause I don't know what to make of this addition 🙈
I would like to reduce the number of projects and packages.