High latency due to dns refresh on https every x minutes

Elastic.Clients.Elasticsearch version:  8.15.8
Elasticsearch version: 8.14.0
.NET runtime version: 8.x
Operating system version: Linux
Description of the problem including expected versus actual behavior:
Depending on parameter DnsRefreshTimeout we see longer requests every X minutes.
I attached example with settings 10 minutes and 5 minutes so there is strong correlation
The issue doesn't occure on HTTP connection its only on HTTPS.
Also this is new index/cluster so there is not many request to it yet.
I would provide more diagnostic details if you tell me what to check. All I know its caused by Https and DNS refresh
but i just can't find any proof its due to client or some network configuration issue. ever
We had no issue like that with older elastic versions like 6
Steps to reproduce:
0. Its hard to reproduce

There should be enabled https
DnsRefreshTimeout set to some value like 5 minutes
.Run the same request in loop for idk 1h and you will see huge spikes every 5 minutes

Expected behavior
No timeouts/long requests during queries to elatic
Provide ConnectionSettings (if relevant):
Provide DebugInformation (if relevant):

Hi @waszak, I'll transfer this issue over to the https://github.com/elastic/elastic-transport-net repository.


CircleCI caching on the first commit of a PR does not pick up the cache from master as it should

That means that the build of the first commit is very long 15-20 minutes but the following ones (providing the first build succeeded and upload its cache with the branch name)
https://app.circleci.com/pipelines/github/INRIA/scikit-learn-mooc/1346/workflows/125ad0b9-f1bf-49a2-90da-676a55be16a9/jobs/1349

It should work in principle according to the doc: https://circleci.com/docs/2.0/caching/#restoring-cache but maybe there is some subtlety there ...

I figured this one out you need to enable "Pass secrets to build from forked pull requests" in the CircleCI advanced settings:

The caching isolation (depending on the repo where the PR is originating from) is mentioned very briefly here:
https://circleci.com/docs/2.0/oss/#caching

So I enabled "Pass secrets to build from forked pull requests"
From the CircleCI doc this allows sharing this:

Environment variables set through the application.
Deployment keys and user keys.
Passphraseless private SSH keys you have added to CircleCI to access arbitrary hosts during a build.
AWS permissions and configuration files.

I think that's fine but ccing @ogrisel and @glemaitre for visibility.


typescript-language-features remote ssh crash

Type: Bug
This was an annoyingly difficult issue to track down as due to the builtin typescript-language-features.  I do NOT need to have this fixed as I can disable the extension.  It would be nice to have fixed so I can run the stock builtin extensions.  Posting here in case someone else has similar issue.  They might be able to see a pattern with mine.
Reproducing

In shell create new folder t1.
Create file z.js with one line in it.
Open code using remote ssh.
Open folder t1.
Open z.js in code.

Wait a few seconds.  Get notification of:
Remote Extension host terminated unexpectedly 3 times within the last 5 minutes.
If I disable the buildin extension typescript-language-features then it loads with no issues.
Log message in Output - Server has:
2024-05-04 13:11:43.726 [info] Last EH closed, waiting before shutting down
2024-05-04 13:12:03.761 [error] [<unknown>][ea8b9349][ExtensionHostConnection] Unknown reconnection token (seen before).
2024-05-04 13:12:03.853 [info] [<unknown>][5eb9842f][ExtensionHostConnection] New connection established.
2024-05-04 13:12:03.863 [info] [<unknown>][5eb9842f][ExtensionHostConnection] <111457> Launched Extension Host Process.
2024-05-04 13:12:04.710 [info] [<unknown>][5eb9842f][ExtensionHostConnection] <111457> Extension Host Process exited with code: null, signal: SIGSEGV.
2024-05-04 13:12:04.711 [info] Cancelling previous shutdown timeout
2024-05-04 13:12:04.711 [info] Last EH closed, waiting before shutting down

I turned on typescript logfile with the following setting in settings.json:
 "typescript.tsserver.log": "verbose"

I will attach a tsserver logfile from this.  There is an exception there about executing command and message about "Error: write EPIPE"
I have tried both vscode version 1.89.0 and code - insiders.  The vscode works okay in Remote SSH to other ubuntu servers.   Just this docker instance has an issue.  Remote (1and1 hosted) and local (docker hosted) installed similarly with ansible install scripts.  There could be differences due to 1and1 hosted image vs docker initial image. (?)
I have tried disabling all extensions (Except Remote ssh).
VS Code version: Code - Insiders 1.90.0-insider (Universal) (816c6f7d7bc4da9796cd19ac6b122b6e26f50d7f, 2024-05-03T17:56:40.522Z)
OS version: Darwin x64 23.4.0
Modes:
Remote OS version: Linux x64 6.6.22-linuxkit

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-1068NG7 CPU @ 2.30GHz (8 x 2300)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
3, 4, 4

Memory (System)
32.00GB (0.08GB free)

Process Argv
--crash-reporter-id 10fe963a-6fd1-43cd-bb5d-2c14a220c1b1

Screen Reader
no

VM
0%

Item
Value

Remote
SSH: dev.docu.team

OS
Linux x64 6.6.22-linuxkit

CPUs
Intel(R) Core(TM) i7-1068NG7 CPU @ 2.30GHz (8 x 2304)

Memory (System)
7.66GB (4.80GB free)

VM
0%

Extensions (3)

Extension
Author (truncated)
Version

remote-ssh
ms-
0.112.2024042515

remote-ssh-edit
ms-
0.86.0

remote-explorer
ms-
0.5.2024031109

A/B Experiments
vsliv368cf:30146710
vspor879:30202332
vspor708:30202333
vspor363:30204092
vsc_aacf:30263846
vscod805:30301674
vsaa593:30376534
py29gd2263:31024238
c4g48928:30535728
a9j8j154:30646983
962ge761:30841072
pythongtdpath:30726887
welcomedialog:30812478
pythonidxpt:30768918
pythonnoceb:30776497
asynctok:30898717
dsvsc013:30777762
dsvsc014:30777825
dsvsc015:30821418
pythontestfixt:30866404
pythonregdiag2:30926734
pyreplss1:30879911
pythonmypyd1:30859725
pythoncet0:30859736
h48ei257:31000450
pythontbext0:30879054
accentitlementst:30870582
dsvsc016:30879898
dsvsc017:30880771
dsvsc018:30880772
cppperfnew:30980852
ccp2r3:30958153
pythonait:30973460
showvideot:31016890
chatpanelt:31014475
724cj586:31013169
pythoncenvpt:31022790
a69g1124:31018687
j0bi8933:31021562
dvdeprecationcf:31036533
pythonprt:31036556
dwnewjupytercf:31035177

@sheetalkamat Any ideas what would cause this exception:
Info 40   [13:04:59.294] TIAdapter:: Scheduling request for: /home/ectown/t1/jsconfig.json
Info 41   [13:04:59.295] event:
    {"seq":0,"type":"event","event":"projectLoadingFinish","body":{"projectName":"/home/ectown/t1/jsconfig.json"}}
Info 42   [13:04:59.298] event:
    {"seq":0,"type":"event","event":"telemetry","body":{"telemetryEventName":"projectInfo","payload":{"projectId":"fd8df32e40d854c1593aaf349b9ba040a6cda4e77181d5febce07669c49557dd","fileStats":{"js":1,"jsSize":18,"jsx":0,"jsxSize":0,"ts":0,"tsSize":0,"tsx":0,"tsxSize":0,"dts":18,"dtsSize":1656617,"deferred":0,"deferredSize":0},"compilerOptions":{"allowJs":true,"maxNodeModuleJsDepth":2,"allowSyntheticDefaultImports":true,"skipLibCheck":true,"noEmit":true,"module":"commonjs","target":"es6"},"typeAcquisition":{"enable":true,"include":false,"exclude":false},"extends":false,"files":false,"include":false,"exclude":true,"compileOnSave":false,"configFileName":"jsconfig.json","projectType":"configured","languageServiceEnabled":true,"version":"5.4.5"}}}
Info 43   [13:04:59.299] event:
    {"seq":0,"type":"event","event":"configFileDiag","body":{"triggerFile":"/home/ectown/t1/z.js","configFile":"/home/ectown/t1/jsconfig.json","diagnostics":[]}}
Info 44   [13:04:59.300] Project '/home/ectown/t1/jsconfig.json' (Configured)
Info 44   [13:04:59.300] 	Files (19)

Info 44   [13:04:59.300] -----------------------------------------------
Info 44   [13:04:59.300] Open files:
Info 44   [13:04:59.300] 	FileName: /home/ectown/t1/z.js ProjectRootPath: /home/ectown/t1
Info 44   [13:04:59.300] 		Projects: /home/ectown/t1/jsconfig.json
Perf 44   [13:04:59.300] 2::updateOpen: elapsed time (in milliseconds) 986.2554
Info 45   [13:04:59.301] response:
    {"seq":0,"type":"response","command":"updateOpen","request_seq":2,"success":true,"performanceData":{"updateGraphDurationMs":936.4607949852943},"body":true}
Err 46    [13:04:59.319] Exception on executing command unknown:

    write EPIPE

    Error: write EPIPE
        at target._send (node:internal/child_process:872:20)
        at target.send (node:internal/child_process:745:19)
        at IpcIOSession.writeMessage (/home/ectown/.vscode-server-insiders/cli/servers/Insiders-816c6f7d7bc4da9796cd19ac6b122b6e26f50d7f/server/extensions/node_modules/typescript/lib/tsserver.js:189210:15)
        at IpcIOSession.send (/home/ectown/.vscode-server-insiders/cli/servers/Insiders-816c6f7d7bc4da9796cd19ac6b122b6e26f50d7f/server/extensions/node_modules/typescript/lib/tsserver.js:185763:10)
        at IpcIOSession.event (/home/ectown/.vscode-server-insiders/cli/servers/Insiders-816c6f7d7bc4da9796cd19ac6b122b6e26f50d7f/server/extensions/node_modules/typescript/lib/tsserver.js:185772:10)
        at IpcIOSession.event (/home/ectown/.vscode-server-insiders/cli/servers/Insiders-816c6f7d7bc4da9796cd19ac6b122b6e26f50d7f/server/extensions/node_modules/typescript/lib/tsserver.js:189180:15)
        at IpcIOSession.defaultEventHandler (/home/ectown/.vscode-server-insiders/cli/servers/Insiders-816c6f7d7bc4da9796cd19ac6b122b6e26f50d7f/server/extensions/node_modules/typescript/lib/tsserver.js:185654:14)
        at _ProjectService.eventHandler (/home/ectown/.vscode-server-insiders/cli/servers/Insiders-816c6f7d7bc4da9796cd19ac6b122b6e26f50d7f/server/extensions/node_modules/typescript/lib/tsserver.js:185557:83)
        at getOrCreateFileWatcher (/home/ectown/.vscode-server-insiders/cli/servers/Insiders-816c6f7d7bc4da9796cd19ac6b122b6e26f50d7f/server/extensions/node_modules/typescript/lib/tsserver.js:181149:15)
        at Object.watchDirectory (/home/ectown/.vscode-server-insiders/cli/servers/Insiders-816c6f7d7bc4da9796cd19ac6b122b6e26f50d7f/server/extensions/node_modules/typescript/lib/tsserver.js:181125:12)
        at watchDirectory (/home/ectown/.vscode-server-insiders/cli/servers/Insiders-816c6f7d7bc4da9796cd19ac6b122b6e26f50d7f/server/extensions/node_modules/typescript/lib/tsserver.js:121585:67)
        at Object.watchDirectory (/home/ectown/.vscode-server-insiders/cli/servers/Insiders-816c6f7d7bc4da9796cd19ac6b122b6e26f50d7f/server/extensions/node_modules/typescript/lib/tsserver.js:121653:92)
        at createDirectoryWatcherWithLogging (/home/ectown/.vscode-server-insiders/cli/servers/Insiders-816c6f7d7bc4da9796cd19ac6b122b6e26f50d7f/server/extensions/node_modules/typescript/lib/tsserver.js:121638:44)
        at Object.watchDirectory (/home/ectown/.vscode-server-insiders/cli/servers/Insiders-816c6f7d7bc4da9796cd19ac6b122b6e26f50d7f/server/extensions/node_modules/typescript/lib/tsserver.js:121603:283)
        at ConfiguredProject2.watchTypeRootsDirectory (/home/ectown/.vscode-server-insiders/cli/servers/Insiders-816c6f7d7bc4da9796cd19ac6b122b6e26f50d7f/server/extensions/node_modules/typescript/lib/tsserver.js:178911:45)
        at createTypeRootsWatch (/home/ectown/.vscode-server-insiders/cli/servers/Insiders-816c6f7d7bc4da9796cd19ac6b122b6e26f50d7f/server/extensions/node_modules/typescript/lib/tsserver.js:128548:60)
        at /home/ectown/.vscode-server-insiders/cli/servers/Insiders-816c6f7d7bc4da9796cd19ac6b122b6e26f50d7f/server/extensions/node_modules/typescript/lib/tsserver.js:20160:21
        at Set.forEach (<anonymous>)

We've seen a few reports of folks hitting EPIPE, typically on linux or windows


zipkin service not access, response HTTP 404

Describe the bug
request zipkin service, response HTTP 404

Expected behavior
{{ What did you expect to happen? }}
Steps to reproduce the bug
kubectl apply -f istio-1.0.0/install/kubernetes/helm/istio/templates/crds.yaml
kubectl apply -f install/kubernetes/istio-demo.yaml

wait all pod running. and access zipkin service
[root@k8s-master1 ~]# kubectl get services -n istio-system |grep zipkin
zipkin                     ClusterIP      <IP_ADDRESS>     <none>        9411/TCP                                                                                20d
[root@k8s-master1 ~]# curl http://<IP_ADDRESS>:9411 -i
HTTP/1.1 404 Not Found
Content-Type: text/plain; charset=utf-8
X-Content-Type-Options: nosniff
Date: Mon, 10 Sep 2018 08:35:06 GMT
Content-Length: 19

404 page not found

Version
[root@k8s-master1 ~]# kubectl version
Client Version: version.Info{Major:"1", Minor:"11", GitVersion:"v1.11.0", GitCommit:"91e7b4fd31fcd3d5f436da26c980becec37ceefe", GitTreeState:"clean", BuildDate:"2018-06-27T20:17:28Z", GoVersion:"go1.10.2", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"11", GitVersion:"v1.11.0", GitCommit:"91e7b4fd31fcd3d5f436da26c980becec37ceefe", GitTreeState:"clean", BuildDate:"2018-06-27T20:08:34Z", GoVersion:"go1.10.2", Compiler:"gc", Platform:"linux/amd64"}
[root@k8s-master1 ~]# is
isosize   istioctl
[root@k8s-master1 ~]# istioctl version
Version: 1.0.0
GitRevision: 3a136c90ec5e308f236e0d7ebb5c4c5e405217f4
User: root@71a9470ea93c
Hub: gcr.io/istio-release
GolangVersion: go1.10.1
BuildStatus: Clean

Is Istio Auth enabled or not?
not enabled
Environment
Linux k8s-master1 3.10.0-862.6.3.el7.x86_64 #1 SMP Tue Jun 26 16:32:21 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
CentOS7
Cluster state
{{ If you're running on Kubernetes, consider following the
instructions
to generate "istio-dump.tar.gz", then attach it here by dragging and dropping
the file onto this issue. }}

[root@k8s-master1 ~]# kubectl logs istio-tracing-7596597bd7-sckcm -f -n istio-system
{"level":"info","ts":1536376627.6249406,"caller":"healthcheck/handler.go:99","msg":"Health Check server started","http-port":14269,"status":"unavailable"}
{"level":"info","ts":1536376627.757525,"caller":"tchannel/builder.go:89","msg":"Enabling service discovery","service":"jaeger-collector"}
{"level":"info","ts":1536376627.7576222,"caller":"peerlistmgr/peer_list_mgr.go:111","msg":"Registering active peer","peer":"<IP_ADDRESS>:14267"}
{"level":"info","ts":1536376627.8226492,"caller":"standalone/main.go:179","msg":"Starting agent"}
{"level":"info","ts":1536376627.8947153,"caller":"standalone/main.go:219","msg":"Starting jaeger-collector TChannel server","port":14267}
{"level":"info","ts":1536376627.894826,"caller":"standalone/main.go:229","msg":"Starting jaeger-collector HTTP server","http-port":14268}
{"level":"info","ts":1536376628.848809,"caller":"standalone/main.go:290","msg":"Registering metrics handler with jaeger-query HTTP server","route":"/metrics"}
{"level":"info","ts":1536376628.8489106,"caller":"standalone/main.go:296","msg":"Starting jaeger-query HTTP server","port":16686}
{"level":"info","ts":1536376628.8489366,"caller":"healthcheck/handler.go:133","msg":"Health Check state change","status":"ready"}
{"level":"info","ts":1536376628.849857,"caller":"peerlistmgr/peer_list_mgr.go:157","msg":"Not enough connected peers","connected":0,"required":1}
{"level":"info","ts":1536376628.849926,"caller":"peerlistmgr/peer_list_mgr.go:166","msg":"Trying to connect to peer","host:port":"<IP_ADDRESS>:14267"}
{"level":"info","ts":1536376628.878678,"caller":"peerlistmgr/peer_list_mgr.go:176","msg":"Connected to peer","host:port":"[::]:14267"}
{"level":"info","ts":1536376629.678079,"caller":"standalone/main.go:249","msg":"Listening for Zipkin HTTP traffic","zipkin.http-port":9411}

@virnet where are you running this ? GKE ? Minikube ?
try accessing it from something like a busybox

The zipkin service is the endpoint for receiving zipkin compatible span data - but the backend is provided by Jaeger.
If you want to try out zipkin as an alternative, then you should use --set tracing.enabled=false on helm install, and then install zipkin separately in the relevant namespace (e.g. istio-system).

The aim is to provide alternative tracing backends at some point, so would be able to do --set tracing.provider=zipkin, but that is not currently supported. Contributions welcome :)

++

+++


how to use script to grant the report access to all the users

Hello,
Is anyway to achieve this, basically I am trying to implement RLS and there are thousands of users, it's impossible to manually add them to the report, is there quick way to add those users to particular report by  sql script?  Thanks!

This question is not related to this project, has nothing to do with security extension.
Nevertheless if you really want to do this:

Catalog stores your reports, folders
Roles stores the roles you want to set for users
Users holds your users that exists

Build a valid sql statement that fills PolicyUserRole table with valid data and your job is done.


Request: An indicator of the true cooldown speed on kitties

For all but Fast or Catatonic speeds, cooldowns have two possible time values: the faster version and the slower version.  It would be nice to be able to visually distinguish which is in effect on a kitty at a glance.
Would it be possible to add the actual time value after the cooldown name on inventory and detail pages, so instead of "Cooldown: Swift" it would say "Cooldown: Swift 2m" or "Cooldown: Swift 5m"?
(or just "Swift 2m" / "Swift 5m" on the inventory/marketplace pages?)
That would go a long way towards distinguishing which kitties are more valuable on a Marketplace search versus their competition at the same cooldown, and tell people more accurately how long the cooldown will actually be on each kitty when it is bred?
Thanks in advance for your consideration of this.  Love this extension!

I have implemented this in my private version that I use.
https://raw.githubusercontent.com/adolg/LeavingHarlemIF/master/CryptoKittyInfoExtension.js
It shows "gen5 2sw k0" (kids 0 - instead of "Virgin") on top of the cattributes column.
It also shows kitty's date and time of birth, and if you click on that it goes to marketplace
with the given gen, cooldown and the most rare cattribute as the filter.
So it's useful also for quickly evaluating parents/children and cats on kittyexplorer.com
(I also added kittyexplorer.com to the list of supported sites)


Filter accounts by (spoken) language

It would be great to be able to filter accounts by the language that they speak/write. To resolve this we'd need to do the following:

Find a Python library/3rd party API for detecting language from text.
Run the detection on people's BIOs and their posts and save the results in a filter table.
Add the UI to filter accounts/posts by spoken language.

We use openai to classify accounts. Maybe we could change it so that AI could also try to guess the users language: https://github.com/anze3db/fedidevs/blob/main/accounts/management/commands/classify.py


have s3 torch.Datasets cache tiles

Currently the Datasets responsible for tile loading open s3:// endpoints every time there's a __getitem__, which is slow and high-bandwidth. We should cache the files on first load into /tmp or something, and delete when the object is closed. Could put in __del__?

not gonna do this, EC2/S3 training is sufficiently quick + bandwidth is free.


review under folders or tags

It is inconvenient for me to set review: false on hundreds of notes since I have more than 2 thousands notes.
I only need to review certain notes under some folders or tags, especially tags.
Anyway ,it is a great plugin.

Hello, I'll work on fixing that in a few. I'll start with the tags.

I've added a mechanism to review only notes with certain tags.
This is available in the settings of the latest release. Do check it out.

This is a great update


[docs] Smaller select demos

[x] I have followed (at least) the PR section of the contributing guide.

Closes #21642
Same effort as #17483
I created smaller demos only for the Select component.
Please give some feedback so that I can proceed with all the other components.
Can you also tell me, when does the docs show smaller snippets from the demos and when it doesn't? From what is it dependent?

@cjoecker Thanks for working on it. A couple of early feedback:

You need to cherry-pick the commits on the next branch
I would move Advanced features higher in the page, developers should be able to quickly identify that they have this alternative.
We aim to remove the full capitalization from the headers: Filled and Outlined Variants -> Filled and Outlined variants.
We might want to group the states "Disabled", "Error", "Required", "Ready only". 4 seems fine.
In the "Native Select" we might want to use the uncontrolled API to shrink the demo to enable "inline preview".

Do I normally need to do a cherry-pick from next branch? Should then we add it the Readme.md?
When are the inline previews shown? From what is it dependent?

Do I normally need to do a cherry-pick from next branch?

If you start your work from the master branch, yes. Sebastian has fixed the CONTRIBUTING.md file recently.

When are the inline previews shown? From what is it dependent?

The render block has to have less than 15 lines.


Unable to install with Laravel 5.4.*

I'm trying to get this added to my Laravel 5.4 install using the following command:
composer require phpzen/laravel-rbac
The output of this command is:
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Installation request for phpzen/laravel-rbac 0.2 -> satisfiable by phpzen/laravel-rbac[0.2].
    - Conclusion: remove laravel/framework v5.4.15
    - Conclusion: don't install laravel/framework v5.4.15
    - phpzen/laravel-rbac 0.2 requires illuminate/support 5.2.* -> satisfiable by illuminate/support[v5.2.0, v5.2.19, v5.2.21, v5.2.24, v5.2.25, v5.2.26, v5.2.27, v5.2.28, v5.2.31, v5.2.32, v5.2.37, v5.2.43, v5.2.45, v5.2.6, v5.2.7].
    - don't install illuminate/support v5.2.0|don't install laravel/framework v5.4.15
    - don't install illuminate/support v5.2.19|don't install laravel/framework v5.4.15
    - don't install illuminate/support v5.2.21|don't install laravel/framework v5.4.15
    - don't install illuminate/support v5.2.24|don't install laravel/framework v5.4.15
    - don't install illuminate/support v5.2.25|don't install laravel/framework v5.4.15
    - don't install illuminate/support v5.2.26|don't install laravel/framework v5.4.15
    - don't install illuminate/support v5.2.27|don't install laravel/framework v5.4.15
    - don't install illuminate/support v5.2.28|don't install laravel/framework v5.4.15
    - don't install illuminate/support v5.2.31|don't install laravel/framework v5.4.15
    - don't install illuminate/support v5.2.32|don't install laravel/framework v5.4.15
    - don't install illuminate/support v5.2.37|don't install laravel/framework v5.4.15
    - don't install illuminate/support v5.2.43|don't install laravel/framework v5.4.15
    - don't install illuminate/support v5.2.45|don't install laravel/framework v5.4.15
    - don't install illuminate/support v5.2.6|don't install laravel/framework v5.4.15
    - don't install illuminate/support v5.2.7|don't install laravel/framework v5.4.15
    - Installation request for laravel/framework (locked at v5.4.15, required as 5.4.*) -> satisfiable by laravel/framework[v5.4.15].

Installation failed, reverting ./composer.json to its original content.

Is anyone else having this error? What did you do to resolve?
Thanks,

Same issue here. Too bad i am unable to use this nice package.


Unable to initiate docker

Hi,
Tried to initiate the docker like in the example.
I think the example referring to ./docker-entrypoint.sh? Didn't find any Docker file to run.
Also when did that, the script prompt:

Help would be very much appreciated!

Hi,
To clarify the structure of the repository - Dockerfiles are within the folders, that refer to specific version of overpass-api.
Regardless to the error that you stumbled on - yes the message comes from docker-entrypoint.sh, but as you didn't show full commandline, with which you tried to start, it's hard for me to reproduce it.
Chekcing the code, it looks like this line if failing:
echo "# Netscape HTTP Cookie File" > /db/cookie.jar

Which could fail only, if /db directory didn't exist. Can you tell more about your environment? Which docker runtime you're using and what kind of filesystem is under /data/transfer/open_street_map/overpass_clone_db/ ? Does this directory exists? If not, do you (or docker daemon user) have rights to create this directory?

Thank you for the fast reply!
First time raising an env.
So I might did some mistakes.
I thought the script will create a docker for me.
I just clone the git to /data/transfer/open_street_map.
Then I created the /data/transfer/open_street_map/overpass_clone_db/ directory and tried to run the script.
From your reply I'm guessing I missed a stage?

the environment is a private server. I created the /db dir as you can see:

There are few things here:

If you want just to run the image, you don't need to clone the repository, just invoke

docker run [...]

As it is written in README. docker will automatically try to download the image from Docker Hub, where the image is hosted. Though right now, you'll need an account to download. You may create one on https://hub.docker.com/

If you don't want to use image from Docker Hub or just want to build the software yourself, clone the repo, choose the version of the overpass-api, such as 0.7.56, and then from the root of the repo:

cd 0.7.56
docker build -t overpass-api .
docker run [...]

But in the last line, replace wiktorn/overpass-api with overpass-api (the image that you've built yourself). Of course, you may use any name you find suitable (provided that it's allowed by docker).
PS. Please do not paste commands that you run as images as this prevents copying the command and running it myself. Use "insert code" from formatting menu

Hi,
Thank you very much for yours support!
Unfortunately, after checking my proxy settings, I noticed I am not getting any data from those sites:
Err:1 http://deb.debian.org/debian stretch InRelease
Could not connect to <IP_ADDRESS>:3128 (<IP_ADDRESS>), connection timed out
Err:2 http://security.debian.org/debian-security stretch/updates InRelease
Could not connect to <IP_ADDRESS>:3128 (<IP_ADDRESS>), connection timed out
Err:3 http://deb.debian.org/debian stretch-updates InRelease
Unable to connect to <IP_ADDRESS>:3128:
Reading package lists...
W: Failed to fetch http://deb.debian.org/debian/dists/stretch/InRelease  Could not connect to <IP_ADDRESS>:3128 (<IP_ADDRESS>)
W: Failed to fetch http://security.debian.org/debian-security/dists/stretch/updates/InRelease  Could not connect to <IP_ADDRESS>
W: Failed to fetch http://deb.debian.org/debian/dists/stretch-updates/InRelease  Unable to connect to <IP_ADDRESS>:3128:
W: Some index files failed to download. They have been ignored, or old ones used instead.
I double checked it with my browser.
Further more, our Devops guy told me that we are using Ubuntu and this Debian dist won't be compatible. Is it hopeless to try? Is there an Ubuntu version for that image?
Also, I looked online for an Ubuntu version and found a version of a tile server. In the requirements it was said that the data is 1TB for the whole planet. Is it true?

From the error messsage it looks like you have misconfigured your proxy server or it's unreachable from your docker container. Switching to ubuntu would not change this problem.
There is no ubuntu version of this image. It shouldn't be a problem to change the base image from nginx to Ubuntu. For sure you'll need:

