Using 'unityLicensingServer' in game-ci/unity-builder@v2 reports a warning and error, refusing to build

Bug description
Your docs say that you now support 'unityLicensingServer' as a parameter to unity-builder.
We tried to use this parameter for doing an enterprise build, instead of building using a professional license w/ serial+username+password. The parameter is reported as not supported, with a warning message saying Unexpected input(s) 'unityLicensingServer'. Then we get an error message saying Missing Unity License File and no Serial was found.
How to reproduce
Use the unityLicensingServer parameter with game-ci/unity-builder@v2
Expected behavior
I expect to be using the Unity license server.
Additional details
Our code, which seems to match your example in the docs:
      # Build the project using GameCI (https://game.ci/docs/github/builder)
      - name: Build Unity project
        uses: game-ci/unity-builder@v2
        with:
          targetPlatform: ${{ matrix.targetPlatform }}
          unityLicensingServer: https://our-licensing.server-name.com:8080
          versioning: None

Warnings and errors in the log:
Warning: Unexpected input(s) 'unityLicensingServer', valid inputs are ['targetPlatform', 'unityVersion', 'customImage', 'projectPath', 'buildName', 'buildsPath', 'buildMethod', 'customParameters', 'versioning', 'version', 'androidVersionCode', 'androidAppBundle', 'androidKeystoreName', 'androidKeystoreBase64', 'androidKeystorePass', 'androidKeyaliasName', 'androidKeyaliasPass', 'androidTargetSdkVersion', 'sshAgent', 'gitPrivateToken', 'chownFilesTo', 'allowDirtyBuild', 'postBuildSteps', 'preBuildSteps', 'customJobHooks', 'customJob', 'awsBaseStackName', 'cloudRunnerCluster', 'cloudRunnerCpu', 'cloudRunnerMemory', 'cachePushOverrideCommand', 'cachePullOverrideCommand', 'readInputFromOverrideList', 'readInputOverrideCommand', 'kubeConfig', 'kubeVolume', 'kubeStorageClass', 'kubeVolumeSize', 'cacheKey', 'checkDependencyHealthOverride', 'startDependenciesOverride']
 [...]
Error: Missing Unity License File and no Serial was found. If this
                          is a personal license, make sure to follow the activation
                          steps and set the UNITY_LICENSE GitHub secret or enter a Unity
                          serial number inside the UNITY_SERIAL GitHub secret.

The warning is just because nobody added the input parameter to the action.yml file in the root.
The error should be accurate. I don't believe this is a bug.

Hi, thanks for the quick reply!
I looked at the code, and it does seem to me like the way to use this feature to add the unityLicensingServer parameter.
I did add:
unityLicensingServer: "https://unity-license.***.com:8080"
But it seems to be failing here, which I guess it should not, since I did set that input?
    if (Input.unityLicensingServer === '') {
      if (!process.env.UNITY_SERIAL && Input.githubInputEnabled) {
        // No serial was present, so it is a personal license that we need to convert
        if (!process.env.UNITY_LICENSE) {
          throw new Error(`Missing Unity License File and no Serial was found. If this
                          is a personal license, make sure to follow the activation
                          steps and set the UNITY_LICENSE GitHub secret or enter a Unity
                          serial number inside the UNITY_SERIAL GitHub secret.`);

It's a bit difficult to debug this locally until we finish our CLI (roadmap). Maybe you can see the logs in the licensing server or add a proxy to see what the request looks like (if it's connecting to the server at all).


FR: Nozzle Temp alert

Hi,
I can turn my printer on/off remotely. I often leave my printer to cool down whilst I'm elsewhere. I don't like to turn the printer off until the nozzle has cooled.
Therefore a notification similar to the bed temperature but for the nozzle would be fantastic.
I could leave the printer to cool, go away and then shut the power down when it's cooled.
Thanks for a great plugin.

Great idea. Scheduled for next release.

Thanks!


CrowdStrike OAuth API Identifier Activity Analysis

Please check all that apply:

[x] New playbook
[ ] Bugfix
[ ] Feature add
[ ] Code style update (formatting, renaming)
[ ] Documentation
[ ] Other (please describe):

Release Notes
Replace the following list with release notes that describe the high level components of the PR:

[x]  playbooks/CrowdStrike_OAuth_API_Identifier_Activity_Analysis.json
[x]  new file:   playbooks/CrowdStrike_OAuth_API_Identifier_Activity_Analysis.png
[x]  new file:   playbooks/CrowdStrike_OAuth_API_Identifier_Activity_Analysis.py
[x]  new file:   playbooks/CrowdStrike_OAuth_API_Identifier_Activity_Analysis.yml

Playbook quality checklist
Please check if your PR fulfills the following requirements.
Requirements for Settings

[x] Playbook name is A-Z in Title case with underscores between words. (e.g. MS_Graph_Search_and_Purge)
[x] Category in Title case with spaces between words (e.g. Identifier Reputation Analysis)
[x] Description is free of grammatical errors and describe what the playbook does.
[x] Notes list any setup required on the third-party API as well as intended areas for customization.
[x] Label is set to '*'

Requirements for all playbooks

[x] Playbook block count not greater than 15 (not including Start and End blocks).
[x] No more than 3 branching paths.
[ ] If referencing a custom list, Notes document what the expected values are in that custom list.

Requirements for all playbook blocks

[x] All blocks have a custom name no more than 4 words, all lowercase, and separated by space (e.g. close workbook task)
[ ] All blocks that support a Notes Tooltip have it filled out. Must be grammatically correct and describes the intended purpose of that block.
[ ] Where custom code is used, block notes indicate presence of custom code (e.g. "This block uses custom code")
[x] No block is disabled by custom code
[x] Custom code is documented with notes

Requirements for specific blocks
Action

[x] Use apps available on Splunkbase
[x] Use asset names that are the app name, all lowercase separated by underscores (e.g. Azure AD Graph becomes azure_ad_graph)

Utility

[x] Block is using community version

Playbook

[ ] Block is using local version

Requirements for specific playbooks
Input playbooks

[x] Start blocks use ocsf variable names and a minimum of one data type per variable name (e.g. device (type: host name))
[x] Has at least one category tag (e.g. reputation)
[x] Playbook has a tag for each vendor app used

Other considerations (PR type specific)

[x] If new playbook, there is a screenshot ending in .png with the same name as the playbook .json
[x] Playbook major minor version matches repo (e.g. 5.5 != 6.0)
[x] PR contains both .py and .json

Thanks for contributing!

Filter_3 and Filter_2 should be renamed to something like "file results filter" and "domain results filter."

Otherwise, looks great. After that change you are good to submit it and also submit to phantomcyber.


[CWS] Index workload per cgroup

What does this PR do?
Refactor the cgroup resolver to index workloads per cgroup ID instead of container ID.
Motivation
Make the cgroup resolver handle cgroups not tied to a container.
Describe how you validated your changes

Possible Drawbacks / Trade-offs
Additional Notes

/merge


Getting build error while creating package.

I just cloned the kafka-manager and started to build.  While packaing I got this error.
Is it something wrong
[info] LESS compiling on 1 source(s)
[info] Compiling 122 Scala sources and 2 Java sources to /Users/anand.ranganathan/apache/kafka-manager-9/kafka-manager/target/scala-2.11/classes...
[info] Main Scala API documentation to /Users/anand.ranganathan/apache/kafka-manager-9/kafka-manager/target/scala-2.11/api...
[info] Packaging /Users/anand.ranganathan/apache/kafka-manager-9/kafka-manager/target/scala-2.11/kafka-manager_2.11-<IP_ADDRESS>-web-assets.jar ...
[info] Done packaging.
[error] /Users/anand.ranganathan/apache/kafka-manager-9/kafka-manager/app/controllers/Topic.scala:209: type mismatch;
[error]  found   : Option[Iterable[(String, kafka.manager.model.ActorModel.ConsumerType)]]
[error]  required: Option[Iterable[String]]
[error]       Ok(views.html.topic.topicDeleteConfirm(clusterName,topic,errorOrTopicIdentity,errorOrConsumerList))
[error] /Users/anand.ranganathan/apache/kafka-manager-9/kafka-manager/app/controllers/Topic.scala:209: type mismatch;
[error]  found   : Option[Iterable[(String, kafka.manager.model.ActorModel.ConsumerType)]]
[error]  required: Option[Iterable[String]]
[error]       Ok(views.html.topic.topicDeleteConfirm(clusterName,topic,errorOrTopicIdentity,errorOrConsumerList))
[error]                                                                                ^
[error]                                                                                ^
[error] one error found
[info] No documentation generated with unsuccessful compiler run
[error] one error found
[error] (compile:doc) Scaladoc generation failed
[error] (compile:compileIncremental) Compilation failed
[error] Total time: 17 s, completed Mar 24, 2016 1:00:53 PM

Got same error too!

+1

+1

[info] Done updating.
[warn] There may be incompatibilities among your library dependencies.
[warn] Here are some of the libraries that were evicted:
[warn]  * org.webjars:jquery:1.11.1 -> 2.1.4
[warn] Run 'evicted' to see detailed eviction warnings
[info] Wrote /tmp/kafka-manager/target/scala-2.11/kafka-manager_2.11-<IP_ADDRESS>.pom
Warning: node.js detection failed, sbt will use the Rhino based Trireme JavaScript engine instead to run JavaScript assets compilation, which in some cases may be orders of magnitude slower than using node.js.
[info] Main Scala API documentation to /tmp/kafka-manager/target/scala-2.11/api...
[error] /tmp/kafka-manager/app/controllers/Topic.scala:209: type mismatch;
[error]  found   : Option[Iterable[(String, kafka.manager.model.ActorModel.ConsumerType)]]
[error]  required: Option[Iterable[String]]
[error]       Ok(views.html.topic.topicDeleteConfirm(clusterName,topic,errorOrTopicIdentity,errorOrConsumerList))
[error]                                                                                ^
[info] No documentation generated with unsuccessful compiler run
[error] one error found
[info] Compiling 122 Scala sources and 2 Java sources to /tmp/kafka-manager/target/scala-2.11/classes...
[error] /tmp/kafka-manager/app/controllers/Topic.scala:209: type mismatch;
[error]  found   : Option[Iterable[(String, kafka.manager.model.ActorModel.ConsumerType)]]
[error]  required: Option[Iterable[String]]
[error]       Ok(views.html.topic.topicDeleteConfirm(clusterName,topic,errorOrTopicIdentity,errorOrConsumerList))
[error]                                                                                ^
[error] one error found
[info] LESS compiling on 1 source(s)
[info] Packaging /tmp/kafka-manager/target/scala-2.11/kafka-manager_2.11-<IP_ADDRESS>-web-assets.jar ...
[info] Done packaging.
[error] (compile:doc) Scaladoc generation failed
[error] (compile:compileIncremental) Compilation failed
[error] Total time: 98 s, completed Mar 25, 2016 10:51:25 AM

Same, +1

Fixed and pushed to master.

Will the fix get released? I'd like to pin this.


Add Filter Tag Command

Closes #224
Changes made

Add abstract class for FilterCommand
Add FilterTagCommand and FilterTagCommandParser
Update CommandType, Messages, FoodRemParser accordingly
Add test cases for FilterTagCommand, FilterTagCommandParser
Update test cases for FoodRemParser, HelpCommand, HelpCommandParser

Codecov Report

Merging #302 (6a1b383) into master (5717dda) will decrease coverage by 2.98%.
The diff coverage is 41.39%.

@@             Coverage Diff              @@
##             master     #302      +/-   ##
============================================
- Coverage     78.06%   75.07%   -2.99%
- Complexity      691      703      +12
============================================
  Files           117      120       +3
  Lines          2074     2203     +129
  Branches        223      241      +18
============================================
+ Hits           1619     1654      +35
- Misses          367      453      +86
- Partials         88       96       +8

Impacted Files
Coverage Δ

src/main/java/seedu/foodrem/MainApp.java
0.00% <0.00%> (ø)

...main/java/seedu/foodrem/commons/core/Messages.java
0.00% <ø> (ø)

...in/java/seedu/foodrem/model/tag/UniqueTagList.java
55.81% <0.00%> (ø)

.../java/seedu/foodrem/model/util/SampleDataUtil.java
0.00% <0.00%> (ø)

src/main/java/seedu/foodrem/ui/MainWindow.java
0.00% <0.00%> (ø)

src/main/java/seedu/foodrem/ui/UiManager.java
0.00% <0.00%> (ø)

src/main/java/seedu/foodrem/model/item/Item.java
73.91% <71.42%> (-9.43%)
:arrow_down:

src/main/java/seedu/foodrem/model/FoodRem.java
87.50% <76.47%> (-5.84%)
:arrow_down:

...rser/itemcommandparser/FilterTagCommandParser.java
90.00% <90.00%> (ø)

.../java/seedu/foodrem/commons/enums/CommandType.java
100.00% <100.00%> (ø)

... and 9 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

I suspect this will have merge conflicts after #286

I suspect this will have merge conflicts after #286

Yup, the GUI still has some ways to go, so we'll need to merge that first and block this, unfortunately.


Make copied expression URL easy to read & edit

When a link is copied for an expression like 0 0 12 * */2 Mon, the link reads as follows:
https://ncrontab.swimburger.net/?expression=0%200%2012%20*%20*%2F2%20Mon

The %20 makes it unreadable or difficult to hand-edit. Using application/x-www-form-urlencoded instead allows use of plus (+) to replace space; that combined with the fact that / and * don't need to be escaped in the query data per Section 3.4 of RFC 3986, the expression in the URL becomes super friendly to read and edit:
https://ncrontab.swimburger.net/?expression=0+0+12+*+*/2+Mon

This is a great suggestion. I'm not sure why the / is being escaped by QueryHelpers.AddQueryString.
I'll start with replacing spaces with + and then later see if I can work around '/'

I'm not sure why the / is being escaped by QueryHelpers.AddQueryString.

I think it's because they take an intersection of what's safe for the various parts of an IRI.

I stopped using QueryHelpers.AddQueryString because it would encode '+' and '/' too.
The tool uses '+' as a separator when copying the URL now.


Banshee simulator error:  Unsupported instruction

I want to simulate sw/apps/blas/gemm/build/gemm.elf with src/banshee.yaml by banshee, but encountered the following errors:
ERROR banshee::tran > Unsupported instruction 0x800027b8: <illegal 0xf60b80d3>
ERROR banshee::tran > Unsupported instruction 0x800027c0: <illegal 0xe6010bd3>
ERROR banshee::tran > Unsupported instruction 0x800027e0: <illegal 0xf60c01d3>
ERROR banshee::tran > Unsupported instruction 0x800027e4: <illegal 0xf6078253>
ERROR banshee::tran > Unsupported instruction 0x800027e8: <illegal 0xf60b82d3>
ERROR banshee::tran > Unsupported instruction 0x800027f4: <illegal 0xe6028bd3>
......
All these unsupported instructions are fmv.b.x and fmv.x.b. How to resolve these errors?
By the way,  banshee's README suggests using .bin files as input, but the snitch_cluster tutorial uses .elf files as input for banshee. What's the difference? And how to run banshee simulation for sw testcases in snitch_cluster correctly? I tried using llvm_objcopy to convert the .elf file into a .bin file, but the generated bin file could not be properly received by banshee.

Please post Banshee-related issues in the Banshee repository. https://github.com/pulp-platform/banshee


Not working as of 03/07/2022

Script is Active but not working on the new Aliexpress web design.

Yes, but now is very hard.
They are blocking and loading the content dynamic, in my attempts i'm unable to detect the elements...
New design is awful and less usable

I too have noticed this.
Although I don't think the new design is awful (the previous design already was pretty bad, the one before that was much better), I agree that it fails to deliver in many aspects.


Add files via upload

School URL: http://scoala79.ro/
School org. chart: http://scoala79.ro/wp-content/uploads/2015/05/Organigrama-05-2015.jpg (TIC - middle right orange box - means IT)
e-mail domain: is actually name.surname@official_domain -- the e-mail domain is the same as the school domain.

@iedutu Pull request merged. Thank you!


Nesting support

Thanks for a great project.
Do you plan to support nesting nodes?
There are two types of nesting:

When we need to find out and display which block (group) the nodes belong to.

When we need to be able to connect groups in each other

Hi and thank you for using Vue Flow :)
I am not sure I am fully understanding what you're trying to accomplish.

You're trying to assign a "group" to a node and then order nodes by their group?

Can you possibly draw what you're trying to connect? So I can see what you specifically mean. Is it supposed to be a connection from Node 4 to Node 4b?

But:
I'll have to think about this suggestion and see what's already possible and how a "nesting" feature would even look like.

@vladsvd
I've thought about this and I don't think "nesting" is something that's in scope of the library.
You can however very well create a feature to create a sort of "nesting" of your nodes.
I've created an Example and I'm guessing you're looking for something like that.
Connecting Nodes that are inside of other nodes can become problematic due to the z-index. You'll have to play around with that yourself, for example set a z-index of 2 for "group" nodes and a z-index of 3 for simple nodes.

<template>
  <VueFlow v-model="elements" :node-types="['group-a', 'group-b']" @connect="onConnect">
    <template #node-group-a="aProps">
      <GroupNode v-bind="aProps" />
    </template>
    <template #node-group-b="bProps">
      <GroupNode v-bind="bProps" />
    </template>
  </VueFlow>
</template>

// GroupNode.vue

<template>

    <Handle type="target" :position="Position.Top" />
    Group {{ data.label }}
    <Handle type="source" :position="Position.Bottom" />

</template>

A group node basically checks whether a dragged node is inside of it's boundaries and if it is, it's set props.data.group to it's id, i.e. you assign a group to a node.
What is missing in the library currently is an export of the getNodesInside utility, which I will provide with the next version for exactly this use-case.
So my conclusion is that it's up to you to play around with existing features to create the nesting feature you're after as I do believe the scope of this library is to simple render the elements to the view, not actually provide logic to group / sort etc.
It's a similar but not identical case to what is pictured in the Layout example.
I hope this somewhat answers your question, if not, please feel free to ask! :)

