Tooltips throw console error when closing/reloading the page

With the latest versions the tooltips finally disappear properly, thanks for that! Unfortunately now the console prints an error every time the page is unloaded. Note that this only happens when the session is completely disposed, e.g. when reloading the page or closing the tab. It produces quite a lot of console spam.
To Reproduce

Open a page with a tooltip
Press ctrl+r or close the browser tab
The following error is logged

warn: Microsoft.AspNetCore.Components.Server.Circuits.RemoteRenderer[100]
      Unhandled exception rendering component: JavaScript interop calls cannot be issued at this time. This is because the circuit has disconnected and is being disposed.
      Microsoft.JSInterop.JSDisconnectedException: JavaScript interop calls cannot be issued at this time. This is because the circuit has disconnected and is being disposed.
         at Microsoft.AspNetCore.Components.Server.Circuits.RemoteJSRuntime.BeginInvokeJS(Int64 asyncHandle, String identifier, String argsJson, JSCallResultType resultType, Int64 targetInstanceId)
         at Microsoft.JSInterop.JSRuntime.InvokeAsync[TValue](Int64 targetInstanceId, String identifier, CancellationToken cancellationToken, Object[] args)
         at Microsoft.JSInterop.JSRuntime.InvokeAsync[TValue](Int64 targetInstanceId, String identifier, Object[] args)
         at Microsoft.JSInterop.JSRuntimeExtensions.InvokeVoidAsync(IJSRuntime jsRuntime, String identifier, Object[] args)
         at BlazorBootstrap.Tooltip.DisposeAsync(Boolean disposing)
         at BlazorBootstrap.BlazorBootstrapComponentBase.DisposeAsync()
         at Microsoft.AspNetCore.Components.RenderTree.Renderer.<>c__DisplayClass74_0.<<Dispose>g__HandleAsyncExceptions|1>d.MoveNext()
fail: Microsoft.AspNetCore.Components.Server.Circuits.CircuitHost[111]
      Unhandled exception in circuit 'CDFABjckZDgYPYT3GDGbO7fishAa_8DFTTIo0P7caM8'.
      Microsoft.JSInterop.JSDisconnectedException: JavaScript interop calls cannot be issued at this time. This is because the circuit has disconnected and is being disposed.
         at Microsoft.AspNetCore.Components.Server.Circuits.RemoteJSRuntime.BeginInvokeJS(Int64 asyncHandle, String identifier, String argsJson, JSCallResultType resultType, Int64 targetInstanceId)
         at Microsoft.JSInterop.JSRuntime.InvokeAsync[TValue](Int64 targetInstanceId, String identifier, CancellationToken cancellationToken, Object[] args)
         at Microsoft.JSInterop.JSRuntime.InvokeAsync[TValue](Int64 targetInstanceId, String identifier, Object[] args)
         at Microsoft.JSInterop.JSRuntimeExtensions.InvokeVoidAsync(IJSRuntime jsRuntime, String identifier, Object[] args)
         at BlazorBootstrap.Tooltip.DisposeAsync(Boolean disposing)
         at BlazorBootstrap.BlazorBootstrapComponentBase.DisposeAsync()
         at Microsoft.AspNetCore.Components.RenderTree.Renderer.<>c__DisplayClass74_0.<<Dispose>g__HandleAsyncExceptions|1>d.MoveNext()

Versions (please complete the following information):

BlazorBootstrap: 1.10.2
Blazor Server
NET 7

Sample code
<Tooltip Title="Tooltip">
    Text
</Tooltip>

Desktop (please complete the following information):

OS: Windows
Browser Chrome, Firefox

I think a simple try/catch would do the job, since we can be sure that the tooltips disappear if the entire page is closed, so it's not a problem that the JSInterop call fails.

I started experiencing the same error after updating to 1.10.2.
I use Modal, ToastMessages, and Tooltips. I removed Modal and ToastMessages and continued to see the error.
Best I can tell is that 52 of the errors are tooltip related. Maybe the other 2 is a modal and the setup for toast.
I reverted back to 1.10.1 and it no longer happens.


Pelican API timeout when CBOM trying to get information about the dataset

Chile compra record (pelican_id = 1057) contains more than 1.6M compiled releases and it looks like it takes too much time to get the results
dqv_django_1 log
[2021-07-22 07:38:54 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:17)
[2021-07-22 07:38:55 +0000] [7] [WARNING] Worker with pid 17 was terminated due to signal 9
[2021-07-22 07:38:55 +0000] [176] [INFO] Booting worker with pid: 176

