Mo Tiwari

```python mnist_test.py --datasize 40000 --valsize 10000 --lrh 0.01 --epochs=100 --hepochs=10 --l2=1e-5 --restart=10 --model=mlp --dataset=mnist --num_layers=1 --hessian=kfac --jacobian=direct``` should be ```python mnist_test.py --datasize 40000 --valsize 10000 --lrh 0.01 --epochs=100 --hepochs=10 --l2=1e-5...

`joblib` `numba` `pytz` `ruamel.yaml` etc. Might want to use https://pypi.org/project/pipreqs/ via `pipreqs --savepath requirements.txt .`

Thanks for providing this code for the associated paper. There are a few typos in the README: "estiamte", "difined", etc. This line: `difined in line 627 in mnist_test.py To use`...

On https://pypi.org/project/gosdt/#files, it appears there's no source distribution or M1 Mac distribution available, which means the package can't be installed from PyPI on M1 Macs

Installation instructions could be clearer

1

I think there were a few non-obvious pitfalls during the installation, that could be made clearer in the readme: 1. the `baselines` package must be installed per the instructions at...

See: https://github.com/rll/rllab/blob/master/scripts/setup_mujoco.sh#L4

This line: https://github.com/roosephu/slbo/blob/master/lunzi/nn/layers.py#L20 Should be replaced with: `self.weight = Parameter(weight_initializer([in_features, out_features], dtype=tf.float32), name='weight')` It's missing the `dtype` parameter, which causes an exception. Will file a PR if I have time...

LoadEdgeListStr should support custom delimiters

1

Instead of only whitespace