LODES and PRISM estimate fix.

[meaghan66]

Small name change to estimate_data() for LODES and PRISM.

[JavadocMD]

Part of the reason this got missed (in whichever refactor changed this) is the way these _estimate_<adrio> functions are constructed. Since they take an untyped self argument, it seems like they were originally a class method rather than a standalone function. A class method in Python is forced to take self as the first argument, so the type checker will infer the type of self. But as a standalone function, type checkers cannot infer the type of self so it assigns it "Unknown". The type checker basically gives up validating usage at that point, and IDE tools also can't analyze this, so the auto-renaming tool failed to catch this and the type checker failed to report usage of an undefined property.

All that's to say: please add a type annotation to self. And in fact, it's weird to use self as a name outside of a class (I was initially a bit confused) so please assign a better name there.

[JavadocMD]

Sorry, I could've been more clear. I think you should pass in the ADRIO instance and use the class_name property (which is defined by BaseSimulationFunction) rather than __class__.__name__.

[JavadocMD]

Looks good, thanks!