tps.ftr.list() doesnt seem to parse parameters

[programme957]

    resp= tps.ftr.setup_listing('SDCARD','IMAGES_OVERVIEW_BMP','Data/Geocom/Images/Overview')
    print("SET LISTING >",GeoComCode(resp.error).value, " ", GeoComCode(resp.error).name," ",resp.params)

    resp= tps.ftr.list()
    print("LIST        >",GeoComCode(resp.error).value, " ", GeoComCode(resp.error).name," ",resp.params)
    print (resp)

This attempt to list files produces a response as shown on the last line but the parameters come back None. I wonder if it's something in the library?

SET LISTING > 0   OK   None
LIST        > 0   OK   None
GeoComResponse(FTR_List) code: OK, tr: 9, params: None, (cmd: '%R1Q,23307,9:0', response: '%R1P,0,9:0,1,"ov100001.bmp",14745654,'0d','36','04','00','10','07','19'')

[programme957]

must be the parsing of the datetime elements

[programme957]

got it - i edited the code in ftr.py

        def transform(
            params: tuple[
                bool, str, int,
                Byte, Byte, Byte, Byte, Byte, Byte, Byte
            ] | None
        ) -> tuple[bool, str, int, datetime | None] | None:
            if params is None:
                return None

            return (
                params[0],
                params[1],
                params[2],
                datetime(
                    int(params[9]) + 2000,
                    int(params[8]),
                    int(params[7]),
                    int(params[3]),
                    int(params[4]),
                    int(params[5]) 
                ) if params[1] != "" else None
            )

[MrClock8163]

I pushed a fix to the main branch of the repo, but it's gonna be sometime before I update the PyPI release.

[programme957]

OK. Is the PyPI version what pip installs?

[MrClock8163]

Yes.

In the mean time, you can clone the repo instead, and use it as a local package.

[programme957]

OK I will have to look into all these things as its new to me. I'm sure I will work it out. In the meantime I've done my own edit on the library on my pc anyway.

[MrClock8163]

@programme957

I did some testing and development on the file transfer commands. It seems, that the TPS1200+ GeoCOM reference incorrectly documented the parameters returned by the FTR_Download command, so the current implementation of that command in GeoComPy doesn't work properly. I opened a pull request with fixes. I'll merge it before the week ends.

I also decided to split out the CLI applications from this package into a separate I-man package. It's not yet published on PyPI, but you can look at the source code in the new repository. I added a file listing and working (tested) file downloader command to it, you can use it as reference.