profile
viewpoint
Timo timotree3 I love to code and learn new languages!

holochain/holochain 70

The new, performant, and simplified version of Holochain on Rust (sometimes called Holochain RSM for Refactored State Model)

timotree3/badger-docs 1

Agile Badger Minecraft server documentation.

timotree3/Battleship 1

My first battleship game. Written in python.

artbrock/cludder 0

This is a lisp implementation of Clutter. A distributed twitter built on holochain.

artbrock/holochat 0

Distributed slack-alike chat application which runs on holochain

timotree3/2048 0

Console based 2048 made to prove to myself I am actually capable at working in python.

timotree3/aHash 0

aHash is a non-cryptographic hashing algorithm that uses the AES hardware instruction

timotree3/aluminum-carbonate 0

A website for ALC students to do blogging. WIP https://agilelearningcenters.org

timotree3/amlion 0

A solver for letter puzzles of various kinds

issue commentZamiell/hanabi-live

feature: Identity list notes affect pips

will resume work once I have a working dev environment.

timotree3

comment created time in 2 days

issue commentZamiell/hanabi-live

feature: Show future needed efficiency in-game

on linux? yes

+ npm install
npm ERR! Cannot read property 'matches' of undefined

online advice says to delete node_modules / .package_lock.json but neither thing exists in my filesystem

timotree3

comment created time in 2 days

issue commentZamiell/hanabi-live

feature: Show future needed efficiency in-game

Couldn't compile the client on my new computer. Giving up for now

~Timo

On Tue, Oct 27, 2020, 3:32 PM James notifications@github.com wrote:

status on this?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Zamiell/hanabi-live/issues/1737#issuecomment-717485770, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABRMNXQ7R35NXUQ3V4L2HT3SM4N47ANCNFSM4RJKSYYQ .

timotree3

comment created time in 2 days

pull request commentZamiell/hanabi-conventions

Proposal: Removed levels 12-13 and replaced with Appendix A - 5 Tech

It seems to me that new players are just practicing the conventions in the order they find most interesting rather than the order presented in the document. Does that have anything to do with overconfidence?

What do you think about documenting what you intend to achieve with the order and division of the levels in a file similar to Convention_Goals.md? That seems like it would be informative to new players choosing their learning path as well as to players like Sephiroth who want to propose changes.

sephirothx

comment created time in 4 days

issue commentZamiell/hanabi-live

feature: Show future needed efficiency in-game

Okay I'm gonna work on this feature. Any ideas for how to present this number? Should I label it?

It also seems useful to display its numerator and denominator. I'll make a strawman that shows too much unlabelled information and we can go from there

timotree3

comment created time in 4 days

issue commentZamiell/hanabi-conventions

Play instructions while 5 Pulled

in the context of the replay, being 5 pulled or being finessed is identical, right?

That's a good point.

We can probably close this issue then. It sounds like consensus among the more experienced players here is that we should think of 5 pulled cards like cards that are finessed are far as stacked bluffs and ejections are concerned. (as opposed to a card that is already play clued)

I have some follow-up questions about stacked bluffs and ejections though that would therefore apply equally to the 5 pulled case.

Is it possible to do a stacked bluff when Bob's First Finesse Position is older than the already finessed card? The way I was thinking of it would suggest yes. It would look like a stacked ejection so I suppose not. But we do do stacked bluffs when the bluffed card is newer than the finessed card. I guess that's because newer cards have less negative information.

Which card is called for when a player who is still playing into a layered finesse and is finessed for their slot 2 is ejected? slot 1 or slot 3? I suppose slot 3 because otherwise it looks like a stacked bluff?

timotree3

comment created time in 8 days

issue openedZamiell/hanabi-live

Table password entry should default to highlighting the whole password

I instinctively type the password without looking and it appends to my previous password and I hit submit before realizing what I'm submitting. Very time consuming..

created time in 12 days

issue commentZamiell/hanabi-conventions

Play instructions while 5 Pulled

That section is talking about an already finessed player which seems different from 5 Pulled

timotree3

comment created time in 19 days

issue commentZamiell/hanabi-conventions

Play instructions while 5 Pulled

if you look at the replay you can see that it covers a situation already specified by the document

Really? Where is it specified in the document?

timotree3

comment created time in 19 days

issue commentZamiell/hanabi-conventions

Play instructions while 5 Pulled

It was concrete https://discordapp.com/channels/140016142600241152/456214043351580674/764267626553016361

timotree3

comment created time in 20 days

issue commentZamiell/hanabi-conventions

Play instructions while 5 Pulled

