Did spotify drop mdns support for spotify connect?

Hi,
I've been trying for days now to use the self discovery of librespot on my local network (not specifying my user account in the configuration).
Librespot implements the local discovery via mDNS protocol. However, using wireshark on my windows PC running official spotify client, I see no traffic on port 5353 (except discovery of proxy viq WPAD, which can be disabled in spotify advanced settings).
Instead, I can see SSDP traffic, with M-SEARCH query.
I can't find anything confirming this on the web.
Does anyone here encounter the same issue, or is able to get some network trace with official spotify connect device to confirm it?
This would mean the whole mDNS part of librespot could be dropped, and should be replaced with SSDP equivalent...
Thanks,
Aurelien

Which version of the Spotify client are you running?
On Mon, Dec 10, 2018, 21:12 agrenott<EMAIL_ADDRESS>wrote:

Hi,
I've been trying for days now to use the self discovery of librespot on my
local network (not specifying my user account in the configuration).
Librespot implements the local discovery via mDNS protocol. However, using
wireshark on my windows PC running official spotify client, I see no
traffic on port 5353 (except discovery of proxy viq WPAD, which can be
disabled in spotify advanced settings).
Instead, I can see SSDP traffic, with M-SEARCH query.
I can't find anything confirming this on the web.
Does anyone here encounter the same issue, or is able to get some network
trace with official spotify connect device to confirm it?
This would mean the whole mDNS part of librespot could be dropped, and
should be replaced with SSDP equivalent...
Thanks,
Aurelien
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/librespot-org/librespot/issues/281, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AOI-8aP9FbVAL3R4XwG0MmdHB7398VV6ks5u3sBJgaJpZM4ZMGDl
.

On windows: <IP_ADDRESS>9.g342899da.
And same with android client <IP_ADDRESS>5.

I've managed to updated the Android client and it doesn't detect librespot-java anymore, I am waiting for the Windows update.

Have you tried building librespot with with-dns-sd, see https://github.com/librespot-org/librespot/wiki/Compiling

Yes, but I'm still stuck on a libsystemd.so not found at link time (while it's there and I've added the path with ld -L argument) - using raspotify cross-compilation docker image tweaked a bit.
But anyway, as I can't even see any mDNS packet going out of the official client, I don't think it would make any difference. Or are you saying avahi-daemon would handle SSDP requests as well for librespot?

<IP_ADDRESS>9.g342899da is also fine on Windows.

What do you mean by "fine"? Do you see mDNS queries sent from the windows client? Are you able to use spotify connect without specifying user/password when starting librespot?

I've not checked the traffic between the clients, but Zeroconf works fine with librespot-java.
If you see some SSDP, maybe they are slowly switching to it, I'll check.

@agrenott There are a lot of SSDP packets being sent on my network, but that's normal. I have not seen anything related to Spotify, could you point out which type of service (NT header in the NOTIFY request) is used?

Here is the SSDP query from spotify client on windows (sent each time you display the list of available devices):
0000   01 00 5e 7f ff fa 70 85 c2 43 63 26 08 00 45 00   ..^.ÿúp.ÂCc&..E.
0010   00 99 26 4d 00 00 01 11 00 00 c0 a8 00 03 ef ff   ..&M......À¨..ïÿ
0020   ff fa df f0 07 6c 00 85 39 58 4d 2d 53 45 41 52   ÿúßð.l..9XM-SEAR
0030   43 48 20 2a 20 48 54 54 50 2f 31 2e 31 0d 0a 48   CH * HTTP/1.1..H
0040   4f 53 54 3a 20 32 33 39 2e 32 35 35 2e 32 35 35   OST: 239.255.255
0050   2e 32 35 30 3a 31 39 30 30 0d 0a 4d 41 4e 3a 20   .250:1900..MAN:
0060   22 73 73 64 70 3a 64 69 73 63 6f 76 65 72 22 0d   "ssdp:discover".
0070   0a 4d 58 3a 20 31 0d 0a 53 54 3a 20 75 72 6e 3a   .MX: 1..ST: urn:
0080   64 69 61 6c 2d 6d 75 6c 74 69 73 63 72 65 65 6e   dial-multiscreen
0090   2d 6f 72 67 3a 73 65 72 76 69 63 65 3a 64 69 61   -org:service:dia
00a0   6c 3a 31 0d 0a 0d 0a                              l:1....

Or a bit more readable view (from wireshark):
Internet Protocol Version 4, Src: <IP_ADDRESS>, Dst: <IP_ADDRESS>
User Datagram Protocol, Src Port: 57328, Dst Port: 1900
Simple Service Discovery Protocol
    M-SEARCH * HTTP/1.1\r\n
        [Expert Info (Chat/Sequence): M-SEARCH * HTTP/1.1\r\n]
        Request Method: M-SEARCH
        Request URI: *
        Request Version: HTTP/1.1
    HOST: <IP_ADDRESS>:1900\r\n
    MAN: "ssdp:discover"\r\n
    MX: 1\r\n
    ST: urn:dial-multiscreen-org:service:dial:1\r\n
    \r\n
    [Full request URI: http://<IP_ADDRESS>:1900*]
    [HTTP request 2/2]
    [Prev request in frame: 40]

And still, I'm almost sure those come from spotify client, as I capture them only when it's started. And the string "ST: urn:dial-multiscreen-org:service:dial:1" appears inside the binary (looking with process explorer).

Arg, got it. Looks like my spotify client correctly sends the mDNS requests, but on an invalid network interface, so it never reaches the "real" network...

That is indeed true, but I cannot force Spotify to use SSDP between two clients. This means that they are using both mDNS and SSDP. I'll try with my smart TV.

Those SSDP requests are likely to be from the Cast SDK that is integrated into the Spotify clients. Whilst I can't be certain, I would say that nothing has changed w/ regards to the mDNS discovery.

I guess we can close the issue, as mDNS support is not dropped, just mis-behaving when several network interfaces are available (in my case using Hyper-V internal interface).
BTW, seems the android spotify client isn't using mDNS (or has similar issue than the windows one).

I had this issue, and the solution in my case was unrelated to librespot. It turns out that Spotify has a bug in their Android app that unintentionally interferes with ad blockers, as discussed in this AdAway issue.
The solution is a fork of AdAway that works around the issue, found here


Implement the pushOnly flag of the pushpull API

What this PR does / why we need it:

Extent PushPull API by adding a pushOnly flag to support Client Sync mode. (#482)
If pushOnly is true, Server makes a push-pull response with empty changes unless data has been changed.
If a push-pull response has changes that have already been performed by local changes on the client side. Send empty operations to prevent redoing those operations.

Due to that modifying operations, Deep coping of chageInfo is needed.

Which issue(s) this PR fixes:

Fixes #482 (Server side only)
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Additional documentation:

Checklist:

[x] Added relevant tests or not required
[x] Didn't break anything

 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.


Single-axis scaling + look_at changes scale on all components [regression]

Godot 3.2 alpha1
I launched one of my projects with 3.2 recently and found out many 3D models I scale at runtime were much larger than usual.
I made a small project showing that:
ScaleBroken.zip
In Godot 3.1.1, this project shows a long cube increase and decrease in Z-length while it changes orientation using look_at. This is expected because only the Z scale is modified.
In Godot 3.2, the cube now scales uniformly, which makes it look too thick or too thin, which is not expected.

Likely due to #26897 (and fixup #29125), CC @rodolforg.


Batch mode: No Grobid home was found in classpath and no Grobid home location was not provided

Version: 0.5.0
System: Ubuntu 16.04 LTS
Problem:
Running grobid in batch mode supplying a grobid home path fails with the following output:
Nov 15, 2017 12:10:38 PM org.grobid.core.main.GrobidHomeFinder getGrobidHomePathOrLoadFromClasspath
WARNING: No Grobid property was provided. Attempting to find Grobid home in the current directory...
Nov 15, 2017 12:10:38 PM org.grobid.core.main.GrobidHomeFinder getGrobidHomePathOrLoadFromClasspath
WARNING: Attempting to find and in the classpath...
Exception in thread "main" org.grobid.core.exceptions.GrobidPropertyException: [GENERAL] No Grobid home was found in classpath and no Grobid home location was not provided

The corresponding batch mode call:
$ java -Xmx1G -jar ./grobid-0.5.0/grobid-core/build/libs/grobid-core-0.5.0-onejar.jar -gH /home/ubuntu/uni.mp/project/backend/webservices/grobid-0.5.0/grobid-home/ -dIn ./pdfs/ -dOut ./pdfs/ -r -exe processHeader

If run from the grobid directory, it works, but specifying the -gH parameter isn't necessary:
$ java -Xmx1G -jar ./grobid-core/build/libs/grobid-core-0.5.0-onejar.jar -gH /home/ubuntu/uni.mp/project/backend/webservices/grobid-0.5.0/grobid-home/ -dIn ./../pdfs/ -dOut ./../pdfs/ -r -exe processHeader

$ java -Xmx1G -jar ./grobid-core/build/libs/grobid-core-0.5.0-onejar.jar -dIn ./../pdfs/ -dOut ./../pdfs/ -r -exe processHeader

Possible reason:
It seems that GrobidHomeFinder only checks env variable but not commandline argument -gH ?
Workaround:
Create a symbolic link to grobid home in the directory grobid is called from:
ln -s ./grobid-0.5.0/grobid-home ./grobid-home

Hi @philonor, thanks for the report. Indeed there is a problem because the grobid home provided is not used to configure the GrobidHomeFinder, which is looking on ../grobid-home as default location.

The issue should be fixed in my last commit, feel free to close the issue or comment back in case of problems

Dear @philonor did you try the fix I've done? Does it works?

Sorry for my late reply, just checked your fix and it works now as expected. Thank you!
I used master @ 593c3b8525f5ee013de527ff74142ab5eb7ceeb6, 3 commits after your specific commit.

hello. same problem happens to me

hello. same problem happens to me


Change Log for specific menus

A change log for specific menus such as sales order menu, purchase order menu, item master menu.    Scope would remain in high traffic menus that have potential for multi user editing.

Added proposed schema to db structure.   See below.   Proposed method would be to grab the returned data structure from getRecord() method call and compare this structure with same data structure used during updateRecord.   Any deviation would result in entries to change_log per id/key.   Changes would be cached in ArrayList and inserted into change_log 'after' updateRecord method is complete.
CREATE TABLE IF NOT EXISTS "change_log" (
"chg_id" INTEGER PRIMARY KEY,
"chg_key" varchar(30) NOT NULL COLLATE NOCASE DEFAULT '',
"chg_table" varchar(30) NOT NULL COLLATE NOCASE DEFAULT '',
"chg_class" varchar(30) NOT NULL COLLATE NOCASE DEFAULT '',
"chg_userid" varchar(8) NOT NULL COLLATE NOCASE DEFAULT '',
"chg_desc" varchar(100) NOT NULL COLLATE NOCASE DEFAULT '',
"chg_ts" timestamp NOT NULL DEFAULT (datetime('now','localtime')),
"chg_type" varchar(30) NOT NULL COLLATE NOCASE DEFAULT '',
"chg_ref" varchar(30) NOT NULL COLLATE NOCASE DEFAULT ''
);

new change log method signature is:
public static  ArrayList<change_log> logChange(String key, String callclass, T x, T y)

prototype mechanism is in place for TermsMaint menu.   Includes pop-up dialog showing change log.

satus update...as of this post date...only a minor few high profile menus have change control logging included.

closing this until further interest in other menus for change log incorporation


travis build failing

This is very suspicious. Until now everything was working fine. Today I bumped faker.js 2.1.3 -> 2.1.4 and suddenly, travis build crashed. So I reverted the commit (which retriggered the travis build) and it failed again. I got no idea on what's going on, but in https://travis-ci.org/pateketrueke/json-schema-faker/builds/65619706 I can see that there are some methods missing:

TypeError: Object [object Object] has no method 'toHaveSchema'
TypeError: Object [object Object] has no method 'toHaveType'

@pateketrueke do you have any idea what can this be connected with?

It seems like jasmine-node is no longer requiring the spec helpers.coffee, I've added it manually and works again.
# main-spec.coffee
require('./helpers')

fs = require('fs')
glob = require('glob')
jsfaker = require('../lib')

# ...

  I ask because I'm working on a PR and I don't want to include that fix if I don't need to, but I do need it to run the test suite.

I've commited to master, please rebase again.

Thanks a lot.

  Any reason you didn't commit there instead?

You're right, I've published to develop too, was my fault. :beers:

Thanks.  :)

Not sure but I think this is solved.


[doc] Getting Started documentation and tutorial

This is proposal for a getting started documentation and step-by-step tutorial.
This complements and replaces the original PR made by @fabianfett (https://github.com/swift-server/swift-aws-lambda-runtime/pull/261)
Motivation:
Provide a step-by-step tutorials to help new comers to get started with the Swift AWS Lambda Runtime
Modifications:

create a top level documentation page for package AWSLambdaRuntimeCore. It contains a brief introduction to the project copied from the main README file) and links to other parts
a getting started guide for the impatient, with just top-level instructions
a step-by-step tutorial based on Swift Docs @Tutorial format

Result:
There is a getting started documentation !
You can see a preview at this ephemeral address : https://staging.stormacq.com/lambda-runtime-doc/documentation/awslambdaruntimecore/

cc @bitjammer

cc @Joannis

this is awesome!

@swift-server-bot add to allowlist

@tomerd I am on PTO - hence the delay to answer questions. Yes, it is good for me and ready to publish

@sebsto no blocker, was confirming this was ready. looks like one CI job is not happy, but lets marge once green

@swift-server-bot test this please

I was inquiring about the failed CI Job :-) Looks like the soundness.sh script fails because swiftformat has not been applied.
But when I run swiftformat it updates many files that are not part of this PR ?
➜  swift-aws-lambda-runtime git:(sebsto/getting-started-doc) ✗ git status --porcelain
 M Examples/Foundation/Lambda.swift
 M Examples/LocalDebugging/MyApp/MyApp/ContentView.swift
 M Sources/AWSLambdaRuntimeCore/ControlPlaneRequest.swift
 M Sources/AWSLambdaRuntimeCore/Documentation.docc/Resources/code/03-02-01-package.swift
 M Sources/AWSLambdaRuntimeCore/Documentation.docc/Resources/code/03-02-02-package.swift
 M Sources/AWSLambdaRuntimeCore/Documentation.docc/Resources/code/03-02-04-package.swift
 M Sources/AWSLambdaRuntimeCore/Documentation.docc/Resources/code/03-02-05-package.swift
 M Sources/AWSLambdaRuntimeCore/Documentation.docc/Resources/code/03-03-01-main.swift
 M Sources/AWSLambdaRuntimeCore/Documentation.docc/Resources/code/03-03-02-main.swift
 M Sources/AWSLambdaRuntimeCore/Documentation.docc/Resources/code/03-03-03-main.swift
 M Sources/AWSLambdaRuntimeCore/Documentation.docc/Resources/code/03-03-04-main.swift
 M Sources/AWSLambdaRuntimeCore/Documentation.docc/Resources/code/03-03-05-main.swift
 M Sources/AWSLambdaRuntimeCore/Documentation.docc/Resources/code/03-03-06-main.swift
 M Sources/AWSLambdaRuntimeCore/HTTPClient.swift
 M Sources/AWSLambdaRuntimeCore/Lambda+LocalServer.swift
 M Sources/AWSLambdaRuntimeCore/Lambda.swift
 M Sources/AWSLambdaRuntimeCore/LambdaConfiguration.swift
 M Sources/AWSLambdaRuntimeCore/LambdaHandler.swift
 M Sources/AWSLambdaRuntimeCore/LambdaRequestID.swift
 M Sources/AWSLambdaRuntimeCore/LambdaRunner.swift
 M Sources/AWSLambdaRuntimeCore/LambdaRuntime.swift
 M Sources/AWSLambdaRuntimeCore/LambdaRuntimeClient.swift
 M Sources/AWSLambdaRuntimeCore/Terminator.swift
 M Sources/AWSLambdaRuntimeCore/Utils.swift
 M Sources/MockServer/main.swift
 M Tests/AWSLambdaRuntimeCoreTests/MockLambdaServer.swift
 M Tests/AWSLambdaRuntimeCoreTests/Utils.swift

