CosmoAI-AES / CosmoSim

Original repo from the BSc project Spring 2022
MIT License
1 stars 3 forks source link

Separate and use model and lens parameters for datagen.py #115

Closed hgeorgsch closed 1 year ago

hgeorgsch commented 1 year ago

This includes cleaning up the C++ code. We should probably have a --lens option for the lens model, and a --sampled flag to choose between direct use and sampling.

hgeorgsch commented 1 year ago

Review this, and close #111 in the process.

hgeorgsch commented 1 year ago

ongoing on branch refactor/split-model-lens

This may be done; testing remains.

hgeorgsch commented 1 year ago

There is a problem with the Sampled Raytrace Model, but it is worse on the develop branch where it segfaults.

hgeorgsch commented 1 year ago

Seems ok; merged.