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

Add "conditionally accepted" #100

Merged
merged 2 commits into from
Aug 7, 2020
Merged

Add "conditionally accepted" #100

merged 2 commits into from
Aug 7, 2020

Conversation

b-studios
Copy link
Contributor

I appreciate having our paper listed, but it is not yet fully accepted at OOPSLA and I do not want to spread the impression it is.
For the time being, maybe we can add "conditionally accepted"?

I appreciate having our paper listed, but it is not yet fully accepted at OOPSLA and I do not want to spread the impression it is. 
For the time being, maybe we can add "conditionally accepted"?
@yallop
Copy link
Owner

yallop commented Aug 7, 2020

Thanks, @b-studios!

@dhil: we could avoid this issue in the future by waiting until a paper is unconditionally accepted before adding it to the bibliography. What do you think?

@b-studios
Copy link
Contributor Author

Another option would be to just mark it as TR. Actually, the title of the OOPSLA submission is also slightly different (due to requirements of double blind reviews)

@dhil
Copy link
Collaborator

dhil commented Aug 7, 2020

@dhil: we could avoid this issue in the future by waiting until a paper is unconditionally accepted before adding it to the bibliography. What do you think?

Yes!! That's what I have done in the past! This is my mistake, apologies! When it was presented to me I understood it to be (fully) accepted, and I didn't bother to check :-) Lesson learnt.

README.md Outdated Show resolved Hide resolved
Co-authored-by: Daniel Hillerström <1827113+dhil@users.noreply.github.com>
@b-studios
Copy link
Contributor Author

Agreed, technical report sounds good to me. I am very happy that it is already listed since I want people to read it :)

@dhil
Copy link
Collaborator

dhil commented Aug 7, 2020

Great! Thanks for correcting my error! I will merge as soon as the CI completes successfully.

@b-studios
Copy link
Contributor Author

Great, thanks for adding it in the first place and sorry, if I communicated it wrong somewhere

@dhil
Copy link
Collaborator

dhil commented Aug 7, 2020

No worries. It was communicated to me by an intermediary. Supposedly, this communication wasn't lossless, or maybe I had instantiated my context with some wrong assumptions ;-))

@dhil
Copy link
Collaborator

dhil commented Aug 7, 2020

It seems like bentnib.org is down at the moment. I will open an issue to notify Bob.

@dhil dhil mentioned this pull request Aug 7, 2020
@dhil dhil merged commit 28dc96d into yallop:master Aug 7, 2020
@b-studios
Copy link
Contributor Author

@dhil now that our paper is published (https://dl.acm.org/doi/10.1145/3428194) should we replace the TR with the officially published version? The TR contains additional material, maybe it can be linked in addition to the official paper?

@dhil
Copy link
Collaborator

dhil commented Nov 17, 2020

Yes!

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

Successfully merging this pull request may close these issues.

None yet

3 participants