fix: Suppress the warning from boost to fix build failure.

Signed-off-by: Jun Zhang<EMAIL_ADDRESS>What problem does this PR solve?
Issue Number: close #4771
Problem Summary:
What is changed and how it works?
Check List
Tests

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

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/cc @JaySon-Huang

/merge

@JaySon-Huang  @fuzhe1989 Can't build-in clang.
/dbms/src/Functions/GeoUtils.h:33:32: error: unknown warning group '-Wunused-but-set-variable', ignored [-Werror,-Wunknown-warning-option]
#pragma GCC diagnostic ignored "-Wunused-but-set-variable"
                               ^

@JaySon-Huang  @fuzhe1989 Can't build-in clang.
/dbms/src/Functions/GeoUtils.h:33:32: error: unknown warning group '-Wunused-but-set-variable', ignored [-Werror,-Wunknown-warning-option]
#pragma GCC diagnostic ignored "-Wunused-but-set-variable"
                               ^

May I ask your clang version? For me everything works fine for Clang trunk.
In fact, this flag is recently added in https://reviews.llvm.org/D100581, and I was surprised they don't even provide a release note...

@JaySon-Huang @fuzhe1989 Can't build-in clang.
/dbms/src/Functions/GeoUtils.h:33:32: error: unknown warning group '-Wunused-but-set-variable', ignored [-Werror,-Wunknown-warning-option]
#pragma GCC diagnostic ignored "-Wunused-but-set-variable"
                               ^

@jiaqizho  What's your error message for compile error? The CI build TiFlash with Clang 13.0.0 and give no warnings/errors https://ci.pingcap.net/blue/organizations/jenkins/tiflash-build-common/detail/tiflash-build-common/4096/pipeline/150

I just realized that the minimal toolchain requirement were not matched in the CMakelists.txt and README. That said cmake will only complain when clang < 5 or gcc < 7. I guess that will confuse some folks especially who first work on the project. If we really just want guarantee tiflash compiles in recent compilers, maybe we should align the requirements in CMakelists.txt @JaySon-Huang @fuzhe1989 @ywqzzy

Now we only declare the minimal toolchain requirement in the README.md (and collapsed by default). We have not changed the related warnings in CMakeLists.txt yet.
/cc @SchrodingerZhu

@JaySon-Huang @fuzhe1989 Can't build-in clang.
/dbms/src/Functions/GeoUtils.h:33:32: error: unknown warning group '-Wunused-but-set-variable', ignored [-Werror,-Wunknown-warning-option]
#pragma GCC diagnostic ignored "-Wunused-but-set-variable"
                               ^

@jiaqizho What's your error message for compile error? The CI build TiFlash with Clang 13.0.0 and give no warnings/errors https://ci.pingcap.net/blue/organizations/jenkins/tiflash-build-common/detail/tiflash-build-common/4096/pipeline/150

Apple clang version 12.0.5 (clang-1<IP_ADDRESS>)

@JaySon-Huang @fuzhe1989 Can't build-in clang.
/dbms/src/Functions/GeoUtils.h:33:32: error: unknown warning group '-Wunused-but-set-variable', ignored [-Werror,-Wunknown-warning-option]
#pragma GCC diagnostic ignored "-Wunused-but-set-variable"
                               ^

@jiaqizho What's your error message for compile error? The CI build TiFlash with Clang 13.0.0 and give no warnings/errors https://ci.pingcap.net/blue/organizations/jenkins/tiflash-build-common/detail/tiflash-build-common/4096/pipeline/150

Apple clang version 12.0.5 (clang-1<IP_ADDRESS>)

Good, that's the issue I guess. I don't have a mac so I can't really test it, but the following godbolt link shows that this flag is only supported after clang 13: https://godbolt.org/z/hWr5bss85 Maybe try to use homebrew to get yourself a more recent compiler?

We can add a line #pragma GCC diagnostic ignored "-Wunknown-warning-option" before "-Wunused-but-set-variable" to make it compatible with older clang https://github.com/pingcap/tiflash/pull/4784/files


chromatic: name build nicely

This is shown in every PR so naming it nicely makes sense.
Bug: T256864

Made me wondering... We should probably only perform the chromatic step if the request of the tests succeeded to go easy on them resources.


Use path() to avoid mixed content error - Fixes #5

Using symfony's path() method creates a relative URL to the dragsort API. Dropping the domain and scheme resolves the mixed content error noted in #5 .
See https://symfony.com/doc/current/templates.html#linking-to-pages

Thanks Michael, I've just created the 1.0.5 release!


[BUG] dingz Lamella Wrong Angle value

homebridge-dingz crashes while moving the lamella blind due to miscalculated angle. If the lamella is set to 100% and 90°, Homebridge value is 100% and 111°. Identified the following two lines which lead to this error, could you take a look? Thanks!
Log:
[dingz] Blinds moving:  down --> 0
[dingz] Setting WindowCovering 1 to position 0 and angle 111°
[dingz] POST http://<IP_ADDRESS>/api/v1/shade/1
[dingz] Error: Request failed with status code 400 (POST http://<IP_ADDRESS>/api/v1/shade/1)
[dingz] HTTP undefined Request failed with status code 400 Bad Request--> <IP_ADDRESS>

https://github.com/johannrichard/homebridge-dingz/blob/a9e848b3727d00fdcceab0056d325c4080af0d2e/src/dingzAccessory.ts#L949
https://github.com/johannrichard/homebridge-dingz/blob/a9e848b3727d00fdcceab0056d325c4080af0d2e/src/dingzAccessory.ts#L992

🙈 Could you please provide additional information The dingz firmware versions are notorious for very varying "interpretations" of the lamella angle and how they report it, and I try (...) to somehow deal with this.
See for example the following lines just a bit further down.
https://github.com/johannrichard/homebridge-dingz/blob/a9e848b3727d00fdcceab0056d325c4080af0d2e/src/dingzAccessory.ts#L1020-L1022
To look into it, I would need at least the version of your dingz' firmware and some additional status dumps. You can get the FW version and the lamella config & status by opening the following in your browser and posting the output here (obfuscate the MAC address from the info endpoint as this is potentially sensitive)
http://<IP>/info
http://<IP>/api/v1/blind_config
http://<IP>/api/v1/state

See #419, #371 for the never-ending story (different FW versions with very different behaviours).

Note to self: irrespective of the cause for the wrong values, some more sanity checks and edge-case provisions would be helpful here.

Can confirm that I'm seeing the same issue.

/info/

{"version":"1.3.30",...}

/api/v1/blind_config

{"blinds":[{"auto_calibration":true,"state":"Initialised","invert_direction":false,"lamella_time":1.8,"shade_up_time":34.12,"shade_down_time":33.78,"type":"lamella_90","min_value":0,"max_value":100,"name":"Motor"},{"auto_calibration":true,"state":"Initialised","invert_direction":false,"lamella_time":1.8,"shade_up_time":0,"shade_down_time":0,"type":"lamella_90","min_value":0,"max_value":100,"name":"Motor"}]}

/api/v1/state

{"dimmers":[],"blinds":[{"moving":"stop","position":100,"lamella":100,"readonly":false,"index":{"relative":0,"absolute":0}},{"moving":"stop","position":100,"lamella":0,"readonly":false,"index":{"relative":1,"absolute":1}}],"led":{"on":false,"hsv":"0;0;100","rgb":"FFFFFF","mode":"hsv","ramp":1023},"sensors":{"brightness":49,"light_state":"night","room_temperature":23,"uncompensated_temperature":40.875,"temp_offset":0,"cpu_temperature":56.11,"puck_temperature":37,"fet_temperature":37.2,"input_state":null,"person_present":0,"power_outputs":[{"value":0},{"value":0},{"value":0},{"value":0}]},"thermostat":{"active":false,"out":0,"on":false,"enabled":true,"target_temp":21,"mode":"heating","temp":23,"min_target_temp":17,"max_target_temp":31},"wifi":{"version":"1.3.30",...},"time":"2023-01-28 15:23:23","config":{"timestamp":1674914600}}

Interestingly, the control works from Homekit itself. It's just from the Homebridge UI itself that the functionality doesn't work correctly.

Interestingly, the control works from Homekit itself. It's just from the Homebridge UI itself that the functionality doesn't work correctly.

Just to be sure I understand correctly: if you use the Homebridge UI to control the blinds, then (and only then) Homebridge crashes?

Interestingly, the control works from Homekit itself. It's just from the Homebridge UI itself that the functionality doesn't work correctly.

Just to be sure I understand correctly: if you use the Homebridge UI to control the blinds, then (and only then) Homebridge crashes?

Yes. To be more precise Homebridge UI doesn’t crash: it logs a failed request with a 400 response code. It just fails to control blinds but keeps operating.

Thanks @pisarenko-net and @whos-c for the reporting and patience. The additional info helped me (finally) identify the culprit. An alpha-version with a fix should be out soon, followed by a beta a few days later. As I don't have all the different FW versions to test with, I will depend on you providing some input on the fix. You can wait for the beta if you wish.


[full-ci] [tests-only] Remove depthinfinity related steps from occ context

Description
This PR removes the following steps and their implementations:

When the administrator has set depth_infinity_allowed to :depth_infinity_allowed

[ ] Forward port to master https://github.com/owncloud/ocis/pull/5679

Related Issue

Part of https://github.com/owncloud/ocis/issues/5337

Motivation and Context

How Has This Been Tested?

:robot:

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Technical debt
[x] Tests only (no source changes)

Checklist:

[ ] Code changes
[ ] Unit tests added
[ ] Acceptance tests added
[ ] Documentation ticket raised:

ocis should support depth:infinity

Ocis supports depth infinity and the default is enabled, but it cannot be disabled or change the configuration. So scenarios for disabled cases are added in expected failures.

The tests still fail, does that mean ocis does not support depth=infinity? If that is the case and that behavior is intended, we should delete the tests alltogether

@individual-it Ocis supports depth infinity and the default is enabled, but it cannot be disabled or change the configuration. So scenarios for disabled cases are added in expected failures.
@micbar Can we disable the depth:infinity? or depth:infinity is always enabled in oCIS? If the latter is the case, we can remove scenarios for the disabled case.

@grgprarup
We should add that setting to the ocdav service. It will be an environment variable. @individual-it you pointed out that that could be difficult to test 😄

we could have an extra pipeline that only runs these tests and sets the correct env. variable


ddev-router fails to become ready on a too slow computer in time

Describe the bug
As experienced in the midcamp contrib zoom chat ddev ran probably in a timeout on a too slow computer (mbp early 2011 - 8gb ram and ssd)
To Reproduce
Steps to reproduce the behavior:

perform a ddev start but is healthy and running afterwards never the less

Expected behavior
ddev-router should become ready in time
Screenshots

Version and configuration information (please complete the following information):

Host computer OS and Version: MBP early 2011 8GB RAM & SSD with macOS High Sierra 10.13.6.
Docker version information (use docker version) and insert it here.

$> docker version
Client: Docker Engine - Community
 Version:           19.03.8
 API version:       1.40
 Go version:        go1.12.17
 Git commit:        afacb8b
 Built:             Wed Mar 11 01:21:11 2020
 OS/Arch:           darwin/amd64
 Experimental:      false

Server: Docker Engine - Community
 Engine:
  Version:          19.03.8
  API version:      1.40 (minimum version 1.12)
  Go version:       go1.12.17
  Git commit:       afacb8b
  Built:            Wed Mar 11 01:29:16 2020
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          v1.2.13
  GitCommit:        7ad184331fa3e55e52b890ea95e65ba581ae3429
 runc:
  Version:          1.0.0-rc10
  GitCommit:        dc9208a3303feef5b3839f4323d9beb36df0a9dd
 docker-init:
  Version:          0.18.0
  GitCommit:        fec3683

ddev version information (use ddev version)

$> ddev version
DDEV-Local version	v1.13.1
commit            	v1.13.1
db                	drud/ddev-dbserver-mariadb-10.2:v1.13.0
dba               	phpmyadmin/phpmyadmin:5
ddev-ssh-agent    	drud/ddev-ssh-agent:v1.13.0
docker            	19.03.8
docker-compose    	1.25.4
os                	darwin
router            	drud/ddev-router:v1.13.0
web               	drud/ddev-webserver:v1.13.1

config.yaml contents for the misbehaving project

APIVersion: v1.13.1
name: d9contrib
type: drupal8
docroot: ""
php_version: "7.3"
webserver_type: nginx-fpm
router_http_port: "80"
router_https_port: "443"
xdebug_enabled: false
additional_hostnames: []
additional_fqdns: []
mariadb_version: "10.3"
nfs_mount_enabled: true
provider: default
use_dns_when_possible: true
timezone: ""

Do you have any custom configuration (nginx, php, mysql) in the .ddev folder? If so, have you tried without them?

Additional context
Add any other context about the problem here. Thanks!

Thanks!

I have to thank you for today! You helped a lot to clear up a few misconceptions and misunderstandings on my side about the process. :) Just had to leave over to the Help Topics room while you were gone, since it was impossible to run two sessions alongside - one in the app the other in the browser. :/

That was delightful working together today!

@rpkoller since you seem to be able to recreate this regularly, could you please provide the output of docker logs ddev-router right after it happens? You can paste it here, or a gist (gist.github.com) might be better. Thanks!

ran the command you provided right after the input prompt was available again after startup and the error showed:
$> docker logs ddev-router
Using the local CA at "/root/.local/share/mkcert" ✨
The local CA is now installed in the system trust store! ⚡

Using the local CA at "/root/.local/share/mkcert" ✨

Created a new certificate valid for the following names 📜
 - "*.ddev.local"
 - "*.ddev.site"
 - "<IP_ADDRESS>"
 - "localhost"
 - "ddev-router"
 - "ddev-router.ddev_default"

Reminder: X.509 wildcards only go one level deep, so this won't match a.b.ddev.local ℹ

The certificate is at "/etc/nginx/certs/.crt" and the key at "/etc/nginx/certs/.key" ✅

Using the local CA at "/root/.local/share/mkcert" ✨

Sorry, I gave you the wrong command @rpkoller . What I need is docker inspect --format "{{json .State.Health }}" ddev-router, and it should be done shortly after your ddev start, thanks!
Also, I'd love it if you could test this build: https://23305-80669528-gh.circle-artifacts.com/0/~/artifacts/ddev_macos.v1.13.0-23-g678d530c.tar.gz and see if it solves your problem. You can just extract the ddev binary and run it manually with a full path.

haha I tried now 6 times in a row with ddev start but unfortunately each time ddev started up properly. unable to reproduce the issue right now. will retry later this afternoon. the only thing that changed to the last times happening is yesterday evenings high sierra security update and that slack rolled out the new functionality on a few channels (which is told to be a bit more performant). no idea if one of those or both might have had an influence on the errors before. will keep on trying later on.

nope chance to far. tried it 13 or 14 time overall at least. stopped counting at a certain point. will try later tonight or tomorrow. but for the moment i can't be of any help here unfortunately :(

Does it matter which project you're doing? Or are you always working with the same project? Does it matter how many projects are running, or whether ddev-router is already running (ddev-router is running if at least one project is up; maybe adding a second project is related? ddev list shows whether router is running)

hmmm i've played around a bit. the last time those errors taking place i had only one drupal 8 project installed and running in ddev and one wordpress project stopped (i've removed all other projects i had in there before due to the playing around with filesharing ressources and settings - where i had to remove all containers, volumes and images - but the issue also still happened yesterday still so the removal was not the reason fixing things).
with my tests earlier today the ddev-router was running. but i did a ddev poweroff and a ddev start so the router got started fresh but still no sign of the timing error.
next i tried to simply spin up one or two more drupal sites with composer to see if that error shows up when three drupal 8 sites run in parallel. but no sign of the error still:

the only thing happened was with the first of the two new projects on ddev configi got :
$> ddev config
Creating a new ddev project config in the current directory (/Users/myuser/Sites/d8test)
Once completed, your configuration will be written to /Users/myuser/Sites/d8test/.ddev/config.yaml

Project name (d8test):

The docroot is the directory from which your site is served.
This is a relative path from your project root at /Users/myuser/Sites/d8test
You may leave this value blank if your site files are in the project root
Docroot Location (web):
Found a drupal8 codebase at /Users/myuser/Sites/d8test/web.
Project Type [drupal6, drupal7, backdrop, magento, magento2, php, drupal8, wordpress, typo3] (drupal8):
Ensuring write permissions for d8test
No settings.php file exists, creating one
Existing settings.php file includes settings.ddev.php
Configuration complete. You may now run 'ddev start'.
segment 2020/03/25 17:12:02 ERROR: sending request - Post https://api.segment.io/v1/batch: net/http: request canceled while waiting for connection (Client.Timeout exceeded while awaiting headers)
segment 2020/03/25 17:12:02 ERROR: 2 messages dropped because they failed to be sent and the client was closed

but the initial issue i am still unable to reproduce :/ was a predictable given until yesterday.

The segment errors are when you're on bad internet or have pi-hole (or similar spam redirector) enabled, https://github.com/drud/ddev/issues/1968
Thanks so much for all your effort on this. Have you increased docker memory since the other day?

no i even lowered the values back to the default ones to provoke the issue.
last time the issues happened:
1cpu(out of 2)/3GB Memory/2,5GBSwap/40GB Diskimage
back to the defaults:
1/2GB/1GB/40GB

ok remembered one other detailed i might have changed since the last time the error showed. i've set the debugging of docker engine to false. reverted to true now and installed another drupal site with composer. but also with debugging set to true again ddev start hasnt ran into the issue. so i got 4 drupal 8 sites running alongside now.


Example does not seem to be working

CREATE WORKLOAD GROUP wgDataLoads WITH
( REQUEST_MIN_RESOURCE_GRANT_PERCENT = 9.6
,MIN_PERCENTAGE_RESOURCE = 9.6
,CAP_PERCENTAGE_RESOURCE = 38.4
,QUERY_EXECUTION_TIMEOUT_SEC = 0)
Fails in Synapse with the following error:
"Msg 105611, Level 16, State 1, Line 44
CREATE/ALTER WORKLOAD GROUP statement failed because a non-integer value 9.6 for MIN_RESOURCE_PERCENTAGE is not allowed.  The value should be an integer between 0 and 100."

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

ID: 06e5e4fb-6dcf-3bef-9123-a8b4a7a690e4
Version Independent ID: 34eb1881-ad77-912e-e164-b888d18b4f85
Content: Convert resource class to a workload group - Azure Synapse Analytics
Content Source: articles/synapse-analytics/sql-data-warehouse/sql-data-warehouse-how-to-convert-resource-classes-workload-groups.md
Service: synapse-analytics
Sub-service: sql-dw
GitHub Login: @ronortloff
Microsoft Alias: rortloff

Hello @Robert-Kostecki , Please share a link to the documentation you are referencing. This will help us jump-start the investigation.

Apologies this was opened from the documentation page and I assumed that it would be linked. Please see below: https://docs.microsoft.com/en-us/azure/synapse-analytics/sql-data-warehouse/sql-data-warehouse-how-to-convert-resource-classes-workload-groups

We have assigned the issue to the content author . They will evaluate & update the document as appropriate

Thanks for the feedback.  I switched the code example to be based off of mediumrc which is 10%.

#please-close


Add regional awareness

Currently, there is only a central cache location for all servers. We will be expanding to multiple geographies as Azure Confidential Computing expands.
The client must become for regionally-aware so we route requests to the correct endpoint.

Instead of using regional awareness, we'll instead rely on performance-based DNS routing.
Repurposing this issue to track the base URL/hostname change to the global endpoint that will be set up soon(ish). Tentative plan is end of month to have fault tolerance and automatic failover in place.

This has been fixed - global endpoint has been up for a while


fix(types): fix type exports

🔗 Linked issue
https://github.com/twoslashes/twoslash/pull/42#discussion_r1631001129
❓ Type of change

[ ] 📖 Documentation (updates to the documentation, readme, or JSdoc annotations)
[x] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
Jiti uses a special build step to compile the default export to export =. The generated type definitions don’t match this. To resolve this, a manual type declaration file was added.
📝 Checklist

[x] I have linked an issue or discussion.
[x] I have updated the documentation accordingly.

If you’re doing a semver major, I suggest another solution: Use a named export.

Defenitly. Default will be for retro compat but yes i am planning to use ESM + named.

In that case I don’t recommend merging this into the 1.x branch. Feel free to merge or close this as you like. :)

Thanks for PR again. jiti v2 is almost ready with named { createJiti } export.


How to install it?

This looks cool, can you a write a brief explanation on how to install it for people who are not experienced with js ?
Thanks.

