Using tmp_path as mentioned in issue #113. In the process of adding a sanity check to test_03_sample_molecules_RNN I fixed a tiny undesirable behavior (the last iteration shooting past the max_molecules mark).
All the changes are minor (introduction of tmp_path), but the diff makes them appear worse. Probably bcompare will do a better job at highlighting these diffs.
write_to_csv_file with a non-dataframe was only used at one place, changed in this PR, so we can remove it in a future commit.
Using
tmp_pathas mentioned in issue #113. In the process of adding a sanity check totest_03_sample_molecules_RNNI fixed a tiny undesirable behavior (the last iteration shooting past themax_moleculesmark).All the changes are minor (introduction of
tmp_path), but the diff makes them appear worse. Probablybcomparewill do a better job at highlighting these diffs.write_to_csv_filewith a non-dataframe was only used at one place, changed in this PR, so we can remove it in a future commit.