Hmm. The text is pretty short and could go in Rarely Used Strategies. It seems like this is something that will come up in #convention-questions again and be met by a lack of consensus there unless we come to a consensus and document it.

timotree3

comment created time in 20 days

issue openedZamiell/hanabi-conventions

Play instructions while 5 Pulled

Objective: Define how Bluffs, Ejections, Discharges, and Charms work on 5 Pulled players.

Some have suggested that a 5 Pulled players blind-playing First Finesse Position (ignoring the 5 Pulled card) should signal an Ejection, Second Finesse Position a Discharge, and so on.

However, such a convention would make it impossible to give a Bluff/non-hidden Finesse to 5 Pulled player.

Instead, I propose the following:

  • We allow Bluffs/non-hidden Finesses on 5 Pulled players as long as both the clue giver and the blind-player (but not necessarily the clue receiver) understand that the 5 Pulled card cannot be the promised card.
    • In other words, after Alice gives a play clue that promises Bob a blind-card, Bob should play his 5 Pulled card if Bob knows that Alice thinks that Bob thinks it can be the promised card, otherwise Bob should immediately play his First Finesse Position as a possible Bluff.
  • Therefore, since First Finesse Position would signal a Bluff to the clue receiver, Second Finesse Position is necessary to signal an Ejection, Third Finesse Position would signal a Discharge, and so on.

This leaves 5 Pulled cards and 5 Pull Promise behaving the same as clued cards and Prompts; a clued card does not occupy a Finesse Position and is prompted by a play clue only when either its holder or the clue giver thinks that it can actually be the promised card.

created time in 20 days

issue commentZamiell/hanabi-live

speedruns should end on negative pace

1, Pace is incredibly easy to calculate. 2. This feature could be implemented in the client

asaelr

comment created time in 23 days

delete branch timotree3/hanabi-live

delete branch : fade-chop-moved-trash-spec

delete time in a month

pull request commentZamiell/hanabi-live

Ignore chop moved notes w.r.t card fade during postgame

does the change to the blank part actually make sense? I don't understand why we want a blank note to affect fading to begin with

timotree3

comment created time in a month

issue openedZamiell/hanabi-live

Chop moved trash cards should still fade during postgame

They currently remain opaque

created time in a month

delete branch timotree3/hanabi-live

delete branch : fix-regex

delete time in a month

create barnchtimotree3/hanabi-live

branch : fade-chop-moved-trash-spec

created branch time in a month

Pull request review commentZamiell/hanabi-live

