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

Port cpp fixes to1.18 #7654

Merged
merged 13 commits into from
Jul 6, 2021
Merged

Conversation

adrpo
Copy link
Member

@adrpo adrpo commented Jul 5, 2021

Copy fixes from master to 1.18

adrpo and others added 13 commits July 5, 2021 18:28
- installs liblpsolve55 in lib64, fix the copy
- add the path to ffi include when building the Compile runtime to fix:
  ./../../3rdParty/libffi/install/include/ffi.h:58:10: error: 'ffitarget.h'
  file not found with <angled> include; use "quotes" instead
- Flatten types in statements.
- Flatten expressions after evaluating bindings in EvaluateConstants,
  since they come from the instance tree and not the flat model.
- Create typed bindings instead of flat bindings in some places where
  the generated binding might not actually be flat.
- Lift the result in evalComponentBinding if necessary to ensure it has
  the same dimensions as the cref being evaluated.
- Apply the array hack for dimensions in Dimension.fromExp instead of
  during typing to simplify problematic dimensions in more cases.
- Don't double delete components.
* Fix loading of subpackages in case of encryption

* Return the class parts
- Expand reductions/array constructors with multiple iterators into
  nested calls when generating the DAE structure, since the code
  generation can't handle multiple iterators.
- Evaluate min/max reductions if possible.
- Fix Expression.typeOf for relations, to avoid using the wrong default
  value when evaluating reductions.
- Don't expand product of an expression with unknown dimensions.
…odelica#7641)

* Implement product operator in Cpp runtime

* Update OMCompiler/SimulationRuntime/cpp/Core/Math/ArrayOperations.cpp

Co-authored-by: phannebohm <phannebohm@fh-bielefeld.de>

* Update test

* Implement DIV_SCALAR_ARRAY in Cpp runtime

See e.g. PowerSystems.Examples.AC3ph.Precalculation.EqCircFromTransDat

Co-authored-by: phannebohm <phannebohm@fh-bielefeld.de>
See e.g. PowerSystems.Examples.AC3ph.Precalculation.EqCircFromTransDat
…7648)

* Fix Cpp writeLhsCref to not use ArraySliceConst on lhs

Call 'daeExpCref(isLhs=true, ...)' instead of 'daeExpCrefRhs(...)' in scalarLhsCref.

See e.g. PowerSystems.Examples.AC3ph.Precalculation.TransDatFromEqCirc raising:
ERROR  : init  : SimManager: Can't assign array to ArraySliceConst

* Whitespaces
* Fix initialization of reductions in Cpp and C runtime

`foldExpPre` had been declared but not used. This resulted in Cpp errors.

See e.g. ModelicaTest.Fluid.TestComponents.Machines.TestWaterPumpCharacteristics
or PowerSystems.Examples.AC3ph.Precalculation.Z_matrixTrDat3
ERROR  : init  : SimManager: Right and left array must have the same size for element wise addition

* Revert fix of initialization of reductions for C runtime
* Fix empty array slices in Cpp runtime

`stop=0` in `start:step:stop` was used to mark `end` so far.
It is needed for empty slices though.

See e.g. ModelicaTest.Math.TestMatrices2b (MSL 4.0.0) raising
ERROR  : init  : SimManager: Wrong slice exceeding array size

* Add missing access operators for 3-5 dims to ArraySliceConst

* Prevent reduction of ArraySlice for index set of size 1
@adrpo adrpo merged commit e2c1a17 into OpenModelica:maintenance/v1.18 Jul 6, 2021
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.

None yet

4 participants