Skip to content

Commit d26ab95

Browse files
Revert "[ML] Add min_version to model snapshot (#17)"
This reverts commit 155d2f8.
1 parent 65f6e5b commit d26ab95

File tree

7 files changed

+0
-171
lines changed

7 files changed

+0
-171
lines changed

include/api/CModelSnapshotJsonWriter.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,6 @@ class API_EXPORT CModelSnapshotJsonWriter
4141
//! Structure to store the model snapshot metadata
4242
struct SModelSnapshotReport
4343
{
44-
std::string s_MinVersion;
4544
core_t::TTime s_SnapshotTimestamp;
4645
std::string s_Description;
4746
std::string s_SnapshotId;

lib/api/CAnomalyJob.cc

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -86,10 +86,6 @@ const std::string HIERARCHICAL_RESULTS_TAG("f");
8686
const std::string LATEST_RECORD_TIME_TAG("h");
8787
const std::string MODEL_PLOT_TAG("i");
8888
const std::string LAST_RESULTS_TIME_TAG("j");
89-
90-
//! The minimum version required to read the state corresponding to a model snapshot.
91-
//! This should be updated every time there is a breaking change to the model state.
92-
const std::string MODEL_SNAPSHOT_MIN_VERSION("6.3.0");
9389
}
9490

9591
// Statics
@@ -1384,7 +1380,6 @@ bool CAnomalyJob::persistState(const std::string &descriptionPrefix,
13841380
if (m_PersistCompleteFunc)
13851381
{
13861382
CModelSnapshotJsonWriter::SModelSnapshotReport modelSnapshotReport{
1387-
MODEL_SNAPSHOT_MIN_VERSION,
13881383
snapshotTimestamp,
13891384
descriptionPrefix + core::CTimeUtils::toIso8601(snapshotTimestamp),
13901385
snapShotId,

lib/api/CModelSnapshotJsonWriter.cc

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,6 @@ namespace
2626

2727
// JSON field names
2828
const std::string JOB_ID("job_id");
29-
const std::string MIN_VERSION("min_version");
3029
const std::string TIMESTAMP("timestamp");
3130
const std::string MODEL_SNAPSHOT("model_snapshot");
3231
const std::string SNAPSHOT_ID("snapshot_id");
@@ -55,8 +54,6 @@ void CModelSnapshotJsonWriter::write(const SModelSnapshotReport &report)
5554

5655
m_Writer.String(JOB_ID);
5756
m_Writer.String(m_JobId);
58-
m_Writer.String(MIN_VERSION);
59-
m_Writer.String(report.s_MinVersion);
6057
m_Writer.String(SNAPSHOT_ID);
6158
m_Writer.String(report.s_SnapshotId);
6259

lib/api/unittest/CModelSnapshotJsonWriterTest.cc

Lines changed: 0 additions & 129 deletions
This file was deleted.

lib/api/unittest/CModelSnapshotJsonWriterTest.h

Lines changed: 0 additions & 30 deletions
This file was deleted.

lib/api/unittest/Main.cc

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,6 @@
3333
#include "CLineifiedJsonOutputWriterTest.h"
3434
#include "CLineifiedXmlInputParserTest.h"
3535
#include "CModelPlotDataJsonWriterTest.h"
36-
#include "CModelSnapshotJsonWriterTest.h"
3736
#include "CMultiFileDataAdderTest.h"
3837
#include "COutputChainerTest.h"
3938
#include "CRestorePreviousStateTest.h"
@@ -68,7 +67,6 @@ int main(int argc, const char **argv)
6867
runner.addTest( CLineifiedJsonOutputWriterTest::suite() );
6968
runner.addTest( CLineifiedXmlInputParserTest::suite() );
7069
runner.addTest( CModelPlotDataJsonWriterTest::suite() );
71-
runner.addTest( CModelSnapshotJsonWriterTest::suite() );
7270
runner.addTest( CMultiFileDataAdderTest::suite() );
7371
runner.addTest( COutputChainerTest::suite() );
7472
runner.addTest( CRestorePreviousStateTest::suite() );

lib/api/unittest/Makefile

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,6 @@ SRCS=\
5151
CMockDataProcessor.cc \
5252
CMockSearcher.cc \
5353
CModelPlotDataJsonWriterTest.cc \
54-
CModelSnapshotJsonWriterTest.cc \
5554
CMultiFileDataAdderTest.cc \
5655
COutputChainerTest.cc \
5756
CRestorePreviousStateTest.cc \

0 commit comments

Comments
 (0)