Any documentation on the parity between sqlite and psql

I just started learning logica. I played one of the examples circulated in this forum. I noticed that the sqlite engine can compile while the Postgres engine generates a type error.  I don't know if there is a feature gap or this script needs to be coded differently. Thanks.
==data.json==
[{"item": "apple", "price": 5, "amount": 2},
 {"item": "pear", "price": 2, "amount": 1},
{"item": "orange", "price": 3, "amount": 4}]
===sample.l==
@Engine("psql");
@Ground(Data);
Fruit() = ReadJson("data.json");
FruitInfo(item:, price:, amount:,
          revenue: price * amount) :-
  {item:, price:, amount:} in Fruit();
FruitInfoObj() List= {item:, price:, amount:, revenue:} :-
  FruitInfo(item:, price:, amount:, revenue:);

==python logica.py sample.l print FruitInfoObj==
Error
Compiling:
FruitInfoObj() List= {item:, price:, amount:, revenue:} :-
  FruitInfo(item:, price:, amount:, revenue:)

[ Error ] Record needs type in PostgreSQL: {item:, price:, amount:, revenue:} was inferred only an incomplete type {amount: Num, item: Any, price: Num, revenue: Num}.

Hello @min-mwei ,
we are designing a better documentation, for now Tutorial is the best start.
But it doesn't go over all the engines. So please don't hesitate to ask questions in this forum, I'll do my best to answer them promptly.
As for JSON: JSON is native to SQLite, all complex types are implemented on JSON when SQLite is the engine. So it's very natural to read it. Postgres is more statically typed, so it's a little more work to operate with JSON. Usually easiest practice is to convert it to native psql types.
PostgreSQL needs to know types of objects and Logica can often infer those, but sometimes it can't, in that case use type hints, using ~ operator. Here is example of how to read the fruits data in psql.
FruitText() = txt :-
  txt = ReadFile("/content/data.json"),
  txt ~ Str;

FruitStrings() = strings :-
  strings = SqlExpr(
    "array(select jsonb_array_elements_text({txt}::jsonb))",
    {txt: FruitText()}),
  strings ~ [Str];

JsonField(record, field_name) = SqlExpr("{record}::json ->> {field_name}",
                                        {record:, field_name:});

Fruit(item: JsonField(line, "item"),
      price: ToInt64(JsonField(line, "price")),
      amount: ToInt64(JsonField(line, "price"))) :-
  line in FruitStrings();

FruitsInfo(fruits? List= item,
           total_cost? += price * amount) distinct :-
  Fruit(item:, price:, amount:);

CoLab: LINK
Note that there is no ReadJSON function when running on psql, we need to use ReadFile and convert to json manually, as in the example.
Thanks for trying out Logica, please reach out if you have any questions or comments!
Evgeny.


Processing of execution-time performance test results

Description

New tool to process the json files that result from running the execution-time benchmark tests with the custom mocha reporter in @fluid-tools/benchmark.
New pipeline that builds the repo, runs execution-time performance tests, runs the tool from the previous bullet point to send relevant metrics from the output to one of our Kusto tenants, and publishes the test results as a pipeline artifact.
New npm script in @fluid-internal/tree and @fluid-experimental/tree to run the execution-time performance tests.

The pipeline will run performance tests for any package that defines a test:benchmark:report npm script. It's assumed that said script already passes all the necessary options to mocha to run the performance tests, including the reporter from @fluid-tools/benchmark to display the results in console. The release-group-level npm script will only add the output location for the reporter.
Reviewer Guidance
General approach
Chatting with @anthony-murphy , we decided to implement this as a separate pipeline and not make it additional steps in the existing Build - client packages one, mainly to keep our public and internal versions of the Build - client packages pipeline identical, and as free as possible of conditional steps. These tests only need to run in our internal pipelines so the definitions would have to diverge, or we'd need conditional logic that defines when to execute the steps for these tests.
This means the repo needs to be built "again" in this pipeline (instead of reusing the build that already happens in Build - client packages). That's one of the most expensive parts of our pipelines so it doesn't feel ideal, but it's the tradeoff we decided on right now. Future work could focus on changing that decision. Work that reduces the repository build time would also be helpful.
New npm script
The new test:benchmark:report npm script in the Tree packages is very similar to the existing bench and perf ones. @CraigMacomber , @Abe27342 do you have any thoughts on consolidating them? Some possible diverging requirements I can see:

cross-env FLUID_TEST_VERBOSE=1 I'd skip that for the script called during CI.
expose-gc is not present in the packages/dds/tree script but that might be an error.
Timeout values

Any relevant logs or outputs
Example of successful pipeline run
Results in Kusto:

AB 1410

Looks good overall, I just don't have enough context to give an actual approval


Audio pauses and then restarts when navigating between screens

Bug
We are using react native video to play audio, the play/ pause button button and video component is in a custom header. When we navigate between screens it pauses shortly before restarting on its own when navigated.
Platform

Which player are you experiencing the problem on:

iOS
Android

Environment info
React native info output:
System:
    OS: macOS 13.3.1
    CPU: (10) x64 Apple M1 Max
    Memory: 62.79 MB / 64.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 16.14.0 - /usr/local/bin/node
    Yarn: 1.22.17 - /usr/local/bin/yarn
    npm: 8.3.1 - /usr/local/bin/npm
    Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
  Managers:
    CocoaPods: 1.12.1 - /Users/jamesparsons/.rbenv/shims/pod
  SDKs:
    iOS SDK:
      Platforms: DriverKit 22.4, iOS 16.4, macOS 13.3, tvOS 16.4, watchOS 9.4
    Android SDK: Not Found
  IDEs:
    Android Studio: 2020.3 AI-203.7717.56.2031.7784292
    Xcode: 14.3/14E222b - /usr/bin/xcodebuild
  Languages:
    Java: 11.0.17 - /usr/bin/javac
  npmPackages:
    @react-native-community/cli: Not Found
    react: 18.0.0 => 18.0.0
    react-native: ^0.69.0 => 0.69.10
    react-native-macos: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

Library version:     "react-native-video": "^5.2.1"
Steps To Reproduce

