Developer Fixme List
FIXME List
A place to note down issues in the code base that you didn't have time to fix. My hope is that the page gets used as a place to note issues with code you aren't familiar with, and the person(s) that are can come along and clean it up.
Please make the descriptions noted clear.
BulkImport Tab
- Multiple file selection
Current Import code
- HandsAction.street is not set to -1 when a player gets to showdown as documented - HudCache appears to get this data. Makes "red line" graphing impossible at the moment (ie. showdown vs non-showdown winnings)
- Probable issue with import times - from Eleatic Stranger
This is a minor problem I discovered when looking around in the database. The time at which a hand is played is not recorded properly. The time stamp is an hour early:
Code:
mysql> select max(handStart) from Hands;
+---------------------+
| max(handStart) |
+---------------------+
| 2009-04-04 18:06:10 |
+---------------------+
1 row in set (0.00 sec)
mysql> select utc_timestamp();
+---------------------+
| utc_timestamp() |
+---------------------+
| 2009-04-04 17:16:49 |
+---------------------+
1 row in set (0.00 sec)
(This was done ten minutes after finishing a session; max(handStart) should have been 17:06 rather than 18:06.)
Grapher
- Graph refresh isn't working for everyone - when refreshing the graph a second/third/… plot appears. Looks like the canveas is failing to be destroyed and that may not be the correct thing to do.
- Whole file needs a refactor… that code is what happens when you start learning a language and cut/paste from everywhere.
- Dates being pulled need to factor in timezone. Attempting to graph a single day appears to get the wrong data depending on timezone.
- Need to add higher granularity to selectable time. Many poker players sessions run past midnight.
HHC
- Player cards dont appear to be in the database (Issue in old import code)
- Fulltilt Razz showdown cards not shown (showdown missing entirely?)
- Fix output of maxseats for all sites (PokerStars kludged) - currently always get 10max hud layout for 6 max game. (Need an optional MAXSEATS attribute in re_HandInfo, then set in each hhc)
Configuration.py
- Could really use the ability for most classes to take a -c argument to specify an alternate config for testing both automated an manual
- Can save back HH_path from autoimport
- Config dialog?
- FIXED? - Make "enabled" tag in site line actually not ruetern the site in conf.supported_sites(.keys())
DB schema
- Hands from mixed games are not flagged in the db—so you can't tell a holdem hand in HORSE from one in a regular holdem game
- Import/export db to file.