GalleryImage and GalleryImageVersion tagged with @Beta in latest azure sdk

I am using methods of classes GalleryImage and GalleryImageVersion in my project. But, I see these classes have @Beta annotation added and java doc of @Beta mentions this:
/**

Indicates functionality that is in preview and as such is subject to change in non-backwards compatible ways in future releases,
including removal, regardless of any compatibility expectations set by the containing library version.
*/

I am using the latest library version https://mvnrepository.com/artifact/com.microsoft.azure/azure/1.31.1.
I have the below questions:

What are the plans to update this class in future releases and removal of @Beta tag?
What are the challenges in using these classes in latest azure SDK library?

Thanks for posting this question @bhavyaSharma-appD. @yaohaizh can you please follow up?

hi @xccc-msft could you pls have a look? thanks.

@anuchandy Would you please share some info about why we set the @Beta annotation at that moment, is it some concern from service side for the preview functionality or from SDK side?

Hi @xccc-msft @anuchandy Any update on this?

@bhavyaSharma-appD I just checked with @anuchandy for the background how we determine to add Beta annotation in the library. It seems we add @Beta for those preview features, which are defined via preview api versions, provided by service providers.
And for your questions, I checked with @yungezz that we don't have plan to change the APIs with Beta tags as no changes for those preview features from resource providers. But at the same time, we might not remove the @Beta as it help track the preview features, which are not placed in the swagger files of stable versions.
So you should be good to use our library. Thanks.

@xccc-msft Could you please explain what do you mean by preview api versions and preview features?

@bhavyaSharma-appD Not sure if you can access the swagger of Azure REST APIs. For each spec, the service team will put two folders, one is named stable, the other is preview. Inside the folder, there are APIs defined by swagger docs.

@xccc-msft I can access these but not sure which is relevant to our usecase. Could you please point me to the one related to our usecase?
Also, do you know why these apis related to shared image are not placed in the swagger files of stable versions?

@bhavyaSharma-appD This should be the swagger related to shared image.

Thanks @xccc-msft . Waiting for your response on second question "Also, do you know why these apis related to shared image are not placed in the swagger files of stable versions?"

@bhavyaSharma-appD Yes, now shared image is under stable folder. I don't have much historical info about this. Normally the @beta is used when feature is in preview. As I mentioned before, we have no plan to change these APIs so you should be good to use our library. Thanks.

The annotations have been removed via Azure/azure-libraries-for-java#1188.
The changes are available in v1.34.0.


The client parameter type's names occur wrong when multipart is include.

sample openapi:
animal.json
sample refitter config file:
Animal.refitter
{
"openApiPath": "./animal.json",
"namespace": "Animal",
"naming": {
"useOpenApiTitle": false
},
"multipleInterfaces": "ByTag",
"operationNameTemplate": "{operationName}Async",
"outputFolder": "../Clients",
"outputFilename": "Animal.cs",
"codeGeneratorSettings": {
"requiredPropertiesMustBeDefined": true,
"generateDataAnnotations": true,
"anyType": "object",
"dateType": "System.DateTimeOffset",
"dateTimeType": "System.DateTimeOffset",
"timeType": "System.TimeSpan",
"timeSpanType": "System.TimeSpan",
"arrayType": "System.Collections.Generic.ICollection",
"dictionaryType": "System.Collections.Generic.IDictionary",
"arrayInstanceType": "System.Collections.ObjectModel.Collection",
"dictionaryInstanceType": "System.Collections.Generic.Dictionary",
"arrayBaseType": "System.Collections.ObjectModel.Collection",
"dictionaryBaseType": "System.Collections.Generic.Dictionary",
"propertySetterAccessModifier": "",
"generateImmutableArrayProperties": false,
"generateImmutableDictionaryProperties": false,
"handleReferences": false,
"jsonSerializerSettingsTransformationMethod": null,
"generateJsonMethods": false,
"enforceFlagEnums": false,
"inlineNamedDictionaries": false,
"inlineNamedTuples": true,
"inlineNamedArrays": false,
"generateOptionalPropertiesAsNullable": false,
"generateNullableReferenceTypes": false,
"generateNativeRecords": false,
"generateDefaultValues": true,
"inlineNamedAny": false,
"excludedTypeNames": []
}
}

@safakkesikci thanks for taking the time to report this
I'm unfortunately not at all experienced in working with multipart/form-data in Refit or multipart/form-data requests in general
Do you know what should the HTTP request look like for this?
"requestBody": {
  "content": {
    "multipart/form-data": {
      "schema": {
        "type": "object",
        "properties": {
          "Name": {
            "type": "string"
          },
          "AnimalClassFile": {
            "type": "string",
            "format": "binary"
          },
          "AnimalCrowdFile": {
            "type": "string",
            "format": "binary"
          }
        }
      },
      "encoding": {
        "Name": {
          "style": "form"
        },
        "AnimalClassFile": {
          "style": "form"
        },
        "AnimalCrowdFile": {
          "style": "form"
        }
      }
    }
  }
}

and what should the Refit method look like?

Hello. When I try to generate client csharp and csharp-dotnet2 via editor.swagger.io, it occurs as follows.
ps: of course type is byte array not streampart.
csharp-client-generated (1).zip
AnimalResponse AnimalsPost (string name = null, byte[] animalClassFile = null, byte[] animalCrowdFile = null);
{
...
csharp-dotnet2-client-generated (1).zip
 public AnimalResponse AnimalsPost (string name, byte[] animalClassFile, byte[] animalCrowdFile)
{
...

@safakkesikci is this a client for an API you developed yourself? If so, if it's an ASP.NET API, what does the code look like for accepting multipart/form-data parameters?

Hello, I tried to go the reverse way :) I created a helloworld webapi project and created a client from the controller with the GenerateAspNetCoreClient package using Refit and examined the document.
example project https://github.com/safakkesikci/refitclienttest
I added AnimalController and created a client with the GenerateAspNetCoreClient package.
swagger.json
I hope it's helpful.

Hello, I tried to go the reverse way :) I created a helloworld webapi project and created a client from the controller with the GenerateAspNetCoreClient package using Refit and examined the document.
example project https://github.com/safakkesikci/refitclienttest I added AnimalController and created a client with the GenerateAspNetCoreClient package.
swagger.json
I hope it's helpful.

@safakkesikci Yes this is really helpful! I'll see what I can do!


[JOSS] protocol version when installing from source vs install through pip

openjournals/joss-reviews#6269

MRdataset version: 0.1.dev
Python version: 3.10.12
Operating System: Ubuntu 22.04.3 LTS

Description
This is part of the JOSS submission review.
In the installation instruction, the stable release will install a key dependency protocol version 0.3, however if we strictly follow the install from source instruction, it will be 0.1.
These two things should be consistent.
What I Did
virtualenv env-stable
source env-stable/bin/activate
pip install -U MRdataset
pip list

From source:
git clone git://github.com/Open-Minds-Lab/MRdataset
cd MRdataset
virtualenv env
source env/bin/activate
pip install git+https://github.com/sinhaharsh/protocol.git#egg=protocol
pip install -r requirements_dev.txt
pip install -e .
pip list

that's a great catch Hao, thank you. I guess we never attempted install from source :)
we will update the docs to make it they should use pip install -U MRdataset when possible, and only when they trying to contribute as a developer should they use the other instructions.
I personally don't understand the parameter #egg=protocol in this command
in pip install git+https://github.com/sinhaharsh/protocol.git#egg=protocol
but my guess is we don't need it. @sinhaharsh and I take a look and fix the docs asap.

Compiling protocol from source was initially required. It's surely not required anymore. It is now addressed in the documentation. Thank you Hao.

The installation instructions are now updated. Installing MRdataset will also install latest protocol from PyPI

LGTM!


Prompt user to create hap.yml file

If we implement #224 then it would be even better to have a prompt that requests the user for basic information to create the hap.yml file:
$ hap init
> repo: 'https://github.com/user/repo.git'
> revision (default: origin/main): production
> host (default: root@localhost<EMAIL_ADDRESS>> port (default: 22): 2222
> Include restart command (y/n)?: y

> Generated hap.yml file

$ cat hap.yml file
> Generated sample hap.yml file
$ cat hap.yml
host<EMAIL_ADDRESS>port: 2222
repo: 'https://github.com/user/repo.git'
revision: origin/main
build_script:
  - echo 'Build steps'
restart_command: echo 'Restart command'

I think repo is the only required property. The other ones could have a default value that the user can change eventually.
(Optional) Also, to create a file with all the default values we could include a flag:
$ hap init -y
> Generated hap.yml file

There is already an implementation for this which was kindly added in a2aa3dba432dc1ad49844cca0f0abc18980c2cc3 but it needs some refinements:

hap.yml file is always created even if the given value is not of the correct type
The parser is using read so Text values need to be wrapped in quotes which can be annoying:

Creating 'hap.yml'
repo (default: git@github.com:stackbuilders/hapistrano.git)?:
"git@github.com:stackbuilders/my-repo.git"

Therefore, I'll leave this issue open to solve those two inconveniences first.

@CristhianMotoche  BTW do you think that implementing repo as the only required value should be good?
The second point (-y flag to default all) sounds reasonable, I'm going to work on that.

Thanks @alexisbcc I see you already worked on this, so I'll close this ticket. Thank you!


README: update compatibility table to match current code

According to the rest of the README and the code comments, enable and disable have been implemented for the web, but in the table they are incorrectly marked as not implemented.

Thanks for the PR!

@all-contributors please add @ptmkenny for bug, code, and doc


Unit fixes

Two fixes in unit.rs.  Also converted most of the unit.rs tests to test-assembler to help in writing tests for those fixes.
Fixes #132

Travis CI had some breakage with rust when they updated their rustup.sh. I think this PR caught that. It's since been fixed, and I've restarted the build.

Coverage increased (+0.5%) to 93.101% when pulling 54e9af47786b68fa715596f1fc823dbcd7701c18 on philipc:unit into 4e44838dd0f9f4771b8505ce31d73743e53e015f on gimli-rs:master.


encoding/binary.ReadVarint does not follow spec

What version of Go are you using (go version)?
go version go1.10.3 linux/amd64
Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/leo/.cache/go-build"
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/leo/src/gopath"
GORACE=""
GOROOT="/usr/lib/go"
GOTMPDIR=""
GOTOOLDIR="/usr/lib/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build048794147=/tmp/go-build -gno-record-gcc-switches"

What did you do?
I tried to decode a Uvarint-encoded integer with ReadVarint. Program: https://play.golang.org/p/nRMSi3PRvi5
What did you expect to see?
The documentation for encoding/binary says:

The varint functions encode and decode single integer values using a variable-length encoding; smaller values require fewer bytes. For a specification, see https://developers.google.com/protocol-buffers/docs/encoding.

The specification says (emphasis added):

Each byte in a varint, except the last byte, has the most significant bit (msb) set – this indicates that there are further bytes to come. The lower 7 bits of each byte are used to store the two's complement representation of the number in groups of 7 bits, least significant group first.

This would imply that, for positive numbers, ReadVarint and ReadUvarint would return the same result when given the same byte slice.
What did you see instead?
ReadVarint does not return the same result as ReadUvarint when given a varint-encoded byte slice of a positive number. Instead, ReadVarint appears to do something completely different, which is not two's complement decoding at all:
// ReadVarint reads an encoded signed integer from r and returns it as an int64.
func ReadVarint(r io.ByteReader) (int64, error) {
	ux, err := ReadUvarint(r) // ok to continue in presence of error
	x := int64(ux >> 1)
	if ux&1 != 0 {
		x = ^x
	}
	return x, err
}

Protocol buffers handles signed integers differently than normal integers. The are not stored in two's complement representation, but use a so called ZigZag encoding. The documentation of ReadVarInt cleary states that it reads an encoded signed integer, which according to the Protocol Buffer documentation uses the ZigZag encoding.
The rationale is that the two's complement representation requires always the maximum number of bytes for negative numbers. The ZigZag encoding avoids that problem.

Working as intended, and as described by @ulikunitz


Finalize BOM Descriptor Schema Extension

Background
The BOM Descriptor extension is intended to provide additional metadata about the asset/software it describes. The schema is currently at 1.0-SNAPSHOT and sources and docs are here:

https://github.com/CycloneDX/specification/blob/master/schema/ext/bom-descriptor-1.0.xsd
https://cyclonedx.org/ext/bom-descriptor/ (v0.9 only)

Community Feedback Requested

BOM-level external references are already supported in the base schema.
Refer to https://github.com/CycloneDX/specification/blob/master/schema/bom-1.1.xsd#L688. The node appears after 'components'.
<bom>
  <components>
    ...
  </components>
  <externalReferences>
    ...
  </externalReferences>
</bom>


Examples; replace Uber with USPTO

This PR replaces the commercial, outdated and oAuth protected Uber example with a public, current and open one from the US Patent & Trademark Office.
Disclaimer: This is a slightly cut-down (bulk download endpoints have been removed for brevity) converted OpenAPI 2.0 definition (thus the Uber example in v2.0 could also be replaced). The response model is not fully defined.
Although it relies on a US Government department, it seems unlikely to come under direct threat from the current administration.

I've replaced the Bulk Data Search API with the Data Set API, as this has a post request as well as get. Hopefully this addresses the other points raised:

[x] uses server variable for scheme
[x] includes tag descriptions
[x] includes an example response
[x] has $ref to #/components/schemas
[x] has been validated & checked against both the draft schemas

Thanks for all your work on this @MikeRalphson !


Out of Date Information

https://forum.unity.com/threads/c-compression-zip-missing.577492/#post-3849472
"Note that in Unity 2018.3, this file should be renamed to csc.rsp, instead of mcs.rsp."
Adding reference to mcs.rsp is insufficient on the latest versions of Unity.

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

ID: 8f103240-7b3b-4732-12a9-39fdef044a08
Version Independent ID: 0c9fb9a9-f254-fa5c-a334-3fff7dfccb92
Content: Using .NET 4.x in Unity - Visual Studio
Content Source: docs/cross-platform/unity-scripting-upgrade.md
Product: visual-studio-windows
Technology: vs-unity-tools
GitHub Login: @therealjohn
Microsoft Alias: johmil

Thanks! I'm in the process of updating these docs and will make sure this info is corrected as well.


Additional Props warning with react-bootstrap

am trying to use "redux-form": "^6.7.0" with "react-bootstrap": "^0.31.0"
I have added the Field to the form as follows:
    class Config extends Component {
        render() {
            const { ServerPort, UserID, PortNumber, WWWUrl, SourcePath, FMEPath, pickFile, pickFolder, handleSubmit } = this.props;
            return (
                <Form horizontal onSubmit={handleSubmit}>
                    <FormGroup controlId="serverPortBox">
                        <Col componentClass={ControlLabel} sm={2}>Watson Port:</Col>
                        <Col sm={10}>
                            <OverlayTrigger placement="left" overlay={(<Tooltip id="tt1">TCP port for Watson to use</Tooltip>)}>
                                <Field name="WatsonPort" component={FormControl}
                                    type="number" min="1024" max="65535" placeholder={ServerPort} />
                            </OverlayTrigger>
                        </Col>
                    </FormGroup>

but I am getting the warning:
    Warning: Unknown props `input`, `meta` on <input> tag. Remove these props from the element.
        in input (created by FormControl)
        in FormControl (created by ConnectedField)
        in ConnectedField (created by Connect(ConnectedField))
        in Connect(ConnectedField) (created by Field)

If I change the Field to not use an JS Object for the component, I do not get the warning, but I also do not see the FormControl appearing.
    <Field name="WatsonPort" component="FormControl" type="number" min="1024" max="65535" placeholder={ServerPort} />

The reason tried changing {FormControl} to "FormControl" was https://github.com/erikras/redux-form/issues/1249 but it did not seem to help.

Oh, this is a great mystery. I followed the advice in https://github.com/react-bootstrap/react-bootstrap/issues/2210 and both the warning about additional props and the empty submit stopped.
and this works, except for one odd thing. When the mouse enters the field the first time, and the field gets selected, then I can press one key, and then field looses the selected state. If I reselect, and continue typing it works fine. It is only the first key press that causes the loss of "selected" or is it focus?
Update
and then I tried converting the component to a stateless functional component, and moved the FieldInput of there, and put it in as an independent SFC. And now it select and focus work fine.
Go figure.
        const FieldInput = ({ input, meta, type, placeholder, min, max }) => {
            return (
                <FormControl
                    type={type}
                    placeholder={placeholder}
                    min={min}
                    max={max}
                    value={input.value}
                    onChange={input.onChange} />
            )
        }

and I invoke it like this:
<Field name="ServerPort" type='number' component={FieldInput} placeholder={ServerPort} min="1024" max="65535" />

see also: https://github.com/erikras/redux-form/issues/1750

Minor question for @DrYSG 's solution .
Would it make sense to add Field name prop as id to the Component? Like:
const FieldInput = ({ input, meta, type, placeholder, min, max}) => {
    return (
        <FormControl
            id={input.name}
            type={type}
            placeholder={placeholder}
            min={min}
            max={max}
            value={input.value}
            onChange={input.onChange} />
    )
}

(with react-forms 7.03 & react-bootstrap 0.31.1)

@szerintedmi  I got pulled into a different project, and don't plan to be using react for a while. So I can't really respond to this since my mind is elsewhere.

@DrYSG your solution worked for me, thanks! :)


🔄 synced file(s) with neurobagel/workflows

synced local file(s) with neurobagel/workflows.

Changed files

created local  .github/workflows/build_docker_nightly.yml  from remote  template_workflows/build_docker_nightly.yml

This PR was created automatically by the repo-file-sync-action workflow run #11109737839

Pull Request Test Coverage Report for Build<PHONE_NUMBER>0
Details

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

Totals

Change from base Build<PHONE_NUMBER>4:
0.0%

Covered Lines:
704

Relevant Lines:
733

💛  - Coveralls


Macros Hygiene

The current macros needs a lot of imports to be used. It would be nice if it just works by importing the macro and not needing to worry about anything else.

Sorry for the.....um.....6 month delay!  I think what you may be asking is why you are required to take dependencies to other crates besides this one just to be able to use it.
The thing to understand is that a procedural macro crate generates code in your project at compile time. It isn't a library that you are calling into, it is adding types and functions to your source. So if that generated code depends on tokio runtime being present, you have to take a dependency to tokio too. Same with async-openai. "Well that's super annoying!" Well...sort of, but macros save a lot of effort, this one included. The source for the clap example is ~260 lines long, but if you use cargo expand to see the generated code, it is something like 6,250 lines. Not all of that is generated off of this crate, but most. We can't use code-generating macros and hide the underlying dependencies simultaneously.
I also thought maybe you were saying there are unused or unnecessary dependencies. I did spend time while updating this library for async-openai changes to try to ensure this library isn't bringing more than it requires. Neither rust-analyzer or clippy throw up any warnings about unused (however they have limitations when it comes to procedural macros). I just ran udeps on it and that tool states that all dependencies are used. I've run that on both the examples as well with the same result.
If I've misunderstood your issue, please let me know!

Heyy, thanks for replying.
Actually, my request was about the need of importing things other than the macro.
I think the dependencies are okay and something to expect your lib user to have. That being said, what I was requesting is to use the fully qualified path while expanding the macro so people don't need to also import other things (use serde::Serialize; for example).
Right now if someone wants to use this crate's macros, they will need to:

You can use this as a guide about macro hygiene.

Ok. If you look at the example code now you will see there are way fewer imports required. The ones that are left are used in the examples. Thanks again for the issue.


Support plugins in subdirectories with ya pack

yazi --debug output
N/A

Please describe the problem you're trying to solve
It appears that ya pack can't support a plugin in a subdirectory of a repository. ya pack -a "imsi32/yazi#plugins" works (Yazi at least trys to install it before failing since the files aren't in the right places), but ya pack -a "imsi32/yazi#plugins/yatline-catppuccin" does not (fails with "Error: Invalid package use").
Would you be willing to contribute this feature?

[ ] Yes, I'll give it a shot

Describe the solution you'd like
Support for installing a plugin from a subdirectory of a repository with ya pack.
Additional context
No response
Validations

[X] I have searched the existing issues
[X] The latest nightly build of Yazi doesn't already have this feature

No, this was a deliberate design choice to only support up to one level of subdirectories because I don't want the ya pack command to become too complex. A long URL like ya pack -a "imsi32/yazi#plugins/yatline-catppuccin" doesn't align with the simplicity philosophy of ya pack, nor is it ergonomic.

Supporting subdirectories isn't ergonomic? Monorepos aren't ergonomic? Why is Yazi so inflexible on every possible instance against users and extension authors? You've made it incredibly annoying to make flavors and have been totally unreceptive to feedback around how to improve that aspect, and now you are refusing to add a this basic feature? I encourage you to listen more to feedback from users and developers interested in Yazi, because right now this is seemingly very opinionated and closed off. Not a great environment for contributing either.

See https://github.com/yazi-rs/plugins/issues/16#issuecomment-2306833496 -- issue is not using the latest version

Nope, I'm on the latest version and installing from a plugin from a non-root folder/subdirectory does not work. Try it yourself!

So is this the same issue as https://github.com/yazi-rs/plugins/issues/16 or something different?

That was the correct format for 0.3.0, the : character is the new syntax introduced in #1471 and released in 0.3.1.


Type Mismatch for Underscore and Question Mark

It seems that underscore (_) is being typed as minus (-) and question mark (?) is being typed as forward slash (/) in current (0.5.4) version.

Thanks for the report. This looks to be related to an issue with keyboard's handling of numpad keys. Tracked in issue 41 on its GitHub issue tracker. I'll leave this open until it's resolved.

The patch has been merged into keyboard's master branch, but a new version has not yet been deployed to Pypi.

This has been fixed in the latest version of keyboard.


Remove penalty on duplicate Status message

This change has been extracted from #5938
We no longer shut down all connections and ban the peer if more than one Status message has been received.
Relates to #2010
Context
At the moment, the network protocol defines that it is forbidden to send messages on notifications substreams as long as the Status messages haven't been exchanged on the legacy substream.
This is currently enforced by the "network worker" (i.e. the main networking task).
Additionally, if multiple TCP connections between the same two peers exist, we only send the Status message on one of them, and not both. If we were to send Status on both, the remote would instantly shut down all connections (because of the code that this PR removes).
Whenever a legacy substream is open, we therefore need to know whether we have to send the Status message on it. As explained above, we cannot just sent it unconditionally, otherwise the remote would shut down all connections.
We also need to know whether or not we are still waiting for a Status message to come.
In order to make progress towards #5670, I would like to make the legacy substream optional.
The objective is that we would no longer actively try to open a legacy substream, and instead only process the legacy substreams that are opened by the remote peer (if any).
The processing of Status messages explained above is already quite complicated, and unfortunately, making the legacy substream optional would introduce even more complexity and would require introducing even more synchronization and messages passing around between the various connection tasks and the network worker.
In order to solve this question, a way more simple fix is proposed by this PR: send a Status message unconditionally whenever a legacy substream is open.
This PR does not send a Status message unconditionally, but prepares the ground to it by removing the code that shut down all connections if more than one Status message has been received from a given peer.

@tomaka do I understand correctly that you want to:

Merge this pull request.

Wait for all Kusama and Polkadot nodes to run this patch.

Merge https://github.com/paritytech/substrate/pull/5938 and thereby

Open a legacy protocol for every connection (not only primary).

Send a handshake on each of these legacy substreams and not get banned due to this pull request. Ignore all but the first received handshake from the remote.

How are you planning to handle the case where the old remote won't send a status message on a non-primary connection? Should we just have the handshake time-out?

Open a legacy protocol for every connection (not only primary).

This is already the case, but other than that yes.

Should we just have the handshake time-out?

Yes.

Note-keeping: will most likely be released as part of 0.8.11.
We should be able to see on the telemetry when that version is widely deployed.

Nodes that send a Status message unconditionally will still be able to connect to nodes that don't have this PR in, but since it is subject to race conditions it might require a few re-tries.

@tomaka just to make sure I understand correctly with "race condition" you are referring to the rare case of two nodes each initiating a connection to the other. In that case a node not running this patch would ban the remote node in case that remote node sends a second status message. At a later point in time one of the two nodes might initiate another connection without the other simultaneously doing the same.

Exactly


Setup

This PR updates the repo re: linting, tests via github actions, python 3 only

sorry wrong place!


[BUG] Term suggestion does not work with terms shorter than 3 chars

When adding e.g. assay parameters of an experiment you have to type in at least 3 characters to get suggestions for the term ontologies. Often used terms (pH or g (g force)) cannot be found.

I adjusted the search to trigger slower but on anything that is not only whitespace. This works well for ph but does not return the expected results for "g" (g force).


how to change width?

I've read and accomplished this http://stackoverflow.com/questions/24363523/mmenu-custom-width-settings
so now I have a .css file and a .css.map file but I don't know how to proceed (complete sass newbie). I'm loading the newly generated .css file, but it makes no difference. (also have an annoying border issue).
Sorry to litter the bug/issues area with support requests. :confused: thanks in advance.

You should add the new classname ("hiddenpanel-customwidth") to the menu.
You can do this manually (in the HTML) or by adding "hiddenpanel-customwidth" to the classes option (in versions older than 5.0.0).
As of version 5.0.0 the classes option is replaced with the extensions option:
$("#menu").mmenu({
    classes: "mm-classname", // older versions
    extensions: ["classname"]
});

Note that the newer versions will prefix all classnames with "mm-" automatically.

thank you for the help. I figured out how to change the width. Any ideas what to do with the border interfering with the counters?

Is there an online example I could have a look at?
The "mm-next" button should have a padding-left due to the preceding counter:
em.mm-counter + a.mm-next {
    padding-left: 40px;
}


New PowerShell on Target Machines 3 has no option to select a subset of machines

Issue Description
I tried to migrate to the new PowerShell on Target Machines Task (3.* preview) from #6789
However, I use tags on the virtual machines to run the script on a subset of the VMs. Are tags deprecated? How should I select a subset of machines with this new version?

@meastp Which tags are you referring to? How did you specify tags in PowerShell on Target Machines Task 2.* ?

@rajatagrawal-dev Advanced Options -> Select Machines By -> Tags (you can also choose machine names instead of tags) and used Filter Criteria to specifi which tags.

@meastp , yes the tags input is deprecated. v3 only supports specifying ip's or fqdn's.


playback preview mode support

[REQUIRED] Use case description
can we support preview function when playback?
we can only preview when transcoding process and we need prview
function when we playback avsync
Proposed solution
we import avsync playback function and add filters or text

ExoPlayer.setVideoEffects lets you pass effects to apply when playing video with ExoPlayer (including color filtering or adding text overlays).


ci(docs): use julia 1.10

Doc build is failing on master as well - https://buildkite.com/julialang/methodoflines-dot-jl/builds/338

@ChrisRackauckas, I think we need one more release as docs build failed buildkite.com/julialang/methodoflines-dot-jl/builds/338 because of julia 1.11. This should fix it. This would also close https://github.com/SciML/MethodOfLines.jl/issues/417

Yes


[SPARK-13791][SQL]Add MetadataLog and HDFSMetadataLog

What changes were proposed in this pull request?

Add a MetadataLog interface for  metadata reliably storage.
Add HDFSMetadataLog as a MetadataLog implementation based on HDFS.
Update FileStreamSource to use HDFSMetadataLog instead of managing metadata by itself.

How was this patch tested?
unit tests

Test build #52801 has started for PR 11625 at commit c32cd96.

cc @marmbrus @tdas

Test build #52801 has finished for PR 11625 at commit c32cd96.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class HDFSMetadataLog[T: ClassTag](sqlContext: SQLContext, path: String) extends MetadataLog[T]
trait MetadataLog[T]

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/52801/
Test FAILed.

Test build #52835 has started for PR 11625 at commit d77bf39.

Test build #52835 has finished for PR 11625 at commit d77bf39.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #53083 has started for PR 11625 at commit 7a52adc.

Test build #53083 has finished for PR 11625 at commit 7a52adc.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/53083/
Test FAILed.

Test build #53103 has started for PR 11625 at commit 4c27c6e.

Test build #53103 has finished for PR 11625 at commit 4c27c6e.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/53103/
Test FAILed.

retest this please

Test build #53112 has started for PR 11625 at commit 4c27c6e.

Test build #53112 has finished for PR 11625 at commit 4c27c6e.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/53112/
Test FAILed.

Test build #53137 has started for PR 11625 at commit 1c82e56.

Test build #53137 has finished for PR 11625 at commit 1c82e56.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/53137/
Test PASSed.

Thanks!  Merging to master.


Update README to reflect user tasks metrics

Which issue does this PR address?
Related to #2062
Acceptance Criteria

[ ] Corresponds to the concept
[ ] Corresponds to the design

Definition of Done

[ ] corresponds to the design principles
[ ] corresponds to the code standards
[ ] passes Continuous Integration checks
[ ] available as feature branch on GitHub

[ ] contains the cleaned up commit history
[ ] commit messages satisfy our commit message guidelines
[ ] a single commit closes the issue via Closes #issuenr

Please add as commit to https://github.com/camunda/camunda-modeler/pull/2067 , it is the same concern and I would not like to merge this PR, before #2067 is ready.

Please add as commit to https://github.com/camunda/camunda-modeler/pull/2067 , it is the same concern and I would not like to merge this PR, before #2067 is ready.


CASM-4819 cray-kyverno-policies

Add image verification check using public keys
Requires #3596

Closing. Having @jpdavis-prof open a new one to workaround the permission issue with the CSM release team being out.


Using Router.push with custom routing

I have some custom routing logic going on with express like this:
  server.get('/posts/:id', (req, res) => {
    return app.render(req, res, '/post', req.params)
  })

And when I use the router to push to this route:
    Router.push(`/posts/${id}`)

I get a 404. When looking at the server logs I see this:
GET /_next/-/pages/posts/recBhAOprhFeugy0z 404 13.273 ms - 43621

And when I refresh on that route, I get this:
GET /posts/recBhAOprhFeugy0z 200 700.074 ms - 861132

So perhaps I shouldnt be using Router.push, with custom routing but I just wanted to check in about that.

For that, you need to do:
Router.push(`/post?id=${id}`, `/posts/${id}`)

Check the Router.push() docs.

interesting... might be a good idea to put include something like this in the express example.

@ccorcos Good idea.

Sorry for hitting up a closed post. @ccorcos - I'm trying basically the same thing you've done. When I try  to access /path/someparam as a direct url, I also get 404. Was there anything additional you did to make this work?

if you have a directory structure that looks like this:
/pages
    /post

Then you the post page will get query params as props in getInitialProps. That's why the first argument to Router.push should have query params. The second argument can be the actual path.

Cool, thanks


UI: Button to stop model check

if we got an infinite model where our model checker can't find any violations we should be able to halt the check manually

Is it necessary to make doModelCheck(...) interuptable?

maybe we should make it run in a thread, and put a progress bar in front?

But how to know about the progress?

I think some icon showing the process is running would be a good Idea.

Seems like Machines with even more than 10000 models just stop.

we can't tell about the progress as we do not know how many states will be found eventually. Maybe just some process bar showing that something is happening? Even better, we could have a dialog showing the current size of the queue and the number of states analysed so far.

We might need change the Modelchecker to work as interruptable thread.

I will implement a method to stop the current checking session, so the executing thread can stop checking on being interrupted.

Model checking can be aborted via ModelChecker::abort() now.


Migrate WAMP load test program to WAMP2

The blog post http://tavendo.com/blog/post/autobahn-pi-benchmark/ describes how we did performance testing and how to repeat, but the code was WAMP1 and has been removed:
https://github.com/tavendo/AutobahnPython/blob/0be0ed31e37540f51652b3de434e9a8f05806280/examples/twisted/wamp/pubsub/loadlatency/client.py
Migrate the code to WAMP2 and update the links in the post

This has moved to CTS


Use Constructor instead of Tang injector

We create physical objects (sources, operators, and sinks) by using Tang injector. However, Tang injection is too slow compared to calling the constructor directly, thus degrading the system performance while creating objects. We should fix this problem.

A simple experimental result:
Time to create 100,000 map operators with Tang injector: 14,989 ms
Time to create 100,000 map operators with constructor: 322 ms


Mobile version

Hi,
Please correct the mobile version.
Recovered and Death blocks not displayed.

thank you! solved!


coming from Node

Where does the putup command come from?
it's not clear where that executable comes from or how to install it

Here is where putup is registered as a command line script: https://github.com/blue-yonder/pyscaffold/blob/9233cd68901732fc8c1ae25f3622d3113187d015/setup.py#L25
putup is available after pip install pyscaffold.

yeah thanks, is it not recommended to install python v3 to python3 and pip to pip3?

also, any idea why PyCharm cannot resolve the pytest dependency:

I looked in .eggs and looks like pytest is installed, so dunno

@ORESoftware no pip3 and python3 mostly exist on system that default to Python 2 but want also to have a co-existing Python 3 installation. Normally you work in a virtualenv created with virtualenv, python(3) -m venv or conda create -n; there python points to your selected python version (which could be any of 2.7, 2.6, 3.5, 3.6, …)

@xhochy  thanks, yeah pycharm created a virtual env for me apparently

@ORESoftware, are your questions answered and can this issue be thus closed?

Since there was not any more activity in the last 30 days and the question seems resolved, I will close this.


[usage] Do not collapse menu when click outside of it

Hello!
First of all, congratulations on the amazing, excelent plugin.
I have a mobile app, that render some information on a bottom panel, powered by mmenu. The div below, is a full screen map.
I was wondering if there is way to keep the menu/panel on canvas while the user interacts with the map. That's because a single click, hides the information we rendered there, and it's somewhat annoying that the user needs to ask the panel to pop back up again.
Any tips?

Not by default, the DIV#mm-blocker closes the menu onMousedown / onTouchstart. I guess you could use CSS to hide the blocker alltogether:
#mm-blocker {
   display: none !important;
}

If you manually want to open/close the menu, try using the API:
http://mmenu.frebsite.nl/documentation/api.html
var $menu = $("#menu").mmenu();
var API = $menu.data( "mmenu" );

API.open();
API.close();

Brilliant! Genius!
Works perfect. Just hid the mm-blocker.


Can it work with node.js 10.24.1?

I have Error: Cannot find module 'pgvector/sequelize'
Any advice besides updating node.js?

No, Node 10 isn't supported.


Mangled Alpine.js attributes when used within slot

I'm trying to add some apline code to make quick shortcuts for links on pages but markup is being mangled.
My current template looks like this with a component with slots:
    <c-card-product-type :product_type="product_type">
      <c-slot name="actions">
        {{ product_type.serial_code|lower }}
        View
      </c-slot>
    </c-card-product-type>

The output looks like this, the attributes on the anchor link are mangled.

V

v
View

Any ideas? Moving the anchor out of the slot does fix the issue. So I assume something is going on with the way slots work.

@destos local tests outputs this as expected.

What version of cotton are you using?
Where does @keyup.vclass come from?

@wrabit, I was on 1.0.8, updated to the latest and it is now working. I had just installed it a few weeks ago to try out and didn't think to update it, ha!


Handle methods with variadic parameters & fixed generated code for existential parameters

1 - handle methods with variadic parameters
2 - fixed generated code for existential parameters

@rokridi Linux tests failed because in the modified stencil there is a call to isVariadic on type String and not a typoe from AST.
See the error text:
SourceryRuntime/TypeName_Linux.swift:31: Fatal error: unable to lookup: isVariadic in String?

@rokridi Linux tests failed because in the modified stencil there is a call to isVariadic on type String and not a type from AST.
See the error text: SourceryRuntime/TypeName_Linux.swift:31: Fatal error: unable to lookup: isVariadic in String?

How can I run tests locally ?

How can I run tests locally ?

I suggest to read Stencil carefully, and check all occurrences of isVariadic , at least one of them is used against String type.
It is faster and more reliable to do than if to setup a VM and dive into debugging in VSCode.
If you won't be able to deduct what's wrong, feel free to tell me, I might have a chance to check it myself later this week.

 Please find the instructions how to update it here.
Generated code for one of the mockable protocols is not valid swift:

class StaticMethodProtocolMock: StaticMethodProtocol {

    static func reset()
    {
         //MARK: - staticFunction
        staticFunctionStringStringCallsCount = 0
        staticFunctionStringStringReceived = nil
        staticFunctionStringStringReceivedInvocations = []
        staticFunctionStringStringClosure = nil

    }

    //MARK: - staticFunction

    static var staticFunctionStringStringCallsCount = 0
    static var staticFunctionStringStringCalled: Bool {
        return staticFunctionStringStringCallsCount > 0
    }
    static var staticFunctionStringStringReceived: (String)?
    static var staticFunctionStringStringReceivedInvocations: [(String)] = []
    static var staticFunctionStringStringReturnValue: String!
    static var staticFunctionStringStringClosure: ((String) -> String)?

    static func staticFunction(_ : String) -> String {
        staticFunctionStringStringCallsCount += 1
        staticFunctionStringStringReceived =
        staticFunctionStringStringReceivedInvocations.append()
        if let staticFunctionStringStringClosure = staticFunctionStringStringClosure {
            return staticFunctionStringStringClosure()
        } else {
            return staticFunctionStringStringReturnValue
        }
    }

}

Everything else seems to be OK within the macOS tests context! Great work!
🤝

👋🏻  Hey @pocketal
Do you need any help with this PR?
I could take over if you want, I am almost done with 2.1.8 release preparation at the moment, and I would love to include this fix into it.
Thanks for your contribution 👍🏻

If I would take over, I would cherry-pick your commits into a new PR and continue from there, persisting attribution.

Hey @art-divin
Sorry I was very busy these days and didn't have time to look into it.
You can take over if you want.
Thanks for you help 🙏

Closing in favor of #1304


Error: object.__switch_error not found. The current runtime does not support generating assert messages, or the runtime is corrupt.

Pretty much the assert(false) which final switch generates

Fixed after importing on object.d core.internal.switch_


[Bug] Error when accessing user's plan as an org admin

Hello everyone,
I found some bugs while accessing a user's plans as an org admin :

On the user profile, when using the pagination in the plans list, the following error appear in the console :

Started GET "/paginable/plans/org_admin_other_user/2?id=1&sort_direction=desc&sort_field=plans.updated_at" for <IP_ADDRESS> at 2019-12-11 16:08:14 +0100
Processing by Paginable::PlansController#org_admin_other_user as JS
  Parameters: {"id"=>"1", "sort_direction"=>"desc", "sort_field"=>"plans.updated_at", "page"=>"2"}
Completed 500 Internal Server Error in 41ms (ActiveRecord: 4.2ms)
NoMethodError - undefined method `org_admin_other_user?' for #<UserPolicy:0x00005570ecef3dc0>:
  app/controllers/paginable/plans_controller.rb:56:in `org_admin_other_user
briri:house_with_garden: 4:51 PM

Not exactly a bug, but the plans pulled from the database differs wether you use the pagination or not on the user profile.
When you display the User Profile, all the user's plans are pulled :
https://github.com/DMPRoadmap/roadmap/blob/f807b5f49ab345eec2b6192039a8fc1cf3e8d3e0/app/controllers/org_admin/users_controller.rb#L13
When you use the pagination, only the plans with public or org visibility are pulled :
https://github.com/DMPRoadmap/roadmap/blob/f807b5f49ab345eec2b6192039a8fc1cf3e8d3e0/app/controllers/paginable/plans_controller.rb#L61
The org_admin_other_user function is called only when using the pagination, here in the code :
https://github.com/DMPRoadmap/roadmap/blob/f807b5f49ab345eec2b6192039a8fc1cf3e8d3e0/app/views/org_admin/users/plans.html.erb#L12

If you want a similar working example, the org admin plan list works the same way :
https://github.com/DMPRoadmap/roadmap/blob/f807b5f49ab345eec2b6192039a8fc1cf3e8d3e0/app/views/org_admin/plans/index.html.erb#L47
I'll try to look into the first bug, I'll keep you updated if I find something.

The first issue seems to be resolved by adding
def org_admin_other_user?
    @user.is_a?(User)
  end

to the policies\user_policy.rb file

Hi @benjaminfaure Is this still a problem or are we okay to close this issue? Thanks for reporting it!

No it should be good, I'm closing it !


Build Error in @hey-api/client-fetch v0.3.2: Unnamed ‘Client’ and ‘RequestOptionsBase’ Issues

Description
For @hey-api/client-fetch version 0.3.2, I get the following error when building:
% npm run build

> tsc -p tsconfig.cjs.json

src/services.gen.ts:6:14 - error TS4023: Exported variable 'client' has or is using name 'Client' from external module "<path>/node_modules/@hey-api/client-fetch/dist/index" but cannot be named.

6 export const client = createClient(createConfig());
               ~~~~~~

src/services.gen.ts:6:14 - error TS4023: Exported variable 'client' has or is using name 'RequestOptionsBase' from external module "<path>/node_modules/@hey-api/client-fetch/dist/index" but cannot be named.

6 export const client = createClient(createConfig());
               ~~~~~~

Found 2 errors in the same file, starting at: src/services.gen.ts:6

This error does not occur in version 0.3.1.
Reproducible example or configuration
https://stackblitz.com/edit/stackblitz-starters-kx7wy2
OpenAPI specification (optional)
3.0.1
System information (optional)
No response

Hey @ygpark80, this is what I'm seeing. How do I get to the error you reported?

Run npm run generate to generate the sources, and then run npm run build.

Thanks, I see. Let me look into this, I just switched the type generator step and broke this in the process

Same issue with @hey-api/client-axios in 0.2.5

And editing client-axios/dist/index.d.ts by adding an export line 72 before the interface seems to fix
// Previous code
export interface Client {
 // Remaining of the client

I also has similar issues. I needed to export the following interfaces for code to compile:
Client, QuerySerializerOptions and RequestOptionsBase.

Wow, thanks a lot! I’ve confirmed that it’s working.

No worries, that's my fault! I should add tests with different configurations as it doesn't always throw depending on your tsconfig.json


Drag from outside to inside an Iframe

There seems to be a lot of talk about Iframe support and you can setup DragDropContextProvider from within an Iframe, however, it doesn't seem possible to drag from outside to inside of an iframe.
<DragDropContextProvider backend={HTML5Backend}>

    <Frame style={{ heigth: '400px', width: '100%' }}>

        <Dustbin />

    </Frame>

      <Box name="Glass" />
      <Box name="Banana" />
      <Box name="Paper" />

</DragDropContextProvider>

Made a quick Backend to support cross iframe support. May not be the best solution but gets the job done for now. Main changes is the how HTML5Backend references this.window. Changed it to this.windows that stores all iframe windows within the parent window.
https://github.com/nbwar/react-dnd-multi-iframe-backend

now react-dnd-multi-iframe-backend can not to use, has other method darg from outside to iframe

now react-dnd-multi-iframe-backend can not to use, has other method darg from outside to iframe

What is the specific method？
@foreverzmy

problem is still present for me
I've a root application with HTML5Backend, and inside iframes driven with react-frame-component.
So the content of these iframes is composed by the top level application window.
When start dragging from outside, canDrop well appears inside the iframe,  but isOver never occurs, so I cannot drop anything
When start dragging from inside the iframe, isDragging never occurs
How can we resolve it ?

I've just found one way oy do it but I access private properties of HTML5Backend for this:
import {useDragDropManager} from 'react-dnd';
import Frame, {FrameContextConsumer} from 'react-frame-component';

...

const dragDropManager = useDragDropManager();

return (
  <Frame>
    <FrameContextConsumer>
      {
        // Callback is invoked with iframe's window and document instances
        ({document, window}) => {
          const backend = dragDropManager.getBackend();
          (backend as any).addEventListeners(window); // => add listeners on the chind iframe
        }
      }
     </FrameContextConsumer>
     <all the frame display I need />
  </Frame>
);

But to be able to do this, I need to be able to access addEventListeners / removeEventListeners method of the backend, that are not public at all.
Is is possible to make them public ?

I have a similar requirement of dragging from the main window and dropping into and iframe.  It would be nice to have a standard solution for this.

Still looking for a solution for this.
Would be great if there was an officially documented solution, or have it support parent=>iframe dropping out of the box.

So apparently I didn't search well enough 🙄
There's a simple solution: https://react-dnd.github.io/react-dnd/examples/dustbin/iframe

So apparently I didn't search well enough 🙄
There's a simple solution: https://react-dnd.github.io/react-dnd/examples/dustbin/iframe

The issue the author needs help with is dragging from outside an Iframe to inside an Iframe.
The link you posted contains the Draggable item and the Droppable in the IFrame itself.


Rename control-plane-operator to hosted-cluster-config-operator

The control-plane-operator is responsible only for managing certain parts
of the configuration of a hosted control plane rather than the control plane
itself as a whole. Rename the operator to reflect its responsibilities and
allow the control-plane-operator name to be reclaimed by a new component
which will assume those broader responsibilities.

/Library/Developer/CommandLineTools/usr/bin/make -C test/e2e run
mkdir -p /Users/dmace/Projects/hypershift/test/e2e/data
KUSTOMIZE_PLUGIN_HOME=/Users/dmace/Projects/hypershift/config/example-cluster/plugin kustomize build --enable_alpha_plugins /Users/dmace/Projects/hypershift/config/example-cluster > /Users/dmace/Projects/hypershift/test/e2e/data/example-cluster.yaml
/Users/dmace/Projects/hypershift/hack/tools/bin/ginkgo -v -trace -tags=e2e -focus="" -nodes=1 --noColor=false  . -- \
          -e2e.data-dir="/Users/dmace/Projects/hypershift/test/e2e/data"
Running Suite: hypershift-e2e
=============================
Random Seed:<PHONE_NUMBER>
Will run 1 of 1 specs

I0203 10:49:33.288383   62035 request.go:645] Throttling request took 1.015257692s, request: GET:https://api.dmace-c0d3.devcluster.openshift.com:6443/apis/helm.openshift.io/v1beta1?timeout=32s
When following the HyperShift quick-start [PR-Blocking]
  Should create a functional guest cluster
  /Users/dmace/Projects/hypershift/test/e2e/quick_start_test.go:65
STEP: Applying the example cluster resources
INFO: Waiting for cluster resource to exist
STEP: Ensuring the guest cluster exposes a valid kubeconfig
INFO: Waiting for guest kubeconfig to become available
STEP: Establishing a connection to the guest apiserver
STEP: Ensuring guest nodes become ready
INFO: found 1 nodes
STEP: Deleting the example cluster
STEP: Ensuring the example cluster resources are deleted
INFO: Waiting for guest cluster namespace to be deleted
INFO: Waiting for the cluster resource to be deleted

• [SLOW TEST:509.694 seconds]
When following the HyperShift quick-start [PR-Blocking]
/Users/dmace/Projects/hypershift/test/e2e/quick_start_test.go:28
  Should create a functional guest cluster
  /Users/dmace/Projects/hypershift/test/e2e/quick_start_test.go:65
------------------------------

Ran 1 of 1 Specs in 512.682 seconds
SUCCESS! -- 1 Passed | 0 Failed | 0 Pending | 0 Skipped
PASS

Ginkgo ran 1 suite in 8m37.252963695s
Test Suite Passed

/lgtm


Rewrite the README file to include the purpose of this project

You can pull the purpose from Notion, if it helps.

@order-desk give some relevant information of this project "order-desk"

Okay, assign to me.

Hi @Soumyajit2825 and @DevTomilola-OS, you've stumbled into a project that we're running internally with our company. As outlined in CONTRIBUTING.md, requests from outside our organization will be rejected. Apologies!


Form vulnerable to malicious fomulae

Hi Jamie, it's best to sanitise the data submitted to Google Sheets by removing any "=" characters. Right now anyone using your code is at risk of all their Sheet's data being exfiltrated from the sheet by a malicious =IMPORTDATA formula.
Kind regards,
Ollie

@Ollie-Boyd Can you please help me with an instance of the same.


Wrong ball position in slider

If min in slider is not 0, then it will be posed incorrect, setValue() method should be something like this:
setValue(final int value) {
        if (!placedBall)
            post(new Runnable() {

                @Override
                public void run() {
                    setValue(value);
                }
            });
        else {
            this.value = value;
            float division = (ball.xFin - ball.xIni) / (max - min); // was - ball.xFin - ball.xIni) / max
            int _value = this.value - min; // this line is new, you were using parameter value (thats incorrect, obviously)
            ViewHelper.setX(ball,
                    _value * division + getHeight() / 2 - ball.getWidth() / 2);
            ball.changeBackground();
        }
}

I am also facing the same issue


No back button in the appointment page

There is no back button in the appointment page so the user can not go back to the home page currently.
I would love to implement a Back or Home button in the page, so the the user can easily be redirected to the Home page.
@tushargupta1504

Please assign this task to me @tushargupta1504

Please assign this issue to me @tushargupta1504
and tag it hacktoberfest, gssoc labels.


Fix: Information related to tokens in README

Fix documentation related to tokens. The secrets.GITHUB_TOKEN provided by the GitHub Actions App can do everything related to the repo if we elevate its permissions, including calling workflow_dispatch and repository_dispatch.
Some people in my organization are using PAT's instead of the secrets.GITHUB_TOKEN when using this action because of the README suggestion, even when they are calling the same repository. Using PATs in that context provides unnecessary security risks.

Testing

Create two workflows:

trigger.yaml
name: Trigger

on:
  workflow_dispatch:

permissions:
  actions: write

jobs:
  workflowdispatch:
    runs-on: ubuntu-latest
    steps:
      - uses: aurelien-baudet/workflow-dispatch@v2
        with:
          workflow: triggered.yaml
          token: ${{ secrets.GITHUB_TOKEN }}

triggered.yaml
name: Triggered

on:
  workflow_dispatch:

jobs:
  read:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/github-script@v6
        with:
          result-encoding: string
          script: |
            const fs = require("fs")
            const eventPath = '${{ github.event_path }}'
            fs.readFile(eventPath, "utf-8", (err, data) => {
              console.log(data)
              console.error(err)
            })
            return eventPath

Manually run the Trigger workflow
See the Triggered workflow will also run


Enable security for CI testing

For whatever reason, I had the --enable=false flag passed into the security-setup script. I have a feeling this was copy-pasta. This will fix a whole class of Travis CI errors.

Mainly for my own benefit, here is the source of the copy pasta: https://github.com/CiscoCloud/mantl/blob/db2884caad33a6f1fd42c06efb3db37afa64ab44/docker_launch.sh#L15

LGTM, merge on green (or at least when this isn't the problem)

GCE passed, AWS failed on consul election, and DO has a fresh bug I haven't seen yet (not directly related here). Merging


Fixes ENYO-2695

add spotlight observer to assist in hiding contextual popups
Enyo-DCO-1.1-Signed-off-by: Ryan Duffy<EMAIL_ADDRESS>

Closing as WNF


Update cortana-in-teams.md

Added the intelligent speaker to the list

@SerdarSoysal PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

Approved by tech reviewer via email.


Programmatically Generated Types/Methods

I've been working on some go language bindings for the SoftLayer API as well and I just recently found this repo.
SoftLayer exposes metadata about the API that can be used programmatically create language bindings. The Java bindings are one example of the metadata being used this way: https://github.com/softlayer/softlayer-java. The metadata is available here: https://api.softlayer.com/metadata/v3.1.
On the go bindings I worked on I focused on auto-generating the types and service methods for all of SoftLayer's API. This results in extremely large structs and one very large package with all the types in it. Attempting to split them up proved near impossible since there are cyclic references within the API.
E.G.: Accounts have VMs. VMs each have an account.
Anyway, here's a working example of what I have so far: https://github.com/sudorandom/softlayer-go. I just wanted to share what I have done in the hopes that it might help.

Hi,
Thanks for sharing. But you might have hit the reason why I created this project. The SL APIs is huge and complicated. And interestingly enough not all of it is needed, or used frequently. I think it's a case of 20% is often used and 80% rarely used or only used in special cases.
This project aims at creating a Golang client for the 20% or so that's needed and also make it easy to add new features as needed.
This is already being used in a few projects. It would make little sense to add the generated types since they (as you said) include too much and would require us to support (all) the APIs which would complicate things and go against our goal.
Cheers,
@maximilien

Cool, thanks for the reply. I have the same views toward the breadth of the SL API. That was the motivation behind the 'manager' concept in the softlayer-python library: https://github.com/sudorandom/softlayer-python/tree/master/SoftLayer/managers.


Problem while installing via addon manager

Hi,
When i try to install GW2Radial via addon manager, it told me it work but it didn't.
Since the last GW2 version on 07/07/2020, the version of d3d9.dll download by addon manager make my game crash immediatly. (i try to download addon manager the 08/07/2020 but it didn't change anything)
So i uninstall my addons and download manually d3d9.dll on the arcdps website to make arcdps and GW2 work again. (and it work now)
Then i used addon manager to install GW2Radial but it's not working in game.
Im playing on W10.
Am i doing it wrong or the last version broke the system? Is there any method to install gw2radial whithout addon manager?
thanks

You cannot use the addon manager to install GW2Radial (or any other addon) if you overwrote d3d9.dll.
Follow the instructions in the readme.

thanks, it works!
I don't know why but when i look the part "how to install" on readme it only says "via addon manager" and i wasn't able to found the .zip and the d3d9.dll you just gave me.
I rename it d3d9_chainload.dll as you mention it and all work perfectly even with arcdps :)


Release python base sdk v0.2.2

Related issue https://linear.app/serverless/issue/SC-564/python-sdk-custom-tags-api
@medikoo This should be merged after https://github.com/serverless/console/pull/526 to make sure lambda SDK does not break with the base v0.2.2, because this release contains a change that pushes storage of captured events to the lambda SDK. It's not a breaking change if we get https://github.com/serverless/console/pull/526 in first.

To further clarify:
Adding a new feature that doesn't affect how the package behaves for dependents, but for which to be used by dependent it needs to upgrade to a new version - is not breaking, but adding changes, which require changes on dependents side as otherwise they will  crash it's breaking
What you describe feels as the second case.
if we release an SDK v0.2.2 version that will crash the AWS Lambda SDK v0.1.7 (as published to PyPI), that's a bug


Cannot get nodes to show on palette

I'm new at node, npm, node-red, etc.  I tried installing node-red-contrib-lifx-api on these versions, and have not been successful getting the nodes to show up on the palette:

beaglebone running: node-red v0.15.2, Node.js v0.12.18, NPM v2.15.11
Ubuntu 16.04 running: node-red XX, Node.js v4.2.6, NPM 3.5.2
Bitanami VM preloaded with node-red v0.16.2, Node.js v7.4.0, NPM v4.0.5

I tried the various versions because I read there were issues with certain versions of these packages, and hoped I'd get lucky and happen to find a combo that works.
The console shows a warning message when starting node-red of "[lifx-api] Error: Cannot find module 'lifx-http-api'"
This could be as simple as I don't know the proper procedure to install the nodejs dependency, but I don't know.  I would really appreciate any help you could offer.
PS - I do have a different lifx node working, but it doesn't provide a way to use the effects.

I just did a clean install on a RPI and experienced the same issue.  One thing that looked odd was the structure of ~/.node-red/node_modules - rather than being a tree of dependencies, it was flat.  I will need to investigate further.

If I install this package using npm@2 (sudo npm install -g<EMAIL_ADDRESS>I get the tree structure expected and the lifx nodes appear in the node-red palette.  (macOS, node-red 0.15.1, Node.js 6.6.0, npm 2.15.11)
I am away from RPI right now, so I can't confirm whether using npm@2 fixes what I observed earlier.  I will try when I get a chance.

Issue appears to have been introduced in v1.0.2 of lifx-http-api.  The package.json on github includes the following change:
-  "main": "source/lifx.js",
+  "main": "dist/lifx.js",

... however the image of the package.json included in npm install (1.0.2) is not reflective of this change and still points to source/lifx.js which does not exist.  Have raised an issue on that project.

npm version 1.0.1 includes a temporary fix for this issue.

Thanks for jumping on this right away and working out the solution!  I can
confirm that installing 1.0.1 did resolve the node-red launch error, and
the nodes are available on the palette.  I didn't have time to test it's
functionality, but I will do that tonight.  Thanks again, I really
appreciate that you created this node!
On Mon, Jan 30, 2017 at 2:27 AM alisdairjsmyth<EMAIL_ADDRESS>wrote:

Closed #1
https://github.com/alisdairjsmyth/node-red-contrib-lifx-api/issues/1.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/alisdairjsmyth/node-red-contrib-lifx-api/issues/1#event-940914523,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AYDtCM99hFYzfv470cWUStFIjO2lGe-aks5rXZDKgaJpZM4Lw3F7
.


false negative Path::getParent

Path::getParent can return null if you are at /
  @Test
  void test() {
    findConfig( Path.of( "" ) );
  }

  Optional<Path> findConfig( Path searchDir ) {
    var configFile = searchDir.resolve("foo.yml");

    if ( Files.exists( configFile ) ) {
      return Optional.of( configFile );
    }

    return this.findConfig( searchDir.getParent() );
  }

here's the relevant portion of my config, it's a buildSrc kotlin plugin but that shouldn't be relevant.
plugins {
  `java-library`
  `java-test-fixtures`
  id("brix.bom")
  id("net.ltgt.errorprone")
  id("org.checkerframework")
}

val checker = the<LibrariesForChecker>()

dependencies {
  errorprone("com.google.errorprone:error_prone_core:2.4.+")
  checkerFramework(checker.processor)
  compileOnly(checker.annotations)
  testFixturesCompileOnly(checker.annotations)
}

checkerFramework {
  excludeTests = true
  if (Files.exists(buildDir.toPath().resolve("../config/stubs"))) {
    extraJavacArgs.addAll(listOf("-Astubs=$buildDir/../config/stubs"))
  }
  checkers.addAll(
    listOf(
      "org.checkerframework.checker.nullness.NullnessChecker"
    )
  )
}

\--- com.uber.nullaway:nullaway:0.8.+ -> 0.8.0
     \--- com.google.guava:guava:22.0 -> 30.1-jre (*)

------------------------------------------------------------
Gradle 7.0
------------------------------------------------------------

Build time:   2021-04-09 22:27:31 UTC
Revision:     d5661e3f0e07a8caff705f1badf79fb5df8022c4

Kotlin:       1.4.31
Groovy:       3.0.7
Ant:          Apache Ant(TM) version 1.10.9 compiled on September 27 2020
JVM:          11.0.10 (AdoptOpenJDK 11.0.10+9)
OS:           Linux 5.11.10-200.fc33.x86_64 amd64

Fedora 33
5.11.10-200.fc33.x86_64
 09:47:23 up 26 days, 10:53,  1 user,  load average: 1.82, 1.38, 1.18
              total        used        free      shared  buff/cache   available
Mem:            15G         11G        1.3G        1.3G        3.4G        3.0G
Swap:           12G        2.6G        9.4G

What's the fully qualified name of Path here? Is this java.nio.file.Path?
Note that for third-party libraries (including JDK libraries), which aren't sources being analyzed by NullAway, the tool will make optimistic assumptions (assume the method can take nulls everywhere and returns non-null). So, this sort of behavior is not unexpected, and can be resolved on your end by adding a custom library model.
That said, happy to look into including java.nio.file.Path in our default models, since it's certainly a frequently used JDK class and the issue you are seeing is likely common to most NullAway users!
Will look into this soon (hopefully within the week), and happy to accept a PR if you/anyone else can get to it before I can 🙂

Actually, got a chance to do this today. Model for Path.getParent() should be part of the next NullAway release!

Thanks and yes it's the jdk Path


Truncate ColumnIndex ByteArray Statistics

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
The usecase is if your data has very large binary or string columns (e.g. you have single elements which are 1MB for instance). Perhaps this is embedded logs or some other large payload.
Without some care, since the parquet statistics have actual column values stored (for their min/max) such columns can cause the resulting metadata / statistics to be pathologically large as a giant value can be stored as the min and max values in the stats for each page
Thankfully, the spec has a solution!

From the specification

These may be the actual minimum and maximum values found on a page, but can also be (more compact) values that do not exist on a page. For example, instead of storing ""Blart Versenwald III", a writer may set min_values[i]="B", max_values[i]="C". This allows writers to truncate large values and writers should use this to enforce some reasonable bound on the size of the index structures.

Describe the solution you'd like

I would like ColumnIndexBuilder to provide an option to limit the maximum length of the generated column index statistics, set by WriterProperties. This should probably have a reasonable default of something like 128 bytes.
The specification isn't clear about whether truncation of column chunk statistics is valid, and so we should ensure that they are not truncated to avoid incompatibility.
Describe alternatives you've considered

Additional context

Marking as "good first issue" as I think this should be a relatively straightforward addition

If no one else picket this up, I might take a stab at it.

label_issue.py automatically added labels {'parquet'} from #4389


telegraf: fix typo & deprecation warning, refactor exec checks

CI is expected to fail due to https://github.com/numtide/mkdocs-numtide/pull/3

bors merge


Add fingerprinting mitigation for getClientRects,  getBoundingClientRect

Scale the result of Range::getClientRects and Element::getBoundingClientRect
by a random +/-5% of the original value for each float in the Rect/Quad.
The random value is generated once for each Document and re-used for all the
attached elements.
The rationale is that such value, albeit constant within the rendered Document,
is within the same order of magniture of the floating point precision being
used for fingerprinting and sufficient to poison the well.
See also: http://www.gsd.inesc-id.pt/~mpc/pubs/fingerprinting-trustcom2016.pdf

The 5% noise is actually from a cited source:
N. Nikiforakis, W. Joosen, and B. Livshits, “Privaricator: Deceiving
fingerprinters with little white lies,” in Proceedings of the 24th
International Conference on World Wide Web, 2015, pp. 820–830

That paper is available here: http://www.www2015.it/documents/proceedings/proceedings/p820.pdf
Unlike this patch, PriVaricator applies 5% offsets to more than just get*ClientRect[s]. Their results have shown minimal to no breakages for the top 1000 Alexa sites by comparing screenshots of the rendered sites.
However, there are still a few concerning aspects:

I can't say with strong confidence that these sites are representative of the designs and structures of sites browsed by ungoogled-chromium users.
The paper's breakage evaluation does not consider dynamic behavior for sites that have a heavier use of JavaScript for rendering, such as web games. Errors in these values may cause accumulative error throughout the runtime of said web applications.

While such concerns may be silly for people like us, I believe it's better to keep cautious when it comes to behavior-mutating changes like these, especially if they can introduce subtle (and potentially frustrating) bugs like this patch.
As such, I would like to have this feature behind a command-line flag or chrome://flag option. Not only does it offer configurability, but it also informs the user of potential issues this can cause.
I'm not too familiar with the DOM side of web application execution, but maybe we could read this flag once per tab, and then have it pass down through Document to individual Elements to reduce the cost of looking up command-line arguments. If we are to implement multiple fingerprinting methods in the future, perhaps we could create a generic fingerprinting argument that accepts a comma-separated list of fingerprinting deception methods, and store these as a bit field to be read by various components of the DOM implementation as necessary.

The 5% noise is actually from a cited source:
That paper is available here: http://www.www2015.it/documents/proceedings/proceedings/p820.pdf

I did not understand that the whole paragraph was about PriVaricator. Thanks.

Unlike this patch, PriVaricator applies 5% offsets to more than just get*ClientRect[s]. Their results have shown minimal to no breakages for the top 1000 Alexa sites by comparing screenshots of the rendered sites.

Yet no source code was published, it would have helped similar implementations.

As such, I would like to have this feature behind a command-line flag or chrome://flag option. Not only does it offer configurability, but it also informs the user of potential issues this can cause.

Sorry, no plans to add this. I will close this PR.

I'm not too familiar with the DOM side of web application execution, but maybe we could read this flag once per tab, and then have it pass down through Document to individual Elements to reduce the cost of looking up command-line arguments.

There are similar flags already in Chromium, like those that are periodically added when a feature is being deprecated (see the deep and shadow CSS selectors for example). One can just look at how those implementations work to do the same.


improve CI tests

This PR introduces a custom action to run tests for different crates, and updates the workflow to use this action for testing:

tuxedo-core;
tuxedo-parachain-core and parachain-piece, which is currently skipped as it fails because it uses too much disk space;
tuxedo-template-runtime, which I think we could skip entirely;
all the other wardrobe pieces, which should be updated manually when the list changes.

This PR is a step towards #155.

I'm open to going this route if it is necessary. But I have some hesitations.

It adds the maintenance overhead of a custom action. I'd rather stick closely to the simple recipes from https://github.com/actions-rs/example
It is kind of a workaround for the underlying problem which is that our code takes way too fing long to compile. Obviously we inherent this problem from Polkadot, but I'd like to try to improve the compile time first.

It adds the maintenance overhead of a custom action.

True to a certain extent, since it is like a code function we call from multiple sources. It has been made just to avoid copy-pasting the same thing over and over again.

I'd rather stick closely to the simple recipes from actions-rs

Well, actions-rs has been archived and is no longer maintained. But I don't see how it would be relevant for us.
Do you mean we should find an "open source" action that does what we need to (eg test and code coverage using cache) and stick with that? I don't necessarily agree.

I'd like to try to improve the compile time first.

Sorry if this wasn't clear, but this PR goes in an opposite direction: since we know where the problem with compile time and testing is, we make sure that everything else works and is tested as expected, otherwise we might overlook on some things and introduce bugs or other issues, or forget to test something because we don't check the coverage.

Okay, I'm coming around to this more and more.

actions-rs has been archived and is no longer maintained.

Damn. I didn't realize that. That is a loss and I hope the community carries on that work somehow. I've found it very useful.
My point wasn't so much that we need to use that specific thing. My point was that I want the CI to be simple, straight-forward, idiomatic, etc. As opposed to custom, complex, hard-to-understand, etc.
We aren't CI experts (at least I'm not) and I don't really want to be one. I just want something simple that tells me if the tests pass. That's what I meant when I was talking about those actions-res examples.
I got the rocks-db dependency removed again over in #156 and I thought maybe that would save us enough disk space to not need this. But it wasn't enough.

We might try to test parachain with the --release flag to see if it takes less space.

superseded by #159


Strip markdown frontmatter

Fixes #4863

Why are we stripping this stuff? This front matter is not part of the CommonMark spec (AFAIK). From what I can tell, YAML front matter is something loosely adopted by some tools but is by no means a Markdown standard. Perhaps I am missing context here!

@joelverhagen This is a feature request from the docs team, which would allow reuse of the same .md files for both docs.microsoft.com and NuGet. It sounds like the metadata is populated automatically by the docs build system; see: https://opsdocs.azurewebsites.net/en-us/opsdocs/partnerdocs/metadata?branch=master
Docs.microsoft.com examples:

https://github.com/MicrosoftDocs/AdaptiveCards/blob/master/AdaptiveCards/index.md
https://github.com/NuGet/docs.microsoft.com-nuget/blob/live/docs/What-is-NuGet.md

I see the NuGet docs metadata contain a list and comments which I didn't account for. I was thinking this would be a quick change, but maybe I'll shelve for now and revisit the requirements when I pick it back up.

Closing - will open new PR once I've investigated requirements more...


Migrate base "wrapper" plugin to Gradle core

And reimplement samples in terms of this base plugin.

@big-guy Do you know what this issue is supposed to represent?

Oh, it's about the wrapper plugin for external build systems.


Fix potential bug of array index overflow

Phenomenon
When training an audio wav file, sometimes it fails with an error output message:

RuntimeError: The expanded size of the tensor (50) must match the existing size (54) at non-singleton dimension 0.  Target sizes: [50, 44].  Tensor sizes: [54, 44]

# record the feats efficiently.. (no concat, constant memory)
start = self.feat_buffer_idx * self.context_size
end = start + feats.shape[0]
self.feat_queue[start:end] = feats   # <<<<<<<<<<<<<<<<<<<<<<<<<< ERROR STACK
self.feat_buffer_idx = (self.feat_buffer_idx + 1) % self.feat_buffer_size

Root cause:
When self.terminated is true, the return value will occasionally be greater than context_size (50).
def frame_to_text(self, frame):
    inputs = self.processor(frame, sampling_rate=self.sample_rate, return_tensors="pt", padding=True)
    with torch.no_grad():
        result = self.model(inputs.input_values.to(self.device))
        logits = result.logits # [1, N - 1, 32]

    # cut off stride
    left = max(0, self.stride_left_size)
    right = min(logits.shape[1], logits.shape[1] - self.stride_right_size + 1) # +1 to make sure output is the same length as input.
    # do not cut right if terminated.
    if self.terminated:
        right = logits.shape[1]   # <<<<<<<<<<<<<<<<<<<<<<<<<<  ROOT CAUSE
    logits = logits[:, left:right]

    ...

Fix
When the iteration is finished (self.terminated==true), both self.feat_queue and self.feat_buffer_idx are of no use at all. We just simply add an if else to avoid this issue.

File "/data/character/digital_people/RAD-NeRF/nerf/asr.py", line 420, in
asr.run()
File "/data/character/digital_people/RAD-NeRF/nerf/asr.py", line 362, in run
self.run_step()
File "/data/character/digital_people/RAD-NeRF/nerf/asr.py", line 223, in run_step
self.feat_queue[start:end] = feats
RuntimeError: The expanded size of the tensor (32) must match the existing size (44) at non-singleton dimension 1.  Target sizes: [30, 32].  Tensor sizes: [30, 44]
I have the same question
The code has been improved, but still reporting this issue


Passing custom arguments to the script

Hey! What do you think of including the ability to pass custom props to the scripts that could be included either as attributes of the script tag, or as a data- attributes
For eg.
<Script
url="https://someUrl.com"
customProp="customPropValue"
onCreate={onCreate}
onError={onError}
onLoad={onLoad}
/>

At quick glance, the only edit necessary would be to change createScript to grab these props and add them to the script:
createScript() {
		const { onCreate, url, ...customProps } = this.props;
		const script = document.createElement('script');

		onCreate();

		if (customProps) {
                        // to include them as script attributes
			Object.keys(customProps).forEach((prop) => {
				script.setAttribute(prop, customProps[prop]);
			});
                       // or something like script.setAttribute(`data-${prop}`, customProps[prop]);
		}

		script.src = url;
		script.async = 1;

I'm working on an integration that would require attributes to be added directly to the script tag (not through data-), although it would be more standard to use data- (https://developer.mozilla.org/en-US/docs/Learn/HTML/Howto/Use_data_attributes)

@julesterrien Hi, thanks for the suggestion! Can you make a PR for that if you have time? I think that naming the prop attributes instead of customProps would work well

Cool can do. Do you think it should handle:

adding custom attributes to the script tag. ie. .
adding custom as data-. ie. <script data-custom="value/>
both (either adding any prop given both as attributes and data- style attributes, or letting the dev define which option he/she prefers using another prop)

I think that all scenarios could be handled by attributes={{ 'data-custom': 'val', 'custom': 'hello' }} would be enough 👍

@jakubkottnauer https://github.com/blueberryapps/react-load-script/pull/10

@julesterrien v0.0.5 published ✅


DNM: Testing Pac 0.15.4

E2E Tests and CI for PAC 0.15.4 before release
Changes

Submitter Checklist

[ ] ♽  Run make test lint before submitting a PR (ie: with pre-commit, no need to waste CPU cycle on CI
[ ] 📖 If you are adding a user facing feature or make a change of the behavior, please verify that you have documented it
[ ] 🧪 100% coverage is not a target but most of the time we would rather have a unit test if you make a code change.
[ ] 🎁 If that's something that is possible to do please ensure to check if we can add a e2e test.
[ ] 🔎 If there is a flakiness in the CI tests then don't necessary ignore it, better get the flakyness fixed before merging or if that's not possible there is a good reason to bypass it. (token rate limitation may be a good reason to skip).

Codecov Report

Merging #1137 (2fc2a84) into release-v0.15.3 (474a9db) will decrease coverage by 0.14%.
The diff coverage is 52.32%.

@@                 Coverage Diff                 @@
##           release-v0.15.3    #1137      +/-   ##
===================================================
- Coverage            62.97%   62.83%   -0.14%
===================================================
  Files                  106      106
  Lines                 6382     6420      +38
===================================================
+ Hits                  4019     4034      +15
- Misses                1978     2000      +22
- Partials               385      386       +1

Impacted Files
Coverage Δ

pkg/cli/webhook/github.go
57.00% <ø> (ø)

pkg/cmd/tknpac/resolve/resolve.go
63.24% <ø> (ø)

pkg/cmd/tknpac/webhook/add.go
55.17% <0.00%> (ø)

pkg/formatting/emoji.go
100.00% <ø> (ø)

pkg/kubeinteraction/events.go
0.00% <ø> (ø)

pkg/kubeinteraction/pod_logs.go
0.00% <ø> (ø)

pkg/params/settings/config.go
60.00% <ø> (ø)

pkg/pipelineascode/match.go
58.82% <0.00%> (ø)

pkg/provider/bitbucketcloud/bitbucket.go
75.79% <ø> (ø)

pkg/provider/bitbucketserver/bitbucketserver.go
82.43% <ø> (ø)

... and 19 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Combine two constraints for people, groups and rooms

Hi,
My goal is to mix children from different classes in different groups for a special event where children have to work with children from other classes.
Then each group will be assigned to available classroom.
The main constraints for the groups are that groups are composed from children from different classes, there are also some skill level constraint.
The constraint for assigning the group to the classroom is the capacity of each classroom but also that some child (very few, we marked them with '*') must be in their classroom (indeed to be under the surveillance of their teacher !!)
I managed to have a correct list of groups.
I managed to have the correct number of groups per classroom, but I cannot figure out how to have the groups with '*' children in the right classroom.
My data contains the list of children:
for e in all_eleves , e[0] is the pupil name, e[1] is its class, e[2] is its skill level, e[3] ='*' for the marked children who should stay in their classroom
I defined the groups as
groups = {}
    for e in all_eleves:
        for g in all_groups:
            groups[(e,g)] = model.NewBoolVar('group_e%sg%i' % (e[0],g))

Adding some constraint such as (there are much more)
for e in all_eleves:
        model.Add(sum(groups[(e,g)] for g in all_groups) == 1)
for c in range(num_classes):
        for g in all_groups:
            model.Add(sum(groups[(e,g)] for e in all_eleves if e[1] == c ) <= 1)

Then for the classroom assignment I defined rooms as
rooms={}
    for r in all_classes:
        for g in all_groups:
            rooms[(r,g)] = model.NewBoolVar('room_r%sg%i' % (r,g))

I added one constraint which use a calculated num_groups_per_room
for r in all_classes:
        model.Add(sum(rooms[(r,g)] for g in all_groups) <= num_groups_per_room[r])

Until there, everything works well. Now I would like to add the constraint that when a marked children (e[3]='*') is in a group, this group should be in the room of the children class (e[1])
I tried
for e in all_eleves:
        if e[3] == '*':
            for r in all_classes:
                if e[1] == r:
                    model.Add(sum(rooms[(r,g)]*groups[(e,g)] for g in all_groups) == 1)
                else:
                    model.Add(sum(rooms[(r,g)]*groups[(e,g)] for g in all_groups) == 0)

But I get the following error:
raise TypeError('Not an integer linear expression: ' + str(arg))
TypeError: Not an integer linear expression: group_eSéraphineg0
How should I link groups[(e,g)] and rooms[(r,g)] so that a condition on e (e[3]=='*') triggers g to be the classroom r ?

What is the faulty line?
What are the types of the arguments ?
Le ven. 7 juin 2019 à 01:11, Yohann Agostini<EMAIL_ADDRESS>a
écrit :

Hi,
My goal is to mix children from different classes in different groups for
a special event where children have to work with children from other
classes.
Then each group will be assigned to available classroom.
The main constraints for the groups are that groups are composed from
children from different classes, there are also some skill level constraint.
The constraint for assigning the group to the classroom is the capacity of
each classroom but also that some child (very few, we marked them with '*')
must be in their classroom (indeed to be under the surveillance of their
teacher !!)
I managed to have a correct list of groups.
I managed to have the correct number of groups per classroom, but I cannot
figure out how to have the groups with '*' children in the right classroom.
My data contains the list of children:
for e in all_eleves , e[0] is the pupil name, e[1] is its class, e[2] is
its skill level, e[3] ='*' for the marked children who should stay in their
classroom
I defined the groups as
groups = {}
for e in all_eleves:

    for g in all_groups:

        groups[(e,g)] = model.NewBoolVar('group_e%sg%i' % (e[0],g))

Adding some constraint such as (there are much more)
for e in all_eleves:
    model.Add(sum(groups[(e,g)] for g in all_groups) == 1)

for c in range(num_classes):
    for g in all_groups:

        model.Add(sum(groups[(e,g)] for e in all_eleves if e[1] == c ) <= 1)

Then for the classroom assignment I defined rooms as
rooms={}
for r in all_classes:

    for g in all_groups:

        rooms[(r,g)] = model.NewBoolVar('room_r%sg%i' % (r,g))

I added one constraint which use a calculated num_groups_per_room
for r in all_classes:
    model.Add(sum(rooms[(r,g)] for g in all_groups) <= num_groups_per_room[r])

Until there, everything works well. Now I would like to add the constraint
that when a marked children (e[3]='*') is in a group, this group should be
in the room of the children class (e[1])
I tried
for e in all_eleves:
    if e[3] == '*':

        for r in all_classes:

            if e[1] == r:

                model.Add(sum(rooms[(r,g)]*groups[(e,g)] for g in all_groups) == 1)

            else:

                model.Add(sum(rooms[(r,g)]*groups[(e,g)] for g in all_groups) == 0)

But I get the following error:
raise TypeError('Not an integer linear expression: ' + str(arg))
TypeError: Not an integer linear expression: group_eSéraphineg0
How should I link groups[(e,g)] and rooms[(r,g)] so that a condition on e
(e[3]=='*') triggers g to be the classroom r ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/1336?email_source=notifications&email_token=ACUPL3KBJ2RIXBG4YBHSPHLPZGDP7A5CNFSM4HVK4M5KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4GYEGQUA,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACUPL3MUSP5GEC3ONH4VN4DPZGDP7ANCNFSM4HVK4M5A
.

OK, Add() expects linear constraints.
Clearly you rooms[(r, g)] * groups[(e, g)] is quadratic.
You need to create intermediate variables
See
https://github.com/google/or-tools/blob/master/ortools/sat/doc/boolean_logic.md#product-of-two-boolean-variables

Thanks a lot,
I set up
# link the pupils, the groups and the room (groups[(e,g)] and rooms[(r,g)] <=> same_room[(e,r,g)])
    same_room={}
    for r in all_classes:
        for g in all_groups:
            for e in all_eleves:
                same_room[(e, r, g)] = model.NewBoolVar(
                    "eleve %s is in group %i in classroom %s" % (e[0], g, r))

    for e in all_eleves:
        for g in all_groups:
            for r in all_classes:
                model.AddBoolOr([groups[(e,g)].Not(),rooms[(r,g)].Not(),same_room[(e,r,g)]])
                model.AddImplication(same_room[(e,r,g)],groups[(e,g)])
                model.AddImplication(same_room[(e,r,g)],rooms[(r,g)])

    # pupils marked with a * seat in their classroom
    for r in all_classes:
        model.Add(sum(same_room[(e,r,g)] for g in all_groups
            for e in all_eleves if (e[1] != all_classes.index(r)) and (e[3] == '*')) == 0)

It works well


dep: bump tm-db to master & remove panics

Test Pr to see what would change with a new breaking release of tm-db.
remvoing panics on the way.

closing this in favor of #279


Explaining abbreviation

Not sure how useful it is, but can't hurt to expand on an abbreviation that not everyone might be familiar with.
Cheers

Hi @kjmears, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@kjmears - Thanks for your contribution. We are in the process of refactoring the Azure technical articles, so we can’t merge this PR. When the refactoring is complete, we can give you a link to the new location of the article you updated so you can resubmit your update, if you want to.

@kjmears - Thanks for your contribution to the Azure docs. On November 16th, we announced the availability of our Azure technical documentation in a new location on docs.microsoft.com. You can read about it on our team blog. This change means the article source lives in a new GitHub repository. Because there are some changes in the source markdown as a result of the move, we can’t pull your contribution over to the new repo without creating a ton of merge conflicts. We invite you to resubmit the change (if you are willing) to the new version of the article at https://docs.microsoft.com/en-us/azure/power-bi-embedded/power-bi-embedded-what-is-power-bi-embedded. Thanks for your understanding.


Try to port test_bessel to Triton

Part of https://github.com/intel/intel-xpu-backend-for-triton/issues/2030

Done in https://github.com/triton-lang/triton/commit/2ea9daa54fd50ffa6363a3f3f9dd7514e16ff4bf


Put Quick Filters in yaml metadata - 1064027

Title
Added quick filters in metadata
Summary
https://mseng.visualstudio.com/CSI/_workitems/edit/1064027
Suggested Reviewers
@jdanyow @adkinn

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

This build shows up as failed in OPS Portal. I'll try to kick it off again.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@mairaw this code change is not published yet so the build might not work as expected. We'll resubmit or let you know when the code change is ready.

Closing pull request and will resubmit once supporting code has been launched.


🐛 Broken link in  biome init

Environment information
CLI:
  Version:                      1.1.2
  Color support:                true

Platform:
  CPU Architecture:             aarch64
  OS:                           macos

Environment:
  BIOME_LOG_DIR:                unset
  NO_COLOR:                     unset
  TERM:                         "xterm-256color"
  JS_RUNTIME_VERSION:           "v20.5.0"
  JS_RUNTIME_NAME:              "node"
  NODE_PACKAGE_MANAGER:         "bun/1.0.0"

Biome Configuration:
  Status:                       Loaded successfully
  Formatter disabled:           false
  Linter disabled:              false
  Organize imports disabled:    false
  VCS disabled:                 true

Workspace:
  Open Documents:               0

What happened?

bunx biome init
instruction shown

 1. Setup an editor extension
     Get live errors as you type and format when you save. Learn more: https://biomejs.dev/editors

https://biomejs.dev/editors is not a valid link
Expected result
I think the correct one is https://biomejs.dev/guides/getting-started#editor-setup.
Not sure If updating references of  https://biomejs.dev/editors to valid links is more appropriate or a redirect.
Code of Conduct

[X] I agree to follow Biome's Code of Conduct

Yes, I'd say that "editors-section" is the most suitable place where to redirect users at the moment :)
Thank you for reporting it

@ematipico I put up a PR fixing it 😄


Allow opting-out of CustomDebugStringConvertible to reduce binary size

This observation comes from researching https://github.com/apple/swift-protobuf/issues/1204.
It appears that CustomDebugStringConvertible's contribution to the binary size adds up and is not insignificant.
A quick check shows that removing it from Message, SimpleExtensionMap, AnyExtensionField would save us ~800kB.
For context, our project has around 1000 message definitions.
This was a modified version 1.17.0,
compiled with -Osize,
Xcode 13.2.1 13C100, Swift 5.5.2, Big Sur
Since conforming to CustomDebugStringConvertible does not seem essential, perhaps it could could be tied to a compilation flag?
I'd like to hear some initial feedback @thomasvl before getting into details.

This sounds sorta like another take on #1240.
With respect to compilation flags - those seems like they never work in practice with build systems - i.e. - how does one make that work (at scale) with SwiftPM/CocoaPods/etc.
TVL

@thomasvl: The DEBUG compilation condition seems fairly universal among build systems:

CocoaPods adds DEBUG to swift compilation conditions under debug in the generated Xcode project (link)
SwiftPM adds -DDEBUG to all compiles in the debug configuration (link)
rules_swift adds it for dbg and fastbuild configs (link)

Yes, DEBUG would be less of an issue, I was taking the initial comment to be something developers directly controlled.

@thomasvl Took a crack at it here: https://github.com/apple/swift-protobuf/pull/1246