GithubHelp home page GithubHelp logo

Comments (4)

cdalitz avatar cdalitz commented on August 12, 2024

With version 1.7.0, VIGRA changed the meaning from angles from counter-clockwise to clockwise, which was an incompatible change that might break end user applications. Before switsching to a newer version of VIGRA, we must thus first find all instances where angles are passed to VIGRA and invert their signs. Or we just ignore this and let and users scratch their heads.

from gamera-4.

FriedrichFroebel avatar FriedrichFroebel commented on August 12, 2024

I have to admit that I am neither familiar with Gamera nor VIGRA in enough detail to comment on which approach should be preferred here. In general I prefer not to bundle external sources wherever possible, as these usually are out-of-date.

Judging from your description, the easiest approach would be to perform the replacements from the upstream PR in our bundled copy? With this we avoid the hassle of dealing with the incompatible angle behaviour for now, leaving this for a possible upgrade later on.

from gamera-4.

cdalitz avatar cdalitz commented on August 12, 2024

Yes, merging these changes into the copy of VIGRA 1.6 and documenting this in a README of the bundled copy would be an option. OTOH, if it's only a wraning and no error, I do nt think that it is worth the trouble.

The reason for bundling some libraries with Gamera is to make it easier to compile on non-Linux platforms.

from gamera-4.

FriedrichFroebel avatar FriedrichFroebel commented on August 12, 2024

Fixing these warnings actually turned out to be rather straightforward, as this affects less than 10 lines. I will send a PR for this as soon as I have isolated the corresponding changes from my latest tests.

from gamera-4.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.