Camera ready #43
Open
Camera ready #43
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Apply acmauthoryear cite style as requested. Small changes to the figures, making lines bolder and adding axis labels. Also add a mention to the Vello rendering engine, which was missing due to anonymity.
armansito
reviewed
Jul 19, 2024
| %\conferenceinfo{Foo}{Month Day-Day, Year, Location} | ||
| \copyrightyear{2024} | ||
| \setcopyright{acmcopyright} | ||
| %\conferenceinfo{High Performance Graphics}{July 26-28 2024, Denver} |
There was a problem hiding this comment.
Should this be uncommented?
There was a problem hiding this comment.
That macro is not defined in the template, so uncommenting it causes an error. I believe the metadata is set correctly now, see eaac725.
This applies the DOI and the rights statement as appropriate for publication in PACMCGIT.
Use a vector rendering of the original Nimbus 'g' in the figure, rather than a bitmap. Visually it is very similar, just crisper.
|
Marking as ready to review, as this is the exact version sent to ACM. There's a short window yet to make revisions, but I believe we should consider this the final version. |
armansito
reviewed
Jul 19, 2024
|
Yep, working on those notes. Some are straightforward, we need |
Use acmauthoryear cite style. Add email addresses. Squash linespace of algorithm 2 to fit page margins. With this cite style, the `\citenum` command is no longer valid, so those are all changed to `\cite`. There are a number of instances that use `\citet` with the author specified in the text, and I believe those are compliant, but I'm not sure that's best. It's possible those should be edited to `\cite` so the author is inside the brackets.
Just use `\cite` for all citations, as there's no longer a need to distinguish `\citenum` from `\citet`. Minor text edits have been made to make this consistent, and the authors of a couple corporately authored artifacts have been changed to mangle the names less.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is a working draft of the "camera ready" version. The final document numbers will be added on the ACM side.
I think this is submittable as is. One thing I see that I would like to address is the bitmap "g" in Fig 8. Also, I'm not completely satisfied with Fig 11, but I might call it "good enough."