Skip to content

Lwtnn Upgrade, master branch (2021.01.26.)

Attila Krasznahorkay requested to merge akraszna/atlasexternals:lwtnnUpdate-master-20210126 into master

This is a replacement for !787 (closed), upgrading lwtnn to version 2.11.1.

@dguest, the issue in !787 (closed) is that the switch from version 2.9 to 2.11.1 is not incremental build friendly. I'm also not a fan of the build warnings in 2.11.1. 😦

[ 76%] Building CXX object CMakeFiles/lwtnn-test-lightweight-graph.dir/src/lwtnn-test-lightweight-graph.cxx.o
Scanning dependencies of target lwtnn-test-fastgraph
/data/hdd-4tb/projects/lwtnn/build/src/lwtnn/tests/test-nn-streamers.cxx: In function 'int main()':
/data/hdd-4tb/projects/lwtnn/build/src/lwtnn/tests/test-nn-streamers.cxx:47:42: warning: missing initializer for member 'lwt::ActivationConfig::alpha' [-Wmissing-field-initializers]
      {}, {}, {}, lwt::Architecture::NONE };
                                          ^
/data/hdd-4tb/projects/lwtnn/build/src/lwtnn/tests/test-nn-streamers.cxx:47:42: warning: missing initializer for member 'lwt::ActivationConfig::alpha' [-Wmissing-field-initializers]
[ 78%] Building CXX object CMakeFiles/lwtnn-test-fastgraph.dir/src/lwtnn-test-fastgraph.cxx.o
[ 81%] Linking CXX executable test-bin/test-nn-streamers
[ 81%] Built target test-nn-streamers
[ 84%] Linking CXX executable bin/lwtnn-dump-config
[ 84%] Built target lwtnn-dump-config
/data/hdd-4tb/projects/lwtnn/build/src/lwtnn/src/lwtnn-test-graph.cxx: In function 'lwt::GraphConfig {anonymous}::dummy_config()':
/data/hdd-4tb/projects/lwtnn/build/src/lwtnn/src/lwtnn-test-graph.cxx:66:66: warning: missing initializer for member 'lwt::InputNodeConfig::miscellaneous' [-Wmissing-field-initializers]
     config.inputs = {{"one", dummy_inputs}, {"two", dummy_inputs}};
                                                                  ^
/data/hdd-4tb/projects/lwtnn/build/src/lwtnn/src/lwtnn-test-graph.cxx:66:66: warning: missing initializer for member 'lwt::InputNodeConfig::defaults' [-Wmissing-field-initializers]
/data/hdd-4tb/projects/lwtnn/build/src/lwtnn/src/lwtnn-test-graph.cxx:66:66: warning: missing initializer for member 'lwt::InputNodeConfig::miscellaneous' [-Wmissing-field-initializers]
/data/hdd-4tb/projects/lwtnn/build/src/lwtnn/src/lwtnn-test-graph.cxx:66:66: warning: missing initializer for member 'lwt::InputNodeConfig::defaults' [-Wmissing-field-initializers]
/data/hdd-4tb/projects/lwtnn/build/src/lwtnn/src/lwtnn-test-graph.cxx:75:55: warning: missing initializer for member 'lwt::NodeConfig::index' [-Wmissing-field-initializers]
     config.nodes.push_back({Type::CONCATENATE, {0, 1}});
                                                       ^
/data/hdd-4tb/projects/lwtnn/build/src/lwtnn/src/lwtnn-test-graph.cxx:84:58: warning: missing initializer for member 'lwt::LayerConfig::bias' [-Wmissing-field-initializers]
       {0, 0, 0, 1,  0, 0, 1, 0,  0, 1, 0, 0,  1, 0, 0, 0}};
                                                          ^
/data/hdd-4tb/projects/lwtnn/build/src/lwtnn/src/lwtnn-test-graph.cxx:84:58: warning: missing initializer for member 'lwt::LayerConfig::U' [-Wmissing-field-initializers]
/data/hdd-4tb/projects/lwtnn/build/src/lwtnn/src/lwtnn-test-graph.cxx:84:58: warning: missing initializer for member 'lwt::LayerConfig::activation' [-Wmissing-field-initializers]
/data/hdd-4tb/projects/lwtnn/build/src/lwtnn/src/lwtnn-test-graph.cxx:84:58: warning: missing initializer for member 'lwt::LayerConfig::inner_activation' [-Wmissing-field-initializers]
/data/hdd-4tb/projects/lwtnn/build/src/lwtnn/src/lwtnn-test-graph.cxx:84:58: warning: missing initializer for member 'lwt::LayerConfig::sublayers' [-Wmissing-field-initializers]
/data/hdd-4tb/projects/lwtnn/build/src/lwtnn/src/lwtnn-test-graph.cxx:84:58: warning: missing initializer for member 'lwt::LayerConfig::components' [-Wmissing-field-initializers]
/data/hdd-4tb/projects/lwtnn/build/src/lwtnn/src/lwtnn-test-graph.cxx:84:58: warning: missing initializer for member 'lwt::LayerConfig::embedding' [-Wmissing-field-initializers]
/data/hdd-4tb/projects/lwtnn/build/src/lwtnn/src/lwtnn-test-graph.cxx:84:58: warning: missing initializer for member 'lwt::LayerConfig::architecture' [-Wmissing-field-initializers]

But I guess we'll live with these for a while...

Merge request reports