Thanks for the quick response.
Yes, that's about what I need.
Since this project is a port from the react-flow project, this innovation will soon be there.
There is a discussion at this link. I would very much like this functionality to appear.
https://github.com/wbkd/react-flow/discussions/1024#discussioncomment-1512393
The main idea is to have descendants in the structure.
const initialNodes = [
  { id: '1', data: { label: 'Node 1' }, position: { x: 50, y: 0 } },
  { id: '2', data: { label: 'Group node 2' }, position: { x: 50, y: 100 }, style: { width: 400, height: 200 } },
  // this position is relative to the position of the parent node '2'
  { id: '2a', data: { label: 'Child node 2a' }, position: { x: 0, y: 0 }, parentNode: '2' }
];

https://github.com/wbkd/react-flow/pull/1555

Well I didn't see that PR tbh as I didn't check back on react-flow for a while 😅
But thankfully you reminded me and I can for sure replicate the functionality that is being offered in their major release. At least I expect to be able to, so with that said I will have to check out the newest changes and work on an implementation that'll work for Vue :)
I might be able to get it done this weekend, depending on the amount of work of course.

Just a heads up!
Nesting will works pretty much now - i've decided to use the children api instead of the parentNode one.
I'll explain it in detail in the docs.
But if you're using Vue Flow currently and want to use the nesting feature be prepared for possible breaking changes between versions

Just a heads up! Nesting works pretty much now - i've decided to use the children api instead of the parentNode one. I'll explain it in detail in the docs.
But if you're using Vue Flow currently and want to use the nesting feature be prepared for possible breaking changes between versions

Ничего страшного. Я сам в поисках решения своих идей.

Another update:
I'm currently in the process of cleaning up and checking for remaining minor bugs.
Expect a stable release some time this week if everything goes well :)

@vladsvd
I'm going to create a pre-release for v0.4.0 (v0.4.0-x) today, which includes the nesting-feature, as it will take a while to re-write the documentation to match the current implementation.
You can check the examples to get an idea on how to use it properly, though I hope to get the documentation done by new years.
If you do check it out, let me know if you have any issues/problems that need attention :)


[BUG] - Byron address attributes required on MC4

Internal
Summary
Mainnet candidates are supposed to mimic mainnet. One of the behaviors for mainnet is that the protocol magic is NOT required inside Byron address attributes (as opposed to a testnet where it IS required)
However, MC4 contains the two following issues:

Cardano-CLI signing using a bootstrap witness adds the protocol magic to the witness
Cardano-Node will reject any transaction where the bootstrap witness does NOT contain the protocol magic

Steps to reproduce
Key file (payment_byron.skey)
{
    "type": "PaymentSigningKeyByron_ed25519_bip32",
    "description": "Payment Signing Key",
    "cborHex": "5880b8f2bece9bdfe2b0282f5bad705562ac996efb6af96b648f4445ec44f47ad95c10e3d72f26ed075422a36ed8585c745a0e1150bcceba2357d058636991f38a37cf76399a210de8720e9fa894e45e41e29ab525e30bc402801c076250d1585bcd91e248de509c070d812ab2fda57860ac876bc489192c1ef4ce253c197ee219a4"
}

Transaction file (tx.raw)
{
    "type": "TxUnsignedShelley",
    "description": "",
    "cborHex": "82a4008182582062c337c54795149010a805ee42eb8554d8db5c327d9d88e0b83529fe42fde57b00018182581d614375a6c397e9c62bf574b9f37bbe25ff4e4a1ac09982117f296c7a521a00895440021a000f4240031a00037460f6"
}

Command
cardano-cli shelley transaction sign --tx-body-file tx.raw --signing-key-file payment_byron.skey --testnet-magic 42 --out-file tx.signed

Expected behavior

Bootstrap witness on a mainnet candidate should NOT require a protocol magic to be accepted by the node
Bootstrap witness generated by cardano-cli should NOT contain a protocol magic

System info (please complete the following information):

1.18.0

