[custom-devices] add screenshotting support

This implements #79678.
I tested it on my device and the flutter SDK part of this works. However it's still not possible to take screenshots of the app, at least when using flutter-pi because the device-part of screenshotting support is missing. I.e. there's currently no ready-to-use command to take screenshots of the Raspberry Pi when using KMS directly without X11 / Wayland. You can get a fbdev screenshot (and that's what I tested) and that works, but only shows the console. So in the future I'll probably add some kind of support inside flutter-pi to take screenshots.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test-exempt.
[x] All existing and new tests are passing.

This would use a platform specific screenshot mechanism right?
Context is that I also have https://github.com/flutter/flutter/pull/80616 as a WIP which will allow falling back to a "rasterizer" screenshot. That should allow all platforms to have some level of screenshot support.

This would use a platform specific screenshot mechanism right?
Context is that I also have #80616 as a WIP which will allow falling back to a "rasterizer" screenshot. That should allow all platforms to have some level of screenshot support.

Yep exactly. Just adds a new command to the config that will take the screenshot and store it in some path. If the command isn't present, the CustomDevice will report that screenshotting isn't supported.

how am I getting this allowlist error? The problem with it started happening way after the base master commit I'm using and the bundle builder refactor (which used the same upstream commit as base) tested fine

There was a bug in the test script, you'll need to pull lastest master to get the fix

@ardera do you need any help getting synced to a good flutter hash?

@ardera do you need any help getting synced to a good flutter hash?

Thanks but I just didn't have time on sunday and wanted to do it later instead 😄 probably one reason why you guys don't give time estimates ;)

Fair enough 😄

So there's one test failing, but I'm not sure it's related. Seems like it times out after 25s, but some other tests take long as well (the Test is taking a long time message is printed out a few other times)
04:38 +9 ~14: test\integration.shard\overall_experience_test.dart: flutter error messages include a DevTools link
Test is taking a long time.
Expected state transitions:
    0 NOW WAITING FOR> /^An Observatory debugger and profiler on Flutter test device is available at: /
    1                  /^The Flutter DevTools debugger and profiler on Flutter test device is available at: /
    2                  /^Performing hot reload\.\.\./
    3                  /^Reloaded 0 libraries in [0-9]+ms./
So far nothing has been logged; use debug:true to print all output.
(streaming all logs from this point on...)
stdout: Syncing files to device Flutter test device...                     265ms
stdout:
stdout: Flutter run key commands.
stdout: r Hot reload.
stdout: R Hot restart.
stdout: h List all available interactive commands.
stdout: d Detach (terminate "flutter run" but leave application running).
stdout: c Clear the screen
stdout: q Quit (terminate the application on the device).
stdout:
stdout:  Running with sound null safety
stdout:
stdout: An Observatory debugger and profiler on Flutter test device is available at: http://<IP_ADDRESS>:53624/49ye2qa6uGk=/
(matched /^An Observatory debugger and profiler on Flutter test device is available at: /)
(process is not quitting, trying to send a "q" just in case that helps)
(a functional test should never reach this point)
stdout:
stdout: Application finished.
(process terminated with exit code 0)
The subprocess terminated before all the expected transitions had been matched.
The transition that we were hoping to see next but that we never saw was:
    1 NOW WAITING FOR> /^The Flutter DevTools debugger and profiler on Flutter test device is available at: /

04:50 +9 ~14 -1: test\integration.shard\overall_experience_test.dart: flutter error messages include a DevTools link [E]
  Missed some expected transitions.
  test\integration.shard\overall_experience_test.dart 275:5  runFlutter

That is unfortunately another broken test, we just landed https://github.com/flutter/flutter/commit/1b4f4482b75b92a95807ee4337798227c439dc61 to work around it

Okay, rebased onto that now


Having trouble with run.py

I'm not computer savvy enough to figure this out myself quickly. I'm working on a laptop with Windows 11. Any guidance or suggestions would be greatly appreciated.

UPDATE:
After doing a little searching I changed run.py, line 60, from (1,8) to (1,1). New error.

UPDATE: After searching Github some more I found a post showing to update checkpoints.py to this, for windows. run.py appeared to be working afterwards but it resulted in system performance being bogged down and nearly unresponsive. System black screened, showing only a python app window and became totally unresponsive when I tried to take a screen shot a few minutes later. My GPU is a "RTX 4060 Laptop GPU"
It may be a good idea to add a warning to the main page stating that (I guess?) using less than x (8?) GPU's will result in significantly diminished performance and using 1 GPU will likely cause your system to become unresponsive and require process termination and/or system reboot.


Document cookie options

Here, we see that the "cookie" object is optional.
But in most libraries, there would be nice JSDoc comments here that describe each individual option and the default value that the library chooses for you. (The JSDoc comments would cause mouse-over documentation to appear in VSCode so that I wouldn't have to leave my code editor to search the documentation.)
Can this be added? It's unclear what I need to specify and/or change in order to make using this product secure.

I have documented the cookie options as well as the other session options in version 3.3.0

thanks


Update query.ex

typo in preload queries for join on.
instead of and it should use where for conditional expressions.

I believe both approaches are fine. Or is there a reason to prefer one over the other?

It all boils down to readability and some specific behavior when using left/right/outer joins. I think it's better to keep filter conditions in where, and as for left/right/outer:
For example, if we have:
TABLE posts(id serial, title varchar, body varchar)
TABLE comments(id serial, post_id fk(posts.id), body varchar, score int)

Doing something like this:
SELECT posts.id, comments.body
FROM posts
LEFT JOIN comments ON (posts.id = comments.post_id AND comments.score > 5)

Could produce something like this:

posts.id
comments.body

1
hello

1
good comment

2
NULL

3
NULL

While this query:
SELECT posts.id, comments.body
FROM posts LEFT JOIN comments ON posts.id = comments.post_id
WHERE comments.score > 5

Will return this data:

posts.id
comments.body

1
hello

1
good comment

It all boils down to readability

I have heard a lot of people with similar opinions so don't have a strong feeling either way. But to me the readability actually goes the opposite way. Putting it in the on clause is actually closer to what is happening. The planner pushing where to on is an optimization not reflected in the query text.

Closing this for now as there is no consensus and it seems to be a matter of preference. Thank you!


There is no way to access original node request

Describe the problem
Sveltekit's handle hook throws away actual node request, and only supplies very minimal request information.
https://github.com/sveltejs/kit/blob/77590cb20e0fba9f942c3c23da279c1807d834f0/packages/kit/src/node.js#L54
The node request information has tons of additional information like connection (ip address) and some critical information which is required for building enterprise apps. While in the handle hook we only get pretty minimal information which is not useful for a lot of use cases.
Describe the proposed solution
One way I can think of, which is pretty straightforward is basically also send in the original node request in the handle hook.
so the event could have request => Sveltekit's request object, and another property called nativeRequest which would actually be the actual node request.
The apps can use the native request object to get the node information which is not available currently.
Alternatives considered
Another solution which I think could work is when building the new Request object, also pass any locals.
so request.locals = req.locals.  This would enable developers using custom node / express server to use any middlewares and then pass down the computed values through req.locals to Sveltekit's request object.
This could even be extended to a list of properties instead of just locals which could be set in adapter-node
Importance
i cannot use SvelteKit without it
Additional Information
I am working on an enterprise level application, and this is a critical functionality we would need in order to distinguish internal & external requests and do some custom authentication for our internal requests. We cannot use Sveltekit without this feature.

Duplicate of #4654.


Debugging inside a container: debugger does not hit breakpoints anymore

Type: Bug

Behaviour
Expected vs. Actual
Context: Debugging code inside a container
Expected: breakpoint are hit when code is executed.
Actual: breakpoints are not hit, and when the debug session the Debug Console shows:
pydev debugger: unable to find translation for: "c:\dev\qxc\model-sample\qxc\app\run.py" in ["C:\dev\qxc\model-sample/qxc/", "C:\dev\qxc\model-sample/qxc"] (please revise your path mappings).

Worth mentioning: rolling back to version v2022.18.2 of the extension fixes the problem without any change in the path mappings. Those mappings were defined weeks ago, and just stopped working with the latest release.
Steps to reproduce:

Configure a project to run and debug inside a container
Set some breakpoints
Run the project
Observe the warning describe above in the Debug Console, and verify that breakpoints are not hit

What seems strange from the message above is that the path inside the container c:\dev\qxc\model-sample\qxc\app\run.py looks like a Windows path, but the container I'm running is from a Linux-based image. I'm on a Windows box with Docker Desktop on WSL2.

Diagnostic data

Python version (& distribution if applicable, e.g. Anaconda): 3.7.4
Type of virtual environment used (e.g. conda, venv, virtualenv, etc.): Global
Value of the python.languageServer setting: Default

Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)

XXX

User Settings

languageServer: "Pylance"

Extension version: 2022.18.2
VS Code version: Code 1.74.0 (5235c6bb189b60b01b1f49062f4ffa42384f8c91, 2022-12-05T16:38:16.075Z)
OS version: Windows_NT x64 10.0.19044
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8650U CPU @ 1.90GHz (8 x 2112)

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_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
31.86GB (11.04GB free)

Process Argv
--folder-uri file:///c%3A/dev/qxc/model-sample --crash-reporter-id d3ea3600-8e75-4ca7-a1d1-6a498426b4ab

Screen Reader
no

VM
0%

Are you using the Docker extension? If so then you may be running into https://github.com/microsoft/vscode-docker/discussions/3750 .

I have encountered a similar issue, but using remote debugger features without docker. After reading this SO post I solved the issue by reverting from 2022.20.0 to 2022.18.2 resolved it.
Seems like whatever broke back then broke again in the latest 2022.20.0
Note: it seems like the debugged program actually halts at the breakpoint, but it's never registered on the clientside within VScode (i.e. no debug control panel pops up as you would normally expect). Once I detach from the debugged process execution continues as normal and I can see output.

I can confirm all that @isears reported on my end also - reverting to  2022.18.2 solved issues with the debugger not attaching to the  remote process to debug

