x/vgo: vgo mod support for -f and .Dir in -json output

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
go version go1.10.3 linux/amd64
vgo commit 0f3e556044ecbd8a0805d098e52a743a1d0e3566

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/myitcv/.cache/go-build"
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/tmp/tmp.7KXsauqfrA"
GORACE=""
GOROOT="/home/myitcv/gos"
GOTMPDIR=""
GOTOOLDIR="/home/myitcv/gos/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build223819470=/tmp/go-build -gno-record-gcc-switches"

What did you do?
cd `mktemp -d`
export GOPATH=$PWD
mkdir hello
cd hello
vgo mod -init -module example.com/hello
vgo mod -json

What did you expect to see?
{
        "Module": {
                "Path": "example.com/hello",
                "Version": "",
                "Dir": "/tmp/tmp.7KXsauqfrA/hello"
        },
        "Require": null,
        "Exclude": null,
        "Replace": null
}

What did you see instead?
{
        "Module": {
                "Path": "example.com/hello",
                "Version": ""
        },
        "Require": null,
        "Exclude": null,
        "Replace": null
}

Having .Dir be part of the output of vgo mod -json would allow other tools to use vgo mod to resolve the containing directory of a module (context). Correspondingly, having a -f flag (equivalent of (v)go list -f) would be useful to programatically extract values returned from -json, including .Module.Dir.
/cc @rsc @bcmills

Does the list -m -json and list -m -f support in https://golang.org/cl/120198 do what you need?

I don't think so - that only gives me (building on the example above):
$ vgo list -m -json
{
        "Path": "example.com/hello",
        "Main": true
}

which looks to be a "subset" of the output from vgo mod -json?


How to configure tabs slide transition?

When I set new index to state of tabview (i.e. switch tabs programatically, from a button click) tabs switch too fast for me, I'd like to have a slower transition. Docs say there's configureTransition callback, but don't say how should it be written and how does that 'transition configuration' that it should return should look like. Could you please, pour some light on it and update the docs?

Hi! You can use configureTransition as following:
  configureTransition={() => ({
    timing: Animated.spring,
    tension: 300,
    friction: 35,
  })}

If it can help you out, you also have TransitionConfigurator's type with Flow and you can check out how the callback is used under the hood right here :)

@CharlesMangwa does this still work as described? For me the configureTransition prop has no effect. It doesn't get executed at all.


Add default value 1 for the rowPixelSpacing and columnPixelSpacing

Please check if the PR fulfills these requirements

[ ] The commit message follows our guidelines
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

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

What is the current behavior? (You can also link to an open issue here)
When there are no pixel spacing data of the image, the values of the angle measurements are not able to be calculated, since the getPixelSpacing function returns null for the rowPixelSpacing and columnPixelSpacing values.

What is the new behavior (if this is a feature change)?
getPixelSpacing function returns default 1 if there is no pixel spacing data.

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
No.

Other information:
Default value 1 is added since in the past every tool had it, but due to some refactoring (in which the getPixelSpacing function was created), this scenario was left out.

This is actually dangerous to do. There's some discussion in this repository, and in the cornerstone-wado-image-loader repository that discuss this.


Add module execution entrypoint

Codecov is grumpy that all of the new lines added by this PR are untested. The code is trivial and I'd rather have an accurate coverage percentage instead of excluding specific lines.


Fix vulnerability update device text

Description
Fixed text from device to system on the system vulnerability tab
Fixes # (issue)
THEEDGE-2507
Type of change
What is it?

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Documentation update
[ ] Tests update

Checklist:

[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I run npm run lint:js:fix to check that my code is properly formatted

Codecov Report

Merging #522 (c44fde3) into master (6fee814) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head c44fde3 differs from pull request most recent head 4188fcb. Consider uploading reports for the commit 4188fcb to get more accurate results

@@           Coverage Diff           @@
##           master     #522   +/-   ##
=======================================
  Coverage   31.07%   31.07%
=======================================
  Files         124      124
  Lines        2616     2616
  Branches      798      798
=======================================
  Hits          813      813
  Misses       1654     1654
  Partials      149      149

Impacted Files
Coverage Δ

src/Routes/DeviceDetail/Vulnerability.js
0.00% <ø> (ø)

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


[grpc] Update to v1.32.0 - use correct port version

Describe the pull request

Fixes PR #14111 so grpc port has correct version number
Patch files updated to match grpc v1.32.0 source code

Thanks for your contribution!


卡在cosine schedule这里不动了

你好，程序应该还是在跑，日志输出是单独保存在log文件夹里面的。

Hi I have the same issues and log file show ieltration but IT dosent move

You mean on GPU??

Yep, I mean the remote GPU server. If you want to train it on the colab, maybe you can explicitly print the loss by changing the logger.info(...) to print(...) here.

HI
sorry to interrupt you but I have a problem with running
your diffusion code. Does this log file mean it is running ?? but log stock
in 400 and do not move.
On Sat, May 4, 2024 at 5:24 PM balabala @.***> wrote:

Yep, I mean the remote GPU server. If you want to train it on the colab,
maybe you can explicitly print the loss by changing the logger.info(...)
to print(...) here
https://github.com/Algolzw/image-restoration-sde/blob/2598d73cde3a1e2cf78c4ba4923414e1f4b90393/codes/config/deraining/train.py#L258
.
—
Reply to this email directly, view it on GitHub
https://github.com/Algolzw/image-restoration-sde/issues/75#issuecomment-2094209655,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALIF3BF5ID3G5P6MLJ4A7EDZATSDHAVCNFSM6AAAAABDCU4MCSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJUGIYDSNRVGU
.
You are receiving this because you commented.Message ID:
@.***>

Hi! Have you solved this issue? You can check the GPU with nvidia-smi to see if the program is still running.


Convert links from http to //

This allows the page to load both HTTP and HTTPS instead of preventing CSS from rendering on HTTPS sites

Thanks!


Query to get current panel brightness

Hi @mrfaptastic ,
i have one queries to get current panel brightness function. by having this. we could get current panel brightness so attempt to update the brightness can only be done only if the current brightness value are different than the panel brightness else every time the loop run setBrightness8(value) will always get executed.
is it ok to update the panel brightness with the same value like in every 5 second loops? any impact?
thank you

Hi @tsctrl. I suggest having a variable within your code to keep track of the last value set to the library.
Trial and error is the best way to see if frequent changing of brightness values causes issues.


Create an endpoint to return a list of dandisets

This would be needed to support #93 .
At the current time, this should be a list of folders in the drafts collection.

@mgrauer Is the intention that eventually this will return a list of folders which spans multiple collections (but just happens to be limited to drafts right now)?

exactly.


Select Python version on OSX Travis Jobs using pyenv

Using pyenv it is possible to select a Python Version on OSX. The drawback is, that the installation requires roughly 3 additional minutes, however, it can be cached. I thought about using the same build matrix for OSX as for Linux, but I did not because the build time of OSX jobs is a lot slower and it seems as there are only two concurrent jobs active. Thus, it might be good if there are fewer OSX jobs running.
The Python version must be one listed in "pyenv install --list". I noticed that different versions of Xcode support different Python versions. Thus, if a newer Python version needs to be supported, a newer Xcode image might also be needed. I choose Xcode9.4, as it was faster compared to newer images (in a single run, so this might be arbitrary).
Furthermore, this PR syncs travis with the recent changes in maint-1.8 (rstcheck currently disabled as tests would fail) and uses SVG for the travis / coveralls / appveyor badges (I'm not sure why it irritates me that they currently do not have the same height...).
Additionally, a Python 3.9-dev job is added, which is marked as allowed to fail. A test fails due to botocore, which is currently not compatible with Python 3.9, as it imports xml.etree.cElementTree, which was removed in Python 3.9.

Coverage decreased (-1.6%) to 85.996% when pulling 5215bc3fbff5e06ee88b85dbe05db9c2efafc571 on rbuffat:update_travis_master into 7f6f9dc3e466036ca196138efc49fc3c281c59b1 on Toblerity:master.

Coverage decreased (-1.6%) to 85.996% when pulling 5215bc3fbff5e06ee88b85dbe05db9c2efafc571 on rbuffat:update_travis_master into 7f6f9dc3e466036ca196138efc49fc3c281c59b1 on Toblerity:master.

I added a configuration for Github CI for OSX builds. It was a bit more complicated than I expected. There are two big differences between travis and github CI. With Github, each step is run in a different process. Sharing environment variables between steps requires thus a workaround. Furthermore, the caching mechanism is slightly different.
While working on this I notices a bug I introduced with the OSX builds on travis. The issue is, that as far I see it is not possible to pin a specific version of homebrew packages. Thus, always the most recent packages are installed. This will lead eventually to problems if the homebrew packages are incompatible with cached gdal / proj binaries (shared libraries such as sqlite, goes, spatiallite, ...).
With the Github CI a key is defined, with them the cache is identified:
brew list --versions | grep 'expat\|libspatialite\|sqlite\|geos\|curl\|libxml2\|freexl\|zstd' > $GITHUB_WORKSPACE/brew_list

      - name: Dependency Cache
        uses: actions/cache@v2
        with:
          path: gdalinstall
          key: ${{ runner.os }}-gdal-${{ matrix.config.GDALVERSION }}-proj-${{ matrix.config.PROJVERSION }}-${{ hashFiles('**/brew_list')}}

Adding ${{ hashFiles('**/brew_list')}} to the cache key should invalidate the cache when a version of a homebrew package changed. I tried to filter packages that are used by gdal, I hope I caught all relevant ones.
I'm not yet sure what has to be done to activate the github ci here. In my repository, it seemed to be enough to add the yml file in .github/workflows.

Travis is obsolete now.


Add method name suffix to the API endpoint url

I.e. /api/v1/proxy for wallet balance method should be /api/v1/proxy/wallet_balance. This would help identifying calls in browser console.

Huge fan of this one!

done on the JS side:
https://github.com/lbryio/lbry-redux/pull/286


Creating my own Index

Hi! I see there are some precomputed indices, but no instructions or code to support creating your own. I can see on huggingface that the provided Wikipedia index is in the form of an .IDX file. Is there any documentation on how to create one? If the code used to create the wikipedia index could be shared that would be very helpful.
Thank you!

It's mostly an IVF-PQ index in FAISS


Correct use of {{ Hugo.Generator }}

Thank you for including the {{ Hugo.Generator }} tag!  :+1:
However, as you may not know, {{ Hugo.Generator }} already includes the entire meta tag, not just the program name and version, so here is a PR to fix that.  :-)

Thanks @anthonyfok  for letting me know and of course your pull request. I'll merge PRs as well.


Removed dependencies related with deleting Person model

Closes #47
Some of the dependencies include:

TestCases

Logic/Commands

AddressBook

@HemanshuGandhi please help review this too when you have time! There are some UI changes involved in this PR

Thanks you @firzanarmani  and @kevinputera , i've made the necessary changes, removed all dependencies related to Person class. Please let me know if there are anymore problems.

@ljiazh3ng Your latest commit message is not in imperative form. Also, the intention of the commit is to point out the change, rather than the build passing (what if it didn't, for some reason). I would suggest renaming it to something like "Comment out broken tests to fix build test"
@kevinputera I think these changes are enough to move forward with passing build checks and proceed with the larger refactoring.
Though, as previously discussed, at first we thought of commenting out methods that broke because of the removal of Person and its dependencies. However, leaving it as returning void or Object for certain methods save us some time for now, and allow us to just carry on with the larger task of refactoring everything else. What's your opinion?

@ljiazh3ng Your latest commit message is not in imperative form. Also, the intention of the commit is to point out the change, rather than the build passing (what if it didn't, for some reason). I would suggest renaming it to something like "Comment out broken tests to fix build test"
@kevinputera I think these changes are enough to move forward with passing build checks and proceed with the larger refactoring.
Though, as previously discussed, at first we thought of commenting out methods that broke because of the removal of Person and its dependencies. However, leaving it as returning void or Object for certain methods save us some time for now, and allow us to just carry on with the larger task of refactoring everything else. What's your opinion?

Hey @ljiazh3ng @firzanarmani thanks a lot for this! Yes, I agree with y'all, let's just stick with the bare minimum change needed to allow others to test out their parts.
However, as it turns out, we cannot run the application now (./gradlew run) due to some NPE. Let me take a look at this and get back to you

Hey @ljiazh3ng @firzanarmani! I pushed a commit where I removed the PersonList and PersonCard views from our app's UI. This should do the work for now. Do try ./gradlew run now, it should work!
Let's merge this in once you all take a look!

@HemanshuGandhi Hey Hemanshu! Could you help us check this PR out and try running it locally too? See if everything's fine from your side. In short, from UI's perspective, the followings are removed:

The PersonCardList fxml and java class
The PersonListPanel fxml and java class

@kevinputera I have tested the pr locally, and on my end, the build is working. No checkstyle errors or any runtime errors.


Bump Sunodo SDK to 0.4.0

Closes #71

Running sunodo build currently raises the following error:
id: unknown user dapp
chown: unknown group
id: unknown user dapp
chown: unknown group
su: unknown user dapp

Halted with payload: 1
Cycles: 43872128
43872128: b9f13b6df3b8868c9b8f921cf4cfc937f64de8ee0e8148815c59796aa74574c6
Storing machine: please wait
    Error: Command failed with exit code 2: docker container exec 17ad6da549d06d490c66408c132cb89c1a556fbc9e4ecb41463e385d71df169e cartesi-machine --assert-rolling-template --ram-length=128Mi
    --flash-drive=label:root,filename:/tmp/.sunodo/image.ext2 --final-hash --store=/tmp/.sunodo/image --append-bootargs=no4lvl --append-init=WORKDIR=/opt/cartesi/dapp --append-entrypoint=export
    PATH=/opt/cartesi/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin --append-entrypoint=rollup-init /opt/cartesi/dapp/dapp

Here is the output of sunodo --version:
@sunodo/cli/0.12.0 linux-x64 node-v20.12.0

I've created #82 for us to fix the Lua tests later.

I'm getting this erro when trying to send a bounty via cli.
❯ go run ./cli send bounty \
    -n "Lua Bounty" \
    -d "Description of Lua bounty" \
    -c ./tests/bounties/lua-bounty/lua-5.4.3-bounty_riscv64.tar.xz
INF fork/exec /usr/local/bin/cast: argument list too long
exit status 1

This is a known issue with the CLI.
With the migration from EggRoll to Rollmelette, we unfortunately don't have suport to a low-level send transaction feature. So, instead of using go-ethereum indirectly, we are (for the time being) calling cast from Go.
We plan to have a Typescript CLI in the future which will use viem to perform these kinds of L1 transactions. You can see there is now a notice in the README stating that the CLI and, therefore, the populate script, are unstable for now. Tests should be done manually through the web front-end for now.
See #76 and #79

Thanks a bunch, guys! :pray:


After forking

SDMM

[ ] Update docker-compose.yml to expose port 5000 on Portal: ports: - 5000:${BACKEND_PORT}
[ ] Get rid of EdXClassPortal and related references (send requests to classy and handle things there)

310 & SDMM
.env
## [SDMM/310 Only] Port that the geo-location service should listen on
## MUST BE SET TO 11316 (this is baked into the service's dockerfile)
GEO_PORT=11316

## [SDMM/310 Only] Port that the reference UI service should listen on.
## MUST BE SET TO 11315 (this is baked into the service's dockerfile)
UI_PORT=11315

## Specifies the mode under which the reference implementation should operate.
## Currently supports values 'd1' and 'd2'.
## Affects the reference UI and the grading container.
PLATFORM=d1

## GitHub token with read access to clone repositories in the org for the particular course offering
COURSE_GH_ORG_TOKEN=asb865...

These have been resolved through the passage of time.


TeamCity support

Hi, I'd like to add TeamCity Test reporting support for Cavy, and am trying to figure out the most reasonable approach.
TeamCity actually supports specially-formatted console output messages that it can translate into its own rich test reporting, so the simplest approach would be to add an option for cavy-cli to switch to that format instead of the default human-readable output. Would it bother you for cavy-cli to have specialized knowledge of a specific CI system?
The option with minimal modification to Cavy would be to use the existing "custom reporting," however in this case we want pretty much everything the default reporter/server already does, just with tweaked output formatting, so that'd be a lot of duplication and managing another express server... (also I'm not sure how compatible cavy-cli is with this... since it won't exit until it receives a report... so the custom reporter would also have to report to cavy-cli...?)
I also see #18, and am wondering if there was a specific plan there? In addition to the special console logging, TeamCity also supports some report formats produced by other testing tools, listed here: https://www.jetbrains.com/help/teamcity/xml-report-processing.html . If you were already planning on using one of those formats for other CI systems, perhaps that would achieve multiple goals, but I'm not sure if that was the plan or how involved it would be to produce one of those reports.
It would be nice if cavy-cli could somehow support custom output formatter plugins, but that may not be feasible or useful outside this particular scenario.
Do you folks have any thoughts on this?

Hi @TheAlmightyBob - this sounds really interesting, and definitely in line with some of our internal ramblings surrounding #18.
Let me chat to the team about possible options and I'll get back to you in the next couple of days.
p.s. thanks for supporting Cavy! ❤

Just in case you were curious, this is the special TeamCity console output format: https://www.jetbrains.com/help/teamcity/build-script-interaction-with-teamcity.html#BuildScriptInteractionwithTeamCity-ReportingTests
(One thing that'd generally help from the core Cavy side would be structured report data rather than (or in addition to) just a combined "message" property... but I expect that's a minor enhancement and not really a blocker here... but probably would also be wanted for #18)

(I'll also note that the console-logging route would be more compatible with #15 , as TeamCity does support that through this mechanism... but #15 seems like a nice-to-have)

Hi @TheAlmightyBob - just to let you know, we've just been pretty stacked this last week at Pixie Labs, but this is still on my radar!

@AbigailMcP Understood. In the meantime I've gone ahead and patched cavy-cli to optionally output TeamCity service messages. It's working, but we'd prefer to have a solution that others could use and doesn't require patching.

Hey @TheAlmightyBob ! We've had a think about this and propose that, for now, Cavy CLI should ship with some in-built formatters, in particular JSON and XML.
Later on, we might think about supporting format plugins, or custom formatters, simple to how we support customer reporters now.
How does that sound from your end? I'll use the JUnit spec for constructing the XML file.

@AbigailMcP That sounds like it should work. I'm not previously familiar with JUnit, but the XSD seems to support the required concepts.
I assume there's no point submitting a PR for the --teamcity flag that we already implemented?

Hi @TheAlmightyBob! Realise this was a while in the making... but we've just released new versions of cavy and cavy-cli that support outputting tests to a JUnit spec XML file! 🎉
This should mean that TeamCity can support Cavy tests :)
Will close this issue for now, but if would love to hear any feedback you have on the new functionality. Thanks!

@AbigailMcP Thanks for notifying me! Finally had a chance to look at this.
It looks pretty promising (and the addition of individual test durations is appreciated!). There's one quirk, which is that the results get duplicated in TeamCity:

I'm curious what system(s) you tested this with and what the expected result was of having the same content in the "message" attribute and the element body?
All the info is present at least, but this isn't great for readability. I would love to use just the "message" attribute content but I don't think I can tell TeamCity to ignore the other data...

Thanks for reviewing!
We based this mainly on the XML we see produced by the rspec-junit-formatter gem, and this JUnit Schema.
The schema says:

a <Failure> "Contains as a text node relevant data for the failure, e.g., a stack trace", and
its message attribute should be "The message specified in the assert"

An example rspec-junit-formatter output would be:
<failure message="Expected to find text "EmployeeList.JimCavy" >

  Failure/Error: expect(page).to have_test "EmployeeList.JimCavy"
  Expected to find text "EmployeeList.JimCavy"
  ./spec/features/employee_list_spec.rb:5:in `block (3 levels);

</failure>

In a similar fashion, an example cavy xml output would be:
<failure message="Could not find component with identifier EmployeeList.JimCavy" >

  Listing the employees: filters the list by search input  ❌
  Could not find component with identifier EmployeeList.JimCavy

</failure>

In both cases, the message is what failed in the assert and the body repeats the failure, but with extra context.
In general, I found it kinda tough to find a consensus for what should be included in each attribute to suit all needs! What would you guys expect to see in both the message and body of a failure?

Thanks for the explanation.

You're repeating the test name in the body, in addition to the error. I don't interpret either the schema or the rspec example as expecting that. That info is already captured by the parent node.
Your example includes blank lines around the body, but that's not what I see in the Cavy XML output, which is part of what makes it hard to read

<failure message="expect(received).toBe(expected) // Object.is equality

Expected: "expected"
Received: "Test text to write to the file"" type="cavy test failure">The file system: can write and read a file  ❌
   expect(received).toBe(expected) // Object.is equality

Expected: "expected"
Received: "Test text to write to the file"</failure>

I realize one difference is that I'm directly using expect rather than Cavy's UI testing functions, so perhaps having a multi-line message with "expected"/"received" is different from what you're used to seeing.
Given that, what I would expect to see (and how I had previously set this up via patching) would be for the message to contain expect(received).toBe(expected) // Object.is equality and for the body to contain the expected/received lines.
Alternatively, since Cavy isn't actually producing stack traces... you could perhaps just leave the body empty and just use the message attribute? TeamCity at least appears to be handling that okay, although I don't know if other systems would expect the message to be single-line


[Optimization] Reduce bundle size

Initial conversation: https://warpcast.com/timdaub/0x1905b4
https://www.framer.com/motion/guide-reduce-bundle-size/
npm ls framer-motion
<EMAIL_ADDRESS>/Users/timdaub/Projects/attestate/kiwistand/src/web
└─┬<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>and so I looked into connectkit's usage of framer motion and saw some optimization potential (e.g. using "m" instead of "motion". Are you interested in me applying the changes from that blog post?
https://github.com/search?q=repo%3Afamily%2Fconnectkit framer-motion&type=code

Hey @TimDaub!
You're welcome to give optimisations a shot and we can give it a look, but worth mentioning that I've spent some time previously trying to migrate motion to m but this seemed to only save us about 20-30kbs, and it also caused some animation issues.
Worth noting that we eventually would like to look into removing framer-motion in the future to lower the file size of ConnectKit, and there's also a few other optimisations that could be done with SVGs and some minor refactoring that we have in our internal to-do list.


[SPARK-29065][SQL][TEST] Extend EXTRACT benchmark

What changes were proposed in this pull request?
In the PR, I propose to extend ExtractBenchmark and add new ones for:

EXTRACT and DATE as input column
the DATE_PART function and DATE/TIMESTAMP input column

Proposed benchmarks for the DATE type requires casting longs to dates. I extended the CAST expression to support casting integral types to dates. CAST interprets the input as number of days since epoch 1970-01-01 which can be a negative number.
Why are the changes needed?
The EXTRACT expression is rebased on the DATE_PART expression by the PR https://github.com/apache/spark/pull/25410 where some of sub-expressions take DATE column as the input (Millennium, Year and etc.) but others require TIMESTAMP column (Hour, Minute). Separate benchmarks for DATE should exclude overhead of implicit conversions DATE <-> TIMESTAMP.
Does this PR introduce any user-facing change?
After the changes, users can convert long, int, short and byte columns to dates:
spark-sql> select cast(1 as date);
1970-01-02

but before the same sql statement fails with the error:
spark-sql> select cast(1 as date);
Error in query: cannot resolve 'CAST(1 AS DATE)' due to data type mismatch: cannot cast int to date; line 1 pos 7;
'Project [unresolvedalias(cast(1 as date), None)]
+- OneRowRelation

How was this patch tested?

Added new tests to CastSuite to check casting integral types to dates
Regenerated results of ExtractBenchmark

@dongjoon-hyun @cloud-fan @maropu Could you take a look at the PR. Should I extract cast-related code to a separate PR? WDYT?

Test build #110502 has started for PR 25772 at commit 6b9ad66.

Can one of the admins verify this patch?

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/testing-k8s-prb-make-spark-distribution-unified/15479/
Test PASSed.

Ur, I see. I didn't notice we've already accepted explicit casts from long to timestamp though, any reason to support this explicit cast? It seems pgSQL doesn't accept that ;
postgres=# select cast(0::bigint as timestamp);
ERROR:  cannot cast type bigint to timestamp without time zone
LINE 1: select cast(0::bigint as timestamp);
               ^
postgres=# select cast(0::int as date);
ERROR:  cannot cast type integer to date
LINE 1: select cast(0::int as date);
               ^

cc: @gengliangwang

Oops. I'm late. :)

hahaha, I was faster

I think long to timestamp is a legacy behavior and we shouldn't go further in this direction. If we do want to convert integrals to timestamp in the benchmark, we can create a UDF/expression to do this in the test package, and use it in the benchmark.


Add FIFO topic

Description

Added fifo_topic option
Motivation and Context

Needed to create a FIFO topic
Breaking Changes

How Has This Been Tested?

[ ] I have tested and validated these changes using one or more of the provided examples/* projects

Created the topics with the changes

Thank you!
v3.1.0 has been just released.


Annotation views are not relocated when annotation coordinate is changed

Annotation views are not relocated when annotation coordinate is changed. Apparently that's missing functionality versus MKMapView. I need to update .coordinate of some of my annotations, and the update is not reflected on the map if the annotations were added via cluster controller - I'm sure that .coordinate property is not even observed in such a case. If I switch to plain MKMapView, it works as expected: annotation views are (automatically) moved when I change .coordinate of annotations.

Closing this for now. Please reopen if there are new ideas on how to implement this. Thanks


WEBAPP-524: Update readme

This pull request belongs to an issue on our bugtracker.
You can find it there by looking for an issue with the key which is mentioned in the title of this pull request.
It starts with the keyword WEBAPP.

Let me know if anything is missing. Specifically @Taggotty


TechDebt: MyPy AWSLambda

Codecov Report

Merging #5586 (6b86c6e) into master (14c6550) will increase coverage by 0.00%.
The diff coverage is 99.35%.

@@           Coverage Diff           @@
##           master    #5586   +/-   ##
=======================================
  Coverage   96.18%   96.18%
=======================================
  Files         742      742
  Lines       72612    72608    -4
=======================================
  Hits        69839    69839
+ Misses       2773     2769    -4

Flag
Coverage Δ

servertests
36.91% <61.53%> (+<0.01%)
:arrow_up:

unittests
96.12% <99.35%> (+<0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

moto/awslambda/models.py
94.23% <98.97%> (+0.15%)
:arrow_up:

moto/appsync/models.py
97.36% <100.00%> (ø)

moto/awslambda/exceptions.py
97.43% <100.00%> (ø)

moto/awslambda/policy.py
95.45% <100.00%> (+0.10%)
:arrow_up:

moto/awslambda/responses.py
93.88% <100.00%> (ø)

moto/awslambda/utils.py
100.00% <100.00%> (ø)

moto/core/utils.py
94.29% <100.00%> (ø)

moto/iam/models.py
95.87% <100.00%> (+0.12%)
:arrow_up:

moto/utilities/docker_utilities.py
100.00% <100.00%> (ø)

moto/ses/models.py
96.95% <0.00%> (ø)

... and 1 more

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


Course name / Publish course name misunderstanding

Currently the project name and publish course name are 100% linked. The way it should work is this:

Project name needs to be set. Should show error if it's not set. It should default on new project to something like "Untitled Project". If the user tries to clear the name in the header, it should revert to the previous value. We just can't accept a blank name.

This will also help with closing an unnamed project, since a name is guaranteed.

In the publish panel, the course name should default to blank. The placeholder (not value... the placeholder text) of the input should be the project name.

If the user puts a value in the course name, it overrides the name in the published course.

If the user puts a value in course name, and publishes, and then goes back and and clears the value from there, is shows blank with the project name as the placeholder again. On re-publish, it would use the project name as the value.

This is important so that people can use internal naming for the project, but have a friendly name when publishing.

This doesn't work properly.
I open the publish pane, I change the course name, hit publish, open publish panel again, and the course name is blank again.


Iris animate docs

I have amended the documentation for the iris animate tool to make it explicitly clear which functions it will accept.

I've rebased this branch off of the current Iris and it now seems to pass all of the tests. @ajdawson and @pp-mo can you see if all of your previous concerns have been addressed?

LGTM. Thanks @LukeC92.


Command 'move' or 'copy'

How to transfer a backup repository from a directory to S3 or another type of repository?
Can you make a new command, for example "move", to be able to completely transfer or copy the repository to another location?

Please ask usage question in the forum.
You can either directly copy the files of a repository to another location (e.g. using rclone, the storage format is identical for each backend), or create a new repository and the use copy to transfer all snapshots.


Chrome , Uncaught TypeError: localStream.stop is not a function

:+1:

Ah sorry @fattalgazi - I just fixed this and missed that you already had a PR open. Fixed now in #161.

Thanks Tim, looks good. Are you able to close this PR?

thanks, I'm closing it.


Add stub for quickly implementing custom templates tags and filters

The end result is when people create their django project using labzero, they can start adding their own custom tags and filters, with all the boilerplate already done.

My plan is to create a directory called templatetags and add an empty init.py to the labzero project or under src folder. This will be generated every time people create their project using labzero. I tried it, and it works on my own project.

Implement some simple tags and filters as well so people can see how it should be implemented and may also use it as a base for their own tags/filters.

I implemented tags. I will look at filter

let me know when you are happy or where to edit with tags so i can go ahead with filters


update glob string for chef-log.dll

Description

chef-18 omnibus build is failing (https://buildkite.com/chef/chef-chef-main-validate-adhoc/builds/1908) to find chef log dll since it expects folder suffix to be x64-mingw-ucrt while the available match is for universal-mingw-ucrt. This change updates the glob to chef-[0-9]*-mingw-ucrt/ext/win32-eventlog/chef-log.dll so it matches both suffixes.
adhoc run: https://buildkite.com/chef/chef-chef-chef-18-validate-adhoc/builds/143
Related Issue

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Chore (non-breaking change that does not add functionality or fix an issue)

Checklist:

[ ] I have read the CONTRIBUTING document.
[ ] I have run the pre-merge tests locally and they pass.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] If Gemfile.lock has changed, I have used --conservative to do it and included the full output in the Description above.
[ ] All new and existing tests passed.
[ ] All commits have been signed-off for the Developer Certificate of Origin.

You'll want to rebase though - and you have some tests failing.

@rishichawda  - the builds are all fixed now, can you rebase?


Fix Ceph image conversion

With the current implementation, we require the human operator to build a PVC for Image Conversion purposes. The PVC, attached to an arbitrary storage_class, can then be injected and mounted to the right path via ExtraMounts. However, this implementation is good only for RWX PVCs, because it realizes a common space for all the glanceAPIs and replicas where the conversion happens. If a RWO PVC is used, a replica deployed on a different node won't be able to mount the it, and even worse, it will be stuck in Pending state due to the Distribution affinity policy implemented in the operator. Even with the NodeSelector usage, there will be cases where this is going to generate issues. This patch fixes the problem described above with the following:

It introduces a PVC, created by the StatefulSet (via templates) only for the "external" glanceAPI instances
It creates the imageConversion PVC only if Ceph is enabled
It automatically sets the Image conversion plugin feature in Glance 00-config.conf when Ceph is part of the enabled_backend string (we compute the hash and store in the .Status.Hash to verify a change in the backends)
Takes care about the StatefulSet update for fields that are forbidden by default

The image conversion PVC is a RWO, stable storage that can be used by each replica (it has been tested with replica:3) and this patch avoids any manual action required by a human operator. ImageConversion is enabled by default with Ceph and it's transparent to the end user.
Fixes: OSPRH-5690

Ok, unblocking the merge on this one.
When Ceph is configured as backend, we can see that:

the glance-conversion PVC is automatically created
os_glance_staging_store is mounted in the external pod and not in the internal pod
glance-external-config-data has the right enabled plugin in the config section


fix: Fix repo URL in go.mod and README.md

@dgzlopes could you create a new release after merging this, plz...


Remove cedar tests

Cedar 14 is deprecated, and thus Travis tests and local docker tests failed on run.
Remove tests for deprecated stack to prevent false negatives:

Remove Travis stack test for Cedar 14
Remove make command test-cedar-14

There are a couple of more places where cedar is referenced if you want to do a deeper GC: https://github.com/heroku/heroku-buildpack-python/search?q=cedar&unscoped_q=cedar


View: Show Inlay Hints vs Editor: Inlay Hints: Toggle

Testing #148040
When editor.inlayHints.toggle is show, inlay hints are never shown regardless of the View: Toggle Inlay Hints checkbox:

Could both of these gestures modify the same value rather than needing to and them together?

related to https://github.com/microsoft/vscode/issues/148151

I don't know yet if both settings should be merged or not... For "not" one could either show the entry as disabled or not at all as soon as the toggle setting is configured. What do you think?

Assuming #148151 comes true how should this work? E.g you have configured the "hideWhenPressed" mode which translates to "on/enabled" and gives a check box. When un-checking and re-checking does it go back to default, e.g just "on" or should it magically remember "hideWhenPressed" and those

Could Editor: Inlay Hints: Toggle be a true/false that when true takes whatever the current View: Show Inlay Hints setting is and flips it? When false the modifier keys would be ignored.

Hm... that's an interesting idea. Currently it would collide with a perf trick I am doing: when "showOnPressed" is used I still  request and update inlay hints (but don't render them) so that they render immediately when holding the keys. That wouldn't be able to do anymore or at least it would mean we always ask for inlay hints (which is unwanted)

As part of https://github.com/microsoft/vscode/issues/148151 I have removed the view menu entry again. The "resolves" the conflict ;-) Turning this into a feature request to make the menu entry a fully fledged submenu with all modes (mutually exclusive)
Inlay Hints > ✔ Enabled
              Show Unless Pressed
              Hide Unless Pressed

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!


[new_basic] New command added to basic.sh script To delete files which are n days old

New command is added to basic.sh script , which will delete files from folder which are n days old
If all looks good you can merge
Thank you

thanks!


How to avoid C26437 warning when slice is required?

[Enter feedback here]
The page explains what slicing is, but does not explain how to avoid the warning when the slice is actually required?  Eg, does using a static_cast avoid the warning?

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

ID: 627fa04e-452e-99f3-3110-9b3ce879c9d1
Version Independent ID: d8dc1078-5e1c-9d89-7318-44c24ccff1ae
Content: C26437
Content Source: docs/code-quality/c26437.md
Product: visual-cpp
Technology: cpp-tools
GitHub Login: @corob-msft
Microsoft Alias: corob

@TonyRietwyk
That's true. You'll find that we don't describe what to do if you want the behavior that code analysis warns about but not the warning in any other articles, either. Instead, elsewhere we describe how to create a ruleset that doesn't include a particular warning, and we also explain how to disable specific warnings at the function level. I'm closing this issue, as I don't expect we'll address it.
A static_cast<base_type>(derived_type) is an explicit "I know what I'm doing" marker, It might work here, though I don't know. I'm very suspicious of claims that slicing is required, instead of, say, constructing a new instance of the needed base type from the derived type and using that instead. You can't make mischief in the sliced members if they're not there in the first place.


Migrate Pull requests from Sway

After https://github.com/swaywm/sway/pull/7541, contributed scripts in contrib/ directory will be managed in this repository.
Here's the list of PR from https://github.com/swaywm/sway

PR from sway
PR in this repo
status
author

https://github.com/swaywm/sway/pull/5919

no PR
@bugrasan

https://github.com/swaywm/sway/pull/6286

no PR
@RagnarGrootKoerkamp

https://github.com/swaywm/sway/pull/6951

no PR
@kraftwerk28

https://github.com/swaywm/sway/pull/7104

no PR
@anpandey

https://github.com/swaywm/sway/pull/7158

no PR
@paulodiovani

https://github.com/swaywm/sway/pull/7238

no PR
@orthros

https://github.com/swaywm/sway/pull/7538

no PR
@buyolitsez

https://github.com/swaywm/sway/pull/7643
#1
merged
@ebbit1q

https://github.com/swaywm/sway/pull/7656
#3
merged
@Omrigan

If you are interested in contributing your work to this repository, please submit a pull request.
I can just cherry-pick some commits, but I think it would be better to be done by original authors.
Issues also could be migrated to this repository.

Some of PRs are still missing. If someone find some PR for contrib/ directory in sway repository. Please share it.

I will migrate my PRs across soon, ~1 week.

@orthros Thank you so much!

https://github.com/swaywm/sway/pull/7197

New PR: Chore / Limit grimshot window grab https://github.com/OctopusET/sway-contrib/pull/10

I made #11 to replace swaywm/sway#6286 .


Feature/1854 OpenCL /prim signatures part 2

Summary
This PR builds on #1859 and does the following changes to the /prim support in OpenCL:

renames elewise_multiplication/elewise_division to elt_multiply/elt_divide
adds crossprod, tcrossprod, fabs, log1m_inv_logit, logit, divide, trunc

Tests
All new functions have tests.
Side Effects
/
Release notes
Renamed elewise_multiplication/elewise_division to elt_multiply/elt_divide in the kernel generator and added OpenCL support to /prim functions crossprod, tcrossprod, fabs, log1m_inv_logit, logit, divide
Checklist

[x] Math issue #(issue number)

[x] Copyright holder: Rok Češnovar
The copyright holder is typically you or your assignee, such as a university or company. By submitting this pull request, the copyright holder is agreeing to the license the submitted work under the following licenses:
- Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
- Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

[x] the basic tests are passing

unit tests pass (to run, use: ./runTests.py test/unit)
header checks pass, (make test-headers)
dependencies checks pass, (make test-math-dependencies)
docs build, (make doxygen)
code passes the built in C++ standards checks (make cpplint)

[x] the code is written in idiomatic C++ and changes are documented in the doxygen

[x] the new changes are tested

This is ready for review.


[DOC] Feedback on configuration/generate-certificates

What do you want to do?

[ ] Request a change to existing documentation
[ ] Add new documentation
[ ] Report a technical problem with the documentation
[x] Other

Tell us about your request. Provide a summary of the request and all versions that are affected.
In this section:
20230517 | /docs/latest/security/configuration/generate-certificates/ | you say we need to add admin.pem and admin-key.pem to each node, yet you use neither in your opensearch.yml config?
What other resources are available? Provide links to related issues, POCs, steps for testing, etc.
Here is the location: https://opensearch.org/docs/2.7/security/configuration/generate-certificates/#add-certificate-files-to-opensearchyml

Thanks for reviewing this one, @scrawfor99!

I will update with what I figure out :)

Should be clarified by #5628
Basically, you don't actually need the admin certificate on every node unless you want to ssh into them and execute the securityadmin script or reload your certificates


Video capture ends with error: 6 on Android

Steps to Reproduce

Download source example from my repo
Run application
Click open camera
Start record a video
Stop record video
Click open camera again
Start record a video
Stop record a video

I see only the first record is success all the action record after that always show Video capture ends with error: 6
Expected results

The video should be record successfully

Actual results

An log show Video capture ends with error: 6
The captureState$ stream does not notify any exception
After tap to stop record MediaCaptureStatus is MediaCaptureStatus.success

About your device

Brand
Model
OS

Samsung
Galaxy A52s
OneUI 5 - Android 13

Your flutter version
Run this in your command line
flutter --version
Flutter 3.7.7 • channel stable • https://github.com/flutter/flutter.git
Framework • revision 2ad6cd72c0 (2 weeks ago) • 2023-03-08 09:41:59 -0800
Engine • revision 1837b5be5f
Tools • Dart 2.19.4 • DevTools 2.20.1

Apparently, this error corresponds to an encoding problem.
I ran your example and found an error in the logcat:
2023-03-27 11:46:10.431 30125-1607  ACodec                  com.example.test_camera_awesome      E  [OMX.qcom.video.encoder.avc] ERROR(0x80001009)
2023-03-27 11:46:10.431 30125-1607  ACodec                  com.example.test_camera_awesome      E  signalError(omxError 0x80001009, internalError -2147483648)
2023-03-27 11:46:10.431 30125-1606  MediaCodec              com.example.test_camera_awesome      E  Codec reported err 0x80001009, actionCode 0, while in state 6
2023-03-27 11:46:10.433 30125-30237 VideoEncoder            com.example.test_camera_awesome      D  Transitioning encoder internal state: STARTED --> ERROR
2023-03-27 11:46:10.449 30125-30237 SequentialExecutor      com.example.test_camera_awesome      E  Exception while executing runnable androidx.camera.video.internal.encoder.EncoderImpl$$ExternalSyntheticLambda6@ed6964b
                                                                                java.lang.IllegalStateException
                                                                                at android.media.MediaCodec.native_flush(Native Method)
                                                                                at android.media.MediaCodec.flush(MediaCodec.java:2298)
                                                                                at androidx.camera.video.internal.encoder.EncoderImpl.lambda$stopMediaCodec$11$androidx-camera-video-internal-encoder-EncoderImpl(EncoderImpl.java:823)
                                                                                at androidx.camera.video.internal.encoder.EncoderImpl$$ExternalSyntheticLambda6.run(Unknown Source:6)
                                                                                at androidx.camera.core.impl.utils.executor.SequentialExecutor$QueueWorker.workOnQueue(SequentialExecutor.java:231)
                                                                                at androidx.camera.core.impl.utils.executor.SequentialExecutor$QueueWorker.run(SequentialExecutor.java:173)
                                                                                at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1167)
                                                                                at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:641)
                                                                                at java.lang.Thread.run(Thread.java:923)

Not sure yet why this happens, I'll investigate a bit more.

Sometimes, The video output can not be played by video_player plugin. I can not replicate this issue. I don't remember exactly the error message but it relates to video codec. Not sure if it relates to this issue

After some tries, I believe that the issues arise when there's a video player in the widget tree.
I think it works until there is a video recorded. Once one is recorded, it's loaded into the main.dart screen and that's when video recording stop working with this strange error.
I commented the part that show the videos and I can't reproduce the bug anymore, even when recording several videos.
Is it the same for you ?

Is it the same for you ?

Yes, When I remove video player out the widget tree. The issue is not happened

Yeah, I think I have a way to workaround. Thank you

Closing since there is a workaround.


Problems about visualizing output on my dataset

Thank you for your work.
When I try the inference code about KITTI-360 on my dataset with the pretrained models on SemanticKITTI, I'm confused on the output: The drivable area is too short. It seems that can't deal with the distant area. Is my config wrong or perhaps the some details I forget.
About the config:
I modify the camera intrinsics and the image resolution form (1408×376) to (1280×720). The 'cam2velo' matrix is not modified.
The output is:

Look forward to your reply. thx :)

Btw, when I run the inference code, the number of images I used was 15. Does a smaller number of images affect the results?

Hi, is it the same for every image?

yes it still does.
I re-modified the image resolution and camera parameters. The effect is a little better but the road length is still very short.
The resolution of the image is from (1280, 720) to (1408, 376). The internal parameters are changed as follows
    orig_size = (1280, 720)
    target_size = (1408, 376)

    scale_width = target_size[0] / orig_size[0]
    scale_height = target_size[1] / orig_size[1]

    adjusted_cam_k = cam_k.copy()
    adjusted_cam_k[0, 0] *= scale_width
    adjusted_cam_k[1, 1] *= scale_height
    adjusted_cam_k[0, 2] *= scale_width
    adjusted_cam_k[1, 2] *= scale_height

Is this a problem with the camera's external parameters? Or is it a problem with image processing? The FOV of the camera I use is 120°.
Thank you.

Yes, except for the data itself, the code has not been changed. I suspect there is something wrong with the depth estimation of the image. Maybe I train the model again on my dataset will solve this problem. Do you have any other good suggestions besides this method?

Hi, is this KITTI360 or your dataset? if it is your dataset then probably you need to retrain since the camera setup is different which is implicitly incorporated into the model during training i.e. through the projection module.


Remove allocation in vtag diff

Disclaimer: I couldn't test it on my system yet.
This PR attempts to remove all allocations (String, Vec and HashSet) used in various vtag diff functions.
I believe this should have an impact both in terms of performance and memory.

Awesome, thanks @tafia, I'll run some benchmarks and see what the improvements are!

I've applied the suggestions. I am a bit surprised that there is no impact, sorry for having you lose your time!
I have some other ideas for change but I am not sure they'll pay out.

Note that I haven't put the not fn in utils because it didn't really match the module comments:
 This module contains useful utils to get information about the current document.

I can put it there if needed.

No worries at all! Happy for your contribution and would gladly accept more PRs like this one. Sorry for the slow review, looks great!!

bors r+

Sorry for the slow review,

Trust me I am much slower nowadays!


Improve FilePicker functionality

Necessary features:

[x] Fill excel cells with file name(s)
[ ] make file names from list selectable -> insert only selected
[ ] fill column range with file names, extend range when file name list is larger than range

Right now it starts to fill all selected cells from top down. If not enough cells are selected for all file names then only the selected cells are filled.
Exmp:

I am not sure if this makes sense, but we can discuss it @kMutagene .
My points for this are:

make file names from list selectable -> insert only selected
This seems like an extra step that will not be used regulary. As we currently allow users to remove unwated file names it is not necessary to select those you want to insert. What we could do, is to allow users to change the order of the filenames via drag 'n drop/small arrows.

fill column range with file names, extend range when file name list is larger than range
As we currently have no 'revert changes' feature this could lead to a user deleting large parts of his file, should he select to many files and the table expend automatically. But then again this is what we do, whenever we add a building block to the column.

Your approach filling only the selected range seems better to me.

This seems like an extra step that will not be used regulary. As we currently allow users to remove unwated file names it is not necessary to select those you want to insert.

Are files taken by the order they are selected, or by the ordering of the explorer (e.g. alphabetical)? If the latter is true, at least reordering of the filelist must be possible if you are against selections

Added a reorder drag and drop for all file names. Will be added in 0.1.4!


Why is the Europe/London time zone set in some of the helper Dockerfiles?

The Europe/London time zone is not the same thing as UTC. Is there a reason for configuring the time zone in the following examples?
Examples
https://github.com/aws-samples/automated-security-helper/blob/7e2ecd8c033115daabdf9f1c246e976cae8ebc99/helper_dockerfiles/Dockerfile-cdk#L5-L6
https://github.com/aws-samples/automated-security-helper/blob/7e2ecd8c033115daabdf9f1c246e976cae8ebc99/helper_dockerfiles/Dockerfile-yaml#L5-L6

The project is moving from a multi-container architecture to a single container, which will be set to UTC. Since those Dockerfiles will be deprecated we will not be changing this.


added test for multiple Y columns. Fix in SGPR and GPRFITC

response to #280
Turns out there was also an error in FITC .
I modified the method equivalence test to have two identical columns of Y. When I tried generating different columns (and higher dimensional inputs) the tests (sometimes) failed at the current tolerances.

Current coverage is 99.63% (diff: 100%)

Merging #283 into master will not change coverage

@@             master       #283   diff @@
==========================================
  Files            25         25
  Lines          2170       2170
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           2162       2162
  Misses            8          8
  Partials          0          0

Powered by Codecov. Last update 3ab7139...184a144

Hi @hughsalimbeni
Yes, I'm not surprised that the tests fail numerically for more difficult problems.
Congrats on your first GPflow PR!


Instantiate RandomBeacon contract for the Beacon Ethereum chain

Depends on #3062
Refs #3006
This PR implements Ethereum chain functions for Random Beacon Sortition Pool. The following functions are implemented for Ethereum:
OperatorToStakingProvider() (chain.Address, bool, error)
EligibleStake(stakingProvider chain.Address) (*big.Int, error)
IsPoolLocked() (bool, error)
IsOperatorInPool() (bool, error)
JoinSortitionPool() error

Some additional changes were necessary to instantiate RandomBeacon and SortitionPool contract references:

The base Ethereum chain needs to provide the chain ID, transaction mutex, nonce manager, and mining waiter.
We need to ensure there is just one instance of transaction mutex. This mutex is held by the base Ethereum chain so there has to be only one instance of Ethereum base chain. The idea is to call global Connect only one time and create all the chain instances only one time. Creating chain instances outside of Connect is not possible from the outside of ethereum package.
We do not need to instantiate ethclient.Client outside of Connect, this is the action that should be executed upon establishing the connection, so Connect function is the right place.

There are no unit tests for Ethereum implementation, as they would require having an Ethereum client instance running. This will be covered by system tests. To test functions during the review, the following simple implementation can be used along with scripts/install.sh and scripts/initialize.sh:
package ethereum

import (
	"context"
	"fmt"
	"testing"

	"github.com/keep-network/keep-common/pkg/chain/ethereum"
)

func TestBeaconChain_KnownOperator(t *testing.T) {
	ctx := context.Background()

	config := ethereum.Config{}

	config.URL = "http://<IP_ADDRESS>:8545"
	config.Account.KeyFile = "/Users/piotr/dev/ethereum/data/keystore/UTC--2018-11-01T06-24-04.418691008Z--65ea55c1f10491038425725dc00dffeab2a1e28a"
	config.Account.KeyFilePassword = "password"
	config.ContractAddresses = make(map[string]string)
	config.ContractAddresses["RandomBeacon"] = "0x33a4bB615E56bB7538cFF511C4dc0A66fb1B27F8"

	beacon, _, err := Connect(ctx, config)
	if err != nil {
		t.Fatal(err)
	}

	stakingProvider, ok, err := beacon.OperatorToStakingProvider()
	if err != nil {
		t.Fatal(err)
	}
	fmt.Printf("stakingProvider = %v, ok = %v\n", stakingProvider, ok)

	eligibleStake, err := beacon.EligibleStake("0x65ea55c1f10491038425725dc00dffeab2a1e28a")
	if err != nil {
		t.Fatal(err)
	}
	fmt.Printf("eligibleStake = %v\n", eligibleStake)

	eligibleStake, err = beacon.EligibleStake("0x33a4bB615E56bB7538cFF511C4dc0A66fb1B27F8")
	if err != nil {
		t.Fatal(err)
	}
	fmt.Printf("eligibleStake = %v\n", eligibleStake)

	isPoolLocked, err := beacon.IsPoolLocked()
	if err != nil {
		t.Fatal(err)
	}
	fmt.Printf("isPoolLocked = %v\n", isPoolLocked)

	isOperatorInPool, err := beacon.IsOperatorInPool()
	if err != nil {
		t.Fatal(err)
	}
	fmt.Printf("isOperatorInPool = %v\n", isOperatorInPool)

	err = beacon.JoinSortitionPool()
	if err != nil {
		t.Fatal(err)
	}

	isOperatorInPool, err = beacon.IsOperatorInPool()
	if err != nil {
		t.Fatal(err)
	}
	fmt.Printf("isOperatorInPool = %v\n", isOperatorInPool)
}


Class not found

i get this error:
Fatal error: Class 'RonMelkhior\CoinpaymentsIPN\IPN' not found in /home/ubuntu/workspace/hello.php on line 5 Call Stack: 0.0010 235288 1.

yes what i did is.......install via composer....a folder called "vendor" got downloaded.....then i created a file hello.php on a level where that vendor folder was present.

and that page.php is where i am writing the code:
`<?php
use CoinpaymentsIPN\IPN;
$ipn = new IPN();
$ipn->setMerchantID('9636bba872a84ce80d4d29f9628cec78')
->setIPNSecret('hello');
try {
if ($ipn->validate($_POST, $_SERVER)) {
// Payment was successful, verify vars such as the transaction ID/email and process it.
} else {
// IPN worked, but the payment is pending.
}
} catch (RonMelkhior\CoinpaymentsIPN\Exceptions\InvalidRequestException $e) {
// The IPN data was not valid to begin with (missing data, invalid IPN method).
} catch (RonMelkhior\CoinpaymentsIPN\Exceptions\InsufficientDataException $e) {
// Sufficient data provided, but either the merchant ID or the IPN secret didn't match.
} catch (RonMelkhior\CoinpaymentsIPN\Exceptions\FailedPaymentException $e) {
// IPN worked, but the payment has failed (PayPal refund/cancelled/timed out).
}
?>`

You have to import vendor/autoload.php.
Like so:
<?php

require __DIR__ . '/vendor/autoload.php';

// ... rest of the code.

well it worked it seems.....dont see any error........one last question.....suppose i want to copy it into another server......just copy pasting the files(vendor,composer.json,composer.lock)....will it still work ?


chore(parameters): apply UA to AWS SDK clients used by Parameters

Description of your changes
Add user agent to clients user by parameters utility.

Related issues, RFCs

Issue number: closes #1537
Checklist

[x] My changes meet the tenets criteria
[x] I have performed a self-review of my own code
[x] I have commented my code where necessary, particularly in areas that should be flagged with a TODO, or hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my change is effective and works
[x] The PR title follows the conventional commit semantics

Breaking change checklist
Is it a breaking change?: NO

[ ] I have documented the migration process
[ ] I have added, implemented necessary warnings (if it can live side by side)

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

e2e tests ✅https://github.com/aws-powertools/powertools-lambda-typescript/actions/runs/5454144683

Good catch with duplicate attachment.

Does this attach the middleware twice? If so, does this result in the UA being appended twice? Same applies to Idempotency and all other providers.

It fails to add two middlewares with identical name, we catch the error and continue with user code execution.
We could extend the header for each utility, similar how .NET does. For example PT/parameters/ PT/idempotency, but it needs more consideration.

It fails to add two middlewares with identical name, we catch the error and continue with user code execution.

It's good that we don't halt execution, but we still log a warning that could be misleading. I'm going to open an issue to remember working on this asap.

In case of #1578 we agreed to attach only one user agent middleware for a client. If the client is used in multiple utilities, we check if a middleware is present and skip the second call, first come first serve.
We will revisit the strategy at later points to align with other languages.


Make REPL not show the day of the week on Windows

Make the output of time on Windows unified with Linux and Mac OS, and fundamentally solve https://github.com/erg-lang/erg/issues/55

Thanks!


added condition for fix the issue: "Cant find variable: styleObject"

When I tried to run, I got this error:  "Cant find variable: styleObject". It because importing not existing file. Files are specialized for Android and iOS. There is no file existing as a baseStyle.

Can you document what error you saw? I can't reproduce, and the import should automatically resolve to the platform:
https://reactnative.dev/docs/platform-specific-code#platform-specific-extensions

I didn't change any configuration or setup, and tested in 2 emulators, but right now I can't reproduce the error again.
Thanks.


HotBillCard

Complete the story in the file below. You can view the figma designs from the "design" tab in storybook. Check out #804 for more details
stories/dashboard/newsfeed/HotBillCard.stories.tsx
https://www.figma.com/file/3ifz37EOwDfmnEG8320KlD/CS1---MAPLE?node-id=158%3A3865

Closed by #867


[ZEPPELIN-791] Build infra: move all RAT to root pom.xml

What is this PR for?
It is better have a single place where we manage project-wise RAT exclusions for a contributions under licenses different from Apache, then let maven sub-modules have them as we do now (makes things harder to track)
What type of PR is it?
[Improvement]
Todos
N/A
What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-791
How should this be tested?
mvn verify -DskipTests
Screenshots (if appropriate)
Questions:
Does the licenses files need update? - No
Is there breaking changes for older versions? - No
Does this needs documentation? - No
Questions:

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

@felixcheung Could you review my PR?

merging if no more comment

merged to master.
thanks!


Available Videos on overview page shows error (using 1.7.1)

I upgraded to 1.7.1 on my grid and noticed this on the overview page :
Available Videos
Could not append videos because of java.util.ArrayList cannot be cast to java.util.Map error

Just curious if this will be addressed soon? (There hasn't been any release in months. Is this project over?) Thanks,
Cktech

@cktech529 I don't think this project is "over". I contribute to it, but haven't lately due to the holidays and work related items.

Thanks!
I see you sent a pull request on the fix. Is it in, awaiting a release? (Something I could pull down, build on my own?) Just curious and thanks again.
cktech

I didn't send in a pull request for this videos issue. I sent in a pull request for browser version detection and also only ask to record videos for nodes (option 1 and 3), not if the machine is only a hub (option 2).

So I'm now able to record videos (although a bunch of them are coming in all black) what I did to avoid the above issue was change the host name of the hub to the IP (and for some reason, videos are recording and I'm no longer getting the above error.)

@cktech529 if you are running Grid Extras as a service on windows it will record a black video. If you log in directly into the user and run it as a batch file on start up the video should record just fine.
Also, if you are using RDP to connect to the machine, everytime you exit the machine the GUI is diabled and the video will be recorded as black. To fix this, use VNC server to access the machine instead of RDP

The error , Could not append videos because of java.util.ArrayList cannot be cast to java.util.Map error , appeared because the video_output directory was not there. We now create this directory if it doesn't exist. The fix for this is in master, and will be in the next release.

Also, my workaround for using RDP and that black screen (at least for the screenshots), was to use the following command to "log off" of the session:
tscon rdp-tcp#0 /dest:console
If I use that, instead of just closing the RDP window, the screenshots appear correctly. However, using VNC might be a better solution.

I just created a brand new Hub by downloading the newest JAR from https://github.com/groupon/Selenium-Grid-Extras/releases (1.7.1-SNAPSHOT).
And, I'm seeing this error on the hub's page (0 nodes are connected yet):

Available Videos
Could not append videos because of java.util.ArrayList cannot be cast to java.util.Map error

@charlescapps The fix isn't in a release yet (it will be in the next release). Though the fix is simple. Create a video_output directory where your jar is.

Thanks!

Regarding the comment on Video black out if running selenium node on windows as a service.  Is there a work around to make video to work on windows as a service?  If I don't use as a service then how do I go about making use of the reboot feature on X number of test run on a node?  Please let me know if you have any suggestion here.

You could put a shell script that starts the selenium-grid-extras jar in the startup directory, and have windows auto login on bootup. Does that work?

Hi Shawn,
I actually tried something similar as well.  I have a Scheduler Task to trigger a batch file that essentially execute the selenium java command.  This task is set to "run when user is log in".
At the same time, I followed this couple links to enable the auto login.  But somehow it doesn't seems to work for me at all.
1. Auto-login in Windows Server 20082.  http://www.win2008workstation.com/configure-auto-logon
I am on VM windows 2008 R2 SP1.
Then I tried to use the "run whether user is log on or not" off the Scheduler Task, and the reboot does bring up the selenium service, but the video is Black screen.
Any ideas here?
Thanks,
David

From: Shawn McCarthy<EMAIL_ADDRESS>To: groupon/Selenium-Grid-Extras<EMAIL_ADDRESS>Cc: David Le<EMAIL_ADDRESS>Sent: Wednesday, May 13, 2015 9:36 PM
Subject: Re: [Selenium-Grid-Extras] Available Videos on overview page shows error (using 1.7.1) (#64)
You could put a shell script that starts the selenium-grid-extras jar in the startup directory, and have windows auto login on bootup. Does that work?—
Reply to this email directly or view it on GitHub.

@lettrung99 Please open a new issue if you think it is a Selenium-Grid-Extras issue. The original issue has been fixed, so I am closing this.


I write mini code for show console.log but have a error is "Uncaught Error: Minified React error #32..."

Please provide a way to reproduce 👍


ARROW-15390: [Dev][C++][Doc] Document the GDB extension

Ok.  I'm fine with leaving versioning out until we introduce some kind of backwards incompatible change.


ianny: 1.0.0beta.1 -> 1.0.0

Description of changes
Update ianny to 1.0.0.
Maintainer: @max-amb

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 313198 run on x86_64-linux 1

1 package built:

ianny

I have updated the hash, I believe it is correct now


Add Station and Trajectory Capability

From OTRS from @amdasilva:

I just spoke with Ben and for my immediate need a small tweak of his Regrid_Util.x will suffice to convert the existing files from cube to lat-lon.We are good for the instrument simulators that I need to implement in the short term.

As we move toward having our GEOS-FP and reanalysis output on cubed-sphere it becomes imperative that we rewrite our tools to work directly with files on the cube, without the need to convert them to lat-lon. Therefore, I’d like to request support from the SI team to create fortran versions of two fundamental utilities needed for OSSE and field campaign data analysis. These are:

Trajectory sampler: reads GEOS collections on the cube and produces a  reduced set on (lon,lat, time) trajectory (similar to my trj_sampler.py)
Station sampler:  similar utility, sampling collections to a fixed (lon,lat) set of stations (no time involved), similar to my stn_sampler.py
After discussing this with Ben we are convinced that these can be developed as extensions of the current Regrid_Utility, adding logic to handle file aggregation (e.g., grads control file) and locstream interpolation.

Also, per a followup, versions of the Python scripts can be found here:
https://github.com/GEOS-ESM/AeroApps/tree/main/src/GMAO_Shared/GMAO_pyobs
Though they might be out-of-date.


upload the new version to PyPi 0.6.2

Hi there,
I see that the code in the git is newer 0.6.2 than the code in PyPi 0.6.1 and i need it to support Django 1.7
could you please upload the new version to PyPi?
I usually don't like using packages from the git repositories.
Thanks :)
Koby

Done!

Thanks :)


Azure: Link kubernetes resources with azure resources

AKS Cluster is not linked to Kubernetes cluster
Virtual Machines to Nodes


Installing JDK after new Oracle JDK License Update

Hello,
I am not able to install JDK after the new Oracle JDK License Update that was made since April 16, 2019.
Can you please let me know how to install the JDK or any other way to install divolte?
Thanks in Advance,
Krishna

Good afternoon,
First of all tell me you operating system if you can.
I work on Linux I hope I can help you!
Thanks!
Manu

I would suggest using OpenJDK instead of the OracleJDK.


feat: update left side as you resize column

Stores leftGridWidthInfo as a state instead of useMemo, thus we have a setter that can update the widths on the fly
export overrideLeftGridWidth, that is used in the ColumnAdjuster as you drag. Not super happy of how many layers we need to pass it through but don't think there is an alternative

As a side note for another PR. Probably want to have a different minimum size for columns on the left side as you can end with need seeing the text at all.

As a side note for another PR. Probably want to have a different minimum size for columns on the left side as you can end with not seeing the text at all.

Yeah the min width is completely arbitrary at this point, and static. We probably want to set it to something more intentional and make it depend on font size

:tada: This PR is included in version 2.4.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Support for Razor Pages

As it stands, [ValidateRecaptcha] cannot be added to a Razor Page's OnPost or OnPostAsync method.
Is there any other way to verify the captcha response that isn't included in the readme?

Yes. You can create an inject an instance of the RecaptchaService class into the razor page. You can then call the validation method directly rather then using the attribute. https://github.com/TimothyMeadows/reCAPTCHA.AspNetCore/blob/master/reCAPTCHA.AspNetCore/RecaptchaService.cs#L62-L71


Update GeneratorApp code to prevent criterionIds/criteria being reported as a discrepancy in alignment.csv

Add the following to fromIds in GeneratorApp.java in the usdm-dictionary-generator repository to prevent criterionIds/criteria being reported as a discrepancy in alignment.csv:
        if (name.contains("criterion") && name.endsWith("Ids")){
            synonyms.add(name.substring(0, name.length() - "Ids".length()).replace("criterion", "criteria"));
        }

Also added synonym to toIds


Merge with ACF Missiles

Merge with ACF 3 Missiles

I don't think this should be done, actually. ACF missiles introduce a few issues, including balance, file size, etc.
Overall, I think ACF 3 should shift more towards being a core system with things like missiles or ACE being separate addons.


fix: fix CometNativeExec.doCanonicalize for ReusedExchangeExec

Which issue does this PR close?

Closes #448.
Rationale for this change

What changes are included in this PR?

How are these changes tested?

Merged. Thanks @sunchao


Dropping SSR support

I'm considering dropping SSR support. It's built on top of many hacks, all of which are entirely circumventing the recommendations of react-native-web.
I've elaborated on this problem at length on many issues and PRs in the past.
If you need Dripsy in production with SSR support, feel free to mention it here.
At the moment, supporting SSR means the web API is complex, kind of confusing, and adds more DOM nodes.
I've been using Dripsy with SSR "disabled" (i.e. we return null at the root of the app and then setMounted(true)) on the beatgig.com. No issues for us with performance.

I'm in favor of this.
I think if there's better support for another method down the line that's in line with react-native-web it can always be added.

I'm thinking we add an ssr option to the provider. If true, then it'll handle blocking rendering on web for you until it's mounted. You could pass a Placeholder component too.

This seems similar to the PWA/Twitter style.

I thought this was a good piece explaining why SSR doesn't work with RNW.
https://lihautan.com/hydrating-text-content/
React's hydration expects the initial render to match what's on the server. However, if we're using JS Dimensions, this isn't possible.

This also means that custom breakpoints should be easy to implement, since we don't need to use them with Fresnel.

I'm working on this update now. It'll be a major version bump.
The only "breaking" changes from the user's standpoint:

If you are using SSR, you can pass ssr={true} to DripsyProvider. In this case, it'll delay rendering until the app mounts. If you already have a gate of your own, then this isn't needed.
I'm removing the already-deprecated ThemeProvider.

All other changes are under the hood.
This will automatically enable custom breakpoints.

This also opens the door to a lot of additions. We can now offer a useSx prop which memoizes styles. We can also allow components to pass custom props to styled. I'm excited for the doors it opens.

If you could test this, it would be great.
yarn add dripsy@canary

That should install 2.0.0-alpha.0

I added notes for making this work well for SEO at the PR: https://github.com/nandorojo/dripsy/pull/101

Removed every instance of webContainerSx from our app. So satisfying.

yarn add dripsy, v2.0.1 removes fresnel and SSR.

Do you know what SEO implications this has? I read the SEO recommendations you referenced, and looked a bit at next-seo, but it remains unclear what, if any, SEO impact it will likely have...?
Have you measured the before/after impact on beatgig.com search index ranking, for instance with Fetch as Google and measuring with the Google Web Vitals ?
It would be interesting to learn the results, to get a feel for the tradeoff.

I need Dripsy in production with SSR support.

@redbar0n you can still use 1.x for this. Unfortunately, maintaining SSR development wise simply isn't worth it. It's the equivalent of maintaining a fork of react-native-web, with style inconsistencies. While I share the concerns about SEO, etc., the problem lies with react-native-web deprecating className, not Dripsy. There are some hacks you could use, if you want, but I don't recommend them.
If you really need SSR support for your render code, you'd need to 1) not use React Native Web, or 2) not use responsive styles. There is no alternative without hacks that are hard to maintain.

If it helps, Twitter doesn't SSR besides meta tags, judging by the fact that opening Twitter shows a loading spinner first

I've written about this on many issues and PRs, but this is where I tried my hardest with RNW: https://github.com/necolas/react-native-web/issues/1318

Thanks, and I understand the concern and your decision.
This was also helpful for me to understand the tradeoffs in dealing with hydration, CSS, SSR, Fresnel etc., like the other link you shared previously re. hydration:
https://aboutmonica.com/blog/server-side-rendering-react-hydration-best-practices#summary
I saw in that issue you mentioned, you said:

The unfortunate part, of course, is that styled-components relies on className exclusively, and doesn't allow you to use a data-*/dataSet.X prop as a CSS selector.

What if styled-components or some similar library allowed dataSet? Could that circumvent the whole issue? Given that styling libraries are quite abundant and change frequently, it might be a hope that there is or will be one out there that suits. It would surely be a shame if this were the only thing to block the awesome combo of RNW+ NextJS from being fully realised.

You could try. But at this point I've come around to agree with Nicolas. Let's stay in JS-land and build apps that are self-contained in React component styles. They should be consistent across platforms and not have CSS side effects. So I have given up on the SSR train when it comes to RNW. Ever since I stopped, my coding speed has been much better, and now I can comfortably use crucial hooks like useWindowDimensions, etc to style content.

Actually, come to think of it, I don't actually need SSR support for the user, it's simply for SEO purposes (search engine crawlers).
I was inspired by the tradeoff this guy made, although he used the more complicated Rendertron instead of NextJS to achieve it.
So, if something (Dripsy?) simply detected the Googlebot User Agent, and used NextJS to SSR a (non-responsive) mobile version of the webpage, it'd be a good tradeoff in most cases, I think.
Then users could have the primary, fully responsive, CSR'ed version. Without the app having to sacrifice SEO in general.
Would that work?

I'm only using static generation, not SSR, which can't conditionally render pages anyway.
Maybe your solution would work. You'd have to set the window dimensions yourself somehow, and Dripsy would read from that if it's a crawler.
Just FYI, this isn't something I find a priority at the moment. I spent too much time optimizing for this and ultimately saw no benefit to our site for it. But if you can come up with a solution that's easy to drop in, I'm open to it.

At the moment, supporting SSR means the web API is complex, kind of confusing, and adds more DOM nodes.

With more DOM nodes, I assume you mean <Responsive> wrapper components emitting co-located

https://necolas.github.io/react-native-web/docs/styling/#how-it-works
The proposed Dripsy output from the wrapped StyleSheet, with one media query per designated viewport breakpoint:

It seems like it is a breeze to get the stylesheet that RNW outputs, by using:
import styleResolver from ‘../StyleSheet/styleResolver’;
const sheet = styleResolver.getStyleSheet();
const content = sheet.textContent(); // to get the atomic CSS for the rules that have been resolved.

From this RNW code.
It would mean that the media queries would be output by Dripsy itself. But the user shouldn't need to care about media queries directly anyway. It's more user-friendly to have a Dripsy separate API for responsivity, that allows users to declare the necessary screen sizes and breakpoints for the media queries.
It would mean to avoid styled-components, and take control of CSS output directly from Dripsy. But as I’m sure you’ve discovered, using styled-components with React Native Web + responsivity, is problematic, to say the least.
To apply styles conditionally in JS, based on window dimensions (client-side), then instead of the JS Dimensions API, you could use:
if (window.matchMedia(“(max-width: {getBreakpoints().width} )”).matches)

It can be run only client-side, by using typeof window === ‘undefined’.
Wrapped in a hook it could be exposed as:
if (clientWindowSize(“medium”)) { /* then apply this style */ }

(It seems matchMedia is also what Fresnel uses in their <Media> components to prevent React from rendering unnecessary breakpoints client-side.) You could either put this into a replacement for RNW’s useWindowDimensions() when on web, or expose it in another hook, perhaps better named. Since media queries are designed so you shouldn't need to know the specific window dimensions.
To avoid React's hydrated HTML not matching what was SSR'ed, then you could use a vanilla JS DOM selector to put the viewport dimensions into a
 outside of the React <App> tree. Breakpoints could be user specified/pre-determined, or server-determined if using Fresnel. Props can be same client-side and server-side, and the component logic is simple: If the
 is there, use it, if not, then set it. If using Fresnel, this way, the client is ensured to use the same media query breakpoints as determined by Fresnel server-side.
Custom breakpoints. Developers could statically set custom breakpoints with Dripsy, which I presume Dripsy could forward to Fresnel custom breakpoints by using its createMedia(). But the breakpoints refer to, and is applied in, the DOM/markup with the <Media> wrapper components that Fresnel uses. So I think you currently cannot escape exposing those or an equivalent wrapper component, like SSRMediaQuery or SSRComponent, to the developer. As of now. But this might change, if Fresnel exposes a useMedia() hook… (which might also remove the extra DOM nodes Fresnel adds).
I see 3 general ways for SSR with RNW:

Googlebot: SSR only a mobile version for the Googlebot, to get SEO. But do Client-Side Rendering (CSR) for users. Like mentioned in my previous comment. Could be done with manual user-agent sniffing, or with a GoogleChrome/Rendertron proxy. Though Google may not like being fed something different than users (different initial markup, even though resulting markup after React’s first CSR should be the same).
Fresnel: SSR all breakpoints. If sniffing the user-agent is considered unreliable. Fresnel might have some RNW compatibility issues. Fresnel is currently most worthwhile if one has radically different layouts (including markup/structural changes), not just classic responsivity (styles only).

Optimise Fresnel with detect-responsive-traits: SSR based on sniffing user-agent where it can, but all breakpoints if not. To optimise bytes sent over-the-wire.

Mobile-first: SSR only mobile page markup, with pre-determined/static breakpoints. Let client choose to "Show desktop version" on first page, then set cookie for all future requests. Avoids rendering multiple layouts/breakpoints/sub-trees, so less CSS & HTML is sent over-the-wire.

Frankly, Mobile-first seems the simplest way to go, with the current limitations. If you are using RNW you presumably want a native mobile app to also work in the browser as a webapp. To simply get the mobile version in the browser, it doesn’t need to be so responsive it can change the layout completely (incl. markup, like Fresnel presumes), to automatically morph to a desktop version too. You just need media queries to make the styles/css responsive enough to fit all mobile screens. If viewed on desktop it can initially center-display the mobile version. The small tradeoff is that it requires a single click from the user to see the full desktop version, on the first visit from a desktop browser. If clicked, then during SSR, you know definitively (without any user-agent sniffing) to render only the breakpoints and markup related to the desktop version.

But common for all ways to get SSR, is to avoid the JS Dimensions API in favor of media queries, to also get responsivity.
I understand that this is not a priority of yours at the moment. I just thought that it would be good to have these ideas out there, in case there is something new here. If not, it could hopefully serve as a (linkable) summary to others going down the same path.

Thanks for the in-depth discussion here. I did read it closely. Hopefully there will be a time when react native can work with SSR without any hacks that circumvent it. This is doubly interesting with the announcement of React 18 and all of its innovations around SSR / Concurrent Mode.


remove unused router channel

https://github.com/thejameskyle/marionette-wires/issues/63

I wanted this to be replaced by something, if you want you can make a PR to https://github.com/thejameskyle/backbone-routing and add events to Backbone.history and then listen to them here.


grammar

Hey @N-C-M ,
it's a nice catch, thank you! Can you please add some more fixes to this pr? We really appreciate every little bit of help, but it will be not fair to other contributors who are doing serious work to accept these kind of pr's.


Nuxt-link to index path "/" not working on error.vue page when already on index page

Version
v2.10.2
Reproduction link
http://www.test.com
Steps to reproduce

Create an Axios call which takes place on the index page, and ensure this call returns any code in the 400/500 range.
Use the sample error.vue page provided in the nuxt documentation, which has the default nuxt-link back to the index page "/"
When the axios call returns with a http code in the 400/500 range the error.vue page will be loaded.
When you try to use the nuxt-link however to 'reload' the index page component, it wont load, presumably because you're already on the "/" path. But this is weird behavior. It should be able to reload the index page component

What is expected ?
That the index page component would reload
What is actually happening?
The index page component simply doesnt reload

This bug report is available on Nuxt community (#c10105)

As a temporary workaround you can use @click="$router.go()" to reload the page, I have a reproducible branch of this issue if anyone wants to try to look better into this.
https://github.com/ocBruno/nuxt.js/tree/fix/reload-error

https://gamecom.app/
I created a website on nuxt and server render. Any thoughts?
FYI you have to login to enjoy the feed :-p

This is still an issue. Has anyone from the core team reviewed this report yet?


Merging qfe/1809 into master

This merge should ideally be a non-squash merge.

This QFE is being considered a 1-off and should not be merged into master.


How to end a competition? / feedback on first "real" use

Hello, me again !
Well we had our first tournament with your software and so first of all thanks !
Everything went rather well except I have no idea how to end a competition so I took screenshots to save the score and leaderboard and then cancelled it.
Other questions/remarks:

I don't know if there is a way to have a 3rd place match in a competition?
Would you have an interest in:

supporting a "gender neutral" category to have singles match with men/women playing against each other (in my club we are not that many so we all play against each other in the tournament) (same for doubles, some teams are mixed, some are not)
Being able to change the score required for matches in the middle of the tournament (for example we do match in the groups as 11 points 2 set wins but we often do the semi-finals and up in 21 points). To workaround this I wanted to create a new competition with only the players that made if to the semi finals but I couldn't create a new one with the same category (since it was considered already running). We ended up tracking the score externally and then just inputting a fake score to get the correct winner.

I understand this isn't the "official" way to play but if you're interested in supporting this I'll try and submit a PR if I manage to add this :)
Thank you again

Hi! I'm glad you could utilize the app, I hope everyone had good fun.
A competition ends when the last match has its result entered. A planned feature is a PDF export of results. But that also works in the middle of the tournament. There is no special state that marks a competition as finished.
You will also be able to save/load entire tournaments so you can start a new one and keep the old results.

I don't know if there is a way to have a 3rd place match in a competition?

Right now that is only possible for the 'Elimination with consolation' mode where you can set how many places to play out. It is planned to give the group tournament (I assume you used this mode) a setting where you can chose the type of elimination tree that follows the group phase. This way you will be able to make a group phase followed by an elimination with consolation and with that a match for 3rd.

"gender neutral" category

I am not planning on implementing this but the app does not enforce any gender specifications when registering players.

change the score required for matches in the middle of the tournament

I have thought about that aswell and I like the option. Right now the scoring options are related to the competitons but it could be remodeled to be a relation of the individual matches.

I wanted to create a new competition with only the players that made if to the semi finals but I couldn't create a new one with the same category

Yeah you'd have to activate the 'Use playing levels' option before starting any competitions. With that you can create arbitrarily many competitions of the same discipline under different categories (or "playing levels").

I'll try and submit a PR

I'm stoked you want to try to contribute! I am currently in the process of a small refactor to make a multi-client-server model work. I am expecting that I can commit my changes to the public repo in about a week. The best point in time to start making changes would be after that. From there it will be easier to manage collaboration! I'll message you when I get there if you want.
Thanks again for testing the app in the wild :)

@EmFl I just pushed the refactoring changes.
The server is no longer the default PocketBase binary but a modified version. You should be able to just replace your pocketbase.exe with the one from the server repository's releases page.
Feel free to open a new issue for your proposed changes where we can discuss and I can answer questions.