The Altair Community is migrating to a new platform to provide a better experience for you. In preparation for the migration, the Altair Community is on read-only mode from October 28 - November 6, 2024. Technical support via cases will continue to work as is. For any urgent requests from Students/Faculty members, please submit the form linked here
"OperatorSelector does not work with Macro Definition"
Hello
Using OperatorSelector with MacroDefiniton I got an NumberFormatError. Diving into the code I found that you have fixed the problem of Integer-Parameters-do-not-use-macros, but not problem of the MacroHandler itself: (i.e. it also occurs in the Zaniah Branch)
Since OperatorSelector try to access the macro BEFORE the process is executed, it gets an error since this macros have not been defined yet. At this step, MacroHandler has only access to predefined macros, the Map "macroMap" is empty. I choose not to add a note in the bugtracker, because this is a quite architectural (does this word exist) problem which may affect other operators, too.
affected methods of OperatorSelector
regards,
Steffen
Using OperatorSelector with MacroDefiniton I got an NumberFormatError. Diving into the code I found that you have fixed the problem of Integer-Parameters-do-not-use-macros, but not problem of the MacroHandler itself: (i.e. it also occurs in the Zaniah Branch)
Since OperatorSelector try to access the macro BEFORE the process is executed, it gets an error since this macros have not been defined yet. At this step, MacroHandler has only access to predefined macros, the Map "macroMap" is empty. I choose not to add a note in the bugtracker, because this is a quite architectural (does this word exist) problem which may affect other operators, too.
affected methods of OperatorSelector
- getInputClasses()
- getOutputClasses()
- getInnerOperatorCondition()
regards,
Steffen
0
Answers
thank you for this detailed description. It made it easy for me to identify the problem in the code and at the first glance it feels like there should be a clean solution for this. But since, as you said, this is quite architectural, I will discuss it with the other deveolpers.
By the way: Zaniah has now become the stable branch.
Greetings,
Sebastian
Thanks,
Keith
this issue is fixed now (and should also have been part of the latest EE update).
Cheers,
Ingo