search

allenai / AMPT

Aquatic Mammal Photogrammetry Tool
Apache License 2.0
3 stars 0 forks source link

Configuration Editor #45

Open palbee opened 3 years ago

palbee commented 3 years ago

Implement functionality accessed via the hamburger menu to edit the configuration.

palbee commented 3 years ago

@APatrickJ What are your thoughts on a configuration editor? I'm leaning towards a simply documenting the configuration for now.

APatrickJ commented 3 years ago

We discussed in person. Agreed on the following (Paul please feel free to add anything that I'm missing):

palbee commented 3 years ago

Seems right to me.

On Sep 24, 2021, at 12:44, Patrick Johnson @.***> wrote:

We discussed in person. Agreed on the following (Paul please feel free to add anything that I'm missing):

Fully documenting the configuration makes a lot of sense for now (existing documentation https://github.com/AllenMLI/AMPT_documentation/blob/main/docs/csv_columns_config.md). A configuration editor will work better once columns have been refactored / reduced (#52 https://github.com/AllenMLI/AMPT/issues/52) That refactoring work depends on moving from CSV to SQLite for configuration. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/AllenMLI/AMPT/issues/45#issuecomment-926878793, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAEE4O3OYIGHXE6E6PTERE3UDTIKXANCNFSM5AYYNW6Q.

palbee commented 2 years ago

Deferred until after the database backend is complete.