add weights to blunderbuss; add missing files to .yml

@brendandburns @thockin
Some output, for example:
I0908 15:17:44.456235   11358 blunderbuss.go:118] Weights: pulls.weightMap{"thockin":67, "brendandburns":67, "davidopp":180, "lavalamp":117, "dchen1107":67}
Sum: 498
I0908 15:17:44.456296   11358 blunderbuss.go:129] would have assigned davidopp to PR 13555

Code LGTM, but I'm afraid it just means those people with their names the most places in the file are going to get all of the assignments...  This will make it a bit less blunderbussy and a bit more pinpointy. And I wonder if pinpointy is going to get @lavalamp (16 references) @thockin (10) and @davidopp (10) the worst. Maybe that's a good thing though, don't get me wrong....

Yeah I agree that file needs a bunch of tweaking. I'm messing with this because some people (@dchen1107) are complaining that they're rarely chosen. I think this is still an improvement over what's currently running?
Here's some output with what it would have done, compared to the current assignee.
I0908 16:05:18.041364    3184 blunderbuss.go:81] Loaded config from blunderbuss.yml
W0908 16:05:18.041470    3184 blunderbuss.go:105] Couldn't find an owner for: api/swagger-spec/v1.json
I0908 16:05:18.041821    3184 blunderbuss.go:118] Weights: pulls.weightMap{"smarterclayton":7152, "eparis":3251, "lavalamp":3294, "jlowdermilk":42, "davidopp":8, "thockin":5768, "brendandburns":3866, "bgrant0607":5810, "erictune":1944, "deads2k":688, "derekwaynecarr":8}
Sum: 31831
I0908 16:05:18.041937    3184 blunderbuss.go:122] Current assignee erictune has a 6.11% chance of having been chosen
I0908 16:05:18.041977    3184 blunderbuss.go:134] would have assigned lavalamp to PR 7893
I0908 16:05:18.042004    3184 pulls.go:105] -=-=-=-=8145-=-=-=-=-
I0908 16:05:26.787466    3184 blunderbuss.go:118] Weights: pulls.weightMap{"thockin":18, "brendandburns":18, "bgrant0607":18, "ixdy":179}
Sum: 233
I0908 16:05:26.787522    3184 blunderbuss.go:122] Current assignee quinton-hoole has a 0.00% chance of having been chosen
I0908 16:05:26.787541    3184 blunderbuss.go:134] would have assigned ixdy to PR 8145
I0908 16:05:26.787550    3184 pulls.go:105] -=-=-=-=8255-=-=-=-=-
I0908 16:05:38.778721    3184 blunderbuss.go:118] Weights: pulls.weightMap{"zmerlynn":28, "brendandburns":28, "eparis":28, "robertbailey":28, "davidopp":28}
Sum: 140
I0908 16:05:38.778778    3184 blunderbuss.go:122] Current assignee brendandburns has a 20.00% chance of having been chosen
I0908 16:05:38.778798    3184 blunderbuss.go:134] would have assigned eparis to PR 8255
I0908 16:05:38.778806    3184 pulls.go:105] -=-=-=-=8647-=-=-=-=-
I0908 16:05:52.791211    3184 blunderbuss.go:118] Weights: pulls.weightMap{"brendandburns":188, "vishh":376, "yujuhong":376, "dchen1107":404, "bgrant0607":160, "lavalamp":12760, "thockin":12920, "quinton-hoole":12732, "smarterclayton":160, "eparis":160, "davidopp":12760}
Sum: 52996
I0908 16:05:52.791379    3184 blunderbuss.go:122] Current assignee dchen1107 has a 0.76% chance of having been chosen
I0908 16:05:52.791451    3184 blunderbuss.go:134] would have assigned thockin to PR 8647
I0908 16:05:52.791482    3184 pulls.go:105] -=-=-=-=8659-=-=-=-=-
W0908 16:06:04.804432    3184 blunderbuss.go:105] Couldn't find an owner for: .travis.yml
W0908 16:06:04.804682    3184 blunderbuss.go:105] Couldn't find an owner for: www/master/components/dashboard/protractor/smoke.spec.js
W0908 16:06:04.804726    3184 blunderbuss.go:105] Couldn't find an owner for: www/master/package.json
W0908 16:06:04.804755    3184 blunderbuss.go:105] Couldn't find an owner for: www/master/protractor/chrome/smoke.spec.js
W0908 16:06:04.804783    3184 blunderbuss.go:105] Couldn't find an owner for: www/master/protractor/conf.js
W0908 16:06:04.804810    3184 blunderbuss.go:105] Couldn't find an owner for: www/test/e2e/protractor.conf.js
W0908 16:06:04.804838    3184 blunderbuss.go:105] Couldn't find an owner for: www/test/e2e/scenarios.js
W0908 16:06:04.804874    3184 blunderbuss.go:105] Couldn't find an owner for: shippable.yml
I0908 16:06:04.804901    3184 blunderbuss.go:118] Weights: pulls.weightMap{"ixdy":172, "eparis":172, "zmerlynn":172, "lavalamp":172}
Sum: 688
I0908 16:06:04.804963    3184 blunderbuss.go:122] Current assignee ixdy has a 25.00% chance of having been chosen
I0908 16:06:04.805002    3184 blunderbuss.go:134] would have assigned zmerlynn to PR 8659
I0908 16:06:04.805026    3184 pulls.go:105] -=-=-=-=8662-=-=-=-=-
W0908 16:06:20.782572    3184 blunderbuss.go:105] Couldn't find an owner for: contrib/completions/bash/kubectl
I0908 16:06:20.782759    3184 blunderbuss.go:118] Weights: pulls.weightMap{"thockin":573, "ixdy":3, "eparis":305, "zmerlynn":3, "deads2k":368, "smarterclayton":941, "brendandburns":941, "bgrant0607":941, "lavalamp":3, "jlowdermilk":368}
Sum: 4446
I0908 16:06:20.782851    3184 blunderbuss.go:122] Current assignee jlowdermilk has a 8.28% chance of having been chosen
I0908 16:06:20.782894    3184 blunderbuss.go:134] would have assigned thockin to PR 8662
I0908 16:06:20.782919    3184 pulls.go:105] -=-=-=-=8832-=-=-=-=-
W0908 16:06:32.790720    3184 blunderbuss.go:105] Couldn't find an owner for: _layouts/generic.html
W0908 16:06:32.790798    3184 blunderbuss.go:105] Couldn't find an owner for: _layouts/gettingstarted.html
W0908 16:06:32.790839    3184 blunderbuss.go:105] Couldn't find an owner for: _sass/_template.scss
W0908 16:06:32.790850    3184 blunderbuss.go:105] Couldn't find an owner for: gettingstarted.md
W0908 16:06:32.790860    3184 blunderbuss.go:105] Couldn't find an owner for: img/desktop/getting_started/k8s-mesos_integration.png
W0908 16:06:32.790871    3184 blunderbuss.go:105] Couldn't find an owner for: img/desktop/getting_started/k8s-on-dcos_arch_overview.png
W0908 16:06:32.790884    3184 blunderbuss.go:105] Couldn't find an owner for: solutions/mesosphere.md
W0908 16:06:32.790895    3184 blunderbuss.go:105] Couldn't find an owner for: Gemfile
E0908 16:06:32.790904    3184 blunderbuss.go:115] No owners found for PR 8832
I0908 16:06:32.790916    3184 pulls.go:105] -=-=-=-=8866-=-=-=-=-
I0908 16:06:42.784588    3184 blunderbuss.go:118] Weights: pulls.weightMap{"eparis":933, "lavalamp":1563, "dchen1107":28, "deads2k":42, "davidopp":98, "bgrant0607":2067, "ixdy":4, "derekwaynecarr":22, "erictune":1134, "zmerlynn":5, "brendandburns":965, "robertbailey":4, "thockin":2095, "smarterclayton":2401}
Sum: 11361
I0908 16:06:42.784721    3184 blunderbuss.go:122] Current assignee erictune has a 9.98% chance of having been chosen
I0908 16:06:42.784771    3184 blunderbuss.go:134] would have assigned lavalamp to PR 8866

this LGTM in general.  I suggested a logarithmic scale for file weights to @lavalamp
--brendan

I started judging the files on a quantized log scale.
Also added a flag to get the output I pasted above.

New sample:
I0908 16:25:33.438162   18276 pulls.go:105] -=-=-=-=6234-=-=-=-=-
I0908 16:25:33.834567   18276 pulls.go:122] Waiting for mergeability on Eliminate redundant code in network volume plugins. 6234
I0908 16:25:38.011420   18276 blunderbuss.go:82] Loaded config from blunderbuss.yml
I0908 16:25:38.011493   18276 blunderbuss.go:123] Weights: pulls.weightMap{"thockin":51, "saad-ali":34, "brendandburns":17, "bgrant0607":17, "smarterclayton":17, "eparis":17}
Sum: 153
I0908 16:25:38.011516   18276 blunderbuss.go:127] Current assignee ArtfulCoder has a 0.00% chance of having been chosen
I0908 16:25:38.011534   18276 blunderbuss.go:139] would have assigned thockin to PR 6234
I0908 16:25:38.011540   18276 pulls.go:105] -=-=-=-=6245-=-=-=-=-
I0908 16:25:38.388990   18276 pulls.go:122] Waiting for mergeability on Generic Config object and it's REST implementation 6245
I0908 16:25:42.455055   18276 blunderbuss.go:123] Weights: pulls.weightMap{"eparis":11, "deads2k":6, "bgrant0607":17, "thockin":17, "erictune":6, "lavalamp":22, "smarterclayton":33, "brendandburns":11}
Sum: 123
I0908 16:25:42.455130   18276 blunderbuss.go:127] Current assignee bgrant0607 has a 13.82% chance of having been chosen
I0908 16:25:42.455184   18276 blunderbuss.go:139] would have assigned bgrant0607 to PR 6245
I0908 16:25:42.455209   18276 pulls.go:105] -=-=-=-=6438-=-=-=-=-
I0908 16:25:43.816185   18276 pulls.go:122] Waiting for mergeability on Kube2sky exports IP:port pairs of all pods. 6438
I0908 16:25:55.794111   18276 blunderbuss.go:123] Weights: pulls.weightMap{"eparis":7, "thockin":21, "ArtfulCoder":21, "robertbailey":7, "davidopp":7, "zmerlynn":7, "brendandburns":7}
Sum: 77
I0908 16:25:55.794228   18276 blunderbuss.go:127] Current assignee thockin has a 27.27% chance of having been chosen
I0908 16:25:55.794279   18276 blunderbuss.go:139] would have assigned robertbailey to PR 6438
I0908 16:25:55.794294   18276 pulls.go:105] -=-=-=-=6477-=-=-=-=-
I0908 16:25:57.831732   18276 pulls.go:122] Waiting for mergeability on Config Resource Proposal 6477
I0908 16:26:17.842918   18276 blunderbuss.go:123] Weights: pulls.weightMap{"thockin":6, "smarterclayton":6, "brendandburns":6, "bgrant0607":6}
Sum: 24
I0908 16:26:17.843028   18276 blunderbuss.go:127] Current assignee pmorie has a 0.00% chance of having been chosen
I0908 16:26:17.843071   18276 blunderbuss.go:139] would have assigned brendandburns to PR 6477
I0908 16:26:17.843098   18276 pulls.go:105] -=-=-=-=6520-=-=-=-=-
I0908 16:26:19.833292   18276 pulls.go:122] Waiting for mergeability on WIP:  Controller Plugin framework 6520
I0908 16:26:39.792415   18276 blunderbuss.go:123] Weights: pulls.weightMap{"lavalamp":15, "dchen1107":5, "davidopp":59, "brendandburns":27, "bgrant0607":22, "thockin":27, "smarterclayton":22, "eparis":22, "derekwaynecarr":44}
Sum: 243
I0908 16:26:39.792604   18276 blunderbuss.go:127] Current assignee thockin has a 11.11% chance of having been chosen
I0908 16:26:39.792652   18276 blunderbuss.go:139] would have assigned derekwaynecarr to PR 6520

