ANN_MLP GetParams returns default values (Bug #4094)

Added by Bruno Nascimento about 10 years ago. Updated over 9 years ago.

Status:Incomplete Start date:2015-01-02
Priority:Low Due date:
Assignee:Maksim Shabunin % Done:

0%
Category:ml
Target version:3.1
Affected version:branch 'master' (3.0-dev) Operating System:Windows
Difficulty: HW Platform:x64
Pull request:

Description

ANN_MLP::Params params = m_classifier->getParams();
returns default params, even m_classifier is already loaded

Associated revisions

Revision d5afd070
Added by Vadim Pisarevsky over 9 years ago

Merge pull request #4094 from kashefy:Mat_push_back_MatExpr_bug_4340

History

#1
Updated by Bruno Nascimento about 10 years ago

layers size is not being retrived correctly you should consider to add on "ann_mlp.cpp" function "void read( const FileNode& fn )" something like:

params.layerSizes = Mat_<int>(_layer_sizes, true);

after read_params(fn);

  • Status changed from New to Incomplete

#2
Updated by Maksim Shabunin almost 10 years ago

  • Target version changed from 3.0-beta to 3.0

#3
Updated by Maksim Shabunin almost 10 years ago

Can not reproduce with the latest master branch:

Ptr<ANN_MLP> ann = ANN_MLP::create();
ann->setLayerSizes(layer_sizes);
ann->save(fname);
cout << "Set " << ann->getLayerSizes() << endl;

Ptr<ANN_MLP> new_ann = Algorithm::load<ANN_MLP>(fname);
cout << "Get " << new_ann->getLayerSizes() << endl;

Layer sizes are read correctly.

Can you, please, check it again?

  • Priority changed from Normal to Low
  • Assignee set to Maksim Shabunin
  • Category set to ml
  • Target version changed from 3.0 to 3.1

#4
Updated by Maksim Shabunin over 9 years ago

Issue has been transferred to GitHub: https://github.com/Itseez/opencv/issues/4896

Also available in: Atom PDF