Eddie Janowicz
Eddie Janowicz
Thanks for the reminder @Eh2406. Prior to the new RuntimeError that this PR introduces, would the existing RuntimeError ("'Maximum number of iterations reached during IPF") already be encountered in situations...
@cvanoli Thanks. Per our existing discussions with @sablanchard, this low-memory change should be made an optional one (e.g. a high-level low_memory parameter that defaults to False), so that those with...
Thanks for reporting this @stelehm. Looks like the notebook may be a bit out of date, as general_synthesizer was renamed to `zone_synthesizer`. Could you please try: `import synthpop.zone_synthesizer`
Thanks very much for this great contribution @mxndrwgrdnr! We'll review this soon, sorry for the delay. There is an existing earlier open PR that was reveiewed, which we kept open...
Awesome, thanks @mxndrwgrdnr! I'll check it out today
This is a really interesting extension to the variable expressions, and in general I like this new optional functionality! Saves a to_frame() call and sets the stage for a column-level...
I've been using this branch of orca for a little while for testing, and I like this new functionality quite a bit. Saves on a lot of to_frame() calls and...
Thanks for this @Eh2406! The passing Travis builds are the relevant benchmark here. The appveyor Windows build are failing due to pandana on Windows, which is unrelated to the present...
Some thoughts (curious what others think too): `test_dcm.py` - Create a "normalized_dcm" fixture that takes the existing basic_dcm fixture as a starting point and set values for normalized/l1/l2 - Using...
Thanks for reporting this. Good to know that it was a control total issue. Agreed that a warning about some agents not being subject to the transition model would be...