10/18/2018 21:54 : Create Pull Request for branch master -> prTesting1_2018101821

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestCon1.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


vcpkg setup-build-depends.sh not work for centos 7/8

Problem description
When trying to run setup-build-dependens.sh to prepare the build env, it reports errors:
++ yum -y install epel-release centos-release-scl
Loaded plugins: fastestmirror, ovl
Determining fastest mirrors
Could not retrieve mirrorlist http://mirrorlist.centos.org/?release=7&arch=x86_64&repo=os&infra=container error was
14: curl#6 - "Could not resolve host: mirrorlist.centos.org; Unknown error"

System information
The repository issue in centos, and could not install cmake and gcc compiler for this info.
CMake log
No response

As a possible temporary solution, would you please try with this method to create a build in docker container? It should be a more general way for building.

Thanks @zhztheplayer, our prod env is centos 7, I'm not sure if the lib built on ubuntu would work fine.

Thanks @zhztheplayer, our prod env is centos 7, I'm not sure if the lib built on ubuntu would work fine.

I see. There might be a glibc compatibility issue since ubuntu 20.04 has higher version. I'll provide another option to build in centos 7 container (or lower) later.


UDP Discovery

Some mechanism for discovering the IP address of each device is necessary (or static IP addresses).
possibilities:

mDNS
broadcast IP address (<IP_ADDRESS>) and have OSC address contain a unique ID. (broadcast UDP appears to slow down the ESP8266 on receiving BUT not on sending).

Currently fixed by having unique ID for each dreamIO device and then using <IP_ADDRESS> to broadcast UDP.


October 15 repeated

Anyone know why this is happening?
https://reddit.com/r/NoStupidQuestions/comments/538zae/why_are_all_of_these_websites_malfunctioning_on/?sort=confidence&utm_source=mweb_redirect&compact=true

See #55 - this happens in Brazil because it starts its daylight savings time at midnight.


Clear XmlDoc caches after closing a solution

Reference #57.
This PR makes sure that XmlDoc caches will be cleared when closing solutions. It helps release locks on Xml documentation files so that external clients can use these files.
As @latkin said in https://github.com/Microsoft/visualfsharp/issues/57#issuecomment-71557706, the issue requires a fix in Visual Studio itself. The doesn't completely solve #57 but the bug should be less severe now.

I'm sorry, but I'm not sure how the bug is any less severe with this PR applied? To build from command-line you'd still need to close the solution first, or get that jarring IO error. In any reasonably sized project, starting VS isn't what takes time, it is loading the solution that takes time. Now VS may have even more work to do, if the Xml doc cache needs to be recreated (not sure how the xml doc cache works).

I think invalidating XmlDoc cache for each solution is an ok behaviour since two subsequent solutions might have very different sets of assembly references.
You're right that opening a solution takes time. But immediately closing and re-opening a solution is fast enough (VS still keep some caches). This is what PR targets for.

I'll have several instances of VS running at any given time, and never close a solution. I'll just close VS instead. This PR won't help this usage pattern. Also, manual action still needs to be taken to build from command line, whether it is close solution or close VS. Let's hope VS can stop locking these files instead. ;)

Yes, I understand your point.
I often get annoyed by this when using Paket to build solutions inside Visual Studio. Whenever, Paket needs to update packages (including Xml files), Visual Studio locks these Xml files and builds fail. I have no way to fix it except closing VS. It happens because Paket is an external process while NuGet is integrated inside Visual Studio.
/cc @forki

From paket's and also FAKE's perspective there is really nothing we can do
other than upvoting every issue and pull request that reduces the pain.
Yes, I understand your point.
I often get annoyed by this when using Paket to build solutions inside
Visual Studio. Whenever, Paket needs to update packages (including Xml
files), Visual Studio locks these Xml files and builds fail. I have no way
to fix it except closing VS. It happens because Paket is an external
process while NuGet is integrated inside Visual Studio.
/cc @forki https://github.com/forki
—
Reply to this email directly or view it on GitHub
https://github.com/Microsoft/visualfsharp/pull/138#issuecomment-71653579.

Have you confirmed this resolves the issue for the limited scenario of closing a solution? There is support for creating and closing solutions in the IDE unit tests, have you looked at adding a case?
Even though it doesn't address the entire problem, this seems like a nice small improvement, and the delta looks innocent enough.
I don't think closing and opening new solutions is a very common action, so even if there is a small perf hit here due to re-creating caches it doesn't strike me as particularly harmful.

@latkin

Have you confirmed this resolves the issue for the limited scenario of closing a solution?

I have tested it with EnableOpenSource build.

There is support for creating and closing solutions in the IDE unit tests, have you looked at adding a case?

Can you elaborate on what kind of tests would you like to add? The fact that Xml files are no longer locked is side effect; I'm not entirely sure how to write tests for it.

Test would attempt the problem scenario:

Create and build project to produce .dll with .xml
Create fresh solution/project, add DLL reference, drive intellisense
Try deleting xml doc file (should throw due to lock)
Close solution
Try deleting xml doc file (should now work)

This should be possible using existing IDE unit test infra (there is already nice support for creating slns/projects and checking intellisense), however it's probably overkill here... Unless someone objects, I guess I'm ok with not adding an explicit test.

I also think that an integration test is overkill.
I manually tested your scenario, (unfortunately) Xml doc files don't get locked inside VS 2015 with F# 4.0. Perhaps it needs other conditions for locking to happens.
I tested with the following scenario:

Open a project using Paket as a package manager
Hover tooltips on a symbol from an assembly referenced as a NuGet package
Increase (decrease) version number of that package in paket.lock
Build the project
The Xml file is locked; Paket can't override the package and build fails
Close the solution
Reopen solution
Build succeeds


100% CPU usage upon import numpy

When doing simple import numpy as np the cpu goes to 100% usage in 7 out of 8 cores available.
Environment
Fresh install Ubuntu 22.04 arm64 python3.11
Python and NumPy Versions:
>>> import sys, numpy; print(numpy.version); print(sys.version)
1.26.0
3.11.7 (main, Dec 8 2023, 18:56:58) [GCC 11.4.0]

Runtime environment
import numpy; print(numpy.show_runtime())
[{'numpy_version': '1.26.0',
'python': '3.11.7 (main, Dec 8 2023, 18:56:58) [GCC 11.4.0]',
'uname': uname_result(system='Linux', node='localhost', release='3.10.61', version='https://github.com/numpy/numpy/pull/1 SMP PREEMPT Tue Dec 5 22:24:50 UTC 2023', machine='aarch64')},
{'simd_extensions': {'baseline': ['NEON', 'NEON_FP16', 'NEON_VFPV4', 'ASIMD'],
'found': [],
'not_found': ['ASIMDHP', 'ASIMDFHM']}},
{'architecture': 'armv8',
'filepath': '/home/android/hass/lib/python3.11/site-packages/numpy.libs/libopenblas64_p-r0-17488984.3.23.dev.so',
'internal_api': 'openblas',
'num_threads': 8,
'prefix': 'libopenblas',
'threading_layer': 'pthreads',
'user_api': 'blas',
'version': '0.3.23.dev'}]
None

Reproducing video can be found at: https://www.youtube.com/watch?v=cba5ntdMRkM
To solve my problem I've installed ATLAS lib, however I do still think I should report the issue in case someone faces the same issue as I did and to make you aware of the issue.

@AyrtonRicardo what I did not get from your video is whether the CPU usage going to 100% is a transient startup effect, or whether the utilization stays that high. I didn't see it go down in the video, but may have missed it.

@AyrtonRicardo what I did not get from your video is whether the CPU usage going to 100% is a transient startup effect, or whether the utilization stays that high. I didn't see it go down in the video, but may have missed it.

Thanks for the reply, it stays like this(100%) as long as the script is running. It went down as soon as I closed the python script I was simulating, you can see that here I was barely using any cpu, I change threads to use only 3, after execute again the script (import numpy), the cpu goes up to 100% in 2 out of 3 cores.
Here is an example over around 1h of this running, avg of 86% while my application was idle. Also comparing to when I applied cpulimit to it with --limit 200(around 35% each core)

*Up and down because I was testing enabling/disabling cpulimit over the process.
That's currently same application running with ATLAS:

