MDL29
/
JacoBot
10
2
Fork 8
Code Issues 25 Pull Requests 8 Projects Releases 1 Activity

Improve JacoVirt Bot window #60

Closed
Bertrand-DUCON wants to merge 6 commits from Bertrand-DUCON/JacoBot:jacovirt_bot_window into main
pull from: Bertrand-DUCON/JacoBot:jacovirt_bot_window
merge into: MDL29:main
Conversation 1 Commits 6 Files Changed 1 +71 -1
Bertrand-DUCON commented 2024-04-25 10:03:37 +00:00
Member
Copy Link
  • Add Sprite Square as the carpet #72
- Add Sprite Square as the carpet #72
Bertrand-DUCON added 2 commits 2024-04-25 10:03:38 +00:00
Bertrand-DUCON changed title from UpJacoVirt Bot window to Improve JacoVirt Bot window 2024-04-25 10:05:16 +00:00
Bertrand-DUCON added the
Feature
JacoVirt
JacoBot
 labels 2024-04-25 10:05:57 +00:00
Bertrand-DUCON added 1 commit 2024-04-25 13:05:40 +00:00
Bertrand-DUCON added 1 commit 2024-04-25 13:12:17 +00:00
Bertrand-DUCON added 1 commit 2024-04-25 13:13:44 +00:00
Bertrand-DUCON added 1 commit 2024-04-25 13:22:10 +00:00
HS-157 requested changes 2024-04-26 20:10:16 +00:00
jacovirt/jacovirt/bot/window.py
@ -12,1 +47,3 @@
BACKGROUND_COLOR = arcade.color.CATALINA_BLUE
BACKGROUND_COLOR = arcade.color.OXFORD_BLUE
class Case(arcade.SpriteSolidColor):
HS-157 commented 2024-04-26 20:04:31 +00:00
Owner
Copy Link

Mettre cette classe dans un autre fichier et je l'aurai comme nom Cell.

Mettre cette classe dans un autre fichier et je l'aurai comme nom `Cell`.
jacovirt/jacovirt/bot/window.py
@ -13,0 +50,4 @@
"""Case sprite"""
# Count instance case classes
ids = count(0)
HS-157 commented 2024-04-26 20:07:06 +00:00
Owner
Copy Link

Pour le moment, je ne vois pas l'utilité d'avoir un compteur pour mettre des ID aux cases.

Pour le moment, je ne vois pas l'utilité d'avoir un compteur pour mettre des ID aux cases.
jacovirt/jacovirt/bot/window.py
@ -24,3 +75,4 @@
"""Set up the pad"""
logging.info("Set up the bot.")
self.case_list = arcade.SpriteList()
HS-157 commented 2024-04-26 20:07:49 +00:00
Owner
Copy Link

Je ne pense pas que cette ligne devrait être dans cette méthode, mais plus dans le __init__().

Je ne pense pas que cette ligne devrait être dans cette méthode, mais plus dans le `__init__()`.
jacovirt/jacovirt/bot/window.py
@ -30,3 +94,4 @@
# Clear the screen
self.clear()
arcade.draw_lrtb_rectangle_filled(0, 0, 150, 150, arcade.color.SHAMROCK_GREEN)
HS-157 commented 2024-04-26 20:09:39 +00:00
Owner
Copy Link

Je ne vois pas où est dessiné cette forme et à quoi elle sert.

Je ne vois pas où est dessiné cette forme et à quoi elle sert.
HS-157 self-assigned this 2024-04-26 20:10:33 +00:00
HS-157 commented 2024-05-12 17:24:53 +00:00
Owner
Copy Link

Canceled & replaced by MDL29/JacoBot#79

Canceled & replaced by MDL29/JacoBot#79
HS-157 closed this pull request 2024-05-12 17:24:53 +00:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
HS-157
Labels
Clear labels
  
Bug

Something is not working

  
Feature

New feature

  
Fix

Fix a feature

  
Hardware

Hardware & device

  
JacoBot

The robot

  
JacoPad

Pad & tokens

  
JacoVirt

Python JacoVirt

  
Wontfix

This won't be fixed

No Label
Bug
Feature
Fix
Hardware
JacoBot
JacoPad
JacoVirt
Wontfix
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
HS-157
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: MDL29/JacoBot#60
No description provided.
Delete Branch "Bertrand-DUCON/JacoBot:jacovirt_bot_window"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?