Add sprite atlases into the new renderer.

Objective
Restore the functionality of sprite atlases in the new renderer.
Note: This PR relies on #2555
Solution
Mostly just a copy paste of the existing sprite atlas implementation, however I unified the rendering between sprites and atlases.

Looks good to me!

bors r+


Missing Pal subspecies

Hello!
the api does not have Pal Subspecies yet

source: https://palworld.fandom.com/wiki/Paldeck

currently working on that! Thanks for open this issue!!!

Subspecies added! Thx!


[Event Request] Report 730 - Copy Item - OnAfterCopyItem with IncludeSender=true

Dear ALAppExtensions team,
Through this issue we would like to request event OnAfterCopyItem to be adjusted to have IncludeSender=true for this event.
That is, so that we can act on additional options added to the request page, which can hopefully be added in the future via report-extensions.
We want to be able to add options via extensions to the request page and subscribe to this event in the report. The subscriber can then use the sender and see if the added option was enabled or not (e.g., via a procedure sender.CopyMySubsidiaryRecordsIsEnabled).
If it was enabled, then the subscriber can trigger to also copy subsidiary records from our extension tables from the source item to the target item.

If you have other/better suggestions of how to let the event subscribers know which options (and additional options) were enabled, please feel free to share.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Fix SSL context

Fixes the recently moved SSL context for servers.

Travis fails on latest as the change is still only in Crystal nightly.

@sdogruyol, Travis fails on latest as the change is still only in Crystal nightly.

Hey @noumar , thanks for the pull request.
Let's wait till this gets published in Crystal latest 👍

Closing in favour of #162


Slack Channel

I'm submitting a ...

[ ] bug report
[ ] feature request
[x] other

Salam,
Could you please add me to the slack channel?
Email<EMAIL_ADDRESS>Thank you!

Could you Please add ME to slack channel?
<EMAIL_ADDRESS>

And one more offer:
Corpus.quran.com
Syntactic Treebank!
The verse 8-59 (between)
Verses does not exist!
THANK YOU very much!

Could you add me to the slack channel<EMAIL_ADDRESS>

I'd like to be added to the slack channel too. The heroku app returns an application error. My email is<EMAIL_ADDRESS>

Please use this app for invitaiton: https://quranslack.herokuapp.com/
App was fixed last week :)


values are not added to instance

Please see updated master branch: https://github.com/davidkarlsen/sundrtest - should the test not pass?

@davidkarlsen: There are 2 issues with your tests:
i) Your classes contain no constructors (builders pass values to the target object via constructor, properties which are not reflected in the constructor arguments are ignored).
ii) You are comparing arrays with equals.

I am wondering if we should completely ignore those properties, or if we should just log a warning.

Well, I decided that if there is a setter for the property we should use the setter.
If neither constructor argument nor setter is found the property will be ignored.

release on its way to central

There are still a few hiccups that need to be addressed.

it should be fine now


conv2d calculation results are inconsistent with pytorch

Please make sure that this is a bug. As per our
GitHub Policy,
we only address code/doc bugs, performance issues, feature requests and
build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock
example script provided in TensorFlow):
OS Platform and Distribution (e.g.,
Linux Ubuntu 16.04):
TensorFlow installed from (source or
binary): - TensorFlow version (use command below):
Python version: 3.6.5
CUDA/cuDNN version: - GPU model and memory:10.1

Describe the current behavior
For some specific data, the calculation results of tensorflow and pytorch are inconsistent
Describe the expected behavior
The calculation results of tensorflow and pytorch are consistent or the difference is small
Standalone code to reproduce the issue
    if target_interface == 'conv1':
        weights_torch = torch.from_numpy(np.full((11, 11, 3, 1), 1, np.float64).transpose((3, 2, 0, 1)))
        output_pytorch_cpu = torch.from_numpy(
            F.conv2d(torch.from_numpy(input_pytorch.numpy().transpose((0, 3, 1, 2))), weights_torch, padding=0,
                     stride=4).numpy().transpose((0, 2, 3, 1)))

Other info / logs
tensorflow_output:
31.42857361,31.4285183,31.42857361
69.24489594,69.24489594,59.3673439
100.89792633,100.89792633,100.89792633
133.26530457,133.26530457,133.26530457
60.22449875,60.22449875,60.22449875
187.46939087,187.46939087,187.46939087
205.918396,205.918396,205.918396
96.42860413,96.42860413,96.42860413
61.42854309,61.42854309,61.42854309
148.89811707,148.89811707,148.89811707
96.28572083,96.28572083,96.28572083
83.02045441,83.02045441,83.02045441
87.89794922,87.89794922,87.89794922
135.12240601,135.12240601,135.12240601
pytorch_output
31.428574,31.428518,31.428574
59.367344,69.244896,69.244896
100.89793,100.89793,100.89793
133.2653,133.2653,133.2653
60.2245,60.2245,60.2245
187.46939,187.46939,187.46939
205.9184,205.9184,205.9184
96.428604,96.428604,96.428604
61.428543,61.428543,61.428543
148.89812,148.89812,148.89812
96.28572,96.28572,96.28572
83.020454,83.020454,83.020454
87.89795,87.89795,87.89795
135.1224,135.1224,135.1224

@lvjunsetup
could you please share the tensorflow version on which the issue was faced for us to replicate this on our local.

Hi @lvjunsetup . Could you please share relevant tensorflow code to reproduce the issue.
Thanks.

@lvjunsetup
please update on the above comment

@lvjunsetup
could you please share the tensorflow version on which the issue was faced for us to replicate this on our local.

Sorry to reply to you so late. The version of tensorflow is 1.90. The error operators are conv2d () and avg_pooling (). Thank you.

Hi @lvjunsetup . Could you please share relevant tensorflow code to reproduce the issue.
Thanks.

This error is a fact, and it can be reproduced by using a large number of random tensors as input to conv2d () or avg_pooling (). Thank you.

@lvjunsetup
could you please provide with simple stand standalone indented code with all dependencies for us to replicate the issue in our local, this would help us resolve the issue faster

@lvjunsetup
could you please update as per above comment

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


Formatting error message

Hi, I am going through your package, and loving it. There is something I am looking for but so far have not found it.  Console log error looks like the following
Error: Value '300 miles/hr' not a float between 10 and 70
    at vfunc (<anonymous>:1131:19)
    at descriptor.set (<anonymous>:1207:19)
    at <anonymous>:114:12
    at hn (https://preview-941011.playcode.io/:193:5449)```

1. Is there a built-in mechanism to retrieve the following data only?:

`Value '300 miles/hr' not a float between 10 and 70`

2. Is there a possibility to write your own error message?

Cheers

See: https://github.com/runtime-data-validation-js/runtime-data-validation-typescript/blob/main/lib/decorators/numerical.ts#L64
The same pattern is used for every decorator.
I suppose that to support custom error messages would require a third parameter something like this:
export function IsFloatRange(min: number, max: number, message?: string) {
    return generateValidationDecorator(
        (value) => validators.isFloatRange(value, min, max),
        message ? message : `Value :value: not a float between ${min} and ${max}`);
}

What about the first question?

Is there a built-in mechanism to retrieve the following data only?:

Suppose you return an Error object or array with the stack trance and the actual error as separate properties of the object or elements in the array?

As you suggested, a third parameter should work. However, you should now populate the message parameter with the error message only. So if the error is logged to the console the error object could show the error message only as a property and the stacktrace as another property.
Then, I would be able to do something like the below...
const {message, stacktrace} = error
What do you think?

Oh, what you're asking for is a more structured error object...?
A quick glance at - https://nodejs.org/api/errors.html#class-error - suggests that
throw {
   message: message,
   stack: ...? -- there is a function Error.captureStackTrace
   code: EINVAL -- there is a bunch of suggested error codes
};

Yes. That would be super!

In the Node.js documentation, the Error class is already described as having message and stack fields that you should be able to access as you suggest.
Start here https://nodejs.org/api/errors.html#class-error -- then scroll down to see the available fields.
What I can do is ensure a better value for code is available.
The only thing that would be unique to this package is perhaps exposing value as a field in Error...


Added python regex cheatsheet from Dataquest

As suggested in #16 : Added the PDF and PNG, and updated Readme to reflect.

Added Business Science University (BSU) Clustering and segmentation cheatsheet to Machine Learning > Unsupervised.

Added BSU's Regression cheatsheet to Machine Learning > Supervised.


device.hardware.py: Add skip_on_error flag to improve Windows experience with USB discovery

If any of the USB devices attached are busy or unavailable, but libusb1 sees them, you get a nice trace on Windows:
$ glasgow list
Traceback (most recent call last):
  File "C:\ChipWhisperer5_64_5_7_0\cw\home\portable\WPy64-31080\python-3.10.8.amd64\lib\runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "C:\ChipWhisperer5_64_5_7_0\cw\home\portable\WPy64-31080\python-3.10.8.amd64\lib\runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "C:\users\c_ofl\.local\bin\glasgow.exe\__main__.py", line 7, in <module>
  File "C:\ChipWhisperer5_64_5_7_0\cw\home\portable\chipwhisperer\jupyter\user\glasgow\glasgow\software\glasgow\cli.py", line 937, in main
    exit(loop.run_until_complete(_main()))
  File "C:\ChipWhisperer5_64_5_7_0\cw\home\portable\WPy64-31080\python-3.10.8.amd64\lib\asyncio\base_events.py", line 649, in run_until_complete
    return future.result()
  File "C:\ChipWhisperer5_64_5_7_0\cw\home\portable\chipwhisperer\jupyter\user\glasgow\glasgow\software\glasgow\cli.py", line 900, in _main
    for serial in sorted(GlasgowHardwareDevice.enumerate_serials()):
  File "C:\ChipWhisperer5_64_5_7_0\cw\home\portable\chipwhisperer\jupyter\user\glasgow\glasgow\software\glasgow\device\hardware.py", line 195, in enumerate_serials
    devices = cls._enumerate_devices(usb_context)
  File "C:\ChipWhisperer5_64_5_7_0\cw\home\portable\chipwhisperer\jupyter\user\glasgow\glasgow\software\glasgow\device\hardware.py", line 99, in _enumerate_devices
    devices.extend(list(usb_context.getDeviceIterator()))
  File "C:\Users\c_ofl\AppData\Local\pipx\pipx\venvs\glasgow\lib\site-packages\usb1\__init__.py", line 2114, in wrapper
    for value in generator:
  File "C:\Users\c_ofl\AppData\Local\pipx\pipx\venvs\glasgow\lib\site-packages\usb1\__init__.py", line 2257, in getDeviceIterator
    device = USBDevice(
  File "C:\Users\c_ofl\AppData\Local\pipx\pipx\venvs\glasgow\lib\site-packages\usb1\__init__.py", line 1784, in __init__
    mayRaiseUSBError(result)
  File "C:\Users\c_ofl\AppData\Local\pipx\pipx\venvs\glasgow\lib\site-packages\usb1\__init__.py", line 127, in mayRaiseUSBError
    __raiseUSBError(value)
  File "C:\Users\c_ofl\AppData\Local\pipx\pipx\venvs\glasgow\lib\site-packages\usb1\__init__.py", line 119, in raiseUSBError
    raise __STATUS_TO_EXCEPTION_DICT.get(value, __USBError)(value)
usb1.USBErrorIO: LIBUSB_ERROR_IO [-1]

With this patch you instead get what you expect:
$ glasgow list
C3-20240303T140525Z

This was something we have added to ChipWhisperer's USB discovery code, you can see here if useful!

Could you change the commit message please to remove the .py suffix?

I think the force-push fixed that, lmk if not. Thanks!


owncloud: don't store plaintext adminPassword in nix store

Rather than using openssl to hash the password at build time, and hence
leaving the plaintext password world-readable in the nix store, we can
instead hash the password in the nix expression itself using
builtins.hashString.
I'm new to nix and nixpkgs so I would appreciate feedback on this PR.  In particular, I note that hashString isn't used anywhere else in nixpkgs so I wonder if this somehow is not idiomatic.

Who cares, looks cool. Merging, cc @edolstra if he's not in favor of using hashString we can revert :)


Update martin binary

Is your feature request related to a problem? Please describe.
The current martin binary (0.5.0) is from 2019.
Describe the solution you'd like
Check if there is a stable tagged version since 1.0.0 is still alpha and latest version should maybe not used as tag
https://hub.docker.com/r/urbica/martin

Unfortunately there is just an alpha release 1.0.0 with no recent updates since 2 years

martin merged with maplibre, so the namespace has changed.


enhance replaceReducer with optional nextState parameter

It helps to recover app state which is rendered on server as sync app, but client is built as code-splitted app.
Each dynamic module injects its reducers when it has been loaded.
But at the moment clients store is initialized with only root reducers from root module and store state lost most of the data from the server.
nextState parameters helps to recover server state for those code-splitted app at the first render

This was last tried with #2551. You can do this on your own: https://gist.github.com/cvbuelow/ea7bd4deb7824fdb65d81cd2b8c60f5e
This doesn't belong in the core library, because it leads to abuse and buggy behavior.

@timdorr thanks for the link.
But why it leads to abuse and buggy behavior?
It's well know design pattern - code split for routes.
In case of a large application with a very big state there's no logic to keep all reducers in one place.


psycopg2.OperationalError: server closed the connection unexpectedly

I'm using docker to run a nominatim instance:
  # Nominatim geocoder
  nominatim:
    image: mediagis/nominatim:4.0
    volumes:
      - nominatim-data:/var/lib/postgresql/12/main
    environment:
      PBF_URL: https://download.geofabrik.de/europe/netherlands-latest.osm.pbf
      REPLICATION_URL: https://download.geofabrik.de/europe/netherlands-updates/
    shm_size: 3gb
    restart: always
    deploy:
      resources:
        limits:
          memory: 6G

At first start the container starts the indexing process, after hours and hours is crashes. I attached a screenshot to show the output:

It then restarts and drops the database content.
Hardware Configuration (please complete the following information):

RAM: 8GiG
number of CPUs: 4
type and size of disks: ssd
bare metal/AWS/other cloud service: Vultr

Seems like you only allow docker to use 6GB of ram, which is probably too low. You can try to set environment variables documented in https://github.com/mediagis/nominatim-docker/tree/master/4.0#postgresql-tuning & https://nominatim.org/release-docs/4.0.1/admin/Installation/#tuning-the-postgresql-database to reduce RAM usage and give it another go.


Auto tmux on startup fails with error "Not a terminal"

Environment
Windows build number: 10.0.19041.630
Windows Terminal version (if applicable): 1.4.3243.0

Any other software?

I am running `zsh` 5.8 (with `oh-my-zsh`) in WSL 2.

Steps to reproduce

Add the following to ~/.zshrc:

# Always work in a tmux session if tmux is installed
# Based on https://github.com/chrishunt/dot-files/blob/master/.zshrc
if which tmux 2>&1 >/dev/null; then
  if [ $TERM != "screen-256color" ] && [ $TERM != "screen" ]; then
    tmux new-session -A -s main; exit
  fi
fi

Set Windows Terminal default to WSL and set default shell to zsh.
Try repeatedly opening Windows Terminal (if it opens successfully, then do CTRL+D and try again).

Expected behavior
Windows Terminal should always open normally with zsh inside tmux.
Actual behavior
Windows Terminal frequently fails on startup with:
open terminal failed: not a terminal

[process exited with code 1]

Does this only happen with zsh? How often does this fail if you try it repeatedly

I was unable to reproduce this with vanilla bash by putting the same code in my .bashrc.  I would say it fails about 1/3 of the time... you need to actually make sure the tmux session closes (rather than just detaching) in order to reproduce this exact error, although sometimes if you just detach you can still get a different error.
Right now I am using a temporary workaround that automatically opens a new Windows Terminal window when tmux fails, basically "trying again" -- so far, it seems like tmux never fails the second time around (in the new window). Also, if you modify the .zshrc to not exit after tmux fails, then you can just run tmux new-session -A -s main manually in the terminal and it also seems to never fail.

Thanks for the report! As far as I can tell, this is either a WSL or tmux issue. The terminal presents the same interface to all client applications, on all startups -- that of an interactive Windows Console console session. This would be the same as running conhost wsl repeatedly -- the window might just disappear too quickly to be seen.
This would probably be a good issue to file on the WSL repo.


chore: 📦 update packages to latest

BREAKING CHANGE: upgrade chakra-ui to v2 and react to v18

https://chakra-ui.com/guides/integrations/with-storybook
https://storybook.js.org/addons/storybook-addon-performance

Dependency:

need this to be merged first and update react-sortablejs package:

https://github.com/SortableJS/react-sortablejs/pull/234
https://github.com/SortableJS/react-sortablejs/issues/233


Cryptic error caused by space in path

I am not sure if this is a Sublime Text issue or an issue with Sublime Debugger but it occurs when I press the Start button of the Sublime Debugger:
Today I tried using the Sublime Debugger and it suddenly didn't work anymore so I tried different things and figured out the cause of the issue and how it can be worked around.
As soon as I pressed the Start button of the Sublime Debugger I got the following cryptic error that is of no help:

The cause: It turned out that the issue was caused by a space in my username ("$firstname $lastname" as created by my employer) which leads to a space in my user folder. The debugger worked just fine after moving the files to another folder (one without a space).
The workaround: When I add the following lines to the *.sublime-project file, the error does not occur:
"folders":
	[
		{
			"path": ".",
		}
	]

This should be fixed


Grid Dynamic size + measureElement examples don't handle going from large to small

Describe the bug
Hello! Right now when you look through the dynamic sizing examples, the base dynamic measureElement works great insomuch it handles variable sizes.
One thing, though, is that it doesn't handle a size of a cell going from large --> small, due to the fact that the cells get a minHeight: row.Size stuck onto them; the measureElement call won't recalculate to go smaller than what it was prior. Is there a good example of what the intended workflow would be?
Your minimal, reproducible example
https://codesandbox.io/p/sandbox/unruffled-pare-j2u30h?welcome=true&file=%2FREADME.md
Steps to reproduce

Render gird
Hit button to swap to a smaller data set
See that the the cells heights are pinned to the larger cells

Expected behavior
As a user, I expect the row to remeasure when the rows get smaller
How often does this bug happen?
Every time
Screenshots or Videos
No response
Platform
MacOS, Chrome
tanstack-virtual version
^3.0.0-beta.54
TypeScript version
No response
Additional context
No response
Terms & Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I understand that if my bug cannot be reliable reproduced in a debuggable environment, it will probably not be fixed and this issue may even be closed.

Ahhhh actually - I think setting getItemKey in the Virtualizer solves this issue :) Closing!


Add more modules to search.nuschtos.de

https://gitea.c3d2.de/c3d2/nix-config/src/branch/master/hosts/nixos-misc/default.nix#L50-L80
look at my flake inputs
look at dd-ix flake inputs

home manager has been added


flexibleSpace not supported

When using the current latest version v0.4.3 it flickers when trying to use flexibleSpace in either of MorphingAppBar or MorphingSliverAppBar.
https://github.com/user-attachments/assets/05f11664-3f1c-4d59-ad0a-affcaa47c101

Released as v0.4.4


Driver-NSQ Syntax Error

Driver-NSQ Benchmark Driver.java has extra bracket at the bottom which prevents a complete build with master.

same error here

https://github.com/openmessaging/openmessaging-benchmark/pull/106
Bug fixed


Error: Failed to quote swap from ADDRESS1 to ADDRESS2. Check that a pool exists for this pair [GOERLI]

When i create a LP en V3 i can't swap from uniswap-cli.
ADDRES1 is my deploy token
ADDRES2 is a deployed token, is not mine and is not wETH
If I go to uniswap UI i can swap them but if i try to do using the client, i'm giving this message error:
`Error: Failed to quote swap from ADDRESS1 to ADDRESS2. Check that a pool exists for this pair.
The code I'm using:
cargo run -- -u https://eth-goerli.public.blastapi.io  -n 5 -a<PHONE_NUMBER>0000000000000000 -s<PHONE_NUMBER>0000000000 -i ADDRESS1 -o ADDRESS2 -b 2 -v
When I'm using the TOKEN is not mine + wETH Goerli is working, because there is a LP with both, but with ADDRESS1 + ADDRESS( I created a pool and on uniswap is working) is not working and my Token + wETH Goerli is no tworing too.
`(Obviously i checked that the Addresses are right). I'm using the Router and Quoter by default on the lib and the LP is on V3.
Some  thoughts about this????
Thank you!

It's so weird.. I'm creating the LP using Uniswap UI, and it's look like there isn't a pool creation. But i can swap on Uniswap... I'm so confused right now.

Okey, if we are using ROUTER and QUOTER to swap two tokens, one of these tokens must be paired with WETH.
To avoid it should use the UniversalRouter, same is using Uniswap DAPP.
So solved!

Okey, if we are using ROUTER and QUOTER to swap two tokens, one of these tokens must be paired with WETH.
To avoid it should use the UniversalRouter, same is using Uniswap DAPP.
So solved!

Sorry for the late reply!
Effectively it seems that UniversalRouter has become the preferred entrypoint for ERC20 swaps.
At first glance it would require a bit of tweaking to support it but you're very welcome to open a PR @DiMiTriFrog. 😊


Nothing is shown in the "02_ShowTexture" demo

I managed to run the program on Mac, but nothing appears in the window.
The error message says "Failed to get image"

According to the error message, the image file might not  be read.
Could you look for the "scene1.jpg" in "pics" directory ?
This program works on my mac book air without problem .. :/

Well, the relative path such as "../../pics/sceine1.jpg" doesn't work for the "out-of-place" build using cmake.
To fix this problem, it is necessary to use the predefined macro.
Look at the CMakeLists.txt in the 03_ShowObj. The path of the resource is defined as:
add_definitions(-DPATH_ROOT_DIR="${PROJECT_SOURCE_DIR}/..")

OK ! I just fixed the CMakeLists.txt.
How about this ?

you need to change this line:
    unsigned char *img = stbi_load("../../pics/scene1.jpg", &width, &height, &channels, 3);

as
    unsigned char *img = stbi_load(std::string(PATH_ROOT_DIR)+"pics/scene1.jpg", &width, &height, &channels, 3);

Sorry, I have written as you show, but it doesn't work...

You can find what's wrong by closely reading the function interface of the "stbi_load".

Look here:
https://github.com/goto-c/OpenGL/blob/32115bd0fc4dc86ecdbb57c806ecca3573b64e7a/02_ShowTexture/stb_image.h#L1280
I think the first variable needs to be changed to const char*. If you are not familiar with C++'s std::string, look at this article:
https://stackoverflow.com/questions/7352099/stdstring-to-char

Thanks, I managed to fix the error, and now the white window is shown with the error message "Failed to get image". I will continue to fix this.

look at the file path
std::cout << std::string(PATH_ROOT_DIR)+"pics/scene1.jpg" << std::endl;

I could finally show the texture ! Thank you so much for helping me.
Could you please check it out ?

Nice! the demo ran successfully on my side.


On homepage, offer simple way for testers to log into fake Civiform + Program Admin views

Our testers won't be able to authenticate with ADCS/IDCS, and yet they'll need to test out the Civiform Admin (formerly UAT Admin) view and the program admin view. So add some simple way for testers to log into a fake Admin view, where they can see and work with fake forms and user data. Again, it's just for testing, and we can remove this "cheat code" before launch.

This is connected with #522 and #523, which will be key for testing.

OK so guest login for applicants is available on seattle.civiform.com. I don't think we have login for program admins yet though.

Do not need program / civiform admin log-in if third party.


ST_Pixelize small polygon error

Expected behavior
ST_Pixelize returns 0 pixels
Actual behavior
ST_Pixelize throw assertion:
Caused by: java.lang.AssertionError: assertion failed
at scala.Predef$.assert(Predef.scala:208)
at org.apache.spark.sql.sedona_viz.expressions.ST_Pixelize.eval(Pixelize.scala:119)
at org.apache.spark.sql.catalyst.expressions.Alias.eval(namedExpressions.scala:160)
Steps to reproduce the problem
case class A(g : String)
val a0 = "3.1" // change a0,a1 to values so an integer point lies between them
val a1 = "3.8"
val d = Seq( A(s"POLYGON (($a0 $a0, $a1 $a0, $a1 $a1, $a0 $a1, $a0 $a0))"))
import spark.implicits._
val df = d.toDS().toDF()  .withColumn("geo", expr("ST_GeomFromWKT(g)"))  .withColumn("area", expr("ST_Area(geo)"))
df.select("geo", "area").show(false)
val df2 = df  .withColumn("px", expr("ST_Pixelize(geo, 10,10, ST_PolygonFromEnvelope(0,0,10,10))"))  .show(false)
Settings
Sedona version = 1.5.0
Apache Spark version = 3.3.0
Apache Flink version = ?
API type = Scala
Scala version = 2.12
JRE version = 1.8
Python version = not tested
Environment = Databricks?

suspect the asset https://github.com/apache/sedona/blob/master/spark/common/src/main/scala/org/apache/spark/sql/sedona_viz/expressions/Pixelize.scala line 119   assert(pixels.size() > 0)

@ricg72 ST_Pixelize is not supposed to return 0 pixel. For any geometry (polygons, points, ...), it should return at 1 pixel. There might be something wrong with the logic itself. Do you want to take a stab?

Hi,
what would the spec be ? -- if the object falls within a single pixel then displaying that single pixel is ok - it's the longer thin items that I am not clear how to display - maybe the algorithm could convert the polygon to a line (skeletonize ?) and then draw those pixels ?
I was actually trying to use ST_PIxelize to get all the pixels coordinates in a polygon to pass to RS_Values to get all the pixel values in a polygon.    Is there a better way to do this ?
It's important to know which pixel came from where and to control precisely which pixels are inside and out of the polygon (a shift of 0.5 of a coordinate caused problems!)

Depending on what you want to do with the resulting pixel values, a few options:

RS_Clip: Clip/Crop the image by the given geometry
RS_AsRaster: Rasterize a geometry to a raster using a reference raster:   . Given two rasters, you can RS_MapAlgebra to perform arbitrary operations on values of two rasters
RS_ZonalStats: calculate the agg values of pixels inside a given geometry

@ricg72

Hi,
thanks for the suggestions - I'll try and test RS_Clip - (the RS_AsRaster / RS_ZonalStats - won't work)
I suspect RS_Clip is going to cause performance issues because there are many geometries per image - the issue is how to prevent the image being read multiple times or being shuffled.  Only way to tell is to try!
Another approach might be to update RS_Values to take an array of polygons instead of just an array of points - it would need to return an array[array[pixel values]] so that we could tell which pixel values belong to which geometry.


Add a way to check if a component responds to an event, or make vm._events public

What problem does this feature solve?
It would be nice to be able to check if a component responds to a given event or not. We can already do so for listeners declared in the template through the vm.$listeners object, but the ones registered with vm.$on() / vm.$off() don't show up in this object, they are stored in the private vm._events object.
I would like to suggest to either add a function that checks if a component responds to a given event, or to make vm._events public, and call it vm.$events
What does the proposed API look like?
A method called vm.$responds(event) could be considered, returning true if a component responds to a given event, false otherwise. If an array of strings is passed to this method instead of a string, it could return true if the component responds to some of these events.
Alternatively, the interface could simply be vm.$events

When do you need this?

It's to permit an optimisation that only builds a larger parameter object for a given even if the event is actually to be received by the component on which it is emitted.

As you explained yourself, for parent/child events, you can already$listener, so I think @posva is interested in understanding what use case requires to emit and listen to an even within the same component., instead of, for example, calling a method if flag: true` is set in the component's local data.
It's not something I've ever come across so far, even though it's technically possible, of course.

I am not listening to the event from the same component. I am simply listening to this event programmatically through a call to $on(), instead of directly from the template. And when doing so, the event doesn't show up in $listeners, only events that are provided in the template do so.
This is something I am using in @ditojs/admin: https://github.com/ditojs/dito/tree/master/packages/admin, a Vue.js based framework to build admin UI from simple Pojo schemas. And since much of the connecting is happening through code and processing of these schema, i cannot listen the vents in a template.
Since one is exposed, I was wondering why isn't the other.

Oh, the use case was clear to me: you don't want to emit if the component isn't listening to the event because building the payload for the event is expensive

Since one is exposed, I was wondering why isn't the other.

$listeners didn#t exist until Vue 2.3 or something, it was added after a lot of people asked for it, it's useful, for example to pass down listeners like you pass down props - which happens a lot in functional components and HOCs.
OTOH, no one asked for what you asked for so far, so we didn't think of adding it. No need for an API that nobody wanted so far.

. And since much of the connecting is happening through code and processing of these schema, i cannot listen the vents in a template.

A quick search didn't bring up any use of $on in the codebase (github code search ignoring special chars is no helping), care to link to one?

It's mostly used from the outside through the schema. Here the events get installed:
https://github.com/ditojs/dito/blob/d5ed796e9705d47a4395e0d788850d9dda91edc1/packages/admin/src/mixins/DitoMixin.js#L294-L298
And here the framework itself uses it internally:
https://github.com/ditojs/dito/blob/d5ed796e9705d47a4395e0d788850d9dda91edc1/packages/admin/src/components/DitoSchema.vue#L189
Here's my current workaround:
https://github.com/ditojs/dito/blob/91d1de30df3795f775c5c73a8db68b9164176e75/packages/admin/src/DitoComponent.js#L43-L97

Just an idea: instead of emitting the costly object you could emit a function that, when called, returns the costly object. Then the costly object is only ever created when someone actually calls the function you emit which in turn should only happen when someone is listening to your event. Just my two cents. Not sure if this any good though. 😁

@ChristianKienle a good idea, but this is for a public-facing API, and I'd like to use a similar approach as Nuxt does for its hooks, where you use destructuring to pluck the parameters that you need out of one single object argument provided to the callback...

In the meantime I've transitioned fully away from Vue's own event mechanism, since I needed the callback to be async and also queued. I guess we can close this issue, but maybe $responds() would still be useful to others? Here's my EmitterMixin: https://github.com/ditojs/dito/blob/master/packages/admin/src/mixins/EmitterMixin.js

@lehni just saw that you are behind Paper.js. 😍
I regularly play with the Voronoi example because I noticed that it calms me down and helps me relax. If you are in the area ping me for a beer/coffee/whatever :)

@ChristianKienle ha that's nice to hear, thank you! Actually, @puckey is the author of that example, he'll be happy to hear that I'm sure.

Given that $on and $off are removed in Vue 3, I don't think there is any intention in implementing something like this.
This could be implemented in user land using an event emitter (https://v3.vuejs.org/guide/migration/events-api.html#migration-strategy) and combining it with composition API to automatically turn off events when the component unmounts


OAK-9911 - Resource leak in ChunkedBlobStream

Backported this from trunk. I had to make minor changes in TemporaryFileStore.java because  FileStoreBuilder.withBinariesInlineThreshold() is not supported in 1.22 (OAK-6911 is not backported to 1.22).


[Streaming] Lambda Timeout when using React Suspense

In a regional deployment with streaming enabled, when the Next.js App uses React Suspense to stream the UI changes, the first chunk is received successfully in browser, but the following changes are not streamed by the lambda and after 10s the lambda timeout and the connection is closed in the browser.
I've added some debug logs to the responseStreaming adapter.

And after some investigation I've noticed that happened only when the this.responseStream.writableNeedDrain is true, which returns false to the socket write function.
When I return true regardless the this.responseStream.writableNeedDrain for my use case it always works.
I've also noticed that in this https://github.com/sst/open-next/pull/213/commits/d32d83d042c3590877c9fedf5a4a1deb90473f1b the return block was changed to return the value based on the this.responseStream.writableNeedDrain.
@khuezy I wonder why you had to do that, I'm assuming there's a reason for that, could you please tell me the reason?
I'm happy to raise a PR to make the return always true, but I'd like to understand the context first.
Thanks

Hey @lucasvieirasilva thanks for investigating. To be honest I don't recall, we had to hack a bunch of stuff b/c AWS' streaming implementation has been poor since they released the feature. I hear they are aware of the issues and have a fix soon.
The streaming functionality on lambda seems to be very brittle, it may work in one scenario and break in others. Here's hoping they fix everything.

Hey @khuezy, yeah, and the AWS docs about it are not good as well, in the meantime, should we change the write function to always return true or leave it as is for now?

I don't know what the state of the lambda runtime is at now. You can try to revert the change to return true and see how the e2e tests behave. iirc, the changed helped w/ the buffered response but that might have been a coincidence when I was testing.

I don't think it's a good idea to touch that, at least not like that.
To solve this we probably need to send a drain event. If we return true the stream will not be able to handle the backpressure. And because of how hacky the implementation is, it might fix something for you but break for someone else.
Streaming is considered experimental in V2 and stuff are expected to break.
Streaming in V3 is totally different than here, so everything done here is useless, and V3 is very close to being ready.

Should be fixed in V3.
Feel free to reopen if that's not the case


Add Feature: Warp between Worlds

A "World" is the area of play space in which the player interacts with the world, mostly by navigating around it with their Ship.
How does the warping feature work?

A: The player can warp freely to any node within their range. Warping longer distance requires more warp fuel.
B: The world nodes are connected through warp lanes. The player must often travel through several world nodes as they follow the path to their destination.

How will worlds be designed & stored?

A: Worlds are hand-made scenes, which are loaded and added into the World node of the game.
B: Worlds are constructed in ConfigFile-type text files, and are generated dynamically onto a blank template world scene.
C: Some mix of A and B.

How will the transition function...um...function?

A: A destination is determined, and the warp sequence triggers the scene change.  Under the Space viewport, the current child is freed, and replaced with an instance of the destination scene. The player is placed at an appropriate position/orientation, and the game picks up from there.
B: That's about the only good way I see that happening...

How does the warping feature work?
Use method B. Will allow for a more cohesive design, and more streamlined game play mechanics.


Error "withConfig is not a function"

Hi,
I just installed the plugin as described but when I try to bundle code using webpack the next error arise:
TypeError: _styledComponents2.default.h1.withConfig is not a function
Am I missing something? Any help?

Are you trying to use it with styled-components v1 possibly?

I am receiving the same error and am using Styled Components v1.
Will try now with v2. Thanks!

This plugin only works with v2—we should add that to the README!


Simplification proposal

Hi @twiecki,
I don't think there's a need to fully clone the John Hopkins repo. It is in fact sufficient to pull the raw data as follows:
import requests

...

base_url = 'https://raw.githubusercontent.com/CSSEGISandData/COVID-19/master/csse_covid_19_data/csse_covid_19_time_series'

...

def load_timeseries(name):
    url = f'{base_url}/time_series_19-covid-{name}.csv'
    csv = requests.get(url, proxies=proxies).text
    df = pd.read_csv(io.StringIO(csv), index_col=['Country/Region', 'Province/State', 'Lat', 'Long'], parse_dates=True)
    ...

Note that I have split the reading of the CSV text and the pandas parsing into two steps because I'm behind a corporate firewall and need proxies. You could combine the two into one by directly specifying the url in the call to pd.read_csv().

Hi @kasparthommen. Thanks, that makes sense.

Implemented.

:+1:


add the hovering effect #40

Adding hover effects #40

@anubhab1001, this issue has already been solved.


[BUG] harvester -installer CI failed (Error response from daemon: read )

Describe the bug

To Reproduce
Steps to reproduce the behavior:
Run harvester-installer ci
Log

Additional context
Add any other context about the problem here.
$ docker save quay.io/kubevirt/virt-launcher:v0.38.1 -o image.out
Error response from daemon: read /var/lib/docker/overlay2/14142766c26e92075713e727c84efc6431f4dbb7fbf72f3fcf5c7ff3c109ef77/diff/lib64: is a directory

Use ctr to walkaround rancher/harvester-installer#49.  Need more investigate.

https://github.com/moby/moby/issues/42114

close in favor of https://github.com/rancher/harvester-installer/pull/49 with changes from docker to ctr.


[Bug]: On importing any json file, we see an error on the modal - Authorization failed, please check your details or try again

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Import any JSON file, observe the error on the modal as attached in the screenshot
https://drive.google.com/file/d/1czMu42-yC9uL-RbDG077sVOO2MHd6Aeg/view?usp=drive_link
Steps To Reproduce
Import any JSON and observe the error in the modal
Public Sample App
No response
Environment
Production
Issue video log
No response
Version
1.9.28

not reproducible anymore, maybe got fixed with something else
https://github.com/appsmithorg/appsmith/assets/94514895/21a3b7c7-56c1-4c38-965d-95a2ab074ee9


Add signal broadcast to server-lambdas

The purpose of this PR is to mainline the dev broadcast-signal branch.
This PR only contains the code to broadcast a signal into the room in server-lambdas. It needs to be released and then consumed by tinylicious to unblock the tinylicious PR which contains the broadcast-signal endpoint in tinylicious..
A third PR that is blocked on both consumes the changes in the server code and showcase how to use it in the external-data app.

Comments below. Also, I think a new file for the new interfaces might be better than adding to index.ts. E.g. scribe has an interfaces.ts.

Fair, I was originally trying to put these into core, but thought better of it. But a new interface file seems like a good in-between.

What was the testing strategy for validating this feature in routerlicious? (apologies if this has been mentioned before - I didn't get a chance to go through the whole discussion)

I have a dev branch here to test out the full flow: https://github.com/microsoft/FluidFramework/tree/dev/external-data-prototyping.

Here are the test results:
New server chages (local tests running against local docker build with new changes):
    "suites": 665,
    "tests": 1687,
    "passes": 444,
    "pending": 962,
    "failures": 473,
    "start": "2023-09-08T00:17:44.725Z",
    "end": "2023-09-08T00:26:27.395Z",
    "duration": 522670

Server code on main (local tests running against our servers that run the end to end tests):
    "suites": 664,
    "tests": 1617,
    "passes": 196,
    "pending": 972,
    "failures": 656,
    "start": "2023-09-08T00:32:31.606Z",
    "end": "2023-09-08T01:27:08.281Z",
    "duration": 3276675

I've also run clicker locally and that works.


only accepts lowercase chars in names to analyze

i will try to fix this soon. could open up a PR if you guys don't get around to it first.

It's fixed now (bit late, I know).

:+1:


feature(code-health): added coveralls and coverage badge in README.md

Description
Added coverage badge to display coverage %.
https://github.com/nickmerwin/node-coveralls
What's included?

coverage badge
Travis CI setup

Test Steps

[ ] See coverage badge in README.md
[ ] Go to https://coveralls.io/github/Teradata/covalent to see more information.

Changes Unknown when pulling 32ae8c645810937129a9bf74008daf63d7a5c970 on feature/coverage-badge into ** on develop**.

Changes Unknown when pulling dca305c2fc1f7e17f13e6988d21498b9372b8d4a on feature/coverage-badge into ** on develop**.

Changes Unknown when pulling baea58a19f46859d3d752620ea88550cb7e0bcf5 on feature/coverage-badge into ** on develop**.


Allow cache store used by console scheduling integration to be overridden

An initial discussion about the implementation of this change can be found here.

Allows the cache store used by ConsoleSchedulingIntegration to be overridden in a user-defined service provider
Uses that cache store for storing state when a job is run in the background

@stayallive @cleptric thanks for the reviews! Any idea when you're planning to cut the next release?

We'll get a new release out next week.

This is now available in 4.9.0.


Enable/Disable Polyglot for /me, /ooc, /desc

At the moment, by default, Polyglot has the following settings:
/me = Disabled
/ooc = Enabled
/desc = Enabled
What I think would be a good setting is to select if Polyglot should run for /me, /ooc, or /desc by selecting if it should be enabled or disabled for these types of communication. It would allow users to select when they expect Polyglot to run on their own settings.
Describe alternatives you've considered
I avoid /me when I need translation, and I use languages all of the on-screen players know to make sure Polyglot does not do translations.

/me is meant to be used as an emote message, like "[Character Name] does thing", it isn't meant for characters to speak.
/ooc can be toggled by the GM for all users, under the General Settings' Chat tab. It is disabled by default for players.
/desc isn't a Core function, it's from the Narrator Tools. Change the Narrator's message type setting back to "Other".

What about /desc from Cautious Gamemaster Pack? It doesn't have an equivalent setting, would this be something that would have to be resolved on the other module's side?

I'll add a check for CGMP's flag. Expect it on the next version (2.3.2+).
Also, after my last reply I introduced a checkbox to disable the language being applied to the message, so you can use that in the meantime.


Unable to hold down the backspace button

React-simple-keyboard version
3.4.51
Describe the bug
https://codesandbox.io/s/github/simple-keyboard/react-simple-keyboard-preventmodefault-caret-pos/tree/master/
I am unable to hold down the backspace button. Sometimes it works but it seems to only occur when controlling the caretPosition.

Hello @ikevinws, thanks for raising this. I have reverted one caret positioning change that might have caused this.
Feel free to try out<EMAIL_ADDRESS>and let me know if you can still repro the issue.
Regards,
Francisco Hodge

@hodgef Thanks, the problem has been fixed on the latest version.


Support of proper Run life-cycle

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Document the changes of scheduler PostFilter extension point

This PR documents the changes of the new scheduler PostFilter extension point.
/sig scheduling

/cc @sftim @alculquicondor @savitharaghunathan

/milestone 1.19

@alculquicondor, I understand the thinking behing https://github.com/kubernetes/website/pull/22429#discussion_r452486722; however, Concepts pages are not tied to a specific version of Kubernetes. This page could feasibly be used to document Kubernetes for many future releases.
When you want to communicate a specific change in behavior between releases, the release notes are the right place for that and the Concepts section is not the right place:

Readers looking to understand the changes between versions won't look there.
Content talking about changes between versions becomes stale, and that staleness is difficult to track when it is mixed into Task or Concept documents.

If it's very important to signpost readers to the v1.19 release notes from the Scheduling Framework concept, we could track a pair of issues to add that signposting and then to remove it again after a few releases. I don't think it's worth it though.

however, Concepts pages are not tied to a specific version of Kubernetes.

I understand this. But in this case, the current wordings are already stale, in the context of 1.19 release. So do think it's better to add a tabbed widget to point out different steps to disable preemption upon different releases?

When you want to communicate a specific change in behavior between releases, the release notes are the right place for that and the Concepts section is not the right place

Given that we only document major API changes between versions (like alpha to beta, beta to GA), and a version may span multiple releases, IMO it's a supplement to release notes guiding users how to upgrade. For now, we document the changes from alpha to beta, and in the future, we only need to document changes from beta to GA. And yes, at some time, if alpha/beta version is basically not used, we can remove them.

I'm also ok with leaving this detail to release notes, but I don't think what the appropriate mechanism is for keeping them together, given that they are extracted from PR descriptions.
@sftim any process around that? Maybe we can just file a release blocking issue so that bring awareness to release managers?

I'm also ok with leaving this detail to release notes, but I don't know what the appropriate mechanism is for keeping them together, given that they are extracted from PR descriptions.

I'm not sure about the way Kubernetes does this. Is it viable to create a PR solely for the side-effect of adding a release note (that summarises the changes), and then get that merged?

Is it important that readers specifically understand how to disable preemption? I don't have a read on how common that task is (I have never needed to do it or needed to explain it to someone who is running a cluster). One option is to drop that detail entirely from the Concept section - I'm not convinced it belongs there.

Is it important that readers specifically understand how to disable preemption?

I'd say 99% (or even more) of the users wouldn't try disabling this. So I'm fine with removing the whole section.

@sftim @alculquicondor Addressed most of the comments except for the release notes part. PTAL.

@kubernetes/release-managers to advice on how to handle Release Notes

One option is to add a README file in the pkg/scheduler/api/configs/beta that describes the changes compared to v1alpha2

I don't think the main repository has enough visibility for people upgrading.
Friendly ping @kubernetes/release-managers for advice

One option is to add a README file in the pkg/scheduler/api/configs/beta that describes the changes compared to v1alpha2

I don't think the main repository has enough visibility for people upgrading.
Friendly ping @kubernetes/release-managers for advice

Hey, I see two ways to handle the release notes scattering across multiple PRs:

collect the user facing changes only in a single PR which is included in v1.19 (by editing the PR body)
reach out to the release notes team of SIG release after code thaw to change the notes as wanted

We right work on a feature to map/collect multiple PRs into a single release note but this is not ready to be used, yet. :)

@pancernik kindly updated this PR description kubernetes/kubernetes#91420
@Huang-Wei could you remove the "changes" section?

@alculquicondor the whole "Upgrading from v1alpha2 to v1beta1" section is removed.
@sftim @saschagrunert PTAL.

Page previews:
https://deploy-preview-22429--kubernetes-io-vnext-staging.netlify.app/docs/concepts/configuration/pod-priority-preemption/
https://deploy-preview-22429--kubernetes-io-vnext-staging.netlify.app/docs/concepts/scheduling-eviction/
https://deploy-preview-22429--kubernetes-io-vnext-staging.netlify.app/docs/reference/scheduling/config/

Hello @Huang-Wei . The changes look good. Thanks for adding the image!
/lgtm


Round down the quota usage

Fixes #2547
I'll propose we render the used amount of data with up to 2 decimals if necessary, rounded down to avoid the rounding to be rendered as "10MB" for this kind of edge-case:

In case the used data has no decimals it renders as before:

Please review @xavijam

Good to me :+1:

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)


Review design with respect to basic UX principles

Peformance load

Hardware: We designed the hardware to decrease both cognitive and kinematic load. We reduced kinematic load by using easy to push buttons and by making the more frequently used buttons bigger and closer to the screen, while settings buttons are in their own separate area. We also reduced cognitive load by designing stickers that easily explain the buttons given purpose. If you want to quickly get the machine working we also pre installed settings and timings so that the user can begin immediately if they don't want to bother playing around with the settings.

Software: We reduced kinematic load by placing the text boxes where you select your desired times or your tasks in the middle of the screen near the start button so you don't have to move the mouse far. We reduced cognitive load by placing the start timer buttons near the boxes where you input the times. We also placed all of the inputs for the timers in a logical order, work time then break time. For the task mode, we put the box where you input the task on the left and the list of all tasks on the right, since westerners go from left to right.

Flexibility - usability tradeoff

While our idea was mostly focused on the accountability of the LEDs to make the person continue working, we wanted to make the product flexible so that as many people as possible would be able to study/ be productive in a way that suits them. We decided that we would have three different modes to try to accommodate as many study styles as possible. The first one is a the standard pomodoro method with a normal working and break time. The second one is where you input a given set of tasks for the day and mark them complete on your own time. The third one you set yourself a given amount of break time for the whole day. You can set these functions on either the hardware or software, but to keep the hardware more simple we allowed only timing changed of 5 minutes but you can get more technical on the software for those who enjoy customizability.

Visibility

On the physical device we will have a display that will show what mode they are editing and if they are currently in a mode which mode they are in. The software you are not allowed to leave the given page so you know what mode you are in. We didn't make it too complicated the use should be able to figure out who to cycle through the modes and input their preferences with relative ease and just scrolling through the options. The settings buttons are also placed off to the side and are smaller so that the user sees them but knows they are not as important.

Feedback

When updating the settings it will be visible on the hardware display that you are updating the desired settings for that mode as the times/ tasks will change. When in an actual mode, there will be like a status pane in the bottom portion of the display that will let the user know their current mode, whether its break time or not for pomodoro mode, how many tasks left for task mode and other useful information. If pressing the play/pause button or the off for day button we can also play a sound on the buzzer to let the user know that the button was pushed or after all the settings have been completed.

Constraint

We don't allow the user to access the turning on or off of LEDs as this would defeat the purpose of the whole design and the idea of them having to be productive to be rewarded to have the LEDs turn on to make the tree look nice. We only allow the user to input their times and they are responsible for being honest with themselves and getting the work done to get the proper visual reward. The user has their mistakes minimized because of the way we organized the buttons and the software interface to make it very clear and easy to how the convention works. When inputting the settings, they have to move to the next "level" of settings selection by once again pressing the settings button which is similar to how those digital watches are programmed. Then we have a basic up down button which will increase or decrease the time as they want.

Forgiveness

Our design also helps minimize error through forgiveness since we have an exit button on the software which will allow the user to exit button on software or the done for day on the hardware to cancel the given mode if they input an incorrect time when initally setting up. If they are midway through  one task and they decide that they do not want to continue with the last pomodoro or they initially inputted too many tasks, they merely have to go back into settings to change it and the LEDs and display will update accordingly.

Affordance

On our software interface, we will have little pictures for each mode that will afford the usage of that mode if the user forgets. For the pomodoro mode we will have a clock, indicative of its timed nature. For the task mode we will have a check, which is indicative of a task list that you check when completed. For the break mode we will have a coffee mug which is indiciative of taking a break. These images will help to negatively afford improper use. On the hardware display since the screen is quite small, we will make the mode be represented by the first letter of the mode. The design as a whole is afforded well as most people understand and want  a tree to flower and blossom, so the goal is to be productive to have the tree flower up with the LEDS.

Confirmation

Before the user can start any task after changing the settings they are prompted with an "OK?" to confirm these settings or else they will have to re input the settings if no action taken.


RFR: Add X-Request-ID API headers so we can trace request in logs easily.

Example:
(virtualenv)/m/s/s/st2 git:support_trace_id_in_cli_run ❯❯❯ grep "a64fd291-f9cf-4797-89e8-b4a58c8f12db" logs/st2api.log
2015-08-19 16:42:38,875<PHONE_NUMBER>36112 INFO log [-] a64fd291-f9cf-4797-89e8-b4a58c8f12db -  GET /v1/rules with filters={} (remote_addr='<IP_ADDRESS>',method='GET',filters={},request_id='a64fd291-f9cf-4797-89e8-b4a58c8f12db',path='/v1/rules')
(virtualenv)/m/s/s/st2 git:support_trace_id_in_cli_run ❯❯❯

(virtualenv)/m/s/s/st2 git:support_trace_id_in_cli_run ❯❯❯ http http://localhost:9101/v1/rules             ✭ ✱ ◼
HTTP/1.1 200 OK
Access-Control-Allow-Headers: Content-Type,Authorization,X-Auth-Token,X-Request-ID
Access-Control-Allow-Methods: GET,POST,PUT,DELETE,OPTIONS
Access-Control-Allow-Origin: http://<IP_ADDRESS>:9101/
Access-Control-Expose-Headers: Content-Type,X-Limit,X-Total-Count,X-Request-ID
Connection: keep-alive
Content-Length: 30806
Content-Type: application/json; charset=UTF-8
Date: Wed, 19 Aug 2015 23:42:38 GMT
X-Request-Id: a64fd291-f9cf-4797-89e8-b4a58c8f12db
X-Total-Count: 38

(virtualenv)/m/s/s/st2 git:support_trace_id_in_cli_run ❯❯❯ http http://localhost:9101/v1/rules                         ✭ ✱ ◼
HTTP/1.1 500 Internal Server Error
Access-Control-Allow-Headers: Content-Type,Authorization,X-Auth-Token,X-Request-ID
Access-Control-Allow-Methods: GET,POST,PUT,DELETE,OPTIONS
Access-Control-Allow-Origin: http://<IP_ADDRESS>:9101/
Access-Control-Expose-Headers: Content-Type,X-Limit,X-Total-Count,X-Request-ID
Connection: keep-alive
Content-Length: 36
Content-Type: application/json
Date: Wed, 19 Aug 2015 23:50:24 GMT
X-Request-Id: 15612f18-21b7-47b9-82fb-34657b7be2d5

{
    "faultstring": "500 testing"
}

(virtualenv)/m/s/s/st2 git:support_trace_id_in_cli_run ❯❯❯

(virtualenv)/m/s/s/st2 git:support_trace_id_in_cli_run ❯❯❯ grep "15612f18-21b7-47b9-82fb-34657b7be2d5" logs/st2api.log
2015-08-19 16:50:24,186<PHONE_NUMBER>15536 INFO log [-] 15612f18-21b7-47b9-82fb-34657b7be2d5 -  GET /v1/rules with filters={} (remote_addr='<IP_ADDRESS>',method='GET',filters={},request_id='15612f18-21b7-47b9-82fb-34657b7be2d5',path='/v1/rules')
(virtualenv)/m/s/s/st2 git:support_trace_id_in_cli_run ❯❯❯

Works for me.
+1

Ideally we would pass this ID through the whole pipeline, but that's a start.

Scrutinizer internal error I suppose. Merging to master.

FYI - https://tools.ietf.org/html/rfc6648
Looks like X- prefixes are not recommended any longer.


ISPN-12867 SoftIndexFileStore LogAppender optimizations

Reuse bytebuffer used by same thread
Offload passing off index submission to non blocking CPU thread instead of log
thread
Do not create a new file object when finishing a file
Delay is held in a List instead of async stage

https://issues.redhat.com/browse/ISPN-12867

Performance numbers compared to CR2
Async writes

Name
CR1
PR
Diff

write
27155.127
27617.127
1.701%

bulk
533.732
693.727
29.976%

write (16t)
54472.216
67034.161
23.061%

bulk (16t)
554.483
687.072
23.912%

ASYNC capped at around 16 threads on my box for performance.
Sync writes

Name
CR1
PR
Diff

write
201.113
202.675
0.776%

bulk
51.999
52.33
0.636%

write (16t)
2874.837
2784.835
-3.13%

bulk (16t)
350.555
381.711
8.887%

write (32t)
5092.437
5032.532
-1.176%

bulk (32t)
410.206
474.033
15.559%

All in all the peformance is nearly the same or better across the various writes. For async it is better in every case.

I have some additional changes I want to try for this. I will update later today with the results.

Added additional changes to move all but the minimum from the LogAppender writing thread.
This should provide for the highest scalability as the thread for the most part should only be writing to disk. In my JMC every thread dump was doing a force, except one. Also the amount of allocations in the log appender thread has been cut to less than 1% that it was before as the nio ByteBuffers are allocated in the invoking threads now intead.

18:09:22  [ERROR] /home/infinispan/workspace/Infinispan_PR-9164/persistence/soft-index/src/main/java/org/infinispan/persistence/sifs/NonBlockingSoftIndexFileStore.java:17:8: Unused import - java.util.concurrent.Executors. [UnusedImports]

18:09:22 [ERROR] /home/infinispan/workspace/Infinispan_PR-9164/persistence/soft-index/src/main/java/org/infinispan/persistence/sifs/NonBlockingSoftIndexFileStore.java:17:8: Unused import - java.util.concurrent.Executors. [UnusedImports]

Oops, used that for my testing. Missed it when I reverted :)

Merged, thanks

In my JMC every thread dump was doing a force, except one.

Maybe a flamegraph from async-profiler would be more relevant? Thread dumps aren't very accurate, especially since you can get a lot less thread dumps compared to profiler samples.

In my JMC every thread dump was doing a force, except one.

Maybe a flamegraph from async-profiler would be more relevant? Thread dumps aren't very accurate, especially since you can get a lot less thread dumps compared to profiler samples.

This included the method hot spot profiling as well btw.


Changing the vanilla workspace bundle to use Azure storage for Terraform backend

Closes #117
Changes from the Porter tracked Terraform state  to the state being managed by Azure backend.
How is this addressed

The environment variable names of the Azure credentials changed from AZURE prefix to ARM

This allows the easiest integration with the Terraform and Azure backend

Adds parameters for Azure backend (names for resource group, storage account, container and Terraform key)
Updates the install and uninstall actions to use the new backend
Updates the documentation to match the changes

Looks good, will run through once get this PR in.
As TF_VAR_tfstate_key needs to be unique per deployment of each bundle, can we actually calculate it in the porter YAML, something like:
key: "{{ bundle.parameters.tre_id }}{{ bundle.parameters.workspace_id }}"

This is fixed.


New version: TechSmith.Snagit.2023 version <IP_ADDRESS>45 (23.2.5)

Created by 🥟 Dumplings in workflow run #5673.

Log
Updated: 23.2.4 → 23.2.5
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request

Microsoft Reviewers: Open in CodeFlow

@wingetbot run


update page transition to allow ^2.0.0

animated_splash_screen 1.1.0 depends on page_transition 2.0.1-nullsafety.0 and no versions of animated_splash_screen match >1.1.0 <2.0.0, animated_splash_screen ^1.1.0 requires page_transition 2.0.1-nullsafety.0.
null safety naming standards have changed, you no longer require the suffix.
page_transition: ^2.0.2 should be valid

Because animated_splash_screen 1.1.0 depends on page_transition 2.0.1-nullsafety.0 and no versions of animated_splash_screen match >1.1.0 <2.0.0, animated_splash_screen ^1.1.0 requires page_transition 2.0.1-nullsafety.0.
So, because ledunia_phone depends on both page_transition ^2.0.2 and animated_splash_screen ^1.1.0, version solving failed.
pub get failed (1; So, because ledunia_phone depends on both page_transition ^2.0.2 and animated_splash_screen ^1.1.0, version solving failed.)


Add browser option to "ionic serve" so that you can specify browser

Add browser option to "ionic serve" so that you can specify browser.
ex) ionic serve --browser chrome --browserOption "/new-window"
(boot chrome with new window)

Nice hack! I love it. I'll be trying this out and merging it soon.

@lamrongol Due to some pretty big changes I made to serve.js, I had to merge this in manually! I really really appreciate the PR. Excellent work.


Backport the NPM fix from master on branch 0.5.6

What is this PR for?
Fix the zeppelin-web build error
[ERROR] npm ERR! registry error parsing json
[ERROR] npm http 200 https://registry.npmjs.org/bower/1.7.2
[ERROR] npm ERR! SyntaxError: Unexpected token
[ERROR] npm ERR! ï¿½ï¿½Y[oï¿½6ï¿½+ï¿½ï¿½6ï¿½ï¿½ulEï¿½ï¿½ï¿½
[ERROR] npm ERR! ï¿½{hï¿½ï¿½Cï¿½Ë‘ï¿½L=RJï¿½ï¿½oï¿½9bï¿½4ï¿½ï¿½ï¿½ï¿½W4ï¿½["ï¿½ï¿½wnï¿½ï¿½ï¿½Eï¿½ï¿½ï¿½2Cï¿½Ï•nï¿½ï¿½ï¿½U`ï¿½ï¿½aï¿½
[ERROR] npm ERR! Gï¿½p^ï¿½ï¿½$eï¿½Ley,ï¿½ï¿½IUï¿½"/Kï¿½,qrï¿½[8ï¿½Fï¿½ï¿½ï¿½^ï¿½ï¿½ï¿½pï¿½ï¿½ï¿½ï¿½ï¿½ï¿½Zï¿½ï¿½ï¿½ï¿½ï¿½=x?ï¿½}ï¿½ï¿½{Wï¿½+ï¿½ï¿½ï¿½Ü³Ð€ìµ±ï¿½ï¿½ï¿½}

What type of PR is it?
Hot Fix
Todos

[x] - Fix zeppelin-web build error

What is the Jira issue?
How should this be tested?
Screenshots (if appropriate)
Questions:

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

@jersub could you elaborate on the purpose of this PR please?
AFAIK there were no plans to do any further Zeppelin releases from 0.5.6 branch

I'm sticking to version 0.5.6 at work and I need to compile it to apply some custom patches. The aim of this PR is to allow everyone to do so.

I think that's a fair point we need to consider to support community. But as @bzz says we don't currently have any plan for more release from 0.5.6.

@jersub Can we close this PR?

Fine, let's reopen it if anyone is interested.


Set the Windows Terminal as the default terminal

Closes https://github.com/mandiant/VM-Packages/issues/971.

I am not sure this works as proposed in #971. @d35ha can you provide more details or how this works and how it looks like (maybe adding a screenshot)?

@Ana06
I've created a console app, here is it how its execution looks before and after setting these registry values:

Also, running cmd or powershell opens it in Windows Terminal.

@d35ha is proposing using the windows terminal to open the shortcuts in the Tools directory, is this PR achieving this?

@Ana06
Any application that requires a console would be assigned the Windows terminal console instead of the the old conhost.exe (that is including the shortcuts under the Tools directory).

I looked into this a little bit more.
I wanted to try and find a way to have any "console app" shortcut open with Windows Terminal (if it exists) or cmd.exe if not, but the way to make that happen would likely require an external script to launch the correct one. I tried creating a singular command that can be done within a shortcut target cmd, but it ended up being too long.
An alternative would be that we force Windows Terminal to be installed, but I believe this would require modification of our install script so that it is always installed if I'm not mistaken?
Let me know if that's the preferred route and I can work on that.

I got something mostly working for Command Prompt, but it seems like Powershell is going to be a lot more of an issue due to a long standing bug with Windows Terminal and how it interprets semicolons: https://github.com/microsoft/terminal/issues/13264
This makes it very difficult, if not impossible to do our powershell shortcuts. I was able to make one work, but it would require removing the "Write-Host" portion (echoing the command we write into the console).
For reference, I modified powersploit to use this as a shortcut target: "C:\Tools\Windows Terminal\terminal-1.19.10573.0\wt.exe" -p "PowerShell" "powershell.exe -NoExit -Command ""Get-Command -Module (Import-Module .\Recon\Recon.psd1)""
This still seems to work as expected, just a re-write of the original command we used: Import-Module .\Recon\Recon.psd1 ; Get-Command -Module Recon

Thanks for the research @emtuls! I suggest we merge this PR as it is, as it seems it is not easy to support it in our Windows version.


save_dir param used during checkpointing, fixes issue #705

Fixes saving issues with GridSearch. This is needed to run multiple, simultaneous jobs.

I.e. why not just pass in save_dir to the fit method, and then pass it through to the model.save call?

That works for discriminative models, where save_dir is already a parameter of TFNoiseAwareModel's train method. For the GenerativeModel , the train class doesn't include the save_dir param which causes an error when it's passed into fit. I just added it for expediency's sake, so I'm not wedded to where it goes.


