cargo-dist: 0.8.0 -> 0.8.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for cargo-dist is: A tool for building final distributable artifacts and uploading them to an archive
meta.homepage for cargo-dist is: https://github.com/axodotdev/cargo-dist
meta.changelog for cargo-dist is: https://github.com/axodotdev/cargo-dist/blob/v0.8.2/CHANGELOG.md
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.8.2 with grep in /nix/store/kw12sqv8w0ax2sfxqqvq97pp41yb88pv-cargo-dist-0.8.2
found 0.8.2 in filename of file in /nix/store/kw12sqv8w0ax2sfxqqvq97pp41yb88pv-cargo-dist-0.8.2

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
cargo-dist

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/kw12sqv8w0ax2sfxqqvq97pp41yb88pv-cargo-dist-0.8.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A cargo-dist https://github.com/r-ryantm/nixpkgs/archive/49ef1a16fbfa81643056b39b920e57e6312ac3c3.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/49ef1a16fbfa81643056b39b920e57e6312ac3c3#cargo-dist

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/kw12sqv8w0ax2sfxqqvq97pp41yb88pv-cargo-dist-0.8.2
ls -la /nix/store/kw12sqv8w0ax2sfxqqvq97pp41yb88pv-cargo-dist-0.8.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

Maintainer pings
cc @figsoda @matthiasbeyer @mistydemeo for testing.

duplicate of #284845


Class should be String errors in FlashDevelop

In FlashDevelop (<IP_ADDRESS>), I'm seeing the following message pop up repeatedly in the Output window when working with Luxe projects:
--macro:1: character 0 : Class<snow.modules.interfaces.Audio> should be String
--macro:1: character 0 : { } should be String
--macro:1: character 0 : For function argument 'module'

I have a feeling this is interfering with code completion in FD also as it seems to popup more frequently when using code completion. Any suggestions on what this could be?

@davisable and others,
Following the discussion and fixes here,
http://www.flashdevelop.org/community/viewtopic.php?f=9&t=3529
Please let us know if that's better.


web: fix update nickname on settings page

We are fixing deletion wallet when user trying to update his nickname.

Codecov Report
Patch and project coverage have no change.

Comparison is base (5cef9ff) 41.63% compared to head (c9a9409) 41.63%.

:exclamation: Current head c9a9409 differs from pull request most recent head 529cd53. Consider uploading reports for the commit 529cd53 to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #394   +/-   ##
=======================================
  Coverage   41.63%   41.63%
=======================================
  Files          15       15
  Lines        1064     1064
=======================================
  Hits          443      443
  Misses        575      575
  Partials       46       46

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


Comparison table should not include changes from all versions

Original report on BitBucket by Taina Jääskeläinen.

It seems that the comparison table has now changed to include changes made to the vocabulary in any version.
The comparison table should only include:

Changes between the current version and the previous published version in that language.
The current version is the latest Editor version and in the general UI the latest published version

As SL Admin, I created a new version of TestingCV2 (en). I added a definition to Code10.
In the ‘Ready to translate SL [language code]’ pop-up the change log only included this change but the comparison table included the changes from all the previous versions as well.

The fix is: include in the comparison table only the changes made in the Editor to the new version. This means the same changes that are displayed in the change log.
This applies also to TL ‘Ready to be published’ pop-up and the language specific comparison tables of published versions.
Not sure if this issue came up because of he work done on the versioning.

don't know, why this issue came up as it was already solved by ae5259a861367872e3553d6f69a32e743a5b174d. closing.


Add looping functionality

Superstake (deposit LST / borrow SOL)
Stable loops (deposit higher yielding stable / borrow lower-yielding stable)

Should be implementable with any pair of assets (pairs can be configured manually)

hey, would like to contribute to this, though can i get a bit more context as for the deliverables?

Some thoughts:
Approach 1: Purely front-end based via manual looping.
Approach 2: Smart contract-based via flash loans.
Pure front-end based:

User deposits UXD, borrows USDC, swaps USDC to UXD, deposits UXD, repeat...
While this introduces no contract risk as it's all front-end based, it's horrible UX as it requires users to sign these txns one by one.
There's also a risk of the user failing any of these steps, e.g. depositing UXD after swapping from USDC, which leads to messy outcomes.
Also incurs swap fees for each looping and extraneous txn cost.

Smart contract-based:

User calls contract with UXD, which flash-loans USDC and swaps it to UXD, deposits UXD into the protocol (with initial input UXD as extra collateral), borrows USDC, and repays the USDC flash-loan.
This replaces swap fees with flash loan fee (which should be similar or cheaper) and lowers txn costs. There's no risk of txn stopping in between as it'll be all atomic.
Downside is it requires a contract, whether peripheral or core (ideally the former). On the bright side, since this functionality simply facilitates looping (like Jupiter swapping), it won't hold any funds and thus introduce any major risk.

Considering that looping significantly increases APY for users and activities for Marginfi, I think it'd be great to materialize the feature. Ideally, a periphery contract is deployed with this feature (and potentially more features) to lower core contract risk and enable easier future upgrades.
cc @edgarpavlovsky

This is live now on https://app.marginfi.com/looper


Analyse stdout instead of hdf5 by default

Small addition to avoid analysing hdf5 file and instead analyse the raw stdout text file. Data extraction is faster and avoids any issues with hdf5 when trying to analyse in the middle of a calculation.
I did notice the script seems slower than it aught to be which looks to be package import related. Worth revisiting.

Can we provide tests or examples for both text-based and hdf5-bases analyses?


EmailStr is parsed as str when setting default for EmailStr

Checks

[x] I added a descriptive title to this issue
[x] I have searched (google, github) for similar issues and couldn't find anything
[x] I have read and followed the docs and still think this is a bug

Bug
Output of python -c "import pydantic.utils; print(pydantic.utils.version_info())":
        pydantic version: 1.8.2
      pydantic compiled: True
            install path: /home/bapu/.cache/pypoetry/virtualenvs/ts-tools-LCrNCC4S-py3.8/lib/python3.8/site-packages/pydantic
          python version: 3.8.10 (default, Jun  2 2021, 10:49:15)  [GCC 9.4.0]
                platform: Linux-5.4.0-84-generic-x86_64-with-glibc2.29
optional deps. installed: ['dotenv', 'email-validator', 'typing-extensions']

from pydantic import BaseSettings, EmailStr
from typing import List

class Settings(BaseSettings):
    admins: List[EmailStr] =<EMAIL_ADDRESS>
The above code gives:
Expression of type "list[str]" cannot be assigned to declared type "List[EmailStr]"

Current workaround:
from pydantic import BaseSettings, EmailStr
from typing import List, cast

class Settings(BaseSettings):
    admins: List[EmailStr] = [cast(EmailStr, "mymail@mail.com")]

There's no parsing error here, just your ide giving a warning

In this case, Do I have to use cast as stated in the workaround to avoid warning?


Abandon

user@ubuntu$ grunt imagemin:task --force
Running "imagemin:task" (imagemin) task
grunt: ../deps/uv/src/unix/stream.c:1319: uv_read_stop: Assertion `!uv__io_activ
>prev) || stream->shutdown_req != ((void *)0) || stream->connect_req != ((void *
Abandon

user@ubuntu$ sudo grunt imagemin:task --force
Running "imagemin:task" (imagemin) task
grunt: ../deps/uv/src/unix/stream.c:1319: uv_read_stop: Assertion `!uv__io_active(&stream->io_watcher, 4) || !(&stream->write_completed_queue == (&stream->write_completed_queue)->prev) || !(&stream->write_queue == (&stream->write_queue)->prev) || stream->shutdown_req != ((void *)0) || stream->connect_req != ((void *)0)' failed.

My code:
imagemin: {
	task: {
		options: {
			progressive: false
		},
		files: [{
			expand: true,
			cwd: './',
			src: ['**/*.{png,jpg,gif,svg}'],
			dest: './'
		}]
	}
}

It is possible to ignore failed files ?

Looks like your Node binary is corrupt. Try upgrading Node. Nothing to do with this plugin though.

Reinstall works, thanks !


fix prodsec issue woth old version of urllib3 lib

Codecov Report

Merging #96 (8e5e941) into develop (29c0ec2) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           develop     #96   +/-   ##
=======================================
  Coverage     7.52%   7.52%
=======================================
  Files            9       9
  Lines          226     226
=======================================
  Hits            17      17
  Misses         209     209

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 29c0ec2...8e5e941. Read the comment docs.

:tada: This PR is included in version 1.0.8-develop.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Float(s) lost

Whenever I try to use a sidenote within one of the environments from mdftheorems I get a "Float(s) lost" error. If I try to use a marginnote it works fine. Is there any way to make it work?

Unfortunately it is a feature of LaTeX that floats cannot be nested, otherwise they 'get lost'. However, if \marginnote works for you, you can add a new macro which will use it to emulate a sidenote:
\makeatletter
\NewDocumentCommand\thmsidenote{ o O{0pt} +m }{%
    \IfNoValueOrEmptyTF{#1}{%
        \refstepcounter{sidenote}% This command has been moved here
    }{%
    }%
    \sidenotemark[#1]%
    \marginnote[#2]{\thesidenote: #3}%
    \@sidenotes@multimarker%
}
\makeatother

This definition should be placed either in the preamble of the main.tex or somewhere in styles/kao.sty.
PS
\marginnote does not work for me inside definitions or theorems: which environment are you using?

I ran into this today as well. Same error with \marginnote too, as @fmarotta mentioned.
@cpazinatto Did you get this fixed?
@fmarotta Any plans to implement a fix?

I will see what I can do in the next few days. As a possible temporary fix, and as a test that would help me to design a better solution, could you please tell me what happens if you add a vertical shift of 0pt to the marginnote? I.e. \marginnote[0pt]{...}

Oh, interesting. Yeah, adding any offset to \marginnote or \sidenote seems to make the error go away. I tested it with the assumption environment and with the margintable environment. Because the margintable is actually in the margin, the \sidenote or \marginnote gets put on top of the margintable. Manual offsetting fixes this. However, manual offsetting is not super fun when writing a book; floats that play well together is definitely preferable.

I totally agree. Thanks for doing that little test: when the offset is added, the \marginnote (or the \sidenote) becomes fixed, rather than floating. So in the latest commit, where, sadly, I forgot to reference this issue, I introduced the \ifInFloatingEnvir conditional: if the note is already inside a floating environment, like a theorem or a margintable, it is printed in non-floating mode.
This should fix the original issue, but I did not understand the problem with margintables: do you @bradyneal want to use \sidenote inside a margintable? If so, should the note refer to the caption or the table itself? And what is the expected behaviour? (Sorry for the many questions, they are really just to understand and see if I can do something about it.)

Nice, it works for me in the simple class, but it gives the same bug as before when the sidenote/sidecite is nested inside an enumerate environment that is inside of a floating environment.
Regarding the margintable, I don't think I was referencing the note inside the table/caption. Rather, I was just using sidenote somewhere near the margintable, and the sidenote was landing on top of the margintable. So the all the sidnotes, marginnotes, and sidecites seemed to float nicely and not land on top of each other, but the margintables and marginfigures did not. Is that more clear?

Yes, thanks. Is this the same problem as in #49? Anyways, does the sidenote overlap with the margintable, or is it just uncomfortably close to it?

Good, I added a comment on that issue trying to address this problem.
As regards the other problem, the one that occurs when the sidenote/sidecite is in an environment which itself is in a floating environment, I think it is because so far I just check the innermost environment, not the parent environments. I'll try to fix it asap.

I have now added a flag that is toggled at the beginning of the mdframed and minipage environments. If that flag is active, then the marginnote is printed in fixed, rather than floating, mode.


New package: Microsoft.VisualStudio.Extensions.TypeScript version 4.3

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[x] Resolve #184547

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.
Manual validation

Microsoft Reviewers: Open in CodeFlow

Hi @,
Do you approve of these changes?
(Automated response - build 897.)

Verify:

This package is an extension for Visual Studio. I'm guessing that a dependency isn't specified, because any version of Visual Studio could be used. The package is mostly .ts files, but has a few .dlls, so I believe it might have enough compiled files to pass that rule.


Add secret link service

closes  https://github.com/inveniosoftware/invenio-rdm-records/issues/530

closed in favor of #https://github.com/inveniosoftware/invenio-rdm-records/pull/595


Resultant Plots

Hi guys, I am wondering that if you can add the plots of your work to the main so that it will be easier to figure out.

Exactly, also we can compare our results with yours.

We have done it now. Please check them.


[Bug] example argument not used in docs ui when using multiple Body() params

Example
from fastapi import Body, FastAPI

app = FastAPI()

@app.post("/")
def test(a: str = Body(..., example="A"), b: str = Body(..., example="B")):
    pass

This results in the following example value for the request body in the docs ui (/docs):
{
  "a": "string",
  "b": "string"
}

Description

Open the browser and call the endpoint /docs.
When using multiple Body() params, the example value for the request body consists of the individual schemas and ignores the provided examples.
But I expected it to create the following example value:

{
  "a": "A",
  "b": "B"
}

Environment

Docker image: tiangolo/uvicorn-gunicorn-fastapi:python3.8-2021-06-09
OS: Linux
FastAPI Version: 0.65.2
Python version: 3.8.6

Вы следующие

it still remains on 0.85.0. When I have a string parameter and its example, it works.
But if I have multiple string parameter and examples, it just shows "string".

Ah, yeah, if you want one example you should have one single Pydantic model and not several independent Body() parameters, that would create independent example objects but Swagger UI can only show one example object.

Sorry for the long delay! 🙈  I wanted to personally address each issue/PR and they piled up through time, but now I'm checking each one in order.


Android Activity com.Mobile has leaked IntentReceiver com.opentok.android.DefaultAudioDevice$1@95eae3e

Android Activity com.Mobile has leaked IntentReceiver com.opentok.android.DefaultAudioDevice$1@95eae3e that was originally registered here. Are you missing a call to unregisterReceiver()? android.app.IntentReceiverLeaked: Activity com.Mobile has leaked IntentReceiver com.opentok.android.DefaultAudioDevice$1@95eae3e that was originally registered here. Are you missing a call to unregisterReceiver()? at android.app.LoadedApk$ReceiverDispatcher.(LoadedApk.java:1159)
I am calling servicce.onpuse and service.onresume But still facing the issue any boday can help me

I am also facing the same issue.

This should be fixed in 2.17.0, please reopen this if it is still crashing.

I'm experiencing the same thing in my app. I tried to clone the samples and the same is happening there.
It happens on screen rotation.
I've tried AudioDeviceManager.getAudioDevice().onPause() (which is suggested in another issue) and it didn't help.
I'm using version 2.17.1
Steps to reproduce:

Connecting to session
Rotate phone, resulting in
Activity has leaked IntentReceiver com.opentok.android.DefaultAudioDevice$1@c9ae257
The app does not crash at this point
Rotate back, resulting in
Activity}: java.lang.IllegalArgumentException: Receiver not registered: com.opentok.android.DefaultAudioDevice$3@1ec17f1
App crashes at this point


When working with tabular.io, they auto create system and default databases. SR cannot "use system" or "use default"

system and default are reserved works.  I get an error.

have to use
use tabular.`default`;


Use visibility from fields

This PR changes the macro to make use of the visibility set for fields and structs. With this, structs and fields are no longer public by default. To make this work with new() and split(), the visibility of the merged struct is used for those generated functions as well.

Thanks!


Improve ContextProvider

Which issue does this PR close?

Renaming like SchemaProvider::table_names, add docs and remove deprecated code.
Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

I'll leave this open for a day as it is an API change, in case anyone else wants a chance to review


test: rfc: refactor: run_float_fn -> run_float_fn_approx

Generally one should not do exact floating-point comparisons anywhere, so maybe we should approx the default thing to do. If you really wanted an exact check you could pass abs=0.0, rel=0.0 I guess....

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 91.67%. Comparing base (992b138) to head (fecff61).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #507   +/-   ##
=======================================
  Coverage   91.67%   91.67%
=======================================
  Files          59       59
  Lines        6138     6138
=======================================
  Hits         5627     5627
  Misses        511      511

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Maintain Chapter 11 Advanced features

[x] should be renamed Advance Cairo (features)
[x] Move custom data structures in this section
[ ] 11-1: maybe add precisions that there is no need to use the #[generate_trait] attribute here as it is just one implementation of Add trait method of corelib
[ ] 11-2: allows -> allow
[ ] 11-3: update_with is a part of the extension trait for hash state accumulators called HashStateExTrait, this should be shortly mentioned with other traits, especially because update_with is used multiple times in the section
[ ] 11-3: A few errors with imports (pedersen instead of poseidon at some place)
[ ] 11-3: Serde and copy attributes are not necessary for StructToHash
[ ]  11-3: Serialization could be shortly introduced as it is used for the poseidon struct example
[ ] 11-3: Advanced hashing : example of hashing a struct (instead of function) that contains a span
[ ] 11-3: following traits and function

Hello @enitrat and @TAdev0 ! I allowed myself to fix some of the points that I could find from the list.
The points I wasn't able to fix/find:
11-1: maybe add precisions that there is no need to use the #[generate_trait] attribute here as it is just one implementation of Add trait method of corelib
11-3: A few errors with imports (pedersen instead of poseidon at some place)
11-3: Serde and copy attributes are not necessary for StructToHash
They might have been already fixed/moved.


[WIP] 5a. Implementation Across Sectors

@vexvexctor has volunteered to lead this section. It may grow to include others' contributions as well.

@ethanwu2011 has volunteered to contribute healthcare-related applications here.


simplify connection-sets output

example:
protocol: TCP src-ports: 1-442,444-65535 dst-ports: 443; protocol: TCP src-ports: 443
should be simplified to
protocol: TCP dst-ports: 443; protocol: TCP src-ports: 443
explore other examples to generalize the problem

Should probably move this issue to models repo

Move to models repo. See here


[PPP-4760]-Vulnerable Component: Apache CXF 3.3.1

PPP-4760-Vulnerable Component: Apache CXF 3.3.1

 Note:

Frogbot also supports Contextual Analysis, Secret Detection, IaC and SAST Vulnerabilities Scanning. This features are included as part of the JFrog Advanced Security package, which isn't enabled on your system.

🐸 JFrog Frogbot

:white_check_mark: Build finished in 23m 39s
Build command:
mvn clean verify -B -e -Daudit -Djs.no.sandbox -pl plugins/connections

:exclamation: No tests found!
:information_source:  This is an automatic message


bug fix reload

Fix the issue where the header background becomes transparent on page refresh.

Hi @Dacong-wu
The header comes out transparent so that if you have an image or gradient in the Hero it can be seen. Then when you scroll, a colored background is applied.
With that fix that you developed in the PR we would lose that possibility that I mentioned to you.
Usually the "bug" appears on very slow connections. It usually doesn't happen.

Surely some corrections need to be made, but I have a feeling that it is more about the script used.


Fix nightly matrix sync by inheriting secrets

Previous syncs failed due to it missing the password. The password needs to be inherited from the calling action


Introduce Tesla.Client with pre/post middleware

This change is backward compatible:

Tesla.build_client API stays the same
one can still use simple function as a client

Tesla.Client struct can hold any custom fun (as before)
but can now hold pre and post middleware lists.
pre works the same way as one given to build_client.
post middleware is injected right before adapter.
It can be used as a regular middleware but can be also
used to dynamically override adapter in runtime.
This is mostly useful in testing:
defmodule MyAPI do
  use Tesla

  def help(client \\ %Tesla.Client{}) do
    get(client, "/help")
  end
end

client = Tesla.build_adapter fn env ->
  %{env | status: 200, body: "new body"}
end

assert %{body: "new body"} = MyAPI.get(client)

I believe these two should be referenced here #94 #97 :)

As far as #94 is concerned there is one more way to do this (and I'm surprised I haven't thought about it earlier)
defmodule Cached do
  use Tesla

  adapter :cache

  def cache(env) do
    cond do
      String.ends_with?(env.url, "/cached") ->
        %{env | body: "cached", status: 304}
      true ->
        Tesla.run_default_adapter(env)
    end
  end
end

All one needs to do is to write a custom adapter that wraps other adapter.
I've added Tesla.run_default_adapter/1,2 as part of this PR to make it more convenient to use the default adapter which can be configured globally.


Error when sheet has +197 rows

Getting this error when use XLSX.utils.sheet_to_json with sheet having +197 rows
(node:990) UnhandledPromiseRejectionWarning: Error: read ECONNRESET
at _errnoException (util.js:992:11)
at TCP.onread (net.js:618:25)
(node:990) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 807)
Any idea why that?

Sorry, it was a problem on other dependency


chore: upgrade pnpm to latest version

Summary of changes

Checklist

[x] I have read and followed the contribution guidelines.
[x] I have read through the Code of Conduct and agree to abide by the rules.
[x] This PR is for one of the available issues and is not a PR for an issue already assigned to someone else.
[x] My PR title has a short descriptive name so the maintainers can get an idea of what the PR is about.
[x] I have provided a summary of my changes.

closes #962

Linting test is failing


No workflow cancellation if event is issue_comment

Seen in jupyterlab/jupyterlab
As soon as a comment on any PR is added, workflows generated from issue_comment get cancelled even if the comment comes from another PR (and it will also happen if the comment comes from the same PR whatever its content). So for now skip looking for duplicated workflow if the event is an issue comment.
Example of unwanted workflow cancellation: https://github.com/jupyterlab/jupyterlab/actions/runs/1952374323

Thanks @blink1073 I'll push the new state on heroku.


When linking statically a the Python wrapper on Macos, also include libicu

Motivation
On boost 1.68 that it's shipped by brew on MacOS the dependencies have changed for boost regex to now include libicu. When statically linking the _pulsar.so Python plugin, we need to ensure libicu is included. (this works fine when doing regular non-static link, since linking with boost_regex will automatically pull the other shared libs).

run java8 tests

run cpp tests

run java8 tests

run java8 tests

run java8 tests

Merged in 2.3.1 at
22c21ae


Fixed bug in compute/forward_pass.py line 359 that caused error with …

…2D patches

Fixes #ISSUE_NUMBER
Proposed Changes

When using a 2D patch such as (2048, 2048), during validation the patch shape is (2048, 2048, 1). This triggers the else statement in forward_pass.py, causing an issue when trying to squeeze().

Checklist

[ x] CONTRIBUTING guide has been followed.
[ x] PR is based on the current GaNDLF master .
[ x] Non-breaking change (does not break existing functionality): provide as many details as possible for any breaking change.
[ ] Function/class source code documentation added/updated (ensure typing is used to provide type hints, including and not limited to using Optional if a variable has a pre-defined value).
[ ] Code has been blacked for style consistency and linting.
[ ] If applicable, version information has been updated in GANDLF/version.py.
[ ] If adding a git submodule, add to list of exceptions for black styling in pyproject.toml file.
[x] Usage documentation has been updated, if appropriate.
[ ] Tests added or modified to cover the changes; if coverage is reduced, please give explanation.
[x ] If customized dependency installation is required (i.e., a separate pip install step is needed for PR to be functional), please ensure it is reflected in all the files that control the CI, namely: python-test.yml, and all docker files [1,2,3].

recheck


⚠ Bofrost.be has degraded performance

In ab85dff, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1737 ms

Resolved: Bofrost.be performance has improved in 2b7e7e7 after 7 minutes.


ADD recent developer name / photo to ASCT+B reporter Pramod Bhakta

Add name and photo to https://hubmapconsortium.github.io/ccf-asct-reporter/ at bottom of page with contributors.
Photo and information from https://cns.iu.edu/previous_team.html

Pramod Bhakta
Angular Developer
<EMAIL_ADDRESS>2022.05.15 - 2023.08.04

Added this in PR https://github.com/hubmapconsortium/ccf-asct-reporter/pull/739

Thanks!


consider adding hostname example in connect_to_cluster.rst

to reinforce the fact that you land on a login node efter login

Will merge in a few minutes.


health-check and storage-metrics support for fluent-bit

PR for adding support to enable health-check and storage-metrics for fluent-bit. (fixes #1286)

Thank you for your contribution! The CI fails because there are generated files which had not been pushed. Could you please add them as well? make check-diff should tell you what the difference is, currently it seems that docs/configuration/crds/v1beta1/fluentbit_types.md is the culprit.

Thank you for your contribution! The CI fails because there are generated files which had not been pushed. Could you please add them as well? make check-diff should tell you what the difference is, currently it seems that docs/configuration/crds/v1beta1/fluentbit_types.md is the culprit.

Thanks @OverOrion :) That should be sorted out now.


Cache charset lookups to avoid synchronization

Charset.forName caches only two entries before blocking threads.  If
abused, as property tests abuse them, this can cause a severe
bottleneck.  We can iterate over the available character sets and
their aliases, prewarm the cache, and avoid the method altogether.

Codecov Report

Merging #1059 into release-0.15.x will increase coverage by 0.04%.
The diff coverage is 100%.

@@                Coverage Diff                 @@
##           release-0.15.x    #1059      +/-   ##
==================================================
+ Coverage           69.45%   69.49%   +0.04%
==================================================
  Files                 212      212
  Lines                4658     4661       +3
  Branches              451      458       +7
==================================================
+ Hits                 3235     3239       +4
+ Misses               1423     1422       -1

Impacted Files
Coverage Δ

core/src/main/scala/org/http4s/Charset.scala
94.11% <100%> (+1.26%)
:arrow_up:

core/src/main/scala/org/http4s/HttpVersion.scala
100% <0%> (+6.25%)
: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 458a8f8...ef9e11a. Read the comment docs.

For future reference, there's an open JDK bug for the only-cache-two behavior:
https://bugs.openjdk.java.net/browse/JDK-6717772
Given its age, it's not likely to be fixed soon.

Cache is about 700 entries on my machine.
A dynamic cache would be a good solution for services that handle multiple real-world character sets.  It does not solve the problem of malicious requests iterating through fake character sets and triggering synchronized lookups.
If we made java.nio.charset.Charset optional on Charset, we would get a few benefits:

An HTTP charset is just a token.  Nothing promises it points to a charset the JVM understands.
Unless we're encoding or decoding, it doesn't need to be resolved.
Users could write their own codecs based on Charset without regard to our resolution strategy, though they'd have to shadow ours.
We could make a configurable org.http4s.Charset => Option[java.nio.charset.Charset] strategy, though I'll bet 99.999% of users aren't registering them dynamically after the app starts.
Though maybe parameterizing the way HTTP is parsed in general is a good way around some other problems, like registering custom headers into a concurrent map.

So a case class Charset private (nioCharset: Either[String, NioCharset]) or pass in the header parser as a Map[String, NioCharset] in case of anything but the default.

I've started experimenting with a Charset that has an optional NIO charset, which is a similar idea to the Either.  I think I like it: we can implement the cache that makes sense for all but the most exotic use cases, and those people are welcome to call Charset.forName at their own risk.
I'm having a hard time conceptualizing how and why a CharsetProvider would be registered after org.http4s.Charset is loaded.

case req @ POST -> _ =>
  req.charset.fold("oops")(_.nioCharset.name)

I ran a quick benchmark with all UTF-8 and by fake-${counter}, where counter is incremented on each request:

Charset
Without Patch
With Patch

constant UTF-8
70000
68000

fake-${counter}
12000
67000

The bottleneck is real, but the implementation may hurt a bit in the common case.

So, I benchmarked Charset directly.
[info] NioCharsetBench.cachedInvalid   thrpt   40    2675162.855 ±   103986.224  ops/s
[info] NioCharsetBench.cachedValid     thrpt   40   83610875.726 ±  2972073.798  ops/s
[info] NioCharsetBench.forNameInvalid  thrpt   40      34237.259 ±      783.657  ops/s
[info] NioCharsetBench.forNameValid    thrpt   40  433837293.317 ± 13169507.210  ops/s

This makes valid lookups 5 times worse to make invalid ones 78 times better.

I added a benchmark, held my nose, and switched to a java.util.HashMap.
Costs:

Cache init at startup.  I haven't noticed.
28 nanoseconds to parse charsets (on my machine).
Can't register CharsetProvider after org.http4s.Charset is loaded.  I don't know how to do this, but I'll bet it was done once, in 2003, on WebSphere, for a since extinct dialect of Klingon.

Benefits:

Can't lock the request pool by repeatedly POSTing Content-Type: text/plain; charset=i-am-a-jerkface
Less allocations of Charset

We'd be in company with Tomcat with this approach.


.NET API Browser main page: missing parts in Description items

Link to article:
https://docs.microsoft.com/en-us/dotnet/api/?view=netcore-2.1
Problem:
In the column Description, parts that refer to links to other sections are missing. e.g. for System.Drawing Namespace:
"The  namespace provides access to GDI+ basic graphics functionality. More advanced functionality is provided in the , , and  namespaces"
should be
"The System.Drawing namespace provides access to GDI+ basic graphics functionality. More advanced functionality is provided in the System.Drawing.Drawing2D, System.Drawing.Imaging, and System.Drawing.Text namespaces."

@dend @presleyy Can you take a loo at this issue?

Yes, please fix this. Tagging @Sampy too.
I've asked about this internally on Teams the other day as well. The API browser simply drops the text from see cref links.

This is also a duplicate of #637

close as duplicate, we will track it on #637


[explorer] add ids to make elements searchable

I am prototyping some external scraping tooling to assess explorer liveness and these ids will make it easier to understand.
We could use other properties to identify these elements but I think element id is the easiest one to get us moving in the near term.
Test Plan
» pnpm explorer test

> sui-monorepo@ explorer /Users/jkjensen/mysten/sui
> pnpm --filter ./apps/explorer "test"

> sui-explorer@ test /Users/jkjensen/mysten/sui/apps/explorer
> pnpm test:unit

> sui-explorer@ test:unit /Users/jkjensen/mysten/sui/apps/explorer
> vitest run

 RUN  v0.32.0 /Users/jkjensen/mysten/sui/apps/explorer

 ✓ src/pages/transaction-result/programmable-transaction-view/__test__/utils.test.ts (3)
 ✓ src/__tests__/unit.test.ts (5)

 Test Files  2 passed (2)
      Tests  8 passed (8)
   Start at  14:11:38
   Duration  914ms (transform 397ms, setup 0ms, collect 609ms, tests 8ms, environment 162ms, prepare 113ms)

If your changes are not user-facing and not a breaking change, you can skip the following section. Otherwise, please indicate what changed, and then add to the Release Notes section as highlighted during the release process.
Type of Change (Check all that apply)

[ ] protocol change
[ ] user-visible impact
[ ] breaking change for a client SDKs
[ ] breaking change for FNs (FN binary must upgrade)
[ ] breaking change for validators or node operators (must upgrade binaries)
[ ] breaking change for on-chain data layout
[ ] necessitate either a data wipe or data migration

Release notes

If you need this can you use data-testid instead  of ids?

But also I’m a little concerned about this approach given we pretty frequently redesign these surfaces.


DasharoBootPolicies: extract IOMMU setting to separate driver

This is the only setting that requires edk2-platforms, and only for PcdVTdPolicyPropertyMask. As not all platforms support VTd, fetching edk2-platforms unconditionally isn't necessary.
This change creates the possibility to build Dasharo edk2 payload without edk2-platform.

Closing this PR as it has been included in rebased branch of edk2: https://github.com/Dasharo/edk2/pull/133#issuecomment-2169499292


Can't import datasets: partially initialized module 'fsspec' has no attribute 'utils'

Describe the bug
Unable to import datasets
Steps to reproduce the bug
from datasets import Dataset, DatasetDict

Expected results
The import works without errors
Actual results
AttributeError                            Traceback (most recent call last)
<ipython-input-37-c8cfcbe62127> in <module>
     11 # from tqdm import tqdm
     12 # import torch
---> 13 from datasets import Dataset
     14 # from transformers import Trainer, TrainingArguments, AutoModel, AutoTokenizer, AutoModelForMaskedLM, DataCollatorForLanguageModeling
     15 # from sentence_transformers import SentenceTransformer

~/.local/lib/python3.8/site-packages/datasets/__init__.py in <module>
     31     )
     32
---> 33 from .arrow_dataset import Dataset, concatenate_datasets
     34 from .arrow_reader import ArrowReader, ReadInstruction
     35 from .arrow_writer import ArrowWriter

~/.local/lib/python3.8/site-packages/datasets/arrow_dataset.py in <module>
     46 )
     47
---> 48 import fsspec
     49 import numpy as np
     50 import pandas as pd

~/.local/lib/python3.8/site-packages/fsspec/__init__.py in <module>
     10 from . import _version, caching
     11 from .callbacks import Callback
---> 12 from .core import get_fs_token_paths, open, open_files, open_local
     13 from .exceptions import FSTimeoutError
     14 from .mapping import FSMap, get_mapper

~/.local/lib/python3.8/site-packages/fsspec/core.py in <module>
     16     caches,
     17 )
---> 18 from .compression import compr
     19 from .registry import filesystem, get_filesystem_class
     20 from .utils import (

~/.local/lib/python3.8/site-packages/fsspec/compression.py in <module>
     68
     69
---> 70 register_compression("zip", unzip, "zip")
     71 register_compression("bz2", BZ2File, "bz2")
     72

~/.local/lib/python3.8/site-packages/fsspec/compression.py in register_compression(name, callback, extensions, force)
     44
     45     for ext in extensions:
---> 46         if ext in fsspec.utils.compressions and not force:
     47             raise ValueError(
     48                 "Duplicate compression file extension: %s (%s)" % (ext, name)

AttributeError: partially initialized module 'fsspec' has no attribute 'utils' (most likely due to a circular import)

Environment info

datasets version: 1.18.4
Platform: Jupyter notebook
Python version: 3.8.10
PyArrow version: 7.0.0

Update: "python3 -c "from from datasets import Dataset, DatasetDict" works, but not if I import without the python3 -c

Also seeing this error: AttributeError: partially initialized module 'fsspec' has no attribute 'utils' (most likely due to a circular import)

Hi @arunasank, thanks for reporting.
It seems that this can be caused because you are using an old version of fsspec: the reason why it works if you run python3 seems to be that python3 runs in a Python virtual env (with an updated version of fsspec); whereas the error arises when you run the import from other Python virtual env (with an old version of fsspec).
In order to fix this, you should update fsspec from within the "problematic" Python virtual env:
pip install -U "fsspec[http]>=2021.05.0"

I'm closing this issue, @arunasank.
Feel free to re-open it if the problem persists.

from lightgbm import LGBMModel,LGBMClassifier, plot_importance
after importing lib getting (partially initialized module 'fsspec' has no attribute 'utils' (most likely due to a circular import) error, can help me

@deepakmahtha I think you are not using datasets: this is the GitHub repository of Hugging Face Datasets.
If you are using lightgbm, you should report the issue to their repository instead.
Anyway, we have proposed a possible fix just in a comment above: to update fsspec.
https://github.com/huggingface/datasets/issues/3902#issuecomment-1066517824


{Containerapp} Update recording files

This checklist is used to make sure that common guidelines for a pull request are followed.
Related command

In azure-cli, the api-version has been upgraded to 2024-03-01, the azure-cli-extensions need to rerun and update recording files.
General Guidelines

[ ] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
[ ] Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)
[ ] My extension version conforms to the Extension version schema

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

Containerapp

If you can help to resolve that will be appreciated.
Hi @zhoxing-ms

We don't know this person, but he auto commented in all PRs.

I just reported abuse or spam for this person.


Enable hover for builtin methods

Adds the method hover also for builtin methods (not even adt knows this btw). I also reformatted all the builtin signatures.

👍


feat: add load more pager to filters

Adding load more pager to filters. Added the pager-has-more property to d2l-filter-dimension-set that when set will render a d2l-pager-load-more button (rendering done in d2l-filter).
The handling of pagination and adding/removing the property from the dimension is still the responsibility of the consumer (see the added demo as an example).
Rally: US153373

Also, just realized the focus is being lost after hitting "load More".  @dbatiste, is this something the list is supposed to handle automatically, or the consumer has to?

Also, just realized the focus is being lost after hitting "Load More". @dbatiste, is this something the list is supposed to handle automatically, or the consumer has to?

I don't think focus should be lost. There were some previous changes that would disable the button while loading, and that did cause focus to be lost, but that was removed a long time ago. Focus should stay on the button while loading, and when the consumer calls e.detail.complete() we focus the first newly loaded item (based on index). We do ignore subsequent clicks while loading.

Also, just realized the focus is being lost after hitting "Load More". @dbatiste, is this something the list is supposed to handle automatically, or the consumer has to?

I don't think focus should be lost. There were some previous changes that would disable the button while loading, and that did cause focus to be lost, but that was removed a long time ago. Focus should stay on the button while loading, and when the consumer calls e.detail.complete() we focus the first newly loaded item (based on index). We do ignore subsequent clicks while loading.

@GZolla Is there a defect or something to track this? Just don't want to forget about it


Try #2 - correct git service handling of non-root git dir

Resolves #142

@nimrodkor Not sure if this is related to your change or a different issue (happy to split off another issue if you'd like). It seems like one of the files in the sub-dir I tested this on was still unable to fetch git blame info:
2021/06/18 09:01:16 [WARNING] Unable to get git blame for file envs/cetus/receipts/receipts.tf: failed to get blame for latest commit of file envs/cetus/receipts/receipts.tf because of error contents and commits have different length
2021/06/18 09:01:29 [WARNING] Failed to tag receipts_bucket with git tags, err: failed to get blame for latest commit of file envs/cetus/receipts/receipts.tf because of error failed to get blame for latest commit of file envs/cetus/receipts/receipts.tf because of error contents and commits have different length

You can see that one of the resources was only tagged with yor_trace:
New Resources Traced (4):
+------------------------------------------+--------------------------------+----------------------+------------------------------------------+--------------------------------------+
|                   FILE                   |            RESOURCE            |       TAG KEY        |                TAG VALUE                 |                YOR ID                |
+------------------------------------------+--------------------------------+----------------------+------------------------------------------+--------------------------------------+
| envs/cetus/receipts/receipts.tf          | receipts_bucket                | yor_trace            | 0a4a88ef-f713-4f9f-8e50-effadb1422b4     | 0a4a88ef-f713-4f9f-8e50-effadb1422b4 |
+                                          +                                +----------------------+------------------------------------------+                                      +
|                                          |                                | yor_trace            | 0a4a88ef-f713-4f9f-8e50-effadb1422b4     |                                      |
+------------------------------------------+--------------------------------+----------------------+------------------------------------------+--------------------------------------+
| envs/cetus/receipts/secretsmgr.tf        | aws_iam_policy.secrets_manager | yor_trace            | 0b7c9745-0ded-438b-8b59-7876b1b4e712     | 0b7c9745-0ded-438b-8b59-7876b1b4e712 |
+                                          +                                +----------------------+------------------------------------------+                                      +
|                                          |                                | git_org              | waveaccounting                           |                                      |
+                                          +                                +----------------------+------------------------------------------+                                      +
|                                          |                                | git_repo             | keller                                   |                                      |
+                                          +                                +----------------------+------------------------------------------+                                      +

Not sure what error contents and commits have different length means :)


Changing themes results in flashing, and theme doesn't get set

Oni Version: <IP_ADDRESS>58
Neovim Version (Linux only): (i'm using neovim version 0.2.2 set with the "debug.neovimPath" key, not sure if it relevant)
Operating System: macOS Mojave 10.14.1
Issue: Changing themes results in window flashing forever, and theme doesn't get set. A related bug is also that selecting the theme with the mouse (not the arrow keys) doesn't result in the flashing, but doesn't actually set the theme, it just closes the command pallet.
In the console I don't get any errors/warnings, but this is some of the output
bundle.js:1 [NeovimEditor] Colors changed: nord - background: #2e3440 foreground: #d8dee9
bundle.js:1 [NeovimEditor] Colors changed: gruvbox - background: #282828 foreground: #ebdbb2
bundle.js:1 [NeovimEditor] Colors changed: nord - background: #2e3440 foreground: #d8dee9
bundle.js:1 [NeovimEditor] Colors changed: gruvbox - background: #282828 foreground: #ebdbb2
bundle.js:1 [NeovimEditor] Colors changed: nord - background: #2e3440 foreground: #d8dee9
bundle.js:1 [NeovimEditor] Colors changed: gruvbox - background: #282828 foreground: #ebdbb2
bundle.js:1 [NeovimEditor] Colors changed: nord - background: #2e3440 foreground: #d8dee9
bundle.js:1 [NeovimEditor] Colors changed: gruvbox - background: #282828 foreground: #ebdbb2
bundle.js:1 [NeovimEditor] Colors changed: nord - background: #2e3440 foreground: #d8dee9
bundle.js:1 [NeovimEditor] Colors changed: gruvbox - background: #282828 foreground: #ebdbb2
bundle.js:1 [NeovimEditor] Colors changed: nord - background: #2e3440 foreground: #d8dee9
bundle.js:1 [NeovimEditor] Colors changed: gruvbox - background: #282828 foreground: #ebdbb2
bundle.js:1 [NeovimEditor] Colors changed: nord - background: #2e3440 foreground: #d8dee9
bundle.js:1 [NeovimEditor] Colors changed: gruvbox - background: #282828 foreground: #ebdbb2
bundle.js:1 [NeovimEditor] Colors changed: nord - background: #2e3440 foreground: #d8dee9
bundle.js:1 [NeovimEditor] Colors changed: gruvbox - background: #282828 foreground: #ebdbb2
bundle.js:1 [NeovimEditor] Colors changed: nord - background: #2e3440 foreground: #d8dee9
bundle.js:1 [NeovimEditor] Colors changed: gruvbox - background: #282828 foreground: #ebdbb2
bundle.js:1 [NeovimEditor] Colors changed: nord - background: #2e3440 foreground: #d8dee9
bundle.js:1 [NeovimEditor] Colors changed: gruvbox - background: #282828 foreground: #ebdbb2
bundle.js:1 [NeovimEditor] Colors changed: nord - background: #2e3440 foreground: #d8dee9
bundle.js:1 [NeovimEditor] Colors changed: gruvbox - background: #282828 foreground: #ebdbb2
bundle.js:1 [NeovimEditor] Colors changed: nord - background: #2e3440 foreground: #d8dee9
bundle.js:1 [NeovimEditor] Colors changed: gruvbox - background: #282828 foreground: #ebdbb2
bundle.js:1 [NeovimEditor] Colors changed: nord - background: #2e3440 foreground: #d8dee9
bundle.js:1 [NeovimEditor] Colors changed: gruvbox - background: #282828 foreground: #ebdbb2
bundle.js:1 [NeovimEditor] Colors changed: nord - background: #2e3440 foreground: #d8dee9
bundle.js:1 [NeovimEditor] Colors changed: gruvbox - background: #282828 foreground: #ebdbb2

A gif demonstrating the problem is shown below:

Expected behavior: The theme should get set
Actual behavior: The theme doesn't get set, and the window flashes
Steps to reproduce:

Open the command pallet with CMD+SHIFT+P
Choose "Themes: Choose Theme"
Select a theme using the arrow keys (not the cursor)

Hello @lukakerr !
AFAICS, this bug seems to have already been reported here:  #2470

Since this looks the be covered already, I'll close this out! Cheers for reporting though, and to @marcagba for finding the duplicate!


mvsim: 0.13.0-1 in 'jazzy/distribution.yaml' [bloom]

Increasing version of package(s) in repository mvsim to 0.13.0-1:

upstream repository: https://github.com/MRPT/mvsim.git
release repository: https://github.com/ros2-gbp/mvsim-release.git
distro file: jazzy/distribution.yaml
bloom version: 0.11.2
previous version for package: 0.12.0-1

mvsim
* Add ROS1 generic launcher
* Port demos to the generic ros2 launch file so all rosargs are now exposed in all demos
* create generic launch_world ros2 file
* ROS node: new param "publish_tf_odom2baselink"
* ROS2 warehouse demo launch: add use_rviz argument
* Update README.md: Mark ROS2 Iron as EOL
* Readme: remove obsolete ROS1 wiki link
* ElevationMap from XML: Ensure trimmed input string in kernel
* Alternative reference method for UTM world coordinates
* Implement world georeferenciation via UTM zone number
* Fix targets order for prev commit
* Fix build without ZMQ
* Contributors: Jose Luis Blanco-Claraco

Holding for Jazzy sync


Create testbed

[x] testbed setup
[x] tests for each different cases
[x] mock of all external network requests
[x] automated unit test based on travis ci
[x] readme update

Just did yarn && yarn test locally and getting this error.

Just did yarn && yarn test locally and getting this error.

This is interesting, I just did clone from scratch and all worked as intended as same as in CI. can you provide more information about your specs?

OS?
Node version?
Yarn version?

I use node v14.15 and OS is MacOS Big Sur (11.4)

Ah got it. It's because I had ENV=local set on .env. Removing it worked


[Symbaroum] Add total corruption hidden input for use in token bar

Changes / Comments
Add an hidden input in corruption section to compute attr_corruption_total as (corruption+corruption_premanent).
This is to be able to use [corruption_total / resolute] as a token bar.
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[ ] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.


Pressing +- buttons on counter with signed url returns exception

Did a fresh install of your project with the signed url's part today. I seem to have configured it right, seeded the db, generated encryption key, but the signed url's do not work. When pressing the + or - buttons on counters I 500 error with this exception:
    "title": "Michalsn\\CodeIgniterSignedUrl\\Exceptions\\SignedUrlException",
    "type": "Michalsn\\CodeIgniterSignedUrl\\Exceptions\\SignedUrlException",
    "code": 500,
    "message": "This URL is not valid.",
    "file": "/tmp/ci-htmx-php8-demo/vendor/michalsn/codeigniter-signed-url/src/Exceptions/SignedUrlException.php",
    "line": 51,
    "trace": [
        {
            "file": "/tmp/ci-htmx-php8-demo/vendor/michalsn/codeigniter-signed-url/src/SignedUrl.php",
            "line": 153,
            "function": "forUrlNotValid",
            "class": "Michalsn\\CodeIgniterSignedUrl\\Exceptions\\SignedUrlException",
            "type": "::"
        },
        {
            "file": "/tmp/ci-htmx-php8-demo/app/ThirdParty/htmx-demo/src/Cells/CounterSigned/CounterSignedCell.php",
            "line": 18,
            "function": "verify",
            "class": "Michalsn\\CodeIgniterSignedUrl\\SignedUrl",
            "type": "->"
        },
        {
            "file": "/tmp/ci-htmx-php8-demo/vendor/codeigniter4/framework/system/View/Cell.php",
            "line": 222,
            "function": "mount",
            "class": "Michalsn\\CodeIgniterHtmxDemo\\Cells\\Counter\\CounterSignedCell",
            "type": "->"
        },
        {
            "file": "/tmp/ci-htmx-php8-demo/vendor/codeigniter4/framework/system/View/Cell.php",
            "line": 99,
            "function": "renderCell",
            "class": "CodeIgniter\\View\\Cell",
            "type": "->"
        },
        {
            "file": "/tmp/ci-htmx-php8-demo/vendor/codeigniter4/framework/system/Common.php",
            "line": 1200,
            "function": "render",
            "class": "CodeIgniter\\View\\Cell",
            "type": "->"
        },
        {
            "file": "/tmp/ci-htmx-php8-demo/app/ThirdParty/htmx-demo/src/Config/Routes.php",
            "line": 52,
            "function": "view_cell"
        },
        {
            "file": "/tmp/ci-htmx-php8-demo/vendor/codeigniter4/framework/system/CodeIgniter.php",
            "line": 883,
            "function": "Michalsn\\CodeIgniterHtmxDemo\\Config\\{closure}",
            "class": "CodeIgniter\\Router\\RouteCollection",
            "type": "::"
        },
        {
            "file": "/tmp/ci-htmx-php8-demo/vendor/codeigniter4/framework/system/CodeIgniter.php",
            "line": 486,
            "function": "startController",
            "class": "CodeIgniter\\CodeIgniter",
            "type": "->"
        },
        {
            "file": "/tmp/ci-htmx-php8-demo/vendor/codeigniter4/framework/system/CodeIgniter.php",
            "line": 368,
            "function": "handleRequest",
            "class": "CodeIgniter\\CodeIgniter",
            "type": "->"
        },
        {
            "file": "/tmp/ci-htmx-php8-demo/public/index.php",
            "line": 67,
            "function": "run",
            "class": "CodeIgniter\\CodeIgniter",
            "type": "->"
        },
        {
            "file": "/tmp/ci-htmx-php8-demo/vendor/codeigniter4/framework/system/Commands/Server/rewrite.php",
            "line": 47,
            "args": [
                "/tmp/ci-htmx-php8-demo/public/index.php"
            ],
            "function": "require_once"
        }
    ]
}``````

Sorry, I can't reproduce it.
Is anyone else experiencing this problem?

@dgvirtual Question. By chance, does your baseURL contain a folder, like: http://localhost/project/public/?

@dgvirtual Question. By chance, does your baseURL contain a folder, like: http://localhost/project/public/?

No, it was a regular codeigniter project run by php spark serve command.
However, I did some tinkering today, and the counters started to work once I replaced
public string $indexPage = 'index.php';
with
public string $indexPage = '';
in app/Config/App.php
So, problem solved, but you might want to update the documentation accordingly.

This sound like a bug in https://github.com/michalsn/codeigniter-signed-url. I will have to look into it soon.
Thanks!


[TC-CONTENTLAUNCHER-] Manual Yaml Script has to be updated as per Verification Step document

Description:
Manual Yaml Script has to be updated as per Verification Step document for below mentioned test cases
TC-CONTENTLAUNCHER-10.4 - command has to be updated as per Verification Step document for all steps.
TC-CONTENTLAUNCHER-10.6 - command has to be updated as per Verification Step document for all steps.
Verification Step Document References:
https://docs.google.com/spreadsheets/d/13zltSz50GRs_kidvobceEuIT8kAAHSK64ANyWcc3xQI/edit?usp=sharing
Yaml-References:
https://github.com/project-chip/connectedhomeip/blob/master/src/app/tests/suites/certification/Test_TC_CONTENTLAUNCHER_10_4.yaml
https://github.com/project-chip/connectedhomeip/blob/master/src/app/tests/suites/certification/Test_TC_CONTENTLAUNCHER_10_6.yaml

https://github.com/project-chip/matter-test-scripts/issues/187


From the docs example, what is supposed to be the correct [ConnectionStringSetting] name?

*Using ["CosmosDBConnection"] as the connection name below fails to publish the code to Azure, have followed the same syntax as per documentation, is there any change in the naming format?
*Using Visual Studio 2022 on Mac

*local.settings.json file

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

ID: f9e3fe74-1d65-41eb-6a5e-0595e6d06029
Version Independent ID: 8933389b-f4c0-acbd-afe3-86f18738c1f9
Content: Azure Cosmos DB input binding for Functions 2.x and higher
Content Source: articles/azure-functions/functions-bindings-cosmosdb-v2-input.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

@Harshal-bhoir
Thanks for your feedback! We will investigate and update as appropriate.

@Harshal-bhoir from the screenshot you shared, you have used Connection and this property is only available in version 4.x or higher of the extension. Can you validate the Cosmos DB extension version being used in your app?
Connection ( or ConnectionStringSetting for older versions) should point to the name of an app setting which appears to be correct in this case i.e. CosmosDBConnection. Also, share the full error message, Functions runtime version, VS Azure function extension version etc. and that will help in understanding the issue better.

Hi, I can confirm I am using the version 4.x of functions. Attached is the below screenshot of the same
Since I am using VS for Mac, I am using the Publish to Azure - Zip Deploy option

Screenshot of VS code installed packages

Deployment Failed screenshot

@Harshal-bhoir Thanks for sharing the details. Zip push deployment appears to have failed and the screenshot or output of the log didn't display the reason for failure. Have you looked at the latest deployment log using https://<app_name>.scm.azurewebsites.net/api/deployments/latest/log API? Also, make sure it follows folder structure described Deployment .zip file requirements doc.
This repo is specifically for docs related feedback. Since the issue described is about deployment, I suggest you posting it in Microsoft Q&A or Stack Overflow and the community experts can assist you. Or you can reach out to Azure Support if you have a support plan.
We will proceed to close this thread and let me know if you have any questions.


Automatically inserts Turbo Stream responses

The user may not realize this how to execute Turbo Stream actions in the response, so it is nice if we can take care of it for them. By automatically inserting the Turbo Stream responses into the body, we can apply those operations automatically for the user.
I couldn't think of any situations where you might want to disable this, but we could add an option to disable it if needed.
Thoughts on this or how it can be improved?
cc @dhh

@marcelolx Thanks! ✌❤
On a related note... do you think we should modify Turbo to automatically set window.Turbo? If Request.js and iOS/Android all need it, seems like it would make sense to out of the box.

Yes, definitely want to set window.Turbo. We have the same issue at Basecamp. In fact, we do that in our pack setup:
const { Turbo } = require("@hotwired/turbo-rails")
window.Turbo = Turbo

@excid3 @dhh I'll take a look on that and send a PR to turbo

@marcelolx I started one already, just need to test it.


Change Tag from GUID to commit SHA

Instead of the new Tag being the TagPrefix+GUID, grab the first 10 character of the commit SHA and use that instead of the GUID. The new Tag will be TagPrefix + _<First 10 digits of the commit SHA>.
Resolves #4299

/azp run tools - test-proxy - tests


Where is the camera_train.pkl?

Can you provide camera_train.pkl and real_train.pkl from the dataset? Or provide its generation script？

Hi, we generate the *.pkl following the shape data preprocessing in SPD. The script is here.
We have done this and you can download our *pkl here.

Thanks for your help！


Add Astarte to Frameworks

Add Astarte (https://github.com/astarte-platform / http://astarte-platform.org) to Frameworks.

Looks good to me. Thanks for the contribution!


Fix ProjectN build breaks

I suppose this is because ProjectN has a lot more warnings as errors?
Shouldn't corert up their warnings as errors then as well?

Maybe once we're caught up, we can backport these to CoreCLR and up the warnings both there and in CoreRT.
No need to make this an extra annoyance for you right now.


Fix for type cast error in Gradle AddDependency recipe.

What's changed?

An instanceof check for JavaSourceFile is added to AddDependency.java.
What's your motivation?

Without this type check, the invocation of UsesType in AddDependency's scanning visitor fails when encountering non-Java files such as a properties file.
Checklist

[x] I've added unit tests to cover both positive and negative cases
[x] I've added the license header to any new files through ./gradlew licenseFormat
[x] I've used the IntelliJ auto-formatter on affected files
[x] I've updated the documentation (if applicable)

Hi @jeremyg484, I'm not sure if I missed something here, but it appears that UsesType's first check is an instanceof JavaSourceFile in it's visit method. So unless I've totally missed something, then removing the if check in AddDependency should still result in a passing test case.

Hi @jeremyg484, I'm not sure if I missed something here, but it appears that UsesType's first check is an instanceof JavaSourceFile in it's visit method. So unless I've totally missed something, then removing the if check in AddDependency should still result in a passing test case.

@shanman190 The problem comes here: https://github.com/openrewrite/rewrite/blob/011685861c1cc49a3b8e8ad77c5d45314ec60f5a/rewrite-java/src/main/java/org/openrewrite/java/search/UsesType.java#L82
The tree is cast to org.openrewrite.java.tree.J. Running the test case I included without the change to AddDependency will result in a ClassCastException at that line.
Note that the Maven version of AddDependency already has the type check:
https://github.com/openrewrite/rewrite/blob/011685861c1cc49a3b8e8ad77c5d45314ec60f5a/rewrite-maven/src/main/java/org/openrewrite/maven/AddDependency.java#L173

@shanman190 It looks like https://github.com/jeremyg484/rewrite/commit/011685861c1cc49a3b8e8ad77c5d45314ec60f5a has taken an alternate (better) approach to fixing this same problem. My test case now passes after pulling in this change.
That said, the above commit does not include a test case, so it might still be worth merging the added test case from this PR. I've updated the PR branch accordingly.

I agree. This test case can help to ensure that specific check is made which would be good going forward.


[FEATURE] Gateway Deposit, Generate QR Code

Аbstract
This is to make the process of deposit through gateway easier for end users.
Motivation and Purposes
When you go to deposit, the QR code should grab the information needed to auto fill when scanned by an outside network client. attachment is filled and only amount is open actually. It should not allow you to add an attachment to transaction to the gateway.

https://docs.waves.exchange/en/waves-exchange/waves-exchange-client-api/waves-exchange-client-payment-api#process gives a nice description of the process.
Quick search of the forked codebase doesn't give any results, could be this is new in waves.exchange


Clarify rules regarding coinbase genesis

Coinbase transactions are fair game for creating a Genesis token transactions, there isn't anything in the SLP token type 1 (including NFT1 parent) rules preventing this.
As a consideration, validator implementations for NFT1 children should just be aware that a coinbase transaction with NFT1 Genesis OP_RETURN can be immediately rejected as invalid since the coinbase input of 0000...0000 is a dead end.
I'm going to leave this open until I have a chance to investigate the NFT1 child genesis edge case for the different validators.

Closing this since it seems like a non-issue, we can reopen if it's still an issue or point of confusion.


SD_WU_3: Moving multiple PDF files from Explorer to SciDrive folder hangs.

Tried multiple PDF files from Windows Explorer to SciDrive chrome browser. 'Loading' message is up on the UI but keeps hanging.

Still an issue? Does upload work at all, for one file?

Tested and fixed.


Change CLA FAQ address

The CLA address still shows the address with Carlos' name and the US office.
Should it be changed to be the Nüremberg office ?
@hodyroff @danimo @dragotin

here https://owncloud.org/contribute/agreement/, search for "carlos"

by intention at the moment - has legal background
will be resolved next week

Addressed by #988


ibverbs-suppot bifrost branch : py3 support

I tried installing the ibverbs-support branch of bifrost in the commensal machine. While I'm able to go through with the install, the libbifrost is not imported by the script and I get error stating "no module libbifrost".
While I do not find the same problem with python 2, I notice that new bifrost blocks written from scratch are not being recognized.

I think this is ended up being a non-issue in the end? Since this was created both @HaraKrish and I have installed and run the ibverb-support in python3.


Added method to pull all Issues in a Repository

Methods Used :
.__scrap_issues_page() .issues_count()
Description
The method has been created to get the number of issues opened in a repository.
Workflow :

Scraped the issues page using a method called .__scrap_issues_page()
then searched the elements with issues in a method called .issues_count()
iterated over each of them and returned the list of allIssues

To what class it belongs?:
Repository
This PR closes #41
Would be grateful if you could add the label of JWOC '23 if this PR is merged. @nikhil25803

Hey @pooranjoyb, we will be including this in another class. We are planning to create a separate class for issues, where we will scrape the data from the issues page.  So I cant accept it rn :))

What I asked to scrape is the issues count. For example in this image, there are 8 issues open.  So scrape this one.

LoL I guess I misunderstood :/ @nikhil25803

No issue @pooranjoyb :))


the implementation for the hard-meta batch

Hello, liu. Your work is insighting and very useful to my work on peptide discovering, will you please update the implementation for the hard-meta batch in the current repository?

Thank you so much for your interest in our work. I am sorry for not integrating the code for the hard task meta-batch into the public repository. As this work was released around five years ago, some packages are already outdated. I will try to update the code, but I cannot guarantee a date.
The implementation details of the hard task meta-batch are available in the journal paper (link). If you have any questions when you're re-implementing this part, please feel free to contact me via email<EMAIL_ADDRESS>


User's photo with bad quality

When i try get the an profile's photo, always return to me an image with really low quality on Android.

@SnowGhost21 this is a good feature request and one we have gotten before.  Since it's outside the control of FirebaseUI I will report it to the team and close this issue.


Sticky Navigation fix

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #100
Test URLs:

Before: https://main--bitdefender--hlxsites.hlx.page/solutions/internet-security
After: https://100-sticky-nav-fix--bitdefender--hlxsites.hlx.page/solutions/internet-security

Something broke if you scroll over hero:

Something broke if you scroll over hero:

done

Opened a ticket regarding sticky nav and menu #105


site url link being displayed as text and not html

Hi,
On chrome I get WPA notifications with the site url link displayed as test and not html.
On FF it's ok.
Any suggestions?
Thanks,Yair

I"m not sure it's related the library.
Maybe it's chrome issue?

You might have more chance to get an answer from people working on browser's implementations on StackOverflow. HTML might not be supported by every browser indeed.


Use shared Clerk functions

Feature Description
We've recently exposed some commonly used functions across SDKs in our @clerk/shared module.

loadClerkJsScript - You can replace your current implementation with this one.

deriveState - We've also exposed this. Feel free to use that instead of your  current implementation.

Use Case
Eliminate duplicate code and make SDKs consistent
Additional context
No response

Thank you for that information. I will tackle that next week!

Shared functions are now being used


Draft: Bevy 0.15

Did a quick stab at getting stylebuilder and quill core working with bevy 0.15. it compiles, I got the simple example to run.
I'm still working on getting my own project moved over to 0.15 using this branch but I'm currently stuck with some other crates just to get it compiling. Posting this early in case someone finds it useful

Thanks for doing this, I looked over the PR and didn't see any problems, so I went ahead and merged it.
I've been thinking about how one might get rid of the requirement for nightly rust, although it would require substantial changes. Currently I use Rust TAIT to define the state type for each view, so that we can store the template state in a component as a set of nested tuples. I've been thinking that instead of doing this, get rid of the tuples and instead have the context include a bumpalo arena. This would allow views to construct their state tree using arbitrary types, and have normal references instead of nested tuples. The arena would be stored in the template's component. However, one problem with this is that objects in an arena are not dropped (that is, they are freed but not destructed). This could be a problem for for loops which maintain a Vec of items.

Cheers!
Some notes from my porting in my project

pointer_events should be done by PickingBehaviour::IGNORE
bevy_mod_picking's On::X can be replaced by inserting Observer::new(...).with_entity(id) but this goes against the docs from bevy of "Observers should always be their own entities!" and probably breaks if you ever have more than 1 event to observe. I think the proper thing do would be to spawn these as child entities, possibly through a new type of Effect.

Depending on nightly is bit of a hassle but for my purposes of tinkering it's acceptable. Is my understanding correct that the use of an arena would remove the need for Element::<> to encode the type of the tuple and thus the type could easily be spelled out in each view instead of relying on impl at all?


Valid Anagram

please merge this pr for hacktoberfest

Give this repo a star and follow me for follow back 😊


RCE in desktop wallet

Hello guys,  I've found a small remote code execution issue in desktop wallet, submitted it to your Hackerone program https://hackerone.com/reports/897121
Hope you find it worth checking.
Thank you.

Hello @slavasn, thank you for your report!
I will elevate the Issue at the person managing our HackerOne program now.
In the mean time, we have now added a fix for this issue here.
Thank you for this discovery, I will come back to you on the HackerOne platform for further questions :+1:

Hello @evias, thank you!


DRM aspect-ratio

Upcoming DRM feature:
https://lists.freedesktop.org/archives/dri-devel/2018-May/176031.html
https://patchwork.freedesktop.org/patch/188125/
Posting this now, so I don't forget about it.

Ahah, I've added this to my TODO list this morning.

It's not upstream yet.


Revert "Nn 3220 use new csra classification prop"

Reverts ministryofjustice/prisonstaffhub#1581

Not needed as we are going to release it


Some thoughts about benchmark fairness/scores

Currently I found that phuslu/log and phsym/zeroslog has no mutex to lock the writer. It is somehow reasonable because we suppose/except that the underlying writer will handle it. see log.FileWriter and lumberjack.Logger. But it seems that unfair for slog.JSONHandler and madkins23/flash because they hold mutex -- Although this does not have much impact on single-threaded throughtput testing.
I notice that the Disabled benchmark result is unstable. It may be that the performance of each handler implementation is too close, so measurement result has jitter in every time.
I guess that the score of samber/slog-logrus is too lower then its score will interfere with the scores of other handlers -- Might consider removing it?
In my experience, allocs is a bit import than nanos.  for example, a 200 ns/op + 0 allocs/op call seems has a higher score than a 150 ns/op + 1 allocs/op call by right.

maybe phuslu/slog also needs introduce a Extras option to define this behavior and others(e.g. formats)
I have no good ideas about it
I raise this query because I found the handler score is too "crowded" in the top-right corner of graph. please ignore me if slog-logrus is still needed.
also "out of thin air", can be ignore too.

And I found another handler authored by go team member https://github.com/seankhliao/svcrunner/blob/main/jsonlog/ , phuslu/slog and its implementation details are similar.

I guess that the score of samber/slog-logrus is too lower then its score will interfere with the scores of other handlers -- Might consider removing it?

I have pushed changes to the scoring page to provide drill-down views of the chart:

In general this isn't so much as an "issue" as it is a "discussion". I noted one fix that I was already working on and I created a separate issue for the new slog handler. The rest of it should be noted in the Discussions area.


Conv1d and Conv3d are not working

Class ConvLoRA currently only works for Conv2d. By inspecting the shape of B @ A, which is (out_channels // groups * kernel_size, in_channels * kernel_size), we can see that it is only compatible with Conv2d.
For reference, weight shape for

Conv1d: (out_channels, in_channels / groups, kernel_size)
Conv2d: (out_channels, in_channels / groups, kernel_size[0], kernel_size[1])
Conv2d: (out_channels, in_channels / groups, kernel_size[0], kernel_size[1], kernel_size[2])

Good catch! I had Conv2d in mind when I first wrote it.
Looks like we just need to instantiate lora_A and lora_B differently depending on the kind of convolution.
Happy to review and merge it if someone wants to implement and test it. Otherwise, I'll do it in the near future.

I have this idea but it will change the way current Conv2d LoRA works. We can treat convolution as a matmul with the input as a flattened "window". For example, for Conv2d, the input is a window with (kernel_size, kernel_size) spatial dimensions, and the flattened input dim is in_channels * kernel_size * kernel_size. This will naturally extend to Conv1d and Conv3d

Conv1d: B @ A = (out_channels, rank) @ (rank, in_channels * kernel_size) = (out_channels, in_channels * kernel_size)
Conv2d: B @ A = (out_channels, rank) @ (rank, in_channels * kernel_size * kernel_size) = (out_channels, in_channels * kernel_size * kernel_size)
Conv3d: B @ A = (out_channels, rank) @ (rank, in_channels * kernel_size * kernel_size * kernel_size) = (out_channels, in_channels * kernel_size * kernel_size * kernel_size)

There are two benefits with the above implementation. (1) kernel size doesn't need to be the same in all spatial dimensions, and (2) we can use convolution in the LoRA branch in the forward pass instead of merging weights, similar to Linear implementation (relevant issue - #54). The first convolution (with lora_A) is normal convolution, with the same kernel size, but the second convolution (with lora_B) will be point-wise (aka 1x1) convolution. I haven't tested it but from what I understand, it should work.
The situation becomes slightly complicated when grouped convolution is involved (groups > 1). I'm thinking of accounting for groups in the input channels of lora_A (so lora_A becomes (rank, in_channels / groups * kernel_size * kernel_size)). We can still implement forward pass of LoRA branch as two convolutions with lora_A and lora_B, where we will use grouped convolution for lora_A, similar to the original convolution branch. A problem might arise when we try to merge weights though. Due to how groped convolution works, I think the merged weights might not be lora_B @ lora_A (I will need to test this). If that's the case, we need to use a different calculation to merge weights.
Another way of using groups > 1 is to follow your current implementation, which puts groups in the output of lora_B (out_channels / groups, rank). However, this would sacrifice the ability to use convolution for forward pass in LoRA branch, but maintains the ability to merge weights with simple lora_B @ lora_A.
Let me know what you think @edwardjhu. Thank you!

Hello, I also encountered the same problem. Is this improvement feasible in your subsequent [experiments? @gau-nernst @edwardjhu


Curl implementation for http client API

This brings the CURL implementation for HTTP Client API.

Codecov Report

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

@@           Coverage Diff           @@
##           master       #6   +/-   ##
=======================================
  Coverage   94.91%   94.91%
=======================================
  Files         155      155
  Lines        6882     6882
=======================================
  Hits         6532     6532
  Misses        350      350


Clicking "Start again" link shows start page with wrong URL path and without related content

Steps to reproduce:

Ensure JavaScript is enabled
View the start page for the "VAT payment deadlines" smart answer directly
In a new browser tab, click the "Start again" link on one of the question pages
In the latter case, you will see that:

the related content on the right-hand side does not appear
the URL path is for the first question page (you can see this by refreshing the page)

I've moved this to Trello in an attempt to end up with a single place containing Smart Answers stories/tasks/bugs.
See the Trello card (not public).
Closing this issue.


[Bug] Navigating back from MediaElement and calling DisconnectHandler results in crash

Description
I have the mediaelement working on windows to play my video.
But when i navigate back and, as per the documentation, try to release the resources by calling
MyMediaElement.Handler?.DisconnectHandler();
during unload. It crashes with a Catastrophic Failure Exception.
I found multiple issues reporting this problem (like this one: https://github.com/CommunityToolkit/Maui/issues/962), but all issues seem to have been closed as in "fixed". So I don't understand why I'm still seeing this behaviour with the latest version.
Or maybe the documentation is outdated and we no longer need to call the DisconnectHandler method??
I'm using

CommunityToolkit.Maui 5.2.0
CommunityToolkit.Maui.MediaElement 2.0.0

Stack Trace

Link to Reproduction Sample
a reproduction sample is here: https://github.com/sjorsmiltenburg/maui_bugs
if you press the button Bug5 you will see the crash on windows
Steps to Reproduce

open repro sample
click the bug 5 button
navigate back

Expected Behavior
no crash
Actual Behavior
crash
Basic Information

Version with issue:
Last known good version:
IDE:
Platform Target Frameworks:

iOS:
Android:
Windows:

Android Support Library Version:
Nuget Packages:
Affected Devices:
MAUI Version:

Workaround

Don't call MyMediaElement.Handler?.DisconnectHandler();
Reproduction imagery

sorry, i seem to have opened this issue on the wrong repo. I've opened a new one on the maui repo: https://github.com/CommunityToolkit/Maui/issues/1245


tips for co-organizers teams

Related to #18 :grimacing:

Add authority & responsability have to align thanks Gwynn Sturdevant


how to require the model to output all entities in a single response.

I didn't find how to do it in the paper

Our current model only supports returning entities of one type at a time. Multiple entity types require multiple rounds. We also tried returning all entities with one query (all-in-one baseline in Fig. 6) but found the performance much lower.

So the released model "Universal-NER/UniNER-7B-all" in huggingface can't output all entity in one response as the paper said? could you share the prompt used when training the "all" type model?


DictConfig doesn't seem to be serializable anymore in latest rc version

Set up
I have installed
pytorch-lightning==0.8.5
hydra-core==1.0.0rc1
omegaconf==2.0.1rc12
tensorboard==2.2.2
PyYAML==5.3.1
Issue
When I want to save the hyper-parameters that are given by a omegaconf.ConfDict with the function save_hyperparameters of the LightningModule of pytorch-lightning, I get a weird error from the PyYAML package saying that there is an AttributeError when trying to represent an object. It seems like the omegaconf.ConfDict is not serializable anymore. However the exact issue is really hard to find.
When I'm using omegaconf==2.0.1rc11 everything works fine.

        # A special case is when function.__name__ == '__newobj__'. In this
        # case we create the object with args[0].__new__(*args).

        # Another special case is when __reduce__ returns a string - we don't
        # support it.

        # We produce a !!python/object, !!python/object/new or
        # !!python/object/apply node.

        cls = type(data)
        if cls in copyreg.dispatch_table:
            reduce = copyreg.dispatch_table[cls](data)
        elif hasattr(data, '__reduce_ex__'):
            reduce = data.__reduce_ex__(2)
        elif hasattr(data, '__reduce__'):
            reduce = data.__reduce__()
        else:
            raise RepresenterError("cannot represent an object", data)
        reduce = (list(reduce)+[None]*5)[:5]
        function, args, state, listitems, dictitems = reduce
        args = list(args)
        if state is None:
            state = {}
        if listitems is not None:
            listitems = list(listitems)
        if dictitems is not None:
            dictitems = dict(dictitems)
>       if function.__name__ == '__newobj__':
AttributeError: 'str' object has no attribute '__name__'

Thanks for reporting.

Please use the latest Hydra (rc4).
Can you create a minimal repro?
Try with a newer Python, there are many type related issues in Python 3.6 and I suspect this is related to an internal change we did in OmegaConf.

@pereman2, mind taking a look?

Also, try with the latest Pytorch Lightning, there have definitely been changes in how PL is saving the OmegaConf config.

Hey, thanks for your reply.

Alright, I will try that one.
I will try to reproduce the issue in a minimal repo, but it might take some time.
Unfortunately I'm bounded to Python3.6 for now. Will you drop support for this python version any time soon?

Alright I will try with latest versions of PL as well and see what happens. It is really hard for me to figure out what is going on here just by the traceback. Since this could be caused by many different things... (I already debugged into PyYAML but it didn't really help)

Okay turned out to be faster than expected.
I created a new conda env with python3.6 and installed omegaconf==2.0.1rc12
import omegaconf
import yaml

hparams = {"architecture": {"cls": "some.path.to.somewhere", "params": {"neurons": 3}}}
hparams = omegaconf.DictConfig(hparams)

# This is how it is done in the current PL version.
omegaconf.OmegaConf.save(hparams, "test1.yml", resolve=True)

# This is how it is done in the PL version that I was using.
hparams = dict(hparams)
with open("test2.yml", "w", newline="") as fp:
    yaml.dump(hparams, fp)

The second one should crash with the same error as in my initial post.

Thanks for the repro.
This is basically wrong usage though, OmegaConf should not be converted to a primitive dictionary with dict() because it's not recursive. the proper way to convert it is OmegaConf.to_container() , which is what OmegaConf.save() is doing.
There have been same changes around yaml serialization which might be causing this new issues, we will take a look and try to fix it. but it may take a while.

@pereman2, mind taking a look?
Sure.

Unfortunately I'm bounded to Python3.6 for now. Will you drop support for this python version any time soon?

No, 3.6.4+ is here to stay for a while. but it is a problematic version. I don't think what you ran into is related to it though.

OmegaConf DictConfig cannot be dumped directly via yaml.dump(), use OmegaConf.to_yaml() or yaml.dump(OmegaConf.to_container(cfg)).
The reason behind this is because pyyaml doesn't support typing types.

Thanks for reporting @pafi-code. Closing as invalid.


Docker desktop can't switch containers on Windows with Bitlocker

Expected behavior
Docker switch default containers to Windows.
Actual behavior

Information

Windows Version:
Windows 10 Enterprise
Domain-joined Work computer with Bitlocker encryption. Not possible to disable bitlocker as it's internal security rule in company.
Steps to reproduce the behavior
Install Docker
Launch Docker
Press in launch bar option -> Switch to Windows containers

Thanks for sharing the issue. To report an issue with the Docker Desktop app, please use https://github.com/docker/for-win


Name and Symbol must be unique

Link to the original issue -> https://github.com/hashgraph/hedera-services/issues/614


Explanations have been replaced with geo-distance

Match explanations now appear only with geo-distance, missing the rest of the explanations. Example - Fayum match in BT.

Fixed


ARROW-11140: [Rust] [CI] Experimenting with Buildkite

I removed all workflows so that they do not get triggered on this PR, as there will be a lot of experimenting and I do not want to trigger those flows.

I set up a pipeline for Rust. You can see the available pipelines here https://buildkite.com/apache-arrow
The dispatcher is responsible to trigger other pipelines, it uses the following configuration:
steps:
  # if push then compare to the previous commit
  - label: "Triggering pipelines"
    # if pull request
    plugins:
      - chronotc/monorepo-diff#v1.2.0:
          diff: .buildkite/pull-request-diff.sh ${BUILDKITE_PULL_REQUEST_BASE_BRANCH} ${BUILDKITE_COMMIT}
          watch:
            - path: go/
              config:
                trigger: arrow-go
            - path: rust/
              config:
                trigger: arrow-rust
            - path: cpp
              config:
                trigger: arrow-c-plus-plus
            - path: python/
              config:
                trigger: arrow-python
```

Since we have no agents running at the moment you won't be able to trigger builds.
I'll try to configure a machine in the upcoming days, but we have network provider issues with our self hosted workers.

I set up a pipeline for Rust. You can see the available pipelines here https://buildkite.com/apache-arrow
The dispatcher is responsible to trigger other pipelines, it uses the following configuration:
steps:
  # if push then compare to the previous commit
  - label: "Triggering pipelines"
    # if pull request
    plugins:
      - chronotc/monorepo-diff#v1.2.0:
          diff: .buildkite/pull-request-diff.sh ${BUILDKITE_PULL_REQUEST_BASE_BRANCH} ${BUILDKITE_COMMIT}
          watch:
            - path: go/
              config:
                trigger: arrow-go
            - path: rust/
              config:
                trigger: arrow-rust
            - path: cpp
              config:
                trigger: arrow-c-plus-plus
            - path: python/
              config:
                trigger: arrow-python
```

Since we have no agents running at the moment you won't be able to trigger builds.
I'll try to configure a machine in the upcoming days, but we have network provider issues with our self hosted workers.

Ok, I think that this is now ready for a spin. It took a bit more than I antecipated, but the core work is ready. This basically deprecates all our rust jobs on github, replacing them by equivalents on buildkite.

Ok, I think that this is now ready for a spin. It took a bit more than I antecipated, but the core work is ready. This basically deprecates all our rust jobs on github, replacing them by equivalents on buildkite.

I can't wait for this! Current CI is slower than little snails!

I can't wait for this! Current CI is slower than little snails!

Note that this has a downside, buildkite won't be available for apache/arrow forks.

Note that this has a downside, buildkite won't be available for apache/arrow forks.

Note that this has a downside, buildkite won't be available for apache/arrow forks.

Yeap. That is a downside. But we plan to have them running for open PRs, right?

Note that this has a downside, buildkite won't be available for apache/arrow forks.

Yeap. That is a downside. But we plan to have them running for open PRs, right?

Preferably, of course it comes with a maintenance cost.

Preferably, of course it comes with a maintenance cost.

@wesm , is there any update around this? If we do not plan to pursue this, please let me know and we close this PR.

@wesm , is there any update around this? If we do not plan to pursue this, please let me know and we close this PR.

@jorgecarleitao
What is the status of this PR?
As part of trying to clean up the backlog of Rust PRs in this repo, I am going through seemingly stale PRs and pinging the authors to see if there are any plans to continue the work or conversation.

@jorgecarleitao  I am closing this PR for the time being to clean up the Rust/Arrow PR backlog.  Please let me know if this is a mistake


[HANDS-ON BUG] Unit 1 problem with LFS files pushing model to the hub

Bug description
Hi there, I've strictly followed the notebook (checking the solutions for each cell) from the hands-on exercise in unit 1 and I always encounter this problem when I try to push the model to the hub:

Basically, the repository is created and the login to HF is fine but it gets stuck indefinitely waiting for the LSF files to be uploaded. I've tried to install LSF again and searched for information about this issue but I can't seem to find what is going on. I'll share the notebook bellow in case that anyone can help and there's anything that I oversaw but I find it weird.
Material

Colab notebook: https://colab.research.google.com/drive/1xWTD_M-1j5ZdlcYQs2SvwG-XubwLVGcj?usp=sharing

I am having same problem

I found that 4 LFS in the official model are all smaller than 200KB. I wonder it may be not a network issue. I also cannot visit the leaderboard after Runtime error during upload with

It appears that one or more of your files contain valid Hugging Face secrets, such as tokens or API keys. It is important to keep your secrets private and not expose them in code that is publicly accessible.

I think I do not do anything else but fill the TODO and run the notebook.

I'm also having the same problem

I am having this issue as well.

I am also having this issue

I've managed to push it successfully by passing token directly to package_to_hub  function like this:
package_to_hub(model=model, # Our trained model                model_name=model_name,                model_architecture=model_architecture,                 env_id=env_id,                 eval_env=eval_env,                 repo_id=repo_id,                 token=TOKEN_AS_STRING,                commit_message=commit_message)
Looks like all this scheme with notebook_login() and git config --global credential.helper store doesn't work as intended in Colab for now.

@Drakrig Thank you so much. Your approach worked for me.

Also worked for me. I've had other similar issues trying to use some HF classes with the usual login instead of just passing the token. Thanks you @Drakrig !!!

Hi everyone, sorry for the inconvenience on this topic. The bug has been identified and fixed in huggingface_hub. More details in https://github.com/huggingface/huggingface_hub/pull/1953#issuecomment-1878364894.
TL;DR: in the meantime you can fix the problem by running !pip install -U huggingface_hub before running anything else in your colab.


Support PBKDF2 keystores

Is your feature request related to a problem? Please describe.
Currently, we want to support client diversity by making it easy for users to migrate to Lodestar with minimal effort, which means being able to handle all types of encryption standards for EIP-2335. Prysm account export feature uses PBKDF2-SHA-256. We should support this in our validator client to help users easily migrate to Lodestar from Prysm by reusing their exported keystores.
Describe the solution you'd like
Include support for PBKDF2-SHA-256 keystores as defined in the EIP-2335 spec:
https://github.com/ethereum/EIPs/blob/master/EIPS/eip-2335.md#json-schema

Already supported. Closing.


pixelRatio value above 1 does not work on echart v5.0.1

Version
5.0.1
Steps to reproduce
try the following option in the online editor and click download button:
option = {
    xAxis: {
        type: 'category',
        data: ['Mon', 'Tue', 'Wed', 'Thu', 'Fri', 'Sat', 'Sun']
    },
    yAxis: {
        type: 'value'
    },
    series: [{
        data: [150, 230, 224, 218, 135, 147, 260],
        type: 'line'
    }],
    toolbox: {
        feature: {
            saveAsImage: {
                pixelRatio:2
            }
        }
    }
};

What is expected?
Download image in higher resolution
What is actually happening?
You will see that download with pixelRatio 1 does work but value 2 causes an error as seen on the browser console logs.

Upgraded from v4 to v5 and everything we use works except this.

I have since noticed that this issue is related to Canvas render mode. Is the only way to increase the resolution for downloads to use SVG rather than canvas ?

Hi, we've fixed this issue in 5.0.2 which is coming soon. You can try the RC version from GitHub releases before publishing the formal version to NPM.
https://github.com/apache/echarts/releases/tag/5.0.2-rc.1


(583) Fix setting of total_value when ingested value contains invalid characters

Several submission entries contain commas and other invalid characters in their total column within their data blob. When this was copied to the total_value field, it was truncated, which threw off the numbers we were reporting.
This fix ensures that these characters are stripped when the total_value is set on an entry. A data migration backfills the totals that were previously affected.

@tahb I've changed the regex to strip out anything that isn't a digit or a period, so this should be good for another look.