re-re updates on Teach page

Changes:

merged teach branch - up to date with the current main branch
added teaching resources submitted on src > templates > pages > teach.hbs  & korean translation in ko.yml when applicable
according addition of lines on i18n-tracking.yml and src > data > en.yml, es.yml, zh-Hans.yml, ko.yml files,
changes in the "teach" section of main.css for a better readability of teaching resources list

@limzykenneth @montoyamoraga thank you again for your time and responses, and hope it works better this way! 😁😁

It seems the teach branch is very outdated. There's too many files to review so I'm going to skip that. As long as the site builds and looks correct to you, I don't mind this being merge as is. We can inspect any additional diff when it comes time to merge into main.

oh i see, i hadn' noticed that, this is for merging @yinhwa's teach branch to this repo's teach branch, ok i agree with @limzykenneth, i merged it :)

Hi @limzykenneth and @montoyamoraga thank you all for checking in!🥳 Thought would be easier to work on main branch directly since the teach branch hasnt been updated for so long 😅 but then i thought i should keep the teach branch stay updated as it seems to remain in the tree for some reason maybe🤔? And I think and hope that merging into main wont be too problematic. All I did for this commit was to pull the updates from main branch into teach branch and made some changes on the teach-relevant files :) anyhow much thanks to all your efforts and time 🙏🙏

@yinhwa Content you are ready to show on the live website should be merged into main as that's where content of the site is generated from but if you are still working on it, merging into teach is fine and we can do a branch to branch PR when you are ready to release the changes. Just let us know if you want the new changes to go live and you are not sure how to create the appropriate PR. :smiley:

@limzykenneth Cool 😁😁 thank you  as always for the kind guideline! I'm just gonna review the contents, and will let you know when I think is ready to merge!


kiss: circular dependency detection

This makes the package manager cleanly exit when circular dependencies
are found rather than segfaulting (in some shells)
I changed the output redirection behavior slightly (no more global
stderr suppression) to allow the error message to appear. It now also checks
for file existence before attempting to read (we would silently ignore read
errors).

Will resubmit.


Add Swift 4 support

IBM-Swift/Kitura#1121

Codecov Report

Merging #4 into master will increase coverage by 7.69%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master       #4      +/-   ##
==========================================
+ Coverage   90.25%   97.95%   +7.69%
==========================================
  Files           4        3       -1
  Lines         154       49     -105
==========================================
- Hits          139       48      -91
+ Misses         15        1      -14

Flag
Coverage Δ

#Health
97.95% <ø> (?)

Impacted Files
Coverage Δ

Sources/Health/Health.swift
100% <ø> (ø)

Sources/Health/Protocols.swift
95% <ø> (ø)

Sources/Health/Utils.swift
100% <ø> (ø)

Tests/HealthTests/HealthTests.swift

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 d2a7f57...7c822f8. Read the comment docs.

@rolivieri Please review


[ci skip] NO_CI Use grayskull in autotick

May help with #10, as package has a fair number of dep updates.
Run autotick analysis via grayskull.

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

@asford I was not aware of this PR / bot. Can you enlighten me with docs or some link on what this PR does and if want to use it to maintain other repositories, what I could possibly do? Thank you.

Totally, this is not a well documented feature but is super useful.
Flytekit also has a bunch of plugins which currently have broken packages that could probably be fixed via an autotick update of this type.


Fix configuration commands

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Bug fix

What is the current behavior? (You can also link to an open issue here)
Commands to change configuration (video/audio input, file format) currently return a 100: Syntax Error response.
I believe the current command string is in the format configuration: <setting>

What is the new behavior (if this is a feature change)?
This should send the correct command syntax, in the form configuration: [video|audio|format]: <setting>

Other information:
It's not tested, as I don't know what I'm doing with TypeScript! ;-)

This would output configuration: videoInput: video input: <setting> because the parameter key is used as key in the output as well.
it should instead be rewritten to if (this.videoInput) res.params['video input'] =

Thanks for the correction - should be fixed now.


SUGGESTION: subreads.bam prefix for ccs_report file

Is it a good idea to make the default output file for the ccs output report something like <subreads.bam>_ccs_report.txt ?

I also thought about that, but I don't want to disrupt existing workflows. That's why there has been --report-file for a long time.


Calling builder.buildStatic without provided option browser: true or node: true results in error

Hey,
I got stuck for several hours :-1:  trying to get buildStatic working properly. Eventually, I realized that I needed to define the environment to target.
However, this is in direct conflict with documentation which states:

By default builds, like bundles, build for the browser in production

This might be true when using the CLI for bundling, but it is not true when running Builder manually in JS.
I don't know if that's an error or not, but there should probably be some safe guards ensuring the proper options have been provided for environment targets.
Thanks! :balloon:

Ok I've created https://github.com/jspm/jspm-cli/issues/1543 to track this further.
In the builder project itself I think we should have a better message when getting the "exact condition" error, for how to proceed.

I've made a note at the API document page. Will tackle this as part of the documentation process to point to a good source of info for this topic.


Couldn't set up WebRTC dataChannel

Thanks for filing a bug report :-)
Steps to Reproduce

Followed steps to deploy artifacts directly from a PR https://github.com/raiden-network/light-client/tree/master/serve_pr
Connect a new address on goerli
mint deposit SVT tokens
Connecting to the TTT token
instead of the default hub account open a channel with another light-client opened similarly  on another browser. Matrix request is success for this.

Expected Result

Webrtc datachannel opening should be possible.

Actual Result

logs of the node.
raiden_0x3de03B8dD112bA12e785f2eFC91D8bf7Ad48806f_2020-09-01T09_13_19.755Z.log

On the current master somehow this issue seems to have solved itself

Hence closing the issue for now. It could be re-opened if we see the issue happening again.


Shutdown server runtime before program exit

We don't ever terminate the server runtime in benchmark when we exit the program. This causes RocksDB running in the server runtime to try to access global state which is cleaned up during exit. This PR terminates the server runtime with a oneshot channel and then exit.
Before this fix, I can reproduce the error in almost every run on my linux computer. After this PR, I did not run into any errors even after 10+ runs.

Great!


oneagent operator deploys oneagent pods with broken readiness probes

After installing releases v0.3.0 the oneagent pods are not ready we get the kubernetes event:
Readiness probe failed: grep: /proc/*/stat: No such file or directory
I see that this is fixed in the master branch but not yet in the v0.3.0 release.

Apparently this was

@MichaelMeelisRTL are you sure that you didn't use the master branch (snapshot) release?
The 0.3 branch does utilize still pgrep as readiness check..
Thank you in advance.

Hi @esukram,
You are right something went wrong with our deployments and this showed up. I will close this ticket.

Thank you, for your quick response!


Terminate runing request before closing the server to fix issue #189

Ensure the api router is completed/terminated before returning to the http_listener request handler.  This will prevent the API routers to be deleted before the http_listeners while deleting the server instance.

@garethsb please review

Is it a good idea to make the api_router::operator()(web::http::http_request req) function blocking? I think this is likely to cause thread pool starvation and deadlock under load. Basically any blocking wait in http_listener handler code is an anti-pattern, isn't it?

Obviously I ought to have added unit tests to show the problem that was solved by 345e344d99710817e1269b7396dbb46f3b1ce5ac, and equally this needs unit tests to show the problem identified by #189.

My suggestion is to bind the api_router into the handlers by value rather than reference, and to give api_router a shared pimpl. And keep the server code unchanged.

PR  superseded by #191


Rails 5 Not compatible?

google-webfonts-rails-0.0.4/lib/google-webfonts/rails/helpers.rb:7:inblock in module:Helpers': undefined method helper_method' for ActionController::API:Class (NoMethodError)
Any idea how to remedy for rails 5?

Hi! Got the same issue here :( Can you please merge https://github.com/misopeso/google-webfonts-rails/pull/2

Hi there! Any news?

@n-rodriguez, no updates that I know of...

@misopeso hi! this issue can be closed. Thank you!


CreateDeltaPackage fails when retrieving DataExtensions

When running createDeltaPackage it sometimes causes to crash when it retrieves DataExtensions with fuel-soap.
Log:
info: Retrieving as Template: dataExtension
events.js:292
throw er; // Unhandled 'error' event
^
Error: Error: Invalid argument for the equals operator. Filter[0] cannot be null or empty.
at /node_modules/mcdev/lib/metadataTypes/MetadataType.js:532:35
at FuelSoap. (/node_modules/fuel-soap/lib/fuel-soap.js:131:6)
at FuelSoap. (/node_modules/fuel-soap/lib/fuel-soap.js:603:5)
at Parser. (/node_modules/xml2js/lib/parser.js:304:18)
at Parser.emit (events.js:315:20)
. . . . . . .

looks like this only happens when retrieveAsTemplate (rt) is executed for a non-existing data extension. In the case of createDeltaPkg I would assume that somebody renamed or deleted a DE that was previously committed to git - but then forgot to commit that rename/delete.
Let me try to improve the error handling & log/cli output for this case.

it now shows proper error messages when rt is run for a name that cannot be found on the server


Impossible to "link to controller"

Hello
I'm on KK M32, I have installed your script and it's work well, thank you!
However, the mixer knobs are locked on volume and panning and i can't link DAW knobs to controller.. I would like to be free to link whatever buttons of the DAW i want with any knobs of my M32 (like the basic fl model). Is there a way to do that ? Modify or delete a part of the script/code?
Thank you

Assuming issue is fixed due to no further discussion.


DATAGO-51728 - Enhance documentation for Discovery offline upload

What is the purpose of this change?
Update documentation for offline upload
How was this change implemented?
Updated README.md
How was this change tested?
N/A
Is there anything the reviewers should focus on/be aware of?
No

Thank you @gregmeldrum. I addressed your comments.


[templates] remove or update: todo-csharp-pgsql-redis-aca

Template seems abandoned/leaked or never hooked to azure-dev repo-man.
@jongio , do you know what's up with this template?
https://github.com/Azure-Samples/todo-csharp-pgsql-redis-aca
It is is not maintained, we should remove it to prevent noise to customers

@ahmelsayed?


feat: add post by Siddharth1006

Made a post on Creating a simple Spring Boot Application.

wrong date + adding images in this repository only work as author image on configuration

So, do I keep the image in the same folder as src/content/posts?

I see. Will fix the date. Also, so what do I do with the image? Do I keep the image in the same folder as src/content/posts?

you can add the image somewhere else and copy the link and add it using <img> tag

I see. Will fix the date. Also, so what do I do with the image? Do I keep the image in the same folder as src/content/posts?

you can add the image somewhere else and copy the link and add it using <img> tag

Is it ok if I create a separate folder like src/content/postImgs and move the image there?

I see. Will fix the date. Also, so what do I do with the image? Do I keep the image in the same folder as src/content/posts?

you can add the image somewhere else and copy the link and add it using <img> tag

Is it ok if I create a separate folder like src/content/postImgs and move the image there?

adding images in this repo wont work

Made the respective changes. Have a look.


Inline error checking in Configuration project

It's very easy to make mistakes in a configuration and not know it.
For instance:

#224 override keys that have conflicts
Use the wrong "syntax" (use keys that have no meaning or in the wrong contexts). Some sort of "schema check" would help here
Set invalid values to keys

etc.
These should be marked inline, as you type.

Using http://json-schema.org/


Where to validate route parameters?

So I'm setting up an express server for custom routing.
Suppose I have a route like this /u/:username . After parsing the query param,  I need to hit the database, find that username and decide whether I should render a profile page or an 404 page.
I have 2 places to do the validating in mind:

At express server

Pros: Sending correct HTTP_STATUS code because I can't find a way to send 404 status within the React app
Cons: Hit the database server twice because in the client side, I need extra information rather than just a validated username to render the page. Query all of the data in the express server and sending it via ctx would be super complicated. For clarification, I'm using Apollo Client for managing the data.

At React app

Pros: Hit the database server only one
Cons: Improper HTTP_STATUS code, complex error handling. (well, try catch everywhere! OMG)

https://github.com/zeit/next.js/issues/746#issuecomment-274850407 is the way we recommend.
Additionally, you can import the default error page as next/error and use it for rendering from the next release.

@nkzawa oh nice. Thank you. I was worry for a while before posting the above post. Putting the logic in express mean I have to keep the code in sync manually.


feat: add collectFromComposedSchemas() and getExamplesForSchema() to utilities

PR summary
Adds a generic collectFromComposedSchemas() function that can be used to build functions like the also-added getExamplesForSchema(), and refactors getPropertyNamesForSchema() to use collectFromComposedSchemas() as well.
PR Checklist
General checklist
Please make sure that your PR fulfills the following requirements:

[x] The commit message follows the Angular Commit Message Guidelines.
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)
[x] Dependencies have been updated as needed
[x] .secrets.baseline has been updated as needed
[x] npm run generate-utilities-docs has been run if any files in packages/utilities/src have been updated

:tada: This PR is included in version 1.6.0 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.26.0 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.30.0 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:


Add info box with description and id on searchable ontology terms

Definition of done
when hovering a particular term from an ontology (on a dropdown or elsewhere) we should display a description dialog/tooltip with the corresponding id and description of the said term. The id could also link to the external page for the given term.

See commit 2826f5

@belkassaby I see the ontology details only in disease Ontology drop down. Is there any other place where I should be viewing the tooltip or dialog?

@belkassaby :looks good now. But I see the ID but  i see prefix from CL and GO and UBERON. Is that correct. I thought we were only iimporting from HPO site. Are we integrating with some other application too inorder to get the ontologies?

@belkassaby yes, the disease ontology for instance comes from an ingested file which is built from the mondo ontology (and which has terms from different ontologies, orpha, go etc). We get the phenotypic features from the HPO ontology.


Could you add a license to the repo?

Hi there, I'm interested in playing with this code and possibly contributing back some improvements... but I'm not sure how it works when there's no license file in the repository.
Could you add a license file? I'm a big fan of the MIT license encase you needed a suggestion :)
Thanks!

+1, thank you for sharing the code. It would be great if the licensing is clear.

Hi,
Thank you very much for your feedback and sorry for the slow response.
I wanted to dig a bit into this topic as i have not much experience with licencing yet, but currently it's a bit hard to find time, so i need to ask a little more patience.

@hullarb can you please add license file to the repo


[RESOURCE] Your Daily Dose of Climate Hope by Climate Action Now

Resource title
Your Daily Dose of Climate Hope
URL
https://climateactapp.substack.com/
Relevant tags (1 per line)
Climate Journalism
Politics
Resource Summary
A newsletter sharing the most inspirational stories in climate action policy and beyond, from the makers of the Climate Action Now app.
Relevance
It’s another news outlet. To quote their mission statement: “Action is the antidote to despair. Subscribe now to get one easy action to take every day, along with stories about the trends, innovations, and people creating a better world!”
Other comments
No response

@JarrodBaniqued I have removed the following tags:

Collective Action
Contacting representatives

I'd like to keep those tags more for tools to help contact representatives and for orginisations/initiatives that people can get involved in. A news source may have some overlap there, but it wouldn't be a main focus.
I have also added #404 after hearing about Climate Action now, let me know what you think :)


add metrics for fast vs slow password hashing

Overview
bump a counter whenever we do a slow or fast password hash, to guide administrators on setting the password cache configuration appropriately and to expose a potential source of slow performance.
Testing recommendations
The counters are visible in the metrics / Prometheus output.
Related Issues or Pull Requests
https://github.com/apache/couchdb/pull/4814
Checklist

[x] Code is written and works correctly
[x] Changes are covered by tests
[ ] Any new configurable parameters are documented in rel/overlay/etc/default.ini
[ ] Documentation changes were made in the src/docs folder
[ ] Documentation changes were backported (separated PR) to affected branches

Nice work!


FN for Flashloan attack bot

BOT ID - 0x55636f5577694c83b84b0687eb77863850c50bd9f6072686c8463a0cbc5566e0
Tx -
https://snowtrace.io/tx/0xab5f6242fb073af1bb3cd6e891bc93d247e748a69e599a3744ff070447acb20f
https://bscscan.com/tx/0x24a2fbb27d433d91372525954f0d7d1af7509547b9ada29cc6c078e732c6d075

for the first one, it must be a dropped tx issue because it normally works. I'll check the performance.
the second one doesn't meet 2% of threshold.


Request: arbitrarily shapped items

Is it planned at all to add the ability to define the shape of something in any shape? the use case is pretty much tetris-shaped items, but if the ability is added to make tetris shaped, might as well make the ability to make it any shape, just in case someday someone wants a weird 8x8 square where it only takes up inventory slots in the 4 corners

Probably, yeah. I'm seeing it as a 1D array of booleans inside an ItemType that stores whether each cell is taken up by it.

[ ] Inspector view to edit ItemType's shape, click to toggle cell, grid size based on item size properties
[ ] Make Inventory respect shape during can_place_item, get_free_position, get_item_at_position
[ ] Allow 90-degree rotations of items in GrabbedItemStackView. Will likely create a Transform2D to rotate, based on an extra_properties key.
[ ] Make ItemInstantiator respect shape during random insert


Single TCP Session for HTTP Requests

Instead of establishing an entirely new TCP session and closing it for each individual request, only one session is created.
This approach saves a lot of time. To be more accurate, for every individual request after the first one that created the socket, the following packets will not be sent (S) or received (R):

(S) SYN
(R) SYN-ACK
(S) ACK
(S) FIN
(R) FIN

I've ran a test (pull list 100 times back-to-back) multiple times in a controlled environment (set the delay to 20ms in VMware) and the results are:

One TCP session per request: 7.536s
One TCP session for all requests: 4.712s

The savings here aren't necessarily on the bandwidth aspect, but the delay one; all of the above packets are blocking (the client/server must wait until the corresponding packet is sent/received).
Furthermore, the delay in the test was set 20ms. The higher the delay and/or the volume of requests, the worse this effect becomes.

I've just realized that there already a pull-request for this... Feel free to close this one :)


Update pause button on paused

 Issue by owalk
Monday Feb 05, 2024 at 18:46 GMT
Originally opened as https://github.com/Sean-Nishi/M1Wengine/issues/411

Button should display the status of if the pause is enabled or disabled

The button class will need to be updated to take in the str to display when in the toggled-on state, and then the parameters will need to change wherever Button classes are initialized to account for those changes

There isn't really any interaction or exposure between logic once the button is initialized which is good, so it makes a bit more sense to keep the scope of the button logic closed off from outside code.
If the feature of changing the button text during runtime is required, that could be a new method in button. But that feature makes more sense to me as a 1 time adjustment rather than a constant per iteration adjustment


go/build: Passes bad $GOROOT to go list subprocess

What version of Go are you using (go version)?

# this sdk came from `go get golang.org/dl/go1.13.3`,
# I'm fiddling with PATH to eliminate other installed Go
# versions to reproduce with a known good version,
# without parent/child version mismatch later.
$ PATH="$HOME/sdk/go1.13.3/bin:/bin:/usr/bin" go version
go version go1.13.3 linux/amd64

Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (go env)?
go env Output
PATH="$HOME/sdk/go1.13.3/bin:/bin:/usr/bin" go env
GO111MODULE="on"
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/tv/.cache/go-build"
GOENV="/home/tv/.config/go/env"
GOEXE=""
GOFLAGS="-trimpath"
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GONOPROXY="mntn.llc/go,*.mntn.llc,eagain.net/2019,eagain.net/tools"
GONOSUMDB="mntn.llc/go,*.mntn.llc,eagain.net/2019,eagain.net/tools"
GOOS="linux"
GOPATH="/home/tv/go"
GOPRIVATE="mntn.llc/go,*.mntn.llc,eagain.net/2019,eagain.net/tools"
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/home/tv/sdk/go1.13.3"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/home/tv/sdk/go1.13.3/pkg/tool/linux_amd64"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/home/tv/z/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/home/tv/tmp/go-build434657787=/tmp/go-build -gno-record-gcc-switches"

What did you do?
~$ mkdir z
~$ cd z
~/z$ go mod init example.com/m
go: creating new go.mod: module example.com/m
~/z$ cat >main.go
package main

import (
	"fmt"
	"go/build"
	"os"
	"runtime"
)

func main() {
	fmt.Printf("runtime.GOROOT=%v\n", runtime.GOROOT())
	_, err := build.Import("example.com/m", ".", build.ImportComment)
	if err != nil {
		fmt.Printf("error: %v", err)
		os.Exit(1)
	}
}
~/z$ PATH="$HOME/sdk/go1.13.3/bin:/bin:/usr/bin" go run .
runtime.GOROOT=go
error: go/build: importGo example.com/m: exit status 2
go: cannot find GOROOT directory: go

exit status 1

What did you expect to see?
Successful run, with ImportDir returning metadata about the package example.com/m that I just created.
What did you see instead?
The go list subprocess is spawned with a broken GOROOT=go in environment, an explicitly set $GOROOT prevents the child from deducing GOROOT from /proc/self/exe, and this causes it to fail.
Confirmation that $GOROOT is passed:
~/z$ PATH="$HOME/sdk/go1.13.3/bin:/bin:/usr/bin" sh -c 'go build && strace -f -o log -v -e execve ./m >/dev/null; grep execve log|tail -1'
1190329 execve("/home/tv/sdk/go1.13.3/bin/go", ["go", "list", "-compiler=gc", "-tags=", "-installsuffix=", "-f={{.Dir}}\n{{.ImportPath}}\n{{.R"..., "example.com/m"], [...skipped things..., "GOOS=linux", "GOARCH=amd64", "GOROOT=go", "GOPATH=/home/tv/go", "CGO_ENABLED=1"]) = 0

It seems golang.org/dl/go1.13.3 contains GOROOT munging logic. But that doesn't help here unless that executable is in PATH as "go" (not "go1.13.3"). This bug could be "fixed" by documenting that nobody must ever call anything under ~/sdk directly.

This only triggers with modules, not with GOPATH (go/build doesn't spawn subprocesses in gopath mode).
Real world code that broke because of this: https://github.com/google/ko

Duplicate of #34860

Wait, no, that's the working directory, not GOROOT.

The fact that the
	fmt.Printf("runtime.GOROOT=%v\n", runtime.GOROOT())

line prints
runtime.GOROOT=go

suggests that this isn't actually coming from go/build at all: it's just passing on what the go command told it.

I'm unable to reproduce the reported behavior.
m$ go1.13.3 mod init example.com/m
go: creating new go.mod: module example.com/m

m$ cat >main.go
package main

import (
        "fmt"
        "go/build"
        "os"
        "runtime"
)

func main() {
        fmt.Printf("runtime.GOROOT=%v\n", runtime.GOROOT())
        _, err := build.Import("example.com/m", ".", build.ImportComment)
        if err != nil {
                fmt.Printf("error: %v", err)
                os.Exit(1)
        }
}

m$ PATH="$HOME/sdk/go1.13.3/bin:/bin:/usr/bin" go run .
runtime.GOROOT=/usr/local/google/home/bcmills/sdk/go1.13.3

m$ PATH="$HOME/sdk/go1.13.3/bin" go run .
runtime.GOROOT=/usr/local/google/home/bcmills/sdk/go1.13.3

m$

Is it possible that you have a GOROOT entry in /home/tv/.config/go/env, or an explicit GOROOT environment variable set?

I have no GOROOT in environment, only GOFLAGS=-trimpath in ~/.config/go/env.
As far as I can understand, the problem comes from go/build using runtime.GOROOT()'s poor guess (internal/sys.DefaultGoroot as the fallback).
I'll try to make an effort to reproduce inside a reproducible container or something.


Update dependencies & fix warnings

Two small unrelated changes.

Sorted, with package sorted according to https://github.com/rust-dev-tools/fmt-rfcs/blob/master/guide/cargo.md


Add Display::Content layout type

What problem does this solve or what need does it fill?
As we move towards a more reactive UI framework, there is a question of how to support conditional rendering and "foreach" nodes.
By conditional rendering, I mean a subtree of the UI that either exists or does not exist based on some condition, such as a modal dialog, a game mode, a tab panel and so on. Generally speaking, when a UI mode is not visible, you don't want to simply "hide" the elements, you want them to not exist at all - because the existence of these nodes often has side-effects beyond simply their impact on the layout.
Similarly, it is often the case that you'll have an array of items that are rendered from some data source that is an iterator, such as a list of saved games.
Immediate-mode frameworks have no problem with either of these patterns. However, a UI that retains elements wants to preserve the node tree where possible, otherwise you lose state (things like cursor position, text selection, focus and accessibility cues). This means that we want to avoid re-generating nodes when they have not changed. There are various strategies for this: the "React" approach is to generate a new list of child entities and compare that list with the old one; The "Solid" approach is to remember each child's index relative to its parent, and then "patch" the list as needed.
However, conditional and iterative UI elements are a challenge because they change the size of the child entity list, and alter all of the indices, making both comparing and patching difficult. For example, if I have a UI node whose children consist of a list of saved games followed by a "save" button, then the list index of that save button is going to change when additional save games are added. This is a challenge for both the React and Solid approaches.
For conditional nodes, a dummy "placeholder" node can be used to take up a child slot when the child is not being rendered. However, you then need to inform the layout engine to ignore this node in layout calculations. For lists, it's more complex, because a simple placeholder won't work.
What solution would you like?
The solution to all these problems is Display::Content. This is an idea that is relatively new in CSS, and is not supported in all browsers yet, but makes sense in the context of Bevy UI. The basic idea is simple: a node with Display::Content is handled by the layout engine as if it were replaced by its children. That is, the children of the element are hoisted up one level by the layout engine and rendered in place of that element. If the element with Display::Content has no children, then it is simply ignored.
For conditional rendering, we create a placeholder node that represents the condition itself, and its single child is actually the conditionally-rendered content (or no child if the condition is not enabled). For iterative rendering, the placeholder node has a variable number of children depending on the number of elements in the array. In both cases, it means we can re-evaluate and re-generate the children of the conditional node, without altering the list indices of any of its siblings.
What alternative(s) have you considered?
The alternative (that I can think of) is to keep track of child node indices as a separate data structure, but this requires a lot more bookkeeping.

Simple example of why this is useful (if I understand it right):
We implement a game menu:

We save a game (I realise now this menu doesn't really make sense, suppose that when you press "New Game" it just makes a save file), which adds a new "Load Save 3" button to the menu:

The menu's implementation inserts the child at an index calculated from the previous number of "Load Save" buttons plus the number of other buttons before the "Load Save" buttons in the children list.
This is fragile and complicated, so we wrap the "Load Save" buttons inside a single child node instead:

This is better. Now we don't have to worry about indices, we just add the "Load Save" button nodes as children to child 1.
Okay but now we're bothered about the empty space at the bottom of the menu.
We decide that we want to divide the space up evenly inbetween the buttons, so we set AlignItems::SpaceBetween on the parent node. But:

child 1 is considered a single item by the layout algorithm and so space isn't added between our "Load Save"
However, if we had a Display::Contents style property then we could set it on child 1. Then child 1's children would be hoisted up the hierarchy and arranged as though they are children of the root parent menu node:

Your analysis is 100% correct.
I realize that the example of a save menu is a bit artificial, but you do find this pattern in React/Solid code a lot where a list of items is rendered at the same level as other items.
Let me take the time to describe how you would handle these cases without implementing Display::Content: Basically when you render the children of a UI node, you would build the same structure as described - that is, conditional nodes and iterative nodes would have placeholder parents. You then run a separate "flatten" step after rendering, and then set the actual children of the node to the flat array. However, you also keep around a copy of the non-flat array in a separate component, and when you are diffing / patching you use that non-flat array as your base of reference.
Before taking this on, I would check to make sure that this is something that the new UI framework is actually going to use.

So, I'm surprised and pleased that you actually implemented this so quickly. I have a few more notes.

Another use case for this is to implement something like React.Fragment.

I've been forging ahead with my own work on UI, and it turns out that the alternative approach is not as hard as I expected. The way this works is that view components which render an element tree return an enum that looks like this:

#[derive(Debug, PartialEq, Clone)]
pub(crate) enum TemplateOutput {
    // Means that nothing was rendered. This can represent either an initial state
    // before the first render, or a conditional render operation.
    Empty,

    // Template rendered a single node
    Node(Entity),

    // Template rendered a fragment or a list of nodes.
    Fragment(Box<[TemplateOutput]>),
}

impl TemplateOutput {
    /// Flattens the list of entities into a vector.
    fn flatten(&self, out: &mut Vec<Entity>) { ... }
}

Once a view element has called the render function of its children, it has a vector of TemplateOutput objects. We can then call flatten to obtain a flat vector of UiNodes entity ids. This list of entities is then passed to .replace_children().
So the question for me now is whether to keep my workaround, or adopt the Display::Contents solution. (Note that Display::Contents has additional use cases beyond flattening template output, so it's not wasted work either way.)

So, as I've continued in my UI research, I've determined that I no longer need this feature; this problem can be solved at a higher level. So if you want to close this, feel free.


Nuspec versioning fails if nuspec file is xml namespaced

I ran into an issue with the nuspec versioning task - it was failing with the following
System.Management.Automation.RuntimeException: The property 'InnerText' cannot be found on this object. Verify that the property exists and can be set.
I tracked this down to this line
$xml.SelectSingleNode("/package/metadata/version").InnerText = $NewVersion
It turned out that my nuspec file had an xml namespace declaration
<package xmlns="http://schemas.microsoft.com/packaging/2010/07/nuspec.xsd">
which causes the call to SelectSingleNode to fail - I resolved it by removing the namespace declaration from my nuspec file
Not sure that this is a simple fix - apart from adding either name space param or nuspec schema version (which would translate to ns) - some discussion here nuspec schema
that's the only hiccup I had - otherwise these tasks have enable me to delete a lot of ps scripts scattered throughout projects ! - nice work

I need to look if I can come up with some more prescriptive xpath to avoid the clash.
Thanks for point this out

Altered the xpath query to account for the namespace
In version 1.7.3 of the extension


Editor remains less bright after window in foreground is closed

Godot version:
Godot 3.2 Alpha 3
OS/device including version:
Win 10 x64
Issue description:
If you use the one click deploy feature for android and you have "use custom build" checked in the export preset then the "Building Android Project (grandle)" window comes into the foreground and the editor gets less bright because it is now in the background. But after finishing, the editor is still less bright even if it is now in the foreground.
There are a few other situations when the editor remains less bright. Easiest to explain:
Steps to reproduce:

Make sure that you can use one click deploy for android
Create a new, blank GLES3 project
Click on the android icon
Now Godot asks for a main scene, click on cancel or in the background
Now you should see that the editor is still less bright

CC @YeldhamDev

Can't really play with one-click deploy stuff here, so this complicates things. Could you tell the other situations where this happen?

Can't really play with one-click deploy stuff here

What about the "one click preview in browser" feature?

Create a new, blank GLES3 project
Add HTML5 as export preset
Click on the HTML5 icon
Now Godot asks for a main scene, click on cancel or in the background
Now you should see that the editor is still less bright

Oh, that. That's because currently, Dialogs check for any open modal before making the editor bright again, but because PopupMenus don't change brightness but are still modals, they end up leaving everything dark.
A quick fix would be to check if the top modal was indeed a dialog, and if not, make the editor bright again. But this would make so that if a dialog has a popup that opens another dialog, closing it would make it erase the dim while there's still another open actual dialog.
It would be better if PopupMenus closed if a Dialog modal opened.

Similar to https://github.com/godotengine/godot/issues/32690


[Feature Request]: add an optional second author to case-study on https://beam.apache.org/case-studies/

What would you like to happen?

update structure of https://beam.apache.org/case-studies/, so it is possible to add 2nd author to case-study
this should an optional param

Issue Priority
Priority: 3
Issue Component
Component: website

.take-issue


Confused about the Equation 6. in paper

hey, i am confused about the x_m-x_n and y_m-y_n when they are equal to zero under special condition using faster rcnn object detector, because center coordinates may be same and results to log(0).  How do solve this problem?

@Jordan-5i You are absolutely right, about the corner case of two bounding boxes with the same center coordinates.
To cover that explicitly in equation 6,  one should replace |x_m-x_n| by min(|x_m-x_n|, eps) for some small constant eps>0.
In our code we do that with eps=0.001*w, and analogously so for the y coordinates.
For reference here is the code I am referencing (notice we also consider the degenerate case of w=0, or h=0).
https://github.com/yahoo/object_relation_transformer/blob/f21674d5c1095fc104ff3b69bfa41cfeea7568db/misc/utils.py#L245-L255

@Jordan-5i You are absolutely right, about the corner case of two bounding boxes with the same center coordinates.
To cover that explicitly in equation 6,  one should replace |x_m-x_n| by min(|x_m-x_n|, eps) for some small constant eps>0.
In our code we do that with eps=0.001*w, and analogously so for the y coordinates.
For reference here is the code I am referencing (notice we also consider the degenerate case of w=0, or h=0).
https://github.com/yahoo/object_relation_transformer/blob/f21674d5c1095fc104ff3b69bfa41cfeea7568db/misc/utils.py#L245-L255


Empty cpu flags on windows

I have been using the systeminformation library for an electron project,
and I'm trying to make a list of cpu flags.
However, the following code only works as expected on linux:
si.cpuFlags(function(data) {
    console.log(`flags: ${data}`);
});

On windows, data returns as an empty string, and I haven't tested mac.
Let me know if you could fix this, or if there's a reason it won't work on windows.

Hi Jelmer,
yes, cpuFlags is currently unsupported for Windows (see documentation). I didn't found a way to determine CPU flags till now. But if you have any idea, that would be great. On OSX, Linux cpuFlags() is working fine.

Hi Sebastian,
I figured it should be possible on windows as well,
so I made a quick implementation for it.
let result = '';
if (_windows) {
  exec(`reg query "HKEY_LOCAL_MACHINE\\HARDWARE\\DESCRIPTION\\System\\CentralProcessor\\0" /v FeatureSet`, function (error, stdout) {
    if (!error) {
      let flag_hex = stdout.split("0x").pop().trim();
      let flag_bin_unpadded = parseInt(flag_hex, 16).toString(2);
      let flag_bin = "0".repeat(32 - flag_bin_unpadded.length) + flag_bin_unpadded;
      // empty flags are the reserved fields in the CPUID feature bit list
      // as found on wikipedia:
      // https://en.wikipedia.org/wiki/CPUID
      let all_flags = [
        "fpu", "vme", "de", "pse", "tsc", "msr", "pae", "mce", "cx8", "apic",
        "", "sep", "mtrr", "pge", "mca", "cmov", "pat", "pse-36", "psn", "clfsh",
        "", "ds", "acpi", "mmx", "fxsr", "sse", "sse2", "ss", "htt", "tm", "ia64", "pbe"
      ]
      for (let f = 0; f < all_flags.length; f++) {
        if (flag_bin[f] === "1" && all_flags[f] !== "") {
          result += " " + all_flags[f];
        }
      }
      result = result.trim();
    }
    if (callback) { callback(result) }
    resolve(result);
  });
}

I tried to write the code in a similar way as the surrounding code,
but it can probably be improved/simplified.
Not sure if these are all the flags, but it should give you a basic idea how to get this done :wink:.

Hi,
Thats GREAT! This is definitely the right direction! I will review it tomorrow and will publish a new version as soon as possible! Thank you so much for your contribution!

Thank you once again! I added this partial flags support for windows in version 3.31.0. The Windows registry seems only to support the EDX CPUID feature bits. For sure, there are more flags (ECX) which is currently not supported and therefore not reflected in the cpuFlags() windows version


Make makehtml.py work with bitRange and on Windows

This makes makehtml.py work with bitRange as well as bitOffset/bitWidth - bitRange is used in some non ST svd files.
Additionally, on Windows multiprocessing.Pool behaves differently - it spawns a new process for each worker, meaning that globals (args) aren't available, using a ThreadPool instead mitigates this and is more suitable as this should be IO bound, not CPU bound.

Additionally, on Windows multiprocessing.Pool behaves differently - it spawns a new process for each worker, meaning that globals (args) aren't available, using a ThreadPool instead mitigates this and is more suitable as this should be IO bound, not CPU bound.

I just tried this and get 21.78 seconds runtime with 102% CPU (i.e. single core), whereas master gets 2.82 seconds with 548% CPU, so I'm pretty sure it is actually CPU bound and this ends up being a big regression. On Linux multiprocessing.Pool also spawns a new process for each worker (that's the point) but globals should still be available. How is this usually dealt with on Windows?
No problem to the bitRange change but I don't want to merge the multiprocessing bit as-is.

Understood - didn't realise that this regressed on Linux. You are correct on Linux it's a new process but the process is forked so it retains the address space of the parent, on Windows this is an entirely separate process.
I'll have to look at this to see if there's some other way of making this work on Windows, though the 'easy' solution might be to use ThreadPool on Windows and Pool on Linux.

I'll have to look at this to see if there's some other way of making this work on Windows, though the 'easy' solution might be to use ThreadPool on Windows and Pool on Linux.

That would be fine with me.

Sorry for taking so long, finally got back to fix this!


Handle ByteBufHttpData in Http1ObjectEncoder.

HTTP1 splits the response when it goes over the size of a TLS frame, so it needs to be aware of pooled objects to not memory leak.

I think this is more complicated than I wished. How about this:

Add a variant of StreamMessage.subscribe() which allows the caller to choose to get the pooled object. e.g. message.subscribe(subscriber, /* poolingAware */ true). When poolingAware is false (default), DefaultStreamMessage could make a copy of it and release the ByteBuf-backed one.
Ensure DefaultStreamMessage releases all unconsumed ReferenceCounteds automatically when it's closed or aborted.

Thanks for the idea! Yeah, you're right the only way for the buffer to be mishandled is after a subscription, so this sounds like it'll work and much more transparently. Let me play with this.

If the other decorator guarding PR makes sense, I'll revert the large commit in this one so it only fixes the HTTPS/1 issue

Yes, please. Thanks!

Reverted down to just the HTTP1 fix

Thanks, @anuraaga


[APP-3434] MRR Deletes: Add API protos

https://viam.atlassian.net/browse/APP-3434
Let me know if I missed anything! This is my first API/proto change

How do we run the final Check Merge Requirements check?

Would Michael un-approving then re-approving the PR not retrigger the compilation?

Emily: you made proto changes after protos were compiled though
Michael: I can re-approve to regenerate the code after the proto is updated.

Would Michael un-approving then re-approving the PR fix this?

it won't if the proto-compiled label is still there. i also recommend clearing the generated files from your PR so there aren't merge conflicts when it tries to regenerate them

Got it, so

I will remove that label
I will remove all the generated files and push again
Emily or Michael will approve again

Ok... that was a non-traditional way of doing the above, but I have removed the tag and the generated files

LGTM!

Omg how do we addressing resolving conflicts in generated files?

Omg how do we addressing resolving conflicts in generated files?

you'll have to rebase main and start the process over again :( sorry it's so finicky and complicated!

Ok the commit history doesn't say it, but I merged in changes from main during my force commit

Cool gonna merge. Thanks for troubleshooting this with me!


Wrote some authoring, and renamed PPP.

Pull Request Test Coverage Report for Build 860

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 81.951%

Totals

Change from base Build 858:
0.0%

Covered Lines:
3142

Relevant Lines:
3834

💛  - Coveralls


Questions on 300VW dataset.

Sorry to disturb,  but I have a problem with 300VW dataset.
The offical web said that it contains 300 videos. But the file I download from the web only contains 114 video clips.
What's the problem, thanks!

Hi,
this is the total number of videos (the workshop was named 300videos in the wild as a follow up to earlier work, the site does not mention there are 300 videos). Feel free to read the actual paper paper: https://ibug.doc.ic.ac.uk/media/uploads/documents/shen_the_first_facial_iccv_2015_paper.pdf


test: refactor oci/helm tests for registryproviders

This refactors the existing helm-sync/oci-sync tests which push artifacts to use the new RegistryProvider interface. This delivers a few benefits:

minimize duplicate test scaffolding
enable running these tests without GKE/AR
enable running these tests with the presubmits in kind

/assign @karlkfi

/cc @nan-yu

/retest

/retest


Investigate nlohmann_json integration

When linking the target in config/CMakeLists.txt using
target_link_libraries(ConfigLib     PRIVATE      nlohmann_json::nlohmann_json     )
the library is not correctly included:
fatal error: 'nlohmann/json.hpp' file not found [build] #include <nlohmann/json.hpp>
It seems to be an access modifier issue as using PUBLIC solves the problem. But exactly why?

From https://cmake.org/cmake/help/latest/manual/cmake-buildsystem.7.html#id25 :

Generally, a dependency should be specified in a use of target_link_libraries() with the PRIVATE keyword if it is used by only the implementation of a library, and not in the header files. If a dependency is additionally used in the header files of a library (e.g. for class inheritance), then it should be specified as a PUBLIC dependency. A dependency which is not used by the implementation of a library, but only by its headers should be specified as an INTERFACE dependency.

@LLagu I'll close this issue, since we already have it integrated.


add minimum games filter

Add a filter to the UI to only show players who have played at least n games.

Fixed in #14


The plugin does not work from today?

Been using this plugin for a year. Starting from today, the plugin stops working & displays the following error:

Uncaught TypeError: Cannot read property 'results' of undefined

This is also happening on the demo.

facing this issue since last week.
please help..

Hello
exactly the same problem for me like anyone else I guess...
Is this possible to fix this ? Thanks

Also facing the same issue from last week

I'm also facing this issue.

I solved like this  (line 26)
Original
weatherUrl += 'select * from weather.forecast where woeid in (select woeid from geo.placefinder where text="'+options.location+'" and gflags="R" limit 1) and u="'+options.unit+'"';
Changed
weatherUrl += 'select * from weather.forecast where woeid in (select woeid from geo.places(1) where text="'+options.location+'") and u="'+options.unit+'"';

@mehmetafsinyuce It worked for me too. Thank you.

Same, it worked for me too. Thank you.

@mehmetafsinyuce
It works! Very nice!

See #174 for more info.


Update importer examples in README.md

At some point the importer changed from being a ReadableStream to a pull-stream.  The examples in the README.md were never changed so they need updating.
Copying some code from the importer tests into the README should be enough.

Fixed in 27565ff


#722 moving domain should be disallowed if overlapping another domain

Oops, this was a bad branch name. I accidentally copied and pasted the incorrect issue. But it is issue #722 being fixed in this PR, not issue #725.


[bug] Name/Location restrictions differ from Home Assistant

Version
Checklist:

[X] I am not using HomeAssistant. Or: a developer has told me to come here.
[X] I have checked the troubleshooting section and my problem is not described there.
[X] I have read the changelog problem was not mentioned there.

Build/Run method

[X] Docker
[ ] PKG
[ ] Snap package
[ ] Manually built (git clone - npm install - npm run build )

zwavejs2mqtt version: 0.13.0
zwave-js version: 7.2.4 / 1.5.0
Describe the bug
The Name and Area fields in zwavejs2mqtt don't allow apostrophes (single quotes). That puts it at odds with the other components in Home Assistant which all do and makes things inconsistent (e.g., I have to leave the fields blank in zwavejs2mqtt which will make it difficult to know what is what as I continue to add more devices).
I opened an issue at HomeAssistant Core and was redirected here.
To Reproduce
Steps to reproduce the behavior:

Edit a name or location on a Zwave node in the ZwaveJS2MQTT console. Attempt to add an apostrophe.

Expected behavior
The field should accept the same types of characters as are otherwise allowed in Home Assistant. For example, I have an area called "Jacob's Room" defined in Home Assistant that I cannot assign in the ZwaveJS2MQTT console because of this restriction.

Nammes are also used on mqtt topics, some chars are not allowed

I've noticed that Home Assistant replaces problem characters with "_" in instances where there are limitations (so for the purposes of MQTT, "Jacob's Room" could be "jacob_s_room"), while keeping the display name intact.

IMO this isn't a bug. As @robertsLando says there are different requirements for mqtt. It would be a lot of unnecessary work to make the topic replace disallowed chars with an _ and then track and somehow display the original name. And it invites confusion among users because they'd need to use the sanitized topic name when interacting with mqtt regardless of what is displayed.

Seems like it's serving a dual purpose today (e.g., setting the parameters there will be copied over to HA upon a server restart - unless maybe it's already defined in HA? I haven't figured out exactly how that works yet) when it should be more explicitly defined.
If MQTT is the target, then it seems like maybe the Location could be eliminated in zwavejs2mqtt and just the name kept while specifying that it's the MQTT label. Let HA handle its areas and labels without trying to coordinate since the requirements are not compatible.

Many people use the location in mqtt. You can sort the topics by location, then device.

Okay. I'll close this since it doesn't seem like something that can be easily improved. I do think the Location integration with HA invites the kind of confusion you're trying to avoid in the topic naming.
Because it's a predefined list in HA, what happens if I assign a non-existing name for the location to approximate one that does exist? Like "Jacobs Room" instead of the already defined "Jacob's Room"? Does it create a second area that I then need to go in and clean up? If it does, can the unwanted name be deleted in HA or will the integration try to recreate it because it exists in zwavejs2mqtt and I want to use the punctuationally incorrect version for MQTT?
Trying to coerce the data in fields for one use-case as input for an incompatible use-case seems messy. But maybe that's a problem for another team to tackle (the HA integration folks). Thanks for your review and feedback!

It'll create a second area. If you change any of this after the fact in HA it doesn't clobber anything. What was in zjs2mqtt stays there and HA keeps the new name/location.


why DateTime compare fails?

it without problem before I using timezones to parse tz info.
....
12:28 PM EST November 16, 2017==2017-11-16T12:28:00-05:00!=0001-00-00T00:00:00+00:00
12-Mar-19==0019-03-12T00:00:00Z!=0001-00-00T00:00:00+00:00
13-Nov-19==0019-11-13T00:00:00Z!=0001-00-00T00:00:00+00:00
1-Jul-15==0015-07-01T00:00:00Z!=0001-00-00T00:00:00+00:00
 March 23, 2019==2019-03-23T00:00:00Z!=0001-00-00T00:00:00+00:00
.....
[FAILED] all
Error: execution of an external program failed: '/Users/bung/nim_works/datetime_parse/tests/test1 '
     Error: Execution failed with exit code 256
        ... Command: "/Users/bung/.nimble/bin/nim" c --noNimblePath -d:NimblePkgVersion=0.1.0 --path:"/Users/bung/.nimble/pkgs/timezones-0.5.1" "-r" "--path:."  "/Users/bung/nim_works/datetime_parse/tests/test1"

https://github.com/bung87/datetime_parse/blob/master/tests/test1.nim

I took a look and it seems like you have test failures that are unrelated to this package. For example, Wednesday, August 21st 2019 fails but Wednesday, August 21 2019 does not and I doubt it's caused by the timezone package. Please post a more reduced test case if you've found a bug in the timezone package.

Wednesday, August 21 2019 that's make even weird,  this date time format not in my pattern so it will not parse correctly

What I mean is that Wednesday, August 21st 2019 (which is included in corpus.txt) fails to parse, but Wednesday, August 21 2019 (which is not included in corpus.txt) does not fail to parse.

I dont have idea now , print outputs as expected , but the check not work as expected.

I fixed pattern to pass Wednesday, August 21st 2019, found there is cst6cdt and no cst makes one pattern not pass.


Entity.getId and Entity.setEntityId don't match

They both modify the same field (entityId), but have differing names for it. Both should probably be just *Id (and the field too) for simplicity.

Yeah, we had better note that this is the network id as well.


fix: show global config in info command

fixes #1013

some findings:

https://pixi.sh/latest/reference/pixi_configuration/
in docs we said that
To find the locations where pixi looks for configuration files, run pixi with -v or --verbose.
Running cargo run -- info --verbose I can't find any logs about where it looks for global configuration.

Running with -vvv I have following output:

DEBUG pixi_config: Loading config from /etc/pixi/config.toml
DEBUG pixi_config: Failed to load system config: /etc/pixi/config.toml (error: failed to read config from '/etc/pixi/config.toml')
DEBUG pixi_config: Loading config from /Users/graf/.config/pixi/config.toml
DEBUG pixi_config: Failed to load global config: /Users/graf/.config/pixi/config.toml (error: failed to read config from '/Users/graf/.config/pixi/config.toml')
DEBUG pixi_config: Loading config from /Users/graf/Library/Application Support/pixi/config.toml
DEBUG pixi_config: Failed to load global config: /Users/graf/Library/Application Support/pixi/config.toml (error: failed to read config from '/Users/graf/Library/Application Support/pixi/config.toml')
DEBUG pixi_config: Loading config from /Users/graf/.pixi/config.toml
DEBUG pixi_config: Failed to load global config: /Users/graf/.pixi/config.toml (error: failed to read config from '/Users/graf/.pixi/config.toml')
DEBUG pixi_config: Loading config from /Users/graf/projects/oss/pixi-3/.pixi/config.toml
DEBUG pixi_config: Failed to load local config: /Users/graf/projects/oss/pixi-3/.pixi/config.toml (error: failed to read config from '/Users/graf/projects/oss/pixi-3/.pixi

maybe we can change the debug message and just tell that config.toml is totally missing?

If I don't have a config location,
instead of this:
Config locations:

maybe we can output something like this:
Config locations: no config file has been found
Other than that, it looks very good!


Refactor and cleanup duplicated code for defining screens menus

Consolidated code for refresh, refresh selector and reset default filters
Removed duplicated close menu from screens
Added unit test for new menu builders

@cristianonicolai Nice cleanup! Can we add some tests to this with a reasonable amount of effort? The whole project seems very low on test coverage. Also, are you sure you want this change on 6.4.x as well (bit of a risk but definitely fine with me).

@csadilek sure, tests are always helpful :) I added one unit test class per builder class that I created for this change. As for 6.4.x I think still worthwhile to merge as we can potentially reduce any effort on maintenance for this branch.

+1, thanks @cristianonicolai.

master: a43a9bb5f6edeae0ec4333979de24c99a38dd953


CreateStrokeFromInkPoints

Hi, the docs say there is a method to create strokes from inkpoints which accepts date time as parameters, however no method seems to exist, is that expected or when will this be available?
https://github.com/MicrosoftDocs/winrt-api/blob/docs/windows.ui.input.inking/inkstrokebuilder_createstrokefrominkpoints_2009821049.md

Hi @Ponant -- Looks like these parameters get pulled in via rendering on the live site doc page, here: https://docs.microsoft.com/en-us/uwp/api/windows.ui.input.inking.inkstrokebuilder#Windows_UI_Input_Inking_InkStrokeBuilder_CreateStrokeFromInkPoints_Windows_Foundation_Collections_IIterable_Windows_UI_Input_Inking_InkPoint__Windows_Foundation_Numerics_Matrix3x2_Windows_Foundation_IReference_Windows_Foundation_DateTime__Windows_Foundation_IReference_Windows_Foundation_TimeSpan__
Please let me know if this addresses your issue and I can close this out.
Thanks! ~Matt

Hi @mattwojo ,
I have only one method publicCreateStrokeFromInkPoints, and it does not allow  for those 2 parameters IReference<DateTime> strokeStartedTime, IReference<TimeSpan> strokeDuration. Indeed, the docs say it is a "pre-release", but I cannot access this extension from a uwp app. So, does this method exist, and if so where can I find it  (Nuget?)?.
Thanks

Hey @Ponant -- Just confirmed that you need to sign up to be a Windows insider and then download the Preview SDK from developer section on Windows Insider in order to get access to this API.
The details of the last flight of the OS/SDK are described in this blog: https://blogs.windows.com/buildingapps/2017/03/14/windows-10-sdk-preview-build-15052-released/#wch3XeeZHLvxmGqM.97.
(I don't believe there's currently a way to get this via Nuget)
If you're not already an Insider, but want to take a crack at it, let me know if you get stuck.
~Matt

@mattwojo , thanks a lot! I think I am already an insider but will check this out again and will let you know if any problems appear. Cheers

@mattwojo , thanks for your help, your insight should be enough. Cheers


Issues upgrading to react-native 0.49.3

Hey I've been trying to get some more insight into whats going on. All i can gather so far is that it seems to be this module, which is showing up some issues with the projects configuration when building for debug with react-native 0.49.3.
When i include this module and try to run:
react-native run-android
The app errors and displays multiple warnings of the following for multiple other packages:
Loading dependency graph...jest-haste-map: @providesModule naming collision:
  Duplicate module name: TestJavaToJSReturnValuesModule
  Paths: /projectdir/node_modules/react-native-keyboard-aware-scroll-view/node_modules/react-native/ReactAndroid/src/androidTest/js/TestJavaToJSReturnValuesModule.js collides with /projectdir/node_modules/react-native-iphone-x-helper/node_modules/react-native/ReactAndroid/src/androidTest/js/TestJavaToJSReturnValuesModule.js

And ends up in throwing an error:
Bundling `index.js`  [development, non-minified]  0.0% (0/1), failed.
error: bundling failed: ambiguous resolution: module `/projectdir/index.js` tries to require `react-native`, but there are several files providing this module. You can delete or fix them:

  * `/projectdir/node_modules/react-native-iphone-x-helper/node_modules/react-native/package.json`
  * `/projectdir/node_modules/react-native-keyboard-aware-scroll-view/node_modules/react-native/package.json`
  * `/projectdir/node_modules/react-native/package.json`

If you have any ideas what could be causing this / if it is something that is being caused by this package please could you let me know.
Thank you for the work on the package it was working well when i was on .47 r-n.
EDIT
Here is the package.json i'm working with.
{
	"name": "projectdir",
	"version": "0.0.1",
	"private": true,
	"scripts": {
		"start": "node node_modules/react-native/local-cli/cli.js start",
		"test": "jest"
	},
	"dependencies": {
		"moment": "^2.18.1",
		"prop-types": "15.6.0",
		"react": "16.0.0-beta.5",
		"react-native": "0.49.3",
		"react-native-ble-manager": "6.2.4",
		"react-native-camera": "^0.10.0",
		"react-native-fcm": "^10.0.3",
		"react-native-keyboard-aware-scroll-view": "0.4.0",
		"react-native-linear-gradient": "^2.3.0",
		"react-native-maps": "0.17.1",
		"react-native-splash-screen": "^3.0.1",
		"react-native-swipe-list-view": "0.4.5",
		"react-native-vector-icons": "4.4.2",
		"react-navigation": "^1.0.0-beta.15",
		"react-redux": "5.0.6",
		"reconnecting-websocket": "^3.2.1",
		"redux": "3.7.2",
		"redux-action-buffer": "^1.1.0",
		"redux-persist": "4.10.1",
		"redux-saga": "0.16.0",
		"sha.js": "2.4.9",
		"styled-components": "2.2.1"
	},
	"devDependencies": {
		"babel-jest": "21.2.0",
		"babel-preset-react-native": "4.0.0",
		"jest": "21.2.1",
		"react-test-renderer": "16.0.0-beta.5"
	},
	"jest": {
		"preset": "react-native"
	}
}

Hi!
Sorry, I accidentally put react-native as a dependency in 0.4.0. Please update to 0.4.1.
Thanks!

Awesome it's all working now thanks!

i have this error any idea?


Short solution needed: "Get year and month from date" (clickhouse)

Please help us write most modern and shortest code solution for this issue:
Get year and month from date (technology: clickhouse)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments to make solution explained.
Link to this issue in comments of pull request.

Get year and month from date


doc: EVP_DigestInit clears all flags

Mention that EVP_DigestInit() also clears all flags.
Fixes: #10031
Signed-off-by: Christian Heimes<EMAIL_ADDRESS>Checklist

[X] documentation is added or updated

Merged to master and 1.1.1.  Thanks for the fix.


[notabug] Hello!

Just chiming in to say thank you for this nifty contribution! :v:

Thank you!
Star it if you like it :-)

Star is a star, but a record in the log makes its own sense :) (I've starred it anyway :) )

I hope the starter will serve you well. Thanks again!


Implement cloud status api

Coverage decreased (-18.5%) to 60.417% when pulling e92b61f7af1409ca65cd473fc956c027b5747b34 on implement_cloud_status_api into 2be38fa88f1c43ba7b3d60fe57b3584b7bb3eb7d on master.

Coverage decreased (-18.5%) to 60.417% when pulling 195f691646767360ff87163faa09e0c37b10ec9a on implement_cloud_status_api into 2be38fa88f1c43ba7b3d60fe57b3584b7bb3eb7d on master.

Coverage decreased (-18.5%) to 60.417% when pulling 195f691646767360ff87163faa09e0c37b10ec9a on implement_cloud_status_api into 2be38fa88f1c43ba7b3d60fe57b3584b7bb3eb7d on master.


Improve validation when adding local users that are already part of an Admin Group of this project/quiz

If a given user is already part of an admin group which is assigned to a project/quize, then do not allow to add that user as a local admin/approver in the Access page of a project/quiz; display an error message, something like "User is already part of an Admin Group and cannot be added as a local admin"; validate on UI and backend

PR: #2977

:+1:


Will blazor version support more charts?

I saw the react antdesign version have plenty of charts like flow charts, map charts, liquid charts. etc. So will all these be done into blazor version? Or is there a guide on how to make a contribute to convert these charts?  thanks!

Hello, thank you for your interest. We want more developers to contribute to these projects. Would you be interested in helping us to contribute this project?

Hello, thank you for your interest. We want more developers to contribute to these projects. Would you be interested in helping us to contribute this project?

would love to. is there a wiki or guide ?

Sorry, not at the moment. But please follow Ant Design Blazor's contribution guidelines.
https://github.com/ant-design-blazor/ant-design-blazor/blob/master/CONTRIBUTING.md


Java indexer: emit actual nodes for JDK classes [phab:D291]

Created by schroederc at 2015-05-27 20:57:54:

zarko commented inline (context lost during migration) at 2015-05-28 17:48:18:
Why isn't the corpus set to "jdk" for java.lang.String in the test below? Is it intended to be set? If not, what's this code doing?

schroederc wrote at 2015-05-28 18:00:33:
Add test for "jdk" corpus.

schroederc commented inline (context lost during migration) at 2015-05-28 18:01:13:
The name node isn't changed.  They shouldn't have a corpus set.  The actual record node does, though.

zarko accepted this revision at 2015-05-28 18:02:06

schroederc closed via commit 3fc9a495f68fbf77374e15c4c1b98431cf8b41e6 at 2015-05-28 18:35:32


Migrate eth-json-rpc-provider into core monorepo (2/2)

Explanation

This is the second of two PRs that will migrate eth-json-rpc-provider into the core monorepo by following these steps: https://github.com/MetaMask/core/issues/1079#issuecomment-1700126302.

The first PR https://github.com/MetaMask/core/pull/1737 focused on migrating the git history of the original package.
This PR integrates eth-json-rpc-provider into the packages/ directory.

Resolves issues arising from applying core monorepo linter/compiler/build settings to the newly added package.
Resolves issues in downstream packages from upgrading to current version (2.1.0) of eth-json-rpc-provider.

Checklist

[x] eth-json-rpc-provider all tests passing.
[ ] Resolve network-controller type errors from eth-json-rpc-provider bump to 2.1.0.
[ ] Add eth-json-rpc-provider to README dependency graph.
[ ] Archive original repo.

Blocked by

[ ] @metamask/utils bump:

[ ] https://github.com/MetaMask/core/pull/1639
[ ] https://github.com/MetaMask/core/pull/1653

References

Closes https://github.com/MetaMask/core/issues/1551
See https://github.com/MetaMask/eth-json-rpc-provider/pull/28 for preparatory steps taken in original repo.

Changelog

@metamask/eth-json-rpc-provider

ADDED: Migrated into core monorepo.
DEPRECATED: Archived original repo - https://github.com/MetaMask/eth-json-rpc-provider

Checklist

[ ] I've updated the test suite for new or updated code as appropriate
[ ] I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
[ ] I've highlighted breaking changes using the "BREAKING" category above as appropriate

@SocketSecurity ignore-all
All packages versions are current with main branch.

Marking this as "request changes" because of the change to network-controller (see comment there).

Are there still any changes to the PR you'd like to see, or maybe a different approach we could try for the network-controller fixes?


Remove govuk_template.html copied in build task from the repository

As described in #210:
The file lib/govuk_template.html is copied from govuk_modules/govuk_template_jinja/views/layouts/govuk_template.html as part of the grunt sync task. This file feels like it shouldn't be committed in the repository as it ultimately comes from govuk_template_jinja
As an aside this was noticed when the govuk_template_jinja dependency was bumped to v0.17.3 this meant that after an npm start, lib/govuk_template.html was showing git file changes.
This PR:

removes lib/govuk_template.html from the repository
adds lib/govuk_template.html to .gitignore

This fixes #210.

Merging this as it does fix the issue, however I'm thinking maybe we should copy the template to govuk_modules, conceptually I think it fits better there instead of lib


Auto flush

@NiteshKant PTAL... this is based on your work but use the correct context etc. I added tests for it.

I took a first pass. I'm assuming this will take precedence over https://github.com/netty/netty/pull/5395#issuecomment-227762708 ... so I only looked at this PR.

@Scottmitch yes this PR contains changes from #5395 so it will override that PR.

@normanmaurer this looks good to me apart from the comment about the fail-fast for wrong Eventloop, I would like to hear what you think about it.

@normanmaurer you want to address @Scottmitch comments or I should?

Fun has moved back to the original PR #5395 😃
The original PR now includes Norman's changes and I have addressed open comments in this PR.
@normanmaurer @Scottmitch PTAL.
Apologies for the back-and-forth between PRs.

Closing this now and move back to original PR.


100% CPU after wakeup from suspend, Linux

lsof:
electron 25432 ninja   21u  unix 0xffff8803ba988700      0t0 1431093 type=STREAM -><EMAIL_ADDRESS>electron 25432 ninja   27u  unix 0xffff88030f8fe580      0t0 1428081 type=STREAM -><EMAIL_ADDRESS>
strace(repeats forever):
poll([{fd=21, events=POLLIN}, {fd=22, events=POLLIN}, {fd=27, events=POLLIN}, {fd=28, events=POLLIN}], 4,<PHONE_NUMBER>) = 1 ([{fd=28, revents=POLLIN}])
recvmsg(27, 0x7ffd90aa93c0, 0)          = -1 EAGAIN (Resource temporarily unavailable)
recvmsg(21, 0x7ffd90aa9380, 0)          = -1 EAGAIN (Resource temporarily unavailable)
recvmsg(27, 0x7ffd90aa9390, 0)          = -1 EAGAIN (Resource temporarily unavailable)

This will happen for every electron app in my tests after waking up from suspend
OS: Arch Linux kernel 4.2.5-1-ARCH
Electron: latest prebuilt from npm repository

Regarding "latest prebuilt from npm repository", is it 0.36.0? And for "every electron app", did you mean apps like VS Code and Atom?

@zcbenz yes the version I tested with was 0.36.0. The apps that were independently causing this problems were Rocket.Chat Desktop and Matterfront (slack-like chat apps). Atom didn't seem to have this problem though.

/cc @paulcbetts Did you ever see similar issues in Slack for Linux?

We've never heard of this afaik

Since Atom doesn't have the problem, and I have to assume it is the app's code that caused the 100% CPU usage. I suggest reporting this to each app's repo instead.


OAuthCache and OIDCAudienceCache use the same cache name.

Both [1] and [2] use the cache name 'OAuthCache'
This makes both cache entry types getting stored into the same backed cache.
This would cause more cache misses and prevents configuring timeouts for each cache using 'IdentityApplicationManagementCacheManager' configurations block.
[1] - https://github.com/wso2-extensions/identity-inbound-auth-oauth/blob/master/components/org.wso2.carbon.identity.oauth/src/main/java/org/wso2/carbon/identity/oauth/cache/OAuthCache.java#L28
[2] - https://github.com/wso2-extensions/identity-inbound-auth-oauth/blob/master/components/org.wso2.carbon.identity.oauth/src/main/java/org/wso2/carbon/identity/oauth/cache/OIDCAudienceCache.java#L25

Thank you for your contribution!
We are closing this issue since it has not been prioritized for a long time. Chances are that it has already been solved in more recent versions. If not, we will be re-evaluating this when it becomes a priority.


Review Final applicants in Outreachy spring 2022

This task is related to #1 to provide final feedback for all applicants who translated a significant part or made a PR. Anyone who submits a PR until the 22nd of April/or submitted a final contribution in Outreachy will be added to the list.

Name
Submitted Final Contribution
Rename Files
Translation in Crowdin
Time Commitment
PR

@AsmaKacem1

✓

@Boutainaelatbaoui

✓

@nancyalaswad90
✓

✓

@ThanaMeri

✓

@amj1337
✓

✓

@Muluh
✓

@AsmaKacem1 @ThanaMeri @Boutainaelatbaoui You submitted a partial application in Outreachy. Can you submit the final application through this link, please?

Hello @BatoolMM  could you help me with this?

I completed mine. it was a pleasure working with you @BatoolMM

Thank you to everyone who submitted their final application 🎉 🎉 . Can I ask everyone one more thing which is to create a simple PR and rename the file you created for docs as follow NAME_review.md e.g. (Asma_review.md, Thana_review.md, nancy_review.md, ..etc). We just want to rename the file you called previously docs.md with your name then underscore review. Can you possibly do that?


Flutter web in profile and realease mode, considere specific variable always null

Steps to reproduce
It has a large flutter system that has worked normally for years, I recently updated flutter to version 3.19.5 and since then I have had several problems only in the web version in profile and release mode, which do not happen anywhere else.
Everyone was able to adapt, but this last one caught my attention, apparently Flutter, when converting the code to js, is assuming that a variable is always null, below are some images:

Below is the generated javascript code:
copyWith$9$cavalo$conditions$id$name$optionalSelected$optionalSelectedDefaultId$optionals$optionalsIds$visualizationIsConditioned(cavalo, conditions, id, $name, optionalSelected, optionalSelectedDefaultId, optionals, optionalsIds, visualizationIsConditioned) {
    var t1, t2, t3, t4, t5, t6, t7, t8, t9, _this = this;
    A.print("copyWith(): this.nameForUser: " + A.S(_this.nameForUser));
    A.print("copyWith(): cavalo: " + A.S(cavalo));
    A.print("copyWith(): nameForUser.runtimeType: " + A.S(cavalo) + " = " + B.JSNull_methods.get$runtimeType(cavalo).toString$0(0) + " = true");
    A.print("copyWith(): nameForUser: " + A.S(cavalo));
    A.print("copyWith(): this.name: " + A.S(_this.name));
    A.print("copyWith(): name: " + A.S($name));
    A.print("copyWith(): name.runtimeType: " + J.get$runtimeType$($name).toString$0(0));
    A.print("*****");
    t1 = id == null ? _this.id : id;
    t2 = _this.nameForUser;
    if (optionalsIds == null) {
      t3 = _this.optionalsIds;
      if (t3 == null)
        t3 = [];
      t3 = A.List_List$from(t3, true, type$.String);
    } else
      t3 = optionalsIds;
    t4 = $name == null ? _this.name : $name;
    if (optionals == null) {
      t5 = _this.optionals;
      if (t5 == null)
        t5 = [];
      t5 = A.List_List$from(t5, true, type$.ProductAcessorie);
    } else
      t5 = optionals;
    if (optionalSelected == null) {
      t6 = _this.optionalSelected;
      t6 = t6 == null ? null : t6.copyWith$0();
    } else
      t6 = optionalSelected;
    t7 = optionalSelectedDefaultId == null ? _this.optionalSelectedDefaultId : optionalSelectedDefaultId;
    t8 = visualizationIsConditioned == null ? _this.visualizationIsConditioned : visualizationIsConditioned;
    t9 = _this.conditionIsValid;
    return A.ProductAcessoriesGroup$(t9, conditions == null ? _this.ProductOptionalSelectorGroupBase_conditions : conditions, t1, t4, t2, t6, t7, t5, t3, t8);
  },

    copyWith$1$id(id) {
      var _null = null;
      return this.copyWith$9$cavalo$conditions$id$name$optionalSelected$optionalSelectedDefaultId$optionals$optionalsIds$visualizationIsConditioned(_null, _null, id, _null, _null, _null, _null, _null, _null);
    },
    copyWith$1$cavalo(cavalo) {
      var _null = null;
      return this.copyWith$9$cavalo$conditions$id$name$optionalSelected$optionalSelectedDefaultId$optionals$optionalsIds$visualizationIsConditioned(cavalo, _null, _null, _null, _null, _null, _null, _null, _null);
    },

In the lines  A.print("copyWith(): nameForUser.runtimeType: " + A.S(cavalo) + " = " + B.JSNull_methods.get$runtimeType(cavalo).toString$0(0) + " = true"); when print the value calls the function "A.S()" but when print the type calls the function "B.JSNull_methods"
On the line t1 = id == null ? _this.id : id; it checks if the id is different from null, already on the line t2 = _this.nameForUser; it disregards the parameter passed in the function
I've already tried changing the name and position of the variable but nothing works.
I found two similar problems:
https://github.com/dart-lang/sdk/issues/44692
https://github.com/dart-lang/sdk/issues/44692
Expected results
That the code does not automatically consider the variable null
Actual results
When converting to javascript, dart is assuming the variable as always null
Flutter Doctor output
[√] Flutter (Channel stable, 3.19.5, on Microsoft Windows [versÆo 10.0.22631.3447], locale pt-BR)
[√] Windows Version (Installed version of Windows is version 10 or higher)
[√] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
[√] Chrome - develop for the web
[√] Visual Studio - develop Windows apps (Visual Studio Community 2019 16.11.32)
[!] Android Studio (version 2021.2)
X Unable to determine bundled Java version.
[√] Android Studio (version 2023.2)
[√] VS Code (version 1.88.1)
[√] Connected device (3 available)
[√] Network resources

Thanks for the report @apps-auth
Is there a way for you to provide us reproducible code that shows this behavior ?


Icon clipping in footer

The VS Code icon in the footer doesn't fully show.

@ddieruf could you please take a look ?

Hi @benfrank241, thank you for contacting us. Are you referring to the VS Code link on langstream.ai (I've attached a shot)? Was there a page on the site where this was getting clipped?

Yeah it’s right there in the image. On langstream.ai in the footer the VS code icon is only partially showing.
Sent from Yahoo Mail for iPhone
On Wednesday, October 11, 2023, 6:06 AM, David Dieruf @.***> wrote:
Hi @benfrank241, thank you for contacting us. Are you referring to the VS Code link on langstream.ai (I've attached a shot)? Was there a page on the site where this was getting clipped?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Ahh yes I see it now. Ok thx.

This is fixed now. Thanks for reporting it, @benfrank241


is there a way to create a rover DF from an ActiveRecord::Result ?

I have some data I pull from AR via a raw SQL statement (it's too complex to use AREL.
data = ActiveRecord::Base.connection.exec_query( ...very complex SQL here ...)
when I try to dataframe it
df = Rover::DataFrame.new(data)
I get the error
Cannot cast to data frame: ActiveRecord::Result
which was a bit surprising. But, the following works:
df = Rover::DataFrame.new(data.to_json)
(having, of course, nothing to do with json, haha)
question: is that as expected?   Could Rover possibly detect it's an AR Result and automatically convert it to an array of hashes?

Good idea, just added.


⚠ ClamAV has degraded performance

In e2ee3cb, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 456 ms

Resolved: ClamAV performance has improved in a5b5f5c after 537 days, 23 hours, 9 minutes.


Remove require hashes from pip install

What? and Why?

What does this pull request change/fix? Why was it necessary?

Some dependent libraries don't have their dependencies pinned with hashes so the --require-hashes switch on pip fails
Checklist

[ ] CHANGELOG.md updated? (if required)

Codecov Report

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

@@           Coverage Diff           @@
##           master     #101   +/-   ##
=======================================
  Coverage   87.25%   87.25%
=======================================
  Files          17       17
  Lines        1342     1342
=======================================
  Hits         1171     1171
  Misses        171      171

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 32711da...336be88. Read the comment docs.


CSS double-colon selector - spaces added

ST3 (3059), CodeFormatter 2.4.0
Double-colon selectors in CSS are getting spaces added.
.div_button::selection {
becomes
.div_button: :selection {

fixed as of v2.6.0


Partial builds with lineups

Lineups broke KOREBUILD_REPOSITORY_INCLUDE/EXCLUDE. We should fix that before we roll out lineups everywhere.
cc @ryanbrandenburg

Fixed with #547


MissingMethod: Marshal.ThrowExceptionForHR

I'm exploring CoreRT and trying to compile one of our applications using CoreRT.
When compiling using dotnet publish -r win-x64 -c release, I get the following warning:

EXEC : warning : Method [System.Runtime.Extensions]System.Environment.ExpandEnvironmentVariablesCore(string)
will always throw because: [TEMPORARY EXCEPTION MESSAGE] MissingMethod: Void System.Runtime.InteropServices.Marshal.ThrowExceptionForHR(Int32)

In certain scenarios, this translates into a runtime exception:
Unhandled Exception: System.MissingMethodException: Method not found: 'Void System.Runtime.InteropServices.Marshal.ThrowExceptionForHR(Int32)'.
   at Internal.Runtime.CompilerHelpers.ThrowHelpers.ThrowMissingMethodException(ExceptionStringID, String) + 0x8
   at System.Environment.ExpandEnvironmentVariablesCore(String) + 0x14

So it looks like System.Runtime.InteropServices.Marshal.ThrowExceptionForHR is missing.
I'm not sure where to look next to actually implement this method, so any guidance would be welcome.

This is because we're using System.Runtime.Extensions.dll that is designed to run on top of CoreCLR, not CoreRT. CoreFX repo only produces a Win32 version of System.Runtime.Extensions for CoreCLR, so that's what we have right now. We can probably make this work with the CoreCLR version by adding an implementation of ThrowExceptionForHR here. We could steal the implementation from here.
@luqunl is working on the unification right now so it's possible a fix for this will naturally fall out of what he's doing.


Update fs2-core, fs2-io, ... to 3.1.0

Updates

co.fs2:fs2-core
co.fs2:fs2-io
co.fs2:fs2-reactive-streams

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

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "co.fs2" } ]

labels: library-update, semver-minor

Superseded by #474.


Kore in k

I've restarted the kore-in-k PR.  I would like to get this merged, but it seems that the latest changes to K make tests fail for the definition.  @hreada could you please take a look at it?
Reviewer checklist

[ ] Test coverage: stack test --coverage
[ ] Public API documentation: stack haddock
[ ] Style conformance: stylish-haskell

@traiansf @hreada There were a couple of problems: one is that the regular expressions for MetaVariable isn't working. I'm not sure what the issue was. The other is that the tests had updated to the new syntax for alias. Simply uncommenting some code fixed that. I've cleaned up the history and force pushed for now.
We also need to get these tests integrated with the CI system. There's probably something in the stack.yaml that needs changing for that, but I'm not sure how.
It'd be awesome if you could rebase instead of merging when trying to bring in the latest code on master. The history becomes much cleaner  to read and debug (tools like git-bisect are great for finding bugs). Besides that, when you resolve a conflict in a merge commit, the conflict resolution gets hidden away in the merge commit and gets overlooked easily.


Materialized table with insert / update instead of replace?

Hi,
would it be possible to include a version of your materialized table (bigquery-materialize-table-web-vitals.sql) where you don't replace data but only update data or insert data that does not match the existing data in the table ?
I'm in the situation that Bigquery events_ table only keeps data for 30 days, so I loose data whenever I replace them in my materialized table, instead of getting a good long performance data history.
Thanks a lot in advance!

I don't have the time at the moment to upgrade the free solution.
My commercial solution for BigCommerce runs every day and replaces the last 3 days. The basic idea:

Determine the day to replace the data from. e.g. 3 days prior to the last date gathered
Delete all rows from that day
Insert new rows with the latest data from that day until now

Here is the SQL template I use to do that:
https://github.com/Tiggerito/GA4-Scripts/blob/e67b801b89298f4a82a1fc3aaab038802db0c6b8/bigquery-tag-rocket-report-data-incremental2.sql#L170
Things like the dataset and variables will need modifying. And the table it updates is probably different to the one for the article and Looker report.


Puppeteer firefox promise sometimes returns array, sometimes object

My code that I execute inside browser:
const getArticles = async (page) => {
    return await page.evaluate(() => {
        ...
        return Promise.resolve(formated.filter(x => x)) // formated is a normal array
    });
}

I have to call it like this:
        let newArticles = await getArticles(page)
        if(typeof newArticles === 'object') {
            newArticles = newArticles._v
        }

It seems very random. Sometimes it returns an array, sometimes object. With chrome, it was array every single time. Any ideas?

@jan0991 did it resolve itself?

I thought so, but nope. Still does this randomly.

@jan0991 so can you please share a small script that will reproduce this?

Thanks for the report. I'm closing this issue since the experimental puppeteer-firefox package is no longer maintained. A replacement will be available
later this year. You can learn more about Mozilla's work on this at https://wiki.mozilla.org/Remote
See also https://github.com/puppeteer/puppeteer/pull/5137


Support for unencrypted DNS

This PR adds support for unencrypted IP:Port based DNS. It could be used to connect to a running instance of PiHole or AdGuard Home.

Thanks a lot, I also need this for my AdGuard :D


Added API Endpoint for Contributor get CLA Manager List

Added v2 endpoint to allow contributors to fetch the list of CLA managers for a given company/cla group.

Signed-off-by: David Deal<EMAIL_ADDRESS>

@prasannamahajan @wanyaland - sorry for so many edits - moved models to a separate file to declutter. Grouped public and private functions together.


Lost Paypal Logo & Remove 'X'

I seem to have lost the Paypal logo and the 'x' to remove items from my mini cart, I have a cart on two pages and both are the same. One page is here http://www.leathersolutions.org/awards.php
The logo is reassuring to customers so any help to retrieve this and the x would be appreciated.

This looks like it's a PayPal issue. Older versions of minicart.js load images from one of their servers and it looks like the configuration for it may be temporarily messed up. I'll email someone over there to resolve it.
In the meantime, this can be fixed quite easily by upgrading to the current version of minicart.js found at https://raw.github.com/jeffharrell/MiniCart/master/minicart.js . In addition to quite a few fixes, the URL the assets are loaded from is different and working.
Let me know if you run into any issues, but it should be a direct swap.

Happy to help. I'm closing this out in the meantime. Feel free to reopen if you're not able to get it resolved.

If I upgrade to the current version of minicart.js the PayPal dropdown cart no longer works. I either get taken to paypal immediately, or no response at all. But tonsai's fix works great.

It's good to hear you got it working. It sounds like a JavaScript error may have been occurring for you with the new version. Did you have any customizations in the JS file and which browser were you using at the time? I'd like to verify it's not an issue for others.

I only got the error in Firefox, and I didn't touch the JS files. Good luck!
On Jan 27, 2012, at 2:49 PM, Jeff Harrell wrote:

It's good to hear you got it working. It sounds like a JavaScript error may have been occurring for you with the new version. Did you have any customizations in the JS file and which browser were you using at the time? I'd like to verify it's not an issue for others.

Reply to this email directly or view it on GitHub:
https://github.com/jeffharrell/MiniCart/issues/44#issuecomment-3694793

I now realize what you were asking. With your new version, I no longer get the dropdown cart in any browser. I just get a new window redirected to PayPal. No customizations.

The upgrade worked fine for me, just a straight swap as you suggested


Support for gzip compression

Would be nice to have gzip compression enabled by default for all responses in a container. This can be built using Jetty Gzip Handler which seems to do all the content negotiation as well actual (de)compression.

Looks like Jetty GzipHandler does not support "deflate" encoding. So this feature will be limited to Gzip.


index: new Promise.resolve > Promise.resolve

This was causing a type error "Promise.resolve" is not a contructor. This
pull request fixes that.

You are correct. Thanks for the PR.


[pcre2] add unofficial-pcre2 cmake config

[pcre2] add unoffical-pcre cmake config
./vcpkg x-add-version --all

[x] Changes comply with the maintainer guide
[x] SHA512s are updated for each updated download
[x] The "supports" clause reflects platforms that may be fixed by this new version
[x] Any fixed CI baseline entries are removed from that file.
[x] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

It's possible the alias I add in pcre2-config.in.cmake isn't actually a good idea, open to input there

usage:
pcre2 provides CMake targets:

    # this is heuristically generated, and may not be correct
    find_package(unofficial-pcre2 CONFIG REQUIRED)
    # note: 1 additional targets are not displayed.
    target_link_libraries(main PRIVATE unofficial::pcre2::pcre2 unofficial::pcre2::pcre2-8-shared unofficial::pcre2::pcre2-16-shared unofficial::pcre2::pcre2-32-shared)

When testing usage, the following error occurs:
>------ Build All started: Project: CMakeFindUsage, Configuration: x64-Debug ------
  MSBuild version 17.5.1+f6fdcf537 for .NET Framework

    Checking Build System
    Building Custom Rule C:/Users/monica/source/repos/CMakeFindUsage/CMakeFindUsage/CMakeLists.txt
    CMakeFindUsage.cpp
C:\Users\monica\source\repos\CMakeFindUsage\CMakeFindUsage\CMakeFindUsage.cpp(3,10): fatal error C1083: Cannot open include file: 'pcre2.h': No such file or directory

Build All failed.

CMakeFindUsage.cpp

#include <iostream>
#include "pcre2.h"
using namespace std;
int main()
{
cout << "Hello CMake." << endl;
return 0;
}

CMakeLists.txt

cmake_minimum_required (VERSION 3.8)
set(CMAKE_TOOLCHAIN_FILE "E:/pcre2/scripts/buildsystems/vcpkg.cmake")
project ("CMakeFindUsage")
find_package(unofficial-pcre2 CONFIG REQUIRED)
add_executable (main "CMakeFindUsage.cpp")
target_link_libraries(main PRIVATE unofficial::pcre2::pcre2 unofficial::pcre2::pcre2-8-shared unofficial::pcre2::pcre2-16-shared unofficial::pcre2::pcre2-32-shared)

pcre2 has official pkg-config files, also for windows. It is not necessary to use unofficial config. (Except for multiconfig generator cmake usage. vcpkg should really solve that instead of adding more lock-in configs.)