make predict a little bit faster

Make predict faster with disabling the second joblib parallel call

How much faster?

How much faster?

Not so much. After 100 times running inference/predict the average runtime increase is 2-3% on small dataset (test size approximately equals to 500 samples) and 3-4% on large dataset (test size approximately equals to 5000 samples)


Use "outer" positions in all window-related operations

See https://docs.rs/winit/latest/winit/window/struct.WindowAttributes.html#method.with_position.
I think this is the best we can do until https://github.com/rust-windowing/winit/issues/2645 is addressed.

Thanks for the fix! We also need to correct comments for window events:
core\src\window\event.rs:
  /// A window was opened.
    Opened {
        /// The position of the opened window. This is relative to the top-left corner of the desktop
        /// the window is on, including virtual desktops. Refers to window's "outer" position,
        /// or the window area, in logical pixels.
        ///
        /// **Note**: Not available in Wayland.
        position: Option<Point>,
        /// The size of the created window. This is its "inner" size, or the size of the
        /// client area, in logical pixels.
        size: Size,
    },

Now I'm also wondering whether we should provide "outer" size for Opened event.


Signups are borked!

Maybe a Minion issue, but emails aren't being sent as far as I can see.  Therefore, no-one can sign up.

Minion is working, it's authenticating to Hover mail that's the problem
"Auth failed: 535 5.7.8 Error: authentication failed: Authentication failed low05_8ec1190b4000f\n at /home/ytfc/html_songs/songs-to-the-siren/script/../lib/SongsToTheSiren/Task/SendMail.pm line 72.\n"

Installing sendemail is probably the best solution, then using that. I've tested it on my  dev Mac and on the production Ubuntu box.
Basically, then I need to do something like:
cat  | sendemail -q -f<EMAIL_ADDRESS>-t  -u  -s mail.hover.com:587 -o tls=yes -xu<EMAIL_ADDRESS>-xp 'password'
Might not work for every mail provider, but it works for Hover and seems faily uncontroversial.

Or...
It's probably worth trying Mojolicious::Plugin::Mail first.

Fixed by PR #365

This is now in production, via 1.7.0 release.


Added holidays for New Zealand

This adds New Zealand as another holiday country class.  Includes provincial anniversary days.

I will fix the flake8 errors.  But not tonight.

There's another change I want to make, so I will close this request and open a new one when NZ holidays is ready.


Can not restart failed pipelines

if the pipeline fails because of an external issue like:
https://ci.woodpecker-ci.org/repos/3780/pipeline/11472
the restarted pipeline do not find the pipeline config:
https://ci.woodpecker-ci.org/repos/3780/pipeline/11475/errors
I would expect that the pipeline restart with the old config and try to fetch external things like e.g. secrets again

duplicate #2982


Add ISO link to ISO download dialog

This changes the discovery modal behavior - after the download is ready
the modal does not close and provides an ISO URL to be coppied and a button
to trigger a download via browser.

Nice! If the user sets a proxy URL, would that be useful to be able to see in the "results" screen too? FYI @nirfarkas

Boom!!!!

Added proxy URL in the ISO summary if provided.


Add boooks

Added html elements.
Added css styling.
Added functionality with javascript

Thank you
Sent from Yahoo Mail on Android
On Wed, Jan 18, 2023 at 6:03 PM, Leonardo Dias @.***> wrote:
@leonardodiasb approved this pull request.
Status: ✔ Project Approved ✔
Hi @Divinity-dev,
Everything is looking great! Please, merge this branch
Congratulations! 🎉
Highlights

Correct use of GitHub flow. ✔
No linter errors. ✔

Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you take them into account as they can make your code better.
Cheers and Happy coding! 👏 👏 👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.
As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think the code review was unfair, you can request a second opinion using this form.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>


Fix duplicate code example

The example for removing the duplicate code would not work because the getGithubLink() is not a function of the manager object, which would cause an error.

Thanks for your contributions! Sorry it took so long to get back to


Unpin CUDA Nightly

Yeah this test is reliably failing   E               python test/prototype/test_quantized_training.py TestFSDP2.test_fsdp2_correctness

I think it's due to this https://github.com/pytorch/pytorch/pull/137005. Will push a fix
@awgu Sorry forgot to inform you, I already jumped ahead and used your new all gather extension signature 😅


GitHub Actions Billing returns total_paid_minutes_used as float64

Fixes #2283
GitHub Actions returns total_paid_minutes_used as floating point value.
The Implementation should accept this floating point value as float64.
Example
curl -H "Authorization: Bearer <TOKEN>" \
-H "Accept: application/vnd.github.v3+json"  \
https://api.github.com/orgs/{org}/settings/billing/actions

{
  "total_minutes_used": 1234,
  "total_paid_minutes_used": 3959.0,
  "included_minutes": 3000,
  "minutes_used_breakdown": {
    "UBUNTU": 1234,
    "MACOS": 1234,
    "WINDOWS": 1234
  }
}

I'm driving several hundred miles today... I may be slow to respond.

@Parker77 - do you have time to review this PR?

Thank you, @Parker77 !
Merging.


ZEPPELIN-992

What is this PR for?
Moving Interpreter.interpret into LazyOpenInterpreterTest in oder to divide test scope between InterpreterFactoryTest and LazyOpenInterpreter. This is related to #987 a little bit.
What type of PR is it?
[Refactoring]
Todos

[x] - Divide tests

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-992

How should this be tested?
Changed only test case
Screenshots (if appropriate)
Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

The failures is related to ZEPPELIN-878

@jongyoul the CI failure I can see does not seem to be part of any existing open sub-task in ZEPPELIN-878
09:02:21,815 ERROR org.apache.zeppelin.AbstractZeppelinIT:153 - Exception in ZeppelinIT while testSparkInterpreterDependencyLoading
org.openqa.selenium.NoSuchElementException: Unable to locate element: {"method":"xpath","selector":"//div[@class='modal-dialog'][contains(.,'Do you want to update this interpreter and restart with new settings?')]//div[@class='modal-footer']//butt

@bzz Thanks for the guide. I'm good at selenium, thus I thought ZeppelinIT.testSparkInterpreterDependencyLoading was related to that issue. I'll look into it but do you know how to reproduce or why it occurs? I re-triggered it and passed all tests.

Looks great to me and CI passes now.


Roll Engine from 64b13b8eff25 to cf20dd0f49e4 (2 revisions)

https://github.com/flutter/engine/compare/64b13b8eff25...cf20dd0f49e4
2021-11-19<EMAIL_ADDRESS>Roll Skia from 940086c81587 to 9b35cd642f98 (4 revisions) (flutter/engine#29852)
2021-11-19<EMAIL_ADDRESS>[ci.yaml] Update engine enabled branches (flutter/engine#29820)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Feature request: add useMutation isLoading and response to the returned value from the hook

for example urql returns an array of [response, execute] from useMutation. Yet react-apollo-hooks useMutation only returns a function to execute the mutation.
Typical usage is for example when I need to get isLoading to disable the button to prevent user from submitting twice.

duplicate of https://github.com/trojanowski/react-apollo-hooks/issues/90


F #2497 Two Factor Authentication for Sunstone

https://github.com/OpenNebula/one/issues/2497

I have a problem with running open nebula from sources. I have tried from two computers so far and get two separate errors:
https://forum.opennebula.org/t/fatal-error-xmlrpc-c-base-hpp-no-such-file-or-directory/6585
https://forum.opennebula.org/t/cp-cannot-stat-home-joasia-one-share-man-oneacct-1-gz-no-such-file-or-directory/6591

I need to test this code but cant install open nebula from sources as per forum links above. Any help?

Ready for review

Thanks, @hauska7  for the contribution, your code is the basis of the next PR https://github.com/OpenNebula/one/pull/3470


Load fixtures.body() as DOM

For traversal etc. I think, it would be handy to have the ability to load content as DOM.
Therefore, I extended your API to realise that.
PR coming soon.

See #28.
After walking through the specs, the following code seem to work, too:
fixtures.load('some.html');
var iFrameWindow = fixtures.window();
var content = iFramWindow.document.body.children();

The content variable looks like I would like to have. What about creating a function for it?


Update Globals “nav”

Automatically generated by Netlify CMS

👷 Deploy Preview for shimmering-marigold-4f5b4a processing.

Name
Link

🔨 Latest commit
a560da284a43f275b2955f8f0487a453766b384a

🔍 Latest deploy log
https://app.netlify.com/sites/shimmering-marigold-4f5b4a/deploys/632cc0a0c5a1ac00081b0d51


Move the trailing

I moved the code and tested in my own copy of samples repo. Things seem to work. The testability is not obvious to me in the Ignite repo, but the existing tests pass.
I considered loading the pure scripts in the head section with a defer parameter but the tool tip script snippet looked like it needed to run last so I did not try that.

Perfect – thank you!


Where is "track_running_stats" implementation code?

Hi!
I looked for "track_running_stats" implementation code. But I could not find.
Since SynchBN inherit nn.modules.batchNrom, It automatically track running statics?
Thank you for your nice code.

https://github.com/vacancy/Synchronized-BatchNorm-PyTorch/blob/master/sync_batchnorm/batchnorm.py#L68-L70
Yes. It always track the running stats.


Duplicate info in the code completion popup

As you can see here:

The field name is seen twice, wich causes the type to be truncated.. wich is not very helpful, specially when the duplicated name is very long, like this:

Hm, weird bug, I will look into it.
This should probably be an issue in the zls repository instead though :)


Icon request: Mattermost

URL for the brand / site / app
https://mattermost.com/
Logo resources for the icon
https://mattermost.com/newsroom/#press-kit-section
Proof of the Aegis-compatible 2FA
Here is their documentation page on their "Multi-factor authentication" functionality. It states that their MFA is implementation relies on TOTP.
Anything else?
No response
Confirmation

[X] I have read and followed the issue guidelines & issue form.

Here you go!
https://github.com/aegis-icons/aegis-icons/blob/master/icons/1_Primary/Mattermost.svg

:question: How to add icon(s) to Aegis?
:star: Support this project by giving a star! Just adding like to this comment is also appreciated.


test: Add libkpod config tests

config_test.go for testing libkpod/config.go
testdata/config.toml as a fixture for config_test.go

Fix spelling error.  We are hoping to make likpod go away as we move towards libpod, but this test should be able to be easily ported.

@rhatdan This isn't really going to end up a libpod test, libpod won't directly deal with loading config files to/from disk. This will probably move back into the CRI-O code at some point. Still a good test, though.
@ashcrow We don't actually have a makefile target for running unit tests right now, as all our tests are integration tests. Would you mind adding one? We can see about getting them running in Travis after that.

@rhatdan will fix.
@mheon will do!

@mheon it looks like there is a make target for unit testing: make testunit:
$ make testunit
can't load package: package github.com/kubernetes-incubator/cri-o/vendor/github.com/ostreedev/ostree-go/pkg/otadmin:
vendor/github.com/ostreedev/ostree-go/pkg/otadmin/admin.go:1:1: expected 'package', found 'EOF'
can't load package: package github.com/kubernetes-incubator/cri-o/vendor/github.com/ostreedev/ostree-go/pkg/otremote:
vendor/github.com/ostreedev/ostree-go/pkg/otremote/remoteadd.go:1:1: expected 'package', found 'EOF'
go test -tags "selinux seccomp  " -cover github.com/kubernetes-incubator/cri-o/cmd/crio github.com/kubernetes-incubator/cri-o/cmd/crioctl github.com/kubernetes-incubator/cri-o/cmd/kpod github.com/kubernetes-incubator/cri-o/cmd/kpod/docker github.com/kubernetes-incubator/cri-o/cmd/kpod/formats github.com/kubernetes-incubator/cri-o/libkpod github.com/kubernetes-incubator/cri-o/libkpod/sandbox github.com/kubernetes-incubator/cri-o/libpod github.com/kubernetes-incubator/cri-o/libpod/common github.com/kubernetes-incubator/cri-o/libpod/ctr github.com/kubernetes-incubator/cri-o/libpod/driver github.com/kubernetes-incubator/cri-o/libpod/images github.com/kubernetes-incubator/cri-o/libpod/layers github.com/kubernetes-incubator/cri-o/libpod/pod github.com/kubernetes-incubator/cri-o/oci github.com/kubernetes-incubator/cri-o/pkg/annotations github.com/kubernetes-incubator/cri-o/pkg/registrar github.com/kubernetes-incubator/cri-o/pkg/storage github.com/kubernetes-incubator/cri-o/server github.com/kubernetes-incubator/cri-o/server/apparmor github.com/kubernetes-incubator/cri-o/server/seccomp github.com/kubernetes-incubator/cri-o/test/bin2img github.com/kubernetes-incubator/cri-o/test/checkseccomp github.com/kubernetes-incubator/cri-o/test/copyimg github.com/kubernetes-incubator/cri-o/utils
?       github.com/kubernetes-incubator/cri-o/cmd/crio  [no test files]
?       github.com/kubernetes-incubator/cri-o/cmd/crioctl       [no test files]
ok      github.com/kubernetes-incubator/cri-o/cmd/kpod  0.069s  coverage: 0.0% of statements
?       github.com/kubernetes-incubator/cri-o/cmd/kpod/docker   [no test files]
?       github.com/kubernetes-incubator/cri-o/cmd/kpod/formats  [no test files]
ok      github.com/kubernetes-incubator/cri-o/libkpod   0.122s  coverage: 3.1% of statements
?       github.com/kubernetes-incubator/cri-o/libkpod/sandbox   [no test files]
?       github.com/kubernetes-incubator/cri-o/libpod    [no test files]
?       github.com/kubernetes-incubator/cri-o/libpod/common     [no test files]
?       github.com/kubernetes-incubator/cri-o/libpod/ctr        [no test files]
?       github.com/kubernetes-incubator/cri-o/libpod/driver     [no test files]
?       github.com/kubernetes-incubator/cri-o/libpod/images     [no test files]
?       github.com/kubernetes-incubator/cri-o/libpod/layers     [no test files]
?       github.com/kubernetes-incubator/cri-o/libpod/pod        [no test files]
?       github.com/kubernetes-incubator/cri-o/oci       [no test files]
?       github.com/kubernetes-incubator/cri-o/pkg/annotations   [no test files]
ok      github.com/kubernetes-incubator/cri-o/pkg/registrar     0.024s  coverage: 84.8% of statements
?       github.com/kubernetes-incubator/cri-o/pkg/storage       [no test files]
ok      github.com/kubernetes-incubator/cri-o/server    0.099s  coverage: 0.8% of statements
?       github.com/kubernetes-incubator/cri-o/server/apparmor   [no test files]
?       github.com/kubernetes-incubator/cri-o/server/seccomp    [no test files]
?       github.com/kubernetes-incubator/cri-o/test/bin2img      [no test files]
?       github.com/kubernetes-incubator/cri-o/test/checkseccomp [no test files]
?       github.com/kubernetes-incubator/cri-o/test/copyimg      [no test files]
?       github.com/kubernetes-incubator/cri-o/utils     [no test files]

@ashcrow Huh. Did not know that. Cool!
In that case, we'll probably want to add some extra targets to run both unit and integration tests, optionally in a container, but I can make that happen in another PR.

LGTM pending tests

Tests are green, and this is just a makefile change - shouldn't need a pass of the E2E tests. @rhatdan let's get this merged.

@mheon Did you comment on the wrong PR?  This is not a Makefile change?

@rhatdan Sorry, test change - just adding new unit tests. Oops. My point stands, though - none of this will break the E2E tests.

Do we run unit tests on E2E?  I guess not, but we really should run these tests in RHEL/Fedora systems.
LGTM


Fundamental issue in the authorship statement

https://github.com/daveshap/HeuristicImperatives/blob/main/Heuristic Imperatives.pdf says most of it is authored by AI with no human editing. Most jurisdictions (the UK is a notable exception here) consider such stuff uncopyrightable. To avoid doubt and prevent the work from uncopyrightable in its entirety, contributions of you and the GPT-4 should be clearly denoted

...copyright does not matter in the least?


Code error in recipe Build Forms with APIs?

📚 Subject area/topic
recipe - building forms
📋 Page(s) affected (or suggested, for new content)
https://docs.astro.build/en/recipes/build-forms-api/
📋 Description of content that is out-of-date or incorrect
From Feedback Fish:
In FeedbackForm.tsx example it gives error on import of type without specifying.
This works:
import { useState } from "react";
import type { FormEvent } from "react";
🖥 Reproduction in StackBlitz (if reporting incorrect content or code samples)
No response

Confirmed. I'll submit a PR 🔜


Add sanitizers

this is independent of my other PR which affects cmakelists #3 because I think I broke tests somehow in that one.  merging is left as an exercise for the amazonian

Hi @larsonmpdx,
Thanks for the detailed steps. I will merge and test the sanitizers and will release it with the next version of the SDK. We have already merged your CMakeLists PR so handling the conflicts should be simple enough.
Varun

Included in v1.2.0.


Code duplication across Standard/Ultra Prover/Verifier

In contrast with Plonk, where one Prover/Verifier was used to handle Standard/Turbo/Ultra via conditionals, we have started going down the path of having a separate prover/verifier for each flavor. This makes it much easier to understand what's going on for a given flavor but currently results in a good deal of code duplication. For example the PCS rounds for standard and ultra are literally identical. Whatever we do should be guided by making things clear and avoiding unnecessary drift between code that should for all intents and purposes be identical. We should not simply try to avoid duplication at all costs.

Handled with the removal of Standard Honk


⚠ SceneTime has degraded performance

In ba73faf, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 3221 ms

Resolved: SceneTime performance has improved in 3213a21.


docker_container redeployed at every chef-run if container exposes ports that are not published

In the same spirit as #444, I have a container that redeploys at every chef run because that container:

exposes ports 80 and 443 (indirectly through the base image, not from my own)
publishes port 80 (but not 443), which implies that we expose port 80

As a consequence, at each chef-run, the container redeploys because a "change" is detected. After instrumenting, I can read in the debug log:
[2015-09-21T14:54:02+02:00] DEBUG: DOCKER: exposed_ports - current:{"443/tcp"=>{}, "80/tcp"=>{}}: serialized:{"80/tcp"=>{}}:

I need to find a way so that when no expose ports are declared then all exposed ports from HostConfig are loaded in the serialized version.

Should be fixed in #448.


printLoginInfo incorrect for non standard ports

When using boringproxy server with both -print-login & -https-port flags. The printed output is displaying an incorrect admin login link.
This happens because the printLoginInfo in boringproxy.go:423 is unaware of httpsPort.
The printLoginInfo need to be updated to include a check for non standard ports and set URL accordingly.
func printLoginInfo(token, adminDomain string, httpsPort int) {
	var url string
	if httpsPort != 443 {
		url = fmt.Sprintf("https://%s:%d/login?access_token=%s", adminDomain, httpsPort, token)
	} else {
		url = fmt.Sprintf("https://%s/login?access_token=%s", adminDomain, token)
	}
	log.Println(fmt.Sprintf("Admin login link: %s", url))
	qrterminal.GenerateHalfBlock(url, qrterminal.L, os.Stdout)
}

I had to declare the url string variable before the IF statement otherwise the var is not set when called in the log function. In not familiar with Golang, so please update code if incorrect.

Your code looks good. Pretty sure that's a proper way to handle this in Golang.
I'm not sure how necessary this is in practice. It doesn't seem like it would be common at all for people to want to run the admin domain on a different HTTPS port. I think you'd either be using boringproxy as your reverse proxy, or if you just want it for the tunneling you'd have it behind another reverse proxy that's handling TLS termination for you.
Can you help me understand your use case better?

For anyone interested, we had a further discussion surrounding this on indiebits.io.
TL;DR: Using non standard ports is included in boringproxy, but its functionality is limited. Please take note that using it might require some additional setup on your end.


Unable to install com.cnuernber/ham-fisted with Neil 0.1.59

I'm seeing this CLI behavior:
$ ls
$ neil dep search ham-fisted
:lib com.cnuernber/ham-fisted :version "1.000-beta-90" :description nil
$ neil dep add com.cnuernber/ham-fisted
$ ls
deps.edn
$ cat deps.edn
{:deps {}
 :aliases {}}

I'd expect neil dep add to add the dependency when we can find it with neil dep search.

From @borkdude on Slack:

I suspect it has to do with neil not adding non-stable versions by default
but if you specify the :version yourself, then it will work

Teodor:

ahh, you're right. setting version explicitly adds everything as expected. neil dep search ham-fisted | fzf | xargs neil dep add also just works.
Any idea on what kind of semantics we want here?
"Do nothing, print nothing and return :exit 0" kind of surprised me.

Fixed with #178 , closing.


Updated wilson corr

The branch contains updates to the wilson line correlator such that hyp smearing can be used and a maximum radius can be specified.
A maximal center gauge fixing has been implemented into the gauge fixing class.
The gauge fixing uses a QR algorithm defined in a class called matrix4x4_notSym.h
A quick wilon loop was also created that is now in the experimental folder.
test for the wilson line and maximal center gauge fixing has been made.
All test has been run. The following error was seen on main and on branch.
OUT_HBOR_multiple_test:4 total processes failed to start
OUT_HBOR_multiple_test:4 total processes failed to start
OUT_HBOR_multiple_test:4 total processes failed to start
OUT_RndMultipleTest:2 total processes failed to start
though no new errors has appeared, which makes sense as the changes should not affect other applications.
Also the checksum for milc does not exist, and is therefore commented out for now.
Regards Rasmus

Looks almost ready to merge. Two things: you commented out the Git Hash in the mpi CommunicationBase constructor. Is that causing issues or can you re-enable that? And: you commented out the checksum check for milc configurations. Can you instead make it optional so that the check is on by default but you can turn it off for your use-case?

Should've read your description more closely... Forget about my milc checksum comment.


Unroll loop and use call when possible

I've unrolled the loop used to create the arguments with a switch, in this way for simple cases we don't need to instantiate an array and we can use call instead of apply.
I've also expanded the tests to make sure the both the "fast" path (switch) and the "standard" slow (loop) are tested.
On my setup this shaves ~10% of execution time.

You should wrap the switch in a function, then it would probably opt & inline, and it might be faster.

I come up with this solution https://github.com/mcollina/fastfall/compare/master...paolochiodi:unroll-loop-function, but it is actually slower.
I think v8 isn't inlining it, while the implementation in this PR is "manually" inlined

Merged with a little addition! Check it out!

I am surprised that changes in https://github.com/mcollina/fastfall/commit/643659b57ebf233c231a691ba418b7a2d0e6c880 result in better performance: you allocate an additional array and call an additional function.
Good to hear it works, though :)

Regarding the array allocation, allocating an array with a fixed size is really fast. Regarding the additional function call, it gets inlined, so no performance drops there.

I took some time to bench the final version with the one in the PR and on my setup the one in the PR seems slightly faster. Do you have different results?

Which version of node are you in?
Il giorno gio 10 dic 2015 alle 21:56 Paolo Chiodi<EMAIL_ADDRESS>ha scritto:

I took some time to bench the final version with the one in the PR and on
my setup the one in the PR seems slightly faster. Do you have different
results?
—
Reply to this email directly or view it on GitHub
https://github.com/mcollina/fastfall/pull/8#issuecomment-163746890.

4.2.3

Ok, reverted :).
I've applied the same technique to the compiled mode, it's as fast as the other one now.

What about creating a benchmark for different versions of fastball?
Ideally it should be a script that run bench.js, then checkout HEAD^ and run it again and compares results.

That would be awesome :). It requires some more tooling that what we have right now. Possibly something that wraps https://github.com/mcollina/fastbench.


Directly call MapController methods for setting position, tilt w/out animation

Did this fix the issues you were seeing in eraser map? After looking more closely, tangram-es does correctly handle eases of zero duration, so I'm not sure what the root of this problem would be.

Hmm thats interesting, it did fix the problem in eraser map. There must be something else going on there?

¯\(ツ)/¯


“May” be that my translations in the web browser are being transferred to the internet service provider's surveillance database

The security of our data is very important due to the fascist, religious, radical, repressive, dictatorial imaginary regime-government in an imaginary country. As an oppositionist, we need to be very careful with this. My translations made with DeepL in the browser appear as URLs. In the HTTPS encryption system, URLs are not hidden, and the internet service provider can see these URLs. Deepl translates translations by showing them as URLs. For example: https://www.deepl.com/en/translator#tr/en-us/ülkedeki faşist dinci radikal baskıcı diktacı rejim sebebiyle verilerimizin güvenliği çok önemli. There are two fictitious internet infrastructure companies in the country. One is Ticktelecomi and the other is Tickcello.
both are in the hands of the regime and the government. Ticktelecomi, the largest internet service provider company in this country, which is the dream of Ticktelecomi, is also the infrastructure company. In addition, there is a fictitious "tick information technology unit" in the country that makes plugging with citizens' data. There's a fictitious company called TickCello that owns a giant data storage building that's also a fictitious data center. Let's assume that this company is transferring data to the information technology unit that ticks. In other words, all dissidents in this imaginary country are monitored, data recorded, reported, plugged and followed.
A frequently used platform such as DeepL also inadvertently learns to the service provider what this opponent is translating, and to whom he is doing this translation to send e-mails about what. Please make your translation system independent of the HTTPS URL. This is a vulnerability for everyone. Not only for the dissident citizens of this imaginary country.

Hi, thanks for your report and sorry to hear that. These repos are unfortunately for our API and client libraries, not for deepl.com. However, you can translate via the API without the query parameters (by sending the translation text as a body parameter), which should cover your texts with the HTTPS encryption. I will forward the feedback about the web issue to the team responsible for that and close here.

Hi, just to follow up here - this behaviour exists on DeepL free to facilitate sharing translations with friends etc. If you require privacy guarantees, we recommend DeepL Pro, where the text will not appear in the URL and is thus covered by TLS encryption. Alternatively you can use the API as i outlined before.

hay capital

Hi I wanted to extend this answer @universish,
DeepL does take data security very seriously. First of all, it is only possible to use deepl.com over SSL, which already offers great data protection of requests sent to the server. AFAIK, snooping an SSL connection will only allow an attacker to see you're using deepl.com, not the query path tr/translate nor the query parameter (usually follows a ?, but isn't used in this case).
Furthermore, I note that the text is included in the anchor part of the URL (after the #). From Mozilla docs:

It is worth noting that the part after the #, also known as the fragment identifier, is never sent to the server with the request.

You can inspect the requests made to DeepL by your browser yourself when you use a link like your one above. AFAIK, only the https://www2.deepl.com/jsonrpc?method=LMT_split_text and https://www2.deepl.com/jsonrpc?method=LMT_handle_jobs requests contain the text for translation, and always in the request body.
I hope that assuages your concerns.

Thank you for your answer. My question was not about requests to deepl servers. It's about requests passing through the servers of the internet service provider. I wonder if the internet service provider also sees the part after the “#” sign? Does it see the part after the “?” sign?

No, this is where I was wrong before. (as an aside, requests to DeepL servers are the requests passing through your ISP's routers and switches) TCP packets expose the DNS of the server you are connecting to, but not any request parameters.
See for example Wikipedia:

Because HTTPS piggybacks HTTP entirely on top of TLS, the entirety of the underlying HTTP protocol can be encrypted. This includes the request's URL, query parameters, headers, and cookies (which often contain identifying information about the user). However, because website addresses and port numbers are necessarily part of the underlying TCP/IP protocols, HTTPS cannot protect their disclosure. In practice this means that even on a correctly configured web server, eavesdroppers can infer the IP address and port number of the web server, and sometimes even the domain name (e.g. www.example.org, but not the rest of the URL) that a user is communicating with, along with the amount of data transferred and the duration of the communication, though not the content of the communication

links not working: https://www2.deepl.com/jsonrpc?method=LMT_split_text and https://www2.deepl.com/jsonrpc?method=LMT_handle_jobs

Sorry I don't understand your question. These are internal (JSON-RPC) requests made by our frontend, not websites, you can't open them in your browser.
If you want to verify what your ISP can see, you can use a tool like wireshark to sniff what packets are sent to your router.

thanks.


Hey my issue is about the Bluestats installer and the Query IP and the Query Port!

Hey like you can See in the Title I don't know what should I write into these two fields?
Which Server Query Port and URL?
The Files are located Website -> stats -> install
Thank you and Sorry for my bad English I'm from Germany!

You have to enable server query on your minecraft server first. Then enter the ip of your minecraft server and the port for the query which is most likely the default 25565


Impressive! Some suggestion how to improve

I never thought that something that possible to do in Swift. Well done! 👍
Here some ideas to improve performance of your algo:

The slowest part of your algo it control point detector. If you will use simplest structure here, you will have better performance and it will be possible to process full image.

  static let lightBackgroundRange = UInt8(0)...UInt8(180) // simple compare better: pixel < 180
  static let darkBackgroundRange = UInt8(100)...UInt8(255) // simple compare better: pixel > 100

I recommend to perform threshold separately. Accelerate framework can help

let bit = pixelThreshold.contains(data[x, y]) ? RCBit.one : RCBit.zero

You probably don't need RCBit abstraction. typealias around bool will works better and pretty informative

pixelPatterns[pixelPatterns.count - 1] = lastPattern

This operation call lot of times. Just with change struct to class and you can remove this line

2) Reed Solomon codes. Additional you will get error restoration. You have interesting idea with something like `hash` with `RCMatrix` (if I understand it right). To make orientation working, you can try 4 possible orientation on decode bits.

You awesome! =)

@AlexandrGraschenkov thank you very much for your feedback and suggestions 👍🏻

I do need to use Accelerate. I realized that during development.
Regarding pixelPatterns[pixelPatterns.count - 1] = lastPattern Turns out using structs were much faster, since classes are stored in the heap.
I was planning to integrate "Reed Solomon", but didn't have enough time to finish. So I ended up with "hash" verification. The parity and matrix multiplication logic is there, so in future versions will add error code.

The goal for this project was to implement everything in Swift without OpenCV. It was harder than I thought. Hopefully, the performance will improve after integrating Accelerate.

I just noticed you have a much much better version of QR scanner. Do you have twitter? I would like to connect with you.

I don't have twitter =)

If so, then better to perform changes directly in array. In Instruments you can see that this line take pretty much time for execution. You copy every time whole structure data.

You can see implementation of dot detector inside my repo. In Swift implementation this code may be more compact and faster current approach.

@aslanyanhaik have you worked out the performance issue @AlexandrGraschenkov mentioned?


D3 switch between different methods for motif and document sizing and colouring

What it says in the title...

Added a checkbox on the page to toggle whether to show MS1 analysis (or not) on the network. Does it do the job, or needs more options?

TODO: make annotated motifs stand out more (bold the lines)

Would be nice to load all the data at first and use javascript only to toggle between the different drawing of the network.


Site is unreachable

www.emacs-bootstrap.com is unreachable.

No news?

Hi @PieterPaulussen and @Ambrevar
Sorry for the late reply... i couldn't find the time to deploy it to a new server. But now it should be up and running.
Out of curiosity, how is emacs-bootstrap useful to you?
Thanks for the report.

It's working, thanks!
To satisfy your curiosity, I don't use emacs-bootstrap directly (my configuration is way to fine-tuned for that) but I think it's great as a starting point suggestion for newcomers.
I see emacs-bootstrap as a big selling point :)

Thanks @Ambrevar, that's the answer i wanted to hear. Am glad you're finding it useful to onboard other people to Emacs.

@rakanalh Thanks for getting the website back up. I was actually just experimenting with your config generator.
Can I make 1 suggestion though? Could you add an option to not specify a default theme, I think both currently available themes are hideous. ;-)


Setting Disabled

Hello,
I know that this issue has been reported before but none of the solution seems to work for me.
After I install TaskBoard it logs me into automaticly and when I click on settings I am unabled to modify anything since everything is gray out. (See screen shot)

Figured out the issue.
forgot to install the apache php lib
ubuntu command: sudo apt-get install libapache2-mod-php

Glad you figured it out. My first thought was that PHP wasn't installed, since it was treating the files as XML.


update table of contents

Current dependencies on/for this PR:

main

PR #528   👈

This comment was auto-generated by Graphite.


Updated the maintainers and codeowners files

Addressing https://github.com/notaryproject/.github/issues/55 https://github.com/notaryproject/.github/issues/56 and https://github.com/notaryproject/.github/issues/57

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (9038355) 89.32% compared to head (d14f76e) 89.32%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #183   +/-   ##
=======================================
  Coverage   89.32%   89.32%
=======================================
  Files          21       21
  Lines        1724     1724
=======================================
  Hits         1540     1540
  Misses        149      149
  Partials       35       35

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Fix crash when "show folder sizes" is enabled

Resolved / Related Issues
Items resolved / related issues by this PR.

Closes #8413
Removes some unused Helper classes

Details of Changes
Add details of changes here.
The issue seems to be due to the fact that the file list is changed on a background thread (in ApplyFilesAndFoldersChangesAsync()) while it's being enumerated in UpdateSelectionSize().
This PR adds a "thread-safe" enumerator to fix this issue. Alternative could have been returning a copy of the list (with ToList()) as in 1e758ef. I'm somewhat unsure what the best option is.
Validation
How did you test these changes?

[x] Built and ran the app

Just for documentation purposes, result of #3386 removing "snapshots" of the underlying collection.
I'll pitch in and argue to reimplement thread-safe enumeration.
Most people would assume a "concurrent" collection should both be read and write safe.
We don't want to cause a similar race condition in the future because of this leaky abstraction.

Do you find any issue with the PR as is? Do you have suggestions for reimplement thread-safe enumeration?
I think in the future we could switch to ReaderWriterLockSlim instead of lock as it supports multiple readers simultaneously or one writer.

Do you find any issue with the PR as is? Do you have suggestions for reimplementing thread-safe enumeration?

It's a bit of a nightmare to synchronize this without some type of drawback.
The only alternative is to reintroduce the "snapshot" pattern but it's expensive for large lists (e.g. a directory with thousands of files), doing this will probably regress the PR I linked above. So... I think what you have is the correct direction to go.

Would it make sense to switch to ReaderWriterLockSlim instead of lock as it supports multiple readers simultaneously?

I did some internal testing (not really conclusive) but we read about 20-25 times more than we write on an average session.
We would need to do some internal testing because despite our reads vs. writes, this would substantially increase overhead and might not be better overall.


make regex in toolchain file case insensitive

