Improved, single-pass templating.

Relates to #5. All tests pass.

Pull Request Test Coverage Report for Build 13

1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 12:
0.0%

Covered Lines:
19

Relevant Lines:
19

💛  - Coveralls

Pull Request Test Coverage Report for Build 13

1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 12:
0.0%

Covered Lines:
19

Relevant Lines:
19

💛  - Coveralls

Pull Request Test Coverage Report for Build 13

1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 12:
0.0%

Covered Lines:
19

Relevant Lines:
19

💛  - Coveralls

Nice test coverage, BTW!


Diagrams and Graphics Not Added

Bug Report: Diagrams and Graphics Not Added
Describe the bug
The application is missing the necessary diagrams and graphics in the relevant sections, leading to confusion for users trying to understand the concepts presented.
To Reproduce
Steps to reproduce the behavior:
Go to the section where diagrams and graphics are expected (e.g., User Interface Overview).
Click on 'View Details'.
Scroll down to the area where the graphics should be displayed.
See that there are no diagrams or graphics present.
Expected behavior
I expected to see relevant diagrams and graphics that illustrate the concepts discussed in the section, aiding in user comprehension.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):
OS: Windows 10
Browser: Chrome
Version: 93.0.4577.63
Smartphone (please complete the following information):
Additional context
Adding diagrams and graphics is essential as they provide visual representations of complex information, making it easier for users to grasp concepts quickly. They enhance user engagement and improve the overall user experience by breaking up text-heavy sections and illustrating key points.
ADD LABELS GSSOC EXT 24 AND hacktoberfest
ASSIGN ME THIS TASK

Please mention in detail the issue you are facing

Please mention in detail the issue you are facing

Here are the key points for the importance of diagrams in a face recognition project:

System Architecture Clarity: Visualizes the workflow and components (image acquisition, preprocessing, feature extraction, classification).
Data Flow Representation: Shows how input images move through layers like CNNs, transforming into outputs.
Simplifies Complex Ideas: Breaks down technical concepts (e.g., neural networks, algorithms) for easier understanding.
Enhances Collaboration: Facilitates communication among team members, ensuring everyone understands the system without code or technical jargon.
Debugging and Optimization: Identifies bottlenecks or inefficiencies by mapping the system visually.


find a issue in guid->test-case.md and fix it