Screenshots and attachments
You can see from the CBOR that the Cardano-CLI generated transaction contains the prtocol magic "42" inside it (whereas it's expected not to contain a protocol magic at all)

Additional context
https://github.com/input-output-hk/cardano-node/blob/master/cardano-api/src/Cardano/Api/Typed.hs#L1325-L1333
Matthias B.  22 hours ago
This seems wrong as it'll insert a protocol magic attribute if an explicit magic is given to the command-line. We had a similar issue in cardano-wallet (treating staging networks as if they were testnets) and resorted to having three modes:
--mainnet
--testnet MAGIC
--staging MAGIC
where --staging implies RequiresNoMagic

Use --mainnet for the signing step.

I've been told this was fixed in Cardano-Rest 2.1.1 (see changelogs here)


Authentication improvements

[x] create JWTs instead of static token (simply use { client: 'tomster-player' } as token payload)
[x] remove account_id from token response
[ ] validate token on requests to albums, songs and comments (validate that client from JWT payload is tomster-player)

Somehow not able to split the string returned from get_req_header("authorization") to "Bearer" and . Returns a pattern matching error.

@ganeshbsub: I fixed it: b5c76d1f8df2ead4e6fe5393c5c2e964fa589ecf
Just need to make sure that any exception that is thrown in that plug results in a 401 response.


Adds Ganja Brownies

About the PR

Adds recipes for basic and "special" brownies. You can make them by the batch of 6 and cut into individual pieces.
Special brownies require cannabis butter to make which also comes with it's own recipe.
In-line with this PR from yesterday, producing canna-butter is a more efficient method than drying/grinding leaves.  See breakdown:

raw leaf=15thc
6leaves(90) + butter = 1 cannabutter= 82thc(10% loss)
2 CB(164) = 6 brownies = 150thc (~9% loss)
1 brownie = 25thc (17% overall loss)

Brownies are also stronger than blunts/joint

Media

[x] I have added screenshots/videos to this PR showcasing its changes in-game, or this PR does not require an in-game showcase

Changelog
:cl: Velcroboy

add: Chef Velcroboy has been in the kitchen messin around with the botanicals. New recipes have been developed for some extra "special" brownies. Careful with these...

the butter sprite does not look like butter imo

the butter sprite does not look like butter imo

It's the same butter we've always had. Just with a green tint. It could always be re-sprited but I didn't see that being w/in the scope of this PR.

resolve conflicts then good to merge @ChilbroBaggins

Conflicts resolved


Rethink2 Chap 16 - Section 16.1

Did this as an exercise for myself as I am learning. Noticed nobody had submitted Chap 16 yet, so this is the first part (Section 16.1). Hopefully followed style guide ok, but please do make suggestions as necessary.

Thanks for the review @aloctavodia ! All done (except was already using az.plot_pair in that figure that matches the corresponding one in the book).

Great! sorry for not being clear I mean to use az.plot_pair with the marginals=True argument. Not sure how easy will be to add the correlation as a subplot, maybe it will work!

Thanks @aloctavodia ! Implemented the Arviz native and retained the 'book' one too. Just added the correlation as text to the main subplot.


Refactor SCAI generator APIs into pkg/

This PR refactors the core SCAI predicate generator APIs into pkg/ for external usage outside the CLI.
An anticipated usage of these refactored APIs will be in the build image attestation PoC for the SLSA workstream on Hardware Attested Platforms.
This also fixes a couple of related documentation bugs.

@alanssitis ptal!

Should we add @alanssitis as a maintainer?
Also, totally agree that scai-gen should eventually be moved out of this repo, and probably into go-witness as part of the broader in-toto golang consolidation efforts.


app.route mixes up parameters passed when multiple regex patterns are used

We recently upgraded from Flask v1.1.2 to Flask v2.2.2 and were surprised when many of our project unit tests started failing because Flask v2.2.2 is not passing the correct parameters to a route function.  This worked correctly in the previous v1.1.2 which we were using but no longer works properly in v2.2.2.
The following python code example of a route which passes 3 parameters with regex expressions will replicate the bug:
ALLOWED_METHODS = ["GET", "HEAD"]
DATE_PATTERN = "((\d{8}T\d{6}([.,]\d{1,3})?)|(\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}([.,]\d{1,3})?))Z"
# Note that 2 date formats are supported using this date pattern
# if you want to see what this regex does, paste it into https://regexper.com/

@app.route('/<regex("\d+"):source_id>/'
        + '<regex("' + DATE_PATTERN + '"):start_time>/'
        + '<regex("' + DATE_PATTERN + '"):end_time>', methods=ALLOWED_METHODS)
def clipslist_id(source_id, start_time, end_time):
    print("clipslist_id parameters source_id {} start_time {} and end time {}".format(source_id, start_time, end_time))
    print("Valid parameters {}".format(end_time > start_time))
    return

Call the app.get function and pass in a uri such one of the following (two date formats are supported):
/22/2023-02-16T23:36:36.266Z/2023-02-16T23:46:36.266Z
/22/20230217T002724.998Z/20230217T003724.998Z

Note that in both of the example uri's above the 3rd datetime parameter value is 10 minutes after the 2nd datetime parameter value
When the uri gets routed to clipslist_id the function receives 3 parameters, but the 3rd parameter is not correct.  Instead of passing through the correct 3rd parameter the route code passes the 2nd parameter twice, once as the 2nd parameter, and again as the 3rd parameter.
The three parameters received by the function should have had the values for: source_id, start_time, and end_time
Instead the three parameters that the function received were: source_id, start_time, start_time
The end_time value was not passed to the function
Environment:
Python 3.8.10 (default, Nov 14 2022, 12:59:47)
Flask 2.2.2
Distributor ID:	Ubuntu
Description:	Ubuntu 20.04.5 LTS
Release:	20.04
Codename:	focal
5.15.0-58-generic

Probably solved by https://github.com/pallets/werkzeug/issues/2582 which will be in 2.3 when it's released. It's also a duplicate of #2491, #2518, #2519, and possibly some others.
Quoting from my comment on #2491:

Note that the regex converter is not provided by Werkzeug. You should report an issue to the library providing that converter.

I looked at the code change you referred to in #2582 and I see that it checks for a / character in the regex expression and then changes a part_isolating flag to false.
In the example I gave above none of the regex expressions contain a / character.  I am not sure exactly what the part_isolating flag alters the processing and it was not obvious to me whether is should be set to False or True when all the regex expression do not contain slashes.
I tried to test this in our code by attempting to explicitly modify the part_isolating flag in the __init__ function.  I experimented by setting it to both False and True, but the app.route code continued to pass the wrong parameter values into the route handing function for every variation that I tried.
This is the code where I attempted to change the part_isolating flag to work around this issue.
 class RegexConverter(BaseConverter):
    """
    This class is used to covert a Regex pattern to a Flask Regex pattern
    """
    def __init__(self, url_map, *items):
        super(RegexConverter, self).__init__(url_map)
        RegexConverter.part_isolating = False
        self.part_isolating = False
        self.regex = items[0]

I also tried the opposite
class RegexConverter(BaseConverter):
    """
    This class is used to covert a Regex pattern to a Flask Regex pattern
    """
    def __init__(self, url_map, *items):
        super(RegexConverter, self).__init__(url_map)
        RegexConverter.part_isolating = True
        self.part_isolating = True
        self.regex = items[0]

Is there a different way that I should have attempted to change the part_isolating flag to work around this issue?  Or is this issue not resolved by changing the value of the part_isolating flag?

https://regexper.com/ shows that the date pattern regex expression we are using has 5 matching groups (see the included regexper.com diagram for this date pattern).

It does not appear that the date pattern regex could be rewritten to have only one matching group.  We need to support two different date formats.  Even supporting only one date format would still require a regex pattern with multiple matching groups.
This route logic has been working properly for us for a long time while using the previous version of Flask.  It is only the new version of Flask which exhibits the incorrect parameter passing issue to the route handler function.
The regex pattern matching portion of the logic seems to work correctly with the new Flask version.  We have many routes registered and the proper route gets matched correctly and the proper route handling function is getting called each time.
The bug seems to be in the logic which passes the parameter values to the route handling function.  For some reason this logic is now passing incorrect parameter values to the route handling function with the new Flask version.
In our code base I have identified 5 separate routes so far where the new version of Flask is passing the parameters incorrectly to the route handling function.  In each of these 5 routes at least one of the parameters uses the regex date pattern to match the expected uri data for that parameter.
As you suggested the date pattern regex we are using does seem to be a consistent factor which triggers the bug that causes incorrect parameters values to be passed to the route handling function.

Please see https://github.com/pallets/werkzeug/pull/2596 as a solution to this

The work around to change the nested groups from (...) to (?:...) worked like a charm.  Our code is fully functional again with the newer version of Flask.
A huge thank you to @pgjones for the quick response in investigating the issue, identifying the cause of the regression, identifying a work around for the problem, and coding a fix for the next release.  You're awesome!


SQLServerBulkCopy with a SQLServerBulkCSVFileRecord on temp file does not delete on exit.

I am using SQLServerBulkCopy with SQLServerBulkCSVFileRecord.
I created a File.createTempFile(..) with my csv data and set tempFile.deleteOnExit().
But this file does not get deleted after the java process exits. The SQLServerBulkCopy is leaving some stream open somewhere which is not letting the file to get deleted.
https://stackoverflow.com/a/28751091/842443

@sathley Hello, thanks for creating the issue. We are just wondering if you close the SQLServerBulkCopy  object? According to this doc, if you close the object, the temp file should be deleted on exit.

Yup. You are correct. I was not closing the SQLServerBulkCSVFileRecord object. It works fine if I close it. Thanks for the help!


Feedback: User Availability Shows Status “Unknown” in Teams Scheduling Assistant

[Enter feedback here]
Issue
An user is trying to create new Microsoft Teams meeting using 'Scheduling Assistant' tab. Some users listed as attendees have their status showing as "Unknown", despite those users not having any events scheduled for the day of the meeting.
This is the issue that Teams product engineering is still working on it. Customer asks MS to document this feature limitation.

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

ID: a162ef5c-3099-7f58-4178-d29e3ec2b5a7
Version Independent ID: 6ca31e56-052d-6c62-1323-b5aaaae1f04a
Content: User presence in Teams - Microsoft Teams
Content Source: Teams/presence-admins.md
Service: msteams
GitHub Login: @msdmaguire
Microsoft Alias: dmaguire

Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.

@scanum   Hi Scanum,
I noticed you pasted link of contacting support. I am MS Teams technical support and this is customer request: "We just need a link or KB from Microsoft mentioned that this is the currently behaviour."
Could you please help on this further?

@scanum  you may contact me at<EMAIL_ADDRESS>


Verify fails if there is only one leaf.

Sometimes when building a merkle tree from data, I might receive only one leaf hash to work with.
If this is the case verify function will return false. I have checked the code and figured out the issue is because of this line.
https://github.com/miguelmota/merkletreejs/blob/d78ea67599962911ae61271ec75fe7c7c155fc76/index.ts#L562
Because there is only one hash the proof length is 0, but the tree will still return a root (hash of that one leaf).
Is this intentional? Removing the line fixes my issue. How is the tree intended to work in cases when there is only one leaf node?

@Alko89 thanks for reporting the issue. I added a test case for this and should be fixed now in the latest version.

Thanks for such a quick response and fix! :smiley: really appreciate it!


[Error][minikube] ENOTFOUND nginx.openwhisk nginx.openwhisk:443

Hi folks!
I deploy OpenWhisk locally on minikube and it's my action:
const openwhisk = require('openwhisk');

function main(params){
  return new Promise((resolve, reject) => {
    ow.actions.list().then(resolve);
  });
}

Result of invoke it is error:
Unhandled rejection Error: Error encountered calling OpenWhisk: Error: getaddrinfo ENOTFOUND nginx.openwhisk nginx.openwhisk:443

Maybe someone experienced similar issue and can help?
Cheers 🥂

Hello @jakub-gawlas, can you give me a link to the instructions you followed for running OW on Minikube? I'll spin it up locally and try it out.

This sounds like a dns issue with kubernetes. If you can't resolve nginx.openwhisk from inside an action container you can't reach the edge gateway.
@DanLavine or @bbrowning may be able to help.

Open issue in kube repo asking how an action can address the controller or nginx from inside.
https://github.com/apache/incubator-openwhisk-deploy-kube/issues
Nothing we can do in this client-js to fix this.


Switch to faster bit reversal algorithm

There's a faster 64-bit reversal algorithm than what we're doing. It's O(log(bits)) instead of O(bits), using clever bit hacks:
x = ((x >> 32) & 0x00000000ffffffff) | (x << 32);
x = ((x >> 16) & 0x0000ffff0000ffff) | ((x & 0x0000ffff0000ffff) << 16);
x = ((x >> 8) & 0x00ff00ff00ff00ff) | ((x & 0x00ff00ff00ff00ff) << 8);
x = ((x >> 4) & 0x0f0f0f0f0f0f0f0f) | ((x & 0x0f0f0f0f0f0f0f0f) << 4);
x = ((x >> 2) & 0x3333333333333333) | ((x & 0x3333333333333333) << 2);
x = ((x >> 1) & 0x5555555555555555) | ((x & 0x5555555555555555) << 1);

This is slightly different to our case because we want to reverse the bit string based on the size of the FFT. So for a 256 element FFT we want to turn 0x1 into 0x80, not 2^63. So we'll need to shift the result based on the log2 of the FFT size.
Also rerun the benchmarks with this improvement.

Fixed by #8


Update Workshops with Upcoming Events

Update: https://dscbuk.club/#workshops with upcoming Events from our Whatsapp group.

No one's done this yet?

No one's done this yet?

No @mustaphagarba :-D


NetworkAnimator does not synchronise values to new clients

Describe the bug
Not current in a position to be able to provide an example project right now, but basically there is no logic at all in NetworkAnimator that would allow it to send the current values of network animator to a client. forceAll in OnSerialize does not pass through to WriteParameters so any non-dirty values are never sent to new clients joining the session (see https://github.com/vis2k/Mirror/blob/master/Assets/Mirror/Components/NetworkAnimator.cs#L291)
To Reproduce
Steps to reproduce the behavior:

Start a server in a scene with a NetworkAnimator that sets a value at the start of the scene and never changes it (so it's never dirty again)
Join it from a client, the value that was set will not be copied to the client

Expected behavior
The NetworkAnimator should sync all values, dirty or not, when forceAll is true
Additional context
Essentially forceAll needs to pass to WriteParameters and dirtyBits should set all bits to dirty so that the reader on the other end also parses all values at the start.
https://github.com/vis2k/Mirror/blob/master/Assets/Mirror/Components/NetworkAnimator.cs#L219

It's working for me. Thanks

:tada: This issue has been resolved in version 3.19.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Task 183 - is it correct?

Hi,
I was checking task 183 https://github.com/allenai/natural-instructions/blob/master/tasks/task183_rhyme_generation.json and it doesn't seem correct to me.
There's no indication on the output field if it rhymes with the input or not. So how would one expect to get the ground truth?
Thanks,
Felipe

Hi,
In all the natural-instructions tasks output is the ground truth and we can have multiple correct outputs as we see in this task. As task 183 is a generation task, all the outputs for each input should rhyme with the input though there might be some noisy outputs as well.


Port 'dcos marathon app version list' in Go.

Solves https://jira.mesosphere.com/browse/DCOS-60278

@ArmandGrillet I think we can merge it without continuous-integration/appveyor/pr to make it all green we need to push to this branch to start appveyor build for this branch


Compilation failed on Mac OS X

s2n_random.c:227:29: error: use of undeclared identifier 'ENGINE_FLAGS_NO_REGISTER_ALL'
        ENGINE_set_flags(e, ENGINE_FLAGS_NO_REGISTER_ALL) != 1 ||
                            ^
1 error generated.

Are you using OpenSSL 1.0.2?

I get the same error.
$ openssl version
OpenSSL 1.0.2d 9 Jul 2015

I orignally encountered the same issue, after having built OpenSSL 1.0.2 separately via macports.
I then followed the instructions here:
https://github.com/awslabs/s2n/blob/master/docs/USAGE-GUIDE.md#building-s2n-with-openssl-102
and this issue was resolved. (You have to compile s2n with a new copy of the OpenSSL source, downloaded into the libcrypto-build directory).

Closing this.


feat(table): 3287-Table-SearhIcon-align-Row-Count

#689
Closes #
Summary
Right align the search icon and toggle open when clicked to display a search component.
Show table row count and should be appear on the left. i.e "Results: X", where X is equal to the number of rows in the table.
This field (Row count) should be configuration via boolean property and Result Label should allow to change.
Change List (commits, features, bugs, etc)
src/components/DataTable/_data-table.scss
src/components/Table/Table.jsx
src/components/Table/Table.story.jsx
src/components/Table/TablePropTypes.js
src/components/Table/TableToolbar/TableToolbar.jsx
src/components/Table/snapshots/Table.story.storyshot
src/components/TableCard/snapshots/TableCard.story.storyshot
src/components/Dashboard/snapshots/Dashboard.story.storyshot
Acceptance Test (how to verify the PR)
Two fields are provided in Knob to test.
Search Icon should be display as right align in table toolbar and when clicked it should display a search component.
Show total row count in table,
Using knob, Row count label input field should allow to change row count label value, also show row count checkbox use to display and hide Row count field.

@davidicus @tay1orjones @stuckless  Please review this PR.

@JordanWSmith15 @lukefirth There is a property on the table options for showRecordCount but this is ONLY affecting the record count in the header.  So a couple of quetions...

should this property only affect showing a record count in the header, and, it will always show the 1-N record count in the footer.  (if yes, then we rename this property to showRecordCountInHeader, if no, then we need to update the code to hide the 1-n count in the footer if this value is false.
From a UX point of view does it make sense to show counts in the header and footer?

@davidicus @tay1orjones @bryancboyd @scottdickerson We should look at how we can inject a data formatter into the table (and possibly other components).  Right now the UI is inconsistent when rendering numbers because, we can format some data in the table (using a cell renderer) but in other places where we are showing numbers, likse "1 of 100000 items" we can't format that.  Or, in the header where we are showing "Records: 100000" we can't format that.  It would seem to me that a component level we should allow some sort of data formatter to be injected as part of the configuration.  The "default" might simply be a String(val) but, at least we could possibly inject a custom formatter if needed.

@stuckless For your first question: Part of the requirements was to make the record count in the table header optional, so the app dev should be able to hide it. I'm guessing a bit here, but if the page does not have pagination because the page size is large, such as https://carbon-addons-iot-react.com/?path=/story/watson-iot-table--simple-stateful-example-with-alignment, you would want the result count in the header (since there is no footer).

Right now the UI is inconsistent when rendering numbers because, we can format some data in the table (using a cell renderer) but in other places where we are showing numbers, likse "1 of 100000 items" we can't format that. Or, in the header where we are showing "Records: 100000" we can't format that. It would seem to me that a component level we should allow some sort of data formatter to be injected as part of the configuration.

@stuckless Those examples you listed should be modified to be functions that return a string, rather than a direct string. Example from the itemRangeText prop in the Carbon <Pagination>

@stuckless @davidicus @tay1orjones  Thanks for your review comments, I will implement the given suggestion.

@tay1orjones I know that IoT uses functions as a way to return localized text, but that problematic.  How is a translation team supposed to translate a function?  In every translation system that I've seen, the localization part is just text with paramater placeholders and then some code inject dynamic parameters.  So the part that gets sent away for localization is just text.
instead of
rowCountLabel: (total) => `Results: ${total}`

it would be (java like)
rowCountLabel: 'Results: {0}'

or (react i18n like)
rowCountLabel: 'Results: {{total}}'

and some other proces basically has to scan the text and replace variables.
In the most simplest form the code to do the replacements is not hard, and, it ensures that i18n strings are in fact strings.
  /**
   * Replace placeholder parameters in string with actual values.
   *
   * <pre>
   * let msg = replaceParams('Hello {0}', ['world']);
   * would output 'Hello world'
   * </pre>.
   *
   * @param {string} message - The message string with optional {0}, {1}, {n} parameters.
   * @param {Array} params - Array containing strings to replace placeholders.
   */
  replaceParams(message, params) {
    if (message && params) {
      params.forEach((param, index) =>  {
        message = message.replace('{' + index + '}', param);
      });
    }
    return message;
  }

And yeah a function like that should be tightened up, because it has holes, but, this could be in a reusalbe util.js file and then included and used in a component when needed.
This is more of a fyi on how we could/should be handling localization and not relying on functions (which can't be localized))

@davidicus @tay1orjones @stuckless  Please review the updated code based on review comments.

@stuckless Regarding translation, I don't see functions under i18n as anything fundamentally different from the approaches you outlined above.  In our application usage we typically wrap the IoT Component in a utility that defines strings and props for the "common" use of that component.  Here's an example for the table:
import { injectIntl, defineMessages } from 'react-intl';
import { StatefulTable } from 'carbon-addons-iot-react';

const messages = defineMessages({
  selectAll: 'Select all',
  ...
  clickToCollapse: 'Click to collapse',
  itemsRange: '{min}–{max} items',
  currentPage: 'page {page}',
  itemsRangeWithTotal: '{min}–{max} of {total} items',
  ...
});

const CommonTable = ({ intl: { formatMessage }, i18n, options, columns, ...props }) => (
  <StatefulTable
    {...props}
    columns={columns.map(column => ({
      ...column,
      filter:
        column.filter || column.isFilterable
          ? { placeholderText: formatMessage(messages.filterColumn) }
          : null,
    }))}
    options={
      options
        ? merge({}, options, {
            hasRowSelection: options.hasRowSelection === true ? 'multi' : options.hasRowSelection,
          })
        : undefined
    }
    i18n={merge(
      {},
      {
        /** pagination */
        pageBackwardAria: formatMessage(messages.previousPage),
        pageForwardAria: formatMessage(messages.nextPage),
        ...
        itemsRangeWithTotal: (min, max, total) =>
          formatMessage(messages.itemsRangeWithTotal, { min, max, total }),
        pageRange: (current, total) => formatMessage(messages.pageRange, { current, total }),
        /** table body */
        overflowMenuAria: formatMessage(messages.moreActions),
      },
      i18n // use whatever props come in
    )}
  />
);
export default injectIntl(CommonTable);

@stuckless I guess the change you are suggesting is that we standardize to where i18n objects passed to IoT components are raw strings in a "react-y" format itemsRangeWithTotal: '{min}–{max} of {total} items', and then have a common util function that does the translation inside of components?  The one benefit to exposing i18n props as functions is that it provides a clear signature for use by consuming components... otherwise I'd have to know that my i18n.itemsRangeWtihTotal prop needs to include some specific named {value}s.
I'm not picky on the approach we take, but I agree we should be consistent here. @tay1orjones could you open a separate issue to discuss our approach for standardizing i18n strings with parameters across the library?
(Also, what does Carbon do?)

Yeah, the translation team wouldn't be translating a function, but instead the keys/strings you provide them via your i18n library. These are then passed to the components, the function params provide the ability to 'replace' the dynamic parts of the string yourself.
I've opened #714 to continue the conversation

@stuckless @tay1orjones latest suggestion are incorporated in Table.jsx and TableToolbar.jsx, Please review.

Yes, Sunil you should just have to pull master into your branch and resolve the conflict.

Sunil this looks good to me. Unfortunately there is a merge conflict with the Table snapshot and I think the only way to resolve that is to merge with master. @davidicus is that how you have to deal with this? Also I see the coverage now fails. Based on the tests becuase hasRowCountInHeader is an option, you shold be adding tests to verify that it shows or not shows. (sort of like what you did with the label)

@stuckless  new PR (https://github.com/IBM/carbon-addons-iot-react/pull/743) raised as yarn test was failing on windows, so I created a new branch and raised new PR.
Please review.

Yes, Sunil you should just have to pull master into your branch and resolve the conflict.

@davidicus I have some issue related to yarn test which was failing on windows. so created new branch and PR from Team member Mac machine. Please review PR 743.
https://github.com/IBM/carbon-addons-iot-react/pull/743

New PR is raised so closing this PR and deleting the branch
New PR : https://github.com/IBM/carbon-addons-iot-react/pull/744


Error loading contacts

level=error ts="2023-01-19 15:48:45.4168740 (UTC)" source=golang err="could not get contacts: could not get fresh data: transaction failed: could not get wanted feeds: could not get feed state: could not get a feed: loading failed: failed to get the message: message not found: Key not found" name=gossb.manager message="send feed to replicate failed"

Apparently none of the messages for this feed exist in the message repository but they are still referenced from the feed repository? This is very strange.

Fixed in https://github.com/planetary-social/scuttlego/pull/175.


strange construction for ShowBadgeOnPluginSettingError

This is a weird construction:
 !Disabled && [ !Disabled && !IsGlobal && string.IsNullOrWhiteSpace(ActionKeyword) ]

Maybe @htcfreek knows something about this?

It should be only visible if the plugin is not disabled.
If you remove the action keyword on an not global plugin these are shown. But if you disable the plugin the warnings are hidden.
Warnings are only important if the plugin is going to be used. And we don't want to spam users with unimportant information.
/helped

But that does mean they are functionally identical, right?

From the point of evaluating the condition yes. (And I think we can remove the !IsDisabled in the badge property. But then please add a comment that the sub property has to have a !IsDisabled ceck.)
We shouldn't remove the badge property as they have different use cases in the ui:

Accessible warning property: Shows an infobar if action key is empty.
Badge property: Shows an error badge in the Plugin list entry header that there is a problem with the plugin. (Currently only on missing keyword but maybe there are other use cases in the future.)

Ah, I understand now 😊


Only add executables from itch

The Itch db can refer to multiple files,
before the first one added was selected,
now the first one that is  executable is selected.

This should fix this issue:
https://github.com/PhilipK/BoilR/issues/26


Segfault on Chapter2 -> Recipe12

On Implement a writing style helper tool for finding very long sentences in text with std::multimap the implementation "segfaults" if the input ends in something else than "." On most systems the input will end in "\n" or "\r\n".
The issue can be fixed if the update to "it2" checks for failure before continuing the loop:
    it1 = next(it2, 1);
    if ((it2 = find(it1, end_it, '.')) == end_it) {
      break;
    }

If we don't "break" out of the loop when updating "it2" then the program segfaults.

Hi Julio,
Thanks for finding this, i did indeed only test it with "nice" input!
If the condition it2 == end_it holds, then the it1 = next(it2, 1); step lets it1 point past the end, indeed.
I would fix this by putting this check before the increment of both iterators. The patch that you suggest would let the program miss the last sentence.
Look for example at input like "this is a sentence. this is another sentence without a dot at its end". Here, we would break out of the loop too early, because it1 points to the space after the only '.' character, and it2 points to the null terminator of the string. We need to iterate once more, but must not increment after that again.
So my fix looks like:
        if (it2 == end_it) {
            break;
        }

        it1 = next(it2, 1);
        it2 = find(it1, end_it, '.');

Do you agree?

Hello Jacek
Yes, I totally agree. You're right about the last sentence being


MQTT payload selection enhancement

Would it be possible to enhance the MQTT publish connector to select a part of the payload rather than the whole? I see this functionality in other nodes and it would save having to add an additional node each time to rewrite the payload.

Hi,
for feature requests/discussions, please can you use the mailing list - link from bottom of nodered.org
We try to keep the this issue like for real bugs or planned features.
Thanks,
Nick

sure - and apologies. I had misunderstood the tickets marked enhancement

The vast majority of nodes operate on payload by convention. Some of the core nodes do let you operate on other properties, but that is by design to avoid having to resort to the Function node - such as the Change and Switch nodes.
So on balance, I don't think this is a change we'd make to the mqtt and (to be consistent) all other output nodes.


Feat: MLL - Add Minimal Live Linux

Why is it delayed?

If you have time, yes. I am having a hard time building MLL on each systems. Best thing would be to build the binaries on RockyLinux 8, and then dispatch these on each distributions containers to package them.


UI improvement of contact us screen

The current UI of the "Contact Us" screen appears uneven and lacks cohesiveness. This could impact user experience and engagement.
I would like to take on this issue and improve the visual layout to ensure the screen is aesthetically pleasing and user-friendly. My goal is to create a more cohesive design that will make the contact page visually appealing.

I'd like to work on this issue

I am working on this

@Shubham-Zone can you please tell me the folder/file name where the "sign up" page codebase is written so that i can modify and work on it.


Code Quality: Update Selectors in checks PPCv2.0

Porting over PPC to the new structure in PPCv2.0 we were just trying to get all the old code into the new structure. The old code took the entire story when seeing if a check was passing and failing. When we moved everything over to the new structure, we were trying to alter everything as little as possible because there were a lot of moving pieces and we didn't want to introduce more surface area for there to be regressions.
Because of the approach above, a lot of the checks are still pulling the entire story object:
const story = useStory();
const isCheckRendered = checkStoryForSomething(story);

However, with the new structure, we have better access to the context selector, so we can fine tune these useStory() uses to have a more fine tuned selector and only perform the check on the part of the story it needs vs the whole story object.
This should increase performance of the checks because they should only be re-rendering when parts of the story they're checking against update.
Completion of this task includes going through every check and making sure each check only re-renders and runs its check of the necessary parts of the story.

WDYT about only giving checklist components the story pages but passed through ChecklistProvider? This would keep the context dependency local to the checklist.

Some checklist checks are for story metadata too, not only pages

WDYT about only giving checklist components the story pages but passed through ChecklistProvider? This would keep the context dependency local to the checklist.

I guess I'm not completely following. I think having each check pull only what's needed from useStory should be the most performant result because it would result in the smallest subtrees getting diffed (only relevant checks and their children) when the story updates.
I think these use-context-selector providers are just subscriptions held by traditional context providers under the hood, so maybe it wouldn't effect it at all perf wise if we passed any story context from useStory through ChecklistProvider . That being said, I think having the smallest chain between between the place the data is coming from and the component receiving it lends the most transparency with regard to perf.
tldr I'm not seeing the benefit, but open to it if there's some nice benefits I'm not quite grokking rn

@choumx Going through it a little more now, I think I might be getting at what you're saying more with regard to perf. Might see a better increase with elements with pageId being hoisted up to the useStory provider so the reduce only runs once every time the pages update, then we can just pull that into a check that runs on every element and only have to run a simple filter.
https://github.com/google/web-stories-wp/blob/9fce684d0a7aedba026b3b115c39e54233885869/assets/src/edit-story/components/checklist/utils/filterStoryElements.js#L29
Not sure if it'd be that much faster, but should be less flatMap calls and object instantiations every time pages updates (probably ~10x less). thoughts?

Oh I was thinking more about context encapsulation rather than a strict performance benefit.
IMO it's cleaner to have a single useStory in the ChecklistProvider vs. a useStory in each checklist component since it keeps component dependencies more local. Though we'd also reduce duplication of code/work in the context selector as you mentioned.

IMO it's cleaner to have a single useStory in the ChecklistProvider

Ah yea, completely agree but I think with this approach we were intentionally moving away from that so each check could have more fine grained logic to re-render based on the selector it passes to the useStory hook vs just running every check every time any part of the story updates.
I agree that I think the tradeoff is that it's not as clean, but I think the fact that we previously had perf issues with a more centralized setup warrants the current approach


Bulk download?

Hey! Really fantastic work on this. I was wondering if there's a clean way to do a bulk download of a site, bulk meaning download everything.
If anyone has any ideas i'd love to hear!
Thanks

I'm just a data hoarder and want to download pretty much want to download all the manga that exists haha.
I do have lists of all the urls from a few sites. Can you give me an example of how I would pass that through poketo? It's one url per line, so nothing crazy.
Thanks

Thanks for the quick reply! I was able to get that going on a my list of manga from mangadex, but it looks like i'm getting some weird characters in my folder names.
First one looks like this:
"Aishiteru", Uso dakedo
Any idea what I can do to get around that? I guess I could just do a mass rename after the fact.

Are those series from Mangadex?
I noticed a bug a few days ago (#37) where series from Mangadex has encoded characters in their titles. " is HTML-speak for ". You might also see × and > and others.
I should be able to push a fix for this later today, but until then, doing a batch rename would be easiest.

I just pushed that fix in a new version of the poketo-cli, 0.0.7. If you upgrade, you should see those " or other weird characters anymore.
I'm gonna close this issue since it sounds like you were able to achieve what you're after.


IOS Build Issue

I am facing on installing the the plugin.
=== BUILD TARGET MarketResearch OF PROJECT MarketResearch WITH CONFIGURATION Debug ===
Check dependencies
“Swift Language Version” (SWIFT_VERSION) is required to be configured correctly for targets which use Swift. Use the [Edit > Convert > To Current Swift Syntax…] menu to choose a Swift version or use the Build Settings editor to configure the build setting directly.
** BUILD FAILED **
The following build commands failed:
Check dependencies
(1 failure)
(node:3787) UnhandledPromiseRejectionWarning: Error code 65 for command: xcodebuild with args: -xcconfig,/Users/algupta/Documents/Compass NZ Servey/MarketResearch/platforms/ios/cordova/build-debug.xcconfig,-workspace,MarketResearch.xcworkspace,-scheme,MarketResearch,-configuration,Debug,-sdk,iphonesimulator,-destination,platform=iOS Simulator,name=iPhone SE,build,CONFIGURATION_BUILD_DIR=/Users/algupta/Documents/Compass NZ Servey/MarketResearch/platforms/ios/build/emulator,SHARED_PRECOMPS_DIR=/Users/algupta/Documents/Compass NZ Servey/MarketResearch/platforms/ios/build/sharedpch
(node:3787) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:3787) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminatethe Node.js process with a non-zero exit code.

I'm also having this issue. It seems like the issue is with the rangeAt method that has ben obsoleted in Swift 4, according to Xcode's Report navigator.
/Users/usuario/Desktop/projects/appcarnabelo2019/platforms/ios/Carnaval Belô/Plugins/cordova-plugin-photo-library/PhotoLibraryService.swift:657:33: 'rangeAt' has been renamed to 'range(at:)'

Created a pull request.
https://github.com/terikon/cordova-plugin-photo-library/pull/138

@viananicolas This change breaks my app, 'range(at:)' has been renamed to 'rangeAt(_:)'. How do i make it use swift 4 ?

@viananicolas This change breaks my app, 'range(at:)' has been renamed to 'rangeAt(_:)'. How do i make it use swift 4 ?

I'm facing the same problem. Anyone knows how to solve it?

Any update on this issue? I see the same issue and using 2.1.1 (or 2.0.0 for that matter) doesn't work.

guys... this is still broken.
it can easily be fixed by changing rangeAt to range(at).
but it's a bit annoying to fix it manually all the time

Would you like to install the master branch plugin?
cordova plugin add https://github.com/terikon/cordova-plugin-photo-library.git#master --variable PHOTO_LIBRARY_USAGE_DESCRIPTION="To choose photos" --save

Or try the version I forked.
cordova plugin add https://github.com/amatakasap/cordova-plugin-photo-library.git#master --variable PHOTO_LIBRARY_USAGE_DESCRIPTION="To choose photos" --save

This is still an issue in October 2019

Just in case anyone is interested, my fork has the fix for this, can be installed using npm and cordova directly, is now mature and has a lot of fixes and is being used by quite a lot of projects (a dozen or more I think):
Instructions to install are on the link below.
https://github.com/xmarkclx/cordova-plugin-photo-library-sism

@xmarkclx - I was struggling for hours until I stumbled upon your fix. Works like a charm. Thanks a million...


I added BUZZER pin functionality.

To notify the client that his card has been read successfully. when mounted to a real system.
Feel free to give notes. I will correct and push again if needed.

@AhmedBHameed thats really awesome!

have you seen this other pull-request where some people refactored to rpio (softpi)?
https://github.com/firsttris/mfrc522-rpi/pull/6
would it be possible to migrate your BUZZER Extension to Softpi as well?
reason: we always had problems with wiring-pi not being updated on kernel updates... rpio was updated more regurlarley

Hello @firsttris , Ok sounds interesting and good idea since the WiringPI is also not working with the latest node version. I will take a look and may be update my PR with SoftSPI code.
Till then, have a great day/night

Hello @firsttris ,
I managed to merge buzzer alert feature to SoftSPI.
Also tested with Raspberry Pi 3 Model B+
Feel free to give review and comments.

@AhmedBHameed
impressed, very good work! i will release this as 2.0 of mfrc522-rpi
if you like i would add you as contributor?

@AhmedBHameed could you add some more notes to the readme how to connect the buzzer? which buzzer was used, and where to buy them?

@firsttris ,
That would be nice of you.
Hmmm, Ok will try to open another pull request explaining about it in more details.

@AhmedBHameed
i already added some more infos to readme.
do you think its sufficient like this ?
https://github.com/firsttris/mfrc522-rpi#buzzer

@AhmedBHameed
added you as collaborator to the repository, you have full access now

Hi @firsttris
I really appreciate your generosity here.
I saw your additional information. Those are enough in case of some experience with a person.
I will add more specific information to make it peace of cake for all people. for example those buzzers have two type the one with 5V DC and the other need extra circuit to produce noise. Also the CARD is working with 3.3 volt and the buzzer is "Better" working with 5V with no sound failure.
It will take time according to my free time. Bit will keep it in mind.


Fix/mlflow sync tag

This PR includes:

Fix in mlflow-sync for when we want to sync a new-experiment that doesn't yet exist on the remote server
Bring a new tag to distinguish between those runs that are synced from offline to online and those are synced 'online' to 'online' (ie between two different remove servers)

Use squash when we merge this one
otherwise it will merge in a lot of unnecessary commits that were squashed in other PRs


classic_filter and non-acronyms

Due to the religiosity of central europe, many city names have a prefix that can be translated to "saint". E.g. "St.Veit", which usually would be written "St. Veit", but, well, that's the Internet.
When using classic_filter: true this string currently returns "stveit". There could be many other edge cases (e.g. "B.Obama" is converted to "bobama", "CNN.com" to "cnncom").
Shouldn't classic_filter only strip dots from acronyms if there's a maximum of ONE letter before and after it?

Thanks! This should be fixed in the most recent version.


Changed the background color of the button to green for dark mode.

This branch has conflicts that must be resolved
Use the web editor or the  to resolve conflicts.
Conflicting files
assets/style.css
index.html

Sir I have resolved merge conflicts please check.

@adithyapaib


correctly judge desired origin url when use GIT proxy

I use a proxy to accelerate git clone and fetch from Github for Doom Emacs,
straight.el always reports inconsistence between desired origin url of
melpa/melpa.git and actual remote origin url.  According to commands
below, it's obvious straight.el should use git config --get instead of
git remote get-url.
$ git config --global url.https://github.com.cnpmjs.org/.insteadOf https://github.com/

$ git config --get remote.origin.url
https://github.com/raxod502/straight.el

$ git remote get-url origin
https://github.com.cnpmjs.org/raxod502/straight.el

@progfolio @raxod502 could you merge this simple patch?

Seems fair enough to me. Thank you.


goimports import the shortest package. But most of the time we need the relative-shortest package.

env:

go version go1.8 darwin/amd64
amd64
Mac mini, macOS 10.12.3

There are two packages:
github.com/longlong/bytes  # long
github.com/short/bytes     # short

I am working in package:
github.com/longlong/work   # work

goimports import the short package when I use bytes in working package. Playground
But I expect import the long one, Because the are under same path github.com/longlong. Playground
goimports source code

Dup of #17557.


Add Stripe Refund Support

Adds the Refund Model
Implements the Stripe Convert action that will convert the Refund Model into Stipe compliant refund query

@gpou you already merged a previous PR making this, but I added new changes.
However, this PR is strange because there are changes already in your fork that show as difference in the commits.
I made this change on top of Payum main branch, so I'll create a new one rebasing on your fork

@bbwebconsult Yes, it's strange because src/Payum/Stripe/Action/Api/CreateRefundAction.php is already there from your last merged pr, but on the new pr it shows as a new file...


Remove auto-generated comment :)

I guess the generated docs files are quite modified compared to what sphinx autogenerates, so we can remove this :)

Apparently I end all sentences with smileys right now :)

Right, makes sense!


/dev/video0 not found

I'm using a rpi 4 (not zero 2w) but I assume it should work as well?
Have the rpi camera v2 plugged in and verified that it works with the traditional rpicam-still and rpicam-vid tools.
I was following the setup steps but after I add the lines in /boot/firmware/config.txt and restarted rpi, I cannot find the /dev/video0 file pointer. There is only a /dev/video19, which doesn't work when used with pi_webrtc:
$ ./pi_webrtc --device=/dev/video19 --fps=30 --width=1280 --height=960 --v4l2_form
at=h264 --hw_accel --mqtt_host=... --mqtt_port=8
883 --mqtt_username=... --mqtt_password=... --uid=home-pi-zero2w
Use h264 format source in v4l2
[v4l2_utils] Error: fd(26) set fps(30): Inappropriate ioctl for device

Using v4l2-ctl to check device:
$ v4l2-ctl --list-devices
rpivid (platform:rpivid):
        /dev/video19
        /dev/media0

Cannot open device /dev/video0, exiting.

The following is the /boot/firmware/config.txt file content for reference:
# For more options and information see
# http://rptl.io/configtxt
# Some settings may impact device functionality. See link above for details

# Uncomment some or all of these to enable the optional hardware interfaces
#dtparam=i2c_arm=on
#dtparam=i2s=on
#dtparam=spi=on

# Enable audio (loads snd_bcm2835)
dtparam=audio=on

# Additional overlays and parameters are documented
# /boot/firmware/overlays/README

# Automatically load overlays for detected cameras

# Automatically load overlays for detected DSI displays
display_auto_detect=1

# Automatically load initramfs files, if found
auto_initramfs=1

# Enable DRM VC4 V3D driver
dtoverlay=vc4-kms-v3d
max_framebuffers=2

# Don't have the firmware create an initial video= setting in cmdline.txt.
# Use the kernel's default instead.
disable_fw_kms_setup=1

# Run in 64-bit mode
arm_64bit=1

# Disable compensation for displays with overscan
disable_overscan=1

# Run as fast as firmware / board allows
arm_boost=1

[cm4]
# Enable host mode on the 2711 built-in XHCI USB controller.
# This line should be removed if the legacy DWC2 controller is required
# (e.g. for USB device mode) or if USB support is not required.
otg_mode=1

[cm5]
dtoverlay=dwc2,dr_mode=host

[all]

camera_auto_detect=0
start_x=1
gpu_mem=16

I image that this file should be part of the rpi OS Lite image so there shouldn't be any problem with that?
Any help is appreciated, thanks!

My Pi 4B used to run on Bullseye(6.1.21) perfectly so far. I just pulled out the Bookworm lite OS (6.6.31) sd card from Pi Zero 2W and inserted it into my Pi 4b. I found the same /boot/firmware/config.txt setting led to different results on Pi 4B!😕
Please increase GPU memory to 256MB in the config.txt and reboot. I guess Pi 4B hardware consumes more resources than Pi Zero, hence 16MB is not enough to start hardwares.
# Automatically load overlays for detected cameras
#camera_auto_detect=1
camera_auto_detect=0
start_x=1
gpu_mem=256

The results of Pi 4B are like the following:
$ vcgencmd get_camera
supported=1 detected=1, libcamera interfaces=0

$ v4l2-ctl --list-devices
bcm2835-codec-decode (platform:bcm2835-codec):
        /dev/video10
        /dev/video11
        /dev/video12
        /dev/video18
        /dev/video31
        /dev/media2

bcm2835-isp (platform:bcm2835-isp):
        /dev/video13
        /dev/video14
        /dev/video15
        /dev/video16
        /dev/video20
        /dev/video21
        /dev/video22
        /dev/video23
        /dev/media1
        /dev/media3

mmal service 16.1 (platform:bcm2835_v4l2-0):
        /dev/video0

rpivid (platform:rpivid):
        /dev/video19
        /dev/media0

$ v4l2-ctl -d /dev/video0 --list-formats
ioctl: VIDIOC_ENUM_FMT
        Type: Video Capture

        [0]: 'YU12' (Planar YUV 4:2:0)
        [1]: 'YUYV' (YUYV 4:2:2)
        [2]: 'RGB3' (24-bit RGB 8-8-8)
        [3]: 'JPEG' (JFIF JPEG, compressed)
        [4]: 'H264' (H.264, compressed)
        [5]: 'MJPG' (Motion-JPEG, compressed)
        [6]: 'YVYU' (YVYU 4:2:2)
        [7]: 'VYUY' (VYUY 4:2:2)
        [8]: 'UYVY' (UYVY 4:2:2)
        [9]: 'NV12' (Y/UV 4:2:0)
        [10]: 'BGR3' (24-bit BGR 8-8-8)
        [11]: 'YV12' (Planar YVU 4:2:0)
        [12]: 'NV21' (Y/VU 4:2:0)
        [13]: 'RX24' (32-bit XBGR 8-8-8-8)

Thank you for the prompt response! I wouldn't think of the gpu_mem to be the issue without your help.
This solution indeed works and I'm able to get live video end-to-end now!
I think it's worth mentioning in the README about this so I created a pull request https://github.com/TzuHuanTai/RaspberryPi_WebRTC/pull/184


Crey/conjurer commit-ref.

"The package "crey/conjurer" is pointing to a commit-ref, this is bad practice and can cause unforeseen issues." is keeping us from Platinum medal.

crey/conjurer will be abandoned soon in favor of subcosm/conjurer. I didn't inspect the source deeply, do this package really use conjurer at all (also an unstable version) ?

for being a mod I really don't know much about industry level (OOP and dependencies mainly) in PHP specifically but it's referenced at https://github.com/php-earth/stats/blob/037923552c0afbade50eeaab6d6b63197b475d86/app/bootstrap.php and initialized for $container

Should be fixed now. For the time being I've added the Symfony Dependency Injection component instead.


[feature request] GithubRepositoryReader seems to ignore all source code files

It seems like only files with supported parsers are parsed
https://github.com/jerryjliu/gpt_index/blob/e5605c171331f29ef3dc00cc21d2149eaed0af05/gpt_index/readers/github_readers/github_repository_reader.py#L322
It would be great to be able to index source code too

Hi @elyase! Thanks for the feature request.
You can turn on/off the parser by passing use_parser = False to the constructor of GithubRepositoryReader.
And for the source code parser, I can't think of a way to parse the source code. What I mean is that the parser for png file tries to extract words from image or the parser for audio converts the unhuman-readable siganls to words, so from these examples I feel like the source code is already natural language. However, I think the parser for specific programming languages could be beneficial for providing more context to specific to that programming language.
Do you have anything in mind for a source code file parser?

The above comment will enable all files to be parsed.
Going to close for now :)


How can I use this library to just show a single image

related to issue #92 ...
If you provide itemCount 1, the library allows you to swipe right left anyway and keep asking the datasourse for the next/ previous item making impossible to actually display a single element.
Any ideas?

you can set the swipeToDismissMode to always.

Thanks @mikengyn 👍


Main 2406 release

update main branch for v2406 release.
Please create merge commit not squash.

This is missing #388.

This is missing #388.

Hi @jlowe , do you mean this commit id?

Yes, I mean that commit. It's weird in that I can see the commit in the commits to be merged, yet when I diff this PR against branch-24.06 there are some missing changes that I thought were coming from #388.  For example:
$ git fetch https://github.com/nvliyuan/spark-rapids-examples.git main-2406-release
$ git diff branch-24.06 FETCH_HEAD
diff --git a/docs/get-started/xgboost-examples/csp/databricks/databricks.md b/docs/get-started/xgboost-examples/csp/databricks/databricks.md
index 1377a15..2d03013 100644
--- a/docs/get-started/xgboost-examples/csp/databricks/databricks.md
+++ b/docs/get-started/xgboost-examples/csp/databricks/databricks.md
@@ -21,8 +21,7 @@ Navigate to your home directory in the UI and select **Create** > **File** from
 create an `init.sh` scripts with contents:
    ```bash
    #!/bin/bash
-   sudo wget -O /databricks/jars/rapids-4-spark_2.12-24.06.0.jar https://repo1.maven.org/maven2/com/nvidia/rapids-4-spark_2.12/24.06.0/rapids-4-spark_2.12-24.06.0.jar
-   ```
+   sudo wget -O /databricks/jars/rapids-4-spark_2.12-24.04.0.jar https://repo1.maven.org/maven2/com/nvidia/rapids-4-spark_2.12/24.04.0/rapids-4-spark_2.12-24.04.0.jar
 1. Select the Databricks Runtime Version from one of the supported runtimes specified in the
    Prerequisites section.
 2. Choose the number of workers that matches the number of GPUs you want to use.
[...]

We can see from the diff above that we're rolling back changes in branch-24.06 as part of this merge of that branch to main.  Not quite sure why these changes were lost despite #388 being part of the commit list in this PR (bad merge conflict resolution maybe?), but it needs to be fixed.  You can do a diff like I show above to verify all of the changes between this PR and branch-24.06 make sense.

Hi @jlowe, thanks for pointing it put, I just checked the diff details, basically there are three part:
part1:  I miss some commits while doing the merge conflict, for example:
diff --git a/docs/get-started/xgboost-examples/csp/databricks/databricks.md b/docs/get-started/xgboost-examples/csp/databricks/databricks.md
index 1377a15..2d03013 100644
--- a/docs/get-started/xgboost-examples/csp/databricks/databricks.md
+++ b/docs/get-started/xgboost-examples/csp/databricks/databricks.md
@@ -21,8 +21,7 @@ Navigate to your home directory in the UI and select **Create** > **File** from
 create an `init.sh` scripts with contents:
    ```bash
    #!/bin/bash
-   sudo wget -O /databricks/jars/rapids-4-spark_2.12-24.06.0.jar https://repo1.maven.org/maven2/com/nvidia/rapids-4-spark_2.12/24.06.0/rapids-4-spark_2.12-24.06.0.jar
-   ```
+   sudo wget -O /databricks/jars/rapids-4-spark_2.12-24.04.0.jar https://repo1.maven.org/maven2/com/nvidia/rapids-4-spark_2.12/24.04.0/rapids-4-spark_2.12-24.04.0.jar
 1. Select the Databricks Runtime Version from one of the supported runtimes specified in the

part2: There are something wrong in branch-2406, since PCA demo only compatible with spark-rapids v23.04, cuspecial demo only compatible with spark-rapids v23.02, we miss this during the previous release, for example:
diff --git a/examples/ML+DL-Examples/Spark-cuML/pca/README.md b/examples/ML+DL-Examples/Spark-cuML/pca/README.md
index 1b2b22d..0c4a514 100644
--- a/examples/ML+DL-Examples/Spark-cuML/pca/README.md
+++ b/examples/ML+DL-Examples/Spark-cuML/pca/README.md
@@ -12,7 +12,7 @@ User can also download the release jar from Maven central:

 [rapids-4-spark-ml_2.12-22.02.0-cuda11.jar](https://repo1.maven.org/maven2/com/nvidia/rapids-4-spark-ml_2.12/22.02.0/rapids-4-spark-ml_2.12-22.02.0-cuda11.jar)

-[rapids-4-spark_2.12-24.06.0.jar](https://repo1.maven.org/maven2/com/nvidia/rapids-4-spark_2.12/24.06.0/rapids-4-spark_2.12-24.06.0.jar)
+[rapids-4-spark_2.12-23.04.0.jar](https://repo1.maven.org/maven2/com/nvidia/rapids-4-spark_2.12/23.04.0/rapids-4-spark_2.12-23.04.0.jar)

 Note: This demo could only work with v22.02.0 spark-ml version, and only compatible with spark-rapids versions prior to 24.06.0 . Please do not update the version in release.

@@ -49,7 +49,7 @@ It is assumed that a Standalone Spark cluster has been set up, the `SPARK_MASTER

     ``` bash
     RAPIDS_ML_JAR=PATH_TO_rapids-4-spark-ml_2.12-22.02.0-cuda11.jar
-    PLUGIN_JAR=PATH_TO_rapids-4-spark_2.12-24.06.0.jar
+    PLUGIN_JAR=PATH_TO_rapids-4-spark_2.12-23.04.0.jar

part3: it is ok to exist some snapshot version diff since we will remove all -snapshot in the main branch, for example:
diff --git a/examples/XGBoost-Examples/taxi/pom.xml b/examples/XGBoost-Examples/taxi/pom.xml
index 7dcfa92..dad8dfa 100644
--- a/examples/XGBoost-Examples/taxi/pom.xml
+++ b/examples/XGBoost-Examples/taxi/pom.xml
@@ -21,7 +21,7 @@
     <parent>
         <artifactId>sample_xgboost_examples</artifactId>
         <groupId>com.nvidia</groupId>
-        <version>0.2.3-SNAPSHOT</version>
+        <version>0.2.3</version>
     </parent>
     <modelVersion>4.0.0</modelVersion>

diff --git a/examples/XGBoost-Examples/utility/pom.xml b/examples/XGBoost-Examples/utility/pom.xml
index e84d9e2..15d5a97 100644
--- a/examples/XGBoost-Examples/utility/pom.xml
+++ b/examples/XGBoost-Examples/utility/pom.xml
@@ -21,7 +21,7 @@
     <parent>
         <artifactId>sample_xgboost_examples</artifactId>
         <groupId>com.nvidia</groupId>
-        <version>0.2.3-SNAPSHOT</version>
+        <version>0.2.3</version>
     </parent>
     <modelVersion>4.0.0</modelVersion>

For part1, I already push some commits to fix that
For part2, I already draft pr #398 to fix the version issue in branch-2406
For part3, I assume we could ignore them.

Yes, snapshot diffs are expected.  The other two need to be fixed.
Nit: Given there were a number of merge mistakes on this PR, it would be cleaner to wait for #398 to be fixed and then re-merge branch-24.06 to main in a fresh PR.  We're not squashing commits for this, and all these "fix" commits will appear in the mainline release.  The mainline history ideally should have a minimal number of divergent commits from the dev branches.  Not must-fix for me, just would be nice.

close this pr and draft new pr #401


Error occurred while deleting the node

Sealos Version
4.1.5-rc2
How to reproduce the bug?
sealos delete   --nodes <IP_ADDRESS>
are you sure to delete these nodes? Yes [y/yes], No [n/no] : y
Error: parameter error: to delete node IP <IP_ADDRESS>:22 must in cluster IP list
What is the expected behavior?
No response
What do you see instead?
No response
Operating environment
- Sealos version: 4.1.5-rc2
- Docker version: containerd v1.6.19
- Kubernetes version: v1.24.11
- Operating system:CentOS Stream 8
- Runtime environment:
- Cluster size:
- Additional information:

Additional information
No response

cat .sealos/default/Clusterfile

sealos delete --nodes <IP_ADDRESS> --force


Added a variable to declare a route concatenated with arguments

Added a route with argument contracts to make it easier to read, but if too many arguments to send, a workaround could be to use parcelable.

Hey, thanks for the contribution! However, we had some internal changes on navigation side just merged, so this PR won't fit in with the structure currently, apologies for that and please keep continue contributing, we really appreciate it :)


[Bug] cast string to float get wrong result

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
2.0.5 && master
What's Wrong?
case:
set experimental_enable_nereids_planner = true;
select cast("123.123" as float);
wrong result on 2.0.5 && master:
MySQL [(none)]> select cast("123.123" as float);
+--------------------------+
| cast('123.123' as FLOAT) |
+--------------------------+
|<PHONE_NUMBER>9863281 |
+--------------------------+
1 row in set (0.010 sec)

right result:
MySQL [(none)]> select cast("123.123" as float);
+--------------------------+
| CAST('123.123' AS FLOAT) |
+--------------------------+
|                  123.123 |
+--------------------------+
1 row in set (0.051 sec)

What You Expected?
work well
How to Reproduce?
No response
Anything Else?
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Hi @zxealous, maybe use decimal instead of float is helpful.
The manual mentions the optimization for decimal in Doris. Therefore, within the precision of 9 digits, decimal consumes 4 bytes which is the same as float.
The below example is from v2.0.5:
mysql> select cast("123.123" as decimal(9,3));
+------------------------------------+
| cast('123.123' as DECIMALV3(9, 3)) |
+------------------------------------+
|                            123.123 |
+------------------------------------+
1 row in set (0.01 sec)


perception: output database evaluation result

The current implementation only outputs evaluation results for each dataset, even if multiple datasets are described in the perception.
In database evaluation, we would like to output the results of multiple datasets together.

PR #17 outputs database_result.json but node exit with failure 1
Updating DataFrame: 100%|████████████████████| 116/116 [00:01<00:00, 103.29it/s]
Updating DataFrame: 100%|█████████████████████| 114/114 [00:01<00:00, 88.15it/s]
Traceback (most recent call last):
  File "/home/hyt/ros_ws/x2/install/driving_log_replayer/lib/driving_log_replayer/perception_database_result_node.py", line 69, in <module>
    main()
  File "/home/hyt/ros_ws/x2/install/driving_log_replayer/lib/driving_log_replayer/perception_database_result_node.py", line 63, in main
    rclpy.spin(perception_database_result)
  File "/opt/ros/humble/local/lib/python3.10/dist-packages/rclpy/__init__.py", line 218, in spin
    executor = get_global_executor() if executor is None else executor
  File "/opt/ros/humble/local/lib/python3.10/dist-packages/rclpy/__init__.py", line 102, in get_global_executor
    __executor = SingleThreadedExecutor()
  File "/opt/ros/humble/local/lib/python3.10/dist-packages/rclpy/executors.py", line 700, in __init__
    super().__init__(context=context)
  File "/opt/ros/humble/local/lib/python3.10/dist-packages/rclpy/executors.py", line 157, in __init__
    self._guard = GuardCondition(
  File "/opt/ros/humble/local/lib/python3.10/dist-packages/rclpy/guard_condition.py", line 23, in __init__
    with self._context.handle:
AttributeError: __enter__
Exception ignored in: <function Executor.__del__ at 0x7f6abf608c10>
Traceback (most recent call last):
  File "/opt/ros/humble/local/lib/python3.10/dist-packages/rclpy/executors.py", line 228, in __del__
    if self._sigint_gc is not None:
AttributeError: 'SingleThreadedExecutor' object has no attribute '_sigint_gc'
[ros2run]: Process exited with failure 1


update ethereumjs-util to solve bug concatenating signature

testrpc is using ethereumjs-util version 4.5.1
in order to solve this bug https://github.com/ethereumjs/ethereumjs-util/issues/57
it needs to be updated to version 5.0.1

Apologies we didn't get to this one sooner. ganache-core has been updated to use ethereumjs-util v5.0.1 for some time now.
Thanks for letting us know!


Hamburger icon disappears when viewing page in device emulator ( Chrome )

Hi , when I use device emulator in chrome on this page.
http://www.getmdl.io/components/index.html#layout-section/layout
The hamburger icon is gone. You can still activate navigation but the icon is invisible.

Works for me

Do you have any error messages in the console?

http://i.imgur.com/VGZ4ta5.png
Hm  , could it be that since I'm connected through Kerio client that it blocks font face?

That’s exactly what I did, @paran0a. Maybe @sgomes can give this a try, but I’m afraid this is a problem on your end :confused:

@paran0a Are you talking about the hamburger icon on the layout demo? If so, it's right there on your screenshot, only hard to see because it's overlaid on top of the building in the image.

@paran0a @surma Just tried this and it shows up for me as well :-/

I tried this at home and it works. It must be something with my network on work I guess.
Thank you for the help and tests!


Orphaned Arbitrary instances are brought into scope by the Blockfrost.Client

Actual situation
Fact 1:
The quickcheck-instances library has this warning in its documentation:

Since all of these instances are provided as orphans, I recommend that you do not use this library within another library module, so that you don't impose these instances on down-stream consumers of your code.

Fact 2:
Blockfrost API library uses quickcheck-instances
Fact 3:
blockfrost-api is a library that ignores the warning and causes problems associated with the orphaned instances to its users, even transitively, via the blockfrost-client dependency.
Desired situation
blockfrost-api doesn't bring orphaned instances to scope for all its downstream users.

Published on Hackage as blockfrost-api <IP_ADDRESS> and blockfrost-client <IP_ADDRESS> that allows api 0.8.


escape codes are not being escaped with echo

\nn.n.n / 2012-03-07 \n===========\n

echo by default unless aliased to echo -e won't escape thus resulting in what you see above.  This should allow it to work even if if there isn't an alias.
ref: http://linux.about.com/library/cmd/blcmdl1_echo.htm

i remember there being issues with this on some platforms, the ultimate fix was to just use printf in place of echo pretty much everywhere haha

printf will work too, but echo doesn't, i've seen a couple machines with the odd printout above, maybe printf with a \n in there ;)

@visionmedia I swapped echo out for printf if that helps

@visionmedia found out that i naively just swapped those two out without testing, and it produces an almost worse result than just having echo.  The only version that works great out of the box for me is:
https://github.com/wavded/git-extras/commit/39b4611d6d2ac601132d39d1156e5c0cbe0ead8e
So you can take it or leave it, either way I think that merge should get reverted as it probably will cause others problems.
On a side note I did some research on echo -e causing probs for people and I didn't come up with much, do you remember the specific problems you faced or references to cases discussion the issue?

crap haha

the problems with echo was related to the ansi escape sequences, so not directly related to this but I figured it should be fine


fix(docs): fixing links in docs

I've come across some broken links in the docs and tried to fix them, but don't know if it's caused by the import to the actual docs on beta.filamentphp.com app. Since it seems to need relative paths on e.g. "04-pages.md" but not on "06-navigation.md"
I hope this can at least show as a guide over which links needs some attention, as they either do nothing, or send the user to the https://beta.filamentphp.com/docs/3.x/panels/installation/ page

Thanks for bringing this up, @sooand.
We're aware of this issue and are working on a proper fix. 🙂


HTTPS certificate not validated by default

By default socketFactory and hostnameVerifier are null, which makes HTTPS connections pointless and is a huge security risk. This should be the default:
Manager.instance.socketFactory = javax.net.ssl.HttpsURLConnection.getDefaultSSLSocketFactory()
Manager.instance.hostnameVerifier = javax.net.ssl.HttpsURLConnection.getDefaultHostnameVerifier()

Yeah, thanks for pointing this out @jurgisp. At that time i naively pushed this to be client's responsibility. I agree that using default is a safer choice. Would you mind open to PR to fix this? I am more than happy to merge it in.

Thanks for quick response. Unfortunately I'm not familiar with the source code enough to know the best place to put the default initialization. From the client perspective I just execute these two lines when the app starts.

Fair enough. No problem. I will squeeze this in the next release. I will
leave this open as a reminder and close this when it is done.
Thanks for using Fuel.
On Sat, Mar 26, 2016 at 7:38 PM jurgisp<EMAIL_ADDRESS>wrote:

Thanks for quick response. Unfortunately I'm not familiar with the source
code enough to know the best place to put the default initialization. From
the client perspective I just execute these two lines when the app starts.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/kittinunf/Fuel/issues/66#issuecomment-201804004


tool tips position

Expected Behavior

the tool tips should show on the right hand side of the cursor
Current Behavior

the tool tips is showing below the cursor
Possible Solution

Steps to Reproduce (for bugs)

Context

Environment

Chart.js version:
Browser name and version:
Link to your project:

Closing due to lack of test case


Refactor ERC20 deposit flow

Issue
https://github.com/omgnetwork/web-wallet/issues/71
Issue when there is a successful approval but user skips the deposit. this blocks the user from making future erc20 deposits of that token due to how the approval/deposit flow was coded.
This removes the 2 popup signing on the standard erc20 deposit flow. Theres still the 2 popup sign if there was a previous allowance, but this should occur less. We could make that 3 steps as well, prompt to reset to 0, then prompt again to approve new amount, then finally prompt to deposit.
Changes
Splits up the erc20 deposit flow into multiple screens.
select erc20 deposit -> input deposit details -> check allowance and prompt user to make an approval/reset/or deposit transaction based on the allowance result.
standard scenario - user prompted to approve first, then asked to deposit

alternative scenario - allowance from a previous approval differs from the current deposit amount, user is prompted to reset their allowance before allowing the deposit transaction

open to suggestions on better wording/messaging.

This looks great, love the explicit requests for approval 😄
One comment - currently the allowance has to be exactly equal to the deposit amount, but what if it's greater than the deposit amount? We could go ahead and do the deposit without requiring an extra 'reset approve' call.

@kevsul ahh ok good point, did not realize it would accept a deposit thats less than whats already allowed 👍

updated to only ask for a reset when new deposit amount > allowance. if less than it will work


"Pătrușești" is listed in the drop-down menu

Description:
"Pătrușești" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Pătrușești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Pătrușești, Alba (Horea)".


Infer BindingSource.Body for collection types (again)

#8536 and changes #7712 resolution
do not infer string.Empty prefix for collections of simple types
also correct ModelMetadata.IsComplexType doc comments

nits:

make InferBoundPropertyModelPrefixes(...) and InferParameterModelPrefixes(...) conditions consistent
correct InferBindingSourceForParameter_ReturnsQueryForSimpleTypes() name
shorten a few lines

@pranavkm bump ⛰
(As discussed offline, this remains relevant when your new compat switch is off or the user doesn't add a binding attribute.)

Closing because @pranavkm is going to mix fixing the inferred BindingSource into his coming "stop when we see any BindingInfo" PR. Will reassign the bugs to him.
@pranavkm do you need my dougbu/collections.again.7712.8536 branch?


Add test to show problem in issue #72.

Issue seems to be solved already.

This turned out to be a problem in the console application, and not in the parser.