Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Improved EndnoteXML and RIS export #4816

Closed
adam3smith opened this issue Jul 6, 2018 · 9 comments
Closed

Improved EndnoteXML and RIS export #4816

adam3smith opened this issue Jul 6, 2018 · 9 comments

Comments

@adam3smith
Copy link
Contributor

adam3smith commented Jul 6, 2018

The current EndnoteXML and RIS export is not as good as it could be.
Working with

I'm proposing the following improvements. I'm happy to provide a mapping as spreadsheet, but have for now just the alternate formats contrasted:
Endnote XML: https://gist.github.com/adam3smith/095f689feadd06946f37026e7dbc8682
RIS: https://gist.github.com/adam3smith/907303c0ed098477430fd87b55443f39

We can help implement these changes, but I wanted to put them up for discussion first.
Note that some mappings may seem a bit arbitrary (e.g. reviewed-item as Geographic coverage), but correspond to what Endnote does and what at least Zotero aims to follow.

I haven't done anything on BibTeX because it's such a poorly defined format to make a definite mapping very tricky.

For reference, @jggautier 's previous work on mapping bib formats:
https://docs.google.com/spreadsheets/d/1ZW5-KmKk5dJWZAD12m7xc1i11xtEY_4vCvTVK72bG38/edit#gid=0

@adam3smith
Copy link
Contributor Author

adam3smith commented Jul 6, 2018

  • ToDo: The same for data files

@pdurbin
Copy link
Member

pdurbin commented Jul 9, 2018

@adam3smith @qqmyers are any of these changes in pull request #4802? I see that BibtexCitation.java has been deleted and the code has been refactored.

@qqmyers
Copy link
Member

qqmyers commented Jul 9, 2018

They aren't yet. I was planning to add them once there are matching suggestions for the file citations - with the refactor, there's common code generating both the dataset and file (indirect and direct) citations, so its useful to know what fields will be in common before coding.) That said, I could go ahead if its useful to see the dataset updates first.

@adam3smith
Copy link
Contributor Author

Let's wrap this in one. I'll make the file citations momentarily -- They're very similar.

@adam3smith
Copy link
Contributor Author

OK, I've updated the files on the gists to include data files, they use the same metadata as the dataset with the following changes:
Dataset title --> secondary title / T2 (previously unused)
Filename --> title / TI
Filename description --> abstract/ AB (replacing dataset description)

This is ready from my end -- @pdurbin did you want to discuss the mapping before Jim implements that?

@pdurbin
Copy link
Member

pdurbin commented Jul 9, 2018

@djbrooke @scolapasta what do you think? Do you want all the work for #4777 and this issue to be combine in the same pull request?

@djbrooke
Copy link
Contributor

djbrooke commented Jul 9, 2018

I won't be picky about small batches here from a process standpoint here. I know Gustavo has some
code review going on for #4777, so it's up to @scolapasta if he's OK including this as well.

@qqmyers
Copy link
Member

qqmyers commented Oct 4, 2018

I think this all went in #4777 and related - is there anything left open?

@pdurbin
Copy link
Member

pdurbin commented Oct 4, 2018

@qqmyers ah, if so, we should have added a "connect to" for this issue in pull request #4971, which was merged and released in Dataverse 4.9.3. But if it's done are you saying it's ready to go to QA? If so, please add a "how to test" comment if you think it would be helpful.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants