Receiving 2 locations with same data

My Environment

Plugin version: 2.1.2
Platform: Android
Device manufacturer and model: Oppo R7plus
Cordova version (cordova -v): 6.2.0
Cordova platform version (cordova platform ls): 5.1
Plugin configuration options:
{name: 'url', defaultValue: 'some url'},
{name: 'stopOnTerminate',  defaultValue: 'false'},
{name: 'startOnBoot', defaultValue: 'true'},
{name: 'debug', defaultValue: 'true'},
{name: 'stopOnStillActivity', defaultValue: 'true'},
{name: 'startForeground', defaultValue: 'true'}
{name: 'stationaryRadius', defaultValue: 50},
{name: 'distanceFilter', defaultValue: 50},
{name: 'desiredAccuracy', defaultValue: 10},
{name: 'notificationIconColor', defaultValue: '#4CAF50'},
{name: 'notificationTitle', defaultValue: 'Asset Management System'},
{name: 'notificationText', defaultValue: ''},
//{name: 'notificationIconLarge', defaultValue: 'icon_large'},
//{name: 'notificationIconSmall', defaultValue: 'icon_large'},
{name: 'locationProvider', defaultValue: 0},        //backgroundGeolocation.provider.ANDROID_DISTANCE_FILTER_PROVIDER,
{name: 'interval', defaultValue: 10000},
{name: 'fastestInterval', defaultValue: 50000},
{name: 'activitiesInterval', defaultValue: 10000},
{name: 'locationProvider', defaultValue: 0},
{name: 'locationProvider', defaultValue: 0},
{name: 'locationProvider', defaultValue: 0},
// {name: 'activityType', defaultValue: 'AutomotiveNavigation'},
{name: 'saveBatteryOnBackground', defaultValue: 'false'}
Link to your project:

Context
why we are receiving 2 same location with only the difference of debug true and debug false, i can assume that one of the location was generated due the debug is true but do we need the 2nd same location just with deference of debug?
Possible Fix
we can set debug to either true or false depending on the value which we set in configuration

Sure. This will change in upcoming version.

All locations updates are recorded in local db. Number of location stored in db is limited by option.maxLocations and never exceeds this number. Instead old locations are replaced by newer ones.

When App is in foreground or background in addition to storing location in local db, location callback function is triggered.

When option.url is defined. Location updates are also stored in local db. Each location is also immediately posted to server url defined by option.url. If post is successful, the location is marked as deleted in local db. All failed to post locations will be coalesced and send in some time later in one single batch. Batch sync takes place only when number of fail to post location exceeds number defined by option.syncThreshold.

Already implemented for ios in next-ios branch.

mean while i was thinking if there is any configuration that plugin should store data for how many days?

Sure it will be. There is ongoing discussion in https://github.com/mauron85/cordova-plugin-background-geolocation/issues/100.

Implemented in 2.2.0.alfa.1


Sense check mid term priorities

The Chore
Barney has previously shared a long list of features / fixes he would like to cover in Console and is also working on a high level Product Strategy for console with Ramsey. We would like to review those things and identify anything missing that we should be including in our mid term prioritisation AND the order of priority based on those things before we finalise our plan forward.
Tasks

[ ] Ramsey and Barney to share their high level product strategy for Console in Slack
[ ] Re-share and review the list that Barney previously shared in Slack adding anything missing onto the prioritisation board in Miro
[ ] Team to review the product strategy and anything missing from the above task add refine the mid term prioritsation plan accordingly
[ ] Share with ELT and refine / finalise
[ ] Share finalised mid term prioritisation in Console slack channel
[ ] Create stories in the backlog for anything not already covered in github

Additional details / background info

A bit more time needed to iterate the strategy before it's ready to share


Add a demo link ;)

Here you are if you want : https://jsfiddle.net/fLajrmun/

Hey @christophepas , thanks! Do you want to update README.md with this demo link and make a PR?


added LogReg and KNN

Added tuned LogReg Notebook to ml-modelle
Added tuned KNN Notebook to ml-modelle

Thanks.


syntax error, unexpected '}', expecting ';' is not send to sentry in dev mode

Hi team,
i am not getting alert for syntax error, unexpected '}', expecting ';' in dev mode.
Even i try with all error_ types
Could you please advice on this.
Thanks

By default, the DSN may not be set in dev environment, you should check on that.
Apart from this, it's possible that the framework is crashing hard before starting, so you are unable to have reports from that event.

I got other error, DSN are there.
Le jeu. 10 oct. 2019 à 17:54, Alessandro Lai<EMAIL_ADDRESS>a
écrit :

By default, the DSN may not be set in dev environment, you should check on
that.
Apart from this, it's possible that the framework is crashing hard before
starting, so you are unable to have reports from that event.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/getsentry/sentry-symfony/issues/258?email_source=notifications&email_token=AH2QLYAK6IFA5PATFCVBHCDQN5F27A5CNFSM4I7MILCKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEA43I5I#issuecomment-540652661,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AH2QLYBPH4CPXQ4EJAKZVBTQN5F27ANCNFSM4I7MILCA
.


bash: virtualenv: command not found

hi there,
Could anyone please help me in this error ??
root@Shinobi:~# cd Desktop/Tool
root@Shinobi:~/Desktop/Tool# git clone https://github.com/davidtavarez/pwndb
Cloning into 'pwndb'...
remote: Enumerating objects: 4, done.
remote: Counting objects: 100% (4/4), done.
remote: Compressing objects: 100% (3/3), done.
remote: Total 142 (delta 0), reused 0 (delta 0), pack-reused 138
Receiving objects: 100% (142/142), 36.95 KiB | 147.00 KiB/s, done.
Resolving deltas: 100% (61/61), done.
root@Shinobi:~/Desktop/Tool# cd pwndb
root@Shinobi:~/Desktop/Tool/pwndb# virtualenv venv
bash: virtualenv: command not found
Appreciate your help.

Probably virtualenv is not installed on your system. Try this:
sudo pip3 install virtualenv
After that:
virtualenv venv

@

Probably virtualenv is not installed on your system. Try this:
sudo pip3 install virtualenv
After that:
virtualenv venv

Thanks for your help!!
It's Fix now. Appreciate your help.

Probably virtualenv is not installed on your system. Try this:
sudo pip3 install virtualenv
After that:
virtualenv venv

still Iam getting the error
$ sudo pip3 install virtualenv
$ virtualenv venv
virtualenv: command not found

Probably virtualenv is not installed on your system. Try this:
sudo pip3 install virtualenv
After that:
virtualenv venv

still Iam getting the error
$ sudo pip3 install virtualenv
$ virtualenv venv
virtualenv: command not found

@avinash5678
You haven't provided proper output i guess, but still i advice you to try
python -m pip install --user virtualenv
Let me know if it's working for you.


Fix CLI argument OpenStruct naming conditions

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your changes? Here's an example.

[ ] Have you successfully run brew tests with your changes locally?

Using OpenStruct prevents us from naming an option after any method defined on OpenStruct, for example tap. This showed up in this Linuxbrew PR: https://github.com/Linuxbrew/brew/pull/667#issuecomment-377372812, but would also bite us were we to convert brew create to use the new parser, since it also has a tap option.
Ideally we'd probably want to only define methods for the defined options themselves, but I think it's fine leaving that as an enhancement for now.

@alyssais: The problem was, I missed declaring --tap here Homebrew/brew#3983 as it was not listed in man page on brew pull. flag "--tap", required: true would have prevented  Linuxbrew/brew#667 (comment)
I don't understand how its going to be an issue in brew create if we were to declare it as a flag

I don't understand how its going to be an issue in brew create if we were to declare it as a flag
Please correct me if I'm wrong

@GauthamGoli Check out this test that fails locally without these changes.
    it "can handle options that are named the same as Object instance methods" do
      parser.switch "--tap"
      args = parser.parse(["--tap"])
      expect(args.tap).to be true
    end

I'd like to give a bit of time here for @GauthamGoli to see if there's an OpenStruct solution that's possible.

@MikeMcQuaid It passes locally for me without these changes.
For this test to pass, expect statement has to be expect(args.tap?).to be true because --tap is a switch

Oh, yes, I should have written the test as a flag, sorry. The names confused be a bit.

@GauthamGoli see that new test. I meant flag rather than switch there, apologies.

@MikeMcQuaid @alyssais This change makes more sense. I was not able to get the above test to pass. Do you know the reason why this is happening for only --tap ?

Do you know the reason why this is happening for only --tap ?

It's because every object in Ruby inherits the #tap method from Object, and OpenStruct seems to prohibit redefining it.

Ah! everything makes sense now 👍 Thanks

What do you know. tap is like tee for pipelines in Ruby. https://ruby-doc.org/core-2.5.0/Object.html#method-i-tap

@alyssais Any idea if there's a way to do this by just defining methods on an OpenStruct subclass instead? Would be nice to save the code from building this ourselves from scratch.


Update MD file on new branch 2017-28-26_17-11-10

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestCon.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Fix crash when old Android Studio (<2.2.0) is installed

Old Android Studio versions prior to 2.2.0 didn't come with Java bundled. Make sure flutter doesn't crash when we try to determine the Java version of those old Android Studio installations.
Fixes #9008.

LGTM


FE: Option to not allow users to change their email addresses

Requires https://github.com/matrix-org/matrix-authentication-service/issues/1663
Where an upstream IdP is used you might want to import the email address on the first sign in, and then not let the user change it.

Use flag from https://github.com/matrix-org/matrix-authentication-service/issues/1663 to know whether email changes are allowed
The UI should not show the options to add/edit emails and only show the current one

https://github.com/matrix-org/matrix-authentication-service/issues/2198


Changing SVG density affect text scaling

I'm not sure if this is a bug or intended functionality but if I renderer the following SVG with different densities the text is scaled out of proportions.
If it's intended, how do I create a text that can scale with the image?
<svg viewBox="0 0 100 24" xmlns="http://www.w3.org/2000/svg"><rect width="100%" height="100%" fill="#4CBB17"/><text xmlns="http://www.w3.org/2000/svg" x="50%" y="75%" font-size="100%" fill="#000000" text-anchor="middle" font-family="Arial,Helvetica">Fancy text</text></svg>
density: 72

density: 144

density: 288

Hello, I had a quick look at this and can reproduce using sharp v0.21.0 / librsvg v2.44.6.
I was unable to reproduce using the older librsvg v2.40.20 so suspect a recent change might be causing this e.g. https://gitlab.gnome.org/GNOME/librsvg/merge_requests/92
I'll try to reproduce using the rsvg-convert command line tool.

This is a regression to librsvg, caused by https://gitlab.gnome.org/GNOME/librsvg/merge_requests/92 in librsvg v2.43.2, that will be fixed by https://gitlab.gnome.org/GNOME/librsvg/issues/348 in librsvg v2.45.0.

What is the status of this?

To take advantage of the upstream fix to librsvg right now, both it and libvips will need to be compiled from source and installed globally before installing sharp.

Okay, I think I'll have a look at that and see if we can solve our problem that way.

Are there any compile flags or dependencies I should be aware of?

@lovell Do you know if there are any configuration before building those libraries so they work with sharp?

The Docker images and build scripts/flags for the pre-compiled binaries are at https://github.com/lovell/sharp-libvips

Hello,
I've got similar issue with customized SVG and overlayWith.
One workaround in my case is to increase the SVG resolution and font size to simulate the density.
By the way, on Windows 10 it works fine without the simulation. The issue appears on Ubuntu Xenial.

+1

sharp v0.22.0 now available, providing libvips v8.7.4 and librsvg v2.45.5.


HTTPS

Make all related servers support HTTPS.
Ref:

https://letsencrypt.org/
https://hackage.haskell.org/package/warp-tls
https://github.com/noteed/acme

SSCCE: https://elm-lang.org

Duplicate of #624


0.2.5

@JuliaRegistrator register

Error while trying to register: Version 0.2.4 already exists

@JuliaRegistrator register

Registration pull request created: JuliaRegistries/General/102268
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.2.5 -m "<description of version>" ad40bb086716caa405fadc9d764371c53fb77199
git push origin v0.2.5


Review all (12) NOTICE files for contents

Track the review of notice files in these individual issues:
openwhisk: https://github.com/apache/incubator-openwhisk-release/issues/75
cli: https://github.com/apache/incubator-openwhisk-release/issues/76
client-go: https://github.com/apache/incubator-openwhisk-release/issues/77
apigateway: https://github.com/apache/incubator-openwhisk-release/issues/78
catalog: https://github.com/apache/incubator-openwhisk-release/issues/79
runtime-docker: https://github.com/apache/incubator-openwhisk-release/issues/80
java: https://github.com/apache/incubator-openwhisk-release/issues/81
nodejs: https://github.com/apache/incubator-openwhisk-release/issues/82
php: https://github.com/apache/incubator-openwhisk-release/issues/83
python: https://github.com/apache/incubator-openwhisk-release/issues/84
swift: https://github.com/apache/incubator-openwhisk-release/issues/85
wskdeploy: https://github.com/apache/incubator-openwhisk-release/issues/86

@mrutkows We are fine with the LICENSE and NOTICE files so far, since we expect source code release only as the first time release.


added opensource  programs section  and code_of_conduct link to readme

issue - #17

Hey, plz dont  take those 2 commits I used the same branch so they are now will be merged under the same issue I am making another PR for the readme updation that will link to the original issue but you can merge the  first pr / commit "added os programs and coc links one

Hey, plz dont  take those 2 commits I used the same branch so they are now will be merged under the same issue I am making another PR for the readme updation that will link to the original issue but you can merge the  first pr / commit "added os programs and coc links one


Automatically create armeria's dependency when missing

Assume we have the following service with a custom dependencyInjector set via Server.builder().dependencyInjector(...)
class MyAnnotatedService {

    @Get("/")
    @RequestConverter(ProtobufRequestConverterFunction.class)
    public HttpResponse foo(...) {...}

}

In this case, armeria is unable to find the appropriate dependency because ProtobufRequestConverterFunction is not registered in BuiltInDependencyInjector.
Unless the user manually registers the dependency manually in his/her custom dependencyInjector, the server would refuse to start.
We probably want to ensure that at least dependencies provided by armeria are always found by doing something like the following:

Define an annotation (e.g. @CreateIfMissing)
Add the custom annotation to armeria classes which may be injected

@CreateIfMissing
class ProtobufRequestConverterFunction {
}

In BuiltInDependencyInjector, check for the annotation @CreateIfMissing and generate a singleton if applicable.

if (type.hasAnnotation(CreateIfMissing.class)) {
    return createSingleton(type);
}

https://github.com/line/armeria/blob/c4ebc7b3a42df17029641caf5f291c31b104e74b/core/src/main/java/com/linecorp/armeria/internal/common/BuiltInDependencyInjector.java#L44-L55

BuiltInDependencyInjector could discover them via Java SPI. Java SPI also uses the default constructor for instantiating the provider, so we could simply define a tag interface and list the types in META-INF/services/....


Removing duplicate lines

Dear @benhoyt, I’m confused right from the start because
awk "!line[$0]++" wordlist.txt > deduplicated.txt

works as expected whereas
goawk "!line[$0]++" wordlist.txt > deduplicated.txt
----------------------------------------------------
!line[$0]++
         ^
----------------------------------------------------
parse error at 1:10: expected lvalue before ++ or --

throws aforementioned error. What would you recommend to do here?

This looks like a legit bug, thank you! I'm very surprised I hadn't noticed this before, as GoAWK has a pretty thorough suite of tests (taken from the awk and gawk test suites). I'll look into this in the next couple of weeks.

If you need a workaround in the meantime you can parenthesize the line[$0]++ expression:
goawk "!(line[$0]++)" wordlist.txt > deduplicated.txt

Thanks for the bug report! Fixed and released https://github.com/benhoyt/goawk/releases/tag/v1.6.1. Can't believe it's been like this since v1.0.0 and hadn't been caught yet. :-)
There was actually a GAWK test for a similar case, but I had it in my "skipped" bucket because I thought it was due to another issue. There's a known issue with parsing double-dollar expressions like $$a++++ -- I'm going to take a closer look at that and see about fixing that one too.

Err… Ben, 1.6.1 sources are present, but binaries are missing.

Aha, oops, will fix tomorrow.

Uploaded!

Dear @benhoyt, I’ve just stumbled upon ~400 KiB size difference after de-duplication of 6 MiB text file via GoAWK and original AWK. The difference is explained by line endings used (Windows vs UNIX). Is there a way to use UNIX line endings for consistency?

Hi @sergeevabc Thanks. Can you please provide a sample script and inputs that reproduce this problem? Do you think this is a goawk-specific issue, or it has more to do with your line ending differences, and you'd just like goawk to normalize it? (Does awk/gawk have the same issue?)

list-20210221-01.7z
$ awk "!a[$0]++" list-original.txt > list-awk.txt
$ goawk "!a[$0]++" list-original.txt > list-goawk.txt

$ wc -c list*
6404594 list-awk.txt
6759614 list-goawk.txt
6404594 list-original.txt

$ xxhsum list*
2d1d8df78c47530b  list-awk.txt
717435dbe29ff67d  list-goawk.txt
2d1d8df78c47530b  list-original.txt

$ diff --brief --report-identical-files list-awk.txt list-original.txt
Files list-awk.txt and list-original.txt are identical

$ diff --brief --report-identical-files list-goawk.txt list-original.txt
Files list-goawk.txt and list-original.txt differ

$ diff --brief --report-identical-files --strip-trailing-cr list-goawk.txt list-original.txt
Files list-goawk.txt and list-original.txt are identical

Thanks @sergeevabc. I removed your attachment because it was a list of porn domains, and I'd prefer that not linked to here. Next time it'd be best to attach a minimal example which reproduces the problem -- in this case I think just a couple of lines with only LF (\n) line endings would suffice.
It looks like you're running on Windows, but awk and gawk on Linux have a similar problem -- Gawk seems to keep the file's existing line endings, whereas GoAWK outputs the platform's line endings. I'm not sure which is better, or which is correct!
Here's a minimal repro script on Linux:
$ python3
>>> f = open('t.txt', 'wb')
>>> f.write(b'one\r\ntwo\r\n')
10
>>> f.close()
>>>
$ hexdump -C t.txt
00000000  6f 6e 65 0d 0a 74 77 6f  0d 0a                    |one..two..|
0000000a
$ awk '!a[$0]++' t.txt | hexdump -C
00000000  6f 6e 65 0d 0a 74 77 6f  0d 0a                    |one..two..|
0000000a
$ goawk '!a[$0]++' t.txt | hexdump -C
00000000  6f 6e 65 0a 74 77 6f 0a                           |one.two.|
00000008
$ awk --version
GNU Awk 5.0.1, API: 2.0 (GNU MPFR 4.0.2, GNU MP 6.2.0)
...
$ goawk --version
v1.6.1

In addition, as you can see below, Gawk seems to include the CR (\r) character of the line ending in the last field (field $2 below). This seems non-ideal:
$ awk '{ printf "%s_%s/", $1, $2 }' t.txt | hexdump -C
00000000  31 5f 6f 6e 65 0d 2f 32  5f 74 77 6f 0d 2f        |1_one./2_two./|
0000000e
$ goawk '{ printf "%s_%s/", $1, $2 }' t.txt | hexdump -C
00000000  31 5f 6f 6e 65 2f 32 5f  74 77 6f 2f              |1_one/2_two/|
0000000c

I gather you're running on Windows given the double quotes used as escape character on the command line? Also, can you send the version of awk and goawk that you're running?
@arnoldrobbins do you have any comment about this, or links to documentation/discussion about this issue?

Gawk is consistent . RS has the default value of \n and that is what terminates records. As far as gawk is concerned, the \r is  no different from any other character, which is why it appears as part of the last field in the record.
That said, on Windows, I believe the default is to work in text mode, in which case gawk never sees the \r\n line ending, it only sees \n.  One can use BINMODE to force gawk to see those characters, in which case you would need to set RS = "\r?\n" in order to get correct processing.
Take the Windows advice with a grain of salt. I have not used a Windows system directly in over two years, and when I did I used Cygwin, so some experimentation may be in order.
If one is processing a Windows file on Linux, then one should use a utility like dos2unix on the file, or tr, before sending the data to GoAwk, which does not (yet! hint, hint) allow RS to be a regular expression.  Using GoAwk on Windows, well, you'll have to figure out what the Go runtime is handing off to your code.
Hope this helps.


[Bug]: jest-runner  Class private methods are not enabled with version 29.0.0

Version
29.0.0
Steps to reproduce
jest --runInBand
Expected behavior
Tests should run without errors
Actual behavior
I receive the following error:
SyntaxError: /Users/myUser/Workspace/projectName/node_modules/jest-runner/build/index.js: Class private methods are not enabled.
  91 |   }
  92 |
