Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Missing column/data in CSV export in NewCantus compared to OldCantus #694

Closed
jackyyzhang03 opened this issue Jun 5, 2023 · 2 comments
Closed

Missing column/data in CSV export in NewCantus compared to OldCantus #694

jackyyzhang03 opened this issue Jun 5, 2023 · 2 comments

Comments

@jackyyzhang03
Copy link
Contributor

I noticed that there is a column in OldCantus CSV export called cao_concordances that is not present in the CSV export of NewCantus.

Additionally, the differentia_new column is not populated in the NewCantus CSV export but is in OldCantus.
@annamorphism
Copy link

I believe the CAO concordances were intentionally left unimplemented (c.f. #125 )
differentia_new should be populated, though (this is related to #693 )

@jacobdgm jacobdgm mentioned this issue Jun 6, 2023
Closed
@jacobdgm
Copy link
Contributor

jacobdgm commented Jun 6, 2023

Good catch, @jackyyzhang03!

The root cause of the empty differentia_new is deeper than just this view - I opened an issue, #701, outlining it, so I'll close this issue. I'll make a note on the front page of the repo that we've made a choice to hide the cao_concordances column.

@jacobdgm jacobdgm closed this as completed Jun 6, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jacobdgm @annamorphism @jackyyzhang03