Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Dec 20 2020 19:07

    gillesdemey on master

    chore: add deprecation note (compare)

  • Oct 29 2017 04:05
    yasushi-saito synchronize #10
  • Oct 24 2017 04:10
    yasushi-saito synchronize #10
  • Oct 23 2017 03:52
    yasushi-saito synchronize #10
  • Oct 22 2017 20:08
    yasushi-saito synchronize #10
  • Oct 10 2017 04:57
    yasushi-saito synchronize #10
  • Oct 06 2017 08:47
    gillesdemey commented #10
  • Oct 06 2017 04:33
    yasushi-saito synchronize #10
  • Oct 05 2017 22:11
    kofalt commented #10
  • Oct 05 2017 20:29
    yasushi-saito synchronize #10
  • Oct 05 2017 03:10
    yasushi-saito synchronize #10
  • Oct 04 2017 18:04
    yasushi-saito synchronize #10
  • Oct 04 2017 05:16
    yasushi-saito synchronize #10
  • Oct 04 2017 00:17
    yasushi-saito synchronize #10
  • Sep 29 2017 04:06
    yasushi-saito synchronize #10
  • Sep 28 2017 20:14
    yasushi-saito synchronize #10
  • Sep 28 2017 04:44
    yasushi-saito synchronize #10
  • Sep 28 2017 04:25
    yasushi-saito synchronize #10
  • Sep 25 2017 02:59
    yasushi-saito synchronize #10
  • Sep 25 2017 02:53
    yasushi-saito synchronize #10
Gilles De Mey
@gillesdemey
Ugh da duurt hier altijd te lang xd
Nog wakker? :p
Die XXXX-XXXX groups zijn allemaal deprecated anyway, dus die moeten we ni opslaan
Gilles De Mey
@gillesdemey
Also, de lookupTag functie gaat die hashmap aanmaken als die nog ni ingevuld is
Da houdt de API nice en clean
Seppe Stas
@Bitbored
*[slice-dictionary][~/go/src/github.com/gillesdemey/go-dicom]$ go test -bench Find
testing: warning: no tests to run
PASS
BenchmarkFindMetaGroupLengthTag 500000000 6.49 ns/op
BenchmarkFindPixelDataTag 500000000 7.06 ns/op
ok github.com/gillesdemey/go-dicom 9.295s
is correcter
uw test deed het marshallen van de file in de benchmark
die van mij niet
mijn code is nog steed way faster though
Gilles De Mey
@gillesdemey
? Het marshallen moet gebeuren wanneer er nog geen dictionary is ingelezen
De user moet da ni manueel doen e :p
Da moet onderdeel zijn van de benchmarks
Seppe Stas
@Bitbored
ja, maar bij u gebeurt dat in de lookup call
waarom?
Gilles De Mey
@gillesdemey
Als hij nog ni bestaat, ja :p
Seppe Stas
@Bitbored
ik vindt mijn implementatie veel loischer
ge kunt als gebruiker definieren welke dictenary er wordt gebruikt
Gilles De Mey
@gillesdemey
Uw implementatie zegt dat de gebruiker zelf de dictionary moet inlezen, glijk in de examples file
Seppe Stas
@Bitbored
ja
Gilles De Mey
@gillesdemey
That's bad
De user should not care
Seppe Stas
@Bitbored
dat geeft de mogelijkheid om bv een string ipv een file te gebruiken
Gilles De Mey
@gillesdemey
Als ge de huidige impementatie extend ook
Seppe Stas
@Bitbored
...
hoe dan?
Gilles De Mey
@gillesdemey
Geef een option mee aan u parser, en ge specified uw dictionary file of string
Seppe Stas
@Bitbored
en hoe wete dan of het een directory file is of een string?
Gilles De Mey
@gillesdemey
Beter om een functie aan te spreken me een option dan eerst de gebruiker wijs maken dat em 100 lijnen code moe schrijven voordat het werkt
Seppe Stas
@Bitbored
euh, de parse neemt ook een byte[] binnen ipv een filename
Gilles De Mey
@gillesdemey
Dan veranderde da wa e :p
Seppe Stas
@Bitbored
NEE
Gilles De Mey
@gillesdemey
De interne API's liggen ni vast, de externe API die we exposen moet zo simpel mogelijk zijn
Seppe Stas
@Bitbored
er is een rede waarom de interfaces van zoveel go libraries byte[] gebruiken
of readers
of een byte[], of een reader
io.Reader
generiek
en liest nog een reader eigenlijk
dan kunde zelfs uw dic file streamen als ge daar goesting in hebt
Gilles De Mey
@gillesdemey
Ksnap ni hoe da onmogelijk is door de externe API ni te veranderen?
Seppe Stas
@Bitbored
een filepath beperkt u tot enkel files inlezen...
= poop
Gilles De Mey
@gillesdemey
Want die dictionary komt van een string, how?
Seppe Stas
@Bitbored
welke string?
skype?
Gilles De Mey
@gillesdemey
Gesprek is eindelijk gedaan xd
Ja cva
Kga om drinken :p
Bel ma