René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Incorrect signature for BMI initialize method","articleBody":"According to the BMI spec, the *filename* argument for the BMI *initialize* method must not be optional and must be a string. Right now `BmiHeat.initialize` accepts either `None`, a file-like object, or a string. I think this should be changed so that it only accepts a string that is a path to a configuration file.\r\n\r\ni.e.\r\n```python\r\ndef initialize(self, filename: str):\r\n    with open(filename) as file_obj:\r\n        self._model = Heat.from_file_like(file_obj)\r\n    ...\r\n```\r\n\r\nActually, in looking at what's there now, I think there's also a bug. `Heat.from_file_like` must take a stream, not a string \r\n\r\ni.e. \r\n```python\r\nHeat.from_file_like(file_obj.read())\r\n```\r\nshould be\r\n```python\r\nHeat.from_file_like(file_obj)\r\n```","author":{"url":"https://github.com/mcflugen","@type":"Person","name":"mcflugen"},"datePublished":"2024-01-13T19:12:56.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":1},"url":"https://github.com/30/bmi-example-python/issues/30"}