Coder Social home page Coder Social logo

Comments (5)

alexdunae avatar alexdunae commented on July 24, 2024

Thanks for reporting this (so embarrassingly long ago). I've picked up work on the gem again and this definitely will be addressed.

:ca is included in us.rb because US holidays currently draw in the informal North American holidays, which apply to Canada, too. It's a bit of circular logic that definitely doesn't make any sense.

I'm clearing out the low-hanging fruit of definition updates before taking a proper look at these more substantial problems like these in the next few weeks (or sooner).

from holidays.

alexdunae avatar alexdunae commented on July 24, 2024

Also of note, d75aca8 will make the UnknownRegionError much less likely. It was way over-keen before.

from holidays.

ppeble avatar ppeble commented on July 24, 2024

Small update, I am in the process of refactoring the codebase so that we don't have a giant single file that contains all of our logic. I'm hoping that one of two things happens during this process:

  1. I figure out what is causing problems like this while refactoring
  2. Once the codebase is split up it will be easier to diagnose problems like this.

from holidays.

ppeble avatar ppeble commented on July 24, 2024

I've merged in v4.0.0, which is a large refactor. This is my next target since it is now the oldest bug that I have inherited.

from holidays.

ppeble avatar ppeble commented on July 24, 2024

Okay! I spent some time looking at this and testing locally and as far as I can tell the core problem has been resolved. The code that handles loading of regions and definitions has changed significantly. I am going to go ahead and close out this issue. Please open a new issue if the problem (or a similar one) persists.

As an aside, once I work through the current crop of issues I am going to A) spend some time with internal refactoring for clarity and B) look at the north american informal holidays. As Alex mentioned it is a bit confusing currently.

from holidays.

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.