[SPARK-31355][SQL][DOCS] Document TABLESAMPLE in SQL Reference

What changes were proposed in this pull request?
Document TABLESAMPLE in SQL Reference
Why are the changes needed?
To make SQL Reference complete
Does this PR introduce any user-facing change?
Yes

How was this patch tested?
Manually build and check.

Merged build finished. Test PASSed.

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

Test build #120840 has started for PR 28130 at commit 88ed6b4.

Test build #120840 has finished for PR 28130 at commit 88ed6b4.

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

Merged build finished. Test PASSed.

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

@maropu A couple of more PRs to review. Thanks!

Test build #120850 has started for PR 28130 at commit 969d4f7.

Merged build finished. Test PASSed.

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

@huaxingao Yea, thanks for the great work!

Test build #120850 has finished for PR 28130 at commit 969d4f7.

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

Merged build finished. Test PASSed.

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

Merged build finished. Test PASSed.

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

Test build #120863 has started for PR 28130 at commit f7e90db.

Test build #120863 has finished for PR 28130 at commit f7e90db.

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

Merged build finished. Test PASSed.

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

Merged build finished. Test PASSed.

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

Test build #120879 has started for PR 28130 at commit acf7e00.

Test build #120879 has finished for PR 28130 at commit acf7e00.

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

Merged build finished. Test PASSed.

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

Test build #120898 has started for PR 28130 at commit c2bcf38.

Build finished. Test FAILed.

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

Test build #120899 has started for PR 28130 at commit 7df973a.

Merged build finished. Test PASSed.

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

Test build #120899 has finished for PR 28130 at commit 7df973a.

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

Merged build finished. Test PASSed.

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

Test build #120898 has finished for PR 28130 at commit c2bcf38.

This patch fails due to an unknown error code, -9.
This patch does not merge cleanly.
This patch adds no public classes.

Build finished. Test FAILed.

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

Jenkins retest this please

Merged build finished. Test PASSed.

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

Test build #120914 has started for PR 28130 at commit 7df973a.

Test build #120914 has finished for PR 28130 at commit 7df973a.

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

Merged build finished. Test PASSed.

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

Looks OK just may need a rebase @huaxingao

Test build #120999 has started for PR 28130 at commit 537c594.

Merged build finished. Test PASSed.

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

Test build #120999 has finished for PR 28130 at commit 537c594.

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

Merged build finished. Test PASSed.

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

Merged to master/3.0

Thanks! @maropu @srowen


Fix UptimeRobot status endpoint

Fixes #6598.
Note that I've already pulled out the custom domain setup from the Cloudfare configuration.

Messages

:book:

Thanks for contributing to our documentation. We :heart: our documentarians!

:book:
:sparkles: Thanks for your contribution to Shields, @PyvesB!

  Generated by :no_entry_sign: dangerJS against 9e37489a45c83f7246a9918fa9215ef026dca6c1


Gofmt

Adds gofmt to Travis pipeline. Also, fixes govet as it was not stopping the build in case of errors. Example output.
Close #819.

@ire-and-curses simpler is always better 😉 . Changed to ls. Can you 👍 again?

Did you forget to change the PR title or you think it’s good? @bartekn

I changed it: https://github.com/stellar/go/commit/0d0c97b319a7a95015835837f62cf8cd72f21129

Ah sorry I missed it because we used to use the PR title to be the squashed commit message.

You can change it just before squashing.


Respect default app for zip files

Resolved / Related Issues
Items resolved / related issues by this PR.

Closes #6381
Closes #6110

Details of Changes
Add details of changes here.
This PR adds the option to open zip archives in an external application e.g 7zip instead of Files.
There are 2 possible implementations to choose from:

a59b29b: queries the default application associated with zip files, if it's Files or explorer the internal zip viewer is used
2565ef9: adds a setting under "Preferences" to use internal zip viewer or not

I'm unsure what's the better option.
Validation
How did you test these changes?

[x] Built and ran the app

Is it possible to allow the user to set the default application for the zip protocol?

@yaichenbaum In any case I'd like to keep the setting to view zip files in the embedded viewer separated from the default zip program setting. I use both Files and Explorer: when I click a zip file in Files it's ok to open it in Files, but when I click a zip file in explorer I want to open it in explorer. Both options 1 and 2 allow this, your suggestion of changing the default zip program makes me think you have different ideas :)
Could you clarify your expectation on this? Do you expect a switch in Files settings that controls whether to use the embedded zip viewer when browsing with Files and also sets Files as default for zip files?

@gave92 option 1 looks good, I was just trying to better understand the different options 👍


fix: kms key tag missing

Resolves #280

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


Integrated terminal closes unexpectedly

Hi, I am running on latest stable VS Code, and since a few weeks I am suffering of an issue: the integrated terminal closes unexpectedly, and I have to open it again. Doing so I get the old terminal instance plus the new one.
How could I solve it?
Thank you so much!

(Experimental duplicate detection)
Thanks for submitting this issue. Please also check if it is already covered by an existing one, like:

 plugin server terminated unexpectedly (#91813)

Unfortunately it doesn't.

I have just discovered that it does not happen with Developer tools open.


add loop

I added loop.
At the end of the song, check the loop value, initialize the current item value, and play.

Hi @ywangnon! Setting a RepeatMode is supported in a release of the maintained fork linked in the README: https://github.com/DoubleSymmetry/SwiftAudioEx/releases


Milestone 2 - deliverables

Deliverables M2
Documentation / Articles

[x] Testing Guide
[x] Article
[x] Inline documentation
[x] Tutorial M2
[x] grants delivery:
https://github.com/Fair-Squares/Grant-Milestone-Delivery/blob/fs-m2/deliveries/fair_squares-milestone-2.md

Pallets

[x] asset-onboarding-pallet.

[x] all functionality in #96 #94
[x] unit testing
[x] cross pallet unit testing / integration testing
[ ] benchmarks

[x] voting-pallet.

[x] all functionality in #94 #98
[x] unit testing
[x] cross pallet unit testing / integration testing
[ ] benchmarks

Formatting,Linting, testing

[x] cargo-fmt
[x] cargo-clippy
[x] test with Polkadot.js

Docs & Spec page

[ ] Work on the docs page in which we can publish our specification of the development chain and the workings of the blockchain. Also a place to form a decent blog where we can publish ourselves.

Docs & Spec page not created + benchmarks has not been extensive as previous Milestone


TASK: Adjust to renamed Object and Resource namespaces/classes

Instructions: https://discuss.neos.io/t/renaming-flow-core-classes-resource-and-object/1200/9


Thanks for this great code

I thank you for this.
I have been searching for some scripts to convert .bmp images.
I found a couple, but they were no good.
I want to make thumbnails in my forum script.
To convert .bmp to .jpg will allow me to resize them easily in php.
Your script is perfect for this.

@halojoy thank you!


This document reverences Azure when it is for Google

Issue in ./help/sources/connectors/cloud-storage/google-pubsub.md
"Platform allows you to bring in data from Azure Event Hubs in real time."

Thank you for submitting this issue. We will review the documentation and make the necessary updates as quickly as possible. Tracking internally in PLAT-139296.


Remove deprecated FaultTolerantFileSystem or alluxio-ft://

alluxio-ft:// has been removed functionality-wise a long time ago.
now really remove the code since we are after 2.0

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4996/
Test FAILed.

Jenkins, test this please

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/5017/
Test FAILed.

Jenkins, test this please

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/5068/
Test FAILed.

Jenkins, test this please

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/5089/
Test FAILed.

Jenkins, test this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/5181/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/5183/
Test PASSed.

alluxio-bot, merge this please.

@apc999 Thanks for getting rid of this


Android Scene-Viewer not openning

I had an experience working just fine a few weeks ago and today openning a ar using scene-viewer (ar-modes="scene-viewer quick-look") is not working. The device is a Samsung S10 which supports ARCore and the "Google Play Services for AR" is up to date.
When the user clicks on the button to open R.A. the page reload and nothing happens (calls the fallback).

model-viewer: v1.2.1

Browser Affected

[X] Chrome
[ ] Edge
[ ] Firefox
[ ] IE
[ ] Safari

OS

[X] Android
[ ] iOS
[ ] Linux
[ ] MacOS
[ ] Windows

Can you please post a link so I can try to repro?

Yes, of course: https://intro-suvinil-xgb.herokuapp.com/home/index.html
Thank you

144d7eeb-b476-442c-9aa1-5d95aafca259.MP4.zip
Follows a video showing what happens in this exactly same URL

I'm seeing ar-modes="webxr quick-look". Did you update it?

Sorry, I was testing last night, I just rollback to scene-viewer.

Those are the Samsung S10+ device configuration

Hmm, SceneViewer seems to be working fine on my Pixel 3 with your site. @tpsiaki since this sounds like a SceneViewer regression.

Thank you. Any suspicious about what is going on? Since in the video provided it won't open?

I found the issue. The user was openning the link through a webview.


read json file as a whole instead of line by line

Request
When scrapping json files, alloy reads individual lines rather than the content of the json file.
alloy sees the following json data and scrapes each data piece as one line
{
	"msec": "foo_mec",
	"connection": "vader",
	"connection_requests": "sith",
	"ssl_cipher": "mandalorian",
	"scheme": "dark",
	"request_method": "force",
	"server_protocol": "bazinga",
	"pipe": "flute",
	"gzip_ratio": "light speed"
}

It scrapes it properly when it is collapsed into one line.
{"msec":"foo_mec","connection":"vader","connection_requests":"sith","ssl_cipher":"mandalorian","scheme":"dark","request_method":"force","server_protocol":"bazinga","pipe":"flute","gzip_ratio":"light speed"}

Use case
Many systems generate json logs, or logs in json format and generally speaking they are not collapsed. Would be hard to change all of these systems to accomodate alloy.
Could there be a feature, maybe via config, to make alloy collapse the data before scrapping it?
Thanks!

Can multi line in Loki.process meet this need?

Let me try that out, thanks


Add Chat

When new adopters have questions they could ask them without creating issues. Besides that contributors can discuss design patterns and work to be done.
I would suggest Gitter chat.

I think I'll wait on this for now to see if there's enough discussion of this package to warrant a chat, but I'll definitely consider adding Gitter if issues stop scaling well. Thanks for the suggestion!


fix: don't panic in leanPosToLspPos

Testing a problem in the REPL.

Please use the same defaulting as in the function above it


Minimum upgrade to async-capable rust-cuda

[x] keep or revert Copy for Location and IndexedLocation?
[x] find a better hack for EventBuffer
[x] fix extraneous pub use for CUDA kernel
[x] check build
[x] check on hardware
[ ] block on CUDA async merge

[x] async launch and capture semantics

Codecov Report
Attention: 579 lines in your changes are missing coverage. Please review.

Comparison is base (8c6e722) 15.19% compared to head (b6dd445) 15.00%.
Report is 10 commits behind head on main.

:exclamation: Current head b6dd445 differs from pull request most recent head 352b4e7. Consider uploading reports for the commit 352b4e7 to get more accurate results

Files
Patch %
Lines

...oalescence/algorithms/cuda/cpu-kernel/src/patch.rs
0.00%
120 Missing :warning:

.../algorithms/cuda/src/parallelisation/monolithic.rs
0.00%
77 Missing :warning:

rustcoalescence/algorithms/cuda/src/launch.rs
0.00%
59 Missing :warning:

rustcoalescence/algorithms/cuda/src/lib.rs
0.00%
56 Missing :warning:

necsim/impls/cuda/src/event_buffer.rs
0.00%
15 Missing :warning:

...csim/plugins/species/src/individual/feather/mod.rs
0.00%
8 Missing :warning:

necsim/core/bond/src/off_by_one_u64.rs
22.22%
7 Missing :warning:

.../plugins/species/src/individual/sqlite/database.rs
0.00%
7 Missing :warning:

necsim/core/src/simulation/mod.rs
0.00%
6 Missing :warning:

.../parallelisation/monolithic/optimistic_lockstep.rs
0.00%
6 Missing :warning:

... and 112 more

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #271      +/-   ##
==========================================
- Coverage   15.19%   15.00%   -0.19%
==========================================
  Files         315      304      -11
  Lines       21401    21407       +6
==========================================
- Hits         3251     3213      -38
- Misses      18150    18194      +44

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

Superseded by #276


add Z3 package

config-x64-windows-out.log
I am seeing build failures on all triplets in the CI system, can you take a look at it?  I have attached the x64-windows log let me know if you would like to see any other logs.
Starting package 1/1: z3:x64-windows
Building package z3[core]:x64-windows...
Could not locate cached archive: C:\vsts\_work\2\s\archives\c8\c880cc98fbafa5f26c872bc7827aef06104749c3.zip
-- Using cached C:/vsts/_work/2/s/downloads/Z3Prover-z3-z3-4.8.4.tar.gz
-- Extracting source C:/vsts/_work/2/s/downloads/Z3Prover-z3-z3-4.8.4.tar.gz
-- Using source at C:/vsts/_work/2/s/buildtrees/z3/src/z3-4.8.4-84e329967a
-- Configuring x64-windows
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:56 (message):
    Command failed: ninja;-v
    Working Directory: C:/vsts/_work/2/s/buildtrees/z3/x64-windows-rel/vcpkg-parallel-configure
    See logs for more information:
      C:\vsts\_work\2\s\buildtrees\z3\config-x64-windows-out.log

Call Stack (most recent call first):
  scripts/cmake/vcpkg_configure_cmake.cmake:269 (vcpkg_execute_required_process)
  ports/z3/portfile.cmake:11 (vcpkg_configure_cmake)
  scripts/ports.cmake:71 (include)

Error: Building package z3:x64-windows failed with: BUILD_FAILED
Elapsed time for package z3:x64-windows: 10.97 s

Thank you @Rastaban!
That one was easy: just missing the python dependency. Let's see if it works now..

build-x64-windows-dbg-err.log
build-x64-windows-dbg-out.log
Awesome!  It gets past the configuration but now there is a build failure.  Once again I am attaching the log, let me know if you need any other information.

Thanks again @Rastaban! Most tests pass now.
There are 3 failures:

2 x ARM64, which Z3 doesn't support. (I added an error message)
x64-uwp, which I don't know how important it is to support. I can have a look at the logs and try to fix it if you send them, though.

I got these two errors:
6>c:\vsts\_work\2\s\buildtrees\z3\src\z3-4.8.4-3c72faa532\src\util\scoped_timer.cpp(148):
error C3861: 'CreateTimerQueueTimer': identifier not found [C:\vsts\_work\2\s\buildtrees\z3\x64-uwp-dbg\src\util\util.vcxproj]

6>c:\vsts\_work\2\s\buildtrees\z3\src\z3-4.8.4-3c72faa532\src\util\scoped_timer.cpp(193):
error C3861: 'DeleteTimerQueueTimer': identifier not found [C:\vsts\_work\2\s\buildtrees\z3\x64-uwp-dbg\src\util\util.vcxproj]

Supported triplets: x64-linux, x64-osx, x64-windows, x64-windows-static, and x86-windows.

Thanks!


Add Resource Owner Password Credentials flow

Closes #746
I have done my best to mimic the patterns and name conventions in the existing code base, but comments are welcome.
Checklist

[X] This PR makes changes to the public API
[X] I have included links for closing relevant issue numbers

Excellent work, small pipeline issue:
<EMAIL_ADDRESS>lint
> eslint --max-warnings=0 --cache .

/home/runner/work/oidc-client-ts/oidc-client-ts/src/ResponseValidator.test.ts
Warning:   4:20  warning  'Timer' is defined but never used  @typescript-eslint/no-unused-vars
ESLint found too many warnings (maximum: 0).

Excellent work, small pipeline issue:
<EMAIL_ADDRESS>lint
> eslint --max-warnings=0 --cache .

/home/runner/work/oidc-client-ts/oidc-client-ts/src/ResponseValidator.test.ts
Warning:   4:20  warning  'Timer' is defined but never used  @typescript-eslint/no-unused-vars
ESLint found too many warnings (maximum: 0).

Thank you. Fixed.

great work, when the readme/docu is in shape i will merge...


Add Weblium site builder detection

Hi,
Weblium is the site builder with more than 100,000 sites.
Main site - (https://weblium.com/)
10 random sites, that has been built on Weblium
5 sites on custom domains

https://shayganlegal.appnector.com
https://madamewu.monobunt.rocks
https://uascomponents.com
https://growthmarketingstage.com
https://marhoum-institute.appnector.com

5 sites on default domain (weblium.site)

https://neotonecraft.weblium.site/
https://equipmentrails.weblium.site/
https://4darabia.weblium.site/
https://itclusteracademy.weblium.site/
https://bogushturbotimecorp.weblium.site/

Thanks!


missing ServiceWorker externs

are there ServiceWorker externs yet?
I can contribute some if there are not
http://www.w3.org/TR/service-workers/

I haven't seen any come through.

OK, I sent a PR here:
https://github.com/google/closure-compiler/pull/936

Added in fd513f84fecbf83642b1f234c0aa45b250923c35 -- I did a quick diff of that version vs. the one you sent and it looked like they were similar but definitely had some differences. Of course, let us know if you see anything that looks wrong.


APT whitelist request for libssl-dev in Trusty

Needed for SIPp/sipp tests.
This used to work previously. Now got: https://travis-ci.org/SIPp/sipp/jobs/169376976
Disallowing packages: libpcap-dev, libsctp-dev, libncurses5-dev, libssl-dev, libgsl0-dev
Not found in: https://github.com/travis-ci/apt-package-whitelist/blob/master/ubuntu-trusty

https://github.com/travis-ci/apt-package-whitelist/blame/e836179192c68d0655f617f9891add0117eacd25/ubuntu-trusty#L862 libssl-dev is already in the image

Okay, so today it is.
Disallowing packages: libpcap-dev, libsctp-dev, libgsl0-dev

Let me close this then.


[Layer's tree] Show the number of layers in each group and the number of activated layers

Example :

@Bo-Duke is going to try different options.


MQTT Service Suddenly Stop

I'm not sure how to debug this, as there's nothing unusual in the logs to suggest a problem, and I'm not sure how hass interacts with the mqtt server.
I'm running a private server and the only clue in the logs is that the mqtt client connecting as home-assistant always seems to exit with a socket error [1].  The only other error I've found is that the client thinks it's already connected - or isn't sending a proper exit signal so tries to open up a second connection:
Jan  2 16:26:41 zrmt mosquitto[18844]: Client home-assistant already connected, closing old connection.
Jan  2 16:26:41 zrmt mosquitto[18844]: Client home-assistant disconnected.
[1] --
Jan  2 21:37:19 zrmt mosquitto[18844]: New client connected from <IP_ADDRESS> as home-assistant (c1, k60).
Jan  2 21:38:09 zrmt mosquitto[18844]: New connection from <IP_ADDRESS> on port 8883.
Jan  2 21:38:09 zrmt mosquitto[18844]: New client connected from <IP_ADDRESS> as paho/7A3A9CC735492929E2 (c1, k60).
Jan  2 21:38:09 zrmt mosquitto[18844]: Client paho/7A3A9CC735492929E2 disconnected.
Jan  2 21:39:01 zrmt CRON[29810]: (root) CMD (  [ -x /usr/lib/php5/maxlifetime ] && [ -x /usr/lib/php5/sessionclean ] && [ -d /var/lib/php5 ] && /usr/lib/php5/sessionclean /var/li
b/php5 $(/usr/lib/php5/maxlifetime))
Jan  2 21:39:10 zrmt mosquitto[18844]: New connection from <IP_ADDRESS> on port 8883.
Jan  2 21:39:10 zrmt mosquitto[18844]: New client connected from <IP_ADDRESS> as paho/8D7966571D3DDC877E (c1, k60).
Jan  2 21:39:10 zrmt mosquitto[18844]: Client paho/8D7966571D3DDC877E disconnected.
Jan  2 21:40:07 zrmt mosquitto[18844]: Socket error on client home-assistant, disconnecting.

I can confirm similar behavior. In my case, HA stops receiving MQTT messages all together until I restart the service. Haven't seen in anything in HA logs. I'll check my mosquito logs later tonight to see if I have the same socket error message.

I can confirm the bug exists also with Eclipse paho. Please find below the log. It seems that somehow home assistant disconnects.
[2016-01-15T01:32:22.323Z]  INFO: ponte/25738 on cubie: subscribed to topic (service=MQTT, client=home-assistant-occult, topic=patio/patio_pir, qos=0)
[2016-01-15T01:32:22.324Z] DEBUG: ponte/25738 on cubie: Storing offline subscriptions (service=MQTT, client=home-assistant-occult)
[2016-01-15T01:32:22.332Z]  INFO: ponte/25738 on cubie: subscribed to topic (service=MQTT, client=home-assistant-occult, topic=apc/ups/status, qos=0)
[2016-01-15T01:32:22.334Z] DEBUG: ponte/25738 on cubie: Storing offline subscriptions (service=MQTT, client=home-assistant-occult)
[2016-01-15T01:34:37.313Z]  INFO: ponte/25738 on cubie: keepalive timeout (service=MQTT, client=home-assistant-occult)
[2016-01-15T01:34:37.314Z] DEBUG: ponte/25738 on cubie: closing client (service=MQTT, client=home-assistant-occult)
[2016-01-15T01:34:37.318Z]  INFO: ponte/25738 on cubie: unsubscribed (service=MQTT, client=home-assistant-occult, topic=apc/ups/bcharge)
[2016-01-15T01:34:37.320Z]  INFO: ponte/25738 on cubie: unsubscribed (service=MQTT, client=home-assistant-occult, topic=apc/ups/timeleft)
[2016-01-15T01:34:37.322Z]  INFO: ponte/25738 on cubie: unsubscribed (service=MQTT, client=home-assistant-occult, topic=patio/sound_detector)
[2016-01-15T01:34:37.324Z]  INFO: ponte/25738 on cubie: unsubscribed (service=MQTT, client=home-assistant-occult, topic=patio/garage_pir)
[2016-01-15T01:34:37.325Z]  INFO: ponte/25738 on cubie: unsubscribed (service=MQTT, client=home-assistant-occult, topic=patio/patio_pir)
[2016-01-15T01:34:37.327Z]  INFO: ponte/25738 on cubie: unsubscribed (service=MQTT, client=home-assistant-occult, topic=apc/ups/status)
[2016-01-15T01:34:37.328Z]  INFO: ponte/25738 on cubie: closed (service=MQTT, client=home-assistant-occult)
Going to try with a higher timeout, currently is 90.

This is probably related to
https://github.com/balloob/home-assistant/issues/873
On Fri, Jan 15, 2016 at 5:02 AM, Esteban Siravegna<EMAIL_ADDRESS>
wrote:

I can confirm the bug exists also with Eclipse paho. Please find below the
log. It seems that somehow home assistant disconnects.
[2016-01-15T01:32:22.323Z] INFO: ponte/25738 on cubie: subscribed to topic
(service=MQTT, client=home-assistant-occult, topic=patio/patio_pir, qos=0)
[2016-01-15T01:32:22.324Z] DEBUG: ponte/25738 on cubie: Storing offline
subscriptions (service=MQTT, client=home-assistant-occult)
[2016-01-15T01:32:22.332Z] INFO: ponte/25738 on cubie: subscribed to topic
(service=MQTT, client=home-assistant-occult, topic=apc/ups/status, qos=0)
[2016-01-15T01:32:22.334Z] DEBUG: ponte/25738 on cubie: Storing offline
subscriptions (service=MQTT, client=home-assistant-occult)
[2016-01-15T01:34:37.313Z] INFO: ponte/25738 on cubie: keepalive timeout
(service=MQTT, client=home-assistant-occult)
[2016-01-15T01:34:37.314Z] DEBUG: ponte/25738 on cubie: closing client
(service=MQTT, client=home-assistant-occult)
[2016-01-15T01:34:37.318Z] INFO: ponte/25738 on cubie: unsubscribed
(service=MQTT, client=home-assistant-occult, topic=apc/ups/bcharge)
[2016-01-15T01:34:37.320Z] INFO: ponte/25738 on cubie: unsubscribed
(service=MQTT, client=home-assistant-occult, topic=apc/ups/timeleft)
[2016-01-15T01:34:37.322Z] INFO: ponte/25738 on cubie: unsubscribed
(service=MQTT, client=home-assistant-occult, topic=patio/sound_detector)
[2016-01-15T01:34:37.324Z] INFO: ponte/25738 on cubie: unsubscribed
(service=MQTT, client=home-assistant-occult, topic=patio/garage_pir)
[2016-01-15T01:34:37.325Z] INFO: ponte/25738 on cubie: unsubscribed
(service=MQTT, client=home-assistant-occult, topic=patio/patio_pir)
[2016-01-15T01:34:37.327Z] INFO: ponte/25738 on cubie: unsubscribed
(service=MQTT, client=home-assistant-occult, topic=apc/ups/status)
[2016-01-15T01:34:37.328Z] INFO: ponte/25738 on cubie: closed
(service=MQTT, client=home-assistant-occult)
Going to try with a higher timeout, currently is 90.
—
Reply to this email directly or view it on GitHub
https://github.com/balloob/home-assistant/issues/846#issuecomment-171954304
.

--
PaulusSchoutsen.nl
It's nice to be important but it's more important to be nice.

If #873 and #922 didn't solve this issue, please re-open it. Thanks.


difficult to pull the flash games,it always halt down there

use command "docker pull quay.io/openai/universe.flashgames:0.20.2", Does anyone have any other way to get the flash game source.

I had the same problem and just had to ctrl-c out and try try again till everything downloaded.  But then when I launched the test script it went and downloaded the 20.1 container which seems to be downloading fine...

how long did you use to download the flash games? @wagonhelm

Took about 15 min for me seems different environments use different docker container versions.

I have the same problem pulling from quay.io/openai/universe.flashgames, it just seems to be taking forever (for the entire night it still says "waiting"). I tried docker pulling from docker hub and it was fine. Seems a problem with the networks

See https://github.com/openai/universe/wiki/Solutions-to-common-problems#docker-wont-pull-images


Add information about Windows charsets

Current coverage is 76.94% (diff: 100%)

Merging #107 into master will increase coverage by 6.37%

@@             master       #107   diff @@
==========================================
  Files             4          4
  Lines           418        603   +185
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            295        464   +169
- Misses          123        139    +16
  Partials          0          0

Powered by Codecov. Last update cd4a68e...7e66d40


[Openstack] move scope logic in its own file

What does this PR do?
The openstack check is the empirical result of individual contributions, bug and performance fix.
As a result the check is complex to understand. This is mainly because everything is in one python file.
This PR isolates scopes and service catalog
It also makes private methods correctly
TODO: Get rid of calls made in except of try/catches (for perf reasons but not only)
Motivation

Better readability
Better testability (the idea is also to add tests to each individual part at some points)
Potentially find improvements (perf and design)

Review checklist

[ ] PR has a meaningful title or PR has the no-changelog label attached
[ ] Feature or bugfix has tests
[ ] Git history is clean
[ ] If PR impacts documentation, docs team has been notified or an issue has been opened on the documentation repo

Additional Notes

Closing this PR in favor of https://github.com/DataDog/integrations-core/pull/2524


Correction for raw affiliation Faculty of Engineering, Mechanical Engineering Department, Lebanese University, Lebanon; Univ Rennes, INSA Rennes,  LGCGM (Laboratoire de Génie Civil et Génie Mécanique), Rennes, France

Correction needed for raw affiliation Faculty of Engineering, Mechanical Engineering Department, Lebanese University, Lebanon; Univ Rennes, INSA Rennes,  LGCGM (Laboratoire de Génie Civil et Génie Mécanique), Rennes, France
raw_affiliation_name: Faculty of Engineering, Mechanical Engineering Department, Lebanese University, Lebanon; Univ Rennes, INSA Rennes,  LGCGM (Laboratoire de Génie Civil et Génie Mécanique), Rennes, France
new_rors: 02fke9256;04xaa4j22;015m7wh34;05x6qnc69;03x0g1010
previous_rors: 02fke9256;04xaa4j22;015m7wh34;05x6qnc69
works_examples: W4385429559
contact: e8240947f79c49368665b72c7c1291ca:9bbfb9f05149f9e22582b24ecc9b439f @ univ-rennes.fr

This issue was accepted and ingested by the OpenAlex team on 2024-10-10. The new affiliations should be visible within the next 7 days.


Add Minimal sentry config

Added Mimimal Sentry config for the application.

This pull request introduces 1 alert when merging ce0aa8c into 0d5b2af - view on LGTM.com
new alerts:

1 for Unused local variable

Done

Please create it as a separate compose file, we want it as an optional addition and not as an integral part of the system.
If you want to install new packages, please add them to requirements.txt/dev_requirements.
Please add the required global variables to the configuration.


Enable Defender on Portal Deployment

Overview/Summary
I added support for Portal deployment option. If enabled, the wizard will deploy an policy initiative with 4 different policies, for Servers, Storage, ARM and Key Vault

As part of this Pull Request I have

[ ] Read the Contribution Guide and ensured this PR is compliant with the guide
[ ] Ensured the resource API versions in .bicep file/s I am adding/editing are using the latest API version possible
[ ] Checked for duplicate Pull Requests
[ ] Associated it with relevant GitHub Issues
[ ] (AVD LZA Team Only) Associated it with relevant ADO Items
[ ] Ensured my code/branch is up-to-date with the latest changes in the main branch
[ ] Performed testing and provided evidence.
[ ] Updated relevant and associated documentation (e.g. Contribution Guide, Module READMEs, Docs etc.)

@sihbher thank you for your contribution, please check the code scanning results, the check is failing.


ERROR: non-ASCII filename not supported on this filesystem

I'm trying to build a document that includes filenames and directories in Japanese in the _static directory, but the HTML build fails.
Steps to reproduce

Go to kanary.rtfd.org
Go to the project "Builds" page
Click the "Build version:" button

Expected Behavior

The Recent builds list in the project "Builds" page should display the latest build as Passed version latest (html).
Going to http://kanary.rtfd.org/ should show the latest HTML build.

Unexpected Behavior

The Recent builds list in the project "Builds" page displays the latest build as Failed version latest (html).
Going to http://kanary.rtfd.org/ shows an old HTML build.

The HTML build error details are available in https://readthedocs.org/builds/kanary/1734409/ . The following is a summary:
ひ:: ERROR: non-ASCII filename not supported on this filesystem encoding 'ANSI_X3.4-1968', skipped.

Traceback (most recent call last):
  File "/home/docs/local/lib/python2.7/site-packages/sphinx/cmdline.py", line 254, in main
    app.build(force_all, filenames)
  File "/home/docs/local/lib/python2.7/site-packages/sphinx/application.py", line 212, in build
    self.builder.build_update()
  File "/home/docs/local/lib/python2.7/site-packages/sphinx/builders/__init__.py", line 214, in build_update
    'out of date' % len(to_build))
  File "/home/docs/local/lib/python2.7/site-packages/sphinx/builders/__init__.py", line 279, in build
    self.finish()
  File "/home/docs/local/lib/python2.7/site-packages/sphinx/builders/html.py", line 482, in finish
    self.copy_static_files()
  File "/home/docs/local/lib/python2.7/site-packages/sphinx/builders/html.py", line 595, in copy_static_files
    ctx, exclude_matchers=matchers)
  File "/home/docs/local/lib/python2.7/site-packages/sphinx/util/__init__.py", line 171, in copy_static_entry
    exclude_matchers=exclude_matchers)
  File "/home/docs/local/lib/python2.7/site-packages/sphinx/util/__init__.py", line 171, in copy_static_entry
    exclude_matchers=exclude_matchers)
  File "/home/docs/local/lib/python2.7/site-packages/sphinx/util/__init__.py", line 167, in copy_static_entry
    if path.isdir(path.join(source, entry)):
  File "/home/docs/lib/python2.7/posixpath.py", line 80, in join
    path += '/' + b
UnicodeDecodeError: 'ascii' codec can't decode byte 0xe3 in position 1: ordinal not in range(128)

Encoding error:
'ascii' codec can't decode byte 0xe3 in position 1: ordinal not in range(128)
The full traceback has been saved in /tmp/sphinx-err-W5Jvn4.log, if you want to report the issue to the developers.

Believe this was fixed.

Now i have the same problem when I'm trying to build a document that includes filenames in Russian.
For example - https://readthedocs.org/projects/diadocsdk-1c/builds/3388391/

This may be due to our new builder backeds, reopening.

I also encounter similar problem. Any clue ? or I can try to fix it with the traceback I get. This is a big problem to me.
Traceback (most recent call last):
  File "/home/docs/checkouts/readthedocs.org/user_builds/wdv4758h-notes/envs/latest/lib/python3.4/site-packages/sphinx/cmdline.py", line 245, in main
    app.build(opts.force_all, filenames)
  File "/home/docs/checkouts/readthedocs.org/user_builds/wdv4758h-notes/envs/latest/lib/python3.4/site-packages/sphinx/application.py", line 264, in build
    self.builder.build_update()
  File "/home/docs/checkouts/readthedocs.org/user_builds/wdv4758h-notes/envs/latest/lib/python3.4/site-packages/sphinx/builders/__init__.py", line 245, in build_update
    'out of date' % len(to_build))
  File "/home/docs/checkouts/readthedocs.org/user_builds/wdv4758h-notes/envs/latest/lib/python3.4/site-packages/sphinx/builders/__init__.py", line 259, in build
    self.doctreedir, self.app))
  File "/home/docs/checkouts/readthedocs.org/user_builds/wdv4758h-notes/envs/latest/lib/python3.4/site-packages/sphinx/environment.py", line 618, in update
    self._read_serial(docnames, app)
  File "/home/docs/checkouts/readthedocs.org/user_builds/wdv4758h-notes/envs/latest/lib/python3.4/site-packages/sphinx/environment.py", line 638, in _read_serial
    self.read_doc(docname, app)
  File "/home/docs/checkouts/readthedocs.org/user_builds/wdv4758h-notes/envs/latest/lib/python3.4/site-packages/sphinx/environment.py", line 797, in read_doc
    self.process_images(docname, doctree)
  File "/home/docs/checkouts/readthedocs.org/user_builds/wdv4758h-notes/envs/latest/lib/python3.4/site-packages/sphinx/environment.py", line 1000, in process_images
    if not os.access(path.join(self.srcdir, imgpath), os.R_OK):
UnicodeEncodeError: 'ascii' codec can't encode characters in position 100-103: ordinal not in range(128)

Encoding error:
'ascii' codec can't encode characters in position 100-103: ordinal not in range(128)
The full traceback has been saved in /tmp/sphinx-err-guzhkhnm.log, if you want to report the issue to the developers.

I think https://github.com/rtfd/readthedocs-docker-images/pull/12 will fix this too.

Has it already deployed ?

@wdv4758h hi, what's you project URL on RTD? are you still having this issue with unicode name files? The kanary project doesn't exist anymore and diadocsdk-1c-docs has only one unicode filename and it's not being used to build the documentation.

@humitos No, I don't think I still having this issue.
Actually, I haven't use readthedocs for a long time. My project is using CI with Sphinx to build now.
And I just check the readthedocs of my project, it doesn't show that problem anymore. It still fail for some reason I can't see, though. Maybe it needs some modification.

I just tried this with diadocsdk-1c-docs project in my local instance using build:2.0 docker image by renaming two different files and adding them to the toctree and it worked.
Here is the diff applied in their project to test this:
diff --git a/docs/source/Organization.rst "b/docs/source/Organizaci\303\263n.rst"
similarity index 100%
rename from docs/source/Organization.rst
rename to "docs/source/Organizaci\303\263n.rst"
diff --git a/docs/source/index.rst b/docs/source/index.rst
index 08f1665..8d1bcfb 100644
--- a/docs/source/index.rst
+++ b/docs/source/index.rst
@@ -13,6 +13,9 @@ AddIn Diadoc API - внешняя компонента, предназначен
    Примеры использования<Examples>
    Ссылки для скачивания <Downloads>

+   официальный <официальный>
+   Organización <Organización>
+
 .. toctree::
    :name: methods
    :maxdepth: 1
diff --git a/docs/source/Official.rst "b/docs/source/\320\276\321\204\320\270\321\206\320\270\320\260\320\273\321\214\320\275\321\213\320\271.rst"
similarity index 100%
rename from docs/source/Official.rst
rename to "docs/source/\320\276\321\204\320\270\321\206\320\270\320\260\320\273\321\214\320\275\321\213\320\271.rst"

Closing issue since it's solved. Feel free to re-open if you find something else.


Add debug messaging around sendbeacon

In reference to https://github.com/ankane/ahoy.js/issues/64
This issue is currently causing a lot of TypeError: Illegal Invocation error metrics. I don't think it will be very easy to provide a way to reproduce but would it be possible to improving the visibility into the circumstances surrounding the exceptions being thrown? The error it self doesn't give very much info on why it's taking place. Would it be possible to add a try catch to give useful debug output and rethrow, such as an output of eventsUrl() and data and index_module_1's return in order to help us with debugging the Illegal Invocations?

Hey @wenqige, you can fork and try that, but I'm not sure it'll provide any more info than the exception itself. It could be related to one of these issues:

https://github.com/vercel/next.js/issues/23856
https://github.com/miguelmota/Navigator.sendBeacon/issues/8


teeworlds: do not allocate for ipv6 servers

improvement: ipv6 server were just not written but an ipv4 slot was still allocated: fixed
for ipv4 only lists, the server->master_pkt array is extended for each packet read
for ipv4/ipv6 mixed list, the server->master_pkt array is extended for each ipv4 read
fix: extra unused ipv4 slot for ipv6 server were mistakenly left uninitialized but readable
in case of ipv6 address a slot was left uninitialized in the server->master_pkt array between
two valid ipv4 address slots (even valgrind was not complaining but the bug was there)
fix: return INPROGRESS after normal packet read too, only legacy ones was explicitely doing it
make ipv4 header variable name more Teeworlds specific since it's a global variable
declare teeworlds-specific global variables as static ones
handle allocation errors
print debug information
improve explanations

I added memory (re)allocation error handling plus debug information printing

Closing due to feedback timeout, feel free to reopen

Well, this Teeworlds effort is already more than 30 months old. So for what I'm used to wait, 6 months is like a day.


Fix typo in LegacyDsl message

This fixes a small typo I noticed in the message for LegacyDsl.

Hi @RyanNielson, could you please that minor Rubocop offence? Thank you for catching that!

Thank you so much!


Add React Native MercadoPago

Why
This PR adds react-native-mercadopago-px to the react-native-libraries.json. MercadoPago is one of the biggest payment platforms from Latam.
Checklist

If you added a new library:

[X] Added it to react-native-libraries.json

If you added a feature or fixed a bug:

[ ] Documented in this PR how to use the feature or replicate the bug.
[ ] Documented in this PR how you fixed or created the feature.

@Simek you're welcome! Thank you for the faster review and merge!

Why
This PR adds react-native-mercadopago-px to the react-native-libraries.json. MercadoPago is one of the biggest payment platforms from Latam.
Checklist
If you added a new library:

[x]  Added it to react-native-libraries.json

If you added a feature or fixed a bug:

[ ]  Documented in this PR how to use the feature or replicate the bug.
[ ]  Documented in this PR how you fixed or created the feature.

amazing thanks for this PR!!

@JonatanSalas  thanks for this PR amazing!!


Reduced number of calls to _determine_vcs_base_and_type

Reduce the number of times that we determine which VCS is being used.
Major changes:

Inlined the _determine_vcs_base_and_type function because we want the environment variables set.
Always set VCS_TYPE and REPOBASE variables
Removed which_vcs as it no longer makes sense

This patch may have a minor conflict with #88 at the beginning of blackbox_deregister_file, but it's a trivial thing to fix.

LGTM

Ugh.  I merged this when I meant to merge the other one.  I've undone the merge.  Sorry for any confusion.

Can you do me a favor and rebase this (if needed) and re-push it?  github is confused and/or I don't know how to use git as well as I should.


Adding false to allowed values for animation property

Related issue: https://github.com/uber/react-vis/issues/162

Thanks for your contribution, @jameskraus !
Before I merge in your changes, I need you to sign our Uber Contributor License Agreement: http://t.uber.com/cla
Could you please do that and then post a comment here when that's done?

I can't sign any CLAs. If you wanted to make a similar, yet unique change to the code, then I guess that would work. It just so happens that there is a built in proptype that encapsulates oneOf([true, false]), so anyone else could make that switch.


Turn Description for first and last walk leg

Why don't you show a Turn Description for the Walk legs at the beginning and at the end?

Assuming is prod
Response: https://gist.github.com/vasile/3eececa6b96fabf71453d86b0937cc81
Looks like a bug on the client, need to patch also the ContinuousLeg - marked as bug.

For Chur / Chur PostautoStation
show at least level / footpath as in the response
<ojp:PathLink>
    <ojp:Transition>level</ojp:Transition>
    <ojp:AccessFeatureType>footpath</ojp:AccessFeatureType>
</ojp:PathLink>

@u233336 @ue71603 - please check also for sharer / bike transfer if we have a response

Self-Drive-Car
https://opentdatach.github.io/ojp-demo-app/?from=46.972505%2C6.832668&to=46.994297%2C6.935128&mot_types=self-drive-car&trip_datetime=2021-08-06+14%3A39&stage=test
There are GuidanceAdvice:
ojp:GuidanceAdviceorigin</ojp:GuidanceAdvice>
ojp:TurnActionleft</ojp:TurnAction>
Sharing
https://opentdatach.github.io/ojp-demo-app/?from=46.965347%2C6.852451&to=46.994297%2C6.935128&mot_types=shared-mobility&trip_datetime=2021-08-06+14%3A39&stage=test
There is also GuidanceAdvice:
ojp:DurationPT0H1M24S</ojp:Duration>
ojp:Length94</ojp:Length>
</ojp:TrackSection>
ojp:GuidanceAdviceturn</ojp:GuidanceAdvice>
ojp:TurnActionleft</ojp:TurnAction>
</ojp:PathGuidanceSection>

always on test now

Actually there is no GUI element to show the turn descriptions (at least none that I know off). They are transmited, but not displayed.
For OJP 2.0 we try to fix this and improve the standard, in order that all possible turn actions can be shown (Some usual sentance are not yet possible).

Is OK for now, so we close.


-[MTLCompileOptionsInternal setMaxTotalThreadsPerThreadgroup:]: unrecognized selector sent to instance

Mac Ventura(Intel) 13.1 fail run after compile.
There a error about setMaxTotalThreadsPerThreadgroup in Metal-cpp. After I comment it, the program run well because it generate shader cache.

Hi, @yangfengzzz
Thanks for reporting this. This method was introduced in macOS 13.3+ so your system failed to find the selector. I fixed it in 4b7857d by adding an availability test before using it.


inference for library jars

This is probably NullAway's current biggest gap; unannotated libraries.  We mostly care about annotations on methods, and we don't care about generating annotations that would make the original source code of the library type check.  So, given a set of jars, we could try the following:

