KAFKA-10023: Enforce broker-wide and per-listener connection creation…

This PR implements the part of KIP-612 that adds broker configurations for broker-wide and per-listener connection creation rate limits and enforces these limits.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

ok to test

There is one test failure in kafka.network.DynamicConnectionQuotaTest.testDynamicListenerConnectionCreationRateQuota that is related to the PR.

@rajinisivaram Thanks for your review.  I updated this PR to expose  'connection-accept-rate'  metrics, addressed the remaining comments, and the test should also be fixed.
It seems like it would also make sense to add and expose throttle time metrics. What do you think? If so, I will have a separate PR to add throttle-time metrics and use Token Bucket for quota calculation that David added as part of KIP-599 that works better with bursty workloads. I will update the KIP accordingly and notify on voting thread.

ok to test

@apovzner DynamicBrokerReconfigurationTest.testAddRemoveSaslListeners failed in all three PR builds, so probably related?
15:55:45 kafka.server.DynamicBrokerReconfigurationTest > testAddRemoveSaslListeners FAILED
15:55:45     org.scalatest.exceptions.TestFailedException: Processors not shutdown for removed listener
15:55:45         at org.scalatest.Assertions.newAssertionFailedException(Assertions.scala:530)
15:55:45         at org.scalatest.Assertions.newAssertionFailedException$(Assertions.scala:529)
15:55:45         at org.scalatest.Assertions$.newAssertionFailedException(Assertions.scala:1389)
15:55:45         at org.scalatest.Assertions.fail(Assertions.scala:1091)
15:55:45         at org.scalatest.Assertions.fail$(Assertions.scala:1087)
15:55:45         at org.scalatest.Assertions$.fail(Assertions.scala:1389)
15:55:45         at kafka.server.DynamicBrokerReconfigurationTest.verifyRemoveListener(DynamicBrokerReconfigurationTest.scala:1178)
15:55:45         at kafka.server.DynamicBrokerReconfigurationTest.testAddRemoveSaslListeners(DynamicBrokerReconfigurationTest.scala:1057)

Hi @rajinisivaram, the test failure turned out to be a bug where I did not remove connection rate sensors on listener removal. I fixed the code and tests now pass.

ok to test

retest this please

ok to test

@apovzner Thanks for the updates, LGTM. Merging to trunk.


Windows support: "epm run" creates a executes a windows command line script

https://github.com/typelead/eta/issues/106

Wonderful work! Merging.


Rename to ⚡ Reducto

Rename to ⚡  Reducto

Changes 👷♀
Changed old and boring name useStateReducer to new, fancy, sexy and shiny useReducto.

😏 🪄 ✨


Datadog tracing issue where Datadog’s “operation name” field would contain what should be in the “resource name” field.

Bug Description
The above mentioned bug was fixed in envoy version 1.26.6 :-
https://www.envoyproxy.io/docs/envoy/v1.26.6/version_history/v1.26/v1.26.6
Current envoy version is 1.27.3-dev when istio 1.19.4 is installed

As per release notes of 1.26.6 envoy version the bug is fixed yet we are facing the issue when istio is upgraded to 1.19.4 and envoy 1.27.3-dev.
Can anyone help here ? @howardjohn https://github.com/istio/istio/discussions/48064#discussioncomment-7794044 as we understand the envoy versions are build and managed by istio team itself.
Version
istioctl version
client version: 1.19.4
control plane version: 1.19.4
data plane version: 1.18.5 (10 proxies), 1.19.4 (3 proxies)

Additional Information
No response

I'm not sure what happen(maybe there is something wrong with backport's process), but can you try with 1.19.5?

After upgrading to istio 1.19.5 resource name value has been changed to istio-virtual service name whereas in istio version 1.18.5 resource name value was with the kubernetes service name.
In 1.18.5 the resource name pattern :- "abc-svc.namespace.svc.cluster.local:8080/api"
In 1.19.5 the resource name pattern :- "app-istio-virtualservice:443/abc/*"

I cannot recall anything related this, could you share the configuration, istioctl pc l and istioctl pc b .

found similar issue with envoy https://github.com/envoyproxy/envoy/issues/30235 @zirain @howardjohn  any idea is it fixed ?

@parshva-shah-dsw can you try with the latest version?

@zirain 1.20.1/1.20.2 which one should we try?


v3 uses generics in schemas so IDEs can see instance and property base types

v3 makes all schema classes based on Generic[T]
So now when models are instantiated, they will show what class they are based on
Like so:
my_model = MyModel(some_str='a')
# my_model is type MyModel[frozendict.frozendict] in the vscode IDE
# my_model.some_str is type SomeStr[str] in the vscode IDE

[x] keep the mixin classes for ide methods (needed to show possible properties)
[x] make Schema generic
[x] when making classes always define new so the generic can be seen
[x] update properties to add the generic type
[x] var equals class define class instead, needed for generics
[x] update ref props to add generic
[x] fix UnsetAnyTypeSchema not being used in AbstractStepMessage
[x] update get_item property with type
[ ] update get_item array with type
[ ] add __init__ method to AnyTypeSchema
[ ] update parameter and header schema and content type definition to be [typed]
[ ] change ref schemas to be type aliases

Out of scope future improvement. Use typing.Never in NotAnyType
PR checklist

[ ] Read the contribution guidelines.
[ ] Pull Request title clearly describes the work in the pull request and Pull Request description provides details about how to validate the work. Missing information here may result in delayed response from the community.
[ ] Run the following to build the project and update samples:./mvnw clean package
./bin/generate-samples.sh
./bin/utils/export_docs_generators.sh

Commit all changed files.
This is important, as CI jobs will verify all generator outputs of your HEAD commit as it would merge with master.
These must match the expectations made by your contribution.
You may regenerate an individual generator by passing the relevant config(s) as an argument to the script, for example ./bin/generate-samples.sh bin/configs/python*.
For Windows users, please run the script in Git BASH.

Closing this and work will continue on in https://github.com/openapi-json-schema-tools/openapi-json-schema-generator/pull/174
Starting a new PR was easier than rebasing this one


Before Turkey CSA P1 and P3 integrations status

Update me on integration status and progress here before Thanksgiving Break.

Link to Integration Planning

link to integration planning

Link to integration planning

Integration Planning

Integrationhttps://github.com/SrinivasNampalli/SRINIVCSA/issues/9

Saathvik's Integration Planning

Link to integration planning

Trevor Huang
Period 1
Link to my team's plans

All planning tickets housed under Team Items with an "in review" status:
All team plans


Pylint is precondition on Azure side

Pull Request Test Coverage Report for Build 537

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 97.374%

Totals

Change from base Build 536:
0.0%

Covered Lines:
445

Relevant Lines:
457

💛  - Coveralls

Pull Request Test Coverage Report for Build 537

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 97.374%

Totals

Change from base Build 536:
0.0%

Covered Lines:
445

Relevant Lines:
457

💛  - Coveralls


SimpleIoc.Register does not enforce generic type TClass: TInterface constraint

SimpleIoc.Register<TInterface,TClass> does not have the generic type constraint that TClass : TInterface.
So you can do this:
class IFoo { }
class Foo : IFoo { }
class Bar {}

SimpleIoc.Register<IFoo,Bar>();

...and you won't get a compiler error, but it will crash at runtime.
I noticed that the Register<TInterface,TClass> method does have  where TInterface : class where TClass : class as a constraint, so I suspect this isn't a simple oversight. Is there a reason it doesn't have a TClass : TInterface constraint?

That's a good point. I am not 100% sure why I went this way. I will add this to the backlog and look into it.

Fixed in https://github.com/lbugnion/mvvmlight/commit/3408287579ace291c9519edefc8017df6a151aac


Update ARFoundation Doc for 2.3

Overview
The ARFoundation capabilities are now in the Foundation package and not the separate UnityAR package, this change is now reflected in the Android/iOS doc.  Also changed 2019.x to 2019.3.
Verification

As a reviewer, it is possible to check out this change locally by using the following
commands (substituting {PR_ID} with the ID of this pull request):
git fetch origin pull/{PR_ID}/head:name_of_local_branch
git checkout name_of_local_branch

/azp run mrtk_docs


should ssl endpoint docs explain how to get a signed cert?

If we get certs-per-app then a wildcard cert, though still helpful, might no longer be necessary to get started.
StartCom has been giving away free class1 certs for ages. Free ones not wildcard, but if the CN is set to deis.domain.com they would still be usable for control surface...which would reduce the risk of control surface credentials being lifted. And seeing as those can do a lot of damage, this seems desirable to me.

That is called SNI, and nginx is capable of it, we just need to make sure nginx and openssl are compiled for it, and that we generate the appropriate nginx.conf template things based on the availability of SSL certs on a per-domain basis.
http://nginx.org/en/docs/http/configuring_https_servers.html#sni

Thanks @ianblenke I'm well aware of SNI but that's not quite what I was thinking about in this one.
It occurred to me that a signed default certificate with a single CN is no worse than a self-signed cert, and that if the CN were to be deis.domain.tld then the control plane would work over https without having to ignore the trust chain because there would actually be one.
Solving the problem of 1 cert per app eg. with SNI is ongoing in other issues.

I think this issue serves as a good pointer should someone need help generating a valid cert. :thumbsup:
The docs in #2911 should help, too.


Show Enter card Manual

I have checked same version in iOS and its provide the facility that Enter Card manual label in bottom same as below image

but in android I am unable to set that label. Please let me know how to set that type of label in android also and get its onClick event. Help would be very appreciate. Thanks.

What do you think @awushensky ?

Thanks for pointing this out. @xsl will take a look at it.

I know might be take long time but let me know approx time.

I believe there is a PR out for this now. I expect this will merge in the next few days.

hey @Zennaxxtech, we just released a change that will allow you to toggle this via an intent.
See the following for examples:
https://github.com/getbouncer/cardscan-android/blob/master/cardscan/src/main/java/com/getbouncer/cardscan/ScanActivity.java#L69
https://github.com/getbouncer/cardscan-android/blob/master/cardscan-base/src/main/java/com/getbouncer/cardscan/base/ScanActivityImpl.java#L27


Update numpy_question.py

Update numpy_question.py -> Done

It works, you finished this lab ✅
I will not merge your PR so that other student do not see your answer from the main


adding spinkube

Closes https://github.com/kairos-io/kairos/issues/635

This was tested on the following distros:
Success

Ubuntu 24.04
Fedora 40

Fail

Alpine 3.19 (needs systemd)

@Itxaka It's an issue with the Kwasm installer: https://github.com/search?q=repo%3AKWasm%2Fkwasm-node-installer systemctl&type=code

Also, @mauromorales do I need to do anything specific to get the spinkube bundle on to Quay?


Unversioned package dependent on versioned package getting changelog entry

Affected Packages
pkg:@changesets/apply-release-plan, I think

Problem
I have an unversioned app which depends on a versioned package. When I run changeset version, it creates a changelog entry for the unversioned app with the version null

