7pm-3 md and bl - Search bars on slack search

In this PR, we add a searchField property to the MessageListView component and set it to false on the Slack Search page to remove the confusing double search bar. The default is true so any other pages will continue to display it. Additionally added tests and storybook stories for true and false values of searchField. Closes #220

I'm now realizing I must have originally done git checkout while on a branch other than the main branch... what is the best way to fix this?

I'm now realizing I must have originally done git checkout while on a branch other than the main branch... what is the best way to fix this?

In addition these old changes are causing mvn tests to fail, while the relevant 4 files are ok

Since this PR isn't ready, can you mark it as a draft for now? You can do that at the bottom of the "Reviewers" section in the right bar.
Let's discuss the issue you're having in section today so we can figure it out together.


Pass Scheduler Backend Config to Childresources

Summary
TBD #256

Checklist

[x] Keep pull requests small so they can be easily reviewed.
[x] Categorize the PR by setting a good title and adding one of the labels:
bug, enhancement, documentation, change, breaking,
as they show up in the changelog
[ ] Update the documentation.
[ ] Update tests.
[x] Link this PR to related issues.

Since you touched resource merging stuff I tested resource templates in this branch, to see how that works now.
We basically have 3 levels where we can specify resources, each overrides the previous one:

Global resource limits and requests
resourceRequirementsTemplate in the schedule Spec
resources of a job spec (e.g. prune)

This PR has the behaviour so that the Schedule after reconciling has additional resources specified in the Job spec.
e.g.
---
apiVersion: backup.appuio.ch/v1alpha1
kind: Schedule
metadata:
  name: schedule-test
spec:
  resourceRequirementsTemplate:
    requests:
      cpu: "250m"
  prune:
    schedule: '*/1 * * * *'

becomes
apiVersion: backup.appuio.ch/v1alpha1
kind: Schedule
metadata:
  name: schedule-test
  namespace: default
spec:
...
  prune:
    resources:
      requests:
        cpu: 250m
    retention:
      keepDaily: 14
      keepLast: 5
    schedule: '*/1 * * * *'
  resourceRequirementsTemplate:
    requests:
      cpu: 250m

(only important thing: spec.prune.resources got added)
And the prune spec gets as expected:
apiVersion: backup.appuio.ch/v1alpha1
kind: Prune
metadata:
  name: prunejob-1609946760
  namespace: default
spec:
  resources:
    requests:
      cpu: 250m
...

All the while the operator has this env var set: kubectl set env deployment/k8up-operator BACKUP_GLOBALMEMORY_LIMIT=76M -n k8up-system. But the prunejob-1609946760 CR doesn't get the memory limit.
There are 2 issues:

The resources of a specific Job spec in test-schedule gets added after reconciling. I think this shouldn't be as it would make changing a value in resourceRequirementsTemplate impossible because spec.<job>.resources is more specific. Only when creating a Prune object, the final values should be in there, but not in the Schedule
The global resource values aren't applied at all. Though this could already be the case in the current master branch. Insufficient testing? :thinking: In any case, if spec.<job>.resources again gets set in the spec, it would disallow cluster administrators to effectively alter resource requirements after the first reconciliation (given users don't override this in the initial source YAML themselves, then it's their responsibility to meet cluster resource policies).

In code this means that merged resources should be kept as runtime variables and stored in the Prune CR. And not stored in the Schedule spec after reconciliation.
See also #143
Would you like a separate issue for this bug? Or do you think you can handle this in this PR when you're already touching the code anyway?

Since you touched resource merging stuff I tested resource templates in this branch, to see how that works now.
We basically have 3 levels where we can specify resources, each overrides the previous one:

Global resource limits and requests
resourceRequirementsTemplate in the schedule Spec
resources of a job spec (e.g. prune)

This PR has the behaviour so that the Schedule after reconciling has additional resources specified in the Job spec.
e.g.
---
apiVersion: backup.appuio.ch/v1alpha1
kind: Schedule
metadata:
  name: schedule-test
spec:
  resourceRequirementsTemplate:
    requests:
      cpu: "250m"
  prune:
    schedule: '*/1 * * * *'

becomes
apiVersion: backup.appuio.ch/v1alpha1
kind: Schedule
metadata:
  name: schedule-test
  namespace: default
spec:
...
  prune:
    resources:
      requests:
        cpu: 250m
    retention:
      keepDaily: 14
      keepLast: 5
    schedule: '*/1 * * * *'
  resourceRequirementsTemplate:
    requests:
      cpu: 250m

(only important thing: spec.prune.resources got added)
And the prune spec gets as expected:
apiVersion: backup.appuio.ch/v1alpha1
kind: Prune
metadata:
  name: prunejob-1609946760
  namespace: default
spec:
  resources:
    requests:
      cpu: 250m
...

All the while the operator has this env var set: kubectl set env deployment/k8up-operator BACKUP_GLOBALMEMORY_LIMIT=76M -n k8up-system. But the prunejob-1609946760 CR doesn't get the memory limit.
There are 2 issues:

The resources of a specific Job spec in test-schedule gets added after reconciling. I think this shouldn't be as it would make changing a value in resourceRequirementsTemplate impossible because spec.<job>.resources is more specific. Only when creating a Prune object, the final values should be in there, but not in the Schedule
The global resource values aren't applied at all. Though this could already be the case in the current master branch. Insufficient testing? :thinking: In any case, if spec.<job>.resources again gets set in the spec, it would disallow cluster administrators to effectively alter resource requirements after the first reconciliation (given users don't override this in the initial source YAML themselves, then it's their responsibility to meet cluster resource policies).

In code this means that merged resources should be kept as runtime variables and stored in the Prune CR. And not stored in the Schedule spec after reconciliation.
See also #143
Would you like a separate issue for this bug? Or do you think you can handle this in this PR when you're already touching the code anyway?

The global resource values aren't applied at all. Though this could already be the case in the current master branch.

I have not confirmed this, but since I did not actually touch that logic I suspect so. I believe that this can be handled in a separate issue.

In code this means that merged resources should be kept as runtime variables and stored in the Prune CR. And not stored in the Schedule spec after reconciliation.

I have changed to logic accordingly. The reconciliation loop still creates the Resources key for some reason I have not been able to figure out yet (but I have stopped looking for now as it's just a Schoenheitsfehler, IMO).

The global resource values aren't applied at all. Though this could already be the case in the current master branch.

I have not confirmed this, but since I did not actually touch that logic I suspect so. I believe that this can be handled in a separate issue.

In code this means that merged resources should be kept as runtime variables and stored in the Prune CR. And not stored in the Schedule spec after reconciliation.

I have changed to logic accordingly. The reconciliation loop still creates the Resources key for some reason I have not been able to figure out yet (but I have stopped looking for now as it's just a Schoenheitsfehler, IMO).

Running the current branch and applying the default sample actually throws panics:
2021-01-08T10:21:15.355Z	INFO	controllers.Schedule	registering schedule for	{"schedule": "default/schedule-test", "type": "prune", "schedule": "*/2 * * * *"}
2021-01-08T10:21:15.355Z	DEBUG	controller	Successfully Reconciled	{"reconcilerGroup": "backup.appuio.ch", "reconcilerKind": "Schedule", "controller": "schedule", "name": "schedule-test", "namespace": "default"}
E0108 10:21:15.356032       1 runtime.go:78] Observed a panic: &reflect.ValueError{Method:"reflect.Value.Type", Kind:0x0} (reflect: call of reflect.Value.Type on zero Value)
goroutine 312 [running]:
k8s.io/apimachinery/pkg/util/runtime.logPanic(0x1514a80, 0xc00067fe20)
	/home/chris/go/pkg/mod/k8s.io/apimachinery@v0.18.10/pkg/util/runtime/runtime.go:74 +0xa6
k8s.io/apimachinery/pkg/util/runtime.HandleCrash(0x0, 0x0, 0x0)
	/home/chris/go/pkg/mod/k8s.io/apimachinery@v0.18.10/pkg/util/runtime/runtime.go:48 +0x89
panic(0x1514a80, 0xc00067fe20)
	/snap/go/6439/src/runtime/panic.go:969 +0x175
reflect.Value.Type(0x0, 0x0, 0x0, 0x18d19a0, 0x1625f20)
	/snap/go/6439/src/reflect/value.go:1894 +0x189
github.com/imdario/mergo.merge(0x15804a0, 0xc00067d6c0, 0x15804a0, 0x0, 0x0, 0x0, 0x0, 0x16e3327, 0xc)
	/home/chris/go/pkg/mod/github.com/imdario/mergo@v0.3.11/merge.go:358 +0x20a
github.com/imdario/mergo.Merge(...)
	/home/chris/go/pkg/mod/github.com/imdario/mergo@v0.3.11/merge.go:291
github.com/vshn/k8up/handler.(*ScheduleHandler).mergeBackendWithDefaults(0xc0006c7c48, 0xc00067d6c0)
	/home/chris/Documents/source/vshn/k8up/handler/schedule.go:152 +0x79
github.com/vshn/k8up/handler.(*ScheduleHandler).mergeWithDefaults(0xc0006c7c48, 0xc0004e3620)
	/home/chris/Documents/source/vshn/k8up/handler/schedule.go:139 +0x65
github.com/vshn/k8up/handler.(*ScheduleHandler).createJobList(0xc0006c7c48, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, ...)
	/home/chris/Documents/source/vshn/k8up/handler/schedule.go:91 +0x7bc
github.com/vshn/k8up/handler.(*ScheduleHandler).Handle(0xc0006c7c48, 0x8, 0xc0006c7b78)
	/home/chris/Documents/source/vshn/k8up/handler/schedule.go:54 +0x85

Running the current branch and applying the default sample actually throws panics:
2021-01-08T10:21:15.355Z	INFO	controllers.Schedule	registering schedule for	{"schedule": "default/schedule-test", "type": "prune", "schedule": "*/2 * * * *"}
2021-01-08T10:21:15.355Z	DEBUG	controller	Successfully Reconciled	{"reconcilerGroup": "backup.appuio.ch", "reconcilerKind": "Schedule", "controller": "schedule", "name": "schedule-test", "namespace": "default"}
E0108 10:21:15.356032       1 runtime.go:78] Observed a panic: &reflect.ValueError{Method:"reflect.Value.Type", Kind:0x0} (reflect: call of reflect.Value.Type on zero Value)
goroutine 312 [running]:
k8s.io/apimachinery/pkg/util/runtime.logPanic(0x1514a80, 0xc00067fe20)
	/home/chris/go/pkg/mod/k8s.io/apimachinery@v0.18.10/pkg/util/runtime/runtime.go:74 +0xa6
k8s.io/apimachinery/pkg/util/runtime.HandleCrash(0x0, 0x0, 0x0)
	/home/chris/go/pkg/mod/k8s.io/apimachinery@v0.18.10/pkg/util/runtime/runtime.go:48 +0x89