For any method directly de-referencing parameter p (unconditionally) or copying p to a known @NonNull location, mark p as @NonNull.
For any method directly returning null or returning a known @Nullable value, mark its return as @Nullable.

We need to iterate 1 and 2 to a fixed point, using a type-based call graph / worklist.  Should be pretty easy to implement using WALA.  Open questions:

How do we deal with virtual calls?
Do we need to make the annotations on the type hierarchy consistent?
Should we do fancier dataflow to discover more possibly-@NonNull parameters and @Nullable returns?  Not clear it's worth it.
Some annotations, while strictly correct, will introduce a lot of noise for clients, as there is some unknown protocol around ensuring (e.g.) null values are not returned.  Not sure if there is any way to account for this in the inference.

Closing this out as it's subsumed by our work on JarInfer


fix(polish): name for pl_a_ogonek

This change updates the name for Unicode "ą/Ą" characters.

Good catch, thanks for fixing it


Fix DAP issues for Dropdown

Basic

Level
Description

V
An element with WAI-ARIA role option is not contained in or owned by an element with one of the following WAI-ARIA roles: listbox. See rule help for remediation guidance.

V
All content must reside within a WAI-ARIA landmark or labelled region role.

Multi-select

Level
Description

V
Each form control must have associated label.

V
An element with WAI-ARIA role option is not contained in or owned by an element with one of the following WAI-ARIA roles: listbox. See rule help for remediation guidance.

V
All content must reside within a WAI-ARIA landmark or labelled region role.

V
Widget with role of option must have no more than one tabbable element.

Basic

We have have to move the role="listbox" up in the tree
association or preview issue

Multi-select

Technically correct, but assuming it's talking about the checkboxes, not valid since they aren't interactive. We could give them an aria-label with the item.content though
We have have to move the role="listbox" up in the tree
association or preview issue
I think this is just some tabindex mangement ... the checkboxes shouldn't be tabbable


Delay evaluation of orElse provider when computing task dependencies

Fixes #13333

@bot-gradle test this

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test this

OK, I've already triggered ReadyForMerge build for you.


LAEA NaN bug and possible solution

The LAEA Transform seems to be broken. Any transform (direct or inverse) is resulting in a NaN value.
DotSpatial version:
I have tested it in the 1.9 binary donwloadable version and after building the dotspatial-9be3db98e21ba9bb7ce0c622dfebbd044beadc84. it
It seems that the initialization of the LAEA Transform is wrong.
File Projections/Transforms/LambertAzimuthalEqualArea.cs, line 328 reads:
_sinb1 = Proj.Qsfn(sinphi, E, OneEs);
And based on PJ_laea.c it should read:
_sinb1 = Proj.Qsfn(sinphi, E, OneEs)/_qp;
Regards!
Iratxo.

Iratxo, please try latest 2.0 version. This issue should be already fixed.

It seems that 2.0 version is not already released, right? I have tried downloading the trunk and the problem appears to be there.

@iratxoalbentia Have a look at line 329 of LambertAzimuthalEqualArea.cs. It contains the change you're talking about. You can find the compiled dlls of the current master version here.

Already fixed.


✨ latest api updates

Changes

Adds ability to trigger plaid refresh
Latest updates from reviewing developer docs
All boolean flags default to None which means less default params get specified on API calls

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


Fix previous PR

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 99.872%

Totals

Change from base Build<PHONE_NUMBER>8:
0.0%

Covered Lines:
3116

Relevant Lines:
3120

💛  - Coveralls


[GH-438] Implement dungeon settlement level up. Fix currency assets

[!CAUTION]
To be merged with https://github.com/lambdaclass/mirra_backend/pull/656

Closes #438
Motivation
We need to level up the Dungeon Settlement, being able to see the costs in the screen.
Summary of changes

Implement a DungeonSettlementManager that will handle most of the gameobjects that display information or perform an action in the Dungeon Settlement scene.
Implement the functionality of the level up button.
Update gateway.proto with the necessary changes

How has this been tested?

Go to the DungeonSettlement scene.
You should see 50 blueprints available to spend in the header, and a cost of 100 gold + 50 blueprints to level up. Your settlement's level should be 1.
Click/tap on the level up button. The level should increment in 1, the blueprints amount in the header should decrease to 0 and the supplies' AFK Rate should not be 0 anymore.
Try to level up again. You should get a pop up informing that currency is not enough.

Checklist

[x] I have tested the changes locally.
[x] I self-reviewed the changes on GitHub, line by line.
[ ] Tests have been added/updated.
[ ] This change requires new documentation.

[ ] Documentation has been added/updated.

[ ] I have tested the changes in another devices.

[ ] Tested in iOS.
[ ] Tested in Android.

Shouldn't the afk rate appear also on the popup? Between the parentheses?

Yeah, I'll tackle that pop up when I implement the rewards claim

Shouldn't the afk rate appear also on the popup? Between the parentheses?

Yeah, I'll tackle that pop up when I implement the rewards claim

Great! Then You have my approve!


[1.5 nightly] getCellHTML use wrong date format

w2utils.formatDate(unixTimestamp, formatString) called with formatted date instead of unix timestamp.
Using dd.mm.yyyy format and the the sends something like this (unix timestamp):
records: [
   {LFDNR: 1, F1STR: "jkjkjkjk", F1NUM: 2, F1DATE:<PHONE_NUMBER>000, F1CHAR: "1",…}, ...
]

Ok it's not a bug from the library. I think, the date format from the server should be the same like in the w2ui configuration.
So I managed to send the same date format like in the locale settings from the grid. Now it works like a charm, but german localization is messed up. When I choose a date with the datepicker 'dd.mm.yyyy' the library convert it to 'mm.dd.yyyy' format and breaks.

This commit should solves the problems with some date formats. Also the en-us.json date_format was different from the default in w2utils.js.

Fix for this is coming in https://github.com/n3tfr34k/w2ui/commit/74c45580fcc79e7d33fd817d31c352bbce64c3be, I am going merge it, so, I am closing this request. Thanks.


New: Initial squeleton for full resync framework on the monitor

Codecov Report

Merging #261 into master will increase coverage by 0.23%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #261      +/-   ##
==========================================
+ Coverage   54.85%   55.09%   +0.23%
==========================================
  Files          38       38
  Lines        4147     4149       +2
==========================================
+ Hits         2275     2286      +11
+ Misses       1678     1667      -11
- Partials      194      196       +2

Impacted Files
Coverage Δ

monitor/rpcmonitor/rpc.go
75.38% <0%> (-1.18%)
:arrow_down:

monitor/linuxmonitor/linuxMonitor.go
70.52% <0%> (+11.57%)
:arrow_up:

Continue to review full report at Codecov.

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

@bvandewalle can you give an update on this? I know that you started it, but is it complete? Can you tag it as ready to merge?

Closing as the new Sync framework was refactored by @satyamsi


launch web browser and open remote URL

I actually need someone else testing :)

thank you!

have you test it ?

no, i did not. i have just reviewed the code

Only have 1 TV to test on :(
Hope it works for more people :)


support birdhouse-deploy>=2.4.0

This updates the scheduler-jobs/deploy_data_pavics_jupyter.env script to allow it to discover the log directory using the BIRDHOUSE_LOG_DIR or the PAVICS_LOG_DIR environment variable.
This is in order to support the variable name changes introduced in birdhouse-deploy versions >= 2.4.0.

Add details in the CHANGES (I see that https://github.com/bird-house/pavics-jupyter-base/tree/0.5.0 exists, but the section is not updated, so need to adjust this).
The edit is manual in this repo.
Once merged, DockerHub should auto-build the latest with master.
Then, a tag can then be pushed with the corresponding update of CHANGES, and DockerHub will once again built it for this tag.
I see that https://github.com/bird-house/pavics-jupyter-base/tree/support-birdhouse-deploy-2.4.0 build fails: https://hub.docker.com/repository/registry-1.docker.io/birdhouse/pavics-jupyter-base/builds/e9ffc5ec-9b3e-4ca0-b5b0-68875900e0c4
Need to address this before merging/tag/release.


Make sure temp whitelist properly matches on subdomains

Reviewer: @MariagraziaAlastra

Description:
I made a change to make sure we weren't over-matching on domains in the temp whitelist, but it ended up under-matching (specifically on subdomains). This change should fix that issue.
Steps to test this PR:

Automated tests:

[x] Unit tests
[ ] Integration tests

Reviewer Checklist:

[ ] Ensure the PR solves the problem
[ ] Review every line of code
[ ] Ensure the PR does no harm by testing the changes thoroughly
[ ] Get help if you're uncomfortable with any of the above!
[ ] Determine if there are any quick wins that improve the implementation

PR Author Checklist:

[ ] Get advice or leverage existing code
[ ] Agree on technical approach with reviewer (if the changes are nuanced)
[ ] Ensure that there is a testing strategy (and documented non-automated tests)
[ ] Ensure there is a documented monitoring strategy (if necessary)
[ ] Consider systems implications

👍


Evaluate audio in iPython notebooks

It would be nice to have the audio work. Do you see a way to make that happen? I could also change how I'm doing the audio in the notebooks if that would make interoperability easier.

Just wanted to double check. Are the sound clips in the notebook supposed to be in the file like the pngs are? Right now it seems like it's looking for them in local files

Oh, hmmm... I hadn't realized that. There might be a way to embed them in the notebook. Keeping them as separate files may be cleaner. I'm open to either.

I think seperate files might be easier. And it wouldn't add that much more work. Could you send me the files for this notebook?

If you don't mind creating an account on Kaggle, it might be easiest to download them from here: https://www.kaggle.com/c/multilabel-bird-species-classification-nips2013/download/NIPS4B_BIRD_CHALLENGE_TRAIN_TEST_WAV.tar.gz
I spoke with the contest organizers and they are fine with my using select files for education purposes.

So the audio is working now. For the moment I'm just converting the notebook and then manually adding in the audio links. How many audio files do you think you'll be adding to the average post? Trying to figure out if it's worth automating that

What is the process? ipynb -> markdown -> html -> html + links?
I'm planning to do several posts involving audio, maybe ~30 audio clips total. What do you recommend?

Are all the audio files from that same set? Either way though it should only take a couple of minutes to add them in so not sure if it's really worth the effort or expense of automating
Right now it's ipynb -> md/html -> add links

Mmmm... good point. The audio files are coming from all over, so there will already be a manually step to host them somewhere (even if just in the repo). Manual sounds good.


Live - "oops error"

Changed our module from test to live, and get this error. Can you help investigate this?

Nevermind, It's the merchant this is not set to "Live" in the Admin portal.
I will close this issue.


fix: incorrect currency displayed in profiledropdown

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR type

[x] Bugfix
[ ] Feature
[ ] Refactoring

What's new?

[x] PR closes #4056
[ ] Requires deployment <rubick/magick_issue>
[ ] <brief_description_of_what_I've_added>

Before submitting Pull Request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality
[x] I've posted a screenshot of demonstrated change in this PR

Optional

[ ] I've tested it at </rmrk/collection/26902bc2f7c20c546a-1FVG7>
[ ] I've tested PR on mobile and everything seems works
[ ] I found edge cases
[ ] I've written some unit tests 🧪

Had issue bounty label?

[x] Fill up your KSM address: Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot
BSX:

[x] My fix has changed something on UI; a screenshot is best to understand changes for others.

hmm, I guess it's not possible to have bsx on rmrk :)

Good catch! Gallery items from different chains may display on the Newest list at the same time.
✅ FIXED

1.0000 KSM ? Like should be 1 KSM I guess?

1.0000 KSM ? Like should be 1 KSM I guess?

on bsx:

on rmrk:

It seem related to https://github.com/kodadot/nft-gallery/issues/3868

hmm so till then I guess we can merge it?

let's see
pay 50

😍 Perfect, I’ve sent the payout
💵 $50 @ 34.44 USD/KSM ~ 1.452 $KSM
🧗 Caiv9TbPz68q5dC8EcHu5xKYPRnremimGzqmEejDFNpWWLG
🔗 0x0cddd54ad5108d54817ded1dd3fa96ce06ed92713e5b3b45f65b223aac85221e
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


Enhancement: Maya fixes, optimizations and logging tweaks

Changelog Description

Tweak logging levels
Fix a lot of validators to now actually report a PublishValidationError for better UX
Fix Collect Inputs for Renderlayers (I suspect this was broken since new publisher)
Optimize get_id_required_nodes (this is much faster e.g. on scene save for many nodes)
Fix Global AOV Mode support for Arnold (to disable/enable all AOVs in a renderlayer)
Add Global AOV Mode support for Redshift (to disable/enable all AOVs in a renderlayer)
Remove some pre-Maya 2017 support
Some cosmetics as well
Only collect file input dependencies if the setting is enabled in project settings

Additional info
Sorry for the massive PR - it was just for much for me to break this into smaller parts. Most commits are 'isolated' so if you have the spare time feel free to pick the commits you need to break this into smaller PRs.
Testing notes:

Go crazy in Maya
Publish a variety of families, models, looks, renders, pointcaches

I've discovered one more issue - some of the validators are not inheriting from the mixin, therefor not respecting "optional" states and so on. This needs to be fixed but from the scope of this PR and possible conflicts, I am wondering if I shouldn't push those fixes here or create separate PR and solve conflicts later.

I had noticed that a lot of things that can be made 'optional' in settings indeed do not inherit from the optional mixin class. Whether it's best kept separate or fixed in this PR I'll leave to others to decide - this one is already quite a hefty one but since we need to test the different cases anyway we might as well put it in here.
@iLLiCiTiT thoughts?

I am wondering if I shouldn't push those fixes here or create separate PR and solve conflicts later.

I did it here #5575

@BigRoy I think this PR might be too big. Could you maybe split the changes up into smaller PRs, maybe 1 per changelog?
I think that'll be faster than getting through this PR.

I think this PR might be too big. Could you maybe split the changes up into smaller PRs, maybe 1 per changelog?
I think that'll be faster than getting through this PR.

Unfortunately I don't have time to do so, but I agree that there are definitely a few features in here that are 'big changes' (like the cbId optimizations). Will close this PR and maybe look into generating tiny PRs from our colorbleed branch over time?
This is just a part of the diff between the branches even though most features should be able to work fine in OP and aren't too specific to our studio.


Rebuilding the images with latest PHP versions

Hi,
Firstly, thanks for your work. 👍
Secondly, could you please rebuild & publish the images with latest PHP versions? I use these images to run some linting & testing steps on CI, and one of my projects fails as it requires PHP 8.1.20.
Cheers.

Done ✅
Thanks for the reminder!


Fix deprecated use of 'unicode' in python3 (v2)

any news?

Oops sorry ! I'm not sure why but I stopped receiving GitHub notifications in my mailbox...


Fix Do Not Disturb on Python 3.5+

An attempt to fix Do Not Disturb on all Python 3.x versions. import distutils.utils failed Travis CI build for 3.4.2.

Coverage remained the same at 100.0% when pulling b9b1cd0390ccf889b3a177fb2ff10cb7a77e4dd5 on dale3h:do-not-disturb into a0f63d92307e68224bacf4f4bf44524a9ebf94e7 on MisterWil:master.

I give up. I am convinced that the lint for 3.4.2 is messed up somehow.


Detect calling 'class' level methods against an instantiated/hydrated model, such as first()

Feature Request
My apologies if this has been discussed before, but I tried hard to search the issues/discussions about this and couldn't find anything ... but it was hard to search for 'exactly' this.
We ran into a bug in our codebase today that was a 'stupid bug', and was something that I would have 100% expected static analysis to find and alert us about.   (To be fair, I would have expected PHP/Laravel to throw an error if you try this, but it appears that it's "by design".
The situation
So the simplest example of this problem, is that you are able in Laravel to make a call to what I would call a 'class level' method (ie, referencing relationships, scopes, etc) against a single instantiated/hydrated model, where you wouldn't expect this to happen.  For example, the first() method.   Which means, if you have a model called Tenant, then the following code 'works':
Tenant::find(56789)->first();

It works, but it doesn't do at all what you would expect, because what it does is return the very first tenant in your system.   Which obviously is not at all what one would expect to have gotten.   While this exact case is somewhat 'made up'.  The real example was more hidden in our code, because we basically had the following:
// in Policy model:
    public function tenants(): BelongsToMany
    {
        return $this
            ->belongsToMany(Tenant::class, 'tenant_policy', 'policy_id', 'tenant_id')
            ->using(TenantPolicy::class)
            ->withPivot('type')
            ->withTimestamps();
    }

    public function primaryTenant(): ?Tenant
    {
        return $this->tenants()->wherePivot('type', PRIMARY)->first();
    }

// elsewhere in a command we were running:
                    $policy = Policy::where('carrier_policy_number', $policy_number)->first();
                    $tenant = $policy->primaryTenant()?->first();

Now obviously when these lines of code are jammed together it's pretty obvious now what the bug was.  But in full context of thousands of lines of code, it wasn't and this problem was caused during a refactor.   originally that primaryTenant method returned the entire pivot of PRIMARY ... and code everywhere called ->first() ... a refactor made us realize that was silly, since you only ever have a single primary tenant anyway, so we updated this method to return just the Tenant model itself.
But that one line of code off in a command that was run in a cron job ... was missed.  And was still calling ->first(), and suddenly 'everyone' became the first Tenant in our system.   (luckily it was only reading them in to do some logging, not changing any data).
The Proposal
I'd propose that if phpstan detects that you have an instantiated/hydrated Model class, that you throw an error if you see a call to any methods that are passthroughs to biulder, such as all, first, where, where*, any scopes, chunk, chunk*, lazy*, select, addSelect ... well the list goes on.
And only would allow direct calls to methods that are actually defined for that model, no passthrough magic methods.
Additional context
FWIW we are currently running on phpstan level:5 ... just in case the answer to all this is "we do that, on level 7" ;)

Hmm, I'm not sure about this---on one hand there's nothing wrong with mixing in the Builder methods on a Model instance, that's on the developer to decide and there could be a valid use-case for doing so; on the other hand it would be nice I suppose to prevent unwanted/accidental queries. In my app I tend to always start off my queries with Model::query() instead of using the static mixins
At best this would probably be an optional rule that could be opted into. However, I do want to point out that you can disable the mixin completely by creating a base Model in your application and overriding __call to prevent the mixin in the first place.

Hi,
I think this is very hard to detect from static analysis point of view.  And I think there might be other cases that would produce false positives.
All that being said, I think this is a good opportunity to write a custom rule for your app! You can read the documentation about how to write rules and let me know if you need any help 😊

Well I was going to say "before we close this lets chat" ... but that seems a touch late now ;) ... but here's some more chatting anyway @calebdw @canvural:
@calebdw wrote:

on one hand there's nothing wrong with mixing in the Builder methods on a Model instance, that's on the developer to decide and there could be a valid use-case for doing so

But what 'is' a valid use case?  I can't think of any valid use cases where you would want to call first() (or any of these) on any instantiated model as a valid use case.  It's sort of a weird/odd passthrough ... going from an instantiated model, to suddenly asking it to give you a completely new query against the database.   Instead of 'starting at the top' again.   I definitely understand the 'there could be' part of this, but I honestly can't think of any valid usecase where you wouldn't be better served by needing to be pedantic and starting back from a static query again against the class.   And static analysis is all about being pedantic :)
@calebdw wrote:

In my app I tend to always start off my queries with Model::query() instead of using the static mixins

Right, which would be perfectly fine.  In fact I could see an argument that any 'static' mixin would be fine, since it's obvious when you are calling a static method, that you aren't operating against the instantiated/hydrated data in the model itself.
@calebdw wrote:

At best this would probably be an optional rule that could be opted into.

That could be fine if so.  Optional is better than nothing.
@calebdw wrote:

However, I do want to point out that you can disable the mixin completely by creating a base Model in your application and overriding __call to prevent the mixin in the first place.

True, I could, but you could also do lots of things in the code to work around things like this.  But IMO that's kinda the point of using static analysis.  To look at the code, and point out potential bugs because of things you did that might be completely 'functional' but have a code smell to them.    This is somewhat like saying:  "Well if you don't like having your undefined variable treated as a null value, you should declare it with a value first".  Yup, you can totally do that, and, PHP is 'ok' with you not doing that.  But we have static analysis to point out "Ummm, you probably didn't mean to do that" :-D
@canvural wrote:

I think this is very hard to detect from static analysis point of view.

This I 100% can't speak to since I don't know any of the internals of how phpstan actually works/tokenizes things/etc, so will completely default to your POV there, and if this is not possible to do, so be it.
@canvural wrote:

If I understood you correctly this new rule would also warn you if you do something like (new SomeModel)->first() While it might be a useless code, it is totally fine and does not have the bug you mentioned.

You are correct it would capture this, because this should just be: SomeModel::first() instead.   My argument is that this isn't just useless code (which, again, static analysis picks up on.  A large part of the static analysis is pointing out useless code so you can clean it up) ... but that it's not 'fine' because it does still describe the bug.
Because you are taking an instantiated object, and calling a function against it to say "give me the first of this resultset" ... when it isn't a resultset.  it's a singular object.
    But instead it returns a completely different object to you.
It's that latter part that makes this really dangerous.   Since going from an instantiated object, to then making a call like first() against it, and assuming that you still have the same object, but having a completely different one ... is potentially major issue.  Especially if say, the model you were operating under was 'User' and you just pulled a completely different user's information.
It does feel to me that this is something that static analysis should be able to pick up on, whether an optional rule, or not, so that it points out the code smell of doing (new SomeModel)->first() instead of SomeModel::first()
@canvural wrote:

All that being said, I think this is a good opportunity to write a custom rule for your app! You can read the documentation about how to write rules and let me know if you need any help 😊

Thanks, but I'm a bit confused now.  You started out saying that this was 'very hard to do' ... and ended with "why don't you write a custom rule"?   So is it something truly possible, or not?
Side topic:  If I were able to get a rule created that did this, would Larastan be willing to accept a PR to address this, as an optional rule?   If I can get a rule working (if it's not impossible), I'd of course be happy to share it here and would hope it would help others.

Thanks, but I'm a bit confused now. You started out saying that this was 'very hard to do' ... and ended with "why don't you write a custom rule"? So is it something truly possible, or not?

The being hard part refers to not generate any false positives. Sorry I wasn't clear. Disallowing certain method calls on certain type of objects is easy to do with PHPStan. There is also a generic extension that does that: https://github.com/spaze/phpstan-disallowed-calls

Side topic: If I were able to get a rule created that did this, would Larastan be willing to accept a PR to address this, as an optional rule? If I can get a rule working (if it's not impossible), I'd of course be happy to share it here and would hope it would help others.

I just don't think this is a good fir for Larastan because it'd generate lots of false positives. I suggest you to start with a custom rule.


fix(chart): changed bar coordinates

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


No license,

Hey, thank you so much for contributing your code. Is it possible that you publish your  code under a specific license. Otherwise, it is unusable for someone who lives in a country with strict law. :D

Done


json print in script tag

<script data-metadata="f42dcf61ee024ceb8f718bcfbc81f212" type="application/json">                  { playerConfig(config) } </script>

Expected Output

<script
      data-metadata="f42dcf61ee024ceb8f718bcfbc81f212"
      type="application/json"
    >
      {
        "src": "https://storage.googleapis.com/shaka-demo-assets/angel-one/dash.mpd",
        "type": "application/dash+xml",
        "title": "Angel One",
        "poster": "https://storage.googleapis.com/shaka-demo-assets/angel-one/poster.jpg",
        "ads": {
          "src": "https://storage.googleapis.com/shaka-demo-assets/angel-one/ads.xml",
          "type": "application/xml"
        },
        "playerConfig": {
          "preferredAudioLanguage": "en",
          "preferredTextLanguage": "en",
          "primaryColor": "#991b1b"
        }
      }
    </script>

But I got this output
``

Answered in https://github.com/a-h/templ/issues/594

@joerdav Thanks


fix type: tags is an array

Currently the swagger options' type is expecting tags to be an object but in reality it is an array of objects.

Coverage remained the same at 98.797% when pulling 9de2de9ca9b2c502fed20c299a26e6d5f660bbcb on will-stone:patch-1 into 40085a953467610fe94e2357d5a8fbf0d19081c5 on glennjones:master.

Thanks! Released in v12.1.3

@robmcguinness thanks! 🥳


How to create a master-detail layout view from meteor-kitchen

Any links would be appreciated. Thanks
This is done using ionic :
http://blog.scottlogic.com/2015/10/02/ionic-master-detail.html
http://blog.scottlogic.com/wreilly/assets/ionic-master-detail/demo/#/0

@scheung38 It's possible, I'll write some examples here (later when I reach my computer).

Thanks maybe modifying the ToDos meteor example as a good starting point?

@scheung38 are you asking how to do it in "meteor kitchen" or this is question on how to do master-detail in "Meteor" itself ?

Sorry in meteor-kitchen

OK, just a moment, writing .json .... :)

OK, here is example based on "example-dataview". I added page "md" to show the principle - no css/formating
{
	"application": {

		"title": "Example application",

		"theme": "bootswatch-darkly",

		"use_collection2": false,

		"collections": [
			{
				"name": "customers",
				"fields": [
					{ "name": "name", "title": "Name", "required": true, "exportable": true },
					{ "name": "phone", "title": "Phone", "default": "-", "exportable": true },
					{ "name": "email", "title": "E-mail", "type": "email", "exportable": true },
					{ "name": "note", "title": "Note", "input": "textarea", "exportable": true },
					{ "name": "active", "title": "Active", "type": "bool", "input": "checkbox", "display_helper": "booleanToYesNo", "exportable": true }
				]
			},
			{
				"name": "items",
				"fields": [
					{ "name": "name", "title": "Item name", "required": true, "exportable": true },
					{ "name": "customerId" }
				]
			}
		],

		"queries": [
			{
				"name": "customers",
				"collection": "customers",
				"filter": {}
			},
			{
				"name": "customers_empty",
				"collection": "customers",
				"filter": {"_id": null},
				"find_one": true
			},
			{
				"name": "customer",
				"collection": "customers",
				"filter": { "_id": ":customerId" },
				"find_one": true
			},

			{
				"name": "customer_items",
				"collection": "items",
				"filter": { "customerId": ":customerId" }
			}
		],

		"free_zone": {

			"components": [
				{
					"name": "main_menu",
					"type": "menu",
					"items": [
						{ "title": "Home page", "route": "home" },
						{ "title": "Customers", "route": "customers" },
						{ "title": "Master-Detail", "route": "md" }
					]
				}
			],

			"pages": [
				{
					"name": "home",
					"title": "",
					"components": [
						{
							"name": "home_jumbotron",
							"title": "Example application",
							"type": "jumbotron",
							"text": "This Meteor application is made with meteor-kitchen without manual coding.
It demonstrates how to use DataView and Form components to create simple database application with view, search, insert, edit and delete functions.
Source code (input file for generator) is here.",
							"button_title": "Continue »",
							"button_route": "customers"
						}
					]
				},

				{
					"name": "md",
					"route_params": ["customerId?"],
					"title": "",
					"components": [
						{
							"name": "master",
							"type": "custom_component",
							"html": "<template name=\"TEMPLATE_NAME\">
Master

{{#each customers}}
{{name}} {{_id}}
{{/each}}
</template>",

							"query_name": "customers"
						},
						{
							"name": "detail",
							"type": "custom_component",
							"html": "<template name=\"TEMPLATE_NAME\">
Detail

{{#each customer_items}}
{{name}}
{{/each}}
</template>",

							"query_name": "customer_items",
							"query_params": [
								{
									"name": "customerId",
									"value": "this.params.customerId"
								}
							]
						}
					]
				},

				{
					"name": "customers",
					"components": [
						{
							"name": "view",
							"type": "data_view",
							"title": "Customers",
							"text_if_empty": "No customers :(",

							"query_name": "customers",
							"query_params": [],

							"insert_route": "customers.insert",
							"edit_route": "customers.edit",
							"edit_route_params": [
								{ "name": "customerId", "value": "this._id" }
							],
							"details_route": "customers.details",
							"details_route_params": [
								{ "name": "customerId", "value": "this._id" }
							]
						}
					],
					"pages": [
						{
							"name": "insert",
							"components": [
								{
									"name": "insert_form",
									"type": "form",
									"mode": "insert",
									"title": "New customer",

									"query_name": "customers_empty",
									"query_params": [],

									"submit_route": "customers",
									"cancel_route": "customers"
								}
							]
						},
						{
							"name": "details",
							"route_params": ["customerId"],
							"components": [
								{
									"name": "details_form",
									"type": "form",
									"mode": "read_only",
									"title": "Details",
									"layout": "horizontal",

									"query_name": "customer",
									"query_params": [],

									"close_route": "customers",
									"back_route": "customers"
								}
							]
						},

						{
							"name": "edit",
							"route_params": ["customerId"],
							"components": [
								{
									"name": "edit_form",
									"type": "form",
									"mode": "update",
									"title": "Edit customer",
									"submit_route": "customers",
									"cancel_route": "customers",

									"query_name": "customer",
									"query_params": []
								}
							]
						}
					]
				}
			]
		}
	}
}

Changes from "example-dataview" are in lines:
21-27 (added collection "items")
49-53 (added query "customer_items")
65 (added navbar menu item for "md" page)
86-112 (added page "md" with two custom components "master" and "detail")
Screenshot:

:+1:

Now, you can format it, add helper to highlight selected item from "master" etc.
Btw, there is also better way - "client" can be in the "md" pages subpage, and in that way, when user clicks for item in master, loading indicator will be shown only in "details" part of the page (not in entire page as it is currently shown).

(To add items into "details" type in your browser console:
Items.insert({ name: "aaaaaaa", customerId: "copy_and_paste_customer_id_here" });

One more thing, if you are confused with ? in "route_params": ["customerId?"], : question mark after route param tells iron-router that parameter is optional (without ? you'll get error on console - required parameter missing).


Refactor yolo example

Codecov Report

Merging #117 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #117   +/-   ##
=======================================
  Coverage   58.02%   58.02%
=======================================
  Files          20       20
  Lines        1601     1601
=======================================
  Hits          929      929
  Misses        672      672

Impacted Files
Coverage Δ

src/problemgenerator/filters.py
72.33% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


GroundPrimitive materials won't work if maximumVertexTextureImageUnits is 0

This is because all the data needed for computing texture coordinates gets packed into the batch table. We should add this to GroundPrimitive.supportsMaterials.

@bagnell recommends rewriting shaders to do the batch table reads in the fragment shader if maximumVertexTextureImageUnits is 0.

Primitives also use VTF for distance display conditions.

The original Batch Table code I wrote had a fallback to the fragment shader.  Can't we use that here?

There are 780 test failures is maximumVertexTextureImageUnits is 0.


vue3-sfc-loader.esm.js should expose the vue is has embedded

My code looks like this:
import * as Vue from '../js_ext/vue.js'
console.log(`vue.js version ${Vue.version}`)    <-----  3.3.4

import { loadModule, vueVersion } from '../js_ext/vue3-sfc-loader.esm.js'
console.log(`vu3-sfc-loader vue version ${vueVersion} `)   <-----  3.4.5

The issue is I have to import Vue twice.
vue3-sfc-loader.esm.js  has it's own internal Vue so it needs to expose that. the solution should allow for the following:
import { loadModule, vueVersion, Vue } from '../js_ext/vue3-sfc-loader.esm.js'
console.log(`vue.js version ${Vue.version}`)    <-----  3.4.5

I'm probably mistaken about this seem like only @vue/compiler-sfc is included in vue3-sfc-loader.esm.js  and doesn't include the runtime.  Please go ahead and close if that is the case.

lfmunoz, you are true, vue3-sfc-loader only embeds the vue compiler, and expose its version in vueVersion


onError triggered outside of websocket

Hello, when I get an error that has nothing to do with the websocket, it pass through the onError method....
I mean the onError method is triggered to display the errors...
(using latest websocket client with android studio)
Do you have an idea of what is causing this strange thing ?

Can you post the specific errors that happen in onError()?
This callback is mostly used for internal errors such as IOException from sockets, thread interrupts, etc.

hello, here is the last error i got : Only the original thread that created a view hierarchy can touch its views. (i was changing the visibility of a view) so I fixed it using "runOnUiThread"... nothing to do with the websocket...
I don't understand why the errors pass by the websocket onError method...

Yes, this is normal behavior when exception is thrown within a callback like onMessage, etc. Otherwise there would be no way to let the user handle such errors without crashing the entire server/client.

Yes, this is normal behavior when exception is thrown within a callback like onMessage, etc. Otherwise there would be no way to let the user handle such errors without crashing the entire server/client.

Oh ok i thank you for explaining me.


just insert one paran if there are any params

@kasperpeulen, if a method is being inserted and there are any params, just append ( instead of (). I think () tells the user that there's no args to the method. We can adjust this behavior to insert placeholder args and back the cursor up, once we have better control over the completion UI.

Seems like a good fix for now. I think to adjust the cursor position, we need to do something with the CodeMirror signal/event "pick":
    pick: function(data, i) {
      var completion = data.list[i];
      if (completion.hint) completion.hint(this.cm, data, completion);
      else this.cm.replaceRange(getText(completion), completion.from || data.from,
                                completion.to || data.to, "complete");
      CodeMirror.signal(data, "pick", completion);
      this.close();
    },

this is from codemirror.js


[dsAudio] Questions on dsAudio interfaces

In file dsAudio.h
*dsError_t  dsGetAudioEncoding(intptr_t handle, dsAudioEncoding_t encoding);
Does it also support for Digital output format?
Seems duplicate with dsGetStereoMode()

This API is not used by Middleware to take any actions. For the time being implementation can be stubbed

In file dsAudio.h *dsError_t dsGetAudioEncoding(intptr_t handle, dsAudioEncoding_t encoding); Does it also support for Digital output format? Seems duplicate with dsGetStereoMode()

This API is not used by Middleware to take any actions. For the time being implementation can be stubbed

No change is required. It is a clarification issue.


Monitor model changed to add an optional fanoutEnabled field

Description
Detectors configured with aggregation sigma rules should ideally generate only one alert in one execution when a set of documents match the rule’s condition. But currently there are duplicate alerts being generated when aggregate sigma rules are matched.
This PR adds a field in monitor  fanoutEnabled (optional field), default value of this flag will be true. When detector is created/updated, the value of this field will be set to false in case chained findings doc level monitor, this will ensure that fanout approach will be disabled when executing doc level monitor
Related Issues
Resolves #[Issue number to be closed when this PR is merged]

Check List

[ ] New functionality includes testing.
[ ] New functionality has been documented.
[ ] API changes companion pull request created.
[ ] Commits are signed per the DCO using --signoff.
[ ] Public documentation issue/PR created.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

UTs will add as a follow up


Setup BigQuery billing to user's Google account

BigQuery studio has moved into the Google cloud console (compared to when it first launched). That means that you have to setup Google cloud projects and billing. I wonder if there's an easy way to do that programmatically via API.
Otherwise, we may need to walk users through how to setup for the first time

Since we are moving to Clickhouse for user-facing queries, going to close this in favor of the billing epic:
https://github.com/opensource-observer/oso/milestone/31


refactor(side-drawer): migrate to fast element

But when the side-drawer is closed you don't see the button to open it again

why dont we use fab to toggle drawer?
https://vivid.vonage.com/?path=/story/alpha-components-fab--mini&args=mini:false;icon:menu-solid
or just the local button with menu-solid icon

I'll use the button and as soon as the fab will be in vivid-3 we can update it

https://github.com/Vonage/vivid/issues/1199
I named the dispatch events side-drawer-closed and side-drawer-opened.
Do you have a preference for another naming?
@yinonov @YonatanKra @rachelbt


need restapi to get user task belong to specific user or group

Description
Is there anyway to filter user task belong to specific actor or group using openapi? Or, I can get group, actor of the task.
I notice that the only way to filter that using graphql/dataindex. I wish to avoid that due to:

it added complexity of infrastructure
i wish my team using generated openapi client instead of code manually

Implementation ideas
I propose:
create api filtered by actor or group
get /{bpmn-flow}/task-byactor/{actorname}
get /{bpmn-flow}/task-bygroup/{groupname}
result will be array of usertask: [{usertaskid:xx1 ,  data: {...}}, {usertaskid:xx2, data:{...}}]

is this not a simple api to get from UserTaskInstance Collection if using MongoDB or any other persistence?
potentialGroups/adminGroups/potentialUsers/ActualOwner etc.

Oh cause official and stable way definitely go through rest api. My plan is We use openapi-generator to generate the typescript client code.

As indicated by @debu999 , Kogito is not going to define specific rest api for any filter combination. The whole idea behind data index is againts that approach.


WebGL renderer has issues with font rendering

Describe the bug
Text on WebGL is not rendered correctly: it's blurred and not cropped correctly
Canvas:

WebGL:

To Reproduce
Create any text with stroke:
return new Style({
    text: new Text({
        font: 'bold 16px Helvetica, sans-serif',
        text: feature.get('jpt_nazwa_'),
        zIndex: 10000,
        stroke: new Stroke({
            color: '#fff',
            width: 2
        })
    }),
    stroke: new Stroke({
        color: '#fff',
        width: 2
    })
})

Expected behavior
Text should look the same as with canvas renderer.

That WebGL renderer has been removed since v5.x.

That WebGL renderer has been removed since v5.x.

I used WebGLMap which is still in 5.x and it is NOT marked as deprecated.
If it will be removed, then how to deal with huge number of points to render? Default renderer gives me poor framerate of 4fps for SVG, while WebGL gives me about 40 fps.
Fiddle with comparison: https://jsfiddle.net/s9pLk4cj/

For points, we have a new WebGL point layer renderer. See the examples in https://openlayers.org/en/master/examples/


added room to onConnected in README.md

Changes

added room parameter to onConnected function in the readme

Reason

Copy pasting the boilerplate from the readme threw an error, room does not exist. Adding room as a parameter for onConnected rectified this.

 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.


[DOCS] PPOTrainer references are missing in the API docs

Hi @younesbelkada,
The API reference does not contain a concise example on using the PPOTrainer.
Perhaps, something like this would already suffice.
But perhaps you prefer the usage displayed in this image?

Hi @davidberenstein1957, the example should just give a concise taste on what the trainers looks like in TRL, thus each example is not a executable code (e.g. dataset loading is missing in everyone). What do you feel is missing exactly?

@lvwerra thanks for the response and great package.
this but for the PPOTrainer, which I think would mean creating a more complete overview as a docs/source/ppo_trainer.mdx.

I see, yes that could nice and a bit more systematic. We already have a lot of info in customization.mdx but we could indeed move it to a ppo_trainer.mdx and add it to the same subsection in the ToC. If you want to take a stab at it I'll be happy to review it.

@lvwerra I have started some efforts and will be able to create a draft PR this coming weekend.

Also, might it more comprehensive to move some of the PPO-logic into the PPOConfig?
Having something like what is shown underneath my unify the API usage a bit more.
from transformers import pipeline
from trl import PPOConfig, PPOTrainer

config = PPOConfig(
    *args,
    **kwargs,
    reward_model: pipeline,
    generation_args: dict
)
trainer = PPOTrainer(
    config=config,
    model,
    tokenizer
)
trainer.train()

Alternatively, we might pass a Callable reward_function to the config?
from transformers import pipeline
from trl import PPOConfig, PPOTrainer

def reward_func(examples):
    return rewards

config = PPOConfig(
    *args,
    **kwargs,
    reward_func: Callable,
)
trainer = PPOTrainer(
    config=config,
    model,
    tokenizer
)
trainer.train()

Indeed, we have been contemplating this, but the evolving logic inside the generation/reward/optimization loop was the main reason we haven't settled, yet. Would you mind opening an issue for that and we can see if there's some community traction for it.


tensorflow 1.2 AttributeError: module 'tensorflow.python.ops.nn' has no attribute 'leaky_relu'

AttributeError: module 'tensorflow.python.ops.nn' has no attribute 'leaky_relu'

I have used Tensorflow 1.6 to develop this. I will update the Readme file. To resolve this issue for earlier versions, you can create a leaky relu function on your own.

Thank you very much!


[Documentation] Zoom Admin for MPWG Assigned Admins

I'll work on a document for securing the Zoom calls, assigning host and co-hosts, etc.

You are amazing, Debbie!
We can test those guidelines with the Context Propagation Team.   I will send the 2 Admins the code today. They have not scheduled the call yet, so perfect timing.
https://github.com/eclipse/microprofile-marketing/issues/420

Document updated with images.

Are we expecting any review comments on this? Looks good to me. Let me know if this to okay to close.
@aeiras @jclingan

Neha,
Thanks for pulling back in. There are a few updates I need to add to the document now that John has inserted the images.
You and Debbie can expect completion latest by next Monday. On that day, you can help Debbie submit the PR under the MPWG pages  https://github.com/microprofile/microprofile-wg/wiki.

Thanks @aeiras for the updates.

Hola @debbiehoffman,
I have added a few things in the document. John did a fantastic job with the screen shots. :)
Can you please review and pull me in if there are any questions on latest additions? Thank you!
With John, we navigated onboarding Context Propagation and learned quite a bit by doing.
If you cannot look into this, do delegate the review to Neha. :) Neha volunteered to help you with the PR submission so the 2 of you are a team!
Thank you both,

@aeiras  Hi! Can you please update this with the info about the PR as discussed in marketing call today? Thank you!

Hola Debbie,
Under https://github.com/eclipse/microprofile/blob/master/CONTRIBUTING.adoc  "Create or fix an issue..."
you find the do a pull request
Have fun!

Hi @aeiras ! Talked with @nsardana-bny today and we agree that the Zoom doc really should be merged with the Mkt- Call Steward Doc (https://docs.google.com/document/d/1ayqaFoXGeTZdneL6U9oekaZP3LPe7PapTlg7jh7eD3M/edit#) as several things mentioned in the original Zoom doc aren't actually steps that we need to do anymore.
We also talked about doing the pull request, and I need to reach out to Cesar H because Neha said we can't actually change the Wiki directly using the instructions provided (she tried with something else previously). So will try and set up a call w/ him for that but in the meantime will work on merging some of the details from the Zoom Admin doc into the Mkt Call Steward document to make being this "Steward" less confusing and easy for anyone to follow.

Sounds fantastic to simplify the documentation, Debbie & Neha!
I think @cesarhernandezgt is going to be glad to be pulled, via the git ticket he created, to unblocking any PR work.
https://github.com/microprofile/microprofile-wg/issues/92

@jclingan,
Can you please help Debbie with the images in the wiki https://github.com/microprofile/microprofile-wg/wiki/MicroProfile-Zoom-Admin?   You volunteered to help her with that step, therefore, thank you, John.
Let's aim at closing this ticket before Nov is done.

completed, you are wonderful, Debbie.
Thank you @jclingan for the extra care importing the images!