I think the blunderbuss.yaml file can use some real help.

LGTM, but I think we should considered some amount of quantization or low-pass filtering to weed out really poor fits before we select the assignee.

We can do that the next time someone complains. I'll go ahead and build and push this.

Built and started. Added new commit with updates to makefile and pod.

I guess this was never merged? Merging it now, since we're running it...


[Spell] Bless

Test if it's possible to make Bless have a clear target selection, instead of selecting the closest to the center

Bane and Aid come to mind as well, fix Aid amount of targets as well.

I got a working version of 3 target of bless. It using magic missile as a template
https://github.com/DubhHerder/DnD-Rebalancing/blob/changingBless/DnDRebalancing/Public/DnDRebalancing/Stats/Generated/Data/Spell_Projectile.txt
not sure how you would like it to be used but it seems to resolve the issue

I'm scared of it being a projectile, though. I'll test if the property AmountOfTargets can be used on Target spells first

I attempted the same and wasn't able to target multiple allies when I combined amountoftargets with a Target version of the spell but i was changing multiple parameters while testing so it might be possible for you to do. For that combination I could only target a single ally.
Can you expand on why changing it to projectile is not a great solution?

Projectiles can possibly be blocked by terrain and other targets in front of the selected target. It can be a possible solution if we find a template which doesn't actually shoot any projectile and just automatically provides the status to the targets

Yeah that's a good reason to avoid this implementation, I hadn't realised the direct line of sight limitation was integrated into projectiles

So, I tested a few alternatives. AmountOfTargets can't be used on Target spells, so I also started testing with Projectile.
If you set ProjectileCount to 0 or remove the Template, the spell goes off and doesn't shoot anything, causing the buff to not go through.
The next best option is to use an invisible projectile Template. I've found this one b64b516c-1afd-4f8d-b624-4c9caf06f1c2, which is used for some trap activations (most notably, the rolling boulder damage). However, even with an invisible Template, the projectile path and travel time is still considered, meaning there's potential for a terrain to block the buff, or even an enemy standing in front of your ally stealing the buff - we can prevent the buff being applied to an enemy, but they might block the projectile regardless, at least denying the buff going through

I'll register here my best attempt into making Bless a projectile
new entry "Target_Bless"
type "SpellData"
data "SpellType" "Projectile"
data "Level" "1"
data "SpellSchool" "Enchantment"
data "SpellProperties" "ApplyStatus(BLESS, 100, 10)"
data "TargetRadius" "9"
data "AmountOfTargets" "3"
data "TargetConditions" "not Item() and not Dead() and not Enemy()"
data "ProjectileCount" "1"
data "Template" "b64b516c-1afd-4f8d-b624-4c9caf06f1c2"
data "Icon" "Spell_Enchantment_Bless"
data "DisplayName" "Target_Bless_DisplayName"
data "Description" "Target_Bless_Description"
data "TooltipStatusApply" "ApplyStatus(BLESS, 100, 10)"
data "PreviewCursor" "Cast"
data "CastTextEvent" "Cast"
data "CastSound" "Spell_Cast_Buff_Bless_L1to3"
data "CycleConditions" "Ally() and not Dead()"
data "UseCosts" "ActionPoint:1;SpellSlot:1:1:1"
data "SpellAnimationArcaneMagic" "414bbf02-2918-4f01-83fb-1ddc7a588d88(SPL_Arcane_Healing_01_Prepare),,;,,;09ae2f11-f5b4-42f5-ae16-687a5b57d500(SPL_Somatic_Aoe_01_Cast),,;10caea0e-c949-4d91-8ab7-3b50019dd054(SPL_Somatic_Aoe_01_Recover),,;,,;,,"
data "SpellAnimationDivineMagic" "414bbf02-2918-4f01-83fb-1ddc7a588d88(SPL_Arcane_Healing_01_Prepare),,;,,;09ae2f11-f5b4-42f5-ae16-687a5b57d500(SPL_Somatic_Aoe_01_Cast),,;10caea0e-c949-4d91-8ab7-3b50019dd054(SPL_Somatic_Aoe_01_Recover),,;,,;,,"
data "SpellAnimationNoneMagic" "414bbf02-2918-4f01-83fb-1ddc7a588d88(SPL_Arcane_Healing_01_Prepare),,;,,;09ae2f11-f5b4-42f5-ae16-687a5b57d500(SPL_Somatic_Aoe_01_Cast),,;10caea0e-c949-4d91-8ab7-3b50019dd054(SPL_Somatic_Aoe_01_Recover),,;,,;,,"
data "VerbalIntent" "Buff"
data "SpellStyleGroup" "Class"
data "SpellFlags" "HasVerbalComponent;HasSomaticComponent;IsConcentration;IsSpell"
data "RechargeValues" "6"
data "HitAnimationType" "MagicalNonDamage"
data "MemoryCost" "1"
data "PrepareEffect" "60ec0a2d-1b72-4fb2-9eaf-571638aca25d"
data "CastEffect" "6d072523-11f5-4b1b-a2d8-24ade34a315d"
data "TargetEffect" "4df0bae6-fccf-48a0-804c-3edc40264d07"
data "PositionEffect" "2832c7dd-14bd-4c51-9d2a-599c2c19005f"

It looks like the difference between the two templates is how the projectile path is specified via PathMaxArcDist, PathRadius, ProjectilePath and travel time via Speed.
It might be possible to tweak those values in a new root template to optimise the projectile paths and travel time.
I guess such curving paths are an attempt to remove direct line of sight limitations for magic missile and the invisible template you found

An alternative solution (that is probably worse) could be to cast bless on one target and have it add extra spells that dont cost a spell slot to cast. This is the approach taken by produce flame and twinned spell in the sorcerer mod on nexus. It would be technically more accurate but the user experience would likely be poorer, especially with how the hotbar works.

Yeah, that might work. I can even use the solution I've made to Produce Flame, so the bless caster always has the "free bless" cast, but can't use it until the first main cast is used (costing a spell slot) and then provides a status to use the free one X times. I'm not sure if that'd work given Bless already is a spell container when you get second level spell slots.
But regardless, I'd avoid doing that, because casting Bless individually 4 times (when lvl 2) seems very tedious and slow

Agreed.
I haven't been able to find a list of what attributes are available for a RootTemplate such as PathMaxArcDist,  PathRadius, ProjectilePath etc. I was hoping that looking through the possible attributes might give additional options for modifying a template for Bless. Do you know if such a list can be found anywhere?

I've never found a compiled list for that, no

That's a shame. I guess then the choice is to go with one of the suggested solutions or put the issue on hold and stick with base version until we see how they handle twinned spell metamagic (or another similar ability). They might add more functionality or they might take the current upcasting approach to switch from a single target to an area target .

I was hoping Twinned Spell from Sorcerer could provide an useful function to use for multiple target selection with Target spells, but here's how they did it:
data "Boosts" "UnlockSpellVariant(TwinnedProjectileSpellCheck(),ModifyNumberOfTargets(AdditiveBase,1,false),ModifyUseCosts(Add,SorceryPoint,SpellPowerLevel,0));UnlockSpellVariant(TwinnedTargetSpellCheck(),ModifyAreaRadius(AdditiveBase,6),ModifyMaximumTargets(Override,2),ModifyUseCosts(Add,SorceryPoint,SpellPowerLevel,0));UnlockSpellVariant(TwinnedTargetTouchSpellCheck(),ModifyAreaRadius(AdditiveBase,2),ModifyMaximumTargets(Override,2),ModifyUseCosts(Add,SorceryPoint,SpellPowerLevel,0))"

