Using a token causes: 500 Internal Server Error

Hi, when I set a token in ~/.grip/settings.pyand load a page from grip, I see "500 Internal Server Error" in the web browser. Here's the dump from the command line:
 * Using personal access token
 * Running on http://localhost:6419/ (Press CTRL+C to quit)
[2016-12-09 16:45:54,144] ERROR in app: Exception on / [GET]
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 1988, in wsgi_app
    response = self.full_dispatch_request()
  File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 1641, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 1544, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 1639, in full_dispatch_request
    rv = self.dispatch_request()
  File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 1625, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/usr/local/lib/python2.7/site-packages/grip/app.py", line 174, in _render_page
    content = self.renderer.render(text, self.auth)
  File "/usr/local/lib/python2.7/site-packages/grip/renderers.py", line 100, in render
    r = requests.post(url, headers=headers, data=data, auth=auth)
  File "/usr/local/lib/python2.7/site-packages/requests/api.py", line 110, in post
    return request('post', url, data=data, json=json, **kwargs)
  File "/usr/local/lib/python2.7/site-packages/requests/api.py", line 56, in request
    return session.request(method=method, url=url, **kwargs)
  File "/usr/local/lib/python2.7/site-packages/requests/sessions.py", line 474, in request
    prep = self.prepare_request(req)
  File "/usr/local/lib/python2.7/site-packages/requests/sessions.py", line 407, in prepare_request
    hooks=merge_hooks(request.hooks, self.hooks),
  File "/usr/local/lib/python2.7/site-packages/requests/models.py", line 306, in prepare
    self.prepare_auth(auth, url)
  File "/usr/local/lib/python2.7/site-packages/requests/models.py", line 527, in prepare_auth
    r = auth(self)
  File "/usr/local/lib/python2.7/site-packages/requests/auth.py", line 68, in __call__
    r.headers['Authorization'] = _basic_auth_str(self.username, self.password)
  File "/usr/local/lib/python2.7/site-packages/requests/auth.py", line 38, in _basic_auth_str
    b64encode(b':'.join((username, password))).strip()
TypeError: sequence item 0: expected string, NoneType found
<IP_ADDRESS> - - [09/Dec/2016 16:45:54] "GET / HTTP/1.1" 500 -

My ~/.grip/settings.py looks like this:
def find_token():
    import subprocess
    import re
    cmd = ['security', 'find-generic-password', '-gs', 'grip_github_token']
    pwinfo = subprocess.Popen(cmd, stdout = subprocess.PIPE,
            stderr = subprocess.PIPE)
    pwline = pwinfo.stderr.read().strip()
    return re.sub('password: "(.*)"', '\\1', pwline)

PASSWORD = find_token()

This has worked fine for the past 2 years or so. I upgraded Grip to 4.3.2, and that hasn't helped.
If I remove the PASSWORD field from settings.py, I am able to load the grip web page..

Interesting! I'll look now more closely tonight, but for now, could you
check that find_token is returning a string. Looks like it might be
returning None instead.
On Fri, Dec 9, 2016, 5:52 PM Nate McCurdy<EMAIL_ADDRESS>wrote:
Hi, when I set a token in ~/.grip/settings.pyand load a page from grip, I
see "500 Internal Server Error" in the web browser. Here's the dump from
the command line:

Using personal access token
Running on http://localhost:6419/ (Press CTRL+C to quit)
[2016-12-09 16:45:54,144] ERROR in app: Exception on / [GET]
Traceback (most recent call last):
File "/usr/local/lib/python2.7/site-packages/flask/app.py", line
1988, in wsgi_app
response = self.full_dispatch_request()
File "/usr/local/lib/python2.7/site-packages/flask/app.py", line
1641, in full_dispatch_request
rv = self.handle_user_exception(e)
File "/usr/local/lib/python2.7/site-packages/flask/app.py", line
1544, in handle_user_exception
reraise(exc_type, exc_value, tb)
File "/usr/local/lib/python2.7/site-packages/flask/app.py", line
1639, in full_dispatch_request
rv = self.dispatch_request()
File "/usr/local/lib/python2.7/site-packages/flask/app.py", line
1625, in dispatch_request
return self.view_functionsrule.endpoint
File "/usr/local/lib/python2.7/site-packages/grip/app.py", line 174,
in _render_page
content = self.renderer.render(text, self.auth)
File "/usr/local/lib/python2.7/site-packages/grip/renderers.py",
line 100, in render
r = requests.post(url, headers=headers, data=data, auth=auth)
File "/usr/local/lib/python2.7/site-packages/requests/api.py", line
110, in post
return request('post', url, data=data, json=json, **kwargs)
File "/usr/local/lib/python2.7/site-packages/requests/api.py", line
56, in request
return session.request(method=method, url=url, **kwargs)
File "/usr/local/lib/python2.7/site-packages/requests/sessions.py",
line 474, in request
prep = self.prepare_request(req)
File "/usr/local/lib/python2.7/site-packages/requests/sessions.py",
line 407, in prepare_request
hooks=merge_hooks(request.hooks, self.hooks),
File "/usr/local/lib/python2.7/site-packages/requests/models.py",
line 306, in prepare
self.prepare_auth(auth, url)
File "/usr/local/lib/python2.7/site-packages/requests/models.py",
line 527, in prepare_auth
r = auth(self)
File "/usr/local/lib/python2.7/site-packages/requests/auth.py", line
68, in call
r.headers['Authorization'] = _basic_auth_str(self.username, self.password)
File "/usr/local/lib/python2.7/site-packages/requests/auth.py", line
38, in _basic_auth_str
b64encode(b':'.join((username, password))).strip()
TypeError: sequence item 0: expected string, NoneType found
<IP_ADDRESS> - - [09/Dec/2016 16:45:54] "GET / HTTP/1.1" 500 -

My ~/.grip/settings.py looks like this:
def find_token():
import subprocess
import re
cmd = ['security', 'find-generic-password', '-gs', 'grip_github_token']
pwinfo = subprocess.Popen(cmd, stdout = subprocess.PIPE,
stderr = subprocess.PIPE)
pwline = pwinfo.stderr.read().strip()
return re.sub('password: "(.*)"', '\1', pwline)
PASSWORD = find_token()
This has worked fine for the past 2 years or so. I upgraded Grip to 4.3.2,
and that hasn't helped.
If I remove the PASSWORD field from settings.py, I am able to load the grip
web page..
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/joeyespo/grip/issues/221, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAmuzaUuA9dlvIEajpnEc5L5HQj718NPks5rGdupgaJpZM4LJcH6
.

Thanks for looking into this!
Confirmed that yes, the find_token() call is returning a string:

Also confirmed that the token is still valid in GitHub.

Looks like the auth argument may have become more strict.
Could you try two things?

See what version of requests you're using
$ pip show requests

(Mine is 2.12.1.)

Try adding the following to your settings.py file
USERNAME = ''

If (2) fixes it, I'll patch the codebase to handle this edge case so it's less of a surprise.
Thanks!

requests version: 2.12.3
That worked!

Looks like it was that USERNAME needed to be set to the empty string.

Ok, excellent!
I just pushed a fix to master. I'll publish a new release soon.
Thanks for the issue, @natemccurdy!


fix: select placeholder color

The placeholder for the select component was black instead of text-muted-foreground because it was using placeholder: instead of data-[placeholder]:
Closes #574

I agree with this change and was about to make a PR myself. A subtle addition I would recommend is explicitly styling the ChevronDown as to prevent it from inheriting the text-muted-foreground. Please see my comment on issue #3737

Could anyone approve this PR? I've fixed manually in my project, but probably some other folks are facing the same issue with the Select component


Simplify the warning when the database does not fully support utf8mb4

Fix #7360

Before

After

@mlocati I'm getting this error via a CLI installation:

Database should support the configured character set and collation... The database does have a complete support for all the Unicode characters. This implies, for example, that you may not be able to save emoticons (it's the same behavior as concrete5 versions older than 8.5.0). You should upgrade your database to fix this issue.

Considering that if the database has complete support for all Unicode characters, why wouldn't it be able to save emoticons?
Or is this a typo, and should it say "The database does not have (...)"?

@a3020 you are right, it's a typo.
It should be "The database does not have..."


fix #285 by updating pomelo e-g-web-sim dependency

note to self: this is probably obsolete because I tend to upstream my e-g-web-sim patches. Need to verify


Integrate key vault into terraform and make sure cluster has access to this key vault resource

All clusters have workloads with secrets. The recommended way to do this on Azure and AKS is using key vault. This task is to bootstrap key vault in terraform and make sure the cluster is also available through a suitable as transparent as possible manner inside of the cluster perhaps using flexvol or some other method.
When this is done whenever a new resource is made using terraform the appropriate connection string should also be inserted into the keyvault so that workloads inside the cluster can get access to it without jumping too many hoops

Now implemented. service bus connection string added as an example


Appstudio update test-component-pac-mdtq

Pipelines as Code configuration proposal

Pipelines as Code CI/test-component-pac-mdtq-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
6 seconds
init

✅ Succeeded
11 seconds
clone-repository

✅ Succeeded
27 seconds
build-container

✅ Succeeded
20 seconds
sanity-inspect-image

✅ Succeeded
25 seconds
deprecated-base-image-check

✅ Succeeded
9 seconds
clair-scan

✅ Succeeded
36 seconds
clamav-scan

✅ Succeeded
10 seconds
sbom-json-check

✅ Succeeded
7 seconds
sanity-optional-label-check

✅ Succeeded
7 seconds
sanity-label-check

✅ Succeeded
8 seconds
show-summary

Pipelines as Code CI/test-component-pac-mdtq-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
6 seconds
init

✅ Succeeded
11 seconds
clone-repository

✅ Succeeded
27 seconds
build-container

✅ Succeeded
25 seconds
deprecated-base-image-check

✅ Succeeded
20 seconds
sanity-inspect-image

✅ Succeeded
36 seconds
clamav-scan

✅ Succeeded
9 seconds
clair-scan

✅ Succeeded
10 seconds
sbom-json-check

✅ Succeeded
7 seconds
sanity-label-check

✅ Succeeded
7 seconds
sanity-optional-label-check

✅ Succeeded
8 seconds
show-summary


Local execution on Google Home optional device

Hello,
I was wondering if I can deploy this solution without a Google Home or Google Nest but instead with a Raspberry PI that simulates a Google Home with the Google Assistant SDK?
Thank you, in advance.

As noted in the documentation:

Google Home or Google Nest devices can load and run your app on-device.

Other devices cannot do this.

Ok thank you.


Investigate additional cuts on smear tuples

Phoebe noticed that no addtional cuts seem to be applied to the smear tuples, regardless of the muid cut. Possibly, these cuts would have to be applied in this block. To be investigated.
Slack discussion:  https://umd-lhcb.slack.com/archives/C0100NU9211/p1715369393999139

It seems I'm using the wrong smearing histograms in the mu misid validation fit (although the file is correct).
The smearing factors are retrieved from the smearing histograms in this function in ApplyMisIDWeight.cpp. By default, these histograms are read from the branches k_smr and pi_smr of dif.root, which is produced by build_histo_dif.py. Additional cuts are applied when producing the smearing histograms (see line 37), so in principle the baseline fit should be fine. Still, I believe we should check if we are missing any (e.g. why no PIDe<1 cut?).
More importantly, build_histo_dif.py produces two pairs of smearing histograms, *_smr and *_smr_ubdt_veto, which differ by the muBDT cut applied. The latter should be used for the misid fit, but currently it isn't. A small modification on ApplyMisIDWeight.cpp should fix that.

I guess I was actually too pessimist. The error above was just the result of performing the append inside a loop. I just didn't leave it running enough time to get something like more obvious like error in opening the tree k_smr_ubdt_veto_ubdt_veto_ubdt_veto_ubdt_veto_ubdt_veto_ubdt_veto.

For reference, these are the current fit templates and results, which use the incorrect smearing and a different combuncertscale as spotted by Phoebe.
fit_result_std_smearing.pdf
templates_std_smearing.pdf

Applying the correct ubdt cut when producing the smearing affects the validation fit, but not by a large margin. Overrall, the misid nirmalization is reduced for D* and D0, and comb_bkg is also reduced for D0.
D0 ISO:

D* ISO:

fit_result_ubdt_veto_smearing.pdf
templates_ubdt_veto_smearing.pdf
std_smr_vs_ubdt_veto_smear.pdf

Modified https://github.com/umd-lhcb/misid-unfold/blob/0fc6b9e2ed8abe3ed319a845b8f80f03e4dae1fa/scripts/build_histo_dif.py#L38-L43 to apply (almost) all cuts to Mc sample used to estimate momentum smearing due o DiF. For reference, these are the cuts we need (in principle) to apply (some redundant cuts included for completeness):
HLT2:

Stripping:

Offline:

The offline $\chi^2_{IP}$ requirement is omitted since it should not affect the smearing and (most importantly) it would reject most signal events, since the MC used is dominantly promt charm.
The MC truth requirements have also been refactored. Previously, the $K / \pi$ candidates were required to have the correct TRUEID. However, due to the in-flight decays, these tracks are most likely randomly truth-matched to hadron, muon or ghost. To avoid signal loss, the truthmatching requirement is now placed on the D meson.
Updated cuts:
https://github.com/umd-lhcb/misid-unfold/blob/c1bfc88f6ec36fb21d1a58d79475b7d9653768d3/scripts/build_histo_dif.py#L39-L62
Updated results:
fit_templates_smr_all_cuts.pdf

Note: This is an attempt to document the discussions from the last few days in Slack. Thanks Phoebe for the valuable insights concerning the MC truth-matching.

As it turns out, the truth-matching requirements can still be improved. The current way is to apply cuts on the K and pi TRUEID, but the in-flight decays make this variable unreliable (see comment above). The D mass distributions below are obtained after placing cuts on TRUEID for D, K and pi (besides momentum and eta cuts for K).

The blue curve represents the "true" D mass, reconstructed from the true K and pi momenta. This distrubution is expected to contain some smearing due to FSR simulated with PHOTOS. The grey bars represent the distribution "correctly" smeared by the kaon decays, which is obtained by reconstructing the D mass from the true pi and reconstructed K momentum. Finally, the black dots represent the distribution obtained by smearing the K true momentum with the technique used in the run 1 ana. Ideally, the dots should agree with the grey bars.
As discussed above, the cuts on pi and K TRUEID are not well defined, so if we rely on the D TRUEID only we get:

This causes a slight increase in the number of events, which seem to concentrate on the right tail of the distribution. In both cases, the tails in the true distribution seem too large to be caused by FSR alone. The D TRUEID requirement might still leave room for incorrect truth-matching.
One way to improve this is to exploit the *_TRUEORIGINVERTEX_TYPE variable for the kaons and pions. The meaning of the assigned values is somewhat described here. We can check the importance of these variables by plotting the pi true end vertex Z against its true vertex type:

Upon inspection most pi tracks have TRUEORIGINVERTEX_TYPE 2 (DecayVertex), however $\mathcal{O}(1000)$ have TRUEORIGINVERTEX_TYPE 100 (HadronicInteraction), which means these pions have originated from nuclear interactions, and $\mathcal{O}(10)$ have TRUEORIGINVERTEX_TYPE 102 (PairProduction). If we now require K and pi to have TRUEORIGINVERTEX_TYPE 2, the result is:

The increase in the right tail persists, but the left tail shrinks significantly.
On top of this, we can also check the position of the true vertices. If we look at the D_TRUEENDVERTEX_Z distribution with the cuts above, we note it doesn't feature a tail:

(Note: The plot above is missing the K eta cut.)
On the other hand, if we look at K_TRUEORIGINVERTEX_Z, we see a long tail extending along the whole tracking system (the end of the tail lies close to the T3 station):

This indicates that we still have incorrectly truth-matched tracks. We can also exploit this variable and require that K_TRUEORIGINVERTEX_Z and pi_TRUEORIGINVERTEX_Z match D_TRUEENDVERTEX_Z exactly. With this, we obtain

We now get a much narrower true distribution, and the smeared distribution also seems to agree better with the reconstructed one. For reference, below you can find the distribution of events for which either of the tracks fail the VERTEX_Z "matching" requirement:

Note how the true distribution does not present a peak anymore, even though the reconstructed distribution still does. This indicates that these events where the K/pi true origin vertex does not match the D true end vertex are, in fact, events in which the wrong MC particle was associated with the track. Therefore, these should also be removed when estimating the smearing.
There is now one more issue to address, which is that when the true PX or PY are too close to zero, the ratio reco/true becomes too large. This can be seen if we plot the ratio for the X and Y components against the corresponding true momentum component:

The extreme ratio values only show up for very small PX/PY values, so we can try to improve this by not using events with PX or PY < 50 MeV to estimate the smearing. Finally, we get

This results in better agreement on the right tail, which was sistematically overestimated before.
As a conclusion, I'm implementing the last solution of performing truth-matching using D_TRUIE, *VERTEX_TYPE and *VERTEX_Z, and excluding tracks for which the true PX or PY is less than 50 MeV.

It turns out the current truth-matching strategy has quite an impact on the number of usefuls events:

The first column represents the current smearing histograms, with K/pi TRUEID-based truth-matching. The second column represents the first attempt at fixing the truthmatching, while also implementing all mu selection cuts (except the IPchi2 cut). Even though several selection cuts are applied, requiring D TRUEID instead of K/pi TRUEID results in increased statistics. The third column represents the strategy presented in the last coment (based on true vertex information). Even though we remove some selection cuts, the new truth-matching strategy caauses a severe reduction in statistics compared to the current approach. Can we do something else to recover statistics?

Following the merge of https://github.com/umd-lhcb/rdx-run2-analysis/pull/114, https://github.com/umd-lhcb/lhcb-ntuples-gen/pull/131 and https://github.com/umd-lhcb/misid-unfold/pull/14, the updated smearing is now on master. Check these slides for the effects on the fit results.


Evaluate, whether conan could be used for dependency management

These talks sum everything up quite nicely:

https://www.youtube.com/watch?v=S4QSKLXdTtA
https://www.youtube.com/watch?v=xBLjXdyh3zs

There is a multitude of package managers out there. At some point I thought about adding TiXI to vcpkg, which I use privatly. Jason Turner also has a short intro to hunter, vcpkg and conan, which might be interesting to you:

https://www.youtube.com/watch?v=9cCQHJ-cNHY
https://www.youtube.com/watch?v=O2_N8OzPGWQ
https://www.youtube.com/watch?v=KOeOLOu6nHw

I do not want to recommend any of those to you. I played only with vcpkg so far and the Visual Studio integration works well, but for cmake you have to pass an annoying cmd line argument.
I am interested in what you will pick ;)

Here's a nice list of pros and cons of the different package managers:
https://www.reddit.com/r/cpp/comments/8t0ufu/what_is_a_good_package_manager_for_c/e141ggo/
I also like to add:

spack (https://spack.io/)
conda (https://conda.io/docs/index.html)

With conda we already have a lot of (mixed) experience but in the end it works pretty well. What conda does really well is to support relocation of binary packages by replacing the original installation directory during build with the user installation directory.

Vcpkg works for Linux as well. And I believe you are right, that you can only have one version (usually the latest) installed. Vcpkg installs packages system wide, which is usually what I want. But yes, if you want to test with different versions, you still have to maintain those manually.
Thank you for the pros/cons list. It is brief but excellent!
Have you thought about providing support for multiple package managers?

I don't really want to support package managers that I don't use, since this is a big maintenance overhead.
I experienced this with the support for several bindings. I cannot guaranty die quality of bindings, which I don't (want) to test. The same if true for packaging system that we don't use.
Right now, I already have to maintain a couple of packaging systems, which are:

conda for the python builds
Debian packages
RPM packages

For vcpkg, you can have different copies of packages by simply having different vcpkg instances -- each "clone" of vcpkg is a totally separate package graph. We do have the option to "promote" one of them to be user-wide for convenience, but all the other operations are totally local. We view it as a "best of both worlds approach" -- convenient when possible, capable when not :)
Additionally, each "commit" to the vcpkg repo is like a lockfile for the universe, specifying exactly what versions of every package work together (and we track the latest stable versions upstream). You can modify what packages to use by simply replacing the corresponding entry in ports/ with your own version.

@ras0219-msft thank you for the insight, I did not think about having multiple installations of vcpkg, which might be a useful concept!
An advantage/disadvantage of vcpkg is also that it builds all the dependencies from source. This is awesome because you can then debug into third party libraries. On the other hand, if for some reason the sources fail to download or compile, you are stuck. One might try to update to some older revisions, but this is tedious. I just had two issues this week with a hash missmatch on the 7zip version and a build fail on OpenSSL for x86-windows.
I have just seen a talk about conan at CppCon 2018 and it also looks good. It seems to be more invasive into your CMake scripts though and has less available packages. But it supports binary distribution, so it might be more stable.
In the end, the decision is yours to make @rainman110 ;)

@ras0219-msft Thanks for the information. I understand that you guys are trying to boost the vcpkg ecosystem and I think it has a very good potential.
For now, vcpkg has - from the practical point of view - some major limitations, as @bernhardmgruber  already mentioned, namely

Missing versioning support
Missing support for binary file distribution

We are using a pretty large third-party library that builds up to an hour. Thus, only source distribution increases the time-to-solution by far. Versioning is also very important for us. We are using patches sources of the third party library and experimenting with several version of it. To be restricted to only one version is too limiting for us right now.
@bernhardmgruber Conan used to be invasing the cmake scripts in the past. Now you can use it completely transparently without modifying the CMakeLists. Otherwise, I would not use it.

@ras0219-msft You don't happen to be this guy: https://www.youtube.com/watch?v=sBP17HQAQjk

Sure, that's him ;)


Trends Research should be Trends & Research

According to the link in the top navigation, the category's name is "Trends & Research", not "Trends Research". This error stems from the mapping document where the ampersand has been omitted.

