fix: add missing operations for distance_op

Ref: https://github.com/davatorium/rofi/blob/next/doc/rofi-theme.5.markdown#calculating-sizes
I saw the test failed when running tree-sitter test, so I fixed them in the 2nd commit.

Hi, thank you for the fixes ! I didn't realize they were new operators.
Can you add the DMENU keyword in the highlights queries as well ?

Done

Perfect thank you 👍


agent plugin: add logback plugin

add a logback plugin to collect the logs in the trace.

Please follow #4545 and #5863


Right Click on Table > Edit Data (DOES NOT WORK)

Issue Type: Bug
Right Click on Table > Edit Data (DOES NOT WORK)
This has been intermittent for a while now and is a huge pain when trying to quickly add/edit a row
Azure Data Studio version: azuredatastudio 1.33.1 (0b9ef714c823d662815f441aec4a94eaa21940f8, 2021-11-04T17:45:15.872Z)
OS version: Windows_NT x64 10.0.19044
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i3-10100F CPU @ 3.60GHz (8 x 3600)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
23.87GB (10.80GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions: none

Hi @hbycr1, do you still face this issue when you try to edit table data in Azure Data Studio? Also, when you encounter this error, do you get an error log or call stack that could help troubleshoot this?

Thanks for reporting this issue.  Closing based on previous comment and lack of details needed to repro & investigate further.  Please let us know if you're still impacted by this issue.


Too many open files

I have a rather large Org document in emacs that I tie into IHaskell for source blocks. It's a really cool setup, but I do have one issue.
One can export the document through various backends, in this case I'm exporting my document to a reveal.js slide show. Since the IHaskell bits are all in one session, the export procedure evaluates every block from the top to make sure the output is consistent, but I find that after two or three exports, my IHaskell process has crashed. I then need to make sure the Emacs bits are killed because they don't do anything intelligent in response to the IHaskell process dying. At that point I can run another export operation, and IHaskell is restarted and all is well until the next time.
This is the only error message I can surface:
ihaskell: getWorkingDirectory: resource exhausted (Too many open files)
The notebook does a fair bit of importing, and outputs a dozen or more images, so there is quite a bit of file IO.
TL;DR: Does anyone have a guess as to how we might reduce the number of file handles IHaskell holds onto?

I've seen this error a few times myself. I don't know where it comes from.
A first step to debug this is to probably look at the /proc filesystem or lsof to figure out what file handles are open. This could be due to a few things and would be necessary to figure out which of them causes it. If the file handles are open to image files, it would be the IHaskell extensions; if it's to Haskell files or the like, it could be due to the hlint support. Nothing else comes to mind immediately, but it's important to identify which part of IHaskell leaks file handles.

I took a quick look at the OS X Activity Monitor process info tab for IHaskell, and all those /dev/ttys005 file descriptors do look suspicious. Here is the full list of open files and ports after two exports of my document.

Huh. I have no idea what those might be; they're Unix serial devices / TTYs?
I wonder if, say, every time a cell is evaluated, a new /dev/ttys005 is opened... Or perhaps every time a cell that generates an image is evaluted. For example, perhaps these are issues being triggered in hlint (which gets run a ~once per cell), or GHC, or IHaskell itself.

Is it correct that there is no line,
voidpf "IHaskellIO.closeFd %s" readVariable in postStmts
in capturedEval in Evaluate.hs? Right here.

Is there an easy way to temporarily disable hlint so I can see if the overall behavior changes?

:hlint off I think will do so.

Also :help will give you the right command if that's not it

And yeah, good catch, closing readVariable is necessary and I think not done. That may be the issue!

Closing readVariable makes it so Jupyter doesn't get output from IHaskell :/
I don't know why that is, but just running jupyter console --kernel haskell and entering an expression produces no output. So, that FD might be problematic, but closing it there does no good.

You may need to close it later, after it has been read from? We spawn a separate thread to read from that file handle, which loops and reads from that handle until the computation is done. See the code -- Once context is finalized, reading can finish. Wait for reading to finish to that the output; after we takeMVar finishedReading it should be safe to close that. So instead of adding it to postStmts (which get evaluated before takeMVar finishedReading, run it after takeMVar finishedReading.
Basically it's possible that the issue is as follows:

We start the loop in another thread that gets delayed. However it does not start running yet; the scheduler has not scheduled it yet. Especially if we only have one OS thread.
We run the preparation statements and the main cell content and the post statement, and write to the completed MVar.
We check for finishedReading, but it's not yet written to, so we block until reading ends.
The scheduler schedules our loop thread, which does the reading. It starts reading. HOWEVER since postStmts have been run, the handle is already closed! It fails at reading.
We now continue onwards, having read nothing.

If we only close the readVariable after we are sure we are done reading, maybe the issue will be fixed?

I tried this, and it broke output to file for images. I don't have any IHaskell error messages, but on the emacs side there was no output to be piped to a file.

That's very surprising. readVariable has, I think,  more or less nothing to do with image writing. Since it causes problems on images, it may be a timing issue with the threading, since images presumably take a long time to write :-/ gnarly issue

Yeah, it's pretty black-and-white for me: if I evaluate the closeFd on readVariable after takeMVar finishedReading my very first image output fails. If I leave the IHaskell side alone, it works fine.
Btw, :option no-lint didn't solve this problem, but it did have the very nice effect of massively reducing memory retention. Peak memory usage is unchanged, but with linting turned off, the memory is freed when IHaskell goes idle, whereas before everything was held.

Has this issue been fixed?  I'm getting the same behaviour with Jupyter console 6.0.0, IHaskell <IP_ADDRESS>.

Do you have a small example to help me reproduce this behaviour?

I'm attaching a Haskell Jupyter notebook (zipped) importing System.Random and calling randomIO 50 times.  After the 47th call, it reports:
ihaskell: /dev/urandom: openFd: resource exhausted (Too many open files)

If that's not the case on your machine, you may have to introduce a few more randomIO calls.
open-files-bug.zip

Should I make a new issue for this?

I'm not sure this is an issue with IHaskell. Does this work in GHCi?

I tried this with 89 cells and could not reproduce the issue you're experiencing.

I cannot reproduce it either with the latest version of master on Linux (aee398a82fb87146c39902dbefea723d1799292e).  It's either a bug that manifests only on Mac or something that got fixed recently (after e15dd63d2efdcfda599dc022732b33589005a6d5).  I'll check again.

It must be a Mac issue because e15dd63 on Linux works OK.  In contrary, the latest aee398a does not work on Mac.


basedir=$(dirname "$(echo "$0" | sed -e 's,\\,/,g')")

When importing vue-test-utils, I get a weird error that doesn't seem to be related to anything else other than the component itself. I am running the test with jest.
Test code:
import { shallow, createLocalVue } from 'vue-test-utils'

describe('App', () => {
  it('should work', () => {
    expect(true).toBe(true)
  })
})

If the import line is commented out, the test runs fine and passes; with the import in, I get the following error:
 FAIL  src\core\App.test.js
  ● Test suite failed to run

    C:\Program Files\nodejs\vue:2
    basedir=$(dirname "$(echo "$0" | sed -e 's,\\,/,g')")
              ^^^^^^^
    SyntaxError: missing ) after argument list

      at ScriptTransformer._transformAndBuildScript (node_modules/jest-runtime/build/script_transformer.js:305:17)
      at Object.<anonymous> (node_modules/vue-test-utils/dist/vue-test-utils.js:5:27)
      at Object.<anonymous> (src/core/App.test.js:2:21)

That done, convinced that the problem was with vue-test-utils, I tried to create tests without using the component:
import Vue from 'vue'
import App from './App.vue'

describe('App', () => {
  it('renders default layout if none is defined in', () => {
    const Ctor = Vue.extend(App)
    const instance = new Ctor().$mount()
    expect(instance.layout).toBe('default')
  })
})

... aaaannnd I get the same error again. Therefore, the issue seems to be somewhere within the jest ecosystem.
I have removed the node_modules folder and reinstalled it all using npm install to make sure it was not a dependency issue with each test.
Jest config:
// package.json
  "jest": {
    "moduleFileExtensions": [
      "js",
      "vue"
    ],
    "moduleNameMapper": {
      "^@/(.*)$": "<rootDir>/src/$1"
    },
    "transform": {
      "^.+\\.js$": "<rootDir>/node_modules/babel-jest",
      ".*\\.(vue)$": "<rootDir>/node_modules/jest-vue"
    },
    "mapCoverage": true,
    "verbose": true
  }

Versions:

Windows 10
npm v3.10.10
node v6.10.3
vue v2.5.2
babel-jest v21.2.0
jest v21.2.1
jest-vue v0.8.1
vue-test-utils with v1.0.0.beta.3, v1.0.0.beta.2, v1.0.0.beta.1

This seems like a problem with vue-test-utils, not a Jest issue.

It is obviously not, as I mention on the original request.
Even without any reference to vue-test-utils, I get the exact same error.
Ricardo Nolde
https://au.linkedin.com/in/nolde
https://stackoverflow.com/cv/nolde
+61 414 097 431
On 24 October 2017 at 15:11, Christoph Nakazawa<EMAIL_ADDRESS>wrote:

Closed #4751 https://github.com/facebook/jest/issues/4751.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/facebook/jest/issues/4751#event-1306939891, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AMNWG1i_AVxe-rEqCvN5Kz4V11QZjGwFks5svWNugaJpZM4QDznk
.

It is not, as I mention on the original request.
Even without any reference to vue-test-utils, I get the exact same error.

require('vue') is pulling in a shell script not JavaScript. Does this happen in the node repl? If not, I'm inclined to calling this a jest bug

Just had a friend set up similar code in OSX; it works on his machine, fails in my Windows device.
Therefore, it is Windows environment related.

Can you delete C:\Program Files\nodejs\vue? And install vue locally in your project instead (and maybe do npm install --global vue)

It is installed locally, and there is no vue installed globally.
The vue that is being picked up seems to be the entry point for vue-cli command vue, which is pretty weird.

if you install the cli globally, what does `node -p "require.resolve('vue')" return?

Running it inside a project that has vue installed, I get C:\git\<project-folder>\node_modules\vue\dist\vue.runtime.common.js
If I run it outside of a project folder, I get the vue-cli bash file.

Yeah, that's a bug in the module resolution of Jest, then.

That said, I can't reproduce on mac. node -p "require.resolve('vue')" outside of a project throws Error: Cannot find module 'vue', even if vue-cli is installed globally.

Yeah, it seems to be related to Windows. My co-worker using mac installed vue-cli as well and nothing came up.

This has not been an issue for me until just recently. I am getting this error now when i try to run npm start. The start command in package.json is
"start": "node node_modules/nodemon/bin/nodemon.js src/app.js --exec "npm run lint && node"",
"lint": "node node_modules/.bin/eslint **..js"
If I change the start command to
"start": "nodemon src/app.js --exec "npm run lint && node"",
"lint": "eslint **..js"
I do not get the error

@Nulimitz not related to jest at all

Just putting it out there that we need help from the community with this one.

@Havunen I had this issue with Nuxt, and your comment was exactly what I needed. It's a shame I only found it after spending the better part of two days on this.

Is there a ./node_modules/.bin/jest.cmd?

Checkout https://github.com/vuejs/vue-test-utils-jest-example
yarn
yarn run test
multiple basedir=$(dirname "$(echo "$0" | sed -e 's,\\,/,g')") errors

There is jest.cmd in ./node_modules/.bin.
vue-cli installed globally

For whomever comes after me, my issue was this was in trying to run this command on a windows machine: node --expose-gc ./node_modules/.bin/jest --runInBand --logHeapUsage, however, I could run it by running this command as suggested above node --expose-gc ./node_modules/jest/bin/jest --runInBand --logHeapUsage

I was using below configuration in package.json file
"scripts": {
 "test": "node --inspect node_modules/.bin/cross-env API_HOST=url ember serve",
}

The above config work for the Linux machine. But I have a windows machine and during npm run test I got this error.
In my case when I removed node --inspect than its work for me.
 "test": "node_modules/.bin/cross-env API_HOST=url ember serve",

This might explain why the above error occurs on windows but not OSX
https://github.com/gotwarlost/istanbul#usage-on-windows


Unable to synthesize getting started example

Steps I took:

Run npx create-serverless-stack@latest my-sst-app
cd my-sst-app
npx sst start

Output:
PS D:\Projects\Mafia\my-sst-app> npx sst start
Using stage: dev
Preparing your SST app

=======================
 Deploying debug stack
=======================

Usage: cdk -a <cdk-app> COMMAND

Commands:
  cdk.js list [STACKS..]             Lists all stacks in the app   [aliases: ls]
  cdk.js synthesize [STACKS..]       Synthesizes and prints the CloudFormation
                                     template for this stack    [aliases: synth]
  cdk.js bootstrap [ENVIRONMENTS..]  Deploys the CDK toolkit stack into an AWS
                                     environment
  cdk.js deploy [STACKS..]           Deploys the stack(s) named STACKS into your
                                     AWS account
  cdk.js watch [STACKS..]            Shortcut for 'deploy --watch'
  cdk.js destroy [STACKS..]          Destroy the stack(s) named STACKS
  cdk.js diff [STACKS..]             Compares the specified stack with the
                                     deployed stack or a local template file,
                                     and returns with status 1 if any difference
                                     is found
  cdk.js metadata [STACK]            Returns all metadata associated with this
                                     stack
  cdk.js acknowledge [ID]            Acknowledge a notice so that it does not
                                     show up anymore              [aliases: ack]
  cdk.js notices                     Returns a list of relevant notices
  cdk.js init [TEMPLATE]             Create a new, empty CDK project from a
                                     template.
  cdk.js context                     Manage cached context values
  cdk.js docs                        Opens the reference documentation in a
                                     browser                      [aliases: doc]
  cdk.js doctor                      Check your set-up for potential problems

Options:
  -a, --app                REQUIRED: command-line for executing your app or a
                           cloud assembly directory (e.g. "node bin/my-app.js")
                                                                        [string]
  -c, --context            Add contextual string parameter (KEY=VALUE)   [array]
  -p, --plugin             Name or path of a node package that extend the CDK
                           features. Can be specified multiple times     [array]
      --trace              Print trace for stack warnings              [boolean]
      --strict             Do not construct stacks with warnings       [boolean]
      --lookups            Perform context lookups (synthesis fails if this is
                           disabled and context lookups need to be performed)
                                                       [boolean] [default: true]
      --ignore-errors      Ignores synthesis errors, which will likely produce
                           an invalid output          [boolean] [default: false]
  -j, --json               Use JSON output instead of YAML when templates are
                           printed to STDOUT          [boolean] [default: false]
  -v, --verbose            Show debug logs (specify multiple times to increase
                           verbosity)                   [count] [default: false]
      --debug              Enable emission of additional debugging information,
                           such as creation stack traces of tokens
                                                      [boolean] [default: false]
      --profile            Use the indicated AWS profile as the default
                           environment                                  [string]
      --proxy              Use the indicated proxy. Will read from HTTPS_PROXY
                           environment variable if not specified        [string]
      --ca-bundle-path     Path to CA certificate to use when validating HTTPS
                           requests. Will read from AWS_CA_BUNDLE environment
                           variable if not specified                    [string]
  -i, --ec2creds           Force trying to fetch EC2 instance credentials.
                           Default: guess EC2 instance status          [boolean]
      --version-reporting  Include the "AWS::CDK::Metadata" resource in
                           synthesized templates (enabled by default)  [boolean]
      --path-metadata      Include "aws:cdk:path" CloudFormation metadata for
                           each resource (enabled by default)
                                                       [boolean] [default: true]
      --asset-metadata     Include "aws:asset:*" CloudFormation metadata for
                           resources that uses assets (enabled by default)
                                                       [boolean] [default: true]
  -r, --role-arn           ARN of Role to use when invoking CloudFormation
                                                                        [string]
      --staging            Copy assets to the output directory (use --no-staging
                           to disable, needed for local debugging the source
                           files with SAM CLI)         [boolean] [default: true]
  -o, --output             Emits the synthesized cloud assembly into a directory
                           (default: cdk.out)                           [string]
      --notices            Show relevant notices                       [boolean]
      --no-color           Removes colors and other style from console output
                                                      [boolean] [default: false]
      --version            Show version number                         [boolean]
  -h, --help               Show help                                   [boolean]

If your app has a single stack, there is no need to specify the stack name

If one of cdk.json or ~/.cdk.json exists, options specified there will be used
as defaults. Settings in cdk.json take precedence.

There was an error synthesizing your app.
PS D:\Projects\Mafia\my-sst-app>

Normal aws-cdk projects work fine:
(.venv) PS D:\Projects\Mafia\mafia-cdk> cdk synth
Resources:
  MafiaApiTestFunctionServiceRoleC23260D1:
    Type: AWS::IAM::Role
    Properties:
      AssumeRolePolicyDocument:
      ...
      ...

Seems like the command npx sst start is triggering the command cdk but not providing any other inputs such as cdk synth etc.
This issue doesn't happen on my other laptop though. So I have no idea where to look,

Hey @JacksonBowe, this should be fixed in v0.69.2.


docs(CONTRIBUTING): add section on testing

Checklist

[x] updated CHANGELOG.md

I am strongly in favor of of the 3 test levels, but I miss a definition of the definition and objective of each level. E.g. integration testing could be seen as the interface to to VScode.

but I miss a definition of the definition and objective of each level. E.g. integration testing could be seen as the interface to to VScode

Great feedback! I think I should clarify this a bit more on what we mean by each.

Sweet! Thanks for the reviews @janiversen @jawnsy - going to merge


feat: Support major version engine upgrades

Exposes the same aws_rds_cluster resource variable as a module variable. Defaults to the same value to preserve backwards compatibility.
see:
https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/rds_cluster#allow_major_version_upgrade

Any progress about this feature?

@antonbabenko how do we support this pr? Is there a process or a list of moderators to help manage it?

Hi! There are several people who can review and speak about PRs. I usually listen to their feedback and merge/review things as soon as I have a possibility.
I flag this PR for the upcoming review cycle and hopefully will be able to merge this during this week.

PS: There is GitHub Sponsors feature if you want to support my work. I often try to prioritize PRs and issues from sponsors higher for obvious reasons.

Thanks @tpounds for the PR!
v3.5.0 has been just released.


use powershell install scoop,return MethodInvocationException

PS C:\Windows\system32> iex (new-object net.webclient).downloadstring('https://get.scoop.sh')
使用“1”个参数调用“DownloadString”时发生异常:“操作超时” 所在位置 行:1 字符: 1

iex (new-object net.webclient).downloadstring('https://get.scoop.sh')
CategoryInfo          : NotSpecified: (:) [], MethodInvocationException
FullyQualifiedErrorId : WebException

I guess you are suffering network problem, maybe you should try to use proxy while installation.
Tip: use below command to display error message in English.
[Threading.Thread]::CurrentThread.CurrentUICulture = 'en-US'; iex (new-object net.webclient).downloadstring('https://get.scoop.sh')

yes,you are right,thank you so much


feat: support for injector library

add support for injector library, which is often used in Python for dependency injection pattern implementation

I like the staff you implemented in this linter; we found it useful in our projects. We faced a conflict with the injector project because it used typing annotation in runtime.
For example:
from injector import Inject
from service_b import ServiceB

class ServiceA:
     def __init__(self, service_b: Inject[ServiceB]) -> None:
         ...

In this pull request I want to add the support for such functionality for ones using this library as we do.


Unable to create postgresflex_database specifiying an owner having hyphens in username

Terraform Version 1.9.3
Stackit Provider Version 0.26.1
When using terraform in order to create PostgreSQL Flex databases, it will end up in an inconsistent result when the specified owner username contains hyphens.
Working example from documentation:
resource "stackit_postgresflex_database" "example" {
  project_id  = "xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx"
  instance_id = "xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx"
  name        = "mydb"
  owner       = "myusername"
}

Non-working example:
resource "stackit_postgresflex_database" "example" {
  project_id  = "xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx"
  instance_id = "xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx"
  name        = "mydb"
  owner       = "my-username"
}

will produce the following error:

Error: Provider produced inconsistent result after apply
When applying changes to stackit_postgresflex_database.example, provider "provider["registry.terraform.io/stackitcloud/stackit"]" produced an unexpected new value: .owner: was
cty.StringVal("my-username"), but now cty.StringVal(""my-username"").
This is a bug in the provider, which should be reported in the provider's own issue tracker.

Hi @lordon,
Thanks for opening the issue. I will be having a look and will provide a fix soon.

Hi @lordon,
The fix was released in v0.26.2


Conflicting names with version 1.9.0

In package.json
"react-native-gainsight-px": "https://github.com/Gainsight/px-react-native-sdk.git#d7467353910e5dad13b11652bcb8e4d127129ede",

In the Podfile:
pod 'PXKit', :git => 'https://github.com/Gainsight/px-ios.git', tag: '1.9.0'

When I do pod install
[!] The 'Pods-App-Mobile' target has frameworks with conflicting names: pxkit.xcframework.

What's wrong?

Please do not use or remove this line 'PXKit', :git => 'https://github.com/Gainsight/px-ios.git', tag: '1.9.0' from project pod spec

Ok it works, thank you.


Display all on module pages

https://github.com/zsol/py.wtf/pull/33 added explicit exports to the index; let's show them! Probably as a dedicated section near the top of the module contents page.

#33 is not enough, we'll need proper xrefs even for intra-linked symbols here unfortunately (as described in #8), because for this
from foo import bar as baz
__all__ = ["baz"]

We'll want to generate a link that's called baz, but points to the symbol foo.bar. Can't do that with plain strings.

#65 should be all that's needed from the indexer side 😄

Fixed by #79


feat: add batch put command

change
add batch put command. 😁

Thanks for the contribution. Is it just some syntactic sugar for the existing Put? This implementation doesn't improve the performance, you save a bit on acquiring the mutex only once per batch, but on the other hand it may cause issues for reads if the mutex is held for too long when inserting a large number of keys.

Thanks for the contribution. Is it just some syntactic sugar for the existing Put? This implementation doesn't improve the performance, you save a bit on acquiring the mutex only once per batch, but on the other hand it may cause issues for reads if the mutex is held for too long when inserting a large number of keys.

Just to reduce the lock time. 😅  In local kv read/write case, this optimization seems to have no other benefits.


Doc tests

Codecov Report

Merging #7 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master       #7   +/-   ##
=======================================
  Coverage   96.06%   96.06%
=======================================
  Files          40       40
  Lines        4044     4044
=======================================
  Hits         3885     3885
  Misses        159      159

Impacted Files
Coverage Δ

src/creature.rs
95.53% <ø> (ø)
:arrow_up:

src/character.rs
94.69% <ø> (ø)
:arrow_up:

src/team.rs
98.14% <ø> (ø)
:arrow_up:

src/fight.rs
95% <ø> (ø)
:arrow_up:

src/space.rs
95.78% <ø> (ø)
:arrow_up:

src/ability.rs
89.65% <ø> (ø)
:arrow_up:

tests/entropy.rs
100% <ø> (ø)
:arrow_up:

src/event.rs
96.01% <ø> (ø)
:arrow_up:

src/entropy.rs
94.33% <ø> (ø)
:arrow_up:

src/actor.rs
93.61% <ø> (ø)
:arrow_up:

... and 11 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 88eed80...9f3088c. Read the comment docs.


[CORE] Store/Use gemini ref images using S3

Sorry this didn't come earlier. With this change we won't be keeping copies of ref images in our repo. Instead, Travis will sync with the "golden image set" from S3, whose location is defined by our Travis environment variable. The jobs that are taking ref images are going to continue to upload to S3 as before. If we want to update the reference images in the future, we simply need to update that one Travis environment variable.

I don't really understand what's happening here, so I'll refrain from approving blindly. Feel free to merge if you have enough :shipit:.
I'd be interested in a quick explanation on a call next week, if you have the time.

+1 to what Eudes said

Sure, I can schedule a quick call early next week to go over this before merging.

Also, with the docker changes I am working on this change also means core devs will need to have the aws-cli tools installed.

@jeeyun @hippee-lee: What are we doing with this PR? Can we close this?

We should close it for now.


nbhood refactor

Refactor of neighbourhood plugin:

Adds standalone routine for calculating neighbourhood totals using summed-area table method
Brings nbhood-iterate-with-mask in line with changes in nbhood-land-and-sea
Removes weight renormalisation from CollapseMaskedNeighbourhoodCoordinate, as the same is done internally by the np.ma.avarage routine used by iris.analysis.MEAN if the input data is MaskedArray.
Moves cube set-up cubes method and associated tests to RecursiveFilter to keep it working.

Testing:

[x] Ran tests and they passed OK
[ ] Added new tests for the new feature(s)

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- improver/nbhood/square_kernel.py  5

Complexity decreasing per file
==============================
+ improver/cli/nbhood_iterate_with_mask.py  -1

Clones added
============
- improver_tests/nbhood/recursive_filter/test_RecursiveFilter.py  2

Clones removed
==============
+ improver_tests/nbhood/square_kernel/test_SquareNeighbourhood.py  -9

See the complete overview on Codacy

I put this up for review in it's current shape as I struggle to spend more time on it. @fionaRust could have a look at the deleted tests and which of them would be worthwhile keeping/adapting for the new functions?

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- improver/nbhood/square_kernel.py  5

Complexity decreasing per file
==============================
+ improver/cli/nbhood_iterate_with_mask.py  -1

Clones added
============
- improver_tests/nbhood/recursive_filter/test_RecursiveFilter.py  2

Clones removed
==============
+ improver_tests/nbhood/square_kernel/test_SquareNeighbourhood.py  -9

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- improver/nbhood/square_kernel.py  5

Complexity decreasing per file
==============================
+ improver/cli/nbhood_iterate_with_mask.py  -1

Clones added
============
- improver_tests/nbhood/recursive_filter/test_RecursiveFilter.py  2

Clones removed
==============
+ improver_tests/nbhood/square_kernel/test_SquareNeighbourhood.py  -9

See the complete overview on Codacy

There are a few numpy-heavy functions that could do with more inline comments to explain what's happening more clearly.

I'll try to add a few more comments. Let me know if there's anything in particular you would like to see explained better.

I had a bit of a look at the scipy functions scipy.ndimage.correlate and scipy.ndimage.uniform_filter which could be used for this purpose. A quick performance comparison showed this code with boxsum being similar speed to scipy uniform_filter and 1.5-2x quicker than scipy correlate. uniform_filter works as a combination of 1D filters, and output seemed to be slightly different leading to acceptance tests failing.

I gave this a quick try. In micro benchmarks scipy.ndimage.uniform_filter seemed faster than both: the built-in numpy.cumsum and the bespoke implementation with outer loop in Python. This didn't seem to carry over into an CLI run, though, in which case it was slower than both for some reason (less CPU usage and more system time). I also didn't manage to make it pass the tests and it doesn't support complex numbers, which are needed for the wind direction.

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- improver/nbhood/square_kernel.py  3

Complexity decreasing per file
==============================
+ improver/cli/nbhood_iterate_with_mask.py  -1

Clones added
============
- improver_tests/nbhood/recursive_filter/test_RecursiveFilter.py  2

Clones removed
==============
+ improver_tests/nbhood/square_kernel/test_SquareNeighbourhood.py  -9

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 3

Complexity increasing per file
==============================
- improver/nbhood/square_kernel.py  3

Complexity decreasing per file
==============================
+ improver/cli/nbhood_iterate_with_mask.py  -1

Clones added
============
- improver_tests/nbhood/recursive_filter/test_RecursiveFilter.py  2

Clones removed
==============
+ improver_tests/nbhood/square_kernel/test_SquareNeighbourhood.py  -9

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- improver/utilities/neighbourhood_tools.py  1
- improver/nbhood/square_kernel.py  3

Complexity decreasing per file
==============================
+ improver/cli/nbhood_iterate_with_mask.py  -1

Clones added
============
- improver_tests/nbhood/recursive_filter/test_RecursiveFilter.py  2

Clones removed
==============
+ improver_tests/nbhood/square_kernel/test_SquareNeighbourhood.py  -9
+ improver_tests/nbhood/use_nbhood/test_CollapseMaskedNeighbourhoodCoordinate.py  -19

See the complete overview on Codacy

This looks great. I have left very few comments, and I'm afraid I have run out of time to go through all the tests. The acceptance test data that have changed are:
...
The changes are on average around precision level. The min and max value changes are somewhat larger in places, though I am not sure of the origin of these changes.

Yes, these tests were originally run in float32 precision and suffer a lot from round off errors, cf. https://github.com/metoppv/improver/pull/1251#discussion_r420690197.

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- improver/utilities/neighbourhood_tools.py  1
- improver/nbhood/square_kernel.py  3

Complexity decreasing per file
==============================
+ improver/cli/nbhood_iterate_with_mask.py  -1

Clones added
============
- improver_tests/nbhood/recursive_filter/test_RecursiveFilter.py  2

Clones removed
==============
+ improver_tests/nbhood/square_kernel/test_SquareNeighbourhood.py  -9

See the complete overview on Codacy

I'm happy for these changes to be merged. We'll need to move the new acceptance test data @bayliffe created into ppdev


SELinux audit parser fails to parse audit lines with multiple permissions

What happened:
My SELinux test recording emitted the following log line:
type=AVC msg=audit(1666691794.882:1434): avc:  denied  { read write open } for  pid=94509 comm="aide" path="/hostroot/etc/kubernetes/aide.log.new" dev="nvme0n1p4" ino=167774224 scontext=system_u:system_r:selinuxrecording.process:s0:c218,c875 tcontext=system_u:object_r:kubernetes_file_t:s0 tclass=file permissive=1

it did not match the regex at all and wasn't thus parsed
What you expected to happen:
the line should be parsed
How to reproduce it (as minimally and precisely as possible):
not sure, but putting the above line into the unit tests reproduces the issue
Anything else we need to know?:
N/A
Environment:

Cloud provider or hardware configuration:
OS (e.g: cat /etc/os-release): RHCOS 4.11
Kernel (e.g. uname -a): N/A
Others: N/A

/assign

This was fixed in https://github.com/kubernetes-sigs/security-profiles-operator/commit/4aaae010a42ea07c8a56a6bcdb43f4543b3982d8

Sorry, I confused this with another ticket. Reopening.


(#1011) Replaced String.startsWith by cactoos scalar StartsWith

This PR is for issue #1011

This pull request #1034 is assigned to @fabriciofx/z, here is why; the budget is 15 minutes, see §4; please, read §27 and when you decide to accept the changes, inform @paulodamaso/z (the architect) right in this ticket; if you decide that this PR should not be accepted ever, also inform the architect; this blog post will help you understand what is expected from a code reviewer; there will be no monetary reward for this job

@0crat assign me

@0crat assign me (here)

@paulodamaso This pull request #1034 is assigned to @paulodamaso/z, here is why; the budget is 15 minutes, see §4; please, read §27 and when you decide to accept the changes, inform @paulodamaso/z (the architect) right in this ticket; if you decide that this PR should not be accepted ever, also inform the architect; this blog post will help you understand what is expected from a code reviewer; there will be no monetary reward for this job

Manual assignment of issues is discouraged, see §19: -5 point(s) just awarded to @paulodamaso/z

@paulodamaso Please have a look here.
I have created an issue in Cactoos.

@fanifieiev Thank you!

@rultor merge

@rultor merge

@paulodamaso OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@fanifieiev @paulodamaso Oops, I failed.

Locales to be generated: 4a294ce930eef6e17ac4d6b4efb3a904b1cbf35720707f30bca4b34b3300baaa
'pid' file is absent, something was broken

@rultor merge

@rultor merge

@paulodamaso OK, I'll try to merge now. You can check the progress of the merge here

@rultor stop

@rultor stop

@paulodamaso OK, I'll try to stop current task.

@rultor stop

@fanifieiev @paulodamaso Sorry, I failed to stop the previous command, however it has the following result: Oops, I failed.

Locales to be generated: d44101943c1299b9061e54d4a917c42c1da80428066d275fdd05e1ecdfbc6a53
Error response from daemon: Cannot kill container: d44101943c1299b9061e54d4a917c42c1da80428066d275fdd05e1ecdfbc6a53: Container d44101943c1299b9061e54d4a917c42c1da80428066d275fdd05e1ecdfbc6a53 is not running
container d44101943c1299b9061e54d4a917c42c1da80428066d275fdd05e1ecdfbc6a53 is dead
Mon Mar  9 18:54:24 CET 2020

Job #1034 is not in the agenda of @paulodamaso/z, can't retrieve data and time of add

Code review was too long (15 days), architects (@paulodamaso) were penalized, see §55


Confirm retry behaviour when a WriteTimeoutException occurs for Collections

we worked out that the WritetimeoutException is coming back as a WriteType SIMPLE when it fails in the COMMIT phase of the Paxos Tx for collections. We are confirming this with extra logging, if true the retry for collections should work for any write timeout (so not need to check the write type).
Original desc is below...

Confirm what happens in the API when running a write operation for a Collection and a WriteTimeoutException is returned from the DB.
It may be returning the timeout to the client without retrying, we should review if this is the best and if we can retry the request

discussions - also check what retry policy the tables are using. It would be good to have the same retry policy is possible, which would mean removing the test for the write type.
note - we also discussed refactoring the try policy class so:

derives from DefaultRetryPolicy
does not override default methods from the RetryPolicy interface
has better logging that always logs the decisions, and maybe logs a rethrow decision at WARN


TextBox that got focus by clicking on the Clear Button location does not accept key input

Describe the bug
If focus is got by clicking on the "right side" of the TextBox, it does not accept key input.
The "right side" is where the Clear Button appears.
If you move the mouse pointer to the left, the TextBox accepts key input.
Steps to reproduce the bug
Place two fixed-width TextBlocks.
<TextBox x:Name="A" Width="200"/>
<TextBox x:Name="B" Width="200"/>

The operation is as follows.

Enter some text into each TextBox.
Focus on TextBox A.
Click on the "right side" of TextBox B. Do not move the mouse pointer after clicking.
Focus is shifted to TextBox B, but it does not accept key input.

Move the mouse pointer to the left.
The TextBox accepts key input

Expected behavior
No response
Screenshots
No response
NuGet package version
WinUI 3 - Windows App SDK 1.1.4
Windows app type

[ ] UWP
[X] Win32

Device form factor
Desktop
Windows version
Windows 11 (21H2): Build 22000
Additional context
No response

This is still an existing problem in the latest WinUI on .Net 8.0.
Can this be reopened?

For reference, this is the location where if you click, you will get focus by typing doesn't add what you type to the field.

Thanks for that extra image, @Joost-Jens-Luminis. I'm able to repro this issue in the TextBox page of WinUI 3 Gallery. Reopening.

Additional note: When I repro this in W3G, moving the mouse to the left over the textbox selects the text, so it seems the TextBox is also confused in thinking that it was still in a PointerPressed state. The textbox also grew wider when the Clear button was added, which may have somehow triggered confusion.

Hello, I have exactly the same problem with a NumericBox with the latest stable version of WinUIEx v2.3.4 and .NET 7.
The problem occurs as soon as the first click is made on the X zone. Keyboard input is not taken into account until the mouse is moved. Keyboard input is ignored until the mouse is moved. After that, the problem no longer seems reproducible.


datapath: Remove IPV{4,6}_NODEPORT

After introducing IPV{4,6}_MASQUERADE, the former is no longer in use.
For the case when a LB request has to be forwarded to a destination
node, we use IPV{4,6}_DIRECT_ROUTING. For SNAT from endpoint to outside
we use IPV{4,6}_MASQUERADE.
Fix #14273

checkpatch.pl is failing with a false positive:
WARNING:CONSTANT_COMPARISON: Comparisons should place the constant on the right side of the test
#74: FILE: bpf/lib/nodeport.h:914:
+       if (DIRECT_ROUTING_DEV_IFINDEX == NATIVE_DEV_IFINDEX &&

NATIVE_DEV_IFINDEX is set while loading via the templating.

test-me-please


added an annotation to beta in list

Currently, Xcode beta versions are not annotated, when you call xcversion list.
This fix should be applicable for ancient Xcode versions as well, such as "4.3.1 for Lion", although nobody would use them now.
before

after

This is reported in the comment of #291 .

Awesome. 🚀


usage example uses a deprecated google scope

As per google's login scopes doc the scope userinfo.email scope featured in the example in the README is deprecated.
Since many people blindly copy/paste the sample code, me thinks the example should be updated.
For now the non deprecated, oauth based equivalent seems to be https://www.googleapis.com/auth/plus.profile.emails.read, while the "real" equivalent would be OpenID Connect's email but this plugin doesn't handle this, does it?

Thanks for pointing this out! It is difficult for me to keep track of everything, so this is much appreciated.
I wan't to say email won't work, but you can always give it a try.  I'll update the documentation some time this week to reflect.
Thanks,

Tried already :wink:
The API returns an invalid scope error after the user enters its credentials.
I'm not so familiar with it but it seems the OpenID Connect dance is slightly different (even though it's based on Oauth).

Actually simply setting email as the scope instead of https://www.googleapis.com/auth/userinfo.email works like a charm.

Thanks for the update!

This change to the README is now in the master.


Migrate core test packages to null safety

[x] async, @jakemac53
[x] boolean_selector, @jakemac53
[x] charcode
[x] collection, @jakemac53
[x] matcher, @jakemac53
[x] meta, @jakemac53
[x] path, @munificent
[x] pedantic
[x] pool
[x] source_span, @jakemac53
[x] stack_trace, @munificent
[x] stream_channel, @natebosch
[x] string_scanner, @jakemac53
[x] term_glyph, @jakemac53
[x] test, @jakemac53
[x] test_api, @jakemac53
[x] test_core, @jakemac53

All packages have null-safe branches.

[ ] tests pass? No! Tests don't run on 3 of them. Teste failures on ~5 othres
[ ] published? Not yet. Watch https://github.com/dart-lang/sdk/issues/41398

Maps to (Google) internal sheet http://sheets/1eTLiYw1ylopt7loBvKDv9cSRwpNIBsHX9FjfNOAk1nY

@kevmoo what remains here?

@kevmoo Please update, and move to #41611 when it's done for TP2

See page 2 of the sheet linked above which I added to track the merging into sdk/internal and publishing. All packages are now tested and merged into google3, and we should be able to begin publishing tomorrow.

@kevmoo @jakemac53 from the tracking sheet, looks like this is done now?


chore: Merge upstream to Mandalay as of 02182022

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
230

Relevant Lines:
474

💛  - Coveralls

@mukhoakash We'll need to disable the live disk resize test as this seems to be failing in North Europe. Please file an issue to re-enable after root causing the test failure.
Feb 24 06:57:17.186: INFO: At 2022-02-24 06:46:48 +0000 UTC - event for test-volume-1-azuredisk-volume-tester-mfblf-0: {external-resizer disk.csi.azure.com } VolumeResizeFailed: resize volume "pvc-40f2d937-eb82-45a1-bb2e-83ea4effab09" by resizer "disk.csi.azure.com" failed: rpc error: code = Internal desc = rpc error: code = Internal desc = failed to resize disk(/subscriptions/***/resourceGroups/k8s-e2etest-e8ed16a0-9535-11ec-90f3-3f09839f38f8-rg/providers/Microsoft.Compute/disks/pvc-40f2d937-eb82-45a1-bb2e-83ea4effab09) with error(Retriable: true, RetryAfter: 0s, HTTPStatusCode: -1, RawError: Code="LiveResizeFailed" Message="Error resizing disk pvc-40f2d937-eb82-45a1-bb2e-83ea4effab09 while it is attached to running VM(s) /subscriptions/***/resourceGroups/k8s-e2eTest-e8ed16a0-9535-11ec-90f3-3f09839f38f8-rg/providers/Microsoft.Compute/virtualMachines/k8s-agentpool1-39769881-2.")


Missing requirement on plantuml binary package

Description
Apologies that this is coming in as a bug. I cannot seem to select this as a documentation issue. I assume someone else has the ability to change the type after submission.
As outlined in the discussion coming from #3942 and https://github.com/nlohmann/json/pull/4024, as someone who is trying to contribute to the documentation for the first time, it is not clear that plantuml is a depedency required to run the documentation engine.
In order for me to run the mkdocs instance for the first time, I needed to brew install plantuml on my Mac or sudo apt install plantuml on my Ubuntu machine. The README discussing how to run the mkdocs project does not mention this requirement.
I see this as a documentation issue first, but arguably we could add a make directive to or have some other mechanism to ensure the required dependency is installed to the system attempting to run the documentation server.
Reproduction steps

Fork and Clone this project for the first time
Attempt to run the make install_venv serve -C docs/mkdocs command, as described in the docs/README.md file
Expect this to fail if you do not have the plantuml MacOS/Debian/etc package installed previously on your system

Expected vs. actual results
If the docs suggest that all I need to do is run a make command to start the documentation server, I would expect that is all I need to do as a new contributor to the project. This caused some confusion and a potential waste of everyone's time trying to debug something minor.
Minimal code example
No response
Error messages
No response
Compiler and operating system
MacOS/Linux
Library version
development
Validation

[X] The bug also occurs if the latest version from the develop branch is used.
[ ] I can successfully compile and run the unit tests.

I think this can be fixed by adding a remark to the documentation.


Fix may be used unitialized warnings in _render_pass_create() and  _draw_list_insert_clear_region().

Also groups value validation to the beginning of the loop in _render_pass_create().
Fixes #38829
Note: This PR only silences the warning message. However, by using invalid values, the values can be checked to ensure they have been appropriately assigned before being used if required.
Edit: Added checks to ensure structs have been initialized properly.

Thanks!


chapter 4: object detection: ModuleNotFoundError: No module named 'cloud_tpu'

Hi @lakshmanok,
Firstly, thanks for releasing the book, it's really helpful to understand the concepts of vision applications. Currently, I'm working on object detection chapter. I have cloned the repository and trying to replicate the same. But unfortunately, I'm getting this error,
ModuleNotFoundError                       Traceback (most recent call last)
<ipython-input-3-e0ad62b2adca> in <module>()
     10 from official.vision.beta.configs import retinanet as retinanet_cfg
     11 from official.vision.beta.configs import backbones as backbones_cfg
---> 12 from official.vision.beta.serving import export_saved_model_lib
     13 from official.core import train_lib
     14

1 frames
/usr/local/lib/python3.7/dist-packages/official/vision/beta/serving/detection.py in <module>()
     18 import tensorflow as tf
     19
---> 20 from cloud_tpu.models.detection.utils import box_utils
     21 from official.vision.beta import configs
     22 from official.vision.beta.modeling import factory

ModuleNotFoundError: No module named 'cloud_tpu'

I have also tried "pip install cloud-tpu-client" but still its same.
Could you please help me.  I'm executing this "04ab_retinanet_arthropods_train.ipynb" file.
Even I have tried in the colab still its the same.

@takumiohym it worked thanks 😊


Adds <canvas> HTML element

Adds HTML <canvas> element
Note: also removes api.HTMLCanvasElement.transferControlToOffscreen from the Off Screen Canvas file because that property is part of the canvas element.

@ddbeck & @foolip - this one prob needs a little extra scrutiny for how I handled the transferToOffscreen method

@petele I think transferControlToOffscreen ought to be in the offscreen canvas feature. It shipped with OffscreenCanvas and is one way of getting a OffscreenCanvas instance, in addition to the OffscreenCanvas constructor.
It's quite common for an API to have small bits on interfaces that mostly belong to another feature in this way. For example, https://w3c.github.io/selection-api/ has window.getSelection() and document.getSelection(), and those make sense together, not in whatever feature we end up putting Window and Document in.

integrated feedback


Weston 9 compatible

Update the libweston-9 on ivi-id-agent cmake file. Update the libweston-9-protocols on simple-weston-client cmake file.
Signed-off-by: Tran Ba Khang(MS/EMC31-XC<EMAIL_ADDRESS>

This pull request works great.
Tested-by: Au Doan Ngoc<EMAIL_ADDRESS>


docs: quickstart fuel wallet integration

Current documentation uses the CLI wallet and hardcodes the wallet's secret.
Since Fuel Wallet is already out (as an alpha) should be included at least in the quickstart.

Please see #86


Implement RTPSender.ReplaceTrack

Resolves #1020

@lherman-cs @OrlandoCo @at-wat If you think this is an un-needed feature we can table it also!

@lherman-cs @OrlandoCo @at-wat If you think this is an un-needed feature we can table it also!

Well I think it can stay,  maybe add some warning that should be taken in account like it requires to sync the tracks to keep the SN and Timestamp, also waiting for a keyframe if it's video, so dont get many issues.
But with the rtp interceptor it maybe a nice solution for simulcast.


Find-PSResource Always Prints Warning for Nuget v3 Feeds

Tried version: 3.0.0-beta10
Hi,
in case I registered a nuget v3 feed from Azure DevOps, Find-PSResource always prints
"WARNING: This functionality is not yet implemented"
even the remaining functionality is given and the available resources are returned.

Thanks for calling this out @bormm , looks like the issue is occurring here:
https://github.com/PowerShell/PowerShellGet/blob/e562f86317bef8f9ae2d1e7547fb890525f7e799/src/FindPSResource.cs#L211

I do not think its a good idea to write "This functionality is not yet implemented" to output from somewhere in the code. What this "this" means is completely unknown for the user.


Remove "import Bitwise" to fix a warning

Allo 👋🏻 should we bump https://github.com/elixir-tesla/tesla/blob/30c3a298bc0de841d8929055429598342ef75573/mix.exs#LL13C7-L13C24 then?
I was about to do it at some point to deprecate the support for older versions

@yordis yes I think we can use compile_env and this one, bump the requirement to 1.10+, and release 1.7.0?


less 2.2.0

is there any plan to update the LessCss.php commands to Less 2.2.0?

We'd need to keep both less1 and less2, so it sounds like another filter would be needed.

Hi,
I'm just having a crack at this with Lessc 2.7.1, the existing LessCss.php filter just needs this _generateScript updated to use the new syntax.
I'm still using a LessPHP version which needs more ../ on includes, 3 to be exact. And Lessc doesn't want them, so this keeps the LESS compatible
$input = str_replace('../../../', '',  $input);

protected function _generateScript($file, $input) {

		//remove 3 ../ from here
		//..\/..\/..\/..\/..\/   -> ..\/..\/
		$input = str_replace('../../../', '',  $input);

		$text = <<<JS
var less = require('less'),
	util = require('util');

 var options = { paths: %s };
 var lessInput = %s;  //actual LESS code

less.render(lessInput, options)
    .then(function(output) {
            // output.css = string of css
            // output.map = string of sourcemap
            // output.imports = array of string filenames of the imports referenced

            util.print(output.css);
            process.exit(0);
        },
        function(error) {
            less.writeError(error);
            process.exit(1)
        });

JS;
		file_put_contents($file, sprintf($text, str_replace('\/*', '', json_encode($this->_settings['paths'])), json_encode($input)));
	}


[Question] Is it possible to access variables in callbacks?

I have been trying to implement this plugin in my app and so far it's very helpful. But I'm trying to make a geofence solution with it and that's where the problems arise.
So far it looks like both initCallback and callback can access other static variables. But after background locator is registered it can't read the variable's updated value.
example:
static int number = 0
static string uid
...
static _initCallback(Map<dynamic, dynamic> params) async {
   _uid = params["uid"];
  print(number) //will always print 0
 }

static _callback(LocationDto locationDto) {
  final SendPort sendPort = IsolateNameServer.lookupPortByName(_isolateName);
  sendPort?.send(locationDto);
  print(number); //will also always print 0
}

BackgroundLocator.registerLocationUpdate(
  _callback,
  initCallback: _initCallback,
  initDataCallback: {"uid": _user.id},
  disposeCallback: _disposeCallback,
  settings: settings.LocationSettings(
      accuracy: settings.LocationAccuracy.HIGH,
      notificationTitle: "Ride Partner",
      notificationMsg: "Running in Background. Searching for orders",
      wakeLockTime: 1,
      autoStop: false,
      distanceFilter: 0.1,
     interval: 5
  ),
);

number = 1;

Also, like how uid is initiated via initDataCallback, every time I need to register a variable do I need to unregister the service then reregister again or is there a more elegant way of registering a variable?

I think you should use shared preferences

I think you should use shared preferences

is it a good practice to call SharedPreferences.getInstance() every time callback fires?

Which version are you using?
Have you tried doing it as the example says ("location_callback_handler.dart" and "location_service_repository.dart")?

Are you referring to the setting variable from initCallback method? if yes, then you can see it in my code that I have tried it

No. in the example use a whole container class (facade) to execute the callbacks

Tried that but the problem remains. The variables are okay as long as the main thread is on. But when it's restarted everything goes back to null. Looks like the only option is to write to a file and read from it which the example app does too for saving the location logs. But isn't calling await getTemporaryDirectory() expensive?

Update: Tested again with repository method and it actually works. The problem isn't with any of this. I have observed a strange behaviour. If you open and close the main app it's ok BUT if you hot restart once or twice it goes back to null

What device are you using? you might have a power / battery gestures that limit the use of apps in the background (i think huawei has that pre-installed)

Tested on both Asus Zenfone Max Pro M1 (X00TD) and OnePlus 8 and both time hot restarting once or twice wipes the variables of the repository

Right now I'm solving this issue by not enabling restart on boot. Using the repository method to store variables and callbacks and also using local files to save data that needs to be persisted across boot.

I think currently this is the only solution. Closing issue.

Right now I think the best solution is to store you variable into sharePreferences. please take a look at #91 If you decided to use SharedPreferences plugin.

@mehdok  Thank you for the pointer. I didn't know you cloud reload everything from filesystem to memory with shared preferences. My custom solution is in theory like that but there's also managing files and saving the location of those etc. In contrast this'll be much easier.
Although I have one question. Can you use the repository to save SharedPreferences instance from await SharedPreferences.getInstance() in initcallback as a variable so you don't have to call getInstance() on every callback?

@DebkanchanSamadder Honestly I don't know if it is possible:) The initcallback is just for primitive types, but You can test that. But I use the sharedPreferences this way and each time I call SharedPreferences.getInstance(), I think the overhead of calling this is very little;

Thanks @mehdok! I refrained from setting any variables from inside location service repository because it doesn't play nicely with hot restart, basically it wipes those variables. This issue might be more deep rooted so I avoided it totally. Using SharedPreferences.getInstance() and SharedPreferences.reload() worked like charm. In fact it worked so good I think it should be the official way of sharing data and should be in documentation.

Glad to hear that your problem is solved;


GroupBy stucks and never return

For bugs, please provide the following:
What's going wrong?
GroupBy stucks in some query, I didn't figure out the pattern. Let me describe my data and my query. I have a packet index with 5 fields: src_ip, dst_ip, src_mac ,dst_mac, dummy. They are all time field. When I do GroupBy(Rows(src_ip), Rows(dst_ip),Rows(src_mac), Rows(dst_mac), filter=Row(dummy=dummy, from=start_time, to=end_time)), if the filter query is empty, then the GroupBy just stucks, never return.And if I change the start_time and end_time to make the filter query not empty, the GroupBy returns the result.GroupBy(Rows(src_ip), Rows(dst_ip),Rows(src_mac), Rows(dst_mac)) also works normally.It seems that this has something to do with if filter query returns empty.
But, if I do GroupBy(Rows(src_ip), Rows(src_mac)) no matter if I add filter query or if filter query return empty, this works fine. So it seem that this also has something to do with the number of Rows() queries.
In another situation I met before, Changing the order of Rows() can solve the problem.For example,GroupBy(Rows(src_ip), Rows(src_mac)) stucks, but GroupBy(Rows(src_mac), Rows(src_ip)) works.
So, the problem seems to have something to do with the filter query result, number of Rows() and the order of Rows().
And I'm sure it is not slow, it just stucks, like falling into an infinite loop, because the data is very small, I have only 200 columns.And when it stucks, the CPU usage keeps high
What was expected?
Please tell me if it is a bug, or what I did wrong?
Steps to reproduce the behavior
Information about your environment (OS/architecture, CPU, RAM, cluster/solo, configuration, etc.)
ubuntu 18.04
4 core
8G
solo
default configuration
I use docker to deploy pilosa, image: pilosa/pilosa:latest
For feature requests, please provide the following:
Description
Success criteria (What criteria will consider this ticket closeable?)

It seems a bug, I change latest to master, and this problem disappear.

The problem showed up again, I  can not figure out the pattern, but it seems a infinite loop bug in GroupBy operation, with time field filter.

why i performe Rows() method, the error appear, following:
"localhost:10101/index/user/query" -X POST -d 'Rows(age)'
error: {"error":"parsing: parsing: \nparse error near IDENT (line 1 symbol 6 - line 1 symbol 9):\n"age"\n"}
curl "localhost:10101/index/user/query" -X POST -d 'Rows(field="age")'
error: {"error":"executing: map reduce: unknown call: Rows"}
hwo can i use it correctly?
please help


ui for listVolumes by isEncrypted

ISSUE TYPE

Enhancement Request

COMPONENT NAME

CLOUDSTACK VERSION

CONFIGURATION

OS / ENVIRONMENT

SUMMARY

see https://github.com/apache/cloudstack/pull/8643#issuecomment-2129386421
STEPS TO REPRODUCE

EXPECTED RESULTS

ACTUAL RESULTS

backend change is still pending


Remove docs/ since on docs.microsoft.com or stale

docs.microsoft.com docs found here: https://github.com/MicrosoftDocs/xamarin-docs/tree/live/docs/tools/dotnet-embedding

I diffed all of the major pages as I went, but I can generate gists if you want @spouliot .
I'll have to wait for tomorrow due to testing today.

The images are all identical https://gist.github.com/chamons/f4471beb1e607c947f865ca5176d8ba8
Except for weather-from-csharp-macos.png which was unused after my docs rewrite and not removed.
BestPracticesObjC.md - https://git.io/vxjTB
Limitations.md - https://git.io/vxjTK
ObjC.md - https://git.io/vxjTd
android-callbacks.md - https://git.io/vxjTp
errors.md - https://git.io/vxjkJ
getting-started-c.md - https://git.io/vxjkt
getting-started-java-android.md - https://git.io/vxjkG
getting-started-java.md - https://git.io/vxjkl
getting-started-objc-ios.md - https://git.io/vxjkV
getting-started-objc-macos.md - https://git.io/vxjki
getting-started-objective-c.md - https://git.io/vxjky
index.md - https://git.io/vxjkb
objc-platforms.md - https://git.io/vxjIv


pandas.Index.to_numpy() method's arguments not properly typed

Describe the bug
The pandas.Index.to_numpy() method takes arguments, but mypy complains about them, whether they are positional or keyword.
To Reproduce
import pandas as pd
df = pd.DataFrame({"A": [1, 2, 3], "B": [4, 5, 6]})
array = df.index.to_numpy(dtype="int32")
This yields error: Unexpected keyword argument "dtype" for "to_numpy" of "Index"
import pandas as pd
df = pd.DataFrame({"A": [1, 2, 3], "B": [4, 5, 6]})
array = df.index.to_numpy(dtype="int32")
This yields error: Too many arguments for "to_numpy" of "Index
Please complete the following information:

OS: Windows
OS Version 10
python version 3.10
version of type checker: latest
version of installed pandas-stubs<IP_ADDRESS>829

Thanks for the report.
Just need to fix:  https://github.com/pandas-dev/pandas-stubs/blob/80f561fc73f387f81f3876eeef4fe3bae82dae18/pandas-stubs/core/indexes/base.pyi#L207
PR and tests welcome!

What about something like:
from typing import Any, Optional
from pandas._typing import npt
import pandas._libs.lib as lib

    def to_numpy(
            self, dtype: npt.DTypeLike | None = None,
            copy: bool = False,
            na_value: Optional[Any] = lib.no_default,
            **kwargs
    ) -> np.ndarray: ...

If good enough I can send the PR.

Thanks guys.  Also, FYI, it is probably worth investigating whether other the pandas objects that have to_numpy() methods are missing these, so that you can create similar annotations for them.

If good enough I can send the PR.

For a stub, you want to have = ... rather than specify the default arguments.
If you can make that change there, and as suggested above, look at the other to_numpy() calls in pandas-stubs, and also add tests, that would be a welcome PR!

You can also run stubgen against pandas main to get the up to date list of
parameters and often correct type info.
On Wed, Aug 31, 2022, 14:03 Irv Lustig @.***> wrote:

If good enough I can send the PR.
For a stub, you want to have = ... rather than specify the default
arguments.
If you can make that change there, and as suggested above, look at the
other to_numpy() calls in pandas-stubs, and also add tests, that would be
a welcome PR!
—
Reply to this email directly, view it on GitHub
https://github.com/pandas-dev/pandas-stubs/issues/237#issuecomment-1232910900,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABKTSROXK77UBBQ66CFBTPDV35J3NANCNFSM6AAAAAAQAWT4DQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Very good catch. According to grep, there are many places where the fix should be applied:
>> grep -Hrnis "to_numpy" pandas-stubs/
pandas-stubs/_libs/tslibs/nattype.pyi:25:    def to_numpy(
pandas-stubs/_libs/tslibs/timedeltas.pyi:147:    def to_numpy(self) -> np.timedelta64: ...
pandas-stubs/_libs/tslibs/timestamps.pyi:203:    def to_numpy(
pandas-stubs/core/indexes/base.pyi:212:    def to_numpy(
pandas-stubs/core/series.pyi:1626:    def to_numpy(
pandas-stubs/core/arrays/masked.pyi:14:    def to_numpy(self, dtype=..., copy=..., na_value: Scalar = ...): ...
pandas-stubs/core/arrays/numpy_.pyi:112:    def to_numpy(self, dtype=..., copy: bool = ..., na_value=...): ...
pandas-stubs/core/arrays/base.pyi:15:    def to_numpy(self, dtype=..., copy: bool = ..., na_value=...): ...
pandas-stubs/core/frame.pyi:221:    def to_numpy(
pandas-stubs/core/base.pyi:44:    def to_numpy(self) -> np.ndarray: ...

If you guys agree that the type signature should be the same everywhere, I can work on it and send a PR in the next couple of hours.

If you guys agree that the type signature should be the same everywhere, I can work on it and send a PR in the next couple of hours.

It definitely should not be the same everywhere, especially with respect to the type of the return values.  Many of the cases you indicate above are already typed.  You would need to look at the pandas sources, as well as the docs, to see where things are the same or different.
The argument types that are shared (dtype and copy and na_value) should be consistent.  Right now, I don't believe that is the case.  Also, you should take a look at each one and determine what the right values should be based on what is inside of pandas, and what the docs say.
For any signature that you update, you should also add a test.
Thanks for your desire to work on this!

The argument types that are shared (dtype and copy and na_value) should be consistent. Right now, I don't believe that is the case.

Indeed, that is not the case:

for the dtype parameter: sometimes np.dtype, sometimes npt.DTypeLike, sometimes DtypeNp,  Dtype. I went for npt.DTypeLike
for the na_value parameter: sometimes object sometimes Scalar. I went for Scalar

For any signature that you update, you should also add a test.

Do you mind please, giving more indications ? For example, I see
def test_types_to_numpy() -> None:
    df = pd.DataFrame(data={"col1": [1, 1, 2], "col2": [3, 4, 5]})
    df.to_numpy()
    df.to_numpy(dtype="str", copy=True)
    # na_value param was added in 1.1.0 https://pandas.pydata.org/docs/whatsnew/v1.1.0.html
    df.to_numpy(na_value=0)

No call to assert_type ?
Correct me if I'm wrong please, I should add similar functions:

for indexes, create an index, call to_numpy and assert the type
for series, create a serie, call to_numpy and assert the type

Also. It seems like there is an addition here. Should I include the addition in the current PR or create a different one ?
Thanks for your time, advices and suggestions o/

With respect to the tests not having check(assert_type(...)), that's a legacy issue.  The original stubs didn't have them, but when we are adding new tests, we are requiring it.  Eventually, all the tests should use that paradigm.  Would be good if you fix all the to_numpy() tests to use that pattern.

Correct me if I'm wrong please, I should add similar functions:

for indexes, create an index, call to_numpy and assert the type
for series, create a serie, call to_numpy and assert the type

Yes, you should add similar functions for all the to_numpy() variants.

Also. It seems like there is an addition here. Should I include the addition in the current PR or create a different one ?

You can do one PR, that includes fixes to all the to_numpy() stubs, and the appropriate tests.
Thanks in advance for your contribution!

closed via #270


What if the training dataset is too large to fit into memory?

@dnouri
@cancan101
Hey, guys. Thanks for sharing your ideas and bring endless contribution to nolearn community. I have a simple question when I am using nolearn.lasagne.NeuralNet() to do kaggle competition. After we construct a neural network architecture by calling NeuralNet() function, we then train the network by function network.fit(X_train, y_train). For small datasets such as MNIST, this is O.K. However, when the data size is too large so that it can not be loaded into memory one time, what should we do? Thank you for your tips.

@paipai880429 One way is to call fit() multiple times.  Another one is to only put ids (or filenames) into X and then load the actual data based on these ids from disk in your batch iterator.

@dnouri Thank you for your timely reply. It seems that the former is more straightforward. By the way,  the document for nolearn in the index page(https://pythonhosted.org/nolearn/) seems not a detailed and latest version. Will any plans for a document update ?

Yes, you're right.  nolearn.lasagne sadly has no documentation yet.  I'll be hopefully working on that soon.

@dnouri Dainel, would you please  explain Another one is to only put ids (or filenames) into X and then load the actual data based on these ids from disk in your batch iterator. in details? Are you suggesting overriding BatchIterator parameterized with ````file ID list``?

@dnouri if we call fit() multiple times won't it recreate validation set 20%, I am working on a project and I have divided the dataset into chunks of say 15k images and a total of 19 chunks, I have a separate validation set of say 17k images, Now if I call .fit() multiple times, it is creating a separate validation set each time 20% and concept of  epoch is also wrong as the term epoch is used for the whole training_set, Please clarify on this


Parse 2.0 macro syntax

The current syntax looks like this.
pub macro run_passes($tcx:ident, $mir:ident, $def_id:ident, $suite_index:expr; $($pass:expr,)*) {{
    /* same as usual */
}}

I implemented the bare minimum to pass tests in 500d832545075a1f2ffbb07997daa766ac9b38da, will need to revisit what these look like in the AST.

I think that your implementation is fine for now. We can come back to it & clean up the AST once the AST used by rustc stabilizes some more.

I kept the AST but restricted the grammar in e0824038665c8f04dc677e906fa220426d356d6f -- the arguments must be parenthesized and the body must be braced.


YAPKMS - Yet Another PKM Space

Experience
YAPKMS.zip
Title
YAPKMS - Yet Another PKM Space
Screenshots

Description
YAPKMS is an adaptation of the Second Brain's concept so that it can be used daily.
It's meant to be easy to use, yet, as organized as you want it to be. Remember, there's no 'one size fits all'.
Categories
Dashboard, Work, Education, Project tracking, Health & Fitness, Finance, Food & Nutrition, Travel, Hobbies, Home
Language
en-US
Terms

[X] I agree that this Experience is licensed under the MIT License.
[X] I have intellectual property rights and/or permission to contribute the Experience and its content.
[X] I confirm that the Experience does not contain any harmful or malicious content.

Well, that was overwhelming... Do i actually need to do something? I see alot of info that shouldn't be in there. Some deleted stuff appears there. Weird.

Well, that was overwhelming... Do i actually need to do something? I see alot of info that shouldn't be in there. Some deleted stuff appears there. Weird.

Thank you for uploading your experience! Let us take a look into the issues found by validator. If we need your assistance, we'll ask you to re-export space one more time

Well, that was overwhelming... Do i actually need to do something? I see alot of info that shouldn't be in there. Some deleted stuff appears there. Weird.

Thank you for uploading your experience! Let us take a look into the issues found by validator. If we need your assistance, we'll ask you to re-export space one more time

Thank you. I need to make a change to my 'experience', but i need to wait for the issue with deleted objects to be resolved.

Is it possible for me to update the current file still?
I made some important changes.
Oh, and btw, will it possible to keep my file updated after submiting?

@telmob sorry for really prolonged reply. The problem was connected with relation options (tag-like objects) that remained in space despite the very relation was deleted. This bug is already fixed, so I have cleaned your archive from these objects

@telmob sorry for really prolonged reply. The problem was connected with relation options (tag-like objects) that remained in space despite the very relation was deleted. This bug is already fixed, so I have cleaned your archive from these objects

No worries. Thank you :)

/gallery approve

/gallery approve

Not wishing to be picky, but would it be possible to have the first image as cover for the Space in Gallery? It's looking weird.


Update Tutorials on how to conduct product Analysis

Describe your changes
Create an tutorial md about how to compute product metrics
Issue number
Closes #264
Checklist before requesting a review

[x] Performed a self-review of my code
[x] Formatted my code with pkgmt format
[x] Added tests (when necessary).
[x] Added documentation in the docstring and changelog (when needed)

:books: Documentation preview :books:: https://jupysql--321.org.readthedocs.build/en/321/

let me know when this is ready for review!

any updates @chuwpeng123 ?

any updates @chuwpeng123 ?

I built locally and the file looks just fine. However, the DB integration test keeps failing. I am stucked there. Can you take a look at the error message for me?

merge or rebase from master, let's see if that fixed the integration tests. let me know

Thank you for the suggest! It works!


Plasma tornadoes should kill the player

Feature
It'd be cool if tornadoes with fluidType set to 'plasma' kill the player on contact, like funnels do when type is set to 'star' or 'lava'
Context
No response

I would not have these fluid types kill the player by default, but having a separate parameter that makes the tornados into hazard volumes would be good.


Use Knife's cooldown instead of a timed attack.

Title, makes it way faster and reliable.

Can you explain how that makes it faster and more reliable? From the tests i did the hits wouldn't count anymore if i would hit only a little bit faster since the delay. Haven't tested it much online tho.

Can you explain how that makes it faster and more reliable? From the tests i did the hits wouldn't count anymore if i would hit only a little bit faster since the delay. Haven't tested it much online tho.

it relies on how a client set the cooldown, not just  a hardcoded one.

With bypass you mean hit faster while the hits are also working?

With bypass you mean hit faster while the hits are also working?

yeah, you can set it by patching HitKnife and setting timeAtLastDamageDealt to 0, which removes effectively the knife cooldown.

Yea, have already tried that but i would just count that as cheating. I mean a auto clicker is technically cheating as well but swinging at certain intervals is something everybody can do

I'm going to close this for now since i don't see any reason to change this yet.


Updating references in the documentation

Updated the docs to include links to the Transformers4Rec paper at the ACM Digital Library, to the new post about Transformers4Rec and to the session-based recommendation page at NVIDIA Developer

@benfred Can we have the docs updated github.io to reflect those changes?

Pushed out a new version of the docs to https://nvidia-merlin.github.io/Transformers4Rec/main/
(I think Marc has a PR out to automate the docs push)


Support control messages

At the very least, we should support CLOSE_NOTIFY. Ideally we'd provide ways to send/receive other messages as well.

This is important because the tests are currently ignoring an "unexpected EOF error" when reading from the server (the TCP socket is closed without sending a CLOSE_NOTIFY and rustls is unhappy about that)

Right now my thinking is: we should return a wrapper type (KtlsStream<IO>) that implements AsyncRead / AsyncWrite / AsRawFd, and that also sends a CLOSE_NOTIFY in poll_shutdown. Maybe on Drop too? Although this can probably block:
https://github.com/quininer/ktls/blob/a97dcd4727d71aa7b848e2ed75d590af81eb0ba2/src/sys/mod.rs#L64-L89
(but does it block waiting for a buffer, or waiting for the peer?)


Alternative INVOKE implementation

This alternative INVOKE implementation takes full advantage of C++11 features. The old implementation is very conservative, carefully designed to work with compilers lacking full expression SFINAE support (specifically anything mem-ptr related). Now that those compilers are no longer targeted we can have a more straightforward implementation, and one that fully supports member pointers (the old implementation lacks support for ref-qualified mem-funs, noexcept mem-funs, C-style variadics mem-funs, ...).
As a bonus, this straightforward implementation compiles in about half the time for msvc.

@K-ballo would you mind summarizing the main differences from the old implementation?

retest cscs

The gcc builders all seem to have problems. Do you see the results?

I see them, but I'm unable to find the full build log with the complete error messages.

Would you need any more context for those, or one of us to look into them?

I am unable to reproduce on GCC 8.3.1. What other factors may be contributing?

The gcc builders all seem to have problems. Do you see the results?

I see them, but I'm unable to find the full build log with the complete error messages.

Yeah, this is unfortunately as much as we get with CDash...

Would you need any more context for those, or one of us to look into them?

I am unable to reproduce on GCC 8.3.1. What other factors may be contributing?

I'll check if I can reproduce locally and get you a full log.

@K-ballo here's slightly more complete log of the errors: https://gist.github.com/msimberg/a7f0ceb2d27da6c4f73db8b5a5da07a0. It specifically seems to be pointers to functions that the hpx::function constructor doesn't seem to like anymore, but I don't know why. Oddly enough the CUDA build is with gcc 8.3 as well. Both are with C++14 but I'm not sure that makes a difference.

I have been able to reproduce the errors using Rostam's GCC 8.3.1 20191121 (one of those Red Hat dev builds). It's definitely a bug in the compiler, as initialization works fine for a variable instead of a parameter. However the compiler doesn't trip an all fun-ptr to function conversions, I'll try to figure out what the pattern is rather than apply a dumb workaround.

I found a workaround for Red Hat's GCC dev-build, had to revert some aliases back into structs.
@msimberg Please let me know if there are any genuine check failures.

Thank you @K-ballo! Could I just ask you to rebase once more on master? I just merged #4954 which fixes most of the unrelated build failures.


Update contribute-to-an-extension.md to ensure CHANGELOG.md is updated or created

Description
When updating an extension, I discovered after publishing that the CI failed because I didn't update the CHANGELOG.md file.
Before merging this PR, please consider that 103 out of 1520 extensions in this repository do not contain a CHANGELOG.md file.
In resopnse to:

Do you mind opening this PR to main branch (to the folder docs) instead, if we change it in gh-pages then it will be overridden next time anyone adds new content in main 🙂

Can we maybe point out that you always need a CHANGELOG entry, if it's not there you should create it

on https://github.com/raycast/extensions/pull/12407
ping @pernielsentikaer
Screencast
N/A
Checklist
N/A

[ ] I read the extension guidelines
[ ] I read the documentation about publishing
[ ] I ran npm run build and tested this distribution build in Raycast
[ ] I checked that files in the assets folder are used by the extension itself
[ ] I checked that assets used by the README are placed outside of the metadata folder

Thanks @luarmr


investigate: ecdsa created signatures do not validate with coincurve

Investigate why old signatures created using ecdsa do not validate with coincurve:
Old signing code (private_key in this case is instance of ecdsa.SigningKey):
https://github.com/lbryio/lbry-sdk/blob/26d0e87f463cdede9a31468ddcf397351a1aa397/lbry/wallet/transaction.py#L459
New signing code (calls the libsecp256k1 code wrapped by coincurve):
https://github.com/lbryio/lbry-sdk/blob/8076000c2761d7ca077049f1e9e1de177553d6f0/lbry/wallet/bip32.py#L238-L258
New validation code includes the old validation as fallback for old signatures:
https://github.com/lbryio/lbry-sdk/blob/8076000c2761d7ca077049f1e9e1de177553d6f0/lbry/wallet/transaction.py#L428-L440
The following tests illustrate the problem, if you step through debugger you will see that it does not validate with coincurve and instead falls back to using the old validation with cryptography:
Example 1 (older):
https://github.com/lbryio/lbry-sdk/blob/8076000c2761d7ca077049f1e9e1de177553d6f0/tests/unit/wallet/test_schema_signing.py#L74
Example 2 (much more recent):
https://github.com/lbryio/lbry-sdk/blob/8076000c2761d7ca077049f1e9e1de177553d6f0/tests/unit/wallet/test_schema_signing.py#L123
Why can't coincurve validate those signatures? Is there anything we can do to get it to validate?
One interesting thing to note is that the new signatures do validate with the old validation scheme using cryptography library.

posted to stackoverflow
https://stackoverflow.com/questions/70477905/why-are-signatures-created-with-ecdsa-python-library-not-valid-with-coincurve


[POPUPS] Popup fields order gets changed after clicking a new field

Create a point map
Go to popup
Select a field
Move that field to the first position
Click on another field

The order of fields get changed.

It didn't happen to me on hover, but happened to me on click. Not sure if this is a normal rule

submitted the PR https://github.com/CartoDB/cartodb/pull/8192 and reported a bug in the current editor https://github.com/CartoDB/cartodb/issues/8191

should we close this @matallo ?


feat: refactor ElasticsearchProxy queries

Summary of Changes
Minor non-breaking refactor making it easier to use ElasticsearchProxy with custom queries.
Tests
n/a
Documentation
n/a
CheckList
Make sure you have checked all steps below to ensure a timely review.

[x] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"

In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

[x] PR includes a summary of changes.
[x] PR adds unit tests, updates existing unit tests, OR documents why no test additions or modifications are needed.
[x] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain docstrings that explain what it does

cc @dechoma


Is there a date picker for meteor ionic?

I found most of them depend on angular.js.

Would be nice with something extensible! For example the dates could be set in different color depending on some collection data etc.

+1

On a device <input type="date"> will trigger the native datepicker, which is I believe is why Ionic doesn't include their own. I also don't think we want to include any components in meteoric that aren't in Ionic.


Login Error - Generic Request Error

I have tried 2 accounts and get this when I try to log in:
instabot@instabot:~/InstaBot$ python run.py

        InstaBot 1.3.0 by nickpettican
        Automate your Instagram activity

        New features:
        - Added timezone as config parameter
        - Removed arrow as dependency

[ 2019-01-15 23:01:49 ] Trying to sign in as xxxxx:
[ 2019-01-15 23:01:53 ] Error signing in. Request returns 200 but it seems it didn't work.

Instagram error message: None

Error type: generic_request_error

Check you entered the correct details!

Strange, did you manage to sort something out?

Nope, gave up. :(

i have the same error


fix: remove duplicated import from keras in vgg files

Too many changes in one PR; in the future, please keep just one issue per PR.  I have fixed the duplicate import in a separate commit due to no response.


Is it possible to pipe an "incoming" SELECT stream into an "outgoing" INSERT stream?

I'm in need of:

Streaming the output of a SELECT to Haskell
Mapping over the stream in Haskell
Piping the mapped stream into an INSERT

Is this possible? If so, how do I do it?
If no, please mark this issue as a feature request.

You probably want https://hackage.haskell.org/package/streaming-postgresql-simple-<IP_ADDRESS>/docs/Database-PostgreSQL-Simple-Streaming.html#v:copyIn. It's been a long time since I worked on this library though so that's probably the most help I can provide initially.

@ocharles thank you for the advice.
For what it's worth, I ended up using fold from postgresql-simple: https://hackage.haskell.org/package/postgresql-simple-0.6.5/docs/Database-PostgreSQL-Simple.html#v:fold
The copyIn functions seems too low level for my needs (streaming raw ByteStrings without parsing into rows).

If you are ok either collecting all rows and doing a single insert or doing one insert per row that will work well, yep


GitHub Actions: Install netbeans on Mac

Brief summary of changes
TODO
CHANGELOG.md (choose one)

no need to update because...not user-facing

@aymanhab I think the issue is that NetBeans is finding JDK 14, not JDK 8.

Ready for review.

@chrisdembia Do you think now is a good time to stop building on travis so we have one place to maintain?

@chrisdembia Do you think now is a good time to stop building on travis so we have one place to maintain?

As soon as the builds are green for all three OSes, then yes. It seems like this is indeed the case.

@aymanhab I've gone ahead and turned off the Travis-CI and AppVeyor continuous integration. I suggest we leave the related CI scripts in the repo in case we want to fall back on these services.

Merged, thanks @chrisdembia


[T3A5][W13-A1] Lim Hui Qi

Apply Single Responsiblity Principle

Ready to review

good job on this :)


Add PK ordering

Did you check the reason for the failing tests?

@xkubov Seems to be some Dynamo related problem (not related to this PR). I'll check the cause within task dedicated to solve failing Dynamo tests in core.


bin/chpasswd.sh script isa not wrking

Description
In MI the chpasswd.sh script is failing with the attached error even after installing Ant.
Buildfile: MI_HOME/bin/build.xml
[taskdef] Could not load definitions from resource net/sf/antcontrib/antlib.xml. It could not be found.

setup:
   [copy] Copying 28 files to MI_HOME/wso2/lib
   [mkdir] Created dir: ] MI_HOME/wso2/tmp/setup
   [delete] Deleting directory MI_HOME/wso2/tmp/setup
   [move] Moving 59 files to MI_HOME/wso2/lib

BUILD SUCCESSFUL
Total time: 0 seconds
Error: Could not find or load main class org.wso2.carbon.core.util.PasswordUpdater

Steps to Reproduce

Install the ant
Then navigate to the MI_HOME/bin using terminal
Execute the below command
sh chpasswd.sh -U admin -P admin
Then response as below

Affected Component
MI
Version
4.1.0
Environment Details (with versions)
No response
Relevant Log Output
No response
Related Issues
No response
Suggested Labels
No response

Public fixes:
https://github.com/wso2/product-ei/pull/5592
https://github.com/wso2/micro-integrator/pull/2901
have already been into the master branch.


Use Azure Blob Storage for image upload

I am working on the image upload function. Technically, this is very straightforward, and I will start with Azure Blob Storage.
There are some open questions, which might require some more polishing:

Do we take the filename of the uploaded file one-to-one?
If so - that could lead to conflicts. Therefore: Should we just add some small random hash at the end of each file? Does that have implications on SEO?
Do we just use a random guide and basically never run into a collision?

My preferred option is right now: (2)
So basically, if a user uploads a file via Drag 'n Drop, we take that filename and append a random 6 to 8 digit hash at the end to avoid any conflicts.
@ncosentino Do you have any preference here?

My use case, when I migrate, will be that I want to enforce 1:1 naming if
possible in my URLs. I'm not actually convinced I need to keep image URLs
the same to be honest, but it's a fear of mine that I might have something
out there expecting an image at some path and it'll be broken.
(Might be just a false fear I have)
I'm fine with two, generally. Any opportunity for the new guids with some
time element in them?
On Tue, Nov 5, 2024, 2:05 PM Steven Giesel @.***> wrote:

I am working on the image upload function. Technically, this is very
straightforward, and I will start with Azure Blob Storage.
There are some open questions, which might require some more polishing:

Do we take the filename of the uploaded file one-to-one?
If so - that could lead to conflicts. Therefore: Should we just add
some small random hash at the end of each file? Does that have implications
on SEO?
Do we just use a random guide and basically never run into a
collision?

My preferred option is right now: (2)
So basically, if a user uploads a file via Drag 'n Drop, we take that
filename and append a random 6 to 8 digit hash at the end to avoid any
conflicts.
@ncosentino https://github.com/ncosentino Do you have any preference
here?
—
Reply to this email directly, view it on GitHub
https://github.com/linkdotnet/Blog/issues/364, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABHOBADE7V7ZV7JC4S3ZDNLZ7E6KJAVCNFSM6AAAAABRHQLIKGVHI2DSMVQWIX3LMV43ASLTON2WKOZSGYZTMNJWG4YDIOI
.
You are receiving this because you were mentioned.Message ID:
@.***>

I do think I have a version that will gives all the flexibility you want:

Basically it allows you to define the whole path. While kind of an MVP, it circumnavigate a whole lot of edge cases.
While "labour" - it covers your case.

Looks awesome!
On Wed, Nov 6, 2024, 8:05 AM Steven Giesel @.***> wrote:

I do think I have a version that will gives all the flexibility you want:
Feature.gif (view on web)
https://github.com/user-attachments/assets/2a839eb4-cb9f-4f1f-b570-5d80e9ce4abe
Basically it allows you to define the whole path. While kind of an MVP, it
circumnavigate a whole lot of edge cases.
While "labour" - it covers your case.
—
Reply to this email directly, view it on GitHub
https://github.com/linkdotnet/Blog/issues/364#issuecomment-2460178462,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABHOBAFIMOVY2VPY2P6M3D3Z7I44ZAVCNFSM6AAAAABRHQLIKGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINRQGE3TQNBWGI
.
You are receiving this because you were mentioned.Message ID:
@.***>

@ncosentino
I merged the feature. It is basically the version above with some extras:

Cache-control header can be automatically set to a week (for SEO).
Optionally, an Azure CDN can be configured!

The basic setup looks like this: Adapt the following in your appsettings.json:
{
  ...
  "ImageStorageProvider": "Azure",
  "ImageStorage": {
    "AuthenticationMode": "Default", // Can Also be "ConnectionString"
    "ConnectionString": "", // If AuthMode is ConnectionString, here should be the ConnectionString from Azure
    "ServiceUrl": "",  // The Service Url - only used when AuthMode is Default
    "ContainerName": "" // Name of the container
  }
}

And there you go. The whole documentation can be found here: Media Upload


Expose String.get_slice

I wanted to use this method today and was actually surprised it's not exposed.
Although maybe it's not that useful outside core? 🤔 idk, already made a PR

Thanks!

Cherry-picked for 3.5.


How could I get cookie in webview which is set in react-native?

Bug description:
I have fetch a request in react-native, then the server response set a cookie named jssession.
When I send http request in webview, the request header does't contain  the cookie jssessionid.
To Reproduce:

Expected behavior:

If cookie is set in one side (react-native or webview), the other(webview or react-native) will share the same cookie.
Screenshots/Videos:

Environment:

OS: Android and IOS
react-native version: 0.59.10
react-native-webview version:5.12.1

Your react-native-webview version is quite old? There were a huge changes since them, i.e. favoring WKWebView instead of UIWebView.
Which is your iOS version where you experienced your problem?
I have an issue with iOS11 and cookies in webview. They just do not get restored and I get logged out in webview.

Yes, 5.5.0 which I am using.
I am trying to upgrade to latest version now, but there's lots of error when I upgrade react-native to 0.60.x.

+1 cookie problem!

I don't have that issue, since I can login to gmail and other sites and even after I close the App and re-enter I am still logged on or my cart has the items added (even if I'm not logged in).

@canaanites I'm not directly manipulating site cookies, however doing API calls that require session information seems to work, and backend is able to retrieve Session information successfully.

The solution for this is to use react-native-cookies. iOS stores session & http-only cookies on the device and removes them from the response headers. https://github.com/joeferraro/react-native-cookies

Still searching for a solution


Make all addresses clickable using generic address link component

What kind of PR is this?:

/kind feature
What this PR does / why we need it:
Which issue(s) does this PR fixes?:

Fixes #494
Sample Links & Screenshots:

Link:

Desktop Screenshot

Mobile Screenshot

Additional comments?:
Developer Checklist:

[X] Read your code changes at least once
[X] Tested on multiple web browsers
[X] Tested responsiveness (e.g, iPhone, iPad, Desktop)
[X] No console errors
[X] Unit tests*
[X] Added e2e tests*

Made some changes:

Added href attributes test to addresses to ensure that they are linked
Fixed ICXSubmitEXTHTLC


Make MegaStoreProvider higher in root components

No totally sure what I am doing here but I think this could help.
When MegaStoreProvider is mounted it creates an instance of mutiny-node, so if any parent components are changed it would too. So I put the I18nProvider inside of it incase that is changing as well as a second ErrorBoundary that will handle the Router, so if any uncaught errors happen there, we don't create a new instance of mutiny-node.
ChatGPT said it might help

The additional ErrorBoundary seems to be the change pulling the weight, when trying to induce a failure I could not cause a i18n error whether the I18nProvider was in the MegaStoreProvider or not. otherwise tACK on the change, errors now direct to a correct error page instead of the app failing

I did notice there is something wrong with ErrorDisplay.tsx the emergency kit link is incorrect, needs to be "/settings/emergencykit".
figure its small enough to push along with this
<NiceP>
    {i18n.t("error.general.getting_desperate")}{" "}
    {i18n.t("error.emergency_link")}
</NiceP>

I did notice there is something wrong with ErrorDisplay.tsx the emergency kit link is incorrect, needs to be "/settings/emergencykit". figure its small enough to push along with this
<NiceP>
    {i18n.t("error.general.getting_desperate")}{" "}
    {i18n.t("error.emergency_link")}
</NiceP>

I think this fixes an issue someone has seen with an emergency page last week


Fahrenheit

Hi, can you add support to have temperature measured in Fahrenheit?

Any pointers on how to change this to Fahrenheit?


How do I set up wsl audio config?

Hello
I'm trying to log in to x with xrdp and record a sound.
How do I set up wsl audio config?
Thanks, Good-day everyone.

Are you able to play audio from an application running inside WSL? If yes, the program is connected to the PulseAudio daemon and recording should work as well. If it doesn't, that is a problem with WSL, not NixOS

Is this still an issue?

I can hear the sound well now. I apologize for not checking earlier. Thank you for the check