seven segment display help

Swap HIGH & LOW here:

void display_time(int& minute, int& second)
{
  digitalWrite( GND4, HIGH);    //digit 4 <<< Change to LOW for all 4 digits
  pickNumber(second%10);
  delay(DTime);
  digitalWrite( GND4, LOW);               <<< Change to HIGH for all 4 digits

  digitalWrite( GND3, HIGH);    //digit 3
  pickNumber(second/10);
  delay(DTime);
  digitalWrite( GND3, LOW);

  digitalWrite( GND2, HIGH);   //digit 2
  pickNumber(minute%10);
  delay(DTime);
  digitalWrite( GND2, LOW);
 
  digitalWrite( GND1, HIGH);   //digit 1
  pickNumber(minute/10);
  delay(DTime);
  digitalWrite( GND1, LOW);

}

alright so i did that code change and it looks a bit better.. i am getting weird interference now.. when it shows the display it shows 88:00 which should be 06:00... also when i change the seconds it shows the numbers when it gets to 3 or so it shows same number in 10 sec spot.. and when i change 10 sec it shows on all of them also.. and when i change 1min it shows on the 10 min.. ectt.. so they are mashing up?

CrossRoads:
Like so. Arduino drives PNP base low to turn it, and segment cathode low to turn the segment on.
PNP used is not particularly sensitive. Must be able to supply 160mA Id, so a >=200mA continuous on part will do.

It probably doesn't help that your example schematic has NPNs instead of PNPs.

not my schematic

santino:
as my display.. now i have NO resistors and NO transistors .. i was just doing this just to see what it displays quickly and it shows the code no issue? ..

Taking shortcuts is a good way to destroy your Arduino or displays.

Resistors and transistors are used for a reason. Don't omit them, especially not just to "save time". It may create more problems than it's worth.

so any thoughts on my issue still?

How about an updated schematic and code listing? I'm lost as to what is connected and running.

Schematic looks correct (but needs work to make it less like spaghetti).

Now post your whole sketch.

where do R1 to R4 go?

R1 to R4 go to the pins on the arduino i didnt add them as i said it would look too much like a mess on the schematic for everyone..

santino:
R1 to R4 go to the pins on the arduino i didnt add them as i said it would look too much like a mess on the schematic for everyone..

Does that package have inter sheet links?

im sorry i dont understand.

Grumpy_Mike:

santino:
R1 to R4 go to the pins on the arduino i didnt add them as i said it would look too much like a mess on the schematic for everyone..

Does that package have inter sheet links?

i dont understand what you mean.. "inter sheet links"

in inter sheet link is a symbol that "says" this wire is joined to that wire but I am not going to show the wire because it will make the circuit messy. The normally look like a box with an arrowed end.

You can use the convention of a bus for showing multiple wiring and it looks a lot neater.

links.png

seven seg bus.png

ahh ok.. yes i use them.. but as i said i just drew up schematic to show how it was hooked up.. there is WAY more i have to add to it which is why i havent made one yet.. everything is on two breadboards right now.. i have 6 buttons with this circuit also and a few other bits.. but for now i need to fix my display before anything else.. so my schematic was just to show how the display was wired.. its not a finished schematic as i am not sure where all the pins will end up..

You still need to post ALL your code not just bits of it.

this is a joint project with a friend and he doesnt want to give me the full code.

Look this is stupidly simple code.
Your so called friend might think he has invented something clever that needs protecting but he has not.

If you are not going to post all of it you are wasting everyone time. That means posting all the code either in a code window or as an attachment if it is too big.

which is understandable.

Why is that then?

he told me this is plenty of it to get the display working.

If it is working then why are you asking questions here.

I think you need to adjust your attitude. This is an open source community and we have little time for people who want to keep secrets, especially over such a trivial matter.

Grumpy_Mike:
If you are not going to post all of it you are wasting everyone time. That means posting all the code either in a code window or as an attachment if it is too big.

+1