Define DB in snowflake connection rather than environment variable

Hello,
We have one snowflake account and within that account we have all of our lower environments defined as databases that we clone from prod. We also use code generation scripts to generate some of our standard tables and workflows. We are currently defining the environment by setting a variable and manually adding the "use database {{ db_var }};" in front of all of our generated code (and non generated code). We would like to instead have an option to add the DB to the connection string into snowflake within the python Snowchange job. Our branches are named the same as our DBs so then we can dynamically set the DB in the YAML file and not have to include the "use database {{ db_var }};" at the top of all of our SQL scripts. We are finding that difficult to remember to do.
I can make the change and request it to be merged. Would that be acceptable?

Hey there @monikaac, thanks for the suggestion, and I think that makes sense. It sounds like your suggesting adding a command line argument to specify the database name and updating the apply_change_script() function to pass the database name to the main execute_snowflake_query() call. I think that would be a nice addition for snowchange users and still allow snowchange to be very flexible. Please do make the change and submit a PR!

@monikaac, yesterday I released version 2.6 and was able to include this change in that release. So nothing to do from your end. Hope it helps!

@monikaac, yesterday I released version 2.6 and was able to include this change in that release. So nothing to do from your end. Hope it helps!


currently user have no access to what will be the greeting message

Problem
Can't apply message according to my choice.
Request
I want to use this for a repo but want the message according to me. and think this should be a new version as if it is also a most needed feature.

@mayankmohak i'm already working on this. Soon it will be published. Thanks again (from the human) 😄

@mayankmohak I have added the feature, test it and freely use in your repo. Happy Coding :+1:


Really long filter regex causes window malfunction

A really long filter causes the window to be too thin and the trash button no longer appears.
Closing the org settings and re-opening has no effect. Stretching the window's aspect ratio extremely wide has no effect.
Workaround: Inspect elements, set max-width of settings page to 1920 or so.

So it is.  Seems definitely worth fixing.  @synicalsyntax is this something you'd be interested in taking?


Mismatch B/w Different Runs

I tried running depth_fm on an image twice with the same settings but I am getting visual difference between the two runs -- is that due to the generative nature of depth_fm?

Never mind, it looks like, I was comparing between different settings.


Tamper protection for Endpoint DLP?

Does the tamper protection cover Endpoint DLP as well? Since Endpoint DLP rides on the same MDE agent/service. If so, there is no mention about Endpoint DLP on this documentation  or any other documentation. Could we add a mention on that?

To make it easier for you to submit feedback on articles on learn.microsoft.com, we're transitioning our feedback system from GitHub Issues to a new experience.
As part of the transition, this GitHub Issue will be moved to a private repository. We're moving Issues to another repository so we can continue working on Issues that were open at the time of the transition. When this Issue is moved, you'll no longer be able to access it.
If you want to provide additional information before this Issue is moved, please update this Issue before December 15th, 2023.
With the new experience, you no longer need to sign in to GitHub to enter and submit your feedback. Instead, you can choose directly on each article's page whether the article was helpful. Then you can then choose one or more reasons for your feedback and optionally provide additional context before you select Submit.
Here's what the new experience looks like.
Note: The new experience is being rolled out across learn.microsoft.com in phases. If you don't see the new experience on an article, please check back later.
First, select whether the article was helpful:

Then, choose at least one reason for your feedback and optionally provide additional details about your feedback:

Article was helpful
Article was unhelpful

Finally, select Submit and you're done!


Consider adding Limitations section

Noticing that certain tasks don't support Powershell core, which is what Hosted Linux Agents run, powershell wise.
It might make some sense to add a limitation section for tasks to outline known limitations (I don't believe full AzureRM, which this task is, runs on powershell core). Without it, it can be confusing as there is an AzureRM (abliet, a limited one) that works on powershell core, but this task runs full AzureRM, which appears to require full powershell, which requires Windows agent (hosted or private).

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

ID: fb215324-b1a0-93b5-5022-4bcc7da93bb7
Version Independent ID: 9eb2977b-ac34-9d53-670e-04fa027bc8e4
Content: Azure PowerShell
Content Source: docs/build-release/tasks/deploy/AzurePowerShellV3.3.md
Service: devops
GitHub Login: @erickson-doug
Microsoft Alias: dastahel

@DavidStaheli Can you take a look at this one? Thank you!

The Azure powershell step has a built in demand for AzurePS. The job would execute on a compatible agent only. If an agent has powershell core only, then it would not be picked up for execution of the job.


context restore error

Version

Phaser Version: 3.55.2
Operating system:  all
Browser: all

Description
WARNING: Too many active WebGL contexts. Oldest context will be lost.
When encountering this kind of problem, it will trigger the WebGL context loss event, but how to recover from the context loss.
The function contextRestoredHandler in WebGLRenderer only reinitializes WebGLRender, but it does not restore the related Buffer, Texture, and Shader
Example Test Code
Additional Information

Yeah, I'm aware this doesn't work (I'm not sure it ever has, to be honest).
I also don't think this is an easy fix, either. There's so much inside the API now that has no concept of restoring from a context loss that it may be easier just to remove the current handlers and leave it as a known issue rather than half-heartedly try to restore it.

Ok. I've removed all of the half-baked restore handling and the renderer will now just die gracefully and not spam the console with errors. This isn't something we can put in Phaser 3, so it'll have to be done properly in Phaser 4.


add documentation on configuring AWS access

I have so far been sending these instructions in slack messages, let's finally write them down and link to them.

should pass after https://github.com/brain-score/brainio/pull/30 is merged

re-trigger


Having trouble with running Docker images in Jenkins slaves

Hi again!
So I'm making progress on my own build. The pipeline is working great and deploying my applications images just fine. But now I'm trying to compile certain project assets for the images and I'm running into issues.
For example, the project uses npm. In our environment, we don't use native installs of tooling, and instead run everything in Docker images. npm is no exception. The Docker images are working great in our dev environment, and our current CI environment (CircleCI), but whenever I try to run the following command in a stage of the Jenkinsfile:
sh('npm install');

I get the following error in the build logs:
[nds_google_container_engine-NLXIZZR7ZURR4JJEWIOWTD36XE5QABL5DQH7Z73ZQIDCJJIU2IDQ] Running shell script
+ npm install
docker: Error response from daemon: mkdir /root/workspace: read-only file system.

I've ran some tests and it seems that the home folder (/root) in the Jenkins slaves is read only (the whole slave may be read-only). I'm assuming the Docker daemon is trying to create this workspace folder, but I'm honestly unfamiliar with it. I use Docker on Mac, and some of the inner workings are obscured by the way the Hypervisor is used on this platform.
Maybe this is related to the Jenkins Kubernetes Plugin?
Do you have any suggestions on what the problem / resolution might be? I'm continuing my research regardless, but I thought I'd ping you just incase you had some wisdom to share.
Thanks again for all your help!

So it looks like changing the "Jenkins slave root directory" of the Kubernetes Plugin from "/root/" to "/home/jenkins/" may have resolved the issue. Can you comment on whether or not this is a good idea?

Hi Chekote !
I'm having the same problem, I changed the root directory like you did but I can't run any shell script inside the container, when I try sh("node -v") for example, I have those errors:
I'm trying:
docker.image('node:4.1.2').inside {
    print "inside a node server"
    sh("echo test");
  }

And I got:
[Pipeline] sh
[play_PLR-437-jenkins-config-WQ5IB66PEGACJNE6UHFF54RVEEBWEEDWRSVCZM3YSVATI3UYUBXA] Running shell script
+ docker inspect -f . node:4.1.2
.
[Pipeline] withDockerContainer
$ docker run -t -d -u 0:0 -w /home/jenkins/workspace/play_PLR-437-jenkins-config-WQ5IB66PEGACJNE6UHFF54RVEEBWEEDWRSVCZM3YSVATI3UYUBXA -v /home/jenkins/workspace/play_PLR-437-jenkins-config-WQ5IB66PEGACJNE6UHFF54RVEEBWEEDWRSVCZM3YSVATI3UYUBXA:/home/jenkins/workspace/play_PLR-437-jenkins-config-WQ5IB66PEGACJNE6UHFF54RVEEBWEEDWRSVCZM3YSVATI3UYUBXA:rw -v /home/jenkins/workspace/play_PLR-437-jenkins-config-WQ5IB66PEGACJNE6UHFF54RVEEBWEEDWRSVCZM3YSVATI3UYUBXA@tmp:/home/jenkins/workspace/play_PLR-437-jenkins-config-WQ5IB66PEGACJNE6UHFF54RVEEBWEEDWRSVCZM3YSVATI3UYUBXA@tmp:rw -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** --entrypoint cat node:4.1.2
[Pipeline] {
[Pipeline] echo
inside a node server
[Pipeline] sh
[play_PLR-437-jenkins-config-WQ5IB66PEGACJNE6UHFF54RVEEBWEEDWRSVCZM3YSVATI3UYUBXA] Running shell script
sh: 1: cannot create /home/jenkins/workspace/play_PLR-437-jenkins-config-WQ5IB66PEGACJNE6UHFF54RVEEBWEEDWRSVCZM3YSVATI3UYUBXA@tmp/durable-b3b5cf3e/pid: Directory nonexistent
sh: 1: cannot create<EMAIL_ADDRESS>Directory nonexistent
sh: 1: cannot create<EMAIL_ADDRESS>Directory nonexistent
[Pipeline] }
$ docker stop --time=1 e2da950dc015082a0490b97b1b976b3bc1a91f631800284e75df5cb475725e44
$ docker rm -f e2da950dc015082a0490b97b1b976b3bc1a91f631800284e75df5cb475725e44
[Pipeline] // withDockerContainer

Any idea ?
Thanks :)

Here is an example pipeline that works:
podTemplate(label: 'ubuntu-k8s', containers: [
    containerTemplate(name: 'ubuntu', image: 'ubuntu:16.04', ttyEnabled: true,
        command: 'cat')
  ]) {
    node('ubuntu-k8s') {
        container('ubuntu') {
            stage('Run Command') {
                sh 'cat /etc/issue'
            }
        }
    }
}

Closing this in favor of #65 .

why is this closed?
not work with kubernetes and jenkins in GKE now.

OMG..  why is this closed?
point is that inside docker task never works

gcr.io slave image is not ready for jenkins docker pipeline plugin

Hi again!
So I'm making progress on my own build. The pipeline is working great and deploying my applications images just fine. But now I'm trying to compile certain project assets for the images and I'm running into issues.
For example, the project uses npm. In our environment, we don't use native installs of tooling, and instead run everything in Docker images. npm is no exception. The Docker images are working great in our dev environment, and our current CI environment (CircleCI), but whenever I try to run the following command in a stage of the Jenkinsfile:
sh('npm install');

I get the following error in the build logs:
[nds_google_container_engine-NLXIZZR7ZURR4JJEWIOWTD36XE5QABL5DQH7Z73ZQIDCJJIU2IDQ] Running shell script
+ npm install
docker: Error response from daemon: mkdir /root/workspace: read-only file system.

I've ran some tests and it seems that the home folder (/root) in the Jenkins slaves is read only (the whole slave may be read-only). I'm assuming the Docker daemon is trying to create this workspace folder, but I'm honestly unfamiliar with it. I use Docker on Mac, and some of the inner workings are obscured by the way the Hypervisor is used on this platform.
Maybe this is related to the Jenkins Kubernetes Plugin?
Do you have any suggestions on what the problem / resolution might be? I'm continuing my research regardless, but I thought I'd ping you just incase you had some wisdom to share.
Thanks again for all your help!

I know it's a very old issue, but if anyone wants to know a solution here I'll leave it:
The problem is that jenkins mounts the workdir folder as read only, if we want to raise a docker with a volume we will have to move our code to a folder like /home/jenkins/code and execute there the npm install or composer install etc... and exec docker run or docker-compose

I don't know if you everyone gave up, but I've achieved a similar example pattern here. If someone can ping me if a similar pattern works using their Kubernetis plugin, I'd appreciate it. I might want this with Kubernetis some day. Bonus points if you share the kube pattern.
// start 2nd degree container
node('ecs-jnlp-agent') {
    stage("ecs 2nd degree container") {
        withEnv(
            [
                /* JENKINS INCEPTION is required to mount the ecs-ubuntu docker.sock, which mounts the ec2 bare metal
                   docker.sock from the ECS Task definition docker.sock - The 3rd degree introspection is required
                   in order to mount the build workspace from the 2nd degree ecs agent into the 3rd degree agent.
                   Confused yet? Welcome to I.N.C.E.P.T.I.O.N.

                   ec2 docker.sock (volume exports to ) -> ecs docker.sock (volume exports to) -> 3rd degree image
                */
                'DOCKER_SOCKET_INCEPTION=-v /var/run/docker.sock:/var/run/docker.sock', // INCEPTION the docker.sock
                // The variables below you may modify to fit your projects needs
                'GIT_REPO1_URL=https://github.com/YOUR_NAME/hello_world_shell.git', // git code
                'GIT_REPO1_BRANCH=master', //...............................// git this branch
                // 'GIT_REPO1_JENKINS_CREDENTIALS_ID=ffffffff-ffff-ffff-ffff-ffffffffffff', # an ssh key maybe
                'ANSI_BLUE=\033[00;34m', //................................// experiment with color inside 3rd docker
                'ANSI_RED=\033[00;31m', //.................................// experiment with color inside 3rd docker
                'ANSI_END=\033[0m', //.....................................// experiment with color inside 3rd docker
                'BUILD_IMAGE1=ubuntu:bionic', //...........................// use multiple build images, if you wanna
                'BUILD_IMAGE2=alpine', //..................................// use multiple build images, if you wanna
            ]
        )
        {
            // use a private registry with a credentials ID
            docker.withRegistry('https://docker.io', 'ffffffff-ffff-ffff-ffff-ffffffffffff') {
                // start 3rd degree container on the ECS cluster host of 2nd degree container
                docker.image("${env.BUILD_IMAGE1}").withRun("${env.DOCKER_SOCKET_INCEPTION}") {
                    ansiColor('xterm') {
                        stage("3rd degree ubuntu: set 1, do stuff") {
                            // git some code
                            git(
                                url: "${env.GIT_REPO1_URL}",
                                branch: "${env.GIT_REPO1_BRANCH}",
                                // credentialsId: "${env.GIT_REPO1_JENKINS_CREDENTIALS_ID}",
                            )
                            // run some shell
                            sh '''#!/usr/bin/env bash
                            echo -en "${ANSI_BLUE}"
                            bash echo_hello_world.sh
                            echo -en "${ANSI_END}"'''
                        }

                        stage("3rd degree ubuntu: set 2, say hello again") {
                            // shell again
                            sh '''#!/usr/bin/env bash
                            echo -en "${ANSI_RED}"
                            bash echo_hello_world.sh
                            echo -en "${ANSI_END}"'''
                        }
                    }
                }
                // start seperate 3rd degree container to build something else kind of different
                docker.image("${env.BUILD_IMAGE2}").withRun("${env.DOCKER_SOCKET_INCEPTION}") {
                    ansiColor('xterm') {
                        stage("3rd degree alpine: set 1") {
                            // git some code
                            git(
                                url: "${env.GIT_REPO1_URL}",
                                branch: "${env.GIT_REPO1_BRANCH}",
                                // credentialsId: "${env.GIT_REPO1_JENKINS_CREDENTIALS_ID}",
                            )
                            // run some shell
                            sh '''#!/usr/bin/env bash
                            echo -en "${ANSI_BLUE}"
                            bash hello.sh
                            echo -en "${ANSI_END}"'''

                            // 4th degree purgatory just to show off
                            docker.image("${env.BUILD_IMAGE2}").withRun("${env.DOCKER_SOCKET_INCEPTION}") {
                                ansiColor('xterm') {
                                    stage("4th degree alpine: purgatory") {
                                        sh '''#!/usr/bin/env bash
                                        echo -en "${ANSI_RED}"
                                        echo "You shouldn\'t be in here... the world is unstable here."
                                        echo -en "${ANSI_END}"'''
                                    }
                                }
                            }
                        }
                    }
                }
            }
        }
    }
}```


Allow zones within a program to run sequentially

Zones should also be able to run with different durations or be repeated within a program. This will also mean that zones should have a "sleep period"
Current example of program.zones:
[1,2] // run zones 1 and 2 for program.duration minutes

Proposed format:
[
  { zid: 1, duration: 5 }, // run for 5 minutes
  { zid: null, duration 30 }, // wait 30 minutes
  { zid: 1, duration: 10 } // run for another 10 minutes
]

This will only work if program.sequential is true. Otherwise, null zones are ignored, and only the first instance of each valid zone is used. This would mean the above example would just run zone 1 for 5 minutes and complete.

Or maybe running the zones without sequential should run for the sum of zones durations within the program? Using the above example, 15 minutes.


Windows uses batch script different from Unix bash script

Should we just include both tutorial.sh and tutorial.bat in the folder and instruct the user to use the appropriate one?

Yes. I pushed for The APS hydra texture a bat file that seems to work. Please test and close if all good.

The separate bat script is good and I also updated the wiki pages.


text selection not updated when an option w/ same value is selected

When you select an option which has the same value as another option the selection doesnt change.
<select>
<option value="1">item 1</option>
<option value="1">item 2</option>
<option value="2">item 3</option>
</select>

If item 1 or 2 are selected it defaults to item 1.

This appears to have been fixed after the setSelected() function was changed. If not, please link me to a jsfiddle.


[macos] flo-worker not supported on intel arch

Description
flo-worker not provided for intel arch in v0.17.0
Context
flo release v0.17.0 - latest

/Applications/w3champions.app/Contents/Resources/app.asar.unpacked/flo-worker: Mach-O 64-bit executable arm64
flo-worker built for arm architecture supporting apple silicon/m series are working.

flo release v0.16.1

flo-worker: Mach-O 64-bit executable x86_64
built for intel architecture

Possible Implementations
⏩ add flo-worker for intel arch

compile flo-worker for intel arch (some options shown below)
https://github.com/w3champions/launcher/blob/82f6d0190de00976072120e99cfdabc43b812e74/src/background-thread/flo/flo-utils.service.ts#L27
https://github.com/w3champions/launcher/blob/82f6d0190de00976072120e99cfdabc43b812e74/vue.config.js#L20-L29

⏩ Revert back to x86_64 (arm based macs can support this with Rosetta 2).

change github-actions runner label from macos-latest to macos-15-large or macos-14-large - ref
https://github.com/w3champions/flo/blob/8323f8d59c9e09d819cf9ee84e3b4763447dbaf4/.github/workflows/release.yml#L13

currently,

arm: macos-latest --> macOS 14 Arm64
intel: macos-15-large --> macOS 15 beta && macos-14-large --> macOS 14

Split cargo build in build and pass --target for x86_64
https://github.com/w3champions/flo/blob/8323f8d59c9e09d819cf9ee84e3b4763447dbaf4/.github/workflows/release.yml#L45-L50
cargo build --target

assumption is that currently "The default is the host architecture" ref provided by ci/cd runner.

cargo config update

Reverted mac runner to mac-13 to go back to x86_64


Implement additional hooks

Extend the set of hooks by e.g.

[x] terminate when training is overfitted (this might be a base hook from which additional hooks will be derived (one e.g. terminates))
[ ] terminate when some of the statistics is NaN

It would be nice to a have a EveryNEpochs hook, which would be a base hook. The purpose of this hook is to provide an API which allows its descendants to run some actions every e.g. N epochs. Some hooks already does this, to they should be reimplemented.

[x] EveryNEpochs

@gnovis this issue is a top priority, please implement at least basic overfitting detection. Thanks

Assigning @petrbel as @gnovis is not available for a while. Implement it including tests please. Thanks.


Can you generate dependency to gradle implementation ?

Hi! It's not available because of jCenter(), it has been deprecated.
But you can use jitpack:
implementation 'com.github.iRYO400:sliding-panel:1.0.0'

Later will upload it to Maven

Ok I implementation 'com.github.iRYO400:sliding-panel:1.0.0'
but not found
com.akbolatss.workshop.slidingpanel.SlidingPanelLayout
in my layout
<com.akbolatss.workshop.slidingpanel.SlidingPanelLayout
xmlns:android="http://schemas.android.com/apk/res/android"
xmlns:app="http://schemas.android.com/apk/res-auto"
android:id="@+id/sliding_panel"
android:layout_width="match_parent"
android:layout_height="match_parent"
app:orientation="vertical"
app:nonSlidingView="@id/non_sliding_view"
app:slidingView="@id/sliding_view"
app:slidingState="expanded"
app:elevation="4dp" >
and i can't use slidinsgState

Okay, I got you...

Don't know why, jitpack version implementation 'com.github.iRYO400:sliding-panel:1.3.1' doesn't work right now, mby it need some time before being cached, but for now you can use directly .aar from release section
https://github.com/iRYO400/sliding-panel/releases

I call in onCreate() in activity
slidingPanel.slideTo(PanelState.EXPANDED);
but it error:
java.lang.RuntimeException: Unable to start activity ComponentInfo{pl.novitus.bill/pl.novitus.bill.ui.sale.SaleActivity}: kotlin.UninitializedPropertyAccessException: lateinit property slidingView has not been initialized
at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2666)
at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2727)
at android.app.ActivityThread.-wrap12(ActivityThread.java)
at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1478)

you can define default state in the xml, instead of calling it before it's drawn

As stated before library is available via jitpack.io
implementation 'com.github.iRYO400:sliding-panel:1.3.1'


Include version information in CMake and the program

This PR adds version information to the generated CMake files and also a new header that contains the version string (as three parts). This allows software using the CMake find mechanism to know the version of the library and ensure it has the correct version. The header also allows the consuming programs to find the version of the library they are linked against.
An additional change made in this PR is to ensure the CMake find files are installed in the same directory as the library (e.g. lib64 if desired).
Fixes #32

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

Pull Request Test Coverage Report for Build 164

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 100.0%

Totals

Change from base Build 162:
0.0%

Covered Lines:
87

Relevant Lines:
87

💛  - Coveralls

Ping @goulart-paul or @bstellato. Either of you free to take a look at this?

Apologies for delay.   It seems good to me.


feat: Remove support for Node 12

New Pull Request Checklist

[x] I am not disclosing a vulnerability.
[x] I am creating this PR in reference to an issue.

Issue Description

Remove Node 12 support
Upgrade semantic release dependencies
Add backmerge

🎉 This pull request has been released in version 3.0.0-alpha.1

🎉 This pull request has been released in version 3.0.0-beta.1

🎉 This pull request has been released in version 3.0.0


cache.writeFragment does not rerender component

When updating cached data with cache write fragment
I dont see any component using that data beeing refreshed.
I can see the records beeing updated using console.log(cache)
Is this intended?

If at the same update i execute along with writeFragment an updateQuery (In an irrelevant query) then everything works as expected.

Hiya, can you give us more details on this please like a reproduction or some code snippets. Also have you checked your console for any warnings from Graphcache?

Hi
No warnings / errors on console.
See below some pseudo code for what i am trying to achive.
When update balance is pressed the new users balances is not shown. Even though there is an update function in graphcache
Also as said on the previous message the record i updated (I inspect it via console.log(Cache) after upgrade fragment)
const BalancesFragment = gql`
  fragment BalancesFragment on User {
    balances {
      currency
      value
    }
  }
`;

const UserQuery = gql`
  query User($userId: ID!) {
    user(id: $userId) {
      id
      name
      ...BalancesFragment
    }
  }
`;

const UpdateBalance = gql`
  mutation SendBalance($input: SendBalanceInput!) {
    sendBalance(input: $input) {
      userId
      balances {
        currency
        value
      }
    }
  }
`;

export const User = ({ userId }) => {
  const [{ data }] = useQuery({ query: UserQuery, variables: { userId } });
  const [, updateBalances] = useMutation(UpdateBalance);
  return (
    <>

{data.name}

      {data.balances.map(balance => (
        {`${balance.currency}:${balance.value}`}
      ))}
      <button
        onClick={() =>
          updateBalances({
            input: { userId, balances: [{ currency: "EUR", value: 10 }] }
          })
        }
      >
        Update balance
      </button>
    </>
  );
};

const setupUrql = () => {
  return createClient({
    url: "A url",
    exchanges: [
      dedupExchange,
      cacheExchange({
        updates: {
          Mutation: {
            sendBalance: (result, args, cache) => {
              const {
                sendBalance: { userId, balances }
              } = result;
              cache.writeFragment(BalancesFragment, {
                id: userId,
                __typename: "User",
                balances
              });
            }
          }
        }
      }),
      fetchExchange
    ]
  });
};

Forgot something.Dont know if it makes much difference
      cacheExchange({
        keys: {
          Balance: () => null,
        }
      })

That’s indeed a little odd. I’ll have to investigate this further. This might be happening due to how formatting is being skipped for fragments right now, but I’m not quite sure

That’s indeed a little odd. I’ll have to investigate this further. This might be happening due to how formatting is being skipped for fragments right now, but I’m not quite sure

One more thing as said earlier if at the same update any other update is made using updateQuery then all is fine

Before I jump onto investigating this, can you maybe give me a little pointer please? I’m curious whether this only affects the new release of<EMAIL_ADDRESS>or 2.1.1 as well

Both

@zenios I've written a test for this now and I can't reproduce this behaviour unfortunately. It's a slightly simplified case, but covers the exact same methods and code paths that you're describing. Feel free to take a look and modify it :+1: https://github.com/FormidableLabs/urql/pull/578
I can totally see this failing if the data doesn't match what you're expecting, so maybe it's worth double checking whether this isn't something on your data first. But I'd at least expect an Invalid key warning, since I can't reproduce a totally failing behaviour yet.

The major difference from my example is that in my case i am updating an
array while in your case its only a single element.
I will try to create a reproducer and let you know
On Sat, 7 Mar 2020, 10:59 Phil Plückthun<EMAIL_ADDRESS>wrote:

@zenios https://github.com/zenios I've written a test for this now and
I can't reproduce this behaviour unfortunately. It's a slightly simplified
case, but covers the exact same methods and code paths that you're
describing. Feel free to take a look and modify it 👍 #578
https://github.com/FormidableLabs/urql/pull/578
I can totally see this failing if the data doesn't match what you're
expecting, so maybe it's worth double checking whether this isn't something
on your data first. But I'd at least expect an Invalid key warning, since
I can't reproduce a totally failing behaviour yet.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/FormidableLabs/urql/issues/574?email_source=notifications&email_token=AAAV3YJVK6FYWWYPUDLQFETRGIEI5A5CNFSM4LDH5TU2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEODTPVQ#issuecomment-596064214,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAV3YIAX6WZTTNWDJJYMKDRGIEI5ANCNFSM4LDH5TUQ
.

@zenios in theory that won’t matter, since either way we’re getting to the Balance array via a User. We keep track of updates by “dependencies” these are a list of entity keys that a query has read or a result has written. So assuming that we should see that:

the initial user query has a dependency on the user, e.g. dependencies = Set { ‘User:xyz’, ... }
the balances will be embedded but they’ll take on a key that looks something like User:xyz.balances...
the mutation writes to the user, so User:xyz will update (it doesn’t matter if this is automatic or manual via writeFragment or another method)
this causes the original query which depended on the same user to update. We just need some overlap of keys, so the balances wouldn’t even matter in this example

Hope this helps 👍 I’m sure we can figure out what’s going on here. Even if this is expected behaviour, maybe there’s a new useful warning we could add as a result of this. Or maybe we do find this to be a bug 🐛😅 Either way, it’s still a little fuzzy for me

So i went down the rabit hole trying to find what is happening.
I am still investigation but i find something that looks rather odd. Actually its a race condition
The race condition is here
https://github.com/FormidableLabs/urql/blob/cc372aed9fe1cc2036a540e37ac78f65dfe6ed8e/exchanges/graphcache/src/operations/write.ts#L63
Between startWrite and clearDataState
I found out that if two queries are executed such as
useQuery({query:QUERY1);
useQuery({query:QUERY2);

There is a possibility that the dependencies calculated of one query will include dependencies of the second query.
I am not sure if this is intended behaviour.

That’s highly odd! This is all synchronous.
But this may be expected due to our new commutative reordering. The problem is, reordering lead to more dependencies on one query, but that would lead to a potentially additional update, which we think is fine, it should never lead to a missing update 😅

Actually its not all synchronous. Inside clearDataState you have two defer calls.
Odd or not i can see it
In some cases write dependencies are
query 1 = 8
query 2 = 5
And other cases
query 2 = 5
query 1 = 13

The more weired think is that writeFragment only works in cases that one query leaks write dependencies to the second query :)

Yea, the garbage collection is deferred and persistence (which isn’t on by default) is deferred. Either of them don’t operate on dependencies however, and dependencies are kept synchronously. initDataState provides them, then synchronous reads/writes executes, then clearDataState flushes all states and unsets all dependencies.
It’d be really beneficial if you could isolate your behaviour to the tests. But what I’m thinking is, if you’re seeing different dependencies (and you didn’t post the full keys or context here?) then you’re likely also getting different data 😅
So the signs do still point to some kind of usage issue unfortunately 😅🙏

One more.
The reason that the element is not updated after writeFragment is because reflock for that object is never released.
Now i need to find why reflock is increased more than what it should

In theory that shouldn’t be the cause either. refLock only prevents GC from deleting an entity when something else is still pointing to it 😅 nothing else. It doesn’t affect the data itself. You can comment out the GC logic and no behaviour will change

Grrrrrrr :)

I’m sorry to see you having a bad time ♥😅 let’s maybe take a step back? If you’re using the dev tools or debug exchange you should be able to see whether after the mutation you see your operation being reexecuted. That’d be a good sanity check.
If it is, then somehow your query is being reexecuted but isn’t seeing the new data.
If it isn’t, then we must have non-overlapping dependencies is what I’d assume. So maybe we could double check that we are in fact writing the right fragment. If you’re not seeing a warning then we must assume you are (unless the id isn’t correct). So at that point we’d have to explore deeper again

The query is beeing reexecuted but it returns the previous data (Before write fragment)
I can see in the cache that
cache.records.base contains the updated object (After write fragment)
but
cache.records.optimistic contains an entry with the object before write fragment

Ok, that’s a step closer. So we could check what that optimistic key is and which operation it is from (optimistic key is the same as the operation key)
But I suspect it’s from a query layer? In that case though the behaviour shouldn’t actually occur in 2.1.1; also this can only happen if the mutation completes before all queries with that same entity do... which is already a tricky edge case in theory, since it’s hard to do the right thing in this case, due to a preexisting race condition

Optimistic is from the original query (cache-and-network) which is the same one i am updating using writeFragment

I have a reproduction :)
https://codesandbox.io/s/focused-night-ztnmm

As a side note.
updateQuery behaves exactly just like writeFragment
i.e No updates

This actually did turn out to be a regression in<EMAIL_ADDRESS>which didn't exist in v2.1.1 since we introduced commutativity. There were some cases where this didn't work as intended, which I suppose was to be expected, since it was a rather new feature. We're now planning to change its behaviour a little again soon, but for now we've fixed the bugs :tada:
The fixes are included in v2.2.2.


Not able to install modelbuilder

Problem encountered on https://dotnet.microsoft.com/learn/ml-dotnet/get-started-tutorial/install
Operating System: windows
Provide details about the problem you're experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.

Hi, can you provide more information? like which version of Visual Studio do you have, and what error do you get

Close due to no response, feel free to reopen it if you still have question though


[core] Speed-up simulation by avoid unecessary computations

Add option to disable extra computations
Add option to update less frequently the formulation of the constrained problem (the pose of the contact points, the set of active constraints, and the Jacobian of the constraints) because it is costly to evaluate and should be slowly varying.

Add option to disable extra computations

The speed up was very marginal (less than 2%), so I think it is not worth adding an extra option for this.


NuGet Error NU1101 Generated by Telerik UI for Blazor

I cannot get rid of the error mainly because I work on Government computer and cannot install the Progress Control Panel with FIPS enabled. It is not possible to disable FIPS on Government computer. In addition, the Automated Wizzard Installer didn't solve the problem and neither the local NuGet feed setup. Now my manager decided not to use Telerik UI, but I cannot get rid of the error even when I uninstalled the extension from Windows 10 and reinstalled VS 2022. Perhaps I could delete the only Windows registry entry that refers to Telerik, but to mess with Windows registry on Government computer is a very last resort.
Now the error is hanging in the system and I am not able even run a Blazor sample project provided by Visual Studio 2022.
I am curious what would be the solution for this?

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

ID: c334f18d-66ca-28a9-3350-c20e762bbec1
Version Independent ID: 2ea96dfa-e016-ae3b-864c-2a1ce513d90e
Content: NuGet Error NU1101
Content Source: docs/reference/errors-and-warnings/NU1101.md
Product: nuget
GitHub Login: @zhili1208
Microsoft Alias: lzhi

@prager I don't understand the relevance of FIPS.  Are your logs showing HTTPS connection failures?  NU1101, which the docs page says means that the package can't be fond on any configured source, indicates that it didn't fail to contact a package source. In any case, I'd be shocked if any cipher needed for HTTPS TLS is not supposed with FIPS mode on.
NuGet only tries to install packages (and their dependencies) that are defined in MSBuild. I'm not aware of any MSBuild feature that can cause PackageReference items to be added via the registry.
The .NET SDK does however automatically includes the first Directory.Build.props file found in any parent directory. So if you have one of thise in your root directory, for example, that adds a PackageReference to the Telerik package, that would be why your new project is trying to install it.

Thank you for your very much for your advice.
The bottom line is that Progress Control Panel will not install on Government computer with FIPS enabled. The installer throws a specific error message in the sense "...to continue with installation you must disable FIPS...". It is not possible to disable FIPS on a Government computer. It is interesting that the the previous version of Telerik UI works on other Government computers in Blazor project.
Unfortunately, I can't provide you with specifics what our troubleshooting included and because it took too much time, we decided to clean my machine of any reference to Telerik UI and with reinstall of Visual Studio the NU1101 error is finally gone.
And that means that at this time I will stay away from Telerik UI package for Blazor in near future and if such a time comes, I will revisit your advice.
Thanks again!

I don't know what "Progress Control Panel" is. Assuming it's related to Telerik's products, you can report it to them, but as you experienced when Telerik's tools are removed from your computer, NuGet works with FIPS enabled.
My summary is that a package was being added to your project, and you didn't understand why or where it came from. We'll consider what possible product changes we might be able to make to make this easier, but since PackageReference is built on MSBuild, it may be more of an MSBuild topic, rather than NuGet.
Since this issue is reported in the docs issue, I'm going to close it, because I don't believe that there's any actionable docs improvements.


No validation error on parsing wrong month

Description:
Parsing of cron-expression like 0 0 0 ? 13 6 doesn't throw a validation error
Current behavior:
There is no error on parsing\validating cron-expression with month value greater than 12
Expected result:
Parsing\validating of cron-expression with month value greater than 12 throws an error.

Hi @IlyaKrenev Thanks for your review and your PR has been merged. I will publish a new version later.

<EMAIL_ADDRESS>


AVS Internet Connectivity "ADD" Public IP failing.

See screen capture for the failure.

This worked fine around a month ago in our tests, ended up deleting it since at the time there was not a need for the Public IPs.  Now we would like this for our VPN NST-T testing along with HCX.

Opened a MS Support Case:
Incident title:	AVS Internet Connectivity 'ADD' Public IP failing
Support request number:<PHONE_NUMBER>015361
Severity rating:	B

Please update the status based on the outcome of the support case.

MS support is saying there is a broken process for "Internet connectivity" on the Azure Portal AVS page ,  they are working on an internal fix.   The workaround is using the CLI method.
https://learn.microsoft.com/en-us/cli/azure/vmware/workload-network/public-ip?view=azure-cli-latest
which did work for us.

Thanks for sharing, and great that you are unblocked using the workaround.


WebSockets client ip address

Hello. This is a very nice project although i have one question: how can i get a websockets client ip? Thank you.

Hello this is not possible right now, will probably lie in req.getIP() or something like that

Thank you for your answer. How soon do you plan to add this feature?

In what way do you need to know the IP?

Fow what purpose*

For*

For access control managment

So you want to filter certain IP addresses solely based on IP alone?

I want to control the access from ip address another usage is geo location by ip address

I use the IP to count unique users connected to server, instead of counting sockets, which could be many per IP.

Like I said this will be added

I want to control the access from ip address + another usage is geo location by ip address

You'll mess up when two users come to your server through one NAT: they will have same IP. And from my experience, mobile internet providers sometimes use NAT, so even on mobile devices you can't be sure that one IP is definitely linked one user.
My advice is to generate unique hash cookie or localstorage key and use it for combining multiple socket connections of one user.

Hey @alexhultman I've been using this package for 3 years now.  Love your work in general but I really like what you have done here with the new node bindings.  This issue is a blocker for me getting off the old npm package.  I have a branch ready to go as soon as this is closed!  Thanks for your work!

Great project! I would also be interested in request IP addresses. We often deploy web applications on LANs with static IPs within defined IP ranges. Knowing the IP address of a request allows us to make assumptions about where and who the client is, and we often surface the information to administrators.

console.log(Buffer.from(res.getIP()).toString('hex'));
00000000000000000000000000000001 = ipv6 localhost
00000000000000000000ffffc0a80086 = <IP_ADDRESS> mapped as ipv6
res.getIP() should be req.getIP so that websockets can do this while in open

This will be ws.getRemoteAddress() and res.getRemoteAddress()

not req.getRemoteAddress()?

Throught about it but it makes no sense. IP is a property of the socket, not any one particular http request. Both ws and res are derivatives of a socket while req is a stack allocated temporary thing only holding http-related parsed out properties.

Please try latest build,
npm install uNetworking/uWebSockets.js#binaries

Hello,
I have tested the latest #binaries build and ws.getRemoteAddress() returns an ArrayBuffer(16). I guess you output an IPv6 address.
I have tested it with a websocket connected to my server both are on localhost and here is the output:
console.log('RemoteAddress: ' + new Uint16Array(ws.getRemoteAddress()));
> RemoteAddress: 0,0,0,0,0,0,0,256

I was expecting <IP_ADDRESS> in IPv6 format which is 0:0:0:0:0:ffff:7f00:1 as follows:
console.log('ipv6Localhost: ' + new Uint16Array([0, 0, 0, 0, 0, 0xffff, 0x7f00, 1]));
> ipv6Localhost: 0,0,0,0,0,65535,32512,1

Am I doing something wrong or is ws.getRemoteAddress() not implemented yet?

It's fairly uncommon not to have a load balancer, or reverse proxy for other reasons, in between the end-user client and the services (potentially using this lib). So I think it's rather prudent to consider req  the natural go-to place for sourcing it (refer back to #93), since the (real) client's IP would have to be derived from a header that naturally must have precedence over the socket-IP — which very likely just might be localhost, but if there are no such headers, it might be because one has it exposed raw to the public (for instance allowing client who are not blocked by internet cafe proxies where they use wifi, to connect to a non "standard" port, whilst the others have to go through the load balancer. You catch my drift. Common deployment scenarios.

You're mixing different topics. This feature is regarding getting the IP address, not whether any one header value might or might not be more relevant.

My bad.

For those reading this wondering how to get the "real" IP from the headers due to uWS being behind a reverse proxy like what ozra suggests, you simply do this:
open: (ws, req) => {
  console.log('X-Real-IP: ' + req.getHeader('x-real-ip'));
  // or...
  console.log('X-Forwarded-For: ' + req.getHeader('x-forwarded-for'));
}

@PeterHewat it is working, you're messing up with the uint16array. It's easy to see with uint8array:
Uint8Array [ 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 255, 255, 127, 0, 0, 1 ] = <IP_ADDRESS>
Uint8Array [ 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1 ] = ::1
Uint8Array [ 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1 ]

For anyone else looking for how to get the IPv6 string from the res.getRemoteAddress() function, here is what you can do:
let ip='';(new Uint8Array(res.getRemoteAddress())).forEach((a,i)=>ip+=a.toString(16)+(i%2&&i!=15?':':''))
ip will come out in this format: "00:00:00:00:00:ffff:244b:8cca"

You can also just do buffer.toString('hex') it's really not any different from any other binary string

Is there a way to get the ipv4 address from getRemoteAddress?

getRemoteAddress returns either 4 (ipv4) or 16 (ipv6) bytes. That is the address. You may want to format it in human readable ways like mentioned above.

@alexhultman how would you do it? Would it be difficult to add it to the library or should people write own function for it?

 function remoteAddressToString(address) {
        if (address.byteLength == 4) {//IPv4
            return new Uint8Array(address).join('.');
        } else if (address.byteLength == 16) {//IPv6
            return Array.from(new Uint8Array(address)).map(v => v.toString(16)).join(':').replace(/((^|:)(0(:|$))+)/, '::');
        }
    }

with inspiration from ipaddr.js

 function remoteAddressToString(address) {
        if (address.byteLength == 4) {//IPv4
            return new Uint8Array(address).join('.');
        } else if (address.byteLength == 16) {//IPv6
            return Array.from(new Uint16Array(address)).map(v => v.toString(16)).join(':').replace(/((^|:)(0(:|$))+)/, '::');
        }
    }

with inspiration from ipaddr.js
If you need to convert IPv4 addresses mapped to IPv6...
(I'm not sure whether this is my computer, but although netstat shows IPv4 connections, I get a mapped IPv6 address with getRemoteAddress())
function remoteAddressToString(address) {
        if (address.byteLength == 4) {//IPv4
            return new Uint8Array(address).join('.');
        } else if (address.byteLength == 16) {//IPv6
            let arr = Array.from(new Uint16Array(address));
            if (arr[0] == 0 && arr[1] == 0 && arr[2] == 0 && arr[3] == 0 && arr[4] == 0 && arr[5] == 0xffff)  //IPv4 mapped to IPv6
                return new Uint8Array(address.slice(12)).join('.');
            else
                return Array.from(new Uint16Array(address)).map(v => v.toString(16)).join(':').replace(/((^|:)(0(:|$))+)/, '::');
        }
    }

It's helpful for me.

function remoteAddressToString(address) {
        if (address.byteLength == 4) {//IPv4
            return new Uint8Array(address).join('.');
        } else if (address.byteLength == 16) {//IPv6
            let arr = Array.from(new Uint16Array(address));
            if (arr[0] == 0 && arr[1] == 0 && arr[2] == 0 && arr[3] == 0 && arr[4] == 0 && arr[5] == 0xffff)  //IPv4 mapped to IPv6
                return new Uint8Array(address.slice(12)).join('.');
            else
                return Array.from(new Uint16Array(address)).map(v => v.toString(16)).join(':').replace(/((^|:)(0(:|$))+)/, '::');
        }
    }

Even though I was connecting to my server with IPv4, ws.getRemoteAddress() was returning 16 bytes instead of 4. This function converts it nicely, thanks.

In 18.1 you'll have getProxiedRemoteAddressAsText() and getRemoteAddressAsText().
Proxied follows the proxy protocol v2

@alexhultman-2fa How can we get the proxied IP in open handler with 18.4?
getProxiedRemoteAddressAsText() is not available on Websocket, and request/response objects are not available either


Add missing instructions

Yeah I'm looking at it. I'm not sure. I'm wondering if it refuses to forward the codacity secret because it's the first time you contribute. Since there is hopefuly no way you can do malicious things with the README, I'll just merge it :)


Fix interface and proto definition

What this PR does / why we need it:
Which issue(s) this PR fixes:
ref #3442, #3449
Does this PR introduce a user-facing change?:

NONE

Thank you!
I fixed. PTAL.

Thanks!
/lgtm

Here you go
/approve


OHRI Technical Roadmap

Feature Work

[ ] Form Engine (Parity with Angular form engine)

[x] Support rendering type "drug"
[x] Ability to mark a form field as transient.
[x] Support injecting of external data sources.
[x] Support registration of custom expressions.
[ ] Support for creation of reusable components.
[ ] Support for rendering problem by concept class.
[x] Support for file attachment rendering.
[ ] Concept location support.
[ ] Adjust the rest-web-services module to allow filtering of voided forms.
[ ] Form submission should use the form_uuid and encounter_uuid fetched from the backend.
[ ] Support for Z function as part of inbuilt Expression helpers.
[ ] Support translation/localisation (i18n) in labels.
[ ] Support for Drug Orders.

[ ] Form Builder

[x] Display of Audit information on the form editor.
[ ] Support for referencing reusable components.

[ ] Mamba ETL

[ ] Move MambaETL DB flattening Task scheduler to base module.
[ ] Support for flattening concepts rendered as both questions and answers.
[x] A configurable toggle between storing ETL output in the OpenMRS database or in External database.
[ ] Ensure that reports data is available while ETL is running.
[ ] Incremental update of ETL based on underlying data changes.
[ ] Automate addition of Make Scripts in MambaETL.
[ ] Service/API layer data pagination support in MambaETL.
[ ] Automate column naming in the ETL flattened Tables.
[ ] ObsGroups support for MambaETL.
[ ] Publish Version 2.0 of MambaETL.

Bugs

[ ] Mamba ETL

[ ] Fix build failure when reports.json configuration file is missing.
[x] Concepts with multi-select values (e.g. key population) failing to flatten.

[ ] Form Engine

[x] Previous values are picked from old encounters.

House Keeping

[ ] Form Engine

[ ] Update Form Engine Documentation.
[ ] Comparison of validation in RFE against AFE.

[ ] Mamba ETL

[ ] Test report on MambaETL running on MySQL 8.
[ ] MambaETL documentation - v1.0.0.
[ ] Performance improvement during the flattening process.
[ ] Robust Error Handling in MambaETL.
[ ] Code cleanups and Refactors in MambaETL.

Moved to respective repositories


Generics with new keyword

Flow version: 0.98.1
Expected behavior
No flow errors
Actual behavior
Cannot call this.Ctor because property constructor is missing in CacheValue

Try-Flow

https://flow.org/try/#0MYGwhgzhAEDCIEsCmA7ALgHlmYALJAamCAK5IBc0A3tMAPYoRoBOJwadzAFAJSUBudBABNoAXwB81AFDQ4HZpXiQIWHPiKkkE2dF31GLNgq79iZZVCXgoavIXPaeMua+hpcCCADpYC6AC80GZalhC6YtK6AEZg3Hxw6g5aLtAAkMxIaCTMKNAoSADu7p4+fpy8EdJiQA works


move almost all functionality to lib

The only thing main should do is handle HTTP requests and call functions for them. A lot of this is inline right now.

Steps for this:

move almost all functionality to lib
Move all http stuff to a sub-crate
main should compose the two

I think I've done this enough to close this ticket; if anyone else has suggestions, please send them in!


Fix instruction inheritance issues

Fixes #1892
Summary
Modified the way in which type-checking is done in instruction.py to allow QuantumRegister and ClassicalRegister to be inherited from.
Details and comments
Type-equality checking is brittle and should be avoided except in the case where inheritance is to be disregarded for a class. That appears to be the case for neither QuantumRegister nor ClassicalRegister.

This PR was made against the stable branch, and I didn't notice that when I merged. It is now reverted. Please reopen the PR against master.


DDS_zeroshot.ipynb - sds loss derivation

i liked your DDS paper a lot, thanks for sharing your code!
It would be really great if you could explain the math/derivation of the SDS loss you're using;
specifically, the following two lines:
grad_z = alpha_t * sigma_t  * (e_t - eps)
sds_loss = grad_z.clone() * z
or link any relevant references.
Thx a lot!

I have the same question
grad_z = (alpha_t ** self.alpha_exp) * (sigma_t ** self.sigma_exp) * (e_t - eps)
sds_loss = grad_z.clone() * z

you can refer to this: https://github.com/threestudio-project/threestudio/issues/266.
\delta sds_loss / \delta z will direcly be the grad_z (e_t - eps), which can avoid Unet backward required for Chain Rule.


ARROW-3672 & ARROW-3673: [Go] add support for time32 and time64 array

Hello everyone,
My attempt at adding support for time32 and time64 array. Need review because I'm not sure I added all that is needed.
Thanks,
Alexandre

Codecov Report

Merging #2944 into master will decrease coverage by 24.18%.
The diff coverage is 0%.

@@             Coverage Diff             @@
##           master    #2944       +/-   ##
===========================================
- Coverage   86.42%   62.23%   -24.19%
===========================================
  Files         490       56      -434
  Lines       68838     3612    -65226
===========================================
- Hits        59491     2248    -57243
+ Misses       9257     1274     -7983
  Partials       90       90

Impacted Files
Coverage Δ

go/arrow/type_traits_numeric.gen.go
15% <0%> (-2.73%)
:arrow_down:

go/arrow/datatype_fixedwidth.go
15.38% <0%> (-13.19%)
:arrow_down:

go/arrow/array/numericbuilder.gen.go
50.05% <0%> (-9.11%)
:arrow_down:

go/arrow/array/numeric.gen.go
38.73% <0%> (-7.05%)
:arrow_down:

go/arrow/array/array.go
93.1% <0%> (-3.33%)
:arrow_down:

python/pyarrow/ipc.pxi

cpp/src/parquet/column_page.h

cpp/src/parquet/bloom_filter-test.cc

cpp/src/plasma/client.cc

cpp/src/arrow/io/test-common.h

... and 429 more

Continue to review full report at Codecov.

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

Yes, will do that.
What do you think about templating all the tests for numeric arrays/builders?

Still need tests for arrays. I'm working on it and it should be ready soon.

@sbinet looks good to me, too. Thanks for the contribution, @alexandreyc!

Should be good now! Fixed a little typo which caused time64 array tests to fail.

I noticed you are using unicode "mu" for microseconds here https://github.com/apache/arrow/commit/df54da211448b5202aa08ed2b245eb78cfd1e50c. Is that really what you want?

Yes, it's one character like other fractions and translate well to microseconds. I'm open to renaming this if needed.

Some other projects use "us" instead of "μs". I had to Google the "mu" unicode character and copy-and-paste it because I'm on Linux and don't know how to type such characters from an English keyboard layout

Switched to "us".


Support of React Native Web ?

Thank you for this awesome lib that really works :)
The main readme doesn't mention the support of react-native-web .
is this planned in the futur ?

For web you can use the https://github.com/douglasjunior/react-recaptcha-that-works
It's a very similar API, but has some little differences.
Sou you can create two Recaptcha components in yout app, like:
my/components/path/Recaptcha.native.js
my/components/path/Recaptcha.web.js


Please enable discussions

Hi @Blimster
Are there any tests around this?  Or some way I can verify it?  The reason for asking, I've started work on a concept which could potentially be used as a replacement for avro.  Still pretty rough, but I need kafka support from dart.  Originally, I was thinking of interop w/ kafkajs, but this would be way better I think.
In addition, for automated testing while working on this feature, the lib could be really useful.
https://github.com/vectorizedio/redpanda/issues/1667#issuecomment-877130993
Oh, also, if possible, please enable discussions on this repo and babylon_dart.  This will keep me from polluting issues w/ non-feature related chatter.
Thanks!

Discussions feature is enabled now. I enabled it for babylon_dart already last week. I will close this issue. Please, post your topic as a discussion again.


Scaling profiling datasets

Description
This PR restructures the profiling code to do a number of things:

Move the data payload fixtures into a profiling/conftest.py
Separate the pmodel and subdaily profile workflows into separate files
Add custom command line options to pytest to allow the scaling of the payload data to be controlled from the command line. The subdaily and pmodel tests share the same data and scaling factor option.
Change the default scaling to something that ought not to break the GitHub runners (max of 16GB RAM).

These changes make the profiling more flexible and helps set up for looking at the scaling of functions with input sizes. It allows commands like the following:
# Look at the peak memory usage,  run all the profiling tests, using an 8 fold scaling of the inputs
/usr/bin/time -l pytest -m "profiling" --pmodel-profile-scaleup 8
# Look at the peak memory usage,  running the splash profile, with an 100 fold scaling of the inputs
/usr/bin/time -l pytest -m "profiling" tests/profiling/test_profiling_splash.py --splash-profile-scaleup 100

The peak memory size of running the full test set is basically the combined size of each test and is very roughly:
50Mb * splash-profile-scaleup +  1Gb * pmodel-profile-scaleup
The defaults --splash-profile-scaleup=125  and --pmodel-profile-scaleup=6 should give about a 6GB size for each dataset. If we could use a single dataset, we'd be able to use bigger loads....
Fixes #215
Type of change

[x] New feature (non-breaking change which adds functionality)
[ ] Optimization (back-end change that speeds up the code)
[x] Bug fix (non-breaking change which fixes an issue)

Key checklist

[x] Make sure you've run the pre-commit checks: $ pre-commit run -a
[x] All tests pass: $ poetry run pytest

Further checks

[x] Code is commented, particularly in hard-to-understand areas
[ ] Tests added that prove fix is effective or that feature works

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

Project coverage is 95.17%. Comparing base (7e9d326) to head (ed90304).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #216   +/-   ##
========================================
  Coverage    95.17%   95.17%
========================================
  Files           28       28
  Lines         1701     1701
========================================
  Hits          1619     1619
  Misses          82       82

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

Perhaps gc.collect() can be used between profiling pmodel and splash so that the peak memory size would presumably be the larger size of pmodel and splash profiling datasets instead of the sum of their sizes.

LGTM


Gardening tab issue

Any gardening timer set is deleted when changing tab or leaving the app.
Android 10
Latest Resin Timer app update

Also, I just noticed that all my furnishing building list was gone. And only 1 timer is saved, the others added get deleted after changing tab or leaving the app.
Otherwise I wanted to say thank you for this awesome app !

Thanks to report bug issue. I will fix that on next update

Both problems have been fixed with today's update !
Thank you so much for this awesome app !


Feature Request -- Scheduled Download

I have a feature request.  Can a download scheduler be included?  My internet service has a soft cap during daytime hours.  i have unlimited download capabilities from 11pm to 7am.  It would be very helpful to be able to schedule downloads to start during that time.

I've solved the problem myself.  I added my code below for anyone who is wanting the same feature.  You can edit it to your needs.  Just change the time as needed.  It's in military time.  Changes start at line 1442.  Basically added and if else statement.
The important part that allows it to delay downloads until a specific time is this.  After making this change I can now set up a download before going to bed and it will start at 11pm and run until completed.
	         # this delays downloads until after 11pm
             $time = Get-Date '23:00'
             $now = Get-Date

		if ($now -le $time)
        {
			$label_DLTitle.Text = "Waiting until 11pm . . ."
			(New-TimeSpan -End $time).TotalSeconds | Sleep;
....existing download code goes here
       }
else {exiting download code}

Saverr-dl-later2.txt

Thanks for leaving the code for anyone else who wanted something like this. Glad you figured out how to do it.


Update README.md

I would specify that the links are only for the C1 as there is no need to use ARM images on the C2/VC1

Hi also think we need to refactor the links, we will create a dedicated issue so we can merge this quickly

see #60


First commit of the NormalHeightMap command line tool.

With it you can:

pack two separated normal and height map files into a single one
invert the R &| G components of a normal map file to change the normal orientation.

.exe ???
there are Mac and Linux users here also ...
Le 23/02/2016 15:13, nockawa a écrit :

With it you can:

pack two separated normal and height map files into a single one
invert the R &| G components of a normal map file to change the
normal orientation.

    You can view, comment on, or merge this pull request online at:

https://github.com/BabylonJS/Babylon.js/pull/988
    Commit Summary

First commit of the NormalHeightMap command line tool.

The tool uses this library http://imageprocessor.org/ which target only .net40 and .net45.
So I'm not sure for .net core... It would require way more work than I already did to achieve this I think...
Can you elaborate why you would like ideally?

What I meant is more :
We do things for the web, we use OS agnostic languages like JS or TS.
So why not keep on providing tools in this ecosystem (gulp, node, js)
rather than going to .NET, JVM or any other languages ?
Just a question ...
2016-02-23 17:51 GMT+01:00 nockawa<EMAIL_ADDRESS>
The tool uses this library http://imageprocessor.org/ which target only
.net40 and .net45.
So I'm not sure for .net core... It would require way more work than I
already did to achieve this I think...
Can you elaborate why you would like ideally?
—
Reply to this email directly or view it on GitHub
https://github.com/BabylonJS/Babylon.js/pull/988#issuecomment-187786603.

Well, I'm pretty new here, so when I looked at the Tools directory I saw there were already few .net projects and I figured I could contribute with the small one I just wrote yesterday and it was ok.
Now the real question behind this (imho) is: do we have to be platform agnostic for the production pipeline?
I think this question is legitimate and depending on the answer some guidelines could be setup to give the devs clear directions of how things are supposed to be made.
@jbousquie : this tool should have been a node.js  module, ideally for you? Considering it's supposed to be used by the graphists guys.

Interesting debate :) We should as much as possible be platform agnostic but it also depends on everyone competencies and timing.

my question would be (even after merging :) ) if it doesn't belong in the extensions repository

Good question. We have tools about .Babylon file for instance here. Perhaps you're right actually
@nockawa : do you mind moving your tool to /extensions here:https://github.com/BabylonJS/Extensions?

@deltakosh not at all... :)
I'll do it asap, won't be long if Git is friendly with me...so it may takes some time...

Ok, I've made the PR on the Extensions project. Who should take care of deleting the tool from this repo?

:+1:
I'm on both

Thank you !

Thank you !


XKAS -> ARMIPS

I know you've got a lot of stuff already working with xkas but as your aware that's pretty much dead and unavailable these days.
Would you consider moving to something like ARMIPS which is open source and supports all the features of xkas and also has built in support for table files and a lot of other useful features.

I'll definitely take a look -- as helpful as xkas was for me, it has some weird quirks. Thanks for letting me know about ARMIPS!

Looks like it's better than xkas for what I need, so I've gone ahead and migrated everything over. It all seems to work on my end so far so I'll close this.


Show Authenticated Scan Message in vet Initiatialization

Feedback from user:

When I configure the API for the cloud if the key is valid it doesn't printout the successfully valided message, while it does when the API key is invalid. Sucess message can help in normal debugging process?

Ensure a successful verification and acknowledge message is shown during vet auth configure flow

Fix for this included in PR #274


focusmonitor keybind does not emit focusedmon event in socket

If you move the mouse to another monitor, you get a focusedmon event in the socket. However, if you use the focusmonitor keybind, it won't emit an event.
Steps to reproduce

bind focusmonitor to a hotkey
use the hotkey to focus another monitor
check socket, there won't be a focusedmon event

I'm using the nix flake build of hyprland at commit d0e47d9fe09dfa3be0c46833bee832ddbe0fcb51.

fixed in 653b9ed0e4933d83b2e3f378cdc76f1ebeb97e51


Async 1

Merging in some optimizations to how we are using find (rather how we are processing find results).
This should highly improve the performance of traversing large directories over ssh (I don't have this implemented in docker, honestly I don't know that its needed as docker is "local").
Anyone who cares, feel free to test this and if it breaks, lemme know! Or don't, I'm not the police.

Oh! Other added benefits, This is now smart enough to convert expected integers to integers :) Hurray for no more string comparisons to numbers :)


Update programmatically-create-subscription.md

The customerId is requested in the 2nd step. The first step is about listing the billing accounts.

@whaakman : Thanks for your contribution! The author(s) have been notified to review your proposed change. @amberb

Hi @bandersmsft - Could you review the proposed change or clarify the author for review? "AmberB" doesn't resolve as Microsoft-internal in GitHub. Thanks.

@amberbhargava @ktoliver amberbhargava is the correct author.


Boarder Background does not match parent background

I am trying to use a TStyleBitBtn over a background. At design time everything looks correct, however at runtime the background boarder of the button is not correct.  See below.
Thanks
--

Please send me a zip with a test project.

Please take a look at the new StyledComponents\Demos\Delphi??\StyledComponentsDemo.dproj and open the StyledTaskDialog Demo: you can use a more beautiful StyledDialog form also with "animations" (enabling SKIA to the project)...

Any feedback?

Fixed in 3.7.0 version


Add copyObject function to S3Client

This PR adds a copyObject function to the S3Client.

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

Hey @bendennis 🙇🏻
So sorry for coming back to you so late, this PR got completely off my radar, and my email notifications.
Thanks a lot for your contribution, I'll take a look now 👀

I'll merge this and we will release a 0.8.1 or 0.9.0 version as soon as possible.

Here is the documentation for the feature: https://github.com/grafana/k6-docs/pull/1273


Exception: database table is locked

Not always.
~ $ history
Exception: database table is locked: cmd
Traceback:
  [interactive], line 1:
    history

dup #263

duplicated with #263


Debugger: [Unstable] Log are missing some message

Environment data
dotnet --info output:
OS: Ubuntu.16.10-x64
VS Code version: 1.8.1
C# Extension version: 1.6.2
.NET Core version: 1.0.0-preview2 - 003177
Steps to reproduce

Enable MIEngine logging ( .VSCode->project.josn )
Download hwapp folder in attachments directly and open with VSCode
hwapp.zip
Update launch.json as following: "logging":{ "engineLogging": true }
Press F5 to start debugging

Expected  behavior
The log are include ' *stopped, reason="entry-point-hit" '
correctlog.txt
Actual behavior
The log are missing ' *stopped, reason="entry-point-hit" '.
errorlog.txt
Investigation
This issue doesn't appear every time. This issue reproduced frequently at first, but as the test times increase, it reproduced less, as 20~30%.

This is the same issue as https://github.com/OmniSharp/omnisharp-vscode/issues/1011

Got it, thanks!


fix proto_sync.py to ensure that the expected diff content is output

When we use proto_format.sh to generate new generated_api_shadow, a lot of meaningless diff content will be generated.
The reason is that when the current API is copied to tmp dir for diff, part of the v2 API will be removed. But the destination API did not remove the corresponding v2 API.
Risk Level: Low.

@phlax Can you help confirm this at your convenience?

@wbpcode i dont think this is required - iiuc this is probably the same problem as here https://github.com/envoyproxy/envoy/issues/16967

Yep. After i cleand cache in bazel-bin, it is work ok. Thanks.


[BUG] Wrong address for last page of Flash?

It seems that the address used for ADDRESS_LAST_PAGE_FLASH is not suited for STM32F051. It is set to ((uint32_t)0x0801F800):
https://github.com/Luos-io/LuosHAL/blob/5bc3d62a768445a520dab0d9b869db1c0d467eb4/STM32F0/luos_hal_config.h#L133
but according to ST's RM0091 Reference manual (page 55/1004), the main flash memory of STM32F051s stops at  0x0800FFFF. Changing ADDRESS_LAST_PAGE_FLASH to ((uint32_t)0x0800F800) solves the problem.
I am not sure this is actually the problem here, but if I don't make this modification, the creation of the Luos container leads the a hardware fault during this call to memset in LuosHAL_FlashReadLuosMemoryInfo:
https://github.com/Luos-io/LuosHAL/blob/5bc3d62a768445a520dab0d9b869db1c0d467eb4/STM32F0/luos_hal.c#L584

LuosHAL/STM32F0/luos_hal_config.h is a default configuration for STM32F0 family. this default configuration must be change to fit with you design and your MCU by define in your project after calling luos.h ADDRESS_LAST_PAGE_FLASH with the right value find in the datasheet

I think  your problem here is a lack of documentation about how to deal with LuosHAL.
I will move this ticket on documentation.


doc naming confusion

I found documentation of compile.json in cli/compile-json.md file and documentation of manifest.json in compiler/Manifest.md. That has confused me a little bit. Should'nt both be in the same directory and have an equal name schema? I suggest to move compile-json.md to compiler dir and rename manifest.md to manifest-json.md

Since they are both configuration files, maybe they should both be in a directory configuration? Then documentation on other things which has config values can refer to there...


Move from glide to dep

FWIW, the glide documentation says that -
The Go community now has the dep project to manage dependencies. Please consider trying to migrate from Glide to dep. If there is an issue preventing you from migrating please file an issue with dep so the problem can be corrected. Glide will continue to be supported for some time but is considered to be in a state of support rather than active feature development.
I think whenever we have some bandwidth, we should make the move to https://github.com/golang/dep

I don't think that dep is going to solve any of our vendoring issues :-(

@kadel agree, but in general we can make this move later on since glide is supported only for now.


Check now button downloads updates instead of just checking for them

@lee-dohm
The Check now button in the About window starts to download an available update right away instead of just checking for it and letting the user know if an update is available or not. It does this although the Automatically download updates box is unchecked.
While I often like to check if an update is available, I do not always want to download it right away (maybe it's just a minor version, maybe I'm on a slow/unreliable network).
So I think the Check now button should do what it claims to do: check if an update is available and if ask the user if he wants to download it or not. This is how it works in most other software packages.

Ok, I understand what you're asking for now. This is partially because of a limitation in the Electron autoUpdate API. We could theoretically work around this limitation to create a flow like this:

Click "Check Now"
Get notified that there is a new version available or not
Change button to "Download" if new version is available
Click "Download"
Wait for it to download
Click "Restart and Install"

But if 80% of the time people are going to want to download a new release if there is one, why make them push the button a third time? And if they are not going to want to download a new version, then does it matter if a new version is available?
For the people that want to only know that there is a new version available, there are options available such as checking https://github.com/atom/atom/releases, subscribing to the Atom releases RSS feed, watching the Atom blog, etc.

But if 80% of the time people are going to want to download a new release if there is one, why make them push the button a third time?

Because 20% of people will be annoyed that the download starts, although the button only suggested to check for a new update, not download one. I really commonly do this in software packages, just checking for updates. I want to find out if there are new features, what new features that would be and if it is worth the download.
Also, if a button claims to do a certain thing, it should not do another thing. So an alternative would be to change the button's label. But the default behavior in all other software packages is that you can check for updates and then decide to download, I can think of no good reason why it should be different in Atom.
Btw. at the moment you have to an extra click, when there is no update available. Then a popup window will notify you that there is no update. This should probably also be avoided by just placing the info
No update available
Version x.x.x is the latest version.

in the box that currently says
Atom is up to date!

Because 20% of people will be annoyed that the download starts, although the button only suggested to check for a new update, not download one. I really commonly do this in software packages, just checking for updates. I want to find out if there are new features, what new features that would be and if it is worth the download.

Agree, I also do like that. Too many times new updates (in various software) were breaking something.

Thanks for the feedback.
We understand the desire for the feature but this isn't something that we feel is going to justify the development effort and ongoing maintenance cost. There are plenty of other ways to check to see if there is a new version, including the RSS feed for Atom's releases that could be built as a separate community package. Since this isn't something that the maintainer team intends to work on, I'm going to close it out.
Thanks again for the ideas!

For the moment, "Check now" button along with  "Automatically download updates" option  (1) are interpreted as: check for the new version and download it without request if this option is checked, and do not download, if the option is unchecked.
So, yes, the button title should be changed to reflect the behavior. It should be "Check and download now" in place of "Check now".
Also, the option (1) caption should be "Automatically check and download updates", or just "Automatically update" to be more consistent with the message: "Automatic updates are disabled please check manually".

Thanks @ruv, I also still feel the button is simply mislabled and tbh don't quite understand why this would not be an issue anymore. And yes, the simplest fix would be to simple change the button label as you suggested.


Conflict with Meteor 1.3.2.x

It is unclear if this is a Meteor or meteor-scss issue, but updating to Meteor <IP_ADDRESS> or <IP_ADDRESS> results in this error:
/Users/mark/.meteor/packages/fourseven_scss/.3.4.3.ifmf9n++os.osx.x86_64+web.browser+web.cordova/plugin.compileScssBatch.os.osx.x86_64/npm/node_modules/meteor/promise/node_modules/meteor-promise/promise_server.js:116
      throw error;

TypeError: Cannot call method 'substr' of undefined

See also https://github.com/meteor/meteor/issues/6851

I think it's Meteor build error. Maybe related this issue that reported by me later.
I fixed my issue with remove all comments from my codes. Read above issue for more info.

@cyclops24 I think it is unrelated. When I remove fourseven:scss from my app it builds fine (but I don't have any css working in my app of course).

I have the same problem with stylus and Meteor <IP_ADDRESS>

Happened to me with less.
After I removed a .less file import from withing a .jsx file, everything was built properly.
Did you guys import scss from js?

@MasterAM yes I did import scss from js, because my library needs to work both with meteor and with webpack (withouth Meteor). Previous versions of Meteor ignored these imports apparently, but <IP_ADDRESS>+ chokes on it.
I resolved it by moving the "require" into a file outside of the Meteor app directory, see https://github.com/reactionic/reactionic-kitchensink/commit/7fd730bba868f9b012ff321fb7a7976af3b4659f


Design changes in upcoming Select2 4.1.0 release

Hi there, this appears to be a frequently used theme for Select2 and I wanted to give you a heads up about upcoming changes to the Select2 CSS/HTML in the next, release 4.1.0. The release date for this theme has not yet been determined, but we are about to roll 4.1.0-beta.1 which features many of these changes.

Dedicated classes have been introduced to handle highlights instead of only using ARIA attributes: https://github.com/select2/select2/pull/5841
aria-selected no longer reflects selected results and instead reflects the highlighted result: https://github.com/select2/select2/pull/5841
Clear/remove icons have been switched to use <button> tags instead of clickable <span> tags: https://github.com/select2/select2/pull/5842
The location of the clear button has been moved outside of the rendered selection area: https://github.com/select2/select2/pull/5842
The location of the search box on a multiple select has been moved outside of the rendered selection area: https://github.com/select2/select2/pull/5842
Selected choices in a multiple select are no longer aligned using CSS float: https://github.com/select2/select2/pull/5842

While many of these should not significantly impact the display of this theme, some of these are going to require modifying the existing CSS to make it look Boostrap-like again. Unfortunately we do not believe that these changes are able to be made backwards-compatible with existing versions of Select2, largely because of the HTML changes that forced the multiple select to be rendered differently.
Feel free to reach out to us about any bugs you encounter or questions you have about these changes.

Any update on this ? When can we expect the changes to be merged ?

Any update on this ? When can we expect the changes to be merged ?

I would like to know this too. What is the plan for being compatible with select2 v4.1 again?

+1

@ttskch Can you give some feedback please?


:white_check_mark: test to check other project lead can see payments

This item belongs to payment request #F78085 on OnlyDust:

from oscarwroche to oscarwroche
35 items included
$1,000 for 2 days of work


Updated ML.NET to 1.5.4 in addition to other dependencies

Resolves

Description

@Brett-Parker this will fix the ML.NET dependencies updates 👍


ksmbd-tools: init at 3.4.4

Description of changes
This adds the ksmbd-tools for the new in-kernel SMB server implementation, which was added in Linux 5.15.
See https://www.kernel.org/doc/html/latest/filesystems/cifs/ksmbd.html for details.
Closes #141225.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Please rebase

Thanks, done and updated to the latest version.

Result of nixpkgs-review pr 165812 run on x86_64-linux 1

1 package built:

ksmbd-tools

Thank you!


Replaced xtest with test

@gabrielegrillo Was this intended as fix for #1666? There seems to be a misunderstanding. The issue was about adding more xtest (skipped tests), not removing them.

Someone else picked up #1666 in the meantime. Sorry for the confusion. I will close this PR now.


refactor: get hash info for tx

Problem
When creating CheckedTransactionDetails for transactions, the lamports_per_signature value is an Option but actually can never be None. Using an Option rather than directly using the value makes error handling needlessly complex downstream.
Summary of Changes

Add a new method to BlockhashQueue to get the info for a particular hash if it's valid
Rename HashAge to HashInfo and add a method to get the lamports per signature value

Fixes #

Codecov Report
Attention: Patch coverage is 98.21429% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 82.7%. Comparing base (cd7710b) to head (7a0682d).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1480   +/-   ##
=======================================
  Coverage    82.7%    82.7%
=======================================
  Files         876      876
  Lines      371245   371275   +30
=======================================
+ Hits       307360   307394   +34
+ Misses      63885    63881    -4


Update the location of the new option

Is there a related issue for this PR?
Issue number:
Developer's Certificate of Origin (DCO)

Signed-off-by: nannanli<EMAIL_ADDRESS>

Hey @nannanli, I think there is something incorrect happening with your DCO signoff here. Might be easy to just copy the content to a new branch since it's only one page?

@BrandonJenkins14 That's a good idea! I've tried to amend my DCO signoff but it does not work. Will try to create a new branch and redo the content. Thanks.

This PR has been replaced by PR #877 . Closing.


.NET 7 RC1 improvements to Rate limiting middleware

Rate limiting middleware improvements

Add [EnableRateLimitingAttribute("MyControllerPolicy")] to Home2Controller
disable rate limiting on a specific endpoint within that group
apply policies directly to endpoints.
add an instance of it directly to an endpoint
Add

builder.Services.AddRateLimiter(options =>
{
   options.AddTokenBucketLimiter("MyControllerPolicy", options =>
   {
       options.TokenLimit = 1;
       options.QueueProcessingOrder = QueueProcessingOrder.OldestFirst;
       options.QueueLimit = 1;
       options.ReplenishmentPeriod = TimeSpan.FromSeconds(10);
       options.TokensPerPeriod = 1;
   })
   .AddPolicy<string>("MyGroupPolicy", new MyRateLimiterPolicy());
});

Then @Rick-Anderson  can update Rate limiting middleware in ASP.NET Core
@Kahbazi can you do these updates?
Also, the updates requested in #14

@fiyazbinhasan @sammychinedu2ky  anyone want to update this sample code for RC1?

@maartenba  would you be able to update the Code . I'll put you on the authors list of Rate limiting middleware in ASP.NET Core

@fiyazbinhasan @sammychinedu2ky anyone want to update this sample code for RC1?

sorry missed this

Here you go folks - https://github.com/dotnet/AspNetCore.Docs.Samples/pull/82

Fixed in #82


Fix/window size

Simply set Stage not resizable: tried to make it dynamic, not working somehow.

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


Auth getting stuck

Sometimes, connections through WalletConnect do not die after disconnecting

Fixed with https://github.com/WalletConnect/web3inbox/pull/216


Bump actions/checkout from v2 to v2.3.3

Latest release for actions/checkout is v2.3.3
See https://github.com/actions/checkout/releases/tag/v2.3.3 for release notes/change-log.

Good catch! Thanks for the PR! 👍

I've checked if you could use the github-actions package ecosystem with Dependabot to deal with upgrades to the Action runners.
I suspect that Dependabot is hard-coded to only check the .github/workflows directory, so that won't help with this repository.
https://docs.github.com/en/free-pro-team@latest/github/administering-a-repository/configuration-options-for-dependency-updates#directory

You define the directory relative to the root of the repository for all ecosystems except GitHub Actions. For GitHub Actions, set the directory to / to check for workflow files in .github/workflows.

I hoped you might be able to do something like this in a dependabot.yml file in the .github folder:
version: 2
updates:
  - package-ecosystem: "github-actions"
    directory: "/combine-prs.yml"
    schedule:
      interval: "daily"

Thanks for looking into that! I actually have a separate repo for manual testing purposes (e.g. with open dependabot PRs) where the workflow is in the .github/workflows so if nothing else, I can get notified there, and then update here as well.
I was also disappointed that apparently there is currently no way to publish a Workflow like this as a GitHub Action of its own - having it on the Actions Marketplace would be a nice way to help others find it more easily. So a potential future improvement would be to rewrite it as an Action (either by re-writing the branch-creation part to happen from JS, so it can be a pure JS action, or make a docker image)

About getting a Starter Workflow published on GitHub
You used to be able to propose new workflows at https://github.com/actions/starter-workflows, so that they can be included into the upstream GitHub repository, and be available for others, but they're not accepting new ones right now.
See issue 631 on that repo for more on that:

👋🏾 Hi there

As you can probably tell, we haven't been very good at merging in new starter workflows in to this repository.  Thank you for submitting them and our apologies for not accepting them sooner.

We'd love to have more starter workflows appear on the Actions tab, but at this time the page just doesn't work well with lots of new starter workflows. Having a team try to process and understand the starter workflows hasn't worked too well.

Perhaps we need to make starter workflows more like the Marketplace, so a community can publish and update workflows without needing approval. That's something we are exploring for the future.

At this time I think its unfair to starter workflow authors to submit them and then have nothing happen with them for a long period of time. Rather than create unfair expectations we are temporarily going to stop accepting new starter workflows. We will try to merge all the ones already submitted, see below 👇🏾 for that list.

Sorry about that, but hope this is temporary and we'll figure something else out that's better for the future.

About rewrite of JavaScript action
I think this is the template you can try use, if you want to make a JavaScript action:
https://github.com/actions/javascript-action
I don't know if you already found that? So I'm mentioning it here in case you haven't yet seen it. :smile:

Thanks for sharing your research! :) Yeah, it seems like the platform is still evolving. For all their similarities, I'm actually kinda surprised that workflows and actions are treated as two completely different things - maybe they'll converge eventually? At any rate, I think a rewrite based on that JS action you linked is probably the most realistic way of "actionifying" the workflow for now.


Pac 3 restrictions

I am wondering if you are able to make it do pac3 restrictions because the only option is a paid addon ?

I'll look into it, see how much work would be required.

Just to be clear, exactly what do you mean with pac3 restrictions? You want to be able to restict what sounds, models etc that players can use?

Well it will basically be able to restrict certain functions. How many sounds,models, and textures they can have they can have etc. Like this script. https://www.gmodstore.com/market/view/3451

Maybe have it in its own tab as well. Ex. PAC 3 - Limits
- restrictions


eu locale

some numbers needed "e" char before "ko":
...21eko
...41eko
...61eko
...81eko

Coverage increased (+0.06%) to 88.499% when pulling 12e38ba524029aa5e69b58c24150855eeebca082 on ZiTAL:develop into e96809208c9d1b1bbe22d605e76985770024de42 on moment:develop.

I will fix it coding, closing the pull request, and i will create new one


Lightweight Azure Billing assistance service

Describe the issue
When a team like CiTv4 comes to us asking for help organizing their billing dashboard, accounting in cloud, we want to be able to quickly show them how to self-serve find the info.
Additional context
Add any other context, attachments or screenshots
Definition of done

Create a video using either Azure Phase II or our own Azure Dev (playzone) that shows a non-technical expense authority how to direct their technical team members to get the cost usage info easily for themselves using Azure's interface.
Use the example training videos like Prabhu's video in Teams, General, Files, Workshops and Accelerators for Partners, Training Videos, Howto-Magic-Wormhole or Health Billing Example

This will be covered in future tickets


Double Vertical Slider

Is your feature request related to a problem? Please describe.
No its not a problem. I want to add a double vertical slider using html, css and javascript.
Additional context
Please assign this under gssoc

/assign

@Rizwan-S @Ankita297 i want to work on this

@Atif0604 I have assigned you another issue. Please work on issues one at a time.

@Atif0604 I have assigned you another issue. Please work on issues one at a time.

Now can i work on this? @Rizwan-S

@Atif0604 sure!


when i rename var. ,many blank lines are added.

VS Code version
1.59.1
Extension version
v2021.10.1365161279
OS type
Windows
OS version
win10
Python distribution
python.org
Python version
3.9.6
Language server
Default
Expected behaviour
only the var. should be rename.
Actual behaviour

Steps to reproduce
every var. ,every time when rename it
Logs
No response
Code of Conduct

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

This is due to https://github.com/pappasam/jedi-language-server/issues/159

This is due to pappasam/jedi-language-server#159

ths! it works.


dragonflybsd adding few ioctl queries

r? @Amanieu
(rust-highfive has picked a reviewer for you, use r? to override)

@bors r+

:pushpin: Commit b81ce71cd3476d96dd8df3349365abd8f66ee7c8 has been approved by Amanieu

:hourglass: Testing commit b81ce71cd3476d96dd8df3349365abd8f66ee7c8 with merge 16457700e4b290f692e9e0358ccee871dc296864...

:broken_heart: Test failed - checks-actions

:umbrella: The latest upstream changes (presumably #2562) made this pull request unmergeable. Please resolve the merge conflicts.


Add products layout is not showing the information properly on small devices

Image must be resized once the user pick one
Product Currency must be {width screen} aware.
It must implement lazy column or constrain layout in order to show the information properly
SAVE button is not showing up

Bug fixed using lazycolumn, but I am moving forward trying constraints