Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Multi-thread unsafe cards #34

Open
11 tasks done
rnilva opened this issue Aug 3, 2017 · 4 comments
Open
11 tasks done

Multi-thread unsafe cards #34

rnilva opened this issue Aug 3, 2017 · 4 comments
Labels
ongoing

Comments

@rnilva
Copy link
Contributor

rnilva commented Aug 3, 2017 •
edited

Taking a card(s) from the deck

  • Patches the Pirate
  • Madam Goya
  • Finja, the Flying Star
  • Quick Shot
  • Slam
  • Cult Master
  • ...

Choice generating cards

  • Shadow Visions
  • Kazakus
  • Adapt cards
  • ...

Reason Unknown

  • Ethereal Arcanist, Alarm-o-Bot (MaxExecution / crash also happens on single thread)
  • Rarely a minion is contained in BoardZone and GraveyardZone simultaneously => infinite loop of death process (this also happens on single thread) (Observed source : Maelstrom Portal / Totems from Wicked Witchdoctor / games where DK card appears)
@rnilva rnilva closed this as completed Aug 6, 2017
@Bert-Proesmans
Copy link
Contributor

What's the reason for closing this issue?

@rnilva
Copy link
Contributor Author

rnilva commented Aug 6, 2017 •
edited

With recent repairing of cloning issues in sub-tasks, cards in the first category (and also some 2nd category cards) no longer throwing bugs. Kazakus is not yet fixed, but it would be fixed soon because the shared resource for Kazakus is already founded. (KazakusPotionSpells)
I think I hurriedly closed this issue, so I opened it again, sorry for any confusion.

@rnilva rnilva reopened this Aug 6, 2017
@Bert-Proesmans
Copy link
Contributor

no problem, i'll adjust the first post to track which cards are confirmed fixed.
please update that list by enabling/disabling the correct checkbox.

@rnilva
Copy link
Contributor Author

rnilva commented Sep 15, 2017

#71

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
ongoing
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Bert-Proesmans @rnilva