*3 hours time window. Barely invoking any spyke (expected behaviour, since it's almost idle all the time)

what is your hardware please ? just loading openblas by impirting numpy should at most cause a short spike as threads get set up, it certainly shouldn't be busy waiting(?) like that

also did you build openblas and/or numpy from source or use binary packages (from where?) ?

@martin-frbg not sure what other information do you need, so my full setup:
Rooted Galaxy Note 5 running Ubuntu 22.04 via Linux Deploy. without Android GUI:

Exynos 7420 Octa (14 nm) Octa-core (4x2.1 GHz Cortex-A57 & 4x1.5 GHz Cortex-A53)
Ubuntu 22.04 arm64 fresh install
4GB RAM

hwinfo:
hwinfo --short
cpu:
                       AArch64 Processor rev 0 (aarch64)
                       AArch64 Processor rev 0 (aarch64)
                       AArch64 Processor rev 0 (aarch64)
                       AArch64 Processor rev 0 (aarch64)
                       AArch64 Processor rev 0 (aarch64)
                       AArch64 Processor rev 0 (aarch64)
                       AArch64 Processor rev 0 (aarch64)
                       AArch64 Processor rev 0 (aarch64)
keyboard:
  /dev/ram             serial console
network:
                       Broadcom Network controller
network interface:
  ip6tnl0              Network Interface
  ip_vti0              Network Interface
  rmnet4               Network Interface
  rmnet2               Network Interface
  rmnet3               Network Interface
  rmnet7               Network Interface
  rmnet5               Network Interface
  rmnet6               Network Interface
  rmnet1               Network Interface
  rmnet0               Network Interface
  lo                   Loopback network interface
  sit0                 Network Interface
  p2p0                 Ethernet network interface
  wlan0                WLAN network interface
disk:
  /dev/sda             SAMSUNG KLUBG4G1BE-E0B1
  /dev/sdb             SAMSUNG KLUBG4G1BE-E0B1
  /dev/sdc             SAMSUNG KLUBG4G1BE-E0B1
  /dev/sdd             SAMSUNG KLUBG4G1BE-E0B1
  /dev/vnswap0         Disk
partition:
  /dev/sda1            Partition
  /dev/sda2            Partition
  /dev/sda3            Partition
  /dev/sda4            Partition
  /dev/sda5            Partition
  /dev/sda6            Partition
  /dev/sda7            Partition
  /dev/sda8            Partition
  /dev/sda9            Partition
  /dev/sda10           Partition
  /dev/sda11           Partition
  /dev/sda12           Partition
  /dev/sda13           Partition
  /dev/sda14           Partition
  /dev/sda15           Partition
  /dev/sda16           Partition
  /dev/sda17           Partition
  /dev/sdd1            Partition
usb controller:
                       ARM USB controller
                       ARM USB controller
                       ARM USB controller
bridge:
                       Samsung Electronics PCI bridge
memory:
                       Main Memory

Full cpu detail:

android@localhost:~/certbot$ hwinfo --cpu
01: None 00.0: 10103 CPU
[Created at cpu.343]
Unique ID: rdCR.j8NaKXDZtZ6
Hardware Class: cpu
Arch: AArch64
Vendor: "ARM Limited"
Model: 0.2.0 "AArch64 Processor rev 2 (aarch64)"
Platform: "SAMSUNG Exynos7420"
Features: fp,asimd,aes,pmull,sha1,sha2,crc32,
Config Status: cfg=new, avail=yes, need=no, active=unknown

02: None 00.0: 10103 CPU
[Created at cpu.343]
Unique ID: rdCR.j8NaKXDZtZ6
Hardware Class: cpu
Arch: AArch64
Vendor: "ARM Limited"
Model: 0.2.0 "AArch64 Processor rev 2 (aarch64)"
Platform: "SAMSUNG Exynos7420"
Features: fp
Config Status: cfg=new, avail=yes, need=no, active=unknown

03: None 00.0: 10103 CPU
[Created at cpu.343]
Unique ID: rdCR.j8NaKXDZtZ6
Hardware Class: cpu
Arch: AArch64
Vendor: "ARM Limited"
Model: 0.2.0 "AArch64 Processor rev 2 (aarch64)"
Platform: "SAMSUNG Exynos7420"
Features: fp
Config Status: cfg=new, avail=yes, need=no, active=unknown

04: None 00.0: 10103 CPU
[Created at cpu.343]
Unique ID: rdCR.j8NaKXDZtZ6
Hardware Class: cpu
Arch: AArch64
Vendor: "ARM Limited"
Model: 0.2.0 "AArch64 Processor rev 2 (aarch64)"
Platform: "SAMSUNG Exynos7420"
Features: fp
Config Status: cfg=new, avail=yes, need=no, active=unknown

05: None 00.0: 10103 CPU
[Created at cpu.343]
Unique ID: rdCR.j8NaKXDZtZ6
Hardware Class: cpu
Arch: AArch64
Vendor: "ARM Limited"
Model: 0.2.0 "AArch64 Processor rev 2 (aarch64)"
Platform: "SAMSUNG Exynos7420"
Features: fp
Config Status: cfg=new, avail=yes, need=no, active=unknown

06: None 00.0: 10103 CPU
[Created at cpu.343]
Unique ID: rdCR.j8NaKXDZtZ6
Hardware Class: cpu
Arch: AArch64
Vendor: "ARM Limited"
Model: 0.2.0 "AArch64 Processor rev 2 (aarch64)"
Platform: "SAMSUNG Exynos7420"
Features: fp
Config Status: cfg=new, avail=yes, need=no, active=unknown

07: None 00.0: 10103 CPU
[Created at cpu.343]
Unique ID: rdCR.j8NaKXDZtZ6
Hardware Class: cpu
Arch: AArch64
Vendor: "ARM Limited"
Model: 0.2.0 "AArch64 Processor rev 2 (aarch64)"
Platform: "SAMSUNG Exynos7420"
Features: fp
Config Status: cfg=new, avail=yes, need=no, active=unknown

08: None 00.0: 10103 CPU
[Created at cpu.343]
Unique ID: rdCR.j8NaKXDZtZ6
Hardware Class: cpu
Arch: AArch64
Vendor: "ARM Limited"
Model: 0.2.0 "AArch64 Processor rev 2 (aarch64)"
Platform: "SAMSUNG Exynos7420"
Features: fp
Config Status: cfg=new, avail=yes, need=no, active=unknown
android@localhost:~/certbot$

also did you build openblas and/or numpy from source or use binary packages (from where?) ?

Only executed pip install numpy==1.26.0 (version required by Home Assistant).

Hmm. Not that special.. I do not have a rooted phone for this but I can see if I can test in termux.
I rather suspect this is a build error of some sort in the pip-packaged openblas binary (from one of the related tickets, it appears to be DYNAMIC_ARCH - which certainly makes sense - but with the build target for the common code parts left at NeoverseN1 (probably its build host) which might have led the compiler to choose some unfortunate optimizations.

Not reproducible on Odroid C4 (4x CortexA53) running Ubuntu 22.04LTS

also not reproducible on same system with python 3.11 (after figuring out deadsnakes repo & getting a working pip 3.11)

Unable to test under Android/termux as this ends up being a numpy build from source, which fails (apparently due to various idiosyncrasies of the platform). I see no reason to expect any different results from that A53/A57 combo compared to the A55 in any case, as none of the cpu-specific BLAS kernel code would get invoked just from loading the library.

@brokeDude2901 if you have the same issue, can you please add the exact CPU details of your machine, as well as how you installed numpy and the output of import numpy; print(numpy.show_runtime())?


Allow emacs commands for formatting/allow using eglot-format-buffer?

I'd like to be able to use eglot-format-buffer as the command to format the buffer on save.
I'm currently using:
# credit: https://go.googlesource.com/tools/+/refs/heads/master/gopls/doc/emacs.md
(defun elrod/eglot-format-buffer-on-save ()
  "Call `eglot-format-buffer` when saving."
  (add-hook 'before-save-hook #'eglot-format-buffer -10 t))
(add-hook 'eglot-managed-mode-hook #'elrod/eglot-format-buffer-on-save)

but would like to switch to aphelia.
The motivation for going through eglot rather than using the formatter directly is Julia.
Julia has a lot of JIT latency.
Running a script that loads and runs the formatter; fmt.jl:
#!/usr/bin/env -S julia -O1
import JuliaFormatter
foreach(JuliaFormatter.format, ARGS)

takes seconds.
$ time fmt.jl ~/.julia/dev/LoopVectorization/src/broadcast.jl
2.755 secs

However, with eglot, the LanguageServer's persistence means that future formats of the same project are reasonably fast.
I can emulate this via starting a Julia daemon
$ time juliaclient fmt.jl ~/.julia/dev/StrideArrays/
0.467 secs

but, then needing to start and manage the daemon is a bit of a hassle.
Or trying to compile the script into an executable, which is probably an even bigger hassle.
This same slowness is also much of aphelia's appeal: my approach for formatting on save locks up emacs for those seconds while the formatter runs!
Apherlia's approach of running it in the background is much nicer, but I'd still rather not have it take seconds on principle.
(Support for more formatters than supported by eglot is also appealing).

Apheleia does support Elisp based formatters, can you integrate it that way?

Thanks for pointing that out, not sure how I missed it.
I tried
(use-package apheleia
  :config
  (apheleia-global-mode +1))
(require 'cl-lib)
(cl-defun apheleia-indent-eglot-managed-buffer
    (&key buffer scratch callback &allow-other-keys)
  "Copy BUFFER to SCRATCH, then format scratch, then call CALLBACK."
  (with-current-buffer buffer
    (let ((tmp (generate-new-buffer "tmpbuffer")))
      ;; backup
      (copy-to-buffer tmp (point-min) (point-max))
      (eglot-format-buffer)
      ;; now we copy to scratch
      (copy-to-buffer scratch (point-min) (point-max))
      ;; and copy tmp back
      (with-current-buffer tmp
        (copy-to-buffer buffer (point-min) (point-max)))
      (kill-buffer tmp)
      (funcall callback))))

(add-to-list 'apheleia-formatters
                   '(eglot-managed . apheleia-indent-eglot-managed-buffer))

(add-to-list 'apheleia-mode-alist '(julia-mode . eglot-managed))

At least two problems with this:

After formatting, point moves to the start of the buffer. eglot's formatting is normally good about keeping it in place, and apheleia
It is inefficient, probably copying buffer contents a lot more than necessary.

I originally tried something like
(cl-defun apheleia-indent-eglot-managed-buffer
    (&key buffer scratch callback &allow-other-keys)
  "Copy BUFFER to SCRATCH, then format scratch, then call CALLBACK."
  (with-current-buffer buffer
    (copy-to-buffer scratch (point-min) (point-max))
    (eglot-format-buffer)
    )
  (with-current-buffer scratch
    (funcall (with-current-buffer buffer major-mode))
    (eglot-format-buffer)
    (funcall callback)))

which copies less often, but got a wrong-type-argument error from eglot when it looked for scratch's path. I probably messed something up here/need to set something else with scratch?
Formatting buffer itself works, but that means I need to first copy it to a scratch buffer, and then swap them, meaning 3 copies total instead of 1 (0 would of course be ideal).
I suspect the first problem is related, i.e. I shouldn't be formatting buffer and swapping, but need to setup scratch correctly?

So, the scratch buffer provided by apheleia--run-formatter-function will already have the contents of the original buffer copied into it. Also, you should be able to format that buffer directly. The thing to fix here is the

wrong-type-argument error from eglot when it looked for scratch's path
which I think you should be able to resolve by

(let ((buffer-file-name (buffer-local-value 'buffer-file-name buffer)))
  (eglot-format-buffer))
(funcall callback)

This will work with zero additional copies.
I think the issues with point getting reset to the beginning are because you're modifying the original buffer, which messes up point.
Does that seem right?

With
(use-package apheleia
  :config
  (apheleia-global-mode +1))
(require 'cl-lib)
(cl-defun apheleia-indent-eglot-managed-buffer
    (&key buffer scratch callback &allow-other-keys)
  "Copy BUFFER to SCRATCH, then format scratch, then call CALLBACK."
  (with-current-buffer scratch
    (let ((buffer-file-name (buffer-local-value 'buffer-file-name buffer)))
      (eglot-format-buffer))
    (funcall callback)))

(add-to-list 'apheleia-formatters
             '(eglot-managed . apheleia-indent-eglot-managed-buffer))
(add-to-list 'apheleia-mode-alist '(julia-mode . eglot-managed))

I get jsonrpc-error: "No current JSON-RPC connection".
Calling M-x eglot-format-buffer works.
JSON-RPC of course is used with the LSP, so something there seems to be going wrong.

I set up EGLOT on my end and fixed your advice:
(cl-defun apheleia-indent-eglot-managed-buffer
        (&key buffer scratch callback &allow-other-keys)
      "Copy BUFFER to SCRATCH, then format scratch, then call CALLBACK."
      (with-current-buffer scratch
        (setq-local eglot--cached-server
                    (with-current-buffer buffer
                      (eglot-current-server)))
        (let ((buffer-file-name (buffer-local-value 'buffer-file-name buffer)))
          (eglot-format-buffer))
        (funcall callback)))

Here's how I found out the right solution. First I enabled M-x toggle-debug-on-error to see the stacktrace of jsonrpc-error: "No current JSON-RPC connection". That showed me eglot--current-server-or-lose. I tested evaluating (eglot-current-server) in the buffer to make sure it was returning non-nil, as expected. Then the issue is the function is returning nil in the scratch buffer. Looking at eglot-current-server, the reason is pretty clear, it is looking at buffer-local variables and the major mode to look up the server connection. I added a step to copy the server context to the scratch buffer and it started working (very quickly too, great user experience!).

That works for me, many thanks!
Also, thank you for explaining how it was you debugged it.
toggle-debug-on-error is very useful -- I'll use that and take a look at the source next time I run into a problem.


VIP: Check get_current_blog_id() in pre_option_* filters

When adding a pre_option_* filter, we should try to check if get_current_blog_id() is being referenced.

Since the theme is being loaded even in non-standard situations (api, jobs system) you should always check if current blog_id matches the real blog_id of the site running the theme. The pre_option_ hooks you should be careful with are:

pre_option_blogname
pre_option_blogurl
pre_option_post_count

See https://vip.wordpress.com/documentation/code-review-what-we-look-for/#pre_option_

Closing here, since it is not related to anything currently in WPCS, but I've made a note of it for VIPCS.


Add header checks for generated files

PR summary
Generated files are excluded from checkstyle header checks. This PR includes them so that they are verified.
PR Checklist
Please make sure that your PR fulfills the following requirements:

[x] The commit message follows the
Angular Commit Message Guidelines.
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[x] New tests
[ ] Build/CI related changes
[ ] Documentation content changes
[ ] Other (please describe)

What is the current behavior?

What is the new behavior?

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

I didn't touch tslint, because it seems deprecated: https://github.com/palantir/tslint

I didn't touch tslint, because it seems deprecated: https://github.com/palantir/tslint

Thank you for the review, @vmatyusGitHub and @mojito317 ! I'm about to merge this.

Thank you for the review, @vmatyusGitHub and @mojito317 ! I'm about to merge this.


Seems the message handlers for subscribe are not run on a Vert.x event loop thread

found an issue in the NATS Vert.x client. Seems the message handlers for subscribe are not run on a Vert.x event loop thread. I see it is running on a NATS thread (nats:3 thread). In my view, the subscriber handler should run on the verticle context that subscribed. Or at least that is how other http servers or messaging clients work with Vert.x.

, found an issue in the NATS Vert.x client. Seems the message handlers for subscribe are not run on a Vert.x event loop thread. I see it is running on a NATS thread (nats:3 thread). In my view, the subscriber handler should run on the verticle context that subscribed. Or at least that is how other http servers or messaging clients work with Vert.x.
6 replies

	at io.nats.vertx.impl.NatsStreamImpl.lambda$subscribe$4(NatsStreamImpl.java:179) ~[nats-vertx-interface-1.1.1-beta12.jar:na]
	at io.nats.vertx.impl.NatsStreamImpl.lambda$null$5(NatsStreamImpl.java:194) ~[nats-vertx-interface-1.1.1-beta12.jar:na]
	at io.nats.client.impl.NatsJetStream$AsyncMessageHandler.onMessage(NatsJetStream.java:501) ~[jnats-2.15.6.jar:2.15.6]
	at io.nats.client.impl.NatsDispatcher.run(NatsDispatcher.java:97) ~[jnats-2.15.6.jar:2.15.6]
	at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:515) ~[na:na]
	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264) ~[na:na]
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128) ~[na:na]
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628) ~[na:na]
	at java.base/java.lang.Thread.run(Thread.java:834) ~[na:na]

rick

i will create an issue and take a look. Thank you sir.

Thanks

Scott

ok so subscribe handlers require a dispatcher. The dispatcher runs on it's own thread that it gets from the connection executor. You can control the executor from the connection options, but that's it.
There are ways to make your own dispatcher, it's not public scoped but I could do it and maybe make a public hook. You have to manage the dispatcher if you want to control where the subscriber handler runs.
:+1:
1

rick

 does that answer satisfy or do you want us to try to get the dispatcher running inside vertx

rick
  LMK if it is not. I added a number to this issue if you want to talk about it: https://github.com/nats-io/nats-java-vertx-client/issues/44 (You can refer it as 44)

scott says that i would just have to create a dispatcher on the same package.


Follow SOLID

Describe the bug
Remove the logic from the repositories and move it to a separate service.
Also add the test cases wherever necessary.
eg :- https://github.com/sourcefuse/loopback4-microservice-catalog/blob/master/services/authentication-service/src/repositories/user.repository.ts
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
We should follow SOLID and repositories should not have business logic within them.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

repositories to be updated:

authentication service > user.repository.ts
in-mail-service > message.repository.ts, thread.repository.ts
oidc service > user.repository.ts
search-service > recent-search.repository.ts, search-query.repository.ts
survey-service > section.repository.ts, survey-question.repository.ts, survey-responder.repository.ts
user-tenant-service > user.repository.ts

Along with these, the respective sanbox examples will also be updated.

sandbox examples to be updated:

auth-ms-basic-example > user.repository.ts
auth-multitenant-example > user-ext.repository.ts
oauth-example > user.repository.ts
telemed-app > backend > authentication-service > user.repository.ts


Result malformed.

diagrams are not drawn following this error:
after this query:
SELECT * from cypher('text_test', $$
          MATCH (V)-[R:connects]-(V2)
          RETURN V,R,V2
$$) as (V agtype, R agtype, V2 agtype);

$ SELECT * from cypher('text_test', $$ MATCH (V)-[R:connects]-(V2) RETURN V,R,V2 $$) as (V agtype, R agtype, V2 agtype);
Converting circular structure to JSON
--> starting at object with constructor 'Object'
--- property 'properties' closes the circle
info: ::ffff:<IP_ADDRESS> - - [22/Nov/2021:13:44:17 +0000] "POST /api/v1/cypher HTTP/1.1" 500 47
 {"timestamp":"2021-11-22 13:44:17"}
info: ::ffff:<IP_ADDRESS> - - [22/Nov/2021:14:06:22 +0000] "GET /api/v1/db HTTP/1.1" 200 134
 {"timestamp":"2021-11-22 14:06:22"}
info: ::ffff:<IP_ADDRESS> - - [22/Nov/2021:14:06:22 +0000] "GET /api/v1/db/metaChart HTTP/1.1" 500 2
 {"timestamp":"2021-11-22 14:06:22"}
info: ::ffff:<IP_ADDRESS> - - [22/Nov/2021:14:06:23 +0000] "GET /api/v1/db/meta HTTP/1.1" 304 -
 {"timestamp":"2021-11-22 14:06:23"}
error: syntax error at or near "MATCH"
    at Parser.parseErrorMessage (/app/backend/node_modules/pg-protocol/dist/parser.js:287:98)
    at Parser.handlePacket (/app/backend/node_modules/pg-protocol/dist/parser.js:126:29)
    at Parser.parse (/app/backend/node_modules/pg-protocol/dist/parser.js:39:38)
    at Socket.<anonymous> (/app/backend/node_modules/pg-protocol/dist/index.js:11:42)
    at Socket.emit (events.js:376:20)
    at addChunk (internal/streams/readable.js:309:12)
    at readableAddChunk (internal/streams/readable.js:284:9)
    at Socket.Readable.push (internal/streams/readable.js:223:10)
    at TCP.onStreamRead (internal/stream_base_commons.js:188:23) {
  length: 93,
  severity: 'ERROR',
  code: '42601',
  detail: undefined,
  hint: undefined,
  position: '1',
  internalPosition: undefined,
  internalQuery: undefined,
  where: undefined,
  schema: undefined,
  table: undefined,
  column: undefined,
  dataType: undefined,
  constraint: undefined,
  file: 'scan.l',
  line: '1129',
  routine: 'scanner_yyerror'
}
info: ::ffff:<IP_ADDRESS> - - [22/Nov/2021:14:14:52 +0000] "POST /api/v1/cypher HTTP/1.1" 500 47
 {"timestamp":"2021-11-22 14:14:52"}
info: ::ffff:<IP_ADDRESS> - - [22/Nov/2021:14:15:41 +0000] "POST /api/v1/cypher HTTP/1.1" 200 137
 {"timestamp":"2021-11-22 14:15:41"}
TypeError: Converting circular structure to JSON
    --> starting at object with constructor 'Object'
    --- property 'properties' closes the circle
    at JSON.stringify (<anonymous>)
    at stringify (/app/backend/node_modules/express/lib/response.js:1123:12)
    at ServerResponse.json (/app/backend/node_modules/express/lib/response.js:260:14)
    at _callee$ (/app/backend/build/controllers/cypherController.js:59:33)
    at tryCatch (/app/backend/node_modules/regenerator-runtime/runtime.js:63:40)
    at Generator.invoke [as _invoke] (/app/backend/node_modules/regenerator-runtime/runtime.js:293:22)
    at Generator.next (/app/backend/node_modules/regenerator-runtime/runtime.js:118:21)
    at asyncGeneratorStep (/app/backend/node_modules/@babel/runtime/helpers/asyncToGenerator.js:3:24)
    at _next (/app/backend/node_modules/@babel/runtime/helpers/asyncToGenerator.js:25:9)
    at processTicksAndRejections (internal/process/task_queues.js:95:5)
info: ::ffff:<IP_ADDRESS> - - [22/Nov/2021:14:16:03 +0000] "POST /api/v1/cypher HTTP/1.1" 500 154
 {"timestamp":"2021-11-22 14:16:03"}
info: ::ffff:<IP_ADDRESS> - - [22/Nov/2021:14:16:13 +0000] "GET /api/v1/db/meta HTTP/1.1" 200 217
 {"timestamp":"2021-11-22 14:16:13"}

Funny, I ran into the same error today as well.
What I found is that it works fine as soon as the relations have at least one property set.
I.e. I get the graph displayed when creating the relations like this:
MATCH (a:Person),(b:Book) WHERE a.name='Jack' AND b.name='Hitchhiker' CREATE (a)-[r:Owns {name: a.name+'->'+b.name} ]->(b)
while the following results in the described error:
MATCH (a:Person),(b:Book) WHERE a.name='Jack' AND b.name='Hitchhiker' CREATE (a)-[r:Owns]->(b)

Just noticed: This is most likely the same issue as #9

so, each edge must be added a property?

@david-m-m I have a looping issue, can you please have a look at it?

so, each edge must be added a property? Thanks for the insight, I would have to amend my code.

Looks like it. Though I'm not familiar with the code, found out more by trial-and-error.
Hopefully the developers of this project can gain & give more insight into this issue.

@david-m-m I have a looping issue, can you please have a look at it?

First: I'm not a developer and am not (yet?) deep into the matter.
But I had issues addressing objects by their ID (when fiddling around on the console). So it may not be possible or work different that what we think.

Thanks for waiting response.
I can't reproduce this issue. Are you using Docker?

yes, i'm using docker here,

@afidegnum
I just updated it on latest tag. I'll close this issue :)


Reworked deploy-host role to work on CentOS hosts

What does this PR do?

Rearranged enablement of repositories to perform step once
Rearranged installation of packages to perform step once
Provider specific repos are appended to required repos
Centos hosts will install openshift origin release repo
Logic added to avoid running subscription-manager on non RHEL

How should this be manually tested?
Set up a workstation host using CentOS or RHEL and run playbook for provider.
Tested:
CentOS/rhv
RHEL/{rhv,aws,gcp,ocp,vsphere}
Is there a relevant Issue open for this?
Request by @mykaul to enable proper handling of CentOS in deploy-host playbook
Who would you like to review this?
cc: @dav1x @cooktheryan @mykaul PTAL

 (1) Ensure that deploy-host playbook works for CentOS hosts


⚠ Minecraft has degraded performance

In 0b4ea04, Minecraft (https://www.minecraft.net) experienced degraded performance:

HTTP code: 200
Response time: 1776 ms

Resolved: Minecraft performance has improved in e630364.


Error [ERR_PACKAGE_PATH_NOT_EXPORTED]: Package subpath './lib/tree-adapters/default' is not defined by "exports"

I get the following error when trying to import the module:
node:internal/errors:484
    ErrorCaptureStackTrace(err);
    ^

Error [ERR_PACKAGE_PATH_NOT_EXPORTED]: Package subpath './lib/tree-adapters/default' is not defined by "exports" in /home/luis/app/node_modules/parse5/package.json
    at new NodeError (node:internal/errors:393:5)
    at throwExportsNotFound (node:internal/modules/esm/resolve:340:9)
    at packageExportsResolve (node:internal/modules/esm/resolve:619:3)
    at resolveExports (node:internal/modules/cjs/loader:492:36)
    at Module._findPath (node:internal/modules/cjs/loader:532:31)
    at Module._resolveFilename (node:internal/modules/cjs/loader:941:27)
    at Module._load (node:internal/modules/cjs/loader:803:27)
    at Module.require (node:internal/modules/cjs/loader:1021:19)
    at require (node:internal/modules/cjs/helpers:103:18)
    at Object.<anonymous> (/home/luis/app/node_modules/css-select-parse5-adapter/lib/css-select-parse5-adapter.js:16:28) {
  code: 'ERR_PACKAGE_PATH_NOT_EXPORTED'
}

Node.js v18.8.0

It seems like something changed in parse5 and now this package doesn't work.

@luiscastro193
If you want it to work somehow, I dropped the parse5 version to 6.0.1 and it worked.


Close button styling, Background Colour, and Hover not working

I'm having a few problems.
Firstly, this may just be an overall thing, but there seems to be a square around the "close tab" button

Secondly, I'm having trouble getting my background to change, or the sidebar to collapse. I've attached my chrome folder below.
chrome.zip
I appreciate any help!
Thanks for making such an awesome project. Hopefully, I can get it working soon :)

Sorry about the super late reply. The box around close button might have been some change in tst update. I am a little busy these days but I'll try to look at it next week.
As per the sidebar issue, the most glaring problem I see is your vars file is actually named var.css while it's supposed to be vars.css. If fixing that doesn't work then maybe move the userChrome-hover import in your userChrome after all the other imports in case something else is also styling sidebar.

Hi! renaming vars.css seems to have fixed the sidebar collapsing! One new problem the menu toolbar seems to be always showing at the top of the screen, to the left of the nav buttons.

Also, there are some spacing issues with the nav buttons on windows.

Again these are minor aesthetic issues that I can probably fix myself, I just wanted to let you know. I really appreciate all the work that you've put into this project! Much love ❤

I think the menu bar usually only pops up when you press alt. If you don't want it to show up at all you can set ui.key.menuAccessKeyFocuses to false in about:config and that should fix it.
If you check title bar in Firefox customization, the controls should show up on a separate title bar.


reduce docker image size

Clean cache to reduce image size

Thanks for the pull request
It's not clear that this slight decrease in readability is worth the few hundred MB of disk space that we're going to save because of the --no-cache.
Unless there is a strong reason, like increase speed for the tests or saving 1~2 GB of disk space, I would prioritize readability over disk space.
If you're familiar with the dockerfiles, something you could help with is finding a way to centralize the version numbers of packages. Currently they're all over the place and when we'll need to bump them, it will be easy to forget one. Maybe putting them in multiple requirement.txt files?

Understood. Thanks @gabrieldemarmiesse.


Not supported for v1.0 to openapi3

@imranshaik012  I will keep as a feature request but I think having support for v2.0 and v2.1 is more than enough, so will be the last feature in the list sorry. You can use postman to migrate your collections from v1 to a newer one.

Thanks

As v1 has been deprecated by postman don't have sense to support in our library, the approach is to migrate to V2 version as described in postman documentation https://learning.postman.com/docs/getting-started/importing-and-exporting-data/#converting-postman-collections-from-v1-to-v2


pip install spyder does not work any longer (pip version >=1.5)

From<EMAIL_ADDRESS>on 2014-01-19T08:24:34Z
ufechner@uwe-desktop64:~$ sudo pip install -U spyder
Could not find any downloads that satisfy the requirement spyder in /usr/local/lib/python2.7/dist-packages
Some externally hosted files were ignored (use --allow-external spyder to allow).
Downloading/unpacking spyder
Cleaning up...
No distributions at all found for spyder in /usr/local/lib/python2.7/dist-packages
Storing debug log for failure in /tmp/tmpLUids7
ufechner@uwe-desktop64:~$
The suggested solution also fails:
ufechner@uwe-desktop64:~$ sudo pip install -U --allow-external spyder
You must give at least one requirement to install (see "pip help install")
ufechner@uwe-desktop64:~$
More info:
ufechner@uwe-desktop64:~$ pip --version
pip 1.5 from /usr/local/lib/python2.7/dist-packages (python 2.7)
ufechner@uwe-desktop64:~$ http://www.pip-installer.org/en/latest/news.html Ubuntu 12.04 64 bits
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=1701

From contrebasse on 2014-01-20T04:56:01Z
The correct command is:
pip install -U --no-install --allow-external spyder --allow-unverified spyder spyder
I guess that a mirror should be made on pypi as pip doesn't allow the installation from external links by default.

From ccordoba12 on 2014-07-10T18:00:13Z
Ok, I fixed this one for 2.3. Enjoy!
Status: Fixed
Labels: MS-v2.3

From contrebasse on 2014-01-21T00:14:27Z
Sprry, the correct command is :
pip install -U --allow-external spyder --allow-unverified spyder spyder


Sarif Output Format Support

Is your feature request related to a problem? Please describe.
I would like to get the output of ggshield in SARIF format, so it can be process afterwards.
My goal is to unify scanning experience with multiple scanning  tools, so i can show aggregated vulnerability information (number of issues and their severity)  on github workflow runs and / or on PRs.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Current json output is undocumented and does note define a schema
Describe the solution you'd like
Provide option to define additional output format as SARIF instead of json.

Hi @Jiri-Stary
FYI GGShield 1.30 is now supporting an XML output (SARIF format)


Document Room.Exists

Needs to be included in Room entry.

Added here:
https://github.com/adventuregamestudio/ags-manual/wiki/Room#roomexists

Need to indicate it was added in 3.6.0

Added https://github.com/adventuregamestudio/ags-manual/wiki/_compare/4e1fe1c91439c4ca1c184ead90baa378e4a2ae2a...bede3bf58d910efaa0e2f0bd8ac6675ab5371713


Grafana roadmap

In reference to #115
The current behavior of grafana-watcher relies on a ConfigMap as the master dashboard repository.
When you design a new dashboard in Grafana you need to manually export it and merge it into the ConfigMap manifest, which is quite tedious to do. Is there a plan to automate this process ?
In addition, while going through this process, I noticed that we were using the Grafana /api/dashboards/import API, which seems to be undocumented in the Grafana API reference. Why aren't we using the documented API?

The [POST] /api/dashboards/db endpoint was also undocumented at the time of implementing the grafana-watcher :slightly_smiling_face: . As far as I can tell they have the exact same functionality, so it should just be replaceable.

Is there a plan to automate this process ?

Glad you ask, because it already is. It is part of kube-prometheus and the scripts to do that can be found in hack/scripts. You are particularly looking for the generate-dashboards-configmap.sh script executed from the generate-manifests.sh script.

Now using the documented API and he workflow is already automated :)

It turns out, that the /import endpoint actually slightly differs, and does not allow templating, this didn't show in my tests, but in more complex dashboards it does appear. Will have to revert to the /import endpoint again.


Docs: Bump up maxWidth

Increases max width of our docs to our largest break point
Note: This PR makes the PageHeader docs un-ideal (but still useable) - can fix that in a follow up PR
Test Plan

This is 😍 in general! Two concerns:

Text blocks look a little funky, splitting into a left-aligned column for no apparent visual reason. Not sure what the appropriate fix is; we don't want full-width text and centering that column would be a bit odd.
The three column sections could stand to be center-aligned, I think?


App not recognizing presence of audio files

When I select a directory containing audio files, no button appears to allow auditing.

This should now be fixed. The shinyjs::show() function was masked preventing the button being displayed. If problems persist please let us know.


how to switch between video calls and audio calls ?

Dear muaz,
Now I meet a problem. Two users talk on video, and when their network environment is poor, their connection is disconnected. I want to make a button. If two users have a video call, when their network environment is poor, I click the button and let them have an audio call, can you help me? In summary, it is how to switch between video calls and audio calls without refreshing the page.

How to switch
connection.session = {
audio: true,
video: true,
data: true
};
connection.mediaConstraints = {
audio: true,
video: true
};
to
connection.session = {
audio: true,
video: false,
data: true
};
connection.mediaConstraints = {
audio: true,
video: false
};
when the page not reload.


Add EDHEC Business School (https://www.edhec.edu)

this is one of the pages indexing the publications of the school: https://www.edhec.edu/fr/recherche-et-faculte/centres-et-chaires/chaire-prevision-innovation-et-transformation/publications
here is one the articles : https://www.tandfonline.com/doi/abs/10.1080/13675567.2022.2088707
on "Exploration of barriers and enablers of blockchain adoption for sustainable performance: implications for e-enabled agriculture supply chains"
the author is: Sachin S. Kamble with the email<EMAIL_ADDRESS>

@fidelio314 First stage of verification passed, official domain is edhec.edu as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@fidelio314 Thank you for your patience while waiting for a review and processing your request. Pull request merged.


window.navigator.languages returns null on Internet Explorer / Edge

It appears window.navigator.languages returns null on IE/Edge. Thus there doesn't seem to be a way to get at the user's language preferences on IE.
Should navigator abstract away what browser we are running on and do whatever IE specific magic it needs to retrieve this setting for us?

I'm going to close this as IE and legacy Edge are no longer supported.
All modern browsers support this API now and for posterity, legacy Edge added support sometime during EdgeHTML 16 release cycle.
If anyone has issues using these libraries on recent browser versions, please open a new issue. Thanks! :D


Issue with aggregated table

AggregatedData seems to be not ready to handle table data. There are two issues that I encountered when I tried to do that:

I got the error message that: The 'name' entry is missing for constructing a file name
https://github.com/equinor/fmu-dataio/blob/423628941dba731396f8ff4952e6c6f68cc65539/src/fmu/dataio/_filedata_provider.py#L120-L121
This seems to be originating from generating metadata using fakeconfig.
https://github.com/equinor/fmu-dataio/blob/423628941dba731396f8ff4952e6c6f68cc65539/src/fmu/dataio/dataio.py#L1196-L1197

It works fine with test case because for some reason, aggregated surface object has a name in it (the name is "unknown"). From what I can see, this name is not being used in this case, so perhaps the easiest way to fix it is to assign a dummy name or we can just assign the name from dataio object.
       etemp = ExportData(config=fakeconfig, name=self.name)

It assumed that there is entry bbox in the data
https://github.com/equinor/fmu-dataio/blob/423628941dba731396f8ff4952e6c6f68cc65539/src/fmu/dataio/dataio.py#L1210
We can probably fix it by checking for the existence of bbox

        if etempmeta["data"].get("bbox"):
            template["data"]["bbox"] = etempmeta["data"]["bbox"]

Having fixed those issues and got the aggregated metadata, I see that the parameters is not transferred to the aggregated metadata. It could be interesting to see the list of parameters that is input to the aggregated data. From the webviz side, it's probably easier for webviz to extract this information and use it for the various plugins that they already have. Or perhaps there are other reasons for not transfer those information?

Ref 3), including parameters on aggregated data. This is not straight forward, and needs a bit of thinking and discussion. I am unsure how to actually do this properly, as the parameters will be different for all the input data objects. So, the parameters for an aggregated object is really the distribution of each parameter from the source objects.

Ref 1), I think this should trigger the user to actually enter a name in the call to dataio. Perhaps the error message could be improved a bit.

Ref 1), I think this should trigger the user to actually enter a name in the call to dataio. Perhaps the error message could be improved a bit.

But from what I can see, the name is not being used anywhere (only file and tracklog field is extracted from generated etempmeta) So, it is probably fine to just assign any arbitrary name
https://github.com/equinor/fmu-dataio/blob/423628941dba731396f8ff4952e6c6f68cc65539/src/fmu/dataio/dataio.py#L1196-L1212

Ref 3), including parameters on aggregated data. This is not straight forward, and needs a bit of thinking and discussion. I am unsure how to actually do this properly, as the parameters will be different for all the input data objects. So, the parameters for an aggregated object is really the distribution of each parameter from the source objects.

Exactly. So, I was thinking that the parameters in the aggregated class object could be the list of the source object parameters. This will be analog to aggregating parameters.txt in the scratch disc. The drawback is that the metadata could potentially become too big.

Fixed with PR #254


TikTokLive - Patch Report for October 3rd, 2022

TikTokLive - Patch Report for October 3rd, 2022
As per usual, here's a brief explanation as to why the library broke and what we did to fix it.
TikTokLive works by connecting you, the user, to TikTok's internal API via Websocket. When TikTokLive and TikTok-Live-Connector came out earlier this year, TikTok were not too thrilled. They added authentication to their Webcast API to prevent users from using our projects.
Our response was to reverse-engineer their authentication. Zerody figured out how to do this, and together we hid the method behind an API, known as the "Signing API" which signs the requests and returns valid tokens. This way, TikTok would not be able to see what we were doing to reverse engineer their authentication and we could continue without them trying to block us.
Timeline of Changes
Friday, September 30th
TikTok released an update with the intention of completely stopping the library in its tracks. They essentially added a bunch of hidden checks, that, if caught in, would make our generated signatures fail. They even added checks to make it harder to see what checks they added.
Saturday, October 1st
We investigated what was going wrong. A fix wash pushed out. The fix managed to get around some issues, but was still being detected.
Sunday, October 2nd
Today, I am happy to say TikTokLive is once again operational, as is TikTok-Live-Connector.
Monday, October 3rd
TikTok are starting to block the library's usage again.

Supporting Development
TikTokLive has always been free to use and free for Zerody and I to run. Zerody is busy with a full-time job, and I am writing my first-ever midterm in University tomorrow. We both took a lot of time to fix this, for free. Time I did not have.
As of this most recent fix, it now also costs us money to run this library, thanks to our new method of token generation.
If you enjoy this library (and perhaps even made a coin or two from it), consider supporting its development with a donation at https://www.paypal.me/isaackogan. Half will go to me, and the other have to Zerody, the other developer responsible for running this project & TikTok-Live-Connector.

@isaackogan I see that it is not working again, you know when it will work again?

I hope it will be resolved soon.

ye this tiktok devs have nothing to do else anyway even if it works the rate limiting is badly im building a livestream viewers script and this is really bad, i need premium access after this is fixed i pay for it.

Fixed, see updated report

viewerbot not working anymore please help sir

viewerbot not working anymore please help sir

Unfortunately, too bad L

Happy news, after a while of work, we've cracked Tiktok's latest signature algorithm, you can use our endpoint at here


proposal: Write parts of KISS in C for portability/speedup

Write all of kiss in C :P

Relevant:

The C package manager implementation is still in the works though its
development process has changed. A library will be created and slowly over
time individual components of the shell package manager will be replaced with
C equivalents (while the library matures at the same time).
Users will be able to opt-in to running parts of the package manager (the slow
components especially) in C. This allows users to benefit from faster
implementations of components as they become available.
Once the library reaches a good level of usability, a full C package manager
implementation will be made available alongside the shell implementation.
A test-suite will also be created and will be used both by the shell and C
package manager implementations to ensure that no regressions are made between
changes.

Dylan, 11 Jul 2020

https://codeberg.org/kiss-community/repo/issues/91


Editing layouts for specific actions is counter-intuitive

If I want to currently chage the layout for user/registrations#edit to dashboard, I have to add this to the config:
config.to_prepare do
    Devise::SessionsController.layout proc{ |controller| action_name == 'edit' ? "dashboard"   : "application" }
end

This is counter-intuitive. The right way is to edit it like any other controller:
  def edit
    super
    render layout: 'dashboard'
  end

Which currently gives fails with AbstractController::DoubleRenderError because the super probably renders with default in the code. At the very least a helper function should be included to override the default.

I found a better way, do this:
class User::RegistrationsController < Devise::RegistrationsController
  layout 'dashboard', only: [:edit]
  ...
end

Feel free to add this to the documentation! It's better than the methods described!

@amingilani Thank you for the suggestion. Do you remember where did you see these other methods to change the layout?

In the wiki.. sighh, I should've edited that directly. I just realized.

Done, here

@amingilani Awesome! ❤ 💛 💙 💜 💚


The year in the background of chart and over the slider is not same while slider is moving

SEVERITY:  Major
MODULE:
*Bubble chart
*Mountain chart
*Bubble Map
Release: v0.13.3
Platform: IPhone 4 / Safari 9
STEPS TO REPRODUCE:

Navigate to the URL : http://static.gapminderdev.org/vizabi/release/v0.13.3/preview/bubblechart.html
Bubble Chart will be loaded.
Click on USA and china bubble
Click on play and generate trials
Observe the year on the slider

ACTUAL RESULTS:
The year appears over the slider is not the exact as the year appeared in the background
EXPECTED RESULTS:
The year over the slider and in the background should be same while the slider is moving
SCREENSHOT:

https://github.com/Gapminder/vizabi/pull/1596

@sergey-filipenko  the bug persists on OSX + Chrome 49 (mountain chart and bubble map)

https://github.com/vizabi/vizabi/pull/2135

@sergey-filipenko please also fix it on bubble chart now and retest on bar rank chart

https://github.com/vizabi/vizabi/pull/2196

https://github.com/vizabi/vizabi/pull/2200


Bug: Testing of the Navlinks that sorts the displayed data on the homepage

Describe the bug

On your laptop, open this project with your code editor, paste gh pr checkout 57 in the terminal and run it, to have a copy of the changes reflecting on this pull request #57
run npm ci then npm start to spin up a server and go to localhost:3000 to start testing. If you click on any of the secondary nav links it should highlight that link alone with a yellow background

Steps to reproduce

Click on any of the links (Popular | Upvoted | Discussed | Recent) to make sure it sorts the cards based on the criteria given on issue #41
Popular and Upvoted Link should return the same result, while Discussed and Recent should return different order of the cards displayed
If you refresh the page, it should return back to default

Popular

Upvoted

Discussed

Recent

Affected services
opensauced.pizza
Platforms
Desktop, Mobile, Tablet
Browsers
Chrome, Firefox, Safari, Edge, Opera
Environment
Development
Additional context
No response
Code of Conduct

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

.take

To be clear, to resolve this issue we need UI tests. The open sauced repo has some https://github.com/open-sauced/open-sauced/tree/main/src/tests

https://github.com/open-sauced/hot/pull/82/files#r782156878

Reopened, since tests were not added in #82. The approach can use the current static files as mock data and the filter can be tested as a unit test checking the order of the array.
https://github.com/open-sauced/hot/pull/82/files#r782156878

Going to unassigned for now. This is probably dependent on the new API changes as well. FYI for anyone who is picking this up. https://github.com/0-vortex/api.opensauced.pizza

This has been merged to beta. FYI, moving forward we want to follow this pattern. Also added this text to the README.

For writing tests, the rule is move business or service logic to the lib folder and write unit tests. Logic that needs to be in a React component, then leverage tools like Cypress or Vitest mocking to write tests.


Add initial locatableContent traits implementation

Adds locatableContent traits to allow us to start building e2e file-based workflows through OpenAssetIO. Required for https://github.com/TheFoundryVisionmongers/OpenAssetIO/issues/347.
The setup is currently a little manual, as we don't have wheels for OpenAssetIO yet, so simple installation, and CI will come shortly. More here.
We wanted to get these up here to facilitate discussion around the topic sooner rather than later. See the into notes here, and a discussion on the changes in the design to the original DR sketch are in the commit message here (note, that message uses the stable URL for the DR post merge of this PR).

Do we want to merge into main, though, given this currently relies on a feature branch in the OpenAssetIO repo?

I think I'd rather, and just note that we need the other branch of OpenAssetIO - otherwise its just more mess to clean up later - we do say this is in "pre-alpha" state. Added a note here.


Wrong content when updating locales files

When change the content of our language json file locally and deploy it on our production theme we do not get the changes applied. Even more strange if you change the file before in "Edit Code" in Shopify it gets replaced with an old version of the file, not the one edited and saved nor the one pushed from our local machines.
Steps to reproduce the behavior:

Change file content locally
deploy file with themekit

Expected behavior
local changes should be applied to the theme
Environment (please complete the following information):

Environment: MacOS 10.15
Themekit version: ThemeKit 1.1.1 and ThemeKit 1.0.3
Editor: SublimeText3 and PHPStorm

Recorded a video in loom where you can see the case
https://www.loom.com/share/586a4b2c814044e5a6af1d0609a4317a

Hi @stefankunze
Thank for the detailed report, and especially the video.
I'm wondering, when you click on Older Versions in the online editor, do you see any revisions containing the updates you made locally?

@andyw8 Thanks for your response - we also checked the older versions, but the content uploaded wasn't in there either.

Thanks, I've checked our logs for that particular asset and it seems the update is not reaching our servers.
I'd like to first determine if it's related to the watch command or not. Could you try stop the watch, make an edit, and then push it using using theme deploy locales/en.default.json --verbose, and share the output.
(also, if you're on the Partners Slack then you can message me on there, might make it quicker to resolve this).

@andyw8 I also have a locale related issue (ticket id 20789131). And I was able to understand it a bit more.
When I use the watch command for my theme, if I update a locale in the shopify admin interface, my local locale files are not updated. The theme watch command does not download locale updates from shopify.
If I do a brand new theme get command then I do get the updated locale files. BTW I think this watch issue is not just for locale files but for a lot of other files, because at some point I did a deploy of my watched theme and it broke A LOT of my live theme, most changes done on the shopify admin UI itself were not reflected in my local files..

Finally, a brand new get command right now get stuck at 97% (theme id:<PHONE_NUMBER>55) and never finishes.

Please see #801

When I use the watch command for my theme, if I update a locale in the shopify admin interface, my local locale files are not updated. The theme watch command does not download locale updates from shopify.

I was reviewing issues and just wanted to point out that this is not how theme watch works. Shopify does not send themekit events of your changes on shopify. If you want changes made in the admin, then you will need to run theme download.
Watch will only watch for you local changes, and we will not be implementing any sort of events from shopify to your local system.

When I use the watch command for my theme, if I update a locale in the shopify admin interface, my local locale files are not updated. The theme watch command does not download locale updates from shopify.

I was reviewing issues and just wanted to point out that this is not how theme watch works. Shopify does not send themekit events of your changes on shopify. If you want changes made in the admin, then you will need to run theme download.
Watch will only watch for you local changes, and we will not be implementing any sort of events from shopify to your local system.

@stefankunze

When change the content of our language json file locally and deploy it on our production theme we do not get the changes applied. Even more strange if you change the file before in "Edit Code" in Shopify it gets replaced with an old version of the file, not the one edited and saved nor the one pushed from our local machines.

This all sounds like an intermittent cache/cdn failure. Is this problem still happening? What shopify does when you make changes to a file is that they get sent to a cdn to be served for your site however sometimes there are connection issues or other intermittent issues that cause files to not get update or delayed.

@stefankunze

When change the content of our language json file locally and deploy it on our production theme we do not get the changes applied. Even more strange if you change the file before in "Edit Code" in Shopify it gets replaced with an old version of the file, not the one edited and saved nor the one pushed from our local machines.

This all sounds like an intermittent cache/cdn failure. Is this problem still happening? What shopify does when you make changes to a file is that they get sent to a cdn to be served for your site however sometimes there are connection issues or other intermittent issues that cause files to not get update or delayed.


emsdk_portable is now apparently emsdk-portable

Hi there,
The emscripten SDK now untars to a directory named emsdk-portable, at least on OS X. Fips thinks it's named with an underscore instead.
The attached patch makes the obvious change to get things working, but I have only tested it on OS X.
emsdk.patch.txt

Just realised I put this in oryol project instead of fips. Sorry about that.

Ouch, good catch. Thanks a lot for the report, fix is on the way!

Ok, I have committed the fix to fips, and I am currently testing on OSX, Linux and Windows. Thanks again!


Issue with Meteor.call()/Meteor methods after page reload

I was wondering if you have ever experienced an issue like the following.
When people accept our experiment HIT, we initially keep them in the lobby and we set their lobby status to "not ready". Before their status is changed to "ready" they have to go through a couple of pages (one with a short outline of the HIT, and another one with full description of the game). They navigate from one page to the next by simply clicking a button. When the corresponding button is clicked, we invoke a Meteor method on the server side (using Meteor.call()) which updates the document of that user in a collection which keeps track of the user's progress before their lobby status is changed to ready, and this document is used to render the appropriate template on the client side (much like the "Recruiting" collection is used in the Prisoner's Dilemma app of @ldworkin).
Everything seems fine if people do not reload their page - they easily navigate through all these pages. However, if, for some reason, they reload their page, the Meteor.call() statement invoked upon a button click, does not start the execution of the corresponding server-side Meteor method (though, the click event itself is registered on the client side). The funny thing is that upon restarting the server (which I do by simply re-deploying the app on Modulus), all calls to Meteor methods are processed (multiple clicks before the restart result in equally many executions of the corresponding Meteor method). It is as if they are postponed until the client-server connection is "properly" re-established with the server restart.
Do you have any ideas on where the problem may lie?
Any help would be very much appreciated!

I think this may be caused by one of the following:

#44: a temporary bug which I will fix soon. You can see if this is the problem by forcing<EMAIL_ADDRESS>and seeing if the issue goes away.
#37: sometimes multitasking users will open multiple tabs by accident and this can break the login tracking. I know what is causing this but haven't got around to fixing it yet. You can help with that!

Have you been able to reproduce the state on a client which makes the methods non-responsive? I think that is the first step, and if you can do so then we are 90% of the way to a fix.

@ZJoveski: have you managed to track down anything more about this?

No, unfortunately. I am pretty sure the issue is not<EMAIL_ADDRESS>as I am still using the pre-Meteor 1.2 TurkServer version and<EMAIL_ADDRESS>does appear in the "versions" file. The issue is definitely not limited  to multitasking users.
I have been able to reproduce the problem using my worker account on the MTurk sandbox - it really only takes one reloading of the screen and the issue appears.
I have been trying hard to isolate the problem by stripping our application of as much of the code as possible. My best guess is that it has to do something with the publications on the server-side as removing the file containing the publications of collections with experiment-related data, eliminates the problem with button-reponsiveness on the screens prior to the actual experiment. I have tried removing the publications from the file one-by-one, but that's where I get lost, because I could not identify which collection (or a group of collections) is causing the problem.
Perhaps the problem is with the number of publications I have...
I will try to create a repository with the simplest version of the code with which I am able to reproduce the error by the end of today, so that if you have a chance, you could have a look at it.
In the meantime, we tried running some tests where we ask people to not reload their screens and potentially exclude those who gave this issue due to external connection loss from the actual experiment, but we found out that the problem is not limited to just the navigation in the pre-experiment phase - it also appeared in the actual experiment (people we not able to change colors), and it would likely become unmanageabla in this way for significantly larger experiments.

Is your repo public? If so, could you just point me to the code and I can see if anything is strange?
And just to be clear, this issue still happens if the user only has one tab open and they just reload it?

Yes, the repo is public. Here is the file containing the publications
https://github.com/ZJoveski/DC/blob/master/server/app/publications.js
Alse, here is where I am doing the partitioning of the collections
https://github.com/ZJoveski/DC/blob/master/lib/models/app/app_models.js
And yes, this issue still happens if the user only has one tab open and they just reload it (or there is a temporary loss of connection).

I think the following issue may be related to the above one. Sometimes, if I log out from the TurkServer admin account and try to log in again, nothing happens (I do not get to the TurkServer Overview page). But if I restart the app from the server while I am at the login screen, the login request ( Meteor.loginWithPassword) gets processed when the app restarts and I get to the TurkServer Overview page.

I think your publications and collections generally look fine - except maybe the messages publication; if you are trying to do some sort of reactive join, that is not the way to do it. Only the cursor you send will get updated and you will have to make sure to re-subscribe to this whenever the experiment, neighborhood, etc. changes.
With regard to the login problem, do you mean that if you log out and enter the password to log in, nothing happens? That's strange, it sounds like there is some protocol error that is causing the clients to lose connection to the server.
With regard to reproducing the error, is that only when you are running in the sandbox, or can you also reproduce it running locally without the sandbox? If it's only in the sandbox that sounds similar to #37.
Finally, if you experience unresponsiveness of buttons on the client side, that suggests you might be accidentally publishing much more data than you intend to on the client. Maybe you can install something like mongol and see how big your client-side collections are getting.

I apologize for the extremely late response. We did try going with a new version of our app, without TurkServer, and that seemed to fix the issues. We were able to successfully run two short test sessions and one full experiment session.
Regarding the messages publication, that was something that I actually found online. It may not be the the best way to go, and we may change it, but it seemed to do the job with the new version of our app that does not use TurkServer. Do you think it could somehow interfere with the normal functioning of TurkServer?
As for the login problem, yes it was happening very often - if I log out and enter the password to log in, nothing happens, but after I restart the server, the log in request gets processed and I can see the admin screen.
I have not tried reproducing the error locally yet, but I do not think the problem is with the opening of multiple tabs, as I was able to reproduce it on the sandbox with only a single tab being opened. I will try that, as well as looking at the sizes of the client-side collections. But in the next couple of weeks we will probably use the new version of the app because we really need to run a few more experiment sessions and gather data for analysis (though I really miss the convenience of TurkServer in setting up the HIT and processing the assignments and payments).

Hi @ZJoveski, sorry to hear that TurkServer didn't work out for you, and also that you are probably re-implementing a lot of the TurkServer functions from scratch on your own version. This is the type of thing I hoped to avoid, but I guess we're not quite there yet.
I was hoping that if you encountered any issues of this sort you'd be able to help fix them, but at the very least you can perhaps let me know any interesting ideas you get in building your experiment that we can keep in mind for future updates.

Any chance you are using Mongol in your development process? It's possible it may interfere with partitioner.

No, I have not used Mongol at all.


Leo fix downsample

closes #360 by upsampling first to the lowest common multiple sampling rate

I created an issue in github flagging this upsampling for cursor and eye data in our pipeline.


IndexError in run_vacuum() in analyze_vacuum.py

Receive an IndexError when attempting to Vacuum a table with --table-name  specified. The get_vacuum_statement which is returned only queries for two columns:
get_vacuum_statement = '''SELECT 'vacuum %s ' + "schema" + '."' + "table" + '" ; '
                                         + '/* Size : ' + CAST("size" AS VARCHAR(10)) + ' MB'
                                         + ', Unsorted_pct : ' + coalesce(unsorted :: varchar(10),'null')
                                         + ', Stats Off : ' + stats_off :: varchar(10)
                                         + ' */ ;' as statement,
                                         "table" as table_name
                                  FROM svv_table_info
                                  WHERE (unsorted > %s or stats_off > %s)
                                    AND   size < %s
                                    AND  "schema" ~ '%s'
                                    AND  "table" = '%s';
                                        ''' % (
            vacuum_parameter, min_unsorted_pct, stats_off_pct, max_table_size_mb, schema_name, table_name)

but below, after vacuum_statements = execute_query(conn, get_vacuum_statement), each row returned is indexed to three spots:
for vs in vacuum_statements:
        statements.append(vs[0])
        statements.append("analyze %s.\"%s\"" % (vs[2], vs[1]))

Which is raising the IndexError

Were you able to resolve this? @mosaic-john-feilmeier

https://github.com/awslabs/amazon-redshift-utils/pull/472
has the fix for this issue.


Emit variable declarations for imported members so debugging works as expected.

TypeScript Version:  2.6.0-dev.20171010
I searched for emit import debugging  and namespace import emit but didn't find any appropriate issues.
Code
import { bar } from 'foo'

bar.blag = 23
debugger

Expected behavior:
define(["require", "exports", "foo"], function (require, exports, foo_1) {
    "use strict";
    var bar = foo_1.bar; // added line
    Object.defineProperty(exports, "__esModule", { value: true });
    foo_1.bar.blag = 23;
    debugger;
});

Actual behavior:
define(["require", "exports", "foo"], function (require, exports, foo_1) {
    "use strict";
    Object.defineProperty(exports, "__esModule", { value: true });
    foo_1.bar.blag = 23;
    debugger;
});

The reason for this is that, when pausing on the debugger line and evaluating bar in the debugger REPL,  I expect it to work. Currently I get 'bar' not defined.
Note that this is the playground output; I'm currently having the same issue with the commonjs output.
Prepending foo_1 isn't much of an issue here, but when selecting and evaluating entire expressions, having to modify anything is time consuming.

That does not match the ES6 module semanitcs. ES6 allows for live binding where multuple accesses to an import will get you the value at the access time. using a variable at the top means that you capture the value of the export. for instance:
// b.ts
export var x = 0;

export function inc() { x++; }

// a.ts
import { x, inc } from "./b";

console.log(x);
inc();
console.log(x);
inc();
console.log(x);

this should result in:
0
1
2

Well strictly speaking, the above code still accesses the namespace, the variable declaration would just be a debugging help. (Even if it isn't entirely semantically correct.) But I can understand if you don't want to add it...
I'll guess I'll have to stick to bundlers and hope they don't rename symbols too much.

Automatically closing this issue for housekeeping purposes. The issue labels indicate that it is unactionable at the moment or has already been addressed.


Add high-profile sponsors to the main page

Having high-profile sponsor logos on our main page will help strengthen Operation Code, and allow us to recognize sponsors who have contributed so much to our success.

This is a great idea! You're thinking of something like those startup type 'Our Customers' with all their logos and everything right?
I'm going to put this on hold for now. It seems like we may want to do a bit of a redesign on the main page so lets sort that out first. Opened up #355 for that discussion.

Yep, exactly. We should probably identify one row of these (two rows always looks cluttered), validate our permission to use their branding, and add below the main image on the front.

Going ahead and taking this... @rickr @hollomancer

Github should be here, at a minimum.


Nuxt v2 does not see the @nuxtjs/device module

 "dependencies": {     "nuxt": "^2.15.8",   },   "devDependencies": {     "@nuxtjs/device": "^3.0.0"   }
console.log('$device', this.$device)

I found the solution myself, you need to put a lower version
yarn add<EMAIL_ADDRESS>-D

Thank you for reporting.
I've updated README.


Generate FAQ schema from page body

What
At present, we're presenting a whole chapter of a guide in a single question/answer. This limits the impact of the content design we apply, because it doesn't break up the content in rich results in the same way as we do on GOV.UK.
We're also presenting all the chapters of the guide as structured data on the first chapter of the guide. This might run us foul of external search indexers because not all of the text is visible on the first page.
This PR moves to only generating structured markup for the chapter on which it's displayed.  This allows us to split up the page into more bite-sized sections.  We parse the govspeak generated HTML and uses the text in h2 headings as the "question", the id of the h2 to generate URLs with anchors, and the other markup as the "answer".
This enables us to display structured markup on on every chapter in a guide and also on other pages (such as mainstream answers).  It also means that the Google results clash less with step by steps because it's more obvious that this is guidance, and not a "how to".
We will still avoid using this markup on travel advice pages because of the latency in results being updated.
I briefly disabled the  Style/IfInsideElse cop because the alternatives seemed less appealing.
Example usage can be seen on the pages linked in https://github.com/alphagov/government-frontend/pull/1492
Visual Changes
The rich Google result for default page in https://www.gov.uk/apply-to-come-to-the-uk:
Before (one rich result for a whole guide)

After (one rich result per page in the guide)

View Changes
https://govuk-publishing-compo-pr-1127.herokuapp.com/component-guide/machine_readable_metadata/a_guide

Going to hold off merging for a couple of days while we measure the last change


HackTogether: Fix feature listing for registered users

Are you accessing the project from the GitHub Codespaces?

[ ] No, I am running the project on my local machine (I will fill in the OS section below)
[x] Yes, I am using GitHub Codespaces.

Note: The project is optimized to run on Codespaces. We welcome contributions to enhance it for cross-platform local development, but it's not something we will prioritize right now. Thank you!

Describe the bug

The bookmark icon is too close to the image.
The Property description is overlapping the bookmark icon.
The short description for the listed property could be removed in order to create a minimalistic design. This description could be assessed when the user wants to know more about the property.

To Reproduce
Steps to reproduce the behavior

Go to home page after login

Expected behavior

I expect the bookmark icon to be properly aligned/placed
The property name/description should not overlap the bookmark icon
A more aesthetic property listing

Screenshots

OS (please complete the following information):

OS:
OS Version:
npm version:
Node.js version:
output of azd version:

Additional context
Having the long description removed from the featured listing will let each card have equal height.

Hi @mzbhewtee Thanks for reporting! We suggest you investigate this issue as related to Angular material design. Thank you for participating. We look forward to your PR!

Hi @anfibiacreativa Thank you, i will investigate the issue.

Suggestion to tackle with issue #242

Removing long description is not ideal recommendation. we can properly customize scss for having responsive cards with appropriate dimensions.

I am also willing to contribute and investigate together @mzbhewtee

Do we have any updates? Thank you!


Please explicitly mention LocalDB

Is it supported or not?

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

ID: c7f63491-26ec-52f6-e124-dec2ecea8477
Version Independent ID: 72aa55f1-766a-c015-a5f7-104359382d53
Content: Use a SQL Server database in a UWP app - Windows UWP applications
Content Source: windows-apps-src/data-access/sql-server-databases.md
Product: uwp
GitHub Login: @mcleanbyron
Microsoft Alias: mcleans

Oops, I see I asked this already.


Google chrome not generating keys

I go through the process to make a new account via databox.me but it only actually installs a certificate on firefox.
The account was created but the keys were not installed so i can never log back in and the recovery email doesnt seem to be working.

Yes, Chrome has disabled this behavior by default in the latest versions. You can manually enable it by click the padlock next to the URL in the address bar, and allow Key Generation.

Oh thats annoying, thanks.

Hey since its you.. >_> you think you can look at this for 30 seconds?
Go to https://solid.github.io/solid.js/demo/?
->
for the Profile URI enter: http://melvincarvalho.com/
->
Expected results but when you replace the uri with https://leathan.xyz/ it fails.
Im super confused because leathan.xyz is the same as melvincarvalho.com as far as I know. Dont spend any time on it but i think you will know whats wrong pretty quick since pretty much wrote it ^_^

There are a couple of issues with your profile that I was able to easily spot.

It doesn't return text/turtle (which is standard for WebID profiles)
The server hosting the profile doesn't support CORS:

XMLHttpRequest cannot load https://leathan.xyz/. No 'Access-Control-Allow-Origin'
header is present on the requested resource. Origin 'https://solid.github.io' is
therefore not allowed access.

Aside from CORS not being enabled do you see any reason why http://melvincarvalho.com is resolving properly and https://leathan.xyz ?
Because I enabled CORS and I copied all of the other headers I see from melvincarvalho.com.

Sorry to reopen this but I have since spent my time reading about Solid. When I returned to try and make another profile from chrome I had to go to chromes content settings and add an exception for *.databox.me.
1.) When i initially load databox.me i am prompted for a username. When I enter my username it prompts me for my WebID. When i enter my WebID https://leathan.xyz/#i it changes the button from "Check Avialability" to "Create Account".
I then continue to press Create Account but afterwards it still generates a new WebID for me. Shouldnt we be able to log in with any ol WebID and use databox.me ldp servers?
At first I thought it was a problem with my WebID but I have switched my WebID to text/turtle and the problem persists. I cannot log in with my WebID and I can not send recovery emails to<EMAIL_ADDRESS>I have burned through like 10 accounts now :/
I know its a hassle.. but if there is anyway to notify chrome users before key generation it would be awesome and would save a few acnts.

I got caught in this same trap. I created an exception for databox.me. How do I go about trying to create a certificate again? (I'm very new to the whole WebID thing.)

Revisiting web-ids and i still cant seam to get anything to work for me locally, or on databox.me :(
I opened a new account god.databox.me :) but i think that one is burned too. when i click the login button i get an orange banner flash at the top of the screen saying "WebID Authentication failed"


Add PATCH to Azure Firewall supported operations

We are adding PATCH to the list of supported operations for Azure Firewall.
Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Automation for azure-sdk-for-python

Can one of the admins verify this patch?

Automation for azure-sdk-for-go

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-java

Please ignore error in NetworkValidation, it's not related to this PR, it failed due to NuGet not being able to resolve dependencies. I've already reported this error to SDK & Azure PowerShell teams.
D:\a\1\s\Out\azure-powershell\src\Network\Network\Network.csproj : error NU1605: Detected package downgrade: Microsoft.Rest.ClientRuntime from 2.3.20 to 2.3.19. Reference the package directly from the project to select a different version.  [D:\a\1\s\Out\azure-powershell\src\Network\Network.sln]

SDK & Azure PowerShell team

Then can you please help me complete this PR?

Signing off from ARM side.


output can be define brokes by zookeeper

if  output can be define brokes by zookeeper, became kafka brokes could Changed and some broke could be down。
eg.
<match *.**>
type                kafka
zookeepers    localhost:2181
default_topic

Sorry for being super late...
I fixes this issue, and I'll upload to RubyGems as v0.0.9 soon.
Please try this version and let me know if you have still trouble.
Thank you for contributing!


File path using backslash on windows

operating system: Windows 10
fuse-box version: 1.3.37
node version: 6.9.1
npm version: 3.10.8

After upgrading from fuse-box-1.3.33 to 1.3.37, the app builds fine, but I get runtime errors in the browser console. They are 404s, but only for my application-specific imports.
Looking at the bundled js, those file paths have backslashes instead of forward slashes.
For example:
___scope___.file("todo\todo-list.component.js", function(exports, require, module, __filename, __dirname){
Manually changing those backslashes to forward slashes seems to make things work again.

Yeah, i am trying to figure out what's wrong now ..

will be fixed ASAP :)

Fixed in 1.1.38

1.3.38 works great! thanks


Color disappears for links and equations inside itemize

The blue color that is used for \eqref and for \url (to indicate that you can click on them) does not appear inside the \begin{itemize} .. \end{itemize} environment.
For equation labels this is not too bad, but for url's it makes them very difficult to spot, since they look similar to the surrounding text.

Seems to come from
li a {
    color:black;
}

in iLaTeXen.css. I commented this out and the blue reappeared. Right now I cannot remember, why the above was put in iLaTeXen.css.

I think the reason it was used, was to have black text in the menu in the upper left corner.


Builds invalid URL when branch name has /

When my branch is for example feature/foo, the built URL becomes
https://github.com/aelesbao/repo/blob/feature%252Ffoo/README.txt

This should be fixed in version 1.5.4 that has just been released. Could you download the update from your IDE and let me know?

Tested and it's working fine. Thanks!


Expose ClusterNetworkPolicy feature gate in controller

Add ClusterNetworkPolicy feature gate option in controller config yaml and set it to false.

merged as part of https://github.com/vmware-tanzu/antrea/pull/900


Make rustc output parsing more robust

While testing #112 I found at least one case, where the current RegEx for error messages does not work correctly. Given this rustc output
main.rs:118:22: 118:26 error: mismatched types:
 expected `&mut [u8]`,
    found `&[u8; 2]`
(values differ in mutability) [E0308]
main.rs:118     match stdin.read(&buf) {
                                 ^~~~

the RegEx will only extract the first line and discard the additional information, resulting in the error message mismatched types:
I'm not sure how this should be fixed.

I suspect a rule along the lines "If the first error line ends with :, everything up to error code is part of the message" would cover it.
But it's just for this case, there are probably other multiline messages thatw e fail on. We really should comb libsyntax sources for all the [EXXX] messages, convert them to tests and start reworking output parsing from there.

We should probably be using the JSON error format, which has recently landed: https://github.com/rust-lang/rust/pull/30711 (requires invoking a recent rustc with -Z unstable-options --error-format=json)


Configure namespaces build service

Can now specify a new custom namespace in the build service
if one is not specified in the deployment yaml, it will default to the default namespace
tested through CLI with command: python3 build-service.py -vv https://github.com/capstone-kubernetes-aas/test-local-reg -b main
Update https://github.com/capstone-kubernetes-aas/test-local-reg/blob/main/kaas.deploy.yml to the three cases for testing:

no namespace specified
default as namespace in the deployment yaml
custom namespace in the deployment yaml

You can check the namespaces by viewing the admin dashboard: https://dashboard.capstone.detjens.dev/#/login
and selecting the namespace you would like to view.

Yay!


PointDataset.setPointStyle is missing an option to disable it

Checklist

[X] I am able to reproduce the bug with the latest version
[X] I made sure that there are no existing issues - open or closed - which I could contribute my information to.
[X] I have taken the time to fill in all the required details. I understand that the bug report will be dismissed otherwise.
[X] This issue contains only one bug.

Affected version
2.3.1
Steps to reproduce the bug

Check documentation on chartjs.org
Check enum PointStyle in chartjs-java-model

Expected behavior
The documentation on chartjs.org states the following values for Elements => Point Styles:
Info
When a string is provided, the following values are supported:
'circle'
'cross'
'crossRot'
'dash'
'line'
'rect'
'rectRounded'
'rectRot'
'star'
'triangle'
false

Actual behavior
public enum PointStyle { 	CIRCLE, 	TRIANGLE, 	RECT, 	RECT_ROUNDED, 	RECT_ROT, 	CROSS, 	CROSS_ROT, 	STAR, 	LINE, 	DASH; }
The boolean value "false" is missing
Additional information
This is the only value that hides the marking. If there is another way within chartjs-java-model, I couldn't find it

Thank you for the issue.
I can confirm the problem. Changing the type to a Object should make it possible to set the value accordingly.

The documentation on chartjs.org states the following values for Elements => Point Styles

A link would also be nice next time ;)

Fix should be available in the next release :)