Steps to reproduce:
clone and checkout the commit at this repo: https://github.com/willwill96/petfinder-api-apps/tree/55ef85d99344a93777bf1545ab8c6b44cc98a970
run yarn install && yarn changeset version and see the changelog is generated

Proposed solution
[Edit]: I have a PR open here to address this issue: https://github.com/changesets/changesets/pull/967

i'm experiencing the same in an Nx repo.
We are building packages to a root level dist dir, which is the default Nx behaviour.
This means the code we publish does not come from the source dir, so we rewrite the pnpm workspace file during publishing to only point at dist.
Changesets then finds built versions and works well, until it encounters an internal package.
Even though it's marked private: true and unversioned, the bot (i'm guessing changeset version as mentioned above) still tries to version it when its workspace dependent changes.

have you found any workaround?


Test migrating to Dual-Stack on AWS - Test Release KubeOne 1.6

This is not possible. We'll consider in the future if we can make it possible with KubeOne.


Outdated code in Minimal Orleans app tutorial

Using the current 3.6.2 version of Microsoft.Orleans.Server, HostBuilder was not available and I had to modify my code as follows to utilize SiloHostBuilder instead:
static async Task<ISiloHost> StartSiloAsync()
{
  var builder = new SiloHostBuilder()
      .UseLocalhostClustering()
      .Configure<ClusterOptions>(options =>
      {
        options.ClusterId = "dev";
        options.ServiceId = "OrleansBasics";
      })
      .ConfigureApplicationParts(
            parts => parts.AddApplicationPart(typeof(HelloGrain).Assembly).WithReferences())
      .ConfigureLogging(logging => logging.AddConsole());

  var host = builder.Build();
  await host.StartAsync();

  return host;
}

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 1d3e0034-a134-6eab-477f-5c5668eb68a0
Version Independent ID: b61f5003-091e-e698-982b-124316d7eba3
Content: Minimal Orleans app sample project
Content Source: docs/orleans/tutorials-and-samples/tutorial-1.md
Product: dotnet-orleans
GitHub Login: @IEvangelist
Microsoft Alias: dapine

Hi @frank-taylor-pcg from the comments in the recent docs updates it seems like Orleans is migrating to use the generic host (Microsoft.Extensions.Hosting.IHost) - I welcome any corrections if mistaken!
I have submitted a PR to update the sample to use the generic host instead: #29935

Thank you @frank-taylor-pcg for this issue, and @richstokoe for the corresponding PR. Much appreciated on both accounts. With that said, I'm waiting for @ReubenBond to have a look at the PR, assuming all is good there. We'll have that PR close this issue. Thanks again.


Git setting git.countBadge is broken

Found by @avih

This seems to have broken with vscode 1.11.0 (was working in 1.10.2).
It now always works as if git.countBadge is set to all, regardless of any actual value it's set to (specifically, tracked and off are not respected).

It now always works as if git.countBadge is set to all

This is weird.
With 1.12.1, It seems that now tracked behaves for me like an alias to off. I tested it on windows and linux, and also after disabling any git-related extensions (on windows at least), and on both systems, when set to tracked - it now never counts tracked modified files - and doesn't show a counter badge at all.
The "thing" at the bottom left of the status bar does show * for modifications, but no counter badge at all on the big git icon on the left.
If I set it to all, then it does show a count badge and does count modified+untracked.
Could anyone please confirm?

Thanks @avih! #26034


[AutoPR managementgroups/resource-manager] Add Management Groups 2018-03-01-preview version

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/3064

This PR has been merged into https://github.com/Azure/azure-sdk-for-go/pull/1868


[in progress] Update subscriber list title for business finder subscription lists

Superseded by https://github.com/alphagov/finder-frontend/pull/1164


Implement a van subscriber for new pending blocks.

Overview
For block verification process, orchestrator needs to have new block header from pandora client as well from vanguard client. So orchestrator client subscribes to trusted vanguard client for new block headers. After receiving notification of new pending new block, it needs to cache and store into db and key will be slot number and value will be blockHash and blockHeader.
Implementation

Implement subscription method. Api definition: van_subscribe: StreamNewPendingBlocks in vanguard client.
When bootstrapping the orchestrator node, it will subscribe to van client in van_subscribe: StreamNewPendingBlocks event api.
If connection does not successful then should be have option to retry the connection as well retry to subscribe.
If connection is successful and getting block from vanguard, store it into lru cache. There will be a cache for that: slot -> block

I moved it to done, if it is not correct please restore


Add Report with administrators

Identify ANET administrator positions and the person assigned to that position.  See attached specification and report prototype.
Reference item 38

Added to Epic 1958 - Add new insights #1958


[C++]: Don't generate copy and move constructors and operators

They are not needed because they are generated automatically as long as you don't have destructors and just copying all the fields should be the correct way. They are also invalid as they are because they can't copy an ampty flyweight (one that was constructed with a default constructor). This happens because we try to call reset which validates buffer sizes.

This broke my code.  I think it's because without the call to reset(),  m_positionPtr is copied as a pointer into the original object.


Improve GDScript range() description

Clarifies that range() does not allocate array in for loops.
Also linked "GDScript basics" in tutorial section, as I think it should be there.

See #71564.


Added my name

@hemantwasthere I have sent you an invite to the organisation. Please remember to publicise your membership. Welcome to the org 🙂

@all-contributors please add @hemantwasthere for content

Thanks for sending me invitation 😊


Update maxAge-behaviour for /verify & /retrieve

What type of PR is this?
Add one of the following kinds:

correction

What this PR does / why we need it:
Updating the maxAge-behaviour for location-verification and location-retrieval
Which issue(s) this PR fixes:

Fixes #216

@camaraproject/device-location_maintainers this PR is ready to be reviewed 😸

@maxl2287 please give priority to this PR as it is blocking the process for these 2 APIs. Thanks

Update is done.
Please check @jlurien and @bigludo7

We have to add another example to the examples array, e.g.:
`
LOCATION_VERIFICATION_422_UNABLE_TO_FULFILL_MAX_AGE:
summary: Unable to fulfill maxAge
description: the system is not able to provide the fresh location required by the client
value:
status: 422
code: LOCATION_VERIFICATION.UNABLE_TO_FULFILL_MAX_AGE
message: "Unable to provide expected freshness for location"

@bigludo7 take a final look and we can merge it. Thanks


[🐞] Qwik 1.9.0 holds onto searchParams when redirecting from a server action

Which component is affected?
Qwik City (routing)
Describe the bug
Through 1.8.0, our app would successfully redirect from server actions to some URL like /profile from the front page of the app. The front page takes an ?id=XXX param to get some data and then after tapping a consent button, it runs an action, saves data to a cookie and throws a redirect.
It used to drop the search params when redirecting. Since 1.9.0, it holds onto the search params.
Reproduction
https://stackblitz.com/edit/github-cv95eb?file=src%2Froutes%2Findex.tsx
Steps to reproduce
No response
System Info
System:
    OS: macOS 15.1
    CPU: (10) arm64 Apple M1 Pro
    Memory: 63.16 MB / 16.00 GB
    Shell: 3.7.1 - /opt/homebrew/bin/fish
  Binaries:
    Node: 18.20.3 - ~/Library/pnpm/node
    Yarn: 1.22.19 - ~/Library/pnpm/yarn
    npm: 10.7.0 - ~/Library/pnpm/npm
    pnpm: 9.4.0 - ~/Library/pnpm/pnpm
  Browsers:
    Chrome: 129.0.6668.60
    Safari: 18.1
    Safari Technology Preview: 18.0
  npmPackages:
    @builder.io/qwik: 1.9.0 => 1.9.0
    @builder.io/qwik-city: 1.9.0 => 1.9.0

Additional Information
No response

Seems related to this:
https://github.com/QwikDev/qwik/pull/6778/files#diff-6c570481bf47bb1dae75e8d12166dc67e7ed94954b9714816b7378813b20a24d
But if this is what the team thinks should be the correct behavior, how can I drop the searchParams?

@brandonpittman can you provide a repro so we're talking about the same things? I thought that change was to provide search parameters when calling actions?

@wmertens
The repro link above shows this behavior:
if you submit a form from a page with searchParmas and redirect in the action, the page you get redirected to has the previous page’s searchParams even though the redirect was to a route with no searchParams.

@wmertens Just for completeness, here's another Stackblitz pinned at 1.8.0 where redirects don't hold onto the submitting page's searchParams.
https://stackblitz.com/edit/github-cv95eb-7l9fik?file=package.json

Just to confirm, it's holding the search params of the old page after the redirect?

Pretty sure this is what broke things:
https://github.com/QwikDev/qwik/pull/6778

@thejackshelton
If I submit a PR, do I need to fix the regression while leaving the commit that broke it in the first place in-place?

Another problem is that if you redirect to a URL with searchParams, Qwik drops the params in the URL you try to redirect to.

I started a PR that confirms that a change in 1.9.0 broke this functionality. Merely commenting the change out restores the original functionality.
https://github.com/QwikDev/qwik/pull/6927
Can we just revert the bad PR or do I need to rewrite the original PR that broke the existing functionality while preserving the original PR's intent?

I created a PR. Can someone take a look at it, please?


MEX of an array in C

💥 Proposal
(A clear and concise description of what the proposal is.)
Have you read the Contributing Guidelines on Pull Requests?
YES.

/assign


Trainings that the US Tax Court PM attended

Description
Definition of done

[ ] Uchenna to sync with Vicki/ Tax Court PM to identify the training that they attended
[ ] Publish said trainings to the DOJ team - Bill and Dan
[ ]

Attachments
title

Moving this card to done - collating trainings in card #387


loadFromUrl sometimes stalls if loading assets that depend on same material

If I have asset1, and it uses material1 and material2
And I have asset2 and it uses material1 and material3
Then, in certain conditions* loadFromUrl will stall on loading the second asset. Some digging through the engine reveals that onLoadAsset (~line 498 of asset-registry.js) is not being called a sufficient number of times although self.loadFromUrl (~line 510 of asset-registery.js) is being called for each material required. I suspect some cache somewhere is soaking up the request and the callback isn't being run if it's in the cache.
I managed to solve the problem in my project by removing the shared material dependency, but this is not a correct solution.
*Note on the "sometimes" nature of the issue:
If you call app.assets.loadFromUrl while iterating through a list (aka a typical usage), then this problem does not occur. Only once an asset (that uses a shared material) has finished loading before scheduling another loadFromUrl does this happen.
I'll try to make an example for this, but have been chasing this issue all afternoon and need to take a break.

Closing because a reproducible example was never submitted. If you can still provide one (and this issue is still active), please reopen it.


Fixed Zookeeper to restart every time Ansible runs on Debian

Ansible were restarting Zookeeper every time Ansible were running.
Fixed service state=started and added restart handlers to restart
only if configuration changes.

+1


Close Folder shortcut not working

Type: Bug
When presing Ctrl+ K F, the opened folder in Visual Studio Code, should be closed. But it is not the case.
VS Code version: Code 1.70.0 (da76f93349a72022ca4670c1b84860304616aaa2, 2022-08-04T04:38:48.541Z)
OS version: Linux x64 5.15.59-1-MANJARO
Modes:

System Info

Item
Value

CPUs
11th Gen Intel(R) Core(TM) i7-11800H @ 2.30GHz (16 x 1361)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: disabled_softwarevideo_encode: disabled_softwarevulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
1, 1, 1

Memory (System)
15.37GB (9.79GB free)

Process Argv
--unity-launch --crash-reporter-id 939d275c-26ff-43ee-b515-8990918271a8

Screen Reader
no

VM
0%

DESKTOP_SESSION
gnome

XDG_CURRENT_DESKTOP
GNOME

XDG_SESSION_DESKTOP
gnome

XDG_SESSION_TYPE
wayland

Extensions (15)

Extension
Author (truncated)
Version

csharp
ms-
1.25.0

python
ms-
2022.12.1

vscode-pylance
ms-
2022.8.30

jupyter
ms-
2022.7.1102252217

jupyter-keymap
ms-
1.0.0

jupyter-renderers
ms-
1.0.9

vscode-thunder-client
ran
1.18.6

fabric8-analytics
red
0.3.6

java
red
1.9.0

vscodeintellicode
Vis
1.2.22

vscode-java-debug
vsc
0.43.0

vscode-java-dependency
vsc
0.21.0

vscode-java-pack
vsc
0.25.0

vscode-java-test
vsc
0.36.0

vscode-maven
vsc
0.37.0

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonvspyt551cf:30345471
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscoreces:30445986
pythondataviewer:30285071
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593cf:30376535
pythonvs932:30410667
wslgetstarted:30449410
cppdebug:30492333
pylanb8912:30545647
vsclangdf:30486550
c4g48928:30535728
d7fjc500:30543455
dsvsc012cf:30540253
azure-dev_surveyone:30548225

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.70.2. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

Please use the Start Extension Bisect command to investigate whether one of your extensions is causing this.

Extension Bisect does not find the problem. Every time Bisect restarts vscode, I still cannot close the opened folder (by pressing Ctrl + K F )

Please use the advice at https://github.com/microsoft/vscode/wiki/Keybinding-Issues to discover what is happening with that key sequence.

[2022-08-27 23:25:13.551] [renderer1] [info] [KeybindingService]: + Leaving chord mode: Nothing bound to "Ctrl+K Ctrl+F".

[2022-08-27 23:25:13.551] [renderer1] [info] [KeybindingService]: + Leaving chord mode: Nothing bound to "Ctrl+K Ctrl+F".

Sounds like your keybinding got unbound.
Here's a gif on how to reconfigure that keybinding


added support for $ne operator

Describe your changes
Added NeMatcher class implementing LikeMatcher interface. NeMatcher does exact string equality testing.
How best to test these changes
Tests for this functionality are in query/filter/comparison_test.go and test/v1/server/document_test.go, in TestRead_Collation.
Issue ticket number and link
/claim #873 for algora bounty

had a syntax error, due to 'mockery' dependence i am not able to compile on my machine

Try running install_build_deps.sh in scripts.

had a syntax error, due to 'mockery' dependence i am not able to compile on my machine

Try running install_build_deps.sh in scripts.

or just make setup


Create /ca/nt/auroracollege.txt

Aurora College is a post secondary educational institution operating in the Northwest Territories.

@mdares Thank you for your request. Could you please give us a reference to a page describing a long-term (> 1 year) IT-related course which your university offers? You have a significant number of courses, but the courses I found using a keyword "software" appeared to be more or less short (45 to 75 hours).

@mdares Unfortunately, I have to close this request without merging because we don't have an update from you, and have no information confirming that the college actually conducts some long-term IT-related education. For short-term courses you are free to use trial versions of JetBrains tools, or request Classroom license by email to<EMAIL_ADDRESS>Thank you for your understanding.


extend musl weak symbols mapping

Extending the musl weak symbol mappings to resolve https://github.com/rust-cross/cargo-zigbuild/issues/179.
The symbol list was generated via scrapping the header files from the musl source code at (http://git.musl-libc.org/cgit/musl/)
fyi: I'm fairly new at the compiler stuff, so if this PR went to far than I'm happy to adjust as directed.

Thanks!
I don't think we need to add constants because usually they're just a #define in C, you can't refer to it in Rust so it's not needed here.

Thanks for the assist. I leaned towards adding add #defines since it was generally easier to remove than add.

It'd be great if you can confirm that this actually fixes #179 for your project, thanks!

It'd be great if you can confirm that this actually fixes #179 for your project, thanks!

Just added a comment that our project builds on this PR. https://github.com/rust-cross/cargo-zigbuild/issues/179#issuecomment-1713192771


Double encoding or no encoding

If I pass to router.navigate(…, params) a params object containing url encoded value, it's encoded once again, else it's not encoded at all.
I would like to encode value once, but it seems to be impossible now.
Is there any way to force encoding once, or to somehow trigger encoding ?
If there is none I'd be happy to introduce PR with update :)
EDIT: Looks like it's caused by path parser, it happens here:
https://github.com/troch/path-parser/blob/master/modules/Path.ts#L190
I think thaht to fix this an option is needed in both path-parser and router5 to enforce encoding (with encodeURLComponent) or not encoding url param..

This is happening to us as well.

There is a workaround. But it's really ugly and not to safe I'd say:
const original = encodeURI
window.encodeURI = encodeURIComponent
router.navigate(routeName, routeParams)
window.encodeURI = original

:upside_down_face:


Installing via Helm, can't get imagePullSecrets to set correctly

Installing tigera-operator via Helm, I am trying to set the top-level imagePullSecrets variable.
I have created a secret in the appropriate namespace called dockercredentials
Running the Helm install with this argument:
--set imagePullSecrets.name=dockercredentials
gets this output:
Error: INSTALLATION FAILED: Secret "name" is invalid: data[.dockerconfigjson]: Invalid value: "<secret contents redacted>": invalid character 'd' looking for beginning of value
I've tried various other attempts:
--set imagePullSecrets[0].name=dockercredentials
--set imagePullSecrets[0]=dockercredentials
--set imagePullSecrets=dockercredentials
None of the above are accepted as valid syntax.
I have used imagePullSecrets with lots of other Helm charts and Kubernetes configs, but this particular one has me stumped. What's the correct way to set this?

After some investigation, I see that I have to set the imagePullSecrets to the full contents of a ~/.docker/config.json file.

This ain't easy on a command line. Almost every non-alphanumeric character in the string needs escaped in some way.
Even after successfully setting the secret, I couldn't see any evidence of it being used.
I managed to workaround around this entire thing by using Helm's --postRendererHook functionality to set the spec.template.spec.imagePullSecrets property of the tigera-operator Deployment. Once I did that, I didn't need to use set --set imagePullSecrets="...", as I could just use a secret that I had previously created in the same namespace as the Deployment.


[WIP] ADD triggerTest

This is a barebones script for testing the trigger latency.

closes #3

hey @andyh616, if we wanted to update the master branch with the MEAN stack, we can do it in this pull request and we will have the latest on my branch, and yours as well. if you're ok with it, feel free to merge

conferred offline with @andyh616. cherry-pick latest relevant commit from andyh616/mean branch. Merging.


Running the physics on a Windows Form application, timestep and other problems

Hi, I'm running the physics on a Windows Form application. The application is just a server, so it doesn't need to render any game object. It just calculates the positions and collisions then sends the numbers to the clients, who will render the objects.
As we all know, every game API or engine has an Update() method. However, I'm not sure where to call world.Step() on a Windows Form app.
Currently, I'm using a System.Timers.Timer object, and trying different intervals. But the physics behaviour looks weird depending on the interval value.
In my server, so far, I have just a bouncing ball and a ground.
At 1 ms, the ball often doesn't bounce when hitting the ground.
At 15 ms, the ball often doesn't bounce, and it seems slower.
At 30 ms, the ball bounces with a force lower than expected.
The time step call happens at every clock tick, as the following:
Step(1.0f / 100.0f, false);
When and how often should I call Step() on a Windows Forms app?

Step(1.0f / 100.0f, false); integrates your world 10ms forward in time. It should not matter if you call that function every 10ms (that should then be real time) for a second (the simulated second would then be in real-time) or call that function each day at midnight for 10 days.
Your other questions (Windows Forms) are not related to the engine.


Discover how Microcks can speed-up your AsyncAPI adoption - Part 2

2nd and last part of How Microcks Can Speed-Up Your AsyncAPI Adoption series 😉
Post title
How Microcks can speed-up your AsyncAPI adoption - Part 2
Post description
In this post we will show you how Microcks is leveraging the AsyncAPI specification on Kafka in a very pragmatic and powerful approach: way beyond documentation or code generation! We will also go through the different business use-cases implemented by users integrating Microcks in their asynchronous API toolchain.

Now published. Thanks!

Now published. Thanks!


feat(app): expose metrics to prometheus

Fixes #143

@vigsterkr this is marked as draft. But it seems like it may be ready for review?

@olevski sorry!🤦

This is great!
Should we include here the annotations to the service that allows Prometheus to automatically scrape the metrics (when enabled)?

This is great! Should we include here the annotations to the service that allows Prometheus to automatically scrape the metrics (when enabled)?

@aledegano yeah I can do that.

@aledegano prometheus can be setup to scrape pods directly. As a matter of fact that is how ours is setup. The only thing I had to add is allow ingress on the metrics port in the network policy for amalthea. After that everything works:

The helm chart for amalthea already allows you to add custom annotations. So that can be used to enable scraping. There is no need to add more stuff in the already crowded values file. And there is no need to create a service.


Migrate to .Net Standard

As title suggest, I would like to see this library migrated to a cross platform compatible solution, .NET Standard.

In one particular project I have - this is the only dependency that is stopping me being .Net Standard :(

Any updates on this one?  .Net Core is now the standard for any new Microsoft developments.

Bump @myob-developers - any update on this one...?

There are definitely more important elements to work on - couldn't care less about support for .NET Core and it's bugs.

We converted this to use in .net core and I must say the performance while running on a linux dockerised micro-service is way better now. Would be happy to contribute this back once we have it tested and stable.

We actually did the same @FaysalM to run in exactly the same environment, from memory all we did was create a new dotnet core class lib, copy all the files over, compile and package and we haven't had an issue since. So I'm not sure what's stopping the @myob-developers doing the same! @cmsmith-asc compile to dotnet standard and we can run both on full framework and dotnet core, its a win:win for all, imho

I do agree. Our performance gains by just moving our whole project including moving the sdk to core running on a Linux docker has been magnitudes better. Especially if you run scheduled syncs and other background tasks.

We're now looking to integrate with MYOB Advanced and run in .NET 6 / Linux.  @myob-developers Any word of if/when a .NET Standard version of this package will be released?

@MichaelK120 or @alexeygrantmyob you seem to be actively making contributions are there any plans to support this? It's well established now and the migration should not be difficult.

.NET core/standard support is basically essential. Would be great for it to be part of the official package rather than everyone having to keep forks updated.

@myob-developers any update on your package supporting newer .NET version?


Auto Effects:  Abjuration Wizard level 2+

When a user has CPR checked in the settings for the importer and imports a level 2+ abjuration wizard, DDBI still imports its own functionality for the arcane ward alongside CPR's and gets in the way/is unnecessary.  If CPR is enabled it should not be shipping Arcane Ward - Hitpoints item.

This interaction is also breaking CPR's Arcane Ward.  I think what is happening is the link item to resources integration is causing the Item use fields to change on the CPR item cause you probably put the CPR item on, then the popup for resources fires right after.  Either way the Arcane Ward needs the third field populated with charges, but its arriving on a fresh import blank, causing link item resources to break.

This is likely correctable on my end, I can specify fixes for stuff like this.

From a quick test, adding "Arcane Ward - Hitpoints" to the removed items list doesn't appear to be fixing it.

@chrisk123999 it's because it's Arcane Ward - Hit Points - works here if I add it to the removedItems map in your config.

https://github.com/chrisk123999/chris-premades/pull/136


Views dropping

Is there an existing issue for this?

[X] I have searched the existing issues

I'm submitting a ...

[x] bug report
[ ] feature request
[X] support request --> Contact me over mail for support https://github.com/MShawon

Description
I used the bot on an earlier version of chrome(v104) as asked, but still the views are dropping within 1-2 days. What can be wrong? Please help.
Environment
- OS : Windows 10
- Python : EXE
- Script version : 1.8.0

config.json
{
    "http_api": {
        "enabled": true,
        "host": "<IP_ADDRESS>",
        "port": 5000
    },
    "database": true,
    "views": 200,
    "minimum": 5.0,
    "maximum": 9.0,
    "proxy": {
        "category": "r",
        "proxy_type": "http",
        "filename": "",
        "authentication": true,
        "proxy_api": false,
        "refresh": 0.0
    },
    "background": false,
    "bandwidth": false,
    "playback_speed": 1,
    "max_threads": 8,
    "min_threads": 5
}

invest in better proxies.


Row Colors don't working very well

Windows 10

DBeaver 7.1.2

Microsoft SQL Server 2017 (RTM-CU8) (KB4338363) - 14.0.3029.16 (X64) - Developer Edition (64-bit) on Linux (Ubuntu 16.04.4 LTS)

Problem
When I colorized the row and the row is a link to another table, the text in the row is blue, therefore which I use a strong color behind of row, the text is hidden. It would be great if we could change the color of the link as well.

Steps to reproduce

Panel Result
Select a column in the result panel and right-click
Choice View/Format and after in "Row Colors ..."
Select column and click in "Add"
Below, input the value for compare and choice the colors

If I understand you correctly, it is not a bug, because the function (to change link color when row color is changed) is not implemented.
You can change cell data color using Foreground field in Row colors...

Please ask if you have additional questions.

I understand your point of view, but this causes a bug in the visualization of the row, but ok.
So I believe I should open as a new feature?

Yes please


Who is Izy?

This feels like a very sexist repo, like a repeat of the whole factory_girl gem disaster of 2017.

Hi @mylescc,
The script is called lzy, the first character is an 'L', which may look like an 'I' on the typeface used on github.com on MacOS, but in fact isn't.
Interesting bit of history: the script is called lzy, as it's the shortest way to represent something recognizable as 'lazy'. This is to represent lzy.js being a very small script compared to all other lazy-loaders.
Marking this issue as closed.


Add build instructions

How to build, requirements, etc.

After @ForNeVeR created TdLib.Native package build is as simple as dotnet build or "Run" button in IDEs :) Feel free to re-open this issue if it's not.


KAS version is not updating

Hello,
I've updated the version about 10 hours ago, but CKAN still shows the old version. The config of the mod is in: https://github.com/KSP-CKAN/NetKAN/blob/master/NetKAN/KAS.netkan.
Is it something broken in the pipeline, or 10 hours is not enough for it to update?

Thanks for calling this to our attention. The bot seems to not be running at all currently; "15 hours ago" is not a normal value for "last checked": http://status.ksp-ckan.org/
@techman83, can you please give it a good smack?

There it goes. Latest KAS is here: https://github.com/KSP-CKAN/CKAN-meta/commit/5c6325110aa706b6bd3828a32b6baf4e49626aa8
Please let us know if there are any further issues.

Awesome! Thank you very much for the quick resolution.

Usually things are fine, but when a lot of new mods drop or one of the bigger hosting sites are under a bit of strain the indexing takes longer. Sometimes hitting the run time out. After a few runs, the mods get cached up, things get archived and it catches up.
I've dropped the timeout and the schedule to let it do a complete run and check back later tonight.


Fix memory allocation of Cloud Run to 128Mi

What
Current memory allocation of Cloud Run is 256 Mi. 128 Mi is enough to AI Platform notification.

Cloud Run seems to use 50% memory and 258Mi is reasonable.


How to Create a URL Shortener with JavaScript

We ask that you please be patient as our team works through approving and publishing all articles/tutorials in a timely manner. Allow 1-3 days for a topic to be reviewed and/or approved - allow 3-7 days for an articles to be reviewed and/or published.
Be sure to visit our Resources Page for tools, resources, and example articles to go over.
NOTE: (Please ensure that you have only one open issue + linked pull request at a time. This will ensure that we complete the article in a timely manner from inception to publishing.)
NOTE: We tend to stray away or tend not to publish reviews/comparisons of commercial product offerings.
We're looking for the first 2-3 paragraphs of the article that appropriately summarize what your article will be about.
Proposed title of article
How to Create a URL Shortener with JavaScript
Introduction paragraph (2-3 paragraphs):
This guide will show you how to build a URL shortening service like bit.ly or goo.gl using JavaScript. A URL shortener is useful for shortening long URLs so that you can easily share them with your friends or co-workers.
To understand how these things work, we need to take a closer look at an URL shortener, so we’ll be building a simple URL shortener. With that task, we’ll be learning some new things and understanding how URL shortener works.
Key takeaways:

URL shortener
How does a URL shortener works?
Building a URL shortener

References:
Please list links to any published content/research that you intend to use to support/guide this article.
Templates to use as guides

How To Guide Template
Software Review Template
Tutorial Template

Hey @JudyNduati make sure that your article  does not overlap this: https://www.section.io/engineering-education/nodejs-url-shortener/
cc: @hectorkambow

@JudyNduati - Could you please explain how your article will be different from the one mentioned by @Qodestackr.

@Kayere, @Qodestackr This tutorial will guide learners to create a URL shortener with JavaScript and HTML. The URL shortener won't need a database system to host it. Instead, we'll use jsonstore.io

Good topic @JudyNduati - approved. Make sure to link any related Section engineering education articles in your article.

@JudyNduati
Just wanted to follow up on this topic, as we will be clearing up the queue where possible.
Just and FYI I will be closing this topic form just to help clear up the queue where possible - can be REOPENED at anytime when ready. 👍

@hectorkambow, I opened a PR. Please re-open this issue


Civis util uploader

Switch out functions in upload-data.py to use the civis_aqueduct_utils package that we can install. Should be able to upload larger files.

👍


No matter what photo I select, it shows “photo is null”

When I run the nanodet in my android phone, it work well for camera. However,  no matter what photo I select, it shows “photo is null”. How to solve it?

Plz refer to this issue #2


docs: issue with mobile version for 'next'

Users visiting the docs homepage for version “next” on mobile experience a JS error called “unexpected token”. Users bounce off the site after this error, this problem will be reduced if the default is to use the latest supported version of docs (#6499) but it should still be possible to view next on mobile.

I can't seem to reproduce this issue?

I noticed that base docusaurus allows changing version from the sidebar in homepage. But I've removed the sidebar, so the only way to change versions is to change the URL at the moment or click the banner. I guess I can fix this instead

I'm closing this issue after discussion with @merkletreeibc since we have no way of reproducing this issue. It's not clear that this issue has actually occurred as described. However, I will be creating a new issue to allow users to switch versions on mobile


Autoscroll to last message received in DM view

Is your feature request related to a problem? Please describe.
When I open a DM view, the very first message between me and the other user is displayed. I think autoscrolling to the bottom/last message sent/received would make more sense.
Describe the solution you'd like
When I open a DM, it should show me the last message received, i.e. auto scroll to the bottom of the DMs.
Describe alternatives you've considered
N/A
Additional context
N/A

Solved by #250 , thanks for reporting!


Write syntax soft checks

Naming convention from the Supplement of Thiele & Palson's A protocol for generating a high-quality genome-scale metabolic reconstruction. 2010

Once the tests pass I will open a pull request and this is good to go.
I must add though that I couldn't implement all the Reaction/Metabolite suffix standards as outlined in the Supplement of [1]. I left out the tags for reversible, irreversible, Dehydrogenase and Synthetase reactions. The latter two because it is hard to do assert that in code and the former two because I don't fully understand the point. From observing the BiGG Database these are the least-consistent tags. I am open to add them in the future though.

[1 ]Thiele, I., & Palsson, B. Ø. (2010, January). A protocol for generating a high-quality genome-scale metabolic reconstruction. Nature protocols. Nature Publishing Group. http://doi.org/10.1038/nprot.2009.203


ttyd crashes on windows

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

run ttyd.win32.exe cmd
open browser and enter in the ip address:7681
ttyd exits with either "== CreateProcessW failed with error 267: The directory name is invalid." or just nothing at all

Expected behavior
Expected to see a command prompt in the browser when I connect to the port.
Screenshots
N / A
Environment:

OS: Windows 10 and Windows 11
Browser: Edge 121.0.2277.83 (Official build) (64-bit)

Additional context
I traced it with process monitor and see it tries to reference some weird directory with chinese characters for some odd reason.  I created this directory but it still does not fix the crash.  Something is causing the crash right when it tries to run the command.

It has nothing to do with ttyd, you may have other file named as cmd, trying using full path like:
ttyd.win32.exe C:\Windows\System32\cmd.exe

Nope, still the same issue.  I should have listed that in the report, but I figured you would look into it.
Just fyi, I'm not a noob who doesn't know if something is a bug or not.
If I post a report, trust me, it's a bug.
I go to great lengths to make sure its not really something I've done wrong somehow.

The error you encountered is returned from windows API, you may google it yourself.

Well I did more than that, I did a lot of testing against the current and older versions and found that version 1.7.3 works fine on both.  This report shows my exact issue.
https://github.com/tsl0922/ttyd/issues/1207
I would have found it if it had a better title.
Thank you for writing this software, I use it as a quick form of remote access and also works well when the customer is still using his machine.

i met the same problem @tech128
and i tried to set cwd like -w c:\ then it works ...


877 Vacancy listing details

Author checklist

[ ] Include primary ticket number in title - e.g. "#123 New styling for widget" - and any additional tickets in description
[ ] Fill in the details below and delete as appropriate
[ ] Be proactive in getting your work approved 💪

What's included?
This change adds the avertType to the exercise listing
Who should test?
✅ Product owner
✅ Developers
✅ UTG
How to test?
Change the advertType on the exercise on the admin, and check the changes on the apply side.
Risk - how likely is this to impact other areas?
🟠 Medium risk - this does change code that is shared with other areas

PREVIEW:DEVELOP

Can I get a preview URL please :)


Interlock and nginx containers must be co located with swarm manager/host with tls certs

Is it a requirement that the interlock and nginx containers must reside on the same host which contains either a swarm manager container, or a host which has the access to the tls certs for the host which dose have the swarm manager container.
That is how are you guys doing this in the examples, as I cannot get this to work unless I do co locate these containers, or locate them from the management node which has the tls certs (used docker-machine to launch other nodes), and all the tutorials completely gloss over this issue, which makes me wonder if I am doing it wrong.
Are you syncing the tls certs to the other hosts some how, vire rsync or ssh? so that we can mount the folder with the certs e.g.
-v /var/lib/boot2docker:/var/lib/boot2docker

Or do you always have to have a swarm management container around so you can mount the local docker socket
-v /var/run/docker.sock:/var/run/docker.sock

Also without using a volume driver like flocker or rexray how are you supposed to have the nginx and interlock containers on different hosts/ have multiple nginix proxies?

Is it a requirement that the interlock and nginx containers must reside on the same host which contains either a swarm manager container, or a host which has the access to the tls certs for the host which dose have the swarm manager container.

No.  You simply need to have access to Swarm.  This can be from a socket or TCP connection (most likely).  If you use TLS with Swarm, Interlock will need access to the certs as well.  You will need to specify the corresponding certs for the TLSCACert, TLSCert and TLSKey settings.  If you are not familiar with using TLS for Docker, see https://docs.docker.com/engine/security/https/.
Starting in the latest version 1.1.0 you do not even need to use a Docker Volume.  Interlock will locate all of the proxy containers (nginx/haproxy) and update accordingly.

Are you syncing the tls certs to the other hosts some how, vire rsync or ssh?

No we do not do anything with the certs.  If you are using TLS you will need to make sure Interlock has access to the appropriate certificates to access Swarm.  Since Interlock runs in a container you will need to mostly use a volume mount to expose the certificates to the Interlock container.

Or do you always have to have a swarm management container around so you can mount the local docker socket

You need access to the Docker API.  This can be from a socket or TCP.  It's up to you.  The examples show both ways.

Also without using a volume driver like flocker or rexray how are you supposed to have the nginx and interlock containers on different hosts/ have multiple nginix proxies?

Starting in 1.1.0, we use the attach and archive endpoints to inject the configuration to all of the proxy containers.  Thus it is not required to run the Interlock containers on the same hosts.  It is also no longer required to use volumes.

Hey thanks for the speedy reply

No we do not do anything with the certs. If you are using TLS you will need to make sure Interlock has access to the appropriate certificates to access Swarm. Since Interlock runs in a container you will need to mostly use a volume mount to expose the certificates to the Interlock container.

So normally how do you get those certificates onto the host which is running the interlock container?
I am instantiating my host nodes form a "management" node on aws using docker-machine. The certificates for each nodes docker engine therefore reside on this "management" node. Should I just generate some new certificates for the container to authenticate with the swarm master node, and copy those to a mountable location for the interlock container volume?

If you're setting up a swarm cluster using docker-machine, you can actually use the certs in /etc/docker to talk to your cluster as they're signed for client usage as well.  For example with an older version of Interlock:
docker run -d -p 80:80 -p 443:443 --restart=always \
    --name interlock \
    -v /etc/docker:/etc/docker:ro \
    -v /data/ssl:/ssl \
    -e NGINX_SSL_CERT_DIR=/ssl \
    ehazlett/interlock:0.3.3 \
    --swarm-url=tcp://$(docker-machine ip ${NAME_PREFIX}-master):3376 \
    --swarm-tls-ca-cert=/etc/docker/ca.pem \
    --swarm-tls-cert=/etc/docker/server.pem \
    --swarm-tls-key=/etc/docker/server-key.pem \
    --plugin nginx \
    start

You might want to verify the names of the certs in /etc/docker as for all I know, they've changed recently.

Right yeah, just figured out /etc/docker works by doing docker inspect swarm-manager. I then got it running thanks to the latest example .
Interestingly you need to use
configPath = "/etc/nginx/nginx.conf"
pidPath = "/tmp/nginx.pid"

For configuration else nginx falls over on start up before interlock can attach.
Thanks for the help.
As an aside is there any plans to add locations labels so we can proxy off the routes, instead of having to create an external dns entry for every service?
Ric

@Richard-Mathie You can use DNS wildcards.  I'm not what you mean by location labels as the browser sends a host header in order to route.  Unless you want paths on a common host.  Is that what you are looking for?

Yeah so what I was thinking is that there is now way of setting a location label such that you can route some locations to different services. something like this....
#/etc/nginx/nginx.conf
# ...
upstream host.domain {
      zone host.domain_backend 64k;
          server <IP_ADDRESS>:3000;
          server <IP_ADDRESS>:3000;
}
upstream service.host.domain {
      zone service.host.domain_backend 64k;
          server <IP_ADDRESS>:3000;
          server <IP_ADDRESS>:3000;
}
server {
      listen 80;

      server_name host.domain;

      location / {
          proxy_pass http://host.domain;
      }
      location /service {
           proxy_pass http://service.host.domain
      }
}

would that make scene?

Oh gotcha.  Yes you can do that today (just merged the PR)
You will want to use the following options:

interlock.context_root
interlock.context_root_rewrite

See https://github.com/ehazlett/interlock/blob/master/docs/interlock_data.md#context-root for details.
In your example above you should be able to add --label interlock.context_root=/service to any container that you want to serve under /service.
This will come in the 1.2 release but you can test now with the ehazlett/interlock:master image.

Thanks @ehazlett, we got the context_root working. We will keep experimenting and report back if we find any problems (or have more questions).

Cool thx. I'm goign to close this out.  Feel free to open another if you run into issues.

oops looks like the newest master has changed a bit, and seems to be misbehaving a bit. will have to pull and build the old version.

@Richard-Mathie Please file an issue if you notice something.  You can also use ehazlett/interlock:master to check.  That is always built from master.  The latest stable release is 1.1.0.


[Data Table] Overflow Menu not fully visible

Before submitting a bug:

[x] Is this reproducible in the latest version of Shidoka?
[x] Is this reproducible in the Shidoka Storybook?
[x] Is this reproducible in a Sandbox?
If it's not reproducible in any of the above scenarios, please do not open a bug report as it is an application-level issue.

Describe the bug
Overflow menu when opening upward not fully visible
To Reproduce
Steps to reproduce the behavior:
https://codesandbox.io/p/devbox/data-table-jj8dq2?file=%2Fsrc%2Fapp%2Fapp.component.ts%3A57%2C7

This is expected behavior due to absolute position within a scrolling container. The only alternative is to use the fixed position tooltip variant.


Wrong with "not implemented for multigraph type"

@gboeing , when i run OSMnx features demo, there is something wrong ,
when I run the code `
G = ox.graph_from_place('Piedmont, California, USA', network_type='drive')
fig, ax = ox.plot_graph(G)
........
edge_centrality = nx.closeness_centrality(nx.line_graph(G))
it appeared
NetworkXNotImplemented                    Traceback (most recent call last)
 in ()