> 93 |   async #createInBandTestRun(tests, watcher) {
     |   ^
  94 |     process.env.JEST_WORKER_ID = '1';
  95 |     const mutex = (0, _pLimit().default)(1);
  96 |     return tests.reduce(
    at File.buildCodeFrameError (/Users/mihaela.lorincz/Workspace/the-st-james-mobile-app/node_modules/@babel/core/lib/transformation/file/file.js:249:12)
    at NodePath.buildCodeFrameError (/Users/mihaela.lorincz/Workspace/the-st-james-mobile-app/node_modules/@babel/core/node_modules/@babel/traverse/lib/path/index.js:143:21)
    at verifyUsedFeatures (/Users/mihaela.lorincz/Workspace/the-st-james-mobile-app/node_modules/@babel/helper-create-class-features-plugin/lib/features.js:97:18)
    at PluginPass.Class (/Users/mihaela.lorincz/Workspace/the-st-james-mobile-app/node_modules/@babel/helper-create-class-features-plugin/lib/index.js:72:44)
    at newFn (/Users/mihaela.lorincz/Workspace/the-st-james-mobile-app/node_modules/@babel/core/node_modules/@babel/traverse/lib/visitors.js:177:21)
    at NodePath._call (/Users/mihaela.lorincz/Workspace/the-st-james-mobile-app/node_modules/@babel/core/node_modules/@babel/traverse/lib/path/context.js:53:20)
    at NodePath.call (/Users/mihaela.lorincz/Workspace/the-st-james-mobile-app/node_modules/@babel/core/node_modules/@babel/traverse/lib/path/context.js:40:17)
    at NodePath.visit (/Users/mihaela.lorincz/Workspace/the-st-james-mobile-app/node_modules/@babel/core/node_modules/@babel/traverse/lib/path/context.js:100:31)
    at TraversalContext.visitQueue (/Users/mihaela.lorincz/Workspace/the-st-james-mobile-app/node_modules/@babel/core/node_modules/@babel/traverse/lib/context.js:105:16)
    at TraversalContext.visitMultiple (/Users/mihaela.lorincz/Workspace/the-st-james-mobile-app/node_modules/@babel/core/node_modules/@babel/traverse/lib/context.js:74:17)

error Command failed with exit code 1.

Additional context
my jest config:
  "jest": {
    "verbose": true,
    "moduleNameMapper": {
      "^[./a-zA-Z0-9@$_-]+\\.(png|jpg)$": "<rootDir>/node_modules/react-native/jest/assetFileTransformer.js"
    },
    "modulePaths": [
      "<rootDir>/src/"
    ],
    "preset": "react-native",
    "setupFiles": [
      "./node_modules/react-native-gesture-handler/jestSetup.js",
      "./jest/setupTests.js"
    ],
    "testEnvironment": "node",
    "transform": {
      "^.+\\.js$": "babel-jest"
    },
    "transformIgnorePatterns": [
      "node_modules/(?!react-native|react-native-gesture-handler|react-navigation|@react-native-async-storage/async-storage|redux-persist|react-native-keychain|)/"
    ]
  }

I do not get this error for versions 28.1.3 and 26.6.3
Environment
System:
    OS: macOS 12.5.1
    CPU: (12) x64 Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz
  Binaries:
    Node: 18.8.0 - /usr/local/bin/node
    Yarn: 1.22.17 - /usr/local/bin/yarn
    npm: 8.18.0 - /usr/local/bin/npm
  npmPackages:
    jest: 29.0.0 => 29.0.0

Why are you transpiling jest-runner?

I do not have jest-runner in my dependencies, it is installed by jest core library. I have a JS project

It's babel that's complaining about the syntax, not Node itself. If you just do require('jest-runner') it'll work fine for you since the Node version supports it. So something in your setup is running Babel on Jest's modules.
Do you have a reproduction I can check out?

No, it is in my prod app, on a private repo. Is there any other info that might help you? Thanks for help

Without a reproduction there's not much I can do. I'd take a look at your setupFiles, do any of them import jest-runner?

Ran into the same issue with React Native. Seems like metro-react-native-babel-preset is involved. Here is the most basic reproduction I could come up with:
{
  "scripts": {
    "test": "jest"
  },
  "dependencies": {
    "@babel/runtime": "^7.18.9",
    "babel-jest": "^29.0.1",
    "jest": "^29.0.1",
    "metro-react-native-babel-preset": "^0.72.1"
  },
  "jest": {
    "transformIgnorePatterns": []
  }
}

babel.config.json
{ "presets": ["module:metro-react-native-babel-preset"] }

test/basic.test.js
import double from "./double";

test("Doubles the number", () => {
  expect(double(2)).toBe(4);
});
// test/double.js
export default (value) => value * 2;

As a workaround ignoring the already mentioned jest-runner from being transpiled works around the issue:
"transformIgnorePatterns": [
  "jest-runner"
]

Ah, nice 👍 that explains why our integration test doesn't fail: https://github.com/facebook/jest/blob/dde24c85698259d9ce887ede9646847d0fff9554/examples/react-native/jest.config.js#L7
Still, weird it triggers... I'll look into it next week 🙂

Right, transforming all node_modules is kind of a bad idea. The initial post is using a popular hack to ignore all node_modules except some that are published as ESM and therefore need to be transpiled as well.
Noticed that there is a / slash at the end which prevents jest-runner from being ignored for some reason.
"transformIgnorePatterns": [
  "node_modules/(?!react-native|any-react-native-esm-package)"
]

When that slash is removed jest-runner doesn't have to be listed explicitly anymore.

Ah, that explains it (over-eager negation). Not sure there's anything for Jest to do? We target specific node versions - if other tooling has other syntax requirements that's not on Jest I think?

I was able to successfully upgrade a React Native project with the error described here. Definitely nothing for Jest to do here as node_modules generally shouldn't be transformed. Metro could add @babel/plugin-proposal-private-methods to metro-react-native-babel-preset but I don't think that's necessary for now.

Agreed, thanks!


Here wallet connect issue

Describe the bug
Unable to connect with HERE wallet on testnet
To Reproduce
Steps to reproduce the behavior:

Go to wallet selector
Select Here Wallet
Scan QR code with mobile camera
No link or data found

Expected behavior
User should confirm on Here wallet the dapp connection
Screenshots

Device (please complete the following information):
OS: IOS 15.6.1
Browser: Chrome
Additional context

Hey @leomanza , thanks for raising this issue! We tested it, and it was the same for us. We are getting this:

Hey @leomanza , thanks for raising this issue! We tested it, and it was the same for us. We are getting this:

Can you guys @Neafiol @AZbang check this, thanks!

Hey @leomanza to update you that we reached out to the HERE Wallet team on their Discord regarding this issue, and it should be addressed soon. I will leave this open until it's resolved. Thanks!

Thanks @AZbang and @AmmarHumackicSQA for taking a look at this issue.


private void ProcessEvent(NetEvent evt) - CPU 20% Loaded

Information about bug or feature
While(true) PollEvents...
Library version: Latest
Framework:NetStandart 2
OS: Win

@nurbat this is wrong way to call PollEvents in simple while(true) loop. You must use Thread.Sleep() after poll or in case of Unity3d just call this in Update method.

I not used Unity3d

I Find,

Options.Listener.ConnectionRequestEvent += OnConnectionRequest;
request.Accept(); without check if
Result: 20% cpu


Bring forward SPS marketplace adjustments

Purpose

Bring the extra bits SPS requires to safely install from the Unity Marketplace into the develop branch.

Proposed Changes

ADD some extra variables (unused) required by the management console's installation procedures
CHANGE some of the module source references to work inside the management console (ssh vs https fetch issues)

Issues

unity-sds/unity-sps#59

Testing

Currently deployed in unity-venue-dev under btl-df1/dev (standard password)

http://btl-df1-dev-httpd-alb-1474979360.us-west-2.elb.amazonaws.com:8080/btl-df1/dev/sps/
https://www.dev.mdps.mcp.nasa.gov:4443/btl-df1/dev/sps/

Just to note, this PR is actually also including the changes from #228, as they're necessary for the installation's proxies (though #228's changes arent merged yet)

what are the new parameters needed by the Management Console

The management console, when installing terraform modules/applications, does a really odd force-injection of variables here. I think the terraform call includes some 'strict' setting, as the absence of any of the variables included there in the actual terraform config throws a fatal error (in the management console output at least).

can we set "deployment_name" to be the same as "project", so we don't have to define the same value twice for 2 different variables

deployment_name is never actually used by the SPS terraform configuration (nor are tags or install_prefix). It does get injected by the management console though?
I guess we could set its default to something simple (like a "" empty string or something), but it definitely needs to be its own separate variable, if only to keep the management console's installer code happy.

@jpl-btlunsfo : ok, can you set the default value of "deployment_name" to be ""? Then we can merge these changes.

tag updated, just need an approval to merge (will hold merging until I run another test-deploy).

No issues on personal deploy!


fix: remove unneeded redux dep

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


Remove some duplicated files and run RAPIDS CI locally

Most of the changes are doc string adding and duplicated code removal.
Updates:

Remove duplicated files under python/pylibwholegraph/wholegraph_torch, keeping the same files under python/pylibwholegraph/pylibwholegraph/torch_cpp_ext.
Added runtime cpp_extension support for PyTorch and fully removed old c10 API dependent compile option.
Remove duplicated files under pylibwholegraph/pylibwholegraph, keeping the same files under python/pylibwholegraph/pylibwholegraph, merged changes.
Some fix on RAPIDS CI scripts to get CI run locally.
Add doc strings and formated files.

/merge


Remove Nexenta storage volume plugin

I'm running a few tests on the bubble. Please hold on with merging.

go build

Ping @remibergsma @nhelsherif @borisroman
Tests:

Deploy DC

[root@cs1 marvin]# python deployDataCenter.py -i /data/shared/marvin/mct-zone1-kvm1.cfg
[INFO] 2016-05-25 07:58:22,289 - marvin - Parsing Test data successful
[INFO] 2016-05-25 07:58:22,501 - marvin - === Test Client Creation Successful ===
[INFO] 2016-05-25 07:58:22,516 - marvin - === Deploy DC Started ===
[INFO] 2016-05-25 07:58:22,905 - marvin - === Create Zone Successful ===
[INFO] 2016-05-25 07:58:28,074 - marvin - === Creating Physical Network Name : mcct-pnet Id : 3fec07df-eb08-4a05-81bd-005b04ce2631 Successful ===
[INFO] 2016-05-25 07:58:53,439 - marvin - === Update Network Service Provider Successfull ===
[INFO] 2016-05-25 07:59:03,581 - marvin - === Update Network Service Provider Successfull ===
[INFO] 2016-05-25 07:59:08,682 - marvin - === Update Network Service Provider Successfull ===
[INFO] 2016-05-25 08:01:16,882 - marvin - === Add Host Successful ===
[INFO] 2016-05-25 08:02:17,129 - marvin - === Creating Storage Pool Successful ===
[INFO] 2016-05-25 08:02:17,221 - marvin - === Add Image Store Successful ===
[INFO] 2016-05-25 08:02:17,256 - marvin - === Enable Zone Successful ===
[INFO] 2016-05-25 08:02:17,263 - marvin - === Data Center Settings are dumped to dc_entries_May_25_2016_08_02_17.obj ===
[INFO] 2016-05-25 08:02:17,272 - marvin - === Deploy DC Successful ===

VM lifecycle tests

[root@cs1 integration]# less results.txt
Test advanced zone virtual router ... === TestName: test_advZoneVirtualRouter | Status : SUCCESS ===ok
Test Deploy Virtual Machine ... === TestName: test_deploy_vm | Status : SUCCESS ===ok
Test Multiple Deploy Virtual Machine ... === TestName: test_deploy_vm_multiple | Status : SUCCESS ===ok
Test Stop Virtual Machine ... === TestName: test_01_stop_vm | Status : SUCCESS ===ok
Test Start Virtual Machine ... === TestName: test_02_start_vm | Status : SUCCESS ===ok
Test Reboot Virtual Machine ... === TestName: test_03_reboot_vm | Status : SUCCESS ===ok
Test destroy Virtual Machine ... === TestName: test_06_destroy_vm | Status : SUCCESS ===ok
Test recover Virtual Machine ... === TestName: test_07_restore_vm | Status : SUCCESS ===ok
Test migrate VM ... SKIP: At least two hosts should be present in the zone for migration
Test destroy(expunge) Virtual Machine ... === TestName: test_09_expunge_vm | Status : SUCCESS ===ok

----------------------------------------------------------------------
Ran 10 tests in 1392.636s

OK (SKIP=1)
results.txt (END)

go build

go build


Ensured the padding buffer is unlisted and wiped out when leaving VimRoom

Hi Mike,
I've been using VimRoom for a few weeks and find it works great in full screen. I use the LustyJuggler for switching buffers and found that the padding buffer called new was appearing in the buffer list once I'd used VimRoom hence the small changes:
Ensure that the padding buffer is not listed when in VimRoom by specifying:
setlocal nobuflisted
setlocal buftype=nofile
setlocal bufhidden=delete
Make sure the padding buffer is wiped out (and hence does not even appear when you :ls!) when leaving VimRoom
The padding buffer has been given the name of __vimroom__ to minimise clashes with real buffers and a check has been put in when leaving VimRoom to wipeout this buffer if it's found.
I've also refactored the code to set the buffer properties into a function (s:set_up_padding_buffer) to reduce duplication.
Thanks,
Matt.

I'm afraid @mikewest may have notifications on this repo killfiled, but these are all great changes. Probably also fixes #32 and #40.
Thanks for the plugin Mike! Just ribbing you :smile:

After using VimRoom for a couple of years I switched to https://github.com/junegunn/goyo.vim


Feature/issue 37/edit profile

This pull request modifies the edit profile functionality
The changes include:
Fetching and displaying the user's profile data upon loading the edit profile screen.
Allowing the user to update their profile information.
Handling API responses and error cases during the update process.

Closing this PR because I mistakenly pushed duplicate commits after rebasing. I will create a new PR with the correct changes.


feat: vector input/output support

closes: #353
Most notable changes:

Unresolved Bytes
Encoding something via the ABIEncoder will no longer give you raw bytes right away. You get something called UnresolvedBytes. Think of an unresolved symbol during linking :D

If you give UnresolvedBytes the address at which the bytes are going to be placed, then it will reward you with the bytes themselves. This is so that we may resolve any pointers (currently only vectors) within the data.

Reverse decoding
Returning a vector involves logging its elements. We need to know where those logs are in, what could be, a sea of other unrelated logs. We cannot require the vec element logs to be the first thing logged since that would mean whatever you do to calculate the vector must not log.

The only reasonable approach to me was to say that the last logs of the method must be the vector elements themselves.
So that means reverse decoding since you don't know from ParamTypes how big the vector is going to be. So you kind of decode in reverse until everyone is happy :)

Decoding now requires the ReturnData as well as Logs to decode.

To generate methods that actually return vectors, we need the sway contract methods themselves to return vectors.
But returning a vector from sway gives you very little to work with. Only 3 WORDs, the pointer, the capacity and the length of the vector you're returning.
The rest of the information should be logged rather than returned. Until further notice that burden falls on the authors of the contracts.
There is a caveat to logging in sway as well -- if you log a vector you're not going to be logging its elements. Just like when you return a vector -- you'll only get those 3 WORDS.
This results in an edge case where if you have a Vec<Vec<..>> you need to jump through an extra hoop to have it decode properly. This is due to log not penetrating the Vec iron curtain, so if you log the parent vec you're going to just get it's ptr, cap and len and not those of its children elements.
That is why you need to log the inner vectors themselves after you log their elements. This is because we're decoding from the right (aka reverse) and we need to get the vec length info before we get the elements.
And why we're decoding in reverse is explained somewhere above.
There is a sync shortly regarding vectors, hopefully, the compiler team can figure something out to either make this logging automatic or provide a recursive helper which can penetrate the veil of a vector :) This behavior is documented in the SDK book.

