refactor: a lots of perf and test changes

completely rewritten prettyFormat
year in now 365 days.

Codecov Report

Merging #27 (fe3f09d) into main (a0526ef) will increase coverage by 3.15%.
The diff coverage is 98.63%.

@@            Coverage Diff             @@
##             main      #27      +/-   ##
==========================================
+ Coverage   94.79%   97.94%   +3.15%
==========================================
  Files           8        7       -1
  Lines         192      243      +51
  Branches       86      123      +37
==========================================
+ Hits          182      238      +56
+ Misses         10        5       -5

Impacted Files
Coverage Δ

src/relativeTime.ts
98.41% <97.05%> (+11.52%)
:arrow_up:

src/prettyFormat.ts
98.38% <98.33%> (-1.62%)
:arrow_down:

src/constants.ts
100.00% <100.00%> (ø)

src/date.ts
96.55% <100.00%> (-0.06%)
:arrow_down:

src/format.ts
96.00% <100.00%> (-4.00%)
:arrow_down:

src/index.ts
100.00% <100.00%> (ø)

src/parse.ts
100.00% <100.00%> (ø)

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


MD5 deprecated for md5

Pushed to wrong branch, but that's ok. @paulyoung want to merge and release?

@forresto go for it. I usually use the GitHub releases UI and add release notes.
I added you as an owner on npm so you can publish too.

Dang, tagged without the v

You should be able to re-tag at the same commit if you wanted to.


Add LiteralPath parameter

Currently we get errors when interacting with files with certain characters, such as square brackets, in their filename; e.g. $myFile = 'c:\temp\myfile[1].xlsx'.
We can work around this by escaping those characters; e.g. $myFile = [Management.Automation.WildcardPattern]::Escape($myFile).
However, it would be helpful if Import-XSLX provided a LiteralPath parameter, as other standard PS cmdlets do, to accept the un-escaped path and treat it as a literal.

THIS REPO IS NO LONGER MAINTAINED
Closing issue


Pass test data structure object directly to TestSuite.run()

Currently, one can substitute RF suite variables programmatically by sending it as list of strings:
run(variable=["varname:value", "foo:bar"), what seems to be working with scalar types only.
For list/dict type variables, so it is better to dump it to JSON/YAML and pass as variablefile argument.
However, if one generates variables as a dict type, it is required to dump it to a file first, and then pass source path variablefile=source.yml. This reads vars and passes it to VariablesStore()` object.
Proposal is to add extra argument, e.g. run(variablesdata) as:

Optional[dict]: data in pass as a regular dictionary  variabledata={var:value, var2:[1,2,3]}
Optional[str]: requires serialized dict, e.g. using variabledata=json.dumps({})

What is the gain here? For systems generating lots of list/dict test data dynamically, it would remove the need of intermediate file saving.
Programmatically, one generates data, loads ROBOT as model and passes it as model.run(variablesdata=data) - clean and easy.
Effort-wise, does not seem to be complicated, since variablefile, e.g. YAML, is loaded to dictionary anyway - so variabledata skipps file loading part - would load dict directly to VariableStore().

Do I get it right that you want to pass variables as actual objects, not only as strings, to TestSuite.run()? The issue title talks about "test data structure" so I'm a bit confused.

Instead of adding a new argument, I think we could change how the existing variable argument is handled. We could make it so that in addition to specifying variables as a string like 'name:value', you could use tuples like ('name', 'value') which would then allow also non-string values like ('example', 42). We've used similar approach elsewhere and it has worked well.

No updates, closing.


First cut of 1.8 support

Enabled node auth.
Removed all 1.5 versioned template folders.
Still need to remove references and some logic involving 1.5 in follow-up issue.
This PR will run up a 1.8 Kubernetes cluster.

I tested 1.8.1, 1.7.8, and 1.6.11 which each worked for me.

As of today, this code will bring up a healthy 1.8 cluster, and is backwards compatible with versions 1.6 and 1.7.
Followup: there are still some deprecated 1.5 references in kraken-lib that will need to be cleaned up.

Copypasta from Slack:
Apparently, the schema validation demands that the kubeVersionedWeaveFabric have both a default and a versions key.
This is fine if we have different WeaveFabrics. However, they are going to be all the same for supported k8s versions, which means I have to do something like this:
      name: kubeVersionedWeaveFabric
      kind: versionedFabric
      type: weave
      kubeVersion:
        default: *defaultWeaveFabric
        versions:
          v1.6: *defaultWeaveFabric

in order to satisfy the schema validation.
How should I proceed?

I may have discovered a problem when bringing up a cluster with an old config:
bash-4.3# k get pods --namespace=kube-networking
NAME          READY     STATUS             RESTARTS   AGE
canal-1dtl0   2/3       CrashLoopBackOff   8          18m
canal-34k5s   2/3       CrashLoopBackOff   8          18m
canal-4vwcz   2/3       CrashLoopBackOff   8          18m
canal-6jqtj   2/3       CrashLoopBackOff   8          18m
canal-7kp18   2/3       CrashLoopBackOff   8          18m
canal-jj66b   2/3       CrashLoopBackOff   8          18m
canal-k6nrj   2/3       CrashLoopBackOff   8          18m
canal-lh83l   2/3       CrashLoopBackOff   8          18m
canal-lks7r   2/3       CrashLoopBackOff   8          18m
canal-nnnqj   2/3       CrashLoopBackOff   8          18m
canal-p0r4h   2/3       CrashLoopBackOff   8          18m
canal-sfr9p   2/3       CrashLoopBackOff   8          18m
canal-t3qtt   2/3       CrashLoopBackOff   8          18m
canal-tkb8k   2/3       CrashLoopBackOff   8          18m
canal-vrqtr   2/3       CrashLoopBackOff   8          18m
bash-4.3#

Will report back soon.


WIP: Migrate to Play2.6

Some commits are made for adjusting Play Framework to 2.6.9.
These are included:

Messages API changes
Controllers changes (for disabled global context manners)
build.sbt or some files fices

reduce redundant parameters of SecureSocial trait children. but not entirely removed.
also fix some lang related bugs.
This commit suits your expectation...

Play 2.6 support was done by another PR.


Fix the generated_projects.sh again

+ tools/buildgen/generate_projects.sh
tools/buildgen/generate_projects.sh: 23: tools/buildgen/generate_projects.sh: [[: not found

Previous PR overlooked one possibility that the Python command returns nothing in stdout.

Known failures: https://github.com/grpc/grpc/issues/25422


Wolf - Arabian Archers on towers and walls

As mentioned in issue #5  , the Wolf was meant to have Arabian bowmen on his towers and walls, as can be seen in the AIV files.
Would be awesome, if that could be restored / done.

The Wolf uses Arabian archers to defend farms though, so it must be made sure he can still send enough to defend them.
And speaking of archers on walls, can we finally have a King Phillip that posts more than 1 or 2 archers at one spot on his towers :D And Sultan having a reserve of Arabian archers on his keep rather than sending all of them away during sieges.

If I recall correctly, the Caliph also had slinger spots, right?

Possible with AIC edits now.


Enhance_UI

Hello @Prathme07! You've done great work. Just try to keep the background color of the popup as it was, or change the popup text to adaptive text. In dark mode, the text isn't visible properly.

Ok I will do that

Hello, I'm interested in contributing to this project. Please assign this to me.

@sapatevaibhav I have corrected the issue. If any changes needed please inform me

Merging the PR but still need to change the color of SVG files which are in black color which switching to dark mode.


Implement as Extensions of Collections rather than struct

Hey there, this repository implements exactly what I've been looking for, but I didn't really like the fact that it's implemented with a struct and generic static functions.
I've swiftly (hehe) implemented a protocol-oriented version of the permutationsWithRepetition function as an extension of the Collection type. Here's the code for a small working example:
let set : Set = [true,false]
let array : Array = [true,false]

extension Collection {
    func permutationsWithRepetition(length n:UInt) -> [[Element]] {
        guard n > 0 else {
            return [[]]
        }

        return self.map{element in self.permutationsWithRepetition(length: n - 1).map{[element] + $0}}.reduce([],+)
    }
}

print(set.permutationsWithRepetition(length: 2))
print(array.permutationsWithRepetition(length: 2))

Is there a reason why you've chosen static functions? What do you think about using protocol extensions instead? If you're interested, I'll gladly help porting the existing code to extensions.
CV

A reason... mmmh... yes, the reason is my Swift skills weren't probably as sharp as they should when I wrote this small library. Let me have a look at your suggestion, it looks interesting in principle.


Add Rust script to verify toolchain and core versions

main branch has been moved to legacy. Closing draft PRs to main and moving open PRs to legacy where further triage will be necessary.


[folding] File contents scrolls if folding is turned off

On testing #2994

open the User preferences and prepare for editing "editor.folding"
in another VS Code column open a file that uses indentation and scroll to some position
remember the line number of the top line
collapse any region

Observe: top line doesn't change (which is good)

set "editor.folding" to false

Observe: file contents scrolls and the top line changes

I fixed it by making sure the cursor position stays visible when collapsing a region (instead of making sure the top line is the same)

when clicking on a fold icon, I make sure that line stays visible
in all other cases we make sure the current cursor line stays visible


Couldn't you think of another name for this?

"App" and "Configuration" are two of the most common words used in Development so searching for anything related to this product in the future will be super annoying for everyone. Conversely, people might not know this exists because when they search for "app configuration" they get 100s of other results related to other types of apps and configurations!
Even an acronym or something like ADAC (Azure Distributed App Configuration), I don't know but just a thought!

Thanks for sharing your input, @lukos. The full product name is "Azure App Configuration". It should be the first hit on Google and Bing. The product naming went through many considerations, including branding and marketing, so it's not something that can be changed lightly. Again, thanks for sharing your thoughts!


webpack: don't remove patternfly from non-insights builds

As of @redhat-cloud-services/frontend-components-config 4.5.0+, patternfly is removed from the build so as not to clash with other patternfly versions in cloud.
This is not desirable for standalone/standalone-keycloak, disabling for non-insights builds.
https://github.com/RedHatInsights/frontend-components/blob/f4633d4f8927f46d5a259bed1cf573c0410785f6/packages/config/README.md#include-pf-css-modules-in-your-bundle
(waiting for the Bump pr first, we're on 4.3.2)

Apparently https://github.com/ansible/ansible-hub-ui/pull/1233 didn't break anything, will investigate more..

.. ah, bundlePfModules only affects config.resolve.alias, which we override in webpack-ts-overrides

closing in favor of https://github.com/ansible/ansible-hub-ui/pull/1342


Issues with some symbolic shortcuts

Created by @rkirsling in https://github.com/leanprover/leanprover.github.io/issues/30

Some of the backslash shortcuts aren't producing their expected output in the online editor.
According to the table in Chapter 3.3 of the tutorial:

\iff and \lr should produce ↔
\not and \neg should produce ¬

In the first case, \lr works as expected (as does \<-> and \leftrightarrow), but \iff produces ⇔ instead, which results in an unexpected token error. The cause here seems to be input-method.js#L689.
The second case is weirder: \neg works correctly (as does \lnot), but \not produces U+0338, a sort of "combining slash" character. Looking at input-method.js#L701 though, nothing really seems amiss. :confused:

Dear @avigad and @leodemoura,

In the first case, \lr works as expected (as does <-> and \leftrightarrow), but \iff produces ⇔ instead, which results in an unexpected token error.

I've noticed this one too. Do you see any problem in mapping \iff to ↔ instead of ⇔?

The second case is weirder: \neg works correctly (as does \lnot), but \not produces U+0338, a sort of "combining slash" character

I think @avigad mentioned this one in lean-discuss list. If you don't mind, I'm happy to make the change.
FYI, input-method.js is automatically generated from leanprover/lean/src/emacs/lean-input.el.

Looks like the root cause is simply that \iff and \not are being mapped to their MathJax defaults, since these sequences aren't specified in lean-input.el. (Same goes for \leftrightarrow and \lnot, though those are unproblematic.)
I'm assuming this also means that \iff and \not are outright unrecognized in Emacs, but I've only tried the online editor so far.

Yes, both should be added to lean-input.el. @soonhokong, feel free to do this (or I will next time I add to the library).

@rkirsling and @avigad, I will fix it tonight.


Rename Full Stack certification to Legacy Full Stack certification

Once https://github.com/freeCodeCamp/freeCodeCamp/pull/38995 is merged, the settings page will describe how to claim a Legacy Full Stack certification.
However, if you claim the Legacy Full Stack certification, it will show as a Full Stack certification instead. This is a reminder that we need to change this before the new 2020 Full Stack Development Certification is created.
Related to https://github.com/freeCodeCamp/freeCodeCamp/issues/38934 and https://github.com/freeCodeCamp/freeCodeCamp/issues/38943.

@scissorsneedfoodtoo I am assigning this to you, if its all the same considering you have worked closest to this in the last PRs.

Are there any plans to offer a new "full stack" cert? Without "Legacy" stamp?   Legacy means and sounds outdated according to my understanding. :(


Object creation for QILESRVSRC pseudo source object reported as successful when not.

I have some QILESRVSRC psuedo source with a bug in it.  When I run 'makei build', the output says the object was created successfully, and is not listed in the list of failed objects.
Unfortunately the object didn't exist, despite the success message.  Upon further exploration, there were messages in the job log that do in fact indicate a failed build, which led me to the discovery of the bug in my psuedo source cl command.
Here is a the QSRVSRC source with the bug:

And a basic rule for it:

Here is the success message in the output (you can actually see the bug with the missing quote):

Object not included in the list of failures:

Here are the relevant messages in the job log:

Thanks for the thorough investigation.  We will look at this when we get a chance.  I suspect the script/launch is not setting the return code.  I see that it is trying to use $? to get the return code from the cl bash command
cl "$cmd"
exitCode=$?
echo "<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<"
echo
"$SCRIPT_DIR"/getJobLog "$cmd" "$timestamp" "$jobid" -f "$joblog_json"
exit $exitCode

Fixed in https://github.com/IBM/ibmi-bob/pull/317


Disable glide on travis

Centralize the 'vendor' task into the root Makefile
ifdef the vendor task to be empty on CI

Will recreate this PR from a remote repo


Fixes to allow bank charge success

There were some json inconsistencies with the paystack API and some parameters missing or miss named.
Also added a test for the charge function.

Hi @rpip , thanks for the well written lib. Please find the changes made as suggested.


Devfile parser copies whole parent directory content to child directory

Which area this feature is related to?
/kind bug
Which area this bug is related to?
/area library
What versions of software are you using?
 I tested this by running main.go:
# clean working tree before running the parser
$ git -C /tmp/bug-devfile-parser/ status
On branch main
nothing to commit, working tree clean

# Parsing the child Devfile
$ go run . /tmp/bug-devfile-parser/child-devfile.yaml
parsing devfile from /tmp/bug-devfile-parser/child-devfile.yaml
[...]

# dirty working tree
$ git -C /tmp/bug-devfile-parser/ status
On branch main
Untracked files:
  (use "git add <file>..." to include in what will be committed)
        parent-devfile.yaml
        some-additional-file.txt

nothing added to commit but untracked files present (use "git add" to track)

Expected behavior
Maybe I missed something, but I don't understand why the directory content of the parent Devfile is being copied into the child directory.
Any logs, error output, screenshots etc? Provide the devfile that sees this bug, if applicable

Additional context

We ran into this issue in the odo tests, where our local Git repos started to have untracked files every time we run our unit tests - see https://github.com/redhat-developer/odo/issues/6539
This behavior seems to have been introduced in https://github.com/devfile/library/pull/139
Any workaround?

Suggestion on how to fix the bug

Devfile Community Call - 05/15
Expected behaviour for loading devfiles with parent is that devfile library will load the content of resources from the devfile parent location.


Update llvmint

I updated my llvmint repository by including changes from https://github.com/aweinstock314/llvmint which resulted into one new intrinsic.

The problem was that I wasn't regenerating the json file from llvmint. It's now fixed and the list is complete.

To fix the removal issue, I keep both llvmint repositories.


SplitColourComposite error in v.0.16.0 Windows binaries

Hi @jblindsay
I was testing SplitColourComposite in WhiteboxTools 0.16.0, following this fix
https://github.com/alexbruy/processing-whitebox/issues/17#issuecomment-482542035
but now I'm getting the error below:
C:\>"C:\OSGeo4W64\WhiteboxTools_win_amd64\WBT\whitebox_tools.exe" --run=SplitColourComposite --input="D:\orto_teste.tif" --red="D:\orto_teste_r.tif" --green="D:\orto_teste_g.tif" --blue="D:\orto_teste_b.tif" -v
***********************************
* Welcome to SplitColourComposite *
***********************************
Reading data...
thread 'main' panicked at 'called `Option::unwrap()` on a `None` value', src\libcore\option.rs:345:21
note: Run with `RUST_BACKTRACE=1` environment variable to display a backtrace.

Can this be a local problem?

@PedroVenancio That's interesting, I haven't changed the code for this tool in quite some time so I wouldn't expect the behaviour to have changed unless it's related to the file format encoder/decoder. Would you be able to send me a copy of the file you're trying to process, or a small subset of the data? I can then diagnose the problem better.

Hi @jblindsay
Off course, I send you the sample data I'm testing:
orto_teste.zip
This tool was functional, with the same sample data, before the update to have the three (R, G and B) output files specified separately (WhiteboxTools v.0.14 or v.0.15, I believe):
C:\>"C:\OSGeo4W64\WhiteboxTools_win_amd64\WBT\whitebox_tools.exe" --run=SplitColourComposite --input="D:\orto_teste.tif" --output="D:\orto_teste_split_2.tif" -v
***********************************
* Welcome to SplitColourComposite *
***********************************
Reading data...

Output file written
Saving green image...
Output file written
Saving blue image...
Output file written
Elapsed Time (excluding I/O): 0.80s

@PedroVenancio I found the problem. It was actually a change to the SplitColourComposite tool that resulted from my change for the QGIS plugin. Basically, I had an extra dash in the code than I needed, which prevented the input file from being interpreted correctly. I guess when I tested the tool after the change, I must have used the '-i' flag rather than the more common '--input' flag. Anyhow, this was a good catch. I've committed code that provides the fix to the repo now and am closing the issue. Thanks again.

Great @jblindsay !
Using -i flag it works as expected:
C:\>"C:\OSGeo4W64\WhiteboxTools_win_amd64\WBT\whitebox_tools.exe" --run=SplitColourComposite -i="D:\orto_teste.tif" --red="D:\orto_teste_r.tif" --green="D:\orto_teste_g.tif" --blue="D:\orto_teste_b.tif" -v
***********************************
* Welcome to SplitColourComposite *
***********************************
Reading data...

Output file written
Saving green image...
Output file written
Saving blue image...
Output file written
Elapsed Time (excluding I/O): 0.79s

Thanks for the fix!
Best regards!


Always use system rubygem to determine nodocument flag.

(This could only happen on chef 14.x at this moment)
Currently if the user doesn't specify custom rubygem binary path,
chef uses Gem::VERSION to determine if the rubygem version is
compatiable with old nodocument flags like --no-rdoc.
https://github.com/chef/chef/blob/7558c414030d082194d0d2b5f74279ce151517e9/lib/chef/provider/package/rubygems.rb#L601-L611
If the user doesn't specify rubygem binary path:
https://github.com/chef/chef/blob/7558c414030d082194d0d2b5f74279ce151517e9/lib/chef/provider/package/rubygems.rb#L272-L274
If the user specifies rubygem binary path:
https://github.com/chef/chef/blob/7558c414030d082194d0d2b5f74279ce151517e9/lib/chef/provider/package/rubygems.rb#L303-L305
This doesn't work because the Gem::VERSION always points to chef's
embedded rubygem version. But the binary we use to install gems is
the system's rubygem, which might be different from chef's embedded
version even though the user doesn't specify custom binary path.
So when a user upgrades his Ruby version to 2.6 (rubygem 3.0.x) with
Chef 14.x (rubygem 2.7.x), an error will occur:
 STDOUT:
   STDERR: ERROR:  While executing gem ... (OptionParser::InvalidOption)
       invalid option: --no-rdoc

This is because Gem::VERSION would make chef think that it's ok to use
--no-rdoc while it's not. The solution is to always perform the check with
system rubygem's version.
Signed-off-by: st0012<EMAIL_ADDRESS>
Description

Related Issue

Types of changes

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

Checklist:

[x] I have read the CONTRIBUTING document.
[x] I have run the pre-merge tests locally and they pass.
[x] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.
[x] All commits have been signed-off for the Developer Certificate of Origin.

Closing this based on @lamont-granquist's response


[PE-D][Tester E] Missing User Stories in DG

Currently, there is no mention of aggregate commands and its use in any user story in the developer guide.

Labels: type.DocumentationBug severity.Medium
original: tanwartushar/ped#14

Thanks for reporting


Add support to open multiple pdfs on a single page in fulfillment app

What is the motivation for adding/enhancing this feature?

Currently when having multiple pdf urls, each url is opened in a separate tab, but when we are doing a bulk operation then in that case we need to open the pdfs in the same tab
What are the acceptance criteria?

[ ] ...

Can you complete this feature request by yourself?

[x] YES
[ ] NO

Additional information

Explored on multiple pdf packages to merge multiple pdfs in one, but having issue regarding the content not being parsed correctly.


ConnectionPool cachedConnections::cleanUp? Why?

maintenanceThread.scheduleWithFixedDelay(cachedConnections::cleanUp, 1, 1, TimeUnit.SECONDS);

I can't understand the comments...

/**
   * Performs any pending maintenance operations needed by the cache. Exactly which activities are
   * performed -- if any -- is implementation-dependent.
   */
  void cleanUp();

The connection pool is implementing using a Guava Cache.
We configure the maximum connections by setting maximumSize(), use expireAfterAccess() to implement the idle timeout (evict connections that haven't been used for a while), and add a removal listener to close any connections that are removed from the cache due to being "expired" (max size or last access).
This all works as long as connection pool is actively used. Guava Cache will perform periodic cleanup of expired entries during writes or reads. However, if the traffic is bursty and the system goes idle for a while, the connections would sit in the cache longer than they should. Thus, we follow the recommendation from Guava:

If your cache does writes only rarely and you don't want cleanup to block cache reads, you may wish to create your own maintenance thread that calls Cache.cleanUp() at regular intervals. If you want to schedule regular cache maintenance for a cache which only rarely has writes, just schedule the maintenance using ScheduledExecutorService.

To more directly answer your question, the Cache.cleanUp() method will go through the entire cache and remove any items that are expired (and call the removal listener for those).

@electrum ths, 祝您好运


Elaborate on purpose of the WriteAttributes tool

The documentation does not adequately explain why the WriteAttributes tool exists, especially in terms of how it can be used to affect observations. This needs to be improved.

Affects 0.1.9.


Use as a standalone layer 3 tunnel

This is a very specific request but assuming I don't care about encryption I wish to avoid the overhead of using a VPN tool such as WireGuard. I need to cross a firewall that only allows TCP 443 but has no other monitoring.
I do have a working solution written in go that seems to be pretty fast but could probably use more optimization as it was my first attempt at writing Go code. If this feature request is something not on priority then could you provide some documentation for the fake-tcp library? It seems very interesting to me and should help get more performance

If you don't need any encryption, I suppose you could simply run a SOCKS5 server on 443.

Do you need UDP support? If not then a SOCKS5 proxy would probably be as fast as you can get.
With UDP support, I think WireGuard is reasonable. It will unlikely become the bottleneck for ordinary users.

On that thought though, there is also static/unmanaged L2TPv3:
https://man7.org/linux/man-pages/man8/ip-l2tp.8.html

@archit120 Documentation added in https://github.com/dndx/phantun/pull/39.


Several formatting changes

As reading I made some formatting changes to enhance the text.
For some notes about content, having already in my mind, I need to read at least once again.

Would you mind raising it against the open PR on the eclipse repo: https://github.com/eclipse/microprofile-sandbox/pull/2

@mmusgrov sure, no problem. I will close this one and open a new one. This jbosstm repo will not be used for PR anymore, right?

@mmusgrov Tom approved the changes so I merged that but now I realized that it will not go to your pr to eclipse (eclipse#2) as it's pointed from master not from this 0009-LRA branch. Would not be ok to create PR to master branch and you will check and merge it and it will be shown in the eclipse one?

@ochaloup yes that is the correct strategy for all future LRA/SRA spec updates.


[TW#26612] stack overflow in task btuT has been detected

@Weijian-Espressif here is my sdkconfig:
https://gist.github.com/chegewara/bb3ae58044ac8b1bff24740a35e90695
If i change ble logging level from DEBUG to default WARNING then i dont have this issue.
https://gist.github.com/chegewara/bb3ae58044ac8b1bff24740a35e90695#file-sdkconfig-L170-L345

@chegewara please provide the log. is ESP32 scanning?

@Weijian-Espressif Yes, esp32 is scanning. This is not the same app, but the same issue with stack overflow when few bt log levels are higher than WARNING:
I (2824) BT_L2CAP: L2CA_SendFixedChnlData()  CID: 0x0004  BDA: 30aea43f244e
E (2864) BLEDevice:  gattc_if: 3 == 3
E (2864) BLEDevice:  gattc_if: 3 == 3
E (2864) BLEDevice:  gattc_if: 3 == 3
E (2864) BLEDevice:  gattc_if: 3 == 3
I (2864) BLEClient: Get service information from remote device
E (2874) BLEDevice:  gattc_if: 3 == 3
I (2874) SampleClient: 7--> 5404
E (2884) BLERemoteCharacteristic:
E (2884) BLEDevice:  gattc_if: 3 == 3
I (2894) SampleClient: 8--> 5180
I (2894) BT_L2CAP: L2CA_SendFixedChnlData()  CID: 0x0004  BDA: 30aea43f244e
I (2904) BT_L2CAP: L2CA_SendFixedChnlData()  CID: 0x0004  BDA: 30aea43f244e
***ERROR*** A stack overflow in task btuT has been detected.
abort() was called at PC 0x4008fb08 on core 0
0x4008fb08: vApplicationStackOverflowHook at D:/msys32/home/imper/esp/esp-idf/components/esp32/panic.c:660

Backtrace: 0x4008f920:0x3ffd1d70 0x4008faf1:0x3ffd1d90 0x4008fb08:0x3ffd1db0 0x4008c49c:0x3ffd1dd0 0x4008e4c0:0x3ffd1df0 0x4008e476:0x3ffd2100 0x40082c9d:0x3ffd2120 0x400592fe:0x3ffd2140 0x400
5937a:0x3ffd2160 0x40058bbf:0x3ffd2180 0x400d9279:0x00000000
0x4008f920: invoke_abort at D:/msys32/home/imper/esp/esp-idf/components/esp32/panic.c:660

0x4008faf1: abort at D:/msys32/home/imper/esp/esp-idf/components/esp32/panic.c:660

0x4008fb08: vApplicationStackOverflowHook at D:/msys32/home/imper/esp/esp-idf/components/esp32/panic.c:660

0x4008c49c: vTaskSwitchContext at D:/msys32/home/imper/esp/esp-idf/components/freertos/tasks.c:3564

0x4008e4c0: _frxt_dispatch at D:/msys32/home/imper/esp/esp-idf/components/freertos/portasm.S:406

0x4008e476: _frxt_int_exit at D:/msys32/home/imper/esp/esp-idf/components/freertos/portasm.S:206

0x40082c9d: esp_log_write at D:/msys32/home/imper/esp/esp-idf/components/log/log.c:121

0x400d9279: __sprint_r at /Users/ivan/e/newlib_xtensa-2.2.0-bin/newlib_xtensa-2.2.0/xtensa-esp32-elf/newlib/libc/stdio/../../../.././newlib/libc/stdio/vfprintf.c:437

@Weijian-Espressif
I have it enabled all the time and i dont print too much when i scan.

@chegewara please Increase the BTU_TASK_STACK_SIZE  task size and try, thanks.

@Weijian-Espressif
I did, i have 8 kB now and i think i did increase it up to 18-20kB. But sure i will do it now one more time to confirm. But in mean time i think this log can be useful too. When i reset one esp32 peripheral and central lost connection to it, then for some reason (timeout) all connected peripherals are disconnected.
From now on all found peripherals are connected again if they are found:
I (24705) BT_GATT: GATT_GetConnIdIfConnected status=0

W (24705) BT_APPL: bta_gattc_conn_cback() - cif=3 connected=0 conn_id=259 reason=0x0008
W (24705) BT_APPL: bta_gattc_conn_cback() - cif=4 connected=0 conn_id=260 reason=0x0008
W (24715) BT_APPL: bta_gattc_conn_cback() - cif=5 connected=0 conn_id=261 reason=0x0008
I (24725) BT_L2CAP: L2CA_SetDesireRole() new:x1, disallow_switch:0
I (24775) BT_GATT: GATT_Disconnect conn_id=3
I (24775) BT_L2CAP: L2CA_RemoveFixedChnl()  CID: 0x0004  BDA: 30aea449a732
I (24785) BT_GATT: GATT_GetConnIdIfConnected status=0

W (24785) BT_APPL: bta_gattc_conn_cback() - cif=3 connected=0 conn_id=3 reason=0x0016
W (24795) BT_APPL: bta_gattc_conn_cback() - cif=4 connected=0 conn_id=4 reason=0x0016
W (24805) BT_APPL: bta_gattc_conn_cback() - cif=5 connected=0 conn_id=5 reason=0x0016
I (24815) BT_L2CAP: L2CA_SetDesireRole() new:x1, disallow_switch:0
I (24825) BT_GATT: GATT_Deregister gatt_if=3
I (24825) BT_GATT: GATT_Listen gatt_if=3
E (24835) BT_APPL: bta_gattc_deregister Deregister Failedm unknown client cif
I (24835) BT_GATT: GATT_Deregister gatt_if=4
I (24845) BT_GATT: GATT_Listen gatt_if=4
E (24845) BT_APPL: bta_gattc_deregister Deregister Failedm unknown client cif
I (24855) BT_GATT: GATT_Disconnect conn_id=517
I (24865) BT_L2CAP: L2CA_RemoveFixedChnl()  CID: 0x0004  BDA: 30aea443a172
I (24875) BT_GATT: GATT_GetConnIdIfConnected status=0

@Weijian-Espressif
This is library with master and slave code i am using right now:
https://github.com/chegewara/esp32-snippets-enchancements-test/tree/multi_connect_test
There is few cpp files in main folder, but for tests i am using:

central -> SampleClient.cpp
peripheral -> SampleServer.cpp

@chegewara Thanks for your help.  I will retry to reproduce the issue. By the way, when you register multiple gattc callbacks, one device disconnects Bluetooth and all callbacks trigger a disconnect event.

I have to check, but i think i have only one callback registered from which every client events are dispatcher according to gattc_if it is connected. But thanks for useful info.

Is there any particular reason why disconnect event is triggered on all callbacks when one peer device is disconnected?

@Weijian-Espressif btuT stack behaves very strange. Now i have test app as such:

turn off debugging in menuconfig (option to decrease binary size), so there should not be anything that takes stack unnecesary,
after bt initialization i am getting btuT handle and i am using it later to log stack size,
when any device is connected to esp32 i am sending write to remote characteristic every 1000ms and i am printing to logs btuT minimal free stack size from watermark function,
Result:
from time to time i am having very different values of btuT minimal free stack, from 322 to 1708 bytes, its just enough to do reset esp32 few times.

I am only guessing now, but maybe this is somehow related to this issue (btuT stack):
E (857) BT_OSI: osi_mem_dbg_clean invalid

W (30863) BT_L2CAP: L2CA_CancelBleConnectReq - no connection pending
E (30865) BT_OSI: osi_mem_dbg_clean invalid

After turning on Debug bluetooth memory in menuconfig im having BT_OSI error messages all the time.

@chegewara Do you modify BT DEBUG LOG LEVEL in menuconfig ?

@Weijian-Espressif Yes, i have changed few BT DEBUG LOG LEVEL values to higher:
https://gist.github.com/chegewara/bb3ae58044ac8b1bff24740a35e90695#file-sdkconfig-L172-L338

@chegewara  If you print too much debugging information, it will cause btuT exception. I have not found a good solution yet.

@Weijian-Espressif Dont worry about that, leave it for now. Im not complaining, just wanted to inform that some issue exists. Since i am only one, and it is only during extensive debugging it is not very important.
You could increase btuT stack size, but this would be waste of resources in this case. Thanks for support.

@chegewara, For some customers who don't have enough memory, this modification reduce the free memory of ESP32 and will cause new problems, so I can't modify it.
If you need more btuT stack size, you can modify it.  .

I thout about 200-300 bytes, but thats fine. Thanks.


Use upsert with related entities

I can't use upsert with related entities always returns this error even i have created an alternate key i manged to insert the parent record but when i use related entities it returns this error :

{Exception details:
ErrorCode: 0x80060891
Message: A record with the specified key values does not exist in msdyn_requirementresourcecategory entity
TimeStamp: 2019-05-08T15:33:10.0216569Z
--
}

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

ID: b67e1546-a8a1-3620-7952-99179cae9bfe
Version Independent ID: c8489a0e-325a-0801-b23d-8ad83aa4c527
Content: Use Upsert to insert or update a record (Developer Guide for Dynamics 365 for Customer Engagement apps)
Content Source: ce/developer/use-upsert-insert-update-record.md
Service: crm-online
GitHub Login: @JimDaly
Microsoft Alias: jdaly

This now relates to https://docs.microsoft.com/en-us/powerapps/developer/common-data-service/use-upsert-insert-update-record

I have experienced this issue as well. I have an automated process that performs upserts using alternate keys to keep Dynamics entities up-to-date with entities in a separate application database. However it appears that some of these upserts fail with the message 0x80060891: A record with the specified key values does not exist in contact entity. It is not yet clear why this occurs for some records but it remains consistent.

“Thanks for your feedback. We are closing this issue because we are either unable to find relevant information or reproduce this. If you're still experiencing the issue described here, please reopen or create a new issue."


Simplify card picking and tests

This PR:

Simplifies auto-picking of a card.
Minimizes the integration tests related to card creation and picking.
Adds description field to a card (to show optional fields and make the example a bit more real).

@armiol, @mdrachuk, contrary to original intend to re-work the columns (as the branch name suggests), I decided to polish the picking logic. Columns would go next.
PTAL.


Cannot install fcopy-pre-bundled

It seems to be an issue related to the recent update of electron-builder.
https://github.com/electron-userland/electron-builder/issues/1662
https://github.com/electron-userland/electron-builder/issues/1651
https://github.com/chentsulin/electron-react-boilerplate/issues/1101
As mentioned here

Please stop wasting your time. Get rid of NPM and consider to use Yarn.

It sounds like the simple workaround would be locking down the electron-builder version.
https://github.com/hackjutsu/Lepton/blob/master/package.json#L36

Since version 18.7.0, when installing electron-builder with npm, I get an error with fcopy-pre-bundled

Shouldn't the semantic version guarantee the backward compatibility? Updating the minor number should not introduce breaking changes, but electron-builder does...

https://github.com/hackjutsu/Lepton/pull/151


gui: Set a title and an icon to the window

https://github.com/iced-rs/iced/commit/9a037a23e9b32d9dbe7086a54d777b5f0550a660

Did you remove it from the fix list too?

I think you added a title already? It's fine for the first version to not have any icon as long as we have a title.

i think this have been done

Thanks for the reminder, closing this.


This package is awesome!!

Hey,
I love this package, good job for making it!
Looks beautiful and nice UX/UI.
I was wondering, do you have a future plan to support multiple droppables?
(Moving a sortable item from one sortable list to another sortable list).
Example:
https://react-beautiful-dnd.netlify.app/iframe.html?id=board--simple

Hi, @elron! Thanks a lot for reaching out! And congratulations on your talk during the recent Svelte Summit. As a longtime WP user and a Svelte fan, that one was definitely one of my favorites :)
And regarding your query, I’ve been considering working on that specific feature for a while, and since I’ve finally been able to provide a solid ground for this package, this could be a good moment to give that a try.
It’ll definitely be quite tricky for lists to detect and handle foreign items (if even possible :P), so I can’t make any promises, but I will definitely give it my best shot :)

Thanks for the kind words about my Svelte Summit talk! It means a lot, especially coming from someone making such useful Svelte tools. Speaking of which - I really think this library deserves a spotlight at the next Svelte Summit! The UX is so polished, and multiple droppables would make it even more powerful.
I'm actually looking to migrate from a React app that uses react-beautiful-dnd to Svelte, and your library looks perfect except for this one feature. In my current app, I heavily rely on the multiple droppables functionality to move items between different lists. I totally understand the technical challenges involved though - just wanted to give you this context about my use case. Really excited to see how the library evolves!
PS: If it helps, this free Egghead.io course on "Beautiful and Accessible Drag and Drop with react-beautiful-dnd" really helped me understand how droppables work. (On lesson 9 minute 0:20 is where he shows the columns object structure, which might give you inspiration)

Thank you, @elron! Thanks for taking the time to provide all that context for your use case. I love this kind of concrete feedback since it provides an scenario and a feature that I might have not considered. Plus it’s always good to know that I might me helping someone out with the work that I’m doing here :)
While considering our options, I realized that making use of the Drag and Drop API would probably make this task much easier, but in the beginning (and after a lot of testing) I went the custom route because it allowed me for much more control over the visuals. I haven’t discarded this approach yet, and I’ve been considered rewriting the package using that DnD API, and I will definitely give it a go at some point. Plus with all the stuff I’ve learnt while build this, I might be able to sort out the API’s shortcomings this time :)
But at the moment it’s kind of hard for me to tell what approach could work considering the current state of this package. My best bet is creating a new component to wrap multiple lists inside in order to share state from one list to another, which might be what React Beautiful DnD is doing.
I’ve just started to work on it, so hopefully I’ll have an update soon enough :)

Thank you for such a detailed response! Really cool to hear about your different approaches and thinking process. I'm excited that you've started working on it! And yes, you'll be helping not just me, but many others too - I'm sure of it. I'm excited because this feature is essential for it to work and I'm happy to brainstorm ideas if you're open for it! but obviously I think you know better because you actually built everything and knows how it works.
The current behavior is great by the way, and I wonder if there's a way to extend it instead of re-writing everything - might be easier and less time consuming for you. Just throwing an idea here on how it could work:
<SortableContext on:sort={handleSort}>
	{#each lists as list, index (list.id)}
		<SortableList id={list.id}> // check if it has SortableContext or not, it it does not, then allow If on:sort on SortableList. That way the API keeps working as it does now, but only has extensions if its needed to have multiple sortables.
			{#each items as item, index (item.id)}
				<SortableItem id={item.id} {index}>
					<div class="ssl-item__content">
						{item.text}
					</div>
				</SortableItem>
			{/each}
		</SortableList>
	{/each}
</SortableContext>

Here are two possible ways we could structure the data:
Option 1 (Array of lists with nested items, already in the correct order [and when re-ordered the whole array gets modified]):
let lists = [
  {
    id: 'list-1',
    items: [
      {
        id: 'list-1-item-1',
        text: 'List 1 item 1',
      },
      {
        id: 'list-1-item-2',
        text: 'List 1 item 2',
      }
    ]
  },
  {
    id: 'list-2',
    items: [
      {
        id: 'list-2-item-1',
        text: 'List 2 item 1',
      }
    ]
  }
];

Option 2 (Normalized object structure, this is inspired by the React-DnD lib):
data = {
  tasks: {
    'task-1': { id: 'task-1', content: 'Take out the garbage' },
    'task-2': { id: 'task-2', content: 'Watch my favorite show' },
    'task-3': { id: 'task-3', content: 'Charge my phone' }
  },
  columns: {
    'column-1': {
      id: 'column-1',
      title: 'To do',
      taskIds: ['task-1', 'task-2']
    },
    'column-2': {
      id: 'column-2',
      title: 'Done',
      taskIds: ['task-3']
    }
  },
  columnOrder: ['column-1', 'column-2']
}

In my opinion, the first option is more Svelte-like in the philosophy of simplicity, but might need to be careful when actually changing the sort, and the second option is smart and conservative, maybe better for bigger objects, and easier to modify.
Another thing, I noticed the library is in svelte-4. I feel like maybe after migrating to Svelte 5 it will be easier to make this feature, as $state() can hold the whole data object/array and re-render in a more cost efficient way.
What are your thoughts on this? I hope you don't mind me brainstorming ideas - I'm just excited about the possibilities, but of course you know what's best for the library!

After some thinking, I think the second option (like react-beautiful-dnd) of structuring the data is more cost-effective, not just for the front end, but also when saving to the back end.
Because what actually changes during a sort is just:
taskIds: ['task-1', 'task-2']
or/and
columnOrder: ['column-1', 'column-2']
The items data itself (tasks) stays the same and doesn't need to be updated at all. This can be beneficial for two reasons:

It saves unnecessary re-renders in the front end
It avoids re-saving the whole data object to the back end when a sort occurs (which can become quite large over time). This reduces network payload and results in faster saves.

Don’t you even worry, my friend! Your input is very welcome and appreciated! In fact, just yesterday I was thinking how nice it would be to have someone to discuss this stuff with :)
So thank you again for your feedback! And I was glad to see we’re very much in sync, by coincidence I even called this new context component  <SortableContext> just like you did :P
And yes, out of the box the second data structure that you shared seems to be more cost-effective, but I guess that will depend mostly on the kind of structure the user has created for his/her own app. But in all honesty, I prefer the first approach the most since to me it goes more in line with what has already been built for this package. I’m guessing most people will create lists with a small amount of items, so hopefully the difference in performance won’t be perceived :)
This has been way more challenging than I expected, but I’ve been exploring a few different approaches:

Building a new component to serve as context for the contained lists.
Refactor the existing components to avoid creating/using additional components:
a. Combine pointer events with drag events.
b. Explore unused pointer events (pointerenter, pointerleave, pointerover, pointerout).

While walking myself through the drag events once again I remembered the main reason why I discarded them: they do not work on mobile devices  x__x
Getting the first option to work would be the easiest, but I’m putting effort on getting the second one to work since (imo) it would keep the DX/UX simple while handling the tricky stuff behind the scenes (which has always been my main goal for this package) and probably make it much easier to implement something like nested lists in the near future.
That’s all I have for the moment! :P

Wow, that makes me so happy that you're open to brainstorming! And that's such a fun coincidence about the SortableContext name 😄
I really appreciate you explaining your thought process about the data structures. I totally trust your approach - if you feel the first option aligns better with the package's philosophy, that makes a lot of sense! Even if you go with that approach, I'll find a way to migrate my data structure to match it. (I'm thinking maybe to hold the actual data in another object, and store the "order" in your approach. That way we enjoy the best of both worlds!)
Super interesting to hear about the mobile device limitations with drag events - I hadn't considered that challenge, sounds annoying. Actually, I think I saw a video about this drag and drop problem by Alex Reardon (the creator of react-beautiful-dnd who deprecated it). If you haven't seen it, it might give you some inspiration: https://www.youtube.com/watch?v=Kz50msV-zq0. By the way, I'm not sure if it was this video or a different one.
Keep me posted on how it goes - I'm excited to see what you come up with!


Even when specifying the name of source and destination columns, error is thrown expecting "src" or "dst"

When running load_edge_attributes as below:
prop_graph.load_edge_attributes(test_edge_df, source_column="src1", destination_column="dst1", relationship_columns=["data5", "data1"])

The following chain of errors is thrown
KeyError: "Invalid column name 'src'."
KeyError: 'duplicated attribute (column) name in relationships'

This has to do with add_edge_relationships expecting columns named "src" and "dst".

Error also happens for load_node_attributes.


Automated cherry pick of #5608: fix: allow list loadbalancer cluster by default

Cherry pick of #5608 on release/3.0.
#5608: fix: allow list loadbalancer cluster by default

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/3.0@8d9a76e). Click here to learn what that means.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             release/3.0   #5611   +/-   ##
=============================================
  Coverage               ?   6.86%
=============================================
  Files                  ?     539
  Lines                  ?   91349
  Branches               ?       0
=============================================
  Hits                   ?    6271
  Misses                 ?   84557
  Partials               ?     521

Flag
Coverage Δ

#aFlag
6.86% <0.00%> (?)

Continue to review full report at Codecov.

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

/lgtm
/approve


Imprecise calculation results

Actually, I run into this issue while writing unit tests for parts of my code. I am testing some calculation logic and thereby, execute the following calculation in a unit test, written in TypeScript:
import BigNumber from 'bignumber.js';

const a = new BigNumber('90');
const b = new BigNumber('0.05');
const c = new BigNumber('0.1');

const result = a.times(b.plus('1').dividedBy((new BigNumber('1')).minus(c)));

In my unit test, I have the following line
expect(result).toEqual(new BigNumber('105'));

which fails with
 - Expected  - 1
 + Received  + 1

 - "105"
 + "105.0000000000000000003"

According to wolfram alpha, the calculation should exactly result 105. So, as far as I can see, the calculation returns<PHONE_NUMBER>000000000003 but should return 105? I also altered the unit test, as I imagined that maybe toEqual executes some strange transformation, to
expect(result.isEqualTo(new BigNumber('105'))).toBeTruthy();

However, this test fails as well.
I'm not sure whether I misunderstood something here and this is expected behavior. In that case, I'd be thankful for a hint to some resources explaining why this is expected. Otherwise, I hope this helps fixing a potential bug in this great library.
Used version of  bignumber.js is 9.0.1.

There is no bug here.
Your sum is 90 * (1.05 / 0.9) and
1.05 / 0.9 =  7 / 6 which cannot be represented exactly as a decimal floating-point number as it is non-terminating:
1.1666666666666666666666666666666666666666666666666666666666666666...
With this library, the number of decimal places of the result of a division operation depends on the value of the DECIMAL_PLACES setting.
So, for example
import BigNumber from './bignumber.mjs';

const a = new BigNumber('90');
const b = new BigNumber('0.05');
const c = new BigNumber('0.1');

const dp = 20;
const expected = new BigNumber(105);

BigNumber.set({ DECIMAL_PLACES: dp })

const result = a.times(b.plus('1').dividedBy((new BigNumber('1')).minus(c)));

console.log( result.isEqualTo(expected) );                          // false
console.log( result.decimalPlaces(dp - 1).isEqualTo(expected) );    // false
console.log( result.decimalPlaces(dp - 2).isEqualTo(expected) );    // true
console.log( result.integerValue().isEqualTo(expected) );           // true

Oh, thanks! Somehow, I mixed it up in my head and thought that the library worked symbolic. Totally makes sense, sorry for the inconveniences!


JarFile tomitribe-http-signatures-1.0.jar has no entries

Hi, newbie here.  I was hoping I might just have enough Java knowledge to do this; but all I get is "JarFile tomitribe-http-signatures-1.0.jar has no entries".  Can you point me in the right direction at all?

Hi,
I am getting this error. Is it because of the ACL grants?
ERROR:
ORA-29273: HTTP request failed
ORA-06512: at "APEX_190100.WWV_FLOW_WEB_SERVICES", line 1035
ORA-12535: TNS:operation timed out
ORA-06512: at "SYS.UTL_HTTP", line 380
ORA-06512: at "SYS.UTL_HTTP", line 1127
ORA-06512: at "APEX_190100.WWV_FLOW_WEB_SERVICES", line 934
ORA-06512: at "APEX_190100.WWV_FLOW_WEB_SERVICES", line 1580
ORA-06512: at "APEX_190100.WWV_FLOW_WEBSERVICES_API", line 408
ORA-06512: at "TESTSCHEMA.LISTUSERS", line 41
Please help me to understand this issue
Regards
Mellwin


move lambdalib to examples requirements

https://github.com/siliconcompiler/siliconcompiler/actions/runs/8159147682


It's not possible to wait user interaction

The 'timeout' parameter should have a value (for ex. 0) to keep the notification open till the user don't interact

You can use requireInteraction option.
https://developer.mozilla.org/en-US/docs/Web/API/Notification/Notification
And we need skip to call setTimeout when props.timeout <= 0
https://github.com/mobilusoss/react-web-notification/blob/master/src/components/Notification.js#L113

v0.7.0 is available now.
https://github.com/mobilusoss/react-web-notification/releases/tag/v0.7.0

The requireInteraction isn't working properly on firefox in linux. It's getting closed automatically after a few seconds even if the timeout is set to 0. It's working properly on chrome.

It seems to be not a matter of this library, but for Firefox.
https://bugzilla.mozilla.org/show_bug.cgi?id=862395


Avoid escaping semicolon in azure

Fixes Issue #229
Please let me know if I need to do anything additional

Could we get a code review and merge here please?

Could we get a merge/build here? Thanks

Thanks!

Thanks here as well!
May I ask how long until we see this available in pip?

There's some large-ish improvements in flight at the moment that I'm planning to land before cutting another release.
In the meantime you can grab it from pip with a specific sha:
-e git+https://git@github.com/github/octodns.git@<sha>#egg=octodns
`

Great, thank you very much for that syntax.


Remove "+" from addresses tabs

"+" icon should be only on "Contacts" tab. Please make it appear and dissapear smooth, so it won't "Blink" during changing tabs

app version: Testnet 3.1.8
"+" button is available on My active and My expired tabs

checked
app version: Testnet 3.1.9
phone's model: iPhone XR


Value error when used in gradient optimization with equinox latest version

Using diffrax ode integration within a equinox nn training loop throws the error:
ValueError: Closure-converted function called with different dynamic arguments to the example arguments provided: ...
traceback.txt
stderr attached since its too long to paste into the issue.
on equinox version 0.11.6, jax version 0.4.33, and diffrax 0.6.0 while it works perfectly fine on equinox version 0.10.6, jax version 0.4.13, and diffrax 0.4.0.
Here is an MVE that replicates the traceback provided:
import diffrax
import jax.numpy as jnp
import jax
import equinox as eqx
import optax

def odeint(dynamics):
    def integrator(args, ts, y0):
        terms = diffrax.ODETerm(dynamics)
        t0 = ts[0]
        t1 = ts[-1]
        dt0 = ts[1] - ts[0]
        saveat = diffrax.SaveAt(ts=ts)
        sol = diffrax.diffeqsolve(
            terms,
            diffrax.Tsit5(),
            t0, t1, dt0,
            y0,
            args=args,
            saveat=saveat,
        )
        return sol.ys
    return integrator

def dynamics(t, y, args):
    dy = args(t) * y
    return dy

class NN(eqx.Module):
    layer: eqx.nn.Linear

    def __init__(self, key):
        self.layer = eqx.nn.Linear(1, 1, key=key)

    def __call__(self, t):
        t = jnp.array(t).flatten()
        u = self.layer(t)
        return u

def compute_loss(args, integrator, x, ts):
    loss = integrator(args, ts, x)[-1].mean()
    return loss

def make_step(controller, integrator, x, ts, optim, opt_state):
    grads = eqx.filter_grad(compute_loss)(controller, integrator, x, ts)
    updates, opt_state = optim.update(grads, opt_state)
    controller = eqx.apply_updates(controller, updates)
    return controller, opt_state

ts = jnp.arange(0.0, 1, 0.1)
y0 = jnp.array([1])

model_key = jax.random.PRNGKey(0)
neural_net = NN(model_key)
integrator = odeint(dynamics)

optimizer = optax.sgd(learning_rate=3e-3)

opt_state = optimizer.init(eqx.filter(neural_net, eqx.is_inexact_array))

neural_net, opt_state = make_step(
    neural_net,
    integrator,
    y0,
    ts,
    optimizer,
    opt_state,
)

As far as I'm aware there have been no deprecation warnings for any of this code.
Is there a better way to perform this task where an equinox neural network gives the argument for the function to be integrated using diffrax?

This looks like the same issue that in equinox that came from the new weak_type struct in jax 0.4.33 (see https://github.com/patrick-kidger/equinox/issues/854, https://github.com/google/jax/issues/23690).

Thank you so much, freezing the jax version fixes it for now. Hope the upstream issue is fixed soon.

Closing as fixed in Equinox v0.11.7 / https://github.com/patrick-kidger/equinox/pull/856 ! Thanks for the report :)

Closing since fixed and last comment intended to close it


Another token ID to be used - Not sure why keeps fails - Please comment as help is needed

Please note: This repository is being rebuilt to accept the new volume of token additions and modifications. PR merges will be delayed.

I agree to not ping anybody on Discord/Twitter/email about this pull request. Instead I will inquire by posting a new comment in the pull request if needed.

PRs are reviewed in bulk and and can take up to two weeks to be merged.
This repository is managed using an auto merge action. Please ensure your PR has no deleted lines, and it will be merged.
Please provide the following information for your token.
Please include change to the src/tokens/solana.tokenlist.json file in the PR.
DON'T modify any other token on the list.
At minimum each entry should have

Token Address:
Token Name:
Token Symbol:
Logo: (logo should be uploaded under assets/mainnet//*.<png/svg>)
Link to the official homepage of token:
Coingecko ID if available (https://www.coingecko.com/api/documentations/v3#/coins/get_coins__id_):

Auto merge requirements
Your pull request will be automatically merged if the following conditions are met:

Your pull request only adds new tokens to the list. Any modification to existing
tokens will require manual review to prevent unwanted modifications.

Your pull request does not touch unrelated code. In particular, reformatting changes to unrelated
code will cause the auto merge to reject your PR.

Any asset files added correspond to the token address you are adding. Asset files
must be PNG, JPG or SVG files.

Your change is valid JSON and conforms to the schema. If your change failed validation,
read the error message carefully and update your PR accordingly.

No other tokens shares the same name, symbol or address.

For example, this change would be rejected due to unrelated changes:

The bot runs every 60 minutes and bulk-merges all open pull requests to prevent conflicts.
This means that you need to wait up to 60 minutes for your pull request to be merged or reprocessed.

Previous problem was: asset file for unknown token found: "49AAjXJMHqEHKkEhZfHVmtB6YYGRkrmQYZqkCYPGhDS5"
No I what this means. Please advice. It seems there always seems to a new problem. Not sure what I keep doing wrong. I had one successful one work with help, but it was linked a token account wallet, not a newly created token Id. Please advise on what to do.


Fixed "total order" definitions; fixed several typos.

Issue #334: definition 2.16, definition 2.24, and the footnote of section 5.6 provided conflicting definitions for "total order". To address this issue, since definition 2.16 provides what appears to be the intended version of "total order," and the type of order being defined in definition 2.24 (a strict order which is also connected) was already defined under definition 2.22 as a "strict linear order", the following changes were made (including some minor typo fixes and cosmetic changes for section consistency):
Definition 2.24: Removed. The first part of this definition conflicted with definition 2.16, and the second part of this definition was already defined under definition 2.22.
Definition 2.16: Added the label \ollabel{def:linearorder} to this definition and replaced all references to the label \ollabel{def:strictlinearorder}, from definition 2.24, with this label.
Definition 2.22: Added "strict total order" as an appropriate alternative name for "strict linear order" to mirror the approach taken in definition 2.16. However, throughout this section this type of order is still only referred to as a "strict linear order" rather than a "strict total order". This approach was taken to be consistent with the usage of definition 2.16 within the section ("linear order" is preferred over the alternative "total order"). Moved \ollabel{def:strictlinearorder} from the now removed definition 2.24 to this definition.
Proposition 2.25: Modified to refer to "strict linear order" rather than "total" based on the definition fixes. Several typos with "Id_X" were changed to "Id_A". Replaced occurrences of "antisymmetric" with "anti-symmetric" for consistency with the hyphenated spelling of the anti-symmetry definition given in definition 2.6 (but it should be noted that "antisymmetry" also appears elsewhere in the complete text and may be the preferred spelling).
Proposition 2.26: Modified to refer to "strict linear order" rather than "total" based on the definition fixes. The set "X" was changed to "A" for consistency with the rest of the section.
Example 2.27: Removed. This example was incompatible with definition 2.16. Additionally, a similar example, which is consistent with definition 2.16, is already given in example 2.23.
Proposition 2.28: Modified to clarify the use of "total" based on the definition fixes. Label \ollabel{prop:extensionality-totalorders} was changed to \ollabel{prop:extensionality-strictlinearorders} to be consistent with the definition fixes. Added a period to end the proposition sentence. Changed the sentence directly prior to this proposition to properly refer to "strict linear orders" rather than "total orders".
Section 5.6 footnote: This footnote was changed to refer to \ollabel{def:linearorder} (definition 2.16) instead of \ollabel{def:strictlinearorder}. The recalled definition of total order given in this footnote was also missing the anti-symmetric requirement for total orders, as defined in definition 2.16. This requirement is now added. Changed "total ordering" to "total order", for clarity, since the former had not been explicitly defined.

Thanks!


Implement a Parameters type

This PR adds a Patameters type that extends HashMap with helpful parsing methods. This allows parameters to easily be parsed as any type T: FromStr without repeating map(|v| v.parse().ok()) or anything similar.
Part of the purpose of this PR is to see how sane it is to implement this as a separate type, instead of a trait. Thoughts, ideas and feedback are most welcome.
This replaces some occurrences of HashMap with Parameters, which makes it a breaking change.
Closes #55

:umbrella: The latest upstream changes (presumably #58) made this pull request unmergeable. Please resolve the merge conflicts.

I like this. I think I will keep it. What do you think, @tmmcguire?

Sorry I've been out of touch lately, I was out of town and then head-down on a only-semi-related project.
I like it! I'd say it looks exactly like what I was thinking of, but I think it does things better than what I had in mind. Thanks!

Great! I'll just push some other changes I have in store and before merging this on top of them. I have decided to give parts of the request parsing a makeover to preserve more of the original data and it will most likely conflict with this.

It turned out to be worse than expected. I'll merge this first.
@homu r+

:pushpin: Commit 440ffd7 has been approved by Ogeon

:hourglass: Testing commit 440ffd7 with merge eb699b7...

:sunny: Test successful - status


 client-a8606f262e6f.js:2   TypeError: Cannot read properties of undefined (reading 'Memory')

Describe the bug

Link to the blitz that caused the error
https://stackblitz.com/github/remix-run/react-router/tree/main/examples/auth?file=src/App.tsx
Steps to reproduce
[[
](https://reactrouter.com/docs/en/v6/examples/auth)](https://stackblitz.com/github/remix-run/react-router/tree/main/examples/auth?file=src/App.tsx)
Expected behavior

Parity with Local

[ ] I have run the project in my local machine and I could not reproduce the issue.

Screenshots
No response
Platform
Browser name  = Chrome
Full version  = 97.0.4692.99
Major version = 97
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/97.0.4692.99 Safari/537.36 Edg/97.0.1072.76
performance.memory = {
  "totalJSHeapSize": 35804134,
  "usedJSHeapSize": 33730862,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Hash = 5df97b5e61f923919ea927c406a4806b792de8c6

Additional context
No response

@aum-app Thanks for the report.
My guess is that some extension is blocking the usage of WebAssembly. Can you try in a profile without extensions? (incognito mode might or might not work, since additional restrictions might apply to 3rd party scripts and storage).
Alternatively, you could check the value of globalThis.WebAssembly.Memory in the different workers that are spawned.

@aum-app Thanks for the report.
My guess is that some extension is blocking the usage of WebAssembly. Can you try in a profile without extensions? (incognito mode might or might not work, since additional restrictions might apply to 3rd party scripts and storage).
Alternatively, you could check the value of globalThis.WebAssembly.Memory in the different workers that are spawned.

It may cause this problem in Edge browser( I use MacOS so I do not know if it's same in Windows ) that open the feature "Enable security mitigations for a more secure browser experience" even I make "https://stackblitz.com" in the white list.


Add KickoffTime and FinishTime

Feature request
I would like to get these additional sensors/attributes in this new (and better) integration that were also available in the old(er) integration by kloknibor
Kickofftime = now - elapsedTime
FinishTime = now + remainingTime
This could be usefull for multiple types of devices
Offcourse, you can create custom templates but integrating this might be more elegant.
See https://github.com/HomeAssistant-Mods/home-assistant-miele/blob/master/custom_components/miele/sensor.py#L587
What version of the integration are you using?
0.0.20
Anything in the logs that might be useful for us?
No response
Additional information
No response

Please have a look at the discussion here.

Thank you for making this possible!


Add toggle to remove  thumbs up/down in feature selections

Tomek in Discord asked, and it looked simple enough. Until I realized I'd have to add a setting, and put the setting into  a menu, and then update the localization file....
Haven't tested, but the patch is so simple I assume it works as intended. Tomek said they'll let me know.

Update to patch every Recommendation component. Still doesn't work on locked/unavailable Features, but I'm not sure that's important?

I did test this time, and it appears to function correctly to me, except on locked Features.

Oh, is this enough to warrant a new version? I can update both then

Info.json too.
Narria will publish a Nexus release tomorrow or so; so it's better to just push it out now and include it.
Also since I'm not changing the repository.json user won't be aware of the new update, so it wouldn't really matter to those. It's a good way to replace the preview versions we did in the past.


Capabilities Clarification

Not an issue, but I don't see anywhere to just leave a comment (unless I'm blind) so I'll just put this here.
After some testing today, I've found that it's just the owner of the server that the bot does not have permissions to change. In the server settings you can adjust the heirarchy of roles - if you make the bot's role the highest on the list it will have power equal to (or greater than) the other admin roles. But the server owner is 'root', so the bot will never have permissions to change that particular user's nickname. I got around this by creating a second discord account for the sole purpose of transferring ownership of my server to it so my regular account can be manipulated by the bot.

This is the same issue I encountered and lies with Discord permissions. I messaged the Discord Team, if there would be any other possibility than yours, but unfortunately I have never received an answer.


Adding Support Forum?

Hey,
I've been pondering, if I wanted to do some (rust) web development and I ran into some issues, where would I post/ask about them? I feel the rust-lang forums aren't really the right place for it (but maybe that is just me?), so where? reddit/r/rust? stack overflow? If there is a prominent place, should we point people to it? Or if there isn't, should we provide one? Either through reddit (yikes) or maybe a self-hosted flarum instance – community.arewewebyet.org?
@bashyHQ/awwy-curators what do you think?

users.rust-lang.org is totally okay for this. As is /r/rust or stack overflow. Preferably the first two, but we do want to make stack overflow more prominent too (currently it's got less of a community answering).

@Manishearth I see your point and I'd like to promote users.rust-lang.org, too. I guess we should point them to the "help" category?!?

added to the mainpage.


Catch for loops counting up when they should be counting down

Consider a for loop like this:
for i = #a, 1 do
  something(i)
end

This is very likely a mistake, and the intention was to iterate down over some array, ie:
for i = #a, 1, -1 do
  something(i)
end

Is this something luacheck can or should point out?

This sounds helpful, I've made mistakes like that one, should be easy to implement, too, just go through the AST recursively looking for ForNum nodes with limits like that. Just need to figure out the warning code. Probably something in 5xx.

Implemented on master branch.


search bug

Type: Bug
1.After searching for results in the search bar
2.Open file in search results
3.Modify case in file and save
Expected results:
The search bar displays the correct search results
Actual results:
The search results are missing one file that was just saved
VS Code version: Code 1.70.3 (a21a160d630530476218b85db95b0fd2a8cd1230, 2022-08-31T08:03:55.990Z)
OS version: Windows_NT x64 6.1.7601
Modes: Restricted

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6700 CPU @ 3.40GHz (8 x 3408)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
7.88GB (2.19GB free)

Process Argv
C:\Users\hys4600\AppData\Local\Temp\Mxt104\RemoteFiles\283313458_4_1\nginx.conf --crash-reporter-id b61456a3-25c0-4776-b4b6-3a7c2a63bd57

Screen Reader
no

VM
0%

Extensions (5)

Extension
Author (truncated)
Version

vscode-language-pack-zh-hans
MS-
1.70.8170916

python
ms-
2022.16.1

vscode-pylance
ms-
2023.1.10

jupyter
ms-
2022.7.1102252217

volar
Vue
1.6.4

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492:30256859
vslsvsres303:30308271
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscorecescf:30445987
pythondataviewer:30285071
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593:30376534
pythonvs932:30410667
cppdebug:30492333
vsclangdc:30486549
c4g48928:30535728
dsvsc012:30540252
pynewext54:30695312
azure-dev_surveyone:30548225
3biah626:30602489
pyind779:30671433
f6dab269:30613381
pythonsymbol12:30671437
2i9eh265:30646982
6233i204:30730053
azdwalk:30721579
pythonms35:30701012
pythonfmttext:30731395
pythoncmv:30731968
fixshowwlkth:30730052
pythongtdpathcf:30731979

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.78.2. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

Upgraded to the latest version

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.78.2. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

Upgraded to the latest version，The problem still exists

Do you have match case enabled for your query?

Alternatively, do you have smart case enabled?

Do you have match case enabled for your query?
Alternatively, do you have smart case enabled?

regardless of case
1.When opening a file via the search bar
2.Increase characters, then fall back to characters that meet the search criteria and save
3.search result error
release is 1.78.2
This happens only in large projects，I feel that have made a special optimization

Would you be able to send a screen recording of tihs?

Hey @andreamah, this issue might need further attention.
@huang-yf3160, you can help us out by closing this issue if the problem no longer exists, or adding more information.

Hmm, that's weird. Is that a singular file open? Does the regular file find work?

Hmm, that's weird. Is that a singular file open? Does the regular file find work?

This is a regular file from a project.
Is this your optimization mechanism for large projects?

Hey @andreamah, this issue might need further attention.
@huang-yf3160, you can help us out by closing this issue if the problem no longer exists, or adding more information.

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


QA - Site - Product Detail Page - Shipping Section - Edit Address - Page Title needs to Change on Edit Address Page.

QA - Site - Product Detail Page - Shipping Section - Edit Address - Page Title needs to Change on Edit Address Page.

Click on search button and go to search page.
Click on any product and go tot detail page.
Select Place Order tab and Select product Color.
Enter Qty in Qty box.
Select Shipping Address.
Click on Address Book List Button.
Edit any Address.
Observe Page title.

Done


SSAO/threejs

Issue title
Description

the ssao shader  in nunustudio tends to remove -100 fps
but the ssao in the three js examples remove only -10fps
(I'M USING THE NUMBERS AS AN EXPRESSION)
and you should update to the r116 or have u?

Version

Platform

Web Version [X]
Windows [X]
Linux [X]

The SSAO shader is the same as the one from current threejs.
There a slight performance penalty inside of the nunuStudio, but when running the exported app the performance should be exactly the same.
Thanks a lo!

aaah ok thank you :)


[DNM][WIP][do not review yet]Service and Device initialization: automatic priorities

DO NOT REVIEW YET (I just need to see first what CI will find out, my potato PC cannot run the whole test suites)
This is a fix for #76937  after discussing on the RFC #73836
I applied most comments made during these discussion.

Keeping current macros unchanged and able to live with the new ones
Adding new macros: both for device and service (not init, SYS_INIT() will be deprecated as well anyway). Taking the opportunity to reduce the number of macros. Basically merging DT/non-DT based macros, so removing manually introduced device name. This is a legacy thing that needs to disappear. Explicit device name is coming from the time DTS was not used, and during the process of switching to DTS we had to keep it. Now it's not true anymore. The new macros still can instantiate a non-DT device, but it will take the unique token as a name and that's it.
Declaring the explicit dependency resolution file to cmake via cmake functions and not by looking up on pre-determined places (only on the application side, I could not use such cmake function, so there it has to be a specific file, unless I misunderstood something with the build process)
The new service API is extremely minimal as it is meant to just replace SYS_INIT where relevant (like, say: net_init, and not all the places where SYS_INIT should not have been used). BUT: it will grow, there are plenty of rooms for improvements. We could start/stop/pause a service if needed, etc etc...

Added to this I (tried to) switch most device macro calls to new ones. Which make this PR a huge one in files touched and modified lines. There are few corner cases which prevent changes: PTP and ethernet's DSA. These needs to switch to DTS.
Last but not least: brace for impact  /o\

@tbursztyka - it might be worth it to put the PR into draft state until you feel it's ready for review. Thanks for taking the initiative to tackle this issue 👍

@tbursztyka - it might be worth it to put the PR into draft state until you feel it's ready for review.

good point, and I'll remve the DNM label so CI will go through more tests.


feat: Refactor providers

Closes #119

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Finalizing Taweret for BAND v0.3

This is the general pull request to bring in any last-minute changes associated with Taweret for BAND v0.3.
It will start as a draft PR and then when the Taweret team has completed any edits, we can bring mark as ready.

Addressed with #76


Add information about the configuration files to the install docs

Signed-off-by: Daniel J Walsh<EMAIL_ADDRESS>

Attempting to improve install docs to solve problems like https://github.com/projectatomic/buildah/issues/810 where people just come to the github and compile for their native platform and end up being confused when all of the configuration files are not put in place.

Once we have this settled for Buildah will open a similar PR for buildah.
@TomSweeneyRedHat @mheon

https://github.com/LemonIceScream PTAL.  Would this document have helped you when you tried the tools out.

Yes, I believe so. Thanks! I was using the manpages as a reference though. I'll double check things on Sunday / Monday evening.

Ok which man pages did you look at, I will attempt to improve them also.

@TomSweeneyRedHat I went through your comments.  The seccomp.json file is the problem right now that we don't have a nice way to link to it.

@rh-atomic-bot r=TomSweeneyRedHat


Question about architecture with custom VTL (via slot)

Amplify CLI Version
12.1.1
Question
Hi,
We're just starting with AppSync. Is this workflow possible to achieve?
The architecture is for a chat app (with AI on the other end), we're looking to run Lambda functions to get additional data and enrich messages sent to AI API. The reason we're looking at VTL is to avoid adding mutations outside of AppSync "core", e.g. in Lambda functions.
This is the workflow:

Client creates a mutation: The client sends a mutation request to create a new ChatMessage.

AppSync triggers the pipeline resolver: The mutation request is received by AWS AppSync. Given that we defined a custom pipeline resolver, it gets triggered.

Mutation stores the ChatMessage: Before invoking the Lambda function, the mutation request is processed normally, and the new ChatMessage from the user is added to the Chat.

Invoke the Lambda function via the VTL script: The VTL script is executed, which includes a call to invoke our Lambda function. The newly created ChatMessage is passed as an argument to the function.

Lambda function processes the message and sends it to the GPT model: The Lambda function receives the ChatMessage, amends it, and sends it to the GPT model.

Lambda function returns the GPT model's response: The GPT model processes the message and sends a response back to the Lambda function. The function then returns this response.

VTL script processes the Lambda function's response: The VTL script receives the response from the Lambda function. If the function returned a ChatMessage, the script constructs a new ChatMessage with the GPT model's response.

VTL script initiates another mutation request: The script sends another mutation request to create a new ChatMessage in the chat's history. This request is handled by the default AppSync resolvers, which use the auto-generated GraphQL mutations to add the new message to the ChatMessage DynamoDB table.

Client receives the response: Once the mutation is successful, the response is sent back to the client. The client receives the updated Chat object, including the new ChatMessage added by the GPT model.

The alternative is custom Lambda resolver and chained calls of Lambda functions, however that defeats the purpose of AppSync making changes in content.

@OperationalFallacy 👋 thanks for raising this! I'm going to transfer this over to our API repository for better assistance 🙂


Disable SITS if Eirini is enabled

Describe the bug
Right now, SITS is kept enabled even if Eirini is enabled. From the first paragraph in the SITS repository:
This is a set of tests to test the sync functionality between Cloud Controller and Diego.
It forces Cloud Controller and Diego into different states so that the sync process has to reconcile them.

To Reproduce
Enable Eirini and check that SITS is still there.
Expected behavior
SITS should never be added if Eirini is enabled.

I see that CI has some jobs for testing SITS with Eirini. They should be removed.


Enable parameter support

update verify scripts.
update version support for major deps. as kubectl, bcc
update version support for local-cluster-dev

as input parameter is a big change, suggestion as tag with 0.0.0 for action and ... after kepler, kepler-operatror, kepler-helm use 0.0.0 merge this PR. and this feature in kepler-action 0.0.1 release. @rootfs , @marceloamaral

@rootfs , by default, I will create a tag with current https://github.com/sustainable-computing-io/kepler-action on Monday night(GMT 8+)
and create PR in kepler, kepler-helm, kepler-operator.

@rootfs , @marceloamaral I suppose as all dependance are updated, we are able to merge this PR.

@sunya-ch it seems you used this action in your own fork. please update accordingly.

@sunya-ch it seems you used this action in your own fork. please update accordingly.

Will this be applied after just rebasing? Do I need to update anything else?


UX: Remaining balance difference

What UI/UX improvements would you like to be implemented:
• Currently, when user navigates from AMOUNT BALANCE screen to Send to other wallet, balance reflected in Send to other wallet is -0.1 DFI

In AMOUNT BALANCE screen, balance = 0.55496217 DFI
In Send to other wallet screen, balance = 0.45496217 DFI

Suggestion:
• To display real balance, but is user click on MAX, show MAX -0.1
• Add i icon stating why is there a lower balance in other screens except main balance.
• Copy: "0.1 DFI UTXO will be set aside for fees and such, this is your available balance to transact"
Why is this needed:
• Prevent users from being confused on why their balance reflected is different
Which component/screens does this concern?:
• Send to other wallet
• Convert DFI
Anything else we need to know?:

Linked to this ticket - https://github.com/DeFiCh/wallet/issues/736


Example in C#

Any plans to add examples in C#?

C#.NET example now available.

C#.NET example now available.


Replace Image is not updating last modified and by properties

The replace image has been a big deal with a lot of ours users, but we have found that it doesn't update the Last modified time and by in the properties view. see attached screen cap. I uploaded a file, waited 4 minutes, then uploaded another and the properties I highlighted didn't change.

Did this not get added to 1.10.3? @sebastienros @Piedone

Not, unfortunately. The PR was merged into dev, i.e. the upcoming 1.11 version's branch.


Correction for raw affiliation Department of Civil, Architectural and Environmental Engineering, University of Naples Federico II, Italy; Amap, Inra, Ird, Cnrs, Cirad, University of Montpellier, France

Correction needed for raw affiliation Department of Civil, Architectural and Environmental Engineering, University of Naples Federico II, Italy; Amap, Inra, Ird, Cnrs, Cirad, University of Montpellier, France
raw_affiliation_name: Department of Civil, Architectural and Environmental Engineering, University of Naples Federico II, Italy; Amap, Inra, Ird, Cnrs, Cirad, University of Montpellier, France
new_rors: 05290cv24;05kpkpg04;051escj72;020nks034
previous_rors: 05290cv24;05kpkpg04;051escj72
works_examples: W4393809746
contact: 19776b043b612330820f04d12f7df2fd:bc73e1a222fa3cf4c66813389fd1014d4296 @ inrae.fr

This issue was accepted and ingested by the OpenAlex team on 2024-11-18. The new affiliations should be visible within the next 7 days.


Use the numba_dpex_dev image in the CI and document its availability …

…on dockerhub

Looking at the CI failure, it seems we forgot to update the requirement in setup.py to change numba_dppy to numba_dpex:
https://github.com/soda-inria/sklearn-numba-dpex/blob/main/setup.py#L38


initial edits, testing commit

NOT really ready for review quite yet... still working on it.
Please check back later.


Testing the zeromq module in Jupyter Notebooks with Electron 22

Testing the zeromq module in Jupyter Notebooks with Electron 22
Refs: #5206

[ ] macOS @digitarald
[x] linux @chrmarti
[x] windows @joyceerhl

Complexity: 3
Authors: @DonJayamanne  @deepak1556
Create Issue
Create Issue

Requirements

Refer to this extension to get the right VS Code bits https://github.com/microsoft/vscode/issues/177646
Have a python environment installed
Have Python extension installed (preferably latest pre-release)
Have Jupyter pre-release extension installed
On Windows, please ensure you have the C++ runtime files installed, see here https://learn.microsoft.com/en-us/cpp/windows/latest-supported-vc-redist?view=msvc-170

Without this installed the zeromq binary will fail to load.

Steps without Conda

Open a notebook
Add a cell with the following python code

print("Hello World")

Click the Run button
If prompted to install ipykernel, then install it
Verify the cell ran successfully and output is displayed
Very the  messages Exception while attempting zmq : Error: No native build was found for platform is displayed towayds the top of the Jupyter output panel
NOTE: If the above error message is present, then the test has failed

All works for me, except I don't get the output error:

Very the messages Exception while attempting zmq : Error: No native build was found for platform is displayed towayds the top of the Jupyter output panel

What I get:
info 17:58:42.985: Starting Jupyter Session startUsingPythonInterpreter, .jvsc74a57bd07e1998ff7f8aa20ada591c520b972326324e5ea05489af9e422744c7c09f6dad./usr/local/bin/python3.10./usr/local/bin/python3.10.-m#ipykernel_launcher (Python Path: , EnvType: Unknown, EnvName: '', Version: 3.10.10 (main, Feb 13 2023, 03:30:25) [Clang 14.0.0 (clang-14<IP_ADDRESS>)]) for 'Untitled-1.ipynb' (disableUI=false)
warn 17:58:43.035: Failed to get activated env vars for /usr/local/bin/python3.10 in 49ms
info 17:58:43.037: Process Execution: > /usr/local/bin/python3.10 -c "import ipykernel; print(ipykernel.__version__); print("5dc3a68c-e34e-4080-9c3e-2a532b2ccb4d"); print(ipykernel.__file__)"
> /usr/local/bin/python3.10 -c "import ipykernel; print(ipykernel.__version__); print("5dc3a68c-e34e-4080-9c3e-2a532b2ccb4d"); print(ipykernel.__file__)"
warn 17:58:43.038: Failed to get activated env vars for /usr/local/bin/python3.10 in 39ms
warn 17:58:43.039: Failed to get activated env vars for /usr/local/bin/python3.10 in 39ms
info 17:58:43.041: Process Execution: > /usr/local/bin/python3.10 -m ipykernel_launcher --ip=<IP_ADDRESS> --stdin=9016 --control=9014 --hb=9013 --Session.signature_scheme="hmac-sha256" --Session.key=b"a5ff54e2-6eed-44dc-a57b-890f0fb1ffad" --shell=9015 --transport="tcp" --iopub=9017 --f=/Users/digitarald/Library/Jupyter/runtime/kernel-v2-16139zJD3UCAbuswR.json
> /usr/local/bin/python3.10 -m ipykernel_launcher --ip=<IP_ADDRESS> --stdin=9016 --control=9014 --hb=9013 --Session.signature_scheme="hmac-sha256" --Session.key=b"a5ff54e2-6eed-44dc-a57b-890f0fb1ffad" --shell=9015 --transport="tcp" --iopub=9017 --f=/Users/digitarald/Library/Jupyter/runtime/kernel-v2-16139zJD3UCAbuswR.json
info 17:58:43.041: Process Execution: cwd: /
cwd: /
info 17:58:43.277: ipykernel version & path 6.16.0, ~/Library/Python/3.10/lib/python/site-packages/ipykernel/__init__.py for /usr/local/bin/python3.10
info 17:58:43.870: Started Kernel Python 3.10.10 (pid: 17544)


cue-api-python: add bindings to libcue functions

write the tests in parallel with the addition of functionality.
ensure everything is idiomatic (Pythonic).
add Python-level documentation to the API.

This is done, so closing.
GitHub automagic didn't pick it up because we used by mistake Fixes #3073 in the commit message, but the Python code is in a different repository than this issue tracker. We should have explicitly said Fixes cue-lang/cue#3073.


Shinyproxy authentication by supabase.io

Hello everyone,
I'm quite new to the programming world but fascinated by the whole shiny framework. At this moment I am experimenting with shiny apps in different ways. I like to use tools developed by startups instead of using the established version. Therefore, I have the following question: is it possible to integrate the authentication module of supabase with shinyproxy. And if the answer is yes, which settings do I need to adapt?
Thank you all in advance!

Hi
I looked into the documentation of supabase, and I don't see any way of integrating it with ShinyProxy. I think the main idea of supabase is to have authentication in apps built using supabase and not external apps. Therefore this requires additional development effort before we can support this. However, looking at the system it won't be very easy to support it in ShinyProxy.

Thank you for your point of view. I will look for another authentication system.


Importing Vue components

I am trying to build an application using the VuePlugin in Javalin.
I was trying to use the Vuealidate component in my application.
I imported the module for it from webjars and imported the javascript into my layout  html.
gradle:
    compile 'org.webjars.npm:github-com-vuelidate-vuelidate:0.7.4'

layout.html
   <script src="/webjars/github-com-vuelidate-vuelidate/0.7.4/vuelidate.min.js"></script>
In the script block of my vue template I tried to import the javascript modules into my Vue component using
import { validationMixin } from "vuelidate";
import { required, minLength } from "vuelidate/lib/validators";

when I use the import I get the error:
Cannot use import statement outside a module
The only way I was able to get it to work was to add the validators.js file into layout.html
   <script src="/webjars/github-com-vuelidate-vuelidate/0.7.4/validators.min.js"></script>
and then create variables for the functions I wanted to use in my Vue components script section:
const required = validators.required;
const minLength = validators.minLength;

Is there anyway to use the import statements with the Vueplugin?

Is there anyway to use the import statements with the Vueplugin?

You could use native JS imports, but the Vue plugin is not intended for that type of usage. If you want to do imports, I recommend setting up something like Parcel.


Magus metamagic in Kingmaker

I have a Magus with Extend Spell, so I can use Shield, Blur and Mirror Image for extended periods of time. The recording function works fine when I use a metamagic modified spell but then, when I repeat the queue, it only casts the base spell, not the modified one.
post link

One of the reasons was a UI update forcing the buff context

Need to check the metamagic application for normal casters as well


🌐 Provide preview for the German translation progress

This PR is my work branch. As it is in sync with all my other PRs here, and mostly with the PRs by fhabers21 and Johannes Jungbluth, it provides a preview of the current state of the German translation, which helps reviewers and other interested people.
That's the only purpose of this PR. It is not intended to be merged.    Nope, I won't recreate. Will just start a new order here.

Edit: no, I will just close this. No preview anymore.


Chunked may not be final in transfer encoding list

Hello. Please look in RFC 7230 - 3.3.3 Message Body Length
If a Transfer-Encoding header field is present and the chunked
transfer coding (Section 4.1) is the final encoding, the message
body length is determined by reading and decoding the chunked
data until the transfer coding indicates the data is complete.

If a Transfer-Encoding header field is present in a response and
the chunked transfer coding is not the final encoding, the
message body length is determined by reading the connection until
it is closed by the server.  If a Transfer-Encoding header field
is present in a request and the chunked transfer coding is not
the final encoding, the message body length cannot be determined
reliably; the server MUST respond with the 400 (Bad Request)
status code and then close the connection.

As I can understand it means that both Transfer-Encoding: gzip, chunked and Transfer-Encoding: chunked, gzip are possible. chunked, gzip may look meaningless but it is possible and its behaviour is well described.
the message body length is determined by reading the connection until it is closed by the server means that we need to decode gzip first and than chunked.
But it means that all nginx, mongrel, nodejs (and its descendants) http parsers are broken by design. chunked encoder/decoder should be shipped outside http parser as separate library. It should be used by end-user together with gzip, brotli, zstd, compress encoders/decoders.
Another solution is to develop fat parser with gzip, brotli, zstd, compress, etc dependencies.
This issue can't be fixed. I want just to inform everyone using http parser.

I don't understand what you think the issue is.

chunked, gzip may look meaningless but it is possible and its behaviour is well described.

Yes, its behaviour is described:

If a Transfer-Encoding header field is present in a request and the chunked transfer coding is not the final encoding, the message body length cannot be determined reliably; the server MUST respond with the 400 (Bad Request) status code and then close the connection.

@sam-github, this exception is assigned for the message body length cannot be determined reliably + chunked transfer coding is not the final encoding. It means that you can't receive HTTP header with Transfer-Encoding: chunked, gzip and without Content-Length.
But if Content-Length exists than Transfer-Encoding: chunked, gzip is valid header.

From the very same 3.3.3:
   If a message is received with both a Transfer-Encoding and a
   Content-Length header field, the Transfer-Encoding overrides the
   Content-Length.  Such a message might indicate an attempt to
   perform request smuggling (Section 9.5) or response splitting
   (Section 9.4) and **ought to be handled as an error**.

In other words, Transfer-Encoding cannot be present together with Content-Length.

It must be said that the presence of both headers leads to quite practical request smuggling attacks as was demonstrated by security researchers.

chunked encoder/decoder should be shipped outside http parser as separate library. It should be used by end-user together with gzip, brotli, zstd, compress encoders/decoders.

This simply can't work as the presence of chunked TE alters protocol in a way that cannot be offloaded to a 3rd party module.

Hello @indutny, I can't agree with you, please read carefully.

If a Transfer-Encoding header field is present in a response and the chunked transfer coding is not the final encoding, the message body length is determined by reading the connection until it is closed by the server.

If a Transfer-Encoding header field is present in a request and the chunked transfer coding is not the final encoding, the message body length cannot be determined reliably; the server MUST respond with the 400 (Bad Request) status code and then close the connection.

If a message is received with both a Transfer-Encoding and a Content-Length header field, the Transfer-Encoding overrides the Content-Length.  Such a message might indicate an attempt to perform request smuggling.

3 exception is related to request only, response just reads as much as possible data from server and has no possible security exceptions.
Please read RFC 7230 - 3.3.1 Transfer-Encoding for clarification too.

If any transfer coding other than chunked is applied to a request payload body, the sender MUST apply chunked as the final transfer coding to ensure that the message is properly framed.

If any transfer coding other than chunked is applied to a response payload body, the sender MUST either apply chunked as the final transfer coding or terminate the message by closing the connection.

This simply can't work as the presence of chunked TE alters protocol in a way that cannot be offloaded to a 3rd party module.

I see no collisions. Transfer-Encoding: chunked, gzip means just gzip after chunked. User are reading data, than applies gzip and than chunked.

I was wrong about Content-Length: it can't be used to identify the length of body if Transfer-Encoding was used, this usage is forbidden. But responses without Content-Length can have Transfer-Encoding with chunked in any position with other encodings.

Well, FWIW it is allowed for responses: https://github.com/nodejs/http-parser/blob/5c5b3ac62662736de9e71640a8dc16da45b32503/http_parser.c#L1904-L1913

Parser doesn't depend on gzip, brotli, etc it can't decode chunked for user, because there is another encoding on the top of chunked. User have to handle chunked by himself anyway.


[DL-716] Raise exception if dataset too large to delete

🚀 🚀 Pull Request
Checklist:

[ ]  My code follows the style guidelines of this project and the Contributing document
[ ]  I have commented my code, particularly in hard-to-understand areas
[ ]  I have kept the coverage-rate up
[ ]  I have performed a self-review of my own code and resolved any problems
[ ]  I have checked to ensure there aren't any other open Pull Requests for the same change
[ ]  I have described and made corresponding changes to the relevant documentation
[ ]  New and existing unit tests pass locally with my changes

Changes

@farizrahman4u the test fails is pretty strange
>           raise TransformError(e) from e
E           hub.util.exceptions.TransformError: cannot reshape array of size 100 into shape (98,1)