You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I noticed that the Makefile for OpenAD experiments does not make sense. After generating it with genmake2 and running make adAll once, one cannot run make adAll again, because of the linked files xaif.xsd, xaif_base.xsd, xaif_inlinable_intrinsics.xsd, xaif_derivative_propagator.xsd, xaif_output.xsd. After removing them, make adAll, really re-does everything, which is not in the spirit of "make".
Typically I found that I needed to do make CLEAN each time I want to try something, which makes the process very slow and difficult. Also, it is not possible to run testreport -q with this type of makefile. Sometimes, I would like to do that, however, e.g. when I fiddle with the transformed code (i.e. I don't want modified code to be overwritten), or when I just want change experiments parameters that do not require re-generating and recompiling the code. With the current makefile, I have to wait O(10min) each time I make a tiny change.
I suggest that we change the dependency on the *.xsd files, e.g. we could add them into the make adDepend step, if that is used at all. Obviously, I am not very good with makefiles, so I would be happy for other suggestions.
The text was updated successfully, but these errors were encountered:
this is something i've found too. my hacky solution, if i wanted to add debugging print commands to OpenAD-generated code (which i need to do from time to time) to was simply to compile the affected files manually and then do the final command to link the object files manually. I am not well versed at all in Make though so would not know how to begin to change genmake2. If there is a simple way to allow this (re-calling make adAll), then this would be amazing!
since the executable has the same name with TAF and OpenAD, it would make more sense to have the same target name, "adall" for both instead of "adall" and "adAll".
it would be nice to be able to run "make clean" and other cleaning target without having to set env. var. "OPENADROOT".
I noticed that the Makefile for OpenAD experiments does not make sense. After generating it with genmake2 and running
make adAll
once, one cannot runmake adAll
again, because of the linked filesxaif.xsd
,xaif_base.xsd
,xaif_inlinable_intrinsics.xsd
,xaif_derivative_propagator.xsd
,xaif_output.xsd
. After removing them,make adAll
, really re-does everything, which is not in the spirit of "make".Typically I found that I needed to do
make CLEAN
each time I want to try something, which makes the process very slow and difficult. Also, it is not possible to runtestreport -q
with this type of makefile. Sometimes, I would like to do that, however, e.g. when I fiddle with the transformed code (i.e. I don't want modified code to be overwritten), or when I just want change experiments parameters that do not require re-generating and recompiling the code. With the current makefile, I have to wait O(10min) each time I make a tiny change.I suggest that we change the dependency on the
*.xsd
files, e.g. we could add them into themake adDepend
step, if that is used at all. Obviously, I am not very good with makefiles, so I would be happy for other suggestions.The text was updated successfully, but these errors were encountered: