Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Feb 09 00:54
    tec4 opened #220
  • Feb 08 11:05
    Alexandre-T commented #207
  • Feb 08 07:33
    amirsalkhori opened #219
  • Feb 08 07:29
    amirsalkhori commented #207
  • Jan 05 17:57
    prostodima synchronize #218
  • Jan 04 18:59
    prostodima synchronize #218
  • Dec 30 2021 16:14
    prostodima synchronize #218
  • Dec 29 2021 19:24
    srdjan2v2 converted_to_draft #218
  • Dec 29 2021 19:19
    srdjan2v2 edited #218
  • Dec 29 2021 18:50
    srdjan2v2 opened #218
  • Oct 27 2021 13:25
    ottaviano closed #217
  • Oct 27 2021 13:25
    ottaviano opened #217
  • Aug 22 2021 08:45
    Alexandre-T commented #209
  • Aug 22 2021 08:30
    Alexandre-T commented #209
  • Aug 22 2021 03:40
    samwilson commented #209
  • Jul 06 2021 17:42
    sadortun commented #216
  • Jul 06 2021 17:41

    sadortun on master

    Fix typo Merge pull request #216 from pa… (compare)

  • Jul 06 2021 17:41
    sadortun closed #216
  • Jul 06 2021 17:24
    pamuche opened #216
  • Jun 18 2021 22:51
    sadortun commented #209
Derek J. Lambert
@djlambert
argh that was referring to 1 in the list not the issue
Jon West
@ddproxy
I think you are on track with 1
Jon West
@ddproxy
Hey! Either I messed up something... Or I probably messed up something... 
Task Exception: Queue Exception: Warning: unpack(): Type d: not enough input, need 8, have 0 in /opt/reactor/vendor/creof/wkb-parser/lib/CrEOF/Geo/WKB/Reader.php on line 164 in /opt/reactor/src/JobQueueBundle/Command/WorkerCommand.php on line 232
Which means - I need to find the shape, I might have replaced something in here since I'm actually reformatting some semicolon separated to be comma separated....
Derek J. Lambert
@djlambert
which version of the library is loaded?
Jon West
@ddproxy
    "creof/doctrine2-spatial": "~1",
I've honestly probably messed up my shape somewhere.
It was working before ;)
Derek J. Lambert
@djlambert
:) I tweaked line 164 this weekend want to make sure I didn't break it for everyone. What does composer list say?
Jon West
@ddproxy
lol
Derek J. Lambert
@djlambert
it might not be a bad idea to wrap that in a try/catch block and dump the whole input in hex on an error
Jon West
@ddproxy
        "name": "creof/doctrine2-spatial",
        "version": "1.1.1",
Is in my lock file
Derek J. Lambert
@djlambert
what's in there for creof/wkb-parser?
Jon West
@ddproxy
Oh well...
2.1.0
Derek J. Lambert
@djlambert
ok. looks like my change is still in a PR
Jon West
@ddproxy
Mhmm..
Anywhere to test my polygon?
Other than the library ;)
BAh doesn't look like theres anything online to do it...
Derek J. Lambert
@djlambert
:) if I remeber correctly I think the postgres error messages gave some hint where the issue was
a simple CLI validator script wouldn't be a bad idea
Jon West
@ddproxy
Yeah and to compare with postgres error if applicable... anyways moment
Jon West
@ddproxy
ERROR:  Invalid endian flag value encountered.
********** Error **********

ERROR: Invalid endian flag value encountered.
SQL state: XX000
Derek J. Lambert
@djlambert
that's interesting
wkb-parser should have thrown an error for that too
Jon West
@ddproxy
How did I mess that up...
Derek J. Lambert
@djlambert
what's the first byte?
should be 0 or 1
Jon West
@ddproxy
0
Derek J. Lambert
@djlambert
0x00?
Jon West
@ddproxy
0103000020E610000098000000
But, interesting... The difference between my two files is one is wrapped with "s and the other is not..
Derek J. Lambert
@djlambert
polygon?
Jon West
@ddproxy
Oh yeah
donuts ;)
Derek J. Lambert
@djlambert
the byte order is 01 but that's still valid
03 00 00 20 should be polygon with srid
so that doesn't have any strange flags
Jon West
@ddproxy
I'll figure it out tonight, maybe write a CLI tool to debug with...
Need to grab some lunch :S
Samuel Denis-D'Ortun
@sadortun
Hello guys, we could probably close the following issue since they were resolved ; creof/doctrine2-spatial#121 creof/doctrine2-spatial#54 creof/doctrine2-spatial#128
Derek J. Lambert
@djlambert
#121 has a work around but proxies haven't been implemented yet. I'm not convinced #128 is resolved but not much we can do without more information.
Derek J. Lambert
@djlambert
Currently in wkt-parser and wkb-parser the namespace uses WKT or WKB in all uppercase. The class names in geo-obj are currently CamelCased (Wkt, Wkb). Should I uppercase these to be consistent, or should the "standard" be uppercase namespace, CamelCase classes?
Jon West
@ddproxy
Uh, Uppercase all the things. WKB, WKB are acronyms
Derek J. Lambert
@djlambert
ok thanks. That was my thought on the other libraries. Not sure why I CamelCased it here. lazy fingers
Jon West
@ddproxy
Likewise...
Philippe CARLE
@philippecarle
just wanted to say ¸¸.•¨•♫♪ Thanks ! (it's not enough often in OS world)