Closed JeremyNison closed 1 year ago
For the moment, the name of the folder corresponding to an execution does not match the template app_name-PID-start_timestamp
as described in #18 , as getting the executed application name looks tricky. The app name is for the moment replaced by the string run
, as in this example :
We decided not to include the app name in the generated result folder, and removed the "run" keyword. From now on, the result folder generated for each execution is named like this : PID-start_timestamp
.
Implementation of the new result files hierarchy, as discussed in issue #18 .
The new class
ResultTreeManager
is responsible for creating the required folders and provide methods for easily getting the needed filepaths when writing data. Particularly, this class is created in theAgent
and is passed as an argument to the existingMonitoringHandler
andShutdownHandler
class. Those classes uses it's methods to write data at the right place.The methods
saveResults
inMonitoringHandler
and inShutdownHandler
has also been refactored : the filepath creation has been extracted and is now independant from those methods, allowing to use the newResultTreeManager
more easily.Finally, in the
ShutdownHandler
, the writing of the methods consumption evolution files has been refactored. It is now done with thesaveResult
method, instead ofwriteConsumptionEvolution
that became obselete as all the folder creation job is now performed by thecreate()
method ofResultTreeManager
.This pull request closes #18 .