Added private method for checking if user is linked with a given auth type.

Add new private method
Update PFAnonymousUtils to use it

cc @grantland

LGTM


Update the POH website to the latest iterations. Check the design.

Update the POH website to the latest iterations.
-Update the top section to white background and lighter avatar images.
-Include the play button to open the video explainer.
-Include the Pools link on the top navbar which scrolls to the Pools section of the page when clicked.
-Update the button (Share POH with the World) in the bottom section to yellow like the other buttons.
-No texts below section titles: (Unlocking new possibilities) and (Your digital identity).
-POH comparison chart on the right, after social graph analysis.
Check the design:

Done


Add range partitioner on DataFrame

What changes were proposed in this pull request?
Currently, a dataframe can be only hash partitioned using "Repartition", please add a new function that partition a dataframe by range
How was this patch tested?
Existing tests
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)

Can one of the admins verify this patch?

@tanshai can you reopen? the history is messed up here. Perhaps you want to open a pr against master.

@tanshai please close this PR. If you meant to send a patch, you probably did something wrong when creating the PR (like choose the wrong branch).

@tanshai can you please fix this PR or close it?


Client Allocation Request for: Rushradar

Client Allocation Questions
Core Information

Name: Rushradar
Website / Social Media: https://rushradar.com/
Region: North America
DataCap Requested: 5TiB
Addresses to be Notarized: f1onedsxngh2nlnsdc6uad2qljka3q5m6pgvleata
Notary Requested: dannyob

@JackFrancisco Please subscribe to notifications for this Issue to be aware of updates. Notaries may request additional information on the Issue.

Here's your datacap. Please remember that datacap should be used for material that is unencrypted and licensed to be distributed to the public. Please use your datacap to make deals with more than one miner. And please don't use a miner you have a pre-existing relationship with. Spread the love! Use your datacap for the public good! Any questions, mail me at<EMAIL_ADDRESS>or @dannyob on the Filecoin Project slack

Request Approved
Your Datacap Allocation Request has been approved by the Notary
Message sent to Filecoin Network

bafy2bzacebvcnowhgpvjcgwua2eruibrkbiugakizfj7vy2j37vikoi2ycm3e

Address

f1onedsxngh2nlnsdc6uad2qljka3q5m6pgvleata

Datacap Allocated

5TiB

You can check the status of the message here: https://filfox.info/en/message/bafy2bzacebvcnowhgpvjcgwua2eruibrkbiugakizfj7vy2j37vikoi2ycm3e


use yjs to sync project title

PR #205 should address this, vote to close.

Closing. #187 addressed it, not #205.

My bad, not #187, which didn't use Yjs. #187 cannot address #195 😂
This is still open. The title is not synced to peers in real time.


Twitch controls not able to hide when controls set to false

Current Behavior
I want to hide controls from the Twitch stream channel when I set controls props to false but the controls query still shows true.
Expected Behavior
The controls should be hidden when I set the controls props to false.
Steps to Reproduce

set controls props to false
inspect the iframe and you will see the controls query is still true.
Twitch stream control still shows

Environment

URL attempting to play:  https://cookpete.com/react-player/
Browser:  Google Chrome Version 92.0.4515.13
Operating system: Windows 10

Other Information

Mine works fine , when I click the  "Controls" checkbox the iframe source shows appropriately and the player also displays appropriately.

Mine works fine , when I click the "Controls" checkbox the iframe source shows appropriately and the player also displays appropriately.

If you use the Twitch Test A which is only a video then the Controls check box works fine. If you use the Test C and Custom URL with channel format, ex: twitch.tv/{channel name}, it will not work.
in the source code:

If the URL is "twitch.tv/{channel name}" which only contains channel name, it will set the Controls props always to true.

some one has the same issue as me
https://github.com/cookpete/react-player/issues/1236


[Feature Request] [feature request] Add virtual list when large amount of data, solve the problem of stutter

Existing Component
是
Component Name
Cascader Select Transfer
Description
Add virtual list when large amount of data, solve the problem of stutter

Do you have any updates on this?


Does not seem to work in .NET 6 minimal api

Hey,
We've been trying to use the trigger library in our project.
This is using a hosted service and a context + repository that lives in another project. There is a project reference to this. I did read upon the dependency chapter on the wiki, but I was not sure what was meant there with the service provider. Might not be totally aware of something, sorry if I'm not.
(Using version 3.0.0).
The following code does not throw an exception, but also does not trigger the trigger:
Program.cs:
using System.Net;
using EntityFrameworkCore.Triggered;
using Microsoft.AspNetCore.Builder;
using Microsoft.EntityFrameworkCore;
using Microsoft.Extensions.Configuration;
using Microsoft.Extensions.DependencyInjection;
using Microsoft.Extensions.Logging;
using Serilog;

public class Program
{
    public static void Main(string[] args)
    {
        var builder = WebApplication
            .CreateBuilder(args);
        var env = builder.Environment.EnvironmentName;

        var config = new ConfigurationBuilder()
            .AddJsonFile("appsettings.json", true, true)
            .AddJsonFile($"appsettings.{env}.json", true)
            .AddJsonFile("commonsettings.json", true, true)
            .AddJsonFile($"commonsettings.{env}.json", true)
            .AddEnvironmentVariables()
            .Build();

        Log.Logger = new LoggerConfiguration()
            .ReadFrom.Configuration(config)
            .CreateLogger();

        builder.Logging.AddSerilog();

        builder.Services
            .AddDbContext<BooksContext>(options =>
            {
                options.UseTriggers(triggerOptions =>
                {
                    triggerOptions.AddTrigger<BooksTrigger>();
                });
            })
            .AddScoped<IBookRepo, BookRepo>()
            .Configure<DatabaseConfiguration>(config.GetSection("Database"))
            .AddHostedService<BookMessageConsumer>(sp =>
            {
                using var scope = sp.CreateScope();
                var logger = scope.ServiceProvider.GetRequiredService<ILogger<BookMessageConsumer>>();

                return new BookMessageConsumer(sp, logger);
            });

        var app = builder.Build();

        app.MapGet("/alive", () => HttpStatusCode.OK);
        app.MapGet("/test", (BooksContext context) =>
        {
            context.Books.Add(new Book()
            {
                Title = 'Dumpert'
            });
            context.SaveChanges();
        });

        app.Run();
    }
}

Trigger code:
using System;
using System.Threading;
using System.Threading.Tasks;
using EntityFrameworkCore.Triggered;

public class BookTrigger : IAfterSaveTrigger<Book>, IBeforeSaveTrigger<Book>
{
    private readonly BookContext _bookContext;

    public BookTrigger(BookContext context)
    {
        _bookContext = context;
    }

    public Task AfterSave(ITriggerContext<Book> context, CancellationToken cancellationToken)
    {
        Console.WriteLine("trigger is working");
    }

    public Task BeforeSave(ITriggerContext<Book> context, CancellationToken cancellationToken)
    {
        throw new NotImplementedException();
    }
}

I'm unable to reproduce this issue. I copied your code and made it run and it successfully hit the BeforeSave method (And therefore raises an exception).
Some things I had to do to get things to run:

Configure<DatabaseConfiguration>(config.GetSection("Database")) suggests that you're configuring your DbContext outside of the Startup logic. You're possibly implementing OnConfiguring. Please make sure that you accept and use the DbContextOptions in your DbContext, as such:

internal class BooksContext : DbContext {
    public BooksContext(DbContextOptions options) : base(options)  { }

    public DbSet<Book> Books => Set<Book>();
}

Otherwise make sure that you're registering Triggers within the used DbContextOptions instance, as such:
internal class BooksContext : DbContext {
    protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder)  {
        optionsBuilder.UseTriggers(triggerOptions => {
            triggerOptions.AddTrigger<BookTrigger>();
        });
    }

    public DbSet<Book> Books => Set<Book>();
}

Your trigger is called BookTrigger however your registration is for BooksTrigger, please make sure that you're actually registering the expected type

Let me know if the above makes a difference, otherwise please share a repro. Thanks!

Thank you for the response and taking the time to help.
Indeed, I did not set up the context correctly and pass the options through to the base class. After doing that, I indeed also got the trigger to fire. Sorry about this, thought I did everything well as I could write to the db, but it is logic I guess because I know also see the triggers correctly in the extensions and been passed through!


Rewrite embedded FCM intro

I think this version is much clearer. Also fixed a typo or two.

Thanks for the contribution


✨ docker magic

I'm back (until it break again)
PR type

[x] Bugfix

if it works why not!

/giphy magic hand


Remove v11 banner

Closes ##4112
Removed v11 notifications
Additional changes

Color usage tab
-- Kept the migration link in the Usage tab
-- Removed the migration note of v10

I saw "Tabs" mentioned in the issue, but I can't find the v11 callout under that component.

@Kritvi-bhatia17 @thyhmdo Tracey removed the Tabs v11 notification at the top of the page in her recent Tab component docs PR that was merged, so that is why the banner is not there. I accidentally marked her in the issue as having removed the Notification component v11 banner, but that was incorrect because it was the Tabs component. I changed it back in the issue to be accurate. So this looks fine to me.

There's only one inline notification under the Tab usage component. I'm not sure if the "Updates for grid alignment" are available now or not. If so, we can update that message with the latest changes.

I would keep this notification about grid-aware tabs for now. We need to follow up with Alison to see if grid-aware tabs are in the code yet. I know some issues and PRs have been closed around this enhancement, but it would be good to double-check with her before removing it.

Oh sounds good @laurenmrice, thanks for clarifying. ❤

Hi, sorry if I missed any updates, but could you confirm if we are removing the inline notification from the Popover component?

I think we missed removing the v11 inline notification callout in the Popover component.

@Kritvi-bhatia17 Yes, Popover still needs to have its banner removed.


manager: Restore multiple simultaneous tunnels feature

This feature was probably disabled accidentally.

Need this very much! Please enable this.

Nothing changed. Didn't help.
While starting second tunnel the first one become disconnected.

Thanks! That worked!


Make EventWaiter importable.

I just pushed an update that adds a Waiter() class and helper method for event busses which should address this feature request.  lemme know if i can close this one out :)

👍


New expanded mode

Codecov Report

Merging #7 (4eab637) into main (d70dc02) will decrease coverage by 22.05%.
The diff coverage is 46.34%.

@@             Coverage Diff             @@
##             main       #7       +/-   ##
===========================================
- Coverage   95.45%   73.40%   -22.06%
===========================================
  Files           3        3
  Lines          66       94       +28
===========================================
+ Hits           63       69        +6
- Misses          3       25       +22

Impacted Files
Coverage Δ

src/macro.jl
63.07% <46.34%> (-31.52%)
:arrow_down:

Continue to review full report at Codecov.

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


domain expiration check

Describe the bug
It seems that DOMAIN_EXPIRATION check fails whenever Expiry date is few years ahead:
output from whois:
    Relevant dates:
        ...
        Expiry date:  31-Jul-2031
        ...

check is failing:

What do you see?
No response
What do you expect to see?
No response
List the steps that must be taken to reproduce this issue
Use DOMAIN_EXPIRATION check on long lasting domain
Version
No response
Additional information
No response

Hello! @TwiN, can you assign this bug to me? I want to solve it and participate in the project as well.

@lefes sure!
This may or may not involve another project of mine: github.com/TwiN/whois

@ArturZurawski Hello!
I am here to help you! The issue lies with the WHOIS package (like @TwiN said). Unfortunately, due to variations in the formats of returned WHOIS dates, it is impossible to provide a parsing filter for all cases. However, we can add the format of individual domains and zones as needed. It appears that your domain is not in the supported list. To assist you, I need the domain you are trying to add or a list of domains so that I can implement filters for them.

Hello @lefes . I would prefer not to brag about my domains on the public Internet :). Above snippet is from the actual whois output, so my guess is that is enough information to be able to phrase it.
Just in case this is Registrar:
    Registrar:
        Tucows Inc t/a Tucows [Tag = TUCOWS-CA]
        URL: https://www.tucowsdomains.com

@ArturZurawski  Totally understandable :) In this case, the provided information will be helpful, but a crucial piece to assist you will be the TLDs (top-level domains) of what you want to monitor with the Gatus tool (for example, .uk, .com, .th, etc.).

@ArturZurawski , sorry for disturbing you again, but to fix this problem, I need to know the Top-Level Domains (TLDs) you want to use (for example, .uk, .com, .th, etc.). Can you provide that information for me?

@lefes I honestly don't understand how giving specific domains is going to solve this problem. If you are parsing the output from whois then apparently the function/regexp used does not catch the expiry date in this case. The relevant part of output was provided by me and that is all I want to make public in this case.

@lefes According to https://github.com/TwiN/whois/blob/029a9259cbe132dd38eea0e6935e2fbb253ad1ec/whois.go#L153 it seems that there should be a match for 31-Jul-2031 as in output from my whois. Why does it not match then?

@lefes I'm sorry but as mentioned I will not share the domain. I think that I provided enough information to replicate the issue. If there is no way to simulate parsing whois output of long lasting domain (10 yrs) then I guess we need to close the issue or wait for someone else who will have the same problem.

@ArturZurawski , I checked the case where the domain has "Expiry date: 15-Jul-2034," and there is no problem at all. It worked well and looks good. It seems like 10 years is not a problem at all. I'm sorry, but I can't provide any further assistance on this problem without more information.
@TwiN , it's your call then. Are we waiting for someone else with the same problem and more information, or should we just close the issue?

I believe I got the same issue and my domain is woodpeckersnest.eu

@roughnecks Hi there!
The .eu domains are tricky. Due to GDPR, whois information is now private and does not provide expiry dates via default whois services.
Looks like I can't help you there, sorry.

I have 3 domains (2 .bz & 1 .nu) all registered w/ Namecheap
This is what I got when I ran whois from Ubuntu command line:
.bz: "Registrar Registration Expiration Date: 2025-03-17T03:50:42.99Z" & "Registry Expiry Date: 2024-05-06T06:50:04Z"
.nu: "expires:          2024-05-01" (This one is saying eNom is the registrar)

I have the same issue.  It is a .com domain, and the return from GoDaddy's check shows:
Registrar Registration Expiration Date: 2026-09-20T11:03:53Z
A check at IANA returns no expiration date, but does show some of the snippets below:
organisation: VeriSign Global Registry Services
whois:        whois.verisign-grs.com
status:       ACTIVE
remarks:      Registration information: http://www.verisigninc.com
YAML Config snippet:

name: Domain Name Expiration
url: "http://mydomain.com"
interval: 6h
client:
timeout: 10s
conditions:

"[DOMAIN_EXPIRATION] > 720h"

Currently returning "Success", but with the following text:
'Get "http://mydomain.com": context deadline exceeded (Client.Timeout exceeded while awaiting headers)'
The error message seems to suggest it was NOT as successful check.  Granted, the error is with it's ability to query IANA, but shouldn't that be giving me some sort of error?  How do I confirm it could contact IANA successfully AND the expiration date is not too near?

Looking at https://github.com/TwiN/gatus/blob/1eba430797fe331aeaf4b4cc2a9dfeaff1ff47ff/core/dns.go#L53-L58
it's possible that if connection to IANA fails, the [CONNECTED] == true condition will fail, but I'm not sure.

For anybody who wants to support additional domains, as I mentioned in an earlier comment, it goes through https://github.com/TwiN/whois, so I'd suggest submitting a PR if you can, but you can also create an issue on said repo 🙏


Science multiplier shouldn't apply to all techs prior to wood processing

Mod source
Factorio Mod Portal
Which mod are you having an issue with?

[X] pyalienlife
[X] pyalternativeenergy
[ ] pycoalprocessing
[ ] pyfusionenergy
[ ] pyhightech
[ ] pyindustry
[ ] pypetroleumhandling
[ ] pypostprocessing
[ ] pyrawores

Operating system

=Windows 10

What kind of issue is this?

[ ] Compatibility
[ ] Locale (names, descriptions, unknown keys)
[ ] Graphical
[ ] Crash
[X] Progression
[X] Balance
[ ] Pypostprocessing failure
[ ] Other

What is the problem?
Technologies can have a flag set to have any science multiplier not apply to them, which is often used for technologies that are required to fully automate the science that unlocks them. For example, the vanilla assembling machine technology has this to avoid handcrafting large amounts of science, and Krastorio 2 has this on its greenhouse tech and all predecessors, as the first science pack requires wood to craft.
While Py has this flag set on the vanilla automation technology, it doesn't have it set on wood processing and all its prerequisites, which are required to fully automate automation science. Currently, these techs are Wood processing, Moss - Stage 1, Botany - Stage 1, Soil washing, Glass technologies, Mining with fluid, Steel processing, Coal processing 1, Soil washing. To avoid players on science multipliers being forced to gather large amounts of wood (see below) all of these should be set to be unaffected by science multipliers.

(screenshot courtesy of Simon Weinberger on the py discord, who's currently playing with a 100x science multiplier)
Steps to reproduce
No response
Additional context
No response
Log file
No response

This is a bit like putting on a blindfold and then complaining you can't see 🤭
Thanks for the report, fixed here: https://github.com/pyanodon/pypostprocessing/commit/b51ab5024def41e915aba74c47a83b87cd913113


Compliance Report | Adjustments to Part 2 summary section labels Round 2 |1|

Task:
This card is to adjust some of the labels within the Part 2 summary section of the Summary & Declaration page and to place restrictions on the data that can be inputted within the Part 2 summary section.
Line 6, 17, 8, 19:
For these four labels, we would like to add two decimal places to the label. For example, using Line 6, the following change would be made:

Current: "Volume of renewable fuel retained (up to 5% of Line 4 is x,xxx L)"
Revised: "Volume of renewable fuel retained (up to 5% of Line 4 is x,xxx.xx L)"

Restrict inputting data:

As part of this label change, for the four lines mentioned above, we want to restrict the input of data to two decimal places.
We also want to maintain all of the other restrictions that were implemented in Card 1474 and Card 1478. https://trello.com/c/PCWUVZnv/1575-compliance-report-adjustments-to-part-2-summary-section-labels-round-2-1

 Compliance Report | Adjustments to Part 2 summary section labels Round 2 |1|

done in sprint red dwarf


Splunk HEC sends 400 and logs error: "Got unexpected null element while expecting event's raw text"

We have upgraded from Splunk Otel Chart 0.79.0 to 0.86.1. We've been using logsEngine: otel already with 0.79.0. We only collect logs. Splunk server-side is 9.0.5. Distribution is OpenShift 4.12.
We are using the metric otelcol_exporter_send_failed_log_records to detect issues with sending to Splunk.  Since the upgrade, we now get somewhat regularly alerts on this.
We then see splunk-otel-collector logs like:
<...> Exporting failed. The error is not retryable <...>
{\\\"text\\\":\\\"Invalid data format\\\",\\\"code\\\":6,\\\"invalid-event-number\\\":0}\"", "dropped_items": 6}

The splunkd.log shows an ERROR containing:
<...> parsing_err="Got unexpected null element while expecting event's raw text, totalRequestSize=6705"

Seems like the splunk-otel-collector somehow constructs an invalid HEC event.

Full splunk otel collector log:
2023-11-01T11:16:03.860Z        error   exporterhelper/retry_sender.go:145      Exporting failed. The error is not retryable. Dropping data.    {"kind": "exporter", "data_type": "logs", "name": "splunk_hec/platform_logs", "error": "Permanent error: \"HTTP/1.1 400 Bad Request\\r\\nContent-Length: 64\\r\\nConnection:
Keep-Alive\\r\\nContent-Type: application/json; charset=UTF-8\\r\\nDate: Wed, 01 Nov 2023 11:16:03 GMT\\r\\nServer: Splunkd\\r\\nVary: Authorization\\r\\nX-Content-Type-Options: nosniff\\r\\nX-Frame-Options: SAMEORIGIN\\r\\n\\r\\n{\\\"text\\\":\\\"Invalid data format\\\",\\\"code\\\":6,\\\"invalid-event-number\\\":0}\"", "dropped_items": 6}
go.opentelemetry.io/collector/exporter/exporterhelper.(*retrySender).send
        [go.opentelemetry.io/collector/exporter@v0.86.0/exporterhelper/retry_sender.go:145](mailto:go.opentelemetry.io/collector/exporter@v0.86.0/exporterhelper/retry_sender.go:145)
go.opentelemetry.io/collector/exporter/exporterhelper.(*logsExporterWithObservability).send
        [go.opentelemetry.io/collector/exporter@v0.86.0/exporterhelper/logs.go:176](mailto:go.opentelemetry.io/collector/exporter@v0.86.0/exporterhelper/logs.go:176)
go.opentelemetry.io/collector/exporter/exporterhelper.(*queueSender).start.func1
        [go.opentelemetry.io/collector/exporter@v0.86.0/exporterhelper/queue_sender.go:124](mailto:go.opentelemetry.io/collector/exporter@v0.86.0/exporterhelper/queue_sender.go:124)
go.opentelemetry.io/collector/exporter/exporterhelper/internal.(*boundedMemoryQueue).Start.func1
        [go.opentelemetry.io/collector/exporter@v0.86.0/exporterhelper/internal/bounded_memory_queue.go:52](mailto:go.opentelemetry.io/collector/exporter@v0.86.0/exporterhelper/internal/bounded_memory_queue.go:52)

The corresponding Splunkd.log:
11-01-2023 12:16:03.859 +0100 ERROR HttpInputDataHandler [13954 HttpDedicatedIoThread-5] - Failed processing http input, token name=k8scluster, channel=n/a, source_IP=<IP_ADDRESS>, reply=6, events_processed=0, http_input_body_size=6705, parsing_err="Got unexpected null element while expecting event's raw text, totalRequestSize=6705"

Manually crafting and sending a HEC message like:
{
  "host": "x",
  "source": "/var/log/test.log",
  "sourcetype": "text",
  "index": "openshift",
  "event": null
}

triggers the Got unexpected null element while expecting event's raw text error on the splunk side.
Sending completly without the event property, the HEC response contains event field is required.
Im quite sure, the Splunk Otel collector sends event: null.

The default logging pipeline does include config to try and avoid "empty lines".
        - field: attributes.log
          id: handle_empty_log
          if: attributes.log == nil
          type: add
          value: ""

Do you have any log pipeline customizations?
Might be worth flipping on debug or trace on otel or splunk side to identify what the actual invalid event is..

I only get the alert on control-plane nodes.  And it's always only one node out of three. I presume, the one that has the most load. The different to application nodes is, that there are additional audit logs configured for these control-plane machines. The additional (audit) logfiles are the only "customization" I have.
I thought about adding a print to https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/b716a4d89acd497cb1ed044b941c49a7897b72bb/exporter/splunkhecexporter/logdata_to_splunk.go#L86 where lr.Body().AsRaw() is nil to get the context of the event. But I failed to understand/learn how to compile the binary.

Hmm, u might be on to something there. I'll need to check what the extraFileLogs pipeline renders as it may not have the same safeguards built in as the default container pipeline. I'll double check.
In the meantime you can turn on debug logging and should be able to capture the smoking gun.
agent:
  config:
    exporters:
      logging:
        loglevel: debug
    service:
      pipelines:
        logs:
          exporters:
            - splunk_hec/platform_logs
            - logging

I think there is no "handle_empty_log" thing for extraFileLogs. but I haven't found empty lines in the audit logs.  i will configure an extra file to test manually.
the issue with the debug is: it logs really much and the issue only occurs randomly. sometimes it takes more than a day until it happens again. pod logs get rotated.  I will try anyways.

Yeah, wish there was a better way..might open an enhancement to log the payload to console when a 4xx is received..would help..
Yeah i looked at the extrafile log pipelines and they are very simple and dont have the handle empty logs operator.
I would probably add it, regardless. Another good enhancement request, imo.
It may not actually be an empty log, but could be a whitespace line or even control chars etc
Getting closer...

I enabled the debug logging now.
btw.

This exporter is being deprecated in favour of the debug exporter. It will be removed in September 2024.

https://github.com/open-telemetry/opentelemetry-collector/blob/main/exporter/loggingexporter/README.md

yeah that was just the example i had on hand. we do ship that exporter now if you want to try it instead:
https://github.com/open-telemetry/opentelemetry-collector/tree/main/exporter/debugexporter
also, you can disable collection of agent logs while you have debug on if you want. or even just update only that particular daemonset on the node in question.
https://github.com/signalfx/splunk-otel-collector-chart/blob/62e5f2246597f2031bb7b1cc2fe654d20136c322/helm-charts/splunk-otel-collector/values.yaml#L555

Filing https://github.com/open-telemetry/opentelemetry-collector-contrib/issues/29551 and taking a look.

We have upgraded to OpenShift 4.12 to 4.13 (k8s from 1.25.11 to 1.26.9) on 2023-11-15. While the platform-upgrade was in progress, the alert could be observed on different nodes, not only on control-plane nodes. Unfortunately i was not able to gather the debug logs.
But: there was no alert anymore after 2023-11-15.

Closing as done - we have a fix upstream so no nil body is possible. If you see issues again, please open a support case.


Update KV rotation docs for portal UI changes

Reflecting the new location of modules in the runbook blade.

@backerman : Thanks for your contribution! The author(s) have been notified to review your proposed change.

This is great feedback, @backerman. The article has moved location, but I updated https://docs.microsoft.com/en-us/azure/key-vault/secrets/key-rotation-log-monitoring per your suggestion.  Thanks for your contribution!  #please-close