PBP IF,THEN help


Closed Thread
Results 1 to 11 of 11

Hybrid View

  1. #1
    Join Date
    Mar 2003
    Location
    Commerce Michigan USA
    Posts
    1,166


    Did you find this post helpful? Yes | No

    Default

    paulcox82, I notice that you don't wait for the motor to stop before you are telling it to go again.

    if (PT1=1) and (PT2=1) and(PT3=1) and(PT4=0) and(PT5=0) and(PT6=0) Then
    low CCWDRV 'stop motor at 111000 position
    high CCWDRV 'start motor for last leg of travel

    and in the statement,

    if (PT1=1) and (PT2=1) and(PT3=1) and(PT4=0) and(PT5=0) and(PT6=0) Then
    low CWDRV 'stop motor at 111000 position
    high CWDRV 'start motor for last leg of travel

    Why is that? In the time it takes to execute the low statement which is probably in the range of a few uS. the output port is being told to go high once again... Your code will probably overshoot the limit..

    Dave Purola,
    N8NTA

  2. #2
    Join Date
    Sep 2009
    Location
    Olympia, WA
    Posts
    5


    Did you find this post helpful? Yes | No

    Default

    I will modify the AND condition statements and include a 100ms pause between motor movemnts.

    Thanks for the help.

  3. #3
    Join Date
    Sep 2009
    Location
    Olympia, WA
    Posts
    5


    Did you find this post helpful? Yes | No

    Default

    I removed the "AND" functions and replaced them with direct port readings and added 100ms delays between motor commands. I have also added a command to set the inputs to digital. The problem still persists, though my memory usage has greatly reduced.

    Anybody have any other ideas that may work to resolve this issue?

    It may be the hysterisis of the potentiometer, but I am scratching my head on how to eliminate this in code as each comparator is set high.

    Thanks for the help,
    Paul Cox

    here is my updated code:

    TRISA=%11111000 'make A0, A1 & A2 outputs
    TRISB=%11101111 'make B0, B1, B2, B3, B5, B6 & B7 inputs

    CCWDRV var PortA.0 'change port name
    CWDRV var PortA.1 'change port name
    LED var PortA.2 'change port name
    CNTR var PortB.0 'change port name
    PT1 var PortB.7 'change port name
    PT2 var PortB.6 'change port name
    PT3 var PortB.5 'change port name
    PT4 var PortB.3 'change port name
    PT5 var PortB.2 'change port name
    PT6 var PortB.1 'change port name

    CMCON = 7

    cycle:

    if PortB=%00000000 then '000 000 right stop
    low LED
    pause 50
    high LED
    pause 500
    low LED
    pause 500
    goto cycle
    endif

    if PortB=%11101110 then '111 111 left stop
    low LED
    pause 50
    high LED
    pause 500
    low LED
    pause 500
    goto cycle
    endif

    if PortB=%00000001 then '000 000 center from right stop
    low LED
    pause 50
    high CCWDRV
    pause 500
    gosub CCWCENTER 'go to sub-routine to center from right
    goto cycle
    endif

    if PortB=%11101111 then '111 111 center from left stop
    low LED
    pause 50
    high CWDRV
    pause 500
    gosub CWCENTER 'go to sub-routine to center from left
    goto cycle
    endif

    if PortB=%00000011 then '100 000
    low LED
    pause 50
    high CCWDRV
    pause 500
    gosub CCWCENTER 'go to sub-routine to center from right
    goto cycle
    endif

    if PortB=%00000111 then '110 000
    low LED
    pause 50
    high CCWDRV
    pause 500
    gosub CCWCENTER 'go to sub-routine to center from right
    goto cycle
    endif

    if PortB=%00101111 then '111 100
    low LED
    pause 50
    high CWDRV
    pause 500
    gosub CWCENTER 'go to sub-routine to center from left
    goto cycle
    endif

    if PortB=%01101111 then '111 110
    low LED
    pause 50
    high CWDRV
    pause 500
    gosub CWCENTER 'go to sub-routine to center from left
    goto cycle
    endif

    GOTO cycle

    CCWCENTER
    if PortB=%00001111 then
    low CCWDRV 'stop motor at 111000 position
    pause 1000 'change to 250ms after debugging
    high CCWDRV 'start motor for last leg of travel
    pause 2000 'run time to go from 15 degrees to center CCW
    low CCWDRV
    endif
    return

    CWCENTER
    if PortB=%00001111 then
    low CWDRV 'stop motor at 111000 position
    pause 1000 'change to 250ms after debugging
    high CWDRV 'start motor for last leg of travel
    pause 2000 'run time to go from 15 degrees to center CW
    low CWDRV
    endif
    return

    end

  4. #4
    Join Date
    Jun 2007
    Posts
    24


    Did you find this post helpful? Yes | No

    Default Labels

    Don't know if it's just a typo, but the sub-routine labels need colons.

    CCWCENTER:
    CWCENTER:

    I don't think they work otherwise

    Mike

  5. #5
    Join Date
    Mar 2003
    Location
    Commerce Michigan USA
    Posts
    1,166


    Did you find this post helpful? Yes | No

    Default

    paulcox82, Is there any dynamic braking on the motor drive circuits? How big is the motor and is there any gearing involved? Where is the pot in relationship to the motor(ie, any gears in between?) I dont beleive 100 milliseconds is enough to stop the motor especially if there is no dynamic braking of the motor itself.. How long does it take for the motor to come to a complete stop after commanding to shut off? This time should be the minimum time for the "off delay" and the travel in degrees should be the minimum for the comparators sensing position... Could you please supply a circuit diagram to better understand the drive?

    Dave Purola,
    N8NTA

  6. #6
    Join Date
    Sep 2009
    Location
    Olympia, WA
    Posts
    5


    Did you find this post helpful? Yes | No

    Default

    Mike,

    The missing colons were a typo. Thank you. I added them, but still have the same issue.

    Dave,

    The motor is a small piezo driven motor that will eventually move through some gearing, but very slowly. On the order of 2 RPM. I am simulating the motor position potentiometer by moving an external pot by hand. I will increase the wait periods between motor movements and let you know the results.

    Thanks for the help.
    Paul

  7. #7
    Join Date
    Sep 2009
    Location
    Olympia, WA
    Posts
    5


    Did you find this post helpful? Yes | No

    Default PIC schematic

    As requested, here is the schematic for the motor controller section.

    -Paul
    Attached Images Attached Images

Similar Threads

  1. PBP Book
    By Bruce in forum Off Topic
    Replies: 83
    Last Post: - 4th October 2021, 12:55
  2. Extensions to PBP variables
    By John_Mac in forum mel PIC BASIC Pro
    Replies: 5
    Last Post: - 23rd October 2009, 05:21
  3. Compiler differences between PBP 2.33 & 2.46
    By nikopolis in forum mel PIC BASIC Pro
    Replies: 3
    Last Post: - 2nd May 2006, 19:01
  4. Newby- PBP wont compile for 18F (MPLAB)
    By jd76duke in forum mel PIC BASIC Pro
    Replies: 1
    Last Post: - 17th December 2005, 23:30
  5. PBP / XP Crash
    By pondindustrial in forum mel PIC BASIC Pro
    Replies: 1
    Last Post: - 27th November 2005, 03:16

Members who have read this thread : 0

You do not have permission to view the list of names.

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts