Add support for non-standard google identity DeviceAuthorizationResponse verification_url

What happened?
Unfortunately the google identity DeviceAuthorizationResponse responds with a non-standard response:
{
  device_code: "AH...9Q",
  user_code: "NCL-...-ZQK",
  expires_in: 1800,
  interval: 5,
  verification_url: "https://www.google.com/device"
}

As you can see, it returns verification_url instead of verification_uri (https://datatracker.ietf.org/doc/html/rfc8628#section-3.2).
Can this library add support for that quirk (e.g. add support for verification_uri and verification_url)?
There is a google issue tracking this problem since 2020 at https://issuetracker.google.com/issues/151238144... but it seems no effort was made to fix it, hence, it needs a workaround?
Version
3.1.4
Runtime
Deno
Runtime Details
2.1.4
Code to reproduce
// NB this is was copied from https://github.com/panva/oauth4webapi/blob/main/examples/device_authorization_grant.ts with little changes to use google identity details.

import * as oauth from "jsr:@panva/oauth4webapi";

// Prerequisites
let issuer!: URL // Authorization server's Issuer Identifier URL
let algorithm!:
  | 'oauth2' /* For .well-known/oauth-authorization-server discovery */
  | 'oidc' /* For .well-known/openid-configuration discovery */
  | undefined /* Defaults to 'oidc' */
let client_id!: string
// End of prerequisites

// see https://developers.google.com/identity/openid-connect/openid-connect#discovery
issuer = new URL("https://accounts.google.com");
algorithm = 'oidc';

// see https://console.cloud.google.com/apis/credentials?project=example-444918
client_id = "55....apps.googleusercontent.com";
const client_secret = "GO...MR";

const as = await oauth
  .discoveryRequest(issuer, { algorithm })
  .then((response) => oauth.processDiscoveryResponse(issuer, response))

const client: oauth.Client = { client_id }
const clientAuth = oauth.None()
// const clientAuth = oauth.ClientSecretPost(client_secret);

let device_code: string
let interval: number
let verification_uri: string
let user_code: string
let verification_uri_complete: string | undefined

// Device Authorization Request & Response
{
  const parameters = new URLSearchParams()
//   parameters.set('scope', 'openid https://www.googleapis.com/auth/photoslibrary.readonly')
  parameters.set('scope', 'openid')

  const response = await oauth.deviceAuthorizationRequest(as, client, clientAuth, parameters)

  const result = await oauth.processDeviceAuthorizationResponse(as, client, response)

  console.log('Device Authorization Response', result)
  ;({ device_code, verification_uri, verification_uri_complete, user_code, interval = 5 } = result)
}

Required

[X] I have searched the issues tracker and discussions for similar topics and couldn't find anything related.
[X] I agree to follow this project's Code of Conduct

If they include non-standard response parameters you can still access them on the response object. It just won't be typed.

Indeed, I've worked around this with this ugly implementation:
  const originalResponseJson = response.json;

  response.json = async () => {
    const j = await originalResponseJson.call(response);
    j.verification_uri = j.verification_url;
    return j;
  };

But it turns out, I have to give up upon the google device authorization grant, as it cannot really access the google photos scopes (my goal is to use the google photos api), so I had to abandon this route and I'm trying to use https://github.com/panva/oauth4webapi/blob/main/examples/oidc.ts instead.
So maybe this issue can be closed? As even implementing this correctly, only give us a severely limited access to google apis scopes.

No, that's not what i meant.
const result = await oauth.processDeviceAuthorizationResponse(as, client, response)
console.log(result) <- it has whatever the JSON response had.

That will not work, and I'm now seeing that I did not include the error description at all... sorry about that! Without the verification_uri property the oauth.processDeviceAuthorizationResponse call fails at:
assertString(
    json.verification_uri,
    '"response" body "verification_uri" property',
    INVALID_RESPONSE,
    { body: json },
  )

Ah right, because it's required. Yeah, no workaround other than consuming the Response instance .json()


Timezone of cron schedule is not supported in 3.16.8

I tried to upgrade to 3.16.8 but blocked by interface change.
It seems that this function is not supported:
cronSchedule.getExpression().setTimeZone()

Fixed! Thanks for report

Is this issue fixed?

timezon isn't a part of CronExpression anymore. Here is how you can apply it: org.redisson.api.CronSchedule#of(java.lang.String, java.time.ZoneId)


Added Custom Databricks Policies

This PR fixes

Added Custom Databricks Policies

Azure Databricks

Policy Name
Policy Area
Description

Append-Databricks-PublicIp
Network Isolation
Enforces no public access on Databricks workspaces.

Deny-Databricks-Sku
Resource Management
Deny non-premium Databricks sku.

Deny-Databricks-VirtualNetwork
Network Isolation
Deny non-virtual network deployment for databricks.

@krnese I would enforce the following on the Corp Management Group:

Deny-Databricks-PublicIp
Deny-Databricks-VirtualNetwork


[lmdeploy] addsiluffn

Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation and Context
externdiopifunc for lmdeploy only
Description
externdiopifunc for lmdeploy only
Use cases (Optional)
externdiopifunc for lmdeploy only.
BC-breaking (Optional)

Checklist
Before PR:

[ ] I have read and followed the workflow indicated in the Contributors.md to create this PR.
[ ] Pre-commit or linting tools indicated in Contributors.md are used to fix the potential lint issues.
[ ] Bug fixes are covered by unit tests, the case that causes the bug should be added in the unit tests.
[ ] New functionalities are covered by complete unit tests. If not, please add more unit test to ensure the correctness.
[ ] The documentation has been modified accordingly, including docstring or example tutorials.

After PR:

[ ] CLA has been signed and all committers have signed the CLA in this PR.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Corrected the wrong label to the right one 'Decrement'

Updated the wrongly showing label for Decrement from Increment

Already closed with
https://github.com/egoist/vuepack/commit/a6723d198dd22fb9d9a649a10023b7e5d27fe935


Replace C++17 hex-float literal with bitshifts

Thank you for the nice library!
This PR enables compilation with C++11 compiler(tested with clang++ -std=c++11).
Also, added more tests(disabled by default). On my local run I've confirmed the test passes.

@jeaiii Thanks!


Add a checker if indices might overflow UINT32_MAX

69920
may just pick one between this one and https://github.com/bytecodealliance/wasm-micro-runtime/pull/3577

LGTM

can you do the same for other resources as well?
memories, tables, tags.

Sure. But since wamr haven't support multi-tables and multi-memories yet, I'll patch tag only.


Adjust blog entries snippet colors

change the pink highlighted detail of words in b.s to a white. And the pink underling to more of a burgendy red. More subtle but noticeable againts the default silver color

The way it works right now, the blog entries are just using the default style from the rest of the text styling and you'll notice it's using the same red color highlight that the rest of the site uses to display highlighted text.
For the blog entries list snippets, I could just remove all styling if that would look better? Bold text is already colored in white, so that wouldn't make them distinct.

811e45ed120f3bbbc2dad2c87f7620c655c40b25 adds a proper summary property for blog entries, which doesn't allow any markup.


Implement token deployer contract in create fuels template

The current create fuels template uses a simple counter contract. We should migrate to something that is a more real life use case for consumers. This could be a token deployer but we could also take inspiration from other sway applications.
The final solution will implement the new contract and modify the template apps to interact with the contract.
Ideally, we'd also improve the predicate and script examples to something conceptually similar.

After discussing with @Dhaiwat10 and @rishabhkeshan we're going to think about this one some more. We will keep the current counter contract in place as the tool is primarily aimed at onboarding new devs to fuel.
However we can continue to have discussions around a template extension that uses a more advanced use case.
Closing as unplanned for now whilst we have these discussions with Dev Rel.

@Dhaiwat10 feel free to reopen this if new conclusions are made after speaking with Dev Rel.


Do not print coverage warnings if the coverage was disabled

Using coverage: false but still see Node terminal warnings

Still happening

ok, works in 2.6.1


contrib/apparmor: remove version-dependent rules

equivalent of https://github.com/moby/moby/pull/44960 in moby

These conditions were added in docker in https://github.com/moby/moby/commit/8cf89245f5b5f9abb066f599cb69bfe0202bae5d to account for old versions of debian/ubuntu (apparmor_parser < 2.8.95) that lacked some options;

This allows us to use the apparmor profile we have in contrib/apparmor/
and solves the problems where certain functions are not apparent on older
versions of apparmor_parser on debian/ubuntu.

Those patches were from 2015/2016, and all currently supported distro versions should now have more current versions than that. Looking at the oldest supported versions;
Ubuntu 18.04 "Bionic":
apparmor_parser --version
AppArmor parser version 2.12
Copyright (C) 1999-2008 Novell Inc.
Copyright 2009-2012 Canonical Ltd.

Debian 10 "Buster"
apparmor_parser --version
AppArmor parser version 2.13.2
Copyright (C) 1999-2008 Novell Inc.
Copyright 2009-2018 Canonical Ltd.

This patch removes the version-dependent rules.
Signed-off-by: Sebastiaan van Stijn<EMAIL_ADDRESS>

Needs rebase on main for Windows CI fixes?

Ah, thanks for the ping rebased 👍
FWIW; I also have a follow-up to this one in https://github.com/containerd/containerd/pull/8069. I decided to keep them separate (the more trivial bits first in this PR).


Add icons for build and launch targets

When using a setting "cmake.statusbar.visibility": "icon", there is no way to change build and launch targets using the status bar. As variants and kits lists are accessible through their dedicated icons, it would make sense to be able to select build and launch targets using an icon.

Thank you for your feature idea. We may not be able to implement it immediately, but we will track the community reactions to determine where this might fit into our backlog.

Can you try this combination of settings?  This appears to do what you are describing.
    "cmake.statusbar.visibility": "icon",
    "cmake.statusbar.advanced": {
        "buildTarget": {
            "visibility": "default"
        },
        "launchTarget": {
            "visibility": "default"
        }
    },

@bobbrow I've opened this issue as a result of discussion in #1005. For me personally, your solution works fine, but I still decided to open this issue, because I feel like those two things deserve their own icons (kit selection has its own icon), and your solution is more like a workaround for the lack of dedicated icons.
IMO, when someone sets cmake.statusbar.visibility": "icon" it should just work, and cmake.statusbar.advanced" should be there purely for customisability, not for workaround. Just my two cents.

When the feature was added, we didn't think there was any icon that made sense for these two items.  The value is in the text and in icon mode you'd have to hover over the icon to see which target was active.  Instead we put that text in the respective build/launch icon tooltips.
But the feedback is noted.  Perhaps we could add a new hybrid "visibility" for cmake.statusbar.visibility to make this easier.  Something like "iconsAndTargets" which would do what the current combination of icons and advanced settings does above.

Makes sense. What about kits though? It has an icon and serves as selection from the list. So you would have to hover over it to see the current one, like you described.

The non-action icons were made clickable as a convenience to change the state that was tied to it.  They behave the same as the action icons in the sense that they show the state in the tooltip when the text is not displayed in the status bar.


[refactor] combine operator API changes

Overview:
depends on:

[x] https://github.com/apache/pinot/pull/10058

this is one of the 2-part changes to make combine & instance-resonse operators more pluggable.

[ ] Part 1: refactor on combine operator API (#10006)
[ ] Part 2: refactor on instance response operator API and planMaker ("#xxxxx")

Goal
The goal of this change is to easily plug stream combine operators into the mix, this is useful for both GRPCServer as well as the leaf-stage operator chain in multistage engine.
API changes
There's 2 part of the API changes

CombineOperator
Splitting the BaseCombineOperator into:

BaseCombineOperator

BaseSingleBlockCombineOperator

[Aggregate/Distinct/Selection]CombineOperator that uses standard ResultBlockMerger API
[GroupBy/MinMax]SingleBlockCombineOperator that DOES NOT
BaseStreamingCombineOperator

[Selection/Aggregate/Distinct]StreamingOperator
GroupByStreamingOperator

ResultBlockMerger

[Selection/Aggregate/Distinct]ResultBlockMerger APIs

NOTE: Group-By does not use centralized merged, instead uses specialized mergeResults method.
Details

BaseCombineOperator implements the parallel task dispatch and future reduce mechanism
BaseSingleBlockCombineOperator merges all blocks into 1 before returning; using CombineFunction with a standard implementation of processSemgents() and mergeResults() method

[Aggregate/Distinct/Selection]CombineOperator only need to pass in corresponding CombineFunction impl. no override necessary.
[GroupBy/MinMax]SingleBlockCombineOperator doesn't use linked blocking queue to communicate between the parallel tasks and the main thread. Thus they have their own implementations extending BaseSingleBlockCombineOperator, but use their own mergeResults() and processSegments() methods.

BaseStreamCombineOperator will be returning multiple blocks until the getNextBlock() returns a metadata-only block which indicates all tasks have finished processing or the early-termination condition has been reached.

this also means we have to refactor the StreamingInstanceResponseOperator to invoke StreamCombineOperator.getNextblock() multiple times until the final MetadataResultsBlock is returned.
this also works well with the multi-stage engine's MailboxSendOperator, which also invokes getNextBlock() multiple times and sends data one block at a time.
[TODO] because of this change, we have to refactor the InstancePlamMaker implementation to treat "streaming" plan properly via tag, no based on whether the streamObserver is present.

Codecov Report

Merging #10006 (3596bc2) into master (54046e1) will decrease coverage by 56.79%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             master   #10006       +/-   ##
=============================================
- Coverage     70.41%   13.62%   -56.80%
+ Complexity     5680      176     -5504
=============================================
  Files          1991     1942       -49
  Lines        107286   104864     -2422
  Branches      16315    16026      -289
=============================================
- Hits          75542    14284    -61258
- Misses        26464    89455    +62991
+ Partials       5280     1125     -4155

Flag
Coverage Δ

integration1
?

integration2
?

unittests1
?

unittests2
13.62% <0.00%> (+<0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...e/operator/combine/AggregationCombineOperator.java
0.00% <0.00%> (-83.34%)
:arrow_down:

...not/core/operator/combine/BaseCombineOperator.java
0.00% <0.00%> (-85.72%)
:arrow_down:

...erator/combine/BaseSingleBlockCombineOperator.java
0.00% <0.00%> (ø)

...core/operator/combine/DistinctCombineOperator.java
0.00% <0.00%> (-90.00%)
:arrow_down:

.../core/operator/combine/GroupByCombineOperator.java
0.00% <ø> (-84.38%)
:arrow_down:

...nMaxValueBasedSelectionOrderByCombineOperator.java
0.00% <0.00%> (-75.18%)
:arrow_down:

...operator/combine/SelectionOnlyCombineOperator.java
0.00% <0.00%> (-62.97%)
:arrow_down:

...rator/combine/SelectionOrderByCombineOperator.java
0.00% <0.00%> (-65.63%)
:arrow_down:

...tor/combine/function/AggregateCombineFunction.java
0.00% <0.00%> (ø)

...ore/operator/combine/function/CombineFunction.java
0.00% <0.00%> (ø)

... and 1577 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

@siddharthteotia please kindly take a look

@Jackie-Jiang @siddharthteotia any objection on merging this one and start a follow up on the remaining items.

@Jackie-Jiang @siddharthteotia any objection on merging this one and start a follow up on the remaining items.

Good from my side.


Layout for the Keccak witness

Closes https://github.com/o1-labs/proof-systems/issues/1210 https://github.com/o1-labs/proof-systems/issues/1211 https://github.com/o1-labs/proof-systems/issues/1212 https://github.com/o1-labs/proof-systems/issues/1213

LGTM. Does follow the RFC. Only shifts -> reset.

fixed the notation in the RFC so the names match 👍🏻


(scaling branch) Bug: Error thrown finding ipfs repo which crashes node

Error: ENOENT: no such file or directory, lstat '/jsipfs'
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>start: node ./src/node.js
Doesn't happen all of the time, must be some race condition, must not be initializing s3 modules entirely before calls are made against it. Seems to only happen on start, but would have to confirm.

Looked through additional logs, does not only happen on starts, fails randomly at later points as well.

Oh just the disk usage measurement metric! Not a bug, but leaving story as reminder to change some measurements, and remove ones that don't make sense anymore.

Heh, was worried there for a while 😆

Replaced with - https://github.com/3box/3box-pinning-node/issues/113
Closing now


Redfish Set DateTime TC: Using REST API to set to Manual / BMC.

@susilsi7 please talk to @sivassrr and get this done.
Test    Verify_Set_Time_Using_Redfish

@susilsi7 Thank you Sushil for looking into it. Pls take a look at the code which I had shared.
Also in our code - We need to add "To Do:  Need to remove REST calls once Redfish API available" from Redfish/managers code as in located at https://github.com/openbmc/openbmc-test-automation/blob/master/redfish/managers/test_managers_bmc_time.robot

Gerrit Link: https://gerrit.openbmc-project.xyz/#/c/openbmc/openbmc-test-automation/+/22701/


Fix rebase conflict detection in git 2.26

Git changed the technique used in rebase from simple apply-patches to use merge. This breaks our conflict detection code.
Fix #10928
Signed-off-by: Andrew Thornton<EMAIL_ADDRESS>

need backport

@zeripath please send second patch ;)

@6543 So I think the second commit was sent by the time you commented.


Center Player horizontal

Hi.
I set to 720p with autoresolution and the player size is "fit to content". So after loading the page and the video starts, the player scale up to 720, but after that the player isn't in the center of the screen (like on the screenshot below).
Can you set the css to center the player after resizing?
http://www.xup.in/dl,16063814/Bildschirmfoto_vom_2015-01-25_14:52:09.png

Could you take a screenshot of the "Fit to content" settings you have? Capture it like this:

Also the current YTC version you are using and browser as well are always helpful information.

I'm having the same issue :

Here are my settings :

I'm using Firefox 35.0 and YTC 2.1.7 (which I got from here, since Firefox add-on page is stuck to 2.1.0).

UI -> Dark player background, uncheck, problem fixed...
I hope there will be new stable release soon, that will fix all the bugs (also official firefox addon page still has 2.1.0 instead of 2.1.7)

Thanks for the fix !

@grantlee77 The settings you showed are not the settings of the "Fit to content" option, you showed the settings for the "80%" which is not what you described in your original post.
Regardless, I have tested the same settings your screenshot shows and the player displays correctly with the dev build 470 with no need to remove the dark player background.
You can download the latest developer build here: https://github.com/YePpHa/YouTubeCenter/wiki/Developer-Version
@Tiuqu read https://github.com/YePpHa/YouTubeCenter/issues/1325#issuecomment-68047353

I am not the original poster. I thought my problem was similar to him.
Apparently it is not, my apologies for putting a bit of misunderstanding here.
Thank you for taking time to answer me.

Apologies, I thought you were the original poster. But still, the suggestion remains valid for anyone with this problem, try the developer version because I suspect this problem has been fixed already.

I have a similar Issue: When I enable Player size: enable Resize everything works fine, but when I disable it, the Player is too far on the right.


minecraft:queued_ticking block reference page is not listed in the TOC

minecraft:queued_ticking block reference page is not listed in the TOC.
https://learn.microsoft.com/en-us/minecraft/creator/reference/content/blockreference/examples/blocktriggers/minecraftblock_queued_ticking

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

ID: e3b91464-040d-48a2-28fc-300a16cce125
Version Independent ID: e3b91464-040d-48a2-28fc-300a16cce125
Content: Block Documentation - minecraft:queued_ticking
Content Source: creator/Reference/Content/BlockReference/Examples/BlockTriggers/minecraftBlock_queued_ticking.md
Product: gaming
GitHub Login: @mammerla
Microsoft Alias: v-jillheaden

Fixed!  https://github.com/MicrosoftDocs/minecraft-creator-pr/pull/804


Feature Request:- Database Crawler

I'd love to see a database crawler for different database types. I deal with databases that store many tens of thousands of pdf or jpg or tiff as blobs, and I'd love to have a full OCR crawl performed of these documents / images.
Predominantly the database used is MS SQL Server 2008R2 or newer.
I love the concept of what you are doing here
**Just adding this to the wish list... :)

Hi @OutbackMatt ! Thanks for the post, but I think we will not create this type of crawler in near future as it requires a lot of work...

OK, so is there plans for a command line interface (CLI) or another interface so that we can programaticly send documents to be processed ad hoc.
At the moment it looks like I could create a 'drop folder', and send documents to the drop folder, and call a cron to assess all documents in this 'drop folder', and then remove the documents. That's OK, but a CLI  or API would also be useful.

@OutbackMatt you can use our REST API to upload files directly to Ambar. Check the documentation here https://github.com/RD17/ambar/blob/master/API_DOC.md#upload-file and the guide in our blog https://blog.ambar.cloud/ambar-use-case-integrated-parse-and-search-solution/

Check our support options


Emit dense arrays and use dense arrays in tests

The purpose of this change is to ease the transition from I64DenseArrayOrElements1DAttr to DenseI64ArrayAttr. The attrs themselves will be changed in a follow-up.
https://github.com/openxla/stablehlo/issues/1578

I've completed my first integrate so this PR is good to go!

How confident are we that we got everything? Have you tested this change after flipping the attribute from ArrayOrElements --> Array?
Also hold off on merge until @ghpvnist's approval since this will have integration implications.

I would say very confident. I flipped the attributes and fixed all the tests, then I split off the test changes.


[Feature] Let gitea recover from DB deadlocks

Taken from https://github.com/go-gitea/gitea/issues/13513#issuecomment-725451964
"... SQLite makes it easy for gitea to fail, then gitea should probably have error-correcting logic to correct any errors SQLite might cause. ..."
aka be more Fault tolerance

The only option for detecting deadlocks would be to have a timer in Xorm that would limit the length of time that a query could take and to kill it if it took too long. We'd need to have some way of actually ensuring that the killing of the query actually killed the SQLite C query too.

yet again this is another example of why models.x must die.
We can have send in a context with our requests to the db which would allow us to query dbs only within the context, killing our context should kill the request.
from mattn/go-sqlite3:sqlite3.go:1921:
// exec executes a query that doesn't return rows. Attempts to honor context timeout.
func (s *SQLiteStmt) exec(ctx context.Context, args []namedValue) (driver.Result, error) {
	if ctx.Done() == nil {
		return s.execSync(args)
	}

	type result struct {
		r   driver.Result
		err error
	}
	resultCh := make(chan result)
	go func() {
		r, err := s.execSync(args)
		resultCh <- result{r, err}
	}()
	select {
	case rv := <-resultCh:
		return rv.r, rv.err
	case <-ctx.Done():
		select {
		case <-resultCh: // no need to interrupt
		default:
			// this is still racy and can be no-op if executed between sqlite3_* calls in execSync.
			C.sqlite3_interrupt(s.c.db)
			<-resultCh // ensure goroutine completed
		}
		return nil, ctx.Err()
	}
}

(which hopefully would break the deadlock.

Deadlock should be handled by callers/developers, not by framework.

I am running into this problem, using Postgres. Server has a long-running migration. Deployment engine restarts it automatically. The repo is broken and nothing in the web UI will let me fix it. It is 'migrating' forever. :/

@alxpettit that's not database deadlock ....
I guess it's a bug in migration module.


Infinite Got heartbeat info...

This happens when an input is scattered, but is not a list. I think dotproduct scatter is used here.
Bunny gives out:
org.rabix.engine.processor.handler.EventHandlerException: Port rgnames__pl for root.alignment.1.process_alignment and rootId f7efecd6-0f7a-4bd9-b0c7-d8936ac5c862 is not a list and therefore cannot be scattered.
This is followed by infinite:
[2017-02-01 15:13:56.606] [DEBUG] [pool-17-thread-2] o.r.e.r.b.s.i.BackendStubLocal [BackendStub.java:58] Got heartbeat info from 49a35ae9-3d7d-4d50-ab87-3f3036d381cd
[2017-02-01 15:13:57.603] [DEBUG] [pool-17-thread-2] o.r.e.r.b.s.i.BackendStubLocal [BackendStub.java:58] Got heartbeat info from 49a35ae9-3d7d-4d50-ab87-3f3036d381cd
[2017-02-01 15:13:58.603] [DEBUG] [pool-17-thread-2] o.r.e.r.b.s.i.BackendStubLocal [BackendStub.java:58] Got heartbeat info from 49a35ae9-3d7d-4d50-ab87-3f3036d381cd

@sinisa88 will this also be handled by #112?


Write release process for blockchain images

Very similar to the one for the bridge.
I've also clarified/fixed the branch names in the bridge release process description (add bridge/ prefix) to distinguish from the ones for the blockchain.

Codecov Report

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

@@          Coverage Diff          @@
##           master   #550   +/-   ##
=====================================
  Coverage      76%    76%
=====================================
  Files          18     18
  Lines         925    925
=====================================
  Hits          703    703
  Misses        222    222

Flag
Coverage Δ

#bridge_end2end
0% <ø> (ø)
:arrow_up:

#bridge_unit
76% <ø> (ø)
: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 90a46b7...6b0c125. Read the comment docs.


ProximityChecker NonAlloc to avoid GC

Tested in uMMORPG with 800 monsters, proximity cast layers set to Everything.
Alloc:

NonAlloc:

5MB less GC. No real performance difference though.
Thoughts?

Codecov Report

Merging #228 into master will decrease coverage by 0.02%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master    #228      +/-   ##
=========================================
- Coverage    6.96%   6.93%   -0.03%
=========================================
  Files          29      29
  Lines        5919    5923       +4
  Branches      827     829       +2
=========================================
- Hits          412     411       -1
- Misses       5494    5498       +4
- Partials       13      14       +1

Impacted Files
Coverage Δ

Mirror/Runtime/NetworkProximityChecker.cs
0% <0%> (ø)
:arrow_up:

Mirror/Runtime/SyncList.cs
83.26% <0%> (-0.43%)
:arrow_down:

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 c0658b2...ec01ee2. Read the comment docs.

My numbers are nowhere near as high,  but it does show up as a problem in my profiler too.   So I am in favor of this PR.

commited: https://github.com/vis2k/Mirror/commit/400382be761aa95f9ff0b4e48a75561dbed94f38


[DO NOT MERGE] Server side snowpark

DO NOT MERGE.
this branch helps us compare snowpark server side changes vs. current main/HEAD

due to force pushes the git history here is compromised, closing this branch for now.


Bug Report: deploy_diagnostics_for_mg variable does not trigger diagnostics deployment

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Versions

terraform: 1.3.9
azure provider: 3.50.0
module: 3.3.0
Description : Deploy diagnostics for MG is not triggered.
Describe the bug
Although bumped up to module version 3.3.0 and AzureRM version 3.50.0 and adding the variable to the enterprise_scale.tfvars the plan show no changes to be made that reflect the enabling of diagnostics on Management Groups
Steps to Reproduce
Up the version of the terraform-azurerm-caf-enterprise-scale module in our workflow (Rover based), see screenshot

The plan output only show 1 outside changes and 1 policy assignments change, thats are both non-related to enabling diagnostics on Management Groups
Screenshots

Additional context
Please advice.....

@lachaves Do you perhaps have an idea why this is not working for me?

Hi @MarcelHeek
The MG diag settings are deployed by the AzAPI provider. Can you confirm this is being used?
We do not test this module with Rover and the aztfmod caf supermodule. You should raise an issue on that repo as well.

Hi @matt-FFFFFF,
How can I confirm whether this is being used? I am not that much into TF and merely a consumer. I can (at this point) only see that the azure/azapi module v1.5.0 is being loaded through the enterprise-scale module load in the Azure/caf-terraform-landingzone logic
And I will look into raising this issue as well in the Azure/caf-terraform-landingzone repo which calls the enterprise_scale module (as a caf solution add-on)

Hi @MarcelHeek, looking into it right now.

@lachaves It's been a while, but diagnostics for MG's are still not set for our tenants. I have the required versions of the providers in place that are documented here
As you can see in this screenshot the version requirements are all met

Have you any idea what might be the issue?

closed by #1053


Provide a way to remove proxy from vagrant

Proxy can be removed from vagrant by passing empty string as proxy configuration.
Also Fixes #5292

@timabbott Please review :)

@timabbott I also made some changes to fix issue #5292. The only problem is that warning is printed twice when I run vagrant up.
$ vagrant up

You have specified value for proxy in ~/.zulip-vagrant-config file but did not install vagrant-proxyconf plugin. To install run `vagrant plugin install vagrant-proxyconf` in terminal.
You have specified value for proxy in ~/.zulip-vagrant-config file but did not install vagrant-proxyconf plugin. To install run `vagrant plugin install vagrant-proxyconf` in terminal.

hashicorp/vagrant#7504 is why its being printed twice. I decided to just document that fact in the error message, link to the issue in the comments, and merge this. Thanks @hackerkid!


Allow to point to a specific salt dir

Improve development environment by creating a certificate authority,
salt-master with correct configuration for certificates to be remotely
signed and improve the start script to set up everything needed for the
development environment.

Maybe add a something in the README to explain the SALT_DIR usage? Not any extensive documentation but we should leave such things completely undocumented.

Updated with README instructions to use the SALT_DIR.


What does FROZEN deployment status mean?

I am getting sometimes FROZEN deployment status (deploying via API), for example
FROZEN express-sessions-tutorial-holgtyuece.now.sh

and was wondering if the process should just wait, or if FROZEN deploys should be aborted?

Which function is returning that status code?

@OlliV This endpoint: https://zeit.co/api#get-endpoint

yeah, I am deploying via API and then polling the get-endpoint. READY is good,
and if DEPLOYING or BOOTED I am waiting more, but FROZEN is considered an error, and I am not sure if this is correct assumption.
https://github.com/bahmutov/now-pipeline/blob/master/src/index.js#L67

FROZEN means that the deployment hasn't receiven traffic for a really long time, and has been temporarily hibernated. Do you get this while deploying? :thinking:

I was getting this for new deploys, and I think there were code changes in each,
Sent from my iPhone

On Oct 17, 2016, at 22:27, Tony Kovanen<EMAIL_ADDRESS>wrote:
FROZEN means that the deployment hasn't receiven traffic for a really long time, and has been temporarily hibernated. Do you get this while deploying? 🤔
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I had completely forgot this issue. We have since made a ton of improvements to the logic determining if the deployment is idling. Have you still seen this issue during the last weeks?

I will close the issue then for now and if I see it ever again will reopen.

I deployed something, and then had to go do something for an hour.  The deployment had already frozen.  What is the expected age of the deployment before being frozen?

@OlliV I’ve had a 1,5 day old page freeze when I sent it to a client for review. Will this be happening if I alias it to a real domain (and it doesn’t get much traffic), too?
If so, that would be kinda bad.
Thank you in advance!

It's happening to me even with an aliased custom domain bought through now... How do i prevent this? Do i have to set up a ping bot?! Super annoying for small websites that don't get much traffic...

To prevent a deployment from sleeping, you can now scale it.

Which i can't do on the Pro plan... ?

My deploy (which was aliased to a real domain) froze only an hour after deploying though... Surely that can't be right?

@irisSchaffer It's possible, especially if the deployment isn't receiving any traffic. You can now scale <url> 1 though like @leo mentioned to prevent the sleeping.

I guess this is tightly connected to https://github.com/zeit/now-cli/issues/511? Will comment there instead!

@TooTallNate @leo if I run now scale  1

Do I get extra charges?
Why doesn't the new  Scaled to 1 instance: [1s] URL not list when I run now list?


Visual C++ 2010 Download not available on MS site

Hi,
When I tried to install V2.2.0 It tries to download the C++ 2020 runtime and the link the setup uses is no longer valid.  Also I cannot find the redistributable elsewhere.  Any work around?  Thx

This issue will be resolved by the October 2022 version.


Agent should read private.pem and self.cert instead of creating identy.pfx

Instead of calling out to create a tertiary file, private.pem and self.cert should be read into a single buffer and passed to Identity::from_pem() instead of using from_pkcs12_der()
https://github.com/whamcloud/integrated-manager-for-lustre/blob/675374cd47f709d1a3d59381fa6459d4329362f2/iml-agent/src/http_comms/crypto_client.rs#L16-L18
Related to Pull #1730

Switching to from_pem requires switching to rustls backend:
    Client::builder()
        .use_rustls_tls()

This causes nginx to stop accepting connections for whatever reason:
2020/07/03 08:07:10 [crit] 19877#0: *22066 SSL_do_handshake() failed (SSL: error:0D07207B:asn1 encoding routines:ASN1_get_object:header too long error:0D068066:asn1 encoding routines:ASN1_CHECK_TLEN:bad object header error:0D07803A:asn1 encoding routines:ASN1_ITEM_EX_D2I:nested asn1 error:Type=X509 error:1408900D:SSL routines:ssl3_get_client_certificate:ASN1 lib) while SSL handshaking, client: <IP_ADDRESS>, server: <IP_ADDRESS>:443

Working on it.


Fortran arm and c++ clashing compiler ids

As seen in papertrail
Nov 30 18:35:48 25925b04925a unified: error: Compiler ID clash for 'ppc64leg8' - used by lang:c++ name:power64le AT12.0, lang:go name:power64le AT12.0, lang:fortran name:power64le AT12.0
Nov 30 18:35:48 25925b04925a unified: error: Compiler ID clash for 'ppc64g8' - used by lang:c++ name:power64 AT12.0, lang:go name:power64 AT12.0, lang:fortran name:power64 AT12.0
Nov 30 18:35:48 25925b04925a unified: error: Compiler ID clash for 'armg640' - used by lang:c++ name:ARM gcc 6.4, lang:fortran name:ARM gfortran 6.4
Nov 30 18:35:48 25925b04925a unified: error: Compiler ID clash for 'armg730' - used by lang:c++ name:ARM gcc 7.3, lang:fortran name:ARM gfortran 7.3
Nov 30 18:35:48 25925b04925a unified: error: Compiler ID clash for 'armg820' - used by lang:c++ name:ARM gcc 8.2, lang:fortran name:ARM gfortran 8.2
Nov 30 18:35:48 25925b04925a unified: error: Compiler ID clash for 'arm64g640' - used by lang:c++ name:ARM64 gcc 6.4, lang:fortran name:AARCH64 gfortran 6.4
Nov 30 18:35:48 25925b04925a unified: error: Compiler ID clash for 'arm64g730' - used by lang:c++ name:ARM64 gcc 7.3, lang:fortran name:AARCH64 gfortran 7.3
Nov 30 18:35:48 25925b04925a unified: error: Compiler ID clash for 'arm64g820' - used by lang:c++ name:ARM64 gcc 8.2, lang:fortran name:AARCH64 gfortran 8.2

Good spot! Perhaps we need a "test config" mode in app.js and then to run that as part of the CI build


Feature: Redis storage delegate

Changed the storage delegate to use Redis instead of AWS secret manager. Confirmed it working but didn't write any tests yet. I'll update the README with a few lines

I realize the implementation is a bit naive, because it uses the cached value as a source of truth, and rewrites the entire kv-store on every single change. When we settle on what KV store to use, I'll improve it

This won't be merged into code, we'll go for a postgreSQL-backed delegate instead


Security questions

I forgot the answers to security questions
I have 12 digit codes
I sent a mail, but there is no return
Could you please help me?

Please generate support request at bitconnect platform.


Iox 4 document public api

Pre-Review Checklist for the PR Author

[ ] Code follows the Rust coding style and is formatted with rustfmt
[ ] Branch follows the naming format (iox-123-this-is-a-branch)
[ ] Commits messages are according to this guideline

[ ] Commit messages have the issue ID (iox-#42 commit text)
[ ] Commit author matches Eclipse Contributor Agreement (and ECA is signed)

[ ] Update the PR title

Follow the same conventions as for commit messages
Link to the relevant issue

[ ] Relevant issues are linked
[ ] Add sensible notes for the reviewer
[ ] All checks have passed (except task-list-completed)
[ ] Assign PR to reviewer

Notes for Reviewer

This is based on #36 which needs to be merged first.
Checklist for the PR Reviewer

[ ] Commits are properly organized and messages are according to the guideline
[ ] Code according to our coding style and naming conventions
[ ] PR title describes the changes

Post-review Checklist for the PR Author

[ ] All open points are addressed and tracked via issues

References

Closes TBD

Codecov Report

Merging #59 (3c52f18) into master (a37117a) will increase coverage by 3.90%.
The diff coverage is 75.42%.

@@            Coverage Diff             @@
##           master      #59      +/-   ##
==========================================
+ Coverage   51.29%   55.20%   +3.90%
==========================================
  Files          19       20       +1
  Lines         774      942     +168
==========================================
+ Hits          397      520     +123
- Misses        377      422      +45

Impacted Files
Coverage Δ

iceoryx-sys/src/runtime.rs
100.00% <ø> (ø)

src/error.rs
0.00% <ø> (ø)

src/introspection/memory.rs
0.00% <ø> (ø)

src/introspection/port.rs
0.00% <ø> (ø)

src/introspection/process.rs
0.00% <ø> (ø)

src/lib.rs
100.00% <ø> (ø)

src/subscriber.rs
48.57% <25.00%> (+1.34%)
:arrow_up:

src/sample_mut.rs
70.00% <65.38%> (-17.50%)
:arrow_down:

src/sample.rs
59.40% <69.56%> (+4.40%)
:arrow_up:

src/publisher.rs
62.75% <71.66%> (+3.35%)
:arrow_up:

... and 5 more

Continue to review full report at Codecov.

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


just test functionality

https://github.com/ShumovskaIryna/Good-bar/blob/7e14885699dbf0496e9750cee60b99bd4ab60352/package.json#L10
Some text below

I saw this test=)


Authentication failed because the remote party has closed the transport stream.

While downloading file from FTPS once it works but next time the exception message "Authentication failed because the remote party has closed the transport stream." appears. Please suggest me.
I'm connecting like
 private void ConnectFTPS()
        {
            try
            {
                FtpClient myFTPSClient = new FtpClient();
                myFTPSClient.Host = ftpHost;
                myFTPSClient.Port = ftpPort;
                myFTPSClient.Credentials = new NetworkCredential(ftpUser, ftpPwd);
                myFTPSClient.EncryptionMode = FtpEncryptionMode.Explicit;
                X509Certificate2 cert_grt = new<EMAIL_ADDRESS>"123456");
                myFTPSClient.ClientCertificates.Add(cert_grt);
                myFTPSClient.ValidateCertificate += new FtpSslValidation(OnValidateCertificate);
                myFTPSClient.Connect();
                 }

            catch (Exception ex)
            {
                MessageBox.Show(ex.Message, Global.AppName + ": FTP Connection Error", MessageBoxButtons.OK, MessageBoxIcon.Error);
                OpenSettingsWindows();
            }
        }

        static void OnValidateCertificate(FtpClient control, FtpSslValidationEventArgs e)
        {
            e.Accept = true;

        }

