Skip to content

some animations make too many assumptions about the number of pixels  #3

@waldo323

Description

@waldo323

self.strand_length should be used with some additional figuring to replace these instances

example:
def middle_to_edges(self, red=0, green=0, blue=0):
for x in sorted(list(range(0, self.strand_length//2))):
self.lightup(x+5, red, green, blue)
self.lightup(4-x, red, green, blue)
time.sleep(.05)

def edges_to_middle(self, red=0, green=0, blue=0):
    for x in sorted(list(range(0, self.strand_length//2)), reverse=True):
        self.lightup(x+5, red, green, blue)
        self.lightup(4-x, red, green, blue)
        time.sleep(.05)

should be:

def middle_to_edges(self, red=0, green=0, blue=0):
for x in sorted(list(range(0, self.strand_length//2))):
self.lightup(x+self.strand_length//2, red, green, blue)
self.lightup((self.strand_length//2 -1) - x, red, green, blue)
time.sleep(.05)

def edges_to_middle(self, red=0, green=0, blue=0):
    for x in sorted(list(range(0, self.strand_length//2)), reverse=True):
        self.lightup(x+self.strand_length//2, red, green, blue)
        self.lightup((self.strand_length//2-1)-x, red, green, blue)
        time.sleep(.05)

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions