suzytamang / clever-rockies

clinical event recognizer for TIU
2 stars 2 forks source link

Update dictionary to remove "comma" #15

Open dax-westerman opened 1 month ago

dax-westerman commented 1 month ago

Need to remove comma entry from res/dicts/dict.txt

This involves the following steps:

suzytamang commented 1 month ago

The terms are not enumerated from 0, but we can do that. Right now they are grouped in chunks of 1000s, but it's more than fine to enumberate if we have a system.

dax-westerman commented 1 month ago

The terms are not enumerated from 0, but we can do that. Right now they are grouped in chunks of 1000s, but it's more than fine to enumberate if we have a system.

As it pertains to the second point, I should have left that as a separate "idea" rather than include as part of the effort, so sorry for any confusion. This was an area of exploration I'd wanted to include as a potential means of managing the dictionary which would provide a method for validation as well. I'm going line-strike it to keep the issue clean :)

Thanks!