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

Ryddesc #31

Closed
wants to merge 3 commits into from
Closed

Ryddesc #31

wants to merge 3 commits into from

Conversation

patrikrydberg
Copy link

added reference to paper describing the descriptors, and a general description of the descriptors to the javadoc section of the file.
I hope this was done in the correct manner, since it's my first time, please check that it looks ok.

@johnmay
Copy link
Member

johnmay commented Feb 15, 2018

I saw a talk by Martin Šícho yesterday who has been woking on similar descriptors to Patriks FAME 2 and even update SMARTcyp I believe. Anyways, still using the CDK so going to drop a message and see if he would like to take a look at this PR.

@johnmay
Copy link
Member

johnmay commented Feb 15, 2018

In fact I've actually looked at these commits, and there's not really anything to review. Just need to check the final one I guess... others are just maintaince. Could still get Martin to push his new descriptors back to CDK which would be good.

@patrikrydberg
Copy link
Author

patrikrydberg commented Feb 21, 2018 via email

@johnmay
Copy link
Member

johnmay commented Feb 21, 2018

Hi Kim,

We both knew Patrik and are sorry for your loss then and now. Please feel free to unsubscribe from notifications on this thread (button on the right). My additional comments these many years later are because I knew this was still open and heard a talk last week by Martin Šícho that built upon Patrik's work.

Kind Regards,
John

@egonw
Copy link
Member

egonw commented Feb 22, 2018

Dear Kim,

first, my condolences (my relation with Patrik was that of a collaborator, and unfortunately I did not know how to reach you at the time).

The loss of Patrik did most certainly not go unnoticed, neither did his contributions to our field. The latter causes me to regularly think back about Patrik, in fact, just yesterday when a project he started (with @vedina and @olas) and I have been able to contribute to came up at a research conference.

This issue report is a reminder to make sure his contributions are not lost. I second John's comments about unsubscribing.

My best wishes and regards,

Egon

@patrikrydberg
Copy link
Author

patrikrydberg commented Feb 28, 2018 via email

@javadev
Copy link
Contributor

javadev commented Jan 30, 2022

This pull request was created nine years ago.

@egonw
Copy link
Member

egonw commented Jan 30, 2022

it was indeed. There are still some useful bits of CDK extension that has not been ported yet :/

@johnmay
Copy link
Member

johnmay commented Jan 30, 2022

@egonw last time I looked the vast majority was now included just hard to see as not rebased, so I would close it. Please let me know which part is missing and we can pull it in

@egonw egonw closed this Jan 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants