Adding 'apache' to docs

Coverage remained the same at 69.345% when pulling 4e8a18c9f2bc493b39fa7d81b208046c295d288f on mistercrunch:docs_apache into 25c599d0400cf320ce5accf50ed88e76ccff5980 on apache:master.

Coverage decreased (-0.02%) to 69.33% when pulling 6789361ca023f59b1a301e80e4dc8c2c2ba05d61 on mistercrunch:docs_apache into 25c599d0400cf320ce5accf50ed88e76ccff5980 on apache:master.


Enable use of the /safeGL launch command through the CataclysmEngineTweaks.ini ?

To clarify: I get massive texture glitches when running the game (whether modified or otherwise) in OpenGL mode on Windows 10 (on a Sapphire Pulse RX 6700 non-XT 10 GB GPU), but launching the game with /safeGL set fixes this problem.
The CataclysmEngineTweaks.ini does not seem to present a method of setting /safeGL, but interestingly enough, creating a shortcut to the cataclysm_loader.exe and then setting /safeGL in the Target field fixes it, just like it would for a shortcut to the original game exe.
Being able to set it in the CataclysmEngineTweaks.ini would be more intuitive, however.

Yes, I can add this. The command line arguments sent to the loader are forwarded to the game, but if this is needed on some systems it'd certainly be more convenient to have it in the .ini.
I'd look into the root cause of the graphical corruption but unfortunately neither of the PCs I own repro this.

To my understanding, newer AMD GPUs have trouble with older versions of OpenGL. The graphical corruption occurs in the unmodified Cataclysm when running in OpenGL mode, and it also occurs in Homeworld 1 in OpenGL mode (using /safeGL fixes texture corruption in HW1 as well).
My current GPU: AMD Sapphire Pulse RX 6700 non-XT 10 GB
AMD Adrenalin software version: 24.5.1
Let me know if additional hardware and/or software detail is needed to help reproduce the problem.

Added the shortcut in v0.12. In a future release, I may break out the individual toggles that the /safeGL command line modifies into separate options, so we can try to figure out what exactly is causing this bug. I do not have any AMD cards here so can't repro it myself.


[BUG] the ui is very broken for me for whatever reason

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.

What problem did you encounter?

this sort of bug continues for every section of the menu
this is my first time installing it and yes i have all the requirements such as .NET 6 Desktop Runtime

https://github.com/pizzaboxer/bloxstrap/issues/1035#issuecomment-1858774554
See what I said here. I can't do anything about this, be sure you don't have anything running that could potentially be causing this.

i dont have anything running apart from my antivirus in the background

Nothing that can potentially provide graphics overlays?

i dont think so

Well, I'm afraid that I can't really do much for you. All I can say is that there is definitely something running in the background that's causing this conflict. Sorry.


display all steps

I currently run krun with --depth=0, --depth=1, ..., --depth=N to see each step of the execution. This takes about a second each time, regardless of the depth, which indicates that most of the cost is in the startup time and not in the execution itself (at least for my small examples). In order to display step N, the execution must go through each of the previous steps, so I assume it would be moderately easy to print all N steps in a single execution? This would allow me to see all the steps in 1 second rather than N, which would be a huge quality-of-life improvement for me. Thanks!

Thanks! I'm currently using the Haskell backend, but I could switch to a different one if that's easier. So far, it doesn't look easy at all!
In particular, I see that the --debugger flag drops me in C's gdb debugger, not Haskell's ghci debugger, so I don't think I can step through the Haskell code from there. But that gave me an idea: I can use --dry-run to get the kore-exec command, and then run ghci on that command in order to step through the Haskell code interactively. Unfortunately, that didn't go too well:
$ stack ghci kore:lib kore:exe:kore-exec
λ> :break exec
Breakpoint 0 activated at .../kore/src/Kore/Exec.hs:(226,9)-(271,40)
λ> :main .../lambda-kompiled/definition.kore --module LAMBDA --pattern .../.krun-2021-05-11-21-17-03-wDoU3VTzhX/tmp.in.l8PtOjsP0i --depth 1
Stopped in Kore.Exec.exec, .../kore/src/Kore/Exec.hs:(226,9)-(271,40)
<interactive>: internal error: allocation of 1673264 bytes too large (GHC should have complained at compile-time)
    (GHC version 8.10.4 for x86_64_apple_darwin)
    Please report this as a GHC bug:  https://www.haskell.org/ghc/reportabug
Abort trap: 6

The next thing I plan to try is to tweak the LogicT computation to return the list of states it stepped through instead of just the final state.

I see that the --debugger flag drops me in C's gdb debugger, not Haskell's ghci debugger,

You are using the LLVM backend, not the Haskell backend. Although, using --debugger will not get you ghci, but rather our own debugger for Kore.
If you are using krun, you can pass the --search-all to see all the states, though they will all be printed at once and not incrementally.


Push iOS option is now deprecated by HA 2022.02.0

Alarmo Version
v1.8.3
HA Version
2022.02.0
Bug description
Using example for push configuration generates a log line:
2022-02-02 23:28:26 WARNING (MainThread) [homeassistant.components.ios] The 'puh' option near /config/configuration.yaml:16 is deprecated, please remove it from your configuration
Steps to reproduce

Upgrade to 2022.02.2
Enable actionable notifications

Relevant log output
No response

This is an error caused by your configuration.yaml file, not by Alarmo.
You will have to fix that yourself.
I updated the documentation according to the changes in HA, you may need to update your existing notifications to make them work again.
In v1.8.4 some changes are made as well to correctly handle the button actions.
Hope that solves your issue, please close it if so.
Otherwise, let me know.

This is an error caused by your configuration.yaml file, not by Alarmo. You will have to fix that yourself.

True, but my cfg was based on your documentation ;-)
So it was worth reporting
Simone


Book: update reference to vl EDN gallery

Hi! A while ago, I asked to add a link to a Vega Lite gallery with examples in EDN. The author of the example gallery has made a new version, and encourages people to use the new version instead. From https://github.com/behrica/vl-galery :

This PR updates the link to the new address.

@teodorlu thank you and sorry for leaving this sitting forever!


MP session at Red Hat Summit

Title: Java and MicroProfile on Azure: Building microservices in style
Speakers: Brian Benz (Microsoft), James Falkner (Red Hat)
Twitter:  @bbenz; @openatmicrosoft
URL: https://summit.redhat.com/conference/sessions
Date & Time: Wednesday, May 8, 3:30 - 4:15 PM
Abstract: This session dives into the steps required to design, build, deploy, and orchestrate a cloud-native microservices architecture using Java and Eclipse MicroProfile. We'll use Red Hat’s MicroProfile implementation known as Thorntail, optimized for deployment to Red Hat OpenShift Container Platform and integrated with Microsoft Azure services. Hear from Microsoft and Red Hat experts how adopting cloud-native, open source projects and open standards while making use of tools, APIs, and Java specifications that developers are already familiar with, can allow you to achieve superior productivity. This session includes an overview of MicroProfile and demonstrations of how to put it into practice in the cloud.

Thanks Theresa!  Suggested tweet (sucks that we can't link to the session directly but whatevs.)
If you're at the @redhatsummit, @bbenz and @schtool will show you how to work with @java and @microprofileio to deploy microservices to containers, @kubernetesio and @openshift on @azure using free, open source tools on Wednesday, May 8, 3:30PM.

Included all five sessions in one issue for faster turnaround.

@Theresa-Nguyen All session need to be in separate issues. Especially if we get anymore help to push sessions on SM. @bbenz And if you 5 sessions this MUST be sumitted 2 weeks in advance for us to push in on MP SM.

Session scheduled on twitter. It will go out today and the day before the session.


Include database optimization pycsw command in registry

Coverage remained the same at 99.47% when pulling aa0d2e4a584abfc64ac8ae440d8df005b6e56c76 on JorgeMartinezG:optimize_db into 563031f8b0245c685e091212975aa1766672e8bf on boundlessgeo:master.

Coverage remained the same at 99.47% when pulling aa0d2e4a584abfc64ac8ae440d8df005b6e56c76 on JorgeMartinezG:optimize_db into 563031f8b0245c685e091212975aa1766672e8bf on boundlessgeo:master.


fixed javadoc

Codecov Report

Merging #2180 into master will not change coverage.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master    #2180   +/-   ##
=========================================
  Coverage     97.44%   97.44%
  Complexity     5260     5260
=========================================
  Files            92       92
  Lines         12063    12063
  Branches       1595     1595
=========================================
  Hits          11755    11755
  Misses          151      151
  Partials        157      157

Impacted Files
Coverage Δ
Complexity Δ

vavr/src/main/java/io/vavr/Value.java
93.36% <ø> (ø)
144 <0> (ø)
:arrow_down:

vavr/src/main/java/io/vavr/collection/Seq.java
98% <ø> (ø)
50 <0> (ø)
:arrow_down:

vavr/src/main/java/io/vavr/collection/Tree.java
97.92% <100%> (ø)
123 <0> (ø)
:arrow_down:

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 4b473ed...87cad59. Read the comment docs.


light_byteswap.h not found

libappimageupdate build failed because "light_byteswap.h" not found on AppImageKit side.
Can you check this?
https://github.com/AppImage/AppImageUpdate/blob/8eb78102fa7f36d607db007e73b9f001cfc3c325/src/elf/CMakeLists.txt#L25
Build log of ocs-manager: https://gitlab.opencode.net/OCS/ocs-manager/-/jobs/87

Thanks for reporting. I will finally remove that hack, and properly add the brand-new libappimage repository as a submodule and use the library built there.

Had to fix a couple of bugs and issues on the way, but I've finally managed to make this important change. Please reopen if there's more issues.

OK, build passed now. Thanks for your work.


SVG morphing - yoyo support

Here's a quick demo of an SVG morphing between shapes: https://codepen.io/pbj/pen/rzYJNG
At the end of the animation sequence it "snaps" back to the original path value and then starts over again.
Is there any api option that allows it to not snap back but "yoyo" or seamlessly morph back to the original shape without any abrupt change of the SVG?

http://animejs.com/documentation/#alternate
Maybe direction: alternate fits your need.
https://codepen.io/anon/pen/eEyeMX

@thammin is right.


x/net/http2: stream reset on invalid content-length header drops conn flow control

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

$ go version
go version go1.18.4 darwin/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/ronakj/Library/Caches/go-build"
GOENV="/Users/ronakj/Library/Application Support/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/ronakj/gocode/pkg/mod"
GONOPROXY="none"
GONOSUMDB="*"
GOOS="darwin"
GOPATH="/Users/ronakj/gocode"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
GOVCS=""
GOVERSION="go1.18.3"
GCCGO="gccgo"
GOAMD64="v1"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/ronakj/project/http2-issue-repro/go.mod"
GOWORK=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch x86_64 -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/4d/2jw_2tc15x339gr53x6k64hm0000gn/T/go-build1453600270=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
I'm using Go HTTP2 server for proxying gRPC requests, where a client could potentially forward a request with a smaller content-length header.
What did you expect to see?
Stream with an invalid content-length header must result in an RST_STREAM. This must not affect other valid streams and connection level flow control.
What did you see instead?
The client slowly notices degradation in performance and eventually, the client is hung with all requests failing with deadline exceeded errors.
This issue happens as the client flow control eventually runs out of available outward flow control bytes as the HTTP2 server does not return WINDOW_UPDATE when the stream resets due to an invalid content-length header.
Issue repro

CC @neild, @tombergan.


how to build a protocol adaptor

What are the basics of building a protocol adapter in OSGP?

This should give guidelines to build an protocol adaptor: http://documentation.opensmartgridplatform.org/Architecture/Technical-overview/ProtocolLayer.html


Update test.tex

If the message below is truncated, see https://ci.appveyor.com/project/yihui/latex-pass/builds/38304919 for the full version.


fixed "Attention-based Multimodal Neural Machine Translation" link

"Attention-based Multimodal Neural Machine Translation" paper link is wrong...

Hi, thanks for pointing out!


Add docstrings for the various diagnostics-related commands.

I was having difficulty getting diagnostic flags to show up, and found it frustrating that  lsp-diagnostic-lines-enable sounded like it should do what I wanted, but instead it just complained "wrong argument count".
It turns out my actual problem was I hadn't called lsp-enable, but I figure more detailed docstrings never hurt.

Thanks for adding those!


Endpoint return type with generics does not generate right type

Description of the bug
My endpoint returns a data type with generics. The generated type is any instead of the actual type.
Minimal reproducible example
static class PageResponse<T> {
  @Nonnull
  public List<@Nonnull T> content;
  @Nonnull
  public long size;

  PageResponse(List<T> content, long size) {
    this.content = content;
    this.size = size;
  }
}

public PageResponse<Person> getPage(int page, int size) {
  var dbPage = repo.findAll(PageRequest.of(page, size));

  return new PageResponse<Person>(dbPage.getContent(), dbPage.getTotalElements());
}

Expected behavior
I would expect the content array to be of type Array<Person>
Actual behavior
The array is of type Array<any>
Versions:
- Vaadin / Hilla version: 1.2.3
- Node version: 18.8
- Java version: 17
- OS version: MacOS 13.0

Note. This used to work in earlier versions. I ran into this when updating an old project from Vaadin 20.

Honestly I don’t remember that this was ever supported, but anyway let us support it.

It may have worked accidentally. But somehow it did in a video I did https://www.youtube.com/watch?v=qpWrIn5xuAE

Looking more into that, I found that parsing / generating generic entities was never supported. Let me mark this as a feature request, not a bug.

Maybe Hilla can support generics return type by requiring developers to add a public Class<T> getClassType() to their class.
public class GenericClass<T> {

     private final Class<T> type;

     public GenericClass(Class<T> type) {
          this.type = type;
     }

     public Class<T> getClassType() {
         return this.type;
     }
}

@tarekoraby That would mean that you can‘t use generics you don‘t have access to the sourcecode like org.springframework.data.domain.Page

Note: generic endpoint methods, such as below, should be explicitly not supported:
public <T> DbEntity<T> savePerson(T pageItem) {
  //...
}


Improve ack handling

Description
This patch improves the state machines and accepts multiple failures on ACK
and SynAck packets.
Test plan
Patch was tested with manual fault injection. We need a generic framework for
fault injection so that we can probe different sequences of packet losses.

Codecov Report

Merging #371 into master will increase coverage by 0.09%.
The diff coverage is 87.17%.

@@            Coverage Diff             @@
##           master     #371      +/-   ##
==========================================
+ Coverage   58.73%   58.82%   +0.09%
==========================================
  Files          51       51
  Lines        5881     5890       +9
==========================================
+ Hits         3454     3465      +11
+ Misses       2143     2142       -1
+ Partials      284      283       -1

Impacted Files
Coverage Δ

enforcer/utils/packet/packet.go
25.5% <0%> (-0.26%)
:arrow_down:

supervisor/iptablesctrl/acls.go
60.09% <84.61%> (-0.39%)
:arrow_down:

enforcer/datapath_tcp.go
56.59% <95.83%> (+1.73%)
:arrow_up:

Continue to review full report at Codecov.

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


Intellisense does not suggest importing react hooks when hitting CTRL+space

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.71.2, 1.73.1, insiders 1.74.0
OS Version: Windows 11 Pro 22H2 22621.819

Steps to Reproduce:

Set up a react app with create-react-app --template typescript, write a few components. Install redux-toolkit and react-redux as well, but manually, not through the create-react-app command.
Close all files, close vscode
Open vscode.
Open any component
Write useEffect, hit ctrl+space, import it, write parentheses. Do not do anything then
Open another file, make sure you haven't opened it yet (it's important!)
Try to write, for example, 'useLayo' and hit ctrl+space. Vscode does not suggest importing anything, shows 'no suggestions', but it instead should suggest 'useLayoutEffect'

I also attach a video of what to do and what it looks like. Apart from that, vscode shows vague error which you can see on the video.
I checked it on the versions specified above, the video is made on the last one, which is 'insiders'. In the other two it's reproducible as well.
The insiders version I tested it on was pure and just installed. There were no extensions, nothing.
In addition, 'typescript-react' is selected, so everything should work.
Here's the video:
https://drive.google.com/file/d/1F3Jo02nOSi1iuGn8W1AUWQ5KFL-kYhjr/view?usp=sharing

Hey, are there any updates? @mjbvz
Do you need more clarification? I can provide you with all the information you need if there's something vague

Does this reproduce in the latest VS Code insiders build with all extensions disabled?

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Allow non-zero Content-Length for HEAD requests

--- Quote from RFC-2616 ---
" The Content-Length entity-header field indicates the size of the entity-body,
in decimal number of OCTETs, sent to the recipient or, in the case of the HEAD
method, the size of the entity-body that would have been sent had the request
been a GET."

Hi @rmongia,
Thank you for the patch. Just got it merged on nginx_refactoring.


add "Yogesh Telang" to the Contributors list

Hi  @dpgc11, I'm quite elated about your pull request. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a pull request and learned that I merged it?
We’ve recently added social share  to our web app. Could you please go to https://roshanjossey.github.io/first-contributions/#social-share and share your first contribution to open source?
Also, check out projects with easy issues while you’re there.


Config.Builder and Config.create() give environment variables precedence over system properties

Helidon Version: 1.2.1

Helidon MP
 and SE
JDK version: 1.8.0_211

OS: MacOS 10.13.6

Problem Description
:
The Config.Builder and Config.create() give environment variables precedence over system properties.  The config spec has the opposite. See https://download.eclipse.org/microprofile/microprofile-config-1.3/microprofile-config-spec.pdf
Note: The Helidon Javadoc actually documents this incorrect behavior:  See Config.Buider at https://helidon.io/docs/latest/apidocs/index.html?overview-summary.html
Furthermore, If the Server is created without using the Config.Builder then the behavior is correct. This works:
static Server startServer() {
return Server.create().start();
}
Steps to reproduce
:
Create a a project using MP archetype to test this.

Change app.greeting to appGreeting in GreetingProvider.java and META-INF/microprofile-config.properties.
Build and run the application as follows:

export appGreeting=HelloFromEnvironment
mvn package -DskipTests=true
java -DappGreeting="HelloFromJavaProperty"  -jar target/helidon-quickstart-mp.jar

Test the app
curl http://localhost:8080/greet
{"message":"HelloFromJavaProperty World!"}

Now change Main.java as follows:
    return Server.builder()
        .config(Config.builder()
            .sources(classpath("META-INF/microprofile-config.properties"))
            .build())
        .build()
        .start();

Retest the app and you will see the problem:
curl http://localhost:8080/greet
{"message":"HelloFromEnvironment World!"}

There are a couple of things going on here...
First, the Config.builder() and Config.create() methods are actually SE specific, though this is not readily apparent: the call to Server.builder().config(...) takes the SE config and converts it to an MP config using MpConfig.builder().config(…).build(). This conversion does not add the default MP config sources, assuming that the ones provided by the SE config are desired.
Second, sys prop and env var precedence is inverted in SE relative to MP (see #507). Taken together with the above, the behavior you are seeing is actually expected, but… confusing, for sure.
IMO, aligning the default behavior in SE with MP is the right approach, but it is a breaking change. We can certainly consider doing this for the next major release.

You should have the option to define order of env and sys properties (another issue & PR I think).
This behavior is expected (although a bit confusing as @batsatt said).
Explanation:
In Helidon, we have decided to prioritize sources as follows:

environment variables
system properties
other config sources (file, class path etc.)

This is in the order of how hard it is to change these

environment variables can be easily modified when running in a cloud environment (e.g. K8S)
system properties are harder to change, though do not require repackaging of application
config sources may require change to sources (e.g. classpath)

So the ordering is logical and from my point of view the reasoning is sound.
Unfortunately the MicroProfile specification has decided to use a different order - they consider system properties are more significant than environment variables.
You have the option to use both - either MP approach or Helidon approach.
In case you want to combine these, you need to explicitly order these sources if you want MicroProfile behavior.

Even if I explicitly order them with system properties first, the env vars take precedence: see issue 977.  It would be good if you could  fix that.  Note, that meta-config works correctly.  The precedence follows the order of appearance, e.g. I can make system properties have the highest precedence.

In progress, see PR #1002.

PR #1002 merged. Closing this; leaving open #507 to consider the merits of changing precedence.


Update Pravega version to latest master

Problem description
Update Pravega version to 0.6.0-50.51c766c-SNAPSHOT and bump the sub-module commit to 51c766c126b184a6d93f09115c1907f697bb15ee
Problem location
Build configurations
Suggestions for an improvement

It would be nice to update pravega on r0.5 branch too.


Can't navigate to a route already in the route stack

I'm seeing a weird problem where if you try to toRoute to route that already exists in the route stack, route.component is just an empty object in the renderScene function, and the app crashes. Let me know if you want to see code example. Thanks.

I don't have this issue, I would like to see an example of the code.

Also running into this issue.
This will crash as Login is somewhere early in the stack.
    this.props.resetToRoute({
        name: 'Login',
        component: Login,
        trans: true,
        hideNavigationBar: true
    });

This won't, despite being in the same code.
    this.props.resetToRoute({
        name: 'DifferentPage',
        component: DifferentPage,
        trans: true,
        hideNavigationBar: true
    });

I'm having the same issue, bump

Please provide a minimal working example to reproduce this issue.
Thanks
On Thu, Jan 7, 2016 at 8:25 PM Ryan<EMAIL_ADDRESS>wrote:

I'm having the same issue, bump
—
Reply to this email directly or view it on GitHub
https://github.com/MikaelCarpenter/gb-native-router/issues/53#issuecomment-169781082
.


fix build for osx (2.1)

Fix compilation for MacOSX
removed processing ERR_SHADER which already has been removed for other code

Note: ERR_SHADER was not removed from other code, it was added into master in https://github.com/godotengine/godot/commit/1527cf8c0d17891dd0ebf99d484f83daa46eba3c (os_unix.cpp & os_windwos.cpp) and https://github.com/godotengine/godot/commit/68bf2434619e863147a248e5cefeb37bad96fd8f (os_osx.mm), and cherry picked into 2.1 in https://github.com/godotengine/godot/commit/8edc3991fd4b5a6e1668c091afbf6aa958e741a6#.

Yep, my mistake :)


Upgrade Imageinary version

The latest version of Imageinary has a new CLI which isn't compatible with the way it is called in the DALI tests at present.
Closes #50
Signed-Off-By: Robert Clark<EMAIL_ADDRESS>

This is on hold until Imageinary is published with the next nvidia-pyindex update as this update will pull an incompatible version in the meantime.

Imageinary has been updated with nvidia-pyindex and this can now be moved forward.


Failed to disable L1 on Xiaomi mobile phone

I have encountered a problem again, I'm a Xiaomi phone, the current software detection is L1, the use of the document provides a mask plug-in to disable L1, but the disable failed, The device is stuck on the startup screen, Fortunately, the brick saving plugin is installed, how to solve the problem?

@yiz24
The problem you cite seems to be related to an incompatibility of oemcrypto disabler for your device. trying to contact the creators of this plugin directly but doesn't seem to be active for a while..


invalid url in the heading of github

The url of the project is still https://grid.rebassjs.org and should probably be https://rebassjs.org/grid/

Moved to https://rebassjs.org/reflexbox


[QUESTION] How to define empty query parameters as missing?

First check

[x] I used the GitHub search to find a similar issue and didn't find it.
[x] I searched the FastAPI documentation, with the integrated search.
[x] I already searched in Google "How to X in FastAPI" and didn't find any information.

Description
How can I make empty query parameters to be interpreted as missing value (and therefore default value is used) instead of empty string? For example, if I were to use optional int query param, empty string would find its way as value being passed and validation error for invalid int would be  raised.
Additional context
Sample code:
from fastapi import APIRouter

router = APIRouter()

@router.get("/test/", tags=["test"])
def test(q: int = None):
    return {"q": q}

GET http://URL/test/?q=
or
GET http://URL/test/?q
results in:
{
  "detail": [
    {
      "loc": [
        "query",
        "q"
      ],
      "msg": "value is not a valid integer",
      "type": "type_error.integer"
    }
  ]
}

Interesting, I can reproduce this with the latest fastapi, but not with fastapi~=0.49.0.

@phy25 I think it is somehow connected with this PR

same question here, any updates on this?:)

Hmm, yeah, it probably has to do with https://github.com/encode/starlette/pull/672.
I think this would be a request for Starlette. To parse ?q and ?q= as q = None instead of q = "".
Do you wanna request it there?

@tiangolo thanks for the reply!
Actually, I'm not sure which approach is better either, but I think that there should be a way to handle this in both use-cases. Since use-case "empty query parameter (?q or ?q=) has value of empty string" is the default one, I am curios about how to achieve "empty query parameter (?q or ?q=) is missing (e.g. None & default value is used)"?
Appreciate your help, thanks!

@tiangolo thanks for the reply!
Actually, I'm not sure which approach is better either, but I think that there should be a way to handle this in both use-cases. Since use-case "empty query parameter (?q or ?q=) has value of empty string" is the default one, I am curios about how to achieve "empty query parameter (?q or ?q=) is missing (e.g. None & default value is used)"?
Appreciate your help, thanks!

Same requirements here, and handled with a middleware at last.
We can just drop those fields with blank values, which are empty strings or strings contain only whitespaces, default values will be handled by Query models.
# Add blank query parameters processing middleware
@app.middleware('http')
async def drop_blank_query_params_middleware(request: Request, call_next):
    _scope = request.scope
    if request.method != 'GET':
        return await call_next(request)
    if not _scope or not _scope.get('query_string'):
        return await call_next(request)

    def _process_query_params(query_params):
        _query_params = QueryParams(query_params)

        from urllib.parse import urlencode
        return urlencode([
            (k, v) for k, v in _query_params.items() if v and v.strip()
        ])

    _scope['query_string'] = _process_query_params(_scope['query_string']).encode('latin-1')
    return await call_next(Request(_scope, request.receive, request._send))  # noqa

However, as @tiangolo commented, this approach may not be correct, maybe it's not recommended. :)

@tiangolo thanks for the reply!
Actually, I'm not sure which approach is better either, but I think that there should be a way to handle this in both use-cases. Since use-case "empty query parameter (?q or ?q=) has value of empty string" is the default one, I am curios about how to achieve "empty query parameter (?q or ?q=) is missing (e.g. None & default value is used)"?
Appreciate your help, thanks!

Same requirements here, and handled with a middleware at last.
We can just drop those fields with blank values, which are empty strings or strings contain only whitespaces, default values will be handled by Query models.
# Add blank query parameters processing middleware
@app.middleware('http')
async def drop_blank_query_params_middleware(request: Request, call_next):
    _scope = request.scope
    if request.method != 'GET':
        return await call_next(request)
    if not _scope or not _scope.get('query_string'):
        return await call_next(request)

    def _process_query_params(query_params):
        _query_params = QueryParams(query_params)

        from urllib.parse import urlencode
        return urlencode([
            # using `_query_params.items()` will mistakenly process list parameters
            (k, v) for k, v in _query_params._list if v and v.strip()  # noqa
        ])

    _scope['query_string'] = _process_query_params(_scope['query_string']).encode('latin-1')
    return await call_next(Request(_scope, request.receive, request._send))  # noqa

However, as @tiangolo commented, this approach may not be correct, maybe it's not recommended. :)

Note that QueryParams comes from starlette.datastructures. For it to work add from starlette.datastructures import QueryParams.
Works like a charm, btw. Thanks!

@tiangolo thanks for the reply!
Actually, I'm not sure which approach is better either, but I think that there should be a way to handle this in both use-cases. Since use-case "empty query parameter (?q or ?q=) has value of empty string" is the default one, I am curios about how to achieve "empty query parameter (?q or ?q=) is missing (e.g. None & default value is used)"?
Appreciate your help, thanks!

Same requirements here, and handled with a middleware at last.
We can just drop those fields with blank values, which are empty strings or strings contain only whitespaces, default values will be handled by Query models.
# Add blank query parameters processing middleware
@app.middleware('http')
async def drop_blank_query_params_middleware(request: Request, call_next):
    _scope = request.scope
    if request.method != 'GET':
        return await call_next(request)
    if not _scope or not _scope.get('query_string'):
        return await call_next(request)

    def _process_query_params(query_params):
        _query_params = QueryParams(query_params)

        from urllib.parse import urlencode
        return urlencode([
            # using `_query_params.items()` will mistakenly process list parameters
            (k, v) for k, v in _query_params._list if v and v.strip()  # noqa
        ])

    _scope['query_string'] = _process_query_params(_scope['query_string']).encode('latin-1')
    return await call_next(Request(_scope, request.receive, request._send))  # noqa

However, as @tiangolo commented, this approach may not be correct, maybe it's not recommended. :)

Note that QueryParams comes from starlette.datastructures. For it to work add from starlette.datastructures import QueryParams.
Works like a charm, btw. Thanks!

Thanks for the answer. I met this issue when I use amis with FastAPI.
According to above code snippet, I make a litte optimization to simplify code.
from typing import Callable
from urllib.parse import parse_qsl, urlencode

from fastapi import Request

@app.middleware("http")
async def filter_blank_query_params(request: Request, call_next: Callable):
    scope = request.scope
    if scope and scope.get("query_string"):
        filtered_query_params = parse_qsl(
            qs=scope["query_string"].decode("latin-1"),
            keep_blank_values=False,
        )
        scope["query_string"] = urlencode(filtered_query_params).encode("latin-1")
    return await call_next(request)


Fix publish platform external contrib

No need to wait for e2e tests, only publish platform ci was changed in PR


Render text

@ankush-me  The script render text based on segmentation image? Can script render text on image based on depth image?

It is also based on depth image.

So can I have just depth info without seg and then the script will work fine?


feat(napi): add tracing via OXC_LOG:DEBUG

Just a nitpick: DEBUG can imply that there might be other levels of logging. Do you plan to add them?

Just a nitpick: DEBUG can imply that there might be other levels of logging. Do you plan to add them?

I may add other INFO logs when the current information is insufficient for complicated situations.

Oh, I see a trace here tracing::trace!(tsconfig = ?tsconfig, "load_tsconfig"); 😅

It's actually the other way around. Info gives less information, and trace gives more information than debug


Fixes color border around export image dialog icon

The original border was too light. This PR sets the right color for the border:

Original issue: #4925

cc: @xavijam

Frontend tests were OK :+1: (details)

:heart:


native_constant_invocation applies to catch section without variables

Bug report
PHP 8.0.2
PHP CS Fixer 3.0.0

Code snippet that reproduces the problem
return (new PhpCsFixer\Config())
    ->setFinder($finder)
    ->setRules([
        'native_constant_invocation' => true,
    ])
;

try {
    $date = new \DateTime();
} catch (\Exception) {
}

Suggests
try {
    $date = new \DateTime();
} catch (Exception) {
}

@BoShurik can you take a look at the PR ☝🏼 (and maybe drop a review)?

@kubawerlos works for me

The bug is still present in version 3.5.0
try {
    // ...
} catch (\InvalidArgumentException|\LogicException) {
    // ...
}

Suggets :
try {
    // ...
} catch (\InvalidArgumentException|LogicException) {
    // ...
}

thanks for reporting @niji-mneute  , proposed fix is here https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/6264

I can confirm that #6264 fix #6271. Thank you @SpacePossum!


[SPARK-30764][SQL] Improve the readability of EXPLAIN FORMATTED style

What changes were proposed in this pull request?
The style of EXPLAIN FORMATTED output needs to be improved. We’ve already got some observations/ideas in
https://github.com/apache/spark/pull/27368#discussion_r376694496.
Observations/Ideas:

Using comma as the separator is not clear, especially commas are used inside the expressions too.
Show the column counts first? For example, Results [4]: …
Currently the attribute names are automatically generated, this need to refined.
Add arguments field in common implementations as EXPLAIN EXTENDED did by calling argString in TreeNode.simpleString. This will eliminate most existing minor differences between
EXPLAIN EXTENDED and EXPLAIN FORMATTED.

This PR is currently addressing comments 2 & 4, and open for more discussions on improving readability.
Why are the changes needed?
The readability of EXPLAIN FORMATTED need to be improved, which will help user better understand the query plan.
Does this PR introduce any user-facing change?
Yes, EXPLAIN FORMATTED output style changed.
How was this patch tested?
Update expect results of test cases in explain.sql

Can one of the admins verify this patch?

Can one of the admins verify this patch?

cc @gatorsmile @cloud-fan @maropu @dilipbiswal

ok to test

Thanks for your work, @Eric5553 !

Test build #118102 has started for PR 27509 at commit 9d30fad.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/22867/
Test PASSed.

Test build #118102 has finished for PR 27509 at commit 9d30fad.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/118102/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/22875/
Test PASSed.

Test build #118110 has started for PR 27509 at commit 4d0fcff.

Also cc @maryannxue @hvanhovell

Test build #118110 has finished for PR 27509 at commit 4d0fcff.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/118110/
Test FAILed.

retest this please

Test build #118138 has started for PR 27509 at commit 4d0fcff.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/22904/
Test PASSed.

Test build #118138 has finished for PR 27509 at commit 4d0fcff.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/118138/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/23198/
Test PASSed.

Test build #118441 has started for PR 27509 at commit 9d52f92.

Also trying to address the improvement https://github.com/apache/spark/pull/27368#discussion_r376927143 here in this PR. Add new string format for Expression to avoid much footprint on existing behaviors. There are still some #exprId not removed, I'm keep working on this.
@cloud-fan

Test build #118441 has finished for PR 27509 at commit 9d52f92.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/118441/
Test FAILed.

retest this please

Test build #118458 has started for PR 27509 at commit 9d52f92.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/23215/
Test PASSed.

Test build #118458 has finished for PR 27509 at commit 9d52f92.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/118458/
Test FAILed.

retest this please

Test build #118480 has started for PR 27509 at commit 9d52f92.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/23238/
Test PASSed.

Test build #118480 has finished for PR 27509 at commit 9d52f92.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/118480/
Test FAILed.

Test build #118507 has started for PR 27509 at commit f0bcf12.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/23262/
Test PASSed.

Test build #118507 has finished for PR 27509 at commit f0bcf12.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/118507/
Test FAILed.

File "/home/jenkins/workspace/SparkPullRequestBuilder/python/pyspark/sql/dataframe.py", line 282, in pyspark.sql.dataframe.DataFrame.explain
Failed example:
    df.explain(mode="formatted")
Differences (ndiff with -expected +actual):
      == Physical Plan ==
      * Scan ExistingRDD (1)
    + <BLANKLINE>
    + <BLANKLINE>
      (1) Scan ExistingRDD [codegen id : 1]
    - Output: [age#0, name#1]
    + Output [2]: [age#0, name#1]
    ?       ++++
    + Arguments: [age#0, name#1], MapPartitionsRDD[4] at applySchemaToPythonRDD at NativeMethodAccessorImpl.java:0, ExistingRDD, UnknownPartitioning(0)
    + <BLANKLINE>
    + <BLANKLINE>
**********************************************************************
   1 of   3 in pyspark.sql.dataframe.DataFrame.explain
***Test Failed*** 1 failures.

@Eric5553 can you fix the pyspark test? you should update dataframe.py explain method
>>> df.explain(mode="formatted")
        == Physical Plan ==
        * Scan ExistingRDD (1)
        (1) Scan ExistingRDD [codegen id : 1]
        Output: [age#0, name#1]

        .. versionchanged:: 3.0.0
           Added optional argument `mode` to specify the expected output format of plans.
        """

@cloud-fan Sorry for missed the failed unit test(just found where to get the jenkins error log...). I'll fixed the pyspark test and address comments today.
And I got an implementation of removing useless #{exprId.id}, but it needs a lot of changes for existing tests. Should I commit here in this PR or open a separate one? Thanks

Let's open a new PR for that.

Test build #118745 has started for PR 27509 at commit 1290cd5.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/23496/
Test PASSed.

LGTM

Test build #118745 has finished for PR 27509 at commit 1290cd5.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/118745/
Test FAILed.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/23515/
Test PASSed.

retest this please

Test build #118765 has started for PR 27509 at commit 1290cd5.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/23517/
Test PASSed.

Test build #118765 has finished for PR 27509 at commit 1290cd5.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/118765/
Test PASSed.

thanks, merging to master/3.0!

Thanks so much! @cloud-fan @maryannxue @maropu @gatorsmile


Quest v0.2 / G6PD: Develop Results Interpretation Module (RIM)

As a healthcare worker, I should be able to assign patient classification based on predetermined thresholds of G6PD activity.
Based on the configured thresholds, the following classification can be assigned to male and female patients:
Male

G6PD deficient
G6PD normal

Female

G6PD deficient
G6PD intermediate
G6PD normal

Please refer to image below for G6PD status & PQ regimen:

Calculator should be available with or without the link to patient record
Calculation/decision support flow should be available as a stand-alone functionality of the app as well as part of a reference application. The decision support flow will be provided through Results Interpretation Module (RIM).  RIM will be available both as a separate module (and can be integrated by third-party applications) and as part of the reference application.

SubTask

[x] #627

Technical approach is discussed in https://github.com/opensrp/fhircore/discussions/659, specifically https://github.com/opensrp/fhircore/discussions/659#discussioncomment-1548255

think this is complete


Any plan to remove CLANG/LLVM from Cilium Agent docker image?

Proposal: Remove CLANG/LLVM from Cilium Agent image, only deliver eBPF bytecode
I installed cilium 1.4.8, found clang/llvm and other relevant  bpf compiler tool chain are packaged in cilium agent image.
Describe the solution you'd like
Expects cilium can only deliver ebpf bytecode like go-ebpf to avoid having a runtime dependency.

@yongming This can be a possibility. Since cilium 1.5 that we have introduced bpf templates which does not require clang/llvm to compile bpf programs anymore. Currently clang/llvm are present because each different option setup in cilium-agent might require clang to compile a new bpf bytecode. Subsequent compilation are avoided wherever possible.
Is there any particular reason why you would want clang/llvm to be removed from cilium agent?

Docker image of Cilium agent size is about 622M (v1.4.8), LLVM/CLANG and other compiler relevant files occupy pretty much spaces. If only packaging necessary runtime libraries，the image size might be reduced greatly.

Currently clang/llvm are present because each different option setup in cilium-agent might require clang to compile a new bpf bytecode. Subsequent compilation are avoided wherever possible.

What options affect compilation?
We want to include Cilium in a minimum k8s distro and want to minimize our footprint and have full reproducibility for our images. Reproducing the Clang toolchain is quite an effort :-)

Our base datapath (i.e., all except BPF programs for endpoints) is currently compiled when the Cilium agent on the node starts. We have https://github.com/cilium/cilium/issues/9515 to track work on removing that requirement for Clang.
Then, as André said, we also need to compile the BPF programs whenever the Cilium agent configuration options are changed, for example if you want to disable monitor aggregation or enable audit mode on an existing cluster. One option to remove that requirement to have Clang on Cilium pods would be to let cilium-operator handle that compilation. There's a lot more work needed to implement that though.


[ISSUE #4138] Add Unit Test for UrlMappingPattern

Fixes #4138 .
Motivation
The TO-DO mark said that "It is better using Mockito not PowerMockito", so I used Mockito. Although there are private method/field to test in the compile() method of UrlMappingPattern.
Modifications
Use reflections to access private method and field, then write regular expressions.
The Test case passed.
Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)
If a feature is not applicable for documentation, explain why?
If a feature is not documented yet in this PR, please create a followup issue for adding the documentation

It's OK for me after you remove the old comment // Verify that the compiledUrlMappingPattern field is updated.

Of course, I have committed the changes.
I have a slight confusion in that although the assertNotNull() method does not verify that the compiledUrlMappingPattern matches the mockedPattern as the assertEquals() method did before, it still ensures that the compiledUrlMappingPattern has been updated by not being null.
Could you kindly enlighten me on the reasons behind your thoughts, if you don't mind?

Thank you for your guidance! I agree with your viewpoint. I have reinstated this line of comment, and I would appreciate it if you could kindly approve it when you have a moment.

@Pil0tXia CI not pass, pls check it

@mxsm Hello! I have pulled the latest code from master and made modifications accordingly. 😉

PTAL😊 @xwm1992

PTAL😊 @xwm1992

@lrhkobe PTAL😉


Godot 4 Beta 10 big FPS drops Linux Ubuntu

Godot version
4.0 beta 10
System information
Ubuntu Studio 22.04, 12x Intel I5 10500H 2.50GHz  4gb envidia 1650 max Q
Issue description
I have a project done in Godot 4.0 beta 8 that goes perfectly well, it has almost no code yet, only the necessary for debugging. I just opened the project in Godot 4.0 beta 9 and beta 10 and in both the project suffers a drop in FPS from 60 to 10.
I did nothing more than open the project and hit play.

Steps to reproduce
In my case I only have to open the project made in Godot 4.0 beta 8 in godot beta 9 or 10.
Minimal reproduction project
N/A

@CHIESOFTWARE Please upload a minimal reproduction project to make this easier to troubleshoot.

I am currently adding and removing scenes to see which one is causing problems. If I remove some scenes and leave the basics the FPS remains stable. Apparently something in some scene must be causing the bug which doesn't happen with beta 8.

@Calinou Can you send me an email or something to share you my project so you can help me to find what is causing this bug?

Can you send me an email or something to share you my project so you can help me to find what is causing this bug?

We don't have time to inspect full projects individually, so I'm afraid I can't provide that.

@Calinou I don't want you to review my project, I just want you to use it as a test project to find the error.

I found some lines of code in charge of verifying the distance of the player and based on the distance change the material of the object in question. There are at least 600 of these objects, they are only checked if the player is close, i.e. all 600 are never checked at the same time or all of them.
These 10 lines of code did not give me problems in the previous beta and now they do, I am working to create a test project for you to see what causes this bug.

@CHIESOFTWARE Regardless of how you phrase it, Godot 4 is supposed to release before March. To hit that target, contributors need to focus on finishing the last features, not "use your full project to find the error". That's why you're asked to give a MRP - the smallest project you can make which has the bug

@Zireael07 @Calinou I understand, I'm trying to get an alpha of the project for that date. I have already uploaded the test project, certain conditions are needed for the error to occur.
In the past the code you will see there did not cause any problem, now in beta 9 and 10 the process increases the milliseconds up to more than 2000ms. I don't know what changes you have made in that aspect, but it seems that there is a bottleneck when processing many objects.

@CHIESOFTWARE
Flamegraph shows that you use a lot of "material_override" 99% of time... setting material override is very expensive process.

try to minimize the material_override usage (avoid using every frame or avoid using Timer with every 0.5sec timeout.. lol )

@stmSi Hi, Ok I understand, but why does it cause this problem only in beta 9 and 10 and not in beta 8? If that's all it should fail in all beta. I am not an expert in this area, obviously.

i didn't have the debug build for beta 8, i will check it after i build it

By the way, in the process of looking for what was causing the problem, I changed the rendering to mobile-compatible rendering and noticed that the distance fade of the lights does not work well at all.

I changed the rendering to mobile-compatible rendering and noticed that the distance fade of the lights does not work well at all.

Distance fade is broken if you use any lights with projectors in the scene: https://github.com/godotengine/godot/issues/67845

I am using omnilights and still the distance field does not work.


Extension for parsing undocumented message attributes

Some Twitch features such as channel point redeems are not documented but would be nice if we could parse them. I'm thinking the library could include an extension trait with functions to parse these details, e.g. ChannelPointRedemptionInformation from a PrivmsgMessage.
It would also probably not be a bad idea to expose IRCMessageParseExt to allow third parties to parse details on their own, if needed.

See also #161

See also #159


internal/restorable: remove ImageTypeVolatile

Operating System

[ ] Windows
[ ] macOS
[ ] Linux
[ ] FreeBSD
[ ] OpenBSD
[X] Android
[ ] iOS
[ ] Nintendo Switch
[ ] PlayStation 5
[ ] Xbox
[ ] Web Browsers

What feature would you like to be added?
Now we have Hint which is for optimization when overwriting some regions on a target texture, an image which is cleared every frame should no longer be stale. Thus, we don't need a notion of being volatile. In order to simplify the code, let's try to remove this.
Why is this needed?
To simplify the code.

an image which is cleared every frame should no longer be stale

Unfortunately this is not true when too many draw calls happen on the same image.


Use this integration or github.com/absalom-muc?

I currently have two MHI aircons. Both are integrated into HA using an ESP running github.com/absalom-muc. I would like to add two new MHI aircons to my house and learnt that new MHI devices have WIFI adapters already built-in.
I assume this integration (Mitsubishi-WF-RAC-Integration) uses these WIFI adapters to control the aircons.
Can someone confirm that one can simply unplug the internal WIFI adapter and use the ESP solution instead (assuming the internal adapter uses the same CNS interface)?

I have 1x SRK20ZSX-W, 1x SRK25ZSX-W and one SRK35ZSX-WF.
They have exactly the same PCB and connections.
In home assistant the integration of Mitsubishi-WF-RAC is almost the same as MHI-AC-Ctrl-ESPHome. You can just remove the wifi adapter and use an esp.

@royspeed from my understanding the esp offers some more options, e.g. connecting or overwritting the internal temperature sensor with an external bluetooth temperatur sensor.
What are your experience with the ESP option compared to the WIFI and which PCB board did you use?


Tweak setup.py for future PyPi upload

I made a few small changes to the setup.py file. Most importantly,
changed the author from myself to “The PyAbel Team”, to acknowledge the
hard work by all of the current (and hopefully future!) contributors to
the project.
Also, changed the description to mention that we also do the forward
Abel transform. Finally, made it clear that the lack of Cython would
only doom the C implementation of abel.direct, since we now have the
pure-Python fallback, right @rth?

Everything looks great. We can merge this now, or if you intend to upload the package to PyPi, we can also use this PR to add a few more arguments to setup asked by PyPi, I think in particular licence and classifiers (not completely sure).

Okay, I added the license, url, and classifiers field. I wasn't quite sure what to put for some of these, so please review my choices:
license='GPLv3'

Looking on http://choosealicense.com/, GPL version 3 seems like the best license to me. It only allow open source use, which seems nice. The other option is the MIT license, which allows for all uses, including closed-source commercial uses. I'd like to see PyAbel widely used, but also like to support open source. Happy to hear thoughts from other contributors as well (@DhrubajyotiDas @stggh). Currently we have the GPLv2 license in the repo. According to the above link, "V3 is similar to V2, but further restricts use in hardware that forbids software alterations." So, V3 seems better, I suppose.
classifiers=[
# How mature is this project? Common values are
#   3 - Alpha
#   4 - Beta
#   5 - Production/Stable
'Development Status :: 3 - Alpha',

# Indicate who your project is intended for
'Intended Audience :: Science/Research',
'Intended Audience :: Developers',
'Topic :: Scientific/Engineering :: Mathematics',
'Topic :: Scientific/Engineering :: Physics',
'Topic :: Scientific/Engineering :: Medical Science Apps.',
'Topic :: Software Development :: Libraries :: Python Modules'

# Pick your license as you wish (should match "license" above)
'License :: OSI Approved :: GNU General Public License v3 (GPLv3)',

The options for the Intended Audience and Topic are listed here. I took a few guesses. Let me know if you have suggestions. I included "Medical Science Apps." since I thought that perhaps the Abel transform would be used by people dealing with reconstructing objects from X-ray images.
# Specify the Python versions you support here. In particular, ensure
# that you indicate whether you support Python 2, Python 3 or both.
'Programming Language :: Python :: 2',
'Programming Language :: Python :: 2.7',
'Programming Language :: Python :: 3',
'Programming Language :: Python :: 3.3',
'Programming Language :: Python :: 3.4',
'Programming Language :: Python :: 3.5',
],

I included just the python versions that we have on Travis CI. So, we are saying that we officially support Python 2.7, 3.3, 3.4, 3.5, but not 2.6 or anything else. Seems reasonable enough.

Regarding the Python version, yes I think we can say that we support version that are listed in Travis CI setup file. Python 2.6 will probably produce errors  (e.g. it does not support dictionary generators); 3.1, 3.2 might work, but again since we don't tests for them it's hard to tell. So I think that we can say that Python 2.7 & 3.3-3.5 are officially supported.  The classifiers look good too.
Personally, I don't know much about licensing: both GPL v2/v3 or MIT type of license sound good. One point to consider I suppose what would happen if some other project would want to use PyAbel, and decided to distribute binaries of the executable they provide (including PyAbel). In the case of a GPL license, this would only be possible if the project itself is under an open-source license (GPL/MIT/etc).   In the case of an MIT license, they could do it  even if their project is not open source. In the latter case, there still could be contributions back to PyAbel, it just means that they are allowed to distribute PyAbel included in their project. I don't really have strong convictions one way or the other, probably a bit more in favor of the MIT license, but we can  keep GPL v3 for now and see if that's an issue later. @DhrubajyotiDas and @stggh would this work for you?
The only external project I'm aware of that could potentially use PyAbel in Python scripting is Neutrino developed by  @aflux and @iltommi, but it has a GPL license, so it's not an issue either way.

I think you're right. We should let people incorporate PyAbel into binaries. And the more people using it the better. MIT license it is.

My preference is for GPLv3, in support of true open source software. However, I have no great objection your decision to expand the application of PyAbel by using the MIT license.

Personally, I am not very knowledgeable about licenses either, but in the interests of building consensus, I support applying the MIT license as well.

Okay, I'll go ahead and merge this with the MIT license, but we can continue to discuss in the future.


Appsource validation will fail or not for BC17

Hi,
We are getting this warning
Use of implicit 'with' will be removed in the future. Qualify with 'Rec.'. This warning will become an error in a future release.
If we do not solve this warning in our 17.0 apps, will the app source validation will pass or not?
Please let us know?
Thanks & Regards
Varun Reddy

This warning was introduced for BC 17; it is still just a warning. Only errors won't let the validation pass.

Thanks for the update what about another similar warning of with keyword will be deprecated for use in cloud

! The warning will not block the submission to AppSource. It is a warning meant to enforce a coding pattern that allows us to evolve the language by adding new system library functions without breaking extensions downstream.
All the diagnostics that are related to AppSource validation come from AppSourceCop and have the AS* prefix https://docs.microsoft.com/en-us/dynamics365/business-central/dev-itpro/developer/analyzers/appsourcecop


Make top bar button white for accessibility reasons

Design team 'Public'/'Private' button in the top bar be white to match the rest of the other buttons for accessibility reasons.
https://warthogs.atlassian.net/browse/WD-5291

@carkod Could you look at this one also please?

@carkod Could you look at this one also please?

How do I QA this?

Is this the intended look? I just tested this in the style editor, so maybe something is wrong. Do you have more background why a white button will increase accessibility? Is this about contrast ratio?
Design may have a better expert opinion, but I do prefer the additional color the orange button brings in. Note also that some event themes overwrite the button color on event pages, e.g. the 'Canonical' theme makes it aubergine.

@kewisch I wasn't aware there were differences across different themes. I would image this would also be in-accessible. I would suggest we keep the navigation in the default colours in this case. But I will bring this up with design first and see if they are willing to compromise.

Closing as this work is no longer needed.


2.0 Beta - Receiving a trace error: insert into stat_hourly - Duplicate entry '43-1731949200' for key 'stat_hourly_monitor_id_timestamp_unique'

⚠ Please verify that this question has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

📝 Describe your problem
I'm seeing a trace error of duplicate entry for a given monitor stat. I'm not sure why, but I can provide the logs.
📝 Error Message(s) or Log

Trace: Error: insert into `stat_hourly` (`down`, `monitor_id`, `ping`, `ping_max`, `ping_min`, `timestamp`, `up`) values (0, 43, 94.4, 102, 87,<PHONE_NUMBER>, 5) - Duplicate entry '43-1731949200' for key 'stat_hourly_monitor_id_timestamp_unique'
    at Packet.asError (/app/node_modules/mysql2/lib/packets/packet.js:738:17)
    at Query.execute (/app/node_modules/mysql2/lib/commands/command.js:29:26)
    at Connection.handlePacket (/app/node_modules/mysql2/lib/connection.js:481:34)
    at PacketParser.onPacket (/app/node_modules/mysql2/lib/connection.js:97:12)
    at PacketParser.executeStart (/app/node_modules/mysql2/lib/packet_parser.js:75:16)
    at Socket.<anonymous> (/app/node_modules/mysql2/lib/connection.js:104:25)
    at Socket.emit (node:events:519:28)
    at addChunk (node:internal/streams/readable:559:12)
    at readableAddChunkPushByteMode (node:internal/streams/readable:510:3)
    at Readable.push (node:internal/streams/readable:390:5) {
  code: 'ER_DUP_ENTRY',
  errno: 1062,
  sqlState: '23000',
  sqlMessage: "Duplicate entry '43-1731949200' for key 'stat_hourly_monitor_id_timestamp_unique'",
  sql: 'insert into `stat_hourly` (`down`, `monitor_id`, `ping`, `ping_max`, `ping_min`, `timestamp`, `up`) values (0, 43, 94.4, 102, 87,<PHONE_NUMBER>, 5)'
}
    at Timeout.safeBeat [as _onTimeout] (/app/server/model/monitor.js:1012:25)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
2024-11-18T09:59:16-08:00 [MONITOR] ERROR: Please report to https://github.com/louislam/uptime-kuma/issues
2024-11-18T09:59:16-08:00 [MONITOR] INFO: Try to restart the monitor

🐻 Uptime-Kuma Version
2.0.0-beta.0
💻 Operating System and Arch
TrueNAS Scale Electric Eel 24.10.0
🌐 Browser
N/A
🖥 Deployment Environment

Runtime: Docker 27.1.1
Database: External MariaDB
Filesystem used to store the database on: ZFS on nvme
number of monitors: 16

We are also seeing this error for stat_daily. This is preventing our push monitors to come back online again.
Running: louislam/uptime-kuma:nightly2
Trace: Error: insert into `stat_daily` (`down`, `monitor_id`, `ping`, `ping_max`, `ping_min`, `timestamp`, `up`) values (219, 14, 0, 0, 0,<PHONE_NUMBER>, 580) - Duplicate entry '14-1733270400' for key 'stat_daily_monitor_id_timestamp_unique'
    at Packet.asError (/app/node_modules/mysql2/lib/packets/packet.js:738:17)
    at Query.execute (/app/node_modules/mysql2/lib/commands/command.js:29:26)
    at Connection.handlePacket (/app/node_modules/mysql2/lib/connection.js:481:34)
    at PacketParser.onPacket (/app/node_modules/mysql2/lib/connection.js:97:12)
    at PacketParser.executeStart (/app/node_modules/mysql2/lib/packet_parser.js:75:16)
    at Socket.<anonymous> (/app/node_modules/mysql2/lib/connection.js:104:25)
    at Socket.emit (node:events:519:28)
    at addChunk (node:internal/streams/readable:559:12)
    at readableAddChunkPushByteMode (node:internal/streams/readable:510:3)
    at Readable.push (node:internal/streams/readable:390:5) {
  code: 'ER_DUP_ENTRY',
  errno: 1062,
  sqlState: '23000',
  sqlMessage: "Duplicate entry '14-1733270400' for key 'stat_daily_monitor_id_timestamp_unique'",
  sql: 'insert into `stat_daily` (`down`, `monitor_id`, `ping`, `ping_max`, `ping_min`, `timestamp`, `up`) values (219, 14, 0, 0, 0,<PHONE_NUMBER>, 580)'
}
    at Timeout.safeBeat [as _onTimeout] (/app/server/model/monitor.js:1012:25)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
2024-12-04T08:32:56+01:00 [MONITOR] ERROR: Please report to https://github.com/louislam/uptime-kuma/issues
2024-12-04T08:32:56+01:00 [MONITOR] INFO: Try to restart the monitor

Not sure what the desired behaviour is and what the stat tables are for.
But if someone could point us in the right direction, we might be able to contribute a PR.


Changed short array to traditional syntax to make it compatible with php 5.3

Changed short array to traditional syntax to make it compatible with 5.3

The minimum version of this library is PHP 5.4.7. PHP 5.3 is super old and hasn't been supported by PHP itself in years, it makes no sense for us to support that.


Mono, paket 3.31.6, usageStringCharacterWidth error

Description
Getting an error in on Linux, Mono 4.6.2
Paket failed with:
    Must be positive value.
Parameter name: usageStringCharacterWidth

Repro steps

Use a linux/docker container with mono 4.6.2
Run paket restore

CI build shows it in action as well.
https://travis-ci.org/fsprojects/Paket/builds/185498758
Expected behavior
Paket runs
Actual behavior
Paket does not run
Known workarounds
Pinning on paket.exe 3.31.5

Have same issue

Can you please try to send me a PR?
Am 20.12.2016 5:54 nachm. schrieb "Alexey"<EMAIL_ADDRESS>
Have same issue
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fsprojects/Paket/issues/2092#issuecomment-268295954,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNKraPqMJhSY2dLkFcWEKRzAC-9XJks5rKAhMgaJpZM4LSCzS
.

@forki, sorry, but I have no idea how to fix this issue. I just have a suggestion that mentioned commit may be a root cause of this issue.

/cc @eiriktsarpalis

Can confirm. Pinning Paket to 3.31.4 works for now.
+++ b/main/external/fsharpbinding/.paket/paket.targets
@@ -13,7 +13,7 @@
   <PropertyGroup>
     <!-- Paket command -->
     <PaketExePath Condition=" '$(PaketExePath)' == '' ">$(PaketToolsPath)paket.exe</PaketExePath>
-    <PaketBootStrapperExePath Condition=" '$(PaketBootStrapperExePath)' == '' ">$(PaketToolsPath)paket.bootstrapper.exe</PaketBootStrapperExePath>
+    <PaketBootStrapperExePath Condition=" '$(PaketBootStrapperExePath)' == '' ">$(PaketToolsPath)paket.bootstrapper.exe 3.31.4</PaketBootStrapperExePath>
     <PaketCommand Condition=" '$(OS)' == 'Windows_NT'">"$(PaketExePath)"</PaketCommand>
     <PaketCommand Condition=" '$(OS)' != 'Windows_NT' ">$(MonoPath) --runtime=v4.0.30319 "$(PaketExePath)"</PaketCommand>
     <PaketBootStrapperCommand Condition=" '$(OS)' == 'Windows_NT'">"$(PaketBootStrapperExePath)"</PaketBootStrapperCommand>

Ok. Will revert argu update when I'm home and hopefully @eiriktsarpalis can send a pull request with showing me the error. It's especially weird that this only happens on mono.

as @nosami pointed, you just need to specify required version as argument to paket.bootstrapper.exe
paket.bootstrapper.exe 3.31.5
Glad the discussion is live as our FAKE scripts failed this morning :)

Also getting this on Teamcity + Fake builds on mono/linux. Pinning to 3.31.5 works around it, but isn't ideal.

ok I reverted. can someone please confirm 3.31.7 is good again?

I've checked. 3.31.7 works. Thanks!)

Seems to work for me 🎸

Thanks!

@TheAngryByrd @alexey-belous Can you confirm that evaluating the Console.WindowWidth property on docker returns 0? If so, this is related to an issue fixed in Argu 3.6.1

ok I updated argu to 3.6.1 and hope the bug is gone. Can someone please verify that Pakt 3.31.8 still works on mono?

@forki, 3.31.8 works for me.

thanks everyone!


Protect against string formatting None

f{"None:.1f}" throws an error and this should protect against that. Also Ideally sets the drag callback to do it's work if the explore sample widget is currently visible (Is there a way to fully disable the callback rather than returning early?)
Attn: @fdrgsp

Codecov Report

Merging #41 (16d27eb) into main (2a98963) will decrease coverage by 0.07%.
The diff coverage is 27.27%.

@@            Coverage Diff             @@
##             main      #41      +/-   ##
==========================================
- Coverage   43.25%   43.18%   -0.08%
==========================================
  Files          10       10
  Lines        1230     1232       +2
==========================================
  Hits          532      532
- Misses        698      700       +2

Impacted Files
Coverage Δ

micromanager_gui/explore_sample.py
45.09% <27.27%> (-0.45%)
:arrow_down:

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 2a98963...16d27eb. Read the comment docs.


Stop deleting the audio_cache folder

On start up for the Music Bot the audio_cache folder gets deleted then recreated is there a way to make it stop getting deleted?
I'll try to work on this myself but if anyone knows how to please let me know.

Change SaveVideos in the config to yes

Is there a way to do this when SaveVideos in the config is set to no?

This will be changed when I add modes to how this works.  Instead of just on and off, there'll be settings for the max size and deletion strategy.

Ok cool :)


Increase test matrix for smoke tests

This increases the test matrix for the kind smoke tests.

Do not merge until https://github.com/weaveworks/kured/pull/183 has merged.

Rebased, now that 183 has merged.


popup is not keyboard focusable / cannot close with keyboard

After some googling it seems like this is a quirk of WPF, a fix would require making an attached behavior that sets the focus manually. I'll ignore this for now but it's definitely something that will need to be fixed eventually

Agreed. Quirk is one word for it. I have a couple of behaviours that do the trick. Handle the opened event and focus accordingly.

Closing due to inactivity


Fix images in "Guide to creating pizza plots"

Images in plot_pizza() guide are different from the actual output of function. Basic change to pkgdown website to be done at the absolute last before release.

Status on this?

Gonna piggyback fixing the logo in the README in this issue

@abhiamishra I'll do this as the absolute last thing before release to public. Wouldn't want the images to be changed again because of other pr's

Reminder: Remove "distance" as a parameter for plot_pass in "Guide to Version 0.2.0" as it is no longer a parameter in the function

Reminder: Change the guide to pitch plots as the parameter in plot_passnet has been changed

Reminder: Add in the links to the new articles in the "Guide to version 0.2.0"

Changed the parameter names in plot_convexhull and plot_voronoi and removed the "_plot" suffix to the title/subtitle param. Also made the necessary changes to the guides. See issue #98 (closed)

Changed the parameter names in plot_convexhull and plot_voronoi and removed the "_plot" suffix to the title/subtitle param. Also made the necessary changes to the guides. See issue #98 (closed)

Changed the parameter names in plot_convexhull and plot_voronoi and removed the "_plot" suffix to the title/subtitle param. Also made the necessary changes to the guides. See issue #98 (closed)


fix: handle constructors that are static anonymous functions

The MethodObjectBuilder was incorrectly used when a registered constructor is a static anonymous functions — it was handled like a static method closure Class::method(...) and would yield errors like this:
Error: Call to undefined method stdClass::CuyZ\Valinor\Tests\Integration\Mapping\{closure}()

PHP Reflection does not provide any way of telling static functions and closures of static methods apart, other than checking for the name {closure}. We check that {closure} is actually the last part of the fully-qualified name, instead of just checking that the string ends with {closure}.

Thanks @edudobay, good catch and nice PR!
FYI I moved the closure detection inside the function definition (which is kind of an upgraded reflection class).


Translate cond to JavaScript

Will probably end up being represented with if..else if..else statements.

I have a version of without pattern matching support

Closing this as an implementation exists, just not with pattern matching. Pattern Matching needs it's own issue.


[Update] check testcase at cnt60

Purpose
Check correct behavior from waveform.

SIM by ModelSim10.1d.

#24


Tensorflow 2.0's eager_execution for keras is not real.

For normal tensorflow codes, it works well.
However, for tf.keras, it actually performs as a static graph, which is terrible.
An good example is tensorflow 1.12.0, which enables users easily debug keras models.
I just wonder why not inherit the 1.12.0's mode in tf2.0? the former is much better.

Please provide details about what platform you are using (operating system, architecture). Also include your TensorFlow version. Also, did you compile from source or install a binary?
Make sure you also include the exact command if possible to produce the output included in your test case. If you are unclear what to include see the issue template displayed in the Github new issue template.
We ask for this in the issue submission template, because it is really difficult to help without that information. Thanks!


Dynamically created scenes via code share resources although being duplicated

Godot version
4.0.beta1
System information
MacOs 11.5.2, Intel Iris Plus Graphics 655
Issue description
have spent countless of hours trying to apply a texture on a MeshInstance3d within a SubViewport programmatically to no avail.

I have a directory that contains two image files that serve as textures for a MeshInstance3d
This directory is read, the image files are loaded and duplicated (so sharing does not occur)
Each creates a Scene that contains a SubViewport with a MeshInstance3D where the texture is applied

I have two instances of the scene instantiated but both mesh instances always use the material of the latest applied texture, instead of each texture applying seperately.
Script that is reading the directory and instantiating the scenes
@tool
extends BoxContainer

const tiles_path = "res://addons/map_editor/tiles"
const tile_scene = preload("res://addons/map_editor/TileRenderer/TileSubview.tscn")

func _ready():
    var fs = FileSystem.new()

    var files = fs.read_dir(tiles_path).filter(func(f): return ".import" not in f)

    for file in files:
        var texture = load(file).duplicate()
        var scene = tile_scene.instantiate()
        scene.texture = texture
        add_child(scene)

Script for applying textures (the owner node contains the resource as export var and has a getset that calls the set_texture method)
@tool
extends MeshInstance3D

@export var texture: Resource

func set_texture(texture):

    if texture == null:
        set_surface_override_material(0, null)
        return

    var material = StandardMaterial3D.new()
    material.albedo_texture = texture

    var uv_mapper = UvMapper.new()
    var surface_definition = uv_mapper.map_cube_uvs(self.mesh)

    var array_mesh = ArrayMesh.new()
    array_mesh.add_surface_from_arrays(Mesh.PRIMITIVE_TRIANGLES, surface_definition.duplicate())
    mesh = array_mesh

    material.uv1_scale = Vector3(3, 1, 1)
    set_surface_override_material(0, material.duplicate())

Whichever Resource is loaded and applied last, applies it to all scenes. I tried making the resource unique with set_local_to_scene(true) to no avail.
Which leads me to thinking it might be a bug
Steps to reproduce

Open ui_window.tscn within the addons/map_editor directory in the editor
Use "Play Scene"
Observe two MeshInstance3d that share the same resource, namely the last one applied

Minimal reproduction project
godot3d.zip

In your Viewport node named SubViewport in TileSubview.tscn, tick the "Own World" property
What's happening is both meshes are being drawn in the same World (i.e. visual server instance) so they are being drawn one on top of the other in both viewports.
Enabling "own world" gives them different visual spaces to render in.

Thank you so much @lewiji
I can confirm this works flawlessly and thanks for the explanation, it makes total sense. Not a bug and can be closed.


Fix quotation mark when constructing bazel cmds.

Before the change, the bazel cmd constructed is like the following:
bazel build //container/debian8-clang-fully-loaded:bootstrap_rbe_debian8_toolchain --action_env=GIT_ROOT=/my/path '--sandbox_writable_path=/my/path --verbose_failures --sandbox_debug'
This gives error:
I/O exception during sandboxed execution: /my/path--verbose_failures --sandbox_debug (No such file or directory)

It still have some issue, please don't merge yet.

I modified it. Could you take another look?

Thanks for approving! Feel free to merge whenever you think is ready.


update link to latest react docs

name: :book: New/Updated Documentation Content
about: Adding a new docs page, or updating content in an existing docs page
PR Type
Does this PR add a new page, or update an existing page?
updates an existing page
Checklist

[x] Is there an existing issue for this PR?

#4515

[ ] Have the files been linted and formatted?

What docs page is being added or updated?

Section: Redux Terms and Concepts
Page:  here

For Adding New Content
What kind of content category is this page (tutorial, how-to, explanation, reference)?
tutorial -> Redux essentials
Who is the intended target audience?
What knowledge are we assuming they have?
What are the intended results or takeaways from reading this page?
What is the most critical info they should learn?
For Updating Existing Content
What updates should be made to the page?
since React's old docs have been deprecated and aren't recommended, we should update the link to new React docs.
Do these updates change any of the assumptions or target audience? If so, how do they change?

Thanks!

Can't believe my PR is merged into Redux Repo🥳
also @markerikson I want to actively maintain the docs, how can I do that?

@MahendraBishnoi29 yeah, we're always interested in folks helping improve the docs!  There's a lot of open issues tagged docs - generally feel free to look those over, try working on them, and file PRs.


Excessive Metaspace usage when Error Prone enabled

As mentioned in https://github.com/google/error-prone/issues/2786#issuecomment-1003368760, we are planning to move away from forking javac. However, after disabling the fork option, we noticed that the metaspace (off-heap) usage increased significantly (220 MiB -> 1.8 GiB). Please have a look at the graphs below.
I don't have a reproduction case at the moment, but we don't have anything custom, only some command-line options to enable/disable some checks. Maybe it's worth mentioning; we also use NullAway; however, disabling it didn't change the metaspace usage.
Depending on the feedback, I am happy to help reproduce this in an open-source repository. Please let me know.

I run mvn -T 4 clean install -DskipTests and attached jconsole.
Error Prone disabled

Error Prone enabled

(Ex-colleague of @hisener here :upside_down_face:.)
We hit a similar issue in the past with a large build running on Travis CI.  The
+# suitability of these exact parameters was established empirically:
+# - We use a serial rather than a parallel garbage collector because this seems
+#   to perform better in combination with `mvn -T 2` when running inside Travis
+#   CI's two-core build environment.
+# - When Error Prone is enabled, Maven builds with many submodules use a _lot_
+#   of metaspace. This appears to be related to the annotation processor class
+#   path being constructed anew for each module; for `picnic-platform` some
+#   classes are loaded hundreds of times in the metaspace. These are kept alive
+#   using soft references. By default the metaspace is unbounded, preventing
+#   the soft references from being garbage collected. We resolve this by
+#   setting an explicit upper bound; that way a subset of the loaded classes
+#   _is_ garbage collected.
+export MAVEN_OPTS="-verbose:gc -XX:+UseSerialGC -XX:MaxMetaspaceSize=3072m -Xmx3584m"

Later we moved to larger Travis CI machines, and on Mon Jul 13 16:42:42 2020 +0200 the relevant code was updated as follows:
-# The Travis CI build environment is memory constrained, so we configure
-# suitable caps and a garbage collector optimized for throughput. The
-# suitability of these exact parameters was established empirically:
-# - We use a serial rather than a parallel garbage collector because this seems
-#   to perform better in combination with `mvn -T 2` when running inside Travis
-#   CI's two-core build environment.
-# - When Error Prone is enabled, Maven builds with many submodules use a _lot_
-#   of metaspace. This appears to be related to the annotation processor class
-#   path being constructed anew for each module; for `picnic-platform` some
-#   classes are loaded hundreds of times in the metaspace. These are kept alive
-#   using soft references. By default the metaspace is unbounded, preventing
-#   the soft references from being garbage collected. We resolve this by
-#   setting an explicit upper bound; that way a subset of the loaded classes
-#   _is_ garbage collected.
-export MAVEN_OPTS="-verbose:gc -XX:+UseSerialGC -XX:MaxMetaspaceSize=3072m -Xmx3584m"
+# The following configuration properties were established empirically:
+# - The `ParallelGC` garbage collector is configured to optimize for
+#   throughput.
+# - The `Xmx` value is configured to utilize the majority of the available 15GB
+#   of memory, while leaving sufficient space for non-heap memory and other
+#   processes.
+# - The `ReservedCodeCacheSize` value is increased compared to its JDK 11
+#   default value of 240MB to in order to resolve a warning logged during
+#   compilation of large projects ("CodeHeap 'non-profiled nmethods' is full").
+export MAVEN_OPTS="-verbose:gc -XX:+UseParallelGC -Xmx9216m -XX:ReservedCodeCacheSize=512m"

The first analysis is ~2.5 years old, but perhaps it still applies to the latest Error Prone. (I checked: back then we also didn't fork javac.)


Breaking potentialAction in Website type

The schema:
{
    "@context": "http://schema.org",
    "@type": "WebSite",
    "url": "http://example.com/",
    "potentialAction": {
      "@type": "SearchAction",
      "target": "http://example.com/search?&q={query}",
      "query": "required"
    }
}

As specified in https://schema.org/SearchAction doesn't compile. It throws at potentialAction, lots of errors in SearchAction, target, etc. Could you point me to the right direction to fix it? I'd gladly help.
Thanks!
EDIT: Fixed removing target, so I believe the error is from schema-dts. Issue opened there https://github.com/google/schema-dts/issues/45

The workaround is:
      "target": {
        "@type": "EntryPoint",
        "urlTemplate": "http://example.com/search?&q={query}"
      },

instead of just string. will fix in schema-dts.


On ConEmu, Japanese font is not displayed correct shape.

The user says, on the pair nyaos & Conemu, font is displayed correctly.

nyaos uses ANSI-API of Windows. (SAFE)

nyagos users Unicode-API of Windows. (OUT)

ConEmu is unpacked from ConEmuPack.161206.7z

どうやら、折り返すと、フォントが潰れる模様

Also on CMD.EXE, when text is output over the width of screen, fonts are broken.
echo echoがずれる問題日本語fffffffffffffffffffffffffffffffffffffffffffffffffffffffffffff
echo echoがずれる問題日本語fffffffffffffffffffffffffffffffffffffffffffffffffffffffffffff

I wrote the issue on ConEmu site: https://github.com/Maximus5/ConEmu/issues/1103

We should open Setting ,Main and disable Compress long string to fit space, so the problem disappears.


separate tools dependencies from server dependencies

Currently the dependencies for the tools are put into the same directory as those for the CIMI server.  This bloats the classpath for the CIMI server and brings in lots of dependencies that aren't needed.
Separate the dependencies and change the classpath for ssconfig to include the new directory with only the tools dependencies.

Turns out that this is impossible because the dependency plugin does not work as advertised.  If an artifact is excluded in the configuration, then it is included anyway.  If the artifact is explicitly included in another execution, then the transitive dependencies are ignored.  The only way this can be achieved is by creating a separate module (and packaging) for the tools.

Not implemented.


ci: separate benchmarks

What does this PR do?
Separate running benchmarks from unit tests as they turned out to be flaky/sensitive on the actual machine and eventually concurrently running jobs
lets also use upstreamed RunIf via #17504

Before submitting

Was this discussed/agreed via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
Did you make sure to update the documentation with your changes? (if necessary)
Did you write any new necessary tests? (not for typos and docs)
[x] Did you verify new and existing tests pass locally with your changes?
Did you list all the breaking changes introduced by this pull request?
Did you update the CHANGELOG? (not for typos, docs, test updates, or minor internal changes/refactors)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing, make sure you have read the review guidelines. In short, see the following bullet-list:

Reviewer checklist

[ ] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified

cc @carmocca @borda

@lantiga lets get this merge and address the loss difference in another PR
as benchmarks are not required tests (they even were not) so it won't block any dev. 🐿


Tests and artifacts misorganized

In the soopervisor/src/soopervisor/assets/
argo should have it's own folder: argo-workflow.yaml
Same thing happens in the tests folder for argo and airflow.
For both we need to move the files into the right location, make sure the path it's consumed from is updated in the code and check the tests are still passing.

fixed in ddb9f4e3185dfd401ff9526046dace00245b5cd0


Unescape VARBINARY values

Seems like Vertica return VARBINARY data in escaped format (as in vsql):
ci=> SELECT HEX_TO_BINARY('d09fd180d0b8d0b2d0b5d1822c2068656c6c6f21');
                      HEX_TO_BINARY
----------------------------------------------------------
 \320\237\321\200\320\270\320\262\320\265\321\202, hello!
(1 row)

This is mean that you should unescape it to receive raw/original binary.
I think escaped string should contain only oct and visible ASCII characters as described here.
So, this is RP unescape binary value (i.e. convert escaped oct to characters and leave other characters).
@wvanbergen What do you think?

FYI: Seems like 2 CI configurations is failed on bundler install. One was passed with success.

@wvanbergen Could you please review this PR?

This is fixed now. I used a somewhat simplified approach in #42


CD Version 4 Tracking Issue

Tracking issue for version 4.0 of Component Detection. This is a list of changes to make before shipping breaking changes.
### Tasks
- [ ] #233
- [ ] #452
- [ ] Switch to .NET 8 RC
- [ ] #251
- [ ] https://github.com/microsoft/component-detection/issues/772

AB#2102422

Some additional context for .NET 8 Release Candidate's support, from the .NET and .NET Core Support Policy

Go-live releases are supported by Microsoft in production. These are typically our release candidate builds, just before the generally available (GA) release.

Closing as v4 was already released - #1135 will handle tracking for v5.


V0.12.1.x voting fix

Disabled flatfile dat checking on write, something is weird with the code and it's causing the data to be overwritten. I kept getting the same file that was loaded, instead of what was in memory until I disabled it.
Fixed various issues with voting and vote storage

wrong branch!


Fix - Move methods from onHilighted after scroll detection

Fixing scroll issues by moving methods call by Element.onHighlighted after the scroll detection.
This fix my issue with an element at bottom of a scroll container but it's seems to also fix #112
#112 codepen fork with PR

I'm still working on a way to make it work with scrollIntoView behavior set to 'smooth'.

Has there been any updates on this?
I'd still vouch for merging in even if it still doesn't work with "smooth" scrolling; it'll at least work somewhat that way

I am sorry for the delay – I got really busy and wasn't able to get back to it. Thank you!


feat: store packing

Added a chapter on storage packing for efficient storage optimisation
Closes #324

Thanks @glihm !

TYVM for the reviews @milancermak @glihm !


[Sysman] update documentation of  zesSchedulerSetComputeUnitDebugMode

https://spec.oneapi.io/level-zero/latest/sysman/api.html#scheduler-functions  returns handles for each tile level   but the ** zesSchedulerSetComputeUnitDebugMode **   will work at device level as it is global function but documentation doesn't say it. Hence if user try to set for 1 tile, it will affect to other tile also which could be concern. Hence requested to update documentation to reflect the same

@jandres742   @wdamon-intel   please make sure this should be part of spec 1.7 change

based on offline discussion, current recommendation is to carry documentation regarding this in driver repo so that behavior is clearly communicated to user and not modify docs at spec level

@jandres742 please look

This is being addressed with #218, which will deprecate this API.


Samples are using out-of-date information

Operating system: ✍ TODO
Last upstream commit (run git log --author="Robert Lord" | head -n 1): ✍ TODO
Browser version(s): ✍ TODO
Ruby version (run ruby -v):  ✍ TODO

✍ TODO write your issue here
Ref. IBM Forum link
It appears that the samples are using out-of-date values when using the REST API.  In the case the REST GET request (link) the code being used is
Public Property Get OData(Server As String) As String     'OData endpoint     OData = HubEndpoint + "server('" + Server + "')/api/v1" End Property
but as per the referred link on the forums it should use...
Public Property Get OData(Server As String) As String     'OData endpoint     OData = "tm1/" + Server + "/api/v1" End Property
Changing the value for OData resulted in a connection being established.

Document updated
https://ibm.github.io/paxapi/#rest-api
Please check it out and see if it looks good.

https://ibm.github.io/paxapi/#rest-api
Document has been updated. Please check it out and see if it looks fine


Tooltip stuck on hover

Issue happens in latest code. While debugging found out that the "handleLeave" methods doesn't get called.
Tested in codesandbox as well with the link from doc. it only happens occasionally.

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯

Hover on disabled content of tooltip (button) then hovering out. Tooltip is stuck and still shown.
Expected Behavior 🤔
Hover on disabled content of tooltip (button) then hovering out. Tooltip is shown be hidden.

Steps to Reproduce 🕹

https://codesandbox.io/s/usage-4t0vk
Steps:

hover on button
leave hover on button
repeat 1,2 until tooltip will eventually get stuck

Context 🔦

Your Environment 🌎

Tech
Version

Material-UI
v4.3.1

React
16.8.1

Browser
chrome 80

TypeScript
-

etc.
macOS Catalina

I wasn't able to reproduce this - could you perhaps point to where you would expect handleLeave to be called?

It's a question covered in https://material-ui.com/components/tooltips/#disabled-elements.

It would be good if that section explained why pointerEvents: "none" is needed in this instance, but it does fix the issue.

No idea why.


paper_trade should delete orders for market on close/clear

I save stats for a market in process_closed_market which works fine for backtest and live. In paper trade however many of the bets are doubled up. I think this is due to the blotter never being cleared and BF reopening and closing the market - sometimes even a few hours later - causing the closed market function to be called again. Happens a lot on UK dogs. Needs a safe / proper way to wipe the blotter for a market  after saving the stats. Maybe the wipe should be done when BF clear the market rather than on a close since I'm sure they sometimes close a market inadvertently and then open it again ?

This is actually intentional, BF will sometimes open/close a market to change a result in which case you want to update your results.
For example in my loggingcontrol I update any orders if there are further closed market events.

Sorry, my bad - looking closer I can see status /  bsp changes are made so stats do need to change. Seems I even get the dupes on backtest but never picked it up before.


Changes to Classification (Choice) and Confidence

From: TA2
Priority: Eval 4
Components: Python API (make_step_prediction, end_scene)
Blockers: Needs design (TA2) and approval (TA1)
Issues:

The term "confidence" has a specific meaning in machine learning, which has lead to some confusion
We may want to encourage teams to provide a confidence value rather than rely on a default value
We may want to provide feedback (like an error message) if an incorrectly-formatted confidence value is passed
We want the API to support both VoE and future non-binary classification tasks in a clear way

Possible Solutions:

Rename confidence to score (or plausibility_score, or continuous_score), to mimic the terminology used in the Eval 4 Plan
Rename choice to rating (or plausibility_rating, or binary_rating), to mimic the terminology used in the Eval 4 Plan
Update the documentation in GoalMetadata to describe the expectations for intuitive physics versus agents as detailed in the Eval 4 Plan
Reject function calls with incorrectly-formatted data based on the goal category

TA2 internal JIRA ticket: https://nextcentury.atlassian.net/browse/MCS-513

https://machinecommonsense.slack.com/archives/CTR31V12M/p1615651333007600
Ben Zinberg Yesterday at 11:02 AM
Confirming that the confidence kwarg to Controller.end_scene has the meaning of 0 = sure implausible, 1 = sure plausible, 0.5 = totally uncertain (and continuous partial uncertainty in between)?  (If so, perhaps we should rename that kwarg to something like soft_choice) (edited)

Related to: https://github.com/NextCenturyCorporation/MCS/issues/263

Resolved in the previously linked PR + included in the 0.4.4 release.