Because of the file this touches I am running a full test suite in the CI system by hand (Azure DevOps job #2670539)


Bounding Box not surrounding Mask

Hi,
I had an image of shape(1024,1024,3), I used SAMAutomaticMaskGenerator to segment everything , I am using below code to plot bounding box around generated masks  but bounding boxes are not surrounding masks , the bbox's size is also different from that of object. Can someone help understand why this is happening?
results = mask_generator.generate(image_rgb)
  #print(results)
 # masks = [(mask['segmentation'],mask['bbox'])for mask in sorted(results, key=lambda x: x['area'], reverse=True)]

  for res in results:
      mask = res['segmentation']*255
      bbox = res['bbox']
      print(bbox)
      x1 = bbox[0] - (bbox[2]/2)
      y1 = bbox[1] - (bbox[3]/2)
      x2 = bbox[0] + (bbox[2]/2)
      y2 = bbox[1] + (bbox[3]/2)
      image = mask.astype('uint8').copy()
      cv2.rectangle(image,(int(x1),int(y1)),(int(x2),int(y2)),(255,0,0),2)
      plt.imshow(image,cmap="gray")
      plt.show()
<img width="373" alt="Screenshot 2023-07-17 at 1 33 02 PM" src="https://github.com/facebookresearch/segment-anything/assets/133599249/7aeb728d-924b-40ab-a074-5ec23bd0a2b4">

bbox is supposed to be XYWH format. You should try -
x1 = bbox[0]
x2 = bbox[1]
x3 = bbox[0] + bbox[2]
x4 = bbox[1] + bbox[3]


Side navigation bar is missing in certain versions of documentation

Side navigation bar is missing in the following versions of documentation
1.0.0, 0.9.0, 0.8.0 and 0.7.0
It works ok with rest of versions.
Also the landing page on these above versions is 'overview' whereas it should have been 'gettingstarted'
Will be sending a PR to fix it, later on.

Direct access works though, eg. https://docs.litmuschaos.io/docs/1.0.0/openebs-target-network-delay


Add multiple config tests

This commit adds multi-config test to test with different python
versions.
This commit tests the latest minor versions of all python > 3.5.

Try tox rather than running multiple containers from single image w/pyenv.

Close in favor of #16


Fixes #23916 - Fix sockets when template is selected

@iNecas could you please review this?

Tested and works well. Thanks @shiramax

rbovirt 0.1.7 released


Can glamor do true inline styles?

I need a real inline styles solution for building email templates. Is this something thats possible with glamor?

I doubt it :( gmail doesn't take style tags, so that's pretty much a non starter. I'd think plain react style prop would be the way it to go?

Yeah, but I want to render the emails in an online archive as well. So it would be nice to use the same tech in both places. I suppose I could use renderStatic and then use juice to inline...

I think that makes sense...

@ccorcos have you checked out https://github.com/revivek/oy?

hmm. doesnt look like Oy will add vendor prefixes though...


Hi,how can i get the ted-59 and opus-100 dataset?

Hi, thanks for your attention.
The TED-59 dataset can be accessed from http://phontron.com/data/ted_talks.tar.gz. The python script ted_reader.py in this repository can be used to read the parallel corpus.
The OPUS-100 dataset can be found at https://github.com/EdinburghNLP/opus-100-corpus.

Thanks,btw,your work is very exciting!

Thank you. I am really excited that you like our work.

it seems that the ted dataset you mentioned has 60 languages ,but you only use 59?

Yes, it seems that there are 60 language pairs in the raw dataset. But the column named calv is empty, which indicates that there is no parallel corpus between calv and en. Therefore, there are 59 language pairs in the TED-59 dataset.


Add ncc to devDependency & refactor to use setFailed

This is a simple PR. Just to facilitate DX and to return the correct exit code when the action failed.

Return the core.setFailed() when and exception is trowed
Add the ncc dependency directly on the project.

For testing your action on local, you can try this https://github.com/nektos/act

Thanks for the PR @fabriceyopa !


Browse pagination not limiting responses from graphql

Describe the bug
Browse pages always display all the results on the first page.
To Reproduce
Steps to reproduce the behavior:

Go to a browse path with more than the RESULTS_PER_PAGE as defined in conf/Browse.tsx (default is 10).
Verify that more than RESULTS_PER_PAGE are shown

Expected behavior
Only RESULTS_PER_PAGE items are shown with pagination pills filled out.

Is this still broken @gabe-lyons ?

@jjoyce0510 I just tried to reproduce and it appears to be working. Let me close this for now.

Reopening this issue- I was able to reproduce the problem. When on a leaf node, pagination works. However, on a node in the middle of the browse path, pagination does not work.

Fixed. Closing the issue.


FileStreamResult needs Read&Write permission?

I am creating this issue because I am not sure if this was the case in previous versions, but, does FileStreamResult need read & write permission for IIS_IUSRS on the resource in order to be served?
Environment
Visual Studio - 15.5.2
AspNetCore - 2.0.3
Published to IIS 8.5
Repro Project
I have a simple repro project that you can use and publish to IIS at https://github.com/muqeet-khan/VideoProcessingSample . All this project provides is the below code and a sample BigBunny mp4 video.

I am not doing anything special. I am using FileStreamResult to serve a MP4.

         public IActionResult ServeVideo(string filename)
        {
            var fPath = Path.Combine(new string[] { "Media", filename });
            var fullPath = Path.Combine(_appEnvironment.ContentRootPath, "wwwroot\\" + fPath);
            return new FileStreamResult(new FileStream(fullPath, FileMode.Open), "video/mp4");
        }

Upload a video to the Media folder and make sure it has Read permissions only.
After publishing to IIS,
http://IIS_Website/home/servevideo?filename=BigBuckBunny_320x180.mp4 -- Returns 500 with an exception.

fail: Microsoft.AspNetCore.Diagnostics.ExceptionHandlerMiddleware[0]
      An unhandled exception has occurred: Access to the path 'MP4 Path' is denied.
System.UnauthorizedAccessException: Access to the path 'MP4 Path' is denied.
   at System.IO.FileStream.OpenHandle(FileMode mode, FileShare share, FileOptions options)
   at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, FileOptions options)
   at Sample.Controllers.HomeController.ServeVideo(String filename)
   at lambda_method(Closure , Object , Object[] )
   at Microsoft.Extensions.Internal.ObjectMethodExecutor.Execute(Object target, Object[] parameters)
   at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.<InvokeActionMethodAsync>d__12.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.<InvokeNextActionFilterAsync>d__10.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.Rethrow(ActionExecutedContext context)
   at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.Next(State& next, Scope& scope, Object& state, Boolean& isCompleted)
   at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.<InvokeInnerFilterAsync>d__14.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.<InvokeNextResourceFilter>d__22.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.Rethrow(ResourceExecutedContext context)
   at Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.Next(State& next, Scope& scope, Object& state, Boolean& isCompleted)
   at Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.<InvokeFilterPipelineAsync>d__17.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.<InvokeAsync>d__15.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Builder.RouterMiddleware.<Invoke>d__4.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Builder.Extensions.MapWhenMiddleware.<Invoke>d__3.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware.<Invoke>d__6.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Diagnostics.ExceptionHandlerMiddleware.<Invoke>d__6.MoveNext()

However, http://IIS_Website/Media/BigBuckBunny_320x180.mp4 - Static File Middleware works just fine with just read permission.

Give Read and Write permissions on the media file for IIS_IUSRS and the file will be served by the ServeVideo Action just fine.

Add FileAccess.Read as last param when creating the FileStream
return new FileStreamResult(new FileStream(fullPath, FileMode.Open, FileAccess.Read), "video/mp4");

You could use a PhysicalFileResult rather than the FileStreamResult+FileStream?
return new PhysicalFileResult(fullPath, "video/mp4");


Share bookmarks between two devices

Is it possible share bookmarks when using multi device? Maybe store data on the iCloud etc.
Sent with GitHawk

Could totally be possible, but isn't atm. iCloud storage seems like the safest bet for this!

Closing in favor of #702


While the Script Editor Plugin is active, makes it optional to switch Editor Plugins when selecting Nodes in the Scene Tree

Problem

While the Script editor is active, selecting a Node in the Scene Tree in order to inspect it, makes the editor_plugin_screen to be changed to the Node's main plugin.
The same happens when right-clicking a Node
This makes it hard, and tiresome to inspect and contextualize Nodes while writing Scripts, because it requires constant viewport and editor switching when interacting with Nodes while in the Text Editor
It makes it even more cumbersome when dealing with long UI-centric Scene Trees, where it's necessary to gather Node Paths and adjust Inspector properties
It also makes the process of dragging and dropping Nodes into the Text Editor repetitive and tiresome, requiring 4 clicks: (1) click a Node, it switches to the 2D/3D viewport, (2) click to go back to Script, (3) click the Node again, (4) drag and drop it into the Text Editor to get the Node Path.
Examples are reported in this blog post, under "Inconsistent Viewport and Script Editor tab switching when clicking Nodes (2D/3D)".

https://user-images.githubusercontent.com/248383/180605492-0e974253-ec46-474d-b3cf-f9718fb3fa2f.mp4
Solution

Make it optional to switch Editor Plugins when selecting Nodes in the Scene tree while the current active Editor Plugin is the Script Text Editor.
The feature is toggeable via text_editor/behavior/navigation/node_select_toggles_inspector_only.

https://user-images.githubusercontent.com/248383/180605429-7fa86418-e554-4386-afd7-fad42e3763d3.mp4
Is it really needed?

I make complex user interfaces with Godot, such as my project Godello and Dynamic Inventory System, and this has always been in my way when scripting the complex UI Node trees.
My tweet on this subject had a few answers and retweets regarding this to be a welcome addition, including an answer from the prolific Godot's developer MagellanicGames. Also here and here.

Please consider adding this!

i want this!

Please refrain from adding comments that don't add to the discussion of this change. If you just want to express your support for this change, use the reaction on the PR itself for that. Thank you!
(Comments add some noise for maintainers who then see the PR has received an update; some also get an email about it. It also makes discussion harder to follow in longer PR comment chains.)

I don't know if I like this solution, feels not optimal to me.
I'd better like a unified solution that works best almost all the time since the default behaviour is still preferred occasionally.
What about the switching only happening if an already selected node gets clicked again? (click down + release, to allow dragging it without a switch)

Note that this is already available when holding Alt while clicking in the scene tree dock. Unfortunately, most window managers on Linux will intercept Alt + clicking by default, so Godot will never see those clicks.

The Alt was a hack that got removed in #61162
I think the proper solution would be extending #61162 to handle script editor properly (i.e. only switch editor when not dragging the node).

(i.e. only switch editor when not dragging the node).
If it requires you to drag the node to not switch the editor, how would you inspect the Nodes in the Inspector and access their context menu, while staying in the Script editor?

Wouldn't that require you to loosely always drag Nodes and let them go, without the intention of actually dragging, but just because you want to access the Inspector for that Node?
The two biggest pain points to be fixed are getting distraction-free and hassle-free access to the Inspector and access to the context menu of any Node, while staying in the Script Editor, without toggling a context switching.

Ok it makes sense to prevent switching on node click.

Isn't the default behaviour more of a legacy leftover than actually being useful?

idk I sometimes use it, but I agree that more often it's annoying and not switching is more useful.

What about the switching only happening if an already selected node gets clicked again?

Or maybe we could switch when Alt is held or something.
But editor setting is fine too probably; it should be enabled by default though. Also the name could be better, like stay_in_script_editor_on_node_selected (up to discussion).

Hi all, first comment event in github.
I would like introduce another posible solution.
A pin icon in the script tool bar, after File menu o next to rigth buttons,
so you can pin to make script visible even if change the node or unpin to have the current behavior.
This make this function more visible and move easy to switch on/off

I agree with@8to16apps and also think a pin icon in the script toolbar is definitely a cleaner solution. Very discoverable and easy to toggle on and off.

@reduz @8to16apps I'm not sure I understand where exactly it should be. A mockup would be welcome.

@alfredbaudisch @reduz I am thinking in something like this. This is the same concept that the animation tap in the bottom.

@8to16apps Good point. Doing this way would keep it consistent with the pin behaviour of the AnimationPlayer.

btw the pin button can simply toggle the editor setting, so you can keep your current changes.

btw the pin button can simply toggle the editor setting, so you can keep your current changes.

You're fast as I was about to comment: my implementation wouldn't change.
The Pin would simply be a shortcut to toggle my new setting.

We put pins on the right side, so I guess I would put it on the right side here too.

@reduz @8to16apps Done
https://user-images.githubusercontent.com/248383/180646125-b20d86cd-196b-4c9c-8bfa-2012599b5b5b.mp4

The commits need to be squashed before the PR can be merged. You can follow the instructions for that here: https://docs.godotengine.org/en/stable/community/contributing/pr_workflow.html#the-interactive-rebase
You can use git commit --amend in the subsequent commits to avoid this process.

Done

The pin feels like a lot of micromanagement for something that should probably work one way out of the box. Like Emi mentioned on Twitter, I'd probably get annoyed by the pin button wasting space and remove it with a plugin. I don't believe it's something you need to constantly change to justify having it so visible.

The pin feels like a lot of micromanagement for something that should probably work one way out of the box. Like Emi mentioned on Twitter, I'd probably get annoyed by the pin button wasting space and remove it with a plugin. I don't believe it's something you need to constantly change to justify having it so visible.

I'm neutral about it actually, since my intended solution is solved with or without the pin. So how things like this are normally decided? @YuriSizov

So how things like this are normally decided?

I guess we'll discuss it during the next PR review meeting. You are welcome to join, it's on Tuesday (watch out in #devel channel on RocketChat).

The pin has been removed in favor of keeping the first basic implementation of just the Editor setting (on by default), as per some Twitter conversation.

Thanks!

Can this be cherry-picked to 3.x branch?

Cherry-picked for 3.5.
@alfredbaudisch Just noticed it but for future commits, please try to keep the commit message title (first line) at a reasonable length. It should typically be less than 72 chars, the current one is 255 (i.e. your commit title is a full description instead of a title).

@akien-mga Thanks Rémi for fixing the commit message and for the tips! I'll keep those conventions in mind for possible future contributions.


Optimize encodeUvarint

Same as #419, but for encodeUvarint
(pprof) list github.com/cosmos/iavl.encodeUvarint
Total: 492.80GB
ROUTINE ======================== github.com/cosmos/iavl.encodeUvarint in /Users/cuonglm/Work/orijtech/iavl/encoding.go
   10.21GB    10.21GB (flat, cum)  2.07% of Total
         .          .     90:	return encodeUvarintSize(uint64(len(bz))) + len(bz)
         .          .     91:}
         .          .     92:
         .          .     93:// encodeUvarint writes a varint-encoded unsigned integer to an io.Writer.
         .          .     94:func encodeUvarint(w io.Writer, u uint64) error {
   10.21GB    10.21GB     95:	var buf [binary.MaxVarintLen64]byte
         .          .     96:	n := binary.PutUvarint(buf[:], u)
         .          .     97:	_, err := w.Write(buf[0:n])
         .          .     98:	return err
         .          .     99:}
         .          .    100:

If #420 merged, should we use the same pool for encodeUvarint and encodeVarint?
cc @odeke-em

If #420 merged, should we use the same pool for encodeUvarint and encodeVarint?
cc @odeke-em

Absolutely! As long as we track the n count of bytes used and returned we are good.


Update docs with info on running tests

Mention how to run pytype
Mention how to install Spanner Emulator + run tests
Add some files related to pytype and the emulator to .gitignore

CC @akefeli

I prefer to define environment variables on the same line I use them so
they only exist for that single command and don't change the rest of the
commands I run in the shell.
And the current instructions tell people to download the emulator into the
spanner-orm directory. It's not ideal but I did add the files to .gitignore
so they won't get accidentally committed. I'm open to other suggested
locations but downloading into the current directory hopefully works for
the most number of people.
On Mon, Oct 5, 2020 at 4:19 PM Daniel Gorelik<EMAIL_ADDRESS>wrote:

@dgorelik approved this pull request.
In README.md
https://github.com/google/python-spanner-orm/pull/89#discussion_r499825368
:

+ +git clone git@github.com:GoogleCloudPlatform/cloud-spanner-emulator.git +

+To check type annotations, run:
+
+ +pip install pytype +# https://github.com/google/pytype/issues/80#issuecomment-385128856 +pytype -V 3.7 spanner_orm -d import-error +
+
+Then run tests with:
+
+```
+SPANNER_EMULATOR_BINARY_PATH=$(pwd)/emulator_main python3 setup.py test
Should there be a newline in between emulator_main and python3? Also
should there be something about switching directories?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/google/python-spanner-orm/pull/89#pullrequestreview-502359152,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGUGXMCNCHB6DKTMGE4KJTSJIS3LANCNFSM4SFCPJFA
.

I prefer to define environment variables on the same line I use them so they only exist for that single command and don't change the rest of the commands I run in the shell. And the current instructions tell people to download the emulator into the spanner-orm directory. It's not ideal but I did add the files to .gitignore so they won't get accidentally committed. I'm open to other suggested locations but downloading into the current directory hopefully works for the most number of people.
…
On Mon, Oct 5, 2020 at 4:19 PM Daniel Gorelik @.> wrote: @.* approved this pull request. ------------------------------ In README.md <#89 (comment)> : > + +git clone ***@***.***:GoogleCloudPlatform/cloud-spanner-emulator.git + + +To check type annotations, run: + + +pip install pytype +# [google/pytype#80 (comment)](https://github.com/google/pytype/issues/80#issuecomment-385128856) +pytype -V 3.7 spanner_orm -d import-error + + +Then run tests with: + +``` +SPANNER_EMULATOR_BINARY_PATH=$(pwd)/emulator_main python3 setup.py test Should there be a newline in between emulator_main and python3? Also should there be something about switching directories? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#89 (review)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAGUGXMCNCHB6DKTMGE4KJTSJIS3LANCNFSM4SFCPJFA .

SGTM, I did not know about the ability to temporarily define env vars!


Fix - return coroutine value not last calling date

Hello,
Small fix for AsyncTTL decorator
When is used as
@AsyncTTL()
async def testing_func():
    return "Returning Value"

returnig_value = await testing_func()

function testing_func return datetime of last calling + live time of the testing_func not "Returning Value" itself

@jirka-tribi Thanks for adding this fix.


Comparing objects?

Why is does (using java-object-diff:0.94)
DiffNode diff = ObjectDifferBuilder.buildDefault().compare(
			new Person(1, "Bob", "London"),
			new Person(2, "Peter", "Paris"));

result in diff.hasChanges()==false?
import static java.lang.System.*;
import static org.assertj.core.api.Assertions.*;

import org.apache.commons.lang.builder.EqualsBuilder;
import org.apache.commons.lang.builder.HashCodeBuilder;
import org.junit.Test;

import de.danielbechler.diff.ObjectDifferBuilder;
import de.danielbechler.diff.node.DiffNode;

public class Compare {
	static public class Person {
		int id;
		String name;
		String city;

		public Person() {
		}

		Person(int id, String name, String city) {
			this.id = id;
			this.name = name;
			this.city = city;
		}

		@Override
		public boolean equals(Object obj) {
			if (!(obj instanceof Person)) {
				return false;
			}
			if (this == obj) {
				return true;
			}
			final Person otherObject = (Person) obj;
			return new EqualsBuilder().append(this.id, otherObject.id)
					.isEquals();
		}

		@Override
		public int hashCode() {
			return new HashCodeBuilder().append(id).toHashCode();
		}
	}

	@Test
	public void compareMaps() {
		DiffNode diff = ObjectDifferBuilder.buildDefault().compare(
				new Person(1, "Bob", "London"),
				new Person(2, "Peter", "Paris"));
		assertThat(diff.hasChanges()).isTrue();
	}
}

ha!  i ran into this same pretty quick after using this.  you're missing getters.  i really wish you didn't have to provide them as i'm using this lib mostly with my DTOs that are SUPER simple and didn't have getters/setters.

That's why I use lombok, I really dislike all that boilerplate code needed.

In this case I'm sure you guys will be happy about PR https://github.com/SQiShER/java-object-diff/pull/182, which aims to replace the current Introspector with a version that can deal with simple fields. The PR is more or less feature-complete but requires some more testing to make sure it doesn't break anything when it becomes the new default. Feedback is welcome.


E2E- Add to cart from plp

We must be able to add a product to cart from plp, navigate to checkout and verify that this product is available on /checkout

done at https://github.com/vtex-sites/base.store/pull/13


Redeploy and overwrite existing application

Hi,
I wanted to redeploy a newer version of my application and was expecting the uploader to simply overwrite the old one:
Warning: <exc:exception xmlns:exc="http://www.sap.com/abapxml/types/communicationframework">Page, controller, or MIME xxxx.JS already exists in application xxxxxPage, controller, or MIME xxxxx.JS already exists in application xxxxx</exc:exception> Use --force to continue.
It works for regular pages "Pages with Flow Logic", but not for mime objects (e.g. Pictures, Other javascript libraries etc.)
How do you handle redeploying and overwriting such existing applications?

Can you share your project?

My application is not a regular ui5 application, but a compiled react app (should not make a difference since ui5 is also just html / javascript?).
Is consists of 2 files which need to be uploaded:

This is my grunt config:
const webpackConfigDev = require('./config/webpack.dev.js');
const webpackConfigProd = require('./config/webpack.prod.js');

module.exports = function (grunt) {
    var sUser = grunt.option('sap_user');
    var sPwd = grunt.option('sap_password');
    // Project configuration.
    grunt.initConfig({
        webpack: {
            prod: webpackConfigProd,
        },
        nwabap_ui5uploader: {
            options: {
                conn: {
                    server: 'http://server:80',
                },
                auth: {
                    user: sUser,
                    pwd: sPwd
                }
            },
            upload_build: {
                options: {
                    ui5: {
                        package: 'ZPACKAGE',
                        bspcontainer: 'ZBSPPAGE',
                        bspcontainer_text: 'Description',
                        create_transport: true,
                        transport_use_user_match: true,
                        transport_text: 'Transport for xxx container'
                    },
                    resources: {
                        cwd: 'dist',
                        src: '**/*.*'
                    }
                }
            }
        }
    });

    grunt.loadNpmTasks('grunt-webpack');
    grunt.loadNpmTasks('grunt-nwabap-ui5uploader');
};

In the meantime I "fixed" the issue by simply deleting main.bundle.js in the BSP repository. After this, updating works fine, even if the file exists. Strange ..
Running "nwabap_ui5uploader:upload_build" (nwabap_ui5uploader) task
NW ABAP UI5 Uploader: file /index.html updated.
NW ABAP UI5 Uploader: file /main.bundle.js updated.

Done.


added docker-compose run alias

added "docker-compose run" alias

i just saw there is a similar PR https://github.com/robbyrussell/oh-my-zsh/pull/5422
please use this.


Compression an injection feature request

Hello, thanks for sharing!
Is there any plan to implement next features:

possibility to compress/minify  output css
bundle css together with bundle.js (aka inject feature webpack loaders apply)

I guess there is possibility to do point 1 one by own handler inside output() method, right?

It would be nice if the plugin would look at process.env.NODE_ENV and look for minifier in package.json and would enable minification.
Could be an option

I'm open to backwards compatible PRs 👍

See https://github.com/thgh/rollup-plugin-scss/tree/v3#examples
There is now an insert option

See https://github.com/thgh/rollup-plugin-scss/tree/v3#examples
There is now an insert option

when is the insert option，the examples can't find this.


[Backport release-22.05] libuiohook: init at 1.2.2

This is a manual backport of #181660 to release-22.05.

@ofborg build


Updated version and CHANGELOG for prerelease of wct-mocha

A version of wct-mocha 1.0.0-pre.1 was accidentally published as "latest" (and then unpublished) a month or so ago, which consumed the 1.0.0-pre.1 version, so this will be published as 1.0.0-pre.2

Thx @aomarks


List unofficial Elixir client

Link: https://github.com/deadtrickster/prometheus.ex
Thanks!

Thanks!

Thanks, I'd suggest a read of the client library guidelines in particular the section around Children. Users should be passing around Counter objects, not specifying the name of the counter on every call.
https://prometheus.io/docs/instrumenting/writing_clientlibs/#overall-structure

@brian-brazil hey, glad you found a minute to take a look at the library. Erlang/Elixir are pure function (c) i.e. without any state/object/etc. You can store state in a process loop or so called Erlang Term Storage, basically awesomely cool hash-table/bag. I'm using ETS to store metrics, each object here is a tuple {Key, ....}
My keys are {Registry, Name, LabelValues} this is why they always there. I tried to overcome this with as many shortcuts as possible e.g.


Decode a Morse code message

Dear Reviewers,
We've worked on the following in this pull request,

Set Up linters
Create morseCode hash
Create transform_morse_code method

Thanks to reviewers @brainconnect93 and @AbelG101 for the reviews. 😁


Framebuffer does not release depth buffer when destroyed, causing GPU memory leak

I noticed that if you (manually) add a depth texture to a Framebuffer, that texture does NOT get cleaned up by calling destroy() on the framebuffer, although arguably, it should, since it belongs to the buffer.

Expected Behavior
The depth texture gets destroyed alongside all the color textures.
Current Behavior
The color textures get destroyed while the depth texture doesn't, causing a memory leak.
Possible Solution
Add destruction of the depth texture when destroying the framebuffer.
Steps to Reproduce
I created a commented Fiddle that can help with reproducing the problem:
https://jsfiddle.net/ncj95301/
Click "Start", then check your VRAM usage, it should go up quite quickly.
Environment

pixi.js version: 5.2.1
Browser & Version: Chrome 80.0.3987.163
OS & Version: Windows 10.0.18362 Build 18362
Running Example: _https://jsfiddle.net/ncj95301/_

Ha! First user of DepthTexture that I see ! I really dont know the demo it was used before.. OK then, time to find that bug.
Yes, it blongs to the framebuffer: https://github.com/pixijs/pixi.js/blob/dev/packages/core/src/framebuffer/FramebufferSystem.ts#L517 . Yes, as you said, in webgl its separate, but in PixiJS it appears only with framebuffer, its fine to dispose it there.

Haha yeah man, people who DepthTexture might be rare, but we exist! I'm actually surprised they're rarely used, since depth buffers have always been an important part of computer graphics.

@bigtimebuddy This is actually a structural problem with Pixi. The depth-texture is not owned by the framebuffer; the color textures aren't freed either. If you call RenderTexture#destroy, it destroys itself (which happens to be the only color texture) and disposes the framebuffer.

@bigtimebuddy I suggest that RenderTexture#destroy actually does the work of destroying the depth texture. See my proposed fix: https://github.com/pixijs/pixi.js/pull/6561

@Vanilagy Please verify the fix: https://jsfiddle.net/ShukantPal/6jqsfa7v/

Verified, the leak is gone.


What is the reason for the author's error and how can I avoid it

The reason for the issue is that LLM did not return the data in the required format: the "Programming Language" field is missing.
This problem can only be resolved by improving the performance of LLM, either by changing the model or upgrading its version.


[Feature Request] UnoCSS support

Is your feature request related to a problem? Please describe.
I usually use unocss for development, and some existing projects also use unocss, which will lead to the inability to use nextui due to a css solution
Describe the solution you'd like
support unocss
Describe alternatives you've considered
maybe provide a version without tailwind
Screenshots or Videos
No response

Hey @Yiniau I appreciate your suggestion, however removing TailwindCSS to provide support to X frameworks means re-doing all components styles again, which is definitely not a target for us. From my understanding UnoCSS supports TailwindCSS classes so you could give it a try, but I haven't tested it yet.


extra nulls in error messages from barback

Not sure where these are coming from, but I'm seeing extra nulls in the error messages from barback:
    import 'package:barback/barback.dart';

    class A extends Transformer {
      A.asPlugin();
      isPrimary(id) => true;

      apply(Transform transform) =>
        transform.getInput(new AssetId('smoke', 'lib/broken.path'));
    }
running pub-build prints these errors:
Build error:
Transform A on test|lib/transformer.dart threw error: Could not find asset smoke|lib/broken.path.
null
null
Build error:
Transform A on test|web/a.dart threw error: Could not find asset smoke|lib/broken.path.
null
null
Build failed.
I see one instead of 2 nulls if I catch the error and print the contents myself. For example:
    import 'package:barback/barback.dart';

    class A extends Transformer {
      A.asPlugin();
      isPrimary(id) => true;

      apply(Transform transform) =>
        transform.getInput(new AssetId('smoke', 'lib/broken.path'))
            .catchError((e) => print("$e"));
    }
prints:
Building test... Could not find asset smoke|lib/broken.path.
null
Could not find asset smoke|lib/broken.path.
null
Building test...

This issue has been moved to dart-lang/barback#58.


GNOME3 on master lacks ability to "Log Out"

Issue description
On NixOS 16.09 with a GNOME3 desktop, you can click in the upper right corner and select <your user> -> Log Out. But on master branch the whole <your user> menu seems to have vanished. I think this happened with the gnome 3.20 -> 3.22 upgrade, but not sure.

Try pressing the Alt key when the menu is open.

When pressing Alt, the "power" button changes to a "pause" button. Nothing else changes.

Oh right, that was the suspend function. It's been a while since I used gnome3 ....

I've just got a gnome3 open, from current master. It has the user menu along with the logout option. Could you try to start with a fresh user profile?

I'm on 17.03pre100522.01fef3f (Gorilla) and logout works fine for me

Hm, it looks like a new (intended) behaviour in GNOME. If I add another user to my system, the user menu (with logout button) appears. But if I remove that user, the menu disappears.
It is actually possible to test this "live". Add/remove user in /etc/nixos/configuration.nix and run nixos-rebuild test. The user menu / logout button (dis)appears immediately.
I guess it makes a bit sense to disable logout when there is only a single user (although I'd prefer to always have it there, for consistency), but the user menu also contains "Account Settings". So they are killing that shortcut too...
Glad to see that this is not a NixOS issue as I first thought.


Update tapir-core, tapir-http4s-server, ... to 1.11.5

About this PR
📦 Updates

com.softwaremill.sttp.tapir:tapir-core
com.softwaremill.sttp.tapir:tapir-http4s-server
com.softwaremill.sttp.tapir:tapir-json-circe
com.softwaremill.sttp.tapir:tapir-openapi-docs
com.softwaremill.sttp.tapir:tapir-refined
com.softwaremill.sttp.tapir:tapir-swagger-ui

from 1.11.4 to 1.11.5
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.softwaremill.sttp.tapir" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.softwaremill.sttp.tapir" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, version-scheme:early-semver, commit-count:1

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 48.19%. Comparing base (995a6da) to head (262ee9d).
Report is 2 commits behind head on develop.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #589   +/-   ##
========================================
  Coverage    48.19%   48.19%
========================================
  Files            5        5
  Lines           83       83
  Branches         6        6
========================================
  Hits            40       40
  Misses          43       43

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


backport golang.org/x/net bump

What does this PR do?
This PR backport https://github.com/DataDog/datadog-agent/pull/14040 to 7.40.x branch
QA
Here are the packages where golang.orgx/net is used:

cmd/manager to be tested by container-integrations
pkg/autodiscovery/providers to be tested by container-integrations
pkg/logs/client/tcp to be tested by agent-metrics-log
pkg/network/dns to be tested by network
pkg/network/filter to be tested by network
pkg/network/netlink to be tested by network
pkg/util/http to be tested by agent-share-components

CI is green, merging


Print without saving in manual mode

The Hello Kitty rom has a cool feature: when you take a picture, you can save it OR print it without saving by pressing START at the confirmation screen instead of A. It would be use to make manual time lapses I think. Once again, it allows stuffing a printer emulator (infinite saving) instead of the sram.

It 's close to #28 but not the same.


refactor: conver NpcType and ObjType to ts [wip]

Got a bit done on this, ObjType causes some errors in the Player class (invSet, decodeIn, onCheat and invAdd) functions

Line 558: Argument of type 'ObjType' is not assignable to parameter of type 'ScriptArgument[]'
Line 662: Argument of type 'ObjType' is not assignable to parameter of type 'ScriptArgument[]'
Line 798: 'objType' is possibly 'null'
Line 1946: Type 'number | undefined' is not assignable to type 'string | number'

I'm pretty sure I've converted the TS correctly but I don't want to change engine code without getting some input 💬
There are two causes:

ScriptRunner.init is being passed an ObjType as a ScriptArgument. Is this permitted? It seems like ScriptArguments can only be number or string.
invAdd and invSet call ObjType.getId and check for -1 as an "invalid" state. getId can return undefined, we need to apply ?? -1 to the getId lookup if we want reliable failure results.    (probably need to do a bit of a healthcheck of the existing lookup functions, though TS conversions will help highlight these)

Will pick up any changes needed tomorrow, please let me know your suggested solutions and I can make them. Alternatively feel free to make the changes in JS and I can merge them into my branch for TS conversion.

The two ScriptRunner.init taking objType can just be ScriptRunner.init(script, this). These scripts shouldn't be receiving any arguments.

@Joshua-F I converted the two classes to use ConfigType base.
One issue: in ObjType, the decoding of countco relies on count, which isn't made available by the base ConfigType.
I could pass it in like this, thoughts?
        let dat = Packet.load(`${dir}/obj.dat`);
        let count = dat.g2();

        for (let id = 0; id < count; id++) {
            let config = new ObjType(id);
            config.decodeType(dat, count);

Changes made, ready for review

tested and working ✔  ready for merge


[Question] Data standardization / scaling and inverse_transform on predicted series

To be honest, I'm not a data scientist. Just curious: if standardization is applied on the inputs:
https://github.com/Secilia-Cxy/SOFTS/blob/011c805c4b040b0e6431ee3a2451348729bd87ae/run_on_your_own_dataset.ipynb#L171-L181
Would it be necessary or better off to perform inverse_transformation on the predictions?
https://github.com/Secilia-Cxy/SOFTS/blob/011c805c4b040b0e6431ee3a2451348729bd87ae/run_on_your_own_dataset.ipynb#L580-L582
E.g.
scaler.inverse_transform(predictions)

In practical applications, it is indeed possible to use inverse_transform on the predictions. However, the reason we do not use inverse_transform in this context is that when calculating the Mean Squared Error (MSE), applying the inverse transformation can cause the MSE of certain channels to be particularly large due to the inherently high values in those channels. This would not accurately reflect the model's average predictive performance across all channels. Therefore, we choose to compute MSE on the standardized data to better assess the overall predictive ability of the model.

In practical applications, it is indeed possible to use inverse_transform on the predictions. However, the reason we do not use inverse_transform in this context is that when calculating the Mean Squared Error (MSE), applying the inverse transformation can cause the MSE of certain channels to be particularly large due to the inherently high values in those channels. This would not accurately reflect the model's average predictive performance across all channels. Therefore, we choose to compute MSE on the standardized data to better assess the overall predictive ability of the model.

This is a research convention to ensure a fair comparison among channels of different scales. See the first paper to come up with the benchmark and its code.


Add Elastic Stack deployment for flow visualization

This PR is a follow-up of PR #825  for adding Elastic Stack manifests and doc for collecting IPFIX flow records and visualize in Kibana dashboard.

@antoninbas Since we want to modify the cache_ttl parameter in the config files, I adopted the first solution we discussed last time. Reason for the need to modify that parameter is -- tcp template in logstash should not be removed according to RFC, but the Netflow plugin of logstash will clear template cache after some time (by default 4000 seconds). So we decided to adjust that value to be the largest possible. Because the parameter change involves modification of original config files for elastiflow, possible alternatives are 1) upload only the elastiflow image or 2) upload both config files and ipfix definitions. Solution 2) is more light-weight, should I change to that way?