Should I commit those as well ?

try to run in docker install of installing swift-format locally, as you may be using a different version than the one we use
docker-compose -f docker/docker-compose.yaml -f docker/docker-compose.al2.58.yaml run soundness

@tomerd thank you swiftformat is now OK.
But soundness fails because the code snippets in the tutorial part do not have the license header file.
I think adding these to these code snippets will reduce the lisibility of the tutorial.  soundness script makes no difference between source code file and code snippets from the doc, they all are *.swift
Can I modify soudness to skip files in the documentation directories ?

I think adding these to these code snippets will reduce the lisibility of the tutorial.

where would they be visible?

I think adding these to these code snippets will reduce the lisibility of the tutorial.

where would they be visible?

These are visible on the tutorial web page.
The tutorial shows code incrementally, step by step, as reader scrolls down.  Each step is a separate .swift file.
See screenshot attached.

Also, the core runtime does not compile on Swift 5.9 or Swift nightly.  This should be fixed on the main branch.
15:48:54 /code/Sources/AWSLambdaRuntimeCore/LambdaContext.swift:15:17: remark: '@preconcurrency' attribute on module 'Dispatch' is unused
15:48:54 @preconcurrency import Dispatch
15:48:54 ~~~~~~~~~~~~~~~~^
15:48:54 [282/295] Compiling AWSLambdaRuntimeCore LambdaContext.swift
15:48:54 /code/Sources/AWSLambdaRuntimeCore/LambdaContext.swift:15:17: remark: '@preconcurrency' attribute on module 'Dispatch' is unused
15:48:54 @preconcurrency import Dispatch
15:48:54 ~~~~~~~~~~~~~~~~^
15:48:54 [283/295] Compiling AWSLambdaRuntimeCore Lambda+String.swift
15:48:54 /code/Sources/AWSLambdaRuntimeCore/Lambda.swift:88:23: error: 'install()' is deprecated: This is no longer needed in Swift 5.9
15:48:54             Backtrace.install()
15:48:54                       ^
15:48:54 [284/295] Compiling AWSLambdaRuntimeCore Lambda.swift
15:48:54 /code/Sources/AWSLambdaRuntimeCore/Lambda.swift:88:23: error: 'install()' is deprecated: This is no longer needed in Swift 5.9
15:48:54             Backtrace.install()
15:48:54                       ^
15:48:54 error: fatalError
15:48:54 error: fatalError

These are visible on the tutorial web page.

right, but do license header show up as well? is there no way to filter them from DocC?

Also, the core runtime does not compile on Swift 5.9 or Swift nightly. This should be fixed on the main branch.

yes, that should be done in a separate PR

Also, the core runtime does not compile on Swift 5.9 or Swift nightly. This should be fixed on the main branch.

yes, that should be done in a separate PR

I just opened https://github.com/swift-server/swift-aws-lambda-runtime/issues/304

These are visible on the tutorial web page.

right, but do license header show up as well? is there no way to filter them from DocC?
cc @bitjammer

The code visible in the tutorial is assembled from individual *.swift file, one file for each code snippet.
I choose to not include the license header file in these *.swift file to keep the code snippets visible on the tutorial web page clean.
Problem : soundness script checks for the presence of the header in all swift files (find . -name '*.swift' -o -name '*.c' -o -name '*.h')
Proposed solution : add an exception for *.swift files inside *.docc folders
git diff scripts/soundness.sh
diff --git a/scripts/soundness.sh b/scripts/soundness.sh
index bd321f6..353359d 100755
--- a/scripts/soundness.sh
+++ b/scripts/soundness.sh
@@ -130,6 +130,7 @@ EOF
     find . \
       \( \! -path '*/.build/*' -a \
       \( \! -path '*/.git/*' \) -a \
+      \( \! -path '*/Documentation.docc/*' \) -a \
       \( "${matching_files[@]}" \) -a \
       \( \! \( "${exceptions[@]}" \) \) \) | while read line; do
       if [[ "$(cat "$line" | replace_acceptable_years | head -n $expected_lines | shasum)" != "$expected_sha" ]]; then

Proposed solution : add an exception for *.swift files inside *.docc folders

I think that would be fine if we cannot find a way to tell DocC to ignore the license headers

5.9 wanring fix: https://github.com/swift-server/swift-aws-lambda-runtime/pull/305

I don't believe there is a way to hide regions of code in tutorials. They use a different feature from "snippets", which do have different mechanisms for hiding regions of code.

Will be able to merge when https://github.com/swift-server/swift-aws-lambda-runtime/pull/307 is approved

@swift-server-bot test this please

@tomerd thank you for merging https://github.com/swift-server/swift-aws-lambda-runtime/pull/307, this PR is now ready to merge.


Add supported formats in description for RawImage and CompressedImage

Updated in website docs here: https://foxglove.dev/docs/studio/panels/image

Another customer asked this question today. I think we should focus on adding this to the schema docs.
I understand that in a pure theoretical world the schemas are separate from the panels that interpret them, but in the real world the only place these schemas are used is in our app, and the schema docs are the place you are looking when you try to figure out what to put in each field.
cc @esthersweon @defunctzombie

There are still two ways we could implement this:

We could add the information to the schema itself (e.g. as a description on the format field) and it would appear in the generated code comments/docs.
We could keep the information in the website repo, and insert/append it when we generate the docs for the relevant schemas. (The same information could be auto-inserted on the 3D panel page as well as the schema pages.)

There is a third option: you could add a new field to the source definition in this repo, but only use that data in the docs generation and not schema generation.
I don't feel strongly either way. Adding it to the website docs is the highest priority.
I can see arguments for and against also adding it to the schema comments:

Pro: Easy for people to access while they are reading the source (especially if they copy the .proto into their own repo)
Con: Might get out of date if/when we add new formats (especially if they copy the .proto into their own repo)


Noninclusive tests after migrating to VS 2017

Coverage is still being reported correct however AppVeyor shows that some unit tests are non-inclusive as does running the tests via the resharper test runner in visual studio.
All tests are run using the visual studio test runner and local builds?

There seems to be an issue with the ReSharper test runner, ticket logged with JetBrains.
Otherwise all tests run, AppVeyor also shows inconclusive test runs however all tests run as show in the published .trx files on AppVeyor and Code Coverage on coveralls.io

Resharper latest EAP fixes the test explorer issue


will the managed identity works for Networking type "Kubenet"

[Enter feedback here]

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

ID: 6a9c74a8-9f1a-3d3d-3aed-f8ec45f92e05
Version Independent ID: 01e98043-dc80-490e-a761-8d558ff4e6df
Content: Use managed identities in Azure Kubernetes Service - Azure Kubernetes Service
Content Source: articles/aks/use-managed-identity.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: mlearned

Thanks for the question! We will review and get back to you shortly.

@AnandKatta Yes managed identity works for Networking type "Kubenet".
You can refer to this for more info.
Hope it helps!!!

@AnandKatta
Hope the provided information is helpful.
We will now close this issue. If there are further questions regarding this, please tag me in a comment. I will reopen it and we will continue the discussion.


OCM-4571 | fix: Fix CheckAndParseVersion when coming from preRelease

Fixing another helper for version validation:
https://issues.redhat.com/browse/OCM-4571?focusedId=23365966&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-23365966

lgtm


Changed default claim type from one-path to all-path

Fixes https://github.com/kframework/kore/issues/1638

It looks like this causes the output of max-broken-spec.k to change trivially. That test is pretty unstable, it seems to break whenever you look at it. Anyway, it should be fine to update the output, but you should double-check.

Maybe we should figure out what is crucial that we want to test about max-broken-spec.k (making sure we don't just return true for everything?) and maybe write a custom test for it? @dwightguth @ttuegel what do you think?


Docker setup

Is your feature request related to a problem? Please describe.
I'd like to set this project up in a docker contianer.
Describe the solution you'd like
I am no web developer and that's why I am asking the creator here, if he could add a Dockerfile and gave us instructions on how to set the application up in a simple docker image and then run it in a container.

Hi @bojtalepenye, I recommend checking out this beginner-friendly Medium article that provides a step-by-step guide on how to dockerize a Next.js app. It should help you get started with setting up the project in a Docker container.


Whispers with more than 1 space dont get send but it shows as if it would

Whispering someone using "/w   " shows as sent when it is not getting sent.

Fixed in nightly


feat(gotrue): add WhatsApp support for OTP login

What kind of change does this PR introduce?
Adds WhatsApp support for OTP login
JS implementation are the following:

https://github.com/supabase/gotrue-js/pull/616
https://github.com/supabase/gotrue-js/pull/647

Isn't the implementation for signUp() missing? And what about resend()? js doesn't have it implemented, but is it because the server doesn't support it, or they forgot it?

Isn't the implementation for signUp() missing?

Done.

js doesn't have it implemented, but is it because the server doesn't support it, or they forgot it?


Unknown error: 'NoneType' object has no attribute 'getroottree'

I get this error with any version (including zeep) and with any python version. What am I doing wrong? Is it a problem with the wsdl? Thanks. Full stacktrace:
Traceback (most recent call last):
File "/home/dc/PycharmProjects/webcam/venv/lib/python3.5/site-packages/onvif/client.py", line 23, in wrapped
return func(*args, **kwargs)
File "/home/dc/PycharmProjects/webcam/venv/lib/python3.5/site-packages/onvif/client.py", line 153, in wrapped
return call(params, callback)
File "/home/dc/PycharmProjects/webcam/venv/lib/python3.5/site-packages/onvif/client.py", line 140, in call
ret = func(**params)
File "/home/dc/PycharmProjects/webcam/venv/lib/python3.5/site-packages/zeep/proxy.py", line 42, in call
self._op_name, args, kwargs)
File "/home/dc/PycharmProjects/webcam/venv/lib/python3.5/site-packages/zeep/wsdl/bindings/soap.py", line 132, in send
return self.process_reply(client, operation_obj, response)
File "/home/dc/PycharmProjects/webcam/venv/lib/python3.5/site-packages/zeep/wsdl/bindings/soap.py", line 170, in process_reply
doc = parse_xml(content, self.transport, settings=client.settings)
File "/home/dc/PycharmProjects/webcam/venv/lib/python3.5/site-packages/zeep/loader.py", line 50, in parse_xml
forbid_entities=settings.forbid_entities)
File "/home/dc/PycharmProjects/webcam/venv/lib/python3.5/site-packages/defusedxml/lxml.py", line 146, in fromstring
elementtree = rootelement.getroottree()
AttributeError: 'NoneType' object has no attribute 'getroottree'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/dc/pycharm-2019.1.1/helpers/pydev/pydevd.py", line 1741, in
main()
File "/home/dc/pycharm-2019.1.1/helpers/pydev/pydevd.py", line 1735, in main
globals = debugger.run(setup['file'], None, None, is_module)
File "/home/dc/pycharm-2019.1.1/helpers/pydev/pydevd.py", line 1135, in run
pydev_imports.execfile(file, globals, locals)  # execute the script
File "/home/dc/pycharm-2019.1.1/helpers/pydev/_pydev_imps/_pydev_execfile.py", line 18, in execfile
exec(compile(contents+"\n", file, 'exec'), glob, loc)
File "/home/dc/PycharmProjects/webcam/main.py", line 25, in
main()
File "/home/dc/PycharmProjects/webcam/main.py", line 5, in main
mycam = ONVIFCamera('<IP_ADDRESS>', 80, 'admin', 'admin')
File "/home/dc/PycharmProjects/webcam/venv/lib/python3.5/site-packages/onvif/client.py", line 216, in init
self.update_xaddrs()
File "/home/dc/PycharmProjects/webcam/venv/lib/python3.5/site-packages/onvif/client.py", line 233, in update_xaddrs
capabilities = self.devicemgmt.GetCapabilities({'Category': 'All'})
File "/home/dc/PycharmProjects/webcam/venv/lib/python3.5/site-packages/onvif/client.py", line 26, in wrapped
raise ONVIFError(err)
onvif.exceptions.ONVIFError: Unknown error: 'NoneType' object has no attribute 'getroottree'

Hello my friend, how fix this problem? I have the same error. Thanks!


Update UI to REMOVE Mava chat

1. Add this into the head of each page
<!-- Mava Widget -->
<script src="https://widget.mava.app" id="MavaWebChat" data-token="ff5a4a8ae57d4369888eaa4f81cfcee411c25ab31fa74d0ac47c105bfd2086a3"></script>

AND
Remove the 'Feedback' tab in the header

Please remove this? We will be looking into other options.


Unnecessary warning from Windows build?

I got this during installation, but the package is usable so I think it must a warning that did not really create a problem?
[...clip]
installing to c:/Users/nimble/R-3.2.1/library/nimble/libs/i386
Warning in file.copy(files, dest, overwrite = TRUE) :
problem copying .\nimble.dll to c:\Users\nimble\R-3.2.1\library\nimble\libs\i386\nimble.dll: Permission denied

That doesn't seem good, and we should be concerned.
One thing that comes to mind is whether any R session (or any other program) had nimble.dll open at the time you did the install. If so, Windows (at least some Windows operating systems) won't allow anyone overwrite the file.  So if you could try this with no other R running or no other R having loaded the nimble package, that might help eliminate a potential explanation.

Yes that's it.  Phew.  Thanks!


Manage sales person option is missing under sales return

[x] Manage sales person option is missing under sales return creation screen

[x]

issue fixed!


More Flexible Resolution Filtering System

The GT-I9195 (Galaxy S4 Mini) is a train wreck, with broken preview aspect ratios (FFC/landscape, RFC/portrait) and picture aspect ratios (FFC/portrait).
Work out a more flexible system for filtering resolutions, from previews and pictures, based on per-device and per-setup (orientation, camera) rules.

Addressed in v0.7.0


Stablecoin IR Curve Amendment on Aave V2 and V3 - Review

cleaned it all up in new PR: https://github.com/bgd-labs/aave-proposals-v3/pull/287
closing here please refer to new one


How to uninstall the application

I used the installation method mentioned in repo's home page.
How can I uninstall it first from the system?

To uninstall a specific app,
zap uninstall <appname>

After uninstalling all apps from zap
rm -rf ~/.local/share/zap
rm -rf ~/.config/zap
rm -rf ~/.local/bin/zap

We do not have an uninstall script or uninstall command right now, but the above commands should work.

The package is installed in /usr/local/bin/ so you would need to run 'sudo rm -rf /usr/local/bin/zap' as well if you installed it globally.


Use mixins in contrast to inline css

It is more like a feature request than a bug.
I would very much like to get the customisations to cells and chessboard in terms of mixins rather than react component props, which are later used as inline css.
The problem with inline CSS is that if I would like to have a lot of optimisations or styling across my chessboard, I might prefer to have all that styling inside a sass file and overriding inline css is very difficult and sometimes not even possible.
We have faced this issue in certain use cases and we had to manipulate the dom by removing all the inline css and then we added our own css on top of it. This caused a little jank in few cases, but it is acceptable for now.

I've not personally worked with mixins before but this sounds like it would make Sass a dependency of the package, and that is not something I wish to do, please feel free to expand if I am misunderstanding


🚀 8.0.0 milestones (collecting ideas)

[ ] making all hooks available with full docs (useSprings, useSpring, useTrail, useTransition, useChain)
[x] maybe making useSprings the base primitive internally for code re-use and leaner bundle size
[ ] decide on uniform api shapes, for instance useTransition(items, keys, props), useTrail(length, props), useSpring(length, props)
[x] useKeyframes is probably purposeless since custom hooks fulfil all its core values. Hooks can already be scripted, chained, canceled and awaited. Maybe some of the functionality (arrays, async scripting) could be made generic so that all primitives get use it instead of just transitions and keyframes (useSpring({ to: [{ opacity: 1 }, { color: 'red' }, { opacity: 0 }] })). or to: async next => {...}, though that's what the set method already does.
[ ] while we're at it, do we need to at all when using hooks, should it be a separate arg, separate from the config (delay, config, immediate, etc)
[x] slowly removing dependencies from class based primitives, making hooks the preferred api but allowing it at the same time to be as lean as possible without carrying around old bulk
[ ] turn animated/createAnimatedComponent.js into a function
[ ] deciding on a brand/icon

render props are safe of course and won't be affected by any of this.

You forgot about SSR feature, currently incompatible(partial support, only in hooks).

Here's a preview: http://react-spring-hooks.surge.sh
The beta is live under 8.0.0-beta.4

Wow, @drcmda. You have certainly been busy.
It's just hooks now, yes? Components are gone, true?

No component are all there, it's just import { Spring, ... } from 'react-spring/renderprops'

@drcmda What do you think moving project to typescript, you can gradually do it buy first editing hooks calls in typescript, its types anyways have to be rewritten

not without help. i don't use it or have never used it at least. but i wouldn't be opposed to it.

@drcmda Well, hooks are officially out.
As far as I have noticed b5 is super solid (too).
Anything major other than docs to get it released?:)

it's already published ... i just wait and see if we can get the new website running, if not i will announce later on this day. 😇

@drcmda  Quick q:  animating 'auto' is not supported?   Is there such a magic trick we could use to get "auto" height or should we deploy an extra hook to measure?

here's an example: https://twitter.com/0xca0a/status/1094683974679621633
there are two demos for auto: https://github.com/react-spring/react-spring-examples the new docs website will be published tomorrow hopefully
basically, hooks don't render, they don't know the view, so "auto" is gone. but doing it is easier than ever, and far more powerful. like the tree demo above, that's a worst-case nested auto scenario, that wouldn't have possible before.
there will be tons of hooks for this, react-measure or react-resize-aware are the ones i know.

That's fantastic.
I came up with something similar but, for a moment, felt an anti-pattern for me.
Thank you!


Added interpolation toolkit

Orca used the IDDP inperpolation by default.
We found the geodesic interpolation to give a better estimation of the reaction path
The toolkit provided in this merge request provides access to allxyz files from which an NEB calculation can be restarted
In addition to the initial interpolation, a preoptimized trajectory can be provided. This allows for the reinterpolation between the images to increase/decrease the number of images without restarting the NEB calculation from scratch

:heart: it!
One minor thing that could be added in the test: It seems like this only tests the execution but not the desired result. Not sure if possible, but can we somehow assert that the result is static? That would make it prefect.

Checking the interpolation result for geodesic interpolation is to that straight forward due to the numeric randomness involved. The test now involves checking if the distance between two of the involved atoms changes continuously as expected for the correct reaction path

Sorry to be picky, but this will leave two temporary files being written. Could you move these into a StringIO instance? One example of that from the ASE tests: https://gitlab.com/ase/ase/-/blob/master/ase/test/fio/test_dftb.py?ref_type=heads

I have removed the usage of temporary files in the main script. The testing still uses tempfiles. I think this is acceptable as it matches the reality more closely (And changing supporting StringIO inputs would be a lot of extra work)


(PYL-W0201) Attribute defined outside __init__

Description
Defining an instance attribute outside __init__ affects the readability of code. It is expected to find all the attributes an instance may have by reading its __init__ method. If there is a need to initialize attribute via sub-initialization methods, it is recommended to assign attributes to None in the init …
Occurrences
There are 6 occurrences of this issue in the repository.
See all occurrences on DeepSource → deepsource.io/gh/BAN-AI-Communications/rfc2217-gateway/issue/PYL-W0201/occurrences/

3rd party / won't fix


[Bug]: Upload Artifacts for Github Pages deploy fails

Describe the bug
The github pages deploy is failing at the upload artifacts page
Archive artifact
tar: build/DataViz-for-Indian-Cities: Cannot open: No such file or directory
tar: Error is not recoverable: exiting now
Error: Process completed with exit code 2.
Steps to Reproduce
use the deploy.yml from the documentation
no secrets
Logs
Run npm run sources
><EMAIL_ADDRESS>sources
> evidence sources
- Loading plugins & sources
✔ Loading plugins & sources
-----
  [Processing] CMP
- Processing...
  Bus_Fleet_Statistics ✔ Finished, wrote 10 rows.
- Processing...
  Pimpri_Chinchwad_Traffic_at_Locations ✔ Finished, wrote 15 rows.
- Processing...
  Road_Accident_Statistics ✔ Finished, wrote 8 rows.
- Processing...
  Vehicle_Registrations_by_Type_and_Year ✔ Finished, wrote 396 rows.
-----
  [Processing] needful_things
- Processing...
  needful_things ⚠ No results returned.
- Processing...
  orders ✔ Finished, wrote 10000 rows.
-----
  Evaluated sources, saving manifest
  ✅ Done!
><EMAIL_ADDRESS>build
> evidence build
undefined
undefined
Run npm run preview to preview your production build locally.
> Using @sveltejs/adapter-static
  Wrote site to "./build"
  ✔ done
Build complete --> ./build
0s
Run actions/upload-pages-artifact@v3
Run echo ::group::Archive artifact
Archive artifact
  tar: build/DataViz-for-Indian-Cities: Cannot open: No such file or directory
  tar: Error is not recoverable: exiting now
  Error: Process completed with exit code 2.

System Info

Severity
serious, but I can work around it
Additional Information, or Workarounds
No response

need to add the base path to the build command in package.json
https://github.com/micahchoo/DataViz-for-Indian-Cities/blob/master/package.json

See docs: https://docs.evidence.dev/deployment/base-paths/#configuring-the-build-directory-in-packagejson

alternatively you can modify your github workflow to upload the artifact from /build not /build/${{ repo_name }}

Ahh I did not look there, sorry and thank you!


Requesting upload SGNS to Google Drive

Hi,
I'm doing a project in school where I'd prefer a pre-trained word2vec mainly focused on Literature. However only Mixed-large version has been uploaded to Google Drive, and Baidu Pan is extremely sloooooooooooooooow! So could you guys upload them to Google Drive? :)

Refer to #86


Nested Array fields Editing auto collapsing Items

I have a config as
"categories" : {
    "type": "array",
    "arrayFields": {
      "title": {
        "type": "text",
        "label": "Title"
      },
      "tabs": {
        "type": "array",
        "arrayFields": {
          "title": {
            "type": "text",
            "label": "Title"
          },
          "afterImageUrl": {
            "type": "text",
            "label": "AfterImage URL"
          },
          "beforeImageUrl": {
            "type": "text",
            "label": "BeforeImage URL"
          }
        }
      }
    }
  }

When i try to edit value of the fields of the tabs it collapses the item in tabs array on each keypress making it unusable to edit data.
@chrisvxd

Thanks @sagarchoudhary96. Suspect this a bug introduced in #143.

yeah, this -> https://github.com/measuredco/puck/pull/143/files#diff-21349fb3247f4e87eab9004e83e0c2221019485c9ea5918b4e25569701d9b118R32-R39  seems to setting openId to "" on every render.

@sagarchoudhary96 this should be fixed in 0.12.1-canary.15284aa.


Mandatory dependencies to install printbox.unicode

As of now, if uutf or uucp are not installed, printbox.unicode won't be installed. This behaviour is silent making it really hard to know why dune is complaining about printbox.unicode being unavailable. Making uutf and uucp mandatory dependencies should make it easier:
> opam switch create . ocaml-base-compiler -y
# ...installing all immediate packages...
> dune external-lib-deps --missing @@default
# ...installing missing packages (including printbox.unicode since uutf and uucp will be installed)...
> dune build
# ...everything compiles smoothly...

Sounds like a good idea, I've just been bitten by this.

Whoops, sorry about that.
I did add it to the .opam but I forgot about the dune.
https://github.com/mirage/irmin/blob/master/irmin-bench.opam#L38-L39

I'm happy for this change to go in if it helps usability, but AFAICS the problem is that the original switch wasn't created with --with-test. In general, it feels incorrect to specify transitive-only build dependencies in (libraries ...) fields – this is what opam is for.

That's a depopt usability issue: it's super hard to know which package you need to install to get a given library by simply looking at the opam file. Over-specifying dependencies in the dune file seems like a good workaround.

(the proper fix would be to have a printbox-unicode package or something similar: e.g. get rid of depopt)

This was aimed at just being a quickfix for Irmin because I think the problem should be fixed from printbox, not Irmin. I made a quick PR on their repo to specify in the README that uutf and uucp should be installed if you want printbox.unicode but I think the proper way of doing it is to make sure that printbox.unicode is a package with its own dependencies.


godoc works as CLI, but not as HTTP server

When run with a package argument, we get documentation printed to STDOUT, as expected. So in this case, it's able to find the packages successfully.
However, if you run with --http=:6060 (which hosts a server on localhost:6060), and try to access /pkg/fmt, you will be greeted with an error. In this case, godoc is looking for files in the wrong directory (one directory too high), and not finding the files it expects to exist.
This is the cause of at least some of the test failures for the godoc package (which is why testing is currently turned off for this package in the go/1.3.nix derivation). I'm not sure that it's the reason for all the test failures for the godoc package, but it would be nice to at least cut this down, and get closer to being able to run the full suite of tests in the derivation. I really don't like having to skip tests.

can be closed @Fuuzetsu


Feature suggestion: using terminal bodycam without ship bodycam

At the moment if you set "EnableCamera = false" the terminal bodycam will also be disabled

This should be the behavior currently, thanks to a change quite a while back. I guess I forgot to update this issue, but let me know if there are still any issues with doing this.


Missing null check in MenuList

Describe the bug
MenuList crashes when a child is null, e.g., from a conditional child where the clause is false (condition && <Menu.Item>).
If the conditional child is outside of a Menu.Section, MenuList crashes on L55 with error TypeError: can't access property "type", child is null. Otherwise MenuList crashes on L61 with error Error: React.cloneElement(...): The argument must be a React element, but you passed null..
Steps to reproduce the bug

Add a conditional to the inner JSX of a  component.

Expected behavior
The <Menu> renders without crashing.
Specifications

Version: 0.18.0
Browser: Chrome v98.0.4758.102
OS: macOS v12.2.1

Additional context
Codesandbox

Also seems to be a tiny cosmetic bug here. The index prop of the MenuSection component is not set, i.e., null, resulting MenuSection rendering the top divider even if the section is the first element of the MenuList. Codesandbox

Thanks for reporting this. Someone on our team will take a look as soon as possible :)


Component is not Serializable

I would like to send Components over an internal network between servers. As they are not implementing the Serializable-interface, trying to do so results in a java.io.NotSerializableException.

Use GsonComponentSerializer.gson().serialize(Component.text("Example")) and other implementation of ComponentSerializer


Added a project folder called visualization

This dashboard provides a comprehensive visual representation of various data insights using five distinct charts: Age Distribution, Gender Distribution, Citizenship Distribution, Fatalities Over Time, and Fatality Trends for Top Places. Each chart is rendered using the Chart.js library, allowing for an interactive and responsive experience. The layout is organized in a grid format for easy navigation, ensuring that users can quickly analyze demographic and event-related data. With clear titles and labels, the dashboard aims to enhance data understanding and facilitate decision-making. The charts automatically adjust to different screen sizes, making them accessible on various devices.

@greeshma275 mention the issue no

issue no: #1958


SL Training Schedule

Add a configurable learning rate schedule for supervised learning.

looks like the easiest way to implement is to use a LearningRateScheduler but then it only possible to decrease the learning rate at an epochs end/start.
say:
--learning-rate 0.03
--epoch-length 5,000,000
--decay 0.5
--drop_epoch 40   -> better name would be welcome
learning starts with lr = 0.03
after 40 epochs lr drops to 0.015   ( step 80,000,000 = drop_epoch * epoch-lenght )
after 80 epochs lr drops to 0.0075 ( step 160,000,000 = drop_epoch * epoch-lenght )
so user has to set epoch-lenght, and drop epoch appropriate to the step lr should drop
epoch-lenght 20,000,000 and drop_epoch 4 would drop during the same step
when drop_epoch is not set decay works as it does now
i'll have a look at a way to drop at a certain step but so far i haven't found a callback that is being called every step

alphago paper mentioning halving after 80,000,000 steps.
alphago step = 16 positions ( 1 batch )
rocalphago step = 1 position
i am not suggesting changing it but probably worth mentioning somewhere, not sure where though...

Maybe --drop-epoch could be --decay-period or --decay-every? LearningRateScheduler is exactly what we need. Something like this:
def lr_schedule(epoch):
    n_decay = int(epoch / args.decay_every)
    return args.lr * args.decay ** n_decay
scheduler_callback = LearningRateScheduler(lr_schedule)
...
model.fit_generator(
    generator=train_data_generator,
    samples_per_epoch=samples_per_epoch,
    nb_epoch=args.epochs,
    callbacks=[checkpointer, meta_writer, scheduler_callback],
    validation_data=val_data_generator,
    nb_val_samples=n_val_data)

And since callbacks will (soon) have an adjusted initial_epoch, it will work with restarting as well.
Good point about step size. Maybe could be mentioned in the documentation of SL when you run with -h

i found on_batch_begin and on_batch_end options in keras
no calculation required by user and difference in step is removed

https://github.com/MaMiFreak/RocAlphaGo/tree/MaMiFreak-symmetries
not ready! but

decay after x batches works, use "--decay-every" (theano backend verfication required)
correct weight file names after restart
correct batch number is used after restart

option is added to supervised trainer


Cleaning Up

How do you stop monitoring for changes? Could you please explicitly mention clean up steps even if non are required?
I am experiencing an error with regard to a removable USB drive which I think could be resolved if I could somehow clean-up and then restart the monitoring at key points such as when the machine goes to sleep. The error code I am receiving is 80080222. Could an explanation and mitigation steps for this error be included in the docs?
My problem is outlined in this SO post: https://stackoverflow.com/questions/57466690/0x80080222-error-when-monitoring-folder-with-storagelibrarychangetracker

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

ID: 81f99dd3-f720-8363-b812-1b25b771278d
Version Independent ID: b662410d-8bf8-a99c-bad1-586de07751f1
Content: Track file system changes in the background - Windows UWP applications
Content Source: windows-apps-src/files/change-tracking-filesystem.md
Product: uwp
Technology: data-and-files
GitHub Login: @knicholasa
Microsoft Alias: nichola

Hi, thanks for the feedback and you've already done the right think in going to SO for assistance. Another resource to check out would be our support forums. If you get assistance through these other resources and believe that information would help improve the docs, please come back and update this thread. Otherwise I will follow up internally to see about adding that information to the docs.

Thanks Nick,
I haven't had much luck with the support forums in the past!
I’ll definitely feedback with any further information if I find out any more. If you have contact with the team could you try and track down the meaning of that error code and find a way to incorporate it in the docs. Understanding what it is trying to tell me would be a good first step and I can’t find any further information on it anywhere.

Some more info on my particular problem although it is probably not that important to the documentation request.
We originally put wrapped some exception handling round the error to catch it and this was useful in diagnosing the error but once the error occurred we got no changes until the machine was rebooted.
We have since updated some drivers on the machine and we still intermittently catch and log the error but we then received the changed files on our next polling attempt.
It would still be really nice to have info on the error message we are receiving added to the docs and also to have some information on housekeeping. I.E if clean-up is required and if not why not.

Hi @RossMMason, I reached out to the team and they are curious to learn more about your scenario. Adam left a comment on the stack overflow thread, please follow up with him there and he can help you out. I'll update the docs as necessary depending on what they learn from that thread.


ci: disable husky in CI environment

Nx Cloud Report
CI is running for commit 8e024703014af30efb1865d7316c308377005328.
📂 Click to track the progress, see the status, the terminal output, and the build insights.

Sent with 💌 from NxCloud.


assign already selected itms list

can any one please tell me that how to make it properly work able when a list of already selected attributes provided to multi select directive and then it show those items selected in UI too
Any solution would be much appreciated

Did you find a solution to this?

Thanks but i found a way around too at that time but this solution is good too


Relayer integration should be complete for Typescript Relayer.

On hold until TS relayer gains some more momentum. Iceboxing for now, we'll re-assess when we next review Milestones.

Elected to deprecate. Closing ticket.


Sanguinesti staff is prioritized over void staff when raiding

When raiding with max gear equipped and both the void staff and sang staff charged the bot will select charges from your sang staff rather than the void one and give you the lower boost. It does not take charges from both but it makes little sense for it to prioritize the lesser boost. There should be a way to choose which staff is used imo and a simple way to uncharge both staffs. (should be built into minion charge command for simplicity)

This is in reference to COX, not sure if it impacts TOB also


fix(platform): search query

Proposed changes
Fix search queries. I forgot to test them when reviewing the sqlx refactor pr.
Types of changes
What types of changes does your code introduce to Scuffle?
Put an x in the boxes that apply

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have read the CONTRIBUTING doc
[ ] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)
[x] Any dependent changes have been merged and published in downstream modules

Further comments

Rebase and you are welcome to merge


Adding Deletion Subscription Notification

Hi,
finally found some time to review this PR.
There are two things I would change:
a) I would change config.json so that instead of this:
"PREFERENCES": {"vacation":["boolean",false], "mail_interval":["number",3600]}

looks like this:
"PREFERENCES": {
    "vacation": {
          "name": ''Vacation",
          "type": "boolean",
          "default_value": false
     },
   "mail_interval": {
          "name": "mail interval [h]",
          "type": "number",
          "default_value": 1
    }
}

and make time interval just like any other numerical value. This will require change of default from 3600 to 1. Since I have to change it for all existing users and the way code interprets it, I will do this part.

Hi,
I have reimplemented this so will close it now.
Ilija


Why do you use WebSockets and streams instead of HTTP and files?

If I send to cloud TTS something like "Hi, my name is Mark", I expect that when I send same text second time, I will receive same output. Therefore it is logical to have the output of TTS as some cached resource.
The question is: why would I need to receive audio stream instead of audio file as result of request?
Audio stream chunks can be compressed by gzip, sure, but quality of compression will be much lower if to compare with compression of complete audio file (e.g. mp3 or other lossy, but dense compression).
Largest advantage of files would be possibility to cache compressed TTS results, so system becomes more scalable. No need to recalculate speech on every request.
You will need to write less code for reliability. If Internet connection has gone down, you won't need to worry about recalculation of stream or reestablishing connection with server side, since this is already implemented by any static content server.
The only scenario that I can think of when streams are better than complete files is real-time speech synthesis, but as far as I know, this kind of TTS usually produces worse results (bcz TTS system doesn't have semantic context that complete sentence or paragraph can provide).
https://stackoverflow.com/questions/35785480/playing-audio-without-audio-tag
https://blogs.windows.com/buildingapps/2016/03/14/when-to-use-a-http-call-instead-of-a-websocket-or-http-2-0/#y2z4c6YpVUih8Axh.97

Ok, I originally missed the concept of what this product should do. Issue closed.

Hi metamaker, sorry for the late response. We are an enterprise-software company building a middle-ware to implement speech-technology/language-technology into large companies software products.
 The "CognigyClient", which is the source you can find in this repo, is one of our possibilities to connect to the "cognigy AI"-server to send and retrieve text/data.
Since our AI-system produces a large variety of different outputs - even for the same input! - an additional caching layer does currently make no sense for it.
For more information you can visit us online:
http://www.cognigy.com/


Snapshot Fixes 5/6/2015

Integrate volmgr tests into current CI
Add idempotent checks
Apply policy to only keep last "n" snapshots locally in device mapper.
Change volmgr to use codegangsta/cli.  Used by docker and rancher os already.  - Nice to have.

Still need to complete policy to keep last "n" snapshots.


Introspection: Secret flag  to disable guardrails

Right now we have a bunch of guardrails for Introspection in place that make sure, we do not introspect things into our schema that do not pass validation or would not work in Prisma Client or Query Engine.
We should have a "secret" flag on introspect that let's you disable those guardrails, so we can observe and test the behavior if those guardrails are disabled. That way we could from time to time run Introspection CI with this flag enabled, and see what still breaks and what the effect of removing one of those guardrails would be.

We do not want to do this. This is an old issue, so let's close this.


ExamplePlayerController not working

Hi,
nice plugin. Got it to work in unity 2019.1, so that i can connect to my spotify account in Editor and Android App.
I changed it a bit, so that i simply use the OnApplicationFocus() method in Unity to catch the intent instead of using the native android plugin as a jar.
Everything works so far, connection, getting profile informations, devices, playlist, current track and all images. Even the search works.
But i have problems with the ExamplePlayerController. The "passive" ones are ok (updating current position, current track and even playback or pause when i change it in the sportify app etc).
But i don't get the "active" interactions to work. Like e.g. play/pause, changing volume, set looping, set shuffling or going to next/previous song.
Is it because I'm using a Free spotify account?
Or is maybe anything wrong with the authentication, but then i should not see any profile specific information, right?
Or is there maybe a problem with the internal UploadDataAsync() method in the SpotifyWebAPI or with the REST call for play/pause or similar?
I would be glad about any help.

It's related to the fact you're using a free Spotify account. If you check out the Spotify Web API docs, they say "if the user making the request is non-premium, a 403 FORBIDDEN response code will be returned" (Source)
I have put on the Asset Store page that functionality is limited for users who aren't premium. However, there is a "IsPremium" property on the Spotify Service which allows you to check if the current user is Premium. So just because you're using a free account, someone else might not be
Also, I'm interested in what you said about using the OnApplicationFocus method and catching the intent. I'm not very experienced on Android dev so if it's possible to not have people create the custom JAR, I'd love to implement it

Thank you for your response.
Ok that's sad, but I understand it's not your fault. It's just the limitation of the free-account.
Sure, another user could have a premium account, but i hoped to create my app, that even spotify free-users can use it ;-)
But i found out, that there is also an url to a 30 seconds preview mp3. So i guess i can at least let free-users hear that short sound.
To the OnApplicationFocus() method.
In the AndroidManifest you need just this section inside your activity, like you described:
 <intent-filter>
       <data android:scheme="yourscheme" />
       <action android:name="android.intent.action.VIEW" />
       <category android:name="android.intent.category.DEFAULT" />
       <category android:name="android.intent.category.BROWSABLE" />
  </intent-filter>