@brettcannon I do use the Docker extension, but it seems to me it's unrelated to the issue. As others posted, reverting to 2022.18.2 solves it, and it's been working fine for weeks until the upgrade to 2022.20.0. Also, the issue you mention shows side effect I do not experience, like the code not running at all, in my case the code runs normally from the container, only breakpoints are not hit.

2022.20.0 also updated debugpy, so there was a potential bug involved there. But 2022.20.1 went out yesterday with a rollback to debugpy 1.6.3, so you can unpin the Python extension and see if it works again.
Otherwise would you mind opening a bug on debugpy about this so we can try and get this fixed in the next debugpy version?


Add License

What is the license for this library?
Please add a license file to the root dir.

Sorry, it was specified in the build meta data, but the file wasn't added - fixed now.
It's Apache2.


day view displays all hours of the day instead of 6-10

🙌

I'd rather it be configurable like the original calendar, rather than hard coded. Sorry about the delay with doing this myself, but I'm busy 24/7 with a large project at work at the moment.

Since it's currently hardcoded it seemed more logical to include all hours rather than just a subset. I agree that a configurable option in the long run.

Yeah you're right, I'll make it configurable next time I get an evening free.

Thanks for being you. ;)


⭐FEAT: Open in new tab

TODO :

when user clicks on links in the footer, the links should open in a new tab

Fixed in #92


Basic demographics

A test query for extracting basic demographics of patients in eICU-CRD.

Matthieu:
This is a good idea, but I would like to discuss (as a group) about our policy for coding data like this .  In particular, converting text fields to numerics.  I would prefer to keep them as is.  I realize this is provided code that someone would choose to run on the original data, and you don't have to run it, but people use this snippets quite a bit, and it's worth discussing what the best practices are.
My thoughts:

It's unnecessary.  Most software handles strings just fine.  It's also prone to errors when we change the original data.
It's eliminating intuitive and informative information.  I always struggle for instance to figure out if 1 is M or F for coding gender.  This is particularly problematic for non-binary situations. Gender is actually not binary in eICU.
Most software gives you better output with the original string representation (e.g., interpreting effects in regression models).
It slows down people who don't know what they are doing.

I'd be interested in hearing what others think.
Jesse

Jesse,
No problem, I was just trying to post a first example of a SQL query using the eICU-CRD and test a commit. I agree, we need to homogenize to some extent the queries. Although it's going to be difficult to harmonize all contributors from all around the world, no? We would need a "best practices" page?
In addition, I converted to numerics because the models that I work on require numerical values only, and I believe that Matlab does not handle text fields in most functions. It's also very easy to simplify the query and remove the data conversion, if wanted.
Matthieu


add disability attribute/dropdown

Add new field for: I identify as a person with a disability
http://sudcc.syr.edu/LanguageGuide/
At this point we will not ask for details about the disability - we just want to know whether this influences how much one is paid.
Should have the same dropdown yes/no style that the current attributes have.

closed with https://github.com/rubycorns/whatdojuniorsgetpaid/pull/25


opnsense 17.1.9 dnsmasq crash

I have upgraded to 17.1.9, after the upgrade dnsmasq repeatedly automatically stop, every few hours or day, will automatically stop

We need your full version now as printed in the dashbaord, as well as a dnsmasq log snippet and which version you upgraded from...
In 17.1.9 dnsmasq was bumped to version 2.77, let's start with using 2.76 instead:
# opnsense-revert -r 17.1.8 dnsmasq

Thank you for reply

system logs show:

Upgrade to 17.7，dnsmasq crash again，
I cannot use dnsmasq 2.76
opnsense-revert -r 17.1.8 dnsmasq
Fetching dnsmasq.txz failed

when you upgrade to 17.7, you can't install older packages for 17.1 anymore. And since we only keep 3 versions,that is 17.7, 17.1.11 and 1.7.10 now.
there is virtually no info about the crash other than the segfault itself, which, by your tiniest indication seems to be fixed by using 2.76?
If so, it would be nice to know. ;)

PS: you will have to find out why your traffic / your setup crashes 2.77, compile a useful report we can give to dnsmasq or sidestep the issue by using unbound DNS resolver.

