Changes to column_descriptions related to LSS columns

[aureliocarnero]

This pull request will resolve issues 146 and 148, related to some inconsistencies for some LSS columns.

The exact changes applied to column_descriptions.csv are:

• Remove the following columns: sort, elg_mask, Z_not4clus, LOCFULL
• Capitalization of: lrg_mask, o2c, flux_{g,r,z,w1,w2}_dered
• Change/define types:
  LRG_MASK set to uint8
  O2C set to float64
  LOCATION_ASSIGNED set to bool
  IS_QSO_MGII set to bool
  FLUX_{g,r,z,w1,w2}_DERED set to float32
• Units of FLUX_{g,r,z,w1,w2}_DERED set to nanomaggy
• Description of LOCATION_ASSIGNED: True/False for assigned/unassigned for the target in question

[coveralls]

Coverage: 100.0%. Remained the same when pulling d476781 on correct_lss_columns_descriptor into 8f99ae8 on main.

[weaverba137]

This looks fine to me, thank you for the quick update. Please merge when ready.

[weaverba137]

@aureliocarnero, merging #145 created some conflicts with your changes. If you like, I can resolve those.

[aureliocarnero]

Yes please, if you dont mind, it would be great if you can solve it. Cheers El vie, 10 mar 2023, 17:49, Benjamin Alan Weaver ***@***.***> escribió:

…

@aureliocarnero <https://github.com/aureliocarnero>, merging #145 <#145> created some conflicts with your changes. If you like, I can resolve those. — Reply to this email directly, view it on GitHub <#149 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ADXSAV76PUFUCQNMO7VSGNLW3NSTJANCNFSM6AAAAAAVWSKEKM> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[weaverba137]

OK, conflicts are resolved. Please take a final look at the changes to confirm that everything still looks OK.

[aureliocarnero]

I confirm everything is correct. You can go with the changes. Cheers El vie, 10 mar 2023, 18:13, Benjamin Alan Weaver ***@***.***> escribió:

…

OK, conflicts are resolved. Please take a final look at the changes to confirm that everything still looks OK. — Reply to this email directly, view it on GitHub <#149 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ADXSAV3OUPLNYPGY4IEM3SLW3NVK7ANCNFSM6AAAAAAVWSKEKM> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[weaverba137]

Good, thank you!