Chris Singley

Thanks for picking this up > Actually, I think I was misunderstanding the OP. IIUC, the goal is to interpret "" as missing and deserialize to None. Yes, this is...

Read the error message. It doesn’t like your time zone. Sounds like their parser barfed as soon as it hit a + sign. In-house development, not a standard library, with...

Nice catch, thanks for reporting. Should be fixed in [655549a](https://github.com/csingley/ofxtools/commit/655549a357d9e5109460ab4687fa9e304694e17c)

No your're right, it's probably not. If you search `Client.py` (or this bug tracker) for "Amex" you'll find comments from the last round of header-related incantation engineering regarding XML quality...

You're in it to win it! Whatever homegrown jank Amex has up on the web seems to be in the van of the "code to Quicken not the spec" movement....

I mean, you could just [check in the new types](https://github.com/csingley/ibflex/blob/master/ibflex/Types.py). I've gone to some trouble to make adding types as easy as possible.... surely easier than logging into IBKR web...

Autogenerating the types would absolutely be the move - unfortunately there's just not enough there for a machine to reasonably inter the type of data primitives (`str` vs `Decimal`, or...

The right way to do this is to get devs from a few funds & RIAs, who in aggregate can speak for O(US$10^8) net assets, on the same page to...

And thank you for putting in the work here - sorry for the delay in reviewing.

`models.base.Aggregate.groom()` is intended as a hook to address (a). It's so far been done in an ad-hoc manner; for some examples see `grep -A 5 INTU ~/Code/ofxtools/ofxtools/models/*.py` . Christ now...