Yes, I fixed it by using 2.76,but now ,can`t downgrade 2.76

you should have said that fixes it, you can always lock the package to prevent the package from updating :)

what architecture/crypto combo are you using? e.g. amd64/OpenSSL

amd64/OpenSSL

thanks, here's the package...
pkg add -f https://pkg.opnsense.org/snapshots/dnsmasq-2.76,1.txz

I have locked the package in 17.1,but upgrade to 17.7 ,it upgrade Automatically

right, sorry, a cautionary tale from a bad 17.1 upgrade made us unlock all packages to be able to upgrade cleanly.

Thanks a lot ,returned to 2.76
pkg add -f https://pkg.opnsense.org/snapshots/dnsmasq-2.76,1.txz

so there is some problem in 2.77: http://www.thekelleys.org.uk/dnsmasq/CHANGELOG
do you have any idea which it could be? does it crash instantly or after a while?

I have looked at changelog and may be related to the following changes:
Fix logic error in Linux netlink code.
Handle binding upstream servers to an interface (--server=<IP_ADDRESS>@eth0) when the named interface is destroyed and recreated in the kernel.
Make --localise-queries apply to names from --interface-name.
Improve logging of upstream servers when there are a lot of "local addresses only" entries.
Make --rev-server for an RFC1918 subnet work even in the  presence of the --bogus-priv flag
Because in the production environment, so can not do more testing

Timeout.


libturbojpeg is not installable

[x] I confirm that this is an issue rather than a question.

Bug report
tried installing hyperion a few times now, and i always end up with the same error
 hyperion : Depends: libturbojpeg but it is not installable
E: Unable to correct problems, you have held broken packages.

Steps to reproduce
Follow install steps for Hyperion on raspberry pi 64 bit lite os
What is expected?
Install of libturbojpeg package
What is actually happening?
Cannot install libturbojpeg package because it doesn't exist, there exists a libturbojpeg0 package though
System

Can you tell us which hardware or OS you want to install Hyperion on? Greetings Paulchen

Closing as no feedback


QueenNormalInterfaceDeclarationNode.java: Don't forget to write some un...

The puzzle -2133584666 originating from #10 has to be resolved:
https://github.com/jvmqueen/queen-of-java/blob/d6df6f41420c5da9f430a84e78885f7d820a8f34/src/main/java/org/queenlang/transpiler/nodes/QueenNormalInterfaceDeclarationNode.java#L14-L14
"Don't forget to write some unit tests here.".
The puzzle was created by amihaiemil at 2022-12-04 10:58:25 +0200.
Estimation is 60 minutes.
If you have any technical questions, don't ask me, I won't be able to help. Open new issues instead.

@amihaiemil this is your task now, please go ahead. Deadline (when this ticket should be closed) is 2022-12-14T09:00:26.511313.
Estimation here is 60 minutes, that's how much you will be paid.
Remember, you don't have to solve everything in this ticket - you can solve it partially and leave todo markers in the code, which will become future tasks.
If you have any questions don't ask me, I'm not a technical person. Open new tickets instead.
If you don't have time or simply don't want to work on this, you can always resign.

@amihaiemil Don't forget to close this ticket before the deadline (2022-12-14T09:00:27). You are past the first half of the allowed period.

@amihaiemil Looks like you've missed the task deadline (2022-12-14T09:00:27). You are now resigned from this task.
Please stop working on it, you will not be paid. I will assign it to someone else soon.

@amihaiemil I couldn't find any assignee for this task. This is either because there are no contributors with role DEV available or because the project does not have enough funds.
Please, make sure there is at least one available contributor with the required role and the project can afford to pay them.

Puzzle disappeared from the code, that's why I closed this ticket.


spyder crash after viewing big matrix - huge memory leak

Issue Report Checklist

[ ] Searched the issues page for similar reports
[ ] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
[ ] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
[ ] Could not reproduce inside jupyter qtconsole (if console-related)
[ ] Tried basic troubleshooting (if a bug/error)

[ ] Restarted Spyder
[ ] Reset preferences with spyder --reset
[ ] Reinstalled the latest version of Anaconda
[ ] Tried the other applicable steps from the Troubleshooting Guide

[ ] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
What steps reproduce the problem?

create the variable
import numpy as np
a = np.zeros((1400,150,90))
explore the numpy array a, change the slice, scroll the data then quit the explorer view
after somes seconds, the RAM consumption increases from 0 to several Go, until the system is fully freezed

What is the expected output? What do you see instead?
No messages preventing large data from being displayed.
Paste Traceback/Error Below (if applicable)

PASTE TRACEBACK HERE

Versions

Spyder version: 3.2.8
Python version: 3.6.5
Qt version: 5.9.5
PyQt version: 5.9.2
Operating System name/version: ubuntu 18.04, RAM 32 Go

Dependencies

PASTE DEPENDENCIES HERE

Seems fixed with spyder 3.3.1
Thks !
SÉBASTIEN GÉNETÉ
Lead Data Scientist
+33 (0)6 16 81 54 23 <+33616815423>
<EMAIL_ADDRESS>www.insideboard.com
Paris https://goo.gl/maps/fnWi5pmYNpK2 • New-York
https://goo.gl/maps/NJepncnNZST2 • San Francisco
https://goo.gl/maps/9MHLSLHnLdP2
[image: Logotype Insideboard] https://www.insideboard.com/
https://www.linkedin.com/company/5259101/
https://twitter.com/insideboard_tw
https://www.youtube.com/channel/UCfUmKP5HlpC9hBdVglsvm9Q
https://www.facebook.com/M-inside-1582016848711968/
https://plus.google.com/u/0/104161323686518017080
2018-08-31 11:21 GMT+02:00 Jitse Niesen<EMAIL_ADDRESS>
Spyder 3.3 fixes several memory leaks in the Variable Explorer, so I
suggest you upgrade and try again.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/spyder-ide/spyder/issues/7793#issuecomment-417606516,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AnGMAxeafS-nH4dT4DexDrUSGO8cAJ3Uks5uWQAUgaJpZM4WU0zB
.

I'm using Spyder 3.3.3 but still have this issue occasionally.
One example is with a matrix of about 250x16, with ints and floats inside. Unsure how to resolve.


Visualization.py and config.pi wher do i find them and where do they go? I have finally got leds to light in strandtest now im stuck please help?

where do i find them and where do they go? I have finally got leds to light in strandtest now im stuck please help?

Have you tried redownloading the files from github?

I know where the file is but what is the command to get it?
Sent from Mail for Windows 10
From: Joey Babcock
Sent: Friday, May 26, 2017 9:00 PM
To: scottlawsonbc/audio-reactive-led-strip
Cc: gscott1962; Author
Subject: Re: [scottlawsonbc/audio-reactive-led-strip] Visualization.py andconfig.pi wher do i find them and where do they go? I have finally got leds tolight in strandtest now im stuck please help? (#48)
Have you tried redownloading the files from github?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Ok I cloned this file on raspberry and its in its own directory do I need to move it? I can find the directory in the terminal and I can find the visualization.py but I cant run it?

Ok I have gotten a little farther I guess when I run visualization.py I get this error?
Traceback (most recent call last):
File "visualization.py", line 257, in
import pyqtgraph as pg
ImportError: No module named pyqtgraph

Did you follow the tutorial? Run pip install pyqtgraph I think is the command, there's a number of them, just follow the tutorial.

Installation for Raspberry Pi
If you encounter any problems running the visualization on a Raspberry Pi, please open a new issue. Also, please consider opening an issue if you have any questions or suggestions for improving the installation process.
Installing the Python dependencies
Install python dependencies using apt-get
This is the guide I am following its Scott lawsons and in here he doesn't mention Pyqtgraph he does in other dependency's and yes I followed the guide yes I am new at this and no I'm not stupid?
sudo apt-get update
sudo apt-get install python-numpy python-scipy python-pyaudio
Install ws281x library
To install the ws281x library I recommend following this Adafruit tutorial.
sudo apt-get install build-essential python-dev git scons swig
git clone https://github.com/jgarff/rpi_ws281x.git
cd rpi_ws281x
scons
when I run the visualization.py it says it cant find the file but when I go to install it pi says its already there?

Did you cd python into the directory before trying to launch it?

Yes that’s what is confusing me I can find it I can open it its in the same place most everything else is but when I run visualization.py it starts the window then stop saying it cant find the mentioned file? Going to redo everything on another sd card and see what happens then compare the two?
Sent from Mail for Windows 10
From: Ritvars
Sent: Sunday, May 28, 2017 10:46 AM
To: scottlawsonbc/audio-reactive-led-strip
Cc: gscott1962; Author
Subject: Re: [scottlawsonbc/audio-reactive-led-strip] Visualization.py andconfig.pi wher do i find them and where do they go? I have finally got leds tolight in strandtest now im stuck please help? (#48)
Did you cd python into the directory before trying to launch it?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

can you tell me which dir. the visualization file should be installed to?

You are not going to believe this but I have the visualization.py running and it actually is reading the music in the screen ! But I cant figure out why I have no leds? I have checked the wiring 2 times? Any ideas?

Fron python folder to python flolder?
Sent from my Sprint Samsung Galaxy S7.
-------- Original message --------From: Ritvars<EMAIL_ADDRESS>Date: 5/28/17  10:46 AM  (GMT-05:00) To: scottlawsonbc/audio-reactive-led-strip<EMAIL_ADDRESS>Cc: gscott1962<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [scottlawsonbc/audio-reactive-led-strip] Visualization.py and
  config.pi wher do i find them and where do they go? I have finally got leds to
  light in strandtest now im stuck please help? (#48)
Did you cd python into the directory before trying to launch it?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/scottlawsonbc/audio-reactive-led-strip","title":"scottlawsonbc/audio-reactive-led-strip","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/scottlawsonbc/audio-reactive-led-strip"}},"updates":{"snippets":[{"icon":"PERSON","message":"@RichusX in #48: Did you cd python into the directory before trying to launch it?"}],"action":{"name":"View Issue","url":"https://github.com/scottlawsonbc/audio-reactive-led-strip/issues/48#issuecomment-304519503"}}}


Proposal: remove logic for getting OAuth token

Should we remove the OAuth token acquisition logic from the core library and delegate OAuth token retrieval to a third-party library like https://godoc.org/golang.org/x/oauth2 as it handles things like refresh tokens, password credentials token, etc.?

FYI we will not likely support refresh tokens, nor the password credentials token. We only support the Code grant flow.
Therefore, the OAuth2 lib may be overkill. I see a value to provide a simple implementation that covers what we offer in our API, since we will likely also have something similar in other clients.
I'll take a look at the OAuth2.


Add read-through cache for opslevel registrations.

The idea here is that if the parsed registration doesn't change, we don't need to do any work. The cache has an expiry equal to the resync period that is defined. So, if the parsed registration doesn't change, at most we'll hard-resync once per resync interval.
This was added to address the 1000s of ServiceGet requests that we get per day per service, without corresponding ServiceUpdate calls. This means that while the kubernetes kind is updated, the information we parse from it does not change.
Resolves #
Problem

Solution

Checklist

[ ] I have run this code, and it appears to resolve the stated issue.
[ ] This PR has no user interface changes or has already received approval from product management to change the interface.
[ ] Make a changie entry that explains the customer facing outcome of this change

@jasonopslevel - So the reason i didn't implement something like this is because the same "service" in opslevel might be fed data from 2 k8s cluster.  Each cluster has a copy of kubectl-opslevel running in it which means each has its own unique cache which means they don't actually line up with each other meaning you'll break actual reconciliation.
Now customers who are writing the same data fields but different values - one could consider that a bug - but this change will paper over that and if there truely are valid cases out there for that - it causes a hidden bug.
#cacheing-is-hard

Ah ok. I was taking a look at our graphql requests and certain k8s resources are responsible for a very large majority of the requests. I took a 15 min sample and some had 1000s of requests (but no corresponding update requests) meaning there was no diff.
I was mostly curious after diving into our requests + metrics, and wanted to see how hard it may be to add and get more familiar with writing go. I wanted to see what you folks thought before writing tests + doing integration testing- so it's not fully baked either.

@jasonopslevel  ahh ok.  Ya i'm not saying its not a ok thing to add.  But it needs to be off by default and turned on only in known good situations.  Which is kind of hard to communicate to the customer.  We can try but this is a LONG road to getting everyone to use it.
Feels like it would be better / easier to have the app cache service data in redis for some sort of TTL that it can ask for - then fallback to MySQL.  That would likely improve ALOT - not just kubectl-opslevel - but everything.

Now customers who are writing the same data fields but different values - one could consider that a bug - but this change will paper over that and if there truely are valid cases out there for that - it causes a hidden bug.

Just to be clear, this would be 2 clusters, writing to the same service + providing different information? I think that still works in this scenario. We're still only caching the possible updates from the registration.
The part that we do lose (regardless of a multi cluster scenario) is re-reconciling a service, where k8s sets a description, the user sets it to something else -> we have to wait for the resync period to set it back again. (Ideally we lock the individual service properties, but that's a separate discussion.
And i guess now that I think about it, perhaps you mean two clusters writing the same attribute, and wanting it to flip-flip back and forth. I feel like we shouldn't support that? That seems more useful to be an event stream than an attribute.

Ya - our k8s sync has lots of "architectural" problems because we don't treat it as an "event stream" 😭


NAO Simulation with Chrome + iOS

Describe the bug

The NAO Simulation can not be used. The Message is "Safari does not have the technical... Please use a compatible browser (Chrome...)
I use Chrome

To Reproduce
Steps to reproduce the behavior:

Use a iOS device + Chrome browser
Go to lab.open-roberta.org
Click on NAO
Click on SIM
See error

Expected behavior

NAO Sim works within Chrome browser

Screenshots

If applicable, add screenshots to help explain your problem.

Device information

Type smartphone / iPad
OS: iOS
Browser: chrome

It seems Chrome as a userAgent is not received in navigator.userAgent.
userAgent info on iPad 12.9 (BrowserStack):

Request Mobile Site:

Request Desktop Site:

That being said, when I tried CriOS or Version for checking for chrome as the userAgent, the simulation reported the following error:

the problem is in software of the WeBots developer. We believe, that the problems are already fixed in a newer version of that software and that we have to upgrade the version we use


Tests timing out don't say timeout

We've got much more complex suite of tests, but we're finding that we're getting reports of "missing plan" as a failing test, and yet the total number of passing tests doesn't actually match the plan.
This is one example, the only clue that it's actually timed out is the 30s, but otherwise it's not clear.

I'm not sure if this is something that's combined with request, but I'm able to replicate using it.
I've created a package tap-test-timeout-fail that replicates the issue. You'll need to cd into the dir and run npm test to see the issue - this is what it gives me (plan count issue, rather than timeout):

The index.js is this:
var test = require('tap').test;
var request = require('request');

test('test', t => {
  t.plan(2);

  t.pass('normal');
  request({
    url: `http://hang.leftlogic.com/?${32 * 1000}`,
  }, () => {
    t.pass('request complete');
  });
});

Note that the hang url will not respond until 32 seconds - i.e. outside the default timeout.
Should this actually show a timeout error? I tried replicating with unref'ing a timer, but tap exited early, so I don't think it's that.

Yeah, this is a bug for sure.  Here's a simpler reproduction case:
var t = require('./')

var opt = {}
if (!isNaN(process.argv[2])) {
  opt.timeout = +process.argv[2]
}

t.test('test', opt, function (t) {
  t.plan(2)
  t.pass('this is fine')
  setTimeout(function (res) {
    t.pass('request complete')
  }, 2000)
})

