DO unable to change hostname correctly in Google Cloud

Environment

Declarative Onboarding Version: 1.27.0
BIG-IP Version: 16.1.0-0.0.19

Summary
While running DO as part of runtime-init (as part of the startup script) on our templates in Google Cloud, we observe a consistent failure where DO fails to "find the requested system device" by it's hostname. Our DO config is supposed to change the hostname from xyz.internal to xyz.local. However, we observe that it fails to do so and looks for the device by the name xyz.internal. When it fails to find it, DO fails and runtime-init fails as well.
However, when we run the runtime-init command manually afterwards, it completes without any failures and without any complaints from DO. Adding a sleep of 10-30 seconds also doesn't seem to help DO complete it's run in the first shot as part of the startup-script.
Steps To Reproduce
Steps to reproduce the behavior:

Submit the following declaration:

"declaration": {
        "schemaVersion": "1.0.0",
        "class": "Device",
        "label": "Autoscale 1NIC BIG-IP declaration for Declarative Onboarding with BIG-IQ license",
        "async": true,
        "Common": {
            "class": "Tenant",
            "My_DbVariables": {
                "class": "DbVariables",
                "provision.extramb": 1000,
                "restjavad.useextramb": true,
                "ui.advisory.color": "blue",
                "ui.advisory.text": "Provisioned via Runtime Init and DO"
            },
            "My_Dns": {
                "class": "DNS",
                "nameServers": [
                    "x.x.x.x"
                ]
            },
            "My_License": {
                "class": "License",
                "hypervisor": "gce",
                "licenseType": "licensePool",
                "licensePool": "production",
                "bigIqHost": "x.x.x.x",
                "bigIqUsername": "admin",
                "tenant": "tenant_name",
                "skuKeyword1": "LICENSE_SKU",
                "unitOfMeasure": "hourly",
                "reachable": false,
                "overwrite": false
            },
            "My_Ntp": {
                "class": "NTP",
                "servers": [
                    "0.pool.ntp.org"
                ],
                "timezone": "UTC"
            },
            "My_Provisioning": {
                "asm": "nominal",
                "class": "Provision",
                "ltm": "nominal"
            },
            "My_System": {
                "class": "System",
                "autoPhonehome": true,
                "hostname": "dd-hostname.local",
                "consoleInactivityTimeout": 0,
                "cliInactivityTimeout": 0,
                "autoCheck": true,
                "tmshAuditLog": true,
                "guiAuditLog": false,
                "mcpAuditLog": "enable",
                "preserveOrigDhcpRoutes": false
            },
            "admin": {
                "class": "User",
                "shell": "bash",
                "userType": "regular"
            }
        }
    }

Observe the following error response:

{
    "id": "c0306f89-a0d6-439e-8c19-6abd2bdc1063",
    "selfLink": "https://localhost/mgmt/shared/declarative-onboarding/task/c0306f89-a0d6-439e-8c19-6abd2bdc1063",
    "code": 404,
    "status": "ERROR",
    "message": "invalid config - rolled back",
    "errors": [
        "tryUntil: max tries reached: 01020036:3: The requested system device (/Common/dd-hostname.c.orgname.internal) was not found.",
        "tryUntil: max tries reached: 01020036:3: The requested system device (/Common/dd-hostname.c.orgname.internal) was not found."
    ],
    "result": {
        "class": "Result",
        "code": 404,
        "status": "ERROR",
        "message": "invalid config - rolled back",
        "errors": [
            "tryUntil: max tries reached: 01020036:3: The requested system device (/Common/dd-hostname.c.orgname.internal) was not found.",
            "tryUntil: max tries reached: 01020036:3: The requested system device (/Common/dd-hostname.c.orgname.internal) was not found."
        ]
    },
    "declaration": {}
}

Expected Behavior
DO should be able to change the hostname in the first shot and not fail.
Actual Behavior
DO fails with the error that it could not find the system device with the old hostname. It seems to be successful in changing the hostname, but when it lists the info in restnoded (as shown below) it doesn't seem to have changed the name.
Please include any applicable error output.
LOG - restnoded.log
Thu, 10 Feb 2022 19:28:28 GMT - fine: [f5-declarative-onboarding: restWorker.js] modify localhost /tm/sys/global-settings {"hostname":"dd-hostname.local"}
Thu, 10 Feb 2022 19:28:28 GMT - fine: [f5-declarative-onboarding: restWorker.js] {"kind":"tm:sys:global-settings:global-settingsstate","selfLink":"https://localhost/mgmt/tm/sys/global-settings?ver=16.1.0","awsApiMaxConcurrency":1,"consoleInactivityTimeout":0,"customAddr":"none","failsafeAction":"go-offline-restart-tm","fileBlacklistPathPrefix":"{/shared/3dns/} {/shared/bin/} {/shared/core/} {/appdata/core/} {/shared/datasync/} {/shared/em/} {/shared/GeoIP/} {/shared/images/} {/shared/lib/} {/shared/lib64/} {/shared/log/} {/shared/lost+found/} {/shared/mgmt/} {/shared/nfb/} {/shared/ssh/} {/shared/statsd/} {/shared/tmstat/} {/shared/vadc/} {/config/aaa/} {/config/big3d/} {/config/bigip/} {/config/filestore/} {/config/gtm/} {/config/httpd/} {/config/ntp.conf} {/config/rndc.key} {/config/ssh/} {/config/ssl/}","fileBlacklistReadOnlyPathPrefix":"{/etc/shadow}","fileLocalPathPrefix":"{/shared/} {/tmp/}","fileWhitelistPathPrefix":"{/var/local/scf} {/tmp/} {/shared/} {/config/} {/usr/share/aws/}","guiAudit":"disabled","guiExpiredCertAlert":"enabled","guiSecurityBanner":"enabled","guiSecurityBannerText":"Welcome to the BIG-IP Configuration Utility.\n\nLog in with your username and password using the fields on the left.","guiSetup":"disabled","hostAddrMode":"management","hostname":"dd-hostname.local","lcdDisplay":"enabled","ledLocator":"disabled","mgmtDhcp":"enabled","netReboot":"disabled","passwordPrompt":"********","quietBoot":"enabled","sshMaxSessionLimit":10,"sshMaxSessionLimitPerUser":-1,"sshRootSessionLimit":"disabled","sshSessionLimit":"disabled","usernamePrompt":"Username"}
Thu, 10 Feb 2022 19:28:28 GMT - fine: [f5-declarative-onboarding: systemHandler.js] Checking Users.
Thu, 10 Feb 2022 19:28:30 GMT - finest: [f5-declarative-onboarding: restWorker.js] /tm/auth/user/admin exists, modifying
Thu, 10 Feb 2022 19:28:30 GMT - fine: [f5-declarative-onboarding: restWorker.js] modify localhost /tm/auth/user/admin {"name":"admin","password":"********","shell":"bash"}
Thu, 10 Feb 2022 19:28:30 GMT - fine: [f5-declarative-onboarding: restWorker.js] {"kind":"tm:auth:user:userstate","name":"admin","fullPath":"admin","generation":0,"selfLink":"https://localhost/mgmt/tm/auth/user/admin?ver=16.1.0","description":"Admin User","encryptedPassword":"********","sessionLimit":-1,"shell":"bash","partitionAccess":[{"name":"all-partitions","role":"admin","nameReference":{"link":"https://localhost/mgmt/tm/auth/partition/all-partitions?ver=16.1.0"}}]}
Thu, 10 Feb 2022 19:28:30 GMT - info: [f5-declarative-onboarding: restWorker.js] This is a BIG-IP
Thu, 10 Feb 2022 19:28:30 GMT - info: [f5-declarative-onboarding: restWorker.js] Waiting for device to be ready.
Thu, 10 Feb 2022 19:28:30 GMT - fine: [f5-declarative-onboarding: restWorker.js] list localhost /shared/echo/available
Thu, 10 Feb 2022 19:28:30 GMT - fine: [f5-declarative-onboarding: restWorker.js] {}
Thu, 10 Feb 2022 19:28:30 GMT - fine: [f5-declarative-onboarding: restWorker.js] list localhost /shared/identified-devices/config/device-info/available
Thu, 10 Feb 2022 19:28:30 GMT - fine: [f5-declarative-onboarding: restWorker.js] {}
Thu, 10 Feb 2022 19:28:30 GMT - fine: [f5-declarative-onboarding: restWorker.js] list localhost /tm/sys/available
Thu, 10 Feb 2022 19:28:30 GMT - fine: [f5-declarative-onboarding: restWorker.js] {}
Thu, 10 Feb 2022 19:28:30 GMT - fine: [f5-declarative-onboarding: restWorker.js] list localhost /tm/cm/available
Thu, 10 Feb 2022 19:28:30 GMT - fine: [f5-declarative-onboarding: restWorker.js] {}
Thu, 10 Feb 2022 19:28:30 GMT - fine: [f5-declarative-onboarding: restWorker.js] list localhost /shared/iapp/package-management-tasks/available
Thu, 10 Feb 2022 19:28:30 GMT - fine: [f5-declarative-onboarding: restWorker.js] {}
Thu, 10 Feb 2022 19:28:30 GMT - fine: [f5-declarative-onboarding: restWorker.js] list localhost /tm/sys/mcp-state
Thu, 10 Feb 2022 19:28:30 GMT - fine: [f5-declarative-onboarding: restWorker.js] {"kind":"tm:sys:mcp-state:mcp-statestats","selfLink":"https://localhost/mgmt/tm/sys/mcp-state?ver=16.1.0","entries":{"https://localhost/mgmt/tm/sys/mcp-state/0":{"nestedStats":{"entries":{"endPlatformIdReceived":{"description":"true"},"lastLoad":{"description":"full-config-load-succeed"},"phase":{"description":"running"}}}}}}
Thu, 10 Feb 2022 19:28:30 GMT - fine: [f5-declarative-onboarding: restWorker.js] list localhost /shared/identified-devices/config/device-info
Thu, 10 Feb 2022 19:28:30 GMT - finest: socket 428 opened
Thu, 10 Feb 2022 19:28:31 GMT - fine: [f5-declarative-onboarding: restWorker.js] {"baseMac":"{}","hostMac":"{}","halUuid":"{}","chassisSerialNumber":"{}","slots":[{"volume":"HD1.1","product":"BIG-IP","version":"16.1.0","build":"0.0.19","isActive":true}],"license":{"generation":0,"lastUpdateMicros":1644521209391999},"interfaces":["1.0","mgmt"],"isIControlRestSupported":true,"icrdPort":8100,"time":1644521311307,"physicalMemory":30720,"platform":"Z100","cpu":"Intel(R) Xeon(R) CPU @ 2.00GHz","machineId":"{}","address":"{}","hostname":"dd-hostname.c.orgname.internal","version":"16.1.0","product":"BIG-IP","platformMarketingName":"BIG-IP Virtual Edition","edition":"Final","build":"0.0.19","restFrameworkVersion":"16.1.0-0.0.19","managementAddress":"{}","mcpDeviceName":"/Common/bigip1","isClustered":false,"isVirtual":true,"hypervisorType":"gce","generation":0,"lastUpdateMicros":0,"kind":"shared:resolver:device-groups:deviceinfostate","selfLink":"https://localhost/mgmt/shared/identified-devices/config/device-info"}

Thank you for your feedback. I have added this to our internal product backlog as AUTOTOOL-3012.

This issue has been resolved with the 1.30.0 release.


Some tweets labeled as non existant

The title is pretty explicit: on some specific tweets, FixTweet doesn't find the tweet. The first link problem is reproductible at the time I'm writing this, but reposting the second one cause the embed to properly displays, so I guess this isn't a permanent problem...? Don't know, maybe you'll have a clue
https://twitter.com/SheepEsports_FR/status/1719059184191168997

https://x.com/udpj2/status/1718652251919802880?s=20

It seems whatever happened with those particular URLs got cleared up in the time between then and now as they both work.

Twitter's upstream API is preeeeetty unstable these days and it's been pretty much a game of whack-a-mole trying to make my own implementation of it more reliable because of how often their servers have issues. We very rarely had upstream issues until we were forced onto Twitter's GraphQL API by them randomly killing the REST API 2.0 endpoints we were previously relying on.
The good news is that I'm working on some changes in the thread-api branch that should (hopefully) help improve overall reliability as well.
It's worth mentioning that fxtwitter.com in the past month kinda blew up and we are handling nearly 4x the requests per day now than we were a month ago (from ~500k per day 30 days ago to ~2M per day today). From a performance standpoint we can handle a lot more requests than even that, but the bottleneck is always how fat of a pipe I can build to extract data from X/Twitter's API. There's certainly a lot more work to be done to make FixTweet more reliable.

Good to know, thank you!

I'm using the fxtwitter api for a bot of my own and I'm noticing we get 200 roughly half the time and 404 the other half. You can see it for yourself if you load up the api URL for a tweet and do a hard refresh a few times.
I absolutely understand you're under a lot of load right now though, by all means take your time. It's not your fault Twitter is busted.

I managed to reproduce it returning 404 with logging. I think a recent change I made to FixTweet/elongator might be to blame as it was assuming it shouldn't retry if Twitter returns a downstream error (which happens a lot with their GraphQL server...). Hopefully this'll make a difference to ensuring the API is more reliable? I'm still making other changes to try to improve API reliability so hang tight for a bit.

I merged the thread-api changes into main now, Hopefully this will increase API stability!

Thank you! I'll comment this issue if the problem re-appears.

I'll close the issue for now, but it can be reopened if issues arise again.

Reopening this because it does (quite rarely) still happen and I haven't gotten to the bottom of why

Seems happen at increased rate since yesterday.
(maybe something messed up by storm on US last week...)

Example of a tweet labeled as non existant: https://x.com/Aiekillu/status/1793969340750131494

Problem still exists, even without using translation. Updated my last comment to add another example.

FixTweet in general had some hiccups earlier today due to the request volume being 4x normal because of world events: https://www.cnn.com/2024/07/13/politics/trump-injured-pennsylvania-rally/index.html
You can even see exactly when it happened, oops.

Tweaked some stuff, should be better now

Oh, ok, mb. I hadn't made the connection between the two events. Glad it's temporary. Thanks for your reactivity, as always.

New examples has been spotted in the wild. Might be related to nsfw content, or could simply be a coincidence.
Links (⚠ NSFW content ⚠):

https://x.com/MonkeChrome/status/1871311759056978389
https://x.com/k0ok93/status/1826982217282879597


[improve] better scoring

Feature Description
I prefer to see completions sorted via length.
Also maybe you could change score calculation such way to improve current case:
I try to find keyword import and see a lot of variant.
In my opinion when input is impor, then DynamicsCompressorNode shouldn't have bigger score than import.
https://github.com/user-attachments/assets/71881c9f-d3cc-4c2e-aff9-d3f0ef6273b9

Another example of bad scoring. I try to find isMapIterator.
https://github.com/user-attachments/assets/23f06d1f-a000-408a-acac-00af6ec7b505

v0.7.4 much more better scoring
https://github.com/user-attachments/assets/468b4753-03f0-4fd6-aa5f-6be48a9537a2

I'm sorry i think it's feature, but it's definitely bug with 0.7.6 version.

Duplicate of #497


Do we want to use a default python .gitignore file provided by github for the template? this might be a good starting place.

There is already a file that covers most things Python, but we can, of course, adjust it.

Because this is a template item we could easily edit in the future, let's just use the cookie next week and see how it goes.  Thank you for your patience with my long wishlist of things!!


Code refactoring for 0.3.0

"溶解"了template.py
"溶解"了class Helpers
其他小改动

Codecov Report

Merging #285 into dev will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##              dev     #285      +/-   ##
==========================================
- Coverage   99.33%   99.33%   -0.01%
==========================================
  Files          80       78       -2
  Lines        2854     2843      -11
==========================================
- Hits         2835     2824      -11
  Misses         19       19

Impacted Files
Coverage Δ

pyecharts/utils.py
94.11% <ø> (ø)
:arrow_up:

pyecharts/constants.py
100% <ø> (ø)
:arrow_up:

pyecharts/charts/geolines.py
96.77% <100%> (-0.11%)
:arrow_down:

pyecharts/custom/page.py
100% <100%> (ø)
:arrow_up:

test/test_constants.py
100% <100%> (ø)
:arrow_up:

pyecharts/charts/geo.py
100% <100%> (ø)
:arrow_up:

pyecharts/charts/map.py
100% <100%> (ø)
:arrow_up:

pyecharts/base.py
100% <100%> (ø)
:arrow_up:

pyecharts/conf.py
100% <100%> (ø)
:arrow_up:

test/test_engine.py
100% <100%> (ø)
:arrow_up:

... and 4 more

Continue to review full report at Codecov.

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

整体上来说没什么大问题，因为这个 PR 主要是对代码进行了整理，没涉及功能变动。

需要你们点击approve我才能merge. 现在 我们已经正规化了.


Improve export sql query to use a single join

close #1659

Codecov Report

Merging #1666 (97668e4) into master (39323ff) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1666   +/-   ##
=======================================
  Coverage   39.40%   39.40%
=======================================
  Files         167      167
  Lines       13611    13611
=======================================
  Hits         5364     5364
  Misses       7485     7485
  Partials      762      762

Continue to review full report at Codecov.

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

Looks neat. Did you benchmark?

Not since the initial benchmarking. Making some more improvements and then will benchmark again.


[Discuss][Http] http supports page increase？

Search before asking

[X] I had searched in the feature and found no similar feature requirement.

Description
Some interfaces need to turn the page to see if the data needs to be supported, for example, the user provides page number fields, and validation criteria. Can realize the loop query until the query is complete
Usage Scenario
Some interfaces need to turn pages to query data
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

This feature looks good to me. This might be useful if some endpoint can only support page queries.
Are you interested in submitting PR to support this?

I plan to do this so that the user can specify the page field name and stop condition (jsonpath validation), and then increment the page field in turn until the stop condition is met. I don't know if there's a better idea

I plan to do this so that the user can specify the page field name and stop condition (jsonpath validation), and then increment the page field in turn until the stop condition is met. I don't know if there's a better idea

Yes, we can add step for each fetch, this might make the HTTP config complex, could you provide a detail design for this.

Okay, this is a very interesting thing, so let me look at the code

Whether it is necessary to add new components, I think currently, such as gitlab and github connector are based on http-base to add retry configuration, if I change this retry logic to http-base, then other component code will be redundant. Moreover, if the page search is early, it may also be designed to encrypt the request parameters


examples: Upgrade k8s to v1.1.8

Upgrade CoreOS Version to Alpha 962.0.0
rkt run kubelet with the kubelet-wrapper
Switch to quay.io/coreos/hyperkube
Provision an ext4 root partition

Solves #100.
@aaronlevy

lgtm


CI: move to different CI provider

Hi,
The build system for sqlite3 has been affected by the decision of Travis CI to stop their free service.
I'm currently attempting to move everything to AppVeyor as a temporary solution but ideally we would move to Github Actions.
In the mean time, no new builds can be published as I do not have access to AWS S3 nor can I enable Github Actions.
The AppVeyor instance should have the required credentials to allow us to publish builds which is the only reason that I am going down that route right now.
cc @springmeyer

@kewde maybe a move to CircleCI is an option? Since that's also used for the mapbox-gl-js repo; https://github.com/mapbox/mapbox-gl-js/blob/main/.circleci/config.yml
It would be nice if this (and the S3 credential issue) is resolved quickly so the new build can be published.

This should now be done 🙂


gnomeExtensions.freon: 44 -> 45

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/gnome-shell-extension-freon/versions.
meta.description for gnomeExtensions.freon is: "GNOME Shell extension for displaying CPU, GPU, disk temperatures, voltage and fan RPM in the top panel"
meta.homepage for gnomeExtensions.freon is: "https://github.com/UshakovVasilii/gnome-shell-extension-freon"
meta.changelog for gnomeExtensions.freon is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 45 with grep in /nix/store/j65x5v7dd92cpz1k5xzn87brb9dw7aa6-gnome-shell-extension-freon-45

found 45 in filename of file in /nix/store/j65x5v7dd92cpz1k5xzn87brb9dw7aa6-gnome-shell-extension-freon-45

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

3 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
1 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
docbookrx
gnomeExtensions.freon

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/j65x5v7dd92cpz1k5xzn87brb9dw7aa6-gnome-shell-extension-freon-45 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/j65x5v7dd92cpz1k5xzn87brb9dw7aa6-gnome-shell-extension-freon-45
ls -la /nix/store/j65x5v7dd92cpz1k5xzn87brb9dw7aa6-gnome-shell-extension-freon-45/bin

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

1 package built:

gnomeExtensions.freon

Maintainer pings
cc @justinas for testing.

@justinas I tried it out in my VM, and it displays "Install lm-sensors" or something. I could get none of the sensors to work, even those that should not depend on any additional packages. If lm-sensors is required to get anything to work at all, then please patch the paths so that it automatically gets installed.
Generally, I'd also appreciate if this extension was converted to an override with fixes, so that we can use the automatically packaged extensions as basis.

@piegamesde please see https://github.com/NixOS/nixpkgs/pull/150340


jest is not recognized as an internal or external command

npm install jest-cli --save-dev
Then when I run npm test, I got jest is not recognized as an internal or external command on Windows 10.
Thanks

Can you paste your package.json?

{
"name": "reactdemo",
"version": "1.0.0",
"description": "",
"main": "index.js",
"scripts": {
"test": "jest"
},
"author": "",
"license": "ISC",
"dependencies": {
"babel-preset-react": "^6.5.0",
"babelify": "^7.2.0",
"express": "^4.13.4",
"react": "^0.14.7",
"react-dom": "^0.14.7"
},
"devDependencies": {
"jest-cli": "^0.8.2"
}
}
Thank you!
Gary

I don't really know how npm works on windows but npm install jest-cli should create a symlink in node_modules/.bin/jest that you can use to run your scripts. If this doesn't work, your setup might be broken.
This issue seems unrelated to Jest, so I'm going to close it.

I don't really know how npm works on windows but npm install jest-cli should create a symlink in node_modules/.bin/jest that you can use to run your scripts. If this doesn't work, your setup might be broken.
This issue seems unrelated to Jest, so I'm going to close it.

why is this symlink not created with yarn install? Any idea?

Eu realmente não sei como o npm funciona nas janelas, mas deve criar um simelo em que você pode usar para executar seus scripts. Se isso não funcionar, sua configuração pode estar quebrada.npm install jest-cli``node_modules/.bin/jest
Este problema não tem relação com jest, então eu vou fechá-lo.

How to solve this problem? I've tried to put jest as an environment variable and it didn't work.


I have added hover on Shuffle button

Related Issue
Closes: #113
Describe the changes you've made
I have added a hover text on shuffle button
Type of change
What sort of change have you made:

[ ] 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 run the website on my device and tested it.
Checklist:

[x ] My code follows the guidelines of this project.
[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 ] I have made corresponding changes to the documentation.
[ x] My changes generate no new warnings.
[x ] I have added tests that prove my fix is effective or that my feature works.
[ x] Any dependent changes have been merged and published in downstream modules.

Screenshots

Original
Updated

Can you please accept it for hacktoberfest 2023

Hey @Niyati3031 can you add screenshot of the whole page.

Okay i'll use that and update

Can you check now?


FD leak in passthrough mode

I added a few URLs in the passthrough mode.
I am noticing that when there is more traffic on the passthrough sites, the open fd count for SSLProxy process keep on increasing rapidly and a time comes when the system is out of file descriptors.
Internet stops working until the SSLProxy process is restarted.
I am using SSLProxy on openWRT.
Anyone else facing this issue?

This might as well be a bug in sslproxy. But we close the file descriptors of the original protocol in  protopassthrough_engage(). And the fds of the passthrough connection is closed when it is being terminated. I have checked memory leaks using valgrind, but I am not sure about fds now.
So I think it may be something else. For example, it may be due to the URLs being connected, which may be using persistent connections.
Can you enable the DEBUG_PROXY switch in Mk/main.mk, recompile, and start sslproxy with the -D4 option, which will cause sslproxy to print very verbose debug logs. Perhaps this can help you understand the reason for this issue. You can post those logs here if you want.


drivers: dma: intel-adsp-hda: add delay to stop host dma

According to hardware spec, host dma needs some delay to complete stop. In the bug the host dma is disabled in different path  in a few microseonds. The first setting disabled the host dma and called pm_device_runtime_put to power off it. The second setting found the host dma was still alive and calle pm_device_runtime_put again. This results to pm->usage checking failed.
BugLink: https://github.com/thesofproject/sof/issues/8686

Please submit this PR to SOF CI with a one-line sof/west.yml change.

updated with WAIT_FOR, thanks!

. The first setting disabled the host dma and called pm_device_runtime_put to power off it. The second setting found the host dma was still alive and calle pm_device_runtime_put again.

I guess this explains the "unbalanced suspend" warning in the corresponding logs?

Please submit this PR to SOF CI with a one-line sof/west.yml change.

This is what happens when you don't do this; DSP panics merged in the main branch: https://github.com/zephyrproject-rtos/zephyr/pull/67393#issuecomment-1889640994

Lets get a +1 and move this in please, it seems to fix bugs

I don't know about the Zephyr side but from an SOF perspective there is no rush to merge this as the failure is extremely rare.
I'd rather the current major regression (DSP panic above) to be fixed first to keep git history / bisect and validation simpler.


Add --add-registry and --block-registry options to docker daemon

The first option allows to prepend additional registries to a public
one. These will be queried one after another - in the same order
they were given - when pulling images.
For example:

    --add-registry=foo.com
    --add-registry=bar.com

Would cause a

    docker pull myapp

to search

    foo.com/myapp
    bar.com/myapp
    docker.io/myapp

The second option builds a list of banned registries. Docker daemon
will prevent user from any communication with such registries. This
option recognizes a special keyword "public" denoting public Docker
registry.

So for example adding

    --block-registry=public

to flags of previous example would result in searching the same
registries except for `docker.io/myapp`

Both flags are marked as experimental in man pages.

This would allow companies and vendors to specify registries which
contain content that the companies will not allowed to be stored at
docker.io.

The FROM statement in Dockerfile keeps its old behaviour. If the
repository is missing registry, it will be pulled from the official
one.

Docker-DCO-1.1-Signed-off-by: Dan Walsh<EMAIL_ADDRESS>(github: rhatdan)
Signed-off-by: Michal Minar<EMAIL_ADDRESS>Conflicts:
graph/pull.go
registry/config.go
registry/service.go
Conflicts:
daemon/config.go

This adds the omitted patches from 1.4.1 to 1.5.
I've tried to build a binary but failed due to missing package "github.com/docker/docker/pkg/ulimit" which, I guess, should be added with "Allow setting ulimits for containers" patch.

Added the missing files to the 1.5.0 branch.

Ok, I'll submit a PR for 1.5.0 as well. Close this if you want to.


Improve exception safety with smart pointers

:thought_balloon: Would you like to wrap any pointer data members with the class template “std::unique_ptr”?

no thanks, this will add extra complication to the library, and the library itself tracks the ownership of the pointers (and owns all pointers).
In fact we barely use any manual allocation via new at all, except in caches and a couple of internal values.

:thought_balloon: Would you become interested to apply C++ Core Guidelines for any more software components?

In fact we barely use any manual allocation via new at all, …

:thought_balloon: How does such a feedback fit to improvable source code places?
Example:
https://github.com/brainboxdotcc/DPP/blob/d4b74dae5a3fd7d2f7e9f458c90c16c4fc4fa588/src/dpp/discordclient.cpp#L103

In fact we barely use any manual allocation via new at all, …

💭 How does such a feedback fit to improvable source code places?
Example:
https://github.com/brainboxdotcc/DPP/blob/d4b74dae5a3fd7d2f7e9f458c90c16c4fc4fa588/src/dpp/discordclient.cpp#L103

there really is no need for this to be a smart pointer, it is just added complexity. Its lifetime is managed by its owning class.

:thought_balloon: Are smart pointers provided for the avoidance of undesirable software behaviour?
:eyes: Where do you see owning objects for the pointers “etf” and “zlib”?

This is the only place they are instantiated.

I know.
I interpret the linked source code in the way that the two pointer member variables “etf” and “zlib” are not initialised with “nullptr” so far.
Thus such a pointer combination is generally problematic in case an exception like “std::bad_alloc” would be thrown.

:eyes: Would you like to take another look at the advice “Item 10:  Prevent resource leaks in constructors.”?

changing everything to smart pointers is a potentially lazy option, the correct fix is shown here without needing to add complexity to the library.
https://github.com/brainboxdotcc/DPP/commit/3981360a3f5f4480eb8fd3beda0dba39fb7e62a4
We simply need to ensure that nullptr is set for those two values, and rethrow the std::bad_alloc after any cleanup. This is a much simpler change than refactoring an entire stable library (where a bad_alloc on construction of a discordclient has never happened in production in a real world system to our knowledge) to use unique_ptr.


Offer parameter to change range of FFT analyze

As noted in #277,  the range of the output of FFT's analyze() is from 20 Hz to 15000 Hz, which is great, and very useful. I'm working on an application that would be at 20,000 Hz, just outside of human hearing, and I wanted to know how we could set the range via a parameter.
Would love to help with this too!

Maybe related to freqDomain in these lines?
https://github.com/processing/p5.js-sound/blob/1e2af06c26caf89bf385745035dc2739b11bbd7a/src/fft.js#L273-L298

@elizasj @therewasaguy @mkontogiannis sorry to bother you but I wonder if this would be a relatively easy parameter to add and would like to contribute if you could offer any pointers? I couldn't find any instance of 15000 in the code besides the comments, so I wasn't sure where to propose inserting a parameter like:
minFreq = minFreq || 20;
maxFreq = minFreq || 15000;

Thank you so much, this is such a great library! 🎉


[Long Term] Support for Import / Export modules

Description
Having the pleasure and opportunity of authoring daru-io plugin-gem that extends Import / Export modules to the daru gem, I understand the amount of work that is required for adding similar IO modules to networkx. This could be a summer of work by it's own - but it can also be interfaced with daru-io as a shortcut.
If the trade-off of including daru as a runtime dependency of networkx.rb is fine (installation-wise), then bridging networkx with daru-io seems a better alternative than adding custom IO modules.

Relevant Issues : (optional)
Relevant PRs : (optional)
Type of issue :

User mode :

[ ] Installation
[x] Feature request
[ ] Bug in existing feature

Developer mode :

[ ] Code quality
[ ] Tests
[ ] Documentation

Sadly the development of daru, daru-io seems to have been stopped


Fix: Webhook could not be registered with GitHub. #22

Acknowledgments

[x] I confirm that I wrote the code based on the company guidelines.
[x] I confirm that I followed the Git flow described in the company guidelines.

Description
This fixes issue #22 by adding a new parameter for GitHub repo owner, and modifies readme to explain which repos to fork and permissions needed on the GitHub access token.
Issue(s): #22

OK, I'll check this in the upcoming days.

@isgallagher, I'm using this framework https://github.com/0x4447/0x4447-cli-node-grapes for the stack. Please read this section of the README https://github.com/0x4447/0x4447-product-s3-email#how-to-work-with-this-project. This means that you don't edit the final CF file, but the individual files.
Please make your changes accordingly to this new info.

Fixed with #24


[DiscordCoreAPI] Update to version 2022-11-19

Describe the pull request

What does your PR fix?

It improves the voice-connection's connecting and reconnecting logic. As well as significantly reduces compilation time as a result of having wrapped a bunch of inlined functions.

Which triplets are supported/not supported? Have you updated the [CI baseline]
<Linux-x64, Windows-x64>, Yes

Does your PR follow the [maintainer guide]

Yes it does, I believe.

Yes

Thank you!

Thank you!

Np!


Update store-business-verification-reqs.md

Require account level support contact information

Merged!

[like]  Leo Guo reacted to your message:

From: John Kennedy @.>
Sent: Wednesday, October 9, 2024 6:08:25 PM
To: MicrosoftDocs/windows-dev-docs @.>
Cc: Leo Guo @.>; Mention @.>
Subject: Re: [MicrosoftDocs/windows-dev-docs] Update store-business-verification-reqs.md (PR #5158)
Merged!
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/windows-dev-docs/pull/5158#issuecomment-2402967248, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AZBOCTSRHKK755VF4WSYXFLZ2VWJTAVCNFSM6AAAAABPTN7JV6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMBSHE3DOMRUHA.
You are receiving this because you were mentioned.Message ID: @.***>


Use Spark 2.4 in spark pool advance setting to submit sample WordCount Scala Jar, to succeed.

[Enter feedback here]
Use Spark 2.4 in spark pool advance setting to submit sample WordCount Scala Jar, to succeed.

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

ID: 86d23813-785f-73c5-33c6-d03c0861c17c
Version Independent ID: f927bdb0-0e32-068d-5533-e4e4b43d46e8
Content: Tutorial: Create Apache Spark job definition in Synapse Studio - Azure Synapse Analytics
Content Source: articles/synapse-analytics/spark/apache-spark-job-definitions.md
Service: synapse-analytics
Sub-service: spark
GitHub Login: @Jejiang
Microsoft Alias: jejiang

Thanks for reaching out here.  Could you please elaborate a bit in detail on your feedback so that we can take action accordingly? - Thank you

Thanks for the feedback! We have assigned the issue to the content author to further review this and update the document as appropriate.

Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. We are closing this issue for now, but if you feel that it's still a concern, please respond and let us know. If you determine another possible update to our documentation, please don't hesitate to reach out again. #please-close


feat: add TreesitterContextSeparator highlight group

This allows customization of the separator's color without having to change the more general FloatBorder group.

It seems this PR breaks tests on 0.9 but I have no idea why :thinking:


windocker: Figure out how to use longer (original) names for the pipelines

As per the notes in https://github.com/adoptium/infrastructure/issues/3286#issuecomment-2277564763 I had the reduce the job name to windbld instead of the more usual jdk21u-windows-x64-temurin style format in order to avoid exceeding path lengths. We will need to figure out a solution to this before the windows container builds can be fully integrated into the pipelines.
We have had to take this into account in the past and shorten things. I'll also note that F:\workspace is used on some of our machines instead of F:\ws and that seems to work ok, so we also need to understand if this is an extra limitation which is specific to containers. Noting also that we don't seem to be able to use a separate workspace drive like F: which isn't already in the container.

Ref: https://github.com/adoptium/infrastructure/issues/1284#issuecomment-621909378 and the associated PR https://github.com/adoptium/temurin-build/pull/1724
We force the workspace to C:/workspace if TARGET_OS=windows in order to shorten the path lengths. This does not appear to be taking effect on the docker builds (probably a good thing since C:\workspace is not in a shared location so it wouldn't work properly - we should perhaps aim for C:\jw\ws in this case ...)
11:21:50  Running on [dockerhost-azure-win2022-x64-1](https://ci.adoptium.net/computer/dockerhost%2Dazure%2Dwin2022%2Dx64%2D1/) in c:\jw\workspace\build-scripts\jobs\jdk21u\windbld
[Pipeline] {
[Pipeline] timeout
11:21:50  Timeout set to expire in 1 hr 0 min
[Pipeline] {
[Pipeline] cleanWs
11:21:50  [WS-CLEANUP] Deleting project workspace...
11:21:50  [WS-CLEANUP] Deferred wipeout is used...
11:21:51  [WS-CLEANUP] done
[Pipeline] }
[Pipeline] // timeout
[Pipeline] sh
11:21:51  + docker inspect '--format={{.Id}}' notrhel_build_image
11:21:51  + /bin/cut -d: -f2
[Pipeline] sh
11:21:52  + docker --version
11:21:52  + grep podman
[Pipeline] isUnix
[Pipeline] withEnv
[Pipeline] {
[Pipeline] bat
11:21:53
11:21:53  c:\jw\workspace\build-scripts\jobs\jdk21u\windbld>docker inspect -f . "notrhel_build_image"
11:21:53  .
[Pipeline] }
[Pipeline] // withEnv
[Pipeline] withDockerContainer
11:21:53  dockerhost-azure-win2022-x64-1 does not seem to be running inside a container
11:21:53  $ docker run -d -t -e BUILDIMAGESHA=5e151976cf54d8cb4ccd90f08eb7514c1aaf115964dd99249e4c0f0b1f81c93e
11:21:53   --init -w c:/jw/workspace/build-scripts/jobs/jdk21u/windbld/ -v c:/jw/workspace/build-scripts/jobs/jdk21u/windbld/:c:/jw/workspace/build-scripts/jobs/jdk21u/windbld/ -v c:/jw/workspace/build-scripts/jobs/jdk21u/windbld@tmp/:c:/jw/workspace/build-scripts/jobs/jdk21u/windbld@tmp/ -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** notrhel_build_image cmd.exe
11:21:57  $ docker top 1db84a1852ce39ae38058844790a46eb15c30f12c92025164d685d85567f1239
[Pipeline] {
[Pipeline] stage
[Pipeline] { (build)
[Pipeline] echo
11:21:57  USER_REMOTE_CONFIGS:

but is in the main jdk21 pipelines:
20:01:19  Running on [build-azure-win2022-x64-3](https://ci.adoptium.net/computer/build%2Dazure%2Dwin2022%2Dx64%2D3/) in c:\workspace\workspace\build-scripts\jobs\jdk21u\jdk21u-windows-x64-temurin
[Pipeline] {
[Pipeline] echo
20:01:19  checking windows
[Pipeline] echo
20:01:19  changing C:/workspace/openjdk-build/
[Pipeline] ws
20:01:19  Running in C:/workspace/openjdk-build

[Pipeline] {
[Pipeline] stage
[Pipeline] { (build)
[Pipeline] echo
20:01:19  USER_REMOTE_CONFIGS:
[Pipeline] echo

The above change to the workspace directory is in an else section which only takes place if the DOCKER_IMAGE variable is not set so this should be relatively simple to resolve, and also resolve in a way that I can use a shared folder C:\jw\ws instead of C:\workspace :-)

Testing at https://ci.adoptium.net/job/build-scripts/job/jobs/job/jdk21u/job/jdk21u-windows-x64-temurin/142/console which has started the build at the time of writing this comment (although that took three attempts due to https://github.com/adoptium/infrastructure/issues/3714


Fix bug of incorrect touch event name

Beginning of touch event is called touchstart, not gesturestart as referenced here.

Thanks @beol!


Scheme inlining works incorrect in SerializeAsV2

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
When serialize OpenApi with structure with multiple response schemes with different http codes and same models with InlineLocalReferences = true
it inlines only first model, other models are replaced with refs
  /sample:
    post:
      responses:
        '200':
          description: Success
          schema:
            type: array
            items:
              $ref: '#/definitions/PostRespDto'
        '202':
          description: Accepted
          schema:
            $ref: '#/definitions/PostRespDto'

with inlining will be
  /sample:
    post:
      responses:
        '200':
          description: Success
          schema:
            type: array
            items:
              type: object
              properties:
                SampleProperty:
                  type: string
              additionalProperties: false
        '202':
          description: Accepted
          schema:
            $ref: '#/definitions/PostRespDto'

Expected behavior
paths:
  /sample:
    post:
      responses:
        '200':
          description: Success
          schema:
            type: array
            items:
              type: object
              properties:
                SampleProperty:
                  type: string
              additionalProperties: false
        '202':
          description: Accepted
          schema:
            type: object
            properties:
              SampleProperty:
                type: string
            additionalProperties: false

Screenshots/Code Snippets
Merge request will be linked

#1151


1.15 docs

There is a list of things to be done to make the Guides v1.15 conformant:

Update Quickstart to use new testing APIs - PR #431
Components page - PR #437
async/await guide - PR https://github.com/san650/ember-cli-page-object/pull/435
dedicated moduleFor page. Initial efforts can be extracted from https://github.com/san650/ember-cli-page-object/pull/415. Or we can just provide a link to the v1.14 guides

For visibility...
Working on navigation updates for v1.15. Just created a new navigation section called "Migrating", it now includes "To v1" page, formely known as "Migrating", "Native Events" and the new one "@ember/test-helpers" page.
Here is how an updated navigation structure does look like locally now:

Getting started

Overview
Installation
Quickstart
Components
Extend page objects

Migrating

To v1
Native Events Mode
@ember/test-helpers

API Reference
... remains unchanged

@mistahenry huge thanks again for your incredible reviews and suggestions!


No deterministic/easily findable name for debug symbol directory on macOS

I recently attempted to debug a Rust program with LLDB. I was surprised to find out that LLDB wouldn't find the debug symbols, even I had been sure to compile them in.
The problem was that on macOS, the debug symbols are stored in a directory that is separate from the binary. This directory has an extension .dSYM. (To be sure, there is also symbol table compiled in the binary, and LLDB was able to show stack traces because of this.), This .dSYM directory was produced in the build, but it's stored in target/debug/deps/ instead of target/debug/. Therefore, debuggingtarget/debug/$BINARY_NAME with LLDB caused LLDB not to find the debug symbols.
This is a problem, since it's surprising; I, and I bet that many others too, expect to be able to debug the main build artefact of a debug build. Many don't necessarily find the "debuggable" binary in the subdirectory.
There's a second problem too: using VS Code, I can modify the settings of the Launch debugger command. I can set the binary name that is going to be debugged. But the binary in the directory target/debug/deps/ doesn't have a deterministic name, because of the hash appended to the name (which is of course the point there...). One could argue that being not able to set a glob filename for the debugging command is just a shortcoming of VS Code, but I think that there should also be a deterministically named debuggable binary.
The easy fix, of course, is to create a .dSYM directory in target/debug/ too, corresponding to the binary there.

Note that this is also an issue on Windows where the .pdb file takes after the name of the .exe which has a hash in its name, so the .pdb does as well.

Curiously, my friend doesn't seem to have this problem, and actually my another Mac works too.
I've had some problem with debug info before with this computer: https://github.com/brson/error-chain/issues/129#issuecomment-280523181
However, the computer is almost new (unboxed just a few weeks ago) and I haven't installed anything significant to it, which suggests that it's not about just messing with the settings.
My friend suggested that it might be that an antivirus program or something is messing with things. (It's a work computer, and I'm not allowed to uninstall the thing.)
Anyway, it seems that on some Macs, the debug tools have no problems finding the debug info, and on my case, there's some problems. But the problems go away if you have the debug info in the same folder.

@golddranks For the machines where it works, is that because SpotLight is indexing the folders involved and on the machines where it doesn't work, SpotLight is not indexing?

I actually just commented on what I think is a very related issue. @golddranks mind peeking at that and giving your thoughts about that? Specifically, is it acceptable to not lift up the *.dSYM folder from deps/ up one level? (but assume that we can drop the "weird" hash at the end)

Okay, the mystery why LLDB was unable to find the debug symbols is now solved. Indeed, like @waywardmonkeys said, my "repos" folder was in the "folders excluded from Spotlight search" on the "Privacy" tab of Spotlight settings. I haven't got a slightest idea why it was there.
Like @alexcrichton asked in the comment above, I also tested if it was able to find the symbols, if they were in the deps folder but with the same name as the main executable. It could'n find the symbols.
Long story short: if Spotlight works flawlessly, the debug symbols will be found. If it doesn't, it won't find the symbols, unless they are in the same directory, and with the same name as the binary.
However, one more thing: I also experienced some lag for them to be found. (For example, when printing a debug trace of a freshly produced binary, it wouldn't show all the info when running for the first time, but only after some seconds (or even tens of seconds) after the first run. This is described in detail here: https://github.com/brson/error-chain/issues/129#issuecomment-281214798 )
I don't think the situation is desirable even if the specific problem I had was solved. The debug symbol finding mechanism depends on Spotlight, which I find unreliable and magical.

Hi, I also came up with this issue: when cargo build hello_world example. then
gdb  ../target/debug/hello_world ,and enter the gdb command line, typing b hello_world::main with complain: Function "hello_world::main" not defined. does anyone who meet the same situation as me?
I've copy the dSYM directory to target/debug/ but nothing help.

I believe these issues have since been fixed, so closing.


Using api in Python

How can I use the API in the web based python application. Do I include the js source code in html code of the app and then directly import with load() in the python code or what?

The handtrack.js library is not callable in python.
I have previously published a python version of the same model ... https://github.com/victordibia/handtracking
-V


Drum Kit virtual version of Real Drum

Title : Drum Kit
Issue No. : #175
Code Stack : HTML, CSS & JS
Close #175
Video (mandatory)
https://github.com/apu52/METAVERSE/assets/114601400/5cbbdab1-74ce-4671-9cbe-51247cc5d3ec
Checklist:

[x] I have mentioned the issue number in my Pull Request.
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md
[x] I have updated the projectData.json file for my contribution
[x] I have added the img of my project in the assets/img section
Additional context (Mandatory )

It is virtual version of Drum Kit used in most of Rock musics and concert. This is a web app on which we can get the same feel of the Drum using Keyboards or Mouse.
Are you contributing under any Open-source programme?
CodePeak'23

@singhcoder694 Please sync your fork and update the projectData.json file....there is a merge conflict...


pipeline(ticdc): push resolved ts before handle processor position

What problem does this PR solve?

Issue Number: close #7405
What is changed and how it works?

push resolved ts before handling processor position
remove unnecessary code from the sort node

Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
`None`.

/run-all-tests


added sanity check and exceptons for non-relative urls

Added code is the almost same as:
https://github.com/python/cpython/blob/3.7/Lib/urllib/parse.py#L482-L485
and
https://github.com/python/cpython/blob/3.7/Lib/urllib/parse.py#L493-L494

Ah, nice catch! Thank you so much for this.


automatically open the worksheet URL in the browser

Many notebooks automatically open the notebook URL in the browser upon launch. This functionality is easy to implement in Java. If desired, it can be made optional, with either default, but I thought that the fewer options the better.

I like this. It might be worth trying to catch UnsupportedOperationException just in case.

I've argued against this feature a number of times, for several reasons (see #46 and #180). I'm still not that keen on it, but will perhaps have to cave in to public pressure as it's clearly something that people want!
Part of me still doesn't think it's in the right place, though, as the gorilla server is often run server side, where opening a browser doesn't make any sense. @j-pb suggested incorporating it instead into lein-gorilla. I'm not sure about that either, for the same reason. It feels to me like the real solution is to have some other thing which makes using Gorilla on the desktop more user friendly. I talked about this a bit on #184, and have experimented a bit with a few of the ideas in there, but am not really happy with any of them. I guess my overall feeling is that adding browser-auto-open is a solution to a problem that does exist, but is not the right solution in the right place. Ideas very welcome, and I'm also not absolutely against having it in core Gorilla if that works out to be a reasonable compromise. But I would like to think a bit more about whether there's a better way.

I agree that this isn't quite the right place for this code.  I have run gorilla server from a docker container with some things preinstalled and configured.
However, I don't see any issue with the feature provided a command line option to enable or disable the feature.  Since this is probably the most common use case open by default with an option such as --no-open-browser would still allow for headless deployment if desired.

Hi,
thanks for the feedback. I do not mean to insist on usefulness of the patch, I just want to provide more information.

On the other hand, I think that the hesitation is at least partly because of a more general design problem. In my opinion HTTP client/server should not ever be allowed (or at least shouldn't be encouraged) to be a remote protocol for gorilla repl.  The way the server is implemented (and it is entirely appropriate  to implement it this way), it is essentially single-user.
Instead of allowing to connect remotely via HTTP, gorilla-repl should be able to connect to a remote NREPL. That's much better from many points. I actually was going to submit a patch for that.

Hi David,
always very happy to see patches and opinions :-)
That's an interesting point you make about using Gorilla remotely over HTTP. I'm not sure that nREPL solves it though. I think the fundamental problem is that there's no such thing (that I'm aware of) of per-user port-level security. So if a user exposes an nREPL server on a port, then any other user of that machine can execute commands as the nREPL user. I think, in this regard, that it's pretty much equivalent to a user running a Gorilla process on the server.
That said, it would be extremely useful to be able to connect to remote nREPLs, so if you do have code for that then a patch would be great!
Jony


Send through link clicks events as unstructured events rather than custom contexts

Currently link_click event jsons are sent in as part of the contexts array e.g.
{
    "schema": "iglu:com.snowplowanalytics.snowplow/contexts/jsonschema/1-0-0",
    "data": [
        {
            "schema": "iglu:com.snowplowanalytics.snowplow/link_click/jsonschema/1-0-1",
            "data": {
                "targetUrl": "http://www.google.com"
            }
        },
        {
            "schema": "iglu:com.snowplowanalytics.snowplow/flash_context/jsonschema/1-0-0",
            "data": {
                "hasLocalStorage": true,
                "stageSize": {
                    "height": 893,
                    "width": 1131
                },
                "version": "WIN 11,1,102,63",
                "isDebugger": true,
                "playerType": "ActiveX",
                "hasScriptAccess": true
            }
        }
    ]
}

These should instead be sent in as an unstructured event. This means moving the JSON out of the contexts array and passing it in as a separate ue_px or ur_pr parameter (depending on whether it is base64 or url encoded)

I used that in order to move away from the fake example schema that was ported over from the java tracker tests.  What would you recommend as a small and simple schema to use as a context for the tests?

This one is not too bad, it's all mostly optional JSON numbers: https://github.com/snowplow/iglu-central/blob/master/schemas/com.snowplowanalytics.snowplow/geolocation_context/jsonschema/1-0-0


Type icons lack enum and flag types

Godot version:
133942c
Issue description:
Now that type icons have been re-implemented (#15505 🎉 ), the only types that seem to be left are the enum (which just displays an error for now) and flag icons.

(CC @djrm)

But not thinking about it, they are just ints, so not really.


[Fix] Cases alpha = 0 or alpha = 1

Fixes #35.
TODO list :

[x] Fix the issue for the dense solver
[x] Fix the issue for the sparse solver
[x] Implement new tests failing under the previous conditions

It's good practice to add a test corresponding to the configuration that you have addressed.

It's good practice to add a test corresponding to the configuration that you have addressed.

Indeed I added a set of alphas = [0.0, 0.5, 1.0] to both test_dense_solvers and test_sparse_solvers parametrizations to test all possible cases.


[SPARK-17870][MLLIB][ML]Change statistic to pValue for SelectKBest and SelectPercentile because of DoF difference

What changes were proposed in this pull request?
For feature selection method ChiSquareSelector, it is based on the ChiSquareTestResult.statistic (ChiSqure value) to select the features. It select the features with the largest ChiSqure value. But the Degree of Freedom (df) of ChiSqure value is different in Statistics.chiSqTest(RDD), and for different df, you cannot base on ChiSqure value to select features.
So we change statistic to pValue for SelectKBest and SelectPercentile
How was this patch tested?
change existing test

Test build #66783 has started for PR 15444 at commit 59ee17d.

Test build #66783 has finished for PR 15444 at commit 59ee17d.

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

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

Merged build finished. Test FAILed.

Test build #66787 has started for PR 15444 at commit b98ccdf.

Test build #66787 has finished for PR 15444 at commit b98ccdf.

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

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

Merged build finished. Test PASSed.

Merged to master


js

Checklist

[ ] You have read the Contribution Guidlines before creating this PR.
[ ] You created a branch from dev branch
[ ] Blog follows Blog Guidelines.
[ ] Checked for all grammatical errors in the blog.
[ ] Checked for any duplicate blog entry.
[ ] You PR is raised on dev branch and not onmaster.

Description of change

Author Details for Blog Post Entry

@Harsha-Ambati Thanks for contributing. This is a blog portal, we do not maintain any kind of listing. Please feel free to work on any other issues if you want. Also, go through guidelines first before creating any PR.


[BUG] Unexpected polymorphism in chains of LET-INs

Here is an MWE:
------------------ MODULE Test ---------------------
VARIABLES
    \* @type: Str -> Int;
    fun

\* @type: (a -> b) => (a -> b);
Foo(f) ==
    LET \* @type: a -> b;
        f2 == f
    IN
    LET
        d == DOMAIN f2
    IN
    [ x \in d |-> f[x] ]

Init ==
    LET F == Foo([ x \in { "a" } |-> 1 ])
    IN
    fun = F

Next ==
    UNCHANGED fun
====================================================

The checker does not fail on this test. However, in the intermediate output after the type checker, the definition of d has its own independent free type variable:
(*
  @type: (((h -> i)) => (h -> i));
*)
Foo(f) ==
  LET (*@type: (() => (h -> i)); *) f2 == f IN
  LET (*@type: (() => Set(n)); *) d == DOMAIN f2 IN [ x \in d |-> f[x] ]

This causes the type checker to complain in more complex specs.

Just posting this for future reference, this might also unblock #359.

Here is a variation of the above test that makes the type checker fail, though it should not:
------------------ MODULE Test ---------------------

VARIABLES
    \* @type: Str -> Int;
    fun

\* @type: (a -> b) => (a -> b);
Foo(f) ==
    LET \* @type: a -> b;
        f2 == f
    IN
    LET
        d == DOMAIN f2
    IN
    [ x \in d |-> f[x] ]

Init ==
    fun = Foo([ x \in { "a" } |-> 1 ])

Next ==
    UNCHANGED fun
====================================================

Here is the description of the actual bug. The type of f2 was inferred to be polymorphic and, as a result, the type of d also became polymorphic. It’s a bug, but it was impossible to see at the annotation level. I think we should make explicit the type quantifiers, which are currently implicit. With explicit quantifiers, I would have seen the bug immediately.


Feature request: Easily share Wiki Pages

When I create a wiki it is possible to announce the creation on my wall.
When I create or edit a wiki page I need to find out the address to the page and post it as a link.
I would like to easily be able to share a wiki page on my wall and preferable both;

use a wiki-link that is not a full absolute URL when I write a comment on my wall
use a share button from the wiki-edit/view-page (mine and others) where a post is created with this page as a link

I'd personally would like to edit the post before it is created but just creating a standard post would be a good start and makes it simple to continue comment in a new post.

Cheers,
Erik

This feature is now available in dev branch


Fixed the visibility of text in catalog section

Description
This PR fixes #1009

Notes for Reviewers
Fixed the visibility of text in catalog section
Signed commits

[x] Yes, I signed my commits.

LGTM @Savio629 👏

Uh-oh, a merge conflict has cropped up. Will you address, @Savio629?


rffit creation and response y used

Dear chengwang88,
thanks for the tool, I was going to test the feature set and response with another method.
I noticed that the rffit model of randomForest is provided and can be used to make new predictions. However, I would be also interested in the part that creates rffit in particular the y response variable (output value). In the associated paper from 2017 [1], it is stated that this is delta pKd(train), which is equal to pKd(train) - pKd(VINA). If pKd(VINA) is the score from Vina, could you please let me know what pKd(train) corresponds to? I have trouble tracking down this variable. I'd like to make sure to use the same response as yours such that I can make a fair comparison. Many thanks in advance.
[1] Cheng Wang 1 and Yingkai Zhang (2017). Improving Scoring-Docking-Screening Powers of Protein-Ligand Scoring Functions using Random Forest.

I just found out from the paper that pKd(train) is the experimental binding affinity. Would it be anyway possible to provide the full training data set with the response variables and labels. Thanks.


Change all interfaceId fields to use custom.fields.merchantReference instead

This resolves #62
This work should change the reference field from interfaceId to a custom field specified on the payment object. If you could give some feedback on whether this is the correct approach, that'd be great. If you want the name of the field to be different too, I can make that change.
All the unit tests past, one test fails but that seems to be failing on master too right now so it seems to be unrelated.

Thank you for the PR @zackpollard . GJ! 🎉


👋 Who's using React Zoom Pan Pinch?

We're curious to know who's using the React Zoom Pan Pinch library in their projects! 🚀
Drop the info or links to the companies, products, websites or repositories. We'd love to hear about your experiences and use cases to improve the library in the future!
Thank you for being a part of the BetterTyped community! 🙌

Using it in Toolbox

Hey @prc5! Have you thought about fixing this issue: https://github.com/BetterTyped/react-zoom-pan-pinch/issues/336
Thanks in advance!

Me! Great library

Using it on Reactive Resume :) Thank you very much for everything you've done for the library!
Link: https://rxresu.me/
GitHub: https://github.com/AmruthPillai/Reactive-Resume

We are evaluating it at present but this bug may prevent us using it in prod https://github.com/BetterTyped/react-zoom-pan-pinch/issues/369

Couldn't implement on my case because the library doesn't support pinch and pan at the same time.
Overall this looks like the best one in regards of the Api and implementation.

Used in our chat software to preview images, Great library

Using it on Custom Icicle tree. The library disable the user select. Need a feature of content editable and CTRL+V in my node. Couldn't found a way to enable user-select.

I'm a lazy dev and was just looking for a quick and easy zoom and pan tool and this seems to be the "best" one I can find. And I can't tell if I'm just doing something completely wrong but I cannot get this thing to work for the life of me and when I finally did manage to get it to work. it behaves wildly inconsistent in the sense that I can zoom in no problem but if I change anything in the code, such as add or remove a comment then refresh the page, the panning is completely broken. I'm genuinely at a loss as to what can be causing this thing to break but its driving me insane! I'm about to make my own from scratch :(

I'm a dev and using this library for an OCR project to zoom the image

Using it on https://www.notionapps.com. Thank you for the work you've put in!

We use it for https://openclimbing.org.
Example here: https://openclimbing.org/relation/17089246/climbing/Lomy nad Velkou - Borová věž.jpg
It's really good. Thanks for your work! ❤

We're using this at Reversed Digital to pan and zoom a big artboard (like Photoshop/Figma/Miro). We're not using it for images.

We are using it to control game window for our browser games. https://www.playful.cz/. There are some nuances, but overall it is very good, thanks :)

Using it for my https://figma-to-react-native.com plugin:
https://github.com/user-attachments/assets/cde8e8c8-e343-42b0-825b-668d50a5f7b0


zone.js GET http://localhost:5555/main.js 404 (Not Found)

Steps to reproduce and a minimal demo of the problem
Launch project, update and save index.html or any project files
Current behavior
After chaging index.html and attempt to reload application fails with error zone.js:101 GET http://localhost:5555/main.js 404 (Not Found)
stopping and restarting npm reloads webapp successfully
Expected/desired behavior
reload index.html without exception
Other information
Seems like component.js file is not generated in dist/dev folder after saving changes

Occurs when uncommented clean.dev, in gulp.task('build.dev'


File Download and Access Issues on iOS 17.5.1 with Cordova

Versions:
•	iOS Version: 17.5.1
•	Cordova Version: 12.0.0<EMAIL_ADDRESS>•	Cordova Plugin File Version: 8.1.0

Description:
I am experiencing an issue with downloading and accessing media files on iOS 17.5.1 using Cordova. The code works perfectly on Android devices, but on iOS, the files are downloaded but cannot be accessed or played.
Steps to Reproduce:
1.	Prompt the user to download media files.
2.	Download the files and save them in the cache directory using cordova.file.dataDirectory.
3.	Attempt to access and play the downloaded files.

Current Behavior:
On iOS, the files are downloaded successfully, but when attempting to access or play these files, the app fails to locate the files, and they appear to be missing or inaccessible.
Expected Behavior:
The downloaded files should be accessible and playable from the cache directory, similar to the behavior observed on Android devices.
Relevant Code:
setDialog(
"E' necessario scaricare i contenuti audio/video, vuoi continuare?",
"yes-no",
(answer) => {
if (answer === "yes") {
this.downloadMediaFiles();
}
}
);
getCacheDirectoryPath() {
// Ottieni il percorso della directory di cache
window.resolveLocalFileSystemURL(cordova.file.dataDirectory, (dirEntry) => {
this.cacheDirectory = dirEntry.toURL();
console.log("Cache directory: " + this.cacheDirectory);
}, (error) => {
console.error("Errore nel risolvere il file system: " + error.toString());
});
}
async downloadMediaFiles() {
this.downloadProgress = 0;
this.downloadMessage = Scaricamento di 0/${this.mediaFiles.length} file;
for (let i = 0; i < this.mediaFiles.length; i++) {
await this.downloadFile(this.server_base_path + this.mediaFiles[i]);
this.downloadProgress = ((i + 1) / this.mediaFiles.length) * 100;
this.downloadMessage = Scaricamento di ${i + 1}/${this.mediaFiles.length} file;
}
this.downloadMessage = "Download completato!";
localStorage.setItem('downloadedMediaFiles', JSON.stringify(this.mediaFiles));
setTimeout(() => {
this.showDownloadDialog = false;
}, 300);
}
async downloadFile(url) {
console.log("Download URL:", url);
try {
const response = await fetch(url);
if (!response.ok) {
throw new Error('Network response was not ok');
}
const blob = await response.blob();
const fileName = url.split('/').pop();
console.log("File name:", fileName);
window.resolveLocalFileSystemURL(this.cacheDirectory, (dirEntry) => {
  dirEntry.getFile(fileName, { create: true, exclusive: false }, (fileEntry) => {
    fileEntry.createWriter((fileWriter) => {
      fileWriter.onwriteend = () => {
        console.log("Download completato: " + fileEntry.toURL());
      };
      fileWriter.onerror = (error) => {
        console.error("Errore nel download del file: " + error.toString());
      };
      console.log(blob);
      fileWriter.write(blob);
    }, (error) => {
      console.error("Errore nella creazione del file: " + error.toString());
    });
  }, (error) => {
    console.error("Errore nel recuperare il file: " + error.toString());
  });
}, (error) => {
  console.error("Errore nel risolvere il file system: " + error.toString());
});

} catch (error) {
console.error("Errore nel download del file: " + error.toString());
}
}
video source are build like this:
<video preload="auto" playinline="" autoplay="" controls="" controlslist="nofullscreen"><source src="file:///var/mobile/Containers/Data/Application/(APP_ID)/Library/NoCloud/(FILENAME).mp4" type="video/mp4"></video>
Observations:
•	The issue appears to be specific to iOS.
•	The file download and access process works correctly on Android devices.
•	When attempting to download a file through Safari’s debugger, it can be accessed correctly if saved manually.

Possible Causes:
•	Differences in file system access permissions between iOS and Android.
•	Possible bug in cordova-plugin-file or Cordova itself on iOS 17.5.1.

Any insights or suggestions on how to resolve this issue would be greatly appreciated. Specifically, if there are known issues with cordova-plugin-file on iOS 17.5.1 or if additional configurations are required, please let me know.

Same here! Working when reading and writing using the FileSystem-API but not when passing the file-url to for example a HTML-ImageTag. Furthermore my attempt was to convert it using the window.WkWebView.convertFilePath.

The following scenarios were tested, but the issue persisted:

Older Cordova-Lib version<EMAIL_ADDRESS>Older ios-platform version<EMAIL_ADDRESS>Older cordova-plugin-file version (7.0.0)
Deploying older apps from XCode on an iPhone
Older iOS Version (iOS 12)

Workaround (not recommended though):

Converting the file into a Data-URL and then putting this url in the src-Tag of an image

Weird thing is, that when downloaded the already released version of the same app (release was about half a year ago), the issue is gone

If you're app is configured to use schemes (e.g. your html page is loaded over http(s)://... on android or app://... (the scheme is customizable on iOS) then using file:// paths in your DOM will likely get blocked by CORs. If you're app is loaded over the file:// protocol, then using file:// paths in the DOM should work as expected but is considered less secure.
v7 (if I recall correctly) of the file plugin introduce changes to produce a DOM-usable url when using the FileEntry.toURL() method.
The pasted code is not well formatted and is hard to read but here are some things to check:

Make sure .toURL() is being used.
The ionic webview may not parse the URL produced by .toURL() properly, so if the ionic webview is being used, attempt to test using the default cordova webview.

If it still doesn't work, please let us know what version of cordova-ios you're using and whether you're using schemes or loading directly the file:// protocol.

Hi, we are using<EMAIL_ADDRESS>I've already tried using the .toURL function without success. If you assumption that the change was introduced in v7 is true, then i wonder why it also didn't work when i build using an older version.

I also have the problem which @phynae mentioned above.
Passing the file URL to an image tag is not working for me, but when I access files for reading or writing using the filesystem-API everything works fine.
I am operating on<EMAIL_ADDRESS>with<EMAIL_ADDRESS>Additionally I have tried to get rid of the issue using older versions of<EMAIL_ADDRESS><EMAIL_ADDRESS>and<EMAIL_ADDRESS>but this will not change anything.
Moreover the .toURL function also did not help me to solve the problem.

Hi breautek, its been long time since last time i tried using cached file on ios, there's been new implementation or have you implemented new functionality about this problem?
Thank you for your patience

This would be specific to iOS but I've been told that since the addition of schemes on iOS there is a WKWebKit.convertFilePath API available in JS.
It's not ideal since you would have to do a platform check but you could try it as a work around:
let myImage = document.getElementById('myImage');
myImage.src = WKWebKit.convertFilePath(theFilePath);


Support --no-build for package and deploy

Is your feature request related to a problem? Please describe.
The serverless-webpack has option to use --no-build flag so we can run npx sls webpack that build webpack contents  and then run sls deploy --no-build -stage qa that should package and deploy the serverless service using the existing .webpack contents
Describe the solution you'd like
This is the PR that was raised for serverless-webpack. The advantage to use --no-build is for CI/CD where we can create single zip file and deploy the zip to each env.

sls package --package ./dist
sls deploy --no-build --stage qa (this command still generate cloudformation-template-update-stack.json and serverless-state.json files but it uses .zip file generated in step 1)

Seems like closed


Add tags & UI improvements to frontmatter

From 2.20.23 slack feedback:
since we have this nice metadata in frontmatter now, we can add tags and other stuff there - feel free to use your imagination if it helps make the end UI nicer

Keeping it simple for now. frontmatter getting parsed and used where needed.


Write docstrings

Right now the code is full of TODO: write docs comments, we need to resolve all of those before we can release the library.

The docstrings are written, this is done


docs(v1): add LisanJS to users

Motivation
This adds my recently published i18n project https://lisanjs.com to the users page.
Have you read the Contributing Guidelines on pull requests?
Yup :)

Sure. No worries. 🙏 Thank you.


Trailing dot in domain names is stripped causing search domains to be used

When executing requests against domain names such as x.com. (note the dot in the end), Armeria strips the trailing dot here.
As a result search domains are being attempted.
The . should be preserved so that DNS resolution treats the domain name as an FQDN.

It seems a trailing dot has to be removed for SNI and preserved for Host and :authority.
https://lists.w3.org/Archives/Public/ietf-http-wg/2016JanMar/0430.html


Linting error special cases not handled

Hi,
Here a few example of error given by the linting extension that might not be errors:
Error with iterator
def nb_batch = 0
Channel.of("A", "B")
    .collect{ [[ id: "all", batch: nb_batch++ ], it] }

Definition in script use in output closure
process STITCH {
    input:
    tuple val(meta), path(input)

    output:
    tuple val(meta), path("*.txt") , emit: plots , optional: { generate_input }

    script:
    def generate_input  = task.ext.args.contains( "--generateInputOnly TRUE" )
}

Assertion with comma
assert params.test, "Test argument required"

Math plugin in string closure:
def myval = 3.45
println "My val ceiled is: ${(int) Math.ceil(myval)}"
println "My val2 ceiled is: " + (int) Math.ceil(myval)

And small warning when in a closure one of the argument is not used:
ch1 = Channel.of([[id1:"A"], "my_file.vcf", "my_file.csi"])
ch2 = Channel.of([[id2:"B"], "my_file2.vcf", "my_file2.csi"])
ch1.combine(ch2)
    .map { meta1, file1, index1, meta2, file2, index2 -> [meta.id1 + meta2, [file1, file2]] }

Thanks Louis for all of the interesting edge cases. In general I will refer you to our new docs which explain the strict syntax that is enforced by the language server (and eventually Nextflow itself) and some common patterns that need to be migrated

Error with iterator

The issue is the nb_batch++. Prefix/postfix increment isn't supported in the strict syntax, instead you'll need to increment in a separate assignment:
    .collect { nb_batch += 1; [[ id: "all", batch: nb_batch ], it] }

Definition in script use in output closure

Wow, I had no idea this worked. However I'm going to say this is an anti-pattern and instead you should declare the variable without def so that it can be used in the output block without a closure:
    output:
    tuple val(meta), path("*.txt") , emit: plots , optional: generate_input

    script:
    generate_input  = task.ext.args.contains( "--generateInputOnly TRUE" )

This is also a hack but it seems to be more common and it's easier to support in the language server. In the future I aim to make it work without either hack

Assertion with comma

Groovy allows a colon or comma here, I decided it would be better to support only one for consistency and colon seems to be the more common usage by far. Sorry :/

Math plugin in string closure

The issue is actually the static cast (int). Groovy supports both (int) x and x as int, again I opted for only one, and the latter is much simpler to implement.

And small warning when in a closure one of the argument is not used

This warning will be more clear in 1.0.1, you can prefix the parameter name with _ to suppress the warning

Forgot to link the docs:
https://nextflow.io/docs/latest/vscode.html#syntax-guide
https://nextflow.io/docs/latest/reference/syntax.html#syntax-page

Thanks a lot for all the explanation.
This plugin will be really usefull for the whole community !


How to run .each in concurrently?

Like the title says: How can I run the test.each in concurrently ?
I found a stackoverflow post with same question, but it is still unanswered.

https://jestjs.io/docs/api#testconcurrenteachtablename-fn-timeout


Refreshing the page resets the theme to light mode

Describe the bug
When the page is refreshed on dark mode, the theme gets reset to light mode. It should be persisting.
Steps to reproduce

Open web app.
Click on moon icon to set dark mode theme.
Refresh the page.
See theme gets reset to light mode.

Expected behavior
It should persist the theme user has selected even on page refresh or visiting again.
Screenshots
No response
Desktop

OS: [All]
Browser: [all]
Version: [e.g. 22]

Smartphones

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
No response

Assign this to me, I would like to fix this issue.


Remove Cargo.lock from repository

I think Cargo.lock should be added to .gitignore. Now after each compilation on different machine it changes.
Also while changing .gitignore, can we also add the following 2 lines to it:
.idea/
*.iml

To ignore IDEA IDE files?

I think this is the right practise for crates producing a binary.
The cargo.lock is not supposed to be updated if the versions in it are compatible with the cargo.toml.

I think this would be ok if we have exact version number for each component. Not something like:
clap = "2"
Since now when I clone this repo and run cargo build on it Cargo.lock will be modified. It doesn't force exact versions numbers on my side.
It means that if I will create PR it will always contain modified 'Cargo.lock` in it.

I misunderstand your ticket. (the title is quite misleading)
You just want to add it to .gitignore is this correct?
I'd like to keep it as is to ensure I don't forget updating it on releases.
I'm ok with adding
.idea/
*.iml

to .gitignore.

My main point was about adding Cargo.lock to .gitignore and remove it from the git repo.
But after reading rust documentation it looks that they recommend for binaries adding Cargo.lock to the repo.
The problem here is then that cargo build shouldn't update cargo.lock on my computer. It should instead pick the right versions from this file.
Not sure why this happened.

I think the meta reason is that I updated master and not develop. You might want to retry. Closing. Please send a PR if you want to put your editor files in the .gitignore file. I'm ok with that.


#368 Add dark themed CodeMirror support for Expression Builder

Fixes https://github.com/elyra-ai/canvas/issues/368
Also fixes some issues around Empty Canvas Image and text.
Also updates codemirror version for security.
Developer's Certificate of Origin 1.1
   By making a contribution to this project, I certify that:

   (a) The contribution was created in whole or in part by me and I
       have the right to submit it under the Apache License 2.0; or

   (b) The contribution is based upon previous work that, to the best
       of my knowledge, is covered under an appropriate open source
       license and I have the right under that license to submit that
       work with modifications, whether created in whole or in part
       by me, under the same open source license (unless I am
       permitted to submit under a different license), as indicated
       in the file; or

   (c) The contribution was provided directly to me by some other
       person who certified (a), (b) or (c) and I have not modified
       it.

   (d) I understand and agree that this project and the contribution
       are public and that a record of the contribution (including all
       personal information I submit with it, including my sign-off) is
       maintained indefinitely and may be redistributed consistent with
       this project or the open source license(s) involved.

Line height needs to be adjusted

Line height, cursor color, and syntax highlighting has been adjusted, ready for review @matthoward366
line height before:

line height after:

I've also found the source of the highlighting color problem:
.CodeMirror-line::selection, .CodeMirror-line > span::selection, .CodeMirror-line > span > span::selection { background: #d7d4f0; }
.CodeMirror-line::-moz-selection, .CodeMirror-line > span::-moz-selection, .CodeMirror-line > span > span::-moz-selection { background: #d7d4f0; }

This can be adjusted


Jeeminso/parse error

Summary of the changes / Why this improves CrateDB
Fixes https://github.com/crate/crate/issues/13845
Checklist

[ ] Added an entry in CHANGES.txt for user facing changes
[ ] Updated documentation & sql_features table for user facing changes
[ ] Touched code is covered by tests
[ ] CLA is signed
[ ] This does not contain breaking changes, or if it does:

It is released within a major release
It is recorded in CHANGES.txt
It was marked as deprecated in an earlier release if possible
You've thought about the consequences and other components are adapted
(E.g. AdminUI)

More complex scenario that 6c131a3 cannot handle:
cr> create table b ("""a[1]""[2]" int[]);
CREATE OK, 1 row affected  (1.716 sec)
cr> select * from b;
+-----------+
| "a[1]"[2] |
+-----------+
+-----------+
SELECT 0 rows in set (0.008 sec)

cr> select """a[1]""[2]" from b;
SQLParseException[line 1:3: mismatched input 'a' expecting {<EOF>, 'AT', 'OR', 'AND', 'IN', 'NOT', 'BETWEEN', 'LIKE', 'ILIKE', 'IS', '=', NEQ, '<', '<=', '>', '>=', '<<', '~', '!~', '~*', '!~*', '+', '-', '*', '/', '%', '||', '::', '[', '&', '|', '#'}]

Another follow up bug to fix:
cr> create table a ("""A[1]""" int[]);
CREATE OK, 1 row affected  (1.569 sec)
cr> select """A[1]""" from a;
+--------+
| "A[1]" |
+--------+
+--------+
SELECT 0 rows in set (0.002 sec)
cr> select """A[1]"""[1] from a;
+---------------+
| """A[1]"""[1] |   --> expect "A[1]"[1] similar to above
+---------------+
+---------------+
SELECT 0 rows in set (0.002 sec)

Sorry @mfussenegger converting this back to draft,
cr> select """A[1]""[1]" from b01;   --> this is a quoted subscript expression with the base column looking like an array subscript.
UnsupportedFeatureException[Nested array access is not supported]

It is another bug needs to be handled before this or just this fix is incomplete.

See https://github.com/crate/crate/pull/13931, closing.


ignore error on a per line basis

pep8 supports both # noqa and # nopep8 at the end of a line to silence errors regarding that line.
It would be nice if pep257 had a similar capability to silence errors for a specific docstring, so that the entire file doesnt need to be excluded , or the same error in all files doesnt need to be excluded.
e.g. adding '# nopep257' after the docstring

So no(qa|pep8) is easily the biggest regret of the flake8/pep8 projects. We refuse to allow it to silence every error and that decision angers users who like to write terrible code. I'm not sure there's a good reason to add this. It might be nice to have, but the doc-string requirements are relaxed enough that I don't see any particular rule that necessitates this. I work on large projects like OpenStack where noqa is particularly abused and it leads to really inconsistent code-bases where tools like flake8 are supposed to help guide the code and its developers towards a relatively consistent practice.
What particular errors are causing you problems @jayvdb ?

noqa is a tool; how it is used depends on the developers.
It is handy when migrating code to be compliant, which is something that pywikibot is slowly doing.  We have a mandatory tox rule which we are add files to as we complete them, or they near completion and have a # noqa or two to help them across the line, so there cant be any regressions in compliance.
http://git.wikimedia.org/blob/pywikibot%2Fcore.git/b83e2e9e7dbf01800cc4b3bdef7f31d8ee775fdf/tox.ini#L32
We've encountered a few issues with pep257 itself (e.g. issue #78), some which were not even docstring related.  We've had to add ugly hacks to our codebase because of pep257 , but we're all pretty keen on cleaning up the codebase, so it is a net positive ;-).
It would be preferable to be able to use # noqa to workaround pep257 problems, when they occur - until they are fixed and released and deployed onto our build machines, if a more elegant solution isnt available.
http://git.wikimedia.org/blob/pywikibot%2Fcore.git/b83e2e9e7dbf01800cc4b3bdef7f31d8ee775fdf/pywikibot%2F__init__.py#L65
http://git.wikimedia.org/blob/pywikibot%2Fcore.git/b83e2e9e7dbf01800cc4b3bdef7f31d8ee775fdf/pywikibot%2F__init__.py#L91
The main outstanding problem in pywikibot is exceptions which use the class docstring as the exception message.  Changing that message is an interface change, so adding a '.' at the end of the class docstring should be a major version version, or we need to implement funky method of removing the '.' at runtime when the exception is raised.
http://git.wikimedia.org/blob/pywikibot%2Fcore.git/b83e2e9e7dbf01800cc4b3bdef7f31d8ee775fdf/pywikibot%2Fexceptions.py (search for noqa)
As another example, yesterday I tried to cleanup ez_setup.py, which is copied into many repos (including pywikibot), and met with resistance, which are mostly to do with personal taste wrt the pep257 fixes.
https://bitbucket.org/pypa/setuptools/pull-request/117/pep8-and-pep257-compliance/diff
IMO it is a good thing to push a module as far towards compliance as a maintainer is happy with, and noqa can help reach that objective - the maintainer has at least accepted that this line has problems in a lot of other peoples minds.

:+1: on request. any progress on this?

@jayvdb why isn't flake8's noqa directive satisfying your needs?
@oryband - this issue was opened just a couple of days ago and we haven't decided if it will be done yet.

Any progress?

@Nurdok this request is about pep257 and not flake8, which we cant run on pull requests from untrusted devs.

@jayvdb I suggest you use a tool like TravisCI which can run flake8 on PRs in a safe environment. Other than that, if the purpose of this issue is to allow you to fix the code bit-by-bit, you can also use the error selection to enable only a closed set of checks, and add to it over time.
While I do understand the request for a per-line ignore and while I have not yet decided against it, it will probably not happen any time soon due to more pressing issues. If the errors you get are localized to few files, I would just ignore the entire files, for now.

As someone who works with django I would like my classes to have docstrings, but it seems a bit silly to have it on the nested meta class.
Ideally I would like to see the errors for nested classes split from top level classes, but a noqa option would solve the issue too.

@hvdklauw Seems reasonable. Can you open a separate issue for that?

I forgot about this issue.
fwiw, my own need for this evaporated after Wikimedia CI allowed executing flake8, and I created https://github.com/jayvdb/flake8-putty so that I could disable specific codes per file & per line.


Downloading distribution tarball returns 404

The url stated in the README to download the code (https://lloydrochester.com/code/rpi_lsm9ds1-2.0.tar.gz) returns a 404 error.

Sorry about that. Please try again and it should work.


Gladius-like layout for sArena

The idea is to have another predefined layout that looks similar to Gladius:

Example from mag stream:

@devai42coding can you have a look at that?


Duplicate filenames in different paths confuse Xcode 11.3.1

I'm using XCodeGen to generate a fairly large project for a build server out of many smaller components - many of the files included have the same name (ex: LabelViewModel.swift) but different paths ex:Alpha/LabelViewModel.swift vs Beta/LabelViewModel.swift. When using the New Build System for the xcodegen generated projects, I get compiler failures for each of these sets of files:
filename "LabelViewModel.swift" used twice: 'Alpha/LabelViewModel.swift' and 'Beta/LabelViewModel.swift`
note: filenames are used to distinguish private declarations with the same name

Cleaning / clearing derived data / etc doesn't doesn't fix it, however visiting both of the LabelViewModels, removing each from the build target, and readding them will cause just those files to settle. The diff on the generated proj file after removing + readding the files looks straightforward so i'm not sure what to make of it
-               2901F66CA0AD2B6C87F30E6A /* LabelViewModel.swift in Sources */ = {isa = PBXBuildFile; fileRef = 7E9330481E62D35CE93FA361 /* LabelViewModel.swift */; };
+               7E19F7B42409B9E700846CAC /* LabelViewModel.swift in Sources */ = {isa = PBXBuildFile; fileRef = 7E9330481E62D35CE93FA361 /* LabelViewModel.swift */; };
+               7E19F7B62409B9F500846CAC /* LabelViewModel.swift in Sources */ = {isa = PBXBuildFile; fileRef = A80EF6E9F9FC23881314933F /* LabelViewModel.swift */; };
-               877FFACDC67ECFDDD74EFCBF /* LabelViewModel.swift in Sources */ = {isa = PBXBuildFile; fileRef = A80EF6E9F9FC23881314933F /* LabelViewModel.swift */; };
//...
-                               877FFACDC67ECFDDD74EFCBF /* LabelViewModel.swift in Sources */,
-                               2901F66CA0AD2B6C87F30E6A /* LabelViewModel.swift in Sources */,
//...
+                               7E19F7B42409B9E700846CAC /* LabelViewModel.swift in Sources */,
+                               7E19F7B62409B9F500846CAC /* LabelViewModel.swift in Sources */,

I'm going to also attempt to build this on the legacy build system, but that is giving me Build operation failed without specifying any errors. Individual build tasks may have failed for unknown reasons. which could be for a dozen reasons - this is my first pass at making a macro project out over 1000 source files, I'm sure it wouldn't work on the first try.
Any advice why XCode is giving me this usually unrelated bug would be greatly appriciated.

This issue seems not related to XcodeGen since I can produce this error without XcodeGen, and the Xcode pointed out the reason—filenames are used to distinguish private declarations with the same name.

removing each from the build target, and readding them will cause just those files to settle.

I don't know if we should discuss it here.
But I couldn't get rid of that error by re-adding them. Could you try again in a brand new Xcode project without XcodeGen?


a question about quantity imbalance

The 4000 images were clustered, but there were more than 3000 images in one category and less in other categories, resulting in quantity imbalance.   I want to ask how do you make the number of categories not too different.

The number of clusters and thus images per cluster depends on the sim parameter as well as on the quality of feature vectors extracted from the embedding network, currently VGG16 which is hard-coded.
In case you know that images are not supposed to be in the same cluster, try changing sim and see if they separate.
It may also be that the VGG16 features do not capture the content of your images in a way which aligns with your prior knowledge, resulting in images being too close in feature space. After all, using a pre-trained embedding network is just a simple hack that often works well enough in practice where the assumption is that new images are drawn from a distribution not terribly different from the one used to train it. You may try a different embedding net. You can also train an auto-encoder on your dataset and cluster using its latent space.

thanks for your reply. I set sim to 0.3, 0.4, 0.5, and the result is similar. How about k-mean.

Sure, k-means is an option, or any other clustering method for that matter, apart from changing how feature vectors ("fingerprints") are computed, as I mentioned above.
You can grab all fingerprints as an array of shape (n_samples, n_features) and feed that to e.g. sklearn's k-means. Have a look at examples/example_api_detailed.py, replace the line clusters = calc.cluster(...) with something like
X = np.array(list(fingerprints.values()))
labels = KMeans(n_clusters=...).fit_predict(X)

To use the lines after that using clusters, you need to map labels to file names somehow. Check how imagecluster.calc.cluster does it and adapt to the output format of KMeans. I haven't done this so far, so I'd have to look into that myself.
Also you will need to ignore time distance scaling since that is implemented as acting on the feature space distances directly in imagecluster.calc.cluster. If you want that, you need to find a way to feed distances instead of feature vectors to the clustering method of your choice. I haven't checked if sklearn supports that.
Hope that helps.

ok，thanks


Add selection of Salesforce environment type

Users should also be allowed to select the type of Salesforce environment. I could not see any parameters in the Cloudformation template, hence I have added it here.
Also note though the "sf_is_production" environment variable has been added to the "AWSSCV_vmx_packager_" function, still the function is running into error. The reason to this is that "Mode" variable in "sf_config.py" file in the python layer has been set to "Mode", the value of which is picked from "awsscv_salesforce_config_" entry in secret manager. I could not find any variable with the name "Mode" in this this entry. Hence I think the there should be an addition of "Mode" variable to the "awsscv_salesforce_config_*" secret entry while deploying the Voicemail solution.
Error seen with new Voicemail solution:
Traceback (most recent call last):
File "/opt/python/lib/python3.8/site-packages/awsscv/sf.py", line 108, in execute
return method(**kwargs, headers=self.headers)
File "/opt/python/lib/python3.8/site-packages/awsscv/sf.py", line 123, in post
return __check_resp__(r)
File "/opt/python/lib/python3.8/site-packages/awsscv/sf.py", line 162, in __check_resp__
raise AuthError('')
awsscv.sf_auth.AuthError
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/var/task/awsscv_vmx_packager.py", line 162, in lambda_handler
response = sf.create(sobject='Case', data=data)
File "/opt/python/lib/python3.8/site-packages/awsscv/sf.py", line 73, in create
resp = self.execute(self.request.post,
File "/opt/python/lib/python3.8/site-packages/awsscv/sf.py", line 110, in execute
self.init_request(refresh=True)
File "/opt/python/lib/python3.8/site-packages/awsscv/sf.py", line 100, in init_request
self.access_token = SalesforceAuth().get_access_token(refresh=refresh)
File "/opt/python/lib/python3.8/site-packages/awsscv/sf_auth.py", line 34, in get_access_token
mode = settings[SalesforceConfig.MODE_PROP]
KeyError: 'Mode'
Record 1 Result: Failed to create case

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Ignore.