Skip to content

Comments

Add script to reset EVC water pump counters (Zoé)#8

Open
Jodaille wants to merge 13 commits intocedricp:masterfrom
Jodaille:master
Open

Add script to reset EVC water pump counters (Zoé)#8
Jodaille wants to merge 13 commits intocedricp:masterfrom
Jodaille:master

Conversation

@Jodaille
Copy link

@Jodaille Jodaille commented Feb 28, 2025

I don't have DTC error anymore to test in real conditions.

Screenshots in Edition mode:
Capture d’écran du 2025-02-28 22-25-30
Capture d’écran du 2025-02-28 22-29-16

It should reset:
($3349) Time Counter for the driving WEP in Low Speed
($334A) Time Counter for the driving WEP in Middle Speed
($334B) Time Counter for the driving WEP in High Speed
and
($3531) V_Timer_DrivWEP_ON

@Jodaille
Copy link
Author

Jodaille commented Mar 1, 2025

Hello,

I have test reset, it does not work :(

    reset_request = self.evc_ecu.requests[u"DataWrite.($3349) Time Counter for the driving WEP in Low Speed"]
    request_response = reset_request.send_request()

I wonder if it code that do not work or if I need to start the car.
Response seems to be: None.

Reading values are ok:
Capture d’écran du 2025-03-01 14-23-26

<Request Name="DataWrite.($3349) Time Counter for the driving WEP in Low Speed"> <DossierMaintenabilite/> <ManuelSend/> <Sent> <SentBytes>2E334900000000</SentBytes> <DataItem Name="($3349) Time Counter for the driving WEP in Low Speed" FirstByte="4"/> </Sent> <Received MinBytes="3"> <ReplyBytes>6E3349</ReplyBytes> </Received> </Request>

@Jodaille
Copy link
Author

Jodaille commented Mar 2, 2025

I have add buttons for reseting each counter
and tips zone
Capture d’écran du 2025-03-02 14-43-38

@Furtif
Copy link
Contributor

Furtif commented Mar 5, 2025

@cedricp merge this pr please

Furtif added a commit to cedricp/ddt4all that referenced this pull request Mar 6, 2025
@Furtif
Copy link
Contributor

Furtif commented Mar 6, 2025

@Jodaille


Traceback (most recent call last):
  File "./ddtplugins/zoe_waterpump_counter_reset.py", line 212, in reset_ecu
    self.reset_medium_speed_counter()
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'Virginizer' object has no attribute 'reset_medium_speed_counter'. Did you mean: 'get_medium_speed_counter'?

Unresolved attribute reference 'reset_medium_speed_counter' for class 'Virginizer'

miss maybe :

def reset_medium_speed_counter(self):
        pass

@Jodaille
Copy link
Author

Jodaille commented Mar 6, 2025

Hello,

sorry I have missed to rename this one

Furtif added a commit to cedricp/ddt4all that referenced this pull request Mar 7, 2025
@Jodaille
Copy link
Author

Jodaille commented Mar 9, 2025

Thank you !
I didn't know we could associate submodule to a commit in a pull request :)

@Furtif
Copy link
Contributor

Furtif commented Mar 9, 2025

Thank you !
I didn't know we could associate submodule to a commit in a pull request :)

Yep for test, but need merge and fix pointer

I not have rights in this repo need wait @cedricp

@Furtif
Copy link
Contributor

Furtif commented Mar 11, 2025

addeded directly cedricp/ddt4all#1380

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants