Slice numbers in result events should be integers

[dan-distributive]

Issue type

Bug

How did you install PythonMonkey?

Installed from pip

OS platform and distribution

macOS sequoia 15.5

Python version (python --version)

3.12.6

PythonMonkey version (pip show pythonmonkey)

1.3.0

Bug Description

slice number is shown with a float (e.g. "sliceNumber": 10.0 ) whereas it should be an integer

Standalone code to reproduce the issue

my_job.on('result',     lambda res: print(json.dumps(res, indent=4).replace('\\n', '\n')))

Relevant log output or backtrace

{
    "job": "j47az5Gl9lc36jFm8pJnjH",
    "sliceNumber": 10.0,
    "result": 46.0
}

Additional info if applicable

What branch of PythonMonkey were you developing on? (If applicable)

No response

[wiwichips]

Smaller repro:

>>> import pythonmonkey as pm
>>> pm.eval("1")
1.0


>>> pm.eval("1 + 1")
2.0

>>> pm.eval("parseInt")(1) # calling a js function which returns a whole number from python
1.0

---

For extra context, Python has int and float, but JS only has number. In JS, console.log prints whole number floats without the decimal point (at least nodejs does > console.log(3.0) // 3), but Python doesn't do that (>>> print("3.0") # 3.0)

I think it would be a bad idea to convert JS numbers to python ints if they happen to be whole numbers, so I think we might be stuck with this behaviour for whole numbers which come from js ): - unless @Xmader @wesgarland @YarnSaw have any other ideas?

@dan-distributive - If it shouldn't be "fixed" / "isn't a bug" on the pythonmonkey side, but it's important to get the slice-numbers printing properly without the ".0", it could be converted somewhat hackily on the bifrost2 side

[zollqir]

Yeah this was unfortunately a choice between two evils for PythonMonkey. Either we violate the principle of least surprise by having one type in JS correspond to two types in Python, or we violate it by having one type (int) in Python passed to JS and back becoming a different type (float).

Wes and I at the time agreed that the former was probably the worse choice, but I can see it as having gone either way.