panic(0x1514a80, 0xc00067fe20)
	/snap/go/6439/src/runtime/panic.go:969 +0x175
reflect.Value.Type(0x0, 0x0, 0x0, 0x18d19a0, 0x1625f20)
	/snap/go/6439/src/reflect/value.go:1894 +0x189
github.com/imdario/mergo.merge(0x15804a0, 0xc00067d6c0, 0x15804a0, 0x0, 0x0, 0x0, 0x0, 0x16e3327, 0xc)
	/home/chris/go/pkg/mod/github.com/imdario/mergo@v0.3.11/merge.go:358 +0x20a
github.com/imdario/mergo.Merge(...)
	/home/chris/go/pkg/mod/github.com/imdario/mergo@v0.3.11/merge.go:291
github.com/vshn/k8up/handler.(*ScheduleHandler).mergeBackendWithDefaults(0xc0006c7c48, 0xc00067d6c0)
	/home/chris/Documents/source/vshn/k8up/handler/schedule.go:152 +0x79
github.com/vshn/k8up/handler.(*ScheduleHandler).mergeWithDefaults(0xc0006c7c48, 0xc0004e3620)
	/home/chris/Documents/source/vshn/k8up/handler/schedule.go:139 +0x65
github.com/vshn/k8up/handler.(*ScheduleHandler).createJobList(0xc0006c7c48, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, ...)
	/home/chris/Documents/source/vshn/k8up/handler/schedule.go:91 +0x7bc
github.com/vshn/k8up/handler.(*ScheduleHandler).Handle(0xc0006c7c48, 0x8, 0xc0006c7b78)
	/home/chris/Documents/source/vshn/k8up/handler/schedule.go:54 +0x85

Running the current branch and applying the default sample actually throws panics:

Interesting. My operator was running the whole night and I have not observed such a panic. Is it reproducible on your end? And would you mind sharing the actual schedule definition, since there are many floating around in the docs and the repo.

Running the current branch and applying the default sample actually throws panics:

Interesting. My operator was running the whole night and I have not observed such a panic. Is it reproducible on your end? And would you mind sharing the actual schedule definition, since there are many floating around in the docs and the repo.

Interesting. My operator was running the whole night and I have not observed such a panic. Is it reproducible on your end? And would you mind sharing the actual schedule definition, since there are many floating around in the docs and the repo.

make clean e2e-test
k apply -f config/samples/k8up_v1alpha1_schedule.yaml

Interesting. My operator was running the whole night and I have not observed such a panic. Is it reproducible on your end? And would you mind sharing the actual schedule definition, since there are many floating around in the docs and the repo.

make clean e2e-test
k apply -f config/samples/k8up_v1alpha1_schedule.yaml

When I run k apply -f config/samples/k8up_v1alpha1_schedule.yaml against the correct cluster it indeed crashed my k8up as well. This has now been fixed.

When I run k apply -f config/samples/k8up_v1alpha1_schedule.yaml against the correct cluster it indeed crashed my k8up as well. This has now been fixed.

Rebased onto current master.

Rebased onto current master.


Pokemon Detail view error

When I connect the pgog- optimizer server which I put in my mac and start to play, all of my Pokemon Detail page seem to be wrong, I can't see the right data and use the function in that page.

It's a know issue mainly due some wrong protobuf files.

any plan to solve the problem？：）

@toryxxxx You can make this change that i mention on this pull request:
https://github.com/justinleewells/pogo-optimizer/pull/63/files
And reinstall pogo-optimized (Delete node_modules folder and do npm install again)
This will only solve a few problems. We have to wait for the guys from protobuf team solves this.

You don't need to be constantly connected to get the information you need, it's easier just to disable the proxy once you have uploaded your data and then go catch more pokemon and do a mass release, but as @esfomeado says if you grab his change you will always have the latest version of the mitm project used in the project.

thank you~ @Shauneh @esfomeado

I believe this is fixed in the newest version. Feel free to re-open if I'm wrong.


style(tangle-dapp): Implement changes from Tangle account UI audit

Summary of changes
Provide a detailed description of proposed changes.

Update Account Summary Card Logo:

Replace the current account card logo with the new version.
Ensure the logo adapts for the light theme appropriately.

Account Summary Card Layout:

Implement a minimum width of approximately 755px (tbd) to ensure card size responsiveness view.
Below this width, stack components vertically and expand them to fill the space.

Manage Balances Table:

Hide balances in Manage Balances Table to match the total balance's hidden state.
Replace local chips with Webb UI's chip (specifically, use purple chips).
Increase padding at the top table row items to align with Figma designs in locked balances container.

Payouts Button Functionality:

Enhance the 'Payouts' button to navigate to the 'EVM Staking' page and automatically select the 'Payouts' tab.

Other changes & additions

Since taking directly to a specific tab on the Nominations page isn't as straightforward, I added query parameters to handle this.
Created useQueryParamKey re-usable hook. Made it as statically typed as possible to prevent logic bugs in the future and also to get nice intellisense. Because of the static guarantees, it should also make it really easy to refactor anything related to it, without the fear of breaking things.

Proposed area of change
Put an x in the boxes that apply.

[ ] apps/bridge-dapp
[ ] apps/hubble-stats
[ ] apps/stats-dapp
[x] apps/tangle-dapp
[ ] apps/testnet-leaderboard
[ ] apps/faucet
[ ] apps/zk-explorer
[ ] libs/webb-ui-components