data-registry_cbom_1 log
[22/Jul/2021 07:47:38] DEBUG [cbom:20] Processing collection Chile - Compra records (10)
[22/Jul/2021 07:48:09] ERROR [cbom:115] Unable get dataset availability from pelican
Traceback (most recent call last):
  File "/usr/local/lib/python3.8/site-packages/urllib3/connectionpool.py", line 699, in urlopen
    httplib_response = self._make_request(
  File "/usr/local/lib/python3.8/site-packages/urllib3/connectionpool.py", line 445, in _make_request
    six.raise_from(e, None)
  File "<string>", line 3, in raise_from
  File "/usr/local/lib/python3.8/site-packages/urllib3/connectionpool.py", line 440, in _make_request
    httplib_response = conn.getresponse()
  File "/usr/local/lib/python3.8/site-packages/sentry_sdk/integrations/stdlib.py", line 102, in getresponse
    rv = real_getresponse(self, *args, **kwargs)
  File "/usr/local/lib/python3.8/http/client.py", line 1348, in getresponse
    response.begin()
  File "/usr/local/lib/python3.8/http/client.py", line 316, in begin
    version, status, reason = self._read_status()
  File "/usr/local/lib/python3.8/http/client.py", line 285, in _read_status
    raise RemoteDisconnected("Remote end closed connection without"
http.client.RemoteDisconnected: Remote end closed connection without response

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.8/site-packages/requests/adapters.py", line 439, in send
    resp = conn.urlopen(
  File "/usr/local/lib/python3.8/site-packages/urllib3/connectionpool.py", line 755, in urlopen
    retries = retries.increment(
  File "/usr/local/lib/python3.8/site-packages/urllib3/util/retry.py", line 532, in increment
    raise six.reraise(type(error), error, _stacktrace)
  File "/usr/local/lib/python3.8/site-packages/urllib3/packages/six.py", line 769, in reraise
    raise value.with_traceback(tb)
  File "/usr/local/lib/python3.8/site-packages/urllib3/connectionpool.py", line 699, in urlopen
    httplib_response = self._make_request(
  File "/usr/local/lib/python3.8/site-packages/urllib3/connectionpool.py", line 445, in _make_request
    six.raise_from(e, None)
  File "<string>", line 3, in raise_from
  File "/usr/local/lib/python3.8/site-packages/urllib3/connectionpool.py", line 440, in _make_request
    httplib_response = conn.getresponse()
  File "/usr/local/lib/python3.8/site-packages/sentry_sdk/integrations/stdlib.py", line 102, in getresponse
    rv = real_getresponse(self, *args, **kwargs)
  File "/usr/local/lib/python3.8/http/client.py", line 1348, in getresponse
    response.begin()
  File "/usr/local/lib/python3.8/http/client.py", line 316, in begin
    version, status, reason = self._read_status()
  File "/usr/local/lib/python3.8/http/client.py", line 285, in _read_status
    raise RemoteDisconnected("Remote end closed connection without"
urllib3.exceptions.ProtocolError: ('Connection aborted.', RemoteDisconnected('Remote end closed connection without response'))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/src/app/data_registry/cbom/utils.py", line 11, in request
    resp = requests.request(method, url, **kwargs)
  File "/usr/local/lib/python3.8/site-packages/requests/api.py", line 61, in request
    return session.request(method=method, url=url, **kwargs)
  File "/usr/local/lib/python3.8/site-packages/requests/sessions.py", line 542, in request
    resp = self.send(prep, **send_kwargs)
  File "/usr/local/lib/python3.8/site-packages/requests/sessions.py", line 655, in send
    r = adapter.send(request, **kwargs)
  File "/usr/local/lib/python3.8/site-packages/requests/adapters.py", line 498, in send
    raise ConnectionError(err, request=request)
requests.exceptions.ConnectionError: ('Connection aborted.', RemoteDisconnected('Remote end closed connection without response'))

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/src/app/data_registry/cbom/process.py", line 180, in update_collection_availability
    resp = request(
  File "/usr/src/app/data_registry/cbom/utils.py", line 15, in request
    raise RecoverableException(error_msg) from e
data_registry.cbom.task.exceptions.RecoverableException: Request on http://localhost:8001/api/dataset_availability/1057 failed

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/src/app/data_registry/cbom/process.py", line 111, in process
    update_collection_availability(job)
  File "/usr/src/app/data_registry/cbom/process.py", line 185, in update_collection_availability
    raise Exception("Unable get dataset availability from pelican") from e
Exception: Unable get dataset availability from pelican

Increased processing timeout.


change remark for FlushAsync method.

Change looks good.
Once you sign the CLA, you are good to go, I think. cc @Eilon

#1721

@kirthik Okey, I did sign the CLA. I hope it's good now and you can accept my pull requests in the future.

CLA is confirmed.

Thanks for fixing this one @stanislavromanov merged : https://github.com/aspnet/Mvc/commit/990473aebc17505a08f59cd85142b30642bc2a54


authentication error

I've applyed the fix for emire web and if I ran just ./empire it works like a charm but when I supply the user/pass/port parameters I got bunch of errors.
`
./empire --headless --restport 1337 --username admin --password temp1234
Starting a full Empire instance
[*] Loading modules from: /opt/Empire-master/lib/modules/
Starting Empire RESTful API on port: 1337
RESTful API token: gjibeplwpkwkl4auzcj5452kk6vibj0rpkctrm3i
Traceback (most recent call last):
File "./empire", line 1234, in
start_restful_api(startEmpire=True, suppress=True, username=args.username, password=args.password, port=args.restport)
File "./empire", line 1081, in start_restful_api
if not request.json or not 'username' in request.json or not 'password' in request.json:
File "/usr/local/lib/python2.7/dist-packages/werkzeug/local.py", line 343, in getattr
return getattr(self._get_current_object(), name)
File "/usr/local/lib/python2.7/dist-packages/werkzeug/local.py", line 302, in _get_current_object
return self.__local()
File "/usr/local/lib/python2.7/dist-packages/flask/globals.py", line 37, in _lookup_req_object
raise RuntimeError(_request_ctx_err_msg)
RuntimeError: Working outside of request context.

This typically means that you attempted to use functionality that needed
an active HTTP request. Consult the documentation on testing for
information about how to avoid this problem.`

The Empire rest API is still a long ways off from being ready for use. I would almost consider it still in alpha. Unfortunately, this is towards the bottom of the list when it comes to priority and won't be addressed for quite some time.

@tehseensagar We will probably end up addressing this when revamping the REST api towards the tail end of the year. Sorry for the long wait.


Added InitializeProfile methods to ProfileClient

Moved DataStore to AzureSession, added documentation, and updated ListAccount logic

@azuresdkci retest this please


canDrop, hover and drop properties cannot be updated

Describe the bug
useDropHandler memo is only updated if monitor changes, not if spec reference change
in this case, if we update the drop function, the old one will be executed, not the new one.
export function useDropHandler(spec, monitor, connector) {
    const manager = useDragDropManager();
    const handler = useMemo(() => {
        return {
            canDrop() {
                const { canDrop } = spec.current;
                return canDrop ? canDrop(monitor.getItem(), monitor) : true;
            },
            hover() {
                const { hover } = spec.current;
                if (hover) {
                    hover(monitor.getItem(), monitor);
                }
            },
            drop() {
                const { drop } = spec.current;
                if (drop) {
                    return drop(monitor.getItem(), monitor);
                }
            },
        };
    }, [monitor]);
    useIsomorphicLayoutEffect(function registerHandler() {
        const [handlerId, unregister] = registerTarget(spec.current.accept, handler, manager);
        monitor.receiveHandlerId(handlerId);
        connector.receiveHandlerId(handlerId);
        return unregister;
    }, [monitor, connector]);
}

Expected behavior
The latest spec need to be execed

Hey @paztis, I believe the latest version of react-dnd solves this. Let me know if that works out for you!

Thanks for the fix here! I had this same issue and upgrading to 14.0.3 helped. A note however for any future searchers. As well as updating you will need to pass in the dependencies array just like you would with useMemo or useCallback

const onDrop = () => // this function can change dynamically

const d = useDrop(
() => ({
	accept: ["type1", "type2"]
	drop: onDrop,
	collect: (monitor) => ({
		isOver: monitor.isOver({ shallow: true }),
		isDragging: monitor.canDrop(),
		item: monitor.getItem(),
	}),
}),
[onDrop] // pass my onDrop function as dependency so callback is updated if this fn changes
);


Add upstream tests

As discussed in https://github.com/prql/dbt-prql/pull/5

If we add tests which test against dbt-core/main, we catch upstream breaks early, then we can unpin minor versions too...

Partially closed by https://github.com/prql/dbt-prql/pull/11, still need to add the "upstream" part


Add missing required Tech methods

While investigating #30 and #32, I found that there are several functions that Video.js requires a custom Tech to implement that are not documented.
I attempted to make a comprehensive list of required methods and submitted it to Video.js here: https://github.com/videojs/video.js/issues/5737
Additionally, it was unclear what values should be returned for the seekable and buffered functions because that information is not available via the Chromecast API. I asked Video.js what they thought those functions should return, and their response is documented here: https://github.com/videojs/video.js/issues/5742
This issue is a request to add the missing Tech functions to the ChromecastTech.
You can see a demonstration of the error that's thrown when a player function that delegates to a missing Tech function is called here: https://codepen.io/mluedke/pen/zyaVjX

https://github.com/silvermine/videojs-chromecast/pull/43


Bump Bitcoin Guide to 28.0

What
What is the reason of this change?
Update the guide to more accurately follow the process of setting up the latest Bitcoin client
Why
Why is this change important?
Simplicity for beginners to follow along.
Scope

[x] significant change to core configuration
[ ] independent bonus guide
[ ] simple bug fix

We treat Bitcoin Core upgrades as "significant change to core configuration" not "simple bug fix". Upgrading, especially between major versions, may cause various incompatibilities with other software, so needs careful testing.

To allow LND 0.18.3 to work, add the following to bitcoin.conf
Allow LND 18.3 to work
deprecatedrpc=warnings

I will implement a Raspibolt in a couple of days, so I will be able to try this commit with the new core v28. Not going to implement LN, though. Do you suggest anything else to test?

Anything that depends on Bitcoin Core can be affected - Electrum server, Blockchain explorer, JoinMarket, etc. The more things are tested and feedback is given, the better.

if i had seen this PR before... I did not follow the entirely tutorial, and gosh, some minors are really annoying, like this

To allow LND 0.18.3 to work, add the following to bitcoin.conf
deprecatedrpc=warnings

BUT nothing a big deal.
And I installed Alby Hub to manage my channel btw.


Spevacus: Watch medicalnewstoday.com

Spevacus requests the watch of the watch_keyword medicalnewstoday\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

medicalnewstoday\.com has been seen in 1 true positive, 5 false positives, and 0 NAAs.

Whoops, didn't realize this is #notspam - This can be closed.


sentry: Ignore SuspiciousOperation exceptions themselves.

596cf2580b ignored the loggers of all SuspiciousOperation subclasses,
but not SuspiciousOperation itself.  Almost all locations raise one of
the more specific subclasses, with the exception of one location in
the session middleware[1].
Ignore the overall django.security.SuspiciousOperation logger as well.
[1] https://code.djangoproject.com/ticket/31962

Nice, merged and thanks for reporting that bug upstream :+1:


cmd/go: should include packages without tests in cover profile file

What version of Go are you using (go version)?
go version go1.11 linux/amd64
Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOOS="linux"

What did you do?
Created a new project with two sub-packages:
.
├── notests
│   └── notests.go
└── tests
    ├── tests.go
    └── tests_test.go

2 directories, 3 files

notests/notests.go
package notests

import "fmt"

func untested() {
	fmt.Print("untested")
}

tests/tests.go
package tests

import "fmt"

func tested() {
	fmt.Print("tested")
}

tests/tests_test.go
package tests

import "testing"

func TestTested(t *testing.T) {
	tested()
}

Ran go test ./... -v -cover -coverprofile cover.out
What did you expect to see?
$ cat cover.out
mode: set
empty-cover-profile/notests/notests.go:7.17,9.2 1 0
empty-cover-profile/tests/tests.go:7.15,9.2 1 1

Coverage details for both files.
https://github.com/golang/go/issues/18909 requested this functionality, but it wasn't implemented in https://golang.org/cl/76875.
What did you see instead?
$ cat cover.out
mode: set
empty-cover-profile/tests/tests.go:7.15,9.2 1 1

Coverage details only for the file with a test in the same directory.
Workaround
echo "package notests" > notests/notests_test.go

Creating an empty test file creates coverage for both files.

Dupe of #24570


INA240: which gain should be selected for V2.0.2

there 4 different gains.  which gain should be selected?
INA240A1 GAIN(V/V) 20
INA240A2 GAIN(V/V) 50
INA240A3 GAIN(V/V) 100
INA240A4 GAIN(V/V) 200

Hey @gowitty,
So in short, I've taken a choice to use:

Shunt resistor value: 0.01Ohm
Current sense: INA240A2 (gain 50)

Here is why:
So in general there are two parameters when it comes to current sensing hardware:

shunt resistor values
amplifier gain

In the case of the SimpleFOCShield, the BLDC driver chip L6234 has the current limit of 5 Amps. So we need to be able to measure +-5amp current. But in most cases your application with the SimpleFOCShield will remain in the 3Amp range. Higher currents require appropriate heat evacuation: heat-sinks, fans, etc.
So you need to be able to measure +-3amps for example, which is a 6Amp range.  (or +- 5amps -> 10amp range).
The mcus usually have the 3.3V or 5V logic which tells you that you need the transformation ratiofor example:
trans_ratio = 5V / 10amps = 0.5 V/Amp    or    trans_ratio = 3.3V / 10amps = 0.33 V/Amp

So now that we know what is the transformation ratio that we need we can proceed to finding the shunt resistor value and the amplification gain. We know that the trans_ratio = amp_gain*shunt_resistance. Now there is a trade-off here in between amplification gain and shunt value:
Shunt resistance will be placed in-line with the motor and all the phase current of the motor will pass through the shunt resistor so it is important to choose the value that is the smallest possible value to reduce the heat loss. Additionally by adding the shunt resistors in-line with motors we are actually increasing the phase resistance of the motor, and since we are placing shunt resistors only on two phases this can make the motor unbalanced and this is another reason that makes it really important to keep this value small relatively to the phase resistance of the motor. On the other hand, the smaller the shunt resistor value the smaller the voltage across it that we want to measure. Since the voltage across it is smaller it which makes it more difficult to measure the phase current, because the smaller it is the higher the amplification noise.  So we would prefer to have the shunt resistor value high and amplification gain low to reduce the noise.
Since we know that the phase resistances of the BLDC motors to be used with this board will be in the range of >10Ohm we know that we can take the shunt resistor of >0.01Ohm.
Now we can calculate that for shunt resistor of 0.01 Ohm we would need the amplificaiton gain:
amp_gain= 0.5/0.01 = 50     or     amp_gain =  3.3/0.01 = 33

So in order to measure the +-5Amps on 5V logic we can use INA240A2 (gain 50)  but on the 3.3V logic that will give us +-3.3Amp range.
If we want to have higer range on 3.3V logic we need to either:

use INA240A1 (gain 20) which will give us +-8.25Amp range on 3.3V and +-12.5Amp range on 5V.
use INA250A2 (gain 50) and change the shunt value (ex. 6 miliOhm) which would give use +-5.5Amps range on 3.3V and +-8.5Amps range on 5V

So I've decided to stick with the 0.01Ohm shunt and INA240A2 because even +-3.3Amp range is perfectly enough for gimbal motors.

Hey @askuric
appreciate sharing so many details!


Update Modals

@m4rkwallace-SL - can you put something in here (to discuss in refinement)?

Closing this until we have more specific requirements


ProjectionElement stores query layout by record ID which does not roundtrip through export/import

When adding a projection element to a layout the user can choose a query layout like so:

This choice gets persisted in the JSON data of the element as the database record ID of the query layout, like so:
QueryLayoutId=85

Exporting this element to a recipe, the record ID 85 gets embedded in the JSON in the recipe. When importing to a new site (such as when using the recipe as a setup recipe) the query layout will most likely not have the same record ID, which causes the projection element to fall back to the default query layout.
The projection element needs to store the query layout reference by its index which, unlike the DB record ID, will persist across the export/import cycle.

Somewhat related issue:
https://github.com/OrchardCMS/Orchard/issues/4440
That one suggests adding a Name property to layouts. If we do that, obviously that would be a better identifier than the index.

Although QueryLayoutId is indeed exported, as is the identifier of the query and the index of the layout. So while the former one is useless it does not do harm and there is still the necessary information to restore the state. I don't think this is an existing issue (at least in latest 1.9.x).

OK, good. Then it should just be a matter of fixing this in the import code?

It is already. #4440 is a valid issue but if you just export/import Layouts and Queries it will work as expected, at least in my experience (I also fixed some related glitches in 1.9.x because currently I have to do a lot of such export/import).

I just tested it on 1.9.x and it works as expected. Looking at the code, I see that the LayoutIndex is exported as well as imported.
@DanielStolt do you have any repro steps?

I've just come across this bug (or something very similar to it) in 1.9.1 - I've exported a projection and it's query (that only has one custom layout on it) to create a recipe, but when running the recipe, the default query layout is set, not the custom one. Are there any additional steps I need to take to make it work?

It should be fixed in 1.9.2.

OK, thanks!


⚠ ESPN has degraded performance

In 563178a, ESPN (https://espn.com) experienced degraded performance:

HTTP code: 200
Response time: 712 ms

Resolved: ESPN performance has improved in dcb1ab5.


regard SHOW as sql command for mysql, postgres protocol

What does this PR do?
network ebpf mysql, postgres protocol regard show as sql command
Motivation
Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Hey @chengjoey 👋🏻
First of all, thank you for submitting this change, we appreciate the effort!
However, we intentionally decided to keep the list of commands limited as our overall code is pretty complex, which gives us very small room for modifications, as a more complicated code can lead us to reach to the eBPF verifier limitations (on number of branches in the code or number of instructions). We chose the most common commands to be added to the list, and unfortunately, show wasn't one of them. Furthermore, the code lives in our classification code, and for that, it is suffice for us to find a single matched command in the connection to classify it as postgres or as mysql, so the impact of unsupported commands (like show, begin, commit, etc) is narrowed down.


Back Button doesn't work

In Gear VR when i press back button it doesn't work or go back, Is there any specific code for that?

When you say " it doesn't work or go back" - what is it that you expect to happen when you hit back?
The gvr-events sample for example intercepts the back button, leaving it to the developer to handle the action on button press.
For most of the other samples hitting back should take you to the oculus home screen.

If you want you own custom action on back button press, you could look at the dispatchKeyEvent call on the activity:
dispatchKeyEvent
You must also make sure that you call super() for all unhandled events.

I followed the gvr-events example to go for back, Is there any predefined way to record our flow of process or we need to give the way of the process.

Usually on pressing back the exit-to-Oculus-Home prompt is brought up but you could change that in GVRActivity.


Add link to "export to CSV" from batch_list page

We have very thorough tools for importing data from spreadsheets into the We Vote database. We would like to add the ability to export that same data into a standard csv format.

Go to Admin Home > Import Data in Batches > Import Candidates
http://localhost:8001/import_export_batches/batch_list/?kind_of_batch=CANDIDATE
You may need to import some data to start with. Here is sample data: https://docs.google.com/spreadsheets/d/1RXxlUDiW66CvTc4ALbELQFpN-A_w3PVzj38pbNqGsLg/edit#gid=0

We would like to add "export" link next to "review" link:

It should export with the mapped header values. This view might be a good place to start researching: batch_list_process_view
The template file is here: WeVoteServer/templates/import_export_batches/batch_list.html
The view file to update: WeVoteServer/import_export_batches/views_admin.py
A new path will be needed in this file: WeVoteServer/import_export_batches/urls.py

Pull request #672 has been created for this issue.

Great work @marodrig!


import Error dlib.cpython-36m-x86_64-linux-gnu.so: undefined symbol: mkl_dft_dz2_c_dft

Expected Behavior
Should be able to import dlib using python 3.6.8 on a virtualenv installed via pip
Current Behavior
when running import dlib on a python shell it returns the following error:
ImportError: /home/gabhen/venv/lib/python3.6/site-packages/dlib.cpython-36m-x86_64-linux-gnu.so: undefined symbol: mkl_dft_dz2_c_dft
Steps to Reproduce
import dlib on python3.6.8 on a virtualenv shell.

Version: 19.17.0
Where did you get dlib: pip
Platform: 64bit Ubuntu 18.04.2
Compiler: cmake version 3.10.2

Any ideas what might be causing this error ? Already installed Intel MKL and reinstalled dlib, no luck yet.
Thanks for the help!

What, exactly, does someone type to reproduce this?

On Jul 4, 2019, at 11:09 AM, Gabriel Henriques<EMAIL_ADDRESS>wrote:
Expected Behavior
Should be able to import dlib using python 3.6.8 on a virtualenv installed via pip
Current Behavior
when running import dlib on a python shell it returns the following error:
ImportError: /home/gabhen/venv/lib/python3.6/site-packages/dlib.cpython-36m-x86_64-linux-gnu.so: undefined symbol: mkl_dft_dz2_c_dft
Steps to Reproduce
import dlib on python3.6.8 on a virtualenv shell.
Version: 19.17.0
Where did you get dlib: pip
Platform: 64bit Ubuntu 18.04.2
Compiler: cmake version 3.10.2
Any ideas what might be causing this error ? Already installed Intel MKL and reinstalled dlib, no luck yet.
Thanks for the help!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

What, exactly, does someone type to reproduce this?
…
On Jul 4, 2019, at 11:09 AM, Gabriel Henriques @.***> wrote: Expected Behavior Should be able to import dlib using python 3.6.8 on a virtualenv installed via pip Current Behavior when running import dlib on a python shell it returns the following error: ImportError: /home/gabhen/venv/lib/python3.6/site-packages/dlib.cpython-36m-x86_64-linux-gnu.so: undefined symbol: mkl_dft_dz2_c_dft Steps to Reproduce import dlib on python3.6.8 on a virtualenv shell. Version: 19.17.0 Where did you get dlib: pip Platform: 64bit Ubuntu 18.04.2 Compiler: cmake version 3.10.2 Any ideas what might be causing this error ? Already installed Intel MKL and reinstalled dlib, no luck yet. Thanks for the Help!

import dlib

That doesn’t reproduce it.

On Jul 4, 2019, at 11:32 AM, Gabriel Henriques<EMAIL_ADDRESS>wrote:
What, exactly, does someone type to reproduce this?
…
On Jul 4, 2019, at 11:09 AM, Gabriel Henriques @.***> wrote: Expected Behavior Should be able to import dlib using python 3.6.8 on a virtualenv installed via pip Current Behavior when running import dlib on a python shell it returns the following error: ImportError: /home/gabhen/venv/lib/python3.6/site-packages/dlib.cpython-36m-x86_64-linux-gnu.so: undefined symbol: mkl_dft_dz2_c_dft Steps to Reproduce import dlib on python3.6.8 on a virtualenv shell. Version: 19.17.0 Where did you get dlib: pip Platform: 64bit Ubuntu 18.04.2 Compiler: cmake version 3.10.2 Any ideas what might be causing this error ? Already installed Intel MKL and reinstalled dlib, no luck yet. Thanks for the Help!
import dlib
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

That doesn’t reproduce it.
…
On Jul 4, 2019, at 11:32 AM, Gabriel Henriques @.> wrote: What, exactly, does someone type to reproduce this? … On Jul 4, 2019, at 11:09 AM, Gabriel Henriques @.> wrote: Expected Behavior Should be able to import dlib using python 3.6.8 on a virtualenv installed via pip Current Behavior when running import dlib on a python shell it returns the following error: ImportError: /home/gabhen/venv/lib/python3.6/site-packages/dlib.cpython-36m-x86_64-linux-gnu.so: undefined symbol: mkl_dft_dz2_c_dft Steps to Reproduce import dlib on python3.6.8 on a virtualenv shell. Version: 19.17.0 Where did you get dlib: pip Platform: 64bit Ubuntu 18.04.2 Compiler: cmake version 3.10.2 Any ideas what might be causing this error ? Already installed Intel MKL and reinstalled dlib, no luck yet. Thanks for the Help! import dlib

Well it's definitly an environment issue given that it works fine on my MacOs, just haven't been able to figure out exactly what is causing this issue


Ordering is broken (syntax error)

It seems that specifying a sort order is broken somehow, all I get is a response mentioning a syntax error.
I tried this query: https://postgrest.herokuapp.com/speakers?order=name.asc
And get this result (in Postman 1.0.3 on OS X):
{
  "status": "Fatal error",
  "hint": null,
  "code": "42601",
  "message": "syntax error at or near \")"
}

I started the server with postgrest-<IP_ADDRESS> -d pgapitest -U myuser -a myuser.

Oops, I see what you mean. My test cases only cover ordering when it's accompanied with another filter condition, but looks like it doesn't work right all by itself. https://github.com/begriffs/postgrest/blob/master/test/Feature/QuerySpec.hs#L26-L40
I'll fix this today, thanks for the bug report.

Thanks for fixing this so quickly! I hoped I could learn a tiny bit of Haskell by looking at the fix, but oh no, I have no idea why it works now … I guess I'll have to be patient and study a bit more until I see it :smile:

Yeah that's one drawback of picking Haskell for the project. It's kind of esoteric and makes it harder for people to jump in and contribute.
The bug was actually kind of a dumb mistake due to changing code. It's in a function that transforms a query to add a where clause if params are present. However the order param is special and does not affect the where clause. The transforming function should be (and now is) referring to the cols variable [1] which is basically all params except the order by one.

I find the use of Haskell in this kind of project exciting: because it's a binary I can start using it without any barrier of entry, but then I still hope to one day hack a bit on it if it proves useful in our daily work – it basically gives me access to a Haskell code base that works in production at a time when I could not write something like this by myself at all.
Picking up my copy of LYAHFGG now and will keep a reference to this bugfix for when I know more about it :)


Lighting-app-example crashes

Problem
I tried to run chip-lighting-app on raspberry pi4 but the application crashes:
$ ./chip-lighting-app --wifi
CHIP:DL: Device Configuration:
CHIP:DL:   Serial Number: TEST_SN
CHIP:DL:   Vendor Id: 9050 (0x235A)
CHIP:DL:   Product Id: 65279 (0xFEFF)
CHIP:DL:   Product Revision: 1
CHIP:DL:   Setup Pin Code: 20202021
CHIP:DL:   Setup Discriminator: 3840 (0xF00)
CHIP:DL:   Manufacturing Date: (not set)
CHIP:DL:   Device Type: 257 (0x101)
CHIP:SVR: SetupQRCode: [CH:YNJV793700A50324800]
CHIP:SVR: Copy/paste the below URL in a browser to see the QR Code:
CHIP:SVR: https://dhrishi.github.io/connectedhomeip/qrcode.html?data=CH%3AYNJV793700A50324800
CHIP:SVR: Manual pairing code: [34970112332]
CHIP:ZCL: Using ZAP configuration...
CHIP:ZCL: deactivate report event
CHIP:ZCL: Unknown cluster ID: 40
CHIP:ZCL: Unknown cluster ID: 40
CHIP:ZCL: Unknown cluster ID: 40
CHIP:ZCL: Unknown cluster ID: 40
CHIP:ZCL: Unknown cluster ID: 40
CHIP:ZCL: Unknown cluster ID: 40
CHIP:ZCL: Unknown cluster ID: 40
CHIP:ZCL: Unknown cluster ID: 40
CHIP:ZCL: Unknown cluster ID: 40
CHIP:DIS: Init admin pairing table with server storage.
CHIP:IN: TransportMgr initialized
CHIP:IN: local node id is 0x0000000000bc5c01
CHIP:DL: wpa_supplicant: _IsWiFiStationProvisioned: interface not connected
CHIP:BLE: Assigned local session key ID 0
CHIP:BLE: Waiting for PBKDF param request
CHIP:DIS: Start dns-sd server
CHIP:DIS: CHIP minimal mDNS started advertising.
CHIP:DIS: Replying to DNS-SD service listing request
CHIP:DIS: Replying to DNS-SD service listing request
CHIP:DIS: Replying to DNS-SD service listing request
CHIP:DIS: Replying to DNS-SD service listing request
CHIP:DIS: Replying to DNS-SD service listing request
CHIP:DIS: Replying to DNS-SD service listing request
CHIP:DIS: Replying to DNS-SD service listing request
Segmentation fault (core dumped)
Looks like wpa_supplicant reports wpa_supplicant: _IsWiFiStationProvisioned: interface not connected.
wlan0 is connected to wifi and got an IPv4

Dup #6643


RPC call failed: Could not resolve host

Hi, I'm trying to run uNOMP + MPOS pool. I got my coin daemons (Bitcoin and Litecoin) run on the same machine as MPOS. Username, password and port match in daemon config file and MPOS config. Here's RPC configuration:
$config['wallet']['type'] = 'http';
$config['wallet']['host'] = 'localhost:19332';

WIth this config I get 'Could not resolve host' error.
<IP_ADDRESS>:19332 doesn't work too.
I tried to run MPOS on another machine. I put rpcallowip with proper ip address in coin config file. In MPOS config I used ip and domain name. Still couldn't resolve host.
Bitcoin and Litecoin showed same results.
Litecoin daemon responds to this command correctly:
curl --user <username>:<password> --data-binary '{"jsonrpc": "1.0", "id":"curltest", "method": "getinfo", "params": [] }' -H 'content-type: text/plain;' http://localhost:19332/

Strange problem but not something that I'd see fixable in MPOS. Either your setup has Ann issue with PHP doing any kind of lookups or there is a hidden character in your connection string causing this.
Closing this since it's not a bug in MPOS itself but likely a setup issue on your end.


Large number of updates can overload client

When there's a large list of updates to process the client tries to do them all in a huge batch
The batches should be broken up by each call to list_folder or similar

Resolved in d96b2c50025d3c63716e1eddb54e0cfeb6f6a69f


⚠ Bofrost.at has degraded performance

In 730852f, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1029 ms

Resolved: Bofrost.at performance has improved in 82ac3b1 after 14 minutes.


significance: ascii listing of time variable when values are large:

Reported by ansley b manke on 21 Nov 2012 00:21 UTC
Steve reports:
When I do an LAS V7 Save as to CSV from http://ferret.pmel.noaa.gov/geoideLAS/getUI.do?dsid=MODIS_AQUA_CHLOR&varid=MODIS_AQUA_CHLOR-chl_a I get something that looks like this:

    Total Number of Records           5325.
                 DATA SET: http://cwcgom.aoml.noaa.gov/thredds/dodsC/MODISAQUACHLOR/CHLOR_A.nc
                 NETCDF/20121118_TERRA
                 X: 0.5 to 5325.5
     Column  1: DATETIME is Time String    BAD FLAG : -1.E+34
     Column  2: TIME is Time Coordinates (seconds since 01-JAN-1970 00:00:00)   BAD FLAG : -1.E+34
     Column  3: LON is Longitude (degrees_east)   BAD FLAG : -1.E+34
     Column  4: LAT is Latitude (degrees_north)   BAD FLAG : -1.E+34
     Column  5: CHL_A is concentration_of_chlorophyll_in_sea_water (kg m-3)   BAD FLAG : -1.E+34
    DATETIME,TIME,LON,LAT,CHL_A
    "10-APR-2012",1.334E+09,-69.88,41.54,5.317
    "10-APR-2012",1.334E+09,-69.87,41.54,3.77
    "10-APR-2012",1.334E+09,-69.85,41.54,3.561
    "10-APR-2012",1.334E+09,-69.84,41.54,2.061
    "10-APR-2012",1.334E+09,-69.82,41.54,1.765
    "10-APR-2012",1.334E+09,-69.81,41.54,1.442
    "10-APR-2012",1.334E+09,-69.8,41.54,1.26
    "10-APR-2012",1.334E+09,-69.78,41.54,1.101
    "10-APR-2012",1.334E+09,-69.77,41.54,1.163
    "10-APR-2012",1.334E+09,-69.75,41.54,1.188
    "10-APR-2012",1.334E+09,-69.74,41.54,1.293
    "10-APR-2012",1.334E+09,-69.73,41.54,1.353

The TIME (as a numerical value) is not useful, because there aren't enough significant digits in it.   Maybe double precision Ferret will do better on this;  single precision may be seeing all of the time words as the same value!   The auto-formatting algorithm that Ferret uses for output is supposed to address both the data range, and the data mean in order to create a suitable format.
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/1992

Comment by ansley.b.manke on 29 Nov 2012 00:41 UTC
Ferret doesn't do much auto-formatting when it's listing multiple variables in a record.  Double-precision Ferret would make the same listing.  (Note that the data in the above example are all at the same time: it's daily data and the date is 10-apr-2012.)
This does need some smarter code in Ferret, but as a first improvement so that we are more likely to see variation in the numbers, I have set the LAS Ferret scripts so that they write more digits -- 6 instead of the default of 4. Now a short time-x slice from the same dataset looks like this:
Total Number of Records             40.
             DATA SET: http://cwcgom.aoml.noaa.gov/thredds/dodsC/MODISAQUACHLOR/CHLOR_A.nc
             NETCDF/20121126_TERRA
             X: 0.5 to 40.5
 Column  1: DATETIME is Time String    BAD FLAG : -1.E+34
 Column  2: TIME is Time Coordinates (seconds since 01-JAN-1970 00:00:00)   BAD FLAG : -1.E+34
 Column  3: LON is Longitude (degrees_east)   BAD FLAG : -1.E+34
 Column  4: LAT is Latitude (degrees_north)   BAD FLAG : -1.E+34
 Column  5: CHL_A is concentration_of_chlorophyll_in_sea_water (kg m-3)   BAD FLAG : -1.E+34
DATETIME,TIME,LON,LAT,CHL_A
"10-APR-2012",1.33402E+09,-89.3841,25.1952,-1.E+34
"10-APR-2012",1.33402E+09,-89.3702,25.1952,0.048
"10-APR-2012",1.33402E+09,-89.3563,25.1952,0.053
"10-APR-2012",1.33402E+09,-89.3424,25.1952,0.06
"10-APR-2012",1.33402E+09,-89.3285,25.1952,0.051
"10-APR-2012",1.33402E+09,-89.3147,25.1952,0.032
"10-APR-2012",1.33402E+09,-89.3008,25.1952,0.05
"10-APR-2012",1.33402E+09,-89.2869,25.1952,0.048
"11-APR-2012",1.3341E+09,-89.3841,25.1952,-1.E+34
"11-APR-2012",1.3341E+09,-89.3702,25.1952,-1.E+34
"11-APR-2012",1.3341E+09,-89.3563,25.1952,-1.E+34
"11-APR-2012",1.3341E+09,-89.3424,25.1952,-1.E+34
"11-APR-2012",1.3341E+09,-89.3285,25.1952,-1.E+34
"11-APR-2012",1.3341E+09,-89.3147,25.1952,-1.E+34
"11-APR-2012",1.3341E+09,-89.3008,25.1952,-1.E+34
"11-APR-2012",1.3341E+09,-89.2869,25.1952,-1.E+34
"12-APR-2012",1.33419E+09,-89.3841,25.1952,-1.E+34
"12-APR-2012",1.33419E+09,-89.3702,25.1952,-1.E+34
"12-APR-2012",1.33419E+09,-89.3563,25.1952,-1.E+34


Add the GraphQL endpoint support

Crowdin API client doesn't support the GraphQL endpoint yet. We should add support for it in this API client.
References:

GraphQL API documentation
JS API client implementation example

Hi @andrii-bodnar! i would like to contribute to this issue, can you assign me? Thanks!

@andrii-bodnar, I see two possible approaches for implementing GraphQL support:

Adding it directly to the CrowdinClient class (for example in client.py and requester.py)
Creating a new module under api_resources following the project's pattern (for example:
crowdin_api/api_resources/graphql/
├── init.py
├── resource.py
├── types.py
├── tests/
└── test_graphql_resources.py)

Which approach would you prefer? Thanks!

@imprvhub I like the first approach. Also, it's something similar to what we have in JS API Client

@andrii-bodnar It's done! Let me know if you have any suggestions for corrections.


500 Internal error: Connection reset by peer

Hello,
I am looking for some explanation on these error messages
2016-02-14T16:55:27.025Z HTTP: #002: POST /test-syncgw/
2016-02-14T16:55:27.026Z WARNING: Couldn't interpret error type *net.OpError, value read tcp <IP_ADDRESS>:11210: connection reset by peer -- base.ErrorAsHTTPStatus() at error.go:63
2016-02-14T16:55:27.026Z HTTP: #002: --> 500 Internal error: read tcp <IP_ADDRESS>:11210: connection reset by peer (3.5 ms)
My setup:
Have the sync gateway setup to run as a service on Ubuntu 14.01 LTE, with a shadow bucket. Starting the gateway things work as expected docs get synched everything is fine. But after a long gap of no activity, when I try a HTTP PUT request I see this message in the logs. Restarting the service seems to get back to normal again. Its almost like something is timing out when there is no action.
Any help greatly appreciated.

I moved this to https://github.com/couchbase/sync_gateway/issues/1598
This repo (couchbaselabs/sync_gateway) was a temporary fork that was created for testing something in the official repo (couchbase/sync_gateway), and I'm deleting the temp testing repo.


Topics can't start with the same first word

Hi,
If I create a Topic with name "Server Test" and then try to create another topic with name "Server Documents" the 2nd topic is not created.
Please help in resolving.
Thanks
Dan

Hi, I'm not able to replicate your issue.
When did you install it?
Try doing a 'git pull` or a fresh clone and see if the issue exists.

Meh, tried again today and its working fine. Im not sure what happened there.
Thanks for the help though.
Dan


Introduces a change in the REST POST, PATCH and PUT API response construction

Why make this change?

At the moment, REST POST, PUT and PATCH API requests return all the fields of the entity in the response without taking into consideration the read permissions of the given role.
This PR introduces logic for considering the read configuration setup such as included/excluded fields and database policies and construct the response accordingly.

What is this change?

Response construction logic changes:

When no read action setup - An empty response is returned
When database policy for read is setup with/without include/exclude fields: A select database query which honors the database policy as well as included/excluded fields configuration is executed to retrieve the results
When include/exclude fields are setup without database policy: Any additional fields in the response are removed in DAB layer. In this case, a select query is not performed.

dab-config.*sql.json: The sample config files are updated with the way the CLI generates the config files with the latest changes.

More details to be added

How was this tested?

[x] Integration Tests
[x] Manual Tests

To be updated with scenarios
Sample Request(s)
To Be Added

Discussed offline:
For cases where there is no database policy defined for read operation, we don't execute a subsequent select query to return data, but instead use OUTPUT clause. In such a case, to obey the include/exclude field permissions for the READ operation, we then filter out the unexposed fields from the result.
What we can do instead is that instead we can construct OutputColumns such that they don't include unexposed fields in the first place. This will save us from querying unrequired fields from the database and save on network bandwidth.
If we go ahead with this approach, we would still need to include primary keys/order by keys as they are needed for location header/nextLink generation. However from the response, we can filter out the unexposed PK fields (if any) before returning them to the end user.

Discussed offline:
For cases where there is no database policy defined for read operation, we don't execute a subsequent select query to return data, but instead use OUTPUT clause. In such a case, to obey the include/exclude field permissions for the READ operation, we then filter out the unexposed fields from the result. What we can do instead is that instead we can construct OutputColumns such that they don't include unexposed fields in the first place. This will save us from querying unrequired fields from the database and save on network bandwidth. If we go ahead with this approach, we would still need to include primary keys/order by keys in OutputColumns as they are needed for location header/nextLink generation. However from the response, we can filter out the unexposed PK fields (if any) before returning them to the end user.

This optimization introduces logic changes to the query builder classes of each SQL database type. In my next PR (achieving the select in the same round trip), I can include this optimization.

Thanks for addressing the comments with diligence thus far!

Thanks for handling all the edge cases and the patience while resolving all the comment @severussundar. LGTM!


New version: Google.JapaneseIME version 2.28.4650.0

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[X] Does your manifest conform to the 1.1 schema?

Resolves #56788

Microsoft Reviewers: Open in CodeFlow

@SpecterShell could you look into this one?

x64:
https://redirector.gvt1.com/edgedl/release2/mwbfzwv776vr5mu3j7emky7tqi_2.28.4650.0/GoogleJapaneseInput64-2.28.4650.0.msi
https://dl.google.com/release2/mwbfzwv776vr5mu3j7emky7tqi_2.28.4650.0/GoogleJapaneseInput64-2.28.4650.0.msi
https://www.google.com/dl/release2/mwbfzwv776vr5mu3j7emky7tqi_2.28.4650.0/GoogleJapaneseInput64-2.28.4650.0.msi
x86:
Didn't see...
Invoke-RestMethod -Uri 'https://update.googleapis.com/service/update2' -Method Post -Body @"
<?xml version="1.0" encoding="UTF-8"?>
<request protocol="3.0">
  <os platform="win" version="10" arch="x64" />
  <app appid="{DDCCD2A9-025E-4142-BCEB-F467B88CF830}" ap="external-stable-universal">
    <updatecheck />
  </app>
</request>
"@

Confirmed that the installer still downloads and installs x64 version on x86 machine, so it is x64 only.
BTW, @vedantmgoyal2009 https://github.com/microsoft/winget-pkgs/pull/57371#issuecomment-1094993653


update uniprot downloader

[ ] fixes https://github.com/galaxyproteomics/tools-galaxyp/issues/600

[ ] TODO https://github.com/galaxyproteomics/tools-galaxyp/issues/586

py3

use requests library with retry functionality (as in https://github.com/bgruening/galaxytools/pull/1092)

exit with non-zero exit code in case of error

@bernt-matthias there is some stuff that does not belong into its PR.


more cleanup

if you're not going to read code in future then don't pr this otherwise I mean it's totally up to you on what to do

Weather is nice in Canada right now
PR means Pull Request

I did not test it. I am sorry for the inconvenience this has caused you. Click the revert button.


Allow user-supplied SSH Keys

Changes:

Allow user to pass their own ssh keys

Copies supplied keys into project directory when terraform apply is called
If ssh_key is omitted and ssh_user is supplied, an ssh key will be created when terraform apply is called.
output_name is changed to ssh-id_rsa if user does not supply one to keep backwards compatibility
use_agent - true is required if using password-protected keys and setting up of ssh-agent

ssh_key: # optional
  public_path: <path> # optional
  private_path: <path> # optional
  use_agent: false # default
  output_name: 'ssh-id_rsa' # default

Restrict permissions for project directory to 0750 and terraform.tfstate to 0600 since it may contain secrets
additional_volumes output added to each machine module which is a mapping of the device location to its specifications.

Will allow for volume script to eventually move out of edb-terraform so it can be handled as a configuration step instead of during provisioning. Then we can remove use_agent and remote_exec since direct ssh will NOT be needed by Terraform.

Output name used for the specification variable has been changed to base in each provider. module.spec.value -> module.spec.base since each.value is used by terraform for looping with for_each and can cause confusion of what is being referenced.

additional_volumes might not always have the correct device name
https://github.com/EnterpriseDB/edb-terraform/issues/30


[cadl-python] support cadl "unknown" type

When I write cadl class with "unknown" property such as:
 model smart_salmon extends Fish {
   ...
+  additionalProperties?: unknown;
 }

I get below error when compile it with python-client-emitter:

It dumps at emitter.ts:

we have support for this


New Email UI

Feature Request
Description
Implement the new UI designs for emails.
Screenshots

Additional context
Figma
No functionality change here, just UI update.
Acceptance criteria
Ensure that colour contrast passes accessibility standards.
Firefox is good at this. There's an accessibility tab in dev tools where you can see colour contrast warnings.
WHEN the user sends an email
THEN the email is shown in the right panel with the correct content

Found a bug!
Description:
Email outbox section doesn't scroll anymore, isn't keyboard focusable, and there is no padding in between emails.
Steps to reproduce the behaviour:

Go to any level
Ask the chatbot to send multiple emails, until the email outbox section on the right hand side fills up completely
See the emails squished together in the outbox container

Expected behaviour
When the emails are too big, or too many to fit into their container,
the user can scroll through the container to see all emails.
There is padding in between each email, and the email container is keyboard
focusable and can be scrolled by keyboard and mouse.
Screenshot of issue

testing


Release independent single windows app

How can we build an installer or single independent app for windows?
I know that flutter for windows is still not production-ready but is having a final app that works alone under consideration?

How can we build an installer

This isn't within the scope of the current desktop work; existing Windows installer builders could be applied to Windows applications built with Flutter (once there are Release builds).

or single independent app for windows? [...] is having a final app that works alone under consideration?

It's not clear what you are asking. What do you mean by "independent" and "alone"?

@stuartmorgan by independent I mean portable apps that don't need installation on windows.
And by alone I mean that no third-party runner is necessary.

by independent I mean portable apps that don't need installation on windows

The build creates a self-contained folder; no installer is necessary if you don't want to use one.

And by alone I mean that no third-party runner is necessary.

No version of Flutter for Windows has ever used a third-party runner. I suspect this is coming from a misunderstanding of what flutter run is. All it does is do the same thing flutter build does, which generates a native executable (a .exe on Windows, plus dependency libraries and resources), then launches that executable and attaches to the observatory. You can use Flutter on Windows right now without ever using flutter run.
Closing since as far as I can tell this isn't requesting anything that we don't already do.


[11.x] Include ConnectionException in ConnectionFailed events

Currently, you receive an event (\Illuminate\Http\Client\Events\ConnectionFailed) if a HTTP Request has hit a timeout of some kind (or some other error), however, the event only has access to the request, which doesn't include any information about the nature of the failure. There is no response that has been received, but we do have an exception (\Illuminate\Http\Client\ConnectionException) that is now being passed.

This was a breaking change for us...

We have old HTTP clients from the pre-HttpFactory era for which we've written stop-gap Guzzle middleware to dispatch Illuminate events to make them appear on Telescope. Can you please revert this? @driesvints

Case in point:
class HttpClientEventsMiddleware
{
    public static function make(): self
    {
        return app(self::class);
    }

    public function __construct(private readonly Dispatcher $dispatcher) {}

    public function __invoke(callable $handler)
    {
        return function ($request, array $options) use ($handler) {
            $this->dispatcher->dispatch(new RequestSending($illuminateRequest = new Request($request)));

            return $handler($request, $options)->then(
                $this->onSuccess($illuminateRequest),
                $this->onFailure($illuminateRequest)
            );
        };
    }

    protected function onSuccess(Request $request)
    {
        return function ($response) use ($request) {
            $this->dispatcher->dispatch(new ResponseReceived($request, new Response($response)));

            return $response;
        };
    }

    protected function onFailure(Request $request)
    {
        return function ($reason) use ($request) {
            $this->dispatcher->dispatch(new ConnectionFailed($request));

            return Create::rejectionFor($reason);
        };
    }
}

Ah sorry, I didn't realise this would be a feature that people are initiating themselves.
Happy for this to be reverted, and for the original PR to be focused against the next major version so that the BC can be documented properly

Ah sorry, I didn't realise this would be a feature that people are initiating themselves.
Happy for this to be reverted, and for the original PR to be focused against the next major version so that the BC can be documented properly

I'm sorry for this unexpected pushback, but we can't allocate resources to upgrade these old HTTP clients at this time. Thank you so much for your understanding!

Makes perfect sense. No breaking change was intended here, if I knew it would be I would have targeted the correct branch
From: Muhammed Sari @.>
Date: Thursday, 18 July 2024 at 15:03
To: laravel/framework @.>
Cc: Alex Bowers @.>, Author @.>
Subject: Re: [laravel/framework] [11.x] Include ConnectionException in ConnectionFailed events (PR #52069)
Ah sorry, I didn't realise this would be a feature that people are initiating themselves.
Happy for this to be reverted, and for the original PR to be focused against the next major version so that the BC can be documented properly
I'm sorry for this unexpected pushback, but we can't allocate resources to upgrade these old HTTP clients at this time. Thank you so much for your understanding!
—
Reply to this email directly, view it on GitHubhttps://github.com/laravel/framework/pull/52069#issuecomment-2236617158, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAGNZXTMDZQV4OEA4S6QAS3ZM7DJPAVCNFSM6AAAAABKTC4O6OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMZWGYYTOMJVHA.
You are receiving this because you authored the thread.Message ID: @.***>


feat: natively support Granite models

What does this PR do?
Adds support for Granite line of models.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


WinError 1314 when attempting to use diarization

Hello there, sorry to bother you, but I'm getting this error when attempting to use Diarization on windows 11
OSError: [WinError 1314] A required privilege is not held by the client: 'C:\Users\nicho\.cache\huggingface\hub\models--speechbrain--spkrec-ecapa-voxceleb\snapshots\5c0be3875fda05e81f3c004ed8c7c06be308de1e\hyperparams.yaml' -> 'C:\Users\nicho\.cache\torch\pyannote\speechbrain\hyperparams.yaml'
I have confirmed I am gated into all 3 models I need, so I'm not sure what's causing this error.

Looks like this was due to Visual Studio code not running your code as administrator. I was able to get it working from an administrative terminal. Sorry for wasting your time.


UkrFormalityClassification added

Translated English GYAFC data to Ukrainian: https://huggingface.co/datasets/ukr-detect/ukr-formality-dataset-translated-gyafc .

Checklist for adding MMTEB dataset

Reason for dataset addition:

[x] I have tested that the dataset runs with the mteb package.
[x] I have run the following models on the task (adding the results to the pr). These can be run using the mteb run -m {model_name} -t {task_name} command.

[x] sentence-transformers/paraphrase-multilingual-MiniLM-L12-v2
[x] intfloat/multilingual-e5-small

[x] I have checked that the performance is neither trivial (both models gain close to perfect scores) nor random (both models gain close to random scores).
[x] If the dataset is too big (e.g. >2048 examples), considering using self.stratified_subsampling() under dataset_transform()
[x] I have filled out the metadata object in the dataset file (find documentation on it here).
[x] Run tests locally to make sure nothing is broken using make test.
[x] Run the formatter to format the code using make lint.
[x] I have added points for my submission to the points folder using the PR number as the filename (e.g. 438.jsonl).

Should it be +4 for language in general or lang in task @imenelydiaker ? Also, doe we have some convenient way to list of all lang per task?

Also, doe we have some convenient way to list of all lang per task?

I'm not sure what you mean here?

I'm not sure what you mean here?

Like a function that crawls through all tasks and lists unique language for Classification, Retrieval etc. I think as the number of datasets grow, it'd be convenient.

I'm not sure what you mean here?

Like a function that crawls through all tasks and lists unique language for Classification, Retrieval etc. I think as the number of datasets grow, it'd be convenient.

I'm not sure we have this implemented, @KennethEnevoldsen may know if we already have this? Else, feel free to open a PR with the proposed implementation

@dokato please update your branch and add points and we'll be good to merge 🙂


Spider was doing fine job, pointing out errors till it didnot work. There wasno cursor in console when i opened spider.

Description
What steps will reproduce the problem?

What is the expected output? What do you see instead?
Please provide any additional information below
AttributeError: module 're' has no attribute 'search'
Version and main components

Spyder Version: 3.2.4
Python Version: 3.6.3
Qt Versions: 5.6.2, PyQt5 5.6 on Linux

Dependencies
pyflakes >=0.6.0 :  1.6.0 (OK)
pycodestyle >=2.3:  2.3.1 (OK)
pygments >=2.0   :  2.2.0 (OK)
pandas >=0.13.1  :  0.20.3 (OK)
numpy >=1.7      :  1.13.3 (OK)
sphinx >=0.6.6   :  1.6.3 (OK)
rope >=0.9.4     :  0.10.5 (OK)
jedi >=0.9.0     :  0.10.2 (OK)
psutil >=0.3     :  5.4.0 (OK)
nbconvert >=4.0  :  5.3.1 (OK)
sympy >=0.7.3    :  1.1.1 (OK)
cython >=0.21    :  0.26.1 (OK)
qtconsole >=4.2.0:  4.3.1 (OK)
IPython >=4.0    :  6.1.0 (OK)
pylint >=0.25    :  1.7.4 (OK)

While clicking run button the output is like:
Traceback (most recent call last):
File "/home/arjun/anaconda_install/lib/python3.6/runpy.py", line 183, in _run_module_as_main
mod_name, mod_spec, code = _get_module_details(mod_name, _Error)
File "/home/arjun/anaconda_install/lib/python3.6/runpy.py", line 142, in _get_module_details
return _get_module_details(pkg_main_name, error)
File "/home/arjun/anaconda_install/lib/python3.6/runpy.py", line 109, in _get_module_details
import(pkg_name)
File "/home/arjun/anaconda_install/lib/python3.6/site-packages/pylint/init.py", line 8, in
from .pkginfo import version as version
File "/home/arjun/anaconda_install/lib/python3.6/site-packages/pylint/pkginfo.py", line 16, in
from pkg_resources import parse_version
File "/home/arjun/anaconda_install/lib/python3.6/site-packages/pkg_resources/init.py", line 24, in
import re
File "/home/arjun/AnacondaProjects/re.py", line 12, in
searchobj = re.search('(.) / (.)' , text)
AttributeError: module 're' has no attribute 'search'

I think the problem is the file /home/arjun/AnacondaProjects/re.py. This file has the same name as the re module for regular expressions in the standard Python library. I suggest you rename your file as it is generally a bad idea to have the same name as used in the standard library, and hopefully that will fix the issue.

Let us know whether it fixes the issue; if we don't hear from you after a week, we'll assume it did and this issue will be closed. Thanks.

This file has the same name as the re module for regular expressions in the standard Python library

Yep, this is the problem here. @spider00, please rename your file to something else to fix this.


kotlin anonymous callback can't capture

the declare of base callback is like this:
typealias SucceedCallback<T> = ((t: T) -> Unit)
now my capture code is:
argumentCaptor<SucceedCallback<ProfileFoundation?>>().apply {
verify(apiService).getProfileInfo().subscribeRemoteData(capture())
firstValue.invoke(foundation)
}
then kotlin bytecode error:
Caused by: kotlin.KotlinNullPointerException

I'm not sure what you mean exactly. Could you include a MVCE with a full stack trace?

Closing due to inactiveness


clean new project won't start

If you are reporting a bug, please fill in below. Otherwise feel free to remove this template entirely.
Can you reproduce the problem with latest npm?
yes
Description
create-react-app demo
cd demo
npm start

will get the following error message:
Failed to compile.

Error in ./src/index.js
Module build failed: Error: Cannot find module 'electron-to-chromium' (While processing preset: "/mnt/c/Users/kunch/github/demo/node_modules/babel-preset-env/lib/index.js") (While processing preset: "/mnt/c/Users/kunch/github/demo/node_modules/babel-preset-react-app/index.js")
 @ multi main

I looked into node_modules/electron-to-chromium and there's no index.js in it, which might be the cause. however i'm curious why a clean setup of create-react-app would run into this problem.
Expected behavior
Expecting "npm start" to start the dev server.
Actual behavior
get the following error message:
Failed to compile.

Error in ./src/index.js
Module build failed: Error: Cannot find module 'electron-to-chromium' (While processing preset: "/mnt/c/Users/kunch/github/demo/node_modules/babel-preset-env/lib/index.js") (While processing preset: "/mnt/c/Users/kunch/github/demo/node_modules/babel-preset-react-app/index.js")
 @ multi main

Environment
kunchenguid@DESKTOP-284F0JO /m/c/U/k/g/demo> npm ls react-scripts
<EMAIL_ADDRESS>/mnt/c/Users/kunch/github/demo
└──<EMAIL_ADDRESS>
kunchenguid@DESKTOP-284F0JO /m/c/U/k/g/demo> node -v
v7.8.0
kunchenguid@DESKTOP-284F0JO /m/c/U/k/g/demo> npm -v
4.4.4

Operating system: Ubuntu on Windows 10 (windows linux subsystems)
Browser and version:

Reproducible Demo
consistently reproducible with clean new project.
create-react-app demo
cd demo
npm start

Try running
cd demo
rm -rf node_modules
npm cache clear
npm install

I can’t reproduce this with given Node and npm versions so it’s likely a corrupted cache, a bad local registry, or something like this.
Unfortunately we can’t do anything about npm bugs, but you can try using Yarn instead:
npm i -g yarn

Then next time you run CRA, it will use Yarn and hopefully won’t fail.


How to pass a single host network interface to a pod

How do I pass a single host network interface into a pod?
My usecase is, that I want to have a pod reading data from a CAN Bus, und publishing the parsed data as HTTP in a contained network.
For now I am only able to run my container with --net=host to have access to the can0 interface.
Is there a possibility to explicit grant access to the can0 interface? Perhaps I only need to bin-mount something from /dev, but I don't know.

First of all: cool!
I'm not familiar with the CAN support in Linux - is it a network device or a block/character device?
If the latter, it's easy: just bind-mount the device node in to the pod. Rkt will automatically set up the devices.allow cgroup correctly.
If it's a network device, then just run the pod with the CAP_NET_ADMIN capability and have your pod create the interface at start time. Alternatively, you can move an existing link in to an existing namespace with
ip link set dev can0 netns <pid>

As far as I know there is a character device driver and a network device driver. I'm using the network device driver, so I access the bus via the berkley socket api.
Moving the interface into the running pod is no option, this would require timing to wait until the pod is started, I don't think this would work well (a simple ExecStartPost statement in the unit file would surely doesn't do the job)
How would you create the interface at runtime? If I would create another container for that, I would also have timing issues between the container using the can and the one creating it.
I can not imagine a proper way to achieve what I want :(

If the device does not exist on the running host at pod start time, then you can add a prestart hook that issues the correct ip link add ... commands. You can also build this in to your own application.
If the device already exists, then I suggest writing a trivial CNI plugin that just moves the device in on ADD, and moves it out on DELETE.

then I suggest writing a trivial CNI plugin

any docs about that would be really usefull ;)

Sure!
I wrote a sample CNI plugin here - https://github.com/containernetworking/plugins/tree/master/plugins/sample - it should suit your needs nicely.

Cool, thank you for the pointer! The vlan plugin found in the same repo gave me some nice hints on how to handle namespace manipulation.
After reading about network namespaces in general I was able to do, what this plugin should do, by hand.
Move In:
1| systemd-run --unit can-test rkt run --uuid-file-save=./uuid  trusch.io/alpine tail -- -f /dev/null
2| POD_NS=$(ip netns identify $(ps -ef|grep $(cat uuid)|tr -s ' '| cut -d ' ' -f2|head -1))
3| ip link set dev can0 netns $POD_NS

Move Out:
1| touch /run/netns/default && mount --bind /proc/1/ns/net /run/netns/default
2| ip netns exec $POD_NS ip link set dev can0 netns default
3| systemctl stop can-test

I would start writing the plugin. Should I pull-request to github.com/containernetworking/plugins?
How are the chances that this "host-device" plugin will make it into stage1? I think it is a usefull feature, to select which host network interfaces are exposed to a container, instead of running it in host-namespace.
I'm not quiet sure how to test the plugin. Do you think it would be enough to "patch" a stage-1 aci to include my plugin?
Thanks for the input so far @squeed !

Pull request at CNI-Plugins

Well, this worked out nicely.
You don't need to change rkt at all to include your plugin - rkt will look in /usr/lib/rkt/plugins/net for any plugin binaries. Just drop the built plugin there, add the configuration to /etc/rkt/net.d/ and you're good to go.
I'll look at the CNI plugin PR, but we may not merge it - the CNI plugin repository is generally intended for basic functionality shared by most installations. The maintainers meet weekly; we might decide to link to your plugin from our database instead.


Question\Feature - ExpandItem Padding\Margin

Hi There,
Is there a way i can add some padding\margin to the expanditem. Something like eg Padding="16,0" ? I've between applying a margin to the expanditemtemplate but it is being clipped when resizing horizontally.
This clipping only really happens when there is 1 column of data present.
Thanks

Hi,
applying a margin seems to me to be the right way. The problem is that the ExpandItem inclusive margin is to wide to fit into the constraints of the panel. The expand item is not allowed to be wider as the row of the items.
It may help if you enable the StretchItems property. Then the full available width is used for the items.

Thanks, i have enabled the StretchItems property and it seems to have resoved the issue.


update "just text" at the end of file

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

just shirt text


generate_k_shot_data.py

The generate_k_shot_data.py file keeps reporting an encoding error when being loaded

Hi, could you paste the error message here?


chore: hubble-dist pacakge error

closed #276

u could run hubble-dist/assembly/travis/run-api-test.sh in local env first to test it


Check if any relic spawn before X floor, if does, make it spawn after non-endless limit

There isn't any which does.


#gmcr magic will not work if using break to finish while

Don't know, may be it's better to add break to limitations section
script below will not end because break stops switch block in translated code instead of while
#gmcr
#args map
var key = ds_map_find_first(map);
while true
{
if is_undefined(key) then break;
yield [key,map[? key]];
key = ds_map_find_next(map,key);
}
script below will work good without break
#gmcr
#args map
var key = ds_map_find_first(map);
while not is_undefined(key)
{
yield [key,map[? key]];
key = ds_map_find_next(map,key);
}

Fixed for the next version.


[VS Code] The Visual Studio Live Share service is unavailable at this time. Please try again later.

Product and Version [VS/VSCode]: VSCode
OS Version [macOS/Windows]: Windows
Live Share Extension Version: 0.3.623
Target Platform or Language [e.g. Node.js]: Node.js
Steps to Reproduce / Scenario:

Try to live-share a session.
Observe

Expected result: VS Liveshare succeeds and you get a shareable link
Actual result: Liveshare throws a "The Visual Studio Live Share service is unavailable at this time. Please try again later." error.

A team member has assured me this report has been recieved and is being looked into.


Please address me in some small workflows

Hi, thanks by these great nodes, really I got impressed by the paper, I think your tools are the starting point to fix so many issues with robotics these days.
I get laser scans with colored pointclouds, and I would like ,apart can use them as you do in your repository (rover), can export the pointcloud+colour+normals,  meshes+colour ,or both to external software as Cloud Compare, Meshlab, blender or Unreal Engine, to in example can make maps for Gazebo, Isaac Sim...
In your paper I read you use HDF5 files but I couldn't find any software that can import them,  could be possible so it in .ASC format maybe?.
Really I think this should be the fact standard, it is so brilliant.
Thanks!

Hi @FPSychotic,
First of all, thanks for appreciating our work. One workflow I can think of is:

Record a map:

PCL SLAM, PCL -> Mesh (lvr2, ...)
Get it from the architect of your building
...

Automatically transport things from A to B in uneven terrain:

Localization: RMCL
Path planning and robot control: mesh_navigation + move_base_flex

The "how to get a map" is also discussed in issue #20.
After recording a map, you would normally have a standard mesh format such as PLY or OBJ. However, those formats turned out to be not descriptive enough to represent the layered format that we use for navigation. So we agreed on HDF5. Or in more detail, a certain structure within the HDF5 file (Mesh-Tools Paper, Figure 7). There are also tools such as HDFCompass which you can use to visualize the structure of an already existing H5 file. BTW, the structure looks very similar to a standard file-directory. To convert a standard mesh format (such as PLY) to H5 you can use the lvr2_hdf5_mesh_tool from lvr2.
With migrating everything to ROS2, we are working on providing a set of examples/tutorials/documentation that should address all the old issues that have arisen over the last few years. The issue about the conversion of standard mesh formats to HDF5 is now tracked here #37.

I think we can close this issue as I integrated most of it in greater detail into the new docs of mesh_navigation_tutorials


Fix some bug

fix bugs in dataset and  metrics
add some distribute interface for counter

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.xiexionghang seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Potential v2 changes

Would like to make some backwards-incompatible changes. For example:

Change MissingTrieNode exception to stop subclassing from KeyError
Default trie to be immutable, where every change spits out a new root hash, and list of nodes to change, but doesn't affect the original trie. (and stop tracking the root hash as a property of the trie)

Surely there are others, just wanted to start a running list.

:+1: immutability

Drop support for from trie import Trie (which is an alias for HexaryTrie)

Drop deprecated HexaryTrieSync


Proposed name change for MOD

Modified the default manifest to use MOD_SWITCH key instead of MOD. The fix matches naming convention for legacy code.
Proposed changes
Describe the big picture of your changes here to communicate to the maintainers
why we should accept this pull request. If it fixes a bug or resolves a feature
request, be sure to link to that issue.
The proposed change keeps naming consistent for legacy code.
Types of changes
What types of changes does your code introduce to the HE Toolkit project?
Put an x in the boxes that apply

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

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

[x] I have read the CONTRIBUTING agreement
[x] Current formatting and unit tests / base functionality passes locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works (if appropriate)
[ ] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

Further comments
I discussed the changes with Jeremy and Jack on July 24th.

Closed as these changes will need further sync with other parts of the SDK.


[v2] Introduce HistoryLimit with go mock

The Scaled Job consume a lot of objects if you are running for a long time. It will eventually eat all the resource.
So I recommend to remove the old jobs, however keep some for debugging.  In this PR, I did:

Implement History Limit
Introduce go mock for scaleJob testing

History Limit
At the first time, I considered to introduce retention policy, however, I change my mind since I found Jobs History Limits on Cron Job on Kubernetes.

Jobs History Limits

I propose the spec will be like this.
New Configuration
  successfulJobsHistoryLimit: 3   // How many success jobs are preserved. Optional. Default Value is 100.
  failedJobsHistoryLimit: 2          // How many failed jobs are preserved. Optional. Default Value is 100.

Pros and Cons with Retention Policy
Compared with the Retention Policy, this strategy allow you to specify the number of the jobs remain. The naming and system is the same as CronJob on kubernetes. However, Retention Policy is for the time, so we don't know how many resources are consumed.
The week point of the strategy is, amount of the computation. We need to split it and sort the by the current queue length. O(n + nlogn ) , the retention policy will be O(n).
However, it won't very long since it is the list of the completed jobs.
apiVersion: v1
kind: Secret
metadata:
  name: azure-storage-queue-secret
data:
  ConnectionString: YOUR_BASE64_ENCODED_CONNECTION_STRING
  queueName: aGVsbG8=
---
apiVersion: keda.sh/v1alpha1
kind: ScaledJob
metadata:
  name: azure-storage-queue-consumer
  namespace: default
spec:
  jobTargetRef:
    template:
      spec:
        containers:
        - name: azure-storage-queue-receive
          image: tsuyoshiushio/azure-storage-queue-client:dev
          imagePullPolicy: Always
          command: ["receive"]
          envFrom:
            - secretRef:
                name: azure-storage-queue-secret
        restartPolicy: Never
    backoffLimit: 4
  pollingInterval: 30   # Optional. Default: 30 seconds
  cooldownPeriod: 30   # Optional. Default: 300 seconds
  maxReplicaCount: 3  # Optional. Default: 100
  successfulJobsHistoryLimit: 3
  failedJobsHistoryLimit: 2
  triggers:
  - type: azure-queue
    authenticationRef:
      name: azure-queue-auth
    metadata:
      queueName: "hello"
      queueLength: "1"
---
apiVersion: keda.sh/v1alpha1
kind: TriggerAuthentication
metadata:
  name: azure-queue-auth
spec:
  secretTargetRef:
  - parameter: connection
    name: azure-storage-queue-secret
    key: ConnectionString

TODO
I'd like to discuss where to keep the default value on the ScaledJob. Maybe we should place it in once place. Or do we have better way to configure it? I'd like to know it.
Known Issue
The number is not always correct, however, it will eventually will be the configuration value.
The reason is, when the trigger happens, CreateJob and CleanUp will be run. When CreateJob is executed and not finished, then CleanUp runs, in this care, CleanUp can't fetch the non-completed jobs. So it just clean up the current jobs. However, after cleanUp, the new jobs that is created by CreateJob  remains. However, I think it is not very big deal.
go mock
For testing this logic, I introduce go mock. At the first time, I consider to use Unit testing. However, it doesn't cover this scenario. We might be better to use both methods. I believe, go mock will help to write unit testing easily and it is defact standard on go programming.
For the generating mock, I do something like this.  We can include this with Makefile in the near future and include contributor guide about mocking.
mockgen --source<EMAIL_ADDRESS>-destination pkg/mock/mock_interfaces.go

Checklist

[ ] A PR is opened to update the documentation on https://github.com/kedacore/keda-docs
[-] Commits are signed with Developer Certificate of Origin (DCO)
[-] Tests have been added

Fixes #
https://github.com/kedacore/keda/issues/829

Hi @ahmelsayed
I accept your all request and change it and tested it.

@ahmelsayed
I update one thing. There is deepcopy method on the generated code. I'm not sure if we generated it again or keep updated manually, however, it looks not automatically generated. so I added it. If you are ok, could you merge this?

@TsuyoshiUshio there are a couple of conflicts

Thank you @ahmelsayed
I fix the conflict and fix one bug (the cleanup only works when they create a job).
Could you merge it?


Adding 418 as a valid status code.

Justification here, https://tools.ietf.org/html/rfc7168#section-2.3.3

Coverage increased (+0.008%) to 85.189% when pulling 29ffd3fba9add5f670ce1200e919559e0f51e24c on praserocking:master into b04a342a4bcd0ff44be65138acffc73a29381c72 on NanoHttpd:master.

I'm not sure if I should scold you for using an april fools RFC as a justification for adding a joke HTTP error or if I should just accept it for catalog completion reasons even though we lack many more important status codes.

Thank you, I will work on adding all important status codes as next PR.