2 # edge_centrality = nx.closeness_centrality(nx.line_graph(G))
3
----> 4 edge_centrality = nx.closeness_centrality(nx.line_graph(G))
 in line_graph(G, create_using)
D:\Anaconda3\lib\site-packages\networkx\utils\decorators.py in _not_implemented_for(not_implement_for_func, *args, **kwargs)
69         if match:
70             msg = 'not implemented for %s type' % ' '.join(graph_types)
---> 71             raise nx.NetworkXNotImplemented(msg)
72         else:
73             return not_implement_for_func(*args, **kwargs)
NetworkXNotImplemented: not implemented for multigraph type
`
How to deal with it？ thank you. ：）

networkx 2.2rc done

See also https://github.com/gboeing/osmnx-examples/issues/8

I'm having the same problem

How can this be resolved?


Add a rate limited client example to illustrate exponential backoff.

Tested with biotechtx.benchling.com with lowered rate limit threshold
setting.

Open Q: Do we need to add a README.md for this too?

I'm going to close this for now, and the dev-experience manager can revive it once they arrive.
Feel free to re-open if you disagree


Migrate to esbuild

Fixes: None
Description
The current build process leaves room for improvement, so the goal of this PR is to get rid of TSC as a transpiler and migrate to using esbuild. TSC will still be part of the pipeline and handle type checking.
Changes

Remove TSC transpile step
Use esbuild to transpile TS -> JS and bundle
Change exports in package.json to the latest standards

PR Tasks

[x] Open PR
[ ] run npm run test --> no failing unit tests
[ ] run npm run format

This PR is so much out of date that it will be easier to reopen.


Generate GOTK components manifest and upload to S3

To eliminate dependency on Github for Flux CRDs, we should generat ehe GitOps ToolKit CRD manifest matching the same version as the Flux version and upload it to S3. We can consume the same from the CLI.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: abhay-krishna, jiayiwang7
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [abhay-krishna,jiayiwang7]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Page "Item Tracing" (ID 6520) Trigger OnOpenPage possibility to overwrite TraceMethod and ShowComponents default value

We want to open up the "Item Tracing" page completly filtered and with own preset options. This is quite possible using the InitFilters function. But in the OnOpenPage Trigger the new values set up in the event OnAfterInitFilters for TraceMethod and ShowComponents are again overwritten by the default values of the page!
So best way would be to not set the values for TraceMethod and ShowComponents in OnOpen Page again, if the InitFilters procedure was already called or to give a possibility (new event publisher OnAfterOpenPage(var TraceMethod: Option Forward,Backward; var ShowComponents: Option No,"Item-tracked only",All)) in the OnOpenPage Trigger to overwrite the default Values again.
Kind regards,
Carsten Heu.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


No page after login

Hi Perez,
your AccountGo is really grate and help full, currently i want to use you application but i got stuck after login page i tried few time and reconfigure as per you given help but i can't success . i just got blank page with the URL below:
http://localhost:9921/account/signin?returnurl=/
please help me how to fix the error or how to get the error message on this stage?
thanks

Issue resolved .its db connection issue.


Revert "ACPI: remove ACPICA source from the kernel."

Reverts Harvey-OS/harvey#244

LGTM
that went badly, for me anyway.

someone else can push the merge. Not sure what's up but it wouldn't boot any more.

Working for me.


num2date for dates before the start of the Gregorian calendar

xref: pydata/xarray#1929
Using the current version of netCDF4.num2date the following works:
>>> from netCDF4 import num2date
>>> import numpy as np
>>> num2date(np.arange(2), 'days since 1000-01-01', 'gregorian')
array([datetime.datetime(1000, 1, 1, 0, 0),
       datetime.datetime(1000, 1, 2, 0, 0)], dtype=object)

However, using the current netcdftime.num2date, it does not:
>>> from netcdftime import num2date
>>> import numpy as np
>>> num2date(np.arange(2), 'days since 1000-01-01', 'gregorian')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "netcdftime/_netcdftime.pyx", line 283, in netcdftime._netcdftime.num2date
  File "netcdftime/_netcdftime.pyx", line 1165, in netcdftime._netcdftime.utime.num2date
  File "netcdftime/_netcdftime.pyx", line 673, in netcdftime._netcdftime.DateFromJulianDay
ValueError: The truth value of an array with more than one element is ambiguous. Use a.any() or a.all()

This error came up in the Xarray test suite (not in a real-world use-case).  In my reading, it seems the Gregorian calendar does not start until October 15th, 1582, so should this be possible (as it is in netCDF4) or should it not be?  In other words, in this case is Xarray testing something that doesn't make sense?  How does (or should?) netcdftime handle this situation?

Seems to be like this should not be allowed - the proleptic gregorian calendar should be used instead.  Don't know why it works with the netcdftime in netcdf4-python master.

on second though, negative years should be allowed - a fixed for this (ported from netcdf4-python) is included in pull request #29.

Awesome, thanks @jswhit.

closed via #29


Always expose D3 onto the environment global

With regards to #1689 adding AMD compatibility, it seemed odd to me that
D3 opt'd out of exposing itself globally for third-party plugins.
Within a discussion about this very problem, found here:
https://github.com/jakevdp/mpld3/issues/33#issuecomment-32101013
the argument is made that exposing the global defeats the point of
module systems.  I agree that module systems help avoid global
pollution, but that is not something I'd consider the point of module
systems, especially since they are registered in a global namespace
themselves.
This patch will fix third-party modules that are shimmed.

The new ES6 modules specification has almost no momentum

It's no longer the case.


The Variation buttons are not translated, and my translated catagory doesn't work

Hello,
I made a website in Dutch and English. Later I installed Woocommerce for the sale pages.
I Installed yyan woocommerce polylang integration to get my sales pages translated to English.
But the Variation texts (like: sizes), doesn't translate. And the translated catagory page doesn't work.
I hope someone can help me please.
https://custers-photography.com/en/product/mother-earth-stop-digging-en

use product attributes for translating the variations, see:
https://github.com/hyyan/woo-poly-integration/wiki/Product-Attributes
looks like you have already done it


[C++] Competitive (Codechef)

Thanks for contributing! 🎉
Please remove all the files except source files. (i.e. .cpp files)

ohh Sure

You're Welcome


Use class instead of data-id selector in controls

Fixes issue #2077. Uses class kirki-data-holder to select inputs within it's control. This change enables developers to dynamically add controls and settings to the customizer without losing access to the selector which was selected via data-id attribute.

I'm still a bit torn about this one and how to implement it.
The problem is not with what you're doing here, I get that and what you want to do is fine. But as I mentioned in the ticket, data-id was added because I'm slowly refactoring things to allow repeaters to be containers of controls instead of containers of <input> fields (I hope that sentense makes sense).
So right now there's a kirki.input object, a kirki.control object, some views etc. The data-id works the way it does because of this hypothetical future scenario:
A user uses options instead of theme-mods (don't ask me why, people are just weird sometimes).
Then they have a repeater.
Inside the repeater, there's a background control.
Background controls save values as an array like this:
[
	'background-color' => '#ffffff',
	'background-image' => 'https://domain.com/foo.png',
]

In the above scenario, data-id for the background color setting inside the background control inside an item in the repeater would look like this:
mysetting[my-repeater][0][my-background-control][background-color]

So there's the kirki.setting object that reads the data-id and sets the values. The kirki.setting.set method understands settings formatted like the one above and t will check the following:

Does the mysetting[my-repeater][0][my-background-control][background-color] exist in the customizer? No. I need to go back a level.
Does the mysetting[my-repeater][0][my-background-control] exist in the customizer? No. I need to go back a level.
Does the mysetting[my-repeater][0] exist in the customizer? No. I need to go back a level.
Does the mysetting[my-repeater] exist in the customizer? YES! Found it!
remove mysetting[my-repeater] from mysetting[my-repeater][0][my-background-control][background-color] and see what I'm left with.
I've got [0][my-background-control][background-color] so I need to save this:

0 => [
	'my-background-control' => [
		'background-color' => '#ffffff',
	]
]

So.... with the above in mind, how would this PR work?

Hi there,
thanks for the feedback. I get the problem. Did you already test your implementation? I guess the repeater option implies that elements are dynamically added to the DOM? E.g. when adding a new field within the repeater? That would create the same problem that I experienced with the data-id attribute as the controls within the new repeater field won't be initialized correctly.
Also I do not 100% get how the data-id attribute related to the initialization of the control? Is that really necessary? All I've changed within the PR is the way controls are selected for initialization - you may add the class kirki-data-holder (or whatever you might name it) to your repeater items (which I guess does only clone/copy markup) as well.
It is hard for me to understand what you are after without having a look at the actual implementation that's why maybe I'm getting it wrong :)
Cheers,
Dennis

It is hard for me to understand what you are after without having a look at the actual implementation that's why maybe I'm getting it wrong :)

Yeah, it's hard for me to explain it too... So far it's not implemented or tested anywhere, it's all just in my head brewing and mentally debugging.
I've merged this PR, we'll debug the repeater when everything else is finished and there's an actual implementation to test against.

@vendidero I'm afraid I had to revert this.
The problem is the use of control.container.find( 'input.kirki-data-holder' );
We can't rely on the control object...
Forget about repeaters. Think of a very simpler scenario: a background control, or even a typography control.
The typography control for example right now is a separate control, but once the migration is finished it will simply be a collection of select controls, checkboxes etc.
If we use control.container.find( 'input.kirki-data-holder' ); on the select, then it won't be able to properly detect the option.
Perhaps I could take a look at your theme and what it is you're trying to accomplish so we can come up with a solution together?

Hi there,
ok - this is the JS (excerpt) I'm currently using to "clone" existing settings/controls:
// Get the existing "placeholder" section and clone it
var newSectionParams      = $.extend( {}, section.params );
newSectionParams.id 	  = newBaseId;
newSectionParams.priority = 170;

var newSection 			  = new builder.vars.api.Section( newSectionParams.id, newSectionParams );

builder.vars.api.section.add( newSection );

// Add section to builder
builder.sections[ newSection.id ] = newSection;

// Register settings + controls by looping through existing settings and cloning them
$.each( elementType.settings, function( i, setting ) {
	var settingId		  = section.id + '_' + setting.id,
		customizerSetting = customizerSettings[ settingId ],
		newSettingId	  = newBaseId + '_' + setting.id,
		newSettingValue   = customizerSetting.value;

	var newSetting = new builder.vars.api.Setting( newSettingId, newSettingValue, {
		transport : customizerSetting.transport,
		type      : customizerSetting.type,
		previewer : builder.vars.api.previewer,
		dirty     : true
	});

	builder.vars.api.add( newSetting );

	// Send the setting to the preview to ensure it exists there.
	newSetting.previewer.send( 'setting', [ newSetting.id, newSetting() ] );

	// Add controls
	var control 		 = builder.vars.api.control( settingId ),
		newControlParams = $.extend( {}, control.params );

	newControlParams.id 			  = newSettingId;
	newControlParams.section  		  = newSection.params.id;
	newControlParams.content		  = null;
	newControlParams.settings		  = $.extend( {}, newControlParams.settings );
	newControlParams.settings.default = newSettingId;
	newControlParams.instanceNumber   = null;
	newControlParams.link			  = null;

	var Constructor = builder.vars.api.controlConstructor[ newControlParams.type ] || builder.vars.api.Control, options;
	options 		= _.extend( { params: newControlParams }, newControlParams ); // Inclusion of params alias is for back-compat for custom controls that expect to augment this property.
	var newControl  = new Constructor( newControlParams.id, options );

	builder.vars.api.control.add( newControl );
});

What I'm currently trying to do is clone existing sections/controls and add them to the customizer. The problem is, that the Kirki inputs won't fire the change events and won't detect any changes made to the cloned inputs (due to the usage of data-id). I hope this helps you to understand my problem :)
Cheers

Do you have a theme you could send me so I can see this in action later today?
Code is fine in most cases, but for more complicated issues like this one having something I can just install locally and play with is a lot more efficient 😊
You can find my mail on my GitHub profile

@vendidero the dev branch is not stable right now for production use, but it's stable enough to check if this one has been fixed already... I'm getting rid of the previous custom implementation and I'll be working on an alternate approach using the native customizer API. Now that WP has JS hooks it will be possible to handle the things that I was previously unable to do.


Remove package-lock.json (library, not application)

See https://github.com/sindresorhus/ama/issues/479#issuecomment-310661514 why this is important. Maybe you have some special reasons for it that I'm missing?

Coverage remained the same at 64.653% when pulling 4994b76364dd3f9f67aff05cfe589c907fa134c8 on ralphtheninja:master into 54a320de961d52fe47fd5022442376911733911c on solid:master.

It is important to us that bugs are exactly reproducible, and we achieve this with an exactly reproducible environment.
The reasoning in the comment you linked to is incomplete:

If you use a lockfile for packages, your local dependency tree will not match the dependency tree of users having your package as a dependency.

The authors seemingly wants to to imply that, if you don't have a lockfile, your local dependency will match the dependency tree of users. However, this is not true.
The truth is that your local dependency tree will always mismatch with the dependency tree of users. By having a lockfile, it at least exactly matches all of your fellow developers.
And by keeping the lockfile up to date, you can match new installs as well.

It is important to us that bugs are exactly reproducible, and we achieve this with an exactly reproducible environment.

I understand. I still think you're wrong though :smile: Since the package-lock.json file will be ignored when used as a dependency, users can (and will) have bugs but you can't reproduce them because of the package-lock file. Anyway, your call.

Since the package-lock.json file will be ignored when used as a dependency, users can (and will) have bugs but you can't reproduce them because of the package-lock file.

But removing package-lock.json does not make them reproducible either 😄
So I understand your issue with package-lock.json, removing it is just not a fix.
At least with package-lock.json, developers' issues are reproducible, removing it would make us lose this benefit without adding any.
If people have bugs that are due to specific dependencies, they need to report their package versions; and they need to do that regardless of whether package-lock.json is present.


Update dependency com.nimbusds:nimbus-jose-jwt to v8.22.1 [4.5.x]

This is addressing a critical vulnerability in net.minidev:json-smart:2.3.x which is used in com.nimbusds:nimbus-jose-jwt:8.20.1.
https://nvd.nist.gov/vuln/detail/CVE-2021-27568
https://github.com/advisories/GHSA-v528-7hrm-frqp
https://bitbucket.org/connect2id/nimbus-jose-jwt/commits/7b68f934e5b450666473e27592fb64694486fcde

Pac4J 5.0.x is not vulnerable because it's using com.nimbusds:nimbus-jose-jwt:9.8.1 which in turn is using net.minidev:json-smart:1.3.2 in which the vulnerability is already fixed.
https://github.com/pac4j/pac4j/blob/c7eeecbc323bfc3ca6889827165ca1156e44bd3c/pom.xml#L66
https://bitbucket.org/connect2id/nimbus-jose-jwt/commits/05fb055f61129933d83d17bedc013145997023aa

@leleuj Thanks for merging this so quickly! 😄
Is there any chance of getting a Pac4j 4.5.x release including this dependency update?
I know that we can override the transitive dependency in our build, but it would be nice to get rid of that workaround.

I don't plan any new release before September, but maybe more fixes/patches will come in...


Performance of core commands

Following up on Twitter thread.
 All examples below executed in VSC integrated terminal using Bash (installed via MINGW/Git Bash, not WSL).
$ sfdx -v
sfdx-cli/6.50.0-9817aece8a (windows-x64) node-v8.9.4

I am using the pre-release version but pinned to API 44.
$ time sfdx --version
sfdx-cli/6.50.0-9817aece8a (windows-x64) node-v8.9.4

real    0m3.729s
user    0m0.015s
sys     0m0.031s

$ time sfdx force:org:list
WARNING: apiVersion configuration overridden at 44.0
=== Orgs
     ALIAS         USERNAME                      ORG ID              CONNECTED STATUS
───  ────────────  ────────────────────────────  ──────────────────  ────────────────
<EMAIL_ADDRESS>       00D36000000YDboEAG  ENOENT
<EMAIL_ADDRESS>        00D1I000002yJUJUA2  invalid_grant
<EMAIL_ADDRESS> 00D6A000002I66oUAC  invalid_grant
<EMAIL_ADDRESS> 00Df4000003lKwxEAE  ENOTFOUND
<EMAIL_ADDRESS>   00DB0000000Y0oEMAS  Connected
(D<EMAIL_ADDRESS>             00D36000000YDboEAG  Connected
     DevHub<EMAIL_ADDRESS>  00D1N000002cKIQUA2  invalid_grant
     Prod_Deploy<EMAIL_ADDRESS>00D1U000000t6NvUAI  ENOENT
     newhub<EMAIL_ADDRESS> 00D0b000000DNDpEAO  ENOTFOUND
     reports-test<EMAIL_ADDRESS>         00D36000000YDboEAG  ENOENT

No active scratch orgs found. Specify --all to see all scratch orgs

real    0m16.785s
user    0m0.000s
sys     0m0.047s

I do have a lot of trial orgs registered here from before Dev Hub became available on Developer Editions. I removed all orgs with sfdx force:auth:logout -p -a and reran the time test:
$ time sfdx force:org:list
WARNING: apiVersion configuration overridden at 44.0
ERROR:  No orgs can be found.

Try this:
Use one of the commands in force:auth or force:org:create to add or create new scratch orgs.

real    0m12.437s
user    0m0.000s
sys     0m0.077s

Opening an org:
$ time sfdx force:org:open -u<EMAIL_ADDRESS>WARNING: apiVersion configuration overridden at 44.0
Access org 00D36000000YDboEAG as user<EMAIL_ADDRESS>with the following URL: https://david-ktema-dev-ed.my.salesforce.com/secur/frontdoor.jsp?sid=00D36000000YDbo!ARMAQPE7Xnq8.QeUSziUj3HXa3a8nnnExXwDwtGWd.LmoKLuFtH1wlkx_Pe9SJ3KPFSvhn2wAPOspLpdQNFLXO1MjvI52ppi

real    0m20.423s
user    0m0.000s
sys     0m0.078s

Happy to gather more timing and logging data if that is helpful.

@davidmreed Thank you for the feedback.
First off the org:list command is a known issue and we will be working on that in this quarter. It is probably worse on Windows but the performance is bad across the board
Second, for the rest of the commands, this is an issue with Windows that we are experiencing and looking into. Your logs are very helpful, thank you. A few questions:
Is there anything more you can tell me about the setup you have in terms of security?
If you run similar commands on a windows machine that is not on behind your companies firewall do you experience the same performance, better but still slow performance, or speed you expect?
Finally has this issue gotten worse or stayed the same over time?

@davidmreed thank you for the feedback.

@davidmreed a few updates we have updated the CLI to V7 and you should see performance improvements on commands, especially those only run on the client.
For org:list, in particular, we have introduced a new command auth:list (see the command reference guide here). This command lists the usernames you’ve authorized. Unlike org:list, this command doesn’t check whether orgs are still valid.
I am going to close this issue, but we are continuing to work on performance on an ongoing basis


doc update

Its honest work...

Codecov Report

Merging #15 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #15   +/-   ##
=======================================
  Coverage   96.77%   96.77%
=======================================
  Files           9        9
  Lines         775      775
=======================================
  Hits          750      750
  Misses         25       25

Impacted Files
Coverage Δ

src/mutation/structure.jl
98.07% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


update SpanPreparator and PipelineTrigger components

SpanPreparator
Current Behaviour

get the highest number of spans currently available (i.e 1)
plus 1 to get the next span number (i.e 2)
create tfrecord file from the list of raw data
store train and test tfrecord dataset to designated locations (cifar-10/span-2/....) where the ImportExampleGen consumes

Updated Behaviour
after the fourth step, we have to add two more steps.

remove the newly collected data from original GCS location. it will prevent Cloud Scheduler to trigger batch prediction pipeline. Because the triggering condition is number of newly collected image > threshold, if we don't remove them, Cloud Scheduler will trigger the batch prediction pipeline all the time even though they are already handled by the training pipeline.

add OutputArtifact[Dataset] to set the new span number. It will be passed to the downstream component, PipelineTrigger, then PipelineTrigger component will use it to set RuntimeParameter.

PipelineTrigger
Current Behaviour

simply trigger the training pipeline with create_run_from_job_spec function.

Updated Behaviour

add InputArtifact[Dataset] to get the span number from the previous component, SpanPreparator.
add parameter_values in the create_run_from_job_spec to pass RuntimeParameters to the training pipeline. An example can be found in this notebook. Or see the below

_ = pipelines_client.create_run_from_job_spec(PIPELINE_DEFINITION_FILE,
                                              enable_caching=False,
                                              parameter_values={
                                                  'input-config': json.dumps({
                                                        'splits': [
                                                          {'name': 'train', 'pattern': 'span-[12]/train/*.tfrecord'},
                                                          {'name': 'val', 'pattern': 'span-[12]/test/*.tfrecord'}
                                                        ]
                                                  }),
                                                  'output-config': json.dumps({})
                                              })

@deep-diver

store train and test tfrecord dataset to designated locations (cifar-10/span-2/....) where the ImportExampleGen consumes

Here, span-2 is just an example, right? Essentially, it should be the result of "plus 1 to get the next span number ". Correct?

Thank you. I will start working on modifying the span separator.

@deep-diver with our current setup, the following SpanSeparator should do what you specified in https://github.com/deep-diver/Continuous-Adaptation-for-Machine-Learning-System-to-Data-Changes/issues/9#issue-1028089643
def SpanPreparator(
    is_retrain: InputArtifact[Dataset],
    gcs_source_bucket: Parameter[str],
    gcs_destination_bucket: Parameter[str],
    latest_span_id: OutputArtifact[Dataset]
):
  if is_retrain.get_string_custom_property('result') == 'False':
    last_span_str = tf.io.gfile.glob(f"{gcs_destination_bucket}/span-*")[-1]
    last_span = int(last_span_str.split('-')[-1])
    new_span = last_span + 1

    timestamp = datetime.utcnow().strftime("%y%m%d-%H%M%S")

    image_paths = tf.io.gfile.glob(f"gs://{gcs_source_bucket}/*.jpg")
    logging.info(image_paths)
    random.shuffle(image_paths)

    val_split = 0.2

    split_index = int(len(image_paths) * (1 - val_split))
    training_paths = image_paths[:split_index]
    validation_paths = image_paths[split_index:]

    write_tfrecords(training_paths, gcs_destination_bucket, tfrecord_filename=f"new_training_data_{timestamp}.tfrecord", new_span=new_span, is_train=True)
    write_tfrecords(validation_paths, gcs_destination_bucket, tfrecord_filename=f"new_validation_data_{timestamp}.tfrecord", new_span=new_span, is_train=False)

    logging.info("Removing the images from batch prediction bucket.")
    os.system(f"gsutil rm -rf {gcs_source_bucket}/*")

    latest_span_id.set_string_custom_property("latest_span", str(new_span))

@sayakpaul
thanks Sayak,
one minor issue that I want to point out

In SpanPreparator component, I don't see codes to delete the data as described in this issue

remove the newly collected data from original GCS location. it will prevent Cloud Scheduler to trigger batch prediction pipeline. Because the triggering condition is number of newly collected image > threshold, if we don't remove them, Cloud Scheduler will trigger the batch prediction pipeline all the time even though they are already handled by the training pipeline.

SpanSeparator has it:
logging.info("Removing images from batch prediction bucket.")
os.system(f"gsutil rm -rf {gcs_source_bucket}/*")


why use use_layer_scale

thanks for your great contribution!
in the  implement for poolformerblock ,there is a layer_scale after token_mixer. What is the impact of this operation?

Hi @rtfgithub ,
Like stochastic depth, LayerScale can help train the models. For more details, please refer to the following that proposes this operator.
Going deeper with Image Transformers

thanks for your reply！I'd like to ask you another question.
As mentioned in your paper, the Training strategy of poolformer follow DEIT method. Did you  add the distillation token to model and use the distillation method with Hard-label distillation loss?

Hi @rtfgithub ,
We follow the training hyper-parameters of DeiT but we don't add distillation methods.

thank u for your reply!Good luck！

You are welcome :)


Cannot login

Describe the bug
The bot cannot log into Steam.
Log
1|Items  | 2020-07-01 01:16:25 debug: Got pricelist
1|Items  | 2020-07-01 01:16:25 info: Signing in to Steam...
1|Items  | 2020-07-01 01:16:25 debug: Starting login attempt {"loginKey":"xxxxxxxxxxxxxx+7A","private":true}
1|Items  | 2020-07-01 01:16:25 debug: Pricelist changed
1|Items  | 2020-07-01 01:16:25 debug: Signing in using login key
1|Items  | 2020-07-01 01:16:26 info: Signed in to Steam!
1|Items  | 2020-07-01 01:16:26 debug: Waiting for web session
1|Items  | 2020-07-01 01:16:27 debug: New login key
1|Items  | 2020-07-01 01:16:27 debug: New web session
1|Items  | 2020-07-01 01:16:27 info: Initializing bptf-listings...
1|Items  | 2020-07-01 01:16:27 debug: Getting inventory...
1|Items  | 2020-07-01 01:16:27 debug: Initializing bptf-listings...
1|Items  | 2020-07-01 01:16:27 debug: Updating profile settings...
1|Items  | Error: Malformed response
1|Items  |     at SteamCommunity. (/root/tf2-automatic/node_modules/steamcommunity/components/profile.js:166:14)
1|Items  |     at Request._callback (/root/tf2-automatic/node_modules/steamcommunity/components/http.js:67:15)
1|Items  |     at Request.self.callback (/root/tf2-automatic/node_modules/request/request.js:185:22)
1|Items  |     at Request.emit (events.js:321:20)
1|Items  |     at Request. (/root/tf2-automatic/node_modules/request/request.js:1161:10)
1|Items  |     at Request.emit (events.js:321:20)
1|Items  |     at Gunzip. (/root/tf2-automatic/node_modules/request/request.js:1083:12)
1|Items  |     at Object.onceWrapper (events.js:427:28)
1|Items  |     at Gunzip.emit (events.js:333:22)
1|Items  |     at endReadableNT (_stream_readable.js:1201:12)
1|Items  |     at processTicksAndRejections (internal/process/task_queues.js:84:21)
1|Items  | 2020-07-01 01:16:29 error: tf2-automatic failed to start properly, this is most likely a temporary error. See the log:
1|Items  | package.version: 3.5.2; node: v13.9.0 linux x64}
1|Items  | Stack trace:
1|Items  | Error: Malformed response
1|Items  |     at SteamCommunity. (/root/tf2-automatic/node_modules/steamcommunity/components/profile.js:166:14)
1|Items  |     at Request._callback (/root/tf2-automatic/node_modules/steamcommunity/components/http.js:67:15)
1|Items  |     at Request.self.callback (/root/tf2-automatic/node_modules/request/request.js:185:22)
1|Items  |     at Request.emit (events.js:321:20)
1|Items  |     at Request. (/root/tf2-automatic/node_modules/request/request.js:1161:10)
1|Items  |     at Request.emit (events.js:321:20)
1|Items  |     at Gunzip. (/root/tf2-automatic/node_modules/request/request.js:1083:12)
1|Items  |     at Object.onceWrapper (events.js:427:28)
1|Items  |     at Gunzip.emit (events.js:333:22)
1|Items  |     at endReadableNT (_stream_readable.js:1201:12)
1|Items  |     at processTicksAndRejections (internal/process/task_queues.js:84:21)
1|Items  | 2020-07-01 01:16:29 debug: Shutdown has been initialized, stopping... {"err":{}}
1|Items  | 2020-07-01 01:16:29 debug: Handler finished cleaning up
1|Items  | 2020-07-01 01:16:29 debug: Waiting for files to be saved
1|Items  | 2020-07-01 01:16:29 debug: Done waiting for files
1|Items  | 2020-07-01 01:16:29 warn: Exiting...
version: 3.5.2

You need to update

Isn't 3.5.2 the latest version? I tried the wiki method to update and it said it is up to date

If this issue is still happening, its because steam changed their edit profile settings.
Running npm install steamcommunity --save will fix this


Flickering webms

Bug description
Some webms show nothing but a screen that flickers black and white when opened. I can see the webm's thumbnail, and can view the webm with no problem when downloading it and opening it in Windows 10's media player. On the log tab, a blinking webm creates several lines of whitespace.
Steps to reproduce
Some webms flicker, while others play as normal. The only difference I can find is that any webm with a height of 1080 works fine, while it is a toss up between flickering and normal for any other resolution. A normal webm can have the same resolution (eg. 1280x720) as a flickering image. Of the boorus I have tried, realbooru, e621, and rule34.paheal have this problem, while danbooru, gelbooru, and rule34.xxx do not.
Expected behavior
When opening a webm in grabber, a webm should play as normal.
Context

My main.log file: main.log
My settings (needed to convert it into a .txt file because github says it does not support .ini files): settings.txt

Screenshots

System information

OS: Windows 10
Grabber version: 7.71

I did! It's discussed on the bottom of this page on RB's bugs/issues thread. The API misplaces a forward slash. Until they fix this, I'm not really sure what to do.


Fix button (ui-next) -> className

Context
This PR fix the ClassName for the button in ui-next.

Changes & Results

Testing
create a Button and add a className

Checklist
PR

[x] My Pull Request title is descriptive, accurate and follows the
semantic-release format and guidelines.

Code

[x] My code has been well-documented (function documentation, inline comments,
etc.)

Public Documentation Updates

[x] The documentation page has been updated as necessary for any public API
additions or removals.

Tested Environment

[x] OS: Windows 11
[x] Node version: 20.16.0
[x] Browser: Chrome

Cecillia is from Pixilib Team


Fix incorrect authenticate method in README

Just fixing the README in regards to authenticating presence channels. Wrong method name was listed.

Ah thanks! I don't know how this one slipped through the net! Cheers :D


DEV-14733 - Add support for instance fleets

This PR doesn't appear to be linked to a DevOps/SRE jira ticket

Added support for instance fleets by changing resource from instance groups, with a mix of on-demand and spot instances.

This PR doesn't appear to be linked to a DevOps/SRE jira ticket

This PR doesn't appear to be linked to a DevOps/SRE jira ticket

This PR doesn't appear to be linked to a DevOps/SRE jira ticket

This PR doesn't appear to be linked to a DevOps/SRE jira ticket

This PR doesn't appear to be linked to a DevOps/SRE jira ticket

This PR doesn't appear to be linked to a DevOps/SRE jira ticket

This PR doesn't appear to be linked to a DevOps/SRE jira ticket

Updated to make most values configurable
Updated version and changelog

This PR doesn't appear to be linked to a DevOps/SRE jira ticket

This PR doesn't appear to be linked to a DevOps/SRE jira ticket

This PR doesn't appear to be linked to a DevOps/SRE jira ticket

This PR doesn't appear to be linked to a DevOps/SRE jira ticket

This PR doesn't appear to be linked to a DevOps/SRE jira ticket

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


Hypothesis testing

Could you list the tests which are yet to be implemented?
I have some tests implemented in pareto, maybe some of that could be useful for owl as well.

Hi Sergei, that would be awesome! I currently implemented: z-test, t-test (one sample, paried, unpaired), Jarque-Bera test, Chi-square variance test, and runs test.
I still need (you can certainly add more :) -- f-test; Kolmogorov-Smirnov test; Anderson-Darling test; Durbin-Watson test; Fisher's exact test; Lilliefors test; sign test, and so on. I saw several have been implemented already in Pareto!
Actually, if you have a look at this page: Matlab hypothesis test, there are distribution, location, and dispersion tests (with detailed functions listed if you click each category). My goal is to cover all of them in the end.
Another thing: currently, the return of a test is a 3-tuple: (reject or not, p-value, score). In the end, the return should also include confidence interval, and return in a record type (type test_result which is actually already int the code if you look at owl_stats.ml).

Hi Ryan, I am happy to say we are (finally) going to tackle this with @bagmanas in the following couple of month.
I would like to come up with an acceptance criteria for the implemented tests. Should we cross-check against R, Matlab/Octave or SciPy? My previous experience suggest that all three could differ in the way the compute things. If that is the case, which side should we take? By the way, I've looked through the tests already implemented in Owl and it seems there're no unit tests for them. Should we write some for the new tests?

Hi Sergei, that's great if you and  @bagmanas can help in stats module.
For cross-validate, I often use Matlab and R. But I think either is fine as long as you specify that in the (in-line) document so that others can know which has been tested against. You probably know R and Matlab better than me so you can decide.
For API design, I am using a mixture of all three at the moment in Owl (try to get the best of all worlds). I think the API in your Pareto library is very nice so I am already happy if you just use the similar style. Of course, if you have even better design, always welcome :)
You are right. I did not write any unit tests for Stats module at the moment, mostly because the module is far from complete. But after you reshape the current hypothesis tests and put more tests, we should start writing unit tests.


Fix/ convert spatialUnits correctly in updateTrajectoryFileInfoFormat

Problem
Scale bar labels for v1 trajectories look like this:

Solution
I was completely overthinking / mistaken about this before. Sorry about all the confusion during the last 2 meetings. Like @blairlyons was saying all along it's a really simple fix that just requires me to properly convert spatialUnitFactorMeters to spatialUnits during the conversion in updateTrajectoryFileInfoFormat().
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[x] This change requires updated or new tests

Steps to Verify:
Install this branch as a dependency for your local simularium-website and load some v1 trajectory files

in viewer, run npm run build
in website, delete the viewer from node-modules and put this in package.json "@aics/simularium-viewer": "file:../simularium-viewer",
Add path.resolve(__dirname, "../", "../simularium-viewer/style") as one of the CSS rules in webpack.config.js in website
in website, run npm i then npm start

Keyfiles (delete if not relevant):

src/simularium/versionHandlers.ts - the fix
src/viewport/index.tsx - renamed newMsg as trajectoryFileInfo per @meganrm 's suggestion

Thanks for contributing!

Looks good! Does the unit test modification represent properly handling the "bad" behavior?

Yeah, it does! In simularium-website we're multiplying tickLengthInterval by spatialUnits.magnitude, which is usually like 1e-6 currently (as you can see in the test). So that creates a really small number for the scale bar label. We might still have to round the number in simularium-website, but that's no big deal


Circuit needs a Group property

One of the properties in the Hystrix Data is "Group",  https://github.com/Netflix/Hystrix/tree/master/hystrix-contrib/hystrix-metrics-event-stream
We can probably default this to be the same as the name property if nothing is passed in

This can be closed, can't it?

Yup,  just looking for the commit.
closed via 3052f23af43c98a622bf51b25edcef969d9b39c0


Mat/Pat - Partner info in supervisor approval is  not populating

Partner info not populating in the supervisor approval review page even though the fields were filled in the applicant form. And it was showing a validation error that the fields are not filled when the supervisor tried to submit the form.
Private Zenhub Image
AC

[ ] Partner info should populate for supervisor and supervisor should be able to approve the form without the validation error that the field is mandatory

Private Zenhub Image

Can confirm that the error is now fixed! I was not able to reproduce the error.

looks good
Private Zenhub Image


Add HOPE Junocw20

Add HOPE Junocw20

When PR Merge?

@DADDY-XEP  If possible, please use SVG over PNG.

has already been updated.  Thank you

Where has it been updated?

This is still a PNG format?

Could you please make a commit and replace this file with an SVG version? Thank you!

Could you please make a commit and replace this file with an SVG version? Thank you!

it uses svg now

Please start the merging. Thank you

@DADDY-XEP is there a pool for this?

Hi yes, pool 653 is live
https://app.osmosis.zone/pools?page=9


SC changed to SCIP_CALL

Small change to syntax to be more in line with SCIP.

Looks good. As it was an internal change I wouldn't consider this as breaking