beanprice Error: Trim end date is before start date

[Ghanashyam Prabhu]

I see this error recently when I run beanprice --update --update-rate=daily on my beancount file. I don't recollect changing much on my beancount file. Below is the error message. What does 'Trim end date is before start date' error indicate?  

Traceback (most recent call last):

  File "/usr/local/bin/bean-price", line 8, in <module>

    sys.exit(main())

  File "/usr/local/lib/python3.9/site-packages/beanprice/price.py", line 809, in main

    args, jobs, entries, dcontext = process_args()

  File "/usr/local/lib/python3.9/site-packages/beanprice/price.py", line 782, in process_args

    jobs.extend(get_price_jobs_up_to_date(entries,

  File "/usr/local/lib/python3.9/site-packages/beanprice/price.py", line 412, in get_price_jobs_up_to_date

    lifetimes.trim_intervals(intervals,

  File "/usr/local/lib/python3.9/site-packages/beancount/ops/lifetimes.py", line 120, in trim_intervals

    raise ValueError('Trim end date is before start date')

ValueError: Trim end date is before start date

Ghanashyam

[Martin Blais]

There's a bug in beanprice.

This looks related to code submitted from a PR a little while ago that I copied over when I moved the repo.

A test is still missing:

https://github.com/beancount/beanprice/commit/b2685fea866a8696f9d5c25e851d30b62d6a02a2

--
You received this message because you are subscribed to the Google Groups "Beancount" group.
To unsubscribe from this group and stop receiving emails from it, send an email to beancount+...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/beancount/40dc70ea-83e8-4e58-9891-f0f8784f5f37n%40googlegroups.com.

[Ghanashyam Prabhu]

Interesting that it was working a week before. I am not an expert at debugging python but I'll give it a try.

To view this discussion on the web visit https://groups.google.com/d/msgid/beancount/CAK21%2BhOcKVspjQ9OP79VJEO5wQWxp6vUgB8jQhR-z0wvkYR7Rg%40mail.gmail.com.

[Martin Blais]

Just means the inputs have changed.

I don't have time for a while, send me a patch if you find it.

Make sure you update too, I just merged a PR from @kris7t

https://github.com/beancount/beanprice/commit/f993ac4eb33f797695ab3237e11fcf34bf3790cd

Not sure if related.

To view this discussion on the web visit https://groups.google.com/d/msgid/beancount/CABieCESCJKnRHY%2B3hA-zXqgHMZSB4mA5LLN-ROSVLmd99Le6mQ%40mail.gmail.com.