CacheAdminA_trunk2010/APPS/Halux/PPS

Verpakking

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 8
Last Commit: 22 Jun 15
Commits this week: 0
Total Lines of Code (LoC): 1.271
Net change in LoC this week: 0

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Thursday 04 Apr 2024
12:33 pm

Peter Vercoutter

[ICT] Tijdelijk wat code toevoegen voor Sofie zodat ze aansturing van Karton voor lbx en mvx kan testen
Options
    • -0
    • +50
    ./impl/LadeKartonWerkpost.cls.xml
Monday 11 Mar 2024
2:12 pm

Peter Vercoutter

11:30 am

Joannes Lemmens

Friday 08 Mar 2024
12:51 pm

Peter Vercoutter

[ICT-4261] TBX/LBX/MVX/TAX - in alle gevallen met standaard afmetingen van onderdelenhuls werken
[ICT-4261] TBX/LBX/MVX/TAX - in alle gevallen met standaard afmetingen van onderdelenhuls werken
Wednesday 21 Feb 2024
9:41 am

Peter Vercoutter

[ICT-4261] TBX/LBX/MVX/TAX - in alle gevallen met standaard afmetingen van onderdelenhuls werken

- Correctie uitbreidingen standaardHulsDefinities

Options
    • -2
    • +2
    ./utils/StandaardHulsDefinities.cls.xml
9:34 am

Peter Vercoutter

[ICT-4261] TBX/LBX/MVX/TAX - in alle gevallen met standaard afmetingen van onderdelenhuls werken

- Uitbreidingen standaardHulsDefinities

Options
    • -1
    • +113
    ./utils/StandaardHulsDefinities.cls.xml
Wednesday 07 Feb 2024
3:44 pm

Peter Vercoutter

[ICT-4319] Alle onderdelendozen (hulzen) standaardiseren

- Alle LBX hulzen standardiseren

Options
    • -0
    • +70
    ./utils/StandaardHulsDefinities.cls.xml
  2. … 2 more files in changeset.
Tuesday 11 Jul 2023
4:11 pm

Peter Vercoutter

[ICT-3758] Hulzen taor batchen worden gemaakt op verkeerde machine
Options
    • -1
    • +1
    ./impl/DoosLosseOnderdelen.cls.xml
Thursday 22 Jun 2023
3:19 pm

Peter Vercoutter

1:59 pm

Tom Vermeulen

Monday 19 Jun 2023
11:01 am

Peter Vercoutter

[ICT-3721] [rvTVE] Onderdelendozen niet beschikbaar op kartonmachine Lijn 1
[ICT-3721] [rvTVE] Onderdelendozen niet beschikbaar op kartonmachine Lijn 1
11:01 am

Peter Vercoutter

[ICT-3721] [rvTVE] Onderdelendozen niet beschikbaar op kartonmachine Lijn 1

- Onderdelendozen niet enkel naar taor of KraftKlein sturen. Afhandelen naargelang de gebruiker het wenst.

Options
    • -1
    • +4
    ./impl/DoosLosseOnderdelen.cls.xml
Wednesday 25 Jan 2023
12:52 pm

Peter Vercoutter

Monday 09 Jan 2023
11:56 am

Peter Vercoutter

Wordt indien nodig aangevuld in de individuele aanvul method, zie VulAanOverdoos

Wordt indien nodig aangevuld in de individuele aanvul method, zie VulAanOverdoos

11:24 am

Peter Vercoutter

De optimizer gaat data zoeken met dikte 3 voor kraft klein en 4.2 voor bv halux. Nadat data gevonden is moeten we er een geheel getal van maken, anders kan het Kraftmachine dit niet interpreteren. ...

De optimizer gaat data zoeken met dikte 3 voor kraft klein en 4.2 voor bv halux. Nadat data gevonden is moeten we er een geheel getal van maken, anders kan het Kraftmachine dit niet interpreteren. Er is dus een verschil tussen voor en na de optimalisatie

10:02 am

Peter Vercoutter

[ICT-3001] VHDoosService opzetten

- Eerste reeks reviewopmerkingen committen

Options
    • -9
    • +3
    ./impl/KartonSnijVisitor.cls.xml
    • -9
    • +3
    ./impl/DoosLosseOnderdelen.cls.xml
  3. … 13 more files in changeset.
9:00 am

Peter Vercoutter

Ik snap niet wat je hiermee wilt zeggen. De set BreedtePlano wordt in deze optimizer maximum maar 1 keer gezet per recept?

Ik snap niet wat je hiermee wilt zeggen. De set BreedtePlano wordt in deze optimizer maximum maar 1 keer gezet per recept?

Wednesday 04 Jan 2023
9:55 am

Jelle Baele

Tuesday 03 Jan 2023
3:37 pm

Jelle Baele