There is a issue in test-case about Cross-Edge-Cloud.
I think shell scripts should be able to be executed in the terminal, but it can`t now. Because environment variable should be set with export.
So this pr is just to fix that.

/close

As even without export, it can also run normally. It looks like not need for this issue  so I close it.


Do FATPAYLOAD and DEPLOYPAYLOAD work on WIC images that uses bootimg-partition instead of sdcard_image class?

Description
I am trying to deploy some files to a /boot partition.
According to documentation, FATPAYLOAD or DEPLOYPAYLOAD should be enough to do that.
Th proble is that I don't use .sdcard format, but instead I use .wic images.
Do you have any workaround on how to extend FATPAYLOAD and DEPLOYPAYLOAD to work even with .wic images, please?
Steps to reproduce the issue:
set DEPLOYPAYLOAD to whatewertestfile (must be located in DEPLOY_DIR_IMAGE)
set IMAGE_FSTYPES to wic
Describe the results you received:
working image without any extra content deployed to /boot partition
Describe the results you expected:
working image with extra content described in DEPLOYPAYLOAD directive
Additional information you deem important (e.g. issue happens only occasionally):
wic images uses bootimg-partition source plugin which is not extended by sdcard_image-rpi.bbclass

Ok, I have managed to resolve this using https://www.yoctoproject.org/docs/2.1/ref-manual/ref-manual.html#var-IMAGE_BOOT_FILES, which worked great. So maybe only some note in the documentation is needed? Or I am blind.
Closing anyway, just for the reference.

That is indeed the case. FATPAYLOAD is only for the sdcard bbclass. wic comes with the mechanisms defined in Yocto - IMAGE_BOOT_FILES.


feat: user attributes database and lambda added

All changes look good. A full cdk deploy command deploys the HTTP API gateway with a single user route a unified route handler lambda.
Waiting for PR6 before merging to minimize conflicts.


Associate Registry Preview with .reg files

Description of the new feature / enhancement
I would like to be able to double-click .reg files and have them open with PowerToys Registry Preview tool.
Scenario when this would be used?
When opening .reg files
Supporting information
When enabled, PowerToys Registry preview (should) enhance the registry file experience by providing preview when opening the files. The tool also allows for writing the change to the registry allowing for informed consent for what is being imported and allowing user full task completion.
Today, enabling PowerToys Registry Preview doesn't really do anything, certainly that a user might expect when turning on a feature in PowerToys. It doesn't modify any default behavior, or even surface a way to "open with" a .reg file with the tool. Turning of the feature in PowerToys only serves to make the utility discoverable inside the PowerToys tray menu (which even then, is still lacking because the tool/utility is a standalone exe that "should" be able to be found with the search box or in the user's Start menu).
If a user was so inclined they could associate .reg files to the PowerToys tool, but really activating the tool should do this (that's my argument anyway). Or at the very least it should associate the Preview Tool as at least another app capable of handling .reg files, so that the user receives the always/once prompt the next time they open such a file.
I did go ahead and associate the files to the tool, but most users aren't going to so immediate discover C:\Program Files\PowerToys\WinUI3Apps\PowerToys.RegistryPreview.exe  to do so.
In summary, I think that forcing a user to find the tool (already not very discoverable) and then use the File/Open option to browse for and find a .registry file to open, is a very 1980's workflow for a document centered workflow. A user should be able to spawn the preview tool directly from a .reg file they are viewing in File Explorer, and today PowerToys doesn't make that very easy.

You can set this setting in the settings to make it the default app.
/helped


Pyth Sample EVM Price Feeds

What's the purpose of this example?

This is kind of hello-world example that is requested by alot of new devs during hackathons.

I wanted to show that one can update multiple prices in a single call. That's why we have two methods.


Cannot find module '@faire/mjml-react/utils/render'

In a new remix v2 project im getting a
Cannot find module '@faire/mjml-react/utils/render'

the file exists.
I suspect the new remix v2 esm like stuff is breaking this.
What is the best advice to resolve this?

Hi @Jarrodsz,
Unfortunately I have not used remix v2 with mjml-react, so I don't know the best way to fix this. I would need to know more about your setup to help debug the problem.
In general though, the render function in mjml-react is just one way of rendering mjml-react to html and isn't necessarily the best way for every application. It should be easy to implement your own. Under the hood it is just ReactDOMServer.renderToStaticMarkup to convert the react mjml to mjml, mjml2html to convert the mjml to html, and a minifier to minify the html (this is optional, and any minifier should work).


Limit listers to the system names for secrets.

This is to restrict somewhat the memory consumption
of the webhook, which is now linear with number of namespaces
in the system.
For #1152
/assign @dprotaso mattmoor

/close


Use prebuilt Rust binaries

http://www.elebihan.com/posts/using-rust-with-buildroot-pre-built-binaries.html
For Kybra and Azle we'll eventually want to ship some kind of cargo/rustc installation script that can just pull down the prebuilt binaries. This would remove the need for the user to install Rust on their system.

We've cut subsequent compilation times in half, and we should be able to get Azle/Kybra down to ~5 seconds on subsequent compiles once the Wasm binary limit is increased. The problem that remains is the initial compile time, which is around 3-5 minutes now on my machine. I am going to bump the priority of this issue to medium for now.


Populate epoch_stake earlier

Fixes https://github.com/input-output-hk/cardano-db-sync/issues/1406
This populates the table epoch_stake earlier. It starts k blocks after the start of the previous epoch than it does now. It also adds an epoch_stake_progress table which shows if the insertions are completed.
Description
Add your description here, if it fixes a particular issue please provide a link to the issue.
Checklist

[ ] Commit sequence broadly makes sense
[ ] Commits have useful messages
[ ] New tests are added if needed and existing tests are updated
[ ] Any changes are noted in the changelog
[ ] Code is formatted with fourmolu on version <IP_ADDRESS> (which can be run with scripts/fourmolize.sh)
[ ] Self-reviewed the diff

Migrations

[x] The pr causes a breaking change of type ~a,~ b ~or c~
[x] If there is a breaking change, the pr includes a database migration and/or a fix process for old values, so that upgrade is possible
[x] Resyncing and running the migrations provided will result in the same database semantically

A migrations is added with an adjusting procedure. On an upgrade this procedure will insert the missing entries that should have been inserted earlier.

Done here https://github.com/input-output-hk/cardano-db-sync/pull/1484


How to customer the super-pixel window size?

Hello!
I recently use lime in the image classification project. I have test the tutorial examples in the ImageNet images, it works perfectly! Thank you so much.
I would like to implement into my own project. However, unlike the images of ImageNet, the images in my database is small: 20x20 pixels. When I use Lime, it shows the whole region of the image to represent the support region for the prediction. I though this is caused by the size of super-pixel. How can I change the size of searching the super-pixel? Otherwise, it there any other problem cause this results?
Thank you so much! And expect for your response.

Yes, that sounds like segmentation is the issue.
Take a look at this and this.


chore: broker logs should show correlation ID

log lines relating to a request should show the correlation ID so that
they can be differentiated when there are concurrent requests
#177289977

I think the code looks right, but my only concern is that with CF as the platform there is no correlationId or requestIdentity header sent. So even though this will attempt to log the actual change should happen in SAPI.

The brokerapi modules guarantees that there is at least a correlation ID even when none is sent by the platform. This means that it will be possible to distinguish between requests when concurrent requests come in.


Lots of openjdk tests failed on test-aws-rhel8-x64-1, passed on other machines

Quite a few tests on test-aws-rhel8-x64-1.
Test_openjdk17_hs_extended.openjdk_x86-64_linux_testList_0 ❌ FAILURE ❌
jvm_compiler_1 => deep history 0/1 passed | possible issues
jdk_jmx_0 => deep history 0/1 passed | possible issues
jdk_net_0 => deep history 1/2 passed | possible issues
jdk_tools_0 => deep history 1/2 passed | possible issues
Post Test => deep history 2/3 passed | possible issues
jdk_jfr_1 => deep history 0/1 passed | possible issues
To make it easy for the infrastructure team to repeat and diagnose, please
answer the following questions:
Rerun jdk_jfr on same machine got the same failure.
https://ci.adoptopenjdk.net/view/work-in-progress/job/grinder_sandbox_new/359/
Rerun tests on different machines and passed.
https://ci.adoptopenjdk.net/view/work-in-progress/job/grinder_sandbox_new/361/
Rerun other tests on other machine and passed:
https://ci.adoptopenjdk.net/view/work-in-progress/job/grinder_sandbox_new/360/
https://ci.adoptopenjdk.net/view/work-in-progress/job/grinder_sandbox_new/358/
two others wait for the results:
https://ci.adoptopenjdk.net/view/work-in-progress/job/grinder_sandbox_new/362/
https://ci.adoptopenjdk.net/view/work-in-progress/job/grinder_sandbox_new/355/

Running full extended.openjdk in Grinder on x64 and s390x RHEL8 systems for comparison

AWS RHEL7 stopped after 10h (Re-running with TIME_LIMIT=100)
AWS RHEL8 completed in 8h44
Marist RHEL8 completed in 9h05

Full extended.openjdk AWS RHEL7 run completed in 10h20m so just over the 10h limit and not too much slower than the other two machines

For the two RHEL8 jobs, both failed as per #2900 and sun/security/tools/keytool/NssTest.java.NssTest and sun/security/tools/keytool/NssTest.java.NssTest
In addition the Marist machine failed the following:

javax/imageio/plugins/shared/ImageWriterCompressionTest.java.ImageWriterCompressionTest
javax/imageio/plugins/shared/ImageWriterCompressionTest.java.ImageWriterCompressionTest
tools/jlink/JLinkReproducibleTest.java.JLinkReproducibleTest
tools/jlink/JLinkReproducibleTest.java.JLinkReproducibleTest

Compression tests from jdk_imageio_0 are a file length issue so potentially a different algorithm being used. we've seen differences on zLinux compression tests in the past on certain distributions I believe.
The failure in the jlink tests from jdk_tools_0 is not immediaately obvious and would require further analysis

On the above tests it looks like only the three suites jdk_security3, jdk_imageio and jdk_tools have failed suggesting the others have been resolved or disabled

Rerunning https://ci.adoptium.net/job/Grinder/8748/console
Only 2 tests fail
java/net/HttpURLConnection/HttpURLConnectionExpectContinueTest.java.HttpURLConnectionExpectContinueTest
compiler/loopopts/superword/TestMovingLoadBeforeStore.java.TestMovingLoadBeforeStore
Rerunning both tests for 10 iterations

Rerunning compiler/loopopts/superword/TestMovingLoadBeforeStore.java on other x64 linux nodes
https://ci.adoptium.net/job/Grinder/8787/console test-ibmcloud-rhel7-x64-1
https://ci.adoptium.net/job/Grinder/8788/console test-docker-debian12-x64-1
https://ci.adoptium.net/job/Grinder/8789/console test-equinix_esxi-ubuntu2204-x64-1
https://ci.adoptium.net/job/Grinder/8790/console test-docker-ubi8-x64-2
https://ci.adoptium.net/job/Grinder/8791/console test-docker-ubuntu2204-x64-4
https://ci.adoptium.net/job/Grinder/8792/console test-ibmcloud-rhel6-x64-1

Taking a bit of a closer look to what the test is actually doing, https://github.com/adoptium/jdk17u/blob/master/test/hotspot/jtreg/compiler/loopopts/superword/TestMovingLoadBeforeStore.java,
            byte[] a_ref = a.clone();
            byte[] a_res = a.clone();

    static void verify(String name, byte[] ref, byte[] res, byte[] orig) {
        boolean fail = false;
        for (int j = 0; j < ref.length; j++) {
            if (ref[j] != res[j]) {
                System.out.println("Wrong: " + j + ":" + ref[j] + " vs " + res[j] + " from " + orig[j]);
                fail = true;
            }
        }
        if (fail) {
            throw new RuntimeException("wrong result for array " + name);
        }
    }

One array is created, then cloned twice using clone(). The test then goes through all 3 arrays and and checks if each element is the same. If not, the test will fail
Failed test outputs look like this
Wrong: 116:-57 vs -58 from -59
Wrong: 117:-20 vs -21 from -22
Wrong: 118:79 vs 78 from 77

The elements in each array are off by 1

        ref1(a_ref, a_ref, i % 2);

        test1(a_res, a_res, i % 2);

Removing these two functions allows the test to pass. This means the clone() function is running fine. The error lies with these functions

The test1 and ref1 functions look like they do the same thing (the code is exactly the same). If I replace the test1 execution with another ref1, so it looks like this
ref1(a_ref, a_ref, i % 2);
ref1(a_res, a_res, i % 2);

The test passes.
But if I replace the ref1 execution with a test1,
test1(a_ref, a_ref, i % 2);
test1(a_res, a_res, i % 2);

The test fails: (the numbers surrounding the vs should be equal, ie -65 vs -65 or -82 vs -82)
Wrong: 44:-65 vs -66 from -67
Wrong: 45:-82 vs -83 from -84
Wrong: 46:-77 vs -78 from -79

Continuing this test failure in https://github.com/adoptium/infrastructure/issues/3377 since this test failure affects more than just test-aws-rhel8-x64-1

Since we are out of a release period I am adding test-aws-rhel8-x64-1 nack into the test pool


votegame JSON config is broken

Apparently the JSON config is not ok for the currently newest crossbar. Running crossbar check on
https://github.com/crossbario/crossbarexamples/blob/master/work/votegame/config.json
produces:
Error: encountered unknown attribute 'processes' in top-level configuration

fixed


Building a MEX-wrapper using CUDPP in Matlab

There is no bug about it.

I wish I could use a MEX-Wrapper, wich allows me, to use CUDPP in M-Code.
This (CUDA-)MEX-file could be compiled at first use and allow after people
like me, who don't know much C, to use GPGPU very easily in Matlab.

On my site sort seems very interesing, in case it gaves back indices (like
it is need for sortrows).

At the moment there a two different Toolboxes available for using CUDA in
Matlab: Accelereyes' Jacket and GPUmat (from gp-you.org). Booth of them
dont allow a sortrows, sort from Jacket is very slow on the other hand.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 2 Sep 2009 at 2:04

Original comment by<EMAIL_ADDRESS>on 2 Sep 2009 at 9:35

Added labels: Type-Enhancement, Priority-Low, OpSys-All, Component-Scripts, Usability
Removed labels: Type-Defect, Priority-Medium

We don't have any plans to do the work for this -- perhaps the Mathworks or
Accelereyes folks can add algorithms like CUDPP to their products (with a
wrapper on CUDPP or Thrust, for example).

Original comment by<EMAIL_ADDRESS>on 30 Jun 2011 at 4:51

Changed state: WontFix


x64: Implement bmask

👋 Hey,
This PR implements bmask for x86.
It's built on top of #5129, so it might  not be worth reviewing just yet (or just look at the last commit individually).
This also enables the fuzzer for x86, and it has been running for about an hour without any issues.
Fixes #5106

I think this can be rebased now!


openHAB URL should work properly if it has a trailing slash

When configuring for openHAB, the required parameter is the URL of the openHAB instance.
I initially entered it like this: http://<IP_ADDRESS>:8080/
This lead to calls to openHAB with a double slash like this: http://<IP_ADDRESS>:8080//rest/voice/interpreters
Sadly, this always give a 302 HTTP response and does not trigger anything in openHAB
As soon as I have modified the URL not to include the trailing slash, it started giving me the 200 OK expected answer.
I understand that this is likely to be an issue in openHAB itself, but it would be nice if willow was to simply ignore the trailing slash if it is present when concatenating the rest of the URL.

Yes, technically speaking this is an issue in openHAB. Per the HTTP standards and general best practice it's perfectly valid to have multiple slashes. We have a burgeoning relationship with members of the openHAB team and we can raise this with them. However, I'm not a fan of requiring users to upgrade something as significant as openHAB or HA for Willow support.
That said, we take ownership of these little things and the least we can do is update the documentation.

Yes, thanks, updating the documentation is fine by me


Cut release 0.11.1

@juliusv This is based on https://github.com/prometheus/prometheus/pull/563, only the last commit is 'real'.

:+1:


Note on WHfB and Step-up Authentication

This comment is based on details provided by Libby Brown in the below Teams thread:
https://teams.microsoft.com/l/message/19:bf81f7ed6bea41168273c095b6f1b51b@thread.skype/1658406678904?tenantId=72f988bf-86f1-41af-91ab-2d7cd011db47&groupId=3e1eb424-0cb7-4665-b384-4fbfd3cd0f83&parentMessageId=1658406678904&teamName=Identity PM Collaboration&channelName=Credential Management (SSPR%2C MFA%2C Passwordless)&createdTime=1658406678904

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

@Justinha
Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


New package: BridgeLandmarks v0.2.0

Registering package: BridgeLandmarks
Repository: https://github.com/mschauer/BridgeLandmarks.jl
Created by: @mschauer
Version: v0.2.0
Commit: d15aff881f925a44faefa948d010e5ca46f75926
Reviewed by: @mschauer
Reference: https://github.com/mschauer/BridgeLandmarks.jl/issues/19

[noblock]
Thank you for your help!


For now i'll just get error, please help

For now i'll just get error, please help

404 - Error not found.

@fengyuanchen I laughed too hard about your comment... xD


Fixed buttons on Android not showing Ripple effect

Fixes #655

Switched from custom Ripple solution to use RectButton component from react-native-gesture-handler for performance and accessibility.

@amazingphilippe - any comments before this is merged?

I tested it works well :)

Will merge and we can adjust in another PR as need be.


Add labels to telescope buffer keymap

Hello,
Right now when you list the keys in the telescope buffer (with "C_/"), some of the keys are mapped to anonymous functions, which makes it hard to see what those keys do. This PR fixes that.

This is already covered by https://github.com/LazyVim/LazyVim/pull/1294#issue-1838004376, my mistake.


Not able to disableGlobalFilter for a column

Adding a disableGlobalFilter option to a Column object does not exclude it from global filtering.
Use an official filter example to reproduce the issue. Adding disableGlobalFilter: true to a column object (line 358 or any other column) does nothing, the column still is used for filtering.
          {
            Header: 'Age',
            accessor: 'age',
            Filter: SliderColumnFilter,
            filter: 'equals',
            disableGlobalFilter: true,
          },

First found similar behaviour trying to add autoResetGlobalFilter: false to a table. It looked like ignored. Since I am new in React, I thought I was doing something wrong. But with this option it seems obvious. Still, maybe I am doing something wrong?

Sorry, everything works. Had to update to latest.


Update README.md

German

It's not necessary to create a new PR. Please continue with the first one 👉 https://github.com/68publishers/cookie-consent/pull/44
Closing


Detect back button used

Is it possible to know in page:done that the page content is being loaded because the user clicked the back button?
I can't rely on checking whether the content is being loaded using template or partialbecause the user could have gone through a few pages loads and clicked back once, so going back from one partial to another. Does wiselinks have this information available somehow?  Or is this possible to be gathered from thehistory` stack?