If we need to build a custom docker image, we should still be able to have a much simpler process that doesn't involve checking-in all the files: your Dockerfile can inherit from the official image (FROM robcowart/elastiflow-logstash-oss:3.5.3) and simply overwrite the files you need to overwrite (ipfix.yaml and some configuration file under conf.d?).
The alternative is the ConfigMap approach we discussed previously, but it gets a bit complicated if multiple files are involved, so I am fine with the above approach, as long as find a way to not copy all the files from the elastiflow repo. In addition to making this repo larger and more complex, there may be some licensing issues (although I haven't looked at the license in details)?

BTW the docker image that you build should have a different name I think (not robcowart/elastiflow-logstash-oss:3.5.3, but maybe antrea/elastiflow-logstash:3.5.3).

@antoninbas Thanks for the suggestions. Yes it makes sense to not copying all the elastiflow files and building our own image may cause extra work and license issues. I tried to combine the configs into one file and simplify the Configmap(so we don't need to upload netflow definitions). In this case we don't need to build custom image. pls have a look whether this solution is acceptable.

@zyiou this looks fine, but what about the cache_ttl parameter you mentioned previously?

@antoninbas I put it here. This conf file will overwrite all the config files previously in conf.d folder

/test-all

/test-all

/test-all

/test-all

Merging this as there are approvals and Antonin's final comment has been addressed.


A few requests / issues

Hi lgrees, thank you for making resy-cli available to all and opensource.
I had a few questions / requests:

Currently it is very difficult to select the venue when scheduling, can it be made to just require the venue_id  ?
Is there a way to delete scheduled bookings?
Is there a way to book using the 'resy schedule' command without having to go through the whole dialogue, perhaps so we can just add flags to the intial command?

Many thanks!

I would also love a way to list out the currently scheduled bookings and cancel/delete them.


Removing operations of mentorship program

The operations page is out of date so I am deleting it

Having trouble getting the links check to pass. I'm going to close this PR and try a new one.


Attempt to serialize unknown objects

Maybe this would be too much complexity. But I was thinking it might be useful to add a "serialize" flag, and if the write function didn't recognize the input object's class, it would then try to serialize it with cPickle rather than throw an error. The read function could have something similar, so that in the event that unpickling the data became broken, the user could still read in all the other data. E.g., store them in a node called "pickle_objectname". Doesn't tables do something like this?

not sure but I smell maintenance efforts with this sort of magic API where you guess default stuff. H5py is not necessarily efficient with unfavorable data types. Already list of lists can be terribly slow. This is really a bare bones package for our internal needs, as long as you don't want to make some strong commitments to maintenance keep the profile flat ;) my 2 cents.

yeah I definitely understand your concerns. I just find myself using h5io more and more often these days because it's a lot easier than writing all your own tables/h5py code. In my mind, it wouldn't require much more maintenance because it's basically a last-ditch effort at storing things in the same object if h5io doesn't know what to do with them.
The mentality would be "if you're saving an unsupported object then we'll try to serialize it for you, but won't go any further" aka, if the serializing doesn't work then it's the user's problem.
An example of how this might be useful: right now there's a discussion about how to handle the I/O for the timegen class. The problem is that there are some things that would be super easy (e.g., the generalization scores, the model parameters, etc) and some things that aren't easy at all (e.g., the CV object in scikit-learn or the model coefficients). With something like this you could just say "store as much of the timegen object as you can easily, and the rest just pickle it".

Closing this because it doesn't seem like there interest in implementing


Use leading logical operators in multiline statement

:smiling_imp: :rabbit:
It's unfortunate that it requires line continuations to make this possible, but I still feel like this style increases readability quite a bit over the trailing operator alternative.
If people are into this, we can do something similar in JavaScript (without the line continuation!).
For example:
validSelection(target) {
  const classList = target.classList;

  return classList.contains('z-attachable-menu__item')
    || classList.contains('z-attachable-menu__item__title')
    || classList.contains('z-attachable-menu__item__value');
}

@zendesk/darko

Oh yeah, there's also an eslint option for this.

but the line continuations tho 😦

but the line continuations tho 😦

I know, I know, but sometimes you gotta experiment.

👍 let's do this


[docs] Add classNames demos in ts

Sorry but I don't understand why test_types is failing, can someone help me?

Sorry but I don't understand why test_types is failing, can someone help me?

The javascript version is generated automatically using yarn docs:typescript:formatted, run that and commit the result and it will stop complaining

@oliviertassinari Doesn’t the withStyles -> makeStyles need to happen first?

@joshwooding I don't think it's important given #16947. I'm happy to see the TypeScript migration.

@lksilva Thanks


Dataset request

Hello. Could you provide the image and mask datasets used in the "u-net-auto.ipynb"

Hello @caneatonly, this video link will help you download the files first and then you can use Autolabeling_IS2_using_S2_iqr_2023.ipynb to help you create your own images/mask datasets.


Remove outdated CPU information completely

I pulled the trigger too fast on PR https://github.com/powdahound/ec2instances.info/pull/99 and props to @powdahound for merging it before I could fix it ;-)

Doh - didn't realize you had more planned. :) You think this is worth removing because Amazon no longer mentions what the processor types are (presumably because they're changing/changed)?

Yah and because of https://github.com/powdahound/ec2instances.info/issues/97

I think as part of this PR the rendering code looking at cpu_details should get removed since that data won't exist for any instance. I suppose they're all just "Latest Xeon" now and we won't know anything more specific. (Makes sense that they'd obscure it)

Well they do still specify for the latest instance types
http://aws.amazon.com/ec2/instance-types/
but I suppose the cpu_details section is not looking at that?

Yeah, cpu_details was never filled with any scraped data I guess. So I suppose another option would be to populate it with the available data, though I'm not sure how much the processor specifics are relevant to people in the context of this site.

Yah, probably not very high value. Anything else I can do for this PR?

Nope - let's ship it!

Oh and if someone comes along this in the future and wishes we'd decided to add CPU details: feel free to resurrect the relevant code here and open another PR. :)


Mentorship selection process criteria

One of the mentees provided feedback on slack channel that the selection process was not clear for the 2nd beta round. We are still experimenting with the best process for selection. we are getting important feedback on the process from mentors and mentees and adjusting accordingly.
Selection criteria:
This is the current general criteria we are following to choose mentees
Mentors contribute in choosing their mentee based on the following:

Level of experience:
Mentor's preference in the level of experience of the mentee.
Time availability:
At least 2-3 hours intersecting available time between the mentor and mentee.
Providing Feedback:
Mentors go through every applicant that chose their mentorship topic and read their bios and provide feedback to the mentorship team.

Mentorship team selection criteria:

Diversity and Inclusivity:
We ensure that the overall cohort is diverse and inclusive of everyone with different backgrounds, skill sets and experience levels.
Level of interest Contributing back to the community:
Sharing knowledge is the essence of the mentorship program. We are looking for mentees who are willing to share resources about their experience with mentorship and will help anyone in the community and those who did not get the opportunity to get matched this cohort. Resources may vary from videos, blog posts, issues, or even talks and sessions!
Interest level in Node.js and the program:
We are looking at the following metrics to gague the level of interest and commitment of the mentees: Mentioning of OSS contribution, The community, interest in nodeJS, how they are using nodeJS, and why they are joining the program.
Github contributions:
We do check github profile in some occasions to get an indication of activity such as previous contributions to OSS.