And the detail error message is:
Authentication failed because the remote party has closed the transport stream.
   at System.Net.Security.SslState.StartReadFrame(Byte[] buffer, Int32 readBytes, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.StartReceiveBlob(Byte[] buffer, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.CheckCompletionBeforeNextReceive(ProtocolToken message, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.StartSendBlob(Byte[] incoming, Int32 count, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.ProcessReceivedBlob(Byte[] buffer, Int32 count, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.StartReadFrame(Byte[] buffer, Int32 readBytes, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.StartReceiveBlob(Byte[] buffer, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.CheckCompletionBeforeNextReceive(ProtocolToken message, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.StartSendBlob(Byte[] incoming, Int32 count, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.ProcessReceivedBlob(Byte[] buffer, Int32 count, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.StartReadFrame(Byte[] buffer, Int32 readBytes, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.StartReceiveBlob(Byte[] buffer, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.CheckCompletionBeforeNextReceive(ProtocolToken message, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.StartSendBlob(Byte[] incoming, Int32 count, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.ProcessReceivedBlob(Byte[] buffer, Int32 count, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.StartReadFrame(Byte[] buffer, Int32 readBytes, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.StartReceiveBlob(Byte[] buffer, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.CheckCompletionBeforeNextReceive(ProtocolToken message, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.StartSendBlob(Byte[] incoming, Int32 count, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.ForceAuthentication(Boolean receiveFirst, Byte[] buffer, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.ProcessAuthentication(LazyAsyncResult lazyResult)
   at System.Net.Security.SslStream.AuthenticateAsClient(String targetHost, X509CertificateCollection clientCertificates, SslProtocols enabledSslProtocols, Boolean checkCertificateRevocation)
   at FluentFTP.FtpSocketStream.ActivateEncryption(String targethost, X509CertificateCollection clientCerts, SslProtocols sslProtocols)
   at FluentFTP.FtpClient.OpenPassiveDataStream(FtpDataConnectionType type, String command, Int64 restart)
   at FluentFTP.FtpClient.OpenDataStream(String command, Int64 restart)
   at FluentFTP.FtpClient.GetListing(String path, FtpListOption options)
   at FluentFTP.FtpClient.GetListing(String path)

Thank you

I'm closing this since you haven't provided any more data for a week. Write back if you want to continue this issue and I'll reopen it.


[bugfix] Process execution modes before configuring logging

Fixes #1379.

Codecov Report

Merging #1399 into master will increase coverage by 0.31%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1399      +/-   ##
==========================================
+ Coverage   91.46%   91.77%   +0.31%
==========================================
  Files          83       83
  Lines       12277    12708     +431
==========================================
+ Hits        11229    11663     +434
+ Misses       1048     1045       -3

Impacted Files
Coverage Δ

reframe/frontend/cli.py
78.81% <100.00%> (+0.66%)
:arrow_up:

reframe/utility/color.py
100.00% <0.00%> (ø)

reframe/core/containers.py
100.00% <0.00%> (ø)

unittests/test_argparser.py
100.00% <0.00%> (ø)

unittests/test_check_filters.py
100.00% <0.00%> (ø)

unittests/test_sanity_functions.py
100.00% <0.00%> (ø)

...ittests/resources/checks_unlisted/kbd_interrupt.py
100.00% <0.00%> (ø)

unittests/test_deferrable.py
99.62% <0.00%> (+<0.01%)
:arrow_up:

unittests/test_buildsystems.py
99.16% <0.00%> (+<0.01%)
:arrow_up:

reframe/core/buildsystems.py
96.75% <0.00%> (+0.01%)
:arrow_up:

... and 34 more

Continue to review full report at Codecov.

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


⚠ Samsung has degraded performance

In dbbe29d, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2090 ms

Resolved: Samsung performance has improved in 9b44602.


Unable to connect to amqp:// via DialTLS

The documentation for DialTLS says:

DialTLS uses the provided tls.Config when encountering an amqps:// scheme.

This sounds great for us, since we prefer SSL/TLS in production but typically develop without it. I expected I could pass some reasonable tls.Config regardless of the connection string:
package main

import (
    "crypto/tls"
    "testing"

    "github.com/streadway/amqp"
)

func TestDialTLSWithAMQP(t *testing.T) {
    uri :=<EMAIL_ADDRESS>
    conn, err := amqp.DialTLS(uri, &tls.Config{MinVersion: tls.VersionTLS12})
    if err != nil {
        t.Fatalf("Expected no error, got %v", err)
    }
    defer conn.Close()
}

However, using DialTLS this way results in an error:
=== RUN TestDialTLSWithAMQP
--- FAIL: TestDialTLSWithAMQP (0.16s)
        cxn_test.go:15: Expected no error, got tls: oversized record received with length 20480
FAIL
exit status 1
FAIL    test    0.164s

Changing the connection string to<EMAIL_ADDRESS>works perfectly.
Am I doing something wrong? Is the documentation incorrect in this case?
For now we do the following:
var conn *amqp.Connection
var err error

if strings.HasPrefix(uri, "amqps") {
    conn, err = amqp.DialTLS(uri, &tls.Config{MinVersion: tls.VersionTLS12})
} else {
    conn, err = amqp.Dial(uri)
}

Given the commit above that corrects a URI in rabbitmq-tutorials, is this still relevant?

Yes. I can see that I wasn't clear. I'll check again why I said "works perfectly."
The documentation says:

DialTLS will use the provided tls.Config when it encounters an amqps:// scheme and will dial a plain connection when it encounters an amqp:// scheme.

My interpretation is that DialTLS will connect without TLS for an amqp:// URI. Is that how DialTLS is supposed to work?

@cbandy yes, my reading of the docs is the same. Well, that's a curious design choice (compared to all other clients I am aware of, at least).

The relationship between the Dial functions, Config, and URIs is complex.


Footstep sound effect synchronization

Related issue:
https://github.com/webaverse/app/issues/1909
Steps for testing:
Footsteps for walk,run,naruto run should be synced with animation
In fly mode there should not be any sound (jump, jump end etc)

IMO this needs a redo, without any hardcoded constants and simply watching the leg animation to trigger.

New approach to this issue is available here:
https://github.com/webaverse/app/pull/2025
That is the reason for closing


Anyway to prevent Alamofire from escaping forward slash in string?

Hi guys. I am using Alamofire to submit a JSON object to an api which is expecting a base64 image as one of the key-value pairs. Everything is working fine except that the API I am hitting does not decode the request and I need a way to prevent Alamofire from escaping the base64 string in the object because it escapes "/" with "/" (as it should), but unfortunately the API i am working with is rejecting my request because it cannot decode the base64 image due to backslashes in string.
Any help would be appreciated, thank you very much guys.
Details:
Xcode 7.3 | Swift 2.2, Alamofire 3.3.0 installed using Carthage.

oh and my apologies for lack of code samples. This is the request:
`        Alamofire.request(.POST,
submitUrl,
parameters: ["data":completedBlob.object], // The JSON object is a SwiftyJSON object.
encoding: .JSON ,
headers: self.submitHeaders
        ).responseJSON{ response in
            switch response.result {
            case .Success:
                if let value = response.result.value {
                    let json = JSON(value)
                    guard json["result"].isExists() else{ callback?(success: false, jsonObject: nil);return}
                    let result = json
                    callback?(success: true, jsonObject: result)
                }
            case .Failure(let error):
                print("An error occured processing request: \(error)")
                print(response.result.value)
            }
    }`

Can you share your request code? It will help us narrow down where in the process you'd have to customize your encoding to get the behavior you want.

@jshier My apologies for not updating the ticket, I was able to resolve the issue by sending blank parameters and changing the encoding type to the following:
encoding: .Custom({                 (convertible, params) in                 let mutableRequest = convertible.URLRequest.copy() as! NSMutableURLRequest //                mutableRequest.HTTPBody = paramBlob.dataUsingEncoding(NSUTF8StringEncoding, allowLossyConversion: false)                 mutableRequest.HTTPBody = paramBlob.dataUsingEncoding(NSUTF8StringEncoding, allowLossyConversion: false)                 return (mutableRequest, nil)             })


unable to build on CentOS Linux release 7.5.1804 (Core)

I am unable to complete the build on CentOS Linux release 7.5.1804 (Core).
I tied following the instructions but seeing below error:
make go build -i -ldflags "-X github.com/elastic/beats/libbeat/version.buildTime=2020-05-10T21:00:06Z -X github.com/elastic/beats/libbeat/version.commit=8cdb6f8f6994100b312a3095cf0fd4493fb9f62e"# github.com/ebay/nvidiagpubeat./main.go:31:15: undefined: cmd.GenRootCmd make: *** [libbeat] Error 2
Able to resolve go path:
which go /usr/local/go/bin/go

I was able to fix this issue by installing latest version of go using following script:
cd /tmp curl -LO https://dl.google.com/go/go1.14.2.linux-amd64.tar.gz tar -C /usr/local/ -xvzf go1.14.2.linux-amd64.tar.gz echo -e "#Updating go path" >> /etc/profile echo -e 'PATH="/usr/local/go/bin/:${PATH}"' >> /etc/profile echo -e "export PATH" >> /etc/profile source /etc/profile


Improve filtering of version names

Please have a look at the individual commit messages for the details.

This change is

Merging despite the unrelated AppVeyor failure.


support build gpu image with lower cuda version

Feature request
not sure if the current cuda 12.2 is a hard limit or could be optional
if optional, could parametrize the docker base image so user could re-build it easily
Motivation
some server just hard to upgrade the cuda version
Your contribution
parametrized the dockerfile and rebuild it if necessary on users' end

+1 for this, why TEI request cuda12.2, while torch is still with cuda12.1, and there is no nv-driver compatible for both 12.1/12.2.  https://docs.nvidia.com/deploy/cuda-compatibility/index.html

and there is no nv-driver compatible for both 12.1/12.2

From the page you linked:

If you are upgrading the driver to 525.60.13 which is the minimum required driver version for the 12.x toolkits, then 11.x and 12.x applications will be supported due to backward compatibility and future 12.x applications will be supported due to minor-version compatibility.

You can run both torch and TEI with >=525.60.13.
@DeoLeung do you have issues running TEI on a server with >=525.60.13?

and there is no nv-driver compatible for both 12.1/12.2

From the page you linked:

If you are upgrading the driver to 525.60.13 which is the minimum required driver version for the 12.x toolkits, then 11.x and 12.x applications will be supported due to backward compatibility and future 12.x applications will be supported due to minor-version compatibility.

You can run both torch and TEI with >=525.60.13.
@DeoLeung do you have issues running TEI on a server with >=525.60.13?

we managed to do a server upgrade and it's now running fine on
NVIDIA-SMI 550.54.14
Driver Version: 550.54.14
CUDA Version: 12.4

and there is no nv-driver compatible for both 12.1/12.2

From the page you linked:

If you are upgrading the driver to 525.60.13 which is the minimum required driver version for the 12.x toolkits, then 11.x and 12.x applications will be supported due to backward compatibility and future 12.x applications will be supported due to minor-version compatibility.

You can run both torch and TEI with >=525.60.13.
@DeoLeung do you have issues running TEI on a server with >=525.60.13?

othx, maybe I should try downgrade to 525.60.13
I'm with 530.30.02 now, it seems not compatible from the table in the link
starting the tei docker will report a error :
nvidia-container-cli: requirement error: unsatisfied condition: cuda>=12.2, please update your driver to a newer version, or use an earlier cuda container: unknown.


Walk refactored as a component and added to the site

Walk is a component and is available on the site with some example.

This closes #545

@ColinEberhardt @chrisprice we think that this is ready to merge

we think that this is ready to merge

This is looking good - although there is one minor comment that I'd like to see addressed. Thanks.


Fix typo in permissions, was missing (s) in virtualmachines

Signed-off-by: Alexander Wels<EMAIL_ADDRESS>
What this PR does / why we need it:
The permissions were applied to the wrong resource name, it was missing an s.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

NONE

/cherrypick release-0.49
/cherrypick release-0.44

/lgtm

/approve


About the ordering of pick/bans in info.getGameInfo().getDota().getPicksBansList()

Hello there,
I have a question concerning the pick and bans list: are they always given
in the right order if I use the following code? By "right order", I mean
the order in which the picks/bans were done at the beginning of the match.
Thanks a lot for your help, and keep doing this awesome work!
Cheers
Map<String, String> properties = new LinkedHashMap<String, String>();
Demo.CDemoFileInfo info = Clarity.infoForFile(filename);

for (int i = 0; i < info.getGameInfo().getDota().getPicksBansList().size(); i++)
{
	properties.put("pick_ban_heroid_p"+(i+1),   "" + info.getGameInfo().getDota().getPicksBansList().get(i).getHeroId());
	properties.put("pick_ban_is_pick_p"+(i+1),  "" + info.getGameInfo().getDota().getPicksBansList().get(i).getIsPick());
	properties.put("pick_ban_team_p"+(i+1),		"" + info.getGameInfo().getDota().getPicksBansList().get(i).getTeam());
}

Hey,
there is no other indexing info in CDemoFileInfo, so I think it's safe to assume that the order in the file is correct.
A LinkedHashMap is a Map that preserves insertion order and when you iterate over it's elements, you get them in the order you inserted them.
Whether or not  your example is correct depends on how you use the map, and you need to provide the code for that :)

Hello,
Thanks for your reply! So I assume that my code is correct:
the key for each pick/ban is also composed of the index i in the list of getPicksBansList().
When I enter
'''
properties.get("pick_ban_heroid_p1");
'''
I do indeed get the first choice that was made.
If that's correct, I shall close the issue.
Thanks a lot!
Cheers

Correct :)


fix pentest findings

Description
Fix internal pentest findings
Release required?

[ ] No release
[x] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

/run pipeline

/run pipeline

:tada: This PR is included in version 0.5.6 :tada:
The release is available on:

GitHub release
v0.5.6

Your semantic-release bot :package::rocket:


Workaround for scrolling position issue with lazy loading images

Story

https://github.com/thematters/developer-resource/issues/372

dups with https://github.com/thematters/matters-web/issues/4222


CB-11760 Datalake upgrade: NPE when CM does not send host status

During datalake upgrade in the cluster sync step (part of the image change flow) the code asks CM for host statuses. CM server sometimes returns an empty response. This was not expected and an NPE was thrown.
After the NPE, even though the imafge change flow failed, the calling datalake flow proceeded and failed at the replaceVm - upscale step, as the new image was not copied.
This commit fixes the NPE itself only.
See detailed description in the commit message.

these instances would stay in SERVICES_RUNNING state. Is this the right state? Would it cause any issue later?
I think the sync might set the right state eventually.

these instances would stay in SERVICES_RUNNING state. Is this the right state? Would it cause any issue later?
I think the sync might set the right state eventually.

Basically what CM says is that the metrics are NOT_AVAILABLE. Based on that we can't determine whether we should change the state of the instances. I've only seen this happen during upgrade so I think it's safe. The regular sync should set the correct host status if it changes.


Not compatible with Samsung flow.

Description
Samsung flow keeps failing whenever i drag any types of file. (jpg,png,mp4, etc)  But when I tested on Windows Explorer it works fine.
https://user-images.githubusercontent.com/42995601/208242988-08205b42-a5c8-4caa-9609-5a0ea41ff308.mp4
Steps To Reproduce

Choose any type of file.
Make sure your device is connected to Samsung flow.
Drag any files to Samsung flow.

Steps to fix
This could be a problem with Samsung flow but it's worth a try to check on why it keeps on failing even on Administration rights.
Files Version
Version: <IP_ADDRESS> OS Version: 10.0.22000.1335
Windows Version
Windows 11 Home Version 21H2  OS build 22000.1335 Experience
Relevant Assets/Logs
debug.log

This is a known issue and it affects other apps as well, merging with #10171


Regression in fix for #793? MatchSpec parsing appears to fail when channel is passed as url

Checklist

[X] I added a descriptive title
[X] I searched open reports and couldn't find a duplicate

What happened?
Using the latest py-rattler==0.6.3 I am unable to parse MatchSpecs when the channel is passed as a url. In  https://github.com/conda/rattler/issues/793#issuecomment-2262269859 the following example is given (copying for visibility here):

>>> from rattler import MatchSpec
>>> m = MatchSpec("https://conda.anaconda.org/conda-forge::python[version=3.9]")
>>> m
MatchSpec("conda-forge::python ==3.9")
>>> m.channel
Channel(name="conda-forge", base_url="https://conda.anaconda.org/conda-forge/")

However on my machine I cannot reproduce this. I have just done the following in an empty directory (on mac M1):

✨ Create a new pixi project and add py-rattler
$ pixi init
$ pixi add --manifest-path pixi.toml py-rattler
✔ Added py-rattler >=0.6.3,<0.7

🔧 Add test_matchspec.py to the project directory (adapted from the previously-mentioned example in https://github.com/conda/rattler/issues/793#issuecomment-2262269859):
# test_matchspec.py

from rattler import MatchSpec

if __name__ == "__main__":
    with_url_channel = MatchSpec("https://conda.anaconda.org/conda-forge::python[version=3.9]")
    print(f"{with_url_channel = }")
    print(f"{with_url_channel.channel = }")
    with_named_channel = MatchSpec("conda-forge::python[version=3.9]")
    print(f"{with_named_channel = }")
    print(f"{with_named_channel.channel = }")

🤔 Run the script. Note that the with_url_channel does not parse correctly (contrary to the linked example in #793):
$ pixi run --manifest-path pixi.toml -e default python test_matchspec.py
with_url_channel = MatchSpec("*")
with_url_channel.channel = None
with_named_channel = MatchSpec("conda-forge::python ==3.9")
with_named_channel.channel = Channel(name="conda-forge", base_url="https://conda.anaconda.org/conda-forge/")

For reference, here is the output of pixi list for this environment:

$ pixi list --manifest-path pixi.toml -e default
Package          Version    Build               Size       Kind   Source
bzip2            1.0.8      h99b78c6_7          120 KiB    conda  bzip2-1.0.8-h99b78c6_7.conda
ca-certificates  2024.8.30  hf0a4a13_0          154.8 KiB  conda  ca-certificates-2024.8.30-hf0a4a13_0.conda
libexpat         2.6.3      hf9b8971_0          62.4 KiB   conda  libexpat-2.6.3-hf9b8971_0.conda
libffi           3.4.2      h3422bc3_5          38.1 KiB   conda  libffi-3.4.2-h3422bc3_5.tar.bz2
libsqlite        3.46.1     hc14010f_0          810.1 KiB  conda  libsqlite-3.46.1-hc14010f_0.conda
libzlib          1.3.1      hfb2fe0b_1          45.8 KiB   conda  libzlib-1.3.1-hfb2fe0b_1.conda
ncurses          6.5        h7bae524_1          783.5 KiB  conda  ncurses-6.5-h7bae524_1.conda
openssl          3.3.2      h8359307_0          2.7 MiB    conda  openssl-3.3.2-h8359307_0.conda
py-rattler       0.6.3      py312h3ddc590_1     3.9 MiB    conda  py-rattler-0.6.3-py312h3ddc590_1.conda
python           3.12.6     h739c21a_0_cpython  12.3 MiB   conda  python-3.12.6-h739c21a_0_cpython.conda
python_abi       3.12       5_cp312             6.1 KiB    conda  python_abi-3.12-5_cp312.conda
readline         8.2        h92ec313_1          244.5 KiB  conda  readline-8.2-h92ec313_1.conda
tk               8.6.13     h5083fa2_1          3 MiB      conda  tk-8.6.13-h5083fa2_1.conda
tzdata           2024a      h8827d51_1          121.3 KiB  conda  tzdata-2024a-h8827d51_1.conda
xz               5.2.6      h57fd34a_0          230.2 KiB  conda  xz-5.2.6-h57fd34a_0.tar.bz2

Is this user error on my part, or a regression, or something else?
Thank you all so much!
Additional Context
No response

Argh, that looks like a bug / regression on first glance. Woudl you be able to whip up a PR with a test case?

Thank for the prompt reply, and yes would be happy to. Will get that up shortly.

Done in https://github.com/conda/rattler/pull/870. As noted there, the regression is not present for me when running those tests locally (from a fresh build), so maybe the regression is already fixed in main and only present in 0.6.3 release?

It's possible that we need to cut a new release of the Python bindings. I kicked off the CI run for your new tests.

This is fixed, right?! :)

Yes this is working for me in >=0.7.0 🙏


Spintires don't work in Fullscreen using D9VK

Is not possible to use the menu using D9VK in FullScreen Mode (as described here). If I use the gam in a Window it works
Software information
Spintires
System information

GPU: NVIDIA GeForce GTX-1060-6GB
Driver: 430.50
Wine version: Proton 4.11-9
DXVK version: 0.30

Apitrace file(s)
-I can't generate an apitrace
Log files

d3d9.log:
SpinTires_d3d9.log
d3d11.log:
dxgi.log: 0 bytes

Now the apitrace woked. Here is it:
https://mega.nz/#!BcRRhYgI!fXwydGk7R2qZYI3NSej3-dbJ_FO8ma6Ox3siTEljyYY
The game has the same problem that Mudrunner in certain moments:

Is not possible to use the menu using D9VK in FullScreen Mode

Works fine here.

This isn't my bug, use a native d3dcompiler/d3dx9.
Fullscreen with D9VK works fine on Windows too, so it's a Wine bug causing that.


Train from pairs not label

Is it possible to train metric from a list of pairs (similar and dissimilar)? My groundtruth consists of these pairs and thus can't use label.

It depends on the algorithm. In case of LSML, ITML, and SDML, this is possible using the classes named LSML, ITML, and SDML, respectively, whose fit() method takes pairwise constraints or connectivity graphs as arguments. The classes LSML_Supervised, ITML_Supervised, and SDML_Supervised are merely wrappers around those, generating pair-wise similarity and dissimilarity constraints from class labels.


⚠ Avatar API Endpoint has degraded performance

In cd42333, Avatar API Endpoint (https://avatar.roblox.com/v1/avatar-rules) experienced degraded performance:

HTTP code: 200
Response time: 1037 ms

Resolved: Avatar API Endpoint performance has improved in b593267.


fix part.body -> part.data . HttpFormPart doesn't have body

This pull request same as : https://github.com/essenciary/Genie.jl/pull/74

Thank you!


Add stats command

We want a stats command that outputs statistics for a course and possibly limited to a course instance.

We have the data subcommand for ladok which provides the data. We refer to course-analytics for the statistical analysis.


Added note about how secrets are displayed

Hey, I just made a Pull Request!
Added note to README about how secrets are displayed in the Config tab of the DevTools plugin
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

@adamdmharvey any chance you could give this a look? Feel like this could be written a bit better but not sure how.
This stems from this issue: https://github.com/backstage/backstage/issues/17855

Sorry to pester you @adamdmharvey, I know I'm asking for your help here, but could you take a look at this text change? I feel like you would have good feedback on this and I want to make sure I get this right. Many thanks in advance!


Handle race condition when doing multiple updates

Some tasks (indexing, disambiguation) are done asynchronously after updating the record, and rely on the following property:

The previous run of the task was for the previous version of the record.

This property is used implicitly when diffing versions of the record in order to detect

which references have changed and their records need reindexing;
which authors have changed and need disambiguating;
etc.

However, the property as stated is false, due to the asynchronicity: it's possible for two (or more) updates of the record to be made before the task has a chance to run, causing the diff of the changes on the DB not to correspond to the diff of successive runs of the tasks. This should be fixed, as it will cause issues (wrong citation counts when updating references, wrongly disambiguated authors, etc.).
Tech Notes:
Looks like the best solution here would be to check which version is on ES currently (based on updated field from db and updated field on ES) and index/disambiguate ONLY if current version in DB is more recent.
https://github.com/inspirehep/inspirehep/blob/a325629aea4e8717997a26ec0f37b5d2e62ad313/backend/inspirehep/records/receivers.py#L37 In here add a check if InspireSearch.get_record_data_from_es(rec)['_updated'] < rec.updated: then reindex/disambiguate
Remember that _updated is in isoformat, so it should be parsed first.

I don't think that would work, as sometimes we index due to changes not reflected in the DB record (e.g. citation count) which wouldn't update the _updated field.

I don't think that would work, as sometimes we index due to changes not reflected in the DB record (e.g. citation count) which wouldn't update the _updated field.

I Updated tech notes.

I Updated tech notes.

We are changing the idea qhat and how we want to fix here
Tech Notes

https://github.com/inspirehep/inspirehep/blob/a325629aea4e8717997a26ec0f37b5d2e62ad313/backend/inspirehep/indexer/tasks.py#L28 Extend this list with Elasticsearch errors: ConnectionError, ConnectionTimeout, SSLError, NotFoundError'RequestError`
https://github.com/inspirehep/inspirehep/blob/a325629aea4e8717997a26ec0f37b5d2e62ad313/backend/inspirehep/indexer/tasks.py#L87 Gracefully handle error from indexing when we try to update document to older version VersionConflictException so rest of the task continue without a problem

We are changing the idea qhat and how we want to fix here
Tech Notes

https://github.com/inspirehep/inspirehep/blob/a325629aea4e8717997a26ec0f37b5d2e62ad313/backend/inspirehep/indexer/tasks.py#L28 Extend this list with Elasticsearch errors: ConnectionError, ConnectionTimeout, SSLError, NotFoundError'RequestError`
https://github.com/inspirehep/inspirehep/blob/a325629aea4e8717997a26ec0f37b5d2e62ad313/backend/inspirehep/indexer/tasks.py#L87 Gracefully handle error from indexing when we try to update document to older version VersionConflictException so rest of the task continue without a problem


#913 Added networkType enum. Implemented logic based on network type

@andrew-aladev please take a look at this implementation.
Does it look well or not?

This change is

PR is added to the queue for testing as 2. task. (79086ef)

Hi @artemjackson, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

Test (Success) 82203060 build is completed (79086ef)
See https://github.com/ThaliTester/TestResults/tree/8220306079086ef__913_Added_networkType_enum__Implemented_logic_based_on_network_type_artemjackson/ for the logs

Test<PHONE_NUMBER>086ef(79086ef) has failed
See https://github.com/ThaliTester/TestResults/tree/8220306079086ef__913_Added_networkType_enum__Implemented_logic_based_on_network_type_artemjackson/ for the fail logs

PR is added to the queue for testing as 1. task. (2c9dbcd)

Test (Success) 82203060 build is completed (2c9dbcd)
See https://github.com/ThaliTester/TestResults/tree/822030602c9dbcd__913_Added_networkType_enum__Implemented_logic_based_on_network_type_artemjackson/ for the logs

Test 822030602c9dbcd(2c9dbcd) has successfully finished without an error
See https://github.com/ThaliTester/TestResults/tree/822030602c9dbcd__913_Added_networkType_enum__Implemented_logic_based_on_network_type_artemjackson/ for the logs

PR is added to the queue for testing as 1. task. (708b260)

Test (Fail) 82203060 build is completed (708b260)
See https://github.com/ThaliTester/TestResults/tree/82203060708b260__913_Added_networkType_enum__Implemented_logic_based_on_network_type_artemjackson/ for the logs

PR is added to the queue for testing as 2. task. (884b823)

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

Comments from Reviewable

Test (Success) 82203060 build is completed (884b823)
See https://github.com/ThaliTester/TestResults/tree/82203060884b823__913_Added_networkType_enum__Implemented_logic_based_on_network_type_artemjackson/ for the logs

Test<PHONE_NUMBER>4b823(884b823) has successfully finished without an error
See https://github.com/ThaliTester/TestResults/tree/82203060884b823__913_Added_networkType_enum__Implemented_logic_based_on_network_type_artemjackson/ for the logs

Once you fix the switch  issues (and do whatever you want with the NB issue) then

Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, 5 unresolved discussions, some commit checks failed.

thali/NextGeneration/thaliMobile.js, line 62 [r2] (raw file):

var getWifiOrNativeMethodByNetworkType = function (method, networkType) {
  if (networkType === networkTypes.BOTH) {

Please use a switch statement

thali/NextGeneration/thaliMobile.js, line 220 [r2] (raw file):

    var start = getWifiOrNativeMethodByNetworkType('start',
     thaliMobileStates.networkType);
    start(router, pskIdToSecret)

NB: I find this code a bit surprising.  I probably would have done something like getWifiOrNativeMethodByNetworkType('start', thaliMobileStates.networkType)(router, pskIdToSecret). The fact that getWifiOrNativeMethodByNetworkType returns a function rather than a promise was a bit surprising (but understandable) to me. It got in the way of reading the code. By putting the args right next to the function output it's more immediately clear what's going on. In any case, I'm arguing over nits.

thali/NextGeneration/thaliMobile.js, line 386 [r2] (raw file):

  var networkType = thaliMobileStates.networkType;
  return promiseQueue.enqueue(function (resolve, reject) {
    if (networkType === networkTypes.NATIVE ||

Again, please use Switch

Comments from Reviewable

PR is added to the queue for testing as 2. task. (198550b)

PR is added to the queue for testing as 2. task. (a181f1c)

PR is added to the queue for testing as 3. task. (5965752)

Test (Success) 82203060 build is completed (198550b)
See https://github.com/ThaliTester/TestResults/tree/82203060198550b__913_Added_networkType_enum__Implemented_logic_based_on_network_type_artemjackson/ for the logs

Test (Fail) 82203060 build is completed (a181f1c)
See https://github.com/ThaliTester/TestResults/tree/82203060a181f1c__913_Added_networkType_enum__Implemented_logic_based_on_network_type_artemjackson/ for the logs

Test<PHONE_NUMBER>8550b(198550b) has failed
See https://github.com/ThaliTester/TestResults/tree/82203060198550b__913_Added_networkType_enum__Implemented_logic_based_on_network_type_artemjackson/ for the fail logs

Test (Success) 82203060 build is completed (5965752)
See https://github.com/ThaliTester/TestResults/tree/822030605965752__913_Added_networkType_enum__Implemented_logic_based_on_network_type_artemjackson/ for the logs

Test<PHONE_NUMBER>65752(5965752) has successfully finished without an error
See https://github.com/ThaliTester/TestResults/tree/822030605965752__913_Added_networkType_enum__Implemented_logic_based_on_network_type_artemjackson/ for the logs

PR is added to the queue for testing as 6. task. (b7ba7ed)

Test (Fail) 82203060 build is completed (b7ba7ed)
See https://github.com/ThaliTester/TestResults/tree/82203060b7ba7ed__913_Added_networkType_enum__Implemented_logic_based_on_network_type_artemjackson/ for the logs


mysql\Schema::findConstraints matches foreign keys both in comments and MySQL syntax

During creation of migrate files for my existing database, I found it strange, that one of tables had two foreign key declarations, from Yii point of view, while there was only one in MySQL workbench. During debug process, I added following lines in mysql\Schema::findConstraints method (https://github.com/yiisoft/yii2/blob/master/framework/db/mysql/Schema.php#L250)
	        if ($table->name == 'user__oauth') {
		        var_dump($sql);
		        var_dump($matches);
		        die();
	        }

And I got following output
string 'CREATE TABLE `user__oauth` (
  `user_id` int(11) NOT NULL COMMENT 'CONSTRAINT FOREIGN KEY (user_id) REFERENCES user(id)',
  `uid` int(11) NOT NULL,
  `provider` varchar(100) CHARACTER SET utf8 NOT NULL,
  PRIMARY KEY (`user_id`,`uid`,`provider`),
  KEY `uid` (`uid`,`provider`),
  CONSTRAINT `user_id` FOREIGN KEY (`user_id`) REFERENCES `user` (`id`) ON DELETE CASCADE ON UPDATE CASCADE
) ENGINE=InnoDB DEFAULT CHARSET=utf8 COLLATE=utf8_unicode_ci' (length=447)
array (size=2)
  0 =>
    array (size=4)
      0 => string 'FOREIGN KEY (user_id) REFERENCES user(id)' (length=41)
      1 => string 'user_id' (length=7)
      2 => string 'user' (length=4)
      3 => string 'id' (length=2)
  1 =>
    array (size=4)
      0 => string 'FOREIGN KEY (`user_id`) REFERENCES `user` (`id`)' (length=48)
      1 => string '`user_id`' (length=9)
      2 => string '`user`' (length=6)
      3 => string '`id`' (length=4)

It turns out, that the regex match (https://github.com/yiisoft/yii2/blob/master/framework/db/mysql/Schema.php#L249) finds either comment and foreign key in MySQL syntax, which is unexpected.
I am used to "old style" yii1, where I would add in MySQL table column comments foreign key declarations, to aid GII model generation process, and I don't know, if this behavior is feature or bug. Should I avoid adding "constraint" comments for tables, if I use foreign keys in MySQL?
I think, there should be some sort of algorithm that ignores stuff written in comments about constraints, if there exists foreign key declaration in MySQL syntax for that same field.

Constraints for MySQL are extracted from the table create query (SHOW CREATE TABLE) using this regular expression:
/FOREIGN KEY\s+\(([^\)]+)\)\s+REFERENCES\s+([^\(^\s]+)\s*\(([^\)]+)\)/mi

which should probably be ungreedy.
Only MSSQL reads them from the INFORMATION_SCHEMA.REFERENTIAL_CONSTRAINTS table, maybe this is also available for MySQL and would be better?

This changes introduced a regression on my app : Currently with code from master, I always get an empty array in $table->foreignKeys for all models of my database (mysql 5.1.72).
Rolling back 191185f make things work again.
The problem is that this SQL query doesn't return anything:
SELECT
    kcu.constraint_name,
    kcu.column_name,
    kcu.referenced_table_name,
    kcu.referenced_column_name
FROM information_schema.referential_constraints AS rc
JOIN information_schema.key_column_usage AS kcu ON
    kcu.constraint_catalog = rc.constraint_catalog AND
    kcu.constraint_schema = rc.constraint_schema AND
    kcu.constraint_name = rc.constraint_name
WHERE rc.constraint_schema = database() AND kcu.table_schema = database()
AND rc.table_name = "model" AND kcu.table_name = "model"
SQL;

Here are the corresponding records for my "model" table in both information_schema tables:
mysql> select * from information_schema.referential_constraints rc WHERE rc.constraint_schema = database() AND rc.table_name = "model";
+--------------------+-------------------+-----------------+---------------------------+--------------------------+------------------------+--------------+-------------+-------------+------------+-----------------------+
| CONSTRAINT_CATALOG | CONSTRAINT_SCHEMA | CONSTRAINT_NAME | UNIQUE_CONSTRAINT_CATALOG | UNIQUE_CONSTRAINT_SCHEMA | UNIQUE_CONSTRAINT_NAME | MATCH_OPTION | UPDATE_RULE | DELETE_RULE | TABLE_NAME | REFERENCED_TABLE_NAME |
+--------------------+-------------------+-----------------+---------------------------+--------------------------+------------------------+--------------+-------------+-------------+------------+-----------------------+
| NULL               | flycar            | fk_model_brand1 | NULL                      | flycar                   | PRIMARY                | NONE         | NO ACTION   | NO ACTION   | model      | brand                 |
+--------------------+-------------------+-----------------+---------------------------+--------------------------+------------------------+--------------+-------------+-------------+------------+-----------------------+
1 row in set (0.00 sec)

mysql> select * from information_schema.key_column_usage kcu WHERE kcu.table_schema = database() AND kcu.table_name = "model";
+--------------------+-------------------+-----------------+---------------+--------------+------------+-------------+------------------+-------------------------------+-------------------------+-----------------------+------------------------+
| CONSTRAINT_CATALOG | CONSTRAINT_SCHEMA | CONSTRAINT_NAME | TABLE_CATALOG | TABLE_SCHEMA | TABLE_NAME | COLUMN_NAME | ORDINAL_POSITION | POSITION_IN_UNIQUE_CONSTRAINT | REFERENCED_TABLE_SCHEMA | REFERENCED_TABLE_NAME | REFERENCED_COLUMN_NAME |
+--------------------+-------------------+-----------------+---------------+--------------+------------+-------------+------------------+-------------------------------+-------------------------+-----------------------+------------------------+
| NULL               | flycar            | PRIMARY         | NULL          | flycar       | model      | id          |                1 |                          NULL | NULL                    | NULL                  | NULL                   |
| NULL               | flycar            | fk_model_brand1 | NULL          | flycar       | model      | brand_id    |                1 |                             1 | flycar                  | brand                 | id                     |
+--------------------+-------------------+-----------------+---------------+--------------+------------+-------------+------------------+-------------------------------+-------------------------+-----------------------+------------------------+
2 rows in set (0.00 sec)

The problem is that  kcu.constraint_catalog and rc.constraint_catalog are both NULL in my case, and that the expression "null = null" in MySQL doesn't evaluate to TRUE but it evaluates to NULL which makes the condition not verified.
The query needs to be fixed like this:
-    kcu.constraint_catalog = rc.constraint_catalog AND
+    (
+        kcu.constraint_catalog = rc.constraint_catalog OR
+        (ISNULL(kcu.constraint_catalog) AND ISNULL(rc.constraint_catalog))
+    ) AND

https://github.com/yiisoft/yii2/commit/e4f5b91f74e37d5acc2c7d01f977a29e790532b6

Thanks!


Feature request: improve bundle

Hello,
currently the "module" field of the package.json points to your "index.ts" file. This break e.g. creat-react-app. It would be better if you would provide two bundled version, one for cjs and one for esm. Then you could set the "main" field to your index.cjs.js file and your "module" field to "index.esm.js" ... in addition to that a "types" field pointing to typescript declarations.
Would you accept a pull request for those changes?
Besides that, this is a fantastic library btw 😉

We already have a compiled CJS version as the "main" field (dist/fermat.js) which you can use using require('@mathigon/fermat'), and you can definitely send a PR to add types for this.
Can you explain why the "module" field doesn't work with creat-react-app? I'd prefer keeping this as the Typescript source file, so that IDE links remain intact (e.g. command-click goes to the original declaration, rather than a compiled JS file), and so that end-users can take advantage of TS minification (e.g. https://github.com/angular/ts-minify).
Happy to hear that you like this library!

Thanks for your quick reply.
The "module" field of a package.json is intended for a esm js entry point (see https://rollupjs.org/guide/en/#publishing-es-modules) So your "main" field has to point to cjs (which it already does) the optional "module" field has to point to "esm".
Currently create-react-app (with typescript template) throws because it expects a js file at "module":
Failed to compile.

<EMAIL_ADDRESS>10:17
Module parse failed: Unexpected token (10:17)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
|
|
> const absStr = (n: number, suffix?: string) => {
|   const prefix = n < 0 ? '–' : '';
|   if (Math.abs(n) === 1 && suffix) return prefix + suffix;

You won't loose the ability to jump to source by cmd+clicking, because typescript will find the index.ts in the module root.
I'll make a pull request so you can test this approach.

Fixed in https://github.com/mathigon/fermat.js/commit/b20cc2d289bd3f2d134c0d4ca738bb8b1504df78


Move the commands in pkg to internal

For now, we don't want consumers of OpenFGA using them. They should rely on the methods in server file only.

Closing in favor of https://github.com/openfga/openfga/issues/1194


Warnings come up for loan terms that are well within boundaries (W0182)

With values like '715' we get warnings like "when present, 'loan term' should be less than 1200(100 years)."

@MattMcB, just to confirm, this is the bug you mentioned when we met last time?

@MattMcB, just to confirm, this is the bug you mentioned when we met last time?
yis

Great. We'll check it out. Thanks!

Then problem is the is_less_than function doesn't convert the strings to ints.  So we're doing a straight string compare with
    check_result = value < max_value
    return _check_blank_(value, check_result, accept_blank)


Convert fill commands from Word to MkDocs

Need to convert the Fill commands from Word to MkDocs.

I converted the Fill command documentation.  Close the issue.


MyQ not working in 0.92.1

Home Assistant release with the issue:
0.92.1
Operating environment (HassOS/Generic):
hass.io, Raspberry Pi
Supervisor logs:
2019-05-01 17:04:05 ERROR (MainThread) [homeassistant.components.cover] Error while setting up platform myq
Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/homeassistant/helpers/entity_platform.py", line 126, in _async_setup_platform
    SLOW_SETUP_MAX_WAIT, loop=hass.loop)
  File "/usr/local/lib/python3.7/asyncio/tasks.py", line 416, in wait_for
    return fut.result()
  File "/usr/local/lib/python3.7/site-packages/homeassistant/components/myq/cover.py", line 43, in async_setup_platform
    myq = await login(username, password, brand, websession)
  File "/usr/local/lib/python3.7/site-packages/pymyq/api.py", line 291, in login
    await api.authenticate(username, password)
  File "/usr/local/lib/python3.7/site-packages/pymyq/api.py", line 226, in authenticate
    await self._get_security_token()
  File "/usr/local/lib/python3.7/site-packages/pymyq/api.py", line 242, in _get_security_token
    login_request=True,
  File "/usr/local/lib/python3.7/site-packages/pymyq/api.py", line 151, in _request
    return await resp.json(content_type=None)
  File "/usr/local/lib/python3.7/site-packages/aiohttp/client_reqrep.py", line 1036, in json
    return loads(stripped.decode(encoding))
  File "/usr/local/lib/python3.7/json/__init__.py", line 348, in loads
    return _default_decoder.decode(s)
  File "/usr/local/lib/python3.7/json/decoder.py", line 337, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/usr/local/lib/python3.7/json/decoder.py", line 355, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

Description of problem:
myq fails to load at startup after upgrading to 0.92.1 from 0.91.3.

In case this is potentially an issue with my config, here's the relevant section from my configuration.yaml:
cover:
  - platform: myq
    username<EMAIL_ADDRESS>    password: myPassword
    type: chamberlain

This had been working for me for quite some time now, though, and it's a match for the config example in the documentation, and I haven't been able to find mention of breaking changes around myq in the last few releases, so I'm relatively convinced that this isn't the culprit.

Look like the response from server is not a valid json

This appears to have been resolved in 0.92.2:
Fix authentication issue (@ehendrix23 - #23600)
Confirmed that it's working for me after updating.


Assert fix

Adjusted asserting leader in HA setups that is compatible with new broker versions

The new release tag is v2.0.1

Check fail can be ignored, doesn't work for PRs

Reviewed change.  Looks straightforward.


Safely serialize reference fields

What changes are proposed in this pull request?

Enable loading a dataset even when referenced objects cannot be resolved.
Doesn't address the root cause of why this is happening though, but best not to alter/delete anything permanently until it's identified and correctly patched.

We decided to go with https://github.com/voxel51/fiftyone/pull/2970 and https://github.com/voxel51/fiftyone/pull/2971 instead.


bug

Hi @javiergonzper how are you ??
Please test this curiosity : create a file "2009-10-09 23-09-20 0002.jpg" in a subfolder, then move this in / (root) .. error 423 ...
:+1:
bye

Hi @marinotws, fine and you?
Uhm... could be a problem related with the server.. an HTTP423 code is:
423 Locked (WebDAV; RFC 4918)
The resource that is being accessed is locked.[17]

fine thanks ... mm no only this file ... yes 423 Locked ... maybe my database owncloud is dirty, you can try .


Add narwhals.maybe_convert_dtypes

pandas-like libraries have convert_dtypes: https://pandas.pydata.org/docs/reference/api/pandas.DataFrame.convert_dtypes.html
People use this to "promote" types - e.g., to promote all types to use the pyarrow-backed ones.
Polars doesn't have anything like this, because each type is just self-standing. E.g. pl.Int64 is just pl.Int64, there aren't any variants of it
So, the task here is to add a method maybe_convert_dtypes, which for pandas-like calls convert_dtypes, and for the rest is a no-op, just like how Narwhals already has other maybe_* funcs: https://narwhals-dev.github.io/narwhals/api-reference/narwhals/#narwhals.maybe_set_index

Working on this one.


Propose N-D smoothing

Types of changes

[ ] Docs change / refactoring / dependency upgrade
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Motivation and Context / Related issue
For working on arrays.
How Has This Been Tested (if it applies)

Checklist

[x] The documentation is up-to-date with the changes I made.
[x] I have read the CONTRIBUTING document and completed the CLA (see CLA).
[x] All tests passed, and additional code has been covered with new tests.

Thanks for the explanation @jrapin


Symbols should be sorted in doc

The order of the symbols in the document is flaky on every access.
e.g. https://jsr.io/@vcltk/ast/doc/~/Declaration

typealias first
namespace first

This issue may depend on deno_doc.

We are aware of the problem and @crowlKats is working on a fix.


Add typer.enable_rich() to allow Rich formatting to be disabled

Implements a feature similar to the one suggested by @anthonyraf in issue https://github.com/tiangolo/typer/discussions/622 to optionally disable Rich output.
Two simple test cases and a paragraph of documentation has also been added.
It affects all instances of Typer globally.
The code from the test cases illustrates how to use it:
import typer

typer.enable_rich(False)

app = typer.Typer()

@app.command()
def main(arg: str):  # pragma: no cover
    pass

if __name__ == "__main__":
    app()

import typer

typer.enable_rich_traceback(False)

app = typer.Typer()

@app.command()
def raise_():
    raise ValueError  # raise some error to test traceback output

if __name__ == "__main__":
    app()

Related issues:
https://github.com/tiangolo/typer/discussions/578, https://github.com/tiangolo/typer/discussions/622

Thanks for the review. I'll rename it as you suggested. I was thinking the same way about my previous naming choice anyway shortly after I published the PR but wanted to wait for a response before touching the code again. I don't know why I even named it that way to begin with. ;-)

Is there any plan on having this merged and released? It's a really valuable contribution as not scenarios where rich and typer are installed mean users want to use rich with typer.

I renamed the two functions and the thereby affected variables to be more semantically precise and more specific to their purpose. For the exposed functions I settled on the form: set_rich_* which is semantically more appropriate than the previous enable_rich_* form but keeps the boolean parameter to be able to conditionally re-enable it.

Hello everyone,
Is there any plan on having this merged ?
@JacobKochems can you set an assignee ?

@JacobKochems So you can't set @tiangolo as the assignee ?

@JacobKochems So you can't set @tiangolo as the assignee ?

No, I can't set anyone as assignee. Not myself and certainly not tiangolo. Nor would I want to. We are all guests here and it isn't my place to assign any task to anybody in this repository.

@coreegor Since you already @ed him I would assume that he already got an email notification but either didn't have time to respond yet or he time boxes his work on a given project at certain intervals.
If you need this feature rather sooner than later, I see two options:

Use my fork for the time being
Write a nice email directly to tiangolo

I'd prefer option 1. We all have to navigate a world where seemingly each and everything is competing for our attention. There is probably a reason why the last commit is from July 30.

@coreegor I just merged the latest changes from 'tiangolo:master'. Could you also do a review of this PR according to this? You can find this functionality under Files changed on the right hand side, named: Review changes.

To all following this PR: The original motivation for this PR is the issue #646 for which this is a possible workaround. However, I'd rather would like to see that original issue addressed as well. If some of you could review/replicate it, it would be much appreciated.

@JacobKochems I can replicate #646 with the latest Rich (13.7.0). To disable Rich though, I am doing the following:
import typer.core

typer.core.rich = None

...

@JacobKochems I can replicate #646 with the latest Rich (13.7.0). To disable Rich though, I am doing the following:
import typer.core

typer.core.rich = None

...

This is working well, thanks

@JacobKochems I can replicate #646 with the latest Rich (13.7.0). To disable Rich though, I am doing the following:
import typer.core

typer.core.rich = None

...

Thank you for this very simple workaround. Recently, yet another effort related to issues #578 and #622 has been made: #726. This in combination with the kwarg pretty_exceptions_enable=False of typer.Typer() seems to accomplish the same with fewer changes.
Tough, I'm not sure about the intended semantics regarding rich_markup_mode and if the proposed solution in #726 (or mine for that matter, I can't test it at the moment) addresses the printing of the rounded panel boxes and rich markup in general independently from each other. This seems to be a desired feature.

I'd also like to add a vote for and option that allows globally disabling rich. Just because rich is installed and available doesn't always mean it's appropriate to render rich output from the cli. I'd like to not have to revert to disabling features piecemeal e.g. prettr_exceptions_enabled=False + ...

Hi @JacobKochems,
The format-imports.sh script has indeed been deleted in PR #797. format.sh works as before, but ruff is a test dependency of typer and should be installed.
I noticed an issue with the test-cov-html.sh earlier this week as well, but I can't reproduce it anymore and now it just works fine on my system when I run
bash scripts/test-cov-html.sh

If this issue persists on your system, I suggest you open a discussion thread for that specifically so we can follow up there.
With respect to this PR - I want to review it in more detail, but I've looked at the related PR #726 first.

Ok, I did need to install pytest-cov and pytest-xdist. I didn't remember this to be necessary when I set up the repo the first time around but now bash scripts/test-cov-html.sh runs. It produces a lot of test fails but so does the master branch of tiangolo.

@JacobKochems: I put up a related PR here: https://github.com/tiangolo/typer/pull/859 - I think the same mechanism can probably be reused to implement the global switch you want. If you (or others) have time to review #859, I'd appreciate it!

As mentioned in my post on https://github.com/tiangolo/typer/pull/859#issuecomment-2163754301 I will close this PR in favor of #859 and attempt to reimplement this PR on top it.