adapt package names that are installed (AFAIR, not all have the same naming)
configure nginx (see what's done in official nginx image

And maybe adapt some paths (not sure, if they are the same on ubuntu and nginx based image).
1TB of space for the whole planet sound reasonable, though it depends also on the style you want to use. For running overpass for whole planet, you will also need like few hundred GB for its database.

I followed those instructions and managed to change the proxy, and build successfully. Thank you!
Now, I wish to download parts of the world.
Specifically Michigan and Israel.
In the EX you showed, the example was with Monaco.
Could you please elaborate on other available places or link to a reference?
Also, is it possible to get 2 places in 1 image?

I followed those instructions and managed to change the proxy, and build successfully. Thank you!
Now, I wish to download parts of the world.
Specifically Michigan and Israel.
In the example you showed, the example was with Monaco. is it possible to get 2 places in 1 image?

No, it's not supported to include two areas in one docker container. If #43 would be solved, this could be done using polygons containing the areas that you're interested in, but still you would need to feed whole planet to it, so it could be longish to process.
Of course you can adapt this image to accept two sources of data and keeping your instance up-to-date. Overpass itself will not have a problem with that.

Thank you.
I'm trying to run the docker but I get "permission denied" when the script gets to line 44.
I deduct that my env has a problem with the chown command.
trying to do sudo doker run [...] or to add sudo before chwon didn't help and neither the internet. Got any suggestion?


update README to use Spark 2.1

Keeping up with the times. Also rsparkling now supports more than just 1.6.

Thanks! This was a bit overdue!


[PYIC-3472] Refactor common-services files into cimit-services:

Proposed changes
To carve off classes from the over-subscribed common-services:

What changed
Move the following classes to cimit-services:

CiMitJwt
CiMitVc
EvidenceItem
ContraIndicator
ContraIndicators
Mitigation
MitigatingCredential

Why did it change

Reduce dependency redundancy and, with it, unneeded lambda redeploys during dev work on unrelated code

Issue tracking

PYIC-3472

Checklists
Environment variables or secrets

[ ] No environment variables or secrets were added or changed

I should move to common-services...gpg45.domain

Kudos, SonarCloud Quality Gate passed!
  0 Bugs   0 Vulnerabilities   0 Security Hotspots   0 Code Smells
 15.9% Coverage  0.0% Duplication

Check coverage seems low.


feat(logging): Add the new custom process output class and inject it into the default list renderer options

Description

Add the new custom process output class and inject it into the default list renderer options
Fix warnings in IDE - ( type mismatches, etc. )
Add delay on the Node.checkNetworkNodeActiveness since it prevents the gRPC service error

Related Issues

Closes #

581
546

:tada: This PR is included in version 0.31.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Update to work on Windows

I haven't tested this on Linux/OS X yet, but the changes should be superficial for those platforms.  Will run through some tests later today but take a look now.

Thanks!

Sorry for commenting on a closed issue but do you plan on republishing the latest changes to puppet's forge (https://forge.puppetlabs.com/maestrodev/wget)? Your last release is from May 1st. I would need this to get the "Could not find command 'test'" issue fixed.


Service can't be deployed if thats an exist user on Pterodactyl.

Respond from module log
Array
(
    [errors] => Array
        (
            [0] => Array
                (
                    [code] => unique
                    [detail] => The email has already been taken.
                    [source] => Array
                        (
                            [field] => email
                        )

                )

        )

    [status_code] => 422
)

Module Command Error

Fixed in https://github.com/TrixterTheTux/Pterodactyl-WHMCS/commit/ba64afba19408e89a71bdb929770bcdd892098d0


Lift requirement to create streams in order

In swift-nio-http2 today we have an unspoken hard requirement, which is that if you call createStreamChannel multiple times you must send the first write on each of those channels in the order you created them. This is because createStreamChannel allocates a stream ID immediately for each of those channels, but that allocation isn't meaningful until we try to write to the network. If you write out of order, the later streams will use the higher stream ID allocated to them, and will implicitly retire the lower IDs used by the earlier channels. When those earlier channels try to send, the core state machine will reject them for violating stream ID ordering.
After discussing with @glbrntt we think this is made up of several changes:

[x] Define a new internal protocol: HTTP2FrameConvertible. This is a protocol that encapsulates creating HTTP2Frames from a given object and vice-versa. We will have two conforming types: HTTP2Frame (trivial) and HTTP2Frame.Payload (straightforward, but not trivial). (#216)
[x] Define a new protocol: HTTP2FramePayloadConvertible. This is the inverse of the previous protocol: creates a payload from the given object, or vice versa. (#216)
[x] Make HTTP2StreamChannel generic over an outbound payload type conforming to HTTP2FrameConvertible. This will become the inbound and outbound message type. This requires updates to the various inbound and outbound reading and writing functions. This should not require any interface changes on the interface between the multiplexer and the stream. The multiplexer will need to be updated to hold the new type (HTTP2StreamChannel<HTTP2Frame>). (#218)
[x] Define a struct that provides the interface HTTP2StreamMultiplexer expects from HTTP2StreamChannel, that holds the channel privately. This should be backed by an enum, which we will use later to add a new case for the new channel type. Refactor HTTP2StreamMultiplexer to use this type. (#215)
[x] Update HTTP2StreamChannel to tolerate its streamID being optional, and nil on initialisation. This requires a new function on HTTP2StreamMultiplexer to get the next stream ID, which is currently done in createStreamChannel. (#217)
[x] Add a new createStreamChannel function with a new initializer that no longer gets a stream ID. Plumb through this initializer to create new stream channels that use HTTP2Frame.Payload instead of HTTP2Frame. Update the enum created above for the new case. (#221)
[ ] Update the codecs to become generic over HTTP2FramePayloadConvertible, and rename them. Create typealiases for both the old version (keeping its old name) and the new one that uses HTTP2FramePayload. Deprecate the old typealias. (#222)
[ ] Deprecate the old interface, including replacing anything in the HTTP2PipelineHelpers function to use the new initializer.

@sweetbot there are still outstanding tasks to resolve this issue before we can tag a release of SwiftNIO HTTP/2. gRPC will then need to pick up the new version and any new changes that come with it. My hope is that we can tag a gRPC release by the end of the week.


WiX: package up macro support libraries

Add the swift syntax runtime libraries and swiftmodules to enable macros on Windows.

Tested at https://ci-external.swift.org/job/swift-PR-build-toolchain-windows/1010/


add Android apps

@osmlab Is there a chance for reviewing this PR?

Thanks for your contribution @abonte! 👍🏼


Add Dashboard

As a student, I want to view a breakdown of how much I owe versus how much others owe me, so that I can easily track my finances.

Story:
#24 #18


BOM?

Would love a hands down bill of materials for this. Just got my 5 PCB's that I ordered!
The smaller ones look like 2.54mm (0.1")spacing but are the larger ones 5.08mm (0.2") or 3.96mm (0.156")?

Hi Erik, sorry that I haven't upload a BOM, I will make it later, I'm not with my notebook.
Yes, smaller ones are 2.54 mm pitch, like Arduino connections and larger ones are terminal blocks with 5.08 mm pitch. thanks for your support and give me a feedback when you test boards

I would also appreciate a BOM as I'm currently waiting for a review and reply of my order on PCBWay.

I started writing down some that I found, not sure about all the parts, some might not be needed based on what configuration you are running with!
@stefschin if I populate r4 and r5 for the analog sensor (P3) do I still need to use a ADS1115 or is that only needed if I want to use P11?
Resistors:
r1: 4.7k (1-wire)
r2: 4.7k
r3: 4.7k
r4: 10k (adc)
r5: 20k (adc)
r6: 10k (buzzer)
So r4 & r5 is only to make a voltage divider for example a pressure sensor. r6 is if using a buzzer with a mosfet
Not sure about r2 and r3, seems to be for some extra i2c?
Buzzer: 5v, 12mm outer diameter. Leg distance 7.6mm
1x 2N7000 or 2N7002 Mosfet transistor channel N
3x screw terminals 5.08mm spacing, 3-pin (p1, p2, p3)
1x screw terminal 5.08mm spacing 1-pin (J1)
1x pin header, 2.54mm spacing, 1x2 pin (jp1)
1x pin header, 2.54mm spacing, 1x6 pin (p5)
1x pin header, 2.54mm spacing, 1x5 pin (p6)
1x pin header, 2.54mm spacing, 1x3 pin (p7)
3x pin header, 2.54mm spacing, 1x4 pin (p8, p9, p11)
1x IDC male 10-pin (p10)

I started writing down some that I found, not sure about all the parts, some might not be needed based on what configuration you are running with!
@stefschin if I populate r4 and r5 for the analog sensor (P3) do I still need to use a ADS1115 or is that only needed if I want to use P11?
Resistors:
r1: 4.7k (1-wire)
r2: 4.7k
r3: 4.7k
r4: 10k (adc)
r5: 20k (adc)
r6: 10k (buzzer)
So r4 & r5 is only to make a voltage divider for example a pressure sensor. r6 is if using a buzzer with a mosfet
Not sure about r2 and r3, seems to be for some extra i2c?
Buzzer: 5v, 12mm outer diameter.

I started writing down some that I found, not sure about all the parts, some might not be needed based on what configuration you are running with!
@stefschin if I populate r4 and r5 for the analog sensor (P3) do I still need to use a ADS1115 or is that only needed if I want to use P11?
Resistors:
r1: 4.7k (1-wire)
r2: 4.7k
r3: 4.7k
r4: 10k (adc)
r5: 20k (adc)
r6: 10k (buzzer)
So r4 & r5 is only to make a voltage divider for example a pressure sensor. r6 is if using a buzzer with a mosfet
Not sure about r2 and r3, seems to be for some extra i2c?
Buzzer: 5v, 12mm outer diameter.
Thanks answer to other guys with tentative BOM that's ok just J1 it's 2-pin screw terminal 5.08mm spacing
As you say R2 and R3 are Pull ups for I2C if they are note populated in your I2C hardware, what's usual.
R4 and R5 are for setting a Voltage divider to supply ADC input of ESP-32.
Attenuation calculation is AT=(R4+R5)/R5, it's better to use resistores in order of 10k until 500k.
For example if you use R4=10k and R5=20k AT=30k/20k=1.5 and if your max input is 5V, your ADC input will be: 5V/1.5=3.3V
If you want to use ADS1115, it has I2C communication, so I would connect it to P11 and make a short between 2 and 3 of JP2 to supply board with 3.3V.
Any other doubt fill free to ask. Thanks for testing this board!

BOM in pdf and xlsx added

BOM released


NOP

@rpetit
I will delete this branch in 7 days because it's rather outdated. Let me know if you think otherwise.

i think this was actually merged in


Autoplay problem.

Hi!
I am trying to automatically play the HLS stream.
(version 0.2.97
autoPlay: true
mute: true)
Playback does not start automatically on android. But it starts well on the latest version of Clappr.
Please tell me what exactly is the difference so that I can come up with a workaround without switching to the new version?

I think latest "autoplay" modifications for 0.2.x was this PR and this PR.
(just did an autoplay keyword research in merged pull requests)


Add paging for collections

We have over two thousand collections in one database, and loading all the collections nearly causes the browser to lock-up.

Ha! Fair enough. I'll see what I can do :)

There's probably a similar problem with lots of databases, as well.

Would be an awesome bonus to be able to just search for collections as well.

Agreed. I've generally had a pretty manageable number of databases and collections, so I've never had to scratch that particular itch. But I can definitely see where it'd be useful.

@bobthecow hey Justin. Any progress on paging for collections?

I haven't had a chance to work on this yet. As a workaround, if you build a version of genghis that doesn't pull collection stats you'll probably mitigate most of the issues (other than searchability, of course).
For PHP, comment out this line:
https://github.com/bobthecow/genghis/blob/master/src/php/Genghis/Models/Collection.php#L259
For Ruby, comment out this line:
https://github.com/bobthecow/genghis/blob/master/src/rb/genghis/models/collection.rb#L114
… everything should still work, other than the collection stats popup, but it should also be a lot faster with that many collections.


catch SIGINT signal

Notice
If you need support or clarification regarding the usage of Drogon in your project, visit the official Drogon support channel at gitter
Please create a new issue only if you think you have found a bug or if have a feature request/enhancement.
Is your feature request related to a problem? Please describe.
It would be nice if Drogon would catch the SIGINT and allowed me to run a subroutine.
Describe the solution you'd like
catch SIGINT and end program without warnings.
Describe alternatives you've considered
I am not sure how to do this in c but in a shell script I would run something like trap on SIGINT.
Additional context

Drogon has this member function:
HttpAppFramework &setIntSignalHandler(
        const std::function<void()> &handler)

I never try, but something like this should work:
app().setIntSignalHandler(my function);
or even
app().setIntSignalHandler([&]{std::cerr << "Alarm! SIGINT!"});

I added this to main but I am still seeing the warning. Is there a way to get rid of the warning?

Also I have a follow up question, after I print out std::cout "good bye" how can I exit. I tried exit and return 0 to no avail.  How do I get my terminal back. Please advice.
Thanks.

Would you like drogon stop running and exit? I suppose app().quit(); will do this work (stop all threads , plugins and return to main (where you called it from))

Perfect! That worked thanks.


Either fix to work with both 2 and 3 or clearly specify that it only works with 2.?

I just spent a couple of hours discovering that this library only works with Python 2. I only tried it with 2.7 so I don't know which particular versions of 2 it works with.
The library installs without any problem on Python 3.5, and can be imported, but doesn't work.

We're working on version 1.0.0, which will be compatible with both Python 2 and 3. It'll be released in a few weeks, but it's already usable (branch 1.0.0, but please take into account the docs haven't been updated yet).
Sorry for the inconvenience.

Fixed in branch 1.0.0
CI is enabled and running tests over Python 2.7, 3.6 and nightly builds


Add DEBUG and NOTICE severities

I have some logging in a different program that I would like to move from INFO to DEBUG or NOTICE. This PR allows that by adding the those severities.

While I appreciate that you are adding the DEBUG level, and know that NOTICE exists in syslog, what is the benefit of having the latter?
So far, we haven't used the NOTICE level in Mailgun code, as far as I know, and I want to note that this level doesn't exist in the Python Logging Facility. Semantically it doesn't seem much different from INFO.

So I don't specifically require it ATM but it was easy to add at the same time and I've worked with other products that use it and have liked it. Its meant for entries that people should pay attention to but aren't technically errors. An example of this might be something like connection reset. I want to know that it happened but by itself ifs not really an error condition and the program just deals with it.

Right, I'm sure we could use the level, but given that Python apps can't express it, I'd like to suggest to keep with the common denominator. I also don't think in practice we are very consistent in the use of levels (yet?) and adding one to the mix doesn't seem helpful at this point. YMMV.
For completeness, Python also doesn't have ALERT or EMERG.

Make sense and I don't have that strong of an opinion about it. I dropped the NOTICE level support out of the PR. Maybe we can revisit it at a latter time if there is a stronger use case.

👍

lgtm


variable and interactive window test fixes

For #13269

[x] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
[x] Title summarizes what is changing.
[x] Has a news entry file (remember to thank yourself!).
[ ] Appropriate comments and documentation strings in the code.
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[x] Unit tests & system/integration tests are added/updated.
[ ] Test plan is updated as appropriate.
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed).
[ ] The wiki is updated with any design decisions/details.

Codecov Report

Merging #13272 into master will decrease coverage by 59.80%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master   #13272       +/-   ##
===========================================
- Coverage   59.80%    0.00%   -59.81%
===========================================
  Files         670       14      -656
  Lines       37172       38    -37134
  Branches     5265        3     -5262
===========================================
- Hits        22231        0    -22231
+ Misses      13813       38    -13775
+ Partials     1128        0     -1128

Impacted Files
Coverage Δ

src/client/debugger/constants.ts
0.00% <0.00%> (-100.00%)
:arrow_down:

src/client/common/process/constants.ts
0.00% <0.00%> (-100.00%)
:arrow_down:

src/client/common/platform/constants.ts
0.00% <0.00%> (-100.00%)
:arrow_down:

src/client/datascience/notebook/constants.ts
0.00% <0.00%> (-100.00%)
:arrow_down:

src/client/interpreter/autoSelection/constants.ts
0.00% <0.00%> (-100.00%)
:arrow_down:

...t/datascience/jupyter/jupyterDataRateLimitError.ts
0.00% <0.00%> (-50.00%)
:arrow_down:

...ction/interpreterSecurity/interpreterEvaluation.ts

src/client/telemetry/importTracker.ts

...ication/diagnostics/checks/pythonPathDeprecated.ts

...ce/interactive-window/interactiveWindowProvider.ts

... and 652 more

Continue to review full report at Codecov.

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


Optional formatting parameter for log

As title, for replay logging.
This is probably a horrible way of doing this, but I'm not really familiar with Rust. It works though.

hmm yeah ok


minor fixups

@richbrowne
This PR has a few minor fixups to ease testing.  The changes in test_esd remove the use of the 'Iterator.next()' in favor of the simpler pattern of indexing into the list of services.
The practical effect is that test developers can now remove sections of test code without changing the meaning of the "service" variable in subsequent portions of the code.
The only other change is dedentation of a few lines to shut flake8 up.
The third commit builds on the previous two to refactor the tests such that they have proper setup and teardown functionality.   These setups and teardowns are not the ideal of booting snapshotted vms, but they do modularize the tests and speed development in the short term.

test this please

retest this please


Better errors from Web APIs

Right now, the RPC library (pauls-electron-rpc) does a really bad job of constructing the error objects. Ideally, they'd be reconstructed in full. Can we fix that?

Solved with a somewhat janky, eval-based solution, here: https://github.com/pfrazee/pauls-electron-rpc/commit/9e6006b0d21e4c378ad64557bb087193ec550814


redundant module 'buffer' in web environment

When used webpack to pack js files for web instead of nodejs, i found some redundant modules.
See the image below:

and the source code:

upgrade to v2.5.1, then resolve this problem.


Licensing Choice

Setting up this issue for discussion of if we should change our licensing in our Python and R issue. Feel free to leave your thoughts and discuss.

Having taken a look at the lecture 8 material, I would agree with Frank, who said on the call yesterday, in sticking with the MIT license.
Since our package seems more for our own purposes of software development and workflows conceptual understanding MIT seems to fit what we require according to https://choosealicense.com. People can do what they want with it, but we have limitation of liability.
If we go this route, I think we will need to add everyone's names into the License though.

Agreed with your comment above @micahkwok.
I, too, think that the MIT license is more in line with our intentions and the purpose of this package.
So, I propose to not change the license and continue using the MIT license and just update it with our names and the year in the dedicated place in the license file.

For documentation purposes:
It was agreed to by all members of the group that we will simply stick with the MIT license in our weekly meeting. The purposes are already listed above in this thread.
I have added the names to our license in the latest PR request for both Python and R packages.


Testing solution for issue #195

Trying a way to resolve the datasets issue #195. Will need to raise a PR for this to test out the solution

I am not experienced with setuptools and thus cannot comment on the right approach here. Removing me as reviewer here :)

@mhajij  This PR is not ready yet.
I initially raised this PR so as to be able to test my solutions. Converting this to draft for now.

@mhajij did you mean to close this?

@mhajij did you mean to close this?

@ffl096, I reopened it.

setuptools-scm has been added to the build process with #381.


Use background task to batch-verify sr25510 signatures with optimum size

Are you sure there is an optimum size?  I'd wishfully imagined https://github.com/dalek-cryptography/curve25519-dalek/pull/249 made arbitrarily large efficient?  Is this CPU cashing issues with large Vecs?


Update plans that include EDM-based classification

From following information and actual portal UI, latest plan that include EDM-based classification is M365 E5 Compliance and M365 E5/A5 Information Protection and Governance.
Detailed Microsoft 365 Compliance Licensing Comparison (PDF).
https://docs.microsoft.com/office365/servicedescriptions/downloads/microsoft-365-compliance-licensing-comparison.pdf

@chrfox - PR is ready for final review.
thank you.


ENH: Ease programming the C++ interface for SimpleITK

The C++ interface for SimpleITK should provide typedefs
for common concepts used.

The typedef are a wider issue. In some ways, many of the initial developers wanted to see the "real" type of these items to make them less opaque. In other, I know there are currently some inconsistency with these types.
This will need additional consideration and discussion.

@blowekamp Should this old issue be closed, or put this on a backlog project?

This is a reasonable idea and it does clarify the types that are used in SimpleITK. And point out some inconsistency.
It is not clear to me if these types should be member of the Image class or just types in the itk::simpleITK namespace.
The philosophical conflict with this, is that one of the design choices for SimpleITK to was have the types used for SimpleITK be readily apparent and not hidden by typedefs. The practical usefulness comes when getting the doc info for a member or procedure in Python, I believe seeing std::vector<uint32_t> is more useful that IndexType. However, for programming and maintaining consistency the typedef would be easier.
If these typedefs are added, what would be the guidance on their usage? Should they be used in the SimpleITK interfaces primarily?
I am going to leave this topic open for discussion...

Personally I prefer the explicit types over the typedefs, reading the code is easier and it is clear what type is expected.
Using typedefs becomes second nature in ITK as everything is a template and it makes the length of a type name manageable and often more readable. On the other hand it often obfuscates the code.
In the current use case I don't think there is an advantage to using typedef.

@zivy @blowekamp OK.  I wanted to have parity between ITK and SimpleITK, but this is fine.  I'll cancel this pull request.

For the C++ programmer, I think it is reasonable to have an additional header file to "ease the c++ programming" to get more ITK style in C++. We already have an optional header file to add overloaded operators for C++.


Index Creation

Feature request
I noticed that the MongoDBChatMessageHistory class object is attempting to create an index during connection, causing each request to take longer than usual. Index Creation is one time process, So no need create index everytime. By default, index creation is enabled. To address this, add an additional parameter, index_creation. If index_creation is set to False, this step should be ignored.
Current Code:
def __init__(
        self,
        connection_string: str,
        session_id: str,
        database_name: str = DEFAULT_DBNAME,
        collection_name: str = DEFAULT_COLLECTION_NAME,
    ):
        from pymongo import MongoClient, errors

        self.connection_string = connection_string
        self.session_id = session_id
        self.database_name = database_name
        self.collection_name = collection_name

        try:
            self.client: MongoClient = MongoClient(connection_string)
        except errors.ConnectionFailure as error:
            logger.error(error)

        self.db = self.client[database_name]
        self.collection = self.db[collection_name]
        self.collection.create_index("SessionId")

Proposed modification:
def __init__(
        self,
        connection_string: str,
        session_id: str,
        database_name: str = DEFAULT_DBNAME,
        collection_name: str = DEFAULT_COLLECTION_NAME,
        index_creation:bool =True #New argument
    ):
        from pymongo import MongoClient, errors

        self.connection_string = connection_string
        self.session_id = session_id
        self.database_name = database_name
        self.collection_name = collection_name

        try:
            self.client: MongoClient = MongoClient(connection_string)
        except errors.ConnectionFailure as error:
            logger.error(error)

        self.db = self.client[database_name]
        self.collection = self.db[collection_name]
        if index_creation: #Conditional Index Creation
            self.collection.create_index("SessionId")

Motivation
Developer can do custom modification, But if you make this feature that feature comes package.
Your contribution
yes, can do this
Proposed modification:
def __init__(
        self,
        connection_string: str,
        session_id: str,
        database_name: str = DEFAULT_DBNAME,
        collection_name: str = DEFAULT_COLLECTION_NAME,
        index_creation:bool =True #New argument
    ):
        from pymongo import MongoClient, errors

        self.connection_string = connection_string
        self.session_id = session_id
        self.database_name = database_name
        self.collection_name = collection_name

        try:
            self.client: MongoClient = MongoClient(connection_string)
        except errors.ConnectionFailure as error:
            logger.error(error)

        self.db = self.client[database_name]
        self.collection = self.db[collection_name]
        if index_creation: #Conditional Index Creation
            self.collection.create_index("SessionId")

I will make this changes and give pull request


GitVersionTask and Nuget Pack

NuGet isn't able to determine that a build is a prerelease.
With latest GitVersion on development  branch:
PS D:\Code\RobGeoLtd\Measurement> .\GitVersion.exe
{
  "Major":0,
  "Minor":5,
  "Patch":0,
  "PreReleaseTag":"alpha.1",
  "PreReleaseTagWithDash":"-alpha.1",
  "BuildMetaData":1,
  "FullBuildMetaData":"1.Branch.develop.Sha.80517f758b159de4bb6bedf43897f5d57e0427b0",
  "MajorMinorPatch":"0.5.0",
  "SemVer":"0.5.0-alpha.1",
  "LegacySemVer":"0.5.0-alpha1",
  "LegacySemVerPadded":"0.5.0-alpha0001",
  "AssemblySemVer":"<IP_ADDRESS>",
  "FullSemVer":"0.5.0-alpha.1",
  "InformationalVersion":"0.5.0-alpha.1+Branch.develop.Sha.80517f758b159de4bb6bedf43897f5d57e0427b0",
  "BranchName":"develop",
  "Sha":"80517f758b159de4bb6bedf43897f5d57e0427b0",
  "NuGetVersionV2":"0.5.0-alpha0001",
  "NuGetVersion":"0.5.0-alpha0001"
}

Where project has GitVersionTask,
nuget pack .\Measurement\Measurement.csproj -p configuration=release

Generates a nuspec...
<metadata>
    <id>RobGeoLtd.Measurement</id>
    <version><IP_ADDRESS></version>
    <title>RobGeoLtd.Measurement</title>

my understanding is that nuget pack will automatically assume any package with a postfix is a pre-release, via the AssemblyInformationVersion. This implies GitVersionTask isn't setting it correctly?

Duplicate: #212 , Requires: #368


MWPW-134481 - Add section xxs spacing variant

When the spacing variants were added to the section metadata the xxs size was missed. Adding it as an option for block spacing.
Resolves: MWPW-134481
Test URLs:

Before: https://main--milo--adobecom.hlx.page/drafts/klee/doc-test/text?martech=off
After: https://sarchibeque-section-spacing--milo--adobecom.hlx.page/drafts/klee/doc-test/text?martech=off

I fixed the "Before" url. Confirmed additional supported spacing in "After" url.

Thank you!


"Failed to build iOS app" with Facebook Login package

Steps to Reproduce

I put the Facebook Login package into my pubspec.yaml and followed all the steps in the guide.

I ran pod install and pod upgrade just in case the Pods were outdated.

I executed the app in Android and everything worked fine. I did it in iOs and couldn't be able to build. Here's the following output.

Logs

Running flutter run
Launching lib/main.dart on iPhone X in debug mode...
Running pod install...
Starting Xcode build...
Xcode build done.                                            2.1s
Failed to build iOS app
Error output from Xcode build:
↳
    ** BUILD FAILED **

Xcode's output:
↳
    In file included from /Users/javier/Documents/Android_Studio_Projects/squat_app/ios/Pods/FBSDKCoreKit/FBSDKCoreKit/FBSDKCoreKit/Internal/AppEvents/FBSDKPaymentObserver.m:23:
    In file included from /Users/javier/Documents/Android_Studio_Projects/squat_app/ios/Pods/FBSDKCoreKit/FBSDKCoreKit/FBSDKCoreKit/Internal/AppEvents/FBSDKAppEvents+Internal.h:19:
    /Users/javier/Documents/Android_Studio_Projects/squat_app/ios/Pods/FBSDKCoreKit/FBSDKCoreKit/FBSDKCoreKit/FBSDKAppEvents.h:33:19: warning: pointer is missing a nullability type specifier (_Nonnull, _Nullable, or _Null_unspecified) [-Wnullability-completeness]
    FOUNDATION_EXPORT NSNotificationName const FBSDKAppEventsLoggingResultNotification;
                      ^
    /Users/javier/Documents/Android_Studio_Projects/squat_app/ios/Pods/FBSDKCoreKit/FBSDKCoreKit/FBSDKCoreKit/FBSDKAppEvents.h:33:19: note: insert '_Nullable' if the pointer may be null
    FOUNDATION_EXPORT NSNotificationName const FBSDKAppEventsLoggingResultNotification;
                      ^
                                         _Nullable
    /Users/javier/Documents/Android_Studio_Projects/squat_app/ios/Pods/FBSDKCoreKit/FBSDKCoreKit/FBSDKCoreKit/FBSDKAppEvents.h:33:19: note: insert '_Nonnull' if the pointer should never be null
    FOUNDATION_EXPORT NSNotificationName const FBSDKAppEventsLoggingResultNotification;
                      ^
                                         _Nonnull
    /Users/javier/Documents/Android_Studio_Projects/squat_app/ios/Pods/FBSDKCoreKit/FBSDKCoreKit/FBSDKCoreKit/FBSDKAppEvents.h:700:13: warning: pointer is missing a nullability type specifier (_Nonnull, _Nullable, or _Null_unspecified) [-Wnullability-completeness]
    + (NSString *)getUserData;
                ^
    /Users/javier/Documents/Android_Studio_Projects/squat_app/ios/Pods/FBSDKCoreKit/FBSDKCoreKit/FBSDKCoreKit/FBSDKAppEvents.h:700:13: note: insert '_Nullable' if the pointer may be null
    + (NSString *)getUserData;
                ^
                 _Nullable
    /Users/javier/Documents/Android_Studio_Projects/squat_app/ios/Pods/FBSDKCoreKit/FBSDKCoreKit/FBSDKCoreKit/FBSDKAppEvents.h:700:13: note: insert '_Nonnull' if the pointer should never be null
    + (NSString *)getUserData;
                ^
                 _Nonnull
    /Users/javier/Documents/Android_Studio_Projects/squat_app/ios/Pods/FBSDKCoreKit/FBSDKCoreKit/FBSDKCoreKit/FBSDKAppEvents.h:714:44: warning: pointer is missing a nullability type specifier (_Nonnull, _Nullable, or _Null_unspecified) [-Wnullability-completeness]
    + (void)updateUserProperties:(NSDictionary *)properties handler:(FBSDKGraphRequestHandler)handler;
                                               ^
    /Users/javier/Documents/Android_Studio_Projects/squat_app/ios/Pods/FBSDKCoreKit/FBSDKCoreKit/FBSDKCoreKit/FBSDKAppEvents.h:714:44: note: insert '_Nullable' if the pointer may be null
    + (void)updateUserProperties:(NSDictionary *)properties handler:(FBSDKGraphRequestHandler)handler;
                                               ^
                                                _Nullable
    /Users/javier/Documents/Android_Studio_Projects/squat_app/ios/Pods/FBSDKCoreKit/FBSDKCoreKit/FBSDKCoreKit/FBSDKAppEvents.h:714:44: note: insert '_Nonnull' if the pointer should never be null
    + (void)updateUserProperties:(NSDictionary *)properties handler:(FBSDKGraphRequestHandler)handler;
                                               ^
                                                _Nonnull
    /Users/javier/Documents/Android_Studio_Projects/squat_app/ios/Pods/FBSDKCoreKit/FBSDKCoreKit/FBSDKCoreKit/FBSDKAppEvents.h:714:66: warning: block pointer is missing a nullability type specifier (_Nonnull, _Nullable, or _Null_unspecified) [-Wnullability-completeness]
    + (void)updateUserProperties:(NSDictionary *)properties handler:(FBSDKGraphRequestHandler)handler;
                                                                     ^
    /Users/javier/Documents/Android_Studio_Projects/squat_app/ios/Pods/FBSDKCoreKit/FBSDKCoreKit/FBSDKCoreKit/FBSDKAppEvents.h:714:66: note: insert '_Nullable' if the block pointer may be null
    + (void)updateUserProperties:(NSDictionary *)properties handler:(FBSDKGraphRequestHandler)handler;
                                                                     ^
                                                                                _Nullable
    /Users/javier/Documents/Android_Studio_Projects/squat_app/ios/Pods/FBSDKCoreKit/FBSDKCoreKit/FBSDKCoreKit/FBSDKAppEvents.h:714:66: note: insert '_Nonnull' if the block pointer should never be null
    + (void)updateUserProperties:(NSDictionary *)properties handler:(FBSDKGraphRequestHandler)handler;
                                                                     ^
                                                                                _Nonnull
    /Users/javier/Documents/Android_Studio_Projects/squat_app/ios/Pods/FBSDKCoreKit/FBSDKCoreKit/FBSDKCoreKit/FBSDKAppEvents.h:726:43: warning: pointer is missing a nullability type specifier (_Nonnull, _Nullable, or _Null_unspecified) [-Wnullability-completeness]
    + (void)augmentHybridWKWebView:(WKWebView *)webView;
                                              ^
    /Users/javier/Documents/Android_Studio_Projects/squat_app/ios/Pods/FBSDKCoreKit/FBSDKCoreKit/FBSDKCoreKit/FBSDKAppEvents.h:726:43: note: insert '_Nullable' if the pointer may be null
    + (void)augmentHybridWKWebView:(WKWebView *)webView;
                                              ^
                                               _Nullable
    /Users/javier/Documents/Android_Studio_Projects/squat_app/ios/Pods/FBSDKCoreKit/FBSDKCoreKit/FBSDKCoreKit/FBSDKAppEvents.h:726:43: note: insert '_Nonnull' if the pointer should never be null
    + (void)augmentHybridWKWebView:(WKWebView *)webView;
                                              ^
                                               _Nonnull
    /Users/javier/Documents/Android_Studio_Projects/squat_app/ios/Pods/FBSDKCoreKit/FBSDKCoreKit/FBSDKCoreKit/Internal/AppEvents/FBSDKPaymentObserver.m:242:38: error: property 'subscriptionPeriod' not found on object of type 'SKProduct *'
          BOOL isSubscription = (product.subscriptionPeriod != nil) && ((unsigned long)product.subscriptionPeriod.numberOfUnits > 0);
                                         ^
    /Users/javier/Documents/Android_Studio_Projects/squat_app/ios/Pods/FBSDKCoreKit/FBSDKCoreKit/FBSDKCoreKit/Internal/AppEvents/FBSDKPaymentObserver.m:242:92: error: property 'subscriptionPeriod' not found on object of type 'SKProduct *'
          BOOL isSubscription = (product.subscriptionPeriod != nil) && ((unsigned long)product.subscriptionPeriod.numberOfUnits > 0);
                                                                                ^
    /Users/javier/Documents/Android_Studio_Projects/squat_app/ios/Pods/FBSDKCoreKit/FBSDKCoreKit/FBSDKCoreKit/Internal/AppEvents/FBSDKPaymentObserver.m:248:9: error: use of undeclared identifier 'SKProductSubscriptionPeriod'
            SKProductSubscriptionPeriod *period = product.subscriptionPeriod;
            ^
    /Users/javier/Documents/Android_Studio_Projects/squat_app/ios/Pods/FBSDKCoreKit/FBSDKCoreKit/FBSDKCoreKit/Internal/AppEvents/FBSDKPaymentObserver.m:248:38: error: use of undeclared identifier 'period'
            SKProductSubscriptionPeriod *period = product.subscriptionPeriod;
                                         ^
    /Users/javier/Documents/Android_Studio_Projects/squat_app/ios/Pods/FBSDKCoreKit/FBSDKCoreKit/FBSDKCoreKit/Internal/AppEvents/FBSDKPaymentObserver.m:248:55: error: property 'subscriptionPeriod' not found on object of type 'SKProduct *'
            SKProductSubscriptionPeriod *period = product.subscriptionPeriod;
                                                          ^
    /Users/javier/Documents/Android_Studio_Projects/squat_app/ios/Pods/FBSDKCoreKit/FBSDKCoreKit/FBSDKCoreKit/Internal/AppEvents/FBSDKPaymentObserver.m:250:17: error: use of undeclared identifier 'period'
            switch (period.unit) {
                    ^
    /Users/javier/Documents/Android_Studio_Projects/squat_app/ios/Pods/FBSDKCoreKit/FBSDKCoreKit/FBSDKCoreKit/Internal/AppEvents/FBSDKPaymentObserver.m:256:76: error: use of undeclared identifier 'period'
            NSString *p = [NSString stringWithFormat:@"P%lu%@", (unsigned long)period.numberOfUnits, unit];
                                                                               ^
    5 warnings and 7 errors generated.
Could not build the application for the simulator.
Error launching application on iPhone X.

Running flutter analyze
Analyzing squat_app...
No issues found! (ran in 1.0s)

Flutter Doctor
[✓] Flutter (Channel beta, v0.9.4, on Mac OS X 10.12.6 16G1036, locale en-ES)
    • Flutter version 0.9.4 at /Users/javier/Documents/flutter
    • Framework revision f37c235c32 (4 weeks ago), 2018-09-25 17:45:40 -0400
    • Engine revision 74625aed32
    • Dart version 2.1.0-dev.5.0.flutter-a2eb050044

[✓] Android toolchain - develop for Android devices (Android SDK 28.0.3)
    • Android SDK at /Users/javier/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)
    • All Android licenses accepted.

[✓] iOS toolchain - develop for iOS devices (Xcode 9.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 9.0, Build version 9A235
    • ios-deploy 1.9.2
    • CocoaPods version 1.5.3

[✓] Android Studio (version 3.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 29.1.1
    • Dart plugin version 181.5656
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)

[✓] Connected devices (2 available)
    • Mi A1    • 1f95603a0804                         • android-arm64 • Android 8.1.0 (API 27)
    • iPhone X • 32CD2ECE-8C12-4F3A-B768-E1568B9754A7 • ios           • iOS 11.0 (simulator)

• No issues found!

Additional info
This is how my Podfile looks right now
# Uncomment this line to define a global platform for your project
platform :ios, '9.0'

# CocoaPods analytics sends network stats synchronously affecting flutter build latency.
ENV['COCOAPODS_DISABLE_STATS'] = 'true'

def parse_KV_file(file, separator='=')
  file_abs_path = File.expand_path(file)
  if !File.exists? file_abs_path
    return [];
  end
  pods_ary = []
  skip_line_start_symbols = ["#", "/"]
  File.foreach(file_abs_path) { |line|
      next if skip_line_start_symbols.any? { |symbol| line =~ /^\s*#{symbol}/ }
      plugin = line.split(pattern=separator)
      if plugin.length == 2
        podname = plugin[0].strip()
        path = plugin[1].strip()
        podpath = File.expand_path("#{path}", file_abs_path)
        pods_ary.push({:name => podname, :path => podpath});
      else
        puts "Invalid plugin specification: #{line}"
      end
  }
  return pods_ary
end

target 'Runner' do
  #use_frameworks!

  pod 'FBSDKLoginKit'
  pod 'FBSDKCoreKit'

  # Prepare symlinks folder. We use symlinks to avoid having Podfile.lock
  # referring to absolute paths on developers' machines.
  system('rm -rf .symlinks')
  system('mkdir -p .symlinks/plugins')

  # Flutter Pods
  generated_xcode_build_settings = parse_KV_file('./Flutter/Generated.xcconfig')
  if generated_xcode_build_settings.empty?
    puts "Generated.xcconfig must exist. If you're running pod install manually, make sure flutter packages get is executed first."
  end
  generated_xcode_build_settings.map { |p|
    if p[:name] == 'FLUTTER_FRAMEWORK_DIR'
      symlink = File.join('.symlinks', 'flutter')
      File.symlink(File.dirname(p[:path]), symlink)
      pod 'Flutter', :path => File.join(symlink, File.basename(p[:path]))
    end
  }

  # Plugin Pods
  plugin_pods = parse_KV_file('../.flutter-plugins')
  plugin_pods.map { |p|
    symlink = File.join('.symlinks', 'plugins', p[:name])
    File.symlink(p[:path], symlink)
    pod p[:name], :path => File.join(symlink, 'ios')
  }
end

post_install do |installer|
  installer.pods_project.targets.each do |target|
    target.build_configurations.each do |config|
      config.build_settings['ENABLE_BITCODE'] = 'NO'
    end
  end
end

Well, after a lot of hours thinking about it I've found the solution.
Looks like Xcode 9.0 didn't have SKSubscriptionPeriod. Right now I'm not much into native iOs development so I didn't expect that it would be an IDE version issue.
After updating Xcode in the App Store, this is how my Doctor looks
[✓] Flutter (Channel beta, v0.9.4, on Mac OS X 10.12.6 16G1036, locale en-ES)
    • Flutter version 0.9.4 at /Users/javier/Documents/flutter
    • Framework revision f37c235c32 (4 weeks ago), 2018-09-25 17:45:40 -0400
    • Engine revision 74625aed32
    • Dart version 2.1.0-dev.5.0.flutter-a2eb050044

[✓] Android toolchain - develop for Android devices (Android SDK 28.0.3)
    • Android SDK at /Users/javier/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)
    • All Android licenses accepted.

[✓] iOS toolchain - develop for iOS devices (Xcode 9.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 9.2, Build version 9C40b
    • ios-deploy 1.9.2
    • CocoaPods version 1.5.3

[✓] Android Studio (version 3.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 29.1.1
    • Dart plugin version 181.5656
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)

[✓] Connected devices (1 available)
    • iPhone 8 Plus • 7D4E5F57-F307-4BD4-97B2-F88DBE31F89A • ios • iOS 11.2 (simulator)

• No issues found!


Move addressfield into hypha and update jquery.addressfield.js.

Fixes #3939
Test Steps

[ ] Test so the address field on application forms still work as it should. For both new and existing submissions

It looks like this is no longer maintained & it looks like we're still using a release from 2016 - is this something we should be looking for an alternative for?


Create ML_EdgeDetection

please accept my pull request sir then only i can able to share all files regarding to this EDGE DETECTION using ML .....

@saieswari2005 ,you have just added a text file , don't have code files ,try to commit those as well so that we can review and merge pr

@saieswari2005 ,you have just added a text file , don't have code files ,try to commit those as well so that we can review and merge pr

i have try to upload code file but shows message like"you need push access to this repository for upload file" thats why i ccreated new file insert description as well as code also at the end if you accept my request after that i can upload code file for now i given the code at last in created file please observe sir!!

Hi @saieswari2005 still same files visible


fix: saving spinner not visible on android in my collection

The type of this PR is: Bugfix

This PR resolves CX-2252
Description

Make Spinner visible on android
Use Palette Spinner instead of react-native ActivityIndicator
Make sure that the status bar is also dimmed

PR Checklist (tick all before merging)

[x] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[x] I have tested my changes on iOS and Android.
[x] I have added tests/stories for my changes, or my changes don't require testing/stories, or I have included a link to a separate Jira ticket covering the tests.
[x] I have added a feature flag, or my changes don't require a feature flag. (How do I add one?)
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added an app state migration, or my changes do not require one. (What are migrations?)
[x] I have added a changelog entry below or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

Fix: Use Palette Spinner for My Collection Add Artwork Saving Modal - mounir

iOS user-facing changes

Android user-facing changes

Fix: Make My Collection Add Artwork Saving Spinner visible - mounir

Dev changes

This PR contains the following changes:

Android user-facing changes (Fix: Make My Collection Add Artwork Saving Spinner visible - mounir)
Cross-platform user-facing changes (Fix: Use Palette Spinner for My Collection Add Artwork Saving Modal - mounir)

  Generated by :no_entry_sign: dangerJS against deb857c7dd8a5df9f06df0fbbb425edc89e44af4

#squashongreen


How to use Hystrix forward: fallback url with path variables

How can we configure fallback service to primary service which will forwards all failing requests to fallback service (or cache)?
where the endpoint is the same and dynamic but context root is different.

I have below route
routes().route("product_route",
r -> r.path("/student/{ID}").filters(hystrix -> hystrix.hystrix(configConsumer -> {
configConsumer.setName("student_fallback");
configConsumer.setFallbackUri("forward:/studentFallfack/");
})).uri("lb://student-api-load-balanced")) and I have configued route for /studentFallfack/ to send request to fallback service.
now in case of student/{id} route failure i want to send the same {id} variable to studentfallback service but with "forward:" i am not able to pass the generic id as a path variable.
Earlier i was taking another approach as in case of student service failure is it possible that Hystrix fallback can serve the result from the cache, if yes how to configure Hystrix to get the result from cache

I think you should use : r.path("/student/**")

can hystrix failure fetch data from secondary service for dynamic routes

Have you tried accessing the parameters via ServerWebExchange.getRequest().getQueryParams()?

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


Not Implemented Error on Portal

Running the trigger after deploying to my function on App Service seems to throw a NotImplementedException.
The error message looks like:
2019-09-05T16:35:50.230 [Error] Executed 'TriggerFunction' (Failed, Id=d4e9580a-7e66-4a8b-aa0d-23e881ff41df)
The method or operation is not implemented.
Might have something to do with manually clicking 'run' on the portal - I looked for instances of throwing a not implemented error in the solution and couldn't find any.

Closing as it is tooling issue - clicking run on trigger sends dummy data


Issues of camera calibration files and image datasets of Pillcam

Hi,
There are two issues I am concerned about:
(1) According to the README file under the folder EndoSfMLearner, there should be a cam.txt containing the camera calibration parameters. I know that I can get the parameters from the folder Calibration, but I have no idea what the format in cam.txt is. Also, only calibrations of MiroCam and PillCam are provided. Where are the calibration parameters of LowCam and HighCam?
(2) Downloaded from the dropbox link, only images of HighCam, LowCam, and MiroCam are provided under the folder Cameras. Where can I find images of PillCam?
Am I correct that, given the limited data, I can only reproduce the experiments of MiroCam if I get its format of 'cam.txt' correclty?

Hi,
All required parameters are released in Table Appendix C.2:
https://arxiv.org/abs/2006.16670
we will release the whole dataset in a few weeks.
Best Regards,
Bengisu

How about the format of cam.txt? Is there an example for that?


feat: read codetour files from regular files and symbolic links

First of all, thank you for this amazing project!
This is a request to add the ability for the extension to read symlink files so that codetour files can exist outside of the repository
Any feedback is appreciated

Thanks for the contribution! This change looks good to me 👍


develop: Pin alpine version to 3.6 and Python version to 3.6

It's coincidental that alpine latest is 3.6 and python latest is also 3.6, but hey.

@StevenACoffman I'ts already done.


State machine crashes on state transitions

This seems to be caused by state reactors and client behaviours that are destructed while their callback functions are running.
Is there a way to prevent such scenarios?
For custom objects it is possible to use mutexes to circumvent the issue, but for standard objects such as 'sr_all_events_go' this is less than ideal.

We ran into a similar issue. We've had some success using a combination of mutex locks in the CBs' onExit methods to prevent callbacks from running during destruction and using SMACC's built-in asynchronous CBs that works better with long-running onExit calls

@pabloinigoblasco that's a fair point. I tried to make a toy example to see if I can replicate the behaviour, but was unable to since the signals you described seemed to be doing their job preventing the short-lived objects from being dereferenced too early. I'll need to think more on the problem, but I'll try to make a proper sample to recreate the issue for you.
For some additional context on my thinking: in our case, the crashes (always segfaults referencing non-existing objects) were non-deterministic and happened randomly, but always upon a user-triggered state transition and involved long-running CUDA image processing tasks. This is what made me suspect a race condition being somewhere and trying mutexes to prevent premature destruction of the associated CBs. This seems to have helped, but I agree its not a very elegant solution and doesn't quite fit into the SMACC style.


Authentication for Powerpages webextension

Added changes for dataverse authentication for Power pages
Requirements added here: https://github.com/microsoft/vscode-dev/issues/477
Testing:
npm run compile-web
tested with portal route - http://localhost:3000/powerplatform/default/adx_webtemplates/97a5603a-3ab0-ec11-9840-000d3a5a4b78?orgUrl=orgca400f6b.crm.dynamics.com&source=dataverse&schema=portalDataverseoldschema

As an aside, can you tell me why this extension needs to be activated for:

onStartupFinished
onLanguage:yaml
onLanguage:html

From my perspective, this extension isn't going to be used all the time to warrant these events and an extension activating when it shouldn't need to ends up annoying customers and impacts performance.


Expose claims to Authorizator method

Allows bypassing expensive DB checks and lookups, while still allowing them if needed.
This allows one to store basic claim data and handle it with the PayloadFunc and Authorization.
See new TestClaimsDuringAuthorization() test to demonstrate this.  The actual fix was trivial (just set the Env before calling Authorizator

Hi Andrew,
Thanks for the PR & tests. This looks very nice.  :+1:
Totally missed that and actually we could also avoid passing the userid to the authorization function like this. However, I don't want to break the API right now so I will keep that for later.
I will add some small inline comments and once they are resolved we can merge this.

Quite right on the log.Printf("Authentication Attempt: %s\n", userId).  It was a left over as I was tying everything together and in my haste to get this out the door before lunch I missed it.
As far as checking request.Env["REMOTE_USER"]:  I was initial going to ask if we really wanted to do that in this test, as it isn't a claim.  However I think that statement would be incorrect, just because we are not pulling it from the JWT_PAYLOAD does not mean it is not a claim.

I think that keeping the userId in the Authorizator function is probably the right way to do it.  Correctly casting out the claims is a little tedious to have to do for simple checks.  How do you feel about some sort of helper method for this, something along the lines of:
Authorizator: func(userId string, request *rest.Request) bool {
  claims, err := jwt.ExtractClaims(request)
  if err := nil {
    log.Errorf("Claims Extraction Failed: %v", err)
    return false
  }
  isAdmin := claims["isAdmin"].(bool)
  // 'real' auth code
}

This would allow easy access without breaking the current API.   If you are interested I will whip something up.

Yes, you are right. Actually it should go in the original/first authorization tests together with a test whether userId is right which I figured is lacking. I can add that.
Yeah, I was thinking about such a helper function as well. If you want you can go ahead and think about something.

NOTE:  Just noticed you replied, keeping my WOT as it was.  I think this PR is ready to merge then.
I was making the check for REMOTE_USER and it didn't feel right to me (apparently I am having a flip-flop day, perhaps I should go into politics).
Strictly speaking REMOTE_USER does not need to be guaranteed in the Authorizator method, as we are passing in userId.  Meaning that if we were to write the check and the code got refactored so that REMOTE_USER was not set during authentication then this test would (erroneously) fail.
When accessing REMOTE_USER from an actual endpoint method it does not matter if it is set before or after the call to Authorizator, and if it is not set at all then the TestAuthJWT() test will catch it.

Yeah, will merge in a second. Thanks again!
Ok, now I see what you mean. I think that makes sense and we can leave it as it is for now. Yeah, the normal callbacks should be fine. I was just wondering about the authorization callback.


Python 3.12 wheels for Windows

FYI the latest release for some reason is missing these

It seems we never published wheels for python 3.12 on Windows. I'll try and publish v0.7.1 to check what the issue was. Thanks for the heads up @ofek !

Closed in #44.

Thank you!


Add public Schema creation, and schema option to Dataset#create_table and Dataset#load

From @AlfonsoUceda's comments in PR #1533:

is there a change to pass the schema as an option instead a block? for example the following snippet:

bigquery.dataset("my_dataset").load "my_table", file, write: "truncate", schema: schema

We create schema objects #from_gapi(nil) because then we inject them in the table#load using our fork. I know #from_gapi method is private but is there a way to create schema objects from a public interface? have you though in that?

More on  @AlfonsoUceda's usage:

we have our custom reporter component, I mean an ETL process. We have schema separated by files, every schema its class, so we can inject them in our ETL class, every ETL has a file and we define a extractor, transformer and loader (csv). When the CSV file is created then with this file and a generated schema object we inject into our bigquery loaders class to create or truncate the table.

This is because we can't test our schema in a separate test or because we share schemas by ETLs.

bigquery_loader.load(file, schema: schema)

# inside `#load`
table.load file, write: 'truncate', schema: schema # this is because of our fork

@quartzmo thanks!


Current feature disappears when map is moved.

Description
When I am tracing a building or road and need to pan the map to finish tracing the feature, as soon as the map moves, the feature disappears.
Version
2.0.0
What browser are you seeing the problem on? What version are you running?
Chrome v111.0
The OS you're using
mac
Steps to reproduce
No response
The browser URL at the time you encountered the bug
https://rapideditor.org/edit?fbclid=IwAR3v1EtFuOMYp1bz_cGma21kuXA37oBaIGKbP0_NjKdXl1SRvN2gv4mIQVg#background=Bing&datasets=fbRoads,msBuildings&disable_features=boundaries&map=20.57/44.97732/-122.93984
The auto-detected useragent string for your browser (leave blank if you're manually filling this form out)
Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36

Duplicate of https://github.com/facebook/Rapid/issues/879


Refactor Stripe payment intent checking and storage

This PR looks to improve on how payment intents are stored and how to better prevent overlaps when manually placing orders via Stripe and also via the webhook.
PaymentIntent storage and reference to carts/orders
Currently PaymentIntent information is stored in the Cart model's meta, which is then transferred to the order when created.
Whilst this works okay it causes for limitations and also means that if the carts meta is ever updated elsewhere, or the intent information is removed, then it will cause unrecoverable loss.
This PR looks to move away from the payment_intent key in the meta to a StripePaymentIntent model, this allows us more flexibility in how payment intents are handled and reacted on. A StripePaymentIntent will be associated to both a cart and an order.
The information we store is now:

intent_id - This is the PaymentIntent ID which is provided by Stripe
status - The PaymentIntent status
event_id - If the PaymentIntent was placed via the webhook, this will be populated with the ID of that event
processing_at - When a request to place the order is made, this is populated
processed_at - Once the order is placed, this will be populated with the current timestamp

Preventing overlap
Currently we delay sending the job to place the order to the queue by 20 seconds, this is less than ideal, now the payment type will check whether we are already processing this order and if so, not do anything further. This should prevent overlaps regardless of how they are triggered.

@alecritson should we add something to the upgrade guide along the lines of this PR's description?

PaymentIntent storage and reference to carts/orders
Currently PaymentIntent information is stored in the Cart model's meta, which is then transferred to the order when created.
Whilst this works okay it causes for limitations and also means that if the carts meta is ever updated elsewhere, or the intent information is removed, then it will cause unrecoverable loss.
This PR looks to move away from the payment_intent key in the meta to a StripePaymentIntent model, this allows us more flexibility in how payment intents are handled and reacted on. A StripePaymentIntent will be associated to both a cart and an order.
The information we store is now:

intent_id - This is the PaymentIntent ID which is provided by Stripe
status - The PaymentIntent status
event_id - If the PaymentIntent was placed via the webhook, this will be populated with the ID of that event
processing_at - When a request to place the order is made, this is populated
processed_at - Once the order is placed, this will be populated with the current timestamp

Preventing overlap
Currently we delay sending the job to place the order to the queue by 20 seconds, this is less than ideal, now the payment type will check whether we are already processing this order and if so, not do anything further. This should prevent overlaps regardless of how they are triggered.

Added the description to the upgrade guide


v0.12 outputs worse result than v0.11, running same code of lstm

I'm afraid there isn't enough detail here. Please open a new issue with the issue template filled in (OS, setup, code you are running, etc.)


[ ] I have tested all my changes thoroughly.

Hell cat ears can be created at any time if you have the original components. Only the Cat ears themselves are discontinued.
The Red  dye used to be discontinued, but it was (unintentionally?) re-released into the /buy command


Setup Cloud Build triggers for creation by referencing github repo instead of docker image

Is this something which might be possible in the future or are there any obvious methods I'm missing to implement this with the current plugin?

I'm sure it's possible in the future.
If it becomes available for use in https://registry.terraform.io/providers/hashicorp/google/latest/docs/resources/cloud_run_service, then it would become available for use in this module. Until that happens, there's not much more that can be done for this module.
I'm closing this issue for now. If this changes in the future, this issue can either be re-opened, or a new one can be created.