client: Fix note keyword regex

 export const set = (order: number, note: string) => { const getNoteKeywords = (   note: string, ) => {-  const regexp = /\[([^\]]*)\]|\|([^|[]*)$|^([^|]+)$/g;+  const regexp = /\[(.*?)\]|\|([^[|]*$)|(^[^[|]+$)/g;

does what I added help?

timotree3

comment created time in a month

PullRequestReviewEvent

push eventtimotree3/hanabi-live

timotree3

commit sha 126ff36d7761e974abb99b61378a4007d479bd15

client: Add comment to regex

view details

push time in a month

PR opened Zamiell/hanabi-live

client: Fix note keyword regex
+1 -1

0 comment

1 changed file

pr created time in a month

create barnchtimotree3/hanabi-live

branch : fix-regex

created branch time in a month

delete branch timotree3/hanabi-live

delete branch : dda-notification

delete time in a month

pull request commentZamiell/hanabi-live

Fix DDA notification

Diagnosis: DDA notification isn't working whenever the player has no clued cards in hand. Test case: https://hanab.live/replay/305547/12

It doesn't show yellow border currently, and on my branch it shows it

timotree3

comment created time in a month

PR opened Zamiell/hanabi-live

Fix DDA notification
+1 -1

0 comment

1 changed file

pr created time in a month

create barnchtimotree3/hanabi-live

branch : dda-notification

created branch time in a month

pull request commentZamiell/hanabi-live

Vertically flip prism pip

Before: Prism Before After: Prism After

timotree3

comment created time in a month

create barnchtimotree3/hanabi-live

branch : prism-pip-yellow

created branch time in a month

PR opened Zamiell/hanabi-live

Vertically flip prism pip

In order to make it look less like the yellow pip.

+24 -23

0 comment

2 changed files

pr created time in a month

issue commentZamiell/hanabi-live

Difficult to see X on black suit

For me, I skip over the black pip with my eyes because it looks crossed out, so the white X solution wouldn't help me. Should we just change the black pip? Anyone wanna design a new one?

english5040

comment created time in a month

delete branch timotree3/hanabi-conventions

delete branch : patch-3

delete time in a month

issue commentmicrosoft/vscode

Snap version opens links in new Firefox process and profile, not existing process and profile

Wow awesome discovery NotWoods. That solution works perfectly. Thanks so much

BillDietrich

comment created time in a month

push eventtimotree3/hanabi-conventions

Timo

commit sha a43994c1940baab1711a19c3c4101ff48e1b57d4

Update Beginner.md

view details

push time in a month

Pull request review commentZamiell/hanabi-conventions

Mention saving critical 2's with color in the beginner's guide

 In this guide, there will be some screenshots to show off the concepts introduce   * Bob looks through the discard pile and finds that there is a yellow 4 there. Thus, this could be a *Save Clue* on the yellow 4. So, Bob knows that for now, he has to hold on to the card.   * Bob marks the card as either a yellow 2 or a yellow 4, so that he can keep track of it for later. Then, he discards. +

unintentionally

timotree3

comment created time in a month

PullRequestReviewEvent

PR opened Zamiell/hanabi-conventions

Mention saving critical 2's with color in the beginner's guide

I'm tired of hearing that saving critical 2's with color is level 6. That said, I'm not sure whether this change actually improves the beginner's guide.

+2 -0

0 comment

1 changed file

pr created time in a month

push eventtimotree3/hanabi-conventions

Timo

commit sha 33cd4cc810af486f0a01c68197a239ebce204001

Mention saving critical 2's with color in the beginner's guide

view details

push time in a month

delete branch timotree3/hanabi-conventions

delete branch : patch-5

delete time in a month

push eventtimotree3/hanabi-conventions

Timo

commit sha 856aa921130c463c912e6c99b54c22349f362248

Update Reference.md

view details

push time in a month

push eventtimotree3/hanabi-conventions

Timo

commit sha aa194c32cb5794ed0f84aa7bb868e50570c18ec5

Update Reference.md

view details

push time in a month

pull request commentZamiell/hanabi-conventions

Add a caveat to the paused priority finesse

done

timotree3

comment created time in a month

push eventtimotree3/hanabi-conventions

Timo

commit sha 3e260a2b862dc84ae83514fe158c0a597baa7660

Update Reference.md

view details

push time in a month

delete branch timotree3/hanabi-conventions

delete branch : patch-3

delete time in a month

push eventtimotree3/hanabi-conventions

Timo

commit sha 14265e8d4d2f0419ecf6a0888cfc3a0ce390775f

Add a caveat to the paused priority finesse

view details

push time in a month

push eventtimotree3/hanabi-conventions

Timo

commit sha 8f9f4b7f8e292e197c944a7b014c42b4cee986bd

Add a caveat to the paused priority finesse

view details

push time in a month

delete branch timotree3/hanabi-live

delete branch : info-on-cards

delete time in 2 months

delete branch timotree3/hanabi-live

delete branch : patch-1

delete time in 2 months

delete branch timotree3/hanabi-live

delete branch : use-null-coalescing

delete time in 2 months

PR opened Zamiell/hanabi-live

client: Use null coalescing to shorten some code

Unfortunately I can't save the file without it auto-formatting that line to be too long.

+4 -26

0 comment

1 changed file

pr created time in 2 months

create barnchtimotree3/hanabi-live

branch : use-null-coalescing

created branch time in 2 months

delete branch timotree3/hanabi-live

delete branch : notes-cm-f

delete time in 2 months

delete branch timotree3/hanabi-live

delete branch : fix-es2020

delete time in 2 months

push eventtimotree3/hanabi-live

timotree3

commit sha 87d651030e2926d6844838f0ab5fb90bdea8e2d4

client: Have finesse border take precedence over chop move border

view details

push time in 2 months

create barnchtimotree3/hanabi-live

branch : notes-cm-f

created branch time in 2 months

push eventtimotree3/hanabi-live

Timo

commit sha 4528ec3a86a0da2df63987c365619f324c58bb7c

client: Improve note parsing logic (#1741)

view details

timotree3

commit sha c9842bd4141492523555a4c31a1b94e7b78bbb38

client: Revert to es2018

view details

push time in 2 months

create barnchtimotree3/hanabi-live

branch : fix-es2020

created branch time in 2 months

PR opened Zamiell/hanabi-live

Fix es2020
+180 -99

0 comment

8 changed files

pr created time in 2 months

Pull request review commentZamiell/hanabi-live

client: Improve note parsing logic

 module.exports = {   env: {     browser: true,     jquery: true,+    es2020: true,

Actually I don't think this is true. I'll make a PR that switched away from using matchAll

timotree3

comment created time in 2 months

PullRequestReviewEvent

issue commentZamiell/hanabi-live

feature: Identity list notes affect pips

in your implementation, if they want to see the latter, they would have to delete the note and then readd the note, which probably isn't desired behavior. No they wouldn't. I think it would work like identity notes do. As soon as you hold left-click on the card or drag a card it would show the card with notes ignored

timotree3

comment created time in 2 months

delete branch timotree3/hanabi-live

delete branch : note-parsing-1

delete time in 2 months

Pull request review commentZamiell/hanabi-live

client: Improve note parsing logic

 module.exports = {   env: {     browser: true,     jquery: true,+    es2020: true,

Note that I don't think this changes our browser compatibility because the target is still es2018. I imagine matchAll is getting expanded to es2018 compatible code

timotree3

comment created time in 2 months

PullRequestReviewEvent

Pull request review commentZamiell/hanabi-live

client: Improve note parsing logic

 module.exports = {   env: {     browser: true,     jquery: true,+    es2020: true,

regex matchAll

timotree3

comment created time in 2 months

PullRequestReviewEvent

pull request commentZamiell/hanabi-live

client: Improve note parsing logic

I plan to build on this in my implementation of #1739

timotree3

comment created time in 2 months

PR opened Zamiell/hanabi-live

client: Improve note parsing logic

This makes the note parsing logic do far less work. This changes behavior in two ways:

  • Partial identity notes can now combine, for example [r] [1] now produces a morphed red 1
  • Abbreviation identity notes can now have a space in the middle, for example r 1 now produces a morphed red 1

I have tested this locally extensively

+187 -101

0 comment

9 changed files

pr created time in 2 months

create barnchtimotree3/hanabi-live

branch : note-parsing-1

created branch time in 2 months

issue commentZamiell/hanabi-live

right click efficiency to add counter

So then either we ignore in shared review or we keep track of when a note was written.

Zamiell

comment created time in 2 months

issue commentZamiell/hanabi-live

right click efficiency to add counter

Hmm. Then cards with an f note will act like they're gotten even when viewing a turn before they were.

Zamiell

comment created time in 2 months

issue commentZamiell/hanabi-live

right click efficiency to add counter

Clued trash cards do not count towards efficiency in shared review (even if they are unknown trash)

Useful clued cards count towards efficiency in shared review.

Unclued cards with an f note by the player holding them or an identity note by the player holding them both count towards efficiency in a shared review.

Zamiell

comment created time in 2 months

issue commentZamiell/hanabi-live

feature: Identity list notes affect pips

personally i like the shorthand rm23 because it's quicker to parse in my head.

Me too. To be clear, I mean that we would parse that shorthand as though it expanded to the comma-separated list, I don't actually want it to expand that way in the note.

I also think it's a good compromise to keep it for the people who don't like note morphing cards.

Are you saying that you know people that would prefer not to have this feature?

timotree3

comment created time in 2 months

issue commentZamiell/hanabi-live

right click efficiency to add counter

Why do you want that?

Zamiell

comment created time in 2 months

issue commentZamiell/hanabi-live

feature: Identity list notes affect pips

I want to work on this tomorrow

timotree3

comment created time in 2 months

issue commentZamiell/hanabi-live

feature: Identity list notes affect pips

Bonus would be to parse the common shorthand for identity lists that would make r234 expand to r2,r3,r4 and ry23 expand to r2,r3,m2,m3

timotree3

comment created time in 2 months

issue openedZamiell/hanabi-live

feature: Identity list notes affect pips

Parse notes that are a comma separated series of identities and show them in the UI.

For example, if a card had the note y4,m4 it should morph into a 4 and it's color pips should be yellow and multi.

I believe that just like identity notes, these should be ignored in empathy.

created time in 2 months

issue openedZamiell/hanabi-live

feature: Account for notes in efficiency calculation

For example, cards with an f note should count towards current efficiency, and clued cards with a kt note should not.

created time in 2 months

issue openedZamiell/hanabi-live

feature: Show future needed efficiency in-game

Currently we show two efficiencies:

Current efficiency = Cards gotten / Clues spent Total efficiency = Maximum score / Total clues possibly usable

I propose we show a third and in my opinion more useful efficiency:

Future efficiency = Cards not gotten / Clues not spent

In other words: (Maximum score - Cards gotten) / (Total clues possibly usable - Clues spent)

created time in 2 months

PR opened Zamiell/among-us-vpn

Update README.md
+22 -24

0 comment

1 changed file

pr created time in 2 months

push eventtimotree3/among-us-vpn

Timo

commit sha 70e3858985e1fc9974f352b006f02223f3b9bb00

Update README.md

view details

push time in 2 months

delete branch timotree3/among-us-vpn

delete branch : patch-1

delete time in 2 months

push eventtimotree3/among-us-vpn

Timo

commit sha 1dcf31d7af7dd23c4a55ed92283e689db82edaad

Update README.md

view details

push time in 2 months

PR opened Zamiell/among-us-vpn

Update README.md
+51 -4

0 comment

1 changed file

pr created time in 2 months

push eventtimotree3/among-us-vpn

Timo

commit sha 8a79bcc093f1a268529bf9cf82c776e4284400af

Update README.md

view details

push time in 2 months

issue commentZamiell/hanabi-conventions

Convention Proposal(s): the Mirage Bluff & Lucidity

So it's impossible to give a Mirage Bluff where the only valid fix is a heal clue to Cathy?

pianoblook

comment created time in 2 months

issue commentZamiell/hanabi-conventions

Convention Proposal(s): the Mirage Bluff & Lucidity

How can the player who is supposed to play into a Lucidity know that there isn't a plan to fix the Mirage Bluff by cluing a copy of the Miraged card in their hand?

pianoblook

comment created time in 2 months

issue commentZamiell/hanabi-conventions

Convention Proposal(s): the Mirage Bluff & Lucidity

Example 2: A better Fix with 1s

Green to hanabilawyer01 gets r1 to play as a SSF Although a Fix is called for, paulcd can delay: they can 3-bluff sjdrodge, since g2 is now known. efaust can now clue 1s to hanabilawyer01, fixing everything. Overall, this is a 4-for-2 that also greatly helps get other cards

That's cool. At first I thought that paulcd can't 3-bluff because it would just look like a fix but it sounds like because only four kinds of fixes are allowed for mirage bluffs, it actually works.

pianoblook

comment created time in 2 months

PR opened Zamiell/hanabi-conventions

Specify further the rules of 5 Pull Finesses

This is just my working understanding and I would love feedback.

I also have one question. There's a scenario that some experts say is legal but would conflict with this wording.

  • Cathy has one clued 1 in her hand - the red 1.
  • Donald has a red 2 on finesse position
  • Alice does a 5 Pull on Bob's red 3
  • Alice expects Bob to wait for red 1 to play before blind-playing.
  • Bob discards.
  • Cathy plays red 1
  • Donald blind-plays red 2
  • Alice discards
  • Bob blind-plays red 3

Is Bob required to delay his blind-play even if he knows that his pulled card cannot be red 2? For example, if he sees the other copy in Alice's hand

If he is required to delay it, then this clue by Alice seems perfectly legal. In which case the wording used in this commit ".., must be demonstrated with a blind-play before the pulled player's next turn" is incorrect.

+3 -2

0 comment

1 changed file

pr created time in 2 months

push eventtimotree3/hanabi-conventions

Timo

commit sha 3928f349e5b5698d0c9fcba64bd75d3d8f0e37f5

Specify further the rules of 5 Pull Finesses This is just my working understanding and I would love feedback. I also have one question. There's a scenario that some experts say is legal but would conflict with this wording. - Cathy has one clued 1 in her hand - the red 1. - Donald has a red 2 on finesse position - Alice does a *5 Pull* on Bob's red 3 - Alice expects Bob to wait for red 1 to play before blind-playing. - Bob discards. - Cathy plays red 1 - Donald blind-plays red 2 - Alice discards - Bob blind-plays red 3 Is Bob required to delay his blind-play even if he knows that his pulled card cannot be red 2? For example, if he sees the other copy in Alice's hand If he is required to delay it, then this clue by Alice seems perfectly legal. In which case the wording used in this commit ".., must be demonstrated with a blind-play before the pulled player's next turn" is incorrect.

view details

push time in 2 months

issue openedZamiell/hanabi-live

friends: Only show table at the top if friends are actually participating

Currently it also shows shared replays where a friend was in the game but no friends are viewing

created time in 2 months

issue openedZamiell/hanabi-live

Cannot enter perspective of a specific player as spectator

To reproduce:

  • spectate a game
  • try right-clicking a player's username underneath their hand
  • nothing happens

created time in 2 months

issue openedZamiell/hanabi-live

Cannot open action log for a specific player as spectator

To reproduce:

  • spectate a game
  • try clicking a player's name under their hand

created time in 2 months

more