Juan-Antonio S. E. Pedersen

Results 28 comments of Juan-Antonio S. E. Pedersen

Hi! Tjek out the github issue I filed. There is a better Way to index:) tir. 26. okt. 2021 kl. 20.24 skrev runger1101001 ***@***.***>: > Ok, looking at the theoretical...

Sorry, I thought we where in SimpleFoc community. You are right, but this is because the attribute is set in the variant file. There is a conflict between the attribute...

I can confirm, that changing the variant file from PIN_ATTR_PWM_G to PIN_ATTR_PWM_F will make it choose TCC1. So it is on the board maker to make sure the variant file...

I ended up just putting "NOT_ON_TIMER" to make it skip the wrong TC association, and use TCC1. Until there is a proper fix, this will work fine. ![Not on timer](https://user-images.githubusercontent.com/4369438/139205746-25d2f15f-94dd-4c4c-b1bd-b2dba4b09474.JPG)

I did a bit of re-arranging on the wiring_analog file ``` #if defined(__SAMD51__) || defined(_SAME51_) Adc *adc; if(g_APinDescription[pin].ulPinAttribute & PIN_ATTR_ANALOG) adc = ADC0; else if(g_APinDescription[pin].ulPinAttribute & PIN_ATTR_ANALOG_ALT) adc = ADC1;...

I have experienced some shatter while connected to simpleFocStudio. When I disconnect it went away....

This function is calling updatestates(); which is sending MG (monitor Get) commands. class StateUpdateRunner(QtCore.QThread): def __init__(self, connector=None, *args,**kwargs): super(StateUpdateRunner, self).__init__(*args, **kwargs) self._stop_event = threading.Event() self.deviceConnector = connector def run(self): try:...

Ah, Ok my mistake, I now see why the SFoCStudio "Simplefoc digital read out" section is so unresponsive. If you completely change the motor.monitor(); function, so that it uses the...

Yes, the StepperDriver8PWM::setPwm(float Ualpha, float Ubeta) function is intended to be called by the setPhaseVoltage function in order to set the appropriate PWM duty cycles based on the calculated alpha...

> Hey, also it doesn't pass the checks for any architectures, not even STM32... I think adding 8-PWM support would be a cool addition, but the PR needs a bit...