hebrew.to_jd_gregorianyear(...) cannot unpack non-iterable float object about convertdate HOT 5 CLOSED

Hi,
in fact, this seems related to the change of hebrew.to_jd_gregorianyear signature. I'm currently looking for a fix on python-holidays for this new behaviour.
open issue
Although a subsequent call to gregorian.from_jd as gregorian.from_jd(hebrew.to_jd_gregorianyear (..))
seems to me the easiest way to revert to the previously returned format (year, month, day) I noticed some differences (possibly issues?) in the date calculation results (ie: current results don't match the existing unit test expectations) so I'm currently in standby, keeping back convertdate to v2.2.0.

Any update available on this?

from convertdate.

Hi, any updates / thoughts on this @fitnr ? Thx

from convertdate.

Please post the smallest possible code snippet to reproduce the error. I can’t make head or tails of what’s in the original comment

from convertdate.

I reversed the code to an older version:

def to_jd_gregorianyear(gregorianyear, hebrew_month, hebrew_day):
      for y in (gregorianyear + HEBREW_YEAR_OFFSET, gregorianyear + HEBREW_YEAR_OFFSET + 1):
          jd = to_jd(y, hebrew_month, hebrew_day)
          gd = gregorian.from_jd(jd)
          if gd[0] == gregorianyear:
              break
          else:
              gd = None

    if not gd:  # should never occur, but just incase...
        raise ValueError("Could not determine gregorian year")

    # tuple: (y, m, d)
    return (gd[0], gd[1], gd[2])

instead of:

def to_jd_gregorianyear(gregorianyear, hebrew_month, hebrew_day):
    
    for y in (gregorianyear + HEBREW_YEAR_OFFSET, gregorianyear + HEBREW_YEAR_OFFSET + 1):
        jd = to_jd(y, hebrew_month, hebrew_day)
        gd = gregorian.from_jd(jd)
        if gd[0] == gregorianyear:
            break

        jd = None

    if not jd:  # should never occur, but just incase...
        raise ValueError("Could not determine gregorian year")

    return gregorian.to_jd(gd[0], gd[1], gd[2])

The problem is that gregorian.to_jd returns a number instead of a tuple.

from convertdate.

Yes, this method changed. The earlier version didn't adequately support other functions, especially the holiday library.

from convertdate.