<html><head><meta http-equiv="Content-Type" content="text/html; charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">I see the discussion on the pull request itself is happening on the pull request and Michael is assisting.<div class=""><br class=""></div><div class="">Regarding the process, I think in general we need to keep the core system stable and it is best not to commit partial implementations. I myself sometime violate that rule, but try not to. The better approach is to create a branch and do the work there (committing and sharing to others as you wish). Then, when all is done and ready, you can merge that branch in to master and submit a pull request. These branches can be either local (private) or on your GitHub clone (public). In rare cases, we also publish branches on the main openvehicles GitHub repository (for example, the SPIRAM branch that was there for a year before merging to master, and the upcoming 3.3 branch I am about to create for the changes to move from v3.2 -> v3.3).</div><div class=""><br class=""></div><div class="">Regards, Mark.<br class=""><div><br class=""><blockquote type="cite" class=""><div class="">On 14 Aug 2020, at 5:18 PM, Soko <<a href="mailto:ovms@soko.cc" class="">ovms@soko.cc</a>> wrote:</div><br class="Apple-interchange-newline"><div class="">
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8" class="">
<div class=""><p class="">I see the point in not having unfinished code in the master so I
removed my OBD implementation from this pull request.<br class="">
Whats left is just the same source as before but with preparation
for the OBD part.</p><p class="">@others: Please let me know your thoughts<br class="">
</p>
<div class="moz-cite-prefix">On 14.08.2020 10:56, Chris van der
Meijden wrote:<br class="">
</div>
<blockquote type="cite" cite="mid:1597395369.7830.3.camel@arachnon.de" class="">
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8" class="">
<div class="">I agree on not further discussing this. I made my point.</div>
<div class=""><br class="">
</div>
<div class="">Chris</div>
<div class=""><br class="">
</div>
<div class="">Am Freitag, den 14.08.2020, 10:02 +0200 schrieb Soko:</div>
<blockquote type="cite" class=""><p class="">I'm not quite sure if we should discuss this here or in the
pull request itself. Maybe Mark or Michael should say whats
best. Anyhow... Let me put you to ease with my <i class="">answers
below...</i><br class="">
</p>
<div class="moz-cite-prefix">On 14.08.2020 09:18, Chris van der
Meijden wrote:<br class="">
</div>
<blockquote type="cite" cite="mid:1597389518.7830.1.camel@arachnon.de" class="">
<meta http-equiv="content-type" content="text/html;
charset=UTF-8" class="">
<div class="">Hey Soko,</div>
<div class=""><br class="">
</div>
<div class="">sorry I don't agree on this pull request.</div>
</blockquote>
<i class="">Just to make things clear with the other guys: You are
talking about one of the pull requests, the one with the
source split.</i><br class="">
<blockquote type="cite" cite="mid:1597389518.7830.1.camel@arachnon.de" class="">
<div class=""><br class="">
</div>
<div class="">We have already discussed this. </div>
<div class=""><br class="">
</div>
<div class="">Lets put this straight. My code for the T26A part is not
ready yet. Climate control is still very buggy and it would
confuse the user to have such a buggy feature within a
possible OVMS release. It will take me probably a week or
two, maybe longer, to get the climate control fixed. Then we
can do a pull request to the master.</div>
</blockquote><p class=""><i class="">The T26A code in my pull request is not the one from your
fork. It's the one that is officially in the master branch
of openvehicles. So nothing changes for the user and not one
source line is different then before.</i><br class="">
</p>
<blockquote type="cite" cite="mid:1597389518.7830.1.camel@arachnon.de" class="">
<div class=""><br class="">
</div>
<div class="">An other point we also talked about, is that you have not
documented your OBD part at all. This is also not
acceptable. You can't put features with in the official OVMS
repository that are not documented at all. How does the user
use your VW e-Up OBD part? What hardware is needed? Where
does the user see the results in the webfrontend? Does the
app work with your code? ...</div>
</blockquote><p class=""><i class="">The user is not able to use any of the OBD code as pointed
out in the pull request description. This is more an
esthetic question as the code is not active. If the others
feel the same I can remove my obd_*.* files from the pull
request. The outcome is the same... </i><br class="">
</p>
<blockquote type="cite" cite="mid:1597389518.7830.1.camel@arachnon.de" class="">
<div class=""><br class="">
</div>
<div class="">You can't put unready code in the master.</div>
<div class=""><br class="">
</div>
<div class="">That is the reason why we use forks. We merge a fork when
we have kind of stable code with features that are well
documented. </div>
<div class=""><br class="">
</div>
<div class="">This is not the case at the moment with the splitted VW
e-Up code.</div>
</blockquote><p class=""><i class="">To sum up: Nothing at all changes for the user with me my
pull request. Well... besides the name of the VW e-Up
vehicle. But is taken care of in the upgrade method.</i></p><p class=""><i class="">I even prepared everything so you have it as easy as
possible when this pull request is put into the master: When
you pull it your fork you only have to backup your three
t26_*.* files from your fork. Then just take 1:1 the
openvehicles-master and put your backup back.<br class="">
At least that's how I would do it as I'm not familiar how
this is done in git directly ;)</i><br class="">
</p>
<blockquote type="cite" cite="mid:1597389518.7830.1.camel@arachnon.de" class="">
<div class=""><br class="">
</div>
<div class="">Regards</div>
<div class=""><br class="">
</div>
<div class="">Chris</div>
<div class=""><br class="">
</div>
<div class=""><br class="">
</div>
<div class="">Am Freitag, den 14.08.2020, 07:06 +0200 schrieb Soko:</div>
<blockquote type="cite" class="">
<pre class="">Hey guys,
Just want to let you know on this way as well about the two pull
requests I have opened yesterday.
Let me know your thoughts. Would be cool if they will be merged very
soon so I can continue my work on the VW e-Up.
regards,
Soko
_______________________________________________
OvmsDev mailing list
<a href="mailto:OvmsDev@lists.openvehicles.com" moz-do-not-send="true" class="">OvmsDev@lists.openvehicles.com</a>
<a href="http://lists.openvehicles.com/mailman/listinfo/ovmsdev" moz-do-not-send="true" class="">http://lists.openvehicles.com/mailman/listinfo/ovmsdev</a>
</pre>
</blockquote>
<br class="">
<fieldset class="mimeAttachmentHeader"></fieldset>
<pre class="moz-quote-pre" wrap="">_______________________________________________
OvmsDev mailing list
<a class="moz-txt-link-abbreviated" href="mailto:OvmsDev@lists.openvehicles.com" moz-do-not-send="true">OvmsDev@lists.openvehicles.com</a>
<a class="moz-txt-link-freetext" href="http://lists.openvehicles.com/mailman/listinfo/ovmsdev" moz-do-not-send="true">http://lists.openvehicles.com/mailman/listinfo/ovmsdev</a>
</pre>
</blockquote>
<pre class="">_______________________________________________
OvmsDev mailing list
<a href="mailto:OvmsDev@lists.openvehicles.com" moz-do-not-send="true" class="">OvmsDev@lists.openvehicles.com</a>
<a href="http://lists.openvehicles.com/mailman/listinfo/ovmsdev" moz-do-not-send="true" class="">http://lists.openvehicles.com/mailman/listinfo/ovmsdev</a>
</pre>
</blockquote>
<br class="">
<fieldset class="mimeAttachmentHeader"></fieldset>
<pre class="moz-quote-pre" wrap="">_______________________________________________
OvmsDev mailing list
<a class="moz-txt-link-abbreviated" href="mailto:OvmsDev@lists.openvehicles.com">OvmsDev@lists.openvehicles.com</a>
<a class="moz-txt-link-freetext" href="http://lists.openvehicles.com/mailman/listinfo/ovmsdev">http://lists.openvehicles.com/mailman/listinfo/ovmsdev</a>
</pre>
</blockquote>
</div>
_______________________________________________<br class="">OvmsDev mailing list<br class=""><a href="mailto:OvmsDev@lists.openvehicles.com" class="">OvmsDev@lists.openvehicles.com</a><br class="">http://lists.openvehicles.com/mailman/listinfo/ovmsdev<br class=""></div></blockquote></div><br class=""></div></body></html>