Nice work! This is my first review pass. I have to rest a bit and try to wrap my head around the resolve thing smile
Ty.
I'm glad you said that. This means that I probably need to put some comments in.


Copy ES indexes to Google Big Query

Initial implementation: copy Elastic Search indexes from index.multiversx.com to Google Big Query datasets.

Implemented by separate PRs.


Getting 400 UnauthorizedRegistration` response

I am getting a unauthorizedRegistration error. Following is my code. I generated keys using createVapidKeys() function. Any help on this?
`$endpoint = 'https://fcm.googleapis.com/fcm/send/cP2Z2eN3cd4:APA91bFUU3OYDU0mKzrpSSwG4ucSua6BGO0wmNizDb-G1wAfI7F-KfHBXT5GRUTdOPFAKFaPtChCpKRMc0eDqW_exeuFLK0PmHm77SwGhn701WvMAspSAHtNtvzgCBgVu7KEgpuSyTZj'; // Chrome
    $auth = array(

        //'GCM' => 'AIzaSyB_TdkuANMFu1a3O-3Ikx7YzXXXXXXXXX', // API key from Google Developer Console
        'VAPID' => array(
            'subject' =><EMAIL_ADDRESS>// can be a mailto: or your website address
            'publicKey' => 'BJ_zGRSUhYMaRobLaiTkrPHvlsW3jjE3_QfRlvcdkzmwQitGVKN8T0NIcwWF8XWPb-duqcf-8H3kk8CYRQHqTZM=', // (recommended) uncompressed public key P-256 encoded in Base64-URL
            'privateKey' => 'MC2p8KJdwnZ85WDmT5eZnBcEs0XxkKcwaDy9daM9wi0=', // (recommended) in fact the secret multiplier of the private key encoded in Base64-URL
           ),
    );

    $webPush = new WebPush($auth);

    // array of notifications
    $notifications = array(
        array(
            'endpoint' => 'https://fcm.googleapis.com/fcm/send/cP2Z2eN3cd4:APA91bFUU3OYDU0mKzrpSSwG4ucSua6BGO0wmNizDb-G1wAfI7F-KfHBXT5GRUTdOPFAKFaPtChCpKRMc0eDqW_exeuFLK0PmHm77SwGhn701WvMAspSAHtNtvzgCBgVu7KEgpuSyTZj',
            'payload' => 'hello !',
            'userPublicKey' => 'BIo-Vu9j45Kky5pMgT7jgTN-wVVGNycApJiRNhdrGNQmiB966c_nTgnl1MzW74SLr6huE8MYAJewkPPwH5od-sM=', // base 64 encoded, should be 88 chars
            'userAuthToken' => '-x_6bLINs_q6o70VwMfABg==', // base 64 encoded, should be 24 chars
            )
        );
    $result =$webPush->sendNotification(
        $notifications[0]['endpoint'],
        $notifications[0]['payload'], // optional (defaults null)
        $notifications[0]['userPublicKey'], // optional (defaults null)
        $notifications[0]['userAuthToken'], // optional (defaults null)
        true // optional (defaults false)
    );`

I'm facing the same issue , firefox working already , chrome & opera is facing the same error above
I've generated the VAPID keys and resubscribed it after each try
I'm using chrome version : Version 60.0.3112.90 (Official Build) (64-bit)
however I noticed my gcm key format is different from the one mentioned above , mine is numbers - no letters

Tests are passing on Travis so there must be a configuration error on your part. I'll look into your code by this week-end. Could you provide the additional info requested in the pull request template please?

Was a configuration error

@stanlyjohn2 am facing the same issue, could you elaborate on the configuration error that you found?

The vapid public key used here should be same as the application server key we use at the client side while getting the subscription from the user.


Prefix all Germline WGS output files with sample name.

This may belong in genome/genome if we want to add the name during the output gather.

There is a fix for this when running GMS workflows, see https://github.com/genome/genome/pull/1913


Mostly ported to Fabric, just some thread related things left.

Not fully ported, but thread stuff is one thing I'm not very good with threads, so didn't want to mess that up. This does port the config and server starting/stopping systems from the forge version.

I decided to use the multi-loader structure for the mod to support both Forge and Fabric easily in the same project 😄
Greatly appreciate the efforts here though, they helped with my port regardless!


show-whitespace doesn’t work on newly sent suggestions

No, I checked that for PRs.

WHat?

Ok the issue is (From what I see) OnNewComments Does not run on the PR File tab. So any new comments on that tab will not work.
The only issue I have is that it was my own PR so I was only able to comment (not apply a real review)
If someone wants they can open a PR on https://github.com/yakov116/TestR/ so I could test

Checked again, the issue still persists.

@szmarczak can you open a pr there and turn off subscribe so I could test a real review?

@szmarczak I am only seeing it on the PR Files Tab.

I am only seeing it on the PR Files Tab.

That's what I'm talking about... Files changed... I guess I could've said that at the beginning :P

Ok the issue is (From what I see) OnNewComments Does not run on the PR File tab. So any new comments on that tab will not work.

I tried to figure it out... No luck.

While looking for this I figured out an easy way to see GitHub's event names:
d = Node.prototype.dispatchEvent;
Node.prototype.dispatchEvent = function (...a) {
	console.log(...a);
	// debugger; // Uncomment when necessary
	d.apply(this, a);
}

Anyway, no special events seem to be available. The only solution I can think of:
o = new MutationObserver(runUpdates);

for commentHolders of select.all('.diff-table tbody, .js-comments-holder')
	o.observe(commentHolders)

This should probably be part of on-new-comments somehow. After #2987

I really think that https://github.com/sindresorhus/refined-github/issues/3084 should fix all of our problems, it should be revisited: https://github.com/sindresorhus/refined-github/pull/3092
That PR probably just needs an efficient mechanism to avoid applying changes to the same element.

@fregante any reason we can't listen on socket:message events and handle everything from there?

If it works, sure.


Format-Table -Wrap produces strange output for a mix of long lines and lines containing tabs

Steps to reproduce
Windows 7, with pwsh.exe's window size set to 200x75. The width is the relevant thing. Choice of font does not seem to matter.
Edit: Also repros on openSUSE Tumbleweed in a similarly sized ssh window or konsole window.
@((
	New-Object psobject | Add-Member -Passthru Foo "a`tb"
), (
	New-Object psobject | Add-Member -Passthru Foo ("c`t" + 'd' * 200)
), (
	New-Object psobject | Add-Member -Passthru Foo "e`tf"
)) | Format-Table -Wrap