Reference issue to close (if applicable)
Specify any issues that can be closed from these changes (e.g. Closes #233).

Closes #2021

Screen Recording
If possible provide a screen recording of proposed change.
https://github.com/webb-tools/webb-dapp/assets/101931215/3282ed8c-bb7d-4820-9486-445664b1639b

Code Checklist
Please be sure to add .stories documentation if any additions are made to libs/webb-ui-components.

[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

I am experiencing two strange bugs, I have done some changes to circumvent them but couldn't figure them out entirely. One of them is exactly what you were experiencing @vutuanlinh2k2, but in my case even with the type import it'd still error! This occurred after just moving type definitions to another file for organization, without 'changing' anything (just refactoring).
The other bug was related to Next.js build; I was getting an error telling me that PagePath was undefined in sideBarProps.ts, while Next.js was pre-rendering the /claim (Claim Airdrop) page. This is quite strange because I had all the appropriate types imported, and no static errors shown by TypeScript; only when building the Next.js app did this occur.


chore(deps): update terraform-module

This PR contains the following updates:

Package
Type
Update
Change

terraform-ibm-modules/resource-group/ibm (source)
module
patch
1.1.1 -> 1.1.4

terraform-ibm-modules/secrets-manager-secret-group/ibm (source)
module
patch
1.1.0 -> 1.1.3

Release Notes

terraform-ibm-modules/terraform-ibm-resource-group (terraform-ibm-modules/resource-group/ibm)
v1.1.4
Compare Source
Bug Fixes

fix variable name in catalog validation json template (#535) (6cadceb)

v1.1.3
Compare Source
Bug Fixes

fix typo in catalog validation json template (#534) (c4eb24f)

v1.1.2
Compare Source
Bug Fixes

cleanup for catalog onboarding (#533) (ec38fcc)

terraform-ibm-modules/terraform-ibm-secrets-manager-secret-group (terraform-ibm-modules/secrets-manager-secret-group/ibm)
v1.1.3
Compare Source
Bug Fixes

correct the usage in documentation (#91) (29410f7)

v1.1.2
Compare Source
Bug Fixes

fix typo in catalo validation variable values (#90) (6a8d366)

v1.1.1
Compare Source
Bug Fixes

onboard module to IBM Cloud module registry (#88) (cfa5676)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

/run pipeline

:tada: This PR is included in version 1.0.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Where is the 1*1 conv?

Hi, there.
Thanks for your code. But seems your implementation is different from the EEGnet paper. In the paper there is a depthwise conv lary which is a 1*1 conv. So I suppose this is not the same as the implementation in the paper?

EEGnet design has changed in 2018. I don't know of further revisions. This
implement is of the first version. At that time, the authors hadn't
released their code. Now they have. Please use the authors' original
implementation linked in my repo itself.
On Sun, Jan 31, 2021, 3:21 PM xiaolongwu0713<EMAIL_ADDRESS>wrote:

Hi, there.
Thanks for your code. But seems your implementation is different from the
EEGnet paper. In the paper there is a depthwise conv lary which is a 1*1
conv. So I suppose this is not the same as the implementation in the paper?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/aliasvishnu/EEGNet/issues/4, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABFBDDLJ4IVXME2ICDG5XPTS4XQWLANCNFSM4W32GH7A
.


shortcuts.vdf has to be present before using GlosSIConfig.exe

Expected Behavior
GloSIConfig.exe should create shortcuts.vdf if it is not yet present.
Actual Behavior
Nothing happens and in logs GlosSI says that a file named shortcuts.vdf was not found.
Steps to Reproduce the Problem

Delete (steam location)/userdata/(userid)/config/shortcuts.vdf
Try configuring a game using GlosSIConfig.exe

Specifications

Version: GlosSI <IP_ADDRESS>
Platform: Windows 10

Please try the latest snapshot build and report back
https://ci.appveyor.com/project/Alia5/glossi/build/artifacts


Set minimum width

We know that Frontend doesn't work particularly well on mobile (#271). We've decided not to prioritize the mobile experience, but we could prevent unexpected behavior on mobile by setting a minimum width for Frontend. That would prevent overlapping elements or content overflow, for example. By setting a minimum width, the user would need to scroll horizontally on mobile to see content on the righthand side of the page.
@issa-tseng designs for a width of 1000px, so we could use that for the minimum width.
@lognaturel says that even today on mobile, things don't generally break. Typically the user is looking for something on the lefthand side of the page. However, we think that setting a minimum width would be an improvement.
After we set a minimum width, we will have bounds on the width on both ends: getodk/central-frontend#447 was about setting a maximum width. (The minimum width will likely be set on a different element from the maximum width, but this is just to say that the width of most Frontend content will end up becoming fully bounded.)

one thing to think about here is to also set the zoom to a sane number if the mobile presentation is lacking


Gamedata update for CS2 v<IP_ADDRESS>

In latest CS2 update, GiveNamedItem Linux signature has changed. While CS# doesn't update it, you can do it by updating counterstrikesharp/gamedata/gamedata.json:
  "GiveNamedItem": {
    "signatures": {
      "library": "server",
      "windows": "\\x48\\x89\\x5C\\x24\\x2A\\x48\\x89\\x74\\x24\\x2A\\x55\\x57\\x41\\x2A\\x41\\x2A\\x41\\x2A\\x48\\x2A\\x2A\\x2A\\x2A\\x48\\x2A\\x2A\\x2A\\x2A\\x2A\\x2A\\x4D\\x2A\\x2A\\x48",
      "linux": "\\x55\\x48\\x89\\xE5\\x41\\x57\\x41\\x56\\x49\\x89\\xD6\\x41\\x55\\x49\\x89\\xFD\\x41\\x54\\x48"
    }
  },

I am running the following:
Ubuntu 22.04.4
mmsource-2.0.0-git1289-linux
counterstrikesharp-with-runtime-build-215-linux-7cae4be
InventorySimulator-v1.0.0-beta.22
The server boots, throwing a "Failed to load plugin error"
When running css_plugins list, it shows InventorySimulator as LOADING.
I have a separate plugin, CS2Rcon, which shows as LOADED.
The gloves I set are working, while weapon skins and knife skins are not.
Here is the excerpt from the log I believe is relevant:
02:22:55 [INFO] (cssharp:PluginContext) Loading plugin InventorySimulator
02:22:55 [EROR] (cssharp:PluginManager) Failed to load plugin from /home/ubuntu/cs2/game/csgo/addons/counterstrikesharp/plugins/InventorySimulator/InventorySimulator.dll
CounterStrikeSharp.API.Core.NativeException: Invalid function pointer
   at CounterStrikeSharp.API.Core.ScriptContext.CheckErrors() in /home/runner/work/CounterStrikeSharp/CounterStrikeSharp/managed/CounterStrikeSharp.API/Core/ScriptContext.cs:line 176
   at CounterStrikeSharp.API.Core.NativeAPI.HookFunction(IntPtr function, InputArgument hook, Boolean post) in /home/runner/work/CounterStrikeSharp/CounterStrikeSharp/managed/CounterStrikeSharp.API/Core/API.cs:line 1033
   at CounterStrikeSharp.API.Modules.Memory.DynamicFunctions.BaseMemoryFunction.Hook(Func`2 handler, HookMode mode) in /home/runner/work/CounterStrikeSharp/CounterStrikeSharp/managed/CounterStrikeSharp.API/Modules/Memory/DynamicFunctions/BaseMemoryFunction.cs:line 62
   at InventorySimulator.InventorySimulator.Load(Boolean hotReload) in C:\Users\ianlu\Projects\cs2-inventory-simulator-plugin\source\InventorySimulator\InventorySimulator.cs:line 50
   at CounterStrikeSharp.API.Core.Plugin.PluginContext.Load(Boolean hotReload) in /home/runner/work/CounterStrikeSharp/CounterStrikeSharp/managed/CounterStrikeSharp.API/Core/Plugin/PluginContext.cs:line 218
   at CounterStrikeSharp.API.Core.Plugin.Host.PluginManager.LoadPlugin(String path) in /home/runner/work/CounterStrikeSharp/CounterStrikeSharp/managed/CounterStrikeSharp.API/Core/Plugin/Host/PluginManager.cs:line 125
   at CounterStrikeSharp.API.Core.Plugin.Host.PluginManager.Load() in /home/runner/work/CounterStrikeSharp/CounterStrikeSharp/managed/CounterStrikeSharp.API/Core/Plugin/Host/PluginManager.cs:line 93```

I'd recommend you to reinstall everything. The error points to line 50 which is the hook for giving a named item to a player - so it appears to be an issue with CS#. I've personally tested in latest Windows and Linux, so I'm pretty sure it's some misconfiguration on your end.
Make sure gamedata is correct and FollowCS2ServerGuidelines is false.

Btw looks like CS# had a new update specifically for this issue. You may try that as well.

Okay looks like it's broken. I wasn't aware Valve released another update. I'll look into it.

Currently CounterStrikeSharp's GiveNamedItem is outdated. Until it is updated, update your server gamedata.

Worked for me 👍
Thanks

Resolved by https://github.com/roflmuffin/CounterStrikeSharp/releases/tag/v218


Fix Kraken GNO parsing

Fixes #1452.
The existing code seems full of similar hacked corner cases, I'm sorry for one more.

Yeah, Kraken causes some ugly hacks for sure.

@HairyFotr @timmolter I agree that the Kraken code base is getting uglier by all those hacks. Perhaps a redesign should be done. I expect that the asset pair list from Kraken is a start point. In this list the alt name seems always clean from the X or Z adds.


Subcommand 'profile' asking for cutoffs arguments

I tried running cayman v0.10.0 with cayman profile, but it raises this error:
AttributeError: 'Namespace' object has no attribute 'cutoffs'
Looking at the __main__.py, it looks like this is due to the if-clause in line 27-28.
Is the cutoff parameter also essential for the profile subcommand or is it a broken if-clause specific to annotate_proteome?
Another question, it looks like the default db_format was changed from bed format to HMMER format (handle_args.py line 80-86). Was it changed to make it accept the Zenodo annotations by default?

@cschu Thanks for the quick reply.
Here is the command line call I used that resulted in the error.
cayman profile -1 "$ID"_R1.fastq.gz -2 "$ID"_R2.fastq.gz \
--cpus_for_alignment 8 \
--out_prefix "$outdir"/"$ID"_ \
"$dbdir"/annots/GMGC10.human-gut.95nr.no-rare.0.5.percent.prevalence_all_v3_FINAL.csv \
"$dbdir"/bwa_index/human

The BWA index was prepared from GMGC10.human-gut.95nr.0.5.percent.prevalence.fna.gz, with this parameter:
bwa index -p cayman_human -b<PHONE_NUMBER>0 "$refpath".
I'm running it within a Singularity container, built with the Remote Builder based on the latest definition file.
P.S. Thanks for the detailed explanation on the format change. As a user, I was somewhat confused as the Zenodo annotations was not in the 4-column format described in the documentation.

Hi @metaganal ,
Thanks for reporting this. That was an oversight when adding the annotate_proteome mode. Indeed, the cutoffs parameter is not required for the profile mode and the check was misplaced.
I fixed this and there is now a version 0.10.1.
Please let me know if this is working on your side.
Best,
Christian

@cschu Thanks for the quick fix. It ran smoothly after updating to v0.10.1.

Perfect, glad to hear that.
Best,
Christian


Incorrect script content saved

I don't have a good set of repro steps, but when managing functions (as I'm editing, saving and switching between them), I frequently run into issues where the file contents are lost (the function file is empty) or the contents from another function are persisted.

I found a bug where clicking save on a file that never had a change to it ends up saving an empty file. It's fixed by 717d90108e0b5c0566d7f09a153f7f87ff58943e


Implement SampleGrad

This builds on #25, so that should be merged before this one.
This implements SampleGrad for Texture2DResource and Texture2DArrayResource.

I've never used this function so I'm not sure what it does 😅 , but it looks correct to me. I'll wait for the other PR to complete.

I've never used this function so I'm not sure what it does

It lets you provide manual values for ddx and ddy (used for calculating mipmap levels, and with the usual Sample method, calculated internally based on adjacent fragments in a 2x2 fragment quad).
I'm using it because my texture sampling call is divergent (not actually inside a conditional, but the texture array index comes from a structured buffer lookup that can vary pixel-to-pixel). In that case, the default derivative calculation doesn't work so I need to provide my own, based on some values that are consistent pixel-to-pixel.

@tgjones I merged this into master manually. This should finish up all the current PR's. Thanks for all the new features!


onehook CLI improvements

Description
Following easy improvements might be considered for onehook CLI tool:

command doesn't expand to the terminal width as other commands
hook delete fails when provided with hook name

$ onehook delete refresh_dns_create-0
/usr/lib/one/ruby/cli/one_helper.rb:859:in `perform_actions': undefined method `each' for "refresh_dns_create-0":String (NoMethodError)
	from /usr/bin/onehook:175:in `block (2 levels) in <main>'
	from /usr/lib/one/ruby/cli/command_parser.rb:470:in `call'
	from /usr/lib/one/ruby/cli/command_parser.rb:470:in `run'
	from /usr/lib/one/ruby/cli/command_parser.rb:84:in `initialize'
	from /usr/bin/onehook:40:in `new'
	from /usr/bin/onehook:40:in `<main>'

Progress Status

[ ] Branch created
[ ] Code committed to development branch
[ ] Testing - QA
[ ] Documentation
[ ] Release notes - resolved issues, compatibility, known issues
[ ] Code committed to upstream release/hotfix branches
[ ] Documentation committed to upstream release/hotfix branches

PRs to merge in 5.10 and master:

code: https://github.com/OpenNebula/one/pull/4206
docs: https://github.com/OpenNebula/docs/pull/947

Note: change in config is not for one-5.10

ok will split in two so we can have the functionality 5.10.x

@rsmontero change done! @vholer thanks for the heads up!!


VS Code MacOS Universal Icon doesn't show up

From Twitter: https://twitter.com/pjausovec/status/1379877534733529089
Visual Code icon is broken. MacOS version 11.2.3 and VS Code Version: 1.55.0 (Universal)

Duplicate of https://github.com/microsoft/vscode/issues/120305


feat(cli): add default value for invoice description

This PR is to resolve #1863.
The default value of "" is the same as that used by lncli. Please let me know if a different value should be used (or any other comments). Thanks.

Codecov Report
Patch coverage has no change and project coverage change: +0.01 :tada:

Comparison is base (5cf843d) 62.62% compared to head (f119f2c) 62.63%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1996      +/-   ##
==========================================
+ Coverage   62.62%   62.63%   +0.01%
==========================================
  Files         141      141
  Lines       28274    28274
==========================================
+ Hits        17706    17709       +3
+ Misses      10568    10565       -3

Impacted Files
Coverage Δ

client/cli/src/main.rs
0.22% <ø> (ø)

... and 8 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Remove undefined variable from CI

composer-version was not defined.
You can set Composer version in shivammathur/setup-php action.

💡 This is a joke PR.
Would you welcome a complete CI?

Ah that's a holdover from another project. Thank you!

Would you welcome a complete CI?
I don't think so at this point. I've got tests and Pint running styles, so I think that's enough for now. Thanks though!

I don't think so at this point.

At a later point you can copy all my knowledge updated here https://github.com/szepeviktor/debian-server-tools/blob/master/webserver/Continuous-integration-Continuous-delivery.yml
🥑


darwin-nix support

Just now we only support darwin via home-manager.
@Kloenk has started a work-in-progress port that could be picked up: https://github.com/Kloenk/sops-nix/commit/c0864759e5d3f5899195fd9debaeda48e08d47d8
At this time it's unclear if @Kloenk has time to finalize this.

I personally have no plans to pick this task up, but I will merge other people's attempts.

looks like https://github.com/Mic92/sops-nix/pull/647 has resolved this, modulo the README entry mentioned in the thread.

I really let documentation slip this time

Should this be closed then?

Should this be closed then?

Only with documentation being merged.


 I find it annoying that if I restarted the process and the count went back to 0

The tricky part will be to persist the data reliably even if the program doesn't exit nominally. What if the user closes the terminal window that runs chiadog? It seems to me that it's necessary to persist the data continuously and not only before shutdown in order for it to work reliably.
If you find a good way to achieve it, that doesn't add some heavy dependencies and unnecessary complexity, I'd say why not?
But I see it as a rather nice-to-have, low priority feature.


Dispatch API/Service do not use message queue

The dispatch client and API are sending messages without a queue. A queue should be used to ensure that only one simulation is run for each submission. The service will run multiple simulations if it is scaled to more than 1 replica.

I will do it alongside #1479


[Idea]: add holograms that says how much players there are in a specific world

Describe the feature
I hope that MultiVerse can implement the following idea :
a new comman (/mv holo ) that create an hologram that show how many players are in the specified world (for example, "23 players in this world" or somethings like it that can be configurable in config.yml.).
Thank you for read it and think about it.
How is the feature useful to all Multiverse users?
Players and admin can see very quikly how much players there are in the world, that is useful when there are a max players per world, and for the servers that have minigames in a specific world. Like that users can see how many players are playing at this game.
Agreements

[X] I have searched for and ensured there isn't already an open issue regarding this.

Can be solved with MV Placeholders, PlaceholderAPI and a holograms plugin (I like DecentHolograms myself)


Issue: cannot handle input parameters for GP service

There seems to be a collision of names due to a wrong import of BaseGPObject from abstract.py.
File arcrest\ags\_geoprocessing.py:
from .._abstract.abstract import BaseAGSServer, BaseGPObject
This object is used https://github.com/Esri/ArcREST/blob/master/src/arcrest/_abstract/abstract.py#L29 and it doesn't have any attributes, so the code never gets into evaluating the input params type: https://github.com/Esri/ArcREST/blob/master/src/arcrest/ags/_geoprocessing.py#L236.
The solution that worked for me: comment out the BaseGPObject in the import statement.
from .._abstract.abstract import BaseAGSServer #, BaseGPObject
Then all the methods and attributes can be used and executeTask and submitJob work normally.
Am I missing something here or it can be fixed just by deleting the BaseGPObject from the import line?

I moved the BaseGPObject out of the gpobject module and put it in abstract where it belonged.
Thank you

Cloned the update repo, tested, works as expected, thanks!


Clarification: why do binned metrics outnumber bins?

Hi there - this is a very basic question, but couldn't find it in the docs.
For BinnedPrecisionRecallCurve, if you ask for 5 thresholds, you get 6 values of precision and recall. I'm just wondering what this means. To copy and paste the docs:
>>> from torchmetrics import BinnedPrecisionRecallCurve
>>> pred = torch.tensor([0, 0.1, 0.8, 0.4])
>>> target = torch.tensor([0, 1, 1, 0])
>>> pr_curve = BinnedPrecisionRecallCurve(num_classes=1, thresholds=5)
>>> precision, recall, thresholds = pr_curve(pred, target)
>>> precision
tensor([0.5000, 0.5000, 1.0000, 1.0000, 1.0000, 1.0000])
>>> recall
tensor([1.0000, 0.5000, 0.5000, 0.5000, 0.0000, 0.0000])
>>> thresholds
tensor([0.0000, 0.2500, 0.5000, 0.7500, 1.0000])

I would expect that there would be num_thresholds - 1 values for precision and recall, so that, for instance, the first values could sit within the bin 0.0 - 0.25. If num_thresholds and the number of precision / recall values were equal, I could assume that was just calculated for this specific threshold. But I can't fathom out why it's num_thresholds + 1.
Apologies if this is obvious, but would you mind clarifying this?
Kind regards,
Jacan.

Or am I looking at this backwards? Are the bin edges actually the precision and recall, and the threshold is being binned by them?

Many thanks for this - might be worth including in the docs for people similarly confused! :)
Jacan.


Improve cli parameter parsing

Resolves L2B-3069

Closing this PR and putting this issue on hold until further discussions regarding separating CLI and discovery lib.


Building yields existing errors

Curious if this codebase has been built recently from a fresh repo clone? Performing a "yarn build" results in:
$ yarn clean && yarn ts:check && NODE_OPTIONS='--max-old-space-size=8192' tsup && cp -r src/adapters/ibm-vllm/proto dist/adapters/ibm-vllm
$ rimraf dist
$ tsc --noEmit && tsc -p tsconfig.examples.json --noEmit
src/tools/base.test.ts:353:13 - error TS2454: Variable 'description' is used before being assigned.
353             description,
~~~~~~~~~~~
src/tools/base.test.ts:368:13 - error TS2454: Variable 'inputSchema' is used before being assigned.
368             inputSchema,
~~~~~~~~~~~
src/tools/base.test.ts:382:13 - error TS2454: Variable 'name' is used before being assigned.
382             name,
~~~~
src/tools/database/elasticsearch.ts:39:8 - error TS2307: Cannot find module<EMAIL_ADDRESS>or its corresponding type declarations.
39 } from<EMAIL_ADDRESS>~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
tsup.config.ts:2:25 - error TS1543: Importing a JSON file into an ECMAScript module requires a 'type: "json"' import attribute when 'module' is set to 'NodeNext'.
2 import packageJson from "./package.json";
~~~~~~~~~~~~~~~~
tsup.config.ts:6:22 - error TS1543: Importing a JSON file into an ECMAScript module requires a 'type: "json"' import attribute when 'module' is set to 'NodeNext'.
6 import tsConfig from "./tsconfig.json";
~~~~~~~~~~~~~~~~~
vitest.config.ts:3:25 - error TS1543: Importing a JSON file into an ECMAScript module requires a 'type: "json"' import attribute when 'module' is set to 'NodeNext'.
3 import packageJson from "./package.json";
~~~~~~~~~~~~~~~~
Found 7 errors in 4 files.
Errors  Files
3  src/tools/base.test.ts:353
1  src/tools/database/elasticsearch.ts:39
2  tsup.config.ts:2
1  vitest.config.ts:3
error Command failed with exit code 2.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
error Command failed with exit code 2.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

I was not able to reproduce it. I did the following steps.
git clone https://github.com/i-am-bee/bee-agent-framework
cd bee-agent-framework
nvm use
yarn install
corepack enable
yarn install
yarn build

Same for me,
git clone
nvm use
yarn install
branch + changes
git commit
yarn run v1.22.17
$ tsc --noEmit && tsc -p tsconfig.examples.json --noEmit
src/agents/bee/runners/default/runner.ts:62:15 - error TS2322: Type 'string' is not assignable to type 'never'.

62               regex: parserRegex.source,
                 ~~~~~

src/llms/base.test.ts:196:41 - error TS2322: Type 'string' is not assignable to type 'never'.

196       await generate(["a"], { guided: { regex: /.+/.source } });
                                            ~~~~~

src/llms/base.test.ts:198:41 - error TS2322: Type 'string' is not assignable to type 'never'.

198       await generate(["a"], { guided: { regex: /.+/.source } });
                                            ~~~~~

src/llms/drivers/json.ts:62:13 - error TS2416: Property 'guided' in type 'JsonDriver<TGenerateOptions>' is not assignable to the same property in base type 'BaseDriver<TGenerateOptions>'.
  Type '(schema: SchemaObject) => { readonly json: SchemaObject; }' is not assignable to type '(schema: SchemaObject) => (Without<(Without<(Without<(Without<(Without<{ json?: string | Record<string, any> | undefined; }, { regex?: string | undefined; }> & { ...; }) | (Without<...> & { ...; }), { ...; }> & { ...; }) | (Without<...> & ((Without<...> & { ...; }) | (Without<...> & { ...; }))), { ...; }> & { ...; }...'.
    Type '{ readonly json: SchemaObject; }' is not assignable to type '(Without<(Without<(Without<(Without<(Without<{ json?: string | Record<string, any> | undefined; }, { regex?: string | undefined; }> & { regex?: string | undefined; }) | (Without<...> & { ...; }), { ...; }> & { ...; }) | (Without<...> & ((Without<...> & { ...; }) | (Without<...> & { ...; }))), { ...; }> & { ...; }) |...'.
      Type '{ readonly json: SchemaObject; }' is not assignable to type 'Without<{ whitespace_pattern?: string | undefined; }, (Without<(Without<(Without<(Without<{ json?: string | Record<string, any> | undefined; }, { regex?: string | undefined; }> & { ...; }) | (Without<...> & { ...; }), { ...; }> & { ...; }) | (Without<...> & ((Without<...> & { ...; }) | (Without<...> & { ...; }))), {...'.
        Property 'whitespace_pattern' is missing in type '{ readonly json: SchemaObject; }' but required in type 'Without<{ whitespace_pattern?: string | undefined; }, (Without<(Without<(Without<(Without<{ json?: string | Record<string, any> | undefined; }, { regex?: string | undefined; }> & { ...; }) | (Without<...> & { ...; }), { ...; }> & { ...; }) | (Without<...> & ((Without<...> & { ...; }) | (Without<...> & { ...; }))), {...'.

62   protected guided(schema: SchemaObject) {
               ~~~~~~

src/llms/drivers/typescript.ts:54:13 - error TS2416: Property 'guided' in type 'TypescriptDriver<TGenerateOptions>' is not assignable to the same property in base type 'BaseDriver<TGenerateOptions>'.
  Type '(schema: SchemaObject) => { readonly json: SchemaObject; }' is not assignable to type '(schema: SchemaObject) => (Without<(Without<(Without<(Without<(Without<{ json?: string | Record<string, any> | undefined; }, { regex?: string | undefined; }> & { ...; }) | (Without<...> & { ...; }), { ...; }> & { ...; }) | (Without<...> & ((Without<...> & { ...; }) | (Without<...> & { ...; }))), { ...; }> & { ...; }...'.
    Type '{ readonly json: SchemaObject; }' is not assignable to type '(Without<(Without<(Without<(Without<(Without<{ json?: string | Record<string, any> | undefined; }, { regex?: string | undefined; }> & { regex?: string | undefined; }) | (Without<...> & { ...; }), { ...; }> & { ...; }) | (Without<...> & ((Without<...> & { ...; }) | (Without<...> & { ...; }))), { ...; }> & { ...; }) |...'.
      Type '{ readonly json: SchemaObject; }' is not assignable to type 'Without<{ whitespace_pattern?: string | undefined; }, (Without<(Without<(Without<(Without<{ json?: string | Record<string, any> | undefined; }, { regex?: string | undefined; }> & { ...; }) | (Without<...> & { ...; }), { ...; }> & { ...; }) | (Without<...> & ((Without<...> & { ...; }) | (Without<...> & { ...; }))), {...'.
        Property 'whitespace_pattern' is missing in type '{ readonly json: SchemaObject; }' but required in type 'Without<{ whitespace_pattern?: string | undefined; }, (Without<(Without<(Without<(Without<{ json?: string | Record<string, any> | undefined; }, { regex?: string | undefined; }> & { ...; }) | (Without<...> & { ...; }), { ...; }> & { ...; }) | (Without<...> & ((Without<...> & { ...; }) | (Without<...> & { ...; }))), {...'.

54   protected guided(schema: SchemaObject) {
               ~~~~~~

src/tools/base.test.ts:353:13 - error TS2454: Variable 'description' is used before being assigned.

353             description,
                ~~~~~~~~~~~

src/tools/base.test.ts:368:13 - error TS2454: Variable 'inputSchema' is used before being assigned.

368             inputSchema,
                ~~~~~~~~~~~

src/tools/base.test.ts:382:13 - error TS2454: Variable 'name' is used before being assigned.

382             name,
                ~~~~

src/tools/database/elasticsearch.ts:39:8 - error TS2307: Cannot find module<EMAIL_ADDRESS>or its corresponding type declarations.

39 } from<EMAIL_ADDRESS>          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

tsup.config.ts:2:25 - error TS1543: Importing a JSON file into an ECMAScript module requires a 'type: "json"' import attribute when 'module' is set to 'NodeNext'.

2 import packageJson from "./package.json";
                          ~~~~~~~~~~~~~~~~

tsup.config.ts:6:22 - error TS1543: Importing a JSON file into an ECMAScript module requires a 'type: "json"' import attribute when 'module' is set to 'NodeNext'.

6 import tsConfig from "./tsconfig.json";
                       ~~~~~~~~~~~~~~~~~

vitest.config.ts:3:25 - error TS1543: Importing a JSON file into an ECMAScript module requires a 'type: "json"' import attribute when 'module' is set to 'NodeNext'.

3 import packageJson from "./package.json";
                          ~~~~~~~~~~~~~~~~

Found 12 errors in 8 files.

Errors  Files
     1  src/agents/bee/runners/default/runner.ts:62
     2  src/llms/base.test.ts:196
     1  src/llms/drivers/json.ts:62
     1  src/llms/drivers/typescript.ts:54
     3  src/tools/base.test.ts:353
     1  src/tools/database/elasticsearch.ts:39
     2  tsup.config.ts:2
     1  vitest.config.ts:3
error Command failed with exit code 2.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
husky - pre-commit script failed (code 2)

Got it. One should use yarn install --immutable to prevent updating packages.
I updated the docs and related packages. Both ways should now work.
Please verify and let me know.

yarn install --immutable solves it for me


Ensure the layout of SoftAsciiStr is same as str

Without #[repr(transparent)], the there are no guarantee of internal layouts being same, and from_unchecked{,_mut} will have undefied behavior.
The attribute is necessary for pointer casts (such as &*(s as *const str as *const SoftAsciiStr)) to be sound.

Thanks a lot, when I wrote this code this attribute wasn't yet available and I missed it when they added it. (Basically the code relied on this wrappers being "zero overhead" and as such there can't be a different representation in this case ;-).

@lo48576  I published a new version, which also fixes "deprecations".  I will see if I can find some time to go through all newly stabilized methods and add them in the next weeks. So for now I just added trim_start,trim_end,get_unchecked and deprecated trim_left,trim_right,slice_unchecked like in the standard library.
And I forgot to update the change log before publishing :man_facepalming:, well a updated change log on github will follow in a minute or so.


scheduler: add opinfluence for balance leader scheduler

Signed-off-by: lhy1024<EMAIL_ADDRESS>
What problem does this PR solve?

We miss op influence when sort target store in balance leader scheduler. To fix #2571
What is changed and how it works?
add it
Check List

Tests

Unit test

Release note

No release note

/merge

/merge


Missing eventtype GitHub::Release

The dashboard value_stream_analytics.xml refers to an eventtype GitHub::Release.
But this eventtype is missing from the eventtypes.conf

Thanks for pointing that out. I've updated the eventtypes.conf file and the next release will have this update. In the meantime, you can create your own eventtype and provide this as the search:
github_webhooks action IN ("released","published") release.id=*


Example ObjC app fails to build with simulator v.8.3

 --verbose_failures shows:
clang: warning: no such sysroot directory: '/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator8.1.sdk'
Looking into that folder in my machine, there's an iPhoneSimulator.sdk directory, and a iPhoneSimulator8.3.sdk link. Duplicating the 8.3 link and renaming it 8.1 makes everything work. But is super hackish.

I see from another issue that there's a couple of flags that I guess would make it work: --ios_sdk_version=8.3 --ios_simulator_version=8.3. Bazel should assume them if those are the only versions of the SDKs it can find, instead of failing.

We're working on a plan to automatically select the right SDK and Xcode as installed on your machine but as those plans are still fairly nebulous I won't go into details yet. Expect some updates here in the next few months.

If the fix is coming in months, I think we should have an immediate bandaid in the example app, like just mentioning the flags in its README.

(Also, FWIW, this command outputs the SDK version: xcrun --sdk iphoneos --show-sdk-version)

Thanks Jorge. We really are working on it :)
Cheers,
Dave
On Wed, Jun 24, 2015 at 5:04 AM, Jorge Canizales<EMAIL_ADDRESS>wrote:

(Also, FWIW, this command outputs the SDK version: xcrun --sdk iphoneos
--show-sdk-version)
—
Reply to this email directly or view it on GitHub
https://github.com/google/bazel/issues/253#issuecomment-114844972.

This is done for several month now.


Implement Password Recovery Feature #98

Related Issue
Closes #98
Description
This PR implements a Password Recovery feature, allowing users who have forgotten their password to reset it through a recovery process.
Changes Made
Added a "Forgot Password?" link on the login page that navigates to the password recovery page.
Created a password recovery form where users can enter their email to receive a password reset link.
Screenshots / Video
https://github.com/user-attachments/assets/d65d5cf1-b646-4512-b94f-0ad764a11f4e

@gupta-ritik  please assign labels and level


cmd/go: false positive +build comment must appear before package clause and be followed by a blank line

Sometimes you may want to write a test that contains code with a build tag.
Example: https://go-review.googlesource.com/c/tools/+/125040
But it will fail to build due to the build tag in the test file.
@bcmills @rsc

@andybons: what are the reproduction steps for this?

I don’t understand the issue. Is the problem that vet is complaining about it? Or something else? cc @mvdan just in case it is vet.

Vet complains about misplaced build tags. However, it doesn't use the Go parser to look at actual comments, it does a simple string search on the file. Thus, a string containing what looks like a build tag can trigger a vet warning. Since go test nowadays runs vet, it causes hard failure.

OK, so go vet godoc/*.go with the test in https://go-review.googlesource.com/c/tools/+/125040 shows the issue.  I can try to take a look later today.

Re-titling it as a vet issue.

Unless my memory fails me, I improved the vet check so that it would have zero false positives in Go source files, even in the presence of multiline strings. So this sounds like a bug - perhaps an edge case that I missed.

I am clearly missing something here - this only happens with vet from Go 1.10 and not tip, as one would expect after my vet improvement in https://go-review.googlesource.com/c/go/+/111415:
$ git rev-parse HEAD # right before the revert
faa8a71ab532725a7fd9d901a97f74c5038c409b
$ go version
go version devel +5f5402b723 Tue Jul 24 23:54:08 2018 +0000 linux/amd64
$ go vet
$ go1 version
go version go1.10.3 linux/amd64
$ go1 vet
# golang.org/x/tools/godoc
./godoc_test.go:327: +build comment must appear before package clause and be followed by a blank line
./godoc_test.go:328: +build comment must appear before package clause and be followed by a blank line

So as far as I can tell, this is a duplicate of https://github.com/golang/go/issues/13533 and has been fixed for 1.11 since May. Do we need 1.10's vet to also be fixed for some reason? The change in question is quite large, so it's likely too invasive to backport into 1.10.x.

How are we even testing build tags?

See src/cmd/vet/testdata/buildtag/.

this only happens with vet from Go 1.10 and not tip

That seems correct. Although it is very hard to say which version of Go is being tested against for the tools repo here https://build.golang.org/?repo=golang.org%2Fx%2Ftools which caused the test to fail.

Do we need 1.10's vet to also be fixed for some reason?

Assuming we are testing using 1.10, then we cannot add the test done in CL 125040 because the build dashboard becomes red. I am not sure what is to be done here. Maybe we can put the test behind a build tag or something ? (Funny that we are using a build tag to prevent detection of a build tag)

See src/cmd/vet/testdata/buildtag/.

Thanks. Makes sense now since it is fixed at tip. :)

Although it is very hard to say which version of Go is being tested against for the tools repo here

I see three Go versions per each tools repo commit, presumably Go master, 1.10.x, and 1.9.x. So I guess there's our answer.

I am not sure what is to be done here.

One temporary solution, until all 1.12 is out and 1.10 can be dropped, is to "hide" the build tag in the multiline string, like:
const str = `
`+`// +build foo
etc etc
`

@mvdan: updated the reland with a similar workaround: https://go-review.googlesource.com/c/tools/+/126256

@mvdan - Perhaps this should be closed since this is fixed at tip ?

A workaround was used in the CL above, so yes, I agree that this can be closed. Thanks for the ping!


spin gdb failing with file not found error

I tested with 0.11 and 0.12, and I get this error:
(numpy)
~/Projects/gh/me/numpy on  unique-cpp! ⌚ 16:09:25
$ spin gdb /tmp/1.py
Invoking `build` prior to invoking gdb:
$ /home/adrin/micromamba/envs/numpy/bin/python3 vendored-meson/meson/meson.py compile -C build
INFO: autodetecting backend as ninja
INFO: calculating backend command to run: /home/adrin/micromamba/envs/numpy/bin/ninja -C /home/adrin/Projects/gh/me/numpy/build
ninja: Entering directory `/home/adrin/Projects/gh/me/numpy/build'
[1/1] Generating numpy/generate-version with a custom command
Saving version to numpy/version.py
$ /home/adrin/micromamba/envs/numpy/bin/python3 vendored-meson/meson/meson.py install --only-changed -C build --destdir ../build-install
$ export PYTHONPATH="/home/adrin/Projects/gh/me/numpy/build-install/usr/lib/python3.12/site-packages"
$ gdb -ex 'set detach-on-fork on' --args /home/adrin/micromamba/envs/numpy/bin/python3 /tmp/1.py

Traceback (most recent call last):
  File "<frozen os>", line 580, in execvp
  File "<frozen os>", line 622, in _execvpe
  File "<frozen os>", line 613, in _execvpe
FileNotFoundError: [Errno 2] No such file or directory

An internal error has occurred. Please file a bug report at

  https://github.com/scientific-python/spin

including the above traceback and the following information:

  spin: 0.12, package: numpy

Aborting.
(numpy)
~/Projects/gh/me/numpy on  unique-cpp! ⌚ 16:09:30
$ which python
/home/adrin/micromamba/envs/numpy/bin/python
(numpy)
~/Projects/gh/me/numpy on  unique-cpp! ⌚ 16:11:35
$ ls /tmp/1.py
/tmp/1.py

I'm trying to debug a segfault, and as the log shows, both python and /tmp/1.py exist. So not sure what's the issue here.

scratch that. It was gdb which was missing (new machine). Still, the error message could mention that maybe?

We definitely want to give intuitive errors for all such behavior. Let's leave this open and I'll propose a PR.

Please see https://github.com/scientific-python/spin/pull/237


A few of problems, just after waking up

#2770
The "More" dropdown is always like when clicked (in every page). I think we had similar issue before few? Hm. I don't remember how we fixed it.
The extension "Options" is doing something strange, constantly moving/resizing  - can't show that.. Maybe from few pictures below you can see it has different size. It stops when you, for example, somehow successfully click on some of the inputs and "save" the options with "Enter".

Reinstalled the extension
errors in console (when in an Issue)

 ❌ make-discussion-sidebar-sticky
content.js:1896 TypeError: Cannot read property 'offsetHeight' of null
    at updateStickiness (content.js:3269)
    at make_discussion_sidebar_sticky_init (content.js:3276)
    at features_run (content.js:1891)
    at content.js:1944
    at onAjaxedPagesRaw (content.js:1843)
features_run @ content.js:1896

❌ link-to-file-in-file-history
content.js:1896 TypeError: Cannot read property 'children' of null
    at cleanSection (content.js:5986)
    at clean (content.js:6024)
    at clean_sidebar_init (content.js:6039)
    at features_run (content.js:1891)
    at content.js:1944
    at content.js:1848
    at onAjaxedPagesRaw (content.js:1843)

more errors in Repo pages

What permissions the token need? Mine has gist, notifications, repo, user, workflow and it's new.
The "hide marketplace" wraps the anchor a tag with a div that makes too much space between the "Issues" and "Trending"? You can see it from above screenshot.

        Marketplace

I don't know how but you're using an extremely old version of Refined GitHub. make-discussion-sidebar-sticky hasn't existed since August 2019 because it was renamed https://github.com/sindresorhus/refined-github/commits/master/source/features/sticky-discussion-sidebar.tsx

I don't know either. As I said, I just reinstalled it from the chrome store, and it's <IP_ADDRESS>2

In anyway, there is ton of other stuff in this issue other than the sticky thing.
And I'm continuing:

Disabling more-dropdown, removes the releases tab too (while releases-tab is enabled)
disabling depreoritize marketplace seems to not work, it still hides it.
"repo age" doesn't appear too.

This is due to https://github.com/sindresorhus/refined-github/issues/2796
Sorry about that

Ah yea, thanks. I'll just leave it and try later or tomorrow :)


How to add a button into leftBarButtonContainerView?

Sir, could you tell me how to add a button left of the inputToolbar?
I know I need to add it into leftBarButtonContainerView,but when I try ,it doesn't work:
UIButton *button=[[UIButton alloc]initWithFrame:CGRectMake(0, 0, 40, 40)];
[button setTitle:@"emotion" forState:UIControlStateNormal];
[self.inputToolbar.contentView.leftBarButtonContainerView addSubview:button];

what's the problem?
Thanks!!


NPE: CreateExpression is not thread safe, yet is being shared in multithreaded environment

The following line occasionally throws NPEs:
https://github.com/querydsl/querydsl/blob/master/querydsl-core/src/main/java/com/querydsl/core/types/ConstructorExpression.java#L135
This is caused by the fact that the very same instance is being shared across different threads while in multithreaded environment. Supposedly, one of these threads succeeds on executing line 132 on uninitialized instance, while the other thread bypasses the switch on line 131 and directly goes to line 135. In this scenario transformers field is not yet initialized, and for-loop throws an NPE.
The testing conditions could be recreated here: https://gist.github.com/oxygenecore/60c909b9a3814301e4b2e287b4f872b5. I haven't come up with a better solution to expose that the single instance is being initialized concurrently other than just adding debug logging from IntelliJ Idea (ConstructorExpression:132, breakpoint: evaluate and display message 'hashCode()' on pass): while running tests you see multiple repeating lines with the same hashCode.
PR for this issue: https://github.com/querydsl/querydsl/pull/1959

@timowest do you have any estimations on when this fix will be released?


Fix the Github Pages page because images don't work for some reason

Fixed itself? Works now on browser


Android video is saved upside down

Hi, the video when recorded on android device is saved upside down.
Is there a work around for this issue?

I believe this is fixed in release 1.4.6 (and upwards), built from my fork.

It looks like there is only until 1.4.4 in nuget. Do I have to download another nuget? I have fixed it by adding the code directly and it is similar to yours with 270 degree rotation for front camera.

It looks like there is only until 1.4.4 in nuget. Do I have to download another nuget?

I posted the link to the nuget above: https://www.nuget.org/packages/CameraMaui/1.4.6
The name is "CameraMaui".

ah thanks for that. I did not pay attention on the name that minutely and thought it was the same


JWT Decoder

JWT Decoder
Make a JWT Decoder using react
Best of Luck and Happy Coding.

I want to make this project. Please assign me this project

done @ritesh2004 do this project Happy Coding


migrated to VS 2017.

Hi @diegobrum,
I just migrated the SolrExpress to VS 2017.

Hi @UncleZen
Tks a lot!

Task "Migrate projects from xproj to csproj" from #209


Apple Pay instant error when pressing custom button

Describe the bug
When we render the dropin with showPayButton put to false the apple pay directly calls our onSubmit when pressed, this casuses the payment the fail directly since no apple pay token exists in the payload.
To Reproduce
Steps to reproduce the behavior:

Disable showPayButton
Render custom button that calls dropinRef.current.submit()
Enable apple pay
Press the custom button
Error

Expected behavior
We expect the onSubmit not to be called when the payment is still in progress, like google pay.
Smartphone (please complete the following information):

Device: IPhone8
OS: IOS14.4
Browser: Safari
Version: 14.4

When adding the following to the onSubmit we got it working:
if (adyenState?.data?.paymentMethod?.type === 'applepay' && !adyenState?.data?.paymentMethod?.applePayToken) {
   console.debug('Apple pay payment does not have token! Cancel onSubmit')
   return
}

This does feel hacky though.

Hi @TriPSs,
Thanks for reaching out. We've pushed a change in #1085 that should fix this issue. It'll be released in the next minor version of the library. Hope this helps!


ace15: Add communication widget and used memory block reporting to PMC

This PR will add Intel DSP Communication Widget driver for a device for generic sideband message transmit/receive between IPs in a SOC.
To MTL mm driver I add reporting of the number of HPSRAM memory blocks in use. This value is reported to the
PMC via the sideband channel supported by the communication widget.

Can we not use on of the existing IPC mechanism in Zephyr? OpenAMP, mbox, ipc, etc?

It's a piece of SoC hardware. I suspect that it's based on https://patents.google.com/patent/US9213666B2/en .

I wonder if I should move this driver to soc/xtensa/intel_adsp/ace as it is platform specific. What do you think? @lgirdwood  @nashif @andyross @marc-hb @galak

I wonder if I should move this driver to soc/xtensa/intel_adsp/ace as it is platform specific. What do you think? @lgirdwood @nashif @andyross @marc-hb @galak

I think its fine to move under soc/ and just make it some library code that utilizes information from devicetree.  don't see any need for this to be a full device/driver.  It will simplify intel_comm_widget_pmc_send_ipc() to not require a struct device.

This is a real device that can be used in many places for communication. At the moment it is only used in one place, so I gave up with handling interrupts, queueing and synchronization. We can do it in the future, when it becomes necessary to expand it, we will turn it into a real driver.

Some CI checks are still red.

@galak
All internal functions used by driver was named cw_*. Only the one function (adsp_comm_widget_pmc_send_ipc) was named differently. It is exported outside the driver and used by another driver. I prefer keep this name, because cw is meaningless in tlb driver.

@galak @aborisovich @edersondisouza @abonislawski Please review

@galak Please review

@softwarecki please rebase, there is a conflict.


One-line require/export oddity

We have a trampoline module that does nothing but make a native module a bit easier for callers, debugging, etc.
module.exports = require './build/Release/authentication'

There is seemingly no way to convince coffeescope that 'module' and 'require' aren't undefined identifiers.  This is in a large codebase of about 900 other files, all of which it seems quite content with given our 'environments' property.  Not sure why it's got its grump on about just this one.

Could it be folder-structure related? Maybe coffeelint isn't picking up your package.json / coffeelint.json file relevant to that particular file?
Can you reproduce this in a controlled environment? I'd love to help out, but I'm going to need a bit more than that to work with.

Aha, yes, your suggestion was spot on. There's a package.json in that particular directory relevant to building the native module, and it lacked coffeelint configuration directives, which we put in our package.json.  Adding the relevant ones there resolved the issue.  Thanks!


fix: util/check-format-commit.sh to handle one-line diff hunks

For multi-line hunks, 'git diff -U0' outputs a pair of START,COUNT
indicators to show where the hunk starts and ends.  However, if the hunk is
just one line, only START is output, with the COUNT of 1 being implied.
Typically, this happens for copyright change hunks, like this:
--- a/crypto/evp/evp_err.c
+++ b/crypto/evp/evp_err.c
@@ -3 +3 @@
- * Copyright 1995-2023 The OpenSSL Project Authors. All Rights Reserved.
+ * Copyright 1995-2024 The OpenSSL Project Authors. All Rights Reserved.

This is normal unified diff output, and our script must adapt.

There is this commit in @nhorman's PR #24861 (which adds OPENSSL_strtoul() - is that good enough? It has been approved, and is waiting for the timer

There is this commit in @nhorman's PR #24861 (which adds OPENSSL_strtoul()) - is that good enough? It has been approved, and is waiting for the timer

Nope, 'cause it's incorrect.  When no count is given, 1 is implied, not 0.

This pull request is ready to merge

Merged
7821b7b9774d481ae92610e2d132ea34d4aaf407 fix: util/check-format-commit.sh to handle one-line diff hunks


Fix typos in Operation Semantics docs

Fix MathJax beta display.
Update '( assuming' to '(assuming'.
Update 'in a the first' to 'in the first'.
Change 'nop' to 'no-op' to match other occurrences.
Misc other minor updates (periods, hyphen, etc).

Thank you for reviewing, @MarkDaoust


TSL2561 saturation fix

Ben
We (the WCS-MS2TC-1 team and I) ran into another bug this Tuesday while performing a preliminary test of the initial high-altitude balloon sketch I submitted on their behalf (by the way, did you receive it successfully?).  When we programmed the sketch on their SpaceKit and took it outside, we noticed the luminosity readings were lower than they were when we were still indoors; it would consistently read 53.0 outside until you placed an object in front of the sensor, at which time the readings jumped up.  Not sure what was happening, I told them I would look into the problem and have a fix by our meeting next Tuesday.  I also challenged them to try and figure out what was wrong and fix it (not very likely, but you never know...).
I read through the code as well as the TSL2561 datasheet, which led me to suspect the sensor was saturating, and thus causing the bad readings.  To test this, I put together a sketch which replicates the tsl2561_init() and tsl2561_getLux() functions from utility/drivers.cpp so that I could look at the individual photodiode readings as well as the computed lux value.  Sure enough, when exposed to a strong light source (the sun, or the torch LED from a smartphone held close to the sensor), the signal conditioned output of both photodiodes become saturated, causing garbage lux values.  When looking at the TSL2561 library, I noticed the gain was set to 16 by default.  Figuring this would not work well under strong light sources, I used the TSL2561::setGain() function to explicitly set it to 1, and now everything looked reasonable.
Given either the high-altitude balloon or the satellite will be outside and therefore potentially be exposed to the sun, setting the gain to 1 is an appropriate first step to correcting this problem.  I could, however, foresee experiments performed on the satellite where it will be desired to measure weak sources of light.  To handle this will require tsl2561_getLux() to automatically set the gain and/or integration time to maximize resolution without causing the photodiodes to saturate.  Moreover, it is possible, even with the gain turned all the way down, for the sensors to saturate; if so, tsl2561_getLux() will need to handle this as well by indicating this to the user somehow (either via status code or a pre-canned "high" lux value).
While this changeset fixes the immediate problem, I am not sure it is a complete solution.  I think more work needs to be done to insure this sensor interface works under all potential conditions.  Let me know what you think.
Michael Kirkhart

Ben
Since there were no clouds today, I ran a quick test on the changes where I took my SpaceKit outside and exposed the TSL2561 to near-direct sunlight, and as I suspected, the photodiodes saturated, yielding a meaningless reading.  As a result, these changes are not sufficient, and I would not recommened you use them as is.
I did a bit of Internet searching, and found Adafruit has a revised version of the TSL2561 library; this is part of their new "unified" sensor library:
https://github.com/adafruit/Adafruit_TSL2561
This library does handle saturation detection and has AGC functionality as well.  If it detects saturation, it returns 0, which I thought was a bit strange; my preference would be to either indicate saturation via a status value or return a pre-canned "high" value.  It does, however, appear to be an improvement.  The changes can be merged over the old TSL2561 library (the one used by the ArdusatSDK) without a terrible amount of pain, and would preserve the high-level interface functions provided by the ArdusatSDK.  You might want to take a look at this library.
I will leave this pull request open so you can review these comments.  Let me know what you think.
Michael Kirkhart

@mkirkhart - Thanks for doing the testing! I have wanted to add in to the SDK the ability to configure sensor-specific options for a while now, but haven't gotten around to it - this is another use case where it would likely help.
I think that changing over to the auto-adjust TSL2561 drive is probably the right move, so I'm not going to merge this PR right now. I'm away for a little bit starting tomorrow, so it may be a few weeks before I can pull in the Adafruit driver, but we'll make sure to have an auto-adjust driver to fly with at the end of the month.

Ben
I am going to close this pull request because I have a new one.  Stay tuned...


Local completion not working for asdf (as a newbie)

Hey fellows!
I'm new in this zsh plugin manager thing so it can be just a misunderstanding from my part, but I'm just trying to source it and add completions to fpath as documented here:  https://asdf-vm.com/guide/getting-started.html#:~:text=Add the following to ~/.zshrc,see ohmyzsh/ohmyzsh%238837.
So I'm using it in my zsh_plugins.txt:
/home/MY_USER/.asdf kind:zsh
/home/MY_USER/.asdf path:completions kind:fpath
mattmc3/zephyr path:plugins/completion

asdf is sourced for sure because I can use it, but completion is simply not working and I can't figure out why... what am I doing wrong?
For now I'm avoiding envsubst just for sanity reasons, so consider MY_USER as my username...

So... I have moved mattmc3/zephyr path:plugins/completion to the end of the file and now it works. I would like to know why if someone can explain to me.
My .zsh_plugins.txt:
### Sourcing
/home/MY_USER/.asdf kind:zsh

### Completion
/home/MY_USER/.asdf path:completions kind:fpath
#mattmc3/zephyr path:plugins/completion <<<< it was here

### Prompt
sindresorhus/pure

### Framework
belak/zsh-utils path:history
belak/zsh-utils path:utility
belak/zsh-utils path:editor

### Final Plugins
mattmc3/zephyr path:plugins/completion <<<< now I moved it to here
zsh-users/zsh-autosuggestions
zsh-users/zsh-history-substring-search

I'm doing everything based on example file  by the way.

There's a section in the docs that goes over this: https://getantidote.github.io/completions
Essentially, whenever compinit is run, whatever completions paths your fpath variable contains is locked-in. If you add completions later, you've already run compinit and they don't get picked up. Depending on which completions plugin you use, you will likely need to load it towards the end of your config.
These popular ones do it that way:

sorin-ionescu/prezto path:modules/completion
belak/zsh-utils path:completion


JSCS package directory is not found error

this is the bundled jscs plugin, developed by jetbrains, please contact them.


Worker not authorized.

I followed this http://zcash.nicehash.com/getting_started
to start mining zcash
but it ends on "Worker not authorized"
do I need to register somewhere? The tutorial only says that it needs zcash address..

Duplicate of #33

An example
nheqminer_zcash -l eu -u t1NVMexBXCuCz8QqVwqDkMokdHEARDtXiyN -t 4

kamikazechaser: That is what I tried. Two problems:

must be without underscore: nheqminer zcash because nheqminer_zcash doesnt existst (After following the build instructions for linux from the tutorial)
must be btc address, with zcash address it says "worker not authorized"

How would that work, using a BTC address instead? Do they convert it?

Supposedly yes, they somehow convert it. You can check your balance at https://www.nicehash.com/?p=miners&addr=xxx
(replace xxx with your btc address)

How many sol/s are you getting? also specify your hardware


option to not perform git checkout on SIRF and STIR folders

@paskino Is this what you meant? Seems to work for me.
If folder exists, give the user the option to not perform git checkout. If true set GIT_TAG and GIT_URL to blank.

@paskino, @evgueni-ovtchinnikov have you managed to give this a try yet? works very well on my computer so I'm tempted to merge. Any objections @KrisThielemans?

ok, used a macro to avoid duplication.

done


Set expiration date of cookie

What problem does this feature solve?
Cookies set by the onLogin helper do not expire.
js-cookie supports setting the number of days before expiry like this: Cookies.set('name', 'value', { expires: 7 });.
What does the proposed changes look like?
A tokenExpires or cookieExpires option in clientConfigs, which is passed to js-cookie.

This feature request is available on Nuxt community (#c127)

@timhanlon sounds like a good idea to me. I won't have time the following two weeks to work on any features but happy to receive PR's

I exact what I need too :) PR https://github.com/nuxt-community/apollo-module/pull/143

feature is merged


(Android) Fix NullPointerException error on some Android phones

Platforms affected
Android
Motivation and Context
Fix NullPointerException error on some Android phones.
Description
In some Android phones, mimeType is returned as null, this is why a NullPointerException is thrown. This defect is reported by issue #400
Solution
Add a safe null checking before equalsIgnoreCase.
Checklist

[x] I've run the tests to see all new and existing tests pass
[x] Commit is prefixed with (platform) if this change only applies to one platform (e.g. (android))

When can we expect a new build of this plugin with this fix in it? The "latest" build is over a year old.

In the next few weeks or months.

@robwatt there's a fresh release!

Just a tag for now, release is in vote but will follow soon-ish. Watch for the rel/x.y.z tag that is made after it has been published to npm.


[Simplified Collect][Taxes] Create WorkspaceTaxesSettingsPage and subpages

Details

This PR covers changes from 4 attached issues and moreover changes in TaxPicker component. While applying this changes I also replace deprecated OptionsSelector usage in this component with SelectionList.
Fixed Issues

$ https://github.com/Expensify/App/issues/37788
$ https://github.com/Expensify/App/issues/37789
$ https://github.com/Expensify/App/issues/37790
$ https://github.com/Expensify/App/issues/37791
PROPOSAL:
Tests

Login to the app
Make sure you have collect workspace added(possible only in Old Dot)
Go to selected Collect Workspace
Turn Taxes feature if it is not in More Features
Go to Taxes page
Click Settings in the top parto of the page
Try to edit each field and test if all works properly

[x] Verify that no errors appear in the JS console

Offline tests

QA Steps

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

https://github.com/Expensify/App/assets/153545991/da415fd6-6a16-4cc4-a053-32d3481f0696

Android: mWeb Chrome

https://github.com/Expensify/App/assets/153545991/6de4dca6-188d-43c8-a921-41b4e3125397

iOS: Native

https://github.com/Expensify/App/assets/153545991/be5908cd-a337-49c0-a555-ac6983a0018f

iOS: mWeb Safari

https://github.com/Expensify/App/assets/153545991/cf8de12d-536a-4b2e-aedc-369958a7b0b1

MacOS: Chrome / Safari

https://github.com/Expensify/App/assets/153545991/7341c3aa-1353-456d-886e-03b09e03af93

MacOS: Desktop

https://github.com/Expensify/App/assets/153545991/5f5253ab-69ff-4cdd-91dd-af32ba330978

we handle 4 issues in this PR 😄

we handle 4 issues in this PR 😄

The issues are closely related, we can call it one issue: add tax settings :D

@kosmydel nvm ::>> A busy weekend 😄

I will complete my review before Monday

@filip-solecki  I have reviewed the PR. Please help to resolve comments and merge the latest main. Then I can complete the PR checklist and make a final test on the platforms
Note that: I am available from 8 am to 11:59 pm (GMT +7). Let's collab to merge this PR asap

@DylanDylann all comments addressed and bug fixed!

@luacmartins Pattern B added

Checking again

@filip-solecki  Some comments. Everything else is fine

@DylanDylann take a look again pls!

When there is no Tax rate, I dont see anything in the default selector. I think we should add some fun empty state

When there is no Tax rate, I dont see anything in the default selector. I think we should add some fun empty state
We should always have 2 default tax states so there should be no empty state for taxes. We're working on that here

Should they be loaded when the taxes page is open then?

@filip-solecki please fix conflict

Should they be loaded when the taxes page is open then?

Yea, that issue that I linked is working on optimistically creating the taxes and we'll also load it in OpenPolicyTaxesPage

Perfect, thanks for confirming!

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

Tests were passing

🚀 Deployed to staging by https://github.com/luacmartins in version: 1.4.54-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/AndrewGable in version: 1.4.54-4 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


New icons for Sensitivity

Icons look good. High five.

Wonder if we could update + combine the 2 in ProjectOverview

Well the tooltips are trying to provide extra info not immediately visible.
When it's just the icon, we show the text for high/med/low. On the project overview page we show the text plainly there, so I don't see the need to duplicate it with a tooltip.
There is a tooltip there for translation projects showing why the sensitivity is not editable.
So I'm still happy with current functionality.


Splitting first terminal after reload is sometimes really narrow

Moving this milestone bc cannot reproduce the problem

Created a PR #129668 fixing this


Add functions to get topic name to publisher and subscription

The rcl_subscription_get_topic_name and rcl_publisher_get_topic_name functions should be wrapped and added to the Subscription and Publisher, analogously to https://github.com/ros2-rust/ros2_rust/pull/143.

Seems like this was closed by #194 . Am looking for issues to work on.

That's true. How about https://github.com/ros2-rust/ros2_rust/issues/147?


Training and inference examples

Hi, thanks for the great work. Could you please provide some examples about the training and inference if possible? Such as dataset, pretrained model and inference code?

Hi, Thank you very much for your interest in our work. We have updated the documentation and inference examples.

Hi, Thank you so much for your interest in our work. We have updated the documentation and inference examples.


Hackathon

Bug Description
https://www.langflow.org/aidevs-india
is this legit? I was told to star the repo to get enter hackathon but then no update on what to do next
Reproduction
lorem ipsum
Expected behavior
lorem ipsum
Who can help?
No response
Operating System
arch linux
Langflow Version
1.0.9
Python Version
None
Screenshot
No response
Flow File
No response

https://www.langflow.org/aidevs-india/challenges

Yes is correct!
1 - Star the repo
2 - Sign up with the github email you used to star the repository.
Do this and you will be ready to participate, just wait for the challenges.

That is correct . You just need to wait for timeline to start.

Not sure why you ask to star the repo, but ok!


Signals are often unreliable and need to be connected manually through code

Godot version
4.1.2
System information
Godot v4.1.2.stable - Windows 10.0.18362 - Vulkan (Forward+) - dedicated NVIDIA GeForce GTX 1050 Ti (NVIDIA; <IP_ADDRESS>24) - Intel(R) Core(TM) i3-10100F CPU @ 3.60GHz (8 Threads)
Issue description
I've created a timer, set it to Autostart, connected its Timeout signal to my script through the Node tab, and it wouldn't ever trigger.

I've used breakpoints and prints and I'm sure it was never triggered.
I've redone the connection, still nothing. Deleted the whole thing and did it again, nothing.
Then I connected the signal manually through code and it worked.
.TSCN file shows it like this, with no signal whatsoever:

But as you can see from previous screenshots, the UI shows it as if it was connected, despite the .TSCN file having absolutely no signal information.
But there were other times where I've set it up exactly the same and it worked, so it's really unreliable for whatever reason.
Steps to reproduce
Create a timer, set it to Autostart, connect its Timeout signal to a script through the Node tab, see if it triggers. In my experience, sometimes it doesn't.
Minimal reproduction project
N/A

But as you can see from previous screenshots, the UI shows it as if it was connected, despite the .TSCN file having absolutely no signal information.

That doesn't really show that, signal connections are at the very bottom of the file, not next to the node


added radio groups

still need to write tests, but just need to see if this covers what we need.
still need validation to update the component when you update the value.

Coverage decreased (-9.9%) to 90.148% when pulling 33f9de4e47cc2dc6d38477de7882d084c1e8debc on feature/radio-groups into 2116e1246186b9a93ec3e826c36295857b220e6d on master.

AvRadioGroup needs to implement a reset method.

Coverage decreased (-10.7%) to 89.268% when pulling 9de4f17b1a1190c38a6e86bd4ecaccb7e305aa80 on feature/radio-groups into 2116e1246186b9a93ec3e826c36295857b220e6d on master.

Coverage decreased (-10.5%) to 89.487% when pulling c2ce65f17339da06e653b8d15bd591b73c355855 on feature/radio-groups into 2116e1246186b9a93ec3e826c36295857b220e6d on master.

Coverage decreased (-2.9%) to 97.066% when pulling 5f65475e1da9b8142b86f156c1dfeece8cc7dff4 on feature/radio-groups into 2116e1246186b9a93ec3e826c36295857b220e6d on master.

Todo before merge
[ ] reset method
[ ] AvRadioGroup tests
[ ] AvRadio tests

Coverage remained the same at 100.0% when pulling e854928b499b1cf9de5ccc828be7808396bc1bf2 on feature/radio-groups into 9fae3a3648c232fdfb772d7915dbc5fabf8e0fb2 on master.

Released in 1.2.0


Update README.md

Connects to #5, fix #5
@micdenny please review.

badges doesn't work properly

Works properly now

merged, thank you!


Add reunite for BiLock and Split{Stream,Sink}

In both cases, consumes two halves of a split-up value and recovers the original.
let foo: Foo = Foo();
let (sink, stream) = foo.split();
let foo: Foo = sink.reunite(stream).expect("sink and stream aren't from the same original Stream + Sink");

Seems like a slick API to me, thanks! I think the usage of Arc::ptr_eq is rasining the minimum Rust version though, mind bumping that in the travis config?

Thanks again @spinda!


⚠ TiendaEnLinea has degraded performance

In 8d0acfe, TiendaEnLinea (https://www.fahorro.com/) experienced degraded performance:

HTTP code: 200
Response time: 5031 ms

Resolved: TiendaEnLinea performance has improved in ced9bd2.


Method of getting drive size on Windows needs changing

For all the gory details, see my comments in https://github.com/resin-io/etcher/issues/995
But in summary: scripts/win32.bat in this repo currently gets the size of the drive using Win32_DiskDrive which erroneously calculates the size of the raw drive using the CHS values. While this works fine for mechanical hard drives, it doesn't work for flash media (SD cards, USB sticks, etc.) and often under-reports the actual size of the drive. IOCTL_DISK_GET_DRIVE_GEOMETRY_EX needs to be used instead, to get the actual size of the raw drive.
I believe that fixing this will fix all the problems mentioned in https://github.com/resin-io/etcher/issues/995

Fixed by https://github.com/resin-io-modules/drivelist/pull/186


Prevent component updates for k8s

Hot reloading is currently unsupported. This PR moves the partial support for component updates behind an environment variable (off by-default). Once hot reloading is completely implemented, it will be moved behind a feature flag.
Signed-off-by: Shubham Sharma<EMAIL_ADDRESS>Description

Issue reference

Please reference the issue this PR will close: #4432
Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[X] Code compiles correctly
[ ] Created/updated tests
[X] Unit tests passing
[X] End-to-end tests passing
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Specification has been updated / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Provided sample for the feature / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

/cc @yaron2 concerns over making this switch with environment variable?

@daixiang0 for FYI


[Bug] [Knowledge] 最后点击同步无法切片

Search before asking

[X] I had searched in the issues and found no similar issues.

Operating system information
Linux
Python version information

=3.11

DB-GPT version
main
Related scenes

[ ] Chat Data
[ ] Chat Excel
[ ] Chat DB
[X] Chat Knowledge
[ ] Model Management
[ ] Dashboard
[ ] Plugins

Installation Information

[X] Installation From Source

[ ] Docker Installation

[ ] Docker Compose Installation

[ ] Cluster Installation

[ ] AutoDL Image

[ ] Other

Device information
GPU 4090
Models information
LLM :QWEN-14B-CHAT-INT4
What happened
拉了最新的也一样，感兴趣的小伙伴自己可以试一下，最后一步别点切片处理直接关掉对话框，后面再在文件列表中同步。谢谢，删了也一样，主要是我没按pipeline走才有的异常，自己直接程序调用api也会有这个。相信大伙也会遇到。
What you expected to happen
最后一步别点切片处理直接关掉对话框，后面再在文件列表中同步
How to reproduce
最后一步别点切片处理直接关掉对话框，后面再在文件列表中同步
Additional context
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

报错 信息：   dbgpt.serve.rag.api.schemas.DocumentVO' is not mapped"

i'm sorry about that , we will fix that soon.

I will try it soon


Refactor internal client data model to track-centric approach

Reasons for doing so have been identified in #96 #90 #66 #84
Approach has been detailed in #96

This is fairly large effort, but we're currently pushing the bounds of complexity that the current data model will allow.

closed in #137


allow alternate coopr plugins root

first step towards fixing https://issues.cask.co/browse/COOPR-782

[x] allow setup.sh to use an alternate root for plugin data (ie a freshly cloned git repo)

LGTM


[Question] Cache@2 returns cache miss even though a previsous build with the exact same key populated the cache

How is the cache supposed to work?
I almost get a cache miss on every build, even though I'm 100% sure it is using the same cache key. Is it anythin more than this I need to do:
    - task: Cache@2
      inputs:
        key: 'nuget | "$(Agent.OS)" | **/fake.fsx.lock'
        path: $(PACKAGES_PATH)
      displayName: Cache NuGet packages

Is the cache only there for a specific amount of time?

Hi @mastoj, pipeline cache expires after 7 days of inactivity - please see https://docs.microsoft.com/en-us/azure/devops/pipelines/release/caching?view=azure-devops#when-does-a-cache-expire.

Hi @mastoj, pipeline cache expires after 7 days of inactivity - please see https://docs.microsoft.com/en-us/azure/devops/pipelines/release/caching?view=azure-devops#when-does-a-cache-expire.

@anatolybolshakov, hm. Then why do I get so many cache miss even though the key is the same between different builds? Are the cache per branch or something? Is there a way to cache per repository if that is the case?

@anatolybolshakov, hm. Then why do I get so many cache miss even though the key is the same between different builds? Are the cache per branch or something? Is there a way to cache per repository if that is the case?

@mastoj currently cache is scoped by exact branch, project and pipeline for security reasons - here are some docs about it https://docs.microsoft.com/en-us/azure/devops/pipelines/release/caching?view=azure-devops#cache-isolation-and-security
As I see there's no way to cache per repository at the moment - if you want this as enhancement, I would suggest to create a ticket on https://developercommunity.visualstudio.com/spaces/21/index.html, since this seems to be some Azure DevOps feature not related to the task itself.

Got it. Thanks.
On Mon, 1 Feb 2021 at 13:40, Anatoly Bolshakov<EMAIL_ADDRESS>wrote:

@mastoj https://github.com/mastoj currently cache is scoped by exact
branch, project and pipeline for security reasons - here are some docs
about it
https://docs.microsoft.com/en-us/azure/devops/pipelines/release/caching?view=azure-devops#cache-isolation-and-security
As I see there's no way to cache per repository at the moment - if you
want this as enhancement, I would suggest to create a ticket on
https://developercommunity.visualstudio.com/spaces/21/index.html, since
this seems to be some Azure DevOps feature not related to the task itself.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/azure-pipelines-tasks/issues/14290#issuecomment-770827369,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAD2NMCG4IUEUJDIDRRLUC3S42OMFANCNFSM4WVLUYSQ
.

Sure! Let me close this one since feature mentioned above seems to be for Azure DevOps itself.


Make "tern lock" work for multistage docker builds

Describe the Feature
The "lock" feature was scoped for single stage Dockerfiles when it was first introduced. Now that we can analyze multistage docker builds, we can use the results to lock a multistage Dockerfile.

A fix was merged. So closing this.