-
Notifications
You must be signed in to change notification settings - Fork 17
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
Performance update #22
Open
klausschuch
wants to merge
84
commits into
main
Choose a base branch
from
performance_update
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
84 commits
Select commit
Hold shift + click to select a range
ef19637
Fix whitespaces
Vaan5 0666de9
Add capacity to ObjectContainer
klausschuch b15a6e8
Use ObjectContainerResize in ObjectContainerPushBack
klausschuch 6cf1173
Use size within ObjectContainer directly
klausschuch ef60f7b
Add ObjectList
klausschuch 03a2379
Use ObjectList instead of ObjectContainer
klausschuch 5ebc0bd
Use size variable
klausschuch 6cd90eb
Cache ConnectionInfo type
klausschuch 8b5cc36
Adapt coding style
klausschuch a80e917
Remove size check in ObjectList/Container Resize
klausschuch 1433851
Fix TypeConversionConvertBoolDouble
klausschuch 9acf45b
time: Add mcx_time_to_micro_s function
klausschuch 7ea9280
Add calcStart/EndWallClockTime to rt factor results
klausschuch 34c8bee
rtFactor: Use a more consistent naming
klausschuch e121150
Add ComponentBeforeDoSteps()
klausschuch fc70701
Use rtGlobalSimStart as reference for rtCalcStart and rtCalcEnd
klausschuch 48c2dbb
component: Restructue ComponentSetupRTFactor
klausschuch dd5da10
component: Reuse rtData->rtTotalSum_s
klausschuch 0cac58a
Set rtData.rtCompStart in ComponentBeforeDoSteps()
klausschuch 4c34594
Component post do-step hook
Vaan5 11052c6
Do not get FMU values in Fmu2DoStep()
klausschuch 5839809
Add retrieval of observables to components (+submodels)
klausschuch 129f788
Rename Monitoring to Observable
klausschuch 53eeda6
Add STORE_MICRO store level
klausschuch f0f3b71
Databus: Track connected inputs
klausschuch a1e57a2
Update connected inputs
klausschuch 4d00f15
Add function to trigger only connected inputs
klausschuch 5653e8c
Trigger only connected inputs in steptypes
klausschuch 0dc4ac7
Component: Remove redundant ComponentUpdateOutChannels() call
klausschuch ca44ee0
Move DatabusUpdateInConnected() call to ModelConnectionsDone()
klausschuch dde5cc1
Model: Improve error handling in ModelConnectionsDone
klausschuch 4fa9c4f
Expose a getter to values of constants
Vaan5 67467e4
Add explicit conversion functions
Vaan5 5153411
Preprocess connections to constant elements
Vaan5 6973a60
ChannelInUpdate: Trigger conversion only if channel is connected
Vaan5 d2c46a8
Apply range/linear conversions immediately when setting a default value
Vaan5 eac058b
Avoid using magic values
Vaan5 48e2282
Group object creation and failure check
Vaan5 83445f5
Remove obsolete comment
klausschuch a8b0321
Avoid cache misses during coupling step mode switches
Vaan5 9e6c479
Avoid cache misses during synch mode switches
Vaan5 bee00ab
Make IntFilter buffer size configurable
Vaan5 f93c77e
Calculate required size of the IntFilter buffer
Vaan5 557c71d
Add buffer limit and 'safety zone'
Vaan5 6f89ec7
Change log levels
Vaan5 24d9626
fmu2: Filter out connected inputs
Vaan5 98be115
Do not set unconnected variables in every DoStep
Vaan5 eba29f2
component: Add SyncStart/EndWallClockTime timing results
klausschuch 381c2d7
component: Measure additional times
klausschuch 639c44a
Do not use uninitialized rtGlobalSimStart
klausschuch e5da00e
Extract common code into a function
Vaan5 15f5444
Enable profiling times with a development flag
Vaan5 b0680ad
Restructure timings
Vaan5 69f1e8e
Avoid premature simulation end in case of only NeverFinishing components
Vaan5 a63defc
Ignore FLUSH_STORE in case of parallel
Vaan5 7ea746b
Change time format in log file header
Vaan5 cae9ce6
Fix const warnings regarding ChannelValue
klausschuch 1630ee5
Convert path to a normalized absolute path prior to _waccess call
Vaan5 118cb96
Implement a memory filter
Vaan5 be4a159
Forward correct argument to CompPostDoUpdateState
Vaan5 0224c67
Use the MemoryFilter when possible
Vaan5 e291799
Add a switch to disable the MemoryFilter on demand
Vaan5 89d3a9c
Check for cycles when determining the buffer size
Vaan5 51d02a2
Use >= to check for buffer overflow
Vaan5 1c95704
Use SIZE_MAX instead of MEM_FILTER_IDX_FLAG
Vaan5 a7631c1
Print user's input as a string instead of the converted int value
Vaan5 80aa0c3
Fix compiler warnings
Vaan5 254d521
Define a Vector of pure C objects
Vaan5 2702ae2
Inline ConnectionInfo into the Connection
Vaan5 a84098a
Remove ConnectionData
Vaan5 836e97d
Vector: Add SetAt method
Vaan5 563936e
Vector: Hide Resize and expose a Reserve method
Vaan5 edd73e6
Inline ChannelInfo into the Channel
Vaan5 8dc290f
Drop ChannelData
Vaan5 5448247
Inline Channel::GetInfo
Vaan5 27a29e0
Vector optimizations
Vaan5 b5c256b
Remove unused variables
Vaan5 f8bb70f
Move variables closer to the place where they are used
Vaan5 5854fa6
Simplify ResultStorage
klausschuch 1ef31f4
Check that the Databus channels were set before trying to update thei…
klausschuch bdb8500
Define a ContainsComponent method
Vaan5 35259ba
Submodel utility to properly check container-like components
Vaan5 0727977
Make sure ComponentStorage->Finished is called for comp_solver compon…
Vaan5 e08d0dd
common_fmu2: Add Fmi2TypeToString() function
klausschuch File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Check failure
Code scanning / CodeQL
Wrong type of arguments to formatting function High