Use code show below
Add audio stream (example, http://listen.vibenation.live:80/vibe)
Navigate between a page (React Navigation)
...

Expected behaviour

Audio player does not pause and then restart when navigating between pages.

Reproducible sample code
   ` <Video
      source={{uri: 'http://listen.vibenation.live:80/vibe'}}
      audioOnly={true}
      paused={play}
      ignoreSilentSwitch="ignore"
      playWhenInactive={true}
      playInBackground={true}
    />`

Same issue

I am seeing this issue as well on android. 6.0.0-beta.3

Did you check if the <Video component is unmounted ?
If you need a audio player in all tabs why don't you put it in an upper level component ?


Organization Name
Brsk
Your Name
Chris Hills
Your current position
Platform Engineer
(Optional) Company logo
https://cdn.brsk.co.uk/branding/brsk-fullfibre-no-slogan-blue.svg
(Optional) Use case
No response
Also using other Powertools for AWS Lambda languages?

[ ] Java
[ ] TypeScript
[ ] .NET

The URL for our company is https://www.brsk.co.uk

Hey @ChrisHills463! This is fantastic, thanks a lot for supporting us!


Testing for JSON on php 5.2+ seems redundant

https://github.com/drewm/morse-php/blob/master/src/Feature/String.php#L25
Why are you testing for this if you minimum supported php version is 5.3?

Even though it's enabled by default, it can be disabled.


Just a wishlist - would love to see a exec.go that probably parameterizes all this. So I can just do Exec.{{command}}.

Just a wishlist - would love to see a exec.go that probably parameterizes all this. So I can just do Exec.{{command}}.
_Originally posted by @ksatchit in https://github.com/render_node/MDExOlB1bGxSZXF1ZXN0NDQ2OTE4Njgw/pull_requests/unread_timeline

This is fixed in 1.6.0-RC1


Coords reading fix

An error occurred when I loaded in the attached 0011.sdf file that has four new coordinate attributes; Px_px_py/Photon, Py_px_py/Photon, Px_px_py/Photon_mid, and Py_px_py/Photon_mid.
I believe that the route cause to this problem is the generator expression var_coords
var_coords = (
    dim_size_lookup[dim_name][dim_size]
    for dim_name, dim_size in zip(value.grid.labels, value.dims)
)

that gets called lazily on line 98 after the loop that converts SDF variables and meshes to xarray DataArrays and Coordinates
ds = xr.Dataset(data_vars, attrs=attrs, coords=coords)

This seems to lead to it incorrectly capturing each coordinates long_name and array length. It instead opts to try to use the ones set the first time that the function was called which is normally something like { "X": 128, "Y": 128 } when it should be using { "Px": 200, "Py": 200 }. As a remedy to this problem I suggest that we instead opt to utilise list comprehension which is slower but avoids the above issue.

Ah yes, the generator isn't immediately evaluating the expression, so it's just wrong and we indeed need a list comprehension instead. I doubt it'll make much difference to the performance.
Is the new SDF file part of the same sequence? I'm not sure I properly documented how I generated the original set of files -- they're a stripped down version of the epoch1d two-stream example. I'll add a readme with how to regenerate them.
Can we use the existing files for the new test?

The SDF I generated is from the 11 th timestep using the default QED_rese deck. Not sure if our existing files would work, will check now


Unable to create Dto from z.discriminatedUnion

I'm trying to create a dto from a zod discriminated union like this:
const BaseSchema = z.object({
  names: z.string(),
  last_name: z.string(),
  document_id: z.string(),
  document_type: z.nativeEnum(DocumentType),
  nationality: z.string(),
  email: z.string().email(),
});

const EmployeeSchema = BaseSchema.extend({
  user_type: z.literal(UserType.EMPLOYEE), // UserType.EMPLOYEE === "EMPLOYEE"
});

const SellerSchema = BaseSchema.extend({
  user_type: z.literal(UserType.SELLER), // UserType.SELLER === "SELLER"
  code: z.number(),
});

const CreateUserBodySchema = z.discriminatedUnion('user_type', [
  EmployeeSchema,
  SellerSchema,
]);

export class CreateUserBodyDto extends createZodDto(CreateUserBodySchema) {}

createZodDto(CreateUserBodySchema) yields the following error: TS2509
Base constructor return type '{ names: string; last_name: string; document_id: string; document_type: "PASSPORT" | "RUT"; nationality: string; email: string; user_type: "HHRR"; } | { names: string; last_name: string; document_id: string; ... 4 more ...; user_type: "SELLER"; }' is not an object type or intersection of object types with statically known members.

nestjs-zod version: 2.2.0
typescript version: 4.9.5

The only solution to this, since as explained here: A class can implement an interface or type alias, both in the same exact way. Note however that a class and interface are considered static blueprints. Therefore, they can not implement/extend a type alias that names a union type.
Is: to use the schema directly in the constructor of a new instance of the validation pipe, and use it like this:
@Body(new ZodValidationPipe(CreateUserBodySchema)) body: CreateUserBodyDto

CreateUserBodyDto then becomes:
type CreateUserBodyDto = z.infer<typeof CreateUserBodySchema>;


Datadoc is incompatible with jupyterlab=3.6.6 (widgetsnbextension=4.0.5)

We cannot install the latest version of datadoc because of the dependency widgetsnbextension >= 4.0.10. It fails in our jupyterlab environment because we have prevented users from installing/upgrading extensions as a security measure:
https://github.com/statisticsnorway/jupyterlab-common/blob/main/docker/Dockerfile#L229

Closed due to JupyterLab no longer being a supported environment.


Update nunit to 3.5.0

@KevinRansom I'm not sure why those commits keep appearing in your commit lists - perhaps try a hard reset on your working master branch in your repo after this?

I did a merge perhaps that is why.
From: Don Syme<EMAIL_ADDRESS>Sent: Wednesday, October 26, 2016 2:07 AM
To: Microsoft/visualfsharp<EMAIL_ADDRESS>Cc: Kevin Ransom<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Microsoft/visualfsharp] Update nunit to 3.5.0 (#1659)
@KevinRansomhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FKevinRansom&data=02|01|Kevin.Ransom%40microsoft.com|dcc710f7730d4fd4ca8b08d3fd7f6256|72f988bf86f141af91ab2d7cd011db47|1|0|636130695954417610&sdata=d8oQz%2B3vM4RzkWduBKbwKxjGd5hraRzHdy2Z7JAVE4M%3D&reserved=0 I'm not sure why those commits keep appearing in your commit lists - perhaps try a hard reset on your working master branch in your repo after this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fvisualfsharp%2Fpull%2F1659%23issuecomment-256290626&data=02|01|Kevin.Ransom%40microsoft.com|dcc710f7730d4fd4ca8b08d3fd7f6256|72f988bf86f141af91ab2d7cd011db47|1|0|636130695954417610&sdata=pX1zdj7Chcv78gfznFed%2F4pmP9hctg%2FfL2ZRS5lyQkw%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAE76FrjHlLfMaS0RJQ6dCy9SBlmj--6yks5q3xgYgaJpZM4KgwQt&data=02|01|Kevin.Ransom%40microsoft.com|dcc710f7730d4fd4ca8b08d3fd7f6256|72f988bf86f141af91ab2d7cd011db47|1|0|636130695954417610&sdata=WytHa2SqmFcT24Nmqsy7wWuJ6SksRDnuDxzGvEcwTII%3D&reserved=0.


Problem related to multi-root workspace

It's a great extenstion helping me review code. I met a problem these days and please help.
When using CodeTour in my multi-root workspace, I choosed the first folder (e.g., path_a) to save my tour file (path_a/.tours/foo.tour).
I can record the step for file A.md in the second folder (e.g., path_b) of my workspace, which is confirmed according to the content of .tour file. But when playback the tour, steps which are not in the first folder didnot show.
I checked the path to file of the problem step and found it to be right. And I read the #1 and #13 but could not find some way to address the problem.
Any help will be appreciated, thanks :smiley:

Hi @mapleleaf-soar, thanks for flagging. To clarify, are you saying that the tour is missing steps when there is more than one root folder? cc @lostintangent

Sorry for my unclear expression. Some steps are not "missing". The comment contents of these steps are indeed in the .tours/foo.tour. I just can not see them after "starting tour".
My workspace is:

path_a
├── a.md
├── .tours
│   └── foo.tour
path_b
└── b.md

When I click step 1 and 2,

Thanks.

Thanks for this! We will look into this issue and see if we can get it fixed.

This is fixed in v0.0.49 👍🏻

I tested the v0.0.49 and it worked! Thanks!

Awesome! Thanks for confirming 🤗


Error accessing ICH RCRB when running the script on Chromebook Pixel 2013

Hi MrChromebox, thanks for the amazing work, I updated the firmware some time ago and after using it for a while, the boot logo inexplicably disappeared and I couldn't access the settings menu with esc, then I found that the system installation failed when it was trying to set up the boot, so I tried to repair the firmware with the script running on a live usb and this error was reported.
Does this mean some kind of hardware issue or simply the firmware broke? can this be fixed?
Thanks!
Downloading supporting files...

Getting device/system info...

Unable to read current firmware; cannot continue:

flashrom dc8caac9-dirty on Linux 6.1.11-1-MANJARO (x86_64)
flashrom is free software, get the source code at https://flashrom.org

Using clock_gettime for delay loops (clk_id: 1, resolution: 1ns).
coreboot table found at 0xacf67000.
Found chipset "Intel HM75".
Enabling flash write... Error accessing ICH RCRB, 0x4000 bytes at 0x00000000fed1c000
/dev/mem mmap failed: Operation not permitted
FAILED!
FATAL ERROR!
Error: Programmer initialization failed.

your kernel is not allowing flashrom to read the (memory mapped) flash chip. You can try booting with 'iomem=relaxed' added to your kernel boot params, or try live booting another more permissive kernel (Ubuntu 2204 works fine)


RFC: add support for asserting if an object contains a nested key path

Checklist

Please ensure the following tasks are completed before filing an issue.

[x] Read and understood the Code of Conduct.
[x] Searched for existing issues and pull requests.
[x] If this is a general question, searched the FAQ for an existing answer.
[x] If this is a feature request, the issue name begins with RFC: .

Description

Description of the issue (or feature request).

Check whether an object contains a nested key path.
Package: @stdlib/assert/deep-has-own-property.
Alias: deepHasOwn
Example:
var obj = { 'a': { 'b': { 'c': 'd' } } };

var bool = deepHasOwn( obj, 'a.b.c' );
// returns true

bool = deepHasOwn( obj, 'a.b.c.d.e' );
// returns false

Notes:

Should provide a factory method, similar to @stdlib/utils/deep-get.
Should return only true or false (i.e., be a predicate function).
Should check for only own properties.

Related Issues

Does this issue (or feature request) have any related issues?

None.
Questions

Any questions for reviewers?

No.
Other

Any other information relevant to this issue (or feature request)? This may include screenshots, references, stack traces, sample output, and/or implementation notes.

No.

Hey @kgryte, is this still open?
If so, how should it differentiate from the deepGet method?
I suppose this should return a value if the path exists, else undefined should be returned instead?

@ognjenjevremovic The issue still is open. The difference from deepGet is that deepHas should be a predicate function, thus returning only true or false. For deepGet, a path could exist and yet still return undefined.
Feel free to submit a PR. Made edits to the original issue.

Thank you so much for the input and a fast reply @kgryte.
I'm going to take a deeper look at this one.

@ognjenjevremovic Awesome. Thanks!

Just a few more question @kgryte; should this one provide a support for deep paths being specified as either a delimited string or a key array, similar to @stdlib/utils/deep-get?
And should the factory property be exposed for creating a reusable deepHasOwn function by using the same set of provided separator options?

@ognjenjevremovic Yes, sounds reasonable. You can basically recycle the deep-get implementation but modify the return value logic. Accordingly, providing support for paths being specified by either an array or delimited string should follow suit.

Hey @kgryte, sorry for my late response.
I am having huge troubles making the tests run properly, the last couple of days...
Can you please assist me in debugging?
Since I've rebased, I keep getting the following errors :
/home/ognjen/repos/stdlib/tools/make/lib/test/javascript.mk:44: recipe for target 'test-javascript-local' failed make: *** [test-javascript-local] Error 1
That is however, when I initialized  the hooks, for linting/test errors, with
make init
I've written the tests and they seem to be working fine for the @stdlib/assert/deep-has-own-property, when setting the TESTS_FILTER flag, prior to running them.
I thought by making a fresh clone of the repo will perhaps help solve the problem, however the error is still present when running
make test
with no changes in the source code whatsoever.

Can you run the make test command using the VERBOSE flag?
$ VERBOSE=1 make test

The error log shown above is not much to work with. :)