i think its important to say that the entire process was 100% transparent and can be checked in the notes at https://github.com/nodejs/mentorship/tree/master/meeting
One more thing, this round is 1 on 1. So from all the applicants only 10 were selected by the mentioned criteria. We are loking for a way to improve the mentors number and to improve the number os pairing mentors 1:n mentees


Header is NOT responsive

Hi,
I am trying web-starter-kit-0.6.0. When I do gulp server, the page comes up, the header part is not responsive.
Please refer the screen shot attached.

Thanks,
Imteyaz Ahmad

What do you expect exactly?

Hi Garbee,
In desktop view, we get "TECHNOLOGY" and "FAQ" section like below

but on re-sizing it, only "TECHN"  from "TECHNOLOGY" is visible and "FAQ" section is NOT visible at all.
Thanks,
Imteyaz Ahmad

It is working as expected. You need to press & scroll to see the rest of the menu.
On a side note: Device Mode is way better for mobile testing than resizing the viewport. Check it out if you haven't already.

I see what you mean by press and scroll however there really should be some indication to the user that they could scroll the nav horizontally. e.g. first time they visit the site and it's on a mobile device, they would never know that there where other top level pages.

The spec shows we shouldn't have an indicator on mobile. However, we should have something on desktop for users to interact with.
File an issue upstream with Material Design Lite and we can try to get that part fixed. Nothing WSK can do about it since it is simply requiring MDL as a dep for base styling.

Closing as an MDL issue. Ideally, all Material Lite related bugs and feature requests should be filed upstream please :+1:


add favicon

Following tutorial Adding a favicon to a Jekyll-based static website by Paul Cochrane to add a favicon. This will add a little icon in the tab of the blog website!

LGTM! I like how you included the favicon for other platforms.


Discovered device does not added to HomeKit automatically

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version):
0.66.0b1
Python release (python3 --version):
3.6.4
Component/platform:
homekit
discovery
Description of problem:
I have two light setup, one is static light.mqtt and another is auto-discovered light.yeelight. After pairing HA with HomeKit, the MQTT light showing correctly, but the discovered one does not. Seems that HomeKit does not support discovery right now.

Have you tried using a delay before starting the HomeKit platform?

It should work if HomeKit starts after device discovered. HomeKit only scan for devices when it starts.

Does it work now, with a delay?
https://rc.home-assistant.io/components/homekit/

It should work if use the delay start feature.
I switched the auto-discovered component to static configure.

I'm closing this on then.


Add CustomServiceEnpoint support

This PR contains the changes to add service override support for Power VS cloud provider.

/hold

As discussed lets work with the upstream codebase and merge the possible code changes there


wc: print counts for each file as soon as computed

This pull request changes the behavior of wc to print the counts for a file as soon as it is computed, instead of waiting to compute the counts for all files before writing any output to stdout. The new behavior matches the behavior of GNU wc.
The old behavior looked like this (the word "hello" is entered on stdin):
$ wc emptyfile.txt -
hello
      0       0       0 emptyfile.txt
      1       1       6 -
      1       1       6 total

The new behavior looks like this:
$ wc emptyfile.txt -
      0       0       0 emptyfile.txt
hello
      1       1       6 -
      1       1       6 total

Looks good! (test failure is unrelated)