Modify selection to stop highlight at new-line (instead of continuing to end-of-screen)

Continued work by @vjeux from https://github.com/atom/atom/pull/8640
It's mostly the same as what @vjeux had but I further extend the selection to select the new line (as it does in sublime and vs code).

Some general direction on how to go from there and I don't mind going through the specs and adjusting.
Now.. I'm not sure how to get the rounded corners that VS Code and Sublime have on the selection. I tried playing with border-radius and it sort of works but we need these triangles to fill in the bottom-right border radius (but only if the line is shorter than the line under it). An idea I had is we could perhaps switch to constructing a single SVG Path that is filled for selection. Perhaps @simurai knows a creative method to make it work in CSS. If there is a reasonable method to do this with CSS; the rounded selection could be left up to a theme to implement (and leave corner with pointy corner selection).

A question before I delve too much deeper. Is this something that would be accepted if we figure out a good solution? @nathansobo @benogle

@mehcode for your spec failures you might want to take a look at this issue: #10719 (specifically this comment).

Thanks @50Wliu; was having the exact problem.

@mehcode can you post a screenshot to illustrate what you have in mind?

Oh I see some screenshots in the referenced PR. It's funny. We worked pretty hard to go full-width with the selection because that's how TextMate and some other OS X apps looked. Can you talk about why you prefer it to be the other way? I'd prefer not to rock the boat unless there's a good reason.

You could overlap 2 pseudo elements and then add a border-radius to one, like:
atom-text-editor::shadow .selection .region {
  border-top-right-radius: .33em;

  &:before,
  &:after {
    content: "";
    position: absolute;
    top: 0;
    bottom: 0;
    right: -1em;
    width: 1em;
  }
  &:before {
    background-color: @syntax-selection-color;
  }
  &:after {
    background-color: @syntax-background-color;
    border-bottom-left-radius: .33em;
  }
}

But you would need some extra elements if there is more than one corner. SVG might work too, yeah. I guess most important is to not slow down performance. Could this be done as a separate package to test first or does it need to be implemented in core?

Can you talk about why you prefer it to be the other way?

On sublime I'd use the close selection feature to show trailing white-space.
Sublime also has a default config of show invisible on selection.
Here's a screenshot of the same text on both editor, see that for someone interested in white space sublime show a lot more useful informations.

@simurai I think this would be pretty tough to do in a package with current APIs. With clever interaction with the scheduler it might be possible, but it's probably something we should consider changing in core if there's popular demand. Assuming we could apply any CSS classes you wanted, any ideas for creating the desired rounding effect without resorting to SVG?

@jeancroy I have completely forgotten about ST's "show invisible chars only in selection", which is SO useful and makes the editor look so clearer.
From a user POV, I think there should be 2 settings for this:

Enable/disable this (not everybody will like this feature)
Enable/disable showing invisible chars only on selection.

@jeancroy That's a great point.
Trailing whitespace is a lot easier to see with sublime / VS Code / Visual Studio / Eclipse selection highlighting.
My other point was going to be all other editors do it this way (even chrome text areas). But apparently TextMate does not. That seems to be an aberration.
I'd don't have another point. I was thinking this was an oversight as most editors I look at do it one way and I guess TextMate does it another.

Here is how Atom looks on this branch. Slightly different than how it looks on the linked PR.

Does using class for rounding means to conditionally apply classes depending of the length of previous / next line ?

@jeancroy Does using class for rounding means to conditionally apply classes depending of the length of previous / next line ?

Yes, it would need to check if the previous and next line is longer or shorter and then add the a class(es) for it. Since there could be an inverted corner left and right, each .selection .region could have two extra child elements.

Then have some classes. Not sure how to call them, maybe:

.has-corner-top-left or .has-corner-top-left--outwards
.has-corner-top-right or .has-corner-top-right--outwards
.has-corner-bottom-left or  .has-corner-bottom-left--outwards
.has-corner-bottom-right or .has-corner-bottom-right--outwards

Then core or themes could apply the right border-radius.


Not working in Ubuntu 14

First of all thank you for the great work!
I'm experiencing some problems setting the bare-bone project up in Ubuntu:
----------------------- 1st problem ------------------------
My avr-gcc version is 4.8.2, which in the Arduino.mk makes it try avr-gcc-ar
ifeq ($(shell expr $(CC_VERNUM) '>' 482), 1)
    AR_NAME      = avr-gcc-ar
else
    AR_NAME      = avr-ar
endif

which produces the following error when trying to build:
'sorry - this program has been built without plugin support'
I changed $(shell expr $(CC_VERNUM) '>' 482 for
$(shell expr $(CC_VERNUM) '>' 483
to make it use avr-gcc, which compiles without errors
----------------------- 2nd problem ------------------------
Same version problem for avr dude
        ifndef AVRDUDE
            ifeq ($(shell expr $(ARDUINO_VERSION) '>' 157), 1)
                # 1.5.8 has different location than all prior versions!
                AVRDUDE = $(AVR_TOOLS_DIR)/bin/avrdude
            else
                AVRDUDE = $(AVR_TOOLS_DIR)/../avrdude
            endif
        endif

        ifndef AVRDUDE_CONF
            ifeq ($(shell expr $(ARDUINO_VERSION) '>' 157), 1)
                AVRDUDE_CONF = $(AVR_TOOLS_DIR)/etc/avrdude.conf
            else
                AVRDUDE_CONF = $(AVR_TOOLS_DIR)/../avrdude.conf
            endif
        endif

I have version 105 but the paths are just the other, so I changed
($(shell expr $(ARDUINO_VERSION) '>' 157)
for
($(shell expr $(ARDUINO_VERSION) '>' 100)

I know is a dirty work around, and that's why I reported the issue so you guys can think for a better fix! Thank you again for the great work

Ok, the second problem I think it was my fault. I installed arduino long time ago and I may had changed the directories. The first problem though, no clue why I get the 'sorry - this program has been built without plugin support' error

thanks for the feedback.
this is more of an upstream bug from what I see. I'll look into it and report it if it's the case.


Server crash with player login

Whenever one of our players logs onto our server it crashes. We ended up having to delete the region.
https://pastebin.com/qdfHP2jQ

duplicate of #168 (update the mod)


Automated cherry pick of #6636: fix: ignore match weights when searching matched policysets

Cherry pick of #6636 on release/3.1.
#6636: fix: ignore match weights when searching matched policysets

Codecov Report

Merging #6655 into release/3.1 will increase coverage by 0.00%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           release/3.1   #6655   +/-   ##
===========================================
  Coverage         7.48%   7.49%
===========================================
  Files              585     585
  Lines            98953   98949    -4
===========================================
+ Hits              7411    7418    +7
+ Misses           90970   90957   -13
- Partials           572     574    +2

Flag
Coverage Δ

#aFlag
7.49% <0.00%> (+<0.01%)
:arrow_up:

Impacted Files
Coverage Δ

pkg/cloudcommon/policy/resources.go
0.00% <ø> (ø)

pkg/cloudprovider/objectstore.go
2.51% <0.00%> (-0.01%)
:arrow_down:

pkg/compute/guestdrivers/managedvirtual.go
0.00% <0.00%> (ø)

pkg/compute/models/buckets.go
1.24% <0.00%> (ø)

pkg/util/rbacutils/policyset.go
75.00% <0.00%> (+9.78%)
:arrow_up:

pkg/util/rbacutils/rbac.go
55.04% <0.00%> (-0.70%)
:arrow_down:

pkg/appsrv/response.go
66.17% <0.00%> (-2.95%)
:arrow_down:

pkg/cloudcommon/cronman/cronman.go
54.85% <0.00%> (+5.33%)
:arrow_up:

Continue to review full report at Codecov.

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


Add documentation about adding a new product to an existing provider

@alambare I've pushed an initial version of the requested document where I show how to add an entry to product_types.yaml. Is there anything else do do? Until recently I would have thought that it would also be necessary to list the product in providers.yml but while working on another pull request I was told that this actually isn't required. If that is indeed not required, then I'm wondering how EODag can decide what providers support what product types.


Trader-Thumbnail for Desktop-View

When generating a thumbnail of an image we need a intermediate size for dektop view.
Generate new thumbnail sizes for already existing images

What size should this new thumbnail be? Currently we are creating one of the size 200x200.

the desired size is 224x224.
Right now the larger side of the image is scaled to 200px and the smaller side is scaled accordingly to preserve aspect ratio. That is fine, but we have a lot of images that endup having a size of 133x200. That is being upscaled to 224x358 (or so).
We need the smaller side of the image to match the desired size.

Okay. This is not possible withe the resize extension we are using. I will replace the extension with a function. This might also solve the Problem described here #6

nice!!


Android tablets without hardware menu keys

like on Jelly Bean 4.1.2 ...?
there is NO hardware key on the Nexus 7 (2012)
suggest "poking" the background to open the menu

poking the edges should be fine on smaller devices, but the tablets will have lots of empty space


Incorrect instructions

In Step 2, When I view the Data Management section, I don't see  "select Enable access tracking", the instructions need to be updated to include something like, "on the Lifecycle Management page, select "

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

ID: 171a0dc0-35ad-be69-023c-4943195c0d7f
Version Independent ID: 0699b9e6-c72b-3527-7215-f629242828f5
Content: Configure a lifecycle management policy - Azure Storage
Content Source: articles/storage/blobs/lifecycle-management-policy-configure.md
Service: storage
Sub-service: common
GitHub Login: @tamram
Microsoft Alias: tamram

@DeanGross Thanks for your question. We are checking on this and will respond to you soon.

@DeanGross The PR request has been initiated! We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Added Github card

What kind of changes does this PR include?

Added a Github Card (::github{repo="chrismwilliams/astro-theme-cactus" leaf directive) https://talk.commonmark.org/t/generic-directives-plugins-syntax/444#:~:text=2. Leaf Block Directives

Description

Added the remark plugin to handle leaf directives for the Github card. (Using the refractored remark-admonitions.ts as a template, no new deps)
Added the css to handle the card + inlined the icons
The card loads with a pulsating skeleton, when the content is fetched it gets populated. If it cant reach the Github API (connection, rete-limit, etc) only the title/link is shown.
Automatically converts a link or just the username/repo.

It seems that for some reason the remark-admonitions.ts transformUnhandledDirective function breaks the leaf directives. Putting it before the remarkAdmonitions plugin makes the plugin work.


Be able to pull in code samples from googlechrome/samples

We had this in H5R - would like it in Web Fundamentals

I'm going to move this to v2, doesn't seem like a ship stopper.

Not going to implement.


Create a bundle to allow third party app to use the carsonbot

What do you think about creating a CarsonbotBundle to allow people to use it in a regular app ?

:-1: CarsonBot as a feature in an application is not usefull, carson bot is an app on its own.
You can clone this repository locally, create a github account, use it in the code and run it on a web server and set up the GitHub events to trigger carson bot.

Well you're right, carsonbot as a bundle is probably not really useful in the current state as it is not customizable...


test: Bug#10757 automation

Test coverage results :test_tube:
:white_circle: Total coverage has not changed
No changes to code coverage between the base branch and the head branch

/ok-to-test sha = ed1a1ff

/ok-to-test sha=ed1a1ff


Add nix flake

🗣 Description
This PR adds flake.nix to allow for the use of awssh without the following:

Requiring Python to be installed on the host system
Executing setup-env script to install awssh
Managing, starting  or maintaining virtual environments for Python

This does not require or use poetry

💭 Motivation and context
The motivation and purpose of this is to provide access to our tools quickly. Considering the recent re-provisioning of my machine I find it convenient to simply run. nix build so I can execute awssh without having to configure virtualenvs or making other system changes.  This is just the beginning.

🧪 Testing

The  binary was executed and I was able to successfully start a session as well as execute different awssh options. I've included a screenshot.

✅ Pre-approval checklist

[X] This PR has an informative and human-readable title.
[X] Changes are limited to a single goal - eschew scope creep!
[X] All relevant type-of-change labels have been added.
[X] I have read the CONTRIBUTING document.
[X] These code changes follow cisagov code standards.
[X] All relevant repo and/or project documentation has been updated to reflect the changes in this PR.

@jasonodoom Please complete the "Testing" section of the PR description and explain how you tested these changes and verified that they do not negatively impact existing functionality.

FYI, I saw that you removed the pre- and post-merge checklists from the PR template.  I added them back in, since this change is worthy of a minor version bump.  Please make sure that you take care of that.

FYI, I saw that you removed the pre- and post-merge checklists from the PR template.  I added them back in, since this change is worthy of a minor version bump.  Please make sure that you take care of that.

@dav3r Is it though? This PR does not touch the Python code so I'm not sure it should require any version modifications. We do not increment for pre-commit changes for example.

FYI, I saw that you removed the pre- and post-merge checklists from the PR template.  I added them back in, since this change is worthy of a minor version bump.  Please make sure that you take care of that.

@dav3r Is it though? This PR does not touch the Python code so I'm not sure it should require any version modifications. We do not increment for pre-commit changes for example.

Gah, you are correct!  I was thinking of this as a new feature, but it's just a new installation mechanism.  I went ahead and removed those checklists.

@mcdonnnj Please see https://github.com/cisagov/awssh/pull/27/commits/232c6f204f088a4ba9a98db453809e3704f835c6

I hate to bring this up so late, but I just noticed that Nix flakes files are experimental.  I'm still OK with this PR going forward, but I figured folks should know.


feat(Pre-render webpages): Support for prerender.io

https://github.com/quintype/ace-planning/issues/55

https://github.com/quintype/ace-planning/issues/55#issuecomment-790286353


Chaining Promises of different types

#698 Solves the issue of chaining Promises of same type.
I would like to chain Promise of Type K , then Promise of Type L
promiseK.thenPromise(promiseL).thenPromise(promiseM).....

Sagar,
I think this exists already.
It is called thenMap.
https://github.com/advantageous/reakt/wiki/Promise.thenMap
The method Promise.thenMap converts one type of promise into another.
Let's say for example you are storing data into Cassandra.
You want to convert a promise that returns a Cassandra ResultSet into a promise that returns a Boolean.
Example using converting a promise for Cassandra ResultSet into a boolean promise.

        final Promise<ResultSet> resultSetPromise = Promises.<ResultSet>promise();
        final Promise<Boolean> promise = resultSetPromise
                .thenMap(ResultSet::wasApplied) // <-------------- see thenMap
                .catchError((error) -> {
                        if (error instanceof DriverException) {
                            logger.error("Error " + message, error);
                            errorCount.incrementAndGet();
                        }
                });
        /* registerCallback is from Reakt Guava Bridge
           It converts ListableFutures into Reakt promises (to/fro).
        */
        registerCallback(resultSetFuture, resultSetPromise);

Here is a simpler example that turns an Employee Promise into a Sheep Promise.
Converting an employee into a sheep.

        Promise<Employee> employeePromise = Promises.<Employee>promise();

        Promise<Sheep> sheepPromise = employeePromise
                .thenMap(employee1 -> new Sheep(employee1.id));

        employeeService.lookupEmployee("id123", employeePromise);

If that is not what you want, please let me know. Just reopen this issue with a comment.


all: Go 1.14.4 release status

Issue tracking the go1.14.4 release by releasebot.

Log
2020/06/01 13:16:41 starting
2020/06/01 13:16:41 working in /Users/dmitshur/go-releasebot-work/go1.14.4
2020/06/01 13:16:41 $ git clone https://go.googlesource.com/go /Users/dmitshur/go-releasebot-work/go1.14.4/gitmirror
2020/06/01 13:18:12 $ git config gc.auto 0
2020/06/01 13:18:12 $ git fetch origin release-branch.go1.14
2020/06/01 13:18:13 $ git clone --reference /Users/dmitshur/go-releasebot-work/go1.14.4/gitmirror -b release-branch.go1.14 https://go.googlesource.com/go /Users/dmitshur/go-releasebot-work/go1.14.4/gitwork
2020/06/01 13:18:16 $ git codereview change relwork
2020/06/01 13:18:17 $ git config gc.auto 0
2020/06/01 13:18:19 an associated security release branch "release-branch.go1.14-security" does not exist; assuming it has been merged and deleted, so proceeding as usual
2020/06/01 13:18:19 $ git rev-parse go1.14.4
2020/06/01 13:18:19 Release status issue title: "all: Go 1.14.4 release status"
2020/06/01 13:18:20 Release status issue: https://golang.org/issue/39347
2020/06/01 13:18:20 $ git add VERSION
2020/06/01 13:18:20 $ git commit -m go1.14.4

Change-Id: I0daa397bee2ad754fc4860e1365c982de232f171
 VERSION
2020/06/01 13:18:22 $ git codereview mail -trybot
2020/06/01 13:18:26 $ go get golang.org/x/build/cmd/release
2020/06/01 13:21:49 $ /Users/dmitshur/go-releasebot-work/go1.14.4/gopath/bin/release -target linux-ppc64le -user dmitshur -version go1.14.4 -staging_dir /Users/dmitshur/go-releasebot-work/go1.14.4/release-staging/8bf1e1b00eae568f0ea9d506752aaccfb9ef0722_449209711 -rev 8bf1e1b00eae568f0ea9d506752aaccfb9ef0722
2020/06/01 13:21:49 $ /Users/dmitshur/go-releasebot-work/go1.14.4/gopath/bin/release -target linux-386 -user dmitshur -version go1.14.4 -staging_dir /Users/dmitshur/go-releasebot-work/go1.14.4/release-staging/8bf1e1b00eae568f0ea9d506752aaccfb9ef0722_449209711 -rev 8bf1e1b00eae568f0ea9d506752aaccfb9ef0722
2020/06/01 13:21:49 $ /Users/dmitshur/go-releasebot-work/go1.14.4/gopath/bin/release -target linux-armv6l -user dmitshur -version go1.14.4 -staging_dir /Users/dmitshur/go-releasebot-work/go1.14.4/release-staging/8bf1e1b00eae568f0ea9d506752aaccfb9ef0722_449209711 -rev 8bf1e1b00eae568f0ea9d506752aaccfb9ef0722
2020/06/01 13:21:49 $ /Users/dmitshur/go-releasebot-work/go1.14.4/gopath/bin/release -target windows-386 -user dmitshur -version go1.14.4 -staging_dir /Users/dmitshur/go-releasebot-work/go1.14.4/release-staging/8bf1e1b00eae568f0ea9d506752aaccfb9ef0722_449209711 -rev 8bf1e1b00eae568f0ea9d506752aaccfb9ef0722
2020/06/01 13:21:49 $ /Users/dmitshur/go-releasebot-work/go1.14.4/gopath/bin/release -target src -user dmitshur -version go1.14.4 -staging_dir /Users/dmitshur/go-releasebot-work/go1.14.4/release-staging/8bf1e1b00eae568f0ea9d506752aaccfb9ef0722_449209711 -rev 8bf1e1b00eae568f0ea9d506752aaccfb9ef0722
2020/06/01 13:21:49 $ /Users/dmitshur/go-releasebot-work/go1.14.4/gopath/bin/release -target linux-s390x -user dmitshur -version go1.14.4 -staging_dir /Users/dmitshur/go-releasebot-work/go1.14.4/release-staging/8bf1e1b00eae568f0ea9d506752aaccfb9ef0722_449209711 -rev 8bf1e1b00eae568f0ea9d506752aaccfb9ef0722
2020/06/01 13:21:49 $ /Users/dmitshur/go-releasebot-work/go1.14.4/gopath/bin/release -target windows-amd64 -user dmitshur -version go1.14.4 -staging_dir /Users/dmitshur/go-releasebot-work/go1.14.4/release-staging/8bf1e1b00eae568f0ea9d506752aaccfb9ef0722_449209711 -rev 8bf1e1b00eae568f0ea9d506752aaccfb9ef0722
2020/06/01 13:21:49 $ /Users/dmitshur/go-releasebot-work/go1.14.4/gopath/bin/release -target freebsd-386 -user dmitshur -version go1.14.4 -staging_dir /Users/dmitshur/go-releasebot-work/go1.14.4/release-staging/8bf1e1b00eae568f0ea9d506752aaccfb9ef0722_449209711 -rev 8bf1e1b00eae568f0ea9d506752aaccfb9ef0722
2020/06/01 13:21:49 $ /Users/dmitshur/go-releasebot-work/go1.14.4/gopath/bin/release -target freebsd-amd64 -user dmitshur -version go1.14.4 -staging_dir /Users/dmitshur/go-releasebot-work/go1.14.4/release-staging/8bf1e1b00eae568f0ea9d506752aaccfb9ef0722_449209711 -rev 8bf1e1b00eae568f0ea9d506752aaccfb9ef0722
2020/06/01 13:21:49 $ /Users/dmitshur/go-releasebot-work/go1.14.4/gopath/bin/release -target darwin-amd64 -user dmitshur -version go1.14.4 -staging_dir /Users/dmitshur/go-releasebot-work/go1.14.4/release-staging/8bf1e1b00eae568f0ea9d506752aaccfb9ef0722_449209711 -rev 8bf1e1b00eae568f0ea9d506752aaccfb9ef0722
2020/06/01 13:21:49 $ /Users/dmitshur/go-releasebot-work/go1.14.4/gopath/bin/release -target linux-amd64 -user dmitshur -version go1.14.4 -staging_dir /Users/dmitshur/go-releasebot-work/go1.14.4/release-staging/8bf1e1b00eae568f0ea9d506752aaccfb9ef0722_449209711 -rev 8bf1e1b00eae568f0ea9d506752aaccfb9ef0722
2020/06/01 13:21:49 $ /Users/dmitshur/go-releasebot-work/go1.14.4/gopath/bin/release -target linux-arm64 -user dmitshur -version go1.14.4 -staging_dir /Users/dmitshur/go-releasebot-work/go1.14.4/release-staging/8bf1e1b00eae568f0ea9d506752aaccfb9ef0722_449209711 -rev 8bf1e1b00eae568f0ea9d506752aaccfb9ef0722
2020/06/01 13:49:29

The release is ready.

Please review and submit https://go-review.googlesource.com/q/I0daa397bee2ad754fc4860e1365c982de232f171
and then run the release stage.

Latest build: go1.14.4
src (tar.gz)
linux-386 (tar.gz)
linux-armv6l (tar.gz)
linux-amd64 (tar.gz)
linux-arm64 (tar.gz)
freebsd-386 (tar.gz)
freebsd-amd64 (tar.gz)
windows-386 (zip) (msi)
windows-amd64 (zip) (msi)
darwin-amd64 (tar.gz)
linux-s390x (tar.gz)
linux-ppc64le (tar.gz)
Log
2020/06/01 15:38:42 starting
2020/06/01 15:38:42 working in /Users/dmitshur/go-releasebot-work/go1.14.4
2020/06/01 15:38:42 $ git fetch origin master
2020/06/01 15:38:45 $ git fetch origin release-branch.go1.14
2020/06/01 15:38:47 $ git clone --reference /Users/dmitshur/go-releasebot-work/go1.14.4/gitmirror -b release-branch.go1.14 https://go.googlesource.com/go /Users/dmitshur/go-releasebot-work/go1.14.4/gitwork
2020/06/01 15:38:50 $ git codereview change relwork
2020/06/01 15:38:51 $ git config gc.auto 0
2020/06/01 15:38:54 an associated security release branch "release-branch.go1.14-security" does not exist; assuming it has been merged and deleted, so proceeding as usual
2020/06/01 15:38:54 Release status issue title: "all: Go 1.14.4 release status"
2020/06/01 15:38:54 Release status issue: https://golang.org/issue/39347
2020/06/01 15:38:54 $ git rev-parse go1.14.4
2020/06/01 15:39:23 $ git tag go1.14.4 83b181c68bf332ac7948f145f33d128377a09c42
2020/06/01 15:39:23 $ git push origin go1.14.4
2020/06/01 15:39:28 $ go get golang.org/x/build/cmd/release
2020/06/01 15:43:06 $ /Users/dmitshur/go-releasebot-work/go1.14.4/gopath/bin/release -target src -user dmitshur -version go1.14.4 -staging_dir /Users/dmitshur/go-releasebot-work/go1.14.4/release-staging/83b181c68bf332ac7948f145f33d128377a09c42_828683296 -rev 83b181c68bf332ac7948f145f33d128377a09c42 -skip_tests
2020/06/01 15:43:06 $ /Users/dmitshur/go-releasebot-work/go1.14.4/gopath/bin/release -target linux-amd64 -user dmitshur -version go1.14.4 -staging_dir /Users/dmitshur/go-releasebot-work/go1.14.4/release-staging/83b181c68bf332ac7948f145f33d128377a09c42_828683296 -rev 83b181c68bf332ac7948f145f33d128377a09c42 -skip_tests
2020/06/01 15:43:06 $ /Users/dmitshur/go-releasebot-work/go1.14.4/gopath/bin/release -target linux-s390x -user dmitshur -version go1.14.4 -staging_dir /Users/dmitshur/go-releasebot-work/go1.14.4/release-staging/83b181c68bf332ac7948f145f33d128377a09c42_828683296 -rev 83b181c68bf332ac7948f145f33d128377a09c42 -skip_tests
2020/06/01 15:43:06 $ /Users/dmitshur/go-releasebot-work/go1.14.4/gopath/bin/release -target linux-386 -user dmitshur -version go1.14.4 -staging_dir /Users/dmitshur/go-releasebot-work/go1.14.4/release-staging/83b181c68bf332ac7948f145f33d128377a09c42_828683296 -rev 83b181c68bf332ac7948f145f33d128377a09c42 -skip_tests
2020/06/01 15:43:06 $ /Users/dmitshur/go-releasebot-work/go1.14.4/gopath/bin/release -target freebsd-386 -user dmitshur -version go1.14.4 -staging_dir /Users/dmitshur/go-releasebot-work/go1.14.4/release-staging/83b181c68bf332ac7948f145f33d128377a09c42_828683296 -rev 83b181c68bf332ac7948f145f33d128377a09c42 -skip_tests
2020/06/01 15:43:06 $ /Users/dmitshur/go-releasebot-work/go1.14.4/gopath/bin/release -target linux-armv6l -user dmitshur -version go1.14.4 -staging_dir /Users/dmitshur/go-releasebot-work/go1.14.4/release-staging/83b181c68bf332ac7948f145f33d128377a09c42_828683296 -rev 83b181c68bf332ac7948f145f33d128377a09c42 -skip_tests
2020/06/01 15:43:06 $ /Users/dmitshur/go-releasebot-work/go1.14.4/gopath/bin/release -target darwin-amd64 -user dmitshur -version go1.14.4 -staging_dir /Users/dmitshur/go-releasebot-work/go1.14.4/release-staging/83b181c68bf332ac7948f145f33d128377a09c42_828683296 -rev 83b181c68bf332ac7948f145f33d128377a09c42 -skip_tests
2020/06/01 15:43:06 $ /Users/dmitshur/go-releasebot-work/go1.14.4/gopath/bin/release -target windows-amd64 -user dmitshur -version go1.14.4 -staging_dir /Users/dmitshur/go-releasebot-work/go1.14.4/release-staging/83b181c68bf332ac7948f145f33d128377a09c42_828683296 -rev 83b181c68bf332ac7948f145f33d128377a09c42 -skip_tests
2020/06/01 15:43:06 $ /Users/dmitshur/go-releasebot-work/go1.14.4/gopath/bin/release -target linux-arm64 -user dmitshur -version go1.14.4 -staging_dir /Users/dmitshur/go-releasebot-work/go1.14.4/release-staging/83b181c68bf332ac7948f145f33d128377a09c42_828683296 -rev 83b181c68bf332ac7948f145f33d128377a09c42 -skip_tests
2020/06/01 15:43:06 $ /Users/dmitshur/go-releasebot-work/go1.14.4/gopath/bin/release -target windows-386 -user dmitshur -version go1.14.4 -staging_dir /Users/dmitshur/go-releasebot-work/go1.14.4/release-staging/83b181c68bf332ac7948f145f33d128377a09c42_828683296 -rev 83b181c68bf332ac7948f145f33d128377a09c42 -skip_tests
2020/06/01 15:43:06 $ /Users/dmitshur/go-releasebot-work/go1.14.4/gopath/bin/release -target freebsd-amd64 -user dmitshur -version go1.14.4 -staging_dir /Users/dmitshur/go-releasebot-work/go1.14.4/release-staging/83b181c68bf332ac7948f145f33d128377a09c42_828683296 -rev 83b181c68bf332ac7948f145f33d128377a09c42 -skip_tests
2020/06/01 15:43:06 $ /Users/dmitshur/go-releasebot-work/go1.14.4/gopath/bin/release -target linux-ppc64le -user dmitshur -version go1.14.4 -staging_dir /Users/dmitshur/go-releasebot-work/go1.14.4/release-staging/83b181c68bf332ac7948f145f33d128377a09c42_828683296 -rev 83b181c68bf332ac7948f145f33d128377a09c42 -skip_tests
2020/06/01 16:26:42 changing milestone of issue 38030 to Go1.14.5
2020/06/01 16:26:43 changing milestone of issue 38904 to Go1.14.5
2020/06/01 16:26:43 changing milestone of issue 39101 to Go1.14.5
2020/06/01 16:26:44 changing milestone of issue 39279 to Go1.14.5
2020/06/01 16:26:45 changing milestone of issue 39288 to Go1.14.5
2020/06/01 16:26:45 changing milestone of issue 39308 to Go1.14.5
2020/06/01 16:26:46 closing milestone Go1.14.4
2020/06/01 16:26:46

The release run is complete! Refer to the playbook for the next steps.

Thanks for riding with releasebot today.

This release is done.
Announcement: https://groups.google.com/forum/#!msg/golang-announce/R0Ops9_eylQ/Py7xQmwiAwAJ
Tweet: https://twitter.com/golang/status/1267589808529997825


inositol many-phosphate phosphatase terms - query or NTR

Does GO have MF terms for the two phosphatase activities attributed to Asp1/PPIP5K in Figure 1 of PMID:26422458 (full text http://pubs.acs.org/doi/full/10.1021/acs.biochem.5b00532)?
If so, what are they (and apologies for not being able to find them - all those inositol phosphate permutations make my head spin a bit)?
If not, can new terms be added? Requested by a community curator. If you have questions, let me know and I'll try to help, but I don't know much about these activities, so I hope the paper has what you need.
Thanks,
m

Hi Midori - could the curator perhaps specify which terms they request exactly? This is a convoluted highly specialised paper, so it is probably best if they can specify which activities they want to capture.
Some potentials:

We seem to have a bunch of inositol 'many' phosphates as per attached; are those not suitable?

Looking at the paper, results & discussion section

The phosphatase domain of Asp1371–920 has a histidine acid phosphatase signature
Asp1 371–920 only hydrolyzes one of the two diphosphate groups in 1,5-InsP8 (whereas DIPP1 attacks both diphosphates), and (ii) Asp1 371–920 does not hydrolyze monoester phosphate from 1,5-InsP8.
The above data demonstrate that the phosphatase domain in Asp1 is an unusual, highly specialized member of the histidine acid phosphatase family.

Wouldn't GO:0101006, protein histidine phosphatase activity be suitable?

I don't know which, if any, of these terms may be applicable. That's why I asked for help! GO:0004446 may suffice, but I suspect our user may want something more specific. I'll enquire.

In the meantime, what we got with the request was:
name: inositol pyrophosphate 1-phosphatase activity
def: "Catalysis of the reaction: myo-inositol 1-diphosphate pentakisphosphate + H2O = myo-inositol hexakisphosphate + phosphate. Or myo-inositol 1,5-diphosphate tetrakisphosphate + H2O = myo-inositol 5-diphosphate pentakisphosphate + phosphate."
But the name looks less specific than the def(s), and I'm not sure I can reconcile the inclusion of a "tetrakisphosphate" in the suggested def, because Figure 1 shows only inositols with 5 or more phosphates. I know that I did not find any GO terms that I recognized as describing the the removal of one phosphate residue from a diphosphate that is on the 1- (or 5-) position of an inositol molecule that also has 5 monophosphate groups on the other phosphorylatable positions (or 4 monophosphates and one diphosphate).

The observation that Asp1 does not cleave 5-linked diphosphate means that it should be annotated to one or two "1-diphosphate phosphatase" term(s), and not to any "5-diphosphate phosphatase" terms. I don't recall seeing any data in the paper using any phosphoprotein substrates, so I don't see any reason to annotate Asp1 to GO:0101006 or any other phosphoprotein phosphatase activity terms. Also, I'm pretty sure the "histidine" in "histidine acid phosphatase" refers to a His residue that is part of the enzyme, not the substrate, and is involved in catalysis.

Thanks for the follow-up Midori.

I suspect I'm in the same boat as you; I'm not sure which term would be suitable. let me know what you find out, and I'll ping @ukemi in case he has any idea (now that he's back ;))
I was checking the InterPro classification for histidine acid phosphatase, http://www.ebi.ac.uk/interpro/entry/IPR000560 as the paper mentions the 'histidine phosphatase signature': they do indicate phosphorylation on the His residue, so per the definition 'Catalysis of the reaction: protein histidine phosphate + H2O = protein histidine + phosphate' wouldn't that mean that this is a protein histidine phosphatase activity, as at some point the phosphate will be linked to the histidine and then cleaved?
InterPro also indicates a mapping to GO:0003993, acid phosphatase activity.

I've had another look at the structures, and I think I've figured out where the "tetrakisphosphate" that was confusing me comes from. It's the "InsP8" molecule, which has diphosphates at two positions and monophosphates at the other four. So I'm a lot more confident now that what we need are two new MF terms for these reactions:
GO:new1
name: inositol 1-diphosphate 2,3,4,5,6-pentakisphosphate 1-diphosphatase activity
def: Catalysis of the reaction: myo-inositol 1-diphosphate 2,3,4,5,6-pentakisphosphate + H2O = myo-inositol hexakisphosphate + phosphate. [PMID:26422458]
is_a: GO:0052745
synonym: inositol 1-pyrophosphate 2,3,4,5,6-pentakisphosphate 1-pyrophosphatase activity EXACT
GO:new2
name: inositol 1,5-bisdiphosphate 2,3,4,6-tetrakisphosphate 1-diphosphatase activity
def: Catalysis of the reaction: myo-inositol 1,5-bisdiphosphate 2,3,4,6-tetrakisphosphate + H2O = myo-inositol 5-diphosphate 1,2,3,4,6-pentakisphosphate + phosphate. [PMID:26422458]
is_a: GO:0052745
synonym: inositol 1,5-bispyrophosphate 2,3,4,6-tetrakisphosphate 1-pyrophosphatase activity EXACT
(I'm less sure about the names and synonyms than the defs now - change as you see fit!)
As for the histidine phosphatase term, I don't think we usually annotate anything based on what happens with reaction intermediates. for this paper specifically, I would stick to the inositol polyphosphates they actually assayed in the experiments.

Hi Melanie and Midori,
I am back now. These enzymes always make my head spin, but I think based on the request, Midori's proposal seems correct. I agree we should not include the intermediates.
-D

Welcome back, David! I'm glad I'm not the only one with a spinning head ...

Yeah welcome back and Happy New Year ;)

any progress?
thx,
m

Still on the TODO list, but I'm trying to catch up this week, so should be addressed soon. Thanks for the reminder.

Committed revision 32468, thanks for the patience.

They've reached PomBase Canto now, and I've emailed our user. Thanks!


Custom DB backup / export path

Very lazy of me to request this, but:
On running nitro db backup it would be great to be able to pass an optional path of where you want the backup to be sent to i.e.
nitro db backup ~/dev/Craft/tutorial OR to have the option for the backup to be added to the current directory of the associated craft website i.e. the same location of the craft sites .env file.
Simple reason is to stop having to copy and paste the DB file from the default .nitro/db directory to the craft sites website prior to adding the site to a git repository.
Thanks -
W

Hi, we are closing this issue as we have decided to retire Nitro, so no additional work will occur on this project. You can read the official blog post here https://craftcms.com/blog/retiring-craft-nitro. We appreciate everyones feedback and involvement and we look forward to refocusing our efforts on Cloud!
If you're looking for a new local development environment, we recommend DDEV and have a knowledge base article to help you with the transition: https://craftcms.com/knowledge-base/migrating-from-craft-nitro-to-ddev.


Remove Wasmtime ABIs from Cranelift

This commit removes the Wasmtime* family of ABIs from Cranelift. These were originally added to support multi-value in Wasmtime via the TypedFunc API, but they should now no longer be necessary. In general this is a higher-level Wasmtime concern than something all backends of Cranelift should have to deal with.
Today with recent refactorings it's possible to remove the reliance on ABI details for multi-value and instead codify it directly into the Cranelift IR generated. For example wasm calls are able to have a "purely internal" ABI which Wasmtime's Rust code doesn't see at all, and the Rust code only interacts with the native ABI. The native ABI is redefined to be what the previous Wasmtime ABIs were, which is to return the first of a 2+ value return through a register (native return value) and everything else through a return pointer.

cc @uweigand if you're up for it I could use some help on the s390x-side of things. I believe that the Wasmtime ABI was load-bearing in a way not related to multi-value on the s390x backend to handle lane order of vector types, meaning that I think as-is this an't land since it'll fail tests on s390x. I was wondering if you had thoughts or ideas about how to go about fixing this? In my mind the fix would look something along the lines of:

Somehow get all wasm funtions to think they're using little-endian lane ordering. Previously this was done with the Wasmtime* family of calling conventions but I'd ideally like to remove this Wasmtime "brand" from Cranelift to avoid all backends having to deal with it.
Converting between big and little-endian lane orders in the various trampolines as necessary. This may not actually be required since v128 is never passed in registers via TypedFunc (haven't gotten around to implementing that in Wasmtime), so this point may already be "solved". In the future though I think it'd be fine to "just" generate more code in trampolines.

So I guess the question boils down to: should an ABI be added for s390x to specify that the lane order of vectors is little-endian? Or are there other options you can think of to solve this?

I think this will be easier once we switch all internal Wasm functions to using the tail calling convention, and then the tail calling convention can specify a little-endian lane ordering.

Hurray looks like tests are green on s390x! I think this is good to go and we can tackle how to avoid having a specific ABI for s390x, if at all, in the future.


Mouse stuttering when using program

I use a joystick instead of WASD and so im using my mouse at the same time and when I use this software to make my joystick work my mouse stutters every 10-20 seconds. This issue of mouse stuttering doesn't happen ever when im not using the program. I would just like for my joystick to work, does anyone know how to fix this?

If you disable the auto-refresh in the settings that got rid of the issue for me.

@GabeGars > If you disable the auto-refresh in the settings that got rid of the issue for me.
It work for me. thank you.


doesn't copy Fedora

I'm on my laptop (HP BANG % OLUFSEN) with Linux Fedora 38 as OS and when I press the Copy Highlighted button it shows a popup saying "text copied" but when I try to past it inside discord it past what I copied before going on the website, I tried to manually copy from the popup but I can't select the text

Have you allowed using your clipboard in browser settings? Also try to click Ok on popup, maybe browser will ask to use your clipboard as in Edge (idk, never used linux ngl)

I tried pressing ok but nothing happend and I can copy text from internet if I select it and press ctrl + C


Updating issue state - unable to open closed issues

We're trying to use a bot to reopen issues if they've been closed by someone other than the person that opened it. We're running into problems using state: 'open' in a way that isn't adding up.
For example, this code works:
module.exports = robot => {
    robot.on('issues', async (event, context) => {
        const issue = event.payload.issue;
        const action = event.payload.action;

        if (action === 'reopened') {
            return context.github.issues.edit(context.issue({state: 'closed'}));
        }
    });
};

BUT this code does not work:
module.exports = robot => {
    robot.on('issues', async (event, context) => {
        const issue = event.payload.issue;
        const action = event.payload.action;

        if (action === 'closed') {
            return context.github.issues.edit(context.issue({state: 'open'}));
        }
    });
};

Are we doing something wrong?
cc @crichID @jasonetco @hectorsector

@brianamarie Any luck with the latest version?

@kaizensoze Thanks for the ping - @hectorsector Would you be interested in trying this out again tomorrow with me?

It looks like a permissions issue. I added a working example that reopens a closed issue. Have you double checked the permissions of the bot? Reopening an issue vs closing requires different permissions: https://help.github.com/articles/repository-permission-levels-for-an-organization/

@kaizensoze Thank you so much. Admittedly I didn't get around to testing like I hoped I would. I am comfortable closing this issue, and if we run into problems I will circle back. Thank you again! 🎉  😊

👋 just wanted to confirm that this is indeed working now


there is something wrong with AggregateFunction count

Here is the data query every 10s:
sql:
SELECT  * FROM events  where meta(device)="TestModbusDevice" and (Coil_150 !=null or Coil_100 != null) GROUP BY TUMBLINGWINDOW(ss, 10)
query data:
[{"Coil_150":true},{"Coil_100":true}]
[{"Coil_150":true},{"Coil_100":true}]
[{"Coil_150":true},{"Coil_100":true}]
[{"Coil_150":true},{"Coil_100":true}]
[{"Coil_150":true},{"Coil_100":true}]
[{"Coil_150":true},{"Coil_100":true}]
Now I want to calculate the total count  for different element  in every 10s.
Here is my sql:
SELECT  count(Coil_150) as a, count(Coil_100) as b FROM events  where meta(device)="TestModbusDevice" and (Coil_150 !=null or Coil_100 != null) GROUP BY TUMBLINGWINDOW(ss, 10)
the actual result:
[{"a":2,"b":2}]
[{"a":2,"b":2}]
[{"a":2,"b":2}]
[{"a":2,"b":2}]
[{"a":2,"b":2}]
the expect result:
[{"a":1,"b":1}]
[{"a":1,"b":1}]
[{"a":1,"b":1}]
[{"a":1,"b":1}]
[{"a":1,"b":1}]
So it is  something wrong with my sql or it is bug ?
my kuiper version is 0.4.1
Thank you！

Laugh. Thanks for the very clear description, it helps us to identify the problem much faster.


FIFO optimization

Feature Suggestion
I would love to see a certain FIFO optimization introduced to the Market Making strategies.
Currently, the strategies cancel and replace orders even if they are exactly the same since and at the same rate/price. This leads to a very unfavourable trade execution due to the FIFO principle most exchange's matching engines implement.
I propose that there is a FIFO switch introduced with a single threshold parameter. Like so for example:
fifo_optimization_enabled = true/false // simply a switch to enable/disable this mechanism
fifo_threshold = <rate>/<percentage> // a threshold which if exceeded, the normal rules would apply, otherwise - use the fifo optimization and leave the limit order intact. Note
Impact
This would increase order fill rate and profitability in most strategies as orders would fill faster. Some lower cap and low sat pairs especially would benefit but not exclusive to these.
Another benefit would be decreasing the exchange API as well as Humminbot operators machine resource load as orders would be replaced less often and only if the amount/rate changes past the threshold set by the user.
Additional context
I will add some more info/context later.. In a hurry...
┆Issue is synchronized with this Clickup task by Unito

I don't really use it but isn't this what "order_refresh_tolerance_pct" does?

yea it's warm but not quite. order_refresh_tolerance_pct  leaves both bid/ask orders (the entire cycle) untouched unless the reference price moves outside this threshold. I am proposing another threshold that filters which orders if any would need replacing (independent of side) after the refresh cycle is triggered based on the delta of current limit order rate and the proposed new limit order rate is larger than the threshold and independent of all other orders (this is done for each new proposed order) in the refresh cycle.
So in other words - a nuanced filter that saves certain orders from being unnecessarily replaced at the same rate/amount which can be caused by quick rate retracements, sudden changes in the spread thus moving the reference price (e.g. mid price) and so on. You could have a macro regime in the market where bids in the book remain at the same level and only asks oscilate for  while (and vice versa). Without this filter/feature HB would replace orders without checking if the new order would be actually at exactly (I also propose a threshold) the same rate and amount or not.
Am I making any sense here? 😅

That makes sense yes! :)

@cardosofede Is this not best implemented in a Script, perhaps as a proposal for students? - Not an Issue, right?

@zappra @krisj I implemented a component called PositionExecutor for Perps. Soon I would like to implement it for Spot and that component can address what you are looking for.
Essentially, each order has a take profit, stop loss, and time limit managed by the bot, so it will let you close the cycle of a filled order and improve the logic of your strategy.


fillin for fallbacks for filesystem funcs

I guess this makes sense!

Yeah, the docs for https://github.com/GarageGames/Torque3D/blob/2044b2691e1a29fa65d1bdd163f0d834995433ce/Engine/source/console/engineAPI.h#L2591 were wrong. need to feed it both variable fallbacks in multi-param functions. (Do note that does mean there's likely quite a few more with the same misbehavior, if someone wants to go over the full list with a fine toothed comb)


Jersey does not supply "commonly used" native image reflection metadata

A Jersey-based (Helidon MP) application running in native image may need to deal with @PathParam objects of type String, Integer, etc. (or a user-created type). To work with them successfully, Jersey will in many codepaths call their single-String-argument constructors reflectively. Jersey does not supply the relevant reflect-config.json metadata for such invocations. End users may see stack traces similar to the following:
org.graalvm.nativeimage.MissingReflectionRegistrationError: The program tried to reflectively invoke method public java.lang.String(java.lang.String) without it being registered for runtime reflection. Add public java.lang.String(java.lang.String) to the reflection metadata to solve this problem. See https://www.graalvm.org/latest/reference-manual/native-image/metadata/#reflection for help.

The Helidon team's opinion is that—rather than the end user having to follow this advice—Jersey should supply metadata for "commonly used" types, for some value of "commonly used", which might be as simple as "types in java.lang that feature single-String-argument constructors", so that the most basic of JAX-RS applications will work, hopefully, out of the box. This may end up being true as well for static valueOf(String) methods and the like.
In the meantime, Helidon should add this, and/or similar, information to our jersey/server reflection-config.json file.

See #9441, which introduces this resource as described.


ValueError: expected 2D or 3D input (got 1D input)

Hi~
Thanks for releasing code. Would be better if you could answer the following question.
I found that training process will stop after epoch30 with an error " ValueError: expected 2D or 3D input (got 1D input)", could you please help me to solve this problem? ( I have replaced duke dataset with my own dataset, and set the batch size to 32 due to GPU limitations).
Looking forward to your kindly reply.
Best,
Yarloy

hi, I get the same error, are you solve this problem?

Please post your error and corresponding code here.

If you have any problems. Please concat the first author<EMAIL_ADDRESS>


Improve waves-script to allow for restarting Bobtimus.

resolves #96

Oops, I ended up doing this too, but this looks better! I'll drop my minimal changes.

Oops, I ended up doing this too, but this looks better! I'll drop my minimal changes.


[AVM Module Issue]: Diagnostic settings issues

Check for previous/existing GitHub issues

[x] I have checked for previous/existing GitHub issues

Issue Type?
No response
(Optional) Module Version
No response
(Optional) Correlation Id
No response
Description
Please see the feedback from one of the customer
"In this module for AVD insights deployment, we see the variables.tf file has a variable defined for diagnostic_settings and expectation is that, when we call this module with Diagnostic setting specific values defined, the diagnostic setting should be enabled for that specific resource but that does not happen. On checking, we found the main.tf file for this module does not have a resource block needed for creation of the Diagnostic setting which is required"

@PmeshramPM  The insights pattern module is to create AVD Insights which is the workbook configuration for monitoring and your setting the diagnostic settings in the management plane pattern. If you look at the main.tf in the managementplane module you can see that it has diagnostic settings enabled
cc @jensheerin

closing issue answered above. The diagnostic setting is set at on each resource.


assistedinject: Skip static methods on interfaces.

This fixes https://github.com/google/guice/issues/937.

Thanks Tavian!


Add type: majority to zonal statistics (categorical raster analysis)

Feature Request to add allowed statistics type: MAJORITY to zonal statistics to facilitate summarizing categorical/discrete raster time series (especially land cover).
(Minimum and Variety would also be useful.)
Very much appreciate the Zonal Statistics by Group option and all the capabilities of geemap.

GEE does not have a majority reducer. PR is welcome if you know how to implement it.

I think the mode reducer would work ee.Reducer.mode
Returns the most common value
From: Qiusheng Wu @.>
Sent: Thursday, March 3, 2022 3:00 PM
To: giswqs/geemap @.>
Cc: Cathleen Sampselle @.>; Author @.>
Subject: Re: [giswqs/geemap] Add type: majority to zonal statistics (categorical raster analysis) (Issue #960)
GEE does not have a majority reducer. PR is welcome if you know how to implement it.

Reply to this email directly, view it on GitHubhttps://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fgiswqs%2Fgeemap%2Fissues%2F960%23issuecomment-1058432125&data=04|01|csampselle%40ducks.org|691ccc9e1b7c4b8ee22408d9fd50759a|2430c44f94924e6fa57d32257ab4c515|0|1|637819344301929173|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=LGZWEWPLjV3OLpu5TgLEl7tfvEwBZBqyftG4XyCPQ7Y%3D&reserved=0, or unsubscribehttps://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAW2FIUCP4E4EGOP6SQ2JMJ3U6EK5TANCNFSM5P3M752Q&data=04|01|csampselle%40ducks.org|691ccc9e1b7c4b8ee22408d9fd50759a|2430c44f94924e6fa57d32257ab4c515|0|1|637819344301929173|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=zKw6ZRZkAZ5tssN3jCLO%2FGq7xov9Q9YdusHv3UQXIdU%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01|csampselle%40ducks.org|691ccc9e1b7c4b8ee22408d9fd50759a|2430c44f94924e6fa57d32257ab4c515|0|1|637819344301929173|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=3MqE%2B1RzySZLb3X1h4N1EGW4ul9mZbXIneNKtOkoaew%3D&reserved=0 or Androidhttps://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01|csampselle%40ducks.org|691ccc9e1b7c4b8ee22408d9fd50759a|2430c44f94924e6fa57d32257ab4c515|0|1|637819344301929173|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=PXcCoKQdqO%2BkCeBHhI%2BDGwAHS3sam29l6Vl5KKZ562U%3D&reserved=0.
You are receiving this because you authored the thread.Message ID: @.***>

The mode reducer has been added.
statistics_type="MODE"

Hello,
In Jupyter I tried geemap.update_package(), restarting Kernel, and running
geemap.zonal_statistics(crop, habDel, out_crop_stats, statistics_type='MODE', scale=30)
and received the following error
The statistics type must be one of the following: MEAN, MAXIMUM, MEDIAN, MINIMUM, STD, MIN_MAX, SUM, VARIANCE, HIST, FIXED_HIST
From: Qiusheng Wu @.>
Sent: Thursday, March 3, 2022 4:30 PM
To: giswqs/geemap @.>
Cc: Cathleen Sampselle @.>; Author @.>
Subject: Re: [giswqs/geemap] Add type: majority to zonal statistics (categorical raster analysis) (Issue #960)
The mode reducer has been added.
statistics_type="MODE"

Reply to this email directly, view it on GitHubhttps://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fgiswqs%2Fgeemap%2Fissues%2F960%23issuecomment-1058512160&data=04|01|csampselle%40ducks.org|32370ef5658248b11d0d08d9fd5cf08a|2430c44f94924e6fa57d32257ab4c515|0|0|637819397891973963|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=TW%2Fz1r%2Bi63R10xeGuP8jPsFAmZ2ZfWY7AGwj4p0%2FaeA%3D&reserved=0, or unsubscribehttps://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAW2FIUAUKVJBXKW5HJTGBHLU6EVMTANCNFSM5P3M752Q&data=04|01|csampselle%40ducks.org|32370ef5658248b11d0d08d9fd5cf08a|2430c44f94924e6fa57d32257ab4c515|0|0|637819397891973963|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=gu0dwQekkzUEqWwgEghnZFuZB8Z%2Fgx01ktP2rPyOJ4I%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01|csampselle%40ducks.org|32370ef5658248b11d0d08d9fd5cf08a|2430c44f94924e6fa57d32257ab4c515|0|0|637819397891973963|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=FNerHPleEW37jwiKUdCVFzG21ciNgz3%2BPYGFBIFSyII%3D&reserved=0 or Androidhttps://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01|csampselle%40ducks.org|32370ef5658248b11d0d08d9fd5cf08a|2430c44f94924e6fa57d32257ab4c515|0|0|637819397891973963|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=G68zBbOmuKdfhUdAJ9BzQ6jH6%2Bmvwf0xrYaUDmJRXA0%3D&reserved=0.
You are receiving this because you authored the thread.Message ID: @.***>

I just updated it. Please try again.


Bug 2067312: PPT event source is lost when received by the consumer

Fixes lost event source
Also minor fix on  ping status call returning 404 error
plus nil pointer exception when PTP starts with the faulty interface.
Signed-off-by: Aneesh Puttur<EMAIL_ADDRESS>

/retest

/cherry-pick release-4.10

/cherry-pick release-4.10


"mount" command does not work on master branch

Describe the bug
mount command provides a list of mounted folders on v1.8.2 . But does not work on master branch
To Reproduce
Steps to reproduce the behavior:

Upload pixhawk 4 with v1.8.2
open mavlink console and type mount. It'll show the mount points
Upload pixhawk 4 with master branch
open mavlink console and type mount. It'll show the mount points. It asks for arguments and does not have default output of mount points.

Expected behavior
The mount command should show the mount points in master branch. Changes to default mount command behavior should be documented if changed between v1.8.2 and master
Log Files and Screenshots

Had to get a new pixhawk board to solve this. Hardware issue


User Study - Make Python Primer more readable

Users seem to not be able to find content that is actually in the doc. (change fonts, etc.)

I came across this during manual testing. I find the page rather hard to read/skim, and it has a lot of different colors that make it hard to figure out where to focus. There's also lots to absorb within a rather compressed format.
In thinking about the use case, I'm probably (as a user) trying to figure out how to do something in Python that I can't remember how to do. Perhaps it might be better to have a barebones list of common FAQs that I can skim?


upstream-community-operators infinispan (2.1.4)

Thanks submitting your Operator. Please check below list before you create your Pull Request.
New Submissions

[x] Does your operator have nested directory structure?
[x] Have you selected the Project Community Operator Submissions in your PR on the right-hand menu bar?
[x] Are you familiar with our contribution guidelines?
[x] Have you packaged and deployed your Operator for Operator Framework?

[x] Is your submission signed?
[x] Is operator icon set?

Updates to existing Operators

[x] Did you create a ci.yaml file according to the update instructions?
[x] Is your new CSV pointing to the previous version with the replaces property if you chose replaces-mode via the updateGraph property in ci.yaml?
[x] Is your new CSV referenced in the appropriate channel defined in the package.yaml or annotations.yaml ?
[x] Have you tested an update to your Operator when deployed via OLM?
[x] Is your submission signed?

Your submission should not

[x] Modify more than one operator
[x] Modify an Operator you don't own
[x] Rename an operator - please remove and add with a different name instead
[x] Submit operators to both upstream-community-operators and community-operators at once
[x] Modify any files outside the above mentioned folders
[x] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[x] Description about the managed Application and where to find more information
[x] Features and capabilities of your Operator and how to use it
[x] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[x] Human readable name and 1-liner description about your Operator
[x] Valid category name1
[x] One of the pre-defined capability levels2
[x] Links to the maintainer, source code and documentation
[x] Example templates for all Custom Resource Definitions intended to be used
[x] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here

The "Operator test / kiwi / Full operator test" seems to have failed because I built and tagged the operator image in parallel. It should be fine if re-run.

/hold

/hold cancel

/merge possible

/merge possible


Add your own API key

In case some site will block the current API-key if this program is misused by some users.
For instance you can use your personal API-key for Bierdopje.com if you create an acoount.
This is more a feature request

actually you can't ... bierdopje is not giving out personal api keys

You are right!
You can request one for a specific application though.
It was only an example for something to consider in the future ;)
I'm very happy with this program and would like it not to get blocked because of it popularity.

Anyway, this should be a feature I think. Some providers allow the use of individual API keys if I remember well.

One that wants to change the API can edit the source code. This shouldn't be everyone as subliminal needs to be identified as such by subtitles providers so they can report me abuse etc.


Adding Power System Test Data

The file definations.jl should be definitions.jl.

I suggest putting the serialize and deserialize functions in one file.

I suggest a slight reorganization of these files:
src/structs/SystemBuildStats.jl -> src/system_build_stats.jl
src/structs/SystemDescriptor.jl -> src/system_descriptor.jl

The function parse_system_descriptor could be a SystemDescriptor constructor.

Would it be easier to store the system catalog in a Julia file instead of JSON?


Unexpected hash during update

I setup my jade and used Green on Android to try it out and it detected that a firmware update was available.

Sadly the file hash doesn't match what I had previously reproduced:

Is this a binary delta hash or what is it?
Our script for WalletScrutiny.com determines these hashes:
$ scripts/test/hardware/blockstreamjade.sh 1.0.31
...
Results:
26c918f0e7281b9cc8fdfc10baae2e5a8ac6edf0cd08e699a913daea3b97d04e  jade_1.0.31_10_ble.bin
56c3543e2929b4b450d683fcad15a3f1cc6fe0c445db4d33c102985cfcf5bc7e  jade_1.0.31_10_ble.built.bin
56c3543e2929b4b450d683fcad15a3f1cc6fe0c445db4d33c102985cfcf5bc7e  jade_1.0.31_10_ble_stripped.bin
The Jade firmware version 1.0.31 10_ble is reproducible with above hashes.
ba25801fa09ca3d0ddb1831b6384f1122f91804cd0d5c4a578aa422e4486c0da  jade_1.0.31_10_noR.bin
45c7353e9d844a549463ab65504e27fcb4a15d3877789e5d30175cefa0bd9c61  jade_1.0.31_10_noR.built.bin
45c7353e9d844a549463ab65504e27fcb4a15d3877789e5d30175cefa0bd9c61  jade_1.0.31_10_noR_stripped.bin
The Jade firmware version 1.0.31 10_noR is reproducible with above hashes.
b9c5568de7ccc4355f95fae91b35faa9ff85c95c42ef44d6151f242691a29f1a  jade_1.0.31_11_ble.bin
35da48fdb0b478a2a1f889f97dc848af0d8b2c518b79bb0b864950cd16713920  jade_1.0.31_11_ble.built.bin
35da48fdb0b478a2a1f889f97dc848af0d8b2c518b79bb0b864950cd16713920  jade_1.0.31_11_ble_stripped.bin
The Jade firmware version 1.0.31 11_ble is reproducible with above hashes.
4c88942060c01938fcdbf6be118771a9c1610a4647b98ce25e4baa813e36d437  jade_1.0.31_11_noR.bin
4065e64e34c68f55eb28e918bf7982d8f9cb4415be6fa3babbc0638b7dd57df1  jade_1.0.31_11_noR.built.bin
4065e64e34c68f55eb28e918bf7982d8f9cb4415be6fa3babbc0638b7dd57df1  jade_1.0.31_11_noR_stripped.bin
The Jade firmware version 1.0.31 11_noR is reproducible with above hashes.

With my device being "11" and with radio, this should be the relevant line:
b9c5568de7ccc4355f95fae91b35faa9ff85c95c42ef44d6151f242691a29f1a  jade_1.0.31_11_ble.bin

index.py:
  "stable": {
    "full": [
      {
        "filename": "1.0.31_noradio_987136_fw.bin",
        "version": "1.0.31",
        "config": "noradio",
        "fwsize": 987136,
        "cmphash": "cf6510018e588122def598e6be977353f07770b25e74d4cf579839500aef1eca",
        "fwhash": "4c88942060c01938fcdbf6be118771a9c1610a4647b98ce25e4baa813e36d437"
      },
      {
        "filename": "1.0.31_ble_1249280_fw.bin",
        "version": "1.0.31",
        "config": "ble",
        "fwsize": 1249280,
        "cmphash": "9803fd7cea218fcc3637f8c345003acf556d23954eeb32837bc55e8e195bbf33",
        "fwhash": "b9c5568de7ccc4355f95fae91b35faa9ff85c95c42ef44d6151f242691a29f1a"
      }
    ],

So teh has you show above 9803fd...3637f8... looks like the 'cmphash' (ie the hash of the compressed [full fw] file) of "1.0.31_ble_1249280_fw.bin", which has full uncompressed hash of b9c556....
(If Jade screen says: 'Fw Hash' if it is passed [and displays] the full uncompressed fw hash, or 'File Hash:' if it receives only the compressed file hash.)
For some reason the android app must be sending the compressed file hash and not the full fw hash (Jade accepts either to cater for older installs of various companion apps) - so initially I'd suggest it's an issue with the android app (we should have updated all our apps to always pass the full uncompressed hash).
Can I ask what version of the android app that was and I'll chase it up.
Many thanks.
J.

I'm using Blockstream Green 4.0.33.
Showing the compressed file hash is kind of expected and WalletScrutiny not showing the hash was a typo in our script. Thanks for your patience. :pray:

No worries mate.


⚠ Bofrost.fr has degraded performance

In dd2ad63, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1507 ms

Resolved: Bofrost.fr performance has improved in 5c27d27 after 6 minutes.


Needs a license

Hi @jordanorelli I like what you've done here, cool :)
Any chance you can throw a license file into this repo so others can patch/modify/etc without fear of legal repercussions?
If you're unsure about licensing, a good choice for a lib like this might be the LGPL version 3+. It protects you, but also lets others use it.
I'm not a lawyer, but I'm happy to answer any questions you have about this, or write the patch.
Thanks!
James

sure, I'll look into it and post one in a bit.

Awesome, thanks!

Added a license (MIT license) in the LICENSE file. Should be all set now.

Cool, thanks!


Investigate @tf.function models running as a wrapped classifier

Per #243 we should investigate to see any issues that occur for models that utilize @tf.function which are wrapped by the TensorFlowV2Classifer.
You can assign this issue to me.

@seanpmorgan Thank you for looking into this issue.

So I had a chance to look into this a bit...
There should be no issues since gradient tape is able to trace anything decorated with @tf.function.
(x_train, y_train), (x_test, y_test), _, _ = load_mnist()

class TensorFlowModel(Model):
    """
    Standard TensorFlow model for unit testing
    """
    def __init__(self):
        super(TensorFlowModel, self).__init__()
        self.conv1 = Conv2D(filters=1, kernel_size=7, activation='relu')
        self.maxpool = MaxPool2D(pool_size=(4, 4), strides=(4, 4), padding='valid', data_format=None)
        self.flatten = Flatten()
        self.dense1 = Dense(10, activation='softmax')

    @tf.function
    def custom_thing(self, x):
        x += tf.constant(10.0)
        return x

    def call(self, x):
        """
        Call function to evaluate the model
        :param x: Input to the model
        :return: Prediction of the model
        """
        x = self.conv1(x)
        x = self.maxpool(x)
        x = self.flatten(x)
        x = self.custom_thing(x)
        x = self.dense1(x)
        return x

if __name__ == "__main__":
    model = TensorFlowModel()
    loss_object = tf.keras.losses.SparseCategoricalCrossentropy()

    # Create the classifier
    tfc = TensorFlowV2Classifier(model=model, loss_object=loss_object, nb_classes=10, input_shape=(28, 28, 1),
                                 clip_values=(0, 1))

    y_predicted = tfc.predict(x_test[0:1])
    grad = tfc.class_gradient(x_test[0:1], label=0)
    print(y_predicted)
    print(grad)

However, while using this wrapper I found it very un-intuitive for the typical TF2 Keras subclass paradigm. For example:
The above code but running:
tfc.fit(x_train, y_train, batch_size=64, nb_epochs=3)

Will return:
# self._train_step(images, labels)
# TypeError: 'NoneType' object is not callable

A user could specify a _train_step,  but that's a strange function to write because having that determined for you is one of the major reasons to be using Keras models in the first place.
I'm going to close this issue as @tf.function doesn't seem to be causing issues with the wrapper, but I'll continue discussion in #243 about a possible refactor.

@seanpmorgan Thank you very much for looking into this issue and please apologise the delayed response.


Load preview if mime type is image/video

When serving files in the remote explorer, show preview of an image.
Things to consider:

Size of the image
Implement "grid view" for better viewing of image/video files.
Ability to switch between views.
Larger (same sized as image) icons for other types of files.

Done.


Can this trainer be used on single player offline mode?

I wanted to try to use this trainer on new version that just came out but offline mode single player ONLY!!! Is that possible??

I will properly document this repo soon. Yes this is compatible with sptarkov


Add links to slides in the ReadMe

Hello! I came here from twitter to take a look at the slides prior to the meetup tonight but couldn't get to them. Could you please include the direct links in the ReadMe so they are easy to find?
I tried using this link to access Andrea's slides, for example, and it didn't point to the rendered html: https://rladies-eastlansing.github.io/2022-accessible_dataviz/AndreaGV/Andrea-RLadies_EastLansing.html
(original html file is at https://github.com/rladies-eastlansing/2022-accessible_dataviz/blob/main/AndreaGV/Andrea-RLadies_EastLansing.html)
I wonder if GitHub Pages needs to be enabled for this repo?

Thanks for the note, Silvia! I have now added the links for both presentations + code to the README file.

Thank you @jananiravi !


Updated a few files

Fixes #
Describe the changes you have made in this PR - Updated dependency_overrides and also updated iOS files generated during iOS environment runs.
Screenshots of the changes (If any) -
Note: Please check Allow edits from maintainers. if you would like us to assist in the PR.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3879

Relevant Lines:
5365

💛  - Coveralls

Sure, I supposed this to be a minute updation. I will do the required with the next PRs that follow. Though, happy to start contributing to CircuitVerse!


Re-indent a few lines to fix indentation in Demo/ViewController.swift

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1086   +/-   ##
=======================================
  Coverage   73.79%   73.79%
=======================================
  Files          22       22
  Lines         729      729
=======================================
  Hits          538      538
  Misses        191      191

Continue to review full report at Codecov.

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

Thanks again, keep it up!


Openai stream v1 support

Add support for streaming in openai v1

Codecov Report
Attention: 6 lines in your changes are missing coverage. Please review.

Comparison is base (d61d065) 81.85% compared to head (19e6d44) 81.88%.

Files
Patch %
Lines

newrelic/hooks/mlmodel_openai.py
82.85%
1 Missing and 5 partials :warning:

Additional details and impacted files
@@                    Coverage Diff                    @@
##           add-streaming-support    #1061      +/-   ##
=========================================================
+ Coverage                  81.85%   81.88%   +0.02%
=========================================================
  Files                        192      192
  Lines                      20968    21002      +34
  Branches                    3650     3658       +8
=========================================================
+ Hits                       17163    17197      +34
+ Misses                      2751     2748       -3
- Partials                    1054     1057       +3

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


Issues on Installing AmpBench on my PC

hello,
I am struggling currently with installing AMPBench on my PC. I really hope you can help me because I really love your product (the other alternatives are really not a match).
I followed the procedure :

I installed the package Node : node-v12.16.1-x64.msi
then I run the git command git clone https://github.com/ampproject/ampbench.git
and finally launched npm start.

But then when I click any url (http://localhost:8080/ for example) it redirects me to the https://github.com/ampproject/ampbench#status.
Which seems normal because the redirect-all.js script redirects everything to the https://github.com/ampproject/ampbench#status page !
So how is it suppose to work ?
I would really appreciate if you could help
thanks & regards

Oh yeah, the master version has that redirect in place! You need to make a change to your package.json so the start script runs node ampbench_main.js—i.e. reverse this commit.
Note that ampbench might have issues running on Windows—I haven't tested that. It should work fine on macOS and Linux.
Will close this but please re-open if you have any further issues.

Many thanks for your quick answer.
I did what you said so npm start launches node ampbench_main.js but now I have another issue Error: Cannot find module './amp-story/linter'
I have tried on PC and Mac but I had the same issue.  And in fact linter is a directory, there is no script linter.js.
Could you please help ?

Uh sorry yes try running npm install (instead of npm update) to install the dependencies? (I've just updated the instructions.)

many thanks it works !!!
FYI, it works on mac only, on PC I have an issue ont the install command but mac is good for me.


Support extracting phrases from Svelte components

This is a rough cut to support Svelte components.

Eh... nevermind, this doesn't run Svelte preprocessors (e.g. to compile from TypeScript), so it's not useful for me.
Discovered it was easier to compile Svelte down to JS and then run @transifex/cli over the built javascript.


Maximizing then restoring with devtools open causes devtools to go missing

Issue Type: Bug
Steps:

Open a vscode window (not maximized)
Toggle devtools (devtools probably needs to be docked)
Maximize the window
Restore the window
:bug: the devtools window is gone

If you maximize the window again it will come back, or if you toggle devtools twice (since it thinks it is still open) it will come back.
This reproduces in 1.40.1 as well
VS Code version: Code - Insiders 1.41.0-insider (f24f483428f592a262dc73d68a3f05745c562b31, 2019-11-21T10:43:20.960Z)
OS version: Windows_NT x64 10.0.19028

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6700K CPU @ 4.00GHz (8 x 4008)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_offsurface_control: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.93GB (16.91GB free)

Process Argv
--enable-proposed-api eamodio.remotehub

Screen Reader
no

VM
0%

I am able to repro the steps on stable but not on the insiders version.

@deepak1556
I can still repro on today's insiders when the devtools are docked to the right (left, bottom, and undocked are fine).
Although, I also noticed an even stranger issue if the devtools are docked to the left (which I toggled by accident). If they are docked to the left, it seems like electron gets confused as to where the window controls are. They seem like they are layered on top of the devtools -- so you can't interact with some of the buttons at the top of the devtools and double clicking in some areas will maximize/restore the whole window, where as dbl-clicking in the vscode titlebar no longer will.

Thanks for the details, I am able to repro this now.


Adding Gem

Adds the following events or companies:

Gem

Checklist

[x] I have updated the event or company counts
[x] I have put the most recent relevant date in the "Last Update" column
[] I have linked to the article about the change, not the company's website

Hi, do you have a public link about the change in policy by chance?

Just an internal email and document. Thanks for the fast response!

Works for me. Thanks!


https://www.sunstar-foundation.org/en/about-us/history - UAT feedback plus CSS changes for text within Picture element to match the original page

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fixes #221
Test URLs:

Original: https://www.sunstar-foundation.org/en/about-us/history

Before: https://main--sunstar-foundation--hlxsites.hlx.page/en/about-us/history

After: https://issue221-uat-feedback--sunstar-foundation--hlxsites.hlx.page/en/about-us/history

[ ] New Blocks introduced in this PR
If yes, please provide details below

Block Name
Documentation

For e.g. cards
doc-link

[ ] New variations introduced in this PR

Variation Name
Documentation

For e.g. cards (grid)
doc-link

The font of the text content within the picture element is also taken care which is there on the actual site as well.


Transition weave-scope Helm chart into this repository?

https://github.com/helm/charts/tree/master/stable/weave-scope will be retired Aug 18 and it might make sense to transition the chart over into this repository to keep it all in one place and potentially attract contributors here as well?
kured has almost completed the process in https://github.com/weaveworks/kured/issues/140 and

https://github.com/weaveworks/kured/tree/master/charts/
https://github.com/weaveworks/kured/blob/master/.github/workflows/chart-lint.yml
https://github.com/weaveworks/kured/blob/master/.github/workflows/chart-release.yml
https://github.com/weaveworks/kured/tree/gh-pages

kured was very lucky to have @ckotzbauer and @davidkarlsen who helped out with the process.
Maybe it'd be possible to invite some of the folks in https://github.com/helm/charts/commits/master/stable/weave-scope and https://github.com/helm/charts/issues?q=weave-scope+is%3Aopen+ to help out?

Meeting is happening next week 28th July: https://docs.google.com/document/d/103_60TuEkfkhz_h2krrPJH8QOx-vRnPpbcCZqrddE1s/edit# - please join to discuss it there.

I reached out to the authors who contributed to the chart and we have a couple of offers of help.
The way we set this up with kured (if we want to follow the same model) is

two Github Actions that are triggered whenever a release is tagged
in https://github.com/weaveworks/kured/pull/150 we copied the chart as is in the first commit, then applied a number of changes to make the chart more compliant with current standards, resolved a couple of inconsistencies, then added the GH actions
if memory serves, we only needed to set up an empty gh-pages branch and enable Github pages - that was all
it needed a couple more commits to get it all working fine, but it wasn't too much work

Does anyone want to set up an initial PR for this?
Scope maintainers (@qiell @bboreham @fbarl @satyamz): Any opinions on how to move forward?

https://github.com/helm/charts/issues/21103 will need an update too.

I've updated the issue in helm-charts.

Thanks for this!
Could you use help pulling in the chart commit history or setting up the Helm GH Actions? See this git repo for an up to date reference https://github.com/helm/charts-repo-actions-demo
I mentioned chart commit history only because this has proven useful elsewhere. It may even be worth bringing back the chart history for kured. For an example on doing this retroactively, see this issue https://github.com/goharbor/harbor-helm/issues/48 I also don't mind helping with a PR, it would just take a little coordination.
One more thought. An idiomatic way to manage an org's charts is a separate git repo holding the source code for multiple charts. The benefit here is simplifying setup for chart CI (testing/releasing) in only one place whenever updates are needed. I'd be happy to setup an example of this so you can see it using the above history splicing on all the weave charts, if that would be helpful (if so, please ping in k8s slack #charts channel in case I miss a reply here).

Thanks @scottrigby for adding more detail to the discussion - we talked about this in the kured meetings and decided we'd like to have the source available in the same repository to focus everyone's attention on the same code. Not sure how the Scope maintainers feel about this.
Losing commit history in kured is a fair point. We could file an issue for that in kured and see if we can easily bring it back.
Thanks also for your offer of help!

@dholbach I can see that too - keeping chart with app source is also legit 👍  Re help, any time 😄

@cpanato @daixiang0 @ckotzbauer and @davidkarlsen all offered help too.
Does anyone want to take the lead and we discuss in the Scope meeting (28th July) ... or any time on Slack?

@dholbach hi, i would to like to do it, let us talk at Slack.

@dholbach I can join the meeting on 28th.

Hey gang, just following up to see if anything new came out of the scope meeting?

So the helm chart upstream is being retired today. Does anyone have time today (or soon) to take a look at this?

This has happened now: https://github.com/helm/charts/pull/24414

Yup - deprecated in the helm/charts repo. Any progress here?

Any news on the transition? Thanks.

Any news on the transition? Thanks.

where is the new helm chart repo?

Unfortunately the transition hasn't happened yet. Help would be appreciated.

https://github.com/squillace/kube-scope-weave depends on the chart as well.

This would be great to see!


Redbiom help update

Codecov Report

Merging #2382 into release-candidate will not change coverage.
The diff coverage is n/a.

@@                Coverage Diff                 @@
##           release-candidate    #2382   +/-   ##
==================================================
  Coverage              93.76%   93.76%
==================================================
  Files                    163      163
  Lines                  18667    18667
==================================================
  Hits                   17504    17504
  Misses                  1163     1163

Continue to review full report at Codecov.

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


Independent asset should be used in the test cases of e2e-transfer-test

What this PR changes/adds
Made test cases of  e2e-transfer-test use independent asset and contract definition.
Why it does that
Invoking EndToEndTransferPostgresqlTest multiple times causes fails due to unintentional reusing of persisted asset information.
Linked Issue(s)
Closes #1824
Checklist

[x] added appropriate tests?
[x] performed checkstyle check locally?
[x] added/updated copyright headers?
[n/a] documented public classes/methods?
[n/a] added/updated relevant documentation?
[ ] assigned appropriate label? (exclude from changelog with label no-changelog)
[x] formatted title correctly? (take a look at the CONTRIBUTING and styleguide for details)

Codecov Report

Merging #1825 (363695c) into main (c2337fb) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1825   +/-   ##
=======================================
  Coverage   67.92%   67.92%
=======================================
  Files         803      803
  Lines       17051    17051
  Branches     1082     1082
=======================================
  Hits        11582    11582
  Misses       4998     4998
  Partials      471      471

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

rebased the PR to address the change of 33f7f4f1228ad612ca496b5b8a191db52b3da9cc.

Tests (including added declinedContractRequestWhenPolicyIsNotEqualToTheOfferedOne) passed in multiple invocation.
$ ./gradlew clean test -p system-tests/e2e-transfer-test/runner -DincludeTags="PostgresqlIntegrationTest"
...
$ ./gradlew clean test -p system-tests/e2e-transfer-test/runner -DincludeTags="PostgresqlIntegrationTest"
---------------------------------------------------------------
|  Results: SUCCESS (4 tests, 4 passed, 0 failed, 0 skipped)  |
---------------------------------------------------------------

BUILD SUCCESSFUL in 2m 26s
116 actionable tasks: 4 executed, 112 up-to-date

The test failure observed in workflow job is intermittently reproducible. Just extending the timeout to 30s did not fix it.
$ for i in `seq 100` ; do echo $i && ./gradlew clean test -p system-tests/e2e-transfer-test/runner -DincludeTags="IntegrationTest" --tests='*EndToEndTransferInMemoryTest*declinedContractRequestWhenPolicyIsNotEqualToTheOfferedOne' -PverboseTest || break ; done
...
EndToEndTransferInMemoryTest > declinedContractRequestWhenPolicyIsNotEqualToTheOfferedOne() FAILED
    org.awaitility.core.ConditionTimeoutException: Assertion condition defined as a lambda expression in org.eclipse.dataspaceconnector.test.e2e.AbstractEndToEndTransfer that uses java.lang.String
    expected: "DECLINED"
     but was: "REQUESTED" within 10 seconds.
        at app//org.awaitility.core.ConditionAwaiter.await(ConditionAwaiter.java:167)
        at app//org.awaitility.core.AssertionCondition.await(AssertionCondition.java:119)
        at app//org.awaitility.core.AssertionCondition.await(AssertionCondition.java:31)
        at app//org.awaitility.core.ConditionFactory.until(ConditionFactory.java:985)
        at app//org.awaitility.core.ConditionFactory.untilAsserted(ConditionFactory.java:769)
        at app//org.eclipse.dataspaceconnector.test.e2e.AbstractEndToEndTransfer.declinedContractRequestWhenPolicyIsNotEqualToTheOfferedOne(AbstractEndToEndTransfer.java:180)
...

@iwasakims yes, I did that in another PR as well, because I noticed that behavior. thanks :raised_hands:


vendor: Update virtcontainers vendoring

This is a new update of the virtcontainers vendoring based on latest changes related to Pod and Process structures. Now the Pod structure does not hold the proxy URL anymore. The URL is duplicated in every
Process structure because it could be a different one from a container to another.

Depends on https://github.com/containers/virtcontainers/pull/191 before we can merge it.

Coverage remained the same at 33.333% when pulling 6734bf54a8e854ff80650b2c908190f1f2391e24 on sboeuf/update_virtcontainers_vendoring into 0a3d1e2075d31cbfb40882486f4ad906ce38c6f0 on master.

lgtm

Dropping this PR as we won't get those virtcontainers changes in the mainline.


Geolocate endpoint, tests and sample app

@pangea-andrest to fix the linting issue that is failing the pipeline, from this directory: packages/pangea-node-sdk run
yarn run prettier:fix

and then recommit those files that changed to get it to pass the linter


[W6.1d][T13-B1]Arnol Lim

LO_W6.1d

@arnollim
Some comments added. Please ack & close the PR after reading comments.

Thank you Sir for the comments tests and default implementation. I will take note.


edit node object in databse so there is an "isoutside"

To be able to dynamically change the outside distances, the node object needs to have a "isoutside" element also. In teh routing algorithm it will change the distance based on cuser parameters

Also, to store the coordinates to give to the frontend, could there be another attribute that is an array of coordinates. The first element being the start, the last being teh finish, and any intermediate points(ORS gives intermediate points to path around things)

I added 'isInside' field and 'coords' field
isInside: If outside distance(from ORS), then 1, else 0
coords: Under features/geometry/coordinates
JSON.get("features").get("geometry").get("coordinates")

The FromToAction elemnt in the edge JSON will list if it is an outside node
The PathShape will list the coordinates of the path


add json lib to wrap db

I'm trying to add json lib to wrap db, since I don't have permissions to push repo, I can't add it. The releases page even contains MD5 sums.
Here's the original repo - https://github.com/nlohmann/json
Here are the releases - https://github.com/nlohmann/json/releases
Latest release zip - https://github.com/nlohmann/json/archive/v2.0.2.zip
There's nothing to build or do, the meson.build file would only contain -
project('json', 'cpp', version : '2.0.2',
		default_options : ['cpp_std=c++11'])

inc = include_directories('src')

That's it.
It would be very helpful if you could add the following lib to wrap db.

Repository and branch created. You can now do the merge request, but please create a dep with declare_dependency so usage is consistent with other subprojects. Thanks.

@jpakkane can declare_dependency omit link_with param, since it is a header only lib.

So does that mean if I use declare dependency in original subproject, this would code -
uWS_dep = dependency('uWS', required : false)
if uWS_dep.found()
  uWS_inc = []
  uWS_lib = []
else
  uWS_sp = subproject('uWS')
  uWS_inc = uWS_sp.get_variable('inc')
  uWS_lib = uWS_sp.get_variable('uWSlib')
endif

should get reduced to -
uWS_dep = dependency('uWS', required : false)
if uWS_dep.found()
else
  uWS_sp = subproject('uWS')
endif

And how is source_hash calculated?


Revert velero CRD removal

Issue: https://github.com/stolostron/backlog/issues/19853
Signed-off-by: Jakob Gray<EMAIL_ADDRESS>

/lgtm


Add logging to USA facts indicator

Description
Add logging to USA facts indicator
Changelog

Add logging for elapsed time, CSVs processed, max lag, etc.
Update prod param for logging output

Fixes

Incremental on #689

Is this ready to merge?

Is this ready to merge?

Yes please.

Yes please.


Minor infinispan schema fix to allow backup-for and backups to coexist

Needs a JBEAP and 7.x.
@pferraro please ok.

Done.


Respect disable_rules_on_error flag for connection errors

Respect disable_rules_on_error flag for connection errors during init_rule.

I think this solves one of the issues in #1926


go build 编译时报错有遇到过吗？

`#go build
github.com/humpback/common/models
../common/models/container.go:200:20: cannot use origContainer.ContainerJSONBase.HostConfig.Resources.Ulimits (type []"github.com/docker/docker/vendor/github.com/docker/go-units".Ulimit) as type []"github.com/docker/go-units".Ulimit in assignment
`

cd GOPATH/src/github.com/docker
rm -rf docker/vendor


Add Rust Moravia meetup #2

I don't know what is the English name for Zlín, lol.
CC @marekpsenka

@Kobzol Baddietown? JK :D Thanks for updating the community index


Set up i2c connection on separate ports for IMU and atmospheric sensor

This is prudent, considering our high port-availability.

BME280 now uses SPI


catkin: 0.7.7-2 in 'lunar/distribution.yaml' [bloom]

Increasing version of package(s) in repository catkin to 0.7.7-2:

upstream repository: git@github.com:ros/catkin.git
release repository: https://github.com/ros-gbp/catkin-release.git
distro file: lunar/distribution.yaml
bloom version: 0.5.26
previous version for package: 0.7.7-1

catkin
* fix warning in case devel space is nested in the build space (#880 <https://github.com/ros/catkin/pull/880>)
* remove workaround using -l:<libpath> necessary for older pkg-config versions (#879 <https://github.com/ros/catkin/issues/879>)
* replace exec call with runpy.run_path (#873 <https://github.com/ros/catkin/issues/873>)
* use environment variable to extend environment in plain shell (#862 <https://github.com/ros/catkin/issues/862>)
* prefer reporting problems to bugtracker / website before maintainer (#861 <https://github.com/ros/catkin/issues/861>)

So, this is a new release of catkin which actually includes the patch to make it install the setup.* files. I think this patch was lost when I was "fixing" the release repo, see:
https://github.com/ros/rosdistro/pull/15613 and https://github.com/ros/rosdistro/pull/15654


Add feature

Probably worth noting that Gorilla does not emit OpenAI function calling compatible messages. They went an "exec" route: https://github.com/ShishirPatil/gorilla/issues/144


Allow for specialized file extensions

I defined this rule for my repo:
'filename-rules/match': [2, { '.ts': 'camelCase', '.styles.ts': 'PascalCase' }],

Actual result:
/Users/simon/Code/repo/src/MyComponent.styles.ts
  1:1  error  Filename 'MyComponent.styles.ts' does not match camelCase  filename-rules/match

Expected result:
MyComponent should be checked to conform to PascalCase, which it does.

@the21st How about making use of ESLint's overrides:
rules: {
  'filename-rules/match': ['error', { '.ts': 'camelCase' }]
},
overrides: [
  {
    files: ['*.styles.ts'],
    rules: {
      'filename-rules/match': ["error", 'PascalCase']
    }
  }
]


Who & Why

Write about it. #1

Who are we?
Well, our name is The Indian wave and what we do is serve soup to you on demand. Only our soup is a bit different, we all html,css and javascript into our soup. The soup we create is also knowns as a website. So if you are in need of soup, the new soup shop in town ( we,we,we ) deserve at least a chance.
Why?
See amazing feature, wow feature etc. Merge them.


Size Limitations on Boost Allocation

HCL runs into limitations on the size of its data objects. Of course, an obvious one is that if you're allocating on the stack then you're limited to the stack size, which turns out to be a few megabytes on my system. If you use Boost shared allocators, you can get bigger objects, but limitations are still hit eventually. I'm using a map that's created like this:
typedef boost::interprocess::allocator<char, boost::interprocess::managed_mapped_file::segment_manager> CharAllocator;
typedef bip::basic_string<char, std::char_traits<char>, CharAllocator> MappedUnitString;
hcl::unordered_map<struct KeyType, std::string, std::hash<KeyType>, CharAllocator, MappedUnitString> *hcl_string_client;

And I can do hcl_string_client->Put(key, vals) where vals is a std::string that contains up to 64 MiB of characters. Once I double that to 128 MiB of characters, I run into a limitation on the Put that reads:
terminate called after throwing an instance of 'std::length_error'
  what():  basic_string::reserve max_size() exceeded

The issue is hit after the Put, which means that the initial string is not the issue (most systems have large string limits that would never be hit in practice anyway, but I also tested it on a local unordered_map), but rather something HCL is creating on Put must be triggering the error. I'll follow up if I can find more details.

The problem seems to be hit in container.h, when you do GetData with a 128 MiB string you have data size 134217728 but the max_size() of value is 134217711. This is the limit of the MappedUnitString. If you know any way to expand this limitation, I'd appreciate it. It would be nice to get up to a GB max size. If not, then we can probably close this issue, as it isn't quite an issue with HCL (though I think it would be nice to document the known limitations somewhere).

Op sizes close to but under 134217711 still fail, but they get a different error. This one is "bad_alloc" instead of "length_error". Figured this was worth mentioning. I don't know what the largest functioning size is, but I was able to get 134112872 to work (roughly 127.9 MiB).


Issues in Firefox

Shown in the console:
TypeError: t.t0.response is undefined Create.js:139:16

Fixed in d68bbf9e1d2e925ff4cd72cbe4011509760aa529.


If we need more then one service to handle requesr?

Hi!
If we need more then one service to handle request, what is the best way to do this?
Integrate another service into this service or into the api?
Thanks!

same request o different request?
same request: best option is spaw another service with maybe docker swarm, kubernetes ...
different request: is just another microservice or another router handler.
maybe we need more info about your problem...

Both are fine. I would prefer to use a single service to make the api layer dumber.


compilation errors due to missing op classes when using selective registration (cmake windows build)

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
NO.
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Windows 7
TensorFlow installed from (source or binary):
From source
TensorFlow version (use command below):
commit 280374
Bazel version (if compiling from source):
n/a, using cmake build
CUDA/cuDNN version:
n/a
GPU model and memory:
n/a
Exact command to reproduce:

cmake .. -A x64 -DCMAKE_BUILD_TYPE=Release -DCMAKE_INCLUDE_CURRENT_DIR:BOOL=ON -DCMAKE_CXX_FLAGS=-DSELECTIVE_REGISTRATION -Dtensorflow_WIN_CPU_SIMD_OPTIONS=/arch:AVX -Dtensorflow_BUILD_PYTHON_BINDINGS:BOOL=OFF -Dtensorflow_BUILD_CONTRIB_KERNELS:BOOL=OFF -Dtensorflow_BUILD_SHARED_LIB:BOOL=ON
MSBuild /p:Configuration=Release tensorflow.vcxproj
Describe the problem
I'm building with cmake on Windows, using selective registration.  I've put ops_to_register.h in my build dir and specified -DCMAKE_INCLUDE_CURRENT_DIR=ON in my cmake command so that it would be added to the include path.
Due (I'm assuming) to selective registration, the generated code omits many classes which would otherwise be generated in a full build.
When compiling non-generated code, e.g. tensorflow\cc\gradients\array_grad.cc, there are compile errors wherever it references the classes that were not generated due to selective registration, for example:
array_grad.cc(51): error C2653: 'Unstack': is not a class or namespace name [...omitted...\tensorflow\tensorflow\contrib\cmake\build\tf_cc.vcxproj]
There are many of these errors for various referenced classes, in various other source files such as:
tensorflow\cc\gradients\math_grad.cc
tensorflow\cc\gradients\nn_grad.cc
tensorflow\cc\framework\gradients.cc
When using selective registration, what's the right approach to avoid these errors?

I'm not aware of anybody who's successfully using the selective registration feature with the CMake build... those build rules are designed to do the minimal amount possible to get the Windows PIP packages built while we wait for full Bazel for Windows support.
I suspect we'd need more specific targets in the CMake build to support selective registration (e.g. excluding the C++ gradients code). You might be able to start by looking at the dependencies for the Bazel Android targets here:
https://github.com/tensorflow/tensorflow/blob/e4296aefff97e6edd3d7cee9a09b9dd77da4c034/tensorflow/core/BUILD#L835
I'm going to mark this as contributions welcome, but please feel free to ask more questions on here if you're interested in pursuing this feature.

As contrib folder is deprecated , this issue is not relevant.


Caffe.model

Is image-net trained caffe model, corresponding to 23.654 Top-1 error rate, hosted anywhere?  If yes can you pls share a link?

@soyebn I'll tell you when I share this model.

+1

Is the imagenet pretrained model available?


Container fails to run without --init parameter

I couldn't get the container to run without adding the docker --init parameter to the run command.  I don't fully understand why but I think it has to do with the fact I am running docker 1.13.
It took me a while to stumble upon this solution so assuming that this is a correct approach it would be helpful for others to update the instructions.
Thanks for an awesome container by the way!

What was the problem without the --init parameter?

When I run without --init I get the following error:
/usr/bin/docker-current: Error response from daemon: oci runtime error: container_linux.go:247: starting container process caused "process_linux.go:339: running prestart hook 2 caused "error running hook: exit status 1, stdout: , stderr: "".

Closing this issue.  Please re-open if needed.