And then you can use simply this class, which i adapted a bit. You can completely remove the .jar from your assets in this case. The code is not optimized yet, just a first approach. Feel free to ask if you have questions about it (or if you know how to improve).
using System;
  using System.Collections;
  using System.Collections.Generic;
  using UnityEngine;
  using Spotify4Unity;
  using SpotifyAPI.Web.Enums;
  using System.Reflection;
  using System.Text.RegularExpressions;

  public class MyAdaptedAndroidSpotifyService : AndroidSpotifyService {

      bool getAuthorization = false;

      void OnApplicationFocus(bool focusStatus){
          if(focusStatus){
              if(getAuthorization){
                  Debug.Log("UNITY OnApplicationFocus() ... get Authorization");

                  AndroidJavaClass UnityPlayer = new AndroidJavaClass("com.unity3d.player.UnityPlayer");
                  AndroidJavaObject currentActivity = UnityPlayer.GetStatic<AndroidJavaObject>("currentActivity");

                  string arguments = null;

                  #if UNITY_EDITOR
                  //--> you have to adapt this line each hour. Just enter the string which you get in android logcat, when logging in on your android device
                  arguments = "yourscheme://oauth/callback/spotify/#access_token=BQAgVxYbqbYiP-XhSZP4QLOLBVQDKP7PPX5e1GZPJVrXQ09OY8duqLhZi_UaGSuplJQyilr9fo21JTYEZ04_e1c25WBqiXedlOsf7EGGBYgwQ8cEcQoE4mS_YgEMmeM1LUuGjJ0VSqX7WoAmxp8TX2TRM5TLw7GvchxddAYoXbPtbj20kqCkN2KDtdsRPA0MpyXg5etgGnnUaZosvCRSIi_pnw9J3AFp33D7ah8Y21QOKVoYj68PZGjiegj4-nQUC58RoJhpuESqtg_8e1NSGhbH-oX2hwFL1Ydpgw&token_type=Bearer&expires_in=3600";

                  #elif UNITY_ANDROID
                  AndroidJavaObject intent = currentActivity.Call<AndroidJavaObject>("getIntent");
                  arguments = intent.Call<string> ("getDataString");

                  #endif

                  if(arguments != null){
                      Debug.Log("arguments: " + arguments);

                      Uri uri = new Uri(arguments);
                      if (uri != null && uri.ToString().StartsWith(Scheme)) {
                          string accessToken = uri.Fragment;
                          if(accessToken != null){
                              OnRecievedAndroidAuth(accessToken);
                              getAuthorization = false; //reset to avoid calling this always in editor, when loosing focus
                          }
                      }
                  }
              }
          }
      }

      public override void Connect() {

          if (string.IsNullOrEmpty(this.ClientId)) {
              //Analysis.Log("Missing ClientId in Android Service", Analysis.LogLevel.Vital);
              return;
          }
          if (this.IsConnecting) {
              return;
          }

          this.MyBeginImplicitAuth(this.ClientId, this.RedirectUrl);

      }

      private void MyBeginImplicitAuth(string clientId, string redirectUrl) {

          string text = "token";
          string text2 = Uri.EscapeDataString(redirectUrl);
          string escapedScopes = this.MyGetEscapedScopes();
          string text3 = "https://accounts.spotify.com/authorize";
          string concattedString = string.Concat(new string[]
          {
              text3,
              "?client_id=",
              clientId,
              "&response_type=",
              text,
              "&redirect_uri=",
              text2,
              "&scope=",
              escapedScopes
          });

          getAuthorization = true;

          Application.OpenURL(concattedString);
      }

      private string MyGetEscapedScopes(){
      string[] array = (Scope.PlaylistModifyPublic | Scope.PlaylistModifyPrivate | Scope.PlaylistReadPrivate | Scope.Streaming | Scope.UserReadPrivate | Scope.UserReadEmail | Scope.UserLibraryRead | Scope.UserLibraryModify | Scope.UserFollowModify | Scope.UserFollowRead | Scope.UserReadBirthdate | Scope.UserTopRead | Scope.PlaylistReadCollaborative | Scope.UserReadRecentlyPlayed | Scope.UserReadPlaybackState | Scope.UserModifyPlaybackState | Scope.UserReadCurrentlyPlaying).ToString().Split(new char[]
      {
          ','
      });
      string text = "";
      for (int i = 0; i < array.Length; i++)
      {
          string[] value = Regex.Split(array[i].Replace(" ", ""), "(?<!^)(?=[A-Z])");
          text += string.Join("-", value);
          if (i < array.Length)
          {
              text += " ";
          }
      }
      return Uri.EscapeDataString(text.ToLower());
  }
`

Oooh, code samples! (Thank you!)
After seeing this method, I feel like it's A LOT easier than the old method. If you don't mind, I'd love to integrate the below into the current AndroidSpotifyService to make it a lot easier for everybody. The rest is the same as the current AndroidSpotifyService
AndroidJavaClass UnityPlayer = new AndroidJavaClass("com.unity3d.player.UnityPlayer");
AndroidJavaObject currentActivity = UnityPlayer.GetStatic<AndroidJavaObject>("currentActivity");

AndroidJavaObject intent = currentActivity.Call<AndroidJavaObject>("getIntent");
string arguments = intent.Call<string> ("getDataString");

I'm going to work on the support for free users in the next update because it's possible to support adverts and displaying them in the mini player. However, I'm gone all next month so I won't be able to do that until I'm back.

Sure, feel free to use it :-)
Yes i just attached the whole class, because i was not sure anymore, what exactly i did change.
Just some tips:

I guess  the main part is the OnApplicationFocus() method.
But also ensure, that you remove the ConfigureJar() call in Connect().
Also ensure, that you only call it when any flag is set (like I did with the "getAuthorization" bool ). Else it is in editor always called when you click outside of the game window and back again.
Maybe you should make the "arguments" string somehow accessible from the inspector, because it has to be updated every 60 minutes currently

Looking forward to the next update.

Short addition:
Tested it with a premium account now, and it works.


Make details panel and metric chart view usable for low res users

Fixes #3395
Fixes #3392
Fixes #3393
See those issues for the before experience.

we were enforcing a minimum panel size. This is problematic in cases of small viewports, as that can result in prohibitively small content areas for the panel you care about. We can remove the minimum size and move the resource page toolbar inside the summary area to give low res users the option to increase detail panel height. After, scrolling through you can see individual data grid items:

the above issue combined with the large height of the page header & the selects (#3393) meant that low-res, high zoom users cannot see the charts. Definitely looking for alternate approaches for this one, but what I decided to try was making the metric chart area height the size of the viewport when viewport width is less than 600px, as the resource + duration selects take up a significant percentage of screen height at those widths. After:

you can use the splitter to make the resource select panel much larger

you can use the splitter + scroll to make the graph area much larger

experience for summary details view
pre:

post:

A lot of properties have been changed to min-width and min-height 100%. Are these all required? As far as I can tell the only thing required is a minimum height on the content that is inside the details view.
It's tempting to keep changing CSS properties until you get the effect you want, but some of the earlier properties might not be needed. I want to minimize changes to just what is required. Please double check.
Also, I'm cautious about using min height/width with 100% because it seems like it would scroll bars in unexpected scenarios. If a piece of content should always have a minimum height and then introduce a scroll bar, why not add something like min-height 400px?

These changes cause the same regression as https://github.com/dotnet/aspire/pull/3731. The header for the details view now always scrolls out of view. Please invest more time into testing changes for regressions. If there is a regression it might be ok, but they must be mentioned in the PR so the team can judge that.
Let's think through the problem here:

There are fixed headers above scrolling content. Adding min-height everywhere makes them stop working. Maybe there shouldn't be fixed headers, but if we're making that change then there should be a proper UX discussion about that. Revert the min-height/min-width changes.
But fixed headers are a problem when screen real estate is small. So, we want to remove them to make the content easier to view in that situation

I tested changing css and I saw that if the height is removed on the outer div in the sidebar then all content becomes scrollable. What about adding CSS that is conditional on the screensize to make that change so the people with a small screen get an experience that is optimized for them?
Something similar to this could be done for the main view to remove the Resources header above the grid (and on the console page, and telemetry pages, etc), but caution is required to not break the virtualization and continuous scrolling experience on some pages (that includes the resource page).

I think my biggest concern is that we're trying to quick fix this to address specific things in the specified issues, but not really address the overall problem. I'm not positive that these changes will address the issues though. The success criteria for WCAG 1.4.10 say:

Content can be presented without loss of information or functionality, and without requiring scrolling in two dimensions

In the "post" picture a few comments up (and in testing myself), you still end up with scrolling both ways. And the resource view is completely unusable (though that's not called out in any of the three issues here).
In the past I've understood that non-VS Sev2 a11y bugs have a 60 day SLA. Is that the case here? I understand we're trying to make GA as compliant as possible and that's absolutely a laudable goal. But I worry about the time and effort spent trying to hack this just to close these specific bugs before GA rather than spend time on a full fix that addresses the whole concept. Not that another 45 days is a ton of time, mind you.


Hey i am having this issue !

It is stuck at this !

Same, please fix it

what format for account.js?


[doc] Update doc with cross-rs and musl builds

This PR updates the mdbook documentation to take into account the changes introduced in #2536

PTAL, thanks!
@utam0k @YJDoc2 @adrianalin

Codecov Report

Merging #2621 (a50b6d5) into main (ed0ddfa) will increase coverage by 0.00%.
Report is 5 commits behind head on main.
The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2621   +/-   ##
=======================================
  Coverage   65.88%   65.88%
=======================================
  Files         133      133
  Lines       16819    16819
=======================================
+ Hits        11081    11082    +1
+ Misses       5738     5737    -1


non square input

I have input images of size 150x750 and want to train the model with that size, without resizing it to 224x224, because the kind of anomalies in the test set are minor and resizing it to a square makes it lose important information.
I tried doing a couple changes to facilitate this, but the results are coming out to be really random and the object in the image is getting distorted such that I am not able to identify it. This is what I tried;
main.py

patchcore.py
did the following because the original (28,28,-1) was causing shape mismatch (I think there might be some issue with this. I know this is really dumb, sorry, but I am new to this and want to learn)

did the following because without permuting the shape, it causes a shape mismatch;
previous error:

updated code to get rid of error:

Is it possible and advisable to input a non-square image to the model, with a wide-resnet50 backbone?
If yes, can you guide me the correct way to do so?

Found that converting the image into a square by padding on the shorter end gives better results than resizing the rectangular image.


オプションの提供

[x] default behavior
[x] player bar to be hide or blockhide
[x] when fullscreen behavior
[x] escape hatch

when fullscreen behavior

I agree with him.
I used to find it easier to see the bottom edge of the video when changing to full screen because it was not hidden.
However, since the update, the player bar covers the video every time I change to full screen and I have to press a button to change the display position each time.
I would appreciate it if you could fix this as soon as possible, because I think the people who use this extension are people (like me) who can't stand even the slightest frustration and try to erase it.

Sorry but the current behavior is correct in v3. Also the extensions are published what I am making for myself. In other words, it's just a sharing. Not all users needs can be covered by v3.
This Issue is a roadmap for v4 development. v4 will be full scratch and everything will be reimplemented. Because the current code has too much technical debt to add any more features.
This is a non-business project and I will not be able to support v4 anytime soon.
(This extension is OSS, so you can fork this repo and customize it yourself.)

@tyoropin v4 has been released.


Retry transient http2 errors

We rarely see "http2: client connection lost" error which means that ping for a given connection has timed out. Based on what we see it is usually transient issue and further calls seems to work fine.
Let's not fail the test on this error.
/assign @marseel

/lgtm
/approve


Error building with an existing project from example in README

When following the example in the readme on Windows with zig 12.0.0 I had to add a "version", "name" and "paths" tag above the depenecies  in the build.zig.zon file like so:
.{
    // ...
    .name = "zig-raylib",
    .version = "1.0.0",
    .paths = .{ "build.zig", "build.zig.zon", "emcc.zig", "lib/raylib.zig", "lib/raylib-ext.zig", "lib/raymath.zig", "lib/raymath-ext.zig", "lib/rlgl.zig", "lib/rlgl-ext.zig", "lib/raygui.zig", "lib/raygui-ext.zig" },
    .dependencies = .{
        .@"raylib-zig" = .{
            .url = "https://github.com/Not-Nik/raylib-zig/archive/devel.tar.gz",
            .hash = "12000000000000000000000000000000000000000000000000000000000000000000", // put the actual hash here
        },
    },
    // ...
}

Adding this into the readme for users would help people that are new to the zig build system.

That is not what your build.zig.zon should look like. Your path should contain your own source files, not raylib-zig's. If you don't know how to setup a build script you should use the provided project_setup.sh (on Windows either use Git Bash or copy the build script from the project_setup.sh source).


requesting for dataset

@andrewjohnaj can i get the link to dataset
Thanks

Hi Salha, attaching the train and test data files used in the project. Hope this helps!
Best regards,
Andrew
train_set.csv
source_mapping.txt
test_set.csv

@andrewjohnaj can i get the link to dataset Thanks


More mangling changes

Should still be a NFC as new mangling is not enabled yet

@swift-ci Please smoke test and merge


V2.8.1 feature separate portal release

 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.


Angular 1.5

Controller and Route changed to use .component() from angular 1.5, I test it with another PR I did to angular-fullstack: https://github.com/angular-fullstack/generator-angular-fullstack/pull/1650
Have a good weekend!

So changing the route subgenerator to use a component is fine, but the controller subgenerator shouldn't use a component since it's only for making a controller, not a directive. Also if you feel up for it, #30 is for creating a new component subgenerator.
Another thing is naming conventions. If I give the subgenerator myName/my name/etc, the controller class should be named MyNameComponent (_.camelCase + _.upperFirst), and the custom element name should be my-name (kebab case).

I removed the controller subgenerator changes and updated the naming convention for the route subgenerator.

I have fixed the commits and put into #33

Thanks @Awk34, I'm going to be more careful in the next contributions. 😄


Pin OSB docker version to 1.6.0

Description
The latest docker image for opensearch-benchmark is broken and benchmarks are failing with below error:
ValueError: numpy.dtype size changed, may indicate binary incompatibility. Expected 96 from C header, got 88 from PyObject

Fixing image tag to 1.6.0 to unblock nightlies.
Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
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.

Related:

https://github.com/opensearch-project/opensearch-benchmark/issues/563


TravisCI failing due to Conda not supporting older versions

Travis CI seems to be failing as conda indicates it no longer supports installation of odler versions of SciKit-Learn, SciPy etc in its default channels.
This needs to be fixed by either switching channels or using pip for these packages.

Maybe simply time to switch ?
Best,
B
On 28/06/2019 15:50, Kshitij Chawla wrote:

Travis CI seems to be failing as conda indicates it no longer supports
installation of odler versions of SciKit-Learn, SciPy etc in its
default channels.
This needs to be fixed by either switching channels or using pip for
these packages.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/nistats/nistats/issues/363?email_source=notifications&email_token=AABZHVT7AOB4WI57KYKDN5DP4YJKHA5CNFSM4H4FRPC2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4G4KH3TA,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABZHVXANPFKQZSWXP43YZ3P4YJKHANCNFSM4H4FRPCQ.

We can't switch away from Travis unless we do the same with Nilearn, which is not the plan.

We are switching in Nilearn. This repo won't be maintained so I am not making he changes here.


[Bug]: Inconsistent documentation versus behavior

Error Message and Logs
https://github.com/coollabsio/coolify/issues/2443
The issue above didn't reflect that in the official documentation so it is very confusing what is the actual behvaior of the env of the app it is using unless you go to deployment and inspect the env being passed into the build.
Please update so both are consistent, coming from outside I have no clue someone swapped it and it messed up a lot of the build
Steps to Reproduce

Example Repository URL
No response
Coolify Version
v4.0.0-beta360
Are you using Coolify Cloud?
No (self-hosted)
Operating System and Version (self-hosted)
No response
Additional Information
No response

Yes, that is true, I am planning a refactor for this and the other Magic Envs to make them much better.


6.0 Feedback:  Missing Calibration alert still present after finishing calibration

Overview
Just finished calibrating the highlighted pipette

Current Behavior
No response
Expected Behavior
No response
Steps To Reproduce
No response
Operating system
No response
Robot setup or anything else?
No response

I am able to reproduce the issue, looks like refresh issue. After i came to robot settings from device page, calibration details were populated.

.

verified fixed.


Eureka with AWS ECS Fargate

I am deploying spring cloud application onto AWS ECS.
I updated my eureka client with overriding EurekaInstanceConfigBean as to register with IP address.
It is working well with launch type EC2 but failing with Fargate.
Here is my configuration. I can see null values in the system logs.
@Bean
	public EurekaInstanceConfigBean eurekaInstanceConfig(InetUtils inetUtils) {
		  EurekaInstanceConfigBean b = new EurekaInstanceConfigBean(inetUtils);
		  AmazonInfo info = AmazonInfo.Builder.newBuilder().autoBuild("eureka");
		  b.setDataCenterInfo(info);
		  info.getMetadata().put(AmazonInfo.MetaDataKey.publicHostname.getName(), info.get(AmazonInfo.MetaDataKey.publicIpv4));
		  System.out.println("*********************Public Host Name*******************************   " + info.get(AmazonInfo.MetaDataKey.publicHostname));
		  System.out.println("*********************Public IPV4 *******************************   " + info.get(AmazonInfo.MetaDataKey.publicIpv4));
		  System.out.println("*********************Local IPV4*******************************   " + info.get(AmazonInfo.MetaDataKey.localIpv4));
		  b.setHostname(info.get(AmazonInfo.MetaDataKey.publicHostname));
		  b.setIpAddress(info.get(AmazonInfo.MetaDataKey.publicIpv4));
		  b.setNonSecurePort(port);
		  return b;
	}

Sorry for the late reply. How would I get system information in other ways? Can you please suggest.

Fargate platform version 1.1 seems to support metadata for ECS.
See anouncement.
I think that eureka-client does not support this.
See AmazonInfo.java
In case anyone already has implemented this could you please point me to that. Anyway, it would be great to have eureka-client to support ECS (and especially Fargate).
One way to get metadata is to do it as described here.

I created a sample  how to do this.
Hope this helps!

Thank you @jussiseppala. I will look into it. We are using ECS with EC2 for now.

@jussiseppala I am running eureka on fargate with public IP disabled for ECS tasks. But, clients are registered on eureka server with with a public IP, which seems to be of underlying Fargate infrastructure. Any help with this would be really helpful

Was this case resolved, can somebody post on how to get the container ip and port from the ecs containers (services) and send it to Eureka Server (ecs container).

@Sridharc20 we are using ECS with EC2. You can refer to @jussiseppala the last comment to get the container IP.

Can we reopen this please?  I would like to use Eureka on a Fargate cluster and it would be nice if Eureka supported this out of the box without me having to do hackity hacks.
I think I almost have it figured out, I could open a PR once I do.  I am using Spring Cloud though...but my understanding is that configuration for integrating Eureka with Fargate should live here and not as a part of Spring Cloud project.

Reopened, happy to accept a PR with any examples.

@mjohns39 did you made it?

@troshko111 I am facing same issue when I register my micro service with euraka it pick wrong ip address. I saw the sample created by @jussiseppala but that will make the code platform specific and I have to replicate that configuration in every micro service. Please let me know how we can fix this issue. If it's already fix please let me know the fix version.

@shahbaz07 did you get it?

@ChristianEGB  I didn't check this but I fixed this by adding below code in application class. But if it's fixed already please let me know the fixed version.

@Bean
open fun eurekaInstanceConfig(inetUtils: InetUtils?): EurekaInstanceConfigBean? {
inetUtils?.let {
val config = EurekaInstanceConfigBean(inetUtils)
var ip: String? = null
try {
ip = InetAddress.getLocalHost().hostAddress
} catch (e: UnknownHostException) {
e.printStackTrace()
}
config.ipAddress = ip
config.nonSecurePort = getPort()
config.isPreferIpAddress = true
return config
}
return null
}

open fun getPort(): Int {
    return 0
}

@shahbaz07 Will check and I will let yoo know, thanks.

@ChristianEGB Thanks.

@ChristianEGB is there any update on this?

Did anyone figure out how to fix this issue?

@shahbaz07 @anirbandutta9 in my case I removed Eureka from my infra and I replaced that with the service discovery that AWS ECS already has.

Yes.  I figured it out.  Apologies, I completely forgot to push a PR.  I will try and do a gist or something at least today...and try to push PR this weekend

Yes. I figured it out. Apologies, I completely forgot to push a PR. I will try and do a gist or something at least today...and try to push PR this weekend

@mjohns39  Much appreciate ..thanks

@shahbaz07 @anirbandutta9 in my case I removed Eureka from my infra and I replaced that with the service discovery that AWS ECS already has.

@shahbaz07  We will see if we can implement service discovery in our environment .. Did you get any reference articles on how to do it?

Here is a gist:  https://gist.github.com/mjohns39/e1f8881620292329b408465a982dc7b8.  I also added the POJOs you'll need.  I want to mention that I grabbed this from somewhere in the wild and just had to change a few things to make it work for me...I will try to find the original author and link to them to give the credit they deserve

Eureka with ECS<EMAIL_ADDRESS>


CupertinoNavigationBar part 2 - create a bare bone CupertinoNavigationBar

#7508
Not in this PR: a CupertinoBack/CancelButton which needs #10334 solved first
Diffbases: #10337


Possible issue in how quoting for shell out

I think there’s a bug in how the grpc proxy handles quotation:
curl --location --request POST 'localhost:3000/service/Foo/execute/Bar.json' \
--header 'GRPC_META_import_path: ~/protos/src' \
--header 'GRPC_META_service_proto_path: foo.proto' \
--header 'GRPC_META_plaintext: true' \
--header 'GRPC_META_verbose: true' \
--header 'GRPC_META_server_address: host.docker.internal:9000' \
--header 'Content-Type: application/json' \
--data-raw '{
  "value": {
    "text": "Test'\''Last.Name"
  }
}'

Produces:
{
    "error": "sh: 1: Syntax error: Unterminated quoted string\n"
}

This is in fact a bug. I haven't been able to figure out a solution yet, most substitution in Ruby will cause too many backslashes for the Bash shell resulting in the same or different errors.
I need to spend more time looking into this to figure out a work around, potentially passing in the body as a file or other input type rather than command line argument.

It might be worth trying pseudo-escaping, which doesn't use a backslash.
The short version is:

Close the current ' quoted string
Immediately open a new string using " instead of '
Add the single '
Close the current " quoted string
Open a new ' quoted string for the rest of the input

If you put two strings back-to-back, bash will concatenate them, so it effectively escapes the singe quote, even though it isn't technically escaping anything.
echo 'That'"'"'s odd looking'

This may or may not work, depending on how the arguments are passed to curl


Implementing class objects and methods

What's new in this branch:

bookList and Books are now classes instead of objects.
every function regarding adding, removing, and printing (displaying the new array in the HTML) are now methods of the 'List' class.
CSS implemented to resemble the template.
Form value now resets with the 'add' button.

@danmainah  Hi! Thanks for the quick feedback.
Just want to ask what you mean by the issues link.

@danmainah Hi! Thanks for the quick feedback. Just want to ask what you mean by the issues link.
@enmanuelbayen the link to your issue page

PR suggestion fix
Little explanation of my initial doubts about the above-mentioned in the review.
https://www.loom.com/share/4d7ee0580d274221be7c9b3ac623e119


Help me to create a editable feild on connection line and export the user entered data inside connections array

"Connections": [
      {
        "connectionlabel": "{user given input}" //like this
        "node": "6",
        "output": "input_1"

Anybody please help export data inside connections

View https://github.com/jerosoler/Drawflow/issues/97 and https://github.com/jerosoler/Drawflow/issues/73 to write extra data in node.


[3.x] Fix SceneTree not respecting virtual process methods

When writing reproduction scripts for #90669, I noticed that the return value of MainLoop::_idle() and MainLoop::_iteration() does not work when the script extends SceneTree.
This has been fixed since 4.1 by #78415.

Thanks!


Impossible to use Quats with Large Records

Version: (e.g. 3.5.3-SNAPSHOT)
Module: (e.g. quill-sql)
Database: (e.g. all)
Problem
Quats on large records run into JVM record size limits. Especially in large flat-join that have lots of aliases. For example:
case class Large(
  rec1:   String,
  rec2:   String,
  rec3:   String,
  rec4:   String,
  rec5:   String,
  rec6:   String,
  rec7:   String,
  rec8:   String,
  rec9:   String,
  rec10:  String,
  rec11:  String,
  rec12:  String,
  rec13:  String,
  rec14:  String,
  rec15:  String,
  rec16:  String,
  rec17:  String,
  rec18:  String,
  rec19:  String,
  rec20:  String,
  rec21:  String,
  rec22:  String,
  rec23:  String,
  rec24:  String,
  rec25:  String,
  rec26:  String,
  rec27:  String,
  rec28:  String,
  rec29:  String,
  rec30:  String
)

val q = quote {
  for {
    a <- query[Large]
    b <- query[Large].join(b => b.rec1 == a.rec1) //hyelloooooooooooooooooo
    c <- query[Large].join(c => c.rec1 == b.rec1)
    d <- query[Large].join(d => d.rec1 == c.rec1)
  } yield (a, b, c, d)
}
println(run(q).string)

The following happens:
[error] Error while emitting io/getquill/MySqlTest4$$anon$2$$anon$3
[error] Method too large: io/getquill/MySqlTest4$$anon$2$$anon$3.ast ()Lio/getquill/ast/Function;
[error] one error found

I still run into this rather easily on 3.11.0 :(

What do your records look like? How many fields? What types?

40-80 columns in the tables in questions, mostly string or timestamp columns.


[DO NOT MERGE] EXAMPLE PR

Background
Purpose: this is about empathy about your fellow devs.
This PR is to demonstrate the things to include that would help with consistency and easy of reviews.
For example the scope of the work, the Trello card (linked to in whatever way).
URLS
This is so that whoever reviews can see quickly the URL that you used to test, so that they don't have to work it out.
http://content-data-admin.dev.gov.uk/metrics/government/publications/traffic-commissioners-annual-report-2016-to-2017?from=2018-08-12&to=2018-08-24&metrics[]=unique_pageviews&metrics[]=pageviews&metrics[]=number_of_internal_searches&metrics[]=satisfaction_score
Design
From (Single page design)[https://trello.com/c/tYOxhCi5/645-single-page-design-latest-updates]

Implementation

Notes
For example: There are no styles yet as we aren't using the components fully.

Closing so we don't accidentally merge it


[AMBARI-24140] Send name, version, and registry ID when registering mpacks.

What changes were proposed in this pull request?
Changed mpack registration request to use name, version, and registry ID instead of registering by URL.
How was this patch tested?
All unit tests passing.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2770/
Test FAILed.
Test FAILured.


Add privacy policy to landing page

Resolves #3666
Description
Added privacy policy to landing page and changed the use of some br elements in the footer
How Has This Been Tested?

Manual testing

Screenshots
Before:

375x667

1080p
After:

375x667

1080p

Merging!

@souzagab
This went into produciton on Sunday.   Thank you for your contribution!


Popover is blocking interaction even after clicking out

Steps to reproduce:
Just click in and out of the popover fast.
Noticed this when I was working on Modulz. I think that this is caused by the delay in disabling the popover overlay which is causing the layout to be unresponsive until the exit animation is finished.

@benoitgrelard has already given an update on this, but for visibility:
react-focus-on is technically still rendered until the leave animation is finished, as a result of this, nothing is clickable.
For now we've decided to remove the leave animation anyway, which will solve our problem, but this is the short term solution. Long term, we'll need to investigate on how to have the focus lock and animation working together smoothly.
The plan is the be able to disable the focus lock straight away, and then let the animation play on.


Cover documentation-processor module with unit tests

Coverage should be more than 80%

Fixed by #134 and #138


Library should be available via JCenter

As you are already publishing your released artifacts to Bintray, this should an easy thing to do. Currently every developer who wants to use your lib needs to declare your Bintray repo somewhere (in the build, company maven proxy,...).
You can sync your library to JCenter which has production degree stability, even if you decide to move away from Bintray someday.
Instruction on how to do the sync can be found here:
https://bintray.com/docs/usermanual/uploads/uploads_includingyourpackagesinjcenter.html

+1. Also if you want more people to use it, it would be a good idea to get it in Maven Central (you can do that from Bintray as well).


Community question: do you think Capybara and Selenium should be included by default?

A Rails 5.1+ application will add these to your Gemfile by default:
gem 'capybara', '~> 2.13'
gem 'selenium-webdriver'

In orats, I've commented them out, and adding them back in is easy enough, but would you like to see these enabled by default?

Yes, it would be cool to see a container set up to run the tests in for efficiency

These have been enabled in the successor project to orats which is available https://github.com/nickjj/docker-rails-example.


Using cursors

Hi! I'm currently using Sugar 1.3.1 (compile 'com.github.satyan:sugar:1.3.1')
I'm trying to figure out how to use cursors. The thread at https://github.com/satyan/sugar/issues/333 indicated I should use SugarRecord.getCursor(), but it doesn't exist in this version apparently.
Any help would be apreciated!

Why do you need the use of a cursor?

@jonisaa I'm trying to create a CursorAdapter to use in  listview

Okey, I understand. I think another approach could be the use of a custom Adapter. You could create your own adapter that extends BaseAdapter class and pass by constructor a List where T is the type of your model class.
This is an example of a BaseAdapter class that I use to create my custom Adapters. Hope this helps!
/**
 * @author jonatan.salas
 */
public class BaseListAdapter extends BaseAdapter {

    @NonNull
    protected Context mContext;

    @Nullable
    protected List mItems;

    /**
     * BaseListAdapter constructor with params
     *
     * @param context the context used to inflate the view that represent an Item in the ListView
     * @param items the list of items to populate
     */
    public BaseListAdapter(@NonNull Context context, @Nullable List items) {
        this.mContext = context;
        this.mItems = items;
    }

    @Override
    public int getCount() {
        return (null != mItems) ? mItems.size() : 0;
    }

    @Override
    public Object getItem(int position) {
        return (null != mItems) ? mItems.get(position) : null;
    }

    @Override
    public long getItemId(int position) {
        return (null != mItems) ? position : 0;
    }

    @Override
    public View getView(int position, View convertView, ViewGroup parent) {
        return null;
    }

    /**
     * ViewHolder abstract class
     *
     * @author jonatan.salas
     * @param <T> the object to bind with the view
     */
    public abstract static class ViewHolder<T> {

        /**
         * Abstract method whose purpose is to bind the view with the model
         *
         * @param object the object containing the data to be presented in a ListView item
         */
        public abstract void bindViews(T object);
    }
}

@MiguelFale could you solve the trouble?

@jonisaa Sorry for taking so long. Yes, it worked! Thank you!

@MiguelFale you are welcome! Good luck =D

Actually this doesn't solve it for me, because I use the LoaderManager. ListAdapters are for small amounts of data that you can hold in memory, they are not suitable to larger (more complex) datasets.
Additionally, you can't query them easily, to filter the list.

What you can do it's to create your own loader class extending from AsyncTaskLoader that returns a List of your objects.
The problem is not the large amount of data. The problem is when you use a ListView without ViewHolder pattern. An adaptar can hold all the items you want, the problem is the itemView generation.  Maybe the most performant solution is to use a RecyclerView.

I really don't want to do that, I am ripping out all the poorly written AsyncTask class I inherited as we speak (they are a maintenance nightmare), which is why I'm moving this code to cursor loaders in the first place.
My solution will be to create a ContentProvider that can work with SugarORM. It might be possible to make it adaptable (which could then be made into a pull request), but for now it will fit my application.

@bpappin  did you create a ContentProvider using SugarORM easly?

I ended up having to fork the library, as the changes where too extensive to easily merge back in.
Once I decided to fork, I did not try to maintain backward compatibility.
I did make the fork public, in case you want to play with it, but you should consider it working but unstable.
See: https://bitbucket.org/sixgreen/sixgreen-cube
At some point in the future, I'll donate it back to this project and/or make it a more stable library.


[BUG] : Clutered Plot - Total tests performed

Describe the bug
The total tests performed plot has 5 digit (soon 6 digit) numbers. These overlap on the plot and makes the plot unreadable. This information is also duplicated by the information displayed when you hover on a point.
To Reproduce
Steps to reproduce the behavior:
Nothing required, just have to view the plot on the dashboard
Expected behavior
A clear , easy-to-read plot
Screenshots

Desktop (please complete the following information):

OS: Ubuntu 18.04 LTS
Browser : Firefox
Version 74 (6 bit)

Cleaned up a bit @Ari-Ramkilowan


Add missing patterns in the Semigroup instance for Alter a

This PR fixes non-exhaustive patterns in the Semigroup instance for Alter a.

Looks good.


fix typo in table api doc

Hey @nt591, thanks for the PR, sorry its not obvious but these API docs are auto generated from the source code. If you could make the change here: https://github.com/facebook/fixed-data-table/blob/master/src/FixedDataTable.react.js#L237 it wont overwritten the next time we build the docs :)

Fixed via #54 :)