SzFMV2018-Tavasz / AutomatedCar

Working repository for the subject "Szoftverfejlesztés multinacionális vállalatoknál" @OE-NIK 2018 tavasz
9 stars 2 forks source link

Reverse radar miscalculates the distance #405

Closed pintergreg closed 6 years ago

pintergreg commented 6 years ago

20:10:34.543 [main] DEBUG hu.oe.nik.szfmv.automatedcar.systemcomponents.ReverseRadar - Gear state is: R 20:10:34.543 [main] DEBUG hu.oe.nik.szfmv.automatedcar.systemcomponents.ReverseRadar - Now the Reverse Radar activated: true 20:10:34.543 [main] DEBUG hu.oe.nik.szfmv.automatedcar.systemcomponents.ReverseRadar - [Object distance: 1.7976931348623157E308] and [Reverse Radar State is: OK]

The Object distance is not changing. kepernyokep_2018-05-02_20-12-17 kepernyokep_2018-05-02_20-07-09

@SzFMV2018-Tavasz/team4 @SzFMV2018-Tavasz/team5 Based on current master.

bench1947 commented 6 years ago
private void activateReverseRadar() {
        reverseRadarPacket.setActivation(true);
        LOGGER.debug("Now the Reverse Radar activated: " + virtualFunctionBus.reverseRadarPacket.getActivation());
        double objectDistance = Double.MAX_VALUE;

        if (objectDistance >= 0.0 && objectDistance <= DANGER_VALUE) {
            reverseRadarState = ReverseRadarState.DANGER;
        }
        if (objectDistance > DANGER_VALUE && objectDistance <= WARNING_VALUE) {
            reverseRadarState = ReverseRadarState.WARNING;
        }
        if (objectDistance > WARNING_VALUE) {
            reverseRadarState = ReverseRadarState.OK;
        }

        reverseRadarPacket.setDistance(objectDistance);
        reverseRadarPacket.setReverseRadarState(reverseRadarState);

        LOGGER.debug("[Object distance: " + virtualFunctionBus.reverseRadarPacket.getDistance()
                + "] and [Reverse Radar State is: " + virtualFunctionBus.reverseRadarPacket.getReverseRadarState()
                + "]");
    }

Ahogy elnézem a mi (team4) metódusaink még nincsenek használva ebben, hanem a double max értéke van átadva neki, és azt írja ki.

pintergreg commented 6 years ago

@bench1947 Teljesen igazad van, kétszeresen is átsiklottam efelett. :open_mouth:

robertvrg commented 6 years ago

Sziasztok, irtam a PR-ben (azt hiszem) hogy csak előkészítettem a cuccot, de még nem számol sajnos. A kijelzes és a kalkuláció folyamatban, holnapra igyekszem pusholni.

robertvrg commented 6 years ago

@SzFMV2018-Tavasz/team4 segítenétek, hogy ne szenvedjek sokat, mely metódusaitokra gondoltatok? :)

bench1947 commented 6 years ago

Az UltrasonicSensor osztályban van 3 metódus:

Az automated car-ból elérhető a 8 szenzor egy listában. Abból ha jól emlékszem a 2. és a 3. szenzor a két hátsó.

robertvrg commented 6 years ago

Igen, a 2. és 3. szenzor lesz az. Nagyon köszönöm, lesem!

robertvrg commented 6 years ago

Javítva lett, elvileg működik szépen. Remélem nemsokára bekerül a team branchbe és megy a PR a masterba. Köszi! 🍺