I didn't test it in-game, but from what's written there it looks like for the TwinnedTargetSpellCheck() (which checks if the spell is a Target one, doesn't have an area of effect and has a range bigger than touch) does this:
UnlockSpellVariant(TwinnedTargetSpellCheck(),ModifyAreaRadius(AdditiveBase,6),ModifyMaximumTargets(Override,2),ModifyUseCosts(Add,SorceryPoint,SpellPowerLevel,0))

translating to: modify the Area Radius to a flat 6 and makes the target amount 2, while also adding the sorcery point cost. This means that even twinned spell can't make a target spell have a selector, instead using the current area auto-selection that Bless has.
It's very unfortunate :/

Larian actually made Target spells have a AmountOfTargets and already used that for Bless.
FINALLY!

It like i am not crazy when i made the test lol ;P.

Well, maybe they got mad that stuff like upcast Hold Person didn't allow for target selection. So the full release included that new possibility, so they went ahead and changed a lot of the target spells to use that option.
Only downside is they went overboard with that and changed some spells that shouldn't include a target selection, such as Sleep, lol. I already reverted the Sleep changes on my mod, so no worries


implement a11y fixes from svelte-repl

adds the fixes from https://github.com/sveltejs/svelte-repl/pull/168, plus a couple of adjacent tidy-ups

This should resolve https://github.com/sveltejs/sites/issues/90 and https://github.com/sveltejs/svelte-repl/issues/163


Create instance page crashed in mysterious way

Here is the error being displayed:

createFormState/<@http://<IP_ADDRESS>:8888/console/lsm/catalog/user_equipment/app.bundle.js:94629:38
createFormState@http://<IP_ADDRESS>:8888/console/lsm/catalog/user_equipment/app.bundle.js:94607:19
ServiceInstanceForm@http://<IP_ADDRESS>:8888/console/lsm/catalog/user_equipment/app.bundle.js:95183:11
Ch@http://<IP_ADDRESS>:8888/console/lsm/catalog/user_equipment/app.bundle.js:50821:137
ck@http://<IP_ADDRESS>:8888/console/lsm/catalog/user_equipment/app.bundle.js:50931:460
bk@http://<IP_ADDRESS>:8888/console/lsm/catalog/user_equipment/app.bundle.js:50914:349
ak@http://<IP_ADDRESS>:8888/console/lsm/catalog/user_equipment/app.bundle.js:50914:280
Tj@http://<IP_ADDRESS>:8888/console/lsm/catalog/user_equipment/app.bundle.js:50914:138
Lj@http://<IP_ADDRESS>:8888/console/lsm/catalog/user_equipment/app.bundle.js:50907:163
4448/jg/<@http://<IP_ADDRESS>:8888/console/lsm/catalog/user_equipment/app.bundle.js:50787:115
53/exports.unstable_runWithPriority@http://<IP_ADDRESS>:8888/console/lsm/catalog/user_equipment/app.bundle.js:51110:343
gg@http://<IP_ADDRESS>:8888/console/lsm/catalog/user_equipment/app.bundle.js:50786:325
jg@http://<IP_ADDRESS>:8888/console/lsm/catalog/user_equipment/app.bundle.js:50787:63
ig@http://<IP_ADDRESS>:8888/console/lsm/catalog/user_equipment/app.bundle.js:50786:428
Hb@http://<IP_ADDRESS>:8888/console/lsm/catalog/user_equipment/app.bundle.js:50956:101
gd@http://<IP_ADDRESS>:8888/console/lsm/catalog/user_equipment/app.bundle.js:50737:354

Here is how to reproduce it:
https://user-images.githubusercontent.com/20844316/192290839-36aec704-3106-4e51-9cc1-c7f4720fed3c.mp4
Web console commit hash: edd03185e0ba74af29f03b80050702b17e5e3d9d
Service orchestrator version:     5.2.1.dev20220926031235
db_dump.zip

This db dump contains credentials to access the 5g lab (if you are in our internal network), you can mess up with it as much as you like, as long as @bartv is not doing a demo at the same time as you.  (If you leave the environment stopped this is fine)

On the same instance we get a similar issue:
https://user-images.githubusercontent.com/10819/192444601-0f15c633-4615-4206-bc2c-15bcacf98c35.mp4
The error is:
TypeError: Cannot create property 'data_networks' on string ''
    at http://<IP_ADDRESS>:8888/console/app.bundle.js:94629:36
    at Array.reduce (<anonymous>)
    at createFormState (http://<IP_ADDRESS>:8888/console/app.bundle.js:94607:19)
    at ServiceInstanceForm (http://<IP_ADDRESS>:8888/console/app.bundle.js:95183:11)
    at Ch (http://<IP_ADDRESS>:8888/console/app.bundle.js:50821:137)
    at ck (http://<IP_ADDRESS>:8888/console/app.bundle.js:50931:460)
    at bk (http://<IP_ADDRESS>:8888/console/app.bundle.js:50914:347)
    at ak (http://<IP_ADDRESS>:8888/console/app.bundle.js:50914:278)
    at Tj (http://<IP_ADDRESS>:8888/console/app.bundle.js:50914:138)
    at Lj (http://<IP_ADDRESS>:8888/console/app.bundle.js:50907:163)


regular expression not support

why not support the regex in  lambda expression?

Because Regular Expressions are not supported in EntityFramework (cannot be translated into SQL).
An option would be like:
https://stackoverflow.com/questions/5720987/how-to-simulate-regular-expressions-in-linq-to-sql

Thank you for your reply.
I wanna use RulesEngine (https://github.com/microsoft/RulesEngine) in my project,  It's depend on this project, Is there any suggestion for me?
Thanks a lot

A cool, I did not know that project used this one.
I don't have any suggestions for you related to RegEx and using these in Lambda expressions. However, it can be used I think.


Change location/multiple locations for the com.mojang folder?

Because i have a version switcher, i have a different com.mojang folder location (and also multiple profiles).

Sure! That should be possible.
What launcher do you use?
Maybe I could do some kind of general integration...

Sure! That should be possible. What launcher do you use? Maybe I could do some kind of general integration...

I use the launcher from this website https://bedrocklauncher.github.io/ (i think it's the one you have on your links page in the app)

Thanks!
I actually linked MCMrARM/mc-w10-version-launcher in my app, but I've already planned to look at BedrockLauncher/BedrockLauncher.

@Mcfly2340
Sorry, that it took me so long, but you can now (still manually) change the directory of the Minecraft folder.
As still fighting with WinUI3 the view lines of code took a fairly long amount of time 😕.
You should find the latest version in the store within the next hours.

Be aware, that it has to the root Minecraft appdata folder, not just the com.mojang folder as the logs and portfolio images are in different locations.
The default folder would be %localappdata%\Packages\Microsoft.MinecraftUWP_8wekyb3d8bbwe\.

One of the next version should then contain some kind of automatic detection of a launcher and the currently used versions.
If you have some more ideas feel free to poste it here (I'll keep the issue open for now) or open a new issue!


Working with radius

Hi,
I see that Radius is a Distance. is this Distance in KM, Meters or spacial ? What Unit Type is this  ?
thanks

currently we don't deal with projection, so it will be Euclidean distance of whatever comes in..


Re-add the fix removing the dependencies to liblttng-ust

Although the RDEPENDS to lttng.ust no longer is required, we still need to remove the libcoreclrtraceptprovider's dependency to it

I thought I checked the libcoreclrtraceptprovider with objdump when I removed the dependency and didn't see it. Is yocto complaining when you do a build?

Yes, without the patchelf, yocto complains:
ERROR: aspnet-core-8.0.8-r0 do_package_qa: QA Issue: /usr/share/dotnet/shared/Microsoft.NETCore.App/8.0.8/libcoreclrtraceptprovider.so contained in package aspnet-core requires liblttng-ust.so.0()(64bit), but no providers found in RDEPENDS:aspnet-core? [file-rdeps]


added Packer

Please rebase with main.

Rebase has been completed.

@flexiondotorg

Please rebase with main.

Rebase has been completed.

The format of package definitions has changed. Please read 01-main/CONTRIBUTING and EXTREPO, update your implementation and submit a new PR.


It's not possible to have manual sampling done when  traces_sample_rate is 0

Environment

PHP 8.1.11
Symfony 6.1
Ubuntu 20.04
Sentry sdk 4.4.0

Steps to Reproduce

create a project with traces_sample_rate set to 0
create a dummy symfony command
at the beginning of the command force sampling decision by putting

          $transactionContext = new \Sentry\Tracing\TransactionContext();
          $transactionContext->setParentSampled(false);
          $transactionContext->setSampled(true);

Expected Result
All run of this command should be sampled
Actual Result
Nothing, no sample send
Other important information
Setting the  traces_sample_rate to 0.000001  workaround the issue, and the sampling is always sent (as expected)
Is there a special treatment for 0 that is considered as a global switch off that can't be overrided ?

Thanks for letting us now know about this.
Looking at https://github.com/getsentry/sentry-php/blob/master/src/State/Hub.php#L219, we do indeed ignore the passed in sampling context by returning too early.
See https://develop.sentry.dev/sdk/performance/#sampling

@allan-simon Can you let us know which version of the sentry/sentry package you have installed locally by running composer info. 3.12.0 might have actually fixed that.

ah indeed, I will check tomorrow, if it was tagged 5 days ago I think I may have missed it !

null will turn off tracing as a whole. In your case, you should use 0.


no password found

Arch Linux.
pass works perfectly in terminal.
browserpass installed from AUR.
Chrome extension does not find a single password.

This was most likely due to a build error in the previous release. Downloading the latest host binary application from the most recent release should fix it! :tada:


Clarify backup of OUD log files

oud_backup.sh does currently backup all OUD log files (access, error, replication etc) This could somehow be an issue if we do have huge amount of log file. This behavior has to be changed. eg. provide a parameter to backup log file and set default behavior to not backup them.

limit backup to current logs eg. everything which is not *Z from the log rotation. Additional the logs are only backed up when -L is specified


Evaluate NumPy 1.10 behavior with astropy.units

See https://github.com/numpy/numpy/pull/4622.

Notice that this is just an evaluation and cannot translate to changes in the code since we are supporting NumPy 1.9. For now.

At the time of writing, NumPy still does not respect the subclass of the arguments and therefore Quantity might lose their units, see https://github.com/astropy/astropy/blob/v1.1rc2/docs/known_issues.rst.


fix: folder structure list elements now properly disappear when deselected

Closes #984
✅ Checklist

[X] I have followed every step in the contributing guide (updated 2022-10-06).
[X] The PR title follows the convention we established conventional-commit
[X] I performed a functional test on my final commit

Changelog
Folder structure in side bar now properly disappear when their corresponding category is deselected in the main Folder Structure component.

Screenshots
💯

This doesn't fix the "last option never gets highlighted bug" I mentioned on the issue thread( #984 ). I can either add it this PR I'm able to fix it tomorrow, otherwise I can open a new issue if you'd like to keep them separate!

The thought crossed my mind of making the solution more upstream, but I ended up implementing this as it was the simplest solution at the time. I'll look closer at the solutions you proposed later tonight and try to implement them!

The thought crossed my mind of making the solution more upstream, but I ended up implementing this as it was the simplest solution at the time. I'll look closer at the solutions you proposed later tonight and try to implement them!

I could also merge this for now as it works and just keep the issue open if anyone wants to go in and refactor it to a more automatic approach?

Ya that seems like a good solution for now! Sorry I haven't had time to do the refactor as of yet.

Thanks for the PR 🙏🏼. Feel free to keep working on it in a new PR if you're down. I left the issue opened and gave a brief description of what's left


Feature/readme file align for poc

This change is

Review status: 0 of 4 files reviewed at latest revision, 4 unresolved discussions.

ibm-block-storage-via-scbe.md, line 8 at r1 (raw file):

# Ubiquity Dynamic Provisioner
There is no tune for the Provisioner that related to IBM Block Storage.

There is no tuning for the Provisioner related to IBM Block Storage.
Should we follow the same description as Spectrum scale one?
Mention that the storage class should refer to scbe?

ibm-block-storage-via-scbe.md, line 64 at r1 (raw file):

 port = 9999     # TCP port on which the Ubiquity Service is listening

Verify that the logPath, exists on the host before you start the plugin.

can we add some code  (to create the folder if it does not exist os.Mkdirall() ?) to do that instead

ibm-block-storage-via-scbe.md, line 78 at r1 (raw file):

5. Clean up by exiting `container2`, removing the containers and deleting the volume `demoVol`.

TBD

I understand that starting from here, you did not change the docker docs to k8s, but I agree to follow the same direction

README.md, line 142 at r1 (raw file):

### FlexVolume usage examples
For examples on how to start and stop stateful containers\PODs with Ubiquity volumes , refer to the [Available Storage Systems](supportedStorage.md) section, according to your storage system type.

An example on how to start ...

Comments from Reviewable

Review status: 0 of 4 files reviewed at latest revision, 4 unresolved discussions.

ibm-block-storage-via-scbe.md, line 8 at r1 (raw file):
Previously, midoblgsm (Mohamed Mohamed) wrote…
There is no tuning for the Provisioner related to IBM Block Storage.
Should we follow the same description as Spectrum scale one?
Mention that the storage class should refer to scbe?

you right, we should mention that storage class should point to :
provisioner: "ubiquity/flex"
and the following params must specify as well :
parameters:
profile: "SCBE STORAGE SERVICE"
fstype: "FSTYPE"
backend: "scbe"
will add it next week.
Mohamed, please add the relevant Spectrum Scale changed in its md file and add it to the PR.

ibm-block-storage-via-scbe.md, line 64 at r1 (raw file):
Previously, midoblgsm (Mohamed Mohamed) wrote…
can we add some code  (to create the folder if it does not exist os.Mkdirall() ?) to do that instead

open ticket for that, but do it after the POC 0.4(of different branch).
lets try to minimize code changes from now. only critical for k8s POC.

ibm-block-storage-via-scbe.md, line 78 at r1 (raw file):
Previously, midoblgsm (Mohamed Mohamed) wrote…
I understand that starting from here, you did not change the docker docs to k8s, but I agree to follow the same direction

yes, i will add here relevant yml files (will take it from the acceptance test I did)
Please fill up the spectrum scale plugin usage as well in the spectrim scale md file, and add it to the PR.

README.md, line 142 at r1 (raw file):
Previously, midoblgsm (Mohamed Mohamed) wrote…
An example on how to start ...

I don't think we should put here anything  because its diff from scbe to spectrum scale. this is why we have plugin usage in the specific backend md file.

Comments from Reviewable


[WIP] ILP Conv Layer support

Tests:

Compared loss curves with 1 GPU and 2x2 GPUs

TODO:
Add test like FC layer

Can you post the loss curves here too?


Tooltip is shown even when sx modal is being shown

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Steps to reproduce 🕹
Steps:

Add a TextField component to the surface
Access the SX menu

Current behavior 😯
As shown in the image below, the tooltip is still showing while In the SX editor. This is very annoying because you cannot access the save button.

Expected behavior 🤔
once the sx button has been clicked the tooltip should not be visible.
Context 🔦
No response
Your environment 🌎
Toolpad version: 0.1.12
Browser: Firefox 113.0.1
User Agent: "Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:109.0) Gecko/20100101 Firefox/113.0"

once the sx button has been clicked the tooltip should not be visible.

Agreed, this was also reported in https://github.com/mui/mui-toolpad/issues/787#issuecomment-1400286601


Drop external dependencies

This change is part of the following stack:

#2 ◀

Change managed by git-spice.


add custom dependencies

set syntax check to full/strict scope
add custom dependencies
setup downport setup
setup transpiling step

see scripts in https://github.com/SAP/abap-file-formats-tools/blob/hvam/deps2004/package.json#L7-L10
it does not work yet, so CI/actions is not setup yet

requires https://github.com/SAP/abap-file-formats-tools/pull/64 to be merged first

requires https://github.com/SAP/abap-file-formats-tools/issues/67

yes


Add ruby 2.7 to test matrix for future proofing

Coverage remained the same at 99.315% when pulling a3dc3896bf181b1ee3874b6bf75909e93e402c6a on DavidS:ruby27 into 86c9005222ebd3ab6eccf8b99cdd21f7b02897ce on mcanevet:master.

Hmmm... unit tests are broken since I merge https://github.com/mcanevet/rspec-puppet-facts/pull/103 (my bad). Maybe we should try to fix it before merging this?

Looks like just removing the offending tests might suffice, since that method was removed too and the facterversion adjusting is being tested anyways in other tests?

Perfect! Thanks David.


Do not show new features screen if there are no new features

Current Implementation
Currently it is implemented that the "New Features" splash screen after a minor app update is shown, although the release does not contain any new features but only bug fixes (see e.g. https://github.com/corona-warn-app/cwa-app-ios/pull/4707).
Suggested Enhancement
Do not show the new features screen if there are no new features in an app release.
Expected Benefits
From a users perspective, showing a screen which tells me that nothing has changed does not have any value. It is not a helpful information and I can't do anything with the information that bugs have been fixed.

@dsarkar / @larswmh
As this item is relevant for version 2.26, please mirror it to JIRA soon.

@Ein-Tim

There is a merged Android PR https://github.com/corona-warn-app/cwa-app-android/pull/5353 related to this request.

@MikeMcC399 This PR is good, as it prevents the splash screen being shown if I update from 2.24.1 to 2.24.2. However, I was talking about the fact that I will see a splash screen when updating form version 2.25.0 to 2.26.0 with the information that nothing has changed. This does not make sense.

@Ein-Tim
Sorry that I misunderstood your request.

You're referring to the iOS PR https://github.com/corona-warn-app/cwa-app-ios/pull/4707

Is there an equivalent PR for Android? I could not find one.

Is there an equivalent PR for Android? I could not find one.

I don't think so. At least it is not included in https://github.com/corona-warn-app/cwa-app-android/releases/tag/v2.26.0-rc.0.

@Ein-Tim
I would be fine if Android implemented the same message as the pending iOS PR (https://github.com/corona-warn-app/cwa-app-ios/pull/4707) if there are no new features.
In the past, new releases have always had new features, so if there is a release with no new feature I think that the app should say so instead of staying silent.
I realise there are different views on this: Some people would like no information at all. You would like an information screen only if there are new features. I would like an information screen for every minor version release (not patch release) whether or not there are new features.

In the past, new releases have always had new features, so if there is a release with no new feature I think that the app should say so instead of staying silent.

Interesting. Could you explain why you'd like this information? How do you benefit from the information that nothing has changed?

@Ein-Tim
This is just my personal preference. It also corresponds to the Project Office decision, as you will know from https://github.com/corona-warn-app/cwa-app-ios/pull/4707#issuecomment-1203817217.
It will be interesting to hear if there are other opinions about this.

@MikeMcC399 Okay, understood! :)
I will feel free and ping a few other community members to see what they think: @fynngodau, @ndegendogo, @vaubaehn what do you think?

I am a software developer. I am aware that every new version can bring changes / improvements / bugfixes / or, if I am unlucky, it can even introduce new bugs 👹👹
So I don't mind seeing a splash screen for a new release, even when there are no new features.
Actually, I would remove the last sentence if the menu "New functions" is empty.
Especially when I am suffering from a bug and waiting for a fix that is promised for the next version, then this splash screen has definitively a value for me.
Note that a non-developer user might see this differently and might be confused.

@ndegendogo
Thanks for your input!
@GisoSchroederSAP Who is responsible to mirror issues to JIRA in August? Will this be done by you or someone from SAP/TSI?

Ok, seems like this won't be considered. Closing

Thanks for the discussion here, folks.
We also had an internal evaluation whether or not to show the screen. The simple explanation is: Whenever we bring a new release, that is for a reason - something changed: be it a new function or a bug fix, or just a change in text or UI.
To stay with good old practice of documentation, we allow ouselves to document the background of every new release.

Is it crucial for the users? Probably not. As we see in the discussion above, there are pro's and con's
Is it crucial for placing the CWA on the market? Probably not, if no new functions can be offered.
Are there legal, compliance or political requirements forcing us to bring up this "new features" screen? Not of my knowledge, it is just a decision made by our team to stay transparent.

So, instead of going to implement a conditional screen, we just go forward with the screen either documenting new features, give a brief statement about bugfixes or changes.

@GisoSchroederSAP
Thank you for the clarification!

Joke here

To stay with good old practice of documentation, we allow ouselves to document the background of every new release.

I guess that's also why the documentation repo is absolutely up to date, right?
SCNR 😉


[PY] Decompose _on_turn method

Break down the on_turn method into several smaller methods so its more readable

test comment

reviewing the behavior in the "_on_turn" method

moving to review shortly.


add: add write batch points

1、add write batch points；
2、add database interface;

closed by #37 , please submit a new pr for write batch points.


Credentials for mapbox bucket are not obtained on AppVeyor with GDAL 1.11 and 2.2

Works for GDAL 2.3. See https://ci.appveyor.com/project/sgillies/fiona/build/1.0.558. Is this sensitive to the GDAL version or is something else going on?

I think this is related to: https://github.com/curl/curl/issues/1538 and https://github.com/OSGeo/gdal/commit/5f95e03297889f8ddaaddf155203723caef3f86e
As far I understand https://github.com/OSGeo/gdal/blob/release/2.2/gdal/port/cpl_http.cpp#L779-L787
for gdal version 2.1 and 2.2 the path could be set via the environment variables CURL_CA_BUNDLE or SSL_CERT_FILE.

I've noticed that the GDAL builds for the failing jobs are older and am going to try upgrading them. I think that eliminating the 1.11 jobs because there are no more current builds for 1.11 on the gisinternals site would be fine, too. We're testing 1.11 on Linux, at least.


Retrieve all paths for a given object (flatten)

This is a truly awesome library - thank you!
I wanted to check if there's an idiomatic way in this lib to get all the paths for an arbitrary JSON object/array. For example
{ "foo": [{ "bar": "baz" }], "baz": true }

would return keys
"foo.0.bar": "baz"
"baz": true

Thank you!

Unfortunately there's nothing like that built into the library.

Ok thanks, I wrote a simple traverse/flatting logic to address this :)


Create 5.48 Explain everything related to THANOS.md

thanos

Description 📜
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.
Fixes # <issue_number_here>

Type of change 📝

[ ] Audio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)
[ ] Video (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[x] Documentation (Content Creation in the form of codes or tutorials)
[ ] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[ ] Android Dev (Flutter, Java, Kotlin)
[ ] Backend Dev (Java, .NET, Python)
[x] C/CPP
[ ] Competitive Programming
[ ] Cyber Security
[ ] DSA
[ ] Database
[ ] Datascience with Python
[ ] Datascience with R
[ ] Frontend Dev HTML/CSS/JS
[ ] Frontend Dev React/Angular/Vue
[ ] Golang
[ ] Interview Prep
[ ] Java_Domain
[ ] JavaScript
[ ] MERN
[ ] Machine Learning
[x] Open Source
[ ] Python_Domain
[ ] Rust
[ ] Statistics
[ ] UI/UX

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

@sandeepkumar00110
Delete all other domains from your comment.. like mentioned in official GWOC PR template directives
Kindly follow the issue template from next time ... & preach this to your fellow contributors

ok sir

file name changed

file name changed

Add images in your doc @sandeepkumar00110


Required property is not working with Checkbox

What package has an issue
@svelteuidev/core
A clear and concise description of what the bug is
Checkbox does not pass down the required attribute to the  in the DOM.
In which browser(s) did the problem occur?
Chrome
Steps To Reproduce

Create a form with a Checkbox having the required prop.
The form can be submitted even if the Checkbox is not checked.

Do you know how to fix the issue
No
Are you willing to participate in fixing this issue and create a pull request with the fix
No
Relevant Assets
No response

I'll make a patch release this week to fix this, thank you for creating this bug report! :raised_hands:


Return 0 for tf.sign(NaN) to align with TF

This change is

Review status: 0 of 1 files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable

@dsmilkov Isn't tf.sign(x) 0 for x == 0 or NaN?
https://www.tensorflow.org/api_docs/python/tf/sign

Nice catch! Thanks. Fixing


US756631 fix to _parse_aedtplt, cache project_file

@Samuelopez-ansys Sorry, this PR has been open for a while...   Do you want me to address the comments or do you not want to merge the changes?


I want filter to remove the null from the array, however, in the syntax of reactstrap, it doesn't seem to work as intended.
What am I doing wrong, or else is there any special ways to do so in reactstrap?

This has nothing to do with reactstrap and would be a better question for stackoverflow.
A hint would be to format your code to better see the shape of the array you are defining.
You will notice none of the item in the result array are null. Add a console.log within the filter function callback and log x to see.

I'm closing this since the the issue is no longer active and it appears it was answered above. Please comment and it will be opened if you feel the issue has not been resolved / the question has not been answered.


[Feature Request] Utilize Cache for Custom Historical Data

It would be super useful to have an interface for storing custom historical data inside the cache, so that expensive sums of historical data for portfolios would not have to be computed repeatedly or stored using another method, and so they can benefit from the aging/cache validation mechanism.
This could probably be done by allowing the user to register a custom ticker along with a custom fetch function for when the price needs to be updated for a given date.
This is a great package by the way, good job on all of the work so far!

Do you have a specific scenario?

When you need to get the historical value of a portfolio of stocks over time, to get the total value of the portfolio you would need to sum up the individual values for all the stocks, regardless of if they were cached or not. If you need to do this repeatedly (i.e. hosting an api) the calculations add up

You might be overestimating the cost of Numpy summing a few thousand values.
Have you tried using FetchDate to implement your caching layer on top of YFC? if df["FetchDate"].iloc[-1] > lastCalcDate ...

I dont know much about the performance of Numpy, you are probably right about the overestimation. In that case there is no need to cache. Thanks for the quick response!


70/100 for assignment 4

10 points - There is no LWLR model.
10 points - Explain the difference between the different models.
5 points - Choose the best model and explain why you chose it.
5 points - There is no graph comparing all the models.

we edited by pull request in phase 4.


fix(SR6): removed deal and facility import endpoints

Changelog

Removed Deal and Facility import endpoints (removes SR6 vulnerability)

Happy for the files to be deleted as we will not use the import service anymore.


Pagination

As Nick mentioned in #981, the padding of the pagination is a bit unbalanced with the latest changes:

What about switching from an input field to a select box that becomes visible when hovering the pagination?

The reason @nils-werner implemented the box as an input was that it was immediately fast to jump to page 3, 50, 100 or 200.
This isn't a common task (or is it?) and I think a input field is fine.
I don't think a select box will scale very well for larger pagination sets. In which case one might suggest that we display a select box where pagination is less than 10 pages (or something) and display a text input otherwise.
Either way, it's minor.

The reason @nils-werner implemented the box as an input was that it was immediately fast to jump to page 3, 50, 100 or 200.

Correct. We talked about it at the Symposium and agreed that

The oldfashioned links work for up to 5-10 pages
A Select Box works for up to 20-30 pages
An Input Box works for any number of pages

The padding is weird because CSS doesn't offer a way to automatically resize inputs depending on the number of characters entered. Maybe some JS could fix that though.

Maybe some JS could fix that though.

Good idea. With CSS Transitions we could make the width grow smoothly too.


Update installation instructions

With the stake helm chart being published, we should update the installation instructions to use it.

Just did this.


Add images and some textbox also button in each swipe card

Hi
thanks for your great library
Now I need to add image and some text with a button relate to each swipecard
I tried to change that but I was not satisfied
would you mind please help me to change that or give me a small example
thanks
best regards

You need two list one for text another for image(drawable image id), --->  then set to adapter ---> then remove "arraylistofImage(0)" at position zero; Check http://stackoverflow.com/questions/20880841/how-to-add-imageview-array-to-arrayadapter-for-a-listview

Hi Sir,
I been trying to implement it with images but this error keeps appearing "ArrayAdapter requires the resource ID to be a TextView"
Could you please give me an advice?, Does the code work with images?
Thanks

@harolhiguera that simply means that the ID you're calling is not a TextView, have you checked the type of view linked to that ID?

I see, I was afraid of that, although I could not identify what kind of Ids can I used for  the SwingAdapter class, its quite complex.   Thank you Alex for your quick response.  By the way,..what a good library, It sure is very handy for many people.  I am myself working in a project and need this feature.
Thanks.
Harol


retrieve all unique values for a list of strings field.

Summary
As a developer I want to be able to pull all unique values for a field (especially values within lists of strings) and count of matching posts so that I can build tag clouds efficiently
Expected Results
some query goes in taking the field of interest and out comes the values and associated counts:
[
    { "value": "foo", "count": 123 },
    { "value": "bar", "count": 456 },
    { "value": "bad", "count": 789 }
]

Notes
being able to pull unique tags or categories will allow efficient getStaticPath and getStaticProps implementations when dealing with tag or category indexes when values are not set up as a separate collection. Otherwise you have to pull the entire collection and process client side.

Aside from the other issue with filtering here. Would this get you what you want?
{
  postsWithFoo: postConnection(filter: {
    tags: { in: "foo"}
  }) {
    totalCount
  }
  postsWithBar: postConnection(filter: {
    tags: { in: "bar"}
  }) {
    totalCount
  }
  postsWithBad: postConnection(filter: {
    tags: { in: "bad"}
  }) {
    totalCount
  }
}

Would yield:
{
  "postsWithFoo": { "totalCount": 123 },
  "postsWithBar": { "totalCount": 456 },
  "postsWithBad": { "totalCount": 789 },
}

There is still the problem of knowing what the tags are in the first place though...

Ah I see, it seems like it'd make sense for a tag to be it's own record in that case, I know that feels cumbersome to have a whole file dedicated to one simple little string of info. And this gets back to the lack of support for reverse lookups, which aren't supported yet. But in general I would think that would be ideal for this scenario (sorry, this isn't supported yet!):
tagsConnection {
  edges {
    node {
      references { # note that this doesn't exist yet!
        totalCount
      }
    }
  }
}

Another issue we'd like to get to is editing references inline, so you don't have to leave the form when you're working with a reference. Would be great for stuff like this.
As for the issue of having a markdown or json file for one little tag, this is something we've discussed streamlining before https://github.com/tinacms/tinacms/discussions/2948#discussioncomment-2874040

that feels pretty counter intuitive to how tags work in pretty much any other CMS though, they are normally free entry strings, or tag records are managed transparently without the user needing to worry about them...
so unless the CMS can create tags automatically in some kind of hidden collection its going to make for awful UX.

I suspect the answer will probably end up being an immutable collection generated by a GitHub action...

or tag records are managed transparently behind the scenes and look like free entry strings

I believe this is likely how they're most-often handled (or in something like Wordpress they'd be post metadata). In general I think they need to be individual records in our data layer so they can be indexed for querying like this, and our aim would be to provide a seamless ux as well here.

In WordPress, a tag is a row in the taxonomy (or term...?) table, so they are individual records in the database. A document per tag is how I'm handling the migration out of WordPress.

Ah right, taxonomies - forgot about those. In Tina everything is a collection, but we probably want to explore the idea of some collections being less "front-facing", as in - the type of collection which is mainly useful when referenced and should easy to create/edit/delete from within the form of another collection document. I don't see a lot of CMSs handling this stuff very well, perhaps Wordpress is the one that get's it right here.


[SECURESIGN-298] Add uninstall & template for default values

This PR:

adds @JasonPowr 's remove keycloak commit from https://github.com/securesign/sigstore-ocp/pull/102
introduces template for default values that is embedded in binary with go compile
adds helm lister to check for existence of release to either install or upgrade
adds uninstall cmd that
a) performs helm uninstall
b) removes extra namespaces that were created but not managed by helm
in absence of a custom values file, the OIDC issuer endpoint https://oauth2.sigstore.dev/auth will be configured. This is set in the default template/values file.
adds constants for secret file names, certs dir

closing this, it's a duplicate PR to #105


Move plot.py to bigbang.visualize

Looks like there is already (duplicate?) code in bigbang.visualize for the stacked chart
The adjacency matrix visualization can maybe move to bigbang.visualize.graphs ?

Partly addressed with #540 but there are issues of code redundancy here which require a targeted discussion.


Floating label for autocomplete does not animate on focus

Bug, feature request, or proposal:
Bug
What is the expected behavior?
Floating label should float using animation on fucs.
What is the current behavior?
Floating label snaps to floating-position without animation.
What are the steps to reproduce?
http://plnkr.co/edit/mVEqWQ3Xp4Ieep3iDs96?p=preview
What is the use-case or motivation for changing an existing behavior?
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Is there anything else we should know?

Should be resolved by https://github.com/angular/material2/pull/3992.

@crisbeto Could you edit your PR to include "Fixes" and a link to this issue? I created an issue as I didn't find one. Might aswell keep this open until your PR is merged.


Release/0.13.1

Codecov Report

Merging #349 (4a03550) into main (7c31deb) will decrease coverage by 0.12%.
The diff coverage is 97.05%.

@@             Coverage Diff             @@
##              main     #349      +/-   ##
===========================================
- Coverage   100.00%   99.87%   -0.13%
===========================================
  Files           46       46
  Lines         1611     1653      +42
===========================================
+ Hits          1611     1651      +40
- Misses           0        2       +2

Impacted Files
Coverage Δ

pfhedge/instruments/derivative/american_binary.py
100.00% <ø> (ø)

pfhedge/instruments/derivative/european.py
100.00% <ø> (ø)

pfhedge/instruments/derivative/european_binary.py
100.00% <ø> (ø)

pfhedge/instruments/derivative/lookback.py
100.00% <ø> (ø)

pfhedge/instruments/primary/brownian.py
100.00% <ø> (ø)

pfhedge/instruments/primary/cir.py
100.00% <ø> (ø)

pfhedge/instruments/primary/heston.py
100.00% <ø> (ø)

pfhedge/nn/modules/bs/american_binary.py
100.00% <ø> (ø)

pfhedge/nn/modules/bs/european_binary.py
100.00% <ø> (ø)

pfhedge/nn/modules/clamp.py
100.00% <ø> (ø)

... and 8 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7c31deb...4a03550. Read the comment docs.


Don't print debugging things on every run of kubectl.sh

It's intensely weird to see kubectl.sh tell me it's starting a cluster every time I run it.

Shippable is green. Merging.


Reporting improvement on table failure

Expected behavior
When test is failed, row should mark red for failed test.
Actual behavior
Step marked as failure instead of specific row.
Steps to reproduce
Example Scenario:

Login to application
Search shop
|search type| search name|
|location      | New York      |
|ShopName | XYZ               |
|Item Name | Mobile           |
Logout from application

Assume that Shop name test iteration is failed and it has to mark as Red color in report. But step has been marked as red.
Gauge version
Gauge version: 0.8.3

Plugins
-------
csharp (0.10.0)
html-report (3.1.0)
java (0.6.2)
ruby (0.4.0)
spectacle (0.1.0)
xml-report (0.2.0)

The scenario you describe is not table driven execution. Rather you seem to have a step that takes in a table parameter.
Table driven execution illustration: https://docs.getgauge.io/using.html#data-driven-execution
In your scenario, gauge executes a step, which happens to take a table as a parameter. It is up to the implementation to decide what to do with it. Gauge has no context or control on the implementation, rather it only tracks the step's execution.

I am using table as a parameter. When i have 10 rows in a table, i dont know which row is failed while looking into the reports.

I'm afraid what you are asking is not possible. Here's why:
The table parameter serves a different need than "table driven execution". Typically it is used to send complex data to step, i.e something that cannot be represented using primitives such as string or int.
Hence Gauge does not treat the table parameter any different, and does not report at table row level.
Alternatives

Use Custom Message to write extra information (like what @aredz suggests in the gitter chat.)

Convert this scenario into Table driven. An example would be:

# Multi Search

|search type| search name|
|location | New York |
|ShopName | XYZ |
|Item Name | Mobile |

## Search after login

- Login to application
- Search shop <search type> <search name>
- Logout from application

Yes, I would like to do table driven. since there is an enhancement pending for scenario table driven, I am waiting for release. I will wait for that and will modify my scenario later.

And also, we cannot write custom failure message to gauge right? Gauge.WriteMessage(boolean, message)

Gauge.WriteMessage does not carry any state, just content. So you cannot mark it as pass or fail.

Ok, Please close this issue (invalid).

Please update me when this feature 604 will be released.
Is there any custom gauge message to make test fail?

Please update me when this feature 604 will be released.

Sure.

Is there any custom gauge message to make test fail?

No, but if your implementation throws an exception, Gauge will mark the step as failed.
Closing this issue. Thanks.


IPv6 NAT candidate reporting

This is part of the output when I run innernet show on my device. These devices(arch, router, home-server) all have a ipv4 address (a private IP address, like <IP_ADDRESS>), and a ipv6 address(a global unicast address). But it seems that only the ipv4 one is being used in innernet. If the ipv6 address can be used to connect peers, we can simply get rid of annoying NATs, and connect to peers directly through ipv6.
So is it possible to use ipv6 addresses in innernet?
I've been using innernet for a while, it is really a great project! Many thanks for your great efforts.
 peer: arch (2t9yDWwyTY...)
    ip: <IP_ADDRESS>
    endpoint: <IP_ADDRESS>:60688
    last handshake: 19 minutes, 18 seconds ago
    transfer: 3.08 MiB received, 7.34 MiB sent
  peer: router (hLu6TevaY5...)
    ip: <IP_ADDRESS>
    endpoint: <IP_ADDRESS>:55552
    transfer: 0 B received, 1418.57 KiB sent
  peer: home-server (VhItncIvyG...)
    ip: <IP_ADDRESS>
    endpoint: <IP_ADDRESS>:33498
    last handshake: 1 minute, 43 seconds ago
    transfer: 36.27 KiB received, 133.89 KiB sent

innernet can use IPv6 endpoints, and there are a couple reasons these endpoints aren't being used right now in your setup:

Your coordinating server is probably setup with an IPv4 endpoint, so clients are connecting to the server via IPv4 and the server is passing those addresses it sees to fellow peers. If the server were being reached over IPv6 it would be reporting other peers' IPv6 addresses.
Right now the innernet client is only reporting IPv4 addresses as NAT candidates, as it's a relatively new addition. This issue is a good reminder that it's time to add IPv6 addresses as other candidates though!

The quickest interim solution if your IPv6 addresses are not temporary would be to use innernet override-endpoint to manually specify an IPv6 endpoint that peers should use to connect.

I went ahead and changed the title so that I can use it as the issue for adding IPv6 NAT candidates :). Thanks again for opening the issue!

I'm trying to implement this. I found that the client will collect its ip addresses, and then upload them to the server.
    let candidates: Vec<Endpoint> = get_local_addrs()?
        .filter(|ip| !nat.is_excluded(*ip))
        .map(|addr| SocketAddr::from((addr, device.listen_port.unwrap_or(51820))).into())
        .collect::<Vec<Endpoint>>();

I found that get_local_addrs() will only collect ipv4 addresses on my machine (archlinux, Linux 5.15.17-xanmod2-2 #1 SMP Fri, 28 Jan 2022 12:27:47 +0000 x86_64 GNU/Linux).
code:
fn main() {
    let res: Vec<_> = get_local_addrs().unwrap().collect();
    dbg!(res);
}

output:
[client/src/main.rs:1170] res = [
    <IP_ADDRESS>,
]

my ip addresses
❯ ip -c a
1: lo: <LOOPBACK,UP,LOWER_UP> mtu 65536 qdisc noqueue state UNKNOWN group default qlen 1000
    link/loopback 00:00:00:00:00:00 brd 00:00:00:00:00:00
    inet <IP_ADDRESS>/8 scope host lo
       valid_lft forever preferred_lft forever
    inet6 ::1/128 scope host
       valid_lft forever preferred_lft forever
2: eno1: <NO-CARRIER,BROADCAST,MULTICAST,UP> mtu 1500 qdisc fq state DOWN group default qlen 1000
    link/ether b0:25:aa:3a:e8:2e brd ff:ff:ff:ff:ff:ff
    altname enp2s0
3: wlp1s0: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc noqueue state UP group default qlen 1000
    link/ether c8:b2:9b:a4:77:20 brd ff:ff:ff:ff:ff:ff
    inet <IP_ADDRESS>/24 brd <IP_ADDRESS> scope global dynamic noprefixroute wlp1s0
       valid_lft 603722sec preferred_lft 603722sec
    inet6 2409:XXXX:XXXX:XXXX:XXXX:9bff:fea4:7720/128 scope global dynamic noprefixroute
       valid_lft 221625sec preferred_lft 135225sec
    inet6 2409:XXXX:XXXX:XXXX:XXXX:b877:583f:f3ab/64 scope global dynamic noprefixroute
       valid_lft 221624sec preferred_lft 135224sec
    inet6 fe80::3ba8:782e:19d5:bb3b/64 scope link noprefixroute
       valid_lft forever preferred_lft forever
4: mgt: <POINTOPOINT,UP,LOWER_UP> mtu 1420 qdisc noqueue state UNKNOWN group defaultqlen 1000
    link/none
    inet <IP_ADDRESS>/16 scope global mgt
       valid_lft forever preferred_lft forever

After some test, I found that ipv6 addresses do not have a nla label. So I remove this line. And the output:
[client/src/main.rs:1170] res = [
    <IP_ADDRESS>,
    <IP_ADDRESS>,
    2409:XXXX:XXXX:XXXX:XXXX:9bff:fea4:7720,
    2409:XXXX:XXXX:XXXX:XXXX:b877:583f:f3ab,
]

Seems that now ipv6 addresses are collected, but innernet's address(<IP_ADDRESS>) is also collected.
I guess that it might be a bug when ipv6 addresses are not reported as cadidates (due to the experiement above). I'm glad to offer my help if needed. Thank you very much for your reply.

oh, there is already a pr implementing this. I will have a try on that branch.

Oh great! Thanks for testing it!
I'll merge that and get another patch release out soon.

Merged #192, closing. Thanks again for opening the issue.

Hi, I guess I'm facing mtu issues, when using 1.3.4-beta2. And this is possibly related to #102. I'm have a ipv4 CIDR range for innernet(<IP_ADDRESS>/16), and I'm connecting to a ipv6 peer. The mtu is set to 1420(by default). It seems that mtu is wrongly configured. When using iperf3, only the first few packages arrive to server. (See logs below)
❯ ip -c l
6: mgt: <POINTOPOINT,UP,LOWER_UP> mtu 1420 qdisc noqueue state UNKNOWN mode DEFAULT group default qlen 1000
    link/none

  peer: home-server (VhItncIvyG...)
    ip: <IP_ADDRESS>
    endpoint: [2409:xxxx:xxxx:xxxx:xxxx:6bff:feef:1110]:33498
    last handshake: 1 minute, 11 seconds ago
    transfer: 375.96 KiB received, 9.54 MiB sent

❯ iperf3 -c home-server.mgt.wg
Connecting to host home-server.mgt.wg, port 5201
[  5] local <IP_ADDRESS> port 40428 connected to <IP_ADDRESS> port 5201
[ ID] Interval           Transfer     Bitrate         Retr  Cwnd
[  5]   0.00-1.00   sec  76.1 KBytes   622 Kbits/sec    2   1.27 KBytes
[  5]   1.00-2.00   sec  0.00 Bytes  0.00 bits/sec    1   1.27 KBytes
[  5]   2.00-3.00   sec  0.00 Bytes  0.00 bits/sec    0   1.27 KBytes
[  5]   3.00-4.00   sec  0.00 Bytes  0.00 bits/sec    1   1.27 KBytes
[  5]   4.00-5.00   sec  0.00 Bytes  0.00 bits/sec    0   1.27 KBytes
^C[  5]   5.00-5.71   sec  0.00 Bytes  0.00 bits/sec    0   1.27 KBytes

After a I set mtu to 1350 for a while, that the network works. And the iperf3 tests give reasonable results.
❯ iperf3 -c home-server.mgt.wg
Connecting to host home-server.mgt.wg, port 5201
[  5] local <IP_ADDRESS> port 40432 connected to <IP_ADDRESS> port 5201
[ ID] Interval           Transfer     Bitrate         Retr  Cwnd
[  5]   0.00-1.00   sec  3.53 MBytes  29.6 Mbits/sec    0    208 KBytes
[  5]   1.00-2.00   sec  1.98 MBytes  16.6 Mbits/sec    6    183 KBytes
[  5]   2.00-3.00   sec  2.48 MBytes  20.8 Mbits/sec    3    183 KBytes

I guess we need to handle this kind of situation: when connecting to ipv6 peers, but using a ipv4 CIDR range.
In previous tests, I only checked the  /var/lib/innernet/mgt.json, and I found that there is a ipv6 address in candidate list. Sorry for that!

@Enter-tainer funnily enough I was just thinking about this right before you posted! Yes I think the MTU logic needs to be reworked. I think there are a couple routes here:

Set all innernet links to a "pretty safe" magic MTU of 1412, since peers can't be assumed to have an IPv4 or IPv6 endpoint.
Have innernet dynamically set the MTU to either 1412 or 1432 depending on if any peers have an IPv6 endpoint (less safe, since peer endpoints can change outside of the innernet binary setting them).
Do more complex discovery by looking at the routing table and identify the MTU of the interface over which wireguard traffic will be flowing, and calculate directly based on that for the maximum possible MTU.

I think for this upcoming release, I'm leaning option 1 to start with.

I did a traceroute --mtu TARGET test. I found that my mtu is actually 1410! That really surprise me. So I set the mtu to 1410-80 = 1330, then everything works.
I'm testing the innernet connection using a wifi hotspot from my phone, and my phone uses 4G LTE, I guess the strange 1410 might result from 4G LTE.
Maybe we should set mtu to the minimum mtu of ipv6: 1280, which cannot be safer. And I agree with you that the ultimate solution should be some mtu discovery techinques similar to traceroute --mtu . Thank you for your time!

@Enter-tainer sorry for the lag in finishing this (I broke my leg and had to get surgery, lol). I think you're right that maybe it does make sense to set a lower MTU for the time being to guarantee connectivity, and then add PMTU functionality down the line (and the MTU is of course configurable via CLI arguments in the mean time).


fix: sending empty data in sse in js client

Description

from our discussion here https://discord.com/channels/919193495116337154/1213242966592856164 it appears that javascript clients dont "see" an event that has no data, while we explicitly test for this and that httpx-sse receives the event fine in that case.
the spec https://html.spec.whatwg.org/multipage/server-sent-events.html#parsing-an-event-stream is not that clear to me as to whether or not an event without data is ok.
Considering the EventSource "client" is not ok with it, and that it's so easy DX-wise to make the mistake not explicitely sending it we fix it by defaulting to the empty-string

The added app is just to manually test the fix works, I dont know how we could incorporate it in our test suite given it involves running a server and checkin if the html page yields the event fine, maybe in e2e testing ?

@provinzkraut how would we do this?

Probably with playwright or something.

I'm all for it but it seems sse is not supported: https://github.com/microsoft/playwright/issues/7466
this is the best I came upwith and it sends error:
import asyncio
from contextlib import asynccontextmanager
from pathlib import Path
from typing import AsyncIterator

from playwright.async_api import Page, async_playwright

from litestar import Litestar, get
from litestar.config.cors import CORSConfig
from litestar.response import ServerSentEvent, ServerSentEventMessage
from litestar.types import SSEData

@get("/test_sse_empty")
async def handler() -> ServerSentEvent:
    async def generate() -> AsyncIterator[SSEData]:
        event = ServerSentEventMessage(event="empty")
        yield event

    return ServerSentEvent(generate())

html="""<!DOCTYPE html>

Server-Sent Events

"""

from uvicorn.config import Config
from uvicorn.server import Server

@asynccontextmanager
async def run_server(config: Config, sockets=None):
    server = Server(config=config)
    task = asyncio.create_task(server.serve(sockets=sockets))
    await asyncio.sleep(0.1)
    try:
        yield server
    finally:
        await server.shutdown()
        task.cancel()

async def test_get_started_link(page: Page, tmp_path: Path ) -> None:
    app = Litestar(route_handlers=[handler], cors_config=CORSConfig(allow_origins=["*"]))
    index = tmp_path / "index.html"
    index.write_text(html)
    async with run_server(config=Config(app=app, port=8765, lifespan="off")):
        async with async_playwright() as p:
            browser = await p.chromium.launch()
            page = await browser.new_page()
            await page.goto(f"file://{index.as_posix()}")
            # Click the get started link.#
            output_div_content = await page.query_selector('div#output')
            print(await output_div_content.inner_text())
            await browser.close()


Why does the latest build REQUIRE Json,NET >= 8.0.2?

Seems a little odd to require the absolute latest version of Json.NET unless it is really needed.  Is It?

It was just the version I had downloaded and built against at the time. Is there a reason not to upgrade?

There have been a fair amount of changes and a lot of people are still using v6 and v7 -- so I was just curious

I would accept pull requests or releases built against earlier versions.


New command line application mdis2isd for generating ISD files from Messenger cubes

The logic of the original application has been abstracted into the mdis2isd class.

@twilson271828 Not sure that I understand this PR - lots of stuff removed (that should have been removed / moved in the merged PR #53 and only 2 lines added.  Is this perhaps incomplete?

Can one of the admins verify this patch?


Internal optimization

Introduction

Add finalizer
Add event output to help locate faults quickly

Contents

Related work

It's need delete pvcs and then remove the finalizer.

https://github.com/vesoft-inc/nebula-operator/pull/409

https://github.com/vesoft-inc/nebula-operator/pull/411


Add catch2 3.6.0

Add catch2 3.6.0

Hello @curoky, modules you maintain (catch2) have been updated in this PR. Please review the changes.

Need skip-url-stability-check label.


Inlay hints broken in recent dev version of neovim.

Did you check docs and existing issues?

[X] I have read all the LazyVim docs
[X] I have searched the existing issues of LazyVim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
v0.10.0-dev-2951+gf1dfe32bf5
Operating system/version
Arch Linux recent
Describe the bug
Changed how built in inlay hints are enabled again
https://github.com/neovim/neovim/commit/f1dfe32bf5552197e0068298b0527526a4f918b1
vim.lsp.inlay_hint.enable(bufnr:number, enable:boolean) is deprecated, use vim.diagnostic.enable(enable:boolean, filter:table) instead. :help deprecated
Feature was removed in Nvim 0.10-dev

Steps To Reproduce
n/a
Expected Behavior
n/a
Repro
return {
  "neovim/nvim-lspconfig",
  opts = {
    inlay_hints = {
      enabled = true,
    },
    servers = {
      -- Ensure mason installs the server
      clangd = {
        mason = false,
        keys = {
          { "<leader>cR", "<cmd>ClangdSwitchSourceHeader<cr>", desc = "Switch Source/Header (C/C++)" },
        },
      },
    },
  },
}

In my case, I can't seem to disable it ! I have it with enabled = false and it still shows up !

In my case, I can't seem to disable it ! I have it with enabled = false and it still shows up !
Using Leader + u + h seems to work fine to disable it, but I have to do it with every single file

same +1


Fix QID quoting.

This was broken in fd0af75322f57c03e1c8b48d5fa5f7b3c6c9d3dc. Add a test to ensure that this keeps working.

Coverage decreased (-0.6%) to 85.94% when pulling 2993c8d6fcb6b0c2b5afb17989ed6228ee7d9472 on leper:fix_qid_escaping into fd0af75322f57c03e1c8b48d5fa5f7b3c6c9d3dc on djc:master.

Coverage decreased (-0.6%) to 85.94% when pulling 2993c8d6fcb6b0c2b5afb17989ed6228ee7d9472 on leper:fix_qid_escaping into fd0af75322f57c03e1c8b48d5fa5f7b3c6c9d3dc on djc:master.

Thanks!


How deactivate gapps location provider?

I need GAPPs only for the PlayStore. I added the second location provider and added other location services.  is it somehow posible to only activate the new and deactivate the google provider?

Sounds like you may want to be using the microG GMScore package at https://github.com/microg/android_packages_apps_GmsCore rather than the older unifiedNLP. With that some have reported success in running PlayStore without installing GAPPS. And GmsCore has unifiedNLP built into it.


Bot to subscribe folks to issues based on labels being added

A bot that, on some label being added to an issue, subscribes some set of folks to the issue. This can be done via adding a comment with their handles.
This should be configured in a .github/subscribers.json file, with the following format:
{
	"label": [
		"UserName1",
		"UserName2",
		"etc."
	]
}

https://github.com/microsoft/vscode/blob/f7131a4cb33a36a63a7bce06119ccebdde29b8d0/.github/subscribers.json#L2-L5

(Note no double-comment 🙂)


get data from api without importing the model

I've been using vuexorm with such syntax this.$store.getters['entities/user/query']().all() . I liked the idea of calling the global store, not having to import anything at component level.
Now I try to use axios plugin. Following the docs, I made this work import User from '../database/models/User' <...>  User.api().get('http://<IP_ADDRESS>:8000/api/v1/user/') . Am wondering if there is a way to use axios plugin with the syntax i'm used to, i.e. is anything like this.$store.api['entities/users']().get('http://<IP_ADDRESS>:8000/api/v1/user/') possible?
Thanks a lot!

You absolutely can, perhaps not exactly the way you've described but most certainly achievable since @vuex-orm/core v0.35
Vuex ORM v0.35 introduces an additional means of retrieving models without the need to import the model modules directly.
this.$store.$db().model('users').api().get('http://<IP_ADDRESS>:8000/api/v1/user/')

Thanks a lot for guidance!


Interposed functions should be internal not private

Given this sample:
contract Base {
   /// #if_succeeds true;
   function foo() internal returns (uint) {}
}

/// #invariant foo() == 1;
contract Child is Base {
}

The foo function in the base will be renamed to __original_Base_foo and made private.
However in the instrumentation of Child we will try to use the interposed function (__original_Base_foo) which is no longer accessible in Child since its now private.
To fix this we should make interposed functions internal.

This is addressed by #258. Closing.


TDQM hook with overrides

Current behavior
When using hamilton.plugins.h_tdqm.ProgressBar(), the progress bar uses "the number of nodes to return" as denominator and "the number of nodes executed" as numerator. However, passing values to .execute(overrides=...) counts towrds the first, but not the latter. Code runs to completion but the progress bar completes without reaching 100%
Screenshots

Dataflow
# joke.py

def joke_prompt(topic: str) -> str:
    return f"Knock, knock. Who's there? {topic}"

def reply(joke_prompt: str) -> str:
    _, _, right = joke_prompt.partition("? ")
    return f"{right} who?"

def punchline(reply: str) -> str:
    left, _, _ = reply.partition(" ")
    return f"No, {left} MooOOooo"

Execution
import joke
from hamilton import driver
from hamilton.plugins.h_tqdm import ProgressBar
dr = (
    driver.Builder()
    .with_adapters(ProgressBar())
    .with_modules(joke)
    .build()
)

dr.execute(["joke_prompt", "reply"], overrides=dict(punchline="..."), inputs=topic("hello")).

Hmm this might be a bit tricky -- we'll probably want to add a get_execution_plan() for the HamiltonGraph object. Otherwise we won't have access to the number. Otherwise we'll also want a overrides in the GraphExecutionHook (https://github.com/DAGWorks-Inc/hamilton/blob/046ed599e00ad186c361e2ce94192b9f0688e8d1/hamilton/lifecycle/api.py#L271).
Everything will be backwards compatible, but not super simple.

We could just make sure there's a post graph hook implementation that makes sure the bar hits 100%?

Hi @skrawcz following up from issue #1186
and @zilto I have come up with the following:

adjusting pre_graph_execute by adding the overridden node to execution path to reflect progress:

nodes_to_execute = set(all_nodes) - set(user_defined_nodes) | set(overrides.keys())

OR
nodes_to_execute = set(all_nodes)

in post_graph_execute force progress bar to 100% if the execution is successful:

if success and hasattr(self, 'progress_bar'):
       self.progress_bar.update_to_completion()

@rajatkriplani Cool feel free to create a PR with it. If there isn't a test we'll need to add one.


PackFileCacheStrategy warnings with global Promise hook

Bug report

What is the current behavior?
I was runing a nextjs project with webpack5. When I added following code
class HookPromise extends Promise {}
global.Promise = HookPromise

I got some warnings like
<w> [webpack.cache.PackFileCacheStrategy] Restoring failed for Compilation/codeGeneration|external "@babel/runtime-corejs2/core-js/parse-int"|webpack-runtime from pack: Error: A lazy value that has been unmemorized can't be called again
<w> [webpack.cache.PackFileCacheStrategy] Restoring failed for Compilation/codeGeneration|external "next/dist/server/image-config.js"|webpack-runtime from pack: Error: A lazy value that has been unmemorized can't be called again
<w> [webpack.cache.PackFileCacheStrategy] Restoring failed for Compilation/codeGeneration|external "next/dist/shared/lib/to-base-64.js"|webpack-runtime from pack: Error: A lazy value that has been unmemorized can't be called again
<w> [webpack.cache.PackFileCacheStrategy] Restoring failed for Compilation/assets|chunkpages/blog from pack: Error: A lazy value that has been unmemorized can't be called again

If the current behavior is a bug, please provide the steps to reproduce.
see above

What is the expected behavior?
in lib/cache/PackFileCacheStrategy.js:
if (value instanceof Promise) {
  ...
}

I changed the conditional statement, the warnings were disappeared
if (value && typeof value.then === 'function') {
  ...
}

Other relevant information:
nextjs version: 11.1.2
webpack version:  nextjs bundled version
Node.js version:  14.16.0
Operating System:  mac os 11.4
Additional tools:

PR welcome

I think there was a PR for that already..

@alexander-akait Can i work on this issue and create a PR?

Yes, PR welcome

@alexander-akait Can you guide me on how to resolve this issue, the fix is giving an error message -> 'Property 'then' does not exist on type 'PackContentItems | Promise'.

Example of fix inside the issue, we should change value instanceof Promise on value && typeof value.then === 'function'


Notifications do not always show legitimate mentions

Through extensive testing I am sure that the mentions are missed on notifications many times.
Have not found a rhyme or reason as to why they are not indexed.
Have confirmed with @maebeam that I am not Grey or Blacklisted.
Can see the mentions on the blockchain data but they don't show on notifications on local node or Bitclout.com.
This will surely effect user experience, so just trying to help by mentioning it.

Have confirmed other user's posts do not show in mentions as well.
First thought it was from extensive tagging, but after research I do not think is the case.
It does seem to be isolated to certain users or posts, though.
Will keep looking for the rhyme and reason...

On the block explorer you can see AffectedPublicKeys. If someone should have received a notification for a post and their key is missing from AffectedPublicKeys that's a problem. Perhaps try starting there?

Thank you for replying.
I did a quick search on one of the last mentions I noticed were missing.
(This extends to multiple accounts and multiple users who mentions never show, or sometimes users normally show up but some notifications are missed)
From: ClementBourcart  BC1YLgmMPiVu3B1SkMcpHuGpZm5maWqUWdjtR8EruZyfqFdv4YYphis
Menton: RemindMe BC1YLiDHaq9FuvZYiDzXe2z4jbnphbZa7C8cGSL147HWLYHDm9Vea2Y
PostHashHex: 70651c5cfef81f5c801ce7309d67e5698a8273a15e4e0eeae62d018c30f741f7
There are no AffectedKeys on this transaction in the block. (Height: 35772)
I could supply many more if needed.
{
  "BlockHashHex": "000000000000b6695eb5c5efeeca7a799b4d206829c9d0baac239431ef092dbb",
  "TxnIndexInBlock": 216,
  "TxnType": "SUBMIT_POST",
  "TransactorPublicKeyBase58Check": "BC1YLgmMPiVu3B1SkMcpHuGpZm5maWqUWdjtR8EruZyfqFdv4YYphis",
  "AffectedPublicKeys": [
    {
      "PublicKeyBase58Check": "BC1YLgmMPiVu3B1SkMcpHuGpZm5maWqUWdjtR8EruZyfqFdv4YYphis",
      "Metadata": "BasicTransferOutput"
    },
    {
      "PublicKeyBase58Check": "BC1YLhhjeJ3t4EZSkK5w257F2vxuacR4ZSbZL4Qu3axJef495PkP3gQ",
      "Metadata": "ParentPosterPublicKeyBase58Check"
    }
  ],
  "TxnOutputs": [
    {
      "PublicKey": "ApAkayhty2LpY7xaOurKoMMtT7Vnv8D1yJg63wfVecpm",
      "AmountNanos": 3747
    }
  ],
  "BasicTransferTxindexMetadata": {
    "TotalInputNanos": 4089,
    "TotalOutputNanos": 3747,
    "FeeNanos": 342,
    "UtxoOpsDump": ""
  },
  "BitcoinExchangeTxindexMetadata": null,
  "CreatorCoinTxindexMetadata": null,
  "CreatorCoinTransferTxindexMetadata": null,
  "UpdateProfileTxindexMetadata": null,
  "SubmitPostTxindexMetadata": {
    "PostHashBeingModifiedHex": "70651c5cfef81f5c801ce7309d67e5698a8273a15e4e0eeae62d018c30f741f7",
    "ParentPostHashHex": "26cc940acab9e9d1ec1621f40bd8b8cac3e433e58340297bd2c843d4b11f18b2"
  },
  "LikeTxindexMetadata": null,
  "FollowTxindexMetadata": null,
  "PrivateMessageTxindexMetadata": null,
  "SwapIdentityTxindexMetadata": null
}

If you want me to PM you a way to check your own mentions against your notifications - let me know. Might help for you guys to see how prevalent it is.

d33e9cbf9034e11335e7bb575de962e16204e931b8bc6992b8df21cd8f2fd3bf has AffectedKeys, but none for @RemindMe which was mentioned. (BC1YLiDHaq9FuvZYiDzXe2z4jbnphbZa7C8cGSL147HWLYHDm9Vea2Y)
{
  "BlockHashHex": "00000000000029eb014d005d534793fa9de370534292ef1c15d22ae0b0379fdb",
  "TxnIndexInBlock": 332,
  "TxnType": "SUBMIT_POST",
  "TransactorPublicKeyBase58Check": "BC1YLfyWj3NaC2WC3zqvz42KiVC4np9vouf6Zp5zs9HA77HTEv58YUu",
  "AffectedPublicKeys": [
    {
      "PublicKeyBase58Check": "BC1YLfyWj3NaC2WC3zqvz42KiVC4np9vouf6Zp5zs9HA77HTEv58YUu",
      "Metadata": "BasicTransferOutput"
    },
    {
      "PublicKeyBase58Check": "BC1YLfzL4FNZJ8k1xCVvVAA3UeMoQApG9V7VW1aVW9yAQdRNzHfqSWg",
      "Metadata": "MentionedPublicKeyBase58Check"
    },
    {
      "PublicKeyBase58Check": "BC1YLfzL4FNZJ8k1xCVvVAA3UeMoQApG9V7VW1aVW9yAQdRNzHfqSWg",
      "Metadata": "RecloutedPublicKeyBase58Check"
    }
  ],
  "TxnOutputs": [
    {
      "PublicKey": "AigPG5pIpXKAwu7qZigM8EWA9gGS6WXULVfZ+PcI69X+",
      "AmountNanos": 13030505
    }
  ],
  "BasicTransferTxindexMetadata": {
    "TotalInputNanos": 13031140,
    "TotalOutputNanos": 13030505,
    "FeeNanos": 635,
    "UtxoOpsDump": ""
  },
  "BitcoinExchangeTxindexMetadata": null,
  "CreatorCoinTxindexMetadata": null,
  "CreatorCoinTransferTxindexMetadata": null,
  "UpdateProfileTxindexMetadata": null,
  "SubmitPostTxindexMetadata": {
    "PostHashBeingModifiedHex": "d33e9cbf9034e11335e7bb575de962e16204e931b8bc6992b8df21cd8f2fd3bf",
    "ParentPostHashHex": ""
  },
  "LikeTxindexMetadata": null,
  "FollowTxindexMetadata": null,
  "PrivateMessageTxindexMetadata": null,
  "SwapIdentityTxindexMetadata": null
}

Here is one more example missing from another account's mentions, and then I'll leave this be unless asked for more info. 😁
From: MLefebvre - BC1YLh2TefSmULToFzrmXGpAcjKPJwCabU7XHTw6W63XBURSHZQZik8
Mention: CloutDash - BC1YLgD76rZNujWj5vjFhdPYzLgQWQJifh2JJbc17kYz3vgJCbT1x4i
The poster, MLefebvre is tagged as an AffectedPublicKey, as is the original poster, but not CloutDash.

{
  "BlockHashHex": "0000000000009ed456ddb935ff279a644c168779ed612009ed6ecdd2f19c2b77",
  "TxnIndexInBlock": 1805,
  "TxnType": "SUBMIT_POST",
  "TransactorPublicKeyBase58Check": "BC1YLh2TefSmULToFzrmXGpAcjKPJwCabU7XHTw6W63XBURSHZQZik8",
  "AffectedPublicKeys": [
    {
      "PublicKeyBase58Check": "BC1YLh2TefSmULToFzrmXGpAcjKPJwCabU7XHTw6W63XBURSHZQZik8",
      "Metadata": "BasicTransferOutput"
    },
    {
      "PublicKeyBase58Check": "BC1YLiU9Wpk8c7pVx7GCu8fae6AQhdUrGYcWc1Wz9V56HbCi7RPr887",
      "Metadata": "ParentPosterPublicKeyBase58Check"
    }
  ],
  "TxnOutputs": [
    {
      "PublicKey": "ArJyUtDvdyN3jyu9iLbzUl+56/Ue9WK78CUTrFoy054T",
      "AmountNanos": 1650
    }
  ],
  "BasicTransferTxindexMetadata": {
    "TotalInputNanos": 1997,
    "TotalOutputNanos": 1650,
    "FeeNanos": 347,
    "UtxoOpsDump": ""
  },
  "BitcoinExchangeTxindexMetadata": null,
  "CreatorCoinTxindexMetadata": null,
  "CreatorCoinTransferTxindexMetadata": null,
  "UpdateProfileTxindexMetadata": null,
  "SubmitPostTxindexMetadata": {
    "PostHashBeingModifiedHex": "17e7ab69ef66295d3cc8af3f88b6c216228c1cf81eeb6a49f9bb0c335f7209cc",
    "ParentPostHashHex": "c3067a29a4d4ddcc1a5563538ecc26becc87570bde50d42461ab66016b001b5d"
  },
  "LikeTxindexMetadata": null,
  "FollowTxindexMetadata": null,
  "PrivateMessageTxindexMetadata": null,
  "SwapIdentityTxindexMetadata": null
}

I've also noticed this issue. Tested around it a little yesterday, and it seemed to be that if you have some extra characters after the @ mention — e.g. "@maebeam's" instead of "@maebeam" — then no notification will get sent. In the attached example, I didn't get a notification for the mention, but on other posts/replies where it was just the username with no text around it, it worked fine.

Thanks for digging into this issue everyone! It made it super easy for me to fix it
https://github.com/bitclout/core/pull/43
Closing this issue as it will be fixed once that PR is merged later today.

Want to make sure the fix is not to only catch commas and other standard "terminators" other than space?
All the cases I have seen were followed by a space.


Add BigQuery Data Transfer API quickstart

This sample lists the data sources enabled for a project.

Looked like the Kokoro failure was on lint with the new test class. I re-ran Format-Code.

@SurferJeffAtGoogle It looks like Kokoro is happy finally.


Add Facebook Open Graph and Twitter Cards metadata

This PR adds some basic Open Graph and Twitter Cards metadata to each page. I make a distinction here between the index page and documents (i.e. material in /docs). Metadata that applies to all pages is templated in layouts/partials/meta.ace and supplied via the params in config.yaml; metadata that only applies to the index page is templated in layouts/partials/index/index-meta.ace; and metadata for documents is templated in layouts/partials/docs/page-meta.ace.

+1

@billonahill - do you have any comments on this?

I'm not that familiar with the syntax of either but these changes seem reasonable.


v4: AMD support missing

Tether and jquery support AMD loading.  Can this please be added to the v4 release.  I looked and there were quite a few others looking for the same, and as v4 is nearing beta, can't we get this?

If memory serves, and I don't fully understand the UMD/AMD stuff, but I think we stripped it all out in https://github.com/twbs/bootstrap/pull/20072 in favor of ES6.


if cloning the master branch fails, try cloning the main branch

Addresses issue #1 .

Great, thanks for this addition! There actually is branch support, but this PR is much needed to not have to assume the default branch name. I haven't updated docs for branch support yet 😓 .
If you wanted to use a different branch, you can use ?ref query options to your github address. Here's an example:
  terraformModule:
    address: https://github.com/isaaguilar/non-main-default-test.git?ref=branch1
  terraformVersion: 0.12.23

Should also works for the ssh method too:
  terraformModule:
    # ?ref accepts a branch or sha-hash
    address: git@github.com:isaaguilar/non-main-default-test.git?ref=63b5bb4ddb1cf77f6bec8f02dbd04baefe768ffd
  terraformVersion: 0.12.23

Great, thanks for this addition! There actually is branch support, but this PR is much needed to not have to assume the default branch name. I haven't updated docs for branch support yet 😓 .
If you wanted to use a different branch, you can use ?ref query options to your github address. Here's an example:
  terraformModule:
    address: https://github.com/isaaguilar/non-main-default-test.git?ref=branch1
  terraformVersion: 0.12.23

Should also works for the ssh method too:
  terraformModule:
    # ?ref accepts a branch or sha-hash
    address: git@github.com:isaaguilar/non-main-default-test.git?ref=63b5bb4ddb1cf77f6bec8f02dbd04baefe768ffd
  terraformVersion: 0.12.23


Right click in SQL editor deselects selected text

Description
Since the latest build, 22.2.4, selecting text and right clicking on the text deselects the text, making it impossible to use context menu operations such as 'morph to delimited list'.
DBeaver Version
22.2.4
Operating System
Windows 10
Database and driver
No response
Steps to reproduce
Open a SQL editor to a database. Enter text. Select text. Right click. Text will become deselected.
Additional context
No response

Thank you for bug report. It's a duplicate of #18152 Fix will be available in the next release.

Ah, thank you! I did look but couldn't see anything. Good to know a fix is on the way!


Replace the test script with more robust python program

The python test program:

Probes device to determine if it is running bootloader or app
Loads bootloader if appropriate, otherwise runs signature
Traps unexpected disconnects during all device communication
Has a timeout for handling signature app hang

We add a run script that sets up venv and runs the python program. The old test script is moved to attic/.
Co-authored-by: Daniel Lublin<EMAIL_ADDRESS>

@cibomahto does all you need?

Looks great to me!


path caching: make sure intersecting paths are copied

This is part bug-fix, part feature request.
971edd3 fixes a bug where a gem with a path that is similar to the bundled application path would not be copied on package --all. For example, a gem with path /test_gem/ would not be copied into /test/vendor/cache.
570e60d is a feature request to preserve the buggy behavior with an option. In my case, it would be helpful because we can restart a docker container that has the gem path mapped in without having to rerun package and copy it into vendor/cache.

I don't really see why we need to add an option for this, as doing so would obligate us to support and test it for a while. The bug fix looks good, though!

Thanks for the patch!
Honestly, it seems a little weird to me to have a flag where 50% of the reason it exists is to cache path deps, and then have yet another flag whose sole point is to cancel out that half of the first flag. If you don't want to cache path deps, you can always remove them from vendor/cache after you run bundle pack.

Cherry-picked in #3907 as 7e8d399.

Fine by me, just putting it out there. Thanks.

FYI solving this with:

not checking if path gems need to be packaged via BundlePackageCheck ignore_path: true
removing all path gems rm -rf #{Dir["vendor/cache/*"].reject { |x| x.include?("-").join(" ") }


V1.0.25 rc

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: wyyalt:x: github-actionsgithub-actions seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Codecov Report

Merging #298 (509a6ba) into V1.0.X (4dee143) will not change coverage.
Report is 1 commits behind head on V1.0.X.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           V1.0.X     #298   +/-   ##
=======================================
  Coverage   40.80%   40.80%
=======================================
  Files         201      201
  Lines        9474     9474
=======================================
  Hits         3866     3866
  Misses       5608     5608

Files Changed
Coverage Δ

...kit/pipeline_plugins/components/collections/gse.py
41.20% <0.00%> (ø)


Variant calling time

Hi Brad,
I am using four vairant callers: mutect2, freebayes, vardict, varscan and ensemble: numpass: 2 criteria to run the turmor-germline cancer variant calling pipeline for 17 pairs in the new 1.0.0 development version with GATK 3.6 and the most updated dbSNP. I found that the variant calling running time is much longer than previous version, GATK 3.4, mutect1.7, and old dbSNP. Previously, all the running to from the beginning to the final results will be within 4 days. So I just set up 4-day time limit in the slurm file. However, the 4-day time limit has expired just now. Then I re-submit the slurm file but found that the ProgressMeter showed 4-7 days remaining runtime for various locations in variant calling. It seems that the bcbio re-ran the variant calling from the beginning but not from the points that previously had completed. Do you have any solution to let the pipeline start from the points that previously have finished to save time in variant calling stage. Otherwise, it seems that it will take tremendous time if the pipeline breaks during the variant calling stage.
Thanks.

Hi Brad,
Is there way to solve this problem?
Thanks.

Sorry about the issues. bcbio splits variant calling into genomic regions and on restart will not redo any region/caller combinations that finished. However, if you have a long running process that did not finish cleanly, bcbio will not be able to restart that and will have to start that section and caller over again.
It sounds like in this case you're running into long run times for MuTect2. My experience is that it has unpredictably variable run times, and some inputs will cause it to run extremely slowly as you're seeing. I don't have a good solution for this, it's really a Mutect2 limitation.
Hope this helps explain the situation.

Brad,
I see. Thank you for your explanation. If this is the case, is there any way that I can downgrade GATK into 3.6 and MuTect 1.17 that I used previously?
Thanks.

There is no need to downgrade, you can just specify mutect instead of mutect2 in your configuration file. They are two separate callers. You will need to have Java 1.7 available in your PATH (bcbio only ships with Java 1.8) but otherwise everything should work cleanly. Hope this helps.

Yes, this really helps. Thank you, Brad!

This is the Error. Should I copy the jar muTect 1.1.7 into the directory as follows?
ValueError: Could not find jar muTect in /path/to/parallel/bcbio/anaconda/bin/picard:/path/to/parallel/bcbio/anaconda/bin/picard

Brad,
Could you answer my above question?
Thanks!

We're happy to try and help but please don't ask in the same post multiple times within a short time window. We try to answer as fast as we can within our other commitments.
The install documentation describes how to add mutect. We're not allowed to distribute it so it needs a manual download and install (http://bcbio-nextgen.readthedocs.io/en/latest/contents/installation.html#gatk-and-mutect-mutect2):
bcbio_nextgen.py upgrade --tools --toolplus mutect=/path/to/mutect/mutect-1.1.7.jar

Hope this helps.

Thanks you, Brad!


Test failure

The following test fails when test suite is executed:
Failed tests:
ttl(redis.clients.jedis.tests.commands.AllKindOfValuesCommandsTest)
Tests run: 262, Failures: 1, Errors: 0, Skipped: 1

Should be working now :)


Fix binding issue for .NET analyzers  in SDK

Reverts breaking change in PR #1807

@weshaggard @chidozieononiwu @pakrym should we require that any changes to the analyzers be used in Azure/azure-sdk-for-net (otherwise, what's the point of the change generally?), or even set up a pipeline similar to swagger changes that at least tries the change(s) out in the .NET repo?

I should also mention I build a local binary and tried it out in VS and it works fine. Oddly, dotnet build always worked even with the previous value.


TypeScript Support

Will this library receive TS support?

Hey @kevineaton603
Sure it will receive as soon as someone will submit a PR for that 😄

I'm gonna start working on this