I fixed the mapping table. The content will be updated during the next import (next Wednesday ?)

I think we should re-import tomorrow so we're ready to move everything to /en/publish first thing on Wednesday.

Done with today's re-import.


Update README.md

test

Codecov Report

Merging #35 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #35   +/-   ##
=======================================
  Coverage   95.76%   95.76%
=======================================
  Files          21       21
  Lines        5217     5217
=======================================
  Hits         4996     4996
  Misses        134      134
  Partials       87       87

Continue to review full report at Codecov.

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


Error when creating a PR

Not sure what this means:

I want the branch to be published to my fork, which it did, and the PR to go to the upstream repo, but it seems that it can't find it. My remotes look like this:

@roblourens Was this with the latest version of the insiders extension? I made a fix yesterday that addressed a similar problem (https://github.com/microsoft/vscode-pull-request-github/pull/1296)

Sorry, I'm not using the nightly extension right now because then the Rich Code Navigation extension complains.

@roblourens - does Rich Code Navigation prompt you to download the stable version of the extension? It just should work with nightly as well, if that would work better for you.
cc @jepetty

Yes it just prompts me to install the stable version

Closing since I believe this was fixed with https://github.com/microsoft/vscode-pull-request-github/pull/1296, I'm not able to repro it. Let me know if you see it again


Fix recognize complete event construction through two new events

Contributing to the Azure SDK
Add two new events: 1. RecognizeChoiceCompleted 2. RecognizeDtmfCompleted for manual contructions of recognize completed event. Since the recognizecomplete event  only include the recognizeResult which was an abstract class, it is not allowed to serialize this manual constructed class, and only able to be deserialized from the incoming json.
The two new events would help customer to construct the recognize completed event with specific recognize result.
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.Communication.CallAutomation


Update meeting-policies-audio-and-video.md

Under TAC, the "disable" value should be "Not enabled", then make this doc align with TAC's value.

@CarolynRowe PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


[Lesson03] Exercises questions

I have 2 questions about the exercises of Lesson03 that I'm currently trying to complete, respectively exercise 1 and 3.

Is the local timer the Timer (ARM side) of BCM2835 Peripherals manual p.196 ?

Is the ARM generic timer mentioned here the one mentioned in my previous question or The Generic Timer in AArch64 state of Aarch64-Reference-Manual p. 2321 ?

@bl4ckout31, The local timer described in the BCM2835 Peripherals manual was designed for the old single-core Raspberry Pis - it may still work because of compatibility guarantees. However, I recommend you to use the local ARM timer described in the BCM2836 ARM-local peripherals manual. (I mentioned it here but I will also add this link to the exercise definition)
I am not very familiar with the Generic timer mentioned in the AArch64-Reference-Manual p. 2321  but I will try to investigate what exactly it does and how it is different from the ARM local timer.

@bl4ckout31
Hi there, I want to ask some questions about exercise1 :D
I want to emulate the timer exception handling in QEMU while the standard timer seems not supported (https://www.raspberrypi.org/forums/viewtopic.php?t=195565)
So can you tell me how you did it? Did you use a real machine or something else?
thanks a lot!

@f0rm2l1n I used the RPi 3 local timer for the real hardware and the ARM generic timer for QEMU.
I don't remember much to be honest, that's what a gathered from reading my code.
Maybe the local timer is working now with QEMU 4.x, I'm gonna try.

@f0rm2l1n After some testing, it looks like QEMU now implement the RPi 3 local timer with the same behavior, at least on version 4.1.0. For this exercice, the same code apart from boot.S and linker.ld should work on both the real hardware and QEMU.


DM/Openapi: ignore can not connect error when force delete task

What problem does this PR solve?

Issue Number: close #5363
What is changed and how it works?
add check when delete task.
Check List
Tests

Unit test
Integration test

Release note
None

/run-all-tests

/cc @Ehco1996

please help me take a look, tanks. @GMHDBJD @dsdashun

Codecov Report

Merging #5364 (d4e6208) into master (687e248) will decrease coverage by 0.0898%.
The diff coverage is 52.6616%.

Flag
Coverage Δ

cdc
61.0650% <52.6616%> (+0.4664%)
:arrow_up:

dm
52.1201% <ø> (-0.3495%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
@@               Coverage Diff                @@
##             master      #5364        +/-   ##
================================================
- Coverage   56.1240%   56.0341%   -0.0899%
================================================
  Files           522        528         +6
  Lines         65325      69438      +4113
================================================
+ Hits          36663      38909      +2246
- Misses        25094      26819      +1725
- Partials       3568       3710       +142

/merge

/run-dm-integration-test

/run-dm-integration-test

/run-dm-integration-test

/merge

/run-dm-integration-test

/run-dm-integration-test

/run-leak-test


update azdata dependency for schema compare and sql database projects

This PR fixes #18071

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 48.276%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
14748

Relevant Lines:
29008

💛  - Coveralls


Reboot

Hi there,
Is it possible to reboot the module over an API?
Thank you

The general rule of thumb is if you cannot do it from the app, then we also cannot do it from the python library, and thus also cannot do it from Home Assistant.
The GraphQL schema can be found in the pybirdbuddy project. I do not see anything related to reboot, restart, or reset (other than "factory reset").
We can toggle the Off-Grid setting, but that's all we can do. And there are some actions that will indirectly result in the device restarting, such as applying a firmware update - but we can only initiate an update if one is available.


Short days before holiday

Is there any way or any plans to add work hours to specific dates? In our country, we have short days ( usually minus 1 hour) in a day before each holiday.

Hey, @abonec!
You're actually in luck. If I'm understanding you correctly, then the PR to do just this is close to being merged: https://github.com/zendesk/biz/pull/109


Practical examples

Looks very promising! But I'm new to NixOS and also to nftables, so I'm having a hard time putting it to use.
In my case, I would appreciate an example for an internet server with several external IPv4 and v6 IPs and several NixOS containers providing services to the outside world and to each other.  🙂

Hi there, sorry for the wait. I have just finished writing a small quickstart guide.
Also the checks are some example configurations that get tested against some pinned nft ruleset. So it might give you a sense of what is generated, if you supply some configuration.
As I actually use it, there is also some examples somewhere in my nix configurations but I tend to not put my firewall configuration in a single spot, but rather add roles on the go in the modules they belong to. You can just search for nftables or firewall in it.
I hope that helps you. I would actually like to improve the documentation further, but sadly I am really short on time.

i'd like to add to the wishlist :slightly_smiling_face: e.g.

redirect a destination port to a different port on localhost
redirect a destination port to a different host on lan
redirect all traffic except for the ssh port to a host on lan

these should be relatively common use-cases and, being new to both nftables and this library, it's not obvious to me how to do this here.
as this is my first post in this repo, thank you for creating this library :raised_hands:


[Bug]: HtmlPlugin conflicts with EslintPlugin options "threads"

System Info
System:
OS: macOS 14.2.1
CPU: (8) arm64 Apple M1 Pro
Memory: 62.55 MB / 16.00 GB
Shell: 5.9 - /bin/zsh
npmPackages:
@rspack/cli: ^0.5.4 => 0.5.4
@rspack/core: ^0.5.4 => 0.5.4
Details
HtmlPlugin conflicts with the threads options of EslintPlugin

Reproduce link
https://github.com/liangyuetian/rsbuild-hmr-slow-demo/blob/htmlPlugin-eslintPlugin/rsbuild.config.ts#L41
Reproduce Steps

pnpm install
npm run dev

I can't understand what exactly the "conflict" means.

I can't understand what exactly the "conflict" means. Please describe your expected behavior and the problems you encounter in more detail.

sorry，I didn't describe it clearly
in htmlPlugin, it is not equal to false and EslintPlugin options When threads coexist, the second compilation will get stuck，as shown in the above figure

seems that there is some bug in jest-worker which used in eslint-webpack-plugin. The javascript main thread keep running in this line https://github.com/webpack-contrib/eslint-webpack-plugin/blob/master/src/getESLint.js#L73, but the lintFiles in worker.js does not been called. Perhaps you can report this to jest-worker or eslint-webpack-plugin for help.

does webpack has this issue?

does webpack has this issue?

I meet the same issue while using webpack-provider

closed since it seems not a rspack bug, feel free to open if you have further question


DSA CHEATSHEET

Merge it under
Hacktoberfest2022

Hacktoberfest

Don't put any random stuff from Internet. Contribute your own documentation, blog, resources.


⚠ Bofrost.lu has degraded performance

In d35562f, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1759 ms

Resolved: Bofrost.lu performance has improved in 9bd771d after 8 minutes.


Updating Asciidoctor version used for Sample generation

This is the first step at modernizing the Asciidoctor version used for our documentation. It update the version only for the sample. For the user manual, the work will be done later.

Let's park this work for now: https://github.com/gradle-guides/gradle-guides-plugin/pull/52


⚠ Bofrost.de has degraded performance

In 5cd1be3, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1201 ms

Resolved: Bofrost.de performance has improved in d8e79fb after 13 minutes.


Get biomedical scenarios working

Support things like MedQA.

One suggestion would be to wrap the BigBIO datasets (GitHub, HuggingFace). It doesn't have MedQA though.


Add contribution documentation to README.md

Hi there!
I love this plugin, thanks for your hard work! I'm really interested in fiddling around with it, and maybe some day contributing back my changes if I think they could be useful to others, too.
To help with adding new contributions, I wrote a few words about how one could contribute to this repository. Please let me know what you feel about this, and if there's anything I could do to improve these further.
Looking forward for more contributions in the future :rocket:

Thanks a lot!
IMHO it's better to have a separate CONTRIBUTING.md and keep readme simple enough.
And a few more comments:

Thanks! I moved the contributing docs to CONTRIBUTING.md in the root of this project — I can also move it under the .github directory if you so wish.
Please let me know of you have further comments :). I think your original message got cut off or accidentally sent before you got to write any more comments..?

No, they're all shown as outdated now:

Oh, just hadn't refreshed GitHub page and I seemed to have missed them

Oh, and it also would be nice to have a note about linting code with ESLint.
Do you know any nice services with Github integration that can inform about lint errors?

I'd like to test out Danger as I don't really like services like HoundCI that spam your pull requests with inline comments.
I've only setup one linter tool for an open source project in the past, and thats phare in https://github.com/venuu/jsonapi-authorization/pull/34. In our work project we just run npm run lint as one of our test commands and it just fails the build silently if the linting fails.
Setting up a lint service could also be done outside this PR. I could tackle it if I get it working easily. What do you think?

Setting up a lint service could also be done outside this PR.

You're right, lets keep this PR clean.

What do you think of the updates I've done after your comments? I'm not 100% sold on the quality of the git command setup, especially concerning the checkout of dev branch, but unless we would expand the "clone and fork" section, it would be quite difficult to figure out what actually is the upstream origin name inside your local git copy.

Well, I would just write something like Use 'dev' branch as PR base and that's it.
I think if one want to contribute to a project hosted on Github he probably already knows how to use it)


Download Python Client as document steps throw errors

Python Client document writes:
# avro serialization
$ pip install pulsar-client[avro]=='2.10.1'

# functions runtime
$ pip install pulsar-client[functions]=='2.10.1'

# all optional components
$ pip install pulsar-client[all]=='2.10.1'

However, all these optional features return not found:
ERROR: Could not find a version that satisfies the requirement pulsar-client[avro]==2.10.1 (from versions: 2.8.0, 2.8.1, 2.9.1, 2.9.2, 2.10.0)
ERROR: No matching distribution found for pulsar-client[avro]==2.10.1

It seems we don't release pypi packages for 2.10.1?
cc @BewareMyPower @codelipenghui

Could you confirm whether the Python client has been released since you're the release manager of 2.10.1? @codelipenghui
BTW, the command should be pip3 install 'pulsar-client[all]==2.10.1' on macOS.

@BewareMyPower I think the quote issue is introduced by zsh syntax. Here is a pull request to adapt the case #16851

The Python Client for macos haven't push yet, I will push it later.

@codelipenghui still not found. Is there any estimate? Since 2.10.1 is an official release we must fix this issue though ><

This can be related to https://github.com/apache/pulsar/issues/17468.

Works for me now. Closing...


Installation issue (using WSL)

Hello, I was trying to install on WSL by following the guide (https://rhasspy.readthedocs.io/en/latest/installation/#windows-subsystem-for-linux-wsl) and ran the make install command, after that I tried to go to http://localhost:12101 but it says that the page isn't working. Is there a step that I'm missing or something I did wrong? I'm new to Linux so my error might be silly, but I've been searching for a solution for hours now. Thanks in advance!

Did you run the rhasspy command after installation? Does it show any error messages?

Thanks for your response. Yes, I did and it says rhasspy: command not found i think I might've not added it to path correctly.

Yes, adding the location of the rhasspy binary to your path should be the first thing to try.

I have many different ways but it says that the command was not found. Is there a command you recommend or some resource you could point me to? That would be highly helpful. Thanks and stay safe!


manual build for raypyc components

I need some help with documentation about the manual build for all raypyc components.
I started working on some windows documentation because that is the only OS I currently have.
if you have any idea how to compile raypyc components for other OS(s) fill free to edit the manual_build.md file.

windows documentation is done

the Linux (ubuntu) documentation is done now


[example/app] Timeline generation

This is just an intuition but if you spend your time on the computer all day, it would be great if you could get a flexible and granular breakdown of how you spent your time. It would be a brand new era in time management, just like that. Dunno if that's out of scope for the project, maybe the UI bit but the backend could be done here I think ?

super interesting
i actually maintain on my own a timeline for my life in excalidraw but that would be interesting to see a day by day timeline generated by AI based on what you do
i'm sure it could be generated directly in excalidraw, i actually did a LLM that can use excalidraw recently: https://github.com/louis030195/excalidraw-gpt
no need to change the code of screenpipe, can be just an external app that query the data and update the timeline periodically
example that come to my mind could be like

code on vscode for 30 min
scroll twitter for 20 min
send message on linkedin 20 min
etc

with more details and represented as a line on excalidraw eventually with links to the actual content if its possible
what kind of timeline would you like to see?

This is really a cool idea. Would like to know if we are thinking to prioritize this?

This is really a cool idea. Would like to know if we are thinking to prioritize this?

what kind of use case would you like?
would you be able to contribute? that would be great 🙂

The initial use case is to see how I am spending my time but gradually it should be able to suggest also where I am wasting time and able to suggest steps as well.
Regarding my contribution, I am native JS stack not python so not sure how much I would be able to. But yeah I will give a look to it for sure.

@bhupesh-sf thanks! we don't use python
you can look at example, i'd recommend trying to run one of these first and iterate from there

sure, coming 2 weeks are busy after that I will check this out and will be happy to contribute anyways I can.

@bhupesh-sf @oxysoft any news? would love feedback, we shipped plenty of new stuff!

Hey @louis030195 , I apologise for the late response—I've been busy with projects lately. I'm glad to hear that the progress has been going well!
As for contributing, since I'm still new to Rust and my schedule is pretty hectic, I was thinking of focusing on the JS/TS stack instead. I noticed the roadmap mentioned a TypeScript SDK. If you could point me in the right direction, I'd love to take that on. I'm well-versed in the JS/TS stack, so I can get up to speed quickly. Or, if there's anything else in this stack that needs attention, I'd be happy to help out whenever I have some free time.

@bhupesh-sf you don't need to write rust, there is an API you can interact with and we have some examples to run JS or TS inside of the CLI or app, we have a JS runtime that can execute your code inside it:
https://github.com/mediar-ai/screenpipe/tree/main/examples/typescript/pipe-tagging-activity
for example you could do things like:

check if there are security risk on the user screen by querying the API every min and asking ollama model if there is a risk in the OCR text
writing logs of what the user is doing
etc

let me know if you need help on this or join the discord: https://discord.gg/ywCgfGwD


Tag Releases on GitHub

Whenever we make a release on PyPi, Conda, etc. for the python package, we should also tag the release on GitHub so that if things are broken, code can be compared across versions easily.

Yes!!!!

It also helps if the documentation for the theme changes between releases.

Closing since we're doing this now ;-)


Bulk operation csv for new employees

I want to do a csv with groups ou name last name alias password send mail to private email
Does some one already have a good example I can use ?

Full name any thing you can think of


[checkpointio]support asyncio for 3d

📌 Checklist before creating the PR

[ ] I have created an issue for this PR for traceability
[ ] The title follows the standard format: [doc/gemini/tensor/...]: A concise description
[ ] I have added relevant tags if possible for us to better distinguish different PRs
[ ] I have installed pre-commit: pip install pre-commit && pre-commit install

🚨 Issue number

Link this PR to your issue with words like fixed to automatically close the linked issue upon merge
e.g. fixed #1234, closed #1234, resolved #1234

📝 What does this PR do?

Summarize your work here.
if you have any plots/diagrams/screenshots/tables, please attach them here.

💥 Checklist before requesting a review

[ ] I have linked my PR to an issue (instruction)
[ ] My issue clearly describes the problem/feature/proposal, with diagrams/charts/table/code if possible
[ ] I have performed a self-review of my code
[ ] I have added thorough tests.
[ ] I have added docstrings for all the functions/methods I implemented

⭐ Do you enjoy contributing to Colossal-AI?

[ ] 🌝 Yes, I do.
[ ] 🌚 No, I don't.

Tell us more if you don't enjoy contributing to Colossal-AI.

checkpoint benchmark
model size: 7b gpus numbers: 4; tp=2 pp=2
use asyncio: True

use asyncio: False


Remove freakattack.com tests since they have dissapeared

Fixes #293

CircleCI build fixed in #299.

Thanks! 👍


Fix #805 don't blur() the trigger after a clipboard action

What kind of change does this PR introduce? (check at least one)

[x] Bugfix
[ ] Feature
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[x] Yes
[ ] No

If yes, please describe the impact and migration path for existing applications:
This PR reintroduces the behavior that used to be in clipboard.js following https://github.com/zenorocha/clipboard.js/pull/419 but that has since been reverted for some reason. As such, after a clipboard action, focus will remain/be on the trigger element, rather than the body. If applications rely on the focus to be gone/removed from the trigger button, they'll need to adapt to this.
The PR fulfills these requirements:

[ ] It's submitted to the dev branch for v2.x (or to a previous version branch), not the master branch
[x] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[x] New/updated tests are included

I didn't target v2.x branch because it looks wildly out of date.
Other information:
I'm not too hot on writing tests, so rather than just removing the check to see if focus is now on the body, it should probably explicitly check that focus remained on the trigger. However, not quite sure how to do that - my knowledge of test harnesses etc is pretty basic.
Closes #805

wonderful! thank you.


Desktop notifier in Windows that displays a joke

This script will display a joke from the PyJokes package on Windows as a desktop notification with an interval of 30 minutes.

Issue solved by @Namyalg in pull request #4


feat: add IncomingRequest::is() method

Description
Writing code like strtolower($this->request->getMethod()) !== 'post' is a pain.

add IncomingRequest::is()

$request->is('get')
$request->is('post')
$request->is('put')
$request->is('delete')
$request->is('head')
$request->is('patch')
$request->is('options')
$request->is('json')
$request->is('ajax')

Checklist:

[x] Securely signed commits
[x] Component(s) with PHPDoc blocks, only if necessary or adds value
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[x] Conforms to style guide

public function isMethod(string $method) : bool {}
public function isGet() : bool {}
public function isPost() : bool {}
// e.t.c.

the default method name is in uppercase.
The fact that in the framework the method name is in lowercase is the wrong way.

The fact that in the framework the method name is in lowercase is the wrong way.

What do you mean? Why is lowercase wrong way?

I think @iRedds was pointing out that we only need this method because we are inconsistent with our case in the getMethod() return. The $upper parameter is already deprecated which means we are currently forcing lowercase method names. However if we want to go PSR-7 we cannot modify the case:
     * While HTTP method names are typically all uppercase characters, HTTP
     * method names are case-sensitive and thus implementations SHOULD NOT
     * modify the given string.

I'm not sure the best approach here. I am not opposed to this helper method, but I also believe it should be unnecessary.

The lowercase/uppercase "problem" was indeed created by PSR-7 compatibility. I don't see a way to resolve it without introducing quite a big BC break because, by default, everything is in lowercase now (and was the same way in CI3).

Because "the default method name is in uppercase."

The method token is case-sensitive because it might be used as a gateway to object-based systems with case-sensitive method names. By convention, standardized methods are defined in all-uppercase US-ASCII letters.
https://www.rfc-editor.org/rfc/rfc9110#name-overview

It seems method names should be case-sensitive.
The getMethod() returns lowercase now.
So strtolower($this->getMethod()) === $value; is redundant.
But the default $upper = false in getMethod() will be removed.

@iRedds Do you say that the standard HTTP method names are in uppercase?
I changed the implementation a bit.
$ curl -X get https://www.w3.org/ -i
HTTP/2 400
server: cloudflare
date: Wed, 21 Dec 2022 08:40:47 GMT
content-type: text/html
content-length: 155
cf-ray: -

<title>400 Bad Request</title>

400 Bad Request

cloudflare

I think case-sensitivity is in our favor. We don't need to adjust the case on the method return because Get is not GET. Forcing these to lower does appear to be a violation of the spec; maybe we add a new method for now?

The lowercase/uppercase "problem" was indeed created by PSR-7 compatibility. I don't see a way to resolve it without introducing quite a big BC break because, by default, everything is in lowercase now (and was the same way in CI3).

Yes, the current getMethod() returns values that are converted to lowercase, and it seems a violation of the HTTP specification.
The behavior should be fixed but it is a big BC break. So it is better that we wait for v5.0.
@MGatner This is() method is the new method to check HTTP methods in case insensitive manner.

The behavior should be fixed but it is a big BC break. So it is better that we wait for v5.0.

I agree with this.
is() may serve as a handful helper method, and since it handles json and ajax it's not 100% redundant with getMethod().

Rebased and resolved, and all green!

Added docs.

next major version?

@MGatner I'm not sure if this label was meant to be added to this PR, but if so, I would have second thoughts about this PR. v5 means that we already have the getMethod() fixed, so is() method wouldn't make as much sense to me.
Then, I would rather add the isJSON() method.

@michalsn I've been using that label for PRs or threads I want to come back and review for v5 content. It doesn't necessarily mean "this PR belongs in v5" (though sometimes) but more like "read this thread for a relevant v5 idea".

Rebased and pushed two commits.

Then, I would rather add the isJSON() method.

I found isJSON() method. It was deprecated in 4.0.5.
See https://github.com/codeigniter4/CodeIgniter4/pull/7019/files#diff-2cab244a084ce08ac666dff9a88aee13003b4170f6c02a690263e3260dac26d8R35
And it was removed in 4.1.2.
See https://github.com/codeigniter4/CodeIgniter4/commit/55b43083

Well... it's strange to me why we deprecated isJSON()... and now we're, in fact, introducing it again. I tried to follow the related discussions, but I still have no point. Anyway, I don't mind it at all. This is a common functionality that will make the life of developers easier. At least, in my opinion.

I think isJSON() was removed when we were trying to establish PSR-7 compatibility, but before we split framework-specific methods out into traits. At least that's my best recollection. It's convenient, I have no problem with it, but it is one more framework-coupling method that doesn't save on lines of code or much readability.


Postgres error thrown when using advised UUID ID field

Hi all, thank you very much for your work on the Postgres driver for Vapor! I'm running into issues with a Postgres 13.0 (latest from homebrew) database, and Vapor, while using UUID id's.
I declared the ID field as such:
@ID(key: .id)
var id: UUID?

With a migration that uses:
.field("id", .uuid, .identifier(auto: true))

The Postgres driver is setup like this:
app.databases.use(.postgres(
    hostname: Environment.get("DATABASE_HOST") ?? "localhost",
    username: Environment.get("DATABASE_USERNAME") ?? "postgres",
    password: Environment.get("DATABASE_PASSWORD") ?? "",
    database: Environment.get("DATABASE_NAME") ?? "mobile_backend"
), as: .psql)

app.migrations.add(UserMigration())
app.migrations.add(TokenMigration())
app.migrations.add(RuleMigration())

try app.autoMigrate().wait()

Which results in the error: [ ERROR ] identity column type must be smallint, integer, or bigint (init_params) Fatal error: Error raised at top level: previousError(server: identity column type must be smallint, integer, or bigint (init_params)): file /AppleInternal/BuildRoot/Library/Caches/com.apple.xbs/Sources/swiftlang/swiftlang-1<IP_ADDRESS>/swift/stdlib/public/core/ErrorType.swift, line 200

@jbehrens94 This happens because you are trying to use and auto-incrementing identifier of UUID type (.identifier(auto: true)). If you set auto: false it should work.
By the way, you could simplify the property like so:
@ID
var id: UUID?

and the migration:
.id()


[Go Live][Jan 25]

Push to live

Docs Build status updates of commit e2097f2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/zone-pivot-groups.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


AboutUs.md: Change http -> https

As per https://github.com/nus-cs2103-AY2324S2/forum/issues/850

Fixed by #27. Not sure why the issue and PR aren't linked directly when the PR was made


Bug: Copying comment url copies invalid url

In any story for example https://www.hckrnws.com/stories/39079256, if one tries to copy any comment url we get VERCEL_URL/stories/<comment_id>.
I think you can fix this from environment variables exposed via vercel panel if you are using vercel. If not then I think specifying the required variable in next.config.js will fix this bug.

Had to fix the variable in the vercel panel. Now works fine!
https://github.com/rajatkulkarni95/hckrnws/assets/57321156/c3246861-7826-45cb-b8c5-8a1d2f65dafc
Thanks for the report


Prevent sending push notifications to mobile devices if activity on desktop client device

At now Riot mobile apps play sound notifications always on new messages, so if I actively use desktop Matrix client - my device rings very often, so I must turn notifications off, and not forgot to give back it on when I go away from desktop computer, and do this multiple times per day.
For solve this problem - will be good to detect user activity on desktop device client, and suppress (or delay) sending push notifications to mobile devices if activity on desktop device is detected some time ago.

https://github.com/vector-im/riot-web/issues/469


Show current speed of car

Acceptance criteria

[ ] Display the current speed of the car on the application.
[ ] Send the speed of the car from the emulator to the application.
[ ] Get the speed of the car from the mulator.

This task already exists as a user story.


[#608] Don't use exit() on duplicate test descriptions

Instead of exit(1), it now throws std::runtime_error with the details
of the failure. This exception is handled in run() at a higher level where
the log is printed to cerr and the test gracefully exits.

:+1:

@philsquared Could you merge this? It's become a real pain to debug duplicate-named test cases and sections, since the application just aborts. This makes it at least possible to break on exception in Visual Studio and the like. It also gives helpful output on command line.

Phil never was quick to get to issues, was he

Phil never was quick to get to issues, was he

Phil has a rather full life - fixing Catch isn't always right at the top of his priority list.

Guilty as charged!
It's worth the bump as I do try and dip in from time to time even if I'm busy elsewhere (currently preparing to give a course on Swift!). Thanks for the defence, Andy :-)
Anyway, this one looks pretty straightforward and a win-win all round, so I've merged the PR in now: look for build 1.5.3 - and thanks for the PR!

Thanks @philsquared !! Good luck on your other endeavors!


Apply fix to part1/example6/webpack.config.dev.js to enable HMR

This issue is referenced here: https://github.com/webpack/webpack-dev-server/issues/326
Signed-off-by: Ian Hilt<EMAIL_ADDRESS>

Hey appreciate the pull request. Could you explain the reasoning behind this to me? I read through the issue threads, and from what I saw, the issue was closed and a related pull request was merged.
Am I missing something that implies this is still a problem?

Ah, so maybe ipv6 addresses aren't supported? I had ::1 in my hosts file but not <IP_ADDRESS>. After I added <IP_ADDRESS> localhost it started working.


Parsing existing Xcode project

This might be a moot request for a project devoted to project generation, though I'll ask the question anyway.
Is there a way to go in reverse, and parse an existing Xcode project to a spec file? Or something at least close to it? Any recommendations? If manual labor is the only answer, would you happen to have any resources to share?

Hi @drkibitz. That issue is tracked in #9, so I'll close this one.
At this point crafting the spec by hand is the only way. A few people have been asking for this recently so I’ll see if I can implement a basic version of this soon. PRs welcome.
Migrating all the possible mixed sources is tricky, but we can at least generate all the targets, configurations, and build setting as a starting point


Merge of IconExtractor Branch

Merge IconExtractor branch so far.
It's not working currently, but anyways.

Won't merge. Just minor unfinished development


Max encoding length + corresponding tests

The encoding function eases the encoding of sequences across tokenizers. The addition of the head_mask return further removes the pressure on the user to manually check the added special tokens.
There is currently no easy method to truncate the encoded  sequences while keeping the special tokens intact. This PR aims to change this by providing a max_length flag to be passed to the encoding function. This flag works even when no special tokens are involved (e.g. for GPT-2).
The second sequence is truncated while the first stays intact. If the first sequence is longer than the specified maximum length, a warning is sent and no sequences are truncated.

Codecov Report

Merging #1252 into glue-example will increase coverage by 0.08%.
The diff coverage is 90.9%.

@@               Coverage Diff               @@
##           glue-example   #1252      +/-   ##
===============================================
+ Coverage         81.32%   81.4%   +0.08%
===============================================
  Files                57      57
  Lines              8074    8104      +30
===============================================
+ Hits               6566    6597      +31
+ Misses             1508    1507       -1

Impacted Files
Coverage Δ

...h_transformers/tests/tokenization_tests_commons.py
100% <100%> (ø)
:arrow_up:

pytorch_transformers/tokenization_utils.py
90.02% <80%> (+0.57%)
:arrow_up:

Continue to review full report at Codecov.

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


[Suggestion] Disable infinite water source in Reservoir

My suggestion is very simple.
Some people (Like me) want to create hard to play modpacks, and there should be a boolean somewhere (For example in config file) to allow people disable infinite water source in reservoir (After filling with 2 buckets). I want to create modpack without infinite water sources. I'm able to disable minecraft's infinite water sources (Using other mods), but I'm not able to disable infinite water in reservoir.

The config flag is named "reservoirEnabled".


Error:  Failed to find where HEAD diverged from undefined. Does undefined exist?

Problem with<EMAIL_ADDRESS>Newly introduced version 2.27.2 causes problems with the same setup as before (even after a clean NPM install with removed lock file and node_modules folder):
> changeset status --verbose
🦋  error Error: Failed to find where HEAD diverged from undefined. Does undefined exist?
🦋  error     at getDivergedCommit (/builds/.../node_modules/@changesets/git/dist/changesets-git.cjs.js:66:11)
🦋  error     at async getChangedFilesSince (/builds/.../node_modules/@changesets/git/dist/changesets-git.cjs.js:198:22)

Problem
Newly introduced version 2.27.2 causes problems with the same setup as before (even after a clean NPM install with removed lock file and node_modules folder):

2.27.1 is good . 😅

Same, here is the workflow step that shows the error: https://github.com/Sec-ant/zxing-wasm/actions/runs/9153151340/job/25161639566#step:4:10


New feature- support select second under the time type

RT

RT

Is this feature to display/select seconds only for time type?

RT

Is this feature to display/select seconds only for time type?

yes

@shingohu
Hi! I added the feature to select seconds in 2.1.0.
Add an option that is only valid if DateTimePickerType.time is specified. It will work if you specify the parameters as follows.
showBoardDateTimePicker(
  context: context,
  pickerType: DateTimePickerType.time,
  options: BoardDateTimeOptions(
    withSecond: true, // <- added option
  ),
);


rendering not working on macOS for version 2.1.1

Following the readme steps (pip install from last commit) the library can be install, however it can not render despite glfw being installed and the library path being set. Running the sample script:
from dm_control import suite
from dm_control import viewer

env = suite.load(domain_name="humanoid", task_name="stand")
viewer.launch(env)

produces the following error:
dm.py 27 <module>
viewer.launch(env)

__init__.py 39 launch
app = application.Application(title=title, width=width, height=height)

application.py 187 __init__
self._window = gui.RenderWindow(width, height, title)

__init__.py 34 ErrorRenderWindow
raise ImportError(

ImportError:
Cannot create a window because no windowing system could be imported

Using python 3.9, intel-based Mac.
In the same setup, glfw can render mujoco environments using mujoco-py with mujoco version 2.1.0. So glfw installation seems to be fine. (mujoco-py is also broken for version 2.1.1, so right now there is no way of using mujoco on mac with version 2.1.1.).

Thanks for reporting. This appears to be caused by my commit 73c31ac21d1214f0a2eadfc8540a7f76796955cc rather than a breakage in MuJoCo 2.1.1. I'll work on reverting the offending commit now.

Thanks @saran-t , this seems to have fixed rendering GLFW issues on my mac :-)!

@saran-t I'm still getting the same initial issue with the new mujoco installation via pip 😢


tidb_query: bug fix for vectorized casting real as time

What problem does this PR solve?
Issue Number: close https://github.com/pingcap/tidb/issues/19609
Problem Summary:
keep same impl as https://github.com/pingcap/tidb/pull/19485
What is changed and how it works?
What's Changed:
Related changes

Need to cherry-pick to the release branch

Check List
Tests

Integration test

Side effects

n/a

Release note

No release note

This change is

/rebuild

/run-integration-copr-test

/run-integration-copr-test

There seems no unit test covering it?

/merge

/run-integration-common-test
/run-integration-ddl-test

Assertion fail:
[2020-08-31T09:06:00.296Z] [2020/08/31 17:04:35.276 +08:00] [FATAL] [lib.rs:481] ["assertion failed: `(left == right)`
  left: `4`,
 right: `8`"] [backtrace="stack backtrace:
   0: tikv_util::set_panic_hook::{{closure}}
   1: std::panicking::rust_panic_with_hook
             at src/libstd/panicking.rs:524
   2: rust_begin_unwind
             at src/libstd/panicking.rs:431
   3: std::panicking::begin_panic_fmt
             at src/libstd/panicking.rs:385
   4: <tidb_query_vec_executors::table_scan_executor::BatchTableScanExecutor<S> as tidb_query_vec_executors::interface::BatchExecutor>::next_batch
   5: <tidb_query_common::execute_stats::WithSummaryCollector<C,T> as tidb_query_vec_executors::interface::BatchExecutor>::next_batch
   6: tidb_query_vec_executors::runner::BatchExecutorsRunner<SS>::internal_handle_request
   7: <minitrace::future::TraceWrapped<T> as core::future::future::Future>::poll
   8: <tikv::coprocessor::interceptors::tracker::Tracker<F> as core::future::future::Future>::poll
   9: <core::future::from_generator::GenFuture<T> as core::future::future::Future>::poll
  10: <core::future::from_generator::GenFuture<T> as core::future::future::Future>::poll
  11: <yatp::task::future::Runner as yatp::pool::runner::Runner>::handle
  12: std::sys_common::backtrace::__rust_begin_short_backtrace
  13: core::ops::function::FnOnce::call_once{{vtable.shim}}
  14: <alloc::boxed::Box<F> as core::ops::function::FnOnce<A>>::call_once
             at /rustc/16957bd4d3a5377263f76ed74c572aad8e4b7e59/src/liballoc/boxed.rs:1078
      <alloc::boxed::Box<F> as core::ops::function::FnOnce<A>>::call_once
             at /rustc/16957bd4d3a5377263f76ed74c572aad8e4b7e59/src/liballoc/boxed.rs:1078
      std::sys::unix::thread::Thread::new::thread_start
             at src/libstd/sys/unix/thread.rs:87
  15: start_thread
  16: __clone
"] [location=/rustc/16957bd4d3a5377263f76ed74c572aad8e4b7e59/src/libstd/macros.rs:16] [thread_name=unified-read-pool-1]

/run-integration-common-test


How to handle GHC beta versions?

With
  - env: BUILD=cabal-new CABALVER=2.2 GHCVER=8.6.1
    addons: {apt: {packages: [cabal-install-2.2,ghc-8.6.1], sources: [hvr-ghc]}}

I get
Found /opt/ghc/8.6.1/bin/ghc version [<IP_ADDRESS>80823] expecting [8.6.1]
Mismatching ghc version found at [/opt/ghc/8.6.1/bin], removing it from PATH and trying again
Found ghc version [<IP_ADDRESS>80823] expecting [8.6.1]
Error: Wrong ghc version [<IP_ADDRESS>80823] expected [8.6.1]

However if I would specify GHCVER=<IP_ADDRESS>80823, the build would break once hvr releases a new beta version.
Not sure if there's a good way to handle this.

What if you specify GHCVER=8.6? It would match 8.6.*, so that should work I guess.

It would be nice if packcheck could work like this, but it currently doesn't. It seems like packcheck constructs ghc's executable name from GHCVER:
Error: ghc 8.6 not found in PATH [/home/travis/.local/bin:/home/travis/.cabal/bin:/opt/cabal/2.2/bin:/opt/ghc/8.6/bin:/home/travis/.rvm/gems/ruby-2.4.1/bin:/home/travis/.rvm/gems/ruby-2.4.1@global/bin:/home/travis/.rvm/rubies/ruby-2.4.1/bin:/home/travis/.rvm/bin:/bin:/usr/bin]

It does work like that, I just tested it. It may be a problem specifically when using the TOOLS_DIR option perhaps. Can you try passing the path of the ghc 8.6 binary in PATH= environment variable?

Sorry, I probably wasn't quite clear: There is no ghc-8.6 executable (link). There's ghc-8.6.1 and there's something like ghc-<IP_ADDRESS>80823.
I suspect that packcheck can't find either of them given GHCVER=8.6.

Oops, sorry for mistrusting packcheck there! :)
After defining PATH as
  - PATH=/bin:/usr/bin:/opt/ghc/bin:/opt/ghc-ppa-tools/bin

it did find the right compiler! Nice work! :)
I'm wondering though whether packcheck should add these directories to PATH by default?!
haskell-ci also adds $HOME/local/bin but I wasn't sure whether that would work with Travis' env configuration.

The problem may be that when finding ghc binary in the hvr ppa dir tree it matches only the exact compiler version while when finding in PATH it matches the prefix. I will try investigating and fixing it whenever I get some time. Until then you can leave this issue open.

Currently it uses the TOOLS_DIR env var to specify the hvr ppa and then searches for ghc in ${TOOLS_DIR}/ghc/${ver}*/. Maybe I should drop using that mechanism and just ask the users to specify /opt/ghc/bin in PATH. Will that always work for hvr ppa? Or are there cases when we should use ${TOOLS_DIR}/ghc/${ver}*/ instead? I think I added mechanism this because of a problem you faced once in using the hvr ppa, not sure if that could have been resolved using the PATH instead.
I might consider adding this to default PATH, but generally I don't want to have such defaults as it may result in surprises, explicit specification by user always works best. We can add it as default in the CI config files though.

From my local Ubuntu machine:
$ ls /opt/ghc/bin/
cabal
ghc
ghc-7.0.4
ghc-7.10.3
ghc-7.2.2
ghc-7.4.2
ghc-7.6.3
ghc-7.8.4
ghc-8.0.2
ghc-8.2.2
ghc-8.4.1
ghc-8.4.2
ghc-8.4.3
ghc-<IP_ADDRESS>80823
ghc-8.6.1
ghc-8.7
ghc-8.7.20180827
ghc-head
ghci
ghci-7.0.4
ghci-7.10.3
ghci-7.2.2
ghci-7.4.2
ghci-7.6.3
ghci-7.8.4
ghci-8.0.2
ghci-8.2.2
ghci-8.4.1
ghci-8.4.2
ghci-8.4.3
ghci-<IP_ADDRESS>80823
ghci-8.6.1
ghci-8.7
ghci-8.7.20180827
ghci-head
ghc-pkg
ghc-pkg-7.0.4
ghc-pkg-7.10.3
ghc-pkg-7.2.2
ghc-pkg-7.4.2
ghc-pkg-7.6.3
ghc-pkg-7.8.4
ghc-pkg-8.0.2
ghc-pkg-8.2.2
ghc-pkg-8.4.1
ghc-pkg-8.4.2
ghc-pkg-8.4.3
ghc-pkg-<IP_ADDRESS>80823
ghc-pkg-8.6.1
ghc-pkg-8.7
ghc-pkg-8.7.20180827
ghc-pkg-head
haddock
haddock-ghc-7.0.4
haddock-ghc-7.10.3
haddock-ghc-7.2.2
haddock-ghc-7.4.2
haddock-ghc-7.6.3
haddock-ghc-7.8.4
haddock-ghc-8.0.2
haddock-ghc-8.2.2
haddock-ghc-8.4.1
haddock-ghc-8.4.2
haddock-ghc-8.4.3
haddock-ghc-<IP_ADDRESS>80823
haddock-ghc-8.6.1
haddock-ghc-8.7
haddock-ghc-8.7.20180827
haddock-ghc-head
runghc
runghc-7.0.4
runghc-7.10.3
runghc-7.2.2
runghc-7.4.2
runghc-7.6.3
runghc-7.8.4
runghc-8.0.2
runghc-8.2.2
runghc-8.4.1
runghc-8.4.2
runghc-8.4.3
runghc-<IP_ADDRESS>80823
runghc-8.6.1
runghc-8.7
runghc-8.7.20180827
runghc-head

$ ls /opt/ghc/8.6.1/bin/
ghc
ghc-<IP_ADDRESS>80823
ghci
ghci-<IP_ADDRESS>80823
ghc-pkg
ghc-pkg-<IP_ADDRESS>80823
haddock
haddock-ghc-<IP_ADDRESS>80823
hp2ps
hpc
hsc2hs
runghc
runghc-<IP_ADDRESS>80823
runhaskell


URL parts appended twice

Hi,
On the Owner page, after editing a pet (or adding a visit) the URL of "Edit pet" (or "Add visit") is wrong, e.g. "http://localhost:8080/owners/5/pets/6/5/pets/6/edit"
kind regards
geert

See also #3.


Reproduce the bug

Expected behaviour
Error for undefined name 'foo' points at 'f' of 'foo' (the 19th character on the line in the example below)
Actual behaviour
Error points at first 'o' instead (the 20th character)
Steps to reproduce:
Configure both Pylance and mypy. Enter this code in a .py file:
def f(arg: int) -> int:
return (arg + foo + 1)
Hit F8 once to show the Pylance error pointing at 'f', and again to show the mypy error pointing at '

Spend a hour trying to reproduce, couldn't get mypy working properly. Might be something wrong with my python installation.

Tried to reproduce. Got only this message. Pylance and mypy should be ok.

Reproduced bug.

Reproduced bug.


Add drop shadow for cells in categories map filter.

As per design https://www.figma.com/file/DT4GPi0WP1h56IxjV9qk6t/Radzima?node-id=2841%3A3446.


[SERV-464] Update security issues, fix tests

Update Alpine image version
Update Vert.x version
Update ItemsHandler as a result of the Vert.x version update
Remove the mid-method return in ItemsHandler
Fix asynchronous asserts in tests
Get rid of TestConstants class; use freelib-utils' constant instead
Eclipse sorted some import statements

Marked as draft just to do some sanity checks to make sure I didn't mangle anything with the other PR merge.

Made this a draft again because I forgot to add an explanation about the cookie format and the link to the Hyrax code in the README and test configs in the POM. Will do this in the AM.


Rename library entry point

#205 introduced a regression when the entry point file was renamed: lib/ruby-saml.rb → lib/onelogin/ruby-saml.rb. This is causing problems with bundler, and also when integrating with other libraries such as omniauth-saml.

:+1:

@pitbulk this PR failed travis FYI, should ALWAYS wait until we're sure we're not merging a breaking change!
thanks for this fix @phlipper

@Lordnibbler, sorry, but It seems a Travis issue.


Loop optimisation

I've been trying to optimise the code for sequential recombination algorithm in Julia and spotted an interesting fact.
Below you can see a very simple piece of code that is basically a translation of the C++ FastJet code. The profiler shows that lines that only have nndist = Δ2 in them take up as much time as really computationally heavy lines like Δ2 = _dist(i, j, _eta, _phi). I checked the stack that the profiler throws out and it's actually calling the == operation inside of the iterate() function that causes that. So this probably comes not from the assignment itself but from the end of the loop iteration. Simple loops typically get optimised but here it's something strange.
I used Profile.jl and profiled the entire algorithm in a loop. This function is called from there frequently and only exists to be compiled to avoid dynamic dispatch in the main algorithm.
# src/Algo.jl @ line 45
function _upd_nn_nocross!(i::Int, from::Int, to::Int, _eta, _phi, _R2, _nndist, _nn)
    nndist::Float64 = _R2
    nn = i
    Δ2::Float64 = 0.0
    for j in from:(i-1)
        Δ2 = _dist(i, j, _eta, _phi)
        if Δ2 <= nndist
            nn = j
            nndist = Δ2 # too long for an assignment
        end
    end
    for j in (i+1):to
        Δ2 = _dist(i, j, _eta, _phi)
        if Δ2 <= nndist
            nn = j
            nndist = Δ2 # too long for an assignment
        end
    end
    _nndist[i] = nndist
    _nn[i] = nn
    nothing
end

can you confirm this is a representative workload?
julia> const input2 = loadjets("./test/data/11.dat");

julia> f() = for _=1:10 anti_kt(input2, R=0.7) end;

julia> @time f()
  0.549266 seconds (72.79 k allocations: 8.631 MiB)

julia> @benchmark f()
BenchmarkTools.Trial: 10 samples with 1 evaluation.
 Range (min … max):  544.617 ms … 554.028 ms  ┊ GC (min … max): 0.00% … 0.00%
 Time  (median):     546.893 ms               ┊ GC (median):    0.00%
 Time  (mean ± σ):   548.198 ms ±   3.530 ms  ┊ GC (mean ± σ):  0.04% ± 0.12%

  █   ▁      ▁ ▁ ▁       ▁                      ▁       ▁     ▁
  █▁▁▁█▁▁▁▁▁▁█▁█▁█▁▁▁▁▁▁▁█▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁█▁▁▁▁▁▁▁█▁▁▁▁▁█ ▁
  545 ms           Histogram: frequency by time          554 ms <

 Memory estimate: 8.63 MiB, allocs estimate: 72790.

julia> @profview f()

Yes, should be.

I am confused: nndist = Δ2 # too long for an assignment is taking so long? Δ2 is coming from dist(), is that guaranteed to be a Float64 (which nndist is)?


Viewport prioritization

[5:01 PM] blakeman: fucking badass
[5:01 PM] blakeman: I tried the tile spiralling thing once a long time ago and it was a fun challenge
[5:01 PM] blakeman: never completed the server tho
[5:02 PM] blakeman: do you have any prioritization functionality, like say if there's 255 players near me and someone farcasts me in wildy, will the engine show them to me?
[5:02 PM] blakeman: I never did that but wanted to

No longer applies as there's no limit to the number of local players. Could conceivable be done for prioritising appearance updates but again not the most important thing, would need to be hundreds of players in one place for this to be abused


Fixes issue #3454

Fixed "Directive prologues are not preserved"
Fixes issue #3454

By the way @ShyykoSerhiy, when I suggested you format your issues in the way that I did, I meant for the big text to go into the title area.

took care of the comments locally. thanks @ShyykoSerhiy for the PR.


Additional services email #2

Subject ENG: E-commerce Berlin Expo 22: Additional services reminder
Subject DE: E-commerce Berlin Expo - Erinnerung an zusätzlichen Standdiensten
additional-services-emails-#2.zip
Please let me know if the files are prepared correctly. The first paragraph introduces the account manager - I'm unsure if you can fill in the AM name using some kind of a "snippet". If you cannot do it this way I will be happy to prepare 3 separate files.
Also, due to German terms, there are two separate files to be sent in DE - one for DK+GY and one for MM companies.

@jprzybylski4
world seems a better place if keep simple things ....simple?
only two files can be processed (language is the criteria, right? )
re account manager -there is only a placeholder {{$footer}} for auto pasting name and contact data (so i do advise to point to the footer and refrain from poetry in the invocation :)
please fix {{$profile->translate("[[fname]]") to {{ array_get($profile, "fname") }} as translate is not used since over a yr

Please find the updated files attached
additional-services-emails-#2-fixed.zip
s

thx. scheduled for TUE 10-12.


Update ploi deploy script

Use the deploy script var {SITE_PHP} as the complete handler instead of the concatenated php{SITE_PHP_VERSION}

Thanks, I changed this in the README example but forgot this in the docs!


Some refactoring about functions

The following changes have been made:

Some classes have been moved to different files.
Added support for custom function builders (set FunctionsAsObject).
Validation is now optional. If your functions are static, you don't need to validate them every time. This will slightly improve performance.
Added support for minProperties and maxProperties.
Added support for function object types. Unless you have your own custom type, this might be helpful.

@rzubek What do you think about changes?

@rzubek What do you think about changes?

I will check it out in the coming days, thank you!


I always encounter this abnormality. How can I solve it? Thank you.

[2019-04-10 14:14:39,305 DEBUG websockets.server] Invalid handshake
Traceback (most recent call last):
File "/usr/zb/.local/lib/python3.6/site-packages/websockets/server.py", line 213, in read_http_request
path, headers = yield from read_request(self.reader)
File "/usr/zb/.local/lib/python3.6/site-packages/websockets/http.py", line 80, in read_request
request_line = yield from read_line(stream)
File "/usr/zb/.local/lib/python3.6/site-packages/websockets/http.py", line 196, in read_line
raise ValueError("Line without CRLF")
ValueError: Line without CRLF
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/usr/zb/.local/lib/python3.6/site-packages/websockets/server.py", line 117, in handler
extra_headers=self.extra_headers,
File "/usr/zb/.local/lib/python3.6/site-packages/websockets/server.py", line 467, in handshake
path, request_headers = yield from self.read_http_request()
File "/usr/zb/.local/lib/python3.6/site-packages/websockets/server.py", line 215, in read_http_request
raise InvalidMessage("Malformed HTTP message") from exc
websockets.exceptions.InvalidMessage: Malformed HTTP message

Usually this means the server closed the connection without sending anything in response to the websocket handshake request.
The error reporting needs to be improved in this case. This is #494.


GPU node should request multiple jobs at once

I need the ability to request multiple jobs at once from the GPU node such as
client.newJob(16)
to get a bundle of 16 jobs. this can be individual or composed shards as per https://github.com/TheoCoombes/crawlingathome/issues/10 i.e. either 16 individual jobs or less when composed jobs are among them, while 8 composed jobs would be the minimum response to the above request
I guess client.downloadShard() would become client.downloadShards() or client.downloadJob() as for GPU node it makes no sense the naming downloadShard

Similarly to my response on #10, I'll add this to my todo list - it doesn't seem too hard to achieve.

for this client.invalidURL() we need a more elaborated protocol

say I request 16 shards
I try the download links, some fail
I send invalidURL() for those and I need to get new shards instead
looping the above until all 16 are successful and continue

I think this should be done at my end, if you provide a way to request a number of jobs (and limit that to 20 or so to avoid bad requests )

something like client.invalidURL('1b9437eb5f5144b9aeeae13d85909bb2')


Improve grammar, and add new FAQ for Analytics

Help Developers understand the various views and filters available for the charts in Analytics for their applications in DPW.

The Analytics items looks more like documentation than FAQ. We are working on updating the Quickstart guides covering the Developer Portal which this should be a part of. The Graduation Criteria should go into the Developer Guide as well with a link from the FAQ.

@grokify do you want me to close the PR then so we can internalize this to the docs?

@bdeanindy It would be good to update this PR to remove the descriptive info on the charts so we can keep the FAQ focused on Q&A. We can keep the question "The most recent API request data is missing from the charts of the Analytics portion of Developer Portal for my application". If you update this PR, be sure to merge in change in the master branch now.


Loc Automation - Updating ETA of loc job: 204

Loc Automation Updating ETA of loc job: 204

:x: Validation status: errors

File
Status
Preview URL
Details

:x:Error

Details

Line 111: [Error] W:\4t3g-s\jobs\100\Pages\Pricing\docfx.json for docset Pricing doesn't exist

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


⚠ Adacta-fintech.com has degraded performance

In 28369c2, Adacta-fintech.com (https://www.adacta-fintech.com) experienced degraded performance:

HTTP code: 200
Response time: 785 ms

Resolved: Adacta-fintech.com performance has improved in dba88c4 after 11 minutes.


Memory leaks by not closing http.Response.Body

After running colly on very large scraping jobs I found that it gradually eats memory until it runs out of memory and crashes the process.
The leaks come from two places in the code:

The Collector.checkRobots() function uses the httpBackend.Client directly but does not close the response body of the request.
The httpBackend.Do() function leaves the defer call to res.Body.Close() too late so response bodies which error after being identified by headers as gzip compressed are not closed.

I have a small pull request to fix these issues.

Yo Adam. You ever get this fixed?

Hi Vinniyo. These two problems were fixed in pull request #322 around the same time of this issue.


Unit test coverage

When I started writing this package I was still exploring how 'ink' worked so wasn't really writing tests. I've retrospectively written a bunch of tests now I'm up to speed, but I'd like to get things to a place where its got 90% or more test coverage

Pretty much there now


Build error: An error occurred during the fetch of repository 'local_config_cuda'

Description
I'm trying to build jaxlib but I see the following error during installation:
ERROR: An error occurred during the fetch of repository 'local_config_cuda':
   Traceback (most recent call last):
        File "/tmp/spack26d5tykr/c126d21af0d42f631ea076a051f19300/external/org_tensorflow/third_party/gpus/cuda_configure.bzl", line 1401, column 38, in _cuda_autoconf_impl
                _create_local_cuda_repository(repository_ctx)
        File "/tmp/spack26d5tykr/c126d21af0d42f631ea076a051f19300/external/org_tensorflow/third_party/gpus/cuda_configure.bzl", line 1076, column 27, in _create_local_cuda_repository
                cuda_libs = _find_libs(repository_ctx, check_cuda_libs_script, cuda_config)
        File "/tmp/spack26d5tykr/c126d21af0d42f631ea076a051f19300/external/org_tensorflow/third_party/gpus/cuda_configure.bzl", line 606, column 21, in _find_libs
                _check_cuda_libs(repository_ctx, check_cuda_libs_script, check_cuda_libs_params.values())
        File "/tmp/spack26d5tykr/c126d21af0d42f631ea076a051f19300/external/org_tensorflow/third_party/gpus/cuda_configure.bzl", line 501, column 28, in _check_cuda_libs
                checked_paths = execute(repository_ctx, [python_bin, "-c", cmd]).stdout.splitlines()
        File "/tmp/spack26d5tykr/c126d21af0d42f631ea076a051f19300/external/org_tensorflow/third_party/remote_config/common.bzl", line 230, column 13, in execute
                fail(
Error in fail: Repository command failed
Expected even number of arguments

Any idea how to fix this? Here is the full build log:

build log

What jax/jaxlib version are you using?
jaxlib v0.1.74
Which accelerator(s) are you using?
GPU
Additional system info
Python 3.9.13, Linux
NVIDIA GPU info
This error was found in CI, so I don't have access to run nvidia-smi.
@aweits

this is probably the fix: https://github.com/tensorflow/tensorflow/commit/a76f797b9cd4b9b15bec4c503b16236a804f676f

some more detail for anyone else who runs into this:

cuda 11.7.1 added whitespace in front of the definitions for versioning in cusolver include files, causing the error you see.
TF 2.10.0 includes a fix for this: https://github.com/tensorflow/tensorflow/commit/a76f797b9cd4b9b15bec4c503b16236a804f676f
jax 0.3.14 references a version of tensorflow that includes that commit.


ci: codecov settings

Adding a settings file to configure https://codecov.io/. The threshold of 1% will cause the codecov check to fail only when coverage decreases by more than 1%. This is useful for cases like #37. The threshold should be reduced as the codebase expands.

This was accidentally merged twice (possibly due to this github incident https://www.githubstatus.com/incidents/d60fmldlvw48), and then rebased to remove one of the merges. The actual commit to master is 4080aa4.


Parser and execution will halt if statement is ended with extra semicolon

If a statement line ends with a ;; a script execution will freeze.
No further error is reported.
Can be tried in the playground.
For example the first print statement  ends with double semicolon:
fn run(a) {
    let b = a + 1;
    print("Hello world! a = " + a);;
    print("Hello world! a = " + a);
}
run(10);
Thank-you,

I just indipendently discovered this bug, this was hogging a cpu core and causing a DoS when executing untrusted scripts, I bisected the breaking change to this commit:
https://github.com/rhaiscript/rhai/commit/eca8212f389d788cadd6c287135cfd14fafabb8a

Thanks for catching this. I'll fix asap.

Yes this is a bug and is now fixed.  Thanks for pointing it out.  I'll push to master in a bit.

The fix has landed in master: https://github.com/rhaiscript/rhai/pull/346


non use aum math problem

make a plan
find the usage related to the agreement in the db
edit the usage to be zero aums for a year or so
add schedule to plan, set number of animals to 0
see if the plan can roll through statuses without barking about 'invalid schedule rows'

From #870:

From @micheal-w-wells's email:
"Sarah has had this one for awhile, 2 things (may or may not be related):

Animals say as not provided when she selected 0
It doesn't let her recommend ready and says there is a problem with the schedule, the only anomalous bit other than above is that they are non use years, and I'm wondering if there is some funny javascript math going on, playing in the browser console i wonder if this is a hint:

She has had this for awhile and bad on me i lost it in the woodwork and lost track of it, but she has a good point that a lot of other non use agreements would have the same issue so we should get the guys to look at it if that makes sense to you also."


Feature request

Some inverter integrations has the ability to communicate directly with the inverter via the wifi-dongle.
like this one: https://github.com/torsteinelv/tte-home-assistant-addons/tree/main/pysolarmanv5-proxy
(doesn't work with the magpie dongle included with my SS 12kW 3p)
This eliminates the the need for internet communication and is not dependent on the (sunsynk) cloud service to be online to fetch data or change parameters on the inverter.
Is this something that could be integrated in this amazing Sunsynk integration?
BR Jon, Norway

Hi there.
Unfortunately the only way to my knowledge to access data points from your inverter is to either gather them via the SS portal or to make a use of custom or modified hardware attached to the inverter serial or Mod/Can bus serial ports and cannot be done on its own  from a HA addon without intervening on a hardware level.
The SS Dongle does not expose any information that you can connect to unfortunately. That would have been great if it did. I also think its not in the interest Dye/Sunsynk to make that available as it would take away "control" from their business. I don't have any hopes that it will become available in the future.
Also the SS dongle is hard coded to send data to a specific domain name via a hard coded DNS (Google DNS) which makes it harder to intercept. You can however intercept the data by means of a transparent proxy because the gateway on the dongle setup screen is configurable. But then you are dealing both text based proxying as well as DNS routes on a packet level.
(bit out of my league)
Therefore I will not be attempting to add such functionality.
If anything I would most probably start a totally new project for that and making use of a proper coding language as opposed to a simple bash script (SolarSynk is just a bash script)
Sorry  about that ;-)


add document for build app

When you launch you app to live site, you should build with -tags 'bindatafs'.
go build -tags 'bindatafs' -o app main.go

./app

We only find document:

If you started your application with tag bindatafs, AssetFS will access file from generated go file or current binary
go run -tags 'bindatafs' main.go
Or it will access its content from registered view paths of your filesystem, which is easier for your local development
go run main.go

I think @tnclong brings up two points:

The documentation for bindatafs is not self explanatory at the moment. The examples should in particular give more details regarding file names of the given samples.

The use of the tag 'bindatafs' is a really bad choice, because the default is not to use the cached files bindatafs, but access through the file system directly. Thus if you forget to add the parameter during a deploy build, then you forgetting your assets. I think compiling with 'bindatafs' should be the default.


Function Definition AST doesn't say where the name of the function actually starts

Summary of the new feature/enhancement
The Function Definition AST has a few properties:

ast.Extent.StartColumnNumber: tells you where the definition starts (aka where function, or filter keywords begin)
ast.Name: tells you the name of the function or filter (Get-Foo)
ast.Extent.Text: tells you the full text: function foo { ... }

There is no property that says "this is where the name begins (or ends for that matter)". This means that document highlighting in PSES (this is the feature that lets you click on a reference and it will highlight the definition in the file) involves computing this... like so:
https://github.com/PowerShell/PowerShellEditorServices/pull/1279
It's silly that we have to do this when we have the AST at our finger tips.
Proposed technical implementation details (optional)
If the AST has the tokens in it, that would be ideal... but if not, just having something that says "this is the location of the name" would be incredibly helpful.

I don't see any reason not to have the function name itself in that AST member. It's kinda weird that it doesn't tbh. 😁

Wait, it is an AST member:
> { function x {} }.Ast.EndBlock.Statements[0]

IsFilter   : False
IsWorkflow : False
Name       : x
Parameters :
Body       : {}
Extent     : function x {}
Parent     : function x {}

The problem I think is a more general one: every time you need to deal with tokens within an AST that aren't themselves ASTs, you're forced to reinvent basic token processing... So ast.Name has no extent, meaning you're forced to scan through the tokens looking for it. It would be nice if ASTs came with something like their first token index in the array or something, to allow fast random access.

Ideally I'd like ASTs to come with their own token list, but I suspect that wouldn't be nice for allocation/garbage collection

A list of tokens is probably overkill for most things, but something like Name could store the actual token there instead. If that ship has sailed, we could add a NameToken member for it.

@rjmholt { function x {} }.Ast.EndBlock.Statements[0] this doesn't seem to give you the startColumnNumber.

Try
{ function x {} }.Ast.EndBlock.Statements[0].Extent

@rjmholt
{ function x {} }.Ast.EndBlock.Statements[0].Extent.StartColumnNumber
3

That's not the value I'm looking for :(

Oh sorry, I was responding to this:

I don't see any reason not to have the function name itself in that AST member

It seems to me that ASTs would ideally have tokens for their properties rather than strings, but that ship has sailed

Yeah I've been burned by this a few times as well.  It's especially annoying because of how forgiving the parser is here:
function

test

{}

I always wished the name was a StringConstantExpressionAst of some sort.  Though, the name isn't exactly parsed like most bare word strings so that might be misleading. While not impactful in this exact use case, part of the problem is that the extent text doesn't always match up to the function name. Take this example:
{ function t'est' {} }.Ast.EndBlock.Statements[0]
# IsFilter   : False
# IsWorkflow : False
# Name       : test
# Parameters :
# Body       : {}
# Extent     : function t'est' {}
# Parent     : function t'est' {}

Yeah I've been worrying about the extent text too for precisely this reason.
An AST seems more appropriate than a token in this case, although it would be nice to have a way to link ASTs and tokens easily...

Yeah I've been burned by this a few times as well.  It's especially annoying because of how forgiving the parser is here:
function

test

{}

Dang... Does this means I have to fix my PR in PSES to take into account newlines? 😢

An AST seems more appropriate than a token in this case, although it would be nice to have a way to link ASTs and tokens easily...

Yeah if I had a nickle every time that would have come in handy...
In EditorServicesCommandSuite I started sort of linking these together.  Everything gets passed an object that has:

The AST closest to the cursor
A LinkedList<Token> node of the token closest to the cursor

Then a bunch of extension methods to make linking them and searching within an extent easier.  It works, but it's super hacky.  I don't know how much extra GC pressure linking them properly would add, but I'm tempted to say it would be worth it.

Dang... Does this means I have to fix my PR in PSES to take into account newlines? 😢

Well...

Putting this code here for anyone else who stumbles upon this issue but also wants the start column and line of the actual function name:
        private static (int, int) GetStartColumnAndLineNumbersFromAst(FunctionDefinitionAst ast)
        {
            int startColumnNumber = ast.Extent.StartColumnNumber;
            int startLineNumber = ast.Extent.StartLineNumber;
            int astOffset = 0;

            if (ast.IsFilter)
            {
                astOffset = "filter".Length;
            }
            else if (ast.IsWorkflow)
            {
                astOffset = "workflow".Length;
            }
            else
            {
                astOffset = "function".Length;
            }

            string astText = ast.Extent.Text;
            // The line offset represents the offset on the line that we're on where as
            // astOffset is the offset on the entire text of the AST.
            int lineOffset = astOffset;
            for (; astOffset < astText.Length; astOffset++, lineOffset++)
            {
                if (astText[astOffset] == '\n')
                {
                    // reset numbers since we are operating on a different line and increment the line number.
                    startColumnNumber = 0;
                    startLineNumber++;
                    lineOffset = 0;
                }
                else if (astText[astOffset] == '\r')
                {
                    // Do nothing with carriage returns... we only look for line feeds since those
                    // are used on every platform.
                }
                else if (!char.IsWhiteSpace(astText[astOffset]))
                {
                    // This is the start of the function name so we've found our start column and line number.
                    break;
                }
            }

            return (startColumnNumber + lineOffset, startLineNumber);
        }

If we enhanced AST with the name extent what about backward compatibility? No problems?


Adding isDisabled to config

Hi
Thanks a lot for this plugin its been a real use to me .
Is there currently a way to disable interaction with the knob. Im using the react example and I only want to enable the knob when its been enabled. Currently I cant find a way to disable it but still get it to render.

Hi,
Thanks for your feedback.
I need to look in the code. It's an old project and I don't remember all the details.
I'll have a look and get back to you.