It is a bit odd that make test does not work, while make TESTS_FILTER=.*/foo/bar/.* test would work, as the make recipe is the same. What differs is the computed file list.
What is your OS and what version of Node are you running?
Sorry that you are having issues. :(

Sure thing.
Doing it just now, though tests can take around ~15-20mins.
Yeah, I tried to debug it for the last 3 days, as it's actually permitting me to make a few final commits (most of which has to do with linting errors and a few failing test scenarios) and do a PR, as of git hooks.
The error, as far as I could tell, comes from the @stdlib/math/ dir tests.
As running make TESTS_FILTER=.*/math/.* test causes the problem each time (though those could not be the only ones, as I ran most of them separately, directory by directory, until I saw that those are the ones that are causing the errors).
I'm running Ubuntu 17.04 and Node v6.11.2.

You are encountering this error on push, correct? I do not believe that the git hooks run tests on commit. Instead, the commit hook should just be running lint.
You said that you pulled down the latest changes from the upstream repo (i.e., the main project repository)?
When running the tests, you should see the test log. What is the failed test? Or does the command fail silently?

Yeah, that's correct; I can not push the latest changes (the commits are stored locally :)).
Yes, everything's up to date with the latest changes to the main repo develop branch.
Sadly, it doesn't say much about the error; the only output I get is the one I've pasted straight from the terminal window.
/home/ognjen/repos/stdlib/tools/make/lib/test/javascript.mk:44: recipe for target 'test-javascript-local' failed make: *** [test-javascript-local] Error 1
I'm not quite sure if it has anything to do with dependencies.
Though, make runs with no problem.
I'm waiting for the last round of tests, now with the VERBOSE flag, so I can give you more info.
However, running tests with make TESTS_FILTER=.*/some/path/*. test runs with no problem.
Depending on the path that is - as I already mention previously, the make TESTS_FILTER=.*/math/*. test produces the exact same output/error as running the make test.

Last few tests and an error output :
    ✔ alpha: 2.6548627613287, beta: 6.954499271853653, y: 0.36071266603830693, expected: 0.36071266603830693
    ✔ within tolerance. alpha: 2.164144120216137. beta: 7.029800248379493. y: 0.3113947603804268. E: 0.31139476038042674. Δ: 5.551115123125783e-17. tol: 2.765741061775866e-16.
    ✔ alpha: 0.975648844590119, beta: 2.047832948643205, y: 6.131133752538203, expected: 6.131133752538203
    ✔ alpha: 9.425459061679401, beta: 4.480806427513445, y: 2.0117557048152683, expected: 2.0117557048152683
    ✔ alpha: 6.718380617444311, beta: 3.67699328910776, y: 2.291798292638805, expected: 2.291798292638805
    ✔ within tolerance. alpha: 9.680688166086341. beta: 9.568864368066365. y: 0.5638201535279117. E: 0.5638201535279116. Δ: 1.1102230246251565e-16. tol: 5.007728929555025e-16.
    ✔ alpha: 4.6001104596645, beta: 9.453378451224095, y: 0.33576895052121924, expected: 0.33576895052121924
    ✔ alpha: 1.5255040667969844, beta: 7.052887294798185, y: 0.2498980149982572, expected: 0.2498980149982572
/home/ognjen/repositories/stdlib/tools/make/lib/test/javascript.mk:44: recipe for target 'test-javascript-local' failed
make: *** [test-javascript-local] Error 1

This makes it seem like it just died without actually having failed. I think you have reproduced something we see on Travis CI, but have never seen anywhere else in the "wild"!!!!
This is quite interesting. But alas, I have no way of debugging. Would actually need to sit down at your machine. :(
Are you running in a virtual machine, by chance?
As long as the tests for your particular contribution pass, you can bypass the git hook as follows:
$ git push origin <remote> --no-verify

I was able to reproduce locally! So, it is possible that, because we print so much test output, that the terminal session runs out of memory and/or is killed by the kernel. This would be especially likely in the math namespace as some of the tests generate thousands of lines of test results.
Sorry about this. We have an item in our todo list to support only running tests for changed files, rather than for the entire codebase on push. For now, feel free to bypass the lint hook for git push using --no-verify.

Thanks a lot @kgryte !
To be honest, I was fighting this waaay too long.
This is my 3rd sleepless night, trying to figure out what's happening.
I tried to avoid pushing with --no-verify, as it felt like ~cheating~ not right thing to do.

Yeah, sorry about that. :( If you run into an issue that this again, definitely reach out sooner. We are here to help. :)
And yeah, to avoid putting anyone else through what you have been through, I am going to disable running tests in the push hook until we figure out a better solution.

Disabled running tests in the following commit: https://github.com/stdlib-js/stdlib/commit/fcbabafe97c9f80506736312446be4e45fefb3aa.
If you merge/rebase the latest develop changes and run make init, you should no longer have to run tests on push. :)

Thanks so much for the help @kgryte.
I finally made the PR :).
I'll be offline for the next 14-16 hours or so (sleep + working hours).
I'll be sure to check the progress on the build/PR once back.

Sounds good. :)


How to disable audit logging of some HTTP_CODE?

Hi,
we have in folder /var/log/apache2/modsec/www-data/YYYYMMDD/* a lot of http_codes logs what we don't care.
For any internal reasons, we don't want to log http_code for example 406 (because it makes about 90% of logs in this folder)
It is logging from rules (not our), is possible to remove this logging for one specific http_code for example 406?
NOTE:
406 is only example, we use this http code for internal reasons.
Logs and dumps
Audit log (/var/log/apache2/modsec_audit.log):
XXX XXX - - [21/May/2024:14:46:58.119386 +0200] "GET /?sdasdasdasdasdasdasddas HTTP/2.0" 406 249 "-" "-" ZkyXwnl54haRrMnxz27mkQAAzAs "-" /www-data/20240521/20240521-1446/20240521-144658-ZkyXwnl54haRrMnxz27mkQAAzAs 0 1743 md5:b8111bdfcaf29b8a0ad246ca13fb26fe
Detail log:
--6397c64b-A--
[21/May/2024:14:46:58.119186 +0200] ZkyXwnl54haRrMnxz27mkQAAzAs XXX 57709 XXX 443
--6397c64b-B--
GET /?sdasdasdasdasdasdasddas HTTP/2.0
Pragma: no-cache
Cache-Control: no-cache
Sec-Ch-Ua: "Google Chrome";v="125", "Chromium";v="125", "Not.A/Brand";v="24"
Sec-Ch-Ua-Mobile: ?0
Sec-Ch-Ua-Platform: "Windows"
Upgrade-Insecure-Requests: 1
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
Accept: text/html,application/xhtml+xml,application/xml;q=0.9,image/avif,image/webp,image/apng,*/*;q=0.8,application/signed-exchange;v=b3;q=0.7
Sec-Fetch-Site: none
Sec-Fetch-Mode: navigate
Sec-Fetch-User: ?1
Sec-Fetch-Dest: document
Accept-Encoding: gzip, deflate, br, zstd
Accept-Language: en-US,en;q=0.9,cs;q=0.8
Priority: u=0, i
Host: XXX
GEOIP_ADDR: XXX
GEOIP_CONTINENT_CODE: XXX
GEOIP_COUNTRY_CODE: XXX
GEOIP_COUNTRY_NAME: XXX

--6397c64b-F--
HTTP/1.1 406 Not Acceptable
Content-Length: 249
Connection: close
Content-Type: text/html; charset=iso-8859-1
Date: Tue, 21 May 2024 12:46:58 GMT
Server: Apache

--6397c64b-E--
<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN">

<title>406 Not Acceptable</title>

Not Acceptable

An appropriate representation of the requested resource could not be found on this server.

--6397c64b-H--
Stopwatch:<PHONE_NUMBER>111722 7610 (- - -)
Stopwatch2:<PHONE_NUMBER>111722 7610; combined=3446, p1=721, p2=2355, p3=49, p4=156, p5=165, sr=126, sw=0, l=0, gc=0
Response-Body-Transformed: Dechunked
Producer: ModSecurity for Apache/2.9.5 (http://www.modsecurity.org/); OWASP_CRS/3.3.5.
Server: Apache
Engine-Mode: "DETECTION_ONLY"

--6397c64b-Z--

To Reproduce
Expected behavior
Stop logging this HTTP CODE 406.
Server (please complete the following information):

ModSecurity 2.9.5
Apache 2.4.59
Ubuntu 22.04 LTS

Rule Set (please complete the following information):

OWASP Core Rule Set 3.3.5

Additional context
Add any other context about the problem here.

Hi @ShaiMagal,
perhaps I would try something like this:
SecRule RESPONSE_STATUS "@within 406" \
    "id:1000,\
    phase:3,\
    t:none,\
    nolog,\
    pass,\
    ctl:auditEngine=Off"

References:

RESPONSE_STATUS
@within
ctl

I would put it before the other rules, eg. as I see you use CRS, then you can add it into REQUEST-900-EXCLUSION-RULES-BEFORE-CRS.conf file.

Hi @airween,
thank you for example, it's not working, BUT, it was very helpful, and this is working example for us :)
SecRule RESPONSE_STATUS "@streq 406" "id:XXX,phase:5,t:none,nolog,pass,ctl:auditEngine=Off"
Thank you again, you save my day! :)


fix: MergedDataLoader is compatible with load with parameters such as ConfluenceLoader

Replace this entire comment with:

Description: [a description of the change, ](fix: MergedDataLoader is compatible with load with parameters such as ConfluenceLoader)
Issue: https://github.com/langchain-ai/langchain/issues/12923
Dependencies: no
Tag maintainer:
Twitter handle: @chyroc_

Please make sure your PR is passing linting and testing before submitting. Run make format, make lint and make test to check this locally.
See contribution guidelines for more information on how to write/run tests, lint, etc:
https://github.com/langchain-ai/langchain/blob/master/.github/CONTRIBUTING.md
If you're adding a new integration, please include:

a test for the integration, preferably unit tests that do not rely on network access,
an example notebook showing its use. It lives in docs/extras directory.

If no one reviews your PR within a few days, please @-mention one of @baskaryan, @eyurtsev, @hwchase17.
-->

Sorry to bother you, could you please review this PR for me? If there are any problems, I will correct them immediately. @baskaryan

@eyurtsev Sorry to bother you, could you please review this PR for me? If there are any problems, I will correct them immediately.

@baskaryan Sorry to bother you, could you please review this PR for me? If there are any problems, I will correct them immediately.

@eyurtsev Sorry to bother you, could you please review this PR for me? If there are any problems, I will correct them immediately.

we probbly dont want to override the signature of load


The feed should show all notes since I closed Primal

Hi,
I close Primal and open in the next hours, and the feed starts showing messages only from 30 minutes or so. I would like te see the feed start since the moment I closed Primal, so wouldn't lose notes.
Maybe show all notes since I closed Primal, limited by 24 hours.
Thanks.

This is a known issue. Still waiting on the backend to resolve this.
I think you already submitted this issue, something like a gap in the feed.


[bug] Storybook Test Runner Timeout with waitForPageReady

Describe the bug
My Storybook test runner tests are timing out with the error "Exceeded timeout of 30000 ms for a test" when using the waitForPageReady function. This issue occurs specifically in the test "Components/ImageEditor › WithInitialImage › play-test".
To Reproduce
Run your Storybook tests using the provided configuration in .storybook/test-runner.ts.
Observe the timeout error for the "Components/ImageEditor › WithInitialImage › play-test" test.
Expected behavior
Tests should take screenshots after waiting for the content to render completely, including fonts and external images.
Tests should complete within a reasonable timeframe (ideally without exceeding the timeout).
Screenshots
(Optional: Include screenshots of the Storybook test UI and the timeout error message if applicable)
System
System:
OS: Linux 5.15 Ubuntu 20.04.6 LTS (Focal Fossa)
CPU: (12) x64 12th Gen Intel(R) Core(TM) i7-1255U
Shell: 5.0.17 - /bin/bash
Binaries:
Node: 20.10.0 - ~/.nvm/versions/node/v20.10.0/bin/node
Yarn: 1.22.19 - /usr/bin/yarn
npm: 10.2.3 - ~/.nvm/versions/node/v20.10.0/bin/npm <----- active
Browsers:
Chrome: 121.0.6167.160
npmPackages:
@storybook/addon-a11y: ^8.0.2 => 8.0.2
@storybook/addon-actions: ^8.0.2 => 8.0.2
@storybook/addon-backgrounds: ^8.0.2 => 8.0.2
@storybook/addon-controls: ^8.0.2 => 8.0.2
@storybook/addon-coverage: ^0.0.9 => 0.0.9
@storybook/addon-designs: ^8.0.0 => 8.0.0
@storybook/addon-docs: ^8.0.2 => 8.0.2
@storybook/addon-interactions: ^8.0.2 => 8.0.2
@storybook/addon-links: ^8.0.2 => 8.0.2
@storybook/addon-mdx-gfm: ^8.0.2 => 8.0.2
@storybook/addon-measure: ^8.0.2 => 8.0.2
@storybook/addon-outline: ^8.0.2 => 8.0.2
@storybook/addon-storysource: ^8.0.2 => 8.0.2
@storybook/addon-themes: ^8.0.2 => 8.0.2
@storybook/addon-viewport: ^8.0.2 => 8.0.2
@storybook/addon-webpack5-compiler-babel: ^3.0.3 => 3.0.3
@storybook/jest: ^0.2.3 => 0.2.3
@storybook/preview-api: ^8.0.2 => 8.0.2
@storybook/react-webpack5: ^8.0.2 => 8.0.2
@storybook/test-runner: ^0.17.0 => 0.17.0
@storybook/testing-library: ^0.2.2 => 0.2.2
storybook: ^8.0.2 => 8.0.2
npmGlobalPackages:
@storybook/cli: 7.6.17
Additional context
The code uses waitForPageReady to wait for the entire page to be ready before taking screenshots.
While this ensures consistent results, it might be causing the timeout due to slow-loading fonts or external images.
Increasing the timeout to 30000 ms doesn't resolve the issue entirely.
Possible Solutions Explored:
Increasing the timeout (not ideal as it masks the root cause)
Using page.waitForSelector for specific elements (might not be suitable for fonts and external images affecting the entire page)
Here are two potential approaches to consider:
Optimize Font and Image Loading: Investigate the performance of fonts and external images. Look for ways to optimize their loading times (e.g., preloading fonts, using Content Delivery Networks for images).
Custom Waiting Strategy: Create a custom waiting function that combines page.waitForSelector for essential elements (like the ImageEditor component itself) with techniques to wait for network requests to finish, ensuring all resources for the component are loaded before taking screenshots.

it's not working even i have added this
test-runner-jest.js
const { getJestConfig } = require('@storybook/test-runner');
/**

@type {import('@jest/types').Config.InitialOptions}
/
module.exports = {
// The default configuration comes from @storybook/test-runner
...getJestConfig(),
/* Add your own overrides below

@see https://jestjs.io/docs/configuration
*/
testTimeout: 30000,
};

@shajjhusein I had this same issue, it seems there's no built in way to configure the playwright timeouts (e.g. navigation timeout which you are hitting) when using the storybook test runner (possible feature request for this?).
For now I'm using the following workaround (basically replace defaultPrepare from storybook):
.storybook/test-runner.ts
import type { TestRunnerConfig } from '@storybook/test-runner';

const config: TestRunnerConfig = {
  async prepare({ page, browserContext, testRunnerConfig }) {
    page.setDefaultTimeout(120 * 1000); // set here to whatever you need

    const targetURL = process.env.TARGET_URL;
    const iframeURL = new URL('iframe.html', targetURL).toString();

    if (testRunnerConfig.getHttpHeaders) {
      const headers = await testRunnerConfig.getHttpHeaders(iframeURL);
      await browserContext.setExtraHTTPHeaders(headers);
    }

    await page.goto(iframeURL, { waitUntil: 'load' }).catch((err) => {
      if (err instanceof Error && err.message.includes('ERR_CONNECTION_REFUSED')) {
        const errorMessage = `Could not access the Storybook instance at ${targetURL}. Are you sure it's running?\n\n${err.message}`;
        throw new Error(errorMessage);
      }

      throw err;
    });
  },
};

export default config;

I'm having the same issue, and my workaround is to use:
Option 1:
await page.evaluate(() => document.readyState === 'complete');
Option 2:
await page.evaluate(() => document.fonts.ready);

Still an issue.
Work around is to install playwright and create your own helper


[HW] Gökcan Çantalı

Name: Gökcan Çantalı
Email<EMAIL_ADDRESS>Linkedin Profile: https://www.linkedin.com/in/gökcan-çantalı-aa31319a/
Attach the homework step I screenshots below:

Attach the homework step II screenshots below:
From Ania's Tutorial:

Hey Gökcan Çantalı, great job! Congrats! Here is your badge! https://api.badgr.io/public/assertions/WCz4WGRHQ3-LydjYorh6jg


Add html repr for dataset, datagroup, array

I'm not sure this is so useful. The string reprs contain all the info we want, and unless we want to add collapsable sections in Dataset, I don't see much use in having a html repr for an array or Datagroup.
If we really want something pretty for a DataGroup, we can convert it to a DataFrame using to_pandas, or leverage pandas to add a repr?
Something like
import pandas as pd

pd.DataFrame(data={"min": [a.norm.min().values for a in mesh.values()],
                   "max": [a.norm.max().values for a in mesh.values()]},
             index=mesh.keys())


Add resend api

adds resend api used like this:
await client.publishJSON({
  api: {
    name: "email",
    provider: resend({ token: process.env.RESEND_TOKEN! }),
  },
  body: {
    from: "Acme<EMAIL_ADDRESS>    to<EMAIL_ADDRESS>    subject: "hello world",
    html: "
it works!
",
  },
});

body is not typesafe. Out llm api isn't typesafe too. It's not very simple to make it typesafe currently.
We can go ahead with this version and make it typesafe later

cc? bcc?

since it's not typesafe, it will work with all parameters in https://resend.com/docs/api-reference/emails/send-email

I feel like we can still handle batch operation without passing the batch param to the resend client. Like inferring from the object count in the body maybe?
        await client.publishJSON({
          api: {
            name: "email",
            provider: resend({ token: resendToken }),
          },
          body: [
            {
              from: "Acme<EMAIL_ADDRESS>              to<EMAIL_ADDRESS>              subject: "hello world",
              html: "
it works!
",
            },
            {
              from: "Acme<EMAIL_ADDRESS>              to<EMAIL_ADDRESS>              subject: "world hello",
              html: "
it works!
",
            },
          ],
        });
      },

That sounds nice but I would prefer to avoid putting in logic we would have to maintain.
A similar optimization could have been made in qstash-js publishJSON vs batch, or it could have been done by resend itself (having a single API for single & batch email).


Introduce jobtype variable in worker dashboard

The tradeoff is the 2nd target_duration variable. As it's tricky to have a variable with multiple values (so we could define the target duration per jobtype), you sort of need to know which target_duration to set for which jobtype (1792 for osbuild jobs, 32 for depsolve/resolve jobs).
https://grafana.stage.devshift.net/d/image-builder-worker-sanne/image-builder-worker-sanne?orgId=1

Can you explain a bit more why is there a need to specify the target time statically please ? I think I kinda understand but a bit unsure.

Can you explain a bit more why is there a need to specify the target time statically please ? I think I kinda understand but a bit unsure.

So per jobtype we have different targets of what consititutes "fast". We commit to 95% of depsolve jobs taking 32s or lower, and we commit to osbuild jobs taking less than 30 minutes. I haven't really found a way to change the jobtype and the target time at the same time, so I made a 2nd variable.
The target time is needed to correctly calculate the error budget and slow request rates in the job duration and job wait duration sections.

The tradeoff is the 2nd target_duration variable. As it's tricky to have a variable with multiple values (so we could define the target duration per jobtype), you sort of need to know which target_duration to set for which jobtype (1792 for osbuild jobs, 32 for depsolve/resolve jobs).

Awesome stuff. It looks a lot better and less cluttered like this. Pity about the jobtype... but I don't think it's a big deal. I mean this is mostly for us right? And I guess we'll know to switch the target duration for each job. would it make sense then to have. Having said that, would it make more sense to have the input selector be a single selector so you can only view one job at a time rather than a multi select?

Discussed with Tom:

job wait duration per architecture
future: stats per image type

Discussed with Tom:
* job wait duration per architecture

Ok so we don't actually add the arch label to the job_wait_duration metric.

* future: stats per image type

Job wait duration per architecture is now available after #3289


Missing brackets closing, page 24 (latex) line 18/19  [version 2022-03-14]

after "admixture model" bracket opens, but never closes

fixed


Add team insights on weekly contributions

Provide more insights on a team
Description
On team page you are now able to see

CRED earn by a team
Total contribution
Weekly CRED's and the weeks growth

What type of PR is this? (check all applicable)

[x] 🍕 Feature
[ ] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 🎨 Style
[ ] 🧑💻 Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI
[ ] 📦 Chore (Release)
[ ] ⏩ Revert

Added it on the team page as well, PR here for the Overview: https://github.com/armitage-labs/armitage-monorepo/pull/48


memo PictureEditDialog to stop rerenders

Closes #445
Moving your mouse constantly rerenders everything inside the FaceTaggingProvider, including the ImageEditor. The ImageEditor isn't made to be rerendered constantly and is therefore becoming unusable.

Looks good. While testing, I noticed that the code for catching a bug in tui doesn't work on Firefox, since the error message differs. Maybe you could change it to something like error instanceof TypeError && error.message.includes("null")?

The error gets caught now on firefox, but firefox just seems to ignore the event.preventDefault() and event.stopPropagation() anyways.


Aliases in the router - why it is worth to reconsider it

Please reconsider adding aliases to the router.
Why?

It is very useful when porting old application to nuxt, especially when we want to save the old link structure

What about seo?

Some of the application that we are porting are just for the intranet. So the seo is not a problem.
For the rest of it we use meta: "noindex, follow"

Workaround:

Now we have to use extendRoutes but we prefer more clean way to do this.

Thanks in advance!

I'm also looking for a clean way to do this.
If I have a project with the following pages/components:

pages
---- companies
---- inbox
---- calendar

I should be able to have URLs like this:
/companies/{companyID}/inbox
/companies/{companyID}/calendar
without having to put my inbox and calendar pages inside the companies folder. I love the autogenerated routes but it would be very helpful to also be able to define aliases.

In the meantime what I think I would do is have my components outside the pages directory and use symbolic links to build the routes using the schema I need

@sebastianmacias you can import a page component into another page component if you want to reuse it.

It will be possible to update our router.js template to write your own with the modules system in the upcoming release.

Closing it since rc3 is out, to see an example: https://github.com/nuxt/nuxt.js/blob/dev/test/fixtures/module/modules/template/index.js

Hi @Atinux
How can I use this?
I've looking for a while but cannot find how to implemente it.
Thanks!


Adding opensearch-sql-cli artifact sections to the most recent entrie…

…s in _versions.
Replaces #1051 and solves #654
Description
Adds another section, and a new download artifact for our cli interface to the sql plugin.
Issues Resolved
#654
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the BSD-3-Clause License.

Looking forward to see this change goes live!


Add Velero Backup object when ClusterDeployment changes.

[x] Add watch for changes to clusterdeployment objects
[x] Add reconcile that checks all clusterdeployment objects in a given namespace
[x] Add unit tests around core functionality
[x] Test that backup object is created correctly.

/test verify

/test e2e

/test integration
/test unit

@dgoodwin @LalatenduMohanty I rebased and I believe I've addressed all of the feedback.
Please review.

/lgtm


use wizard variant of temptifly

Signed-off-by: John Swanke<EMAIL_ADDRESS>open-cluster-management/backlog#11435
open-cluster-management/backlog#11434
open-cluster-management/backlog#12170
create cluster is a wizard now:

Also has a fix for https://github.com/open-cluster-management/backlog/issues/12132 to center the text


Unable to upload files from CMS...

How do I upload files from the CMS???

Hello. How do you mean? Pleas explain what's happening.

No worries... I figured it out... Actually I was unable to find way to add files.. Then I saw Mediain sidebar 😅

Oh right. This is not production ready yet. Are you considering using this ? I am working to have it production ready this week.

Yes I am thinking of using it in production of my App. Can we integrate Stripe with Tensei??
Well this is an very powerful and awesome library.. I was able to get my backend done in 1-2 hours 😆


If statement wrapping attributes need white at the end

Curlylint wants a space after {% endif %}. But this seems valid scenario to me:
 <form {% if has_file_field %}enctype="multipart/form-data" {% endif %}action="{{ form_url }}"></form>

It raises:
Parse error: expected one of '>', '\\s+', '{#', '{%', '{{' at 45:70	parse_error

Curlylint is okay when 'shifting' the white:
<form{% if has_file_field %} enctype="multipart/form-data"{% endif %} action="{{ form_url }}"></form>

I used curlylint, version 0.12.0

Hey @allcaps, thank you for taking the time to report this!
This needs fixing, I don’t know what change exactly is needed, but the code is here: https://github.com/thibaudcolas/curlylint/blob/aa9c15bde399b4a7ad9541e5a3feac6c1a9a2f83/curlylint/parse.py#L416-L420
I’m working on adding more tests for the parser. Once they’re in place, it should hopefully be much easier to fix things like this.


Panning and Zooming over a double.NaN results in crash (OnNaN set to Gap)

Bug Report
Issue: (Bug description goes here)
When using OnNaN set to Gap, panning or zooming on an area of double.NaN results in this exception.
"An unhandled exception of type 'System.InvalidOperationException' occurred in ScottPlot.dll
Y must be a real number"
It works fine on portions of the scatter with data.
Reproducing: (What do others have to do to reproduce the bug?)
create a scatter plot with double.NaN in a set OnNaN to Gap. Then try and pan or zoom when clicking a section of y with the NaN (i.e. the gap).
// a code sample may improve communication
            WpfPlot1.Plot.Clear();
            WpfPlot1.Plot.Title($"Loaded Time Series Data");
            Sp = WpfPlot1.Plot.AddScatter(xs, channel);
            Sp.OnNaN = ScottPlot.Plottable.ScatterPlot.NanBehavior.Gap;
            CrossHair = WpfPlot1.Plot.AddCrosshair(0, 0);
            CrossHair.IsVisible = false;
            ToolTip = WpfPlot1.Plot.AddTooltip(label: "test", x: 0, y: 0);
            ToolTip.IsVisible = false;
            WpfPlot1.Refresh();

System Details

ScottPlot Version: 4.1.56
Operating System: Windows 10.)
Application Type: ( WPF)
.NET Version: (.NET 6.0)

This also occurs when OnNaN is set to Ignore....

Looks like it something to do with my code, behaviour happens with an overlay enabled and not when it is disabled. Carry On....


IntentIntegrator always returns result -1

First of all, I'm far from an Android professional when it comes to programming so I hope this is actually an issue worth reporting, but I felt it being strange that the result code is "error" when it should be "success" for a successful qr-code detection and data read.
This is my findings;
Description of the problem:
Initializing a new IntentIntegrator and then scanning a QR-code always returns -1 (error), even if the data is complete and has been read successfully.
Which library version are you using?
implementation "com.journeyapps:zxing-android-embedded:3.6.0"
implementation "com.google.zxing:core:3.3.3"

Which phone/tablet are you using, and which Android version does it run?
Samsung Galaxy S7 (Android 8.0.0), OnePlus 6 (Android 8.1.0)
Does the same happen on other devices or an emulator?
I've only tested the two devices.
Can you reproduce the issue in the sample project included with the library? If not, can you
provide your own sample project or sample code that produces this error?
// Initializer
IntentIntegrator qrScannerIntent = new IntentIntegrator(this);
qrScannerIntent.setBeepEnabled(false);
qrScannerIntent.setPrompt("");
qrScannerIntent.setRequestCode(REQUEST_QRCODE_SCAN); // 55
qrScannerIntent.initiateScan();

// This doesn't work as described by your documentation
protected void onActivityResult (int requestCode, int resultCode, Intent data)
{
    if (requestCode == REQUEST_QRCODE_SCAN) {
        // resultCode is always -1, as such intentResult is always null too
        IntentResult intentResult = IntentIntegrator.parseActivityResult(requestCode, resultCode, data);
    }
}

// Solution is to do this instead;
protected void onActivityResult (int requestCode, int resultCode, Intent data)
{
    // QR code scan?
    if (requestCode == REQUEST_QRCODE_SCAN) {
        if (!data.hasExtra(Intents.Scan.RESULT)) {
            return;
        }

        String qrCodeResult = data.getStringExtra(Intents.Scan.RESULT);
        if (qrCodeResult.length() == 0) {
            return;
        }

        // Works!! qrCodeResult contains the data from the QR code, even if resultCode is -1!
    }
}

Same problem here.
Has anyone solved this?

@SvenCarstensen I provided a solution in the code. Just do that and you'll be good. It'll work now and after the bug is fixed.

Where did you get the value of REQUEST_QRCODE_SCAN from? For me the request code is 49374.

I'm not by the computer now and it was a while since I reported this, but I think that is my activity request code, it will be different depending on your implementation, you decide it's value to determine what result is being passed to onActivityResult().

Just to clarify -1 does not mean error. -1 is equivalent to Activity.RESULT_OK. But thanks for the solution @vebut, I've been pulling my hair the whole day!
Here's what's weird, the code you provided above is that same exact code I tried, it works fine when I'm using java but when in a kotlin activity that's when I get the issue. The only difference I think is that in my java code, I passed in MyActivity.this in the contructor parameter of IntentIntegrator instead of just passing 'this'.
Didn't really get it, there must be something that's going on here.

I had similar problem. My problem was that I called IntentIntegrator.parseActivityResult() with intent as argument and not data from onActivityResult().
In Kotlin:
    override fun onActivityResult(requestCode: Int, resultCode: Int, data: Intent?) {
        val scanResult = IntentIntegrator.parseActivityResult(requestCode, resultCode, data)
        if (scanResult != null) {
            AlertDialog.Builder(this)
                .setMessage(scanResult.toString())
                .show()
        } else {
            super.onActivityResult(requestCode, resultCode, data)
        }
    }

If you change the requestCode, you have to use the parseActivityResult(resultCode, data) method (the version without the requestCode parameter).

If you change the requestCode, you have to use the parseActivityResult(resultCode, data) method (the version without the requestCode parameter).

Thanks for this, I wasted hours this evening due to this... Onwards and upwards!


Add CORS headers to response (#665)

Issue #, if available:
#665
*Description of changes:

Allow user config cors headers.
Add cache control headers

To run CI tests on your changes refer README.md
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

To make sure I understand correctly, correct usage would be to enable this in the config:
cors_allowed_origin=thomasdelteil.github.io
Correct? Once this is merged is it going to find its way in the :lastest docker build?

@ThomasDelteil : The docs are updated with the usage. Could you check if that answers your question?


MINOR: revert change in log level for skipped record

It looks like it was a mistake during #6521
Specifically, while addressing code review comments
to change other logs from debugs to warnings, this
one seems to have been included by accident.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Ping @ableegoldman or @cadonna for a review

I replied on the user mailing list thread about my hesitation to "fix" this, but if we do ultimately decide to go back to debug note that we should also fix this in InMemorySessionStore  and AbstractRocksDBSegmentedBytesStores

After discussing this with @cadonna , he filled me in that this change was actually intentional as part of KIP-444. Closing this PR.

@vvcephei  @ableegoldman  Hi, I'm seeing large amount of this WARN log every time our instances restart. I think it's just the same case as described in the mail list. These logs are causing high disk and CPU consumption.
Is there any followup after that mail thread? Thanks!

Hi @IndeedSi ,
No, as far as I know Jiri's solution was to increase the join window size. Judging from the investigation, it seemed like there really were out-of-order records in the repartition topic that needed to be buffered so they could be processed instead of dropped (as opposed to some kind of superfluous logging as we initially thought).
Does you investigation point to the same cause? If so, then really the only thing that can be done is to increase the join window (aka buffer more records), so that you aren't dropping any data.
I hope this helps,
-John

@vvcephei  Thanks, John
I think you are right. We did end up with larger window size (for some other reason) and this log seems no longer happening during restarts.


Cant disable auto focus?

I have setup focusMode with 'FocusMode.FIXED' but seems equals to FocusMode.AUTO, focus still continue automatically

I think there might be some differences between what different devices support. Perhaps the library could check for this better, and give some feedback on the JS side.


Refactor JS bindings

At the moment it's inconsistent the usage of Uint8Array and Buffer and in general bindings can be rewritten to be more performant

Closed by #33


invoke-rc.d: could not determine current runlevel with Openssh

Hello, my windows 10 version is  10.0.18363.836.
I've started with WSL1 and already have some files in the system.
Thus, I don't wish to move to WSL2 even though I think that my version is low so I cannot upgrade.
However, I hope to have GUI with the WSL ubuntu 18.04.
Thus, with my limited cap, I've done some search online and it seems that it is possible to open a port via Openssh and get an access to the GUI VcXsrv in the windows system from the native X xfce in the Ubuntu. Yet, to do so I had to do the following command,
xxx@yyy:~$ sudo dpkg-reconfigure openssh-server
[sudo] password for xxx:
invoke-rc.d: could not determine current runlevel
Why am I getting this error message? What is the reason?
Also, would there be any way that I can access VcXsrv from Ubuntu in WSL1?
Thank you so much!
Any unsolicited ideas are welcome.

Environment
Windows build number: [run `[Environment]::OSVersion` for powershell, or `ver` for cmd]
Your Distribution version: [On Debian or Ubuntu run `lsb_release -r` in WSL]
Whether the issue is on WSL 2 and/or WSL 1: [run `cat /proc/version` in WSL]

Steps to reproduce

Expected behavior

Actual behavior

Why am I getting this error message? What is the reason?

/dupe #573

Also, would there be any way that I can access VcXsrv from Ubuntu in WSL1?

If I list multiple bugs/concerns in this one issue, the core team may close my issue without further explanation or engagement.

https://github.com/rescenic/autossh
https://dev.to/darksmile92/linux-on-windows-wsl-with-desktop-environment-via-rdp-522g
Those are the guides that will lead you to have running xrdp in your ubuntu wsl.

Wow! it works. Thank you so much for your help, Rescenic. Even without autossh, just with the remote desktop connection, even though I had to pay for it to upgrade into pro..., anyhow, rdc made all possible. Thank you again!

Welcome. When connecting to SSH Server (sshd), you can enable X11 Forwarding in SSH Client, like Putty and VcXsrv Display Server.

Thank you for your suggestions. I hope to do so... Yet, I am new to all these so.. is there any website or a thread that explains more step by step how to configure that? Thank you!

There are lots of tutorials in youtube. Just search for the relevant keywords.

Ok. I will look for them. Have a great day!


chore: add lodash as devDependency

I used lodash that was a dependency someone pulled (grunt) but I am using npm3 so lodash was on the root. Better to add it as a proper dependency.
Also my "npm i -S lodash" decided to tweak the package.json, meh, I like the changes tho.

Works for me


Use filename when creating streamlit app, not full path

Fixes #54
This makes streamlit create --file relative/path.py work the same as streamlit deploy --file relative/path.py, so that they both only look at the filename path.py.
Previously, streamlit deploy would create the file path.py in the new stage, but streamlit create would create a streamlit app that was expecting a path to be located at relative/path.py

Beautiful


Feature/issue 864 ard exp quad

Summary
This PR adds support for separate length scales in the squared exponential kernel. For function definitions that have a vector length scale, we use the function divide_columns as in the matern52 kernel to divide each column by the corresponding length scale.
Tests
Tests include column vector for data input, scalar and vector
Side Effects
Changed marginal_variance parameter name to magnitude.
Checklist

[x] Math issue #864

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

[x] the basic tests are passing

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

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

[x] the new changes are tested

@drezap , looks like you're getting the same failure of the upstream cmdstan tests on Windows as I'm getting on https://github.com/stan-dev/math/pull/1093 . (A warning that some files are too big.) I wonder if this is something that should be fixed in stan/cmdstan. @bob-carpenter , this reminds me of something I remember you mentioning a long time ago, about Windows builds failing because of long files, meaning that you've had to split up files. Any idea how we should proceed here?

As for divide_columns there was suggestion to use "divide" but this only supports dividing a vector by a matrix. Can leave as is, or I could update divide.hpp to support dividing a length n vector by n different values, but could have unexpected behavior as divide is exposed to the language, and divide_columns is not.

@syclik

@syclik good comments. I also added a test case that checks numerical accuracy of the ARD. I just chose something that can be trivially calculated by hand, but if you want something more robust I can  update accordingly. Yeah, if you think these docs are in a good state I can propagate the changes to other covariance functions.

Maybe it's time to chat over video or slack

Sure, whatever your preference is. This could probably just happen on discourse, or a wiki, or discourse with a link to a wiki. I don't feel the need for you to show me anything over video, unless you want to. Some aren't fans of meetings. There's some book about it, and about combinatorial issues with business structure and productivity.......

@syclik

@syclik is there any reason this hasn’t gone through yet?

Thanks!

This could probably just happen on discourse, or a wiki, or discourse with a link to a wiki. I don't feel the need for you to show me anything over video, unless you want to. Some aren't fans of meetings.

That’s actually my preference too. I don’t like meetings.

@syclik is there any reason this hasn’t gone through yet?

Nothing other than time.

Appreciate it, thanks.
Andre
On Sunday, February 17, 2019, Daniel Lee<EMAIL_ADDRESS>wrote:

Thanks!
This could probably just happen on discourse, or a wiki, or discourse with
a link to a wiki. I don't feel the need for you to show me anything over
video, unless you want to. Some aren't fans of meetings.
That’s actually my preference too. I don’t like meetings.
@syclik https://github.com/syclik is there any reason this hasn’t gone
through yet?
Nothing other than time.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/stan-dev/math/pull/1095#issuecomment-464570608, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ALHebCypmbYLKNl2WmaOXTig534rcm9oks5vOh_RgaJpZM4ZjU1R
.

--
Best,
Andre Zapico


WIP: fixes #524

Can we follow the same pattern for every mission?

No need to create different Properties files, only the OSIO-MISSION-README.adoc

@gastaldi updated the readmes but kept the properties files due to different file paths to edit.
Let me know what you think.

A lot of the boosters don't work in OSIO (or OSO Starter), but this PR includes READMEs for them as well. I guess that's intentional, but wanted to point out anyway.

Thank you @Ladicek. I added them all for now, but they'll never get used until they are ready in OSIO/OSO.
@gastaldi removed the extra file.
merging.


timezonefinder 6.2.0 :snowflake:

timezonefinder 6.2.0 :snowflake:
Destination channel: Snowflake
Links

PKG-3776
dev_url (pypi): https://github.com/jannikmi/timezonefinder/tree/6.2.0
conda-forge: https://github.com/conda-forge/timezonefinder-feedstock/blob/main/recipe/meta.yaml
pypi: https://pypi.org/project/timezonefinder

Explanation of changes:

fix recipe
skip s390x because numba is not there
add one simple test

Linter check found the following problems:
ERROR conda.cli.main_run:execute(124): `conda run conda-lint /tmp/abs_8ffayud3ii/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:26: host_section_needs_exact_pinnings: Linked libraries host should have exact version pinnings.
===== Final Report: =====
0 Errors and 1 Warning were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(124): `conda run conda-lint /tmp/abs_941gjxmqq0/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:26: host_section_needs_exact_pinnings: Linked libraries host should have exact version pinnings.
===== Final Report: =====
0 Errors and 1 Warning were found


gitin: remove at 0.2.3

gitin depends on a precise version of git2go. Not even the latest
gitin/git2go version is compatible with an up to date libgit2. As the
upstream project hasn't seen activity in some time and doesn't respond
to the related issue (https://github.com/isacikgoz/gitin/issues/71),
drop this package.

Motivation for this change
cc @Artturin @kimat
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

:+1: LGTM


Warning when using AnimationParameter on a disabled GameObject.

This warning is called whenever there is a AnimationParameter on a disabled GameObject:
Animator is not playing an AnimatorController
UnityEngine.Animator:get_parameterCount()
BitStrap.AnimationParameterDrawer:OnGUI(Rect, SerializedProperty, GUIContent) (at Assets/Scripts/BitStrap/Plugins/Editor/Animation/AnimationParameterDrawer.cs:36)
UnityEditor.Editor:DrawDefaultInspector()

Fixed on 175029a51d2f959050f364ee33f81972517a255c
Although the solution is not displaying the dropdown when the GO is inactive or when there's no AnimatorController.
I'm not yet sure if there's a way to query Animator params when the GameObject is not active.

I'll close this for now


FastBuild won't create GetTempDir()/FastBuild folder if not present

Heya!
I noticed that I sometimes had a empty monitor window, if I started the build first and then the monitor.
This is because if a build is started with FastBuild with -monitor, and the folder GetTempDir()/FastBuild doesn't exist, FastBuild won't create it, hence won't be able to write the log file the monitor reads, giving that empty window if the monitor is opened.
The folder is created by the monitor though, so if the monitor is opened first, and then the build is started, everything works :)

Closing this since it was actually a FastBuild issue that was fixed :)


Some properties lacking detail and defaults

Re: the clean property:
"Scorch the repo before fetching?" isn't very informative :)
Does it issue a "git clean"? With what params?
Also, the description for this property doesn't indicate what the default value is. Several other properties don't have their defaults indicated either.
Thank you.

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

ID: b28abf7f-ecdf-8037-acd9-722690424372
Version Independent ID: b28abf7f-ecdf-8037-acd9-722690424372
Content: steps.checkout definition
Content Source: content/steps-checkout.md
Product: devops
Technology: devops-cicd
GitHub Login: @steved0x
Microsoft Alias: sdanie

@PJBowron That is one of the unusual ones in there. These descriptions come from the schema that is used for validating the YAML Pipeline editor, for example:
https://github.com/microsoft/azure-pipelines-vscode/blob/282970bd18b11fe3da3b8c450ee15715b60ac9e4/service-schema.json#L3353
Unfortunately this schema doesn't include all of the defaults, and since this reference is generated from the schema, it doesn't include them. We are about to roll out an update to the engine that generates this documentation, and one improvement will be the ability to add in richer property descriptions to include things like defaults more easily.
There is more information on cleaning options here:
https://docs.microsoft.com/en-us/azure/devops/pipelines/process/phases?view=azure-devops&tabs=yaml#workspace
But that article doesn't explicitly say what the checkout clean does. When checkout clean is true, it runs git clean -ffdx followed by git reset --hard HEAD before fetching.
I'll get that updated both here and in that other article. Thank you for letting us know.


Select not working, Redis.om generate redis query with wrong column count

Okay I think I have run into a bug.
I have this class:
[Document(StorageType = StorageType.Json)]
public record ProductTimelineCache
{
[RedisIdField]
public Ulid Id { get; set; }
[Indexed]
public string? ProductId { get; set; }
[Indexed]
public string? DistributionCenter { get; set; }
[Indexed]
public required string Hash { get; set; }
[Indexed(CascadeDepth = 1)]
public PriceCampaignInterval[] PriceTimeLine { get; set; } = [];
[Indexed(CascadeDepth = 1)]
public ProductExposureCampaignInterval[] ExposureCampaigns { get; set; } = [];
}
Where I want to list all { ProductId, Hash }, excluding all other fields so I write:
await _provider.RedisCollection<ProductTimelineCache>()                 .Select(q => new { q.ProductId, q.DistributionCenter, q.Hash })                 .ToListAsync()                 .WaitAsync(cancellationToken)
This fails with the REDIS Dragonfly syntax error:
ERR syntax error\r\nFailed on FT.SEARCH producttimelinecache-idx * LIMIT 0 100 RETURN 9 ProductId AS ProductId DistributionCenter AS DistributionCenter Hash AS Hash
It fails because of the number "9" above which I presume is the column count. Fixing it to 3 and manually issuing it on CLI makes REDIS respond.

The problem seems to go futher than this, if I remove aliases from the count in redis.om, I only get an empty result because indexed fields are prefixed with $.Name leading to there being no match for the fields in the select part.
Here is my class:
[Document(StorageType = StorageType.Json)]
public record ProductTimelineCache
{
[RedisIdField]
public Ulid Id { get; set; }
[Indexed(IndexEmptyAndMissing = false, JsonPath = "$.ProductId")]
public string? ProductId { get; set; }
[Indexed(IndexEmptyAndMissing = false, JsonPath = "$.DistributionCenter")]
public string? DistributionCenter { get; set; }
[Indexed(IndexEmptyAndMissing = false, JsonPath = "$.Hash")]
public string? Hash { get; set; }
}
This query:
await _provider.RedisCollection()
.Select(q => new { q.ProductId, q.DistributionCenter, q.Hash })
.ToListAsync()
.WaitAsync(cancellationToken))
Generates the wrong query with wrong count and missing paths
FT.SEARCH producttimelinecache-idx * LIMIT 0 1 RETURN 9 ProductId AS ProductId DistributionCenter AS DistributionCenter Hash AS Hash
But should have generated FT.SEARCH producttimelinecache-idx * LIMIT 0 1 RETURN 3 $.ProductId $.DistributionCenter $.Hash

Hi @pabnemlig - Redis OM is creating the correct queries. the count after RETURN is not the number of fields to return, it is the number of arguments that Redis needs to read forward to finish parsing the query. This is an issue with Dragonfly.
For some background, FT.SEARCH / FT.CREATE are part of the RediSearch module (a source available module created by Redis). Dragonfly, in an attempt to provide some level of feature parity, attempted to copy the API. I cannot personally attest to its completeness nor its stability. I can however, speak from my own experience when I say that this is not first API compatibility issue I've heard from Dragonfly's users.


Groupby transform preserves output dtype

Previously transform output was always the same dtype as the groupby
object. This allows the output dtype to differ from the input. Fixes #9807.

looks great! could you add a release note to what's new?

merged via https://github.com/pydata/pandas/commit/30580e7acd6cafd55616d91bedbb8999d65177f5
thanks!


fix(collection): support skipSessions in findOne()

I've been tinkering with transactions in 4.0.0-rc2, came up with the below script:
const assert = require('assert');
const { MongoClient } = require('mongodb');

run().catch(error => console.error(error.stack));

async function run() {
  const client = await MongoClient.connect('mongodb://localhost:31000,localhost:31001,localhost:31002/test?replicaSet=rs');

  const coll = client.db('test').collection('test');
  await client.db('test').dropDatabase();
  await client.db('test').createCollection('test', {});

  const session = client.startSession();
  session.startTransaction();

  let doc = await coll.findOne({}, { session });
  assert.ok(!doc);

  doc = { hello: 'world' };
  await coll.insertOne(doc, { session });

  doc = await coll.findOne({}, { session });
  assert.ok(doc);

  await session.abortTransaction();
  session.endSession();

  console.log('Transaction Done');
  doc = await coll.findOne({}, { skipSessions: true });
  assert.ok(!doc);

  console.log('Done');
}

Without this PR, I get this strange error in that last findOne() because skipSessions isn't actually getting to executeOperation():
$ node .
Transaction Done
MongoError: Must specify autocommit=false on all operations of a multi-statement transaction.
    at queryCallback (/home/val/Workspace/txn/node_modules/mongodb-core/lib/cursor.js:244:25)
    at /home/val/Workspace/txn/node_modules/mongodb-core/lib/connection/pool.js:544:18
    at _combinedTickCallback (internal/process/next_tick.js:131:7)
    at process._tickCallback (internal/process/next_tick.js:180:9)
^C
$

With this PR, everything goes through nicely:
$ node .
Transaction Done
Done
^C
$

@vkarpov15 skipSessions is not part of the public API, and AFAIK should not be used. Every operation that we run against >=3.6 should have either an explicit or implicit session.
This looks very similar to a bug that came up and was fixed in 3.1.0-rc4. Are you using the latest RC?

As an update: I ran your test snippet against node driver 3.1.0-beta4, and it appears to not need the change.

You're right, good catch, I was running -beta2, I'll close this issue. In the future, can you be more consistent with npm tags though? I installed beta2 because I was looking at the beta tag in npm:

@vkarpov15 sorry about the tag confusion, it was released while I was on vacation. Dan and I had not yet discussed the formal strategy for beta tagging, and there was a pressing need from a beta customer so a release was made hastily. There will be an upcoming beta5 which updates to the latest version of the spec, and that will revert back to correct tagging for beta.


[Feature] Support rewriting footer of segment files replicated from another cluster to correct column unique id

Why I'm doing:
If a segment file is replicated from another cluster, the column unique ids in its footer may be not matched with tablet schema, causing a crash in query.
What I'm doing:
Rewrite footer of segment files replicated from another cluster to correct column unique id.
Fixes #issue
What type of PR is this:

[ ] BugFix
[x] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[ ] This is a backport pr

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.2
[x] 3.1
[ ] 3.0
[ ] 2.5

@Mergify backport branch-3.2 branch-3.1

Backport branch-3.2 https://github.com/StarRocks/starrocks/pull/39166
Backport branch-3.1 https://github.com/StarRocks/starrocks/pull/39167


Key to Species graph on YIR and Collection/Umbrella projects stats pages soemtimes partially covered

See https://forum.inaturalist.org/t/iconic-taxa-cut-off-on-project-stats-page-and-year-in-review/9632 I can't replicate on my Macbooks, either with the built-in screen or my 27" 4k monitor, regardless of resolution or browser -  maybe it's Windows-only?
Basically what happens is that at certain screen resolutions and browser window sizes, this list gets cut off:

Some descriptions:

Windows 10 / 2560x1440 / 100% (recommended) / Chrome / any browser zoom level
Once I resize my browser window to a little less than half the width of my screen, I can see all taxa.

Windows 10 / 1800 x 1200 / system set to 125%
in browser 100% is no good, but 125% is ok (Chrome)

Link to screen recording here: https://inaturalist.slack.com/archives/C02F8RN5Q/p1672167698641539

I made an attempt at a fix. It works for the browsers I have access to on my PC, but it's not a guaranteed fix. Positioning text in an SVG, which these are, is trickier than plain CSS. For whatever reason PC browsers are adding more padding to the labels in the SVG legends that I was seeing on MacOS browsers and I couldn't in a reasonable amount of time figure out how to get the padding the same. I ended up just giving the legend a little more space, which is pretty tenuous fix

Thanks Patrick.


NumaflowControllerRolloutReconciler should only update spec of Resources if something has changed

NumaflowControllerRolloutReconciler.Reconcile() can be called for various reasons - need to make sure we only do an Update if the new spec is different from the previous.
(Consider auto-healing case in which somebody has manually modified the Numaflow Controller Deployment spec on the cluster directly and we need to overwrite it. Therefore, it's not sufficient to just make a determination if the Version number in the NumaflowControllerRollout spec changed.)
Note: this issue is related to this issue, so maybe the same person should do both.

This is handled by gitops engine with WithResourceModificationChecker option during syncing. Also, note that controller definition in stored as immutable config map, so it should not change once loaded.


patch: add new hub logo

Change-type: patch
Signed-off-by: Alan Boris<EMAIL_ADDRESS>

@balena-ci I self-certify!


Check before free

-- inode table array element can be null (ex: element zero intentionally null)

I can't see the point of this check.
free(NULL) is perfectly valid C, and has been since at least K&R days. The reference from ISO/IEC 9899:1999 (which is the only one I have by my desk) is <IP_ADDRESS> which quotes for free(ptr):-
If ptr is a null pointer, no action occurs.
See also my comment in #1487
What am I missing? Are you getting a coredump which is pointing you in this area? Are you hitting the issue with 0.9.12 which is addressed by #1487?

Indeed.
Although because I am looking at a production server it's not so easy to debug.
The call stack led me to this quite simple function and my (naive) review caught what look(s/ed) like a potential problem.
I was unaware that free(0) is safe.
So, that implies that something that the array holds is no longer available.
For me debugging beyond this level isn't an option, not due to personal ability, but due to environmental problems.  I really don't have the resources to investige in a meaningful way

I might be able to do something with the stack trace you've provided in #1502 to work out whether this is indeed #1487 or something else.
Could you maybe close this PR, re-open #1502, and add some information about the server operating system and version of the XRDP package you've installed? I couldn't see that information.
Is falling back to v0.9.11 something you could do quite easily? I'm not familiar with the remmina client, but if it allows you to map a network drive, this might be a workaround you could use.


bug: fix rename a container with invalid name bug

Signed-off-by: Lang Chi<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
fix rename a container with invalid name bug
Ⅱ. Does this pull request fix one issue?

NONE
Ⅲ. Why don't you add test cases (unit test/integration test)? (你真的觉得不需要加测试吗？)
Added.
Ⅳ. Describe how to verify it
root@compatibility:~ # pouch run -d --name name1 busybox top
root@compatibility:~ # pouch rename name1 name:new
Error: {"message":"Invalid container name (name:new), only [a-zA-Z0-9][a-zA-Z0-9_.-] are allowed"}

Ⅴ. Special notes for reviews

LGTM


Fluentbit FluentbitTooManyErrors prometheus rule does not correctly match "job" label of metric used

The prometheus rule query for FluentbitTooManyErrors does not match the labels of the scraped metrics
The rendered prometheusRule
rate(fluentbit_output_retries_failed_total{job="logging-operator-logging-fluentbit",namespace="cluster-logging"}[10m]) > 0

and the scraped metrics
{container="fluent-bit", endpoint="http-metrics", instance="<IP_ADDRESS>:2020", job="logging-operator-logging-fluentbit-monitor", name="forward.0", namespace="cluster-logging", pod="logging-operator-logging-fluentbit-t75bf", service="logging-operator-logging-fluentbit-monitor"}

Seems to be a disconnect between the service created for the servicemonitor to scrape metrics and the rule
https://github.com/banzaicloud/logging-operator/blob/master/pkg/resources/fluentbit/prometheusrules.go#L33
https://github.com/banzaicloud/logging-operator/blob/master/pkg/resources/fluentbit/service.go#L30
Expected behaviour:
job label filter of the fluentbit rules matches the scraped job label for the used metrics
Steps to reproduce the bug:

deploy logging-operator chart
deploy logging-operator-logging chart enabling both metrics and prometheusRules for fluentbit
see the discrepancy in prometheus for the job used in the rule vs the scraped metric

Environment details:

Kubernetes version (e.g. v1.15.2): 1.21.2
Cloud-provider/provisioner (e.g. AKS, GKE, EKS, PKE etc): minikube
logging-operator version (e.g. 2.1.1): 3.14.2
Install method (e.g. helm or static manifests): helm
Resource definition (possibly in YAML format) that caused the issue, without sensitive data:

apiVersion: logging.banzaicloud.io/v1beta1
kind: Logging
metadata:
  annotations:
    meta.helm.sh/release-name: cluster-logging
    meta.helm.sh/release-namespace: cluster-logging
  creationTimestamp: "2022-01-07T19:23:00Z"
  generation: 2
  labels:
    app.kubernetes.io/instance: cluster-logging
    app.kubernetes.io/managed-by: Helm
    app.kubernetes.io/name: logging-operator-logging
    app.kubernetes.io/version: 3.14.2
    helm.sh/chart: logging-operator-logging-3.14.2
  name: logging-operator-logging
  resourceVersion: "1029131"
  uid: 2cc6235e-ee02-46f8-8a8e-1b004a8e326f
spec:
  controlNamespace: cluster-logging
  fluentbit:
    inputTail:
      Ignore_Older: 30m
    metrics:
      prometheusRules: true
      serviceMonitor: true
    resources:
      requests:
        cpu: 10m
    tls:
      enabled: true
      secretName: logging-operator-logging-fluentbit-tls
      sharedKey: ""
  fluentd:
    metrics:
      prometheusRules: true
      serviceMonitor: true
    resources:
      requests:
        cpu: 10m
    tls:
      enabled: true
      secretName: logging-operator-logging-fluentd-tls
      sharedKey: ""
  nodeAgents: []
status:
  configCheckResults:
    36df92cc: true

$ kubectl get all,servicemonitor,prometheusrule -n cluster-logging
NAME                                                        READY   STATUS      RESTARTS   AGE
pod/logging-operator-logging-fluentbit-t75bf                1/1     Running     0          30m
pod/logging-operator-logging-fluentd-0                      2/2     Running     0          30m
pod/logging-operator-logging-fluentd-configcheck-36df92cc   0/1     Completed   0          18h

NAME                                                 TYPE        CLUSTER-IP      EXTERNAL-IP   PORT(S)               AGE
service/logging-operator-logging-fluentbit-monitor   ClusterIP   None            <none>        2020/TCP              5d21h
service/logging-operator-logging-fluentd             ClusterIP   <IP_ADDRESS>   <none>        24240/TCP,24240/UDP   5d21h
service/logging-operator-logging-fluentd-headless    ClusterIP   None            <none>        24240/TCP,24240/UDP   5d21h
service/logging-operator-logging-fluentd-metrics     ClusterIP   None            <none>        24231/TCP             5d21h

NAME                                                DESIRED   CURRENT   READY   UP-TO-DATE   AVAILABLE   NODE SELECTOR   AGE
daemonset.apps/logging-operator-logging-fluentbit   1         1         1       1            1           <none>          5d21h

NAME                                                READY   AGE
statefulset.apps/logging-operator-logging-fluentd   1/1     5d21h

NAME                                                                              AGE
servicemonitor.monitoring.coreos.com/logging-operator-logging-fluentbit-metrics   5d21h
servicemonitor.monitoring.coreos.com/logging-operator-logging-fluentd-metrics     5d21h

NAME                                                                            AGE
prometheusrule.monitoring.coreos.com/logging-operator-logging-fluentbit         18h
prometheusrule.monitoring.coreos.com/logging-operator-logging-fluentd-metrics   18h

/kind bug

Thanks for the really detailed issue @wondersd !


Testing a new new PR

Docs Build status updates of commit ac4784f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

advocates/maxime-rouiller.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit ac4784f:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: GitBranchDeletedOrForcePushed] Cannot sync git repo to specified commit because branch testing has been deleted or has been force pushed

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


All allocated in bin sizes is 0

I've been following your blog post and it has been amazingly helpful! The only thing I've found is that with jemalloc 4.0.0 (my only variation from your post), the gem returns a 0 for each and every bin allocated.
e.g.,
jq '.arenas[0].bins[].allocated' log/jemalloc.log returns nothing but zeros.
I'm guessing this is a difference between 3.6.0 and 4.0.0 and it's my own fault. But just curious if you've seen it before or had an idea of what might be going on.
Thanks!

Did find that using stats_print which appears to just call malloc_stats_print does still include allocated for each bin.

Seems that the name is not in 4.x series http://www.canonware.com/download/jemalloc/jemalloc-latest/doc/jemalloc.html, assuming I'm reading that remotely correctly.

So strange that it can still be visible in the stats_print, just not available via mallctl!
bins:           size ind    allocated      nmalloc      ndalloc    nrequests      curregs      curruns regs pgs  util       nfills     nflushes      newruns       reruns
                     ---
                  16   1         1584          100            1            0           99            1  256   1 0.386            1            0            1            0
                  32   2         3168          100            1            0           99            1  128   1 0.773            1            0            1            0
                  48   3         4656          100            3            0           97            1  256   3 0.378            1            0            1            0
                  64   4         3584           64            8            0           56            1   64   1 0.875            1            0            1            0
                  80   5         8000          100            0            0          100            1  256   5 0.390            1            0            1            0
                  96   6         9216          100            4            0           96            1  128   3 0.750            1            0            1            0
                 112   7        10976          100            2            0           98            1  256   7 0.382            1            0            1            0
                 128   8         4096           32            0            0           32            1   32   1 1                1            0            1            0
                 160   9        15680          100            2            0           98            1  128   5 0.765            1            0            1            0
                     ---
                1280  21        20480           16            0            0           16            1   16   5 1                1            0            1            0
                     ---
                2560  25        25600           10            0            0           10            2    8   5 0.625            1            0            2            0
                     ---
                5120  29        51200           10            0            0           10            3    4   5 0.833            1            0            3            0
                     ---
large:          size ind    allocated      nmalloc      ndalloc    nrequests      curruns
                     ---
              131072  48       131072            1            0            1            1
                     ---
huge:           size ind    allocated      nmalloc      ndalloc    nrequests   curhchunks
                     ---


Benchmarks

Hello,
did you test the 2D performance on 300W? Also, is the reported result on AFLW2000-3D the 2D error as the code suggests or the 3D error?

300W is not tested.
3D error is not the same as 2D error. For 3D error, the z-axis must perform the translation. PRNet does this.

Thanks for the response.
Is this model only trained to produce low 2D error? Why do you even guess the z-translation in that case? Is the result in the paper of 3.8 % on AFLW2000-3D also  the 2D error?


Pull Request Preview Environments for increasing maintainer productivity

I would like to make life easier for Livebook maintainers by implementing Uffizzi preview environments.
Disclaimer: I work on Uffizzi.
Uffizzi is a Open Source full stack previews engine and our platform is available completely free for Livebook (and all open source projects). This will provide maintainers with preview environments of their PRs in the cloud, allowing them iterate faster and reduce time to merge.
Uffizzi is purpose-built for the task of previewing PRs and it integrates with your workflow to deploy preview environments in the background without any manual steps for maintainers or contributors.
TODO:

[ ] Intial PoC

Hi @waveywaves! Yes, we would love to give Uffizzi a try, it looks very useful!

Hi @waveywaves, we are seeing some errors from time to time: https://github.com/livebook-dev/livebook/actions/runs/3669750462
Is it something that you need to look at? I am worried that we may not be deleting some images. Thanks!

Hi @waveywaves, we are seeing some errors from time to time: https://github.com/livebook-dev/livebook/actions/runs/3669750462
Is it something that you need to look at? I am worried that we may not be deleting some images. Thanks!

Yes the deletion action before this workflow run failed due to quotes that weren't handled. https://github.com/livebook-dev/livebook/actions/runs/3669749323
I've opened a PR to improve this and several other things, bringing your workflows in line with what we're deploying elsewhere. https://github.com/livebook-dev/livebook/pull/1577
Thanks for using Uffizzi and bringing this to our attention!