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

Lossy compression through coordinate sampling #326

Merged

Conversation

erget
Copy link
Member

@erget erget commented May 4, 2021

See issue #327 for discussion of these changes.

To do before completion of this PR:

"interpolation mapping variable:: A variable used as a container for attributes that define a . The type of the variable is arbitrary since it contains no data."

Release checklist

@erget to complete:

  • Authors updated in cf-conventions.adoc?
  • Next version in cf-conventions.adoc up to date? Versioning inspired by SemVer.
  • history.adoc up to date?
  • Conformance document up-to-date?

For maintainers

After the merge remember to delete the source branch.
Tags are set at the conclusion of the annual meeting; until then master always is a draft for the next version.

davidhassell and others added 30 commits November 30, 2020 08:56
* more text following 2020-11-27 discussions

* bounds

* tidy

* tidy

* tidy

* tidy

* reproducability

* offset

* indices

* indices

* indices

* super
Minor editorial changes
* more text following 2020-11-27 discussions

* bounds

* tidy

* tidy

* tidy

* tidy

* reproducability

* offset

* indices

* indices

* indices

* super

* tie_point_dimension (1)

* tie_point_dimension (2)

* tie_point_dimension (3)

* tie_point_dimension (4)

* tie point

* tie_point_dimension (5)
* more text following 2020-11-27 discussions

* bounds

* tidy

* tidy

* tidy

* tidy

* reproducability

* offset

* indices

* indices

* indices

* super

* tie_point_dimension (1)

* tie_point_dimension (2)

* tie_point_dimension (3)

* tie_point_dimension (4)

* tie point

* tie_point_dimension (5)

* lossy

* lossy

* correct typo

Co-authored-by: AndersMS <[email protected]>
ch08.adoc Outdated Show resolved Hide resolved
@erget
Copy link
Member Author

erget commented Aug 3, 2021

Most recent PDF of all docs, built manually:

cf-conventions.pdf

We have some references missing. Will fix.

@erget
Copy link
Member Author

erget commented Aug 4, 2021

@AndersMS I see boxes also using Adobe Acrobat Reader where @JonathanGregory notes them - this is found on pg. 222. Do you know what's going on here?

@davidhassell
Copy link
Contributor

Using okular, I see the boxes, too. However, viewing the file with the built-in Firefox PDF viewer, I see just a space with no box outlining box.

@AndersMS
Copy link
Contributor

AndersMS commented Aug 4, 2021

@JonathanGregory, @davidhassell, @erget

A special character had sneaked into the formulas cos() and sin(). I have removed it c989276 and the boxes are gone in my preview.

@erget
Copy link
Member Author

erget commented Aug 4, 2021

Thanks very much - I see that the characters are gone now. For reference I built c989276 - here's the PDF:
cf-conventions.pdf

@erget erget merged commit e9a68d5 into cf-convention:master Aug 24, 2021
@davidhassell davidhassell added this to the 1.9 milestone Sep 3, 2021
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.

Lossy Compression by Coordinate Sampling
3 participants