Expected behavior

(from PS 5)
Actual behavior

There is an extra line after the first and last element (in fact there is an extra line after every row except the one with the wrapping value)

The row that has the long value is wrapped strangely.

Environment data

> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      6.0.1
PSEdition                      Core
GitCommitId                    v6.0.1
OS                             Microsoft Windows 6.1.7601 S
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

and
Name                           Value
----                           -----
PSVersion                      6.0.1
PSEdition                      Core
GitCommitId                    v6.0.1
OS                             Linux 4.15.6-1-default #1 SMP PREEMPT Sun Feb 25 18:51:57 UTC 2018 (06f0b06)
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Removing the tab character from the first object's property (or even replacing it with 8 spaces) fixes the extra line between the rows but does not fix the strange wrapping of the long row.
Shortening the long row so that it does not need to wrap also fixes the extra line between the rows.

Anotehr bit of strange behavior for
@((
	New-Object psobject | Add-Member -Passthru Foo "a`tb"
), (
	New-Object psobject | Add-Member -Passthru Foo ("c`t" + 'd' * 100 + ' ' + 'd' * 100)
), (
	New-Object psobject | Add-Member -Passthru Foo "e`tf"
)) | ft -Wrap

PS 5.0:

PS Core:

There is now also an empty line between the two parts of the wrapping second row. This line does not go away if I replace the tab in the first row with spaces, but does go away if I replace the tab in the second row with spaces.

@Arnavion Thanks for you report. Could you please repo with laters daily build?

@Arnavion Thanks for the repo.
/cc @SteveL-MSFT Could you please comment the Issue?

Looks like a bug in the wrapping logic.  I'll look into this.