Running directly with an argument, it prints the timeout pretty nicely:
$ node t.js 1000
TAP version 13
    # Subtest: test
    1..2
    ok 1 - this is fine
    not ok 2 - timeout!
      ---
      expired: test
      timeout: 1000
      at:
        line: 8
        column: 3
        file: t.js
      source: |
        t.test('test', opt, function (t) {
      ...
    # failed 1 of 2 tests
not ok 1 - test # time=1011.291ms
  ---
  timeout: 1000
  at:
    line: 8
    column: 3
    file: t.js
  results:
    plan:
      start: 1
      end: 2
    count: 2
    pass: 1
    ok: false
    fail: 1
    time: 1011.291
  source: |
    t.test('test', opt, function (t) {
  ...

1..1
# failed 1 of 1 tests
# time=2033.463ms

However, when executed with tap, the parent test runner process sends a SIGTERM, which it interprets as being a failure (which is correct), but hte output is kinda funky.  Note the missing test in the planned grandchild test, then the 1..0 of the child t.js test, and then the top level getting that sigterm failure (indicating that the child test ended with a signal, instead of exiting normally).
$ tap -t1 t.js -Rtap
TAP version 13
    # Subtest: t.js
        # Subtest: test
        1..2
        ok 1 - this is fine
    1..0
not ok 1 - t.js # time=1021.292ms
  ---
  results:
    ok: false
    count: 0
    pass: 0
    plan:
      start: 1
      end: 0
  command: /usr/local/bin/node
  timeout: 1000
  signal: SIGTERM
  arguments:
    - t.js
  file: t.js
  failure: timeout
  ...

1..1
# failed 1 of 1 tests
# time=1041.976ms

The "classic" reporter doesn't handle that funky output well at all, and kinda barfs on it:
$ tap -t1 t.js
t.js 1..0
t.js .................................................. 1/2 1s
  test
  not ok test count !== plan
    +++ found
    --- wanted
    -2
    +1
    results:
      ok: false
      count: 1
      pass: 1
      plan:
        start: 1
        end: 2

total ................................................. 1/2

  1 passing (1s)
  1 failing

What's happening is that it's seeing invalid TAP (plan not matching test count), and (correctly?) printing that out as some kind of disaster, but then missing the bigger (and more actionable) problem, that there was a timeout.
This patch is a pretty easy solution:
diff --git a/lib/root.js b/lib/root.js
index 2e16a81..5bdf2e1 100644
--- a/lib/root.js
+++ b/lib/root.js
@@ -96,5 +96,5 @@ onExit(function (code, signal) {
     at: null
   })

-  tap.end()
+  tap.endAll()
 })

which changes the output to be this:
$ tap t.js -t1
t.js .................................................. 1/3 1s
  test
  not ok test unfinished: test
    at:
      line: 8
      column: 3
      file: t.js
    plan: 2
    count: 1
    source: |
      t.test('test', opt, function (t) {

  not ok test point left in queue: not ok - received SIGTERM with pending event queue activity
    requests: []
    handles:
      - type: Timer
        msecs: 2000

total ................................................. 1/3

  1 passing (1s)
  2 failing

It still doesn't actually say "timeout", which is reasonable to infer if a SIGTERM comes, but it's a step in the right direction.
Another solution, maybe worth exploring, is to use the TAP_TIMEOUT environment var that the parent can set for spawned child procs.  The problem there is that it's very hard to distinguish between "the root tap in the parent test timed out" and "the child's root tap timed out based on an environment variable", and also it's totally unclear why that might've happened (an open socket or whatever) which is why the current SIGTERM behavior exists (but it's not being done properly, it seems).
It seems like perhaps the best behavior would be to let the parent runner proc send the SIGTERM, but interpret SIGTERM the same as a test timeout, but include any active handles at the time of getting the signal, so you'd get something like this:
$ tap t.js -t1
t.js .................................................. 1/2 1s
  child test
  not ok timeout!
    signal: SIGTERM
    handles:
      - type: Timer
        msecs: 20000

total ................................................. 1/2

  1 passing (1s)
  1 failing

The issue there is that then you might (in some desert island scenario) have a test where the process is getting a SIGTERM, and you don't want TAP to handle it, or it doesn't indicate a timeout.

Hm....
I think I may be seeing the same issue but wanted to share it to confirm. I've got:
process.env.NODE_ENV = 'test';
var test = require('tap').test;
var config = require('../config');
var server = require('../lib/server');

test('Starting server', function (t) {
  t.plan(2)
  t.pass('Server starting');
  server.listen(config.server.port, function () {
    t.pass('Server started');
  });
});

And I get:
test/00-setup.js ...................................... 2/3 30s
  not ok received SIGTERM with pending event queue activity
    requests: []
    handles:
      - type: Server
        events:
          - connection
          - clientError
          - close
          - error
          - listening
          - secureConnection
          - upgrade
          - checkContinue
          - request
        connectionKey: '6::::8080'

Is this what you are referring to?
Thanks

@lgomez The issue you're running into is that node-tap runs each test script as a separate process.  This is generally good, because you don't have to worry about state leaking between tests, and you always have a clean process environment.
However, if you have your tests structured like you would in mocha or lab, where the first test script starts a server, and then the others all do stuff with it, and then the last one closes it, then that obviously wont work, because the 00-setup.js test will always time out.
I usually approach this in one of two ways:

Every test file starts the server and tears it down.
Run all the tests in a single JavaScript file.
The 00-setup.js script spawns the server in a detached child process, and puts the child process pid in a file.  Then a zz-cleanup.js script reads the pid file, and kills the child process.

Option 1 is simpler, but a little bit slower, and if you want to test things that modify state (create a user, then have that user do a thing, then verify that another user can see the thing, etc.) then it won't work out so great, because the state is gone between processes.
Using Option 2 is pretty easy, but not as organized and can get quite messy.  Also, you give up all the advantages of having clean process env for each test!
Option 3 is a bit more tricky, but I've been meaning to write a module to make it easier, since I've hand-rolled this for a few different things now.  https://github.com/npm/npm-registry-couchapp/blob/master/test/00-setup.js has an example of doing this with CouchDB, but the same basic approach would work just fine for a node server as well.

@lgomez Check out https://github.com/tapjs/t-up

Haha.. You beat me to it. I was planning to code the detached process based on your couchapp reference from before.
This is one of those cases where I just realized I'm overcomplicating my tests based on optimistic expectations instead of keeping them simple and tabling this for when they actually need to share a server. The fact is my current case doesn't have enough tests and I can just keep it to a single file.
Thank you for tup though... I'm going to use it soon.

@remy Can you try with tap version 5.7 and see if it fails more helpfully?

Sorry for the super late reply - but yes. It's all good (though I'm about to raise a new bug related to timeouts! :smile: )

What is the solution to  received SIGTERM with pending event queue activity?
I'm running tap 6.3.2
Here's the repo https://github.com/playground/sample
I'm getting timeout error even though the tests passed.
user.test.js
var request = require('request');
var app = require('../server');
var test = require('tap').test;
test('First test!', function (assert) {
assert.plan(1);
request.get('http://localhost:3000/api/users', function(err, res) {
let json = JSON.parse(res.body);
assert.equal(json[0], 'John', 'name should be John');
//assert.equal(json[1], 'Joe', 'name should be John');
assert.end();
});
});
test('Second test!', function (assert) {
assert.plan(1);
request.get('http://localhost:3000/api/users', function(err, res) {
let json = JSON.parse(res.body);
assert.equal(json[0], 'John', 'name should be John');
//assert.equal(json[1], 'Joe', 'name should be Joe');
assert.end();
});
});
Server running on port 3000
First test!
✓ name should be John
Second test!
✓ name should be John

timeout!

2 passing (2s)
1 failing

test/user.test.js timeout!:
timeout!

@playground The I believe I did, Bob answer is "Don't leave event queue activity running long enough to get a timeout and SIGTERM".
Answered in #293.


kubectl - should go to previous context just like cd -

Kubectl?
If you’re talking about kubectx, that’s pretty much what the entire tool does.

never mind, you are right, also thanks for the -c option
On Sun, Sep 1, 2019 at 10:12 PM Ahmet Alp Balkan<EMAIL_ADDRESS>wrote:

Kubectl?
If you’re talking about kubectx, that’s pretty much what the entire tool
does.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ahmetb/kubectx/issues/172?email_source=notifications&email_token=AAO7Y5DQSCP4RVETFQ7A7ULQHSODHA5CNFSM4ISZRGKKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5UXBYY#issuecomment-527003875,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAO7Y5D7O4I6V2IDCIK6273QHSODHANCNFSM4ISZRGKA
.


ci: add step to cleanup kubernetes resources upon merge

Description:

Add cloudbuild yaml to cleanup the namespace for a PR once it has been merged!

@ckim328 - I've made some more changes after your approval. Can you please take a look at this diff which is between the commit just before your approval and the latest state now

@ckim328  Added a minor change in https://github.com/GoogleCloudPlatform/point-of-sale/pull/84/commits/dd32aad966084520f8566b88f349c0bae786be0e as per this Yaqs answer


[FanzyZones] app history and zone settings granular saving

Summary of the Pull Request
What is this about:
Fix for a FZ regression in 0.31.1 that caused Win+arrow to stop working properly.
The SaveFancyZonesData() method was used to save the app zone history when a window is zoned.
Because  SaveFancyZonesData() also saves the zones-settings.json file, the  zones-settings.json was triggered to reload the fine and recreate some internal data structure, causing win+arrow to not work properly.
What is include in the PR:
This PR replaces some calls to SaveFancyZonesData() with calls to SaveAppZoneHistory() to avoid triggering the file watcher for zones-settings.json
How does someone test / validate:
Turn on these options:

Create a large grid layout (9 or more zones) and try zoning a window using win+arrows and win+ctrl+alt+arrows.
Also test that app-zone-history.json is correctly updated when removing virtual desktops.
Quality Checklist

[x] Linked issue: https://github.com/microsoft/PowerToys/issues/9453
[ ] Communication: I've discussed this with core contributors in the issue.
[ ] Tests: Added/updated and all pass
[ ] Installer: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Docs: Added/ updated
[ ] Binaries: Any new files are added to WXS / YML

[x] No new binaries
[ ] YML for signing for new binaries
[ ] WXS for installer for new binaries

Contributor License Agreement (CLA)
A CLA must be signed. If not, go over here and sign the CLA.

I couldn't reproduce the issue in 0.31.1


Punctuation corrector

I would like to add a project for punctuation corrector using NLP. Here, the punctuations given will be corrected from a set of data take as input.

6 days has passed by.. there is no discussion on this issue

6 days has passed by.. there is no discussion on this issue


fix: update TSLint template to align with DT changes

On DT all definitions have been mass migrated to use
<EMAIL_ADDRESS>as a value. But this is not a case
for newly created DT definitions when using recommended in README
dts-gen .... command.
Thanks!
See: DefinitelyTyped/DefinitelyTyped#57489

@peterblazejewicz Thank you. I want to create the same PR.

Would be nice if this could be merged sometime soon...


TeamCity output only shows summary

I don't see any line, branch or method level output when using under teamcity as documented here.
I am using the following command to generate test coverage data dotnet test UnitTests.csproj /p:CollectCoverage=true /p:CoverletOutputFormat=teamcity
I can see in this pull request that outputting line an method coverage service messages was removed?

@dijitle you did that PR can you take a look?
Actually I see CodeCoverageL and CodeCoverageM can you explain why?

@MarcoRossignoli yes I see those values. I think this might be my misunderstanding of expected output if what you say is correct about the CodeCoverageB, CodeCoverageL, CodeCoverageM, CodeCoverageC lines.

TeamCity should work with service message as explained https://www.jetbrains.com/help/teamcity/build-script-interaction-with-teamcity.html#Reporting-Build-Statistics
Do you see "empty" results or are these "unexpected" in values?

@msnelling let me know of feel free to close if it's conform to team city specifications.

@MarcoRossignoli no I see the correct values, I think this was just my misunderstanding. I thought the CodeCoverageB, CodeCoverageL, CodeCoverageM, CodeCoverageC would give me detailed coverage like in a report.
Feel free to close the issue.


Take into account that database section names can get out of sync with runtime section names

We currently use the name of a section as part of the identity of the evaluation of an exercise, and store it together with the user input whenever they evaluate an exercise.
This is a problem since section names can change or dissapear entirely so we should handle the cases when we have a section in the database that is not available at runtime. This involves

creating a migration every time a section identity that's been published changes
handling the scenario outlined above in our code, specifically when visiting a library and resolving the last seen section from the user's progress.

Duplicate of #508


Requests are not visible in "on" event

Puppeteer version: "puppeteer": "^0.10.2"
Platform / OS version: win 10, 64b

expected: request to home https://www.example.net/ and https://www.example.com/'
actual:
REQUEST TO : https://www.example.net/
GET 200 https://www.example.net/
REQUEST TO : https://www.example.net/favicon.ico
(node:4640) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): Error: Protocol error (Network.continueInterceptedRequest): Invalid InterceptionId. undefined
(node:4640) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
REQUEST TO : https://www.example.com/favicon.ico
GET 404 https://www.example.com/favicon.ico
In log, there is not request to REQUEST TO : https://www.example.com/ only on https://www.example.com/favicon.ico
See code:
const puppeteer = require('puppeteer')
const path = require('path')

const URL = 'https://www.extrategy.net/en/values'

const run = async (browser) => {
	const page = await browser.newPage()
	page.setRequestInterceptionEnabled(true)

	page.on('request', request => {
		console.log("REQUEST TO : " + request.url);
		request.continue(); // pass it through.
	});

	page.on('response', response => {
		const req = response.request();
		console.log(req.method, response.status, req.url);
	});

	await page.goto('https://www.example.net/')
	await page.goto('https://www.example.com/')

}

(async () => {
  const browser = await puppeteer.launch({headless: false})
  await run(browser)
  //return browser.close()
})()

Strangely, the second response is null, despite document opening in the browser (puppeteer v0.11.0-alpha):
'use strict';

const puppeteer = require('puppeteer');

(async () => {
  const browser = await puppeteer.launch({ headless: false });
  const page = await browser.newPage();

  await page.setRequestInterceptionEnabled(true);

  page.on('request', (request) => {
    console.log(`REQUEST TO: ${request.url}`);
    request.continue();
  });

  page.on('response', (response) => {
    const req = response.request();
    console.log(req.method, response.status, req.url);
  });

  const resp1 = await page.goto('https://www.example.net/');
  console.log(resp1 && resp1.status);
  const resp2 = await page.goto('https://www.example.com/');
  console.log(resp2 && resp2.status);

  // await browser.close()
})();

REQUEST TO: https://www.example.net/
GET 200 https://www.example.net/
200
REQUEST TO: https://www.example.net/favicon.ico
null
REQUEST TO: https://www.example.com/favicon.ico
GET 404 https://www.example.com/favicon.ico

I am not sure but sometime when requesting to https://google.com i got error.

full console


PR failure while using NFS or JuiceFS

Description
I'm using docker to run gitea, using volumes: /data:/data to mount space from container to local.
At first, I'm using juicefs as file system, and when i tried to pull request in a large repo, the gitea website will stuck and then break down on code 504. The docker logs showed as below: Unable to run ls-files -u -z! Error: git ls-files -u -z: context deadline exceeded

Then I thought it was the juicefs problem, so I tried nfs instead, but the performance is the same.
At last, I used ubuntu local file system, there was no problem any more, and the speed of PR or push was very quick.
So, can gitea use a network-file-system or cloud-file-system as storage rather than local fs? And what might be the reason of this situation?
Thanks.
Gitea Version
1.16.8
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
No response
Screenshots

Git Version
2.25.1
Operating System
ubuntu
How are you running Gitea?
Docker
Database
MySQL

I would say your git on NFS is just slow ... so increase timeouts?
-> https://docs.gitea.io/en-us/config-cheat-sheet/#git---timeout-settings-gittimeout

I would say your git on NFS is just slow ... so increase timeouts? -> https://docs.gitea.io/en-us/config-cheat-sheet/#git---timeout-settings-gittimeout

Thanks a lot, but I wonder why it is slow for git on NFS? Could you please outline the reason if possible? Thanks again.

if you have high ping in the line ... and git does some sequential work ... it could handle throu thousands of files that depend on each other, so if NFS is not async ... you wait thousand*ping time + the computation work

Inactive for a long time. Feel free to re-open if there are new problems.


Failed to conform to ES5.1, Section <IP_ADDRESS>.5 DeleteBinding (N), Step 4 & 5

It seems to me that S5 does not conform to ES5.1, Section <IP_ADDRESS>.5 DeleteBinding (N), Step 4 & 5.
For the following test program:
// <IP_ADDRESS>.5 DeleteBinding (N), Step 4 & 5

// 4. Remove the binding for N from envRec.
// 5. Return true.

// If a binding of a declarative environment record (i.e., not in a global scope nor an 'with' construct) is deletable, then remove it and return true.

// The only way to create a deletable binding is to call CreateMutableBinding with the second argument 'D' of true,
// while the only time to call CreateMutableBinding with 'D' of true is when 'configurableBindings' is true in "10.5 Declaration Binding Instantiation",
// while 'configurableBindings' is true only when 'code' is eval code.
// Thus, a binding of a declarative environment record is deletable only when
// - a function or a variable name is declared in a eval code
// - a formal parameter of a function or an argument object of a function in a eval code (although, in this case, it is not specified whether 'configurableBindings' is used or not)

// a variable declaration in an eval code
function f11() {
  eval(" var x11 = 1; delete x11; ");
  return x11; // ReferenceError
}
try {
  f11();
  throw "Not here!";
} catch (e) {
  if (!(e instanceof ReferenceError)) {
    $ERROR('function f11() { eval(" var x11 = 1; delete x11; "); return x11; } f11(); throws ReferenceError. Actual: ' + e);
  }
}

S5 throws an exception, while it should have succeeded:
$ ./s5-test262 /tmp/12.js
../envs/es5.env:6:0-5029:0
<unknown>:20:-1--4
<unknown>:20:-1--4
<unknown>:20:-1--4
<unknown>:24:-3-25:-125
<unknown>:25:-5--124
<unknown>:25:-5--11
../envs/es5.env:977:28-983:3
../envs/es5.env:980:11-982:5
../envs/es5.env:287:28-293:1
Uncaught exception: {[#proto: @9, #class: "Object", #extensible: true,]
 'message' : {#value "Unbound identifier: $ERROR" ,
              #writable true ,
              #configurable false}}

Fatal error: exception Failure("Runtime error")

Is there anything that I'm missing?

Is it because S5 supports only strict mode?

Not applicable in the strict mode. Sorry for disturbing.


error 500

i keep getting 500 errors when i try to add something to a list

its been like this since saturday btw


[chore] fix deeplinks

Description
LEARNER-8937
After redesigning the home screen, removing discovery, and adding Profile into a tab, deep links need to be fixed.
Deeplinks that needs to be fixed:

[ ] Profile
[ ] Settings
[ ] Course Tab
[ ] Course details screen
[ ] Program Tab
[ ] Program details screen
[ ] Deeplink to course component

How to test this PR
To test deeplinks, use the following urls using terminal xcrun simctl openurl booted '<url>'
https://edx.app.link/settings
https://edx.app.link/profile
https://edx.app.link/program
https://edx.app.link/discovery_home
https://edx.app.link/discover_course_detail_itpm
https://edx.app.link/cs50itc_dates
https://edx.app.link/cs50itc_component

https://edx.app.link/program_discovery
This is not valid deeplink, removed from description.

https://edx.app.link/cs50itc_dates, https://edx.app.link/cs50itc_component
These 2 links are working fine.


Unhandled exception: Access violation writing location in 0.9 devel

Awesome framework. I haven't gotten it to work yet though.
I grabbed the source from the 0.9-devel branch because older sources didn't compile for me or had other issues.
I used CMake to build cppnetlib-client-connections.lib and cppnetlib-uri.lib. Linked it with my project. Wrote some simple test code, build it with VS2010 and ran my application.
http::client client;
http::client::request request("the url");
request << header("Connection", "close");
http::client::response response = client.get(request);
std::cout << body(response) << std::endl;
The exception occured in: void async_connect(...) in asio/basic_socket.hpp at the following line,
BOOST_ASIO_MOVE_CAST(ConnectHandler)(handler), ec));
It reads: violation writing location 0x00000014.
It always occurs. Is there a simple fix for this? Do you need more info?
Update:
I ran a different target of my application and this time the same exception occurred somewhere else.
This time in handle_resolved(...) ../client/connection/async_normal.hpp
Regards,
Wouter

Unfortunately I don't have access to the compiler+platform, but can you say what version of Boost you're using? Also it seems that this is an issue with the move emulation/implementation in Asio. It may be that this is specific to Boost.Asio and MSVC 2010.

Ah that's a shame. Well I grabbed the latest boost, 1.48 and build it myself. Thanks for replying so soon :)

One suggestion is to use an older version of Boost -- we have been developing against Boost 1.45 to keep with backward compatibility but occasionally do try and build with 1.47. Now that 1.48 is already out we have yet to test against it, but will definitely be something we should be doing.
Thanks for the report -- also can you say whether this is happening in debug or release mode?
If you also don't mind, can you please send in your sample code (as a gist perhaps) and/or resulting stack traces so that we can properly diagnose the situation as well? It may very well be a bug in the way we do use Boost.Asio and I'd like to get that fixed before 0.9.3 is released.
Cheers

Hi Dean. I was busy yesterday and today aswell. I benched my project so I cleaned up all build stuff.
But because this is in the interest of getting 0.9.3 released I will build everything again tonight and get you the information you requested.
Regards,
Wouter

I tried again and this this also in release which also gives the very same exception. I made a dmp of the crash which is around 80MB. Can you use that? It's exported from VS2010.
http://pastebin.com/JtS6MeAh Here is the netlib code I use, the exception and the callstack. Its rather large due to the template stuff :/

This doesn't help. Can you do the following instead:
http::client c;
http::client::request r("http://www.boost.org/");
r << header("Connection", "close");
http::client::response res = c.get(r);
std::string body_ = body(res);
std::cout << body_ << std::endl;
Also, can you say what version of Windows you're running this on?

Alright. Sorry for not giving such good information.
I just build that piece of code in an empty console project and it runs fine.
My code was almost the same except for this line: std::cout << body(response) << std::endl;
Now I just use std::string body_ = body(res); in my project and I can confirm it goes haywire in the body function.
I am running Windows 7 SP1 64bit. I have a downloader.cpp file which simply downloads a string from the given uri (thus containing the code above).
It compiles into a static library which is then linked against a CLI and a GUI version of my application. For the GUI I use Qt 4.8, this however should not matter as the exception occurs in the CLI build as well. An additional library I use is TinyXml.
The CLI version gives the exception differs at bit (the callstack is 3 calls shorter). It strands in handle_resolved(...) in async_normal.hpp 'Line 135 + 0x22 bytes	C++'
Where that is 'Line 135 + 0x18c bytes	C++' for the GUI version.
I saw that the vtable of the argument 'body_callback_function_type callback' is 0 and functor->obj_ptr = 0xcccccccc. Might be nothing.  Especially because of my lack of experience.
I hope this helps.

I have build a version where my library is a sharedlib and it doesnt make a difference. I have used #define BOOST_NETWORK_NO_LIB as well. It might be an issue in my code somewhere but wouldnt know where. I do have some other boost code in there but that is limited to the filesystem and Regex components.

The body callback function isn't used if you don't provide it.
Do you define _SCL_SECURE or have you built Boost appropriately to use/not-use the secure iterators? This has been a constant pain for MSVC builds and because I don't develop on MSVC primarily it's hard for me to track this down.
Your help would be most appreciated if you can provide more information that's relevant in your set-up (or if you have code that others can use to reproduce the problem). It would really help if you can say whether the example applications build and run properly on your system (or if the tests even run/fail).

I'm not entirely sure whether the system_error thing is entirely relevant. That said unless you can provide a minimal example that others can reproduce, I'm afraid we can't really help much. Can you close the issue if you think this is acceptable?

I didn't think it was relevant either. I did think it might be something you didn't know about yet.
Thanks for your time.


Using Custom SSL Certificates

Hi,
I have a .pfx file along with its password obviously... I want to use this while decrypting SSL. This is what I do:
var proxyServer = new ProxyServer("path/to/.pfx_file", "Issuer Name", true, true, true);
proxyServer.CertificateManager.PfxPassword = "password";
var httpProxy = new ExplicitProxyEndPoint(IPAddress.Parse(IP), Port, decryptSsl: true);
proxyServer.AddEndPoint(httpProxy);
proxyServer.BeforeRequest += OnBeforeRequest;
proxyServer.BeforeResponse += OnBeforeResponse;
proxyServer.AfterResponse += OnAfterResponse;

proxyServer.Start();

I had put this .pfx certificate in the client's Trusted Certificate Store. While accessing the proxy from the client it does not work... Shows security issue

The first parameter in ProxyServer is not a path, the name of the root certificate to lookUp. If I recall right it looks at the execution directory where the proxy.dll is located.
You can take a look at certificate manager for details. To use a pfx path directly, I think you would need to set call below method, before starting the proxy.
proxyServer.CertificateManager.LoadRootCertificate(pfxFilePath, password, false);
See CertificateManager implementation in below link for details.
https://github.com/justcoding121/Titanium-Web-Proxy/blob/master/src/Titanium.Web.Proxy/Certificates/CertificateManager.cs

See CertificateManager API docs below.
https://justcoding121.github.io/Titanium-Web-Proxy/docs/api/Titanium.Web.Proxy.Network.CertificateManager.html

It would be nice if you could post a code segment for my use-case... I tried various methods but it seems I am not doing it the correct way


zap exec

Implement the zap exec command.
Make sure running the root command zap <binary> will attempt first to run scripts, and then fallback to exec.

294fd7c


Install Pre-reqs Needs Attention

I'm running into a lot of issues trying to get the pre-req tools installed.  Most of my issues have been trying to figure out what exactly needs to be installed to get azdata to install.  I went down some rabbit holes trying to put the right C++ build tools on my test machine but still couldn't get azdata to install due to Cython issues and CLI issues.  I think the documentation needs some more requirements or guidance on the Cython requirement.

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

ID: 99908ff8-cfba-af0d-1415-c366a9f380c5
Version Independent ID: 30494d83-bf1e-6085-04b3-8b02ff19e301
Content: Install azdata using pip - SQL Server big data clusters
Content Source: docs/big-data-cluster/deploy-install-azdata.md
Product: sql
Technology: big-data-cluster
GitHub Login: @MikeRayMSFT
Microsoft Alias: mikeray

Perhaps a better way to say this:  I have a clean Windows 10 VM from Azure.  What software packages will this workstation need in order to successfully install Azure Data Studiom- Insiders and the required big data tools for building a BDC on AKS.

@cdotzenrod Thank you for the feedback, assigning this to the author for review.

@cdotzenrod Same thing on both Cython and C++ v14.
I did PS> pip install Cython
That got past the Cython error.
Next, C++ error.
I did PS> choco install visualstudio2017enterprise
This is version 14 of C++. Still errored. Next installed 2019, same error.
I let Buck Woody know of the install error.
Environment: Azure D2 VM (Win10 virgin install)

The latest recommendation is to install azdata with a package manager:

Windows installer
Ubuntu


Add lifebank logo to visual resources

Add lifebank logo to visual resources

@JeanVegaD  This issue should be done for both english and Spanish repositories


Register provider via policy For ManagedNetworkFabric.

Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

/azp run java - managednetworkfabric - ci

API change check
API changes are not detected in this pull request.

/azp run java - managednetworkfabric - mgmt - tests


[wasmtime-api] reduce examples complexity: hostref for store/engine

The API examples contain lots of boiler plate stuff. Using HostRef in parameters as reference simplifies how examples are read.

My understanding is that it's generally better to do .clone() on the caller side than the callee side, especially for a new function, so that it's clear from reading the code that the constructed object isn't capturing a reference to the object. Also, if the caller doesn't need the original object, it can avoid the clone.
However, I'm open to opinions here.

We'll need to rebase this for the CI fixes.


[api] Enhancement to support custom action buttons

Added custom_actions virtual attribute to service_templates
Added custom_action_buttons virtual attribute to service_templates
Delegating custom_actions and custom_action_buttons from services to their service_template
When queries services, expand the available actions to include custom action buttons
Update Rest API parser to allow non api.yml specified actions when custom actions are allowed for resources.
Implemented a generic custom action method for verifying and invoking a custom action
Support simple button actions and return REST API action result signature

https://trello.com/c/7C2VEFGM/122-api-add-support-for-custom-service-button-actions

Marked as WIP

Still need to support custom button dialogs
Add specs

custom_actions return the custom buttons as layed out in the UI on the service.  custom_action_buttons returns the flattened array of custom buttons.
Fetching custom actions and custom action buttons for a service_template or a deployed service
GET /api/service_templates/:id?attributes=custom_actions,custom_action_buttons
GET /api/services/:id?attributes=custom_actions,custom_action_buttons

i.e.
{
  "href": "http://localhost:3000/api/service_templates/7",
  "id": 7,
  "name": "SampleServiceCatalogItem2",
  "description": "Sample Service Catalog Item 2",
  "guid": "47edc73a-4cef-11e5-8525-000a27020067",
  "options": {
    "button_order": [
      "cb-1",
      "cb-2",
      "cbg-43",
      "cb-3",
      "cb-4",
      "cb-5",
      "cb-6"
    ]
  },
  "created_at": "2015-08-27T19:10:29Z",
  "updated_at": "2015-09-16T18:24:13Z",
  ...
  "custom_actions": {
    "buttons": [
      {
        "id": 1,
        "guid": "3d5975fe-57bb-11e5-8d22-000a27020067",
        "description": "Rescan Service Offering",
        "applies_to_class": "ServiceTemplate",
        "options": {
          "button_image": 2,
          "display": true
        },
        "userid": "admin",
        "created_on": "2015-09-10T12:55:40Z",
        "updated_on": "2015-09-16T14:28:17Z",
        "name": "Rescan",
        "visibility": {
          "roles": [
            "_ALL_"
          ]
        },
        "applies_to_id": 7
      },
      {
        "id": 2,
        "guid": "7c900d18-57c1-11e5-8d22-000a27020067",
        "description": "Ping Service Status",
        "applies_to_class": "ServiceTemplate",
        "options": {
          "button_image": 3,
          "display": true
        },
        "userid": "admin",
        "created_on": "2015-09-10T13:40:23Z",
        "updated_on": "2015-09-16T14:16:21Z",
        "name": "Ping",
        "visibility": {
          "roles": [
            "_ALL_"
          ]
        },
        "applies_to_id": 7
      }
      ...
    ],
    "button_groups": [
      {
        "id": 43,
        "name": "MyButtonGroup|ServiceTemplate|7",
        "description": "My Custom Buttons",
        "created_on": "2015-09-10T13:41:20Z",
        "updated_on": "2015-09-15T14:53:49Z",
        "guid": "9e4a9900-57c1-11e5-8d22-000a27020067",
        "set_data": {
          "button_order": [
            3,
            4,
            5,
            6
          ],
          "button_image": 1,
          "display": true,
          "applies_to_class": "ServiceTemplate",
          "applies_to_id": 7,
          "group_index": 1
        },
        "owner_type": "ServiceTemplate",
        "owner_id": 7,
        "buttons": [
          {
            "id": 3,
            "guid": "b9afcdfa-57c1-11e5-8d22-000a27020067",
            "description": "Sub Service Ping",
            "applies_to_class": "ServiceTemplate",
            "options": {
              "button_image": 3,
              "display": true
            },
            "userid": "admin",
            "created_on": "2015-09-10T13:42:06Z",
            "updated_on": "2015-09-16T14:27:30Z",
            "name": "Sub-Ping",
            "visibility": {
              "roles": [
                "_ALL_"
              ]
            },
            "applies_to_id": 7
          },
          ...
        ]
      }
    ]
  },
  "custom_action_buttons": [
    {
      "id": 1,
      "guid": "3d5975fe-57bb-11e5-8d22-000a27020067",
      "description": "Rescan Service Offering",
      "applies_to_class": "ServiceTemplate",
      "options": {
        "button_image": 2,
        "display": true
      },
      "userid": "admin",
      "created_on": "2015-09-10T12:55:40Z",
      "updated_on": "2015-09-16T14:28:17Z",
      "name": "Rescan",
      "visibility": {
        "roles": [
          "_ALL_"
        ]
      },
      "applies_to_id": 7
    },
    ...
  ]
}

The custom actions (buttons) are exposed as actions on the deployed services from the REST API, not the service_templates. The action name to trigger the action is the lower-cased name of the custom button.
As an example, here's a service without custom buttons:
{
  "href": "http://localhost:3000/api/services/90",
  "id": 90,
  "name": "new_service_90",
  "description": "This is a new description for the service 90",
  ...
  "actions": [
    {
      "name": "edit",
      "method": "post",
      "href": "http://localhost:3000/api/services/90"
    },
    {
      "name": "retire",
      "method": "post",
      "href": "http://localhost:3000/api/services/90"
    },
    {
      "name": "delete",
      "method": "delete",
      "href": "http://localhost:3000/api/services/90"
    }
  ]
}

Here is an example of a service with custom buttons:
{
  "href": "http://localhost:3000/api/services/105",
  "id": 105,
  "name": "SampleServiceCatalogItem2",
  "description": "Sample Service Catalog Item 2",
  "guid": "24c193a2-57c8-11e5-9332-000a27020067",
  "service_template_id": 7,
  "options": {
    "button_order": [
      "cb-1",
      "cb-2",
      "cbg-43",
      "cb-3",
      "cb-4",
      "cb-5"
    ],
    "dialog": {
      "dialog_DynamicMemory": "t"
    }
  },
  "display": true,
  "created_at": "2015-09-10T14:28:02Z",
  "updated_at": "2015-09-10T14:28:02Z",
  "evm_owner_id": 1,
  "miq_group_id": 1,
  "actions": [
    {
      "name": "edit",
      "method": "post",
      "href": "http://localhost:3000/api/services/105"
    },
    {
      "name": "retire",
      "method": "post",
      "href": "http://localhost:3000/api/services/105"
    },
    {
      "name": "delete",
      "method": "delete",
      "href": "http://localhost:3000/api/services/105"
    },
    {
      "name": "rescan",
      "method": "post",
      "href": "http://localhost:3000/api/services/105"
    },
    {
      "name": "ping",
      "method": "post",
      "href": "http://localhost:3000/api/services/105"
    }
  ]
}

Here, the rescan and ping are custom buttons for the service and are returned after the common actions.
Triggering those actions is simply a
POST /api/services/:id

{
  "action" : "<custom_action_name>"
}

@gtanzillo @imtayadeway wip, let me know if direction ok so far.
@dclarizio @eclarizio interested to know the custom_actions hash returned is good enough for the UI to show the button/button group hierarchy in the ssui or if you need additional info.
Thanks

Looks good so far :+1:

@gtanzillo @imtayadeway  added specs, un-wipped.

:+1:


Adding ability to set Varnish TTL with Class params

Adding ability to override the default Varnish TTL with Class params.

Is it possible to rerun the travis-ci test. Looks like it failed due to a gem installation issue for 1.8.7?
I just ran the same checks under my travis-ci account and they all passed. Wondering if it was just a fluke w/ rubygems at the time.
https://travis-ci.org/cybo42/puppet-varnish/builds/105492424

It looks like a problem with the Gemfile.lock not specifying a version for mini_portfile2.  I'll be looking at this tomorrow and I'll re-run both the Travis CI jobs and merge.
Thanks for contributing!

Awesome, thank you for the module! Let me know if I could help in anyway.


os: TestOpenFileLimit fails on openbsd

For #46279, we added os.TestOpenFileLimit, which checks that it can open 1200 files.
This test fails on openbsd, even after doing setrlimit(RLIMIT_NOFILE, lim) with lim.Cur raised to lim.Max.
Perhaps the builder runs with a hard limit < 1200.
Whatever the reason, OpenBSD is not a first-class port,
so the test will be skipped with a pointer to this issue,
and we will leave the issue for any OpenBSD maintainers
that want to take a look.

FYI: It still failed on openbsd/mips64
https://build.golang.org/log/8d2a91e75462acd10f0dc525f53e25004631e444


Outlook Exchange Online: When Plan to GA: Enable SSO in an Office Add-in using nested app authentication (preview)

we notice Enable SSO in an Office Add-in using nested app authentication (preview) is still in preview. but exchange online is will be turned off by default for all Exchange Online tenants in October 2024. https://devblogs.microsoft.com/microsoft365dev/new-nested-app-authentication-for-office-add-ins-legacy-exchange-tokens-off-by-default-in-october-2024/?ocid=microsoft365dev_eml_tnp_autoid106_image
So, what's the plan for this issue?

Hi @hello-chenchen,
Nested app authentication (NAA) is still in preview. Although we are unable to provide a set release date at this time, we are trying to launch the feature as soon as possible. In the interim, we recommend using Single sign-on with the on-behalf-of flow .
I will assign this issue to @davidchesnut, in case there is other guidance we can provide.

Hi @hello-chenchen, Sorry cross-posting this from the duplicate issue. We plan to get NAA to GA very soon now. We'll also provide more information as we go on either the Office Add-ins community call or the M365 developer blog. Also we will keep updating the sample with any changes that come along. Please take this time to try out NAA and let us know if you spot any issues. You're in the right location to let us know about any issues, or questions you have!
Thanks!
David


EmbedsMany::detach() doesn't working

https://github.com/jenssegers/laravel-mongodb/blob/1303b5fb05d1c8f06d29c55b79e3e5468c946f45/src/Relations/EmbedsMany.php#L181
It seems that EmbedsMany::detach($ids = []) won't remove all embedded documents if no $ids inputted.
Should the code returns all embeds in this case?
        $models = $this->getResults();
        if (!empty($ids)) {
            $models = $models->only($ids);
        }

Sorry I found the EmbedsMany::destroy() does the thing.


linear_model_issue3

Hi @jstac , I removed one of two X_0 in lecture.

Thanks a lot, @sayaikegawa and @jstac .
Good morning, @jstac , as we discussed, in the subsection Time Trends of lecture linear_model, I calculated G = [2a a+b c], added G and kept x_0 = [0 0 1]  (as @sayaikegawa did).
You can also see the ipynb file compiled by the rst file of this subsection with these changes, here.

Thanks guys, nice PR.
When I look at the bottom of this display, I feel like the last line should change.  Perhaps it should be something like
"You can now confirm that :math:y_t = G x_t has the correct form."
What do you think?

Thanks, @jstac . I think it is a good idea.
Given your advice, I kept our previous changes and rephrased the last sentence as you instructed. Please see the new ipynb file compiled by the rst file with this new modification, here.

Thanks @shlff and @sayaikegawa  --- this is a valuable improvement and a nice, clear PR.  Great work.


Add steamid to UserSummary

Te field steamid is returned by the Steam Web API's GetPlayerSummaries endpoint but being ignored by this library.

Please, change the field name to camel case (steamid -> steamId)

Sure, I can change it. I just wanted to be somewhat consistent in this code.
But the code actually contains all of this occurrences: steamid, steamId and steamID 😉

Done.


I still see check announcement message happening every 1 minute

I still see the below, I thought the recent YouTube video (https://www.youtube.com/watch?v=4gul6AqAoEo) mentioned that the announcement check time is now reduced to milliseconds.
No coins announced, or coin has already been bought/sold. Checking more frequently in case TP and SL need updating. You can comment me out, I live on line 176 in main.py
No coins announced, or coin has already been bought/sold. Checking more frequently in case TP and SL need updating. You can comment me out, I live on line 176 in main.py
Checking for coin announcements every 1 minute (in a separate thread)

Is this still a problem?


feat: add a prototype  generator

This prototype is meant to be run as a protoc plugin and yield a HTTP/JSON client library. It is missing many features and has many todos. Currently this is only tested by one "golden" style test that validates generated output matches cached testdata.

Let's make sure test is run on pull request before we merge this.


How can i Convert *bin and Textures to Only One *.gltf  to use?

I buyed this plugins ,but i dont know how to use in my work. I DownLoad some gltf from Sketch.com or Export from UE4, then
Called glTFLoadAssetfromFileName failed. here is the  Error: LoadPrimitive(): Unable to load POSITION attribute.

Hi, you should use gltf embedded files if you want the single-asset approach.
Otherwise just use the github version that includes support for loading external files.
Fell free to join the discord channel for help.


Improve the precommit hook

In this PR:

I follow the more recent guidance from the rush team on how to write a precommit hook: https://rushjs.io/pages/maintainer/enabling_prettier/. Fixes #11076.
The prettier wrapper is switched from precise-commits to pretty-quick. The main difference between the two is that the latter works on whole files instead of changed regions only, so no more bizarre formattings. Furthermore, .prettierignore is picked up correctly and is used to filter out staged files that we do not want to run prettier on. Fixes https://github.com/Azure/azure-sdk-for-js/issues/10770 and fixes https://github.com/Azure/azure-sdk-for-js/issues/11077.

Epic: #11105.

@ramya-rao-a I rebased your branch in this issue #10770 on this one and found the formatting changes applied to be reasonable:

@ramya-rao-a can you confirm if this change is good for you? I'd love to get it into master soon.


[Snyk] Security upgrade com.amazonaws:aws-java-sdk from 1.12.318 to 1.12.415

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `maven` dependencies of this project.
:sparkles: Snyk has automatically assigned this pull request, set who gets assigned.
As this is a private repository, Snyk-bot does not have access. Therefore, this PR has been created automatically, but appears to have been created by a real user.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

uasdm-server/pom.xml

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Upgrade
Breaking Change
Exploit Maturity

616/1000  Why? Proof of Concept exploit, Has a fix available, CVSS 5.9
Denial of Service (DoS) SNYK-JAVA-COMFASTERXMLJACKSONCORE-3038424
com.amazonaws:aws-java-sdk:  1.12.318 -> 1.12.415
No
Proof of Concept

616/1000  Why? Proof of Concept exploit, Has a fix available, CVSS 5.9
Denial of Service (DoS) SNYK-JAVA-COMFASTERXMLJACKSONCORE-3038426
com.amazonaws:aws-java-sdk:  1.12.318 -> 1.12.415
No
Proof of Concept

(*) Note that the real score may have changed since the PR was raised.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
👩💻 Set who automatically gets assigned
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Denial of Service (DoS)
🦉 Denial of Service (DoS)

Out of date


NTR transcription termination signal

synonym
polyA site downstream element
polyA site associated transcription termination signal
Nrd1-dependent terminator
Nrd1 binding motif
motif UGUAA/G and UCUUG motifs also reported as (A(U)GUA)
parent transcription_regulatory_region (SO:0001679)
Ref: PMID:28367989 (and references therein)
Def: termination signal preferentially observed downstream of polyadenylation signal

I will work on this and get it added by tomorrow.

Thanks Nicole!

We have SO:0000141 terminator and SO:0000951 eukaryotic_terminator terms as children of transcription_regulatory_region. Would it be better for this term to be placed under one of these terms?

Oh yes please. Under SO:0000951 eukaryotic_terminator
v


Link

fix naive bayes and countvectorizer model to fit word classification

May I ask you to modify the title of this PR to make it meaningful

May I ask you to modify the title of this PR to make it meaningful

feel free to change it

There are some conflicting files, you should resolve them through merge or rebase.


Update AppRegistryData data fields to be in sync with the new apps_me…

it fixes https://github.com/aiidalab/aiidalab-docker-stack/pull/174


Add option to disallow modifying properties of parameters

PR checklist

[ ] Addresses an existing issue: #0000
[x] New feature, bugfix, or enhancement

[x] Includes tests

[x] Documentation update

Overview of change:
Added an option to no-parameter-reassignment to disallow adding/modifying/deleting the properties of argument objects as well.
Is there anything you'd like reviewers to focus on?

CHANGELOG.md entry:
[new-rule-option] no-parameter-reassignment: Add props option

When implementing this rule I decided not to add this code.
The purpose of this rule is to prevent reassigning parameters, because you cannot declare them as const.
Modifying objects passed as parameter can be prevented by choosing the correct type. You can declare it as follows:
type DeepReadonly<T> = {
     readonly [K in keyof T]: DeepReadonly<T[K]>
};
function foo(bar: Readonly<A>, baz: DeepReadonly<B>) {
    bar.prop++; // compile error
    bar.nested.prop++; // no compile error, because Readonly is shallow

    baz.prop++; // compile error
    baz.nested.prop++; // compile error
}

After all you should prefer compile errors over lint errors. That's why I'm 👎  on this

Btw: "parameter properties" !== "parameter's properties"

That's true. But I can't enforce using DeepReadonly over a whole project. Adding a linter rule makes this behaviour consistent over all functions.

Ok, if I leave my opinion aside, there are still some cases that are currently not detected by your implementation:
foo.bar.baz++;
foo[0]++;
(foo).bar++;
// and combinations of the above

// this is a bit tricky and maybe not worth the effort
const {bar} = foo:
bar.baz++; // DeepReadonly would catch this one

Also note that your parameters are not immutable with this option. Collections can still be modified through their methods, e.g. Array.prototype.push.

I can't enforce using DeepReadonly over a whole project

You can. A rule to enforce that would be way easier than this rule.

I have the same concerns as @ajafff; I don't think this belongs in the core rule set since it's tricky to get right and is much more natural as a compiler check. Feel free to implement it as a custom rule though.

OK, I agree. I kinda underestimated the impact/complexity of this. Thanks for reviewing so quickly, @ajafff @adidahiya.


feat(ci): use Actions cache for docker builds

Description
Use Github Actions cache for storing Docker build cache.
Type of Change
[ ] Bug Fix
[x] New Feature
[ ] Breaking Change
[ ] Refactor
[ ] Documentation
[ ] Other (please describe)
Checklist

[x] I have read the contributing guidelines
[x] Existing issues have been referenced (where applicable)
[x] I have verified this change is not present in other open pull requests
[x] Functionality is documented
[x] All code style checks pass
[x] New code contribution is covered by automated tests
[x] All new and existing tests pass

I'll experiment with optimizing the Docker builds further in a separate PR based on this: https://docs.docker.com/build/ci/github-actions/cache/#cache-mounts

Closing as GH Actions cache is limited and I'll try to find a way to mount the cache entries created at verification steps into the build in order to avoid duplicating the cached data and to hit the cache storage limit.


Creating a new project with UI library does not work (antd as an example)

trying to create a POC nuxt app using https://github.com/AmpleOrganics/Blaze.vue, could not get my component library to install so i created a default create-nuxt-app using the antd and it does not register the components either as part of the default scaffolding
create-nuxt-app version: 2.4.3
yarn: 1.13.0
node: 10.15.0 (minor is the lts)
OS: osx
Steps to reproduce:

yarn create nuxt-app test or whatever npx equivalent
when selecting a ui library choose antd
in the default pages/index.vue paste the following:

<v-button type="primary">
  main button
</v-button>

i notice that only the text is rendered and the html tag is v-button instead of the compiled button
would like to understand the underlying issue as the goal is to actually use a different component library from antd

made a mistake apologies
its a-button, although i still dont know why my component library is not working but that's probably my fault.
Apologies


Travis CI: Add Python 3 in allow_failures mode

Master is in Python2, the branch python3 is in Python3

Why not do it the other way around?  Master is Python 3 and then another branch for legacy Python.

The project was build on top of a Python2 library to control the robot car.


feat: workbench flat appearance

Addressed #57
What changed

Sidebar headers
Tabs background
Breadcrumb Background

Idea
The main idea was to make flat mode for the theme, based on the old look before v2.2.0.
What to do next

[ ] The changes still pretty WIP, need to check if any other parts in editor still not flat.
[ ] Need to take aesthetic and UX to consideration.
[x] Need option to choose between default and flat mode.

Screenshot

I merged #65 into main already, so I updated your PR. I'm waiting on this for v2.3.0.

Happy to merge this if you're okay with those suggested changes.
We can happily introduce more minor changes later, I'd rather get this option out there sooner rather than later.
Thanks for getting started on this PR!

Thank you it looks better!


Mouse Position(within Canvas Space)

Translating Canvas(web page) mouse click position into Canvas Space mouse position

This will also include changing the rendering to orthographic perspective


Bumped ccx-rules-ocp version

Codecov Report
Patch and project coverage have no change.

Comparison is base (c884741) 73.46% compared to head (3890668) 73.46%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #455   +/-   ##
=======================================
  Coverage   73.46%   73.46%
=======================================
  Files          11       11
  Lines         603      603
=======================================
  Hits          443      443
  Misses        135      135
  Partials       25       25

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

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


add certificates

Adds additional capability to add a configmap file containing a PEM certificate bundle.

Mounts certificate to /etc/ssl/certs in both the scanner and server deployments
Adds configurable settings to the values file
Cleans up volumes and volumemounts, wrapping them in an if block

@semyonmor updated the PR against 6.5.  Thanks!

@ebram-va - the implementation is very functional, rather than having manual steps to create a configmap with certs and mounting, we can add them from secrets conditionally maybe we can use a range to load multiple custom certs to /etc/ssl/certs/
by so adding simple instruction to readme will be easy.

@KoppulaRajender sure, I see how you've implemented it below, sorry I missed that earlier, just used to doing it as configmaps.  I'll try to get to that today or tomorrow, test it out on our dev instance and then update the PR!

@KoppulaRajender I am running this live now in our env.  The logic ended up making more sense just doing 1 key per secret instead of 1 secret with multiple keys.  Let me know if that's OK!

Hi @ebram-va
Our apologies for the delay with your PR
Can you please rebase you branch to solve conflicts and we ill try to merge you PR tomorrow?

No problem, I did a rebase so let me know!


HBASE-26351 Implement a table-based region grouping strategy for Re…

Implement a table-based region grouping strategy for RegionGroupingProvider

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
9m  9s
Docker mode activated.


docs: removes unnecessary information from API docs

Fixes

Fixes #3208 by @AetherUnbound
Description

Just removed the cURL instructions from every endpoint:

Testing Instructions

just api/up

Checklist

[x] My pull request has a descriptive title (not a vague title likeUpdate index.md).
[x] My pull request targets the default branch of the repository (main) or a parent feature branch.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[x] I added or updated tests for the changes I made (if applicable).
[x] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no visible errors.
[x] I ran the DAG documentation generator (if applicable).

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

Thanks for the contribution @bermr. I approved before seeing Olga's comment, so I'll wait for Dhruv as well. @dhruvkb if you approve go ahead and merge :slightly_smiling_face:


Add Image Styling

@ConnorLand

Could you also apply this change to the Mx9 docs?

Could you also apply this change to the Mx9 docs?

Yes I will.