helllo , sorry brother but the projetc is not complete yet , just a start
of work in 2 days i will try to finish it ;) and i will add description :D
2017-01-08 21:03 GMT+01:00 mohamed aziz knani<EMAIL_ADDRESS>
This looks cool, can you a write a brief explanation on how to install it
for people who are not experienced with js ?
Thanks.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Rebaiahmed/TuniRap/issues/1, or mute the thread
https://github.com/notifications/unsubscribe-auth/AMcvP3yncai_zT_cgcnrocdqL_ROyADJks5rQUEbgaJpZM4LdyEN
.


New version: tencent.tencentmeeting <IP_ADDRESS>6

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

This seems to be a beta version... 🤔

How can you tell? I don't read structure letters so if it says that go ahead and fix it 😁

https://meeting.tencent.com/download-win.html?from=1000&fromSource=1
This page downloads stable version. It is now at <IP_ADDRESS>4.
https://meeting.tencent.com/download-beta.html?from=1000&fromSource=1&fromSource=1&type=3
This page downloads beta version. It is now at <IP_ADDRESS>6.
I am working on pushing this package to the automation (https://github.com/vedantmgoyal2009/winget-pkgs-automation/issues/230). However the current version (<IP_ADDRESS>6) in winget-pkgs is newer than that in the stable channel (<IP_ADDRESS>4), which blocks the process.
@lychichem How about waiting for newer stable version or removing existing beta versions so that the automation process could get passed?

https://meeting.tencent.com/download-win.html?from=1000&fromSource=1 This page downloads stable version. It is now at <IP_ADDRESS>4.
https://meeting.tencent.com/download-beta.html?from=1000&fromSource=1&fromSource=1&type=3 This page downloads beta version. It is now at <IP_ADDRESS>6.
I am working on pushing this package to the automation (vedantmgoyal2009/winget-pkgs-automation#230). However the current version (<IP_ADDRESS>6) in winget-pkgs is newer than that in the stable channel (<IP_ADDRESS>4), which blocks the process.
@lychichem How about waiting for newer stable version or removing existing beta versions so that the automation process could get passed?

If it blocks automation, I need to say apologize to you. The reason for adding beta version is even if I installed a stable version of tencentmeeting on my machine, every time when I start it, the software will check for beta version and show a red dot, so I will install a new version...


Feature / expand height while keyboard is visible

This PR resolves #43 issue.
Huge thanks to @arunim3405 and his comment that helped us get on the correct path to resolve that.
https://user-images.githubusercontent.com/39670088/143773571-52f4af17-c384-4842-b18b-4f82f36de340.mov

Glad I could help, awesome work 🙌

Nice job guys!


Blueprint does not consider extension installation recommendations

Bug Description:
If a repository recommends extensions to be installed via extensions.json, Theia blueprint does not suggest to install them.
Steps to Reproduce:
Checkout a repository with recommended extensions, e.g. https://github.com/eclipse-theia/theia
No recommendations are shown.
Should suggest eslint, see https://github.com/eclipse-theia/theia/blob/master/.vscode/extensions.json

@JonasHelming it should be available when bumping the theia version https://github.com/eclipse-theia/theia/pull/9043.

Great! Let us leave this open until then, I assigned a label to keep track

Closed thanks to #141.

@vince-fugnitto : Have you verified this in the Blueprint build or only upstream?

@vince-fugnitto : Have you verified this in the Blueprint build or only upstream?

During the review, please feel feee to confirm as well: https://github.com/eclipse-theia/theia-blueprint/pull/141#pullrequestreview-728546106.

During the review, please feel free to confirm as well: #141 (review).

I have not re-tested this, but I have confirmed that the new Blueprint installers, based on Theia v1.16.0, were correctly deployed. (linux: https://www.eclipse.org/downloads/download.php?file=/theia/latest/linux/TheiaBlueprint.AppImage&r=1)


Outdated TypeScript definitions

The bundled typings file (https://github.com/zurb/foundation-sites/blob/develop/js/typescript/foundation.d.ts) seem to be outdated, referring to Foundation 6.3.
I'm not sure how in sync the file is but I can tell that at least responsiveMenu is missing.


Fix duplicate link and duplicate note bugs

Previously the code would break if duplicate links or notes would be added to a person.
Let's do exception handling and return an appropriate message for the same!

Codecov Report
Base: 50.05% // Head: 49.91% // Decreases project coverage by -0.14% :warning:

Coverage data is based on head (65f6168) compared to base (9e38aa0).
Patch coverage: 34.78% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #253      +/-   ##
============================================
- Coverage     50.05%   49.91%   -0.15%
  Complexity      743      743
============================================
  Files           143      143
  Lines          3544     3562      +18
  Branches        420      424       +4
============================================
+ Hits           1774     1778       +4
- Misses         1615     1626      +11
- Partials        155      158       +3

Impacted Files
Coverage Δ

...main/java/seedu/clinkedin/model/person/Person.java
62.71% <ø> (ø)

...clinkedin/storage/JsonSerializableAddressBook.java
93.33% <0.00%> (-6.67%)
:arrow_down:

...ava/seedu/clinkedin/storage/JsonAdaptedPerson.java
75.49% <15.38%> (-8.21%)
:arrow_down:

...seedu/clinkedin/logic/commands/AddLinkCommand.java
79.31% <50.00%> (-5.31%)
:arrow_down:

...seedu/clinkedin/logic/commands/AddNoteCommand.java
86.20% <100.00%> (+1.59%)
:arrow_up:

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


validate Ducktype files?

There's any Ducktype validator?
How can I debug what I'm writing? AFAIK now I must first convert to xml using mallard-ducktype and then use yelp-check validate. But this is cumersome, as I have to check the .page file and then work on the .duck file.
I also wonder what's the adoption status of Ducktype. I think that all GNOME projects use the Mallard spec only, right? What is preventing the use of Ducktype?

A validator would be very nice. For most of the checks in yelp-check, we could just teach yelp-check how to convert behind the scenes. But for validation, as you mention, there's line numbers involved, and that's annoying.
I'm going to move this issue to the mallard-ducktype repo, since that's where tooling like this would probably happen.
As for GNOME adoption, I think part of it is that the build system integration isn't there, especially for translations. But a bigger part is that there are only a few active contributors, and when there's already so many maintainable docs in XML, people don't see switching as a good use of time. Probably the most likely route is that some new apps will use it before you see a full conversion of something like gnome-user-docs.


Fix #257 Apply full width to spin outside of table so that the table overflows only within itself

What type of PR is this?
/kind ui-change
What this PR does / why we need it:
The longest table cell (corresponding to log field of action receipt table) overflows from the screen and it needs a fix.
Which issue(s) this PR fixes:
Fixes #257
Special notes for your reviewer:
This is my first PR. Feel free to let me know if there's anything done inappropriately.
Does this PR introduce a user-facing change?:
NONE

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


SVD implementation is based on GPL code

svd.js code is based on Python SVD implementation which is licensed under GPL license:
https://github.com/sloisel/numeric/blob/master/src/svd.js
http://stitchpanorama.sourceforge.net/Python/svd.py
Following GPL license terms numeric.js should be released under GPL license as well so MIT license is incorrect.
Could you please check this legal issue?
Possible solution can be to make SVD part optional and released under GPL.

But that Python SVD implementation is based on an algorithm:

Almost exact translation of the ALGOL SVD algorithm published in Numer. Math. 14, 403-420 (1970) by G. H. Golub and C. Reinsch

So one could argue that both this and Python code is just code translation of the algorithm from the book and as such are not really enough new contribution to be even copyrightable.


chore(main): release 0.2.2

:robot: I have created a release beep boop
0.2.2 (2022-11-20)
Bug Fixes

release: publish snapshots as well (460f5ec)

Miscellaneous

main: release 0.2.2-SNAPSHOT (bd3c904)
main: release 0.2.2-SNAPSHOT (#10) (9b9857f)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/paulschuberth/mdgen/releases/tag/v0.2.2 :sunflower:


How to get mix task exceptions to work?

Hi, I'm loving this project, thanks a lot!
We have some big tasks (running via Heroku scheduler) that we would like to track in bugsnag, still on a phoenix app
We tried this test exception task but couldn't get it to work
defmodule Mix.Tasks.Carwow.TestException do
  def run(_args) do
    {:ok, _started} = Application.ensure_all_started(:carwow_api, :permanent)
    try do
      raise "Test Exception"
    rescue
      e -> Bugsnag.report(e)
    end
  end
end

This will be fixed when this PR will be merged:
https://github.com/jarednorman/bugsnag-elixir/pull/35

Can you bump the dependency of bugsnag-elixir to version:
1.4.0-beta1 ?

At the moment I'd prefer not to – there's no strict dependency relationship between the two versions.
I'd suggest adding {:bugsnag, "~> 1.4.0-beta1", override: true} to your mixfile.

Additionally I'm going to close this issue as it pertains to bugsnag-elixir and not plugsnag.


Publishers' extended archive creation tools

Some ideas:

Create a CLI, or desktop application (even inside Etcher itself)
Maybe expose a service with a nice wizard-like web UI that generates the final package and allows the user to download
Create a tool that "lints" a manually built package (for more hardcore users)

Just to add:

Simple UI customising tool with UI preview and CL style export of selected settings
Implement instructions in a responsive and adaptive UI to accommodate relevant info and steps
Accessible instructions without the need to flash an img

This is not going to be needed anymore given recent changes to the extended archive format.


Exception java.lang.NullPointerException in the middle of native Gradle build

In the middle of a native Gradle build (with Gradle 2.14.1 still) we see the following error message:
<<<
FAILURE: Build failed with an exception.
12:40:16
12:40:16 * What went wrong:
12:40:16 java.lang.NullPointerException (no error message)
12:40:16
12:40:16 * Try:
12:40:16 Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.
12:40:16
12:40:16
12:40:16 BUILD FAILED

The error is not reproducible and randomly appearing, esp. in Jenkins automation (CI server).
Question is mainly if this behavior was seen by others and might be fixed in later Gradle versions.
Expected Behavior
Running-through build :)
Current Behavior
Most of the time 99% the build is working as expected. In 1% we get the above-described error message.
Context
Affects mainly CI Server (Jenkins). Might be related to destroyed Gradle caches. The error we see more often when we had to restart Jenkins master and Jenkins clients thus aborted execution.
Steps to Reproduce (for bugs)
Arbitrary native C and ASM build. Breaks randomly in between.
Your Environment
Gradle version is 2.14.1. Jenkins ver. 1.580.3, Java VM 1.8.0_25-b18.

Thanks for reporting this issue @FranzKrainer. Could you turn on the stack trace for all CI build by passing --stacktrace to Gradle? This would help us get more context. Alternatively, if you can use build scan it would be even better.
Here's a couple more question to help with the debugging:

Do you use configure on demand in your native build?
What recently changed in your build logic?
Did you upgrade to Gradle 2.14.1 recently?

Hi,
thanks for the fast reply! Much appreciated :-)!
I will answer one by one:
@ --stacktrace: Yes, that's what we gonna do for CI builds as default. Question here: Does this have any negative side effects? Is there something slowed down etc. in normal execution? If no, why is not the default in Gradle execution :-)? Guess it is too verbose...
@ Build Scan: Nope, we are not Gradle Enterprise users and bec. of sec. limitations not allowed to use free-variant of it (upload to net)
@ Configure on Demand: Yes, we use it. So far it caused no issues.
@ Recent Changes: Actually nothing. We kept the actual structures stable. The problem 'just' appeared.
@ Gradle 2.14.1: Nope, we have this for several months now. All stable. also no complaints from human users.
One further question: Could parallel execution lead to such effects here? so, we have not modeled something correctly and in some very rare cases not all preconditions are met e.g.?

--stacktrace: Yes, that's what we gonna do for CI builds as default.

Seems like it is not properly configured. At least the Gradle error message seems to indicate that. Maybe you can try to reproduce the issue locally. We'll need to have the stack trace to be able to pinpoint the issue.

Does this have any negative side effects? Is there something slowed down etc. in normal execution?
There's no performance impact.

Guess it is too verbose...

Correct. Most user would rather have less information or only the relevant information. A NullPointerException is most certainly a bug in Gradle core.

Gradle 2.14.1: Nope, we have this for several months now. All stable. also no complaints from human users.

I'd strongly recommend upgrading to Gradle 4.0.1. It's possible that the issue has already been fixed. BTW: It looks like you are concerned about performance. Gradle 4 is much, much faster than Gradle 2. That by itself is a good argument to upgrade.

One further question: Could parallel execution lead to such effects here? so, we have not modeled something correctly and in some very rare cases not all preconditions are met e.g.?

Yes, there could be an issue with parallel execution.

Hi,
thanks a lot for the precise answers!
We were really able to catch a build using --stacktrace.
The error log / stacktrace:
<<<
FAILURE: Build failed with an exception.

What went wrong:
java.lang.NullPointerException (no error message)

Try:
Run with --info or --debug option to get more log output.

Exception is:
java.lang.NullPointerException
at org.gradle.api.internal.changedetection.state.TreeSnapshotRepository.removeTreeSnapshotUsage(TreeSnapshotRepo
sitory.java:85)
at org.gradle.api.internal.changedetection.state.CacheBackedFileSnapshotRepository.remove(CacheBackedFileSnapsho
tRepository.java:47)
at org.gradle.api.internal.changedetection.state.CacheBackedTaskHistoryRepository$1$1.run(CacheBackedTaskHistory
Repository.java:81)
at org.gradle.internal.Factories$1.create(Factories.java:22)
at org.gradle.cache.internal.DefaultCacheAccess.useCache(DefaultCacheAccess.java:192)
at org.gradle.cache.internal.DefaultCacheAccess.useCache(DefaultCacheAccess.java:175)
at org.gradle.cache.internal.DefaultPersistentDirectoryStore.useCache(DefaultPersistentDirectoryStore.java:106)
at org.gradle.cache.internal.DefaultCacheFactory$ReferenceTrackingCache.useCache(DefaultCacheFactory.java:187)
at org.gradle.api.internal.changedetection.state.DefaultTaskArtifactStateCacheAccess.useCache(DefaultTaskArtifac
tStateCacheAccess.java:60)
at org.gradle.api.internal.changedetection.state.CacheBackedTaskHistoryRepository$1.update(CacheBackedTaskHistor
yRepository.java:66)
at org.gradle.api.internal.changedetection.changes.DefaultTaskArtifactStateRepository$TaskArtifactStateImpl.afte
rTask(DefaultTaskArtifactStateRepository.java:135)
at org.gradle.api.internal.tasks.execution.SkipUpToDateTaskExecuter.execute(SkipUpToDateTaskExecuter.java:68)
at org.gradle.api.internal.tasks.execution.ValidatingTaskExecuter.execute(ValidatingTaskExecuter.java:58)
at org.gradle.api.internal.tasks.execution.SkipEmptySourceFilesTaskExecuter.execute(SkipEmptySourceFilesTaskExec
uter.java:52)
at org.gradle.api.internal.tasks.execution.SkipTaskWithNoActionsExecuter.execute(SkipTaskWithNoActionsExecuter.j
ava:52)
at org.gradle.api.internal.tasks.execution.SkipOnlyIfTaskExecuter.execute(SkipOnlyIfTaskExecuter.java:53)
at org.gradle.api.internal.tasks.execution.ExecuteAtMostOnceTaskExecuter.execute(ExecuteAtMostOnceTaskExecuter.j
ava:43)
at org.gradle.execution.taskgraph.DefaultTaskGraphExecuter$EventFiringTaskWorker.execute(DefaultTaskGraphExecute
r.java:203)
at org.gradle.execution.taskgraph.DefaultTaskGraphExecuter$EventFiringTaskWorker.execute(DefaultTaskGraphExecute
r.java:185)
at org.gradle.execution.taskgraph.AbstractTaskPlanExecutor$TaskExecutorWorker.processTask(AbstractTaskPlanExecut
or.java:66)
at org.gradle.execution.taskgraph.AbstractTaskPlanExecutor$TaskExecutorWorker.run(AbstractTaskPlanExecutor.java:

at org.gradle.internal.concurrent.ExecutorPolicy$CatchAndRecordFailures.onExecute(ExecutorPolicy.java:54)
at org.gradle.internal.concurrent.StoppableExecutorImpl$1.run(StoppableExecutorImpl.java:40)

BUILD FAILED
Total time: 12.81 secs

Any idea what stands behind this stacktrace? Sounds/feels like a cache issue, right?
How can such issues with Cache been triggered (so, what are we doing wrongly that causes this behavior)?
@Move to Gradle 4.0.1: Yes, indeed, that's what we gonna do and are preparing currently :).
And yes, we are hoping for an even better performance - it can never be fast enough ;-)!
BR,
Franz

Hi @bmuschko and @lacasseio,
any update on the stack trace I provided? Any ideas here?
Should we let this topic go and concentrate on the fact if we see similar things in Gradle 4.0.2?
Am eager to know if it is worth investing into this random issue (and if there is a likelihood that this persists in many/later Gradle versions).
Thanks a lot in advance!
BR,
Franz

@FranzKrainer I think the best step forward would be to migrate to Gradle 4.0.2 and soon to be released 4.1. Lots have changed in that part of the code base (the class TreeSnapshotRepository doesn't exist anymore) that it wouldn't surprise me the flaky issue has already been fixed. With the latest version, we would be in a better position to help find the issue and roll a fix if needed.

Hi,
sounds great and fair to me. In the meantime we switched to Gradle 4.0.2 and did not yet see this there. Let's refrain from analyzing old stuff and concentrate on new one :-).
Will come back to you if this re-appears in new Gradle versions of course.
Think we can close this one!
Thanks & bye,
Franz


[Feature] Benchmarking PostgreSQL vs. SQLite for Large-Scale VPN Networks Using Headscale

Use case
Although Headscale is primarily designed for small-scale VPN environments (e.g., home VPNs), there is increasing interest in using Headscale to deploy large-scale VPN networks with over 500 clients. However, there is a lack of comprehensive guides and benchmark materials for such use cases. This benchmark provides recommendations for database configurations and comparison metrics to assist users in setting up large-scale VPN networks with Headscale.

The aim of this benchmark was to evaluate the suitability of SQLite (with the WAL option) versus PostgreSQL for large-scale client networks. The evaluation criteria included error occurrence frequency, client request processing time, and profiling results.
The benchmark results showed that PostgreSQL outperformed SQLite in terms of total client creation time, error occurrence frequency, and profiling results.
Based on these results, it is recommended to update the documentation and config.yaml to include criteria for selecting a database based on different use cases.

Description
Benchmark Environment

A single Headscale server was used to create and connect 60 Tailscale clients on each of 10 EC2 instances.

Headscale Instance: t2.large instance deployed in the ap-northeast-2 region
Tailscale Clients: 10 t2.medium instances (5 in ap-northeast-2 and 5 in us-east-1), with 60 Tailscale clients deployed per instance (total 600 tailscale client)
- The benchmark was performed in a consistent environment where the only variable was the choice of database specified in the headscale config.yaml: either PostgreSQL or SQLite (with the WAL option).

Comparison Metrics and Results
The benchmark evaluated SQLite (with the WAL option) and PostgreSQL based on the following three criteria: total client creation time and error occurrence rate.

Total Client (600) Creation Time and Error Occurrence Rate

SQLite: sqlite-node-list.json

Total Creation Time: 3.46 hours (First user created at:<PHONE_NUMBER>,  Last user created at:<PHONE_NUMBER>)
Successful Client Creations: 442
Errors Encountered: 158 (ERR: context deadline exceeded)

PostgreSQL: postgres-node-list.json

Total Creation Time: 2.47 hours (First user created at:<PHONE_NUMBER>,  Last user created at:<PHONE_NUMBER>)
Successful Client Creations: 501
Errors Encountered: 99 (ERR: context deadline exceeded)

Error Occurrences
The following provides a verification of the errors that occurred in Issue #1966, including their occurrence rates:

ERROR-1: The "ERR update not sent, context cancelled..." error occurred more frequently with SQLite. PostgreSQL experienced the Cannot create user: context deadline exceeded error.
ERROR-2: The issue where some clients go from online to offline was caused by insufficient resources on the EC2 instances. This was not related to Headscale itself. With t2.medium instances, up to 60 Tailscale clients could be deployed effectively, but deploying more than 100 clients led to resource shortages, causing some clients to go offline.

Profiling Results

The profiling results show that SQLite had higher query and database-related time compared to PostgreSQL. Please refer to the attached files for detailed profiling data.

Profiling Results: cpu
postgres

sqlite

Profiling Results: memory
postgres

sqlite

Conclusion

Client Creation Time: PostgreSQL outperformed SQLite in terms of faster client creation and a higher number of successful client creations.
Error Occurrence Frequency: PostgreSQL had a lower error(context deadline exceeded) occurrence rate compared to SQLite.
Profiling Performance: PostgreSQL demonstrated better performance in query execution and database-related operations compared to SQLite.

Recommendation
Based on the benchmark results, PostgreSQL is recommended for large-scale client environments over SQLite. It is suggested to update the documentation and config.yaml to include guidelines for selecting the appropriate database based on the use case.
Attachments

Headscale Node Lists:

postgres-node-list.json
sqlite-node-list.json

pprof Profiling and Heap Measurement Results:

cpu-postgres.png
cpu-sqlite.png
heap-postgres.png
heap-sqlite.png

Contribution

[X] I can write the design doc for this feature
[X] I can contribute this feature

How can it be implemented?
No response

Does this highlight a need (regardless of database) to be keeping more info in memory, or doing less database work every time a peer update occurs (or a mixture of both)?
I've been diving into the source, and I see a couple of TODOs by @kradalby mentioning caching. Would there be appetite for that?

Thanks for your meticulous benchmark.
It astonishes me that creating just 600 clients will take hours both in postgresql and sqlite. Intuitively speaking, I think the vital bottleneck here maybe not the database🤔

I've posted a comment touching on some of these topics here: https://github.com/juanfont/headscale/issues/1993#issuecomment-2226829944


update parent command

Not sure on the need of this but I have sometimes wanted to switch a branches parent and wanted all unique commits on that branch (relative to its current branch) cherry-picked onto the new parent.
Not certain if merge conflict resolutions would get carried over in this case or not.

I think this should be the default behavior. When I change the parent of the branch, I also always want that the commits I made are moved over to a new branch that starts at the new branch. I think we can simply rebase the current branch against the new parent and change the Git Town config.

I think this is implemented in the form of the set-parent command.


python311Packages.ics: refactor

Description of changes
Fix build (https://hydra.nixos.org/build/246469124)

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

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

6 packages built:

calcure
calcure.dist
python311Packages.ics
python311Packages.ics.dist
python312Packages.ics
python312Packages.ics.dist


APM compared to AppScope

How does APM work
How does AS work
Why are they different
Normalized data
Any and all apps
cost
overhead, performance
visualization options
data destinations
locked in platform

Closing as part of clean-up at the moment
If needed, will be reopened to track the relevant work


Is default branch being next intentional?

Hi,
The default branch of this repository has recently become next. Up until now, it has always been master. Was this change intentional? It affects the go get command in GOPATH mode, since it fetches commits from the default remote branch.
Thanks!

Hi @dmitshur, yes this change was intentional but the effect it had on go get was unanticipated and undesirable, so I have switched it back to master. Apologies if this caused you inconvenience or confusion, and thanks for rasing it 🙏.
The intention was to have an integration branch where we could land approved PRs without causing a release. Then when we choose to release, we make a single merge from next -> master. Using the default branch setting meant PRs would automatically use next as the base branch, but I guess this will just have to be manually set each time due to go get's / GitHub's behaviour in this instance.
Thanks again!

Apologies if this caused you inconvenience or confusion, and thanks for rasing it 🙏.

No problem. This did not cause me inconvenience, I just wanted to report it in case it was an unintentional change.
Thank you!


ember build breaks in npm@2 and npm@3 as well

Getting the below error

Buildingui.writeInfoLine is not a function
TypeError: ui.writeInfoLine is not a function

Update ember-cli in your app.


each with round brackets breaks highlighting

Hi! Thanks for your work! :slightly_smiling_face:
I've found an issue with highlighting when round brackets are present in the each block.
{#each someFunc(someArr) as someItem (someItem.id)}
{/each}

{#each someArr.filter(someFunc) as someItem (someItem.id)}
{/each}

Also do you know by any chance where should I post about indent issues? :thinking:

Thanks for pointing out this. I will fix this up in a few days time.
@elianiva Can you look over the indentation issues from the ./queries folder?

if you use Neovim, you should post indentation issue on nvim-treesitter repo because the algorithm for indentation is there

It is interesting that having highlighting broken by this each changes how the indents work too. So looks like they kinda related. :thinking:
When I press o to add a line after Code component it gives 2 excess spaces of indent:
                {#each user.codes.filter(codeNotBlocked) as code (code.id)}
                  <Code {code} {codeBlocked} {revokeCode} {codeStatuses} />
                {/each}

When I remove .filter(codeNotBlocked) it goes on the same indent level as the Code component.
There's much more indent issues as a try to reindent svelte component with more logic, or try to add some divs here and there. :thinking:
I think it is better for me to make concise reproductions to be more helpful...

algorithm for indentation is there

So does it mean that pressing V, selecting a bunch of lines and pressing = uses the algorithm implemented in nvim-treesitter repo?

It is interesting that having highlighting broken by this each changes how the indents work too. So looks like they kinda related.

In this case, yes, because the AST is probably broken so the indent algorithm can't figure it out correctly.

So does it mean that pressing V, selecting a bunch of lines and pressing = uses the algorithm implemented in nvim-treesitter repo?

Correct

I've played around with indents and it seems = works fine if I don't use each as described above, or :else if.
:else if actually doesn't mess up indent that bad - just pushes :else if and /if forward by 2 spaces. :thinking:
But 'as I type' indent is kinda gets in the way. I think I'll shoot some asciinemas or something in the nvim-treesitter repo. Thanks.

I'm having the same issue where the parentheses () inside the #each block break the highlighting. Any idea if a fix is coming soon?
Thank you

Hi @Himujjal, I'm still getting issues with the each block

I uninstalled svelte via :TSUninstall svelte and reinstalled with the :TSInstall svelte command.
Am I still using an outdated version of the svelte parser somehow or was this not in the scope of this fix?

This is how it looks when I take a pair of parentheses out:

Hold on. Filing a PR on the nvim-treesitter repo.


Update the first paragraph for clarity

Make it more clear what the scaffolding does and is for, and why it's necessary. Also use a bit more plain language.

@rhires  thanks for your PR but I prefer the original.


Adding Oracle Cloud Infrastructure as a datasource

Is your feature request related to a problem? Please describe.
please add Oracle cloud as a datasource .
Is there any plan to add this in the roadmap?

Hi, we have it in the roadmap for the beginning of 2024. Please email info (at) hystax.com if you want us to prioritize this or if you would like to contribute. Here is a guide: https://github.com/hystax/optscale/blob/integration/documentation/new_cloud_integration.md.


Create GitHub Action that runs cargo-deny

We're switching over more of our repos to use GitHub Actions for CI and would be great if we create and support a good action specifically for cargo-deny.
We want this both for our own open source Rust repos and have heard from other projects that want it as well, so would be super nice if it is easy for everyone to set up and use the action (esp. when Actions is out of beta).
Let's do it 🚀

Have implemented a first version of it here: https://github.com/EmbarkStudios/cargo-deny-action


Suggested synonym: preventative/preventive

So far as I am aware, "preventative" and "preventive" mean exactly the same thing, so any MAxO term or synonym that contains the word "preventative" could just as easily and correctly be written with the word "preventive". I'm wondering do you want to add these alternative forms as synonyms? The first MAxO term that contains "preventative" is MAXO:0000017. There are 18 occurrences of "preventative" in the latest release of maxo-full.obo.

I'll add 'preventive' as a synonym. Although, just a note: there are only 3 labels or terms that this applies to. I am not exactly sure where the 18 number comes from, but could it be that these are synonyms and/or subclasses to these terms? In any event, I will add synonyms to MAXO_0000118, MAXO_0000020, and MAXO_0000017

Sorry if I was unclear. The number 18 refers to occurrences of "preventative" both in term names and in synonyms you already defined for those terms. So all the occurrences of "preventative" that are highlighted in yellow in the screen shot above would be included in the total count of 18. I was looking at the "raw" obo file, not the interface you use. But the information should be the same no matter how you see it.


devede: fix runtime error

Motivation for this change
https://github.com/NixOS/nixpkgs/issues/61578
Needs backport to 19.03 as devede is currently broken there as well.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

backported in 51cc0ed1461e290fa05f1989a29b0a1fe5f9f8a3


Build adapter to pull fees from fees-service

Fixes https://github.com/omisego/feat-fee/issues/18
Overview
Following the introduction of an adapter layer in omisego/feat-fee#17 and the implementation/deployment of the fees-service, we can add a new adapter to the fee sources to connect to that service.
Pulling
Fee-feed will offer a /fees.all endpoint from which the fees can be retrieved. See format below.
Updating
The childchain should continously pull fees from fee-feed, but should only update a stored fee every X minutes, unless it detects that an amount has changed more than Y%.
X and Y should be configurable.
Format
Format of the returned from fee-feed service response follows standard OmiseGO http-rpc response object schema. data element format is the same as fee-specs JSON file format, see schema.
Changes
Describe your changes and implementation choices. More details make PRs easier to review.

Added FeedAdapter module responsible for ensure configuration and fetching prices from the Feed service. Frequency of polls is described by fee_adapter_check_interval_ms configuration variable and is used by FeeServer.
Added FeeUpdater module that contains business logic behind when to update the fees from feed
Added HttpRPC helpers similar to these in Watcher. In child chain fetching fees is the first purpose for this module.
Added child chain's release task which sets Feed adapter's options from environment
Refactors in supportive module, e.g. JSONFeeParser, renamings around error messages, swagger specs

Testing
Describe how to test your new feature/bug fix and if possible, a step by step guide on how to demo this.

Test business logic for fee update mechanics in FeeUpdaterTest
Integration tests for adapter (with fake server)

Coverage decreased (-78.4%) to 7.959% when pulling cb53906133e5332d19815e48a322c4abc51d915c on papa/fee-18-adapter-to-pull-fees-from-feefeed into d6c50112df1a5b92151006dd0f64835ead3eea62 on master.

sorry, @pnowosie I didn't see I had unsubmitted comments pending

🤔 we are no longer posting to "get" endpoints???

@boolafish - no :D it's GET because feed service don't follow http-rpc like ch-ch. There is nothing wrong to get data with POST

🤔 we are no longer posting to "get" endpoints???

@boolafish - no :D it's GET because feed service don't follow http-rpc like ch-ch. There is nothing wrong to get data with POST

factually incorrect :)

no one follows http-rpc, what we have on all services is a mish mush
to GET data with POST is against RFC. clients that follow RFC closely cannot make these requests.

factually incorrect :)

Possible ;)

1. no one follows http-rpc, what we have on all services is a mish mush

Or maybe you just though about json-rpc? Http-rpc is kind of mish mush ;)

2. to GET data with POST is against RFC. clients that follow RFC closely cannot make these requests.

Not exactly. Some poor clients cannot make post request without body correctly, because setting Content-Length: 0 is too much... 👐  but yeah one can argue that sending post without body makes no sense. (I don't think it matters much)


Ideas for the future.

I was thinking of a few things I would eventually like to do with this platform.
(1) Send texts. Im going to do some research and see if I can find a way to pair Melissa with my phone to send texts to people in my contact lists. I also plan to do a similar thing to answer and place phonecalls.
(2) I'd love to have a module that specifies in more advanced image recognition - although I know that will take along time. A module that could recognize faces, places and pull up a profile on that person/place. I know, its futuristic and somewhat outlandish but it would be amazing! This would take a long time to design though, years maybe.
(3) Finally, I'd love to have Melissa be able to locate buildings and use google maps to find directions.
Again, these are just some ideas I was thinking about. Im not sure if I can commit to any of them, Im probably not an experienced enough coder to do that. I will try to work on these things though just for fun, but I hope someone more expert than I will come along and make these changes.

@SLaVey1 Interesting ideas! Looking forward to your pull requests :D

Yeah I have been busy. I made an sms sender with python a few weeks ago (uses sms gateways), as well as a python code that can search the internet or Wikipedia. If i give you my scripts do you think you could do anything with them? If yes I will post them here.

We can add text support via TextMagic. However, I don't think it's worth trying to integrate text from someone's own phone number. We can easily add  email support though.

@SLaVey1 @tanay1337 And we should use the Wolfram Alpha API to add  mathematical functionality. I used it in a project last year and it's both simple and functional.

That would be cool - can you integrate it into Melissa?

Closing this because this does not match our roadmap and we don't have the bandwidth to work on it. Ideas 2 and 3 do sound cool and it might make sense to attempt such a solution in the future. Thanks for your ideas @SLaVey1.


refactor(live-status-gw): reassign log levels

About the Contributor

This PR is being opened on behalf of TV 2 Norge.
Type of Contribution
This is a:

Code improvement / Log level improvement
Current Behavior

The Live Status Gateway outputs lots of excessive info level logs. When in use, it can output hundreds of thousands log entries a day. Some of them inform about individual document updates, heartbeat (sent out every 2 seconds), information about collection handlers and topics receiving or sending updates, etc.
New Behavior

info level is now mostly used for lifecycle events (initialization, connecting/disconnecting, subscribing/unsubscribing). Less relevant messages are demoted to debug or in few cases to silly.
Testing Instructions

Other Information

It's a draft, awaiting merge of https://github.com/nrkno/sofie-core/pull/1131 after which I'll rebase this to release51
Status

[ ] PR is ready to be reviewed.
[x] The functionality has been tested by the author.
[ ] Relevant unit tests has been added / updated.
[ ] Relevant documentation (code comments, system documentation) has been added / updated.

#1131 has been merged. Besides a rebase, is there anything else this is waiting for? We will most likely freeze R51 development soon so it would be nice to get this in before.


Popup messages are not displayed correctly if they contain angle brackets

Sublime Text 3.1.1 (Build 3176), Rust Language Server

Good find, we can never trust server-returned content that doesn't identify as html or markdown. I suspect html encoding should do the trick.

While this may fixed the issue, it introduced another. But it is an easy fix.
Here is the problem.
See the error popup, it should't look like this.
There is also an error in the console: Parse Error: #x27; code: Unknown entity.

To fix this, in plugin/hover.py file the escape function takes an additional optional argument.
If the optional flag quote is true (the default), the quotation mark characters, both double quote (") and single quote (') characters are escaped.
Which in this case is not preferred and setting it to False fixes the error popup and it looks like this:

Pushed a fix for this just now, thanks for the research and instructions!


Battery Icon display is duplicated.

Which version of HA do you use?
core-2021.6.2
Which version of the card do you use?
power-wheel-cardVersion: 0.1.5L
Describe the bug
Somtimes the Battery Icon is duplicated. But it vanishes also on reatltime.
Configuration of the card
type: custom:power-wheel-card
title: Power wheel
solar_power_entity: sensor.power_photovoltaics_fronius_power_flow_4_http_10_13_24_193
grid_power_entity: sensor.power_grid_fronius_power_flow_4_http_10_13_24_193
battery_power_entity: sensor.power_battery_fronius_power_flow_4_http_10_13_24_193
battery_soc_entity: sensor.state_of_charge_fronius_power_flow_4_http_10_13_24_193
home_energy_entity: sensor.power_load_fronius_power_flow_4_http_10_13_24_193
charging_is_positive: false
Dev console debug output
power-wheel-cardVersion: 0.1.5Lovelace resource: power-wheel-card.jsHA version: 2021.6.2Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/91.0.4472.77 Safari/537.36Report issues here: https://github.com/gurbyz/power-wheel-card/issuesProcessed config: { "type": "custom:power-wheel-card", "title": "Power wheel", "solar_power_entity": "sensor.power_photovoltaics_fronius_power_flow_4_http_10_13_24_193", "grid_power_entity": "sensor.power_grid_fronius_power_flow_4_http_10_13_24_193", "battery_power_entity": "sensor.power_battery_fronius_power_flow_4_http_10_13_24_193", "battery_soc_entity": "sensor.state_of_charge_fronius_power_flow_4_http_10_13_24_193", "home_energy_entity": "sensor.power_load_fronius_power_flow_4_http_10_13_24_193", "debug": true, "charging_is_positive": 1, "production_is_positive": 1, "title_power": "Power wheel", "title_energy": "Power wheel", "title_money": "Power wheel", "power_decimals": 0, "energy_decimals": 3, "money_decimals": 2, "money_unit": "€", "color_icons": true, "consuming_color": "var(--label-badge-yellow, #f4b400)", "producing_color": "var(--label-badge-green, #0da035)", "invert_grid_colors": false, "active_arrow_color": "var(--paper-item-icon-active-color, #fdd835)", "initial_view": "power", "initial_auto_toggle_view": false, "auto_toggle_view_period": 10}Registered sensors: [ "sensor.power_photovoltaics_fronius_power_flow_4_http_10_13_24_193", "sensor.power_grid_fronius_power_flow_4_http_10_13_24_193", "sensor.power_battery_fronius_power_flow_4_http_10_13_24_193", "sensor.power_load_fronius_power_flow_4_http_10_13_24_193", "sensor.state_of_charge_fronius_power_flow_4_http_10_13_24_193"]Views object: { "power": { "title": "Power wheel", "oneGridSensor": true, "twoGridSensors": false, "batteryCapable": true, "capable": true, "unit": "W" }, "energy": { "title": "Power wheel", "oneGridSensor": false, "twoGridSensors": false, "capable": false }, "money": { "title": "Power wheel", "oneGridSensor": false, "twoGridSensors": false, "capable": false, "unit": "€" }}

This is expected.

For now battery support is implemented in the current layout grid. This is why the battery shows up in two different places and is visible only when in use. This however should be temporary. First priority is to get all the values and arrows right and improve later. If it functionally works with a battery completely, then it's time to change the layout into a better interface. Many suggestions for the final UI have been given.

I'm just starting to try and use this power wheel card on a new solar/battery integration into HA 2022.8 latest.  Has there been any development on the card to improve battery visuals or is the development stalled?


Purpose of Read/Write inner classes

As @karroffel mentioned in the last progress report, we have 2 new inner classes in PoolArray's API created in this commit https://github.com/GodotNativeTools/godot-cpp/commit/e3b20a28a93c35ef33865d2f05ba6ac82fc170ae
What's the purpose of these classes? I think it should be commented on top of the header or documented somewhere. If it is, could you please link me that information.

PoolArrays need to be locked in order to write or read from them. By using set or the index operator on a PoolArray this locking happens automatically. But this also means that when you use those to access and modify the data that there's a lot of overhead due to the constant locking and unlocking.
The Read and Write access objects lock the PoolArray when they are created and unlock automatically when they get leave the scope.
So they are used to get better performance when accessing data in blocks.
set/get way:
// p is a PoolIntArray
for (int i = 0; i < p.size(); i++) {
    printf("%d ", p[i]); // p[i] will lock and unlock - every iteration
}

read way:
{
    auto read = p.read(); // locking happens here

    for (int i = 0; i < p.size(); i++) {
        printf("%d ", read[i]); // just plain access, almost no overhead
    }

} // unlocking happens here

As this has been explained, closing :)


Document events: attaching, attached, detaching and detached.

Events are undocumented. They need to have event doclets (like fetching etc.) and attach and detach need @throws tags.

any updates on this????


When I'm using graphql for authentication, the errorHandle function params is [object, object]

{error: Error: GraphQL error: [object Object]     at new ApolloError (webpack-internal:///./node_modules/ap…}
and My graphql returns an error is
errors: [{message: {statusCode: 401, error: "Unauthorized"}, locations: [{line: 2, column: 3}],…}] 0: {message: {statusCode: 401, error: "Unauthorized"}, locations: [{line: 2, column: 3}],…} extensions: {code: "INTERNAL_SERVER_ERROR",…} locations: [{line: 2, column: 3}] message: {statusCode: 401, error: "Unauthorized"} path: ["get_user_total"]

Even I am getting this. The error returned from server is as below:

{"errors":[{"message":"Invalid Email or password.","locations":[{"line":2,"column":3}],"path":["signIn"],"extensions":{"code":"UNAUTHENTICATED","exception":{"stacktrace":["AuthenticationError: Invalid Email or password.","

However in the handleError function I get just this

Error: GraphQL error: Invalid Email or password.
at new ApolloError (bundle.esm.js?74ca:63)
at Object.next (bundle.esm.js?74ca:1004)
at notifySubscription (Observable.js?a7b2:130)
at onNotify (Observable.js?a7b2:165)
at SubscriptionObserver.next (Observable.js?a7b2:219)
at eval (bundle.esm.js?74ca:866)
at Set.forEach ()
at Object.next (bundle.esm.js?74ca:866)
at notifySubscription (Observable.js?a7b2:130)
at onNotify (Observable.js?a7b2:165)

I can show the error "Error: GraphQL error: Invalid Email or password." on the UI by striping the "Error: GraphQL error:", but if I have to perform some logic based on error code, like redirecting based on different codes then it is a problem. Will be good if I can get the whole object which is returned from the server.

From memory you can do error.graphQLErrors[0].extensions.code


(FACT-1377) Include key type in ssh fact

Previously the ssh fact returned host keys by their general type, e.g. ecdsa, but did not include information about their real type, e.g. ecdsa-sha2-nistp256.
This adds a type attribute for each host key containing the real type.

Ping @branan; we chatted about this yesterday.

CLA signed by all contributors.

Seems reasonable.

Actually, I'd rather not, since the fact would fail if a new key type is introduced.


Address duplicate work/chapter creation by setting ID before submit

User created duplicate works/sections but claims not to have pressed submit/save buttons multiple times, instead blaming connection issues.
My response was:
I suppose it's theoretically possible that could happen. I was thinking already that we should generate the ID beforehand rather than when you submit the story/chapter, which could help prevent the duplication from happening.

this should no longer happen as loading state prevents the Create buttons from accepting further clicks until the current request has finished


Change React function bindings to pass on return values

Fix as described in https://github.com/jeffreylanters/react-unity-webgl/issues/40
In summary, a bound React function called from a jslib will always return 'undefined' as the wrapper function does not pass on the return value.
Added fix to legacy binding code too (line 10).

@jeffreylanters I switched the base to master, but it seems you've added a couple of commits to 'development' in the meantime which you may not wish to be merged (I branched off 'development' in my local fork). Let me know if you want me to branch off master instead.

I'm completely reworking the library from the ground up in development, could you branch off master instead? Sorry about that.


Concurrency Problems

Is there any recommended way to deal with 1000+ simultaneous users? I use the library in the Data Access and it gets so slow when multiple users are interacting with the database. Any help is appreciated. Here's a snippet of the way i'm interacting with my Oracle Database:
using (DbAccess db = DbPackage.CreateConnection())                 {                     db.ExecuteReader(DbPackage.GetProcedure("trk_pkg_tracking.ConsultarMisCasosConFiltro"),                         parameters =>                         {                             ((OracleCommand)parameters.Command).BindByName = true;                             parameters.Add("pUsuario", pUsuario);                             parameters.Add("pCasoID", pCasoID);                             parameters.Add("pCasoTitulo", pCasoTitulo);                             parameters.Add("pProcesoID", pProcesoId);                             parameters.Add("pActividadDescripcion", pActividadDescripcion);                             parameters.Add("pEnviadoPor", pEnviadoPor);                             parameters.Add("pUsuarioAsignado", pUsuarioAsignado);                             parameters.Add("pIdentificacionCaso", pIdentificacionCaso);                             if (pFechaDesde.HasValue)                             {                                 parameters.Add("pFechaDesde", pFechaDesde.Value);                             }                             if (pFechaHasta.HasValue)                             {                                 parameters.Add("pFechaHasta", pFechaHasta.Value);                             }                             parameters.AddRefCursor("pRegistros");                         },                         resultReader =>                         {                             dtCasos.Load(resultReader);                         }, bulkRead: true);                 }

Concurrent planning should be carried through your whole solution desian which depends on your use cases. And your description doesn't reveal any information about your application architecture. So let's discuss concurrency problems in a general sense here.

When large numbers of concurrent clients are executing the same stored procedure, the query efficiency should be considered as the most critical factor. Other than simplifying the algorithm, try to minimize the frequency of I/O on related tables is a common direction of endeavors. Like, boosting throughput with read-ahead and write-behind in-memory optimizes etc. Optimizations should be applied in both database server and application server. For database side, a variety of techniques can be referenced from Oracle document library, see also:

https://docs.oracle.com/database/121/CNCPT/consist.htm
https://docs.oracle.com/en/database/oracle/oracle-database/12.2/vldbg/degree-parallel.html
http://www.dba-oracle.com/t_initrans.htm
https://oracle-base.com/articles/12c/in-memory-column-store-12cr1
http://www.oracle.com/technetwork/database/in-memory/overview/index.html
https://blogs.oracle.com/in-memory/getting-started-with-oracle-database-in-memory-part-i-installing-enabling

When large numbers of concurrent clients are connecting the same database, the database connections is also a precious resource. Pooling is a common approach to economize database connections. That's why connection pool tuning is important to the application server for a high concurrency system. Other than testing purposes, it's too extravagant to let thousands of end-clients connect to a database directly - without an application server.

I understand, i only needed to know if concurrency and parallelism have been taken into account while designing the library because i faced a terrible downside while using the library in a SOAP Webservice developed using .NET framework 4.5 and C#. The webservice architecture is using basic authentication and the connections are handled using the structure provided by the library when i installed the nugget package, so i'm not using any special architecture or a different way to handle connections through the SOAP webservice. The client application is a legacy WebForms application and also a new interface in development using ASP.NET MVC 5 but without using Entity Framework and just the SOAP service.

Specific to the method used in your snippet, its library implementation is just a very lightweight syntax encapsulation/extensions on the low level ODP.NET OracleCommand.ExecuteReader/ExecuteNonQuery, it doesn't add any concurrency or parallelism mechanism on top.
To make it easy to figure out where's the bottleneck - in database side (big topic), connection pooling, the library, C# WCF service (InstanceContextMode and ConcurrencyMode), ASP.NET MVC (Asynchronous Controller vs. Synchronous Controller) or Application Pool settings in IIS (such as MinWorkerThreads, MaxWorkerThreads) ... etc, could you try following small test code in your environment:
   class Program
    {
        static void Main(string[] args)
        {
            const int concurCnt = 1000;             // Simulating 1000 times
            Console.WriteLine("Testing {0} times ...", concurCnt);
            DateTime startTime = DateTime.Now;

            // Sequential execution - Run 1000 (concurCnt) times in sequence.
            for (int i = 0; i < concurCnt; i++)
            {
                var ret = TestDbCall();
            }
            Console.WriteLine("Sequential execution, average elapsed time = {0}, average response time = {0}",
                TimeSpan.FromTicks((DateTime.Now - startTime).Ticks / concurCnt));

            ////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
            long elapsedTicks = 0L;
            startTime = DateTime.Now;

            // Concurrent execution - Run 1000 (concurCnt) times concurrently.
            Parallel.For(0, concurCnt,
                new ParallelOptions() { MaxDegreeOfParallelism = 100 }, // By default, "Max Pool Size" = 100 in Oracle connection string, see https://docs.oracle.com/database/121/ODPNT/featConnecting.htm#ODPNT164
                i =>
                {
                    DateTime taskStart = DateTime.Now;

                    var ret = TestDbCall();

                    Interlocked.Add(ref elapsedTicks, (DateTime.Now - taskStart).Ticks);
                });
            Console.WriteLine("Concurrent execution, average elapsed time = {0}, average response time = {1}",
                TimeSpan.FromTicks((DateTime.Now - startTime).Ticks / concurCnt), TimeSpan.FromTicks(elapsedTicks / concurCnt));
        }

        static string TestDbCall()
        {
            using (DbAccess db = DbPackage.CreateConnection())
            {
                DbParameter returnParam = null;

                db.ExecuteNonQuery("SYS.DBMS_RANDOM.STRING", parameters =>
                {
                    parameters.BindByName = true;
                    returnParam = parameters.AddReturn("RETURN_VALUE", DbType.String, 100);
                    parameters.Add("OPT", "a");
                    parameters.Add("LEN", 32);
                });

                return returnParam.Value as string;
            }
        }
    }

After the first test, please append "Min Pool Size=8" to your connectionString, then test again. It would double the speed for concurrent execution (you can try out a better number to adjust to your environment);
Then, replace the Oracle build-in function "SYS.DBMS_RANDOM.STRING" by your own stored procedure "trk_pkg_tracking.ConsultarMisCasosConFiltro", and test again.
So on and so forth...


chore: install k8s molecule pip dependency

It is not possibly to start Ansible workspace and run any command from the DevSpaces Ansible sample in AirGap environment, because it requires kubernetes dependency.
Can we include Kubernetes python client into the image?
Related issue: https://issues.redhat.com/browse/CRW-6562

@NilashishC @alisonlhart @priyamsahoo could you take a look please


Add a license to the repository

I recently came across your project QHarmony and I noticed that the repository github.com/adiazcarral/QHarmony currently does not have a license specified.
Why Adding a License is Important?
As GitHub documents

Public repositories on GitHub are often used to share open source software. For your repository to truly be open source, you'll need to license it so that others are free to use, change, and distribute the software.

Would be possible to add a valid license to this repository? Probably something really OpenSource such as the Apache 2.0 license would fit really well.
Thanks!

Thanks!


Conversion between chrono_tz::Tz and zoneinfo.ZoneInfo

Issue #3266

CI fails because of a more aggressive dead_code lint


build docs and push to branch in CI

This creates a job to build docs and push them. The basic idea is to reuse the install part of the ~unittest~ smoke_test jobs, and then do
pushd doc
pip install -r requriements.txt
build html
popd

Then a separate step commits the built documentation, checks out the gh-pages branch, copies the rendered documentation into the correct directory, commits it, and pushes the result to the branch.
Things I don't understand about the config.yml, I hope I did it right
~- how do the unittest_linux_cpu jobs know not to run if the build job fails? There is no requires clause~

git push hopefully will JustWork when a PR is merged, although it will fail when done on a fork (like when people submit a PR). Should I limit the job to only run on nightly to avoid the problem?

ping @brianjo

hmm, I wanted to use the built wheel which should be in ./dist but instead seem to be building the project. @seemethere any ideas what I am doing wrong?

hmm, I wanted to use the built wheel which should be in ./dist but instead seem to be building the project. @seemethere any ideas what I am doing wrong?

The script used in the preparation steps (.circleci/unittest/linux/scripts/install.sh and .circleci/unittest/linux/scripts/setup_env.sh) are intended for installing torchaudio from source.
If the binary package is preferable, you can set up things like smoke_test do, which depends on the binary build process.
https://github.com/pytorch/audio/blob/b7c17f80b8f8f5cd25a167dc1589ada4c5b0c7cc/.circleci/config.yml.in#L275-L292

how do the unittest_linux_cpu jobs know not to run if the build job fails? There is no requires clause

unittest workflow and binary workflow are completely independent. This is because binary workflow can fail due to external reasons like network issue, but most often we still want to run unit tests for the PR we are working on.

git push hopefully will JustWork when a PR is merged, although it will fail when done on a fork (like when people submit a PR). Should I limit the job to only run on nightly to avoid the problem?

@seemethere Does the CI job executor have a push privilege?

@seemethere build_docs is failing, it is not "continuing" from where the binary_linux_wheel_3.8 is finishing. What do I need to preserve to not have to rebuild from 0?

OK, CI passes. There are warnings when building the docs, but that is not new: brianjo/audio#1.

Looks good. Let's see how the build process goes. Thanks!


Sign in With Apple Not Found in Firebase Unity SDK

Hi All
I am working in unity 2019.4.30. In my project i have to use Sign In With Apple through Firebase. I have enabled Sign in with Apple option in Firebase console. While I am using Firebase unity Auth Project. Sign in with apple option is not found in this sample project. But  other option  Microsoft,Yahoo Found.
I have to work in Sign in With Apple. How can i make  Sign in with Apple through Firebase?

Hi @CLOUDFIRE91,
While Sign in With Apple isn't available in the quickstart, this feature is included in the SDK. A prerequisite for this is joining the Apple Developer Program to configure Sign In With Apple for your build. You may check out our documentation for more detailed information.
I'll be closing this for now. Let me know if an issue arises.


Data.Char functions

Should these be free functions?

They're probably fine as is.

OK.  Submitted the same PR to Swiftz.

:boat:


Canvas add a drawMesh api demo needed.

Canvas add a drawMesh method on API 34. and there is no api demo how to use it, it is possiable to add thie api in this demo?

Will look into adding this 👍🏼

I‘m working on this， and it already works。
But i still don't how to access the paint shader in mesh fragment shader.

Canvas add a drawMesh method on API 34. and there is no api demo how to use it, it is possiable to add thie api in this demo?

It would be extremely necessary


Issue 2779 one try: add one flash message after successfully 'add annotation'

After click the save, it will show like this

@bzhou26 yeah, this is on the right track. I believe there should be some harder issues in general, but you did it right here.


Start With Debugging Fails with Error Message

Today I installed VSC v1.18.1 (x64) on Windows 10 and wrote a simple "Hello World" program (hello.rs).  However, when I try to start it with debugging, three notifications are displayed at the top of the application:

"Error: Cannot read property 'openConfigFile' of undefined"
"Info: LLDB self-test FAILED."
"Info: Since this is the first time you are starting LLDB, I'm going to run some quick diagnostics"

The Output panel (when "LLDB" is selected from the drop down) shows the following error information:
"An exception was raised while launching debugger: { Error: The debugger exited without completing startup handshake.
at ChildProcess.lldb.on (C:\Users\ssabu.vscode\extensions\vadimcn.vscode-lldb-0.7.1\out\startup.js:224:27)
at emitTwo (events.js:106:13)
at ChildProcess.emit (events.js:194:7)
at Process.ChildProcess._handle.onexit (internal/child_process.js:215:12) code: 'Handshake', stdout: '' }"
The only extensions I currently have installed are:
LLDB Debugger v0.7.1
Rust v0.4.2
Rust (rls) v0.3.2
I also ran "rustup update" to ensure there were no updates and it showed everything was indeed up to date:
"stable-x86_64-pc-windows-msvc unchanged - rustc 1.22.1 (05e2e1c41 2017-11-22)
nightly-x86_64-pc-windows-msvc unchanged - rustc 1.24.0-nightly (6a3601944 2017-12-12)"
I also checked my Environment Variables to ensure that the Python and LLVM path are present.  The paths listed include:
%USERPROFILE%\AppData\Local\Programs\Python\Python37-32\Scripts
%USERPROFILE%\AppData\Local\Programs\Python\Python37-32
%USERPROFILE%.cargo\bin
C:\Program Files\Microsoft VS Code\bin
C:\Program Files (x86)\LLVM\bin
I have tried restarting VSC several times but the exact same notification and Output error are shown.  I have also confirmed that all the extension are enabled.  Any help or suggestions with this issue would be greatly appreciated as I would really like to start programming/debugging Rust applications in VSC.  Thanks in advanced for any help you can provide.

LLDB requires Python 3.5 specifically.  Looks like you've got only 3.7.

Did you find out what was the issue?

Sorry about the delayed response (just started a new job and have been very busy).  I have installed Python 3.5.4 (2017-08-08) and confirmed that the PATH variable has been updated with %USERPROFILE%\AppData\Local\Programs\Python\Python35-32\Scripts\ and %USERPROFILE%\AppData\Local\Programs\Python\Python35-32.  I created a new project with "cargo new" and tried debugging and still get the following displayed at the top of VSCode:

"Error: Cannot read property 'openConfigFile' of undefined"
"Info: LLDB self-test FAILED."
"Info: Since this is the first time you are starting LLDB, I'm going to run some quick diagnostics"

If I check the Output window it says:
(lldb) command script import 'C:\Users\ssabu.vscode\extensions\vadimcn.vscode-lldb-0.7.3\adapter'
(lldb) script adapter.main.run_tcp_session(0, 'e30=')
Listening on port 65014
Any suggestions?

Not sure what's going on, but looks similar to https://github.com/Microsoft/vscode/issues/38674.
Also, can you please upgrade to the latest extension version (0.7.3), run LLDB: Run diagnostics command and post its output here?

@DustinByfuglien I got the same error. Turns out I was trying to debug a 64-bit binary with 32-bit LLVM. You need to install the Rust i686 toolchain instead of the x86_64 toolchain. I installed the gnu toolchain, so I'm not sure if the msvc toolchain would actually work. Also, remember to build your binaries with -g to generate debug symbols.

I am having trouble using lldb with this extension as well.
Running lldb from the terminal works fine, but attempting to debug something with vscode-lldb spews the following:
lldb version 4.0.1
--- Checking Python ---
(lldb) script import sys, io, lldb
Traceback (most recent call last):
  File "<string>", line 1, in <module>
ImportError: No module named lldb.embedded_interpreter
Traceback (most recent call last):
  File "<string>", line 1, in <module>
NameError: name 'run_one_line' is not defined
Traceback (most recent call last):
  File "<string>", line 1, in <module>
NameError: name 'run_one_line' is not defined
Traceback (most recent call last):
  File "<string>", line 1, in <module>
NameError: name 'run_one_line' is not defined
Traceback (most recent call last):
  File "<string>", line 1, in <module>
NameError: name 'run_one_line' is not defined
Traceback (most recent call last):
  File "<string>", line 1, in <module>
NameError: name 'run_one_line' is not defined
Traceback (most recent call last):
  File "<string>", line 1, in <module>
NameError: name 'run_one_line' is not defined
error: python failed attempting to evaluate 'import sys, io, lldb'
Traceback (most recent call last):
  File "<string>", line 1, in <module>
NameError: name 'lldb' is not defined

*** An exception was raised during self-test ***
{ Error: The debugger exited without completing startup handshake.
	at ChildProcess.lldb.on (/home/user/.vscode/extensions/vadimcn.vscode-lldb-0.7.5/out/startup.js:288:27)
	at emitTwo (events.js:106:13)
	at ChildProcess.emit (events.js:194:7)
	at Process.ChildProcess._handle.onexit (internal/child_process.js:215:12)
  code: 'Handshake',
  stdout: '(lldb) script import sys, io, lldb\n' }

Installed Python version is 3.6, the module "six" is installed.

@hyperfekt: looks like LLDB python bindings are missing.   Not sure how LLDB package is structured on Fedora...  Could it have been installed without python support?
What does lldb-4.0 -P output?  Is there a "lldb" module at that location?

lldb -P puts out /usr/lib64/python2.7/site-packages for bot lldb 4.0 and 5.0, but no lldb module is to be found there. In fact, it's nowhere on the system.

Ok, I figured it out. Fedora ships the Python module separately in a package called python-lldb. After installing that, things work just fine. If you could add that to the install instructions, that'd be neat.

Can you please post the exact commands that were needed to install all required packages?  Thanks!

This is it: sudo dnf install lldb python-lldb.

On Ubuntu 18.04 and vscode 1.30.1, python --version 2.7.15rc1 (but I have also phyton3 installed)
I got LLDB self-test has failed!.
*** An exception was raised during self-test ***
{ Error: The debugger exited without completing startup handshake.
at ChildProcess.process.on (/home/user/.vscode/extensions/vadimcn.vscode-lldb-1.1.2/out/util.js:200:27)
at emitTwo (events.js:126:13)
at ChildProcess.emit (events.js:214:7)
at Process.__dirname.ChildProcess._handle.onexit (internal/child_process.js:198:12)
code: 'Handshake',
stdout: '(lldb) script import sys, io, lldb\n' }

On Ubuntu 18.04 and vscode 1.30.1, python --version 2.7.15rc1 (but I have also phyton3 installed).
lldb --version
lldb version 6.0.0
I got LLDB self-test has failed!.
*** An exception was raised during self-test ***
{ Error: The debugger exited without completing startup handshake.
at ChildProcess.process.on (/home/user/.vscode/extensions/vadimcn.vscode-lldb-1.1.2/out/util.js:200:27)
at emitTwo (events.js:126:13)
at ChildProcess.emit (events.js:214:7)
at Process.__dirname.ChildProcess._handle.onexit (internal/child_process.js:198:12)
code: 'Handshake',
stdout: '(lldb) script import sys, io, lldb\n' }

Same issue. Did you manage to fix this?

Did you manage to fix this?

No.  Looks like lldb crashes, but I'm not sure why.   Does it work from the command line?   Can you execute any Python commands in it? (e.g. script version).
Also, you may want to try  lldb.adapterType="bundled"

Hi, I am facing a similar issue in installing codelldb on ubuntu 14. It says, handhake did not complete. Attached is the output from the diagnostics. Any help with this issue is deeply appreciated.
--- Checking version ---
lldb version 3.9.1 ( revision )
--- Checking Python ---
(lldb) script import sys, io, lldb
(lldb) script print(lldb.SBDebugger.Create().IsValid())
True
(lldb) script print("OK")
OK
--- Done ---

CodeLLDB now bundles its own version of liblldb, I wonder if this fixed the issue.

Just installed on Mac OS, latest VSCode version - ran into the named issue

Just installed on Mac OS, latest VSCode version - ran into the named issue

Ah interesting, I add numbers to the args instead of strings

Ah interesting, I add numbers to the args instead of strings