The way -Wrap works is that it tries not to split a word.  So in the second case above (w/ latest master), the wrap inserts a newline after the first 'd' * 100 since it fits, but puts the second one on a new line rather than split it.  In the first case, the tab character's display size isn't taken into account, so it treats it as size 1.  'd' * 200 is wider than the width of the console, so it has to split it in half, but due to the wrong total length calculation of the tab, it inserts the newline in the wrong place.
Not sure how to fix this.  On macOS, the default tab stop is 8, so "atb" would show "a       b".  However you can use tabs -4` to change the tab stop to 4 or even set different sizes for tab stop 1, 2, ... so we can't make assumptions about the tab stops.

There are already some places in the PowerShell code base that replaces tabs with four spaces, although it's not consistent everywhere.


[AUTOPATCHER-CORE] Upgrade kubernetes to 1.30.3 Fix CVE-2024-10220

[AUTOPATCHER-CORE] Upgrade kubernetes to 1.30.3 Fix CVE-2024-10220
Upgrade pipeline run -> https://dev.azure.com/mariner-org/mariner/_build/results?buildId=692673&view=results

Auto cherry-pick results:

3.0-dev :white_check_mark: -> https://github.com/microsoft/azurelinux/pull/11649

Auto cherry-pick pipeline run -> https://dev.azure.com/mariner-org/mariner/_build/results?buildId=696637&view=results


UML-959 change url for survey URL

Purpose
UML-959
Approach
change survey link.
Learning
Checklist

[x] I have performed a self-review of my own code
[ ] I have added relevant logging with appropriate levels to my code
[ ] I have updated documentation (Confluence/GitHub wiki/tech debt doc) where relevant
[ ] I have added tests to prove my work
[ ] The product team have tested these changes

Coverage remained the same at 81.604% when pulling 3e112d14106be043246b4086a3d4a122b3c576f4 on UML-959-fix-survey-url into 93d644c659b83376e4d10d57661764137db99d1f on master.

closing as added to #435


Select correct ticker function during initialization, not only when document visibility changes

Fixes #611

ugh, it didn't work. I'm pretty sure it did when I tested it year ago

Was finishing up getting the pause in and eyeing this up next, any idea why it doesn't fix it?

I'm not sure yet. Animation starts playing in the background as it supposed to, but instead of stopping at the end it plays it backwards to initial state with the same speed, then forward again and so on. I was thinking it might be because of tick() call which is unnecessary upon initialization, but that's not it.
I'll take another look this evening (in like 6 hours)

I run a D&D game on Thursday evenings, so won't be able to look till tomorrow anyway - and trying to update QUnit to v2 to improve the testing right now ;-)

Hi. Diidn't had time to take a look into this yesterday, so I'm just getting started. Something has changed indeed. I tried to roll back to ce90611 (year ago), apply the patch and it did the thing. Also works with 1.3.1, but breaks on 1.3.2 release.
I also noticed another new behavior with 1.3.2, which is not mentioned in release notes: all running animations are getting fast-forwarded to the end with the next tick after tab is moved to background.
Here's JSFiddle: 1.3.1 and 1.3.2. I don't know if it's intended or not. I can definitely see how this could be a desired behavior, but personally I would consider this a breaking change.
Thought, I would let you know asap. Will keep you updated.

Caused by 130d2e6.

So, to summ this up, what should be done is:

130d2e6 should probably be rolled back and fixed
Rewrite function we're assigning to ticker when page goes to background so it would resemble implementation from rAFShim more and would pass correct time since last call in first argument rather than true. I think it's also safe to lower framerate twice there, even though browsers slow down setTimeouts themselves..... Actually, since browsers do delay setTimeouts for background tabs we could rely on this behavior and just use setTimeout-based implementation from rAFShim - just need to make it visible to other parts of Velocity code. And there also will be less dupplicated code. At which point timestamp option will not be needed?

Expect another pull request landing in few minutes.

Not going to roll that back - but definitely needs fixing - if it's in the background then it should use the "normal" time, and not the high resolution time.

Hopefully this fixes it - can't have a "perfect" timer as the start time isn't known, but looks to be within a couple of ms of the expected time to me - which should hopefully let this PR work properly again :-)

Hopefully this fixes it - can't have a "perfect" timer as the start time isn't known

Here was my take on this. Instead of creating local polifyll for performance.now() I updated rAFShim. It uses performance.now() when present, so we can safely interchange it with native implementation of requestAnimationFrame and timestamp returned to callback will be on the same scale.
When performance.now() is not available - native version of requestAnimationFrame probably isn't present as well, so we shouldn't care, but timeCurrent is measured from the moment code definyning polifill was executed.
With this
var timeCurrent = Velocity.timestamp && timestamp !== true ? timestamp : performance.now();

could be simplified to
var timeCurrent = timestamp;

....
Now thinking of what I did with timestamps when performance API is not available, that might lead to problems in environments where window.requestAnimationFrame was monkey patched. Because verison from another polifyll will probably use just new Date().getTime() returned to the callback.
With all this in mind, I think you should update that line to simply
var timeCurrent = performance.now();

Only this way we know for sure returned values will be count from the same origin in every situation.

which should hopefully let this PR work properly again :-)

Can confirm - it works properly as is with current master. It's up to you now

Checking on MDN, performance.now() is available somewhat later than requestAnimationFrame (but both quite a few years ago) - so either way works.
If you look at #684 (specifically the graphs in it) you can see that performance.now() is definitely not good enough for normal use (it's barely better than Date.now()) - so it does need to use the rAF timestamp when available. The possibility of a "broken" performance.now() polyfill is there - but as Velocity is planning on dropping the legacy browsers that don't support it at some point I think it's ok to use.
Saying that, it might be worth improving the new polyfill code to try and detect broken API access - maybe check for decimal places, and if none then assume broken?
Not quite sure how we can make this part of the test suite either...


Difficulty recognizing Genesis 1:1 John 1:1

When attempting to parse two passages where the second passage is only separated by a single space and contains a book name that is also found in multiple books, i.e. John is found 1 John, 2 John, and 3 John etc., current parser finds a bc Genesis 1 and bcv 1 John 1:1.

Both for the sake of any human readers and the parser sequential references like this should be separated (e.g. with a coma or semicolon) to disambiguate the potential meanings.

I agree that references like these should be written in comma-separated sequences. However, there is a deficiency since the book numbers of 1, 2, or 3 John should never be attached to a previous reference via a colon like this. Any human reader would not see Genesis 1:1 John 1:1 and think the verse of Genesis 1:1 belonged to the next reference. And given the use case I'm currently working on, the references are not separated by any punctuation marks.


[new][atlas] Add "score" attribute to all the operators

Atlas search supports "score" parameter in other operators, not just the "leaf" ones (regex, queryString, range, etc.).
Add scoring support for all the operators that support it on the Atlas side.

Thanks for the feedback.

Please note that "If you have a feature request, please first raise it on the mailing list."


fix(material/chips): changed double to single click for edit

changed double click to single click to edit chips. Talkback mobile does not have double click functionality, so users cannot edit chips that are selected
Please let me know if there are any concerns with changing from double to single click!
fixes b/290806246

I'd be a bit concerned with the UX here since it can cause users to accidentally trigger edits. Also if we wanted to support selectable chips in the future, we wouldn't be able to have them be both selectable and editable if we switch to a single click.

Yeah I agree with what @crisbeto said. Another alternative might be to introduce something like a chip edit button, similar to our existing chip remove button


split: replace string checking with Strategy enum

Previously, the code was checking strings to decide which chunking strategy to use. This pull request creates a Strategy enum and moves all the responsibility of parsing the Strategy to the same time as all the other Settings fields are parsed.

Cool change! thanks


Fixed 'IndexError' error in downloading samples step (#106)

Changed langchain URL docs path, because wget don't follow redirects

I'm sure the Langchain team has a lot going on, so chiming in to verify this is the same changes I made to get the ingest step up and running.

I'm going to close this pull requests, because it was deprecated


add onstart callback fetaure

In the onStart method, you may need to interrupt dragging or placing elements, or rendering and placing elements requires obtaining data from the terminal, so the following is added:

Allow the clone method to be an asynchronous function
Add the return type of clone method: Promise and false can be returned. When false is returned, it will not be updated.

This asynchronous behavior may lead to the correct value not being obtained if the promise has not been resolved when the update is triggered.

This asynchronous behavior may lead to the correct value not being obtained if the promise has not been resolved when the update is triggered.

This asynchronous behavior may lead to the correct value not being obtained if the promise has not been resolved when the update is triggered.

Yes. I've updated this method to use Promise. resolve to ensure the return value is as expected

If it's asynchronous, then no other events should be triggered before the promise is resolved. If that's the case, it could cause the view to be blocked, which I personally think is not a reasonable requirement.

yes.

If it's asynchronous, then no other events should be triggered before the promise is resolved. If that's the case, it could cause the view to be blocked, which I personally think is not a reasonable requirement.

Yes. Asynchronous may cause render to be blocked. We can also render page by asynchronous data after the element is placed, but it will cause rendered twice. Is it bad? so I want to solve it on start.
In fact, delayed rendering after dragging (may need to be combined with remote data for page rendering, may require the user to input again for confirmation, etc.) is also a normal requirement. For example

Drag and drop to render remote components
Before placed, user needs to input some information.
Remote data needs to be verified before be placed.


Speech to Text translator using Azure cognitive services API

I have made a translator with a little of GUI work left. Can i submit my work to this repo?

Sure @HemanthSai7!. Assigning this to you right now.


integrate handlebars/Google Docs thing.

Already done! instructions in README here, and full blown markdown tutorial here.


Upload signed agreement to proposal 318

Manually add this to 318 and update the portal to reflect all 3 have signed. Still showing 1/3 here:

The “activate grant” button should be activated so that an  ADMIN can click it and activate Grant 318.

Commit https://github.com/ledgerleapllc/devxdao-backend/commit/52aa607dbc9e14b7fe1c9258cd4de73c364899f2


Standard users are unable to provision RKE1/RKE2 clusters with PSACT due to being unable to view global PSACTs

Setup

Rancher version: v2.7-head https://github.com/rancher/rancher/commit/6316baf1697cca085e073ccd5e9e7a3bc542d0e5
Browser type & version: Chrome

Describe the bug

When working with Pod Security Admission Configuration Templates (PSACT), it appears that only the admin user is able to do so successfully. In Cluster Management, below is the admin left view pane:

And here is the standard user view pane:

Standard users need to be able to work with PSACT when provisioning clusters; adding a standard user as a Cluster Owner to an admin-provisioned cluster isn't a viable workaround. As a consequence of standard users not being able to view PSACT in Cluster Management, when provisioning a cluster, the following issue is seen:

I should be able to see the default privileged, restricted options and other PSACT options that I created; only admin users are able to do this. Standard users are the de facto standard for provisioning clusters, so we'll need to ensure that PSACT is properly work.
In other words, we need the ability to view global PSACTs and apply it to a cluster spec for standard users.
To Reproduce

Setup Rancher, create a standard user and login as that user.
Navigate to Cluster Management and note that PSACT is absent; you are unable to view global PSACTs.
Attempt to provision an RKE2 cluster and underneath the Security tab, click PSA.

Verify that you are unable to select any PSACT options.

Result
Standard users are unable to provision clusters with PSACT enabled due to being unable to view global PSACTs.
Expected Result

Standard users should be able to provision clusters with PSACT enabled and should be able to view global PSACTs.

This needs to be broken into 2 separate issues.

Enhancement - Creating a global role for Managing PSACTs
Bug - Ability to view global PSACTs and applying it to a cluster spec for standard users

Here is the enhancement issue: https://github.com/rancher/rancher/issues/40289

Assigning @HarrisonWAffel to handle this.
I believe it should be about just adding the same access we have for PSPs to PSACTs in role_data.go, e.g.:

https://github.com/rancher/rancher/blob/1e993de98e9b303a4a8b1b57b583cc0f7ce16e4b/pkg/data/management/role_data.go#L50
https://github.com/rancher/rancher/blob/1e993de98e9b303a4a8b1b57b583cc0f7ce16e4b/pkg/data/management/role_data.go#L99
https://github.com/rancher/rancher/blob/1e993de98e9b303a4a8b1b57b583cc0f7ce16e4b/pkg/data/management/role_data.go#L116

Though addressing #2 may be part of https://github.com/rancher/rancher/issues/40289
Also, I'm not sure why we're adding it explicitly for Create Cluster role (#1) when it's already a part of User role (#3).
@cbron , we may need some help from your team on this. Please suggest/assign someone for this.

I'm taking a look at this as well.  Those 3 lines in role_data appear to be right.  I'm checking to see if there are any other changes needed.

@crobby , there are more hits on podsecuritypolicytemplates in role_data.go but they seem much more related to adding a separate role for handling permissions for actual CRD instances which is part of https://github.com/rancher/rancher/issues/40289.

Marking this as closed as standard users are now able to provision PSACT-enabled clusters. The internal test plan is keeping track of all the other open/blocking issues, but this issue can be closed.


[INLONG-245] add tubemq-proxy module for restrictive accessing

we may have multi masters and a lot of brokers in a productive environment,  and in many cases, there is network restriction for accessing services, it's better to have tubemq-proxy service to solve the problems.
related to https://issues.apache.org/jira/browse/TUBEMQ-244.

JIRA link - [INLONG-245] created by dockerzhang

Closed due to long periods of inactivity
Please reopen it if needed


Cast from 'MDLMaterialProperty?!' always fails

Adding SCRecorder to my project via CocoaPods causes the 'MDLMaterialProperty?!' always fails error to appear while casting from AnyObject.
For example, the warning is thrown from the following code (this example context is within the completion handler of a graph request using the Facebook SDK to extract user data):
let userId:String = result["id"] as! String

And the warning is:
Cast from 'MDLMaterialProperty?!' to unrelated type 'String' always fails

The warning appears in various other areas within the project where syntax is similar and disappears as soon as I remove header import for SCRecorder in my bridging file. This warning causes crashes unfortunately, or data not to be loaded properly.
Xcode version 7.2, Swift version 2.1.1

I'm dealing with the same problem, seems it become an issue starting with Xcode 7.2. Anyone found a solution for this yet?

Update: You need to cast 'result' as a dictionary and then you can access it like normal.

I found the solution of this problem.
Go to the SCImageView.h remove these imports:
#import <GLKit/GLKit.h>
#import <CoreImage/CoreImage.h>
And add it to the SCImageView.m file. I still don't know why, but must be a conflict with the GLKit.

Thanks for all the contributions guys @mitchellporter @marcelocotrim, going to give this fixes a try myself in the next few days.

Yeup. @marcelocotrim your fix did the trick for me, thanks.

Thank you So much @marcelocotrim , Really it fix my Problem after lots of effort.


💡 Feature: Implement file deletion

Type of Feature
💡 Feature
Current Behavior
Currently there is UI to delete a file from the project, but the action does not anything on the wails backend.
Suggested solution
The idea is to be able to delete the file from the project, if it is not needed. But if the file has references in the timeline then the option should be disabled. If the file has a reference counter of 0 then it can be deleted from the project.
Additional Context
No response

closed by (#19)


[V2] MathJax taken no effect on bing.com pages.

I was trying to typeset some LaTeX on bing.com pages, tried the following code, however it taken none changes, the formulas won't be typeset at all.
var ele = document.createElement('script');
document.head.appendChild(ele);
ele.src = 'https://cdn.bing.com/npm/mathjax@2/MathJax.js?config=TeX-AMS_CHTML';
ele.async = true;

MathJax.Hub.Queue(["Typeset", MathJax.Hub]);

I want to debug and find out why, however I don't know how to do it.
If anyone can help, I will be grateful, thanks.

The URL you have used is not a valid on, as bing.com doesn't host a copy of MathJax.  You need old load MathJax from one of the CDNs that actually provides MathJax, e.g., http://cdn.jsdelivr.net/npm/mathjax@2/MathJax.js?config=TeX-AMS_CHTML.  You may also need to set the src value before appending the scrip tag.  You don' need to set the async attribute, as a dynamically created script is already async.  Finally, you should not need to queue the typeset action, as that should be done automatically when MathJax is loaded.
Also, I'm not sure why you are using version 2 for this, a version 3 is faster and more up to date.  Try
const script = document.createElement('script');
script.src =<EMAIL_ADDRESS>document.head.appendChild(script);


new feat : password_generator in casual programs

Closes: Password Generator Program #53

Title : Password Generator Program
Name: Akarsh Ghildyal
Idenitfy yourself: SSOC Contributor

Describe the add-ons or changes you've made 📃
Made a password generator program using python, streamlit.
Give a clear description of what have you added or modifications made
Created a password generator that has option to have uppercase, lowercase, digits and special characters. User can choose at-least anyone and at-max all the options. The user also has the option of determining the length of the password, from 6-24 characters, default is set to 12. Copy to clipboard button is also available to directly copy the generated password, instead of selecting the entire text and copying it manually. This reduces the friction and enhances user experience.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested? ⚙

I have tested the changes in my local machine.
Checklist: ☑

[x] My code follows the Contributing Guidelines & Code of Conduct of this project.
[x] This PR does not contain plagiarized content.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly wherever it was hard to understand.
[x] My changes generate no new warnings.

Screenshots 📷

https://github.com/Avdhesh-Varshney/Jarvis/assets/129004417/dd38bb91-fdb7-40f5-a006-07425c0413b9

Note to reviewers 📄

None.

@Avdhesh-Varshney Okay, will get it done by tonight.

@Avdhesh-Varshney I have updated it as per your review. Kindly see the video for reference.
https://github.com/Avdhesh-Varshney/Jarvis/assets/129004417/17081a21-55e6-44be-8f58-777d052d35c1


ExpressiveModal: correct close button position

Detailed description
The close button doesn't match the specs exactly. Right now it's flushed top right at all times, and need to be adjusted. Notice how in the specs it's flushed top: 0; right: 0; on the sm breakpoint, but anything larger then that it goes to top: 16px; right: 16px;. To set the correct value we might need to use the grid gutter token.
As-is
sm, md, lg, xlg, max

To-be
sm

md, lg, xlg, max

The other thing we should do is clear the close button on sm, and md by adding an additional 48px to the top of the content container area.

Additional information

LightboxMediaViewer Functional Specs
ExpressiveModal Functional Specs

Acceptance criteria

[ ] sm breakpoint top: 0; right: 0;
[ ] md, lg, xlg, max breakpoint top: 16px; right: 16px;
[ ] Leverage correct spacing tokens.
[ ] sm, md Clear close button with additional 48px

That's how we spec'ed everything originally. But based on this study we should start changing it to flushed to corner.
@larahanlon2 @oliviaflory @JennySanchez @jacobottesen-dgc @sethibm any thoughts?


Jetson Nano - GStreamer and FFmpeg error with Reolink cameras

I am trying to get my Reolink cameras working over rtsp or rtmp but have not been able to get GStreamer or FFmpeg to properly connect to the camera.
Both FFmpeg and GStreamer will not stream my Reolink cameras on the Jetson Nano. I spent a while debugging but don't have enough knowledge about FFmpeg or GStreamer to know what is happening here. The camera will stream fine in VLC on my desktop and through a few other programs I have tested including Viseron on my x86_64 Intel 11th Gen Nuc running Debian. FFmpeg will work on the Jetson if I force the codec to 'h264' in the config but then this is using the cpu to decode so is not optimal. Forcing this codec has no change with GStreamer.
FFmpeg outputs: "Did not receive a frame" then "Restarting frame pipe"
GStreamer outputs:
ERROR: from element /GstPipeline:pipeline0/GstSplitMuxSink:mux/GstMP4Mux:mp4mux0: Could not multiplex stream.
Additional debug info:
gstqtmux.c(4561): gst_qt_mux_add_buffer (): /GstPipeline:pipeline0/GstSplitMuxSink:mux/GstMP4Mux:mp4mux0:
Buffer has no PTS.
GStreamer process has exited

These are just the last messages in the logs, I have included the full debug log for FFmpeg and the info log for GStreamer. I tried to get the debug logs for GStreamer but it output over 100MB in under a minute. I can grab this if it is needed.
When I run the FFmpeg command from the terminal it runs for a few seconds and then get a segmentation fault. If I modify the command and remove the "pipe:1" at the end it seems to work fine. I tried to remove it with output_args but then found those are only for the recorder settings so that didn't work.
Command used for testing. I got this from catching the command Viseron is running. Error output is attached.
ffmpeg_back_porch -hide_banner -loglevel info -avoid_negative_ts make_zero -fflags nobuffer -flags low_delay -strict experimental -fflags +genpts -use_wallclock_as_timestamps 1 -vsync 0 -stimeout 5000000 -c:v h264_nvv4l2dec -rtsp_transport tcp -i rtsp://user:pass@IPADDRESS:554/h264Preview_01_sub -f segment -segment_time 5 -reset_timestamps 1 -strftime 1 -c:v copy -c:a copy /tmp/back_porch/%Y%m%d%H%M%S.mp4 -vf fps=1.0 -f rawvideo -pix_fmt nv12 pipe:1

I have included logs for both GStreamer and FFmpeg. I am not sure exactly what additional info is needed but I can always provide more if required. I have tested things such as changing the password length, putting the user/pass in the url string, using rtmp, using main/sub stream from camera, a few different codecs, messing with /dev/dri permissions, and possibly some other things as I have been working on this on and off for a week or two.
viseron gstreamer error from docker.txt
viseron ffmpeg error from docker.txt
viseron ffmpeg error from terminal.txt
config settings.txt

Thank you for looking into this! I appreciate adding the option so I can test out how to get it to work. I will gladly report back and provide any logs or details that would be helpful since you said you don't have a Jetson.

No need to rush on my behalf, I'll be here when it's ready.

Sorry this took me longer than i had anticipated.
I have pushed a new update to dev which lets you set a raw_pipeline.
I dont really know what to change at this point tho.
Can you try increasing the gstreamer_loglevel and see if we get some more information?

I can definitely get more detailed logs. Unfortunately this may have to wait a short time because I needed to return the cameras I had. The Wi-Fi was unstable so I need to pull cables and deploy the poe version of the cameras. I will report back with the logs and details once I get everything set up.

I apologize for lack of updates on this. I have not been able to get time to redeploy my cameras. I do plan to come back to this once I have my new camera set up, will probably be a few more weeks though.

I will not be able to troubleshoot this issue anymore. I have decided not to be using my Reolink cameras with the Jetson. This issue can be closed if you want.


Aligning problem when doing evaluation on the replica dataset

Hi，thanks for your great work!
I'm trying to doing evaluation on replica using your scripts in replica_eval/eval_recon.py. I did the virualization and found I had problem in aligning the reconstruction mesh with the groundtruth mesh in rotation. I tried using the ICP algorithm offered in your code and tested several different threshlods. But due to the fact that the shape of the replica mesh is like a symethical cuboid, the two meshes always differ in rotation by 180 degrees on one axis. Did you encounter similar problems? Is there any solution in additon to adjust the parameters?
def get_align_transformation(mesh_rec, mesh_gt):
    """
    Get the transformation matrix to align the reconstructed mesh to the ground truth mesh.
    """
    o3d_gt_mesh = o3d.geometry.TriangleMesh(
        vertices=o3d.utility.Vector3dVector(mesh_gt.vertices),
        triangles=o3d.utility.Vector3iVector(mesh_gt.faces)
    )
    o3d_rec_mesh = o3d.geometry.TriangleMesh(
        vertices=o3d.utility.Vector3dVector(mesh_rec.vertices),
        triangles=o3d.utility.Vector3iVector(mesh_rec.faces)
    )
    o3d_gt_pc = o3d.geometry.PointCloud(points=o3d_gt_mesh.vertices)
    o3d_rec_pc = o3d.geometry.PointCloud(points=o3d_rec_mesh.vertices)
    trans_init = np.eye(4)
    threshold = align_threshold
    reg_p2p = o3d.pipelines.registration.registration_icp(
        o3d_rec_pc, o3d_gt_pc, threshold, trans_init,
        o3d.pipelines.registration.TransformationEstimationPointToPoint(),
        o3d.pipelines.registration.ICPConvergenceCriteria(max_iteration = 3000)
)
    transformation = reg_p2p.transformation
    return transformation

def calc_3d_metric(rec_meshfile, gt_meshfile, align=True):
     ...
     if align:
            transformation = get_align_transformation(mesh_rec, mesh_gt)
            mesh_rec = mesh_rec.apply_transform(transformation)

Hi @Isabel-jin I encountered the same issue. I just quickly tried to comment out lines 122-124 and that did the trick for me.
But let's wait for the official response.
`def calc_3d_metric(rec_meshfile, gt_meshfile, align=False):
"""
3D reconstruction metric.
"""
mesh_rec = trimesh.load(rec_meshfile, process=False)
mesh_gt = trimesh.load(gt_meshfile, process=False)

