tardis icon indicating copy to clipboard operation
tardis copied to clipboard

Clean up custom abundance input and documentation

Open andrewfullard opened this issue 4 years ago • 4 comments

Problem description

Currently the two custom input options are in the documentation "ascii" and "csv". Neither are truly descriptive of the file types used or the formatting requirements.

I propose changing the "ascii" input type to be "complex_composition" (because it requires all isotopes from Z=0-30) and the "csv" input type to be "simple_composition" because it requires only isotope names in the header. The user should be able to choose a delimiter for either input file type as part of the configuration.

This would involve changes primarily to tardis/io/model_reader.py and the model input schema.

andrewfullard avatar Jan 21 '21 18:01 andrewfullard

Hello, I wish to work on the same with some guidance for the exact changes that are needed to reach the required goal :) I do understand that we need to keep one type of the file format specifically for complex_composition & the other one for simple_composition but how can it be elaborated in the documentation is a matter of concern 🤔

DhruvSondhi avatar Jan 24 '21 06:01 DhruvSondhi

I would recommend providing examples following the existing documentation style for custom compositions

andrewfullard avatar Jan 25 '21 14:01 andrewfullard

Hi, I would like to take up this issue. After going through the documentation i got to know that the 'ascii' input files use a ‘.dat’ format and csv input files use a ‘.csv’ format.Our objective is to change ‘ascii’ input type to ‘complex_composition’ and ‘csv’ to ‘simple_composition’.Going through the model_reader.py file i saw many funtions like ‘read_simple_ascii_abundance’ in which i need to make changes to the input names. Please tell if i am thinking in the right direction and do elaborate on what all should be done.

karban8 avatar Mar 20 '21 08:03 karban8

That's correct @karban8

andrewfullard avatar Mar 24 '21 14:03 andrewfullard