Jae-Seung Yeom

Results 12 comments of Jae-Seung Yeom

> This worked for me; see my blog post for today's mtg. Only comment: I think the "index_list:" field in the data reader should be renamed to "sample_list" for clarity....

Unless anyone has any additional comment, I will merge this PR at the end of the day.

Rebased and compiles. Will test tomorrow.

> This PR is still not ready to merge. There are no clear sample lists in the imagenet datasets directory. `/p/lustre2/brainusr/datasets/ILSVRC2012/sample_list`

Hopefully, this helps. Otherwise, we can discuss more during the meeting. `/p/lustre2/brainusr/datasets/ILSVRC2012/Single_train_c0-9_01_filenames.txt` and `/p/lustre2/brainusr/datasets/ILSVRC2012/sample_list/Single_train_c0-9_01_image_list.txt` are same files in the original format except that the former has the full path to...

> I would prefer if we made a copy of `execution_algorithms/ltfb/checkpoint_file.{cpp,hpp}` so we can have a dedicated class for `checkpoint_file_dyad`. That might help avoid some complicated control flow. Got it.

Currently two steps are required to use dyad with spack before it is officially merged into spack. - add the dyad package.py under `${SPACK_ROOT}/var/spack/repos/builtin/packages/dyad/` ``` from spack import * class...

The example seems to rely on that the class is composed of arithmetic type members, which are already recognized by cereal. Can it be done for a non-standard type or...

I am trying the following. Can this be a reasonable approach? ``` namespace cereal { //! Saving for POD types to binary template inline typename std::enable_if::value && std::is_trivially_copyable::value, void>::type CEREAL_SAVE_FUNCTION_NAME(BinaryOutputArchive...