if align:
    transformation = get_align_transformation(rec_meshfile, gt_meshfile)
    mesh_rec = mesh_rec.apply_transform(transformation)

# found the aligned bbox for the mesh
# to_align, _ = trimesh.bounds.oriented_bounds(mesh_gt)
# mesh_gt.vertices = (to_align[:3, :3] @ mesh_gt.vertices.T + to_align[:3, 3:]).T
# mesh_rec.vertices = (to_align[:3, :3] @ mesh_rec.vertices.T + to_align[:3, 3:]).T

min_points = mesh_gt.vertices.min(axis=0) * 1.05
max_points = mesh_gt.vertices.max(axis=0) * 1.05

`

Hi，thanks for your great work! I'm trying to doing evaluation on replica using your scripts in replica_eval/eval_recon.py. I did the virualization and found I had problem in aligning the reconstruction mesh with the groundtruth mesh in rotation. I tried using the ICP algorithm offered in your code and tested several different threshlods. But due to the fact that the shape of the replica mesh is like a symethical cuboid, the two meshes always differ in rotation by 180 degrees on one axis. Did you encounter similar problems? Is there any solution in additon to adjust the parameters?
def get_align_transformation(mesh_rec, mesh_gt):
    """
    Get the transformation matrix to align the reconstructed mesh to the ground truth mesh.
    """
    o3d_gt_mesh = o3d.geometry.TriangleMesh(
        vertices=o3d.utility.Vector3dVector(mesh_gt.vertices),
        triangles=o3d.utility.Vector3iVector(mesh_gt.faces)
    )
    o3d_rec_mesh = o3d.geometry.TriangleMesh(
        vertices=o3d.utility.Vector3dVector(mesh_rec.vertices),
        triangles=o3d.utility.Vector3iVector(mesh_rec.faces)
    )
    o3d_gt_pc = o3d.geometry.PointCloud(points=o3d_gt_mesh.vertices)
    o3d_rec_pc = o3d.geometry.PointCloud(points=o3d_rec_mesh.vertices)
    trans_init = np.eye(4)
    threshold = align_threshold
    reg_p2p = o3d.pipelines.registration.registration_icp(
        o3d_rec_pc, o3d_gt_pc, threshold, trans_init,
        o3d.pipelines.registration.TransformationEstimationPointToPoint(),
        o3d.pipelines.registration.ICPConvergenceCriteria(max_iteration = 3000)
)
    transformation = reg_p2p.transformation
    return transformation

def calc_3d_metric(rec_meshfile, gt_meshfile, align=True):
     ...
     if align:
            transformation = get_align_transformation(mesh_rec, mesh_gt)
            mesh_rec = mesh_rec.apply_transform(transformation)

Do you solve the problem? I met the same problem!


Prevent confirming transactions with insufficient funds

The wallet allows users to confirm transactions without enough funds to complete them. Preventing this action would save time and resources from transactions that are going to fail. Today, we have a state on the transaction signing page that warns the users when the funds are not enough for to cover the TX fees and disallows the submit button. Still, we are not checking if the funds are enough for the whole transaction.

Steps to reproduce

Go to https://gamma.io/collections/the-explorer-guild/2497
Attempt to buy the NFT with a wallet balance between 1-59 STX

View in Explorer
https://user-images.githubusercontent.com/239215/169360100-be6d9ef7-216d-4daf-b3c7-68017a810f62.mp4
Existing warning state

Solution
A. Re-use the UX pattern for insufficient fees and display something like: "You don't have enough STX for this transaction". We would keep the state for insufficient fees for the network fees, but it should only show when the balance covers the tx but not the fees.
B. Create a single state for insufficient fees that has a more generic copy like: "You don't have enough STX for this transaction"
Question: for option A, I believe the intent of having 2 messages is in the use-case where the user sees an NFT at 60 STX, has exactly 60 STX on the wallet, but it's unaware they need extra for the fees. Is this right?

The tricky part in this case is that the contract may transfer less than or equal to 60 STX with a balance of 23.47 STX or 0 STX, which can possibly succeed in the event that the actual amount attempted is as low as 0 STX.
The post condition would have to place the constraint of more than or equal to 60 STX (or simply equal to 60 STX) for the wallet to know ahead of time that the execution will fail if attempted.

the contract may transfer less than or equal to 60 STX

What's that depending on? When would it transfer less than 60 STX?

The contract logic could determine that it's appropriate to send less than 60 STX for whatever conditional reason it finds during execution. So, it depends on things like network state.
E.g. the contract could be designed to transfer 60 STX if the block height is less than N but 30 STX if it's more than N (as a artificial example).

So, should we do something to improve this state? Or just close this issue?

I may suggest we take a broad look at this as part of the "Improve transaction signing UX" project on the roadmap (which will probably need decomposition before we take it on, so it's not too broad).


[PE-D][Tester B] Edit command does not check for duplicate tags

The edit command does not enforce the same check as the add command where duplicate tags where simply treated as the same tag.
Command Used: edit 5 t/          test 1             t/test1

Labels: severity.Medium type.FunctionalityBug
original: nicholascher/ped#6

Looks like the tags shown are not duplicates due to the spacing.
We could investigate more regardless. See how adding and editing differs for handling those two tags. And see how each of them handles actually duplicate tags.

Rejected
From testing, the edit command does act like the add command in that it automatically condenses truly duplicate tags.


correct way to set an Editable (and prefilled) Field?

So in my online template via the Hellosign template drag + drop GUI i setup a field called 'patient_fullname'.
I set its 'Me (when sending)' to signify the API will prefilll it.
I noticed you can't set in the GUI if its editable also (and if so, if its required).
So i have to when i send up the data set if its editable.
The usual code i have for other prefilled fields looks like:
request.setCustomFieldValue ("___patient_fullname___", "John TestingFullname");
but there is no optional params to do something like so (is this correct?):
request.setCustomFieldValue ("___patient_fullname___", "John TestingFullname", "AllowedEditorRole", isRequiredBoolean);
so i had to create a helper class:
public class HellosignCustomFieldHelper
{
    public static void setCustomField (TemplateSignatureRequest request, String key, String value, String editor, boolean isRequired ) {
        CustomField x = new CustomField();
        x.setEditor (editor); //this seems needed to make the field required by the 1 role we assigned to the Template in the GUI settings
        x.setIsRequired (isRequired);
        x.setName (key);
        x.setValue (value);
        //x.setType (FieldType.text);
        request.addCustomField (x);
    }
    public static void setCustomField (TemplateSignatureRequest request, String key, String value) {
        request.setCustomFieldValue (key, value);
    }
}
...
//regular non-editable field
HellosignCustomFieldHelper.setCustomField (request, "___patient_address___", "101 Cool lane");
//convert to editable field; overloaded method
HellosignCustomFieldHelper.setCustomField (request, "___patient_fullname___", "John TestingFullname", "AllowedEditorRole", true);

Does this all look like the correct way to set a prefilled field thats editable by the user (a mix of GUI and non-GUI api setting set via CustomField class)?

@armyofda12mnkeys Yes, this looks correct (I agree that the API for CustomField is verbose). I'll look into adding a convenience method for setCustomFieldValue() with my next update.

@armyofda12mnkeys It looks like we actually have that convenience method I mentioned above. Try using TemplateSignatureRequest.setCustomFieldValue(fieldName, value).


Chrome warning: /deep/ combinator is deprecated

The issue
I just updated Aurelia - TodoMVC to latest packages and structure of skeleton-navigation.
But, when running the app in Chrome 46 I see this warning in console:
/deep/ combinator is deprecated. See https://www.chromestatus.com/features/6750456638341120 for more details.

This was already mentioned in Gitter.
Quick fix
I was able to fix it locally by removing the /deep/ from here https://github.com/aurelia/templating-resources/blob/00e639/src/aurelia-templating-resources.js#L24
See also

https://groups.google.com/a/chromium.org/forum/#!msg/blink-dev/68qSZM5QMRQ/pT2YCqZSomAJ
https://drafts.csswg.org/css-scoping/#deep-combinator

It will get fixed in the future when we implement ShadowDOM v1. It's not a bug though, it's just a feature that is going away in Chrome in the near future.


Autobahn Failure: Many failures when in non-Windows over SSL

Most of the test cases fail when running on a non-Windows platform using Kestrel's built-in SSL implementation.

'Most'? Does the client just not like the self-signed cert?

No, the handshake completes and then the connection terminates abruptly at some point.

Putting in 1.1 for now for further investigation to see if there are possible product bugs in the stack

This issue was moved to aspnet/Home#2477


Service Account Annotation Formatting Error

Is this the right place to submit this?

[X] This is not a security vulnerability or a crashing bug
[X] This is not a question about how to use Istio

Bug Description
The istiod helm chart does not correctly format the serviceaccount.yaml file when pilot.serviceAccountAnnotations is provided.
When this values file is supplied:
pilot:
 serviceAccountAnnotations:
   test: test

The service account template produces:
apiVersion: v1
kind: ServiceAccount
metadata:
  name: istiod
  namespace: istio-system
  labels:
    app: istiod
    release: release-nameannotations:    test: test
---

This is the expected Output:
apiVersion: v1
kind: ServiceAccount
metadata:
  name: istiod
  namespace: istio-system
  labels:
    app: istiod
    release: release-name
  annotations:
    test: test
---

I believe this can be solved by removing the extra dashes in this file as such:
apiVersion: v1
kind: ServiceAccount
  {{- if .Values.global.imagePullSecrets }}
imagePullSecrets:
  {{- range .Values.global.imagePullSecrets }}
  - name: {{ . }}
  {{- end }}
  {{- end }}
metadata:
  name: istiod{{- if not (eq .Values.revision "") }}-{{ .Values.revision }}{{- end }}
  namespace: {{ .Values.global.istioNamespace }}
  labels:
    app: istiod
    release: {{ .Release.Name }}
  {{ if .Values.pilot.serviceAccountAnnotations -}}
  annotations:
{{ toYaml .Values.pilot.serviceAccountAnnotations | indent 4 }}
  {{- end }}
---

Version
$ istioctl version
client version: 1.22.0
control plane version: 1.22.0
data plane version: 1.21.2 (14 proxies), 1.22.0 (9 proxies)
$ helm version --short
v3.14.1+ge8858f8

Additional Information
No response

If no one is working on this issue,I would like to work on this issue

/assign  @visheshtha

Fixed, thanks all


Beta Support and Feedback Form

Task Description
Per previous discussion, we'll use a form to receive user support requests and feedback during the Beta release period.
See the following for review:

Support and Feedback Form
Form flow reference

Some pending questions:

Should we allow uploads?
Anything to omit or include?
Feedback on which responses are required or optional?

cc: @o-fernandez , @amedina , @swissspidy

Nice work on the form! 👍

Should we allow uploads?

Yes, it would be useful if people could upload screenshots 👍

Anything to omit or include?

For the "Report an issue" step, we should add add a "error message / details" field?
This would need to support pre-filling. When users click on the "Report Error" button implemented in #2256, the field should be pre-filled with the error message / stack trace.
I'm gonna take a stab at this...
Should we add the "Provide an email address where we can contact you." field to all cases?
Should we add the "How would you rate your experience with the Web Stories Editor plugin?" field to all cases?

Feedback on which responses are required or optional?

For the "Report an issue" form, should we perhaps consolidate the 3 fields into 1 longer one?
Having to fill out 3 required fields there might be too much hassle.

Note to self: we need to make the form public once it's finished 🙂

Looks like prefilling would work like this:
https://docs.google.com/forms/d/e/1FAIpQLSfRK9gFWMavvGbEWPFUFz8PDgrDE6XdVuh3B9e1Fu549V8BxQ/viewform?entry.1908864204=Report+an+issue&entry.770137249=This+is+an+error%0A%0Aon+multiple+lines

Where This+is+an+error%0A%0Aon+multiple+lines needs to be replaced with the actual error message.

Should we allow uploads?

Yes, it would be useful if people could upload screenshots 👍

+1

@ernee Could you please update the form as per above's feedback?

Hi @swissspidy!  Currently the form (and reference spreadsheet) have been updated per feedback above and separate discussion.
Special note:

we determined it was best not to allow uploads at this time
an error field is available; pending support for pre-filling from the workspace I think

Feel free to let us know if we should make any further adjustments at this time (cc: @o-fernandez)

@ernee What is the reason why we can't do upload at this time? Sometimes you need a screenshot to really explain what's going on.

@ernee What is the reason why we can't do upload at this time? Sometimes you need a screenshot to really explain what's going on.

We decided to do email follow-ups to get screenshots, for a variety of privacy and security considerations.

@ernee since we're probably gonna do a mailing list, time to close this issue?

@swissspidy we are still awaiting final confirmation about change in direction. I'll update the topic when it's complete.


Another mosquito genome

[Current coverage][cc-pull] is 100%

Merging [#68][cc-pull] into [master][cc-base-branch] will not change coverage

@@            master       #68   diff @@
========================================
  Files           17        17
  Lines         1938      1938
  Methods          0         0
  Messages         0         0
  Branches       193       193
========================================
  Hits          1938      1938
  Misses           0         0
  Partials         0         0

Powered by Codecov. Last updated by [b926aec...811d08b][cc-compare]
[cc-base-branch]: https://codecov.io/gh/standage/genhub/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/standage/genhub/compare/b926aec4d77611f99ff6ac8521135ec89718cfd0...811d08bd2e16d0f131e0fe8b7f1bbf85b90d51c8
[cc-pull]: https://codecov.io/gh/standage/genhub/pull/68?src=pr


uProxy for Android: write logic that detects if uProxy was just installed.

Depending on special behaviour that we might need (e.g. in the browser, we want the icon to be green on install), we should update CordovaBrowserApi's hasInstalledThenLoggedIn to false if uProxy Android was just installed.

Probably should use this https://developer.chrome.com/extensions/runtime#event-onInstalled

Is there any Android-relevant behavior that uses this currently?  I'm not aware of any.


upgrade to z3 4.8.11

wip

I just saw https://github.com/FStarLang/FStar/blob/master/INSTALL.md#runtime-dependency-particular-version-of-z3
I will open an issue instead.


Update SDK link to point to public github

This updates the README to point to the public repos for obtaining the SDK Python Language Bindings

Resolves Issue #2

This fix will go in with a new PR tomorrow.


Support eq and notEq

I don't think they add much value, but why not.
Maybe that loosely type narrowing helps prevent errors somewhere :)

Please add some tests that might trick the current implementation and the problems we're trying to solve, like $a == $a (always true) and also createStdClass() == createStdClass() (not always true). Thanks 😊

added some more tests, but I'm not entirely sure what I'm finding here and what is expected and what not. I think this needs a double-check from you :)

Btw. I going to add more impossible-check tests for === in a similar fashion in another PR

There's still one failure and I have no idea why.

Alright ,the error "+54: Call to static method Webmozart\Assert\Assert::eq() with stdClass and mixed will always evaluate to true." should not be reported, but the root issue here is that PHPStan actually doesn't see the function declaration, because it's not loaded.
Please rewrite these tests to use classes+methods, it's more reliable (and you don't have to require_once those files).

Thank you!