c.info requires native types

[qt-thomas]

I caught an untested bug on the straight component: c.info["length"] = length. Pydantic complained that length was not int or float (and it did not coerce it). Instead it was a numpy int which was returned by a rounding function. I fixed with c.info["length"] = float(length).

I assumed this was the right solution as I saw it somewhere else in the codebase. Hope others find this error here. I am pasting the error in case someone might google:

pydantic_core._pydantic_core.ValidationError: 1 validation error for Info
  Value error, Values of the info dict only support int, float, string or tuple.length: 960, <class 'numpy.int64'> [type=value_error, input_value={'length': 960}, input_type=dict]
    For further information visit https://errors.pydantic.dev/2.6/v/value_error

[sourcery-ai]

Hey @qt-thomas - I've reviewed your changes and they look great!

Here's what I looked at during the review

• 🟢 General issues: all looks good
• 🟢 Security: all looks good
• 🟢 Testing: all looks good
• 🟢 Complexity: all looks good
• 🟢 Docstrings: all looks good

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment to tell me if it was helpful.}

[joamatab]

thank you Thomas!