Nevermind this kind of thing seems to work:
var userInteraction = false;

$(document).on('click', 'a[data-push], a[data-replace]', function (e) {
  userInteraction = true;
});
// do equivalent for onsubmit of wiselinks forms as well

$(document).on('page:done', function (event, $target, status, url, data) {
// userInteraction = true if link clicked / form submitted
// userInteraction = false if link not clicked / form not submitted
});

$(document).on('page:always', function (event, xhr, settings) {
  userInteraction = false;
});


Update presentations.md

Added a new video listing

@packtpartner it looks like this content is gated, and none of the other presentations on the list are. Do you have a non-gated version to link to instead?

No we do not have a non-gated version of this. But we Packt have the copyrights to this video.

Hi Judith,
Would it be possible to merger this PR?

I'm checking with a committer now 🙂

Alright will wait for an update about it.

Merged in https://github.com/apache/mesos/commit/f79a09a4e2caeaa215a3acc52b85ca50362ad171, closing.


Parameter that prints video info to json only

Checklist

[x] I'm reporting a feature request
[x] I've verified that I'm running youtube-dl version 2019.11.28
[x] I've searched the bugtracker for similar feature requests including closed ones

Description

So basically, I've been facing issues with downloading the direct video link, the library hangs and loses connection and its probably my server's firewalls. Proxies work great for this, however proxy availability is limited. --write-info-json continues the download but prints valid direct-urls in the json file, even for licensed content. I would love to access those without actually downloading the video.
Perhaps a new parameter, something like --dont-download so that once it prints the json, it stops.

Read FAQ?
--skip-download                  Do not download the video

Huh, odd. Thank you.


Store Stripe webhook data

We should store data from Stripe webhook so we know, for example, exactly when a CC payment was made and what fees were charged. I already created a model in backend/models/external_payment.js - we just need to populate it

I've got this coded. Just need master checkout to work to make sure it's correct.

I'm closing this, since the PR is up.


Labels not working

Labels don't have a "for" attribute when generated, so they aren't linked to any element.
This means that when the user clicks a label, input isn't focused.
Same goes with accessibility, screen readers don't know what label belongs to what element.

This is resolved in the latest release.  Closing this issue.


Pure GraphQL Peloton update script

Uses GraphQL for both queries and mutations. This allows the inclusion of Discussions, and greatly reduced runtime.
Uses sgqlc for a Pythonic representation of GraphQL.
This is a big change. I recommend reading through the main() function of update_project.py to get a feel for the workflow, before diving into how the individual queries/mutations are constructed.
The Peloton in its current guise has been successfully populated by this script on several occasions, both locally and via GitHub Actions.
GitHub actions examples:

Standard run: trexfeathers/SciTools.github<PHONE_NUMBER>
5min looping run, via workflow dispatch: trexfeathers/SciTools.github<PHONE_NUMBER>

Closes #14
Closes #12
Closes #11
Closes #10
Closes #9

@trexfeathers Awesome! Thanks for pushing this forwards, this'll make a huge difference to the peloton 🚀 🥳 🎉


setIn() for numerical indexes fails on empty lists

I'd like to better understand the intended behavior of setIn() regarding integers used in the set path.
This is from @axefrog on #55

When the path does not match what exists already, what exists should be replaced with the best collection type for the path element type (a string would suggest a map key, whereas a number would suggest a list index, unless there is already a map at that location).

Consider this example:
setIn(
  ['a', 0],         // at: input.a[0]
  true,             // set: true
  from({a: []})     // for: {a:[]} ❌
);
// expected result: from({ a: [true] })

This use of a numerical index throws the following error:
     Error: Index -1 is out of range
      at Object.setValueAtOrdinal
(node_modules/@collectable/list/lib/commonjs/internals/values.js:15:15)

It looks like this is because the intended path violates a safeguards against setting values at indexes beyond the length of the list. For example:
setIn(
  ['a', 9],         // at: input.a[9]
  true,             // set: true
  from({a: []})     // for: { a: [] } ❌
);
// what is this supposed to do? what goes in the first 8 indexes?
// undefined? just crash

This has several implications:

You cannot set the last value of a list (that seems problematic). Solution: change the safeguard logic.
You cannot have arrays in the path of a deep setIn(), like so:

setIn(
  ['a', 0, 'b', 0, 'c'], // at: input.a[0].b[0].c
  true,                  // set: true
  from({a: []})          // for: {a:[]}
);
// expected: from({ a: [ { b: [ { c: true} ]}]}) ❌

I'm not sure if [2] is confusing or not. I'm not sure whether people would want to do that or not. This comment from @axefrog suggests that something like that is desirable though. Maybe all that it means is that existing lists and indexes along the path should be settable, but not dynamically filled in where missing.
setIn() seems to work just as expected for existing lists and indexes:
setIn(
  ['a', 0, 'b', 4, 'c'],              // at: input.a[0].b[4].c
  true,                               // set: true
  from({ a: [ { b: [ 0, 1, 2, 3 ]}]}) // for: { a: [ { b: [ 0, 1, 2, 3 ]}]}
);
// expected: from({ a: [ { b: [ 0, 1, 2, 3, { c: true} ]}]}) 👌🏻

Solution: change the safeguard logic.

I'd agree with that. The safeguard logic should allow index n to pass validation and defer its resolution to the normal code path for appending values.

I'm guessing that in
function verifyIndex(size: number, index: number): number {
  index = normalizeIndex(size, index);
  return index === size ? -1 : index;
}

What we really want is index > size ? -1 : index

Hey, sorry about the slow reply. Disappeared off my notifications list and then forgot it was here. Feel free to bump an issue if I'm ever slow to reply, or jump on Gitter and ping me (@axefrog). Same on Hangouts if you want to ping me directly.
Sounds about right, I guess. Only hesitation is that if a person has, for example, a list of length 3, then tries to set the value of index 7, the change you suggest would suggest insertion at index [3], which may not be what they expected, and feels wrong to me, somehow. I think insertion at the last index (list.size) is ok, but beyond that my instinct is that I'd expect an error, or to be forced to grow the list. I'm open to your thoughts on the topic.

@axefrog no worries.
Maybe I'm still confused about verifyIndex(). Do these examples look right? They purport to show that index > size ? -1 : index allows setting the last index, but does not allow setting an index beyond the size of the list. So trying to set [7] for [1,2,3] would fail. Trying to set [3] at [1,2,3] would pass.
// example of failed verification
// index: 7, size: 3 --- can't set an index beyond the end of the list
// is this the case you're worried about?

index > size ? -1 : index
7     > 3    ? -1 : 7
true         ? -1 : 7
-1

// example of accepted verification
// index: 3, size: 3 --- you can set the last member though
// i think this is desirable?

index > size ? -1 : index
3     > 3    ? -1 : 3
false        ? -1 : 3
3

I guess it should be index >= size actually because lists are 0 indexed. but still should not have the problem you described

It's been a little while since I wrote that code, so I just dove back in to try and remember exactly what I had in mind when I write it. To recamp, the code is:
export function verifyIndex(size: number, index: number): number {
  index = normalizeIndex(size, index);
  return index === size ? -1 : index;
}

export function normalizeIndex(size: number, index: number): number {
  return max(-1, min(size, index < 0 ? size + index : index));
}

There are a few things to keep in mind here:

If the index argument is negative, it's a backtracking offset from the end of the list
If -1 is returned, it indicates that the index was invalid
normalizeIndex() computes a new index argument to transform a backtracking index into an actual index, and to ensure that if the argument is ultimately out of bounds, -1 is returned. It caters for both a backtracking index that would push the index too far to the left of 0, and for clamping excessive upper indexes back to the list size, which retains the property that the index is out of bounds, while ensuring that it does not exceed the list size.
verifyIndex() just adds the extra step of ensuring that if the index is normalized to match the list size (exclusive upper bound), then it is converted to -1, thus verifying that the specified index is actual an existing location in the list.

There are two functions there because I found several scenarios where I needed to reuse the code for incorporating a backtracking index calculation into the normalized index but where it was still permissable to return an exclusive upper bound equal to the list size.
I would suggest changing setValueAtOrdinal() to use normalizeIndex() instead, and then if the normalized index equals the list size, divert further operations to appendValues() instead.

Is this still an issue? If you've moved on, I'll go ahead and close this.


HAWQ-1431: Do not use StatsAccessor when column specified in SELECT c…

…lause.

@sansanichfb If the code change has been in main, please close this PR. Thanks.


volume button in phone

Can you add volume control from mobile app by volume button?

Thanks for the suggestion! If you rotate the screen it probably will be there already.
It's only missing in portrait mode on my phone

I think he means to actually use the volume controls (the physical buttons) to control volume of the active player in MA

I think he means to actually use the volume controls (the physical buttons) to control volume of the active player in MA

Yes, physical buttons. It's natural I think

This request is a duplicate of the linked discussion #378


Remove dotnet5.0 lambda runtime to avoid confusion

Description of changes:
The dotnet5.0 runtime never existed.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Oddly, the web page also includes dotnet7 which is also isn't mentioned in this markdown, that also isn't a lambda runtime, how can I update this?


sbt 1.0.3 can't overwrite SNAPSHOT artifacts on sonatype snapshot repository

steps
Publish configuration:
publishTo in ThisBuild := {
  if (version.value.trim.endsWith("SNAPSHOT")) Some(Opts.resolver.sonatypeSnapshots)
  else Some(Opts.resolver.sonatypeStaging)
}

Plugin configuration:
addSbtPlugin("com.jsuereth" % "sbt-pgp" % "1.1.0")

and run sbt publishSigned twice.
problem
First publishing is succeeded, but second is failed with the following error:
[error] java.io.IOException: destination file exists and overwrite == false
[error] 	at org.apache.ivy.plugins.repository.url.URLRepository.put(URLRepository.java:75)
[error] 	at sbt.internal.librarymanagement.ConvertResolver$LocalIfFileRepo.put(ConvertResolver.scala:365)
[error] 	at org.apache.ivy.plugins.repository.AbstractRepository.put(AbstractRepository.java:130)
[error] 	at sbt.internal.librarymanagement.ConvertResolver$ChecksumFriendlyURLResolver.put(ConvertResolver.scala:117)
[error] 	at sbt.internal.librarymanagement.ConvertResolver$ChecksumFriendlyURLResolver.put$(ConvertResolver.scala:104)
[error] 	at sbt.internal.librarymanagement.ConvertResolver$$anonfun$defaultConvert$lzycompute$1$PluginCapableResolver$1.put(ConvertResolver.scala:164)
[error] 	at org.apache.ivy.plugins.resolver.RepositoryResolver.publish(RepositoryResolver.java:216)
[error] 	at sbt.internal.librarymanagement.IvyActions$.$anonfun$publish$5(IvyActions.scala:492)
[error] 	at sbt.internal.librarymanagement.IvyActions$.$anonfun$publish$5$adapted(IvyActions.scala:491)
[error] 	at scala.collection.TraversableLike$WithFilter.$anonfun$foreach$1(TraversableLike.scala:789)
[error] 	at scala.collection.Iterator.foreach(Iterator.scala:929)
[error] 	at scala.collection.Iterator.foreach$(Iterator.scala:929)
[error] 	at scala.collection.AbstractIterator.foreach(Iterator.scala:1417)
[error] 	at scala.collection.IterableLike.foreach(IterableLike.scala:71)
[error] 	at scala.collection.IterableLike.foreach$(IterableLike.scala:70)
[error] 	at scala.collection.AbstractIterable.foreach(Iterable.scala:54)
[error] 	at scala.collection.TraversableLike$WithFilter.foreach(TraversableLike.scala:788)
[error] 	at sbt.internal.librarymanagement.IvyActions$.publish(IvyActions.scala:491)
[error] 	at sbt.internal.librarymanagement.IvyActions$.$anonfun$publish$3(IvyActions.scala:141)
[error] 	at scala.runtime.java8.JFunction0$mcV$sp.apply(JFunction0$mcV$sp.java:12)
[error] 	at sbt.internal.librarymanagement.IvyActions$.withChecksums(IvyActions.scala:154)
[error] 	at sbt.internal.librarymanagement.IvyActions$.withChecksums(IvyActions.scala:148)
[error] 	at sbt.internal.librarymanagement.IvyActions$.$anonfun$publish$1(IvyActions.scala:141)
[error] 	at sbt.internal.librarymanagement.IvyActions$.$anonfun$publish$1$adapted(IvyActions.scala:131)
[error] 	at sbt.internal.librarymanagement.IvySbt$Module.$anonfun$withModule$1(Ivy.scala:229)
[error] 	at sbt.internal.librarymanagement.IvySbt.$anonfun$withIvy$1(Ivy.scala:190)
[error] 	at sbt.internal.librarymanagement.IvySbt.sbt$internal$librarymanagement$IvySbt$$action$1(Ivy.scala:70)
[error] 	at sbt.internal.librarymanagement.IvySbt$$anon$3.call(Ivy.scala:77)
[error] 	at xsbt.boot.Locks$GlobalLock.withChannel$1(Locks.scala:93)
[error] 	at xsbt.boot.Locks$GlobalLock.xsbt$boot$Locks$GlobalLock$$withChannelRetries$1(Locks.scala:78)
[error] 	at xsbt.boot.Locks$GlobalLock$$anonfun$withFileLock$1.apply(Locks.scala:97)
[error] 	at xsbt.boot.Using$.withResource(Using.scala:10)
[error] 	at xsbt.boot.Using$.apply(Using.scala:9)
[error] 	at xsbt.boot.Locks$GlobalLock.ignoringDeadlockAvoided(Locks.scala:58)
[error] 	at xsbt.boot.Locks$GlobalLock.withLock(Locks.scala:48)
[error] 	at xsbt.boot.Locks$.apply0(Locks.scala:31)
[error] 	at xsbt.boot.Locks$.apply(Locks.scala:28)
[error] 	at sbt.internal.librarymanagement.IvySbt.withDefaultLogger(Ivy.scala:77)
[error] 	at sbt.internal.librarymanagement.IvySbt.withIvy(Ivy.scala:185)
[error] 	at sbt.internal.librarymanagement.IvySbt.withIvy(Ivy.scala:182)
[error] 	at sbt.internal.librarymanagement.IvySbt$Module.withModule(Ivy.scala:228)
[error] 	at sbt.internal.librarymanagement.IvyActions$.publish(IvyActions.scala:131)
[error] 	at sbt.Classpaths$.$anonfun$publishTask$4(Defaults.scala:2217)
[error] 	at sbt.Classpaths$.$anonfun$publishTask$4$adapted(Defaults.scala:2215)
[error] 	at scala.Function1.$anonfun$compose$1(Function1.scala:44)
[error] 	at sbt.internal.util.$tilde$greater.$anonfun$$u2219$1(TypeFunctions.scala:42)
[error] 	at sbt.std.Transform$$anon$4.work(System.scala:64)
[error] 	at sbt.Execute.$anonfun$submit$2(Execute.scala:257)
[error] 	at sbt.internal.util.ErrorHandling$.wideConvert(ErrorHandling.scala:16)
[error] 	at sbt.Execute.work(Execute.scala:266)
[error] 	at sbt.Execute.$anonfun$submit$1(Execute.scala:257)
[error] 	at sbt.ConcurrentRestrictions$$anon$4.$anonfun$submitValid$1(ConcurrentRestrictions.scala:167)
[error] 	at sbt.CompletionService$$anon$2.call(CompletionService.scala:32)
[error] 	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
[error] 	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
[error] 	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
[error] 	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
[error] 	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
[error] 	at java.lang.Thread.run(Thread.java:745)

expectation
SNAPSHOT artifacts should be overwritten.
notes
sbt version: 1.0.3
Some users mentioned about this problem in #3431.

Not a bug.
It's just a default behaviour which is very easy to change
Full example: https://github.com/pshirshov/izumi-r2/blob/develop/sbt-izumi/src/main/scala/org/bitbucket/pshirshov/izumi/sbt/PublishingPlugin.scala

It is definitely a bug, snapshots should be overwritten by default at sonatype just like they do locally. That's why snapshots exists after all. As for your example I wouldn't say that it's easy, thats looks like a hack.

@dwijnand we're running into this too, any idea how we can fix this?  we would like to publish snapshots to maven central, and this seems to be a pretty big impediment.

Look at my message above, there is a link to a plugin which is fixing the issue.

@pshirshov hmm, OK, I'll try it out.  thanks!

@mosesn did @pshirshov's plugin work for you? I made those changes to my build but it made no difference for me.

Plugin worked for me with a minor modification to not depend on other plugins in @pshirshov repo. I would rather have this be the default behavior for publishing snapshots though.

Another workaround is to remove the SNAPSHOT packages manually from Sonatype. Click here, then log in, find the package(s) in the  "Snapshots" pane, right-click and select "Delete".

Maybe a little update on this. I found out that in sbt-pgp the overwrite flag is hard coded. This why I did a PR to this plugin:
https://github.com/sbt/sbt-pgp/pull/121

Still happening on sbt 1.1.1, easiest workaround I've found is adding these lines to build.sbt:
// FIXME: remove setting of overwrite flag when the following issue will be fixed: https://github.com/sbt/sbt/issues/3725
publishConfiguration := publishConfiguration.value.withOverwrite(isSnapshot.value)
com.typesafe.sbt.pgp.PgpKeys.publishSignedConfiguration := com.typesafe.sbt.pgp.PgpKeys.publishSignedConfiguration.value.withOverwrite(isSnapshot.value)
publishLocalConfiguration := publishLocalConfiguration.value.withOverwrite(isSnapshot.value)
com.typesafe.sbt.pgp.PgpKeys.publishLocalSignedConfiguration := com.typesafe.sbt.pgp.PgpKeys.publishLocalSignedConfiguration.value.withOverwrite(isSnapshot.value)

Credits goes to https://github.com/plokhotnyuk/jsoniter-scala

Thanks @maqdev, dropping those lines in my ~/.sbt/1.0/global.sbt seems like the best work-around until @ffinfo signs the typelevel CLA and we get a new sbt-pgp release 😆

This is finally fixed due to sbt-pgp 1.1.1 is released!


disable-nls breaks build

When building with --disable-nls and no gettext library:
depbase=`echo src/commons.o | sed 's|[^/]*$|.deps/&|;s|\.o$||'`; cc -DLOCALEDIR=\"/usr/local/share/locale\" -DHAVE_CONFIG_H -I. -I./src     -O2 -DSYSCONFDIR=\"/usr/local/etc\" -Wall  -Wextra -Wnested-externs -Wformat=2 -g  -Wmissing-prototypes -Wstrict-prototypes  -Wmissing-declarations -Wwrite-strings -Wshadow  -Wpointer-arith -Wsign-compare  -Wbad-function-cast -Wcast-align  -Wdeclaration-after-statement -Wshadow  -Wold-style-definition -O2 -pipe  -I/usr/include -I/usr/local/include -fstack-protector-strong -fno-strict-aliasing  -pthread -MT src/commons.o -MD -MP -MF $depbase.Tpo -c -o src/commons.o src/commons.c && mv -f $depbase.Tpo $depbase.Po
In file included from src/commons.c:44:
./src/labels.h:33:10: fatal error: 'libintl.h' file not found
#include <libintl.h>
         ^~~~~~~~~~~
1 error generated.
*** Error code 1

Stop.
make[3]: stopped in /wrkdirs/usr/ports/sysutils/goaccess/work/goaccess-1.7.1

src/goaccess.c calls bindtextdomain() and textdomain() without guarding ENABLE_NLS. Similarly, src/labels.h includes libintl.h and defines some gettext macros without ENABLE_NLS guards.
I patched this up in FreeBSD a while ago but I guess I never upstreamed it. I'm not convinced that the patch to labels.h is correct (it just creates some stubs) so I wanted to check with you. If it looks right I'm happy to create a PR for it.
Patches here: https://github.com/freebsd/freebsd-ports/tree/main/sysutils/goaccess/files

Yeah, that looks good to me. Please feel free to create the PR. Thanks!

Thank you. To ship these changes, I plan to release v1.7.2 over the weekend.


Ignore inscriptions on a given sat after the first

Fixes #1128.

Must also modify get_inscription_by_id to not return skipped inscriptions


Sorted value column in Addresses

Unless there's a good reason to prevent sorting on the Value column, I think it's useful to be able to sort the wallet's addresses by value. 🤞

Addresses should be sorted by index IMO, and I can't see a good reason to do otherwise. Also, I haven't tested this fully, but I believe it has the potential to introduce subtle bugs in transaction loading. You'll also note that once you sort by value, you can't "undo" that sort - there is no way to return to the original "by index" sorting (I have no desire to introduce an index column).

Thanks for the response.  I'm looking to view the addresses in my wallet by value so I can see the ones that have "a lot" of BTC, all the way down to those that barely have any.  If it doesn't make sense to do it on this screen (due to potential side-effects), is there another place in Sparrow to get this information?

Yes, the UTXOs tab.

The problem is that I have several addresses in my wallet with multiple outputs.  So while the UTXO screen allows me to sort by the Value column, it's at the UTXO level.  What I'm hoping to see is a sorted list of addresses with their total value.

The problem is that I have several addresses in my wallet with multiple outputs.

This is not ideal from a privacy perspective - to the extent that Sparrow is opinionated, it is opinionated in trying to dissuade from poor privacy practices.


Which WA API?

Hi, thank you for your work on MathSensei. Could you clarify which of the WolframAlpha APIs was used for obtaining the results in the paper? Is it the LLM API? Many thanks

We used the Free API (2000 non-commercial free calls per month) : https://products.wolframalpha.com/api


feat: panic hook

Closes #3402
A panic hook was added which adds the following info to stderr:

reth version
os version
link to bug report page
cmd entered

A handler for panics was added which turns panics to errors.
TODO: add reth db version to panic hook

Output from Ubuntu22.
PR panic output

...

Current panic output from main branch

PR panic output from test

Current main branch panic output from test


Fixed debugging on device crash

Fixed a issue that running on debugging iPad device crashes app.
Crash log:
dyld: Library not loaded: @rpath/FLAnimatedImage.framework/FLAnimatedImage
  Referenced from: /var/mobile/Containers/Bundle/Application/CD971193-C95F-401A-BA4A-8CBC422CCAEE/FLAnimatedImageDemo.app/FLAnimatedImageDemo
  Reason: image not found

Awesome, thanks!


chore(main): add release-please

So maybe we should add some link in the CONTRIBUTION.md to docs about those prefixes, like chore:) I needed to google it, to learn the meaning...  found this one: https://ec.europa.eu/component-library/v1.15.0/eu/docs/conventions/git/#:~:text=chore%3A Changes to the build,libraries such as documentation generation

@amabelleS feel free to open an issue about it.


Unable to configure Enhanced Block Entities via NeoForge's "Mods" menu

Mod Name
Enhanced Block Entities
Mod Homepage
https://github.com/FoundationGames/EnhancedBlockEntities
Minecraft version
1.21.1
Describe the bug
Unable to configure Enhanced Block Entities via NeoForge's "Mods" menu
Steps to reproduce
No response
Logs
https://mclo.gs/8nHXdJM
Additional context
latest.log

Do you have Connector Extras installed?


made render_images more robust to channel/cmap choices

Yeah, overhauled the entire render_images function because there were a bunch of edge cases for the permutations of the channel, palette and cmap options. @MobiTobi found a few bugs

I'll add tests to lock in this behaviour later and will then merge 👌

Perfect thanks. One last question, how do you use images with tests? Do you assert that the plot needs to look like the images you put in the repo? If so, great idea, I need to do it as well when I transform data!

Codecov Report

Merging #118 (4c44c82) into main (bde6827) will increase coverage by 0.71%.
The diff coverage is 91.89%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #118      +/-   ##
==========================================
+ Coverage   75.84%   76.56%   +0.71%
==========================================
  Files           9       10       +1
  Lines        1151     1199      +48
==========================================
+ Hits          873      918      +45
- Misses        278      281       +3

Impacted Files
Coverage Δ

src/spatialdata_plot/pl/basic.py
83.69% <80.00%> (-0.20%)
:arrow_down:

src/spatialdata_plot/pl/render.py
88.20% <92.00%> (+0.64%)
:arrow_up:

src/spatialdata_plot/_logging.py
92.85% <92.85%> (ø)

src/spatialdata_plot/pl/utils.py
62.65% <100.00%> (+0.31%)
:arrow_up:

Yeah exactly, it's taken from Squidpy where @giovp uses that a lot. Super nice for end-to-end tests in plotting.

Looks good to me.


BitwiseNot issue

I have an enum like this:
    [Flags]
    public enum AxesMask
    {
        None = 0,
        X = 1,
        Y = 2,
        Z = 4,
        A = 8,

        XY = X | Y,
        XYZ = X | Y | Z,
        All = X | Y | Z | A,
    }

if I do:
AxesMask axes = AxesMask.Y | AxesMask.A;
var notResult = EnumUtil<AxesMask>BitwiseNot(axes);

I would expect "notResult" to be AxesMask.X | AxesMask.Z;
but instead I get -11 ???
Is that a known  issue?
Is there a work around?

@clemexjeap That is what bitwise not is supposed to do. You should get the equivalent of ~(1|8).

That would be nice. The idea is just to "NOT" all the flags

Sounds good to me.
One other nice to have:
// Determine if multiple flags are set
bool isCombined = EnumUtil.IsCombinedFlag(YourEnum.Foo | YourEnum.Bar, YourEnum.Bar);

@clemexjeap I do not understand how that would be helpful. Did you actually mean to pass two arguments to IsCombinedFlag() or did you only mean to pass one argument?
Please open a new issue with a description of what that is and what utility it has. We can discuss it further and, if it seems useful to me, I could prototype an implementation.
P.S., as it is, I am not sure if this library is even going to accept my PR or if a new release will be published to NuGet for my PR. This library has two purposes:

Provide a type-safe wrapper for enum.
Provide high performance enum manipulation.

On the new dotnet core framework, the method used by this library to attain high performance actually results in lower performance. So the library author has less motivation to continue updating it.
I personally think that having the type-safe wrapper for enum manipulation is useful, even if high performance cannot be achieved. But, yeah, this just isn’t as active anymore.
@arogozine Do you have an idea of if you can check these things out someday? ;-)

Thanks for your comment @binki.
For the IsCombinedFlag() thing. I would like to know if multiple flags (bits) are set.
ex/
EnumUtil.IsCombinedFlag(YourEnum.Foo | YourEnum.Bar) == yes
EnumUtil.IsCombinedFlag(YourEnum.Foo) == no
Sorry my previous example was wrong


Preact with Kea?

Hi folks,
There is already a framework(ish) thingy based on redux called Kea and it simplifies connecting state with HoC(s). Is there any examples or gists you have about integrating those?
Cheers
Baris

Hi @hwclass,
I 've just tested there kae-example with preact and it works just fine.
All you need todo is:

clone there example run npm i --save-dev
npm i --save-dev preact preact-compat
add'react': 'preact-compat',
'react-dom': 'preact-compat'

to the alias-section of their webpack.config.js.

Now everything works just like when using react.

Thanks @Kanaye, I'll look at it ;)

Hi @hwclass.
To start a new project with preact and keajs you just need keajs (and it 's dependencies excluding react), preact and preact-compat.
Imagine that we have a new project with webpack, babel and all the other stuff you need already installed as dev dependencies.
Run npm install --save preact preact-compat kea react-redux redux reselect prop-types to add preact, preact-compat, kea and it 's dependencies.
npm will warn you that react is missing but because we replace react with preact-compat we can just ignore this warning.
In the webpack configuration we will need an alias section and need to add
'react': 'preact-compat',
'react-dom': 'preact-compat'

to it.
This will tell webpack to use preact-compat whereever kea or react-redux will import (or require) react.
Know you can use kae with preact just as you woul use it with react.
I haven't used kea before but I tried this config with there counter example.
You can find this test here and the repository with everything I did here

That's awesome, seems like you already found a solution 👍


[BUG] useRegister() deletes register form data values other than email and password

Describe the bug
I was trying to customize RegisterPage to add Name and Company fields to the registration form. I confirmed the parameters passed to useRegister containing name and company fields but when the request was on the fly, only email and password are submitted. Is it impossible to add additional fields in Register Page?
Steps To Reproduce
/RegisterPage.tsx/
import { useRegister } from "@refinedev/core";
type RegisterVariables = {
email: string;
password: string;
name: string;
};
export const RegisterPage = () => {
const { mutate: register } = useRegister();
const onSubmit = (e: React.FormEvent) => {
e.preventDefault();
const values = {
    name: e.currentTarget.name.value,
  email: e.currentTarget.email.value,
  password: e.currentTarget.password.value,
};
console.log(values);
register(values);

};
return (

Name

Email

Password

Submit

);
};
Expected behavior
in the console, the values passed to register() function are printed as {name: '...', email: '..', password:'...'}. But when you inspect in the network window using Google Chrome, auth/signup request payloads are something like {email: '...', password: '...'} omitting name on the fly.
Screenshot
No response
Desktop
No response
Mobile
No response
Additional Context
No response

Oh, I've found the reason. I had to change the authProvider.ts file. I posted email & password in axios request. That was not a bug in refine but my fault.


MultiUser: As soon as another user (my wife) uses the car, data is not updated anymore

As soon as another user (my wife) uses the car, provided data is not updated anymore (Mileage, Battery Level, etc)
Expected behavior
Data should be updated, independend of the user
Is it possible to enter multipe Username/Password, so that "WeConnect-python" will always provide the data no matter which user is active in the car?

I'm working on a new version of the library (https://github.com/tillsteinbach/CarConnectivity) that will support also skoda and seat/cupra, there it will be possible to add several accounts. For your use-case I would need to think about a way to merge the data when the same VIN is in multiple accounts. Sounds doable, but will probably cause some more headache.


Jump search in c++

Aim
To write a program that searches an element in a given array using jump search
Details

What features will your script have
It has a jump search function that searches an element in a given array and the main code

Programming language

[ ] C
[x] C++
[ ] Java
[ ] Python

I want to also work on this issue in java language


Namespace selection dropdown enhancement

What would you like to be added:
Regarding the dropdown that allows selection of a namespace : once you select a namespace its marked as "checked", but if you want to unmark it you have to scroll all the way to find it and click it again. Or I have to type and search for it again before clicking and unmarking it.
It would be easier if the selected namespaces are moved to the top of the dropdown so its easy to see which ones are selected and also easy to uncheck them.
Why is this needed:
Improve usability in cases where there are a large number of namespaces to deal with.
Environment you are Lens application on:

Kubernetes distribution: [EKS, AKS, GKE, Rancher, ...] EKS
Desktop OS: macOS (Catalina)

Another enhancement would be to avoid reset the scroll of the namespace dropdown so is more easy to select multiple namespace adiacent.
Currently you need to scroll down all the time.

@smark91 The issue you described currently fixed and will be available in version 4.1.0.

Both the mention of @smark91 and the request from @kavishdahekar are not yet addressed.

Thanks for pointing it out @matglas.
@aleksfront I'm currently on 4.2.5 and I still see the issues I mentioned initially.


Set minimum buffer size of BytesPool to maxChunk size

This PR fixes #1580
It fixes the immediate issue, but it could be worthy to discuss if pool.BytesPool is necessary at all or could be replaced by a simpler direct usage of sync.Pool

Match store bytes pool size to size of chunks used by chunkByteReader.

Changes

extract constant nested in method to package level
change pool.BytesPool construction to use chunk size as smallest bucket size

Verification
The store does no longer exhaust its byte pool size on larger buckets

More sizes we have it defeats the purpose of sync pool indeed. But for queries that uses lot's of single chunks (e.g especially for downsampled data) having smaller pools might make sense. Any chance for some benchmarks before & after? (: In terms of memory usage for the same requests?

The memory usage is only positively affected as described in the linked issue.
The main issue is that the individual chunk files are fetched in parallel in 16k byte parts when the chunk is read.
So this one:

would be fetched in ~ 1,244 concurrent requests - each downloader requests 16k from the byte pool which caused in the previous version to grab 248.75 MB from the byte pool (because the 16k request was allocated the smallest buffer size: 200k), with the changed version it is only 24,875 MB (smallest buffer: 20k)
The issue appears on primarily when quering as far as I can tell, because the byte chunks are allocated in https://github.com/thanos-io/thanos/blob/4c7ecd3f86c5ce1f59f54f03a6dceec82d21b77e/pkg/store/bucket.go#L1272
and only returned when the reader is closed in https://github.com/thanos-io/thanos/blob/4c7ecd3f86c5ce1f59f54f03a6dceec82d21b77e/pkg/store/bucket.go#L1887
For some of our index chunk files with 500+MB this amounted to a required pool size of 6.25GB to successfully read that chunk file, which blew our Max BytePool size of 4GB, resulting in the byte pool exhausted error when querying.
With the change it 'only' allocates 625 MB for reading the 500MB chunk, which is not optimal but OK and fixes the issue for us for now (on second thought I should have set the smallest bucket to 16000, which is pretty much the only used size).

OK I think there are few misunderstandings but overall this fix makes sense (:

000001 chunk file stores multiple chunks (!). At least one for each series.
Partitioning tries to compose request against bucket (and that's the byte slice we take) into bigger one: https://github.com/thanos-io/thanos/blob/138909636249036fec1b6f3922317e7e01fb586b/pkg/store/bucket.go#L1789 This means that we will request a minimum 16k ONLY if for some reason our query touches only single chunk (it can happen though for short time range queries and when we touch downsampled data).

would be fetched in ~ 1,244 concurrent requests - each downloader requests 16k from the byte pool which caused in the previous version to grab 248.75 MB from the byte pool (because the 16k request was allocated the smallest buffer size: 200k), with the changed version it is only 19,904 MB (smallest buffer: 16k)

Partitioning makes this really unlikely - however still you can exhaust pool with multiple specific queries at the same time.

and only returned when the reader is closed in

Yes - which is done once the requests finish so it should be fine (?)

For some of our index chunk files with 500+MB this amounted to a required pool size of 6.25GB to successfully read that chunk file, which blew our Max BytePool size of 4GB, resulting in the byte pool exhausted error when querying.

That makes sense, but I guess increasing the limit (and memory) will work.

With the change it 'only' allocates the chunk size (plus a few bytes if the chunk is not evenly divisible by 16k) for reading the 500MB chunk, which is a lot better and fixes the issue for us for now

This means to me that your queries are hitting small chunks a lot as I mentioned before.


Fix invalid account address

High Level Overview of Change
Update the test on master which was failing due to an invalid address which was passing silently incorrectly.
Context of Change
Originally found as a result of the behavior changing in rippled 1.11 - it now throws invalidAccount as expected on that version.
Type of Change

[X] Tests (You added tests for code that already exists, or your new feature included in this PR)

Test Plan
CI Passes

Since they are different there should be a malformed address test still.

I changed this PR to point at the 2.0 branch because it already has a similar change in place, and if this were to be committed against master it'd create a merge conflict which may require a rebase to fix. Since this is all going to master anyway and is just a test, I think it's easier to just fix it all with the 2.0 merge.


7.2.9 unable to install GitHub package issue with URLMonLibrary

system does not install rko281/GitHub
after a full install Dolphin7Setup.exe
on trying to evaluate
SourceManager default fileIn: (URLMonLibrary default urlDownloadToCacheFile: 'https://raw.githubusercontent.com/rko281/GitHub/master/install.st').
in workspace
status bar shows error on URLMonLibrary

I should draw your attention to the comment at the top of the readme if you are using any 7.2 build:
Please note: The master branch is now the development branch for version 7.2. This will add support for namespaces, but is currently in an interim and unstable state. It also changes the package and class file formats, and consequently code saved from 7.2 cannot be loaded directly into earlier versions of Dolphin. Please use the release/7.1 branch if you need a stable version. PRs for bug fixes, etc, will still be accepted against the release/7.1 branch for the foreseeable future.
The comment is slightly out of date in that old format packages should save back out in the same format, although of course that will only work as long as dependencies on new stuff (or namespace classes) are not added.
URLMonLibrary is in the namespace OS.COM:

Classes in certain namespaces (Core, Kernel, UI, OS, External and Tools) are imported into Smalltalk, and so remain globally visible to old code and scripts. OS.COM is not imported. In some other cases aliases for "old" class names are installed into the Smalltalk namespace.  URLMonLibrary is not one of those so far, but given the pre-existing use in a useful 3rd-party extension it perhaps should be. You could try loading the up the package using the alternative instructions given, but it may have other dependencies that don't work.


Make nutrition fact input more accessible

There's the case where we need to input > or <. That's why we had opted for a full keyboard.
We should perhaps have something like this:

There's the case where we need to input > or <. That's why we had opted for a full keyboard.
We should perhaps have something like this:

We don't need this for the serving, which is what this PR is about. I'm dealing with what you are saying in another branch, not pushed yet.


Renable disabled tests

From #56

We deleted these disabled tests as part of #141, and decided to punt on implementing jets in Rust for now.


Use the OpenAPI spec to generate the client

Clubhouse now publishes the spec for their API.
It might make sense to use it for code generation instead of manually maintaining the lower-level details and Type hints.
I am not sure which code generator should be used and if any of them can generate Type hints for example.

Thanks, @j-martin ! I'll look into this in more detail this week.

Hey @j-martin it's been awhile, but I've gotten back to working on this project. I spent some time learning about OpenAPI and client generators, and thanks for introducing these concepts.
I'm going to continue developing this package manually because I value the learning experience of creating an API wrapper from scratch for the first time. However, I'm open to re-working this in the future using an OpenAPI code generator.

No worries. Looking at the dev branch you are doing admirable work with the heredocs and everything else. :) My gut reaction is always "can't a computer do that for me instead" though. ;)


[cxx-interop] Enable a test on Windows

The Cxx module is present on all platforms, so we can enable at least the typechecker test unconditionally.

@swift-ci please smoke test


D&D 4e Compatibility

Hiya, I hope you don't mind but I took the liberty of adding some code to improve Active Auras' compatibility with D&D 4e (there are dozens of us! :p). Since the 4e Foundry implementation is built using 5e as a base, AA functions quite well out of the box, but there are some specifics that benefited from additional tweaks. Specifically, I've added 4e to the lists of systems to check for non-Euclidean diagonals as is its default, and I've added a 4e-specific type check function to let us apply auras based on creature origin, type and subtype.

Thanks!


Default port is going to change in the future

See https://github.com/nodejs/node/pull/7212

Closed in 39660868acce62b2cf3e393c2f59c4184fa13573.