Wat met fouten en standaardgedrag naar nodered? Lijkt mij niet zo heel veel werk aangezien je al een deel kan kopiëren / dezelfde mocks kan gebruiken http://subversion02.vanhoecke.be/static/ogdo0b/...

Wat met fouten en standaardgedrag naar nodered? Lijkt mij niet zo heel veel werk aangezien je al een deel kan kopiëren / dezelfde mocks kan gebruiken .

3:34 pm

Jelle Baele

Dit allemaal bevat enorm veel logica en komt nogal overweldigend over (net zoals Wim aanhaalt in zijn review hierboven) en maakt het vrij moeilijk om te lezen + onder UT te plaatsen. Ik persoonlijk...

Dit allemaal bevat enorm veel logica en komt nogal overweldigend over (net zoals Wim aanhaalt in zijn review hierboven) en maakt het vrij moeilijk om te lezen + onder UT te plaatsen. Ik persoonlijk zou dit allemaal wat meer afzonderen en toch wat meer onder unittest plaatsen. Zeker omdat als hier ooit een wijziging aan gebeurt, je mooi kan zien waarom iets faalt. (Ik vind het persoonlijk ook moeilijk hoor om hier een goede structuur voor te bedenken, maar indien nodig wil ik gerust eens mee nadenken )

3:26 pm

Jelle Baele

Ik zou dit ook in een aparte method plaatsen (Do ..Swap(...)) opdat dit alles iets beter leesbaar wordt

Ik zou dit ook in een aparte method plaatsen (Do ..Swap(...)) opdat dit alles iets beter leesbaar wordt

3:25 pm

Jelle Baele

Waarom staat deze hier? Is dat ook niet recept specifiek? EDIT: Dit wordt ook in de optimizer aangepast, dus misschien is het logisch om deze naar daar / aparte klasse te plaatsen?

Waarom staat deze hier? Is dat ook niet recept specifiek?

EDIT: Dit wordt ook in de optimizer aangepast, dus misschien is het logisch om deze naar daar / aparte klasse te plaatsen?

3:14 pm

Jelle Baele

Als algemene opmerking voor deze klasse zou ik toch een aantal dingen afzonderen in aparte methods / klasses. Zoals deze kan gerust in een aparte method (Set Prioriteit = ..GeefPrioriteit(MachineTe...

Als algemene opmerking voor deze klasse zou ik toch een aantal dingen afzonderen in aparte methods / klasses. Zoals deze kan gerust in een aparte method (Set Prioriteit = ..GeefPrioriteit(MachineTechnologie).

3:02 pm

Jelle Baele

Lijntje commentaar mag weg?

Lijntje commentaar mag weg?

2:59 pm

Jelle Baele

Is maar een idee, maar aangezien er hier veel 'code duplication' is, kan je dit eventueel in een aparte method steken a la '..GeefParam(Key, Value)

Is maar een idee, maar aangezien er hier veel 'code duplication' is, kan je dit eventueel in een aparte method steken a la '..GeefParam(Key, Value)

2:54 pm

Jelle Baele

Ik zou dit als property in de recepten apart plaatsen en opvullen zoals hieronder. Als dit later zou wijzigen, of er meerdere nodig zijn, moet je die sowieso ergens ophalen lijkt mij

Ik zou dit als property in de recepten apart plaatsen en opvullen zoals hieronder. Als dit later zou wijzigen, of er meerdere nodig zijn, moet je die sowieso ergens ophalen lijkt mij

2:52 pm

Jelle Baele

Ik zou dit en de overigen in een aparte folder plaatsen als dat nog lukt, iets met dto of iets dergelijks. EDIT: Ik zie dat Wim deze opmerking ook bij de 'Doos.cls' heeft geplaatst, negeer deze da...

Ik zou dit en de overigen in een aparte folder plaatsen als dat nog lukt, iets met dto of iets dergelijks.

EDIT: Ik zie dat Wim deze opmerking ook bij de 'Doos.cls' heeft geplaatst, negeer deze dan maar

Wednesday 21 Dec 2022
6:27 pm

Wim Vermeulen

6:26 pm

Wim Vermeulen

Deze, en volgende grote methods, zal ik niet reviewen ... :-P

Deze, en volgende grote methods, zal ik niet reviewen ... :-P

6:24 pm

Wim Vermeulen

Bij sommige OptimizeTralala methods staat enkel een IF (...) then set BreedtePlano = iets Wanneer deze method een tweede keer wordt opgeroepen, en de IF conditie is niet voldaan, dan kon je in een ...

Bij sommige OptimizeTralala methods staat enkel een IF (...) then set BreedtePlano = iets
Wanneer deze method een tweede keer wordt opgeroepen, en de IF conditie is niet voldaan, dan kon je in een gevaarlijke situatie.

Mogelijke oplossing : in t begin van de Optimize() method de waarde resetten.