[api-replacement] replace json_rpc_invocations with Dev API in deploy.rs

Motivation
Currently, we are using a hard coded json-rpc value to deploy modules onchain. We want to remove all hard-coded json-rpc values from shuffle and replace them with the dev API. This PR replaces the json rpc value in deploy.rs with the dev API.
Test Plan
The goal is to mimic what deploy currently does. So I ran shuffle console and inside the console ran Shuffle.accountTransactions(). Notice how there are no transactions since I haven't deployed anything yet.

Then I ran shuffle deploy "project_path". Notice that 3 modules have deployed and hence 3 transactions have been created.

Now when I rerun Shuffle.accountTransactions() in console, I get these transactions. Notice the hash on the first transaction matches the hash of the first deployed module transaction in the picture above.

Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/diem/diem/tree/main/developers.diem.com, and link to your PR here.)
If targeting a release branch, please fill the below out as well

Justification and breaking nature (who does it affect? validators, full nodes, tooling, operators, AOS, etc.)
Comprehensive test results that demonstrate the fix working and not breaking existing workflows.
Why we must have it for V1 launch.
What workarounds and alternative we have if we do not push the PR.

/land


Add storage sdk methods to update flags

The current storage interface does not let you set/unset flags. We should add additional methods to update the flag field so users can set/unset the NO_AUTOBUMP flag.

As part of this, the NO_AUTOBUMP tests removed as part of https://github.com/stellar/stellar-core/pull/3824/commits/bef8df46049eb36964181dd8de1785d99d1b2276 should be added back in.

Closing, storage flags have been removed


Measure Interpreter call stack footprint

For some tests that call multiple contracts in higher depths, revm has a overflow. It will be great to measure and quantitate where is the place that takes most data and how much does the stack rises with every call.
For info pthread_crate default is around 2-4MB. Check Notes here https://man7.org/linux/man-pages/man3/pthread_create.3.html

This transaction causes the revm to blow the standard stack size: https://etherscan.io/tx/0x0c10fafe0cdbfff32abfe53d57ec861d09986cc1050c850481f79b1a862bb10a
It reaches 266 levels of recursion before it blows up.

Looks like each call is consuming around 7500 bytes of stack. The way I measured this is by tracking the remaining stack using stacker::remaining_stack() in https://github.com/bluealloy/revm/blob/main/crates/revm/src/inspector/customprinter.rs#L30
After a CALL opcode there is around 7500 bytes of consumed stack.
Still not sure what is the thing that consumes that much stack. I tried to Box the Interpreter as a potential large struct, but it didn't help https://github.com/bluealloy/revm/blob/v24/crates/revm/src/evm_impl.rs#L846
Any ideas are more than welcome 🤔

This transaction causes the revm to blow the standard stack size: https://etherscan.io/tx/0x0c10fafe0cdbfff32abfe53d57ec861d09986cc1050c850481f79b1a862bb10a
It reaches 266 levels of recursion before it blows up.

There are even some tests that fail for SO, so it can be easier to find those tests and just call revme (with e) and do tests there, should be easier. revme can be called as cargo run --release -- statetest path_to_folder.
https://github.com/bluealloy/revm/blob/main/bins/revme/src/statetest/runner.rs#L360

Looks like each call is consuming around 7500 bytes of stack. The way I measured this is by tracking the remaining stack using stacker::remaining_stack() in https://github.com/bluealloy/revm/blob/main/crates/revm/src/inspector/customprinter.rs#L30
After a CALL opcode there is around 7500 bytes of consumed stack.
Still not sure what is the thing that consumes that much stack. I tried to Box the Interpreter as a potential large struct, but it didn't help https://github.com/bluealloy/revm/blob/v24/crates/revm/src/evm_impl.rs#L846
Any ideas are more than welcome 🤔

Oh didn't know about stacker looks nice. 7.5kb per call
I would just spread the println and try to find out where it grows the most. Would focus when call/create is done.
My best bet was that Interpreted, maybe there are some other variables made inside function, or maybe it is related to function inputs as they are put to stack, but idk.

Oh didn't know about stacker looks nice. 7.5kb per call
I would just spread the println and try to find out where it grows the most. Would focus when call/create is done.

Unfortunately stacker is not that granular and doesn't update the estimation of the remaining stack as you progress inside the function. I managed to hook up lldb debugger, which allows to examine the stack frames and the variables in each frame in detail. As next steps I might try to dump the stack memory, using the stack register and try to get some info from there...

this is another tx that resulted in an overflow
https://etherscan.io/tx/0x4135170d3cdb3c4bf7029d52134b997661b71c3b6509b672e8d77f53bd1026a7/advanced#internal
1024 zero value calls

this is another tx that resulted in an overflow
https://etherscan.io/tx/0x4135170d3cdb3c4bf7029d52134b997661b71c3b6509b672e8d77f53bd1026a7/advanced#internal
1024 zero value internal transactions

lol, someone was live testing evm


WIP: Fixes #589. Allows 2+ roles to delegate to same role

Removes an incorrect check that prevents delegating to a role if any role has previously delegated to that role. (Previously, if X delegated to A, then Y was prevented from delegating to A.)
Adds a few tests.
See #589 for more details. (Note that the central fix, a removal of a few lines, has been merged already in #638, and so has been removed from this commit history.)

[x] The code follows the Code Style Guidelines
[x] Tests have been added for the bug fix or new feature
[x] Docs have been added for the bug fix or new feature

Will rebase and conflict resolve this due to merging of #638, which performs part of what this pull request was intended to do. This PR will then add the tests I recommended.

Rebased. The check I removed here, which you also removed in the later-merged #638, is no longer in the commit history here. I've changed the docs changes as well: while the changes were previously in tuf/README.md, that has been split into docs/CLI.md and docs/TUTORIAL.md, and so the same notes as were added before to tuf/README.md have now been added to those two documents. You may want to take a peek at the changed docs/CLI.md and make sure the extra note isn't too disruptive.

I don't find the changes to CLI.md too disruptive.

@awwad
I want to validate the correctness of the API redesign against this pull request, if it makes sense.
Has the task/test in scenario 2 (https://github.com/theupdateframework/tuf/pull/590#issuecomment-359841774) been implemented in this pull request?  I see that Travis is failing, but that is expected, correct?

The checks in the comment capture the current state -- the last two tests in the list still need to be added. There are more tests I'd want for the API redesign, I think, though I'd have to make sure I understand what you mean by that: is the redesign the fix for #660? (For example, is the expectation that the redesign will result in validation of a delegated role to always consult the appropriate delegating role to determine the keys to expect?)

is the redesign the fix for #660? (For example, is the expectation that the redesign will result in validation of a delegated role to always consult the appropriate delegating role to determine the keys to expect?)

That's correct!  The redesign of the API is outlined in https://github.com/theupdateframework/tuf/issues/574#issuecomment-358115382 and being worked on in https://github.com/vladimir-v-diaz/tuf/tree/refactor_low_level_api.

@awwad Is it okay if I finish the last two tests in https://github.com/theupdateframework/tuf/pull/590#issuecomment-359841774?  I'd like to use this pull request to test https://github.com/theupdateframework/tuf/pull/757.
I can let you know when pull request #757 is ready for a review.

Certainly

Due to imminent refactor efforts this code is unlikely to merge.


angular universal

do you have an example how to use it with angular universal ?
where to put the SvgServerLoader class becouse angular nodemodules dosen't have fs module

I do not have an example at hand, but maybe @freezy or @edulelis does?

any updates?)

push

Updates ?

@edulelis created an example project demo-universal-angular-svg-loader.

The basic stuff to get this working is:

As per the suggestion here.
Add this snippet to the package.json file to prevent compilation issues.

"browser": {
   "fs": false,
   "path": false,
   "os": false
}

Add ServerTransferStateModule to app.server.module
Add BrowserTransferStateModule to app.module
The loader should be different per platform, so the factory should receive the PLATFORM_ID and load the  correct class appropriately (this is already added in the example).

Thank you so much, we have great community here !

Cant this be closed? There is this documentation: https://github.com/czeckd/angular-svg-icon#usage-with-angular-universal


ARROW-5314: [Go] fix bug for String Arrays with offset

PTAL @stuartcarnie @alexandreyc

LGTM
I've identified the same problem with FixedSizeBinary and fixed it in #4334. Looks like there is no problem with Binary.


Initial implementation

Here is some code :D

how about it, @Matt-Esch ?

@nrw
We gave this module a proper 1.0.0 now. Added tests and docs as seperate PRs


feat: support adding traceID and spanID

We're currently processing your upload.  This comment will be updated when the results are available.


[ALLUXIO-3288] skip conf validation for getConf

https://alluxio.atlassian.net/browse/ALLUXIO-3288
If I set alluxio.zookeeper.enabled=true alone in alluxio-site.properties without setting alluxio.zookeeper.address, this error shows before this patch:
$ bin/alluxio getConf alluxio.zookeeper.enabled
Exception in thread "main" java.lang.ExceptionInInitializerError
	at alluxio.cli.GetConf.getConfImpl(GetConf.java:176)
	at alluxio.cli.GetConf.getConf(GetConf.java:141)
	at alluxio.cli.GetConf.main(GetConf.java:261)
Caused by: java.lang.IllegalStateException: Inconsistent Zookeeper configuration; alluxio.zookeeper.address should be set if and only if alluxio.zookeeper.enabled is true
	at com.google.common.base.Preconditions.checkState(Preconditions.java:176)
	at alluxio.conf.InstancedConfiguration.checkZkConfiguration(InstancedConfiguration.java:405)
	at alluxio.conf.InstancedConfiguration.validate(InstancedConfiguration.java:280)
	at alluxio.Configuration.validate(Configuration.java:375)
	at alluxio.Configuration.reset(Configuration.java:97)
	at alluxio.Configuration.<clinit>(Configuration.java:60)
	... 3 more

After this change:
$ bin/alluxio getConf alluxio.zookeeper.enabled
false

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/86/
Test FAILed.

@aaudiber PTAL.
I will cherry-pick this to master and remove the isHidden in master

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/95/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/97/
Test FAILed.

Only failing test is a known issue being fixed in another PR


Partial link styling in custom epub.css is overridden by ebook defaults.

I have a document with many links in the body, and therefore having many underlined links make the text unwieldy to read. One solution is to restyle the links as italics for example (in epub3.css), and attempt to remove the underlining.
  /* Modified epub3.css in local css folder */

a {
  background: transparent;

  /* Custom link styling */
  font-style: italic;
  text-decoration: none;
}

And build with:
$ asciidoctor-epub3 -a epub3-stylesdir=css@ master-epub.asciidoc

Although the italic font-style will take effect in iBooks or Kindle Previewer3, all links remain underlined. Given the experience of creating the epub3, epub3-css-only style sheets, I wonder whether the maintainers of this project know if is possible to style links in ebook readers without these attributes being overriden in the ebook reader? Many thanks!

My Google-fu and some experiments say that it is impossible to disable link underlining on Kindle.
There's underlining even with the following CSS:
a         { text-decoration: none !important; }
a:link    { text-decoration: none !important; }
a:visited { text-decoration: none !important; }
a:hover   { text-decoration: none !important; }
a:active  { text-decoration: none !important; }


Reorder pages in the introductory docs

:tada: This PR is included in version 3.12.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


From cdnjs, about library request.

Hi @nosir  ~
This is  cdnjs.
There is a lib request :point_right: https://github.com/cdnjs/cdnjs/issues/9631 , and we desire to contain cleave.js.:smiley:
However, I must to make a check for the files that we need to added.
The files I would like to add are : all the files under dist folder.
Is that complete for the user? Or other files required?
Thanks for your great work ✨
Piicksarn

Yep! Only the dist folder is needed to run cleave.js

Sorry for the late guys, still in vacation.
Yes, only the dist folder is complete for the user.
Thanks


termbg: init at 0.6.0

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 352093 run on x86_64-linux 1

1 package built:

termbg

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/2101


Migrate pygls to use new lsprotocol

Done https://github.com/openlawlibrary/pygls/pull/487

This was verified upstream by pygls.


Invariant Violation: Text strings must be rendered within a  component.

I'm facing the same errors.
react-native-webview = ^10.9.0
react-native = 0.61.5

npm install --save react-native-webview
simply put <WebView source={{ uri: 'https://www.example.com' }} />;
error appear

I also encountered the same problem and I finally found the reason is that the Usage Example contained a semicolon so that throw error.
import React, { Component } from 'react';
import { StyleSheet, Text, View } from 'react-native';
import { WebView } from 'react-native-webview';

// ...
class MyWebComponent extends Component {
  render() {
    return <WebView source={{ uri: 'https://reactnative.dev/' }} />;   // here
  }
}

Solution:
Just remove the semicolon.


Document the format of image digest status message from /push endpoint

The docker-maven-plugin can be used for Maven projects to push an image to the user's configured Docker registry at certain points in the build process. The plugin uses the docker-client library, which communicates with the docker daemon using the Remote API.
When pushing an image, we would like for the plugin to be able to be aware of the registry-assigned digest for the image, so that it can be communicated to downstream plugins / build steps.
docker-client tries to find the image digest by basically grepping through the JSON status messages sent by the /push endpoint. We (docker-maven-plugin maintainers) discovered that in docker v1.8 the format of the message changed from
Digest: sha256:dbad7f727f340f4bfa1160689eaf57a60242bf9a48cc2c4c14f383c9b66e8f06

to something like
0.2.1-SNAPSHOT-efd3f71.DIRTY: digest: sha256:8138f3c93c333aa1db6634ae77fce1a50febb9b189ba6fb2121f22810b7841b3 size: 41358

Would it be possible to document the format of the status message containing the image digest to make it less susceptible to unintended changes in future docker versions?

Is there a smarter way for the docker-client library to get the image digest after/during a push?

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

This is not going to be something which is documented since it is used like this internally only and not intended to be used in this manner. There is a smarter way to get the digest through using a custom JSONMessage and intercepting the message before it is formatted for the console. Some recent refactoring is making this look like a much more reasonable change and one we plan to make. This is mainly intended to be used for content trust but it may be possible to add a summary message at the end with all the digests which were pushed. I might suggest adding a proposal after the update to use JSONMessage directly is done.

@dmcgowan when you say

using a custom JSONMessage and intercepting the message before it is formatted for the console

do you mean adding something to the docker CLI to handle this?
Basically for our purposes, all we would really like is a clear way to be able to find out the digest of an image after it has been pushed, preferably through the remote API used to talk to the docker daemon.

@dmcgowan @stevvooe @aaronlehmann do you know if something was done about this? Is this still worth to implement?

The digest was added to the JSONMessage so it doesn't need to be parsed out of the textual output.
I think we can close this.

@aaronlehmann Thanks!


Find and Fix Bugs | Start here

Background
This issue provides directions on how you can contribute to Legit-Info by documenting bugs and fixing them. Please keep this issue open.
Tasks

[ ] Check out the open issues, ensure you're not replicating existing issues
[ ] Read & follow the instructions in the Getting Started section of the readme.md to set the solution up

As you run across any bugs (error messages, etc)

[ ] if your issue is more of a suggestion for improving the UI go to this issue instead
[ ] create a new issue using the bug template
[ ] label it 'hacktoberfest', 'bug'
[ ] Discuss the bug you found in the issue with @upkarlidder @tonypearson @demilolu

If you are able to fix the bug

[ ] assign yourself to the issue
[ ] add details on your proposed fix to the issue
[ ] Discuss the issue  with @upkarlidder @tonypearson @demilolu
[ ] submit a pull request to main branch
[ ] PR merged by Legit-Info team
[ ] Congratulations!

If you are unfamilar with pull requests, committing code on Github, we recommend you check out our free course on open source: ibm.biz/opensourcecourse

@demilolu Created. I can't assign it to myself or add labels, so if you could handle that part, I'd appreciate it. It's #117


rats-tls: how to support trusted communication between SGX node and CSV node

Background:
rats-tls need to support Secure and trusted communication of heterogeneous HW-TEE nodes.

As shown in above picture:
Step 1: enclave-tls-server running in SGX node will generate ecdsa cert and verify CSV cert.
Step2: enclave-tls-client running in CSV node will generate csv cert and verify SGX ecdsa cert.
Problem:
Right now, I have added CSV verifier libverifier_csv.so and attester libattester_csv.so.
The problem is how can enclave-tls-server running in SGX node verify CSV cert with libverifier_csv.so while generating SGX cert?  Since all codes are running enclave in SGX enclave, how can enclave-tls-server running in enclave registers csv verifier and calls csv verifier to verify  CSV cert?

It may be necessary to put the csv verifier related code in the enclave. Since csv verifier relies on SM2, rats-tls need to be adapted to babassl first.

Some comments.

In theory, CSV node and SGX node can serve both client and server.
In SGX build mode, all verifier instances have to be implemented in enclave.

A simpler way, you can consider porting the patchset of making OpenSSL in enclave to BabaSSL.

According to the discussion, there are 3 options:

OPT 1：cherry pick the necessary SM2/3 source code chunks to openssl used by rats-tls
OPT 2：call openssl/babassl high level APIs for SM2/3
OPT 3：implement enclavazation for babassl

Blocked by https://github.com/alibaba/inclavare-containers/issues/1349.


Refresh Tasks fails with Gradle 3.5.1 on Ubuntu 18.04 with JDK 1.8

Extension Name: vscode-gradle
Extension Version: v3.11.0
OS Version: Win 10 20H2 - OS Build 19042.1586
VSCode version: 1.66.2
Describe the bug
When opening a folder in VS Code in Ubunto 18.04 (WSL 2.0,) the task list fails to load in vscode even though the running gradlew from the terminal returns a list.
** Output from Gradle **
~/development/CFW/component-sample$ ./gradlew tasks --all
The ProjectDependency.getProjectConfiguration() method has been deprecated and is scheduled to be removed in Gradle 4.0.
`ModuleDependency.getConfiguration() has been deprecated and is scheduled to be removed in Gradle 4.0. Use ModuleDependency.getTargetConfiguration() instead.`
:tasks
------------------------------------------------------------
All tasks runnable from root project
------------------------------------------------------------
Application tasks
-----------------
bootRun - Run the project with support for auto-detecting main class and reloading static resources
Build tasks
-----------
assemble - Assembles the outputs of this project.
sample-client:assemble - Assembles the outputs of this project.
.... <long list of tasks > ...
BUILD SUCCESSFUL
Total time: 2.895 secs
To Reproduce
Run the Refresh tasks action
Expected behavior
The tasks from the dependencies should appear along with those from the build.gradle
Screenshots

Output from "Gradle for Java"
[debug] Refreshing tasks
[debug] Get daemon status
[info] Java Home: /usr/lib/jvm/adoptopenjdk-8-hotspot-amd64
[info] JVM Args: -XX:MaxPermSize=256m,-XX:+HeapDumpOnOutOfMemoryError,-Xmx1024m,-Dfile.encoding=UTF-8,-Duser.country,-Duser.language=en,-Duser.variant
[info] Gradle User Home: /home/rizzorc/.gradle
[info] Gradle Version: 3.5.1
[debug] Get daemon status: Cancelled on client
[debug] Get daemon status
[info] The ProjectDependency.getProjectConfiguration() method has been deprecated and is scheduled to be removed in Gradle 4.0.
ModuleDependency.getConfiguration() has been deprecated and is scheduled to be removed in Gradle 4.0. Use ModuleDependency.getTargetConfiguration() instead.
CONFIGURE SUCCESSFUL
Total time: 3.611 secs
[error] [gradle-server] Could not run build action using connection to Gradle distribution 'https://services.gradle.org/distributions/gradle-3.5.1-bin.zip'.
[error] Error getting build for /home/rizzorc/development/CFW/component-sample: Could not run build action using connection to Gradle distribution 'https://services.gradle.org/distributions/gradle-3.5.1-bin.zip'.
[info] Found 0 tasks
Does the bug still exist if you disable all other extensions?
Yes
Additional context
See attached txt file for sample build.gradle
build.gradle.txt

What you have seen is the default view, since the extension is trying to get the available tasks, the extension will show this view temporarily and after the exact tasks are available this view will be replaced.
The problem here is why the extension fails to get the tasks. Des your redhat-java extension work well?


initial commit

Description

[x] I have added a folder(Named it as my GitHub user id)
[x] I have added a Readme.md file inside the folder
[x] I understand the GitHub Forking Concept fully

Thanks again.


New command - Update-DbaSqlServiceAccount

Type of Change

[ ] Bug fix (non-breaking change, fixes #)
[x] New feature (non-breaking change, adds functionality)
[ ] Breaking change (effects multiple commands or functionality)
[x] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[x] Pester test is included
[ ] Nunit test is included
[ ] Documentation
[ ] Build system

Purpose
Configuring SQL Server service accounts using dbatools
Approach
Utilizing SetServiceAccount and ChangePassword methods of the ManagedComputer.Services collection.
Commands to test
See examples/pester
Screenshots

Learning

that detach test is bitchy - im going to disable it for now

Thanks so much, btw! Looking forward to adding this - should be tonight, after I disable that test 🌮

No prob, turns out that SetServiceAccount actually takes care of the service master key as well, so it was pretty easy to just wrap it with some digital noise and write a test 😄

Service master key and permissions! And probably other stuff too. It's awesome!

It still took around a minute or two to complete the test 🤢 Let me know if that's too much, I'll remove SQL Server engine from the test keeping only agent changes - it should cut the execution time at least in half - or even more!

We can keep as is until we decide against it 👍 💯 Helllllllll yeahhhhh! thanks for another awesome command!


How to get the panel to show and not my local web server?

Firstly just want to say thank you for this amazing library. Saves soo much time in kick starting a project compared to manually setting up a CEP extension.
That said though I'm having some issues with Adobe Animate and getting a panel to show. I can create a new adobe extension just fine and I can see it in Adobe Animate under the extensions without any issue. However when the panel opens it shows me my local web server's base page - Laragon in my case. Not quite sure what I'm doing wrong but guess it might be something to do with port numbers maybe?
Any help would be greatly appreciated. Thank you.

Thanks! Your port number is likely the issue. By default the bombino templates are set up to use 8080 because this is the default for Vue-CLI, but you can easily change this.
In your ./vue.config.js file at the root of the panel:
module.exports = {
  publicPath: "./",
  configureWebpack: {
    target: "node-webkit",
    node: false,
  },
  devServer: {
    port: 9090, // You can manually set a different port number here
  },
  lintOnSave: false,
};

Then in your ./public/index-dev.html file, we need to change the src of the iframe to match the port you've set above:
<!DOCTYPE html>
<html lang="en">
  <head>
    <meta charset="utf-8" />
    <title>docker-host</title>
  </head>
  <body style="margin: 0; border: 0; overflow:hidden;">
    <iframe
      enable-nodejs
      src="http://localhost:9090"
      style="border: 0; width: 100vw; height: 100vh;"
    ></iframe>
  </body>
</html>

Notice the src="http://localhost:9090" value in the <iframe> element above.
Just for some clarification, the way the panel works in a developer context is that it relies on a standalone port to serve the Vue content, which you must run with an npm run serve command on the root of the extension before the extension will load properly.
Adobe recently made changes to CORS values that messed up certain parts of bombino as well. Admittedly Animate wasn't one of the original apps that bombino was made for and I added support in for it later -- I had to recently change the way evalScript works but I don't know that I added support for Animate. I can fix this for you now that I know someone's out there actually in need of this. If you notice issues with trying to run scripts through the evalScript command (which must be used through the workaround package to work in both DEV and PROD contexts as of CEP 11) be sure to copy the recent version of this file to your ./public/createEventChain.js file.

Thank you soo much once again for this and for you speedy reply. Figured it has something to do with ports but just wasn't sure where to start. That works perfectly now. I'll keep testing and test you know. I've wanted to make a banner ad extension for adobe animate for some time but could never wrap my head around setting up a CEP extensions from scratch.
This project you've made is absolutely perfect and it uses Vue as well. Love you work!

Thanks!
Just so you know, CEP support for Animate isn't that great. Certain aspects of CEP integration in AN seem like an afterthought because they work so differently from the way other host apps do, and I've filed several bugs in AN that have never been resolved. The panel background color is not accurate and CEP only reports 2 of a possible 4 UI themes, so the workaround for this is that the context menu has a "Switch Theme" option when using <Menus> components that will toggle between Dark/Darkest variants (but this only needs to be set once).
Unfortunately CEP 11 has broken the <Menus> component for flyout and context menus in bombino templates, but only while in Dev context. These work inside production context still. I'm still working on solving this but running into even more bugs with CEP 11 where I've not been able to get window.postMessage to accurately handle the menu interaction -- or, I have, but there's very buggy behavior I haven't been able to pin down and solve for yet. You may have a better DX overall if you were to use AN 2020 rather than '21 to bypass this, but as of right now the only issues in bombino left are the flyout/context menus (which frustratingly used to work beautifully).
Thanks! I'll follow you and keep an eye out for your projects. Feel free to file other issues, though I'm going to assume we can close this one now 👍

Ah that should be OK. I'm using it for an older copy of Animate. 2019 Which I believe uses CEP 9. The new Animate (2020 and 2021) are buggy as and it seems like Adobe breaks more stuff than they fix with it. Just need an app to churn out banner ads quicker and this will get me there now.

In that case, I'd recommend using the evalScript method from the package named cluecumber, and <Panel>/<Menu> components from the package named lokney. These were both pre-CEP 11 and you should be set rather than using the post-11 versions in the current brutalism/workaround packages. Sorry if that seems confusing, ha.

Updated the above to link to the packages directly 👍


Add JRuby executable example, fixes #2412

This is supposed to fix JRuby executable detection. See #2412 for details.

@yegortimoschenko Thanks! Could you also add a sample for rbx? (It's better if it comes from a real application)

@pchaigno Done! Drop me a line if you'd like a rebase.

It looks good to me. You'll have to wait for someone from GitHub to review it now ;)
For reviewers: jruby examples and rbx examples.

:+1: thanks @yegortimoschenko. This will go live in the next release of Linguist (likely another week or so)


It is possible to have separate ReactSiteConfigurations for each MVC area?

We're currently using MVC areas in our project and would love to have separate ReactSiteConfigurations per site. Currently, even if you put the ReactSiteConfiguration within the AreaRegistration, the configuration gets shared between all configured areas.
Is there any easy fix for being able to separate the ReactSiteConfigurations on a per-area basis?

Most likely no. This library makes heavy use of static properties to access
the current React configuration and JS instance. It probably could be
reworked to use the .NET Core service injection system but I don’t know if
that is area specific.
I recommend separate Web projects instead of using Areas, with common logic
in a shared helper class library.
On Wed, Apr 22, 2020 at 06:56, ipoley-r7<EMAIL_ADDRESS>wrote:

We're currently using MVC areas in our project and would love to have
separate ReactSiteConfigurations per site. Currently, even if you put the
ReactSiteConfiguration within the AreaRegistration, the configuration
gets shared between all configured areas.
Is there any easy fix for being able to separate the
ReactSiteConfigurations on a per-area basis?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/reactjs/React.NET/issues/1100, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHGCFT7ABXWXZ2DIUPXPHTRN3ZRFANCNFSM4MOGAWTQ
.


Support meta type of [object Arguments]

When making a call such as winston.debug("myFunction", arguments); from within a function, the arguments object is appended to the message after being formatted with util.format. This doesn't happen with 'regular' object. Since Arguments is a valid built-in type of Object, it should be supported.
Example:
http://jsbin.com/sefetileve/1/edit?js,console

Howdy! Thank you for your contribution. This PR is against the<EMAIL_ADDRESS>release line. Since<EMAIL_ADDRESS>is now out we are only accepting major bug fixes.

It would have been nice to have this merged in more than a year ago then. Do you happen to know if this has been addressed in winston@3?


adding missing documentation and readinessProbe

Which issue this PR fixes : fixes https://github.com/helm/charts/issues/7569
Special notes for your reviewer:

/assign @davidkarlsen

/lgtm


How can read from a different app?

I have two apps that have common users and can go from one app to another. I need to save data like password and email so that I can access it from the other application. Is it possible and safe?

Hey @endritgojani, sorry for the late response.
The plugin is a thin wrapper on top of iOS Keychain and KeyStore/SharedPref. So you need to check the capabilities of them. Afaik it's possible for iOS, but I am not sure about Android.
Please use Stackoverflow for general questions.


Remove app name specification from library

Library should not specify application label and app_name string
Came across this while setting up new flavor. While app_name string was still missing my app name was set to TrustKit

Thanks!


Mirage creating new id rather than using client-generated id

I'm using generateIdForRecord in a DS.Adapter to pass a client-side generated id when creating.  Mirage is generating it's own id value when it stores the record causing Ember to return the error "An adapter cannot assign a new id to a record that already has an id." when the payload returned uses a different id than the id already specified on the DS model.  I've verified that the "id" is passed within the payload as expected.

Is this using the new Mirage 0.2 betas?

I just updated to the latest beta (0.2.0-beta.3) and tested again with the same result.  Previously I was on a fork of one of the 0.2 betas.

Would you be able to produce a failing test that shows this edge case? There are provisions in the DB's IdentityManager to allow for setting, and a few test cases (example). That indicate this functionality should be working as expected.
I'm not terribly familiar with DS Adapter's generateIdForRecord, so I could use a hand :)

I'm trying to figure out a test case (I'm new to ember mirage), but I think I see the problem in the code.  The check for an id is only looking at the request parameters.  https://github.com/samselikoff/ember-cli-mirage/blob/master/addon/route-handlers/shorthands/base.js#L49
But the id is actually set within the request body.  It is sent in the JSONapi document that is parsed out here: https://github.com/samselikoff/ember-cli-mirage/blob/master/addon/route-handlers/shorthands/base.js#L49
For example, by data (jsonApiDoc variable) looks like:
{   data {     attributes: {       label: 'test'     },     id: 'my-client-side-id',     type: 'my-model-type'   } }

Whoa! Very interesting. Are you good to go from here at this point, or do you need a hand putting together a pull request? I'm more than happy to help if need be.

I just started to put one together but I noticed that the id gets forced to an integer.  This wouldn't be the case with client-side generated ids, necessarily.  Do you know if there is a requirement that ids be integers for mirage tables?

I think you just found a second bug by accident. Theoretically all IDs in Mirage should be treated as strings (see #462). This parsing shouldn't be happening.

Yay - my first pull request for an Ember project :)

Congrats! :cake:
The only thing I'd recommend is noting in your commit message why you're removing the ID integer parsing, or at least noting that it should have been removed in #462 .

Thanks - valid point :+1:


Local variable gets overwritten causing wrong gdb debugging display

Reporting a bug

[x] I have tried using the latest released version of Numba (most recent is
visible in the change log (https://github.com/numba/numba/blob/master/CHANGE_LOG).
[x] I have included a self contained code sample to reproduce the problem.
i.e. it's possible to run as 'python bug.py'.

# File simple_kernel.py
from numba import jit
import numpy as np

@jit(nopython=True, debug=True)
def double_add_1(a):
    one = 1.0
    result = a * 2 + one # breakpoint
    return result

def main():
    sz = 100
    a = np.ones((sz,), np.float32)
    out = double_add_1(a)

if __name__ == "__main__":
    main()

Reproducing debugging commands:
# shell
NUMBA_OPT=0 gdb python
# gdb commands
b simple_kernel.py:8
r simple_kernel.py
p one

one is expected to be 1.0, but is printed 0. The lines around the stopped at assembly from disass:
   0x00007fffe15f1339 <+825>:   movabs $0x3ff0000000000000,%r10
   0x00007fffe15f1343 <+835>:   mov    %r10,0x2b8(%rsp)
   0x00007fffe15f134b <+843>:   movq   $0x0,0x2b8(%rsp)
=> 0x00007fffe15f1357 <+855>:   movq   $0x2,0x2b0(%rsp)
   0x00007fffe15f1363 <+867>:   movq   $0x0,0x2b0(%rsp)

It looks like the constant 1.0 is loaded to 0x2b8(%rsp) but was immediately set to 0 again. In gdb, info one address gives
Symbol "one" is a variable at frame base reg $rsp offset 0+696.
The below contains full disass output and llvm_dump
disass_llvm_dump.zip

There were a few recent GDB related PRs merged. This might be fixed. Need to verify on the latest commit.

Hey @sklam, just built numba from the latest commit and noticed that the above behavior still exists.

Thanks for reporting this. I think this is down to a mixture of constant propagation/rewrites and variable lifetimes. This is the Numba IR that is lowered. Note how one is defined and then immediately deleted, and the arrayexpr doesn't refer to one but instead refers to a propagated const(float, 1.0).
$ NUMBA_DEBUG_PRINT_AFTER='ir_legalization' NUMBA_OPT=0 python issue7616.py
---------------------------------__main__.double_add_1: nopython: AFTER ir_legalization---------------------------------
label 0:
    a = arg(0, name=a)                       ['a']
    one = const(float, 1.0)                  ['one']
    del one                                  []
    $const8.2 = const(int, 2)                ['$const8.2']
    del $const8.2                            []
    result = arrayexpr(expr=(<built-in function add>, [(<built-in function mul>, [Var(a, issue7616.py:7), const(int, 2)]), const(float, 1.0)]), ty=array(float64, 1d, C)) ['a', 'result']
    del a                                    []
    $20return_value.7 = cast(value=result)   ['$20return_value.7', 'result']
    del result                               []
    return $20return_value.7                 ['$20return_value.7']

One of the improvements to debugging on HEAD (that will be in 0.55) is accessed via a new environment variable NUMBA_EXTEND_VARIABLE_LIFETIMES. This will extend the lifetime of variables to the end of a basic block, this giving a more Python-like debugging experience.
The docs built from mainline are here: https://numba.readthedocs.io/en/latest/user/troubleshoot.html#debugging-jit-compiled-code-with-gdb and have been updated to reflect the improved debugging experience. Setting the environment variable https://numba.readthedocs.io/en/latest/reference/envvars.html#envvar-NUMBA_EXTEND_VARIABLE_LIFETIMES will prevent the variable one from being zero'd (the zeroing is a side effect of deletion). Example:
$ NUMBA_OPT=0 NUMBA_EXTEND_VARIABLE_LIFETIMES=1 gdb -q --args python issue7616.py
Reading symbols from python...
(gdb) set breakpoint pending on
(gdb) b issue7616.py:8
No source file named issue7616.py.
Breakpoint 1 (issue7616.py:8) pending.
(gdb) r
<snip>
Breakpoint 1, __main__::double_add_1_241[abi:c8tJTC_2fWgEeGLSgydRTQUgiqKEZ6gEoDvQJmaQIA](Array<float, 1, C, mutable, aligned>)
    (a=...) at issue7616.py:8
8           result = a * 2 + one # breakpoint
(gdb) p one
$1 = 1
(gdb) disassemble
Dump of assembler code for function __main__::double_add_1_241[abi:c8tJTC_2fWgEeGLSgydRTQUgiqKEZ6gEoDvQJmaQIA](Array<float, 1, C, mutable, aligned>):
   <snip>
   0x00007fffd933d339 <+825>:   movabs $0x3ff0000000000000,%r10
   0x00007fffd933d343 <+835>:   mov    %r10,0x2b8(%rsp)
=> 0x00007fffd933d34b <+843>:   movq   $0x2,0x2b0(%rsp)
   0x00007fffd933d357 <+855>:   mov    0x2c0(%rsp),%r10

We're hoping to enhance debugging further in a future release (probably 0.56) so as to put more of this configuration into the debug= kwarg on the @jit decorator.
Hope this helps.


Disable edit button on learn.microsoft.com

Pages on learn.microsoft.com have an "edit" pencil icon which will take the user to the docs repo. Since the overwhelming majority of content in those repos is automatically generated, opening PRs against auto-generated content is inadvisable. Remove the opportunity to make those mistakes.

Users can still report feedback by scrolling to the bottom of the page and clicking on the "This page" button in the "Feedback" section. This will direct the user to open an issue in the SDK source repository with relevant information for product teams to review and act.

To do this set the open_to_public_contributors property in the .openpublishing.publish.config.json file.
Example:
https://github.com/Azure/azure-docs-sdk-dotnet/blob/main/.openpublishing.publish.config.json#L74
Documentation:
https://review.learn.microsoft.com/en-us/help/onboard/admin/publish-configuration?branch=main

This does not impact upcoming changes.


Nomination for contributor triage rights for marco85mars20

Nomination
I would like to nominate contributor marco85mars20 with
Triage permission level, which allows the user to add reviewers to a pull
request and be added as a reviewer by other users.
GitHub User
Marco Argiolas
marco85mars20
FTP Technologies
Supporting Documents
https://github.com/zephyrproject-rtos/zephyr/pulls?q=is%3Apr+is%3Aclosed+author%3Amarco85mars20
Thanks

unassigning from myself, usually @nashif or @stephanosio look into these
I thought you were going to self nominate :-)
Can we give triage permissions to @nixward as well while at it? Plenty of code in the repo: https://github.com/zephyrproject-rtos/zephyr/pulls?q=is%3Apr+is%3Aclosed+author%3Anixward+

No problem @fabiobaltieri. I believe I already have triage permissions.

invite sent https://github.com/zephyrproject-rtos/zephyr/invitations


Baseline MAINTAINERS, CODEOWNERS, and external collaborator permissions

Follow https://github.com/opensearch-project/.github/issues/125 to baseline MAINTAINERS, CODEOWNERS, and external collaborator permissions.
Close this issue when:

MAINTAINERS.md has the correct list of project maintainers.
CODEOWNERS exists and has the correct list of aliases.
Repo permissions only contain individual aliases as collaborators with maintain rights, admin, and triage teams.
All other teams are removed from repo permissions.

If this repo's permissions was already baselined, please confirm the above when closing this issue.

Permissions are incorrect. Please work with @bbarani's team to remove the group and add individuals with maintain access.

Fixed the permissions so closing this issue.

Verified, thx.


imports correction

changed imports to relative in order for installed module to work.
pip install -e . works!
start-bot works from bash

 looks like that correction breaks the code. at least at my system

what exactly is brocken?

 looks like that correction breaks the code. at least at my system

this is for installation and running of module by command start-bot, not just python main.py


Create User database to store miles, lastAnsweredAt

The users collection firebase now stores the following fields upon registering an account:
{
    miles: int;
    lastUpdatedAt: DateTime;
}

Adds functions in user_database.dart to update the database and user_database_providers to retrieve from the database. The functions in user_database_providers might not work lol.

gonna move the files to lib/features/user


Add support for .mcas.ms domain

Describe your changes
This PR adds support for the .mcas.ms domain, which is used when the org is protected behind Microsoft Defender for Cloud Apps.
Note: In that situation, the sid cookie is never usable by the extension, because the sid cookie available to the .mcas.ms domain is encrypted by Microsoft Defender for Cloud Apps, and cannot be used as-is.
This means that Microsoft Defender for Cloud Apps users must always use Salesforce Inspector with a Connected App.
With Microsoft Defender for Cloud Apps, a valid access token must be generated before using the extension.
Issue ticket number and link
Solves #234
Checklist before requesting a review

[X] I have read and understand the Contributions section
[X] Target branch is releaseCandidate and not master
[X] I have performed a self-review of my code
[X] I ran the unit tests and my PR does not break any tests
[X] I documented the changes I've made on the CHANGES.md and followed actual conventions
[ ] I added a new section on how-to.md (optional)

Hi @mehdisfdc ,
Thanks for the PR ! 👏
I have to perform some tests on .mil org but everything seems to be perfect 🎉

Works like a charm on .mil org and no regression with connected app (without microsoft domain)


Changelog on PR's since last tag?

Sorry, more of a question but ... I have release workflow that is triggered manually on the main branch. In that workflow I have these steps: 1) checkout 2) install GitVersion 3) build and package in zip-file 4) generate changelog 5) create tag&release and publish.
The problem that I have is that the tag is created in step 5, where step 4 is taking the PR's from tag n and n-1. Is there a way to have the action get the PR's from the last tag until 'now' (so, without a new tag)? Or what would be a better way to solve this?

@mtirion you could provide the Git SHA-1 to the toTag config, and the previous tag as fromTag so the action won't try to identify these based on the found tags on the repo.

Would it be a possibility to place the git tag before step 4 and do the release afterwards?

Thanks for the quick reply @mikepenz! I will try that, makes sense. For creating the tag and release and upload I use the GitHub Action ncipollo/release-action@v1 that does it all. I've tried to change it to separate steps, and than rearrange the steps, but I ran into all kinds of other issues. I'll try your solution first ... and will post the result here for others to learn :)

This action itself for example does it like this: https://github.com/mikepenz/release-changelog-builder-action/blob/develop/.github/workflows/ci.yml#L105-L119
(Tag being the trigger for the release procedure to run)

Here is what I did now (which works). I've set the fromTag to ${{ steps.gitversion.outputs.VersionSourceSha }} and toTag to ${{ steps.gitversion.outputs.Sha }}. These are the variables provided by GitVersion. This gives me the correct result in the CHANGELOG.
The reason that I didn't chose for tag-creation to be the trigger, is that I want the versions to be done using GitVersion and not manually. This way I can also create a tag for bumping up the major version for instance without immediately publishing the release for it.
Thanks for thinking about another option though. Much appreciated.

@mtirion thank you for the details on how you solved it.
Always great to see the action in use in different environment.


Delete memorytamer.rb

Don’t see any link on the website (without having to buy).

Merged as 369f9c312b792623c2dfe4928c42bfddad1fedef.


Docs: with-docker example apk update before apk add

What is the improvement or update you wish to see?
The Dockerfiles in the with-docker example show the following code:
RUN apk add --no-cache libc6-compat
RUN apk update

I'm no alpine expert, but from what I understand it doesn't make any sense to run apk update after apk add. The alpine wiki specifically mentions that you should run apk update before apk add.
Our linter has caught this and warns that performing the operations this way can increase the Docker image size.
Is there a reason for performing these commands in this order? If so, can we have the reason for this added to the documentation (e.g. a comment on the Dockerfiles)?
Is there any context that might help us understand?
Examples pulled from:
https://github.com/vercel/turbo/blob/main/examples/with-docker/apps/web/Dockerfile
https://github.com/vercel/turbo/blob/main/examples/with-docker/apps/api/Dockerfile
Does the docs page already exist? Please link to it.
https://turbo.build/repo/docs/guides/tools/docker#example

Thanks for pointing this out! Example fixed and docs update is rolling.


call ACK when receive message

Hi
it seems to be no way to call ack when receive message
sio::socket::event_listener_aux([&](string const& name, message::ptr const& data, bool isAck, message::list& ack_resp)
what is ack_resp for? It suppose to be a function to call instead of message

this is how java implement ack.
public void call(final Object... args) {
EventThread.exec(new Runnable() {
public void run() {
if (!sent[0]) {
sent[0] = true;
Socket.logger.fine(String.format("sending ack %s", args.length != 0 ? args : null));
int type = HasBinary.hasBinary(args) ? 6 : 3;
Packet packet = new Packet(type, new JSONArray(Arrays.asList(args)));
packet.id = id;
Socket.this.packet(packet);
}
}
});
}
Need to find out

I got it works by this:
packet p(packet::type_ack);
send_packet(p);
this works as Ack.call() of java

Hi,
I'm having the same issue: I just want to send response on a request. Can you please extend your solution?


fix: hot type

Summary

Test Plan

Require Documentation?

[ ] No
[ ] Yes, the corresponding rspack-website PR is __

!canary


This repo is not currently updated, use matterinc/web3swift instead

As the main contributors of this project I request to put this notice back.
We are agreed to leave this project "as is" for the good of BANKEX team.
Since this repo not supported by bankex, please make a reference to actually updated library in order to not confuse the developers.
You are welcome to contribute to this project, but leave this to mention on the place as a kind of respect to our work.

Important notice:
This version is not currently supported updated.
Use matterinc/web3swift instead.

As we've left the company BANKEX and I was the main enthusiast maintainer of this project, all the further work and updates will be done in the new repo.
Sincerely,
Alex (@shamatar)

We will continue working at web3swift library within our core team in our new company.
We are committed to make it more powerful and push new updates to matterinc/web3swift
Regards,
Petr Korolev (@skywinder)

This is revert commit of https://github.com/BANKEX/web3swift/commit/fd6e78b3530980b15a0c0858d445a006cc039364#diff-04c6e90faac2675aa89e2176d2eec7d8

Related to the last update in this repo too: https://github.com/BANKEX/web3swift/pull/295


Summary - System Tasks

Link

https://netflix.github.io/conductor/metadata/systask/

Dynamic Task
Parameters

dynamicTaskNameParam

Name of the parameter from the task input whose value is used to schedule the task. e.g.
if the value of the parameter is ABC, the next task scheduled is of type 'ABC'.

Example
{
  "name": "user_task",
  "taskReferenceName": "t1",
  "inputParameters": {
    "files": "${workflow.input.files}",
    "taskToExecute": "${workflow.input.user_supplied_task}"
  },
  "type": "DYNAMIC",
  "dynamicTaskNameParam": "taskToExecute"
}

If the workflow is started with input parameter user_supplied_task's value as user_task_2
Conductor will schedule user_task_2 when scheduling this dynamic task.

Decision

A decision task is similar to case...switch statement in a programming language.
The task takes 3 parameters

Parameters

name
description

caseValueParam
Name of the parameter in task input whose   value will be used as a switch.

decisionCases
Map where key is   possible values of caseValueParam with value being list of tasks to be executed.

defaultCase
List of tasks to be   executed when no matching value if found in decision case (default condition)

Example
{
  "name": "decide_task",
  "taskReferenceName": "decide1",
  "inputParameters": {
    "case_value_param": "${workflow.input.movieType}"
  },
  "type": "DECISION",
  "caseValueParam": "case_value_param",
  "decisionCases": {
    "Show": [
      {
        "name": "setup_episodes",
        "taskReferenceName": "se1",
        "inputParameters": {
          "movieId": "${workflow.input.movieId}"
        },
        "type": "SIMPLE"
      },
      {
        "name": "generate_episode_artwork",
        "taskReferenceName": "ga",
        "inputParameters": {
          "movieId": "${workflow.input.movieId}"
        },
        "type": "SIMPLE"
      }
    ],
    "Movie": [
      {
        "name": "setup_movie",
        "taskReferenceName": "sm",
        "inputParameters": {
          "movieId": "${workflow.input.movieId}"
        },
        "type": "SIMPLE"
      },
      {
        "name": "generate_movie_artwork",
        "taskReferenceName": "gma",
        "inputParameters": {
          "movieId": "${workflow.input.movieId}"
        },
        "type": "SIMPLE"
      }
    ]
  }
}

Fork

Fork is used to schedule parallel set of tasks.

Parameters

forkTasks

A list of list of tasks.
Each sublist is scheduled to be executed in parallel.
However, tasks within the sublists are scheduled in a serial fashion.

Example
{
  "forkTasks": [
    [
      {
        "name": "task11",
        "taskReferenceName": "t11"
      },
      {
        "name": "task12",
        "taskReferenceName": "t12"
      }
    ],
    [
      {
        "name": "task21",
        "taskReferenceName": "t21"
      },
      {
        "name": "task22",
        "taskReferenceName": "t22"
      }
    ]
  ]
}

When executed, task11 and task21 are scheduled to be executed at the same time.

Dynamic Fork

A dynamic fork is same as FORK_JOIN task.
Except that the list of tasks to be forked is provided at runtime using task's input.
Useful when number of tasks to be forked is not fixed and varies based on the input.

name
description

dynamicForkTasksParam
Name of the parameter that contains list of   workflow task configuration to be executed in parallel

dynamicForkTasksInputParamName
Name of the   parameter whose value should be a map with key as forked task's reference   name and value as input the forked task

Example
{
  "inputParameters": {
     "dynamicTasks": "${taskA.output.dynamicTasksJSON}",
     "dynamicTasksInput": "${taskA.output.dynamicTasksInputJSON}"
  }
  "type": "FORK_JOIN_DYNAMIC",
  "dynamicForkTasksParam": "dynamicTasks",
  "dynamicForkTasksInputParamName": "dynamicTasksInput"
}

Consider taskA's output as:

{
  "dynamicTasksInputJSON": {
    "forkedTask1": {
      "width": 100,
      "height": 100,
      "params": {
        "recipe": "jpg"
      }
    },
    "forkedTask2": {
      "width": 200,
      "height": 200,
      "params": {
        "recipe": "jpg"
      }
    }
  },
  "dynamicTasksJSON": [
    {
      "name": "encode_task",
      "taskReferenceName": "forkedTask1",
      "type": "SIMPLE"
    },
    {
      "name": "encode_task",
      "taskReferenceName": "forkedTask2",
      "type": "SIMPLE"
    }
  ]
}

When executed, the dynamic fork task will schedule two parallel task of type "encode_task" with reference names "forkedTask1" and "forkedTask2" and inputs as specified by _ dynamicTasksInputJSON_

Join

Join task is used to wait for completion of one or more tasks spawned by fork tasks.

Parameters

joinOn

List of task reference name, for which the JOIN will wait for completion.

Example
{
    "joinOn": ["taskRef1", "taskRef3"]
}

Join Task Output

Fork task's output will be a JSON object with key being the task reference name and value as the output of the fork task.

Sub Workflow

Sub Workflow task allows for nesting a workflow within another workflow.

Parameters

subWorkflowParam

List of task reference name, for which the JOIN will wait for completion.

Example
{
  "name": "sub_workflow_task",
  "taskReferenceName": "sub1",
  "inputParameters": {
    "requestId": "${workflow.input.requestId}",
    "file": "${encode.output.location}"
  },
  "type": "SUB_WORKFLOW",
  "subWorkflowParam": {
    "name": "deployment_workflow",
    "version": 1
  }
}

When executed, a deployment_workflow is executed with two input parameters requestId and file.
The task is marked as completed upon the completion of the spawned workflow.
If the sub-workflow is terminated or fails the task is marked as failure and retried if configured.

Wait

A wait task is implemented as a gate that remains in IN_PROGRESS state unless marked as COMPLETED or FAILED by an external trigger.
To use a wait task, set the task type as WAIT

External Triggers for Wait Task

Task Resource endpoint can be used to update the status of a task to a terminate state.
Contrib module provides SQS integration where an external system can place a message in a pre-configured queue that the server listens on.
As the messages arrive, they are marked as COMPLETED or FAILED.

SQS queue

SQS queues used by the server to update the task status can be retrieve using the following API:

GET /queue

When updating the status of the task, the message needs to conform to the following spec:

Message has to be a valid JSON string.
The message JSON should contain a key named externalId with the value being a JSONified string that contains the following keys:

workflowId: Id of the workflow
taskRefName: Task reference name that should be updated.

Each queue represents a specific task status and tasks are marked accordingly. e.g. message coming to a COMPLETED queue marks the task status as COMPLETED.
Tasks' output is updated with the message.

Example SQS Payload:
{
  "some_key": "valuex",
  "externalId": "{\"taskRefName\":\"TASK_REFERENCE_NAME\",\"workflowId\":\"WORKFLOW_ID\"}"
}

HTTP

Event

Chech it
Dynamic Task

not clearly understand
see code and demo, and figure out how it works

Decision

no defaultCase at example, figure out how it works
What if case is not that simple? Production level workflow is not simple

Dynamic Fork

How to generate well?
not clearly understand about FORK_JOIN_DYNAMIC limitation and how join works

Wait

Depends on implementation, waiting could be a point of stress.
figure out how it works
SQS is not for my case, how can we replace it to rabbitmq or kafka?

Common

System failure could be happened across multiple servers
How can we monitor it, counteract it, and recover it?


deploying vault using vault-operator

Is this project still active? i see no issues after Apr 2019 for this project. I am trying to setup vault in kubernetes, unfortunately I could not deploying using vault-helm then landed here to use vault-operator and able to deploy the vault-operator and there are no issue/error message but example vault is not showing the status in the service section. I do not know how to access the vault URL and start using it.
if this project active, I shall post the output of vault-operator to understand better about my issue.

Here is the status of what I have done so far. The vault is not initialized. And in my k8s cluster, i can only get the external IP using servicetype LoadBalancer, so how to make this vault to use the servicetype Loadbalancer to get the external IP.
/github/vault-operator# kubectl get pods
NAME                              READY   STATUS    RESTARTS   AGE
etcd-operator-f59c5ddc9-zh2hr     1/1     Running   0          32m
vault-operator-7985d585d7-kx6jm   1/1     Running   0          31m
~/github/vault-operator# kubectl get service
NAME      TYPE        CLUSTER-IP      EXTERNAL-IP   PORT(S)                      AGE
example   ClusterIP   <IP_ADDRESS>           8200/TCP,8201/TCP,9102/TCP   6m42s
~/github/vault-operator# kubectl get vault example -n vault -o json | jq .status
{
"clientPort": 8200,
"initialized": false,
"phase": "Running",
"serviceName": "example",
"vaultStatus": {
"active": "",
"sealed": null,
"standby": null
}
}
ERROR: logging before flag.Parse: I1203 16:07:25.007388       1 leaderelection.go:184] successfully acquired lease vault/vault-operator
time="2019-12-03T16:07:25Z" level=info msg="starting Vaults controller"
time="2019-12-03T16:07:25Z" level=info msg="Event(v1.ObjectReference{Kind:"Endpoints", Namespace:"vault", Name:"vault-operator", UID:"fecf1ffb-15e6-11ea-b650-005056bbf202", APIVersion:"v1", ResourceVersion:"5933846", FieldPath:""}): type: 'Normal' reason: 'LeaderElection' vault-operator-7985d585d7-kx6jm became leader"
time="2019-12-03T16:07:25Z" level=info msg="Vault CR (vault/example) is created"
time="2019-12-03T16:09:06Z" level=error msg="error syncing Vault (vault/example): reconcile Vault failed: deploy etcd cluster failed: still failing after 10 retries"

Since CoreOS has been acquired by RedHat I don't think they are still maintaining this one...
It's a pity tho, could've been a good project.
Anyway, that doesn't stop you from forking the project and contributing 😄

this is the most advanced one https://github.com/banzaicloud/bank-vaults that I've come across and the team over there is amazing too which included my fav @bonifaido


[#112422463] Know immediately if I am successful in my framework application on an individual framework's (dashboard) page

This pull request completes the DOS framework agreements page story. The first part of that story had to do with uploading/downloading the framework agreements, so this the "know immediately if I am successful" part of it.
Changed a few of the messages on the frameworks dashboard such that:

If you've been allowed onto the framework, it should be abundantly clear
If not, we stick with the current messaging which doesn't really say anything either way.  Basically it's: 'Here's your letter. Read it.'

And then -- because I found the whole thing super confusing -- I've broken the template up into smaller ones like we've done in the supplier template.
>> Story on Pivotal

Accepted to framework (in standstill)

Not accepted to framework (in standstill)

Here's all of the various states for the frameworks dashboard page for a supplier who is/has applied to the framework.  Page is dependent on (a) framework status and (b) supplier actions.
Open
Pending
Standstill + not on framework
Standstill + on framework
Standstill + returned countersigned agreement
Live + not on framework
Live + on framework
Live + returned countersigned agreement

Has this been tested with :sparkles: functional tests :sparkles:? And have any new scenarios been added?

@robyoung I've tested that the existing functional tests work.
Didn't add any new scenarios because it seems like a lot of moving parts (ie, supplier has to have applied to a framework which is in a specific state for each on-screen message).
Discussed it with Andrew about it earlier and he agreed.
Do you think we need them?

Sounds good to me :+1:


OpenId UserInfo endpoint hint and POST

Resolves #842 by updating the hint below the UserInfo endpoint with the correct path and, on request of @PinpointTownes, added POST to the Me action.
Note: I chose to use the AcceptVerbsAttribute to specify multiple verbs as a clean solution (AFAIK you can't specify both HttpGet and HttpPost on the same action). The alternative would be to create a MePOST action with [HttpPost, ActionName = nameof(Me), but that seemed overkill.

@anoordende,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot


add tests for CldOgImage

We use vitest for our testing suite, and the tests are located in packages/svelte-cloudinary/tests. We just need some simple tests that cover basic functionality for now, similar to the example below:
Example: https://github.com/cloudinary-community/svelte-cloudinary/blob/main/packages/svelte-cloudinary/tests/CldImage/CldImage.test.ts

@ghostdevv, I can work on these tests. Can you assign this to me?


WPF - Custom Title Bar and Control Buttons

Objective
The objective of this issue is to implement a custom title bar and control buttons (minimize, maximize/restore, close) for a WPF application to enhance its visual appeal and provide a unique user interface.
Proposed Actions

Custom Title Bar Design:

Design a custom title bar that complements the application's overall theme and branding.
Ensure the custom title bar is visually distinct and aligns with the desired UI/UX.

Custom Control Buttons:

Create custom control buttons (minimize, maximize/restore, close) that blend seamlessly with the custom title bar.
Implement appropriate behaviors for these custom control buttons (e.g., minimize the window, maximize/restore the window, close the application).

Integration with WPF Application:

Integrate the custom title bar and control buttons into the existing WPF application.
Ensure the custom title bar and control buttons function correctly and align with the application's functionality.

Technical Details

Custom Title Bar: The custom title bar will be designed using XAML (eXtensible Application Markup Language) to define the layout, styles, and interactions.
Custom Control Buttons: The control buttons will be created using XAML and will be associated with appropriate event handlers to achieve the desired functionality.
Integration: The custom title bar and control buttons will be integrated into the main window's XAML file, replacing the default title bar and control buttons.

Clarification for the UI

Custom Title Bar: The custom title bar will typically feature a unique design, possibly incorporating branding elements, icons, or a specific color scheme to differentiate it from a standard title bar.
Custom Control Buttons: The control buttons will have a distinctive design, aligning with the overall theme of the application. They should be intuitive and recognizable for users to interact with easily.

Expected Outcome
A WPF application with a custom title bar and control buttons that enhance the visual aesthetics, provide a unique user interface, and seamlessly integrate with the application's functionality.

Not a priority. Will work on this later


Update .travis.yml to use faster docker containers

"your builds now start in less than 10 seconds" - http://blog.travis-ci.com/2014-12-17-faster-builds-with-container-based-infrastructure/

Coverage remained the same at 87.33% when pulling 5a9ad8b09c73aa2adf2976d85ba8b227c9c38556 on clark800:travis-docker into 5d4db2a1022093d2983ca3f82b998f568f209e2d on ripple:develop.

This may not help much:

The default behavior when no sudo usage is detected is to send Linux builds to this infrastructure.

http://docs.travis-ci.com/user/workers/container-based-infrastructure/

Another thing we can do is run the linting and testing in parallel, or run many tests in parallel

It takes 5-10 minutes to start quite frequently, so if their 10 second statistic is anywhere near valid, then maybe it wasn't able to determine that sudo wasn't used. Linting only takes a few seconds and the unit tests only take about 12 seconds, so these aren't a big issue. Also, some of the unit tests are not parallelizable currently. For example, the ones that use withDeterministicPRNG override the global SJCL random number generator and could get different answers if they ran concurrently.

This did have an effect. Looking at the travis build logs, they are now running on worker-linux-docker-* whereas before they were running on worker-linux-*.


Impose hierarchy

Things were getting a bit unwieldy, so this organizes things based on folders.  Some things are not a perfect fit for one directory; that can always be cleaned up later.  The folders are

graphics/ -- anything to do with rendering to the screen.   (This should maybe be subdivided further, by DOM/Canvas etc)
isometric/ -- For the couple of files related to isometric display
spatial/ -- The 2D components, collisions, and similar functionality
controls/ -- Various input functionality
sound/ -- Just sound.js right now
debug/ -- Just DebugLayer.js right now
core/ -- core functionality that isn't specific to the above

It was necessary to change the paths of some of the require statements, of course.
I also split off the "Image" component from drawing.js (there was no reason for it to be there) and renamed files such that they were consistently lower case.

Looks like if those goes in I'd have to merge it manually, but I've checked locally that it won't interfere with other PRs -- they rebase just fine on top of this.

I have thought the same, but would it be an idea to split up the components into a ./components folder?

Yeah, I'm kind of ambivalent about that, since it's nice to have components split out, but having them all in one place is a bit weird.  Possibly each top level directory could have a separate components folder?

This doesn't affect functionality at all, so I'm going to go ahead and merge.

Looks like when I renamed the files lowercase, I didn't update all of them in the crafty.js source file.  This worked fine on my mac, which I guess doesn't care about case, but failed on TravisCI.  (This sort of inconsistency is actually why folk recommend just using lower-case everywhere!)  I'll fix that and push the fix.


QUESTION: Any way to specify the worker to launch the model

Note that the issue tracker is NOT the place for general support.
I launch 2 workers in a cluster which are in different server, one have 2 A100 40g and the other have 2 A100 80g.
Now i launch qwen-chat-14b in the 40g A100 which use 1 gpu.
Then i want to launch Yi-chat-34b in A100 80g,but when i launch using cli , it default launch in the other A100 40g and there is not enough gpu mem for it , so i cant chat with it.
So is there any way to specify the worker to launch the model?

For this kind of scenario, could you please join our slack for a discussion? https://join.slack.com/t/xorbitsio/shared_invite/zt-1o3z9ucdh-RbfhbPVpx7prOVdM1CAuxg


Improvements on working with bare repos and worktrees

Hi !
Thanks a lot for your work, that solves a lot of my recent problems with tmux.
These changes are a few things I had to do to make the tool work with my setup. They basically all stem from a bare repo I have whose name starts with a dot and that has worktrees I declared outside of the bare repo. (I was trying things...)
$ git clone --bare git@github.com:jrmoulton/tmux-sessionizer.git .tms
$ cd .tms
$ git worktree add ../main
$ git worktree add ../add-license-1
$ cd ../main
$ cargo run

This gives me .tms main and add-license-1 in the list. I opted to just skip every worktree during the search for repos. It seems like this is the expected behaviour according to your readme but it could be a config option. I'm up to do it if you feel it's more sensible. c1a2974
When I select .tms in the list, The session opens with the default window instead of its worktrees. This is because of the leading dot which gets translated as an underscore by tmux when creating the session but it still tries to assign a window for each worktree to .tms instead of _tms. I make this translation before all commands so that the same name is used for each one. 5f880a6
I would always be missing the second window in the session and had the bare repo itself as the first window. This is because the window at index 1 was deleted instead of the first window. 5a3ddf1
The original name prefixed with a dot is used when checking if the session exists, this ends up launching the new-window commands again and duplicating windows. d4998f9 fixes that by using the underscore prefix for the check instead.
The command doesn't work if started without tmux open. This is because the switch-command is used and it fails if no client is open. 313feb9 checks the exitstatus from the switch-client command and runs the attach command if it fails.
This might be a bit too broad for a single PR so I can spread it if needed.

This might be a bit too broad for a single PR so I can spread it if needed.

No worries these look like good changes. Everything seems to be working as I would expect but there is that line that checks for bare repos and then continues and I don't understand its purpose. But everything looks good

but there is that line that checks for bare repos and then continues and I don't understand its purpose.

https://github.com/Gawise/tmux-sessionizer/blob/313feb97417c0ec746bcacb7121236fadc6b9854/src/main.rs#L235-L237
If that's the section of code you're talking about, I'm doing this because if my worktrees are not inside the bare repo folder, but instead worktrees and bare repo are in the same folder, the worktrees are going to appear in the list of selectable repos to start a session from. I'm guessing that this is not what we want since they are supposed to be windows in a session. So when searching for repos to start as sessions, I'm skipping those that are worktrees.

Ahh that makes sense. All these changes look good. Thanks for the PR!


Dispatch from onEnter route

I'm trying to dispatch an action when the route change but I can't figure out what's the correct way.
the idea is trigger an action when the route change, but the <Provider> is placed inside Root.js and has the store like a property.
My idea is use something like
<Route path='/foo' component={Foo} onEnter={method(store)}/> from routes/index.js
How can I get access to the store from this point?
Any ideas?
Thanks

There was a gist posted the other day that may be useful https://gist.github.com/EmperorEarth/474373cb16b006fc8fbc I am not sure if its working or best practice but it may unblock you for now.

Yes, createRoutes(store) is how we recommend to get store into your routes. No magic!

Dan I was just trying to achieve this using a closure around the store object but store is unavailable in my debugger point here https://gist.github.com/marr/a63f4ada9239610fd0be. Is that not the intended method to use?

if I replace that debugger with this.store = store it does keep a reference to the store variable passed in to the wrapping function. I guess I need to brush up on Functional programming stuff!

solved
routes/index.js
export default (store) => {
  return (
    <Route path='/' component={CoreLayout}>
      <IndexRoute component={HomeView} />
      <Route path='/login' component={Login} onEnter={method(store)} />
    </Route>
  );
};

Root.js
  get content () {
    return (
      <Router history={this.props.history}>
        {this.props.routes(this.props.store)}
      </Router>
    );
  }

Thanks for help @gaearon @marr

@gaearon Any idea why this dispatch might not be working? https://github.com/marr/console/blob/master/app/app.js#L40
I am running the app from a browserify build with node-static and I see the URL in the browser change, but the page itself doesn't change to reflect the new route.
When I console.log out store.getState and the nextState argument in the onEnter handler I see a difference in the keys of the state object:
store.getState: Object {routing: Object, home: Object, user: Object}
nextState: Object {routes: Array[2], params: Object, location: Object}

It's best to ask in a for related to react-router-redux. For example, on Discord.

@5h1rU  I have the same issue but According to your method
TypeError: this.props.routes is not a function
What else do you need to change?

@leeir You need to import the routes file.
export default (store) => {
  return (
    <Route path='/' component={CoreLayout}>
      <IndexRoute component={HomeView} />
      <Route path='/login' component={Login} onEnter={method(store)} />
    </Route>
  );
};

This ^^
And from there you pass in the store from your root component.


/etc/nvidia-container-runtime/config.tom not present in your last driver packages

Hello,
The "/run is full" issue happened to me again. You solved it by changing the log-level in /etc/nvidia-container-runtime/config.toml.
This file is not present in your last driver packages. Looks like nvidia-container-toolkit doesn't create this file anymore.
Some ways to fix this:

Add config file to driver package.
In nvidia-driver.plg, if config file doesn't exist, you can generate it with this command
mkdir -p /etc/nvidia-container-runtime && nvidia-ctk config default > /etc/nvidia-container-runtime/config.toml
or create it with this content:

[nvidia-container-runtime]
log-level = "warning"

Unraid: 6.12.4 (kernel: 6.1.49-Unraid)
Nvidia driver unraid plugin: 2023.08.31
Nvidia driver: 535.104.05 (ctk 1.14.0)

Sorry, my build machine pulled in the wrong nvidia-container-toolkit and therefore the file isn't there, you can see that the package that I've built has the file in it here, but I really can't tell why it did pull in the wrong file. Hmmm, have to look into this.
I will trigger a rebuild from the Nvidia drivers which will take about an hour.

The build for the packages is now almost finished. I also pushed a update to the plugin itself for users who have the plugin currently installed but this of course means that they have to reboot at least once.
This file was never auto created in my build process btw, just read your full question and possible fix.
Thank you for the report, if you experience further issues feel free to open up a new issue.

Thank for your help too and quick fix!


Add support for border-radius CSS property

I am trying to round the corners of my table, border-radius doen't seem to work when I convert the below HTML to PDF using xhtmltopdf pdf generator. Below is the HTML written for content file name is sticker_print.html :

Batch Sticker
Batch ID
MFG Date
 Bar Code

QR Code

    Can you suggest me alternative way.

@monirul89
border-radius isn't supported CSS property. Please see the docs.

Is the no way to make rounded corners?

Not for now. Nevertheless, you are of course invited to propose a corresponding PR to add support for it.


Typos in function comments? "iff" vs. "if"

This question or problem report applies to the codebase, not the documentation files. I've just noticed that comments for many functions in "Return Value:" part often contains "True iff ..." or "true iff ...". Currently there are 72 such lines in 19 files. In some cases "iff" is in the middle of sentence. I suspect it is typo, since many other function comments have expected "if " e.g. "True if there was a title to output.", but I still not sure that it does not have some special meaning in automatic documentation generation from code or something like that.
If that "iff" is typo that has somehow multiplied all over codebase, are you interested in pull request to fix that? There are other typos like "dsipatch"

As far as "iff" is concerned - we use that as a shorthand for "if and only if". So those are totally intentional.
Other typos like "dsipatch" are probably unintentional. If you open a PR, we'd be happy to review those and make sure :)

Ok. Thanks for clarification!

indeed, “iff” is a very common shorthand (in our field)


How to get libjre_native.jar and jre_bundle_deploy.jar and jre_bundle_deploy-src.jar?

I am building j2cl source. But I found there was not these file in project:
libjre_native.jar
jre_bundle_deploy.jar
jre_bundle_deploy-src.jar

How do I get these files above?

I found it


Missing "ipywidgets>=7.0.0" on the examples' requirements

Please make sure that this is a Bug or a Feature Request and provide all applicable information asked by the template.
If your issue is an implementation question, please ask your question on StackOverflow or on the TensorTrade Discord #help-desk channel instead of opening a GitHub issue.
System information

Have I written custom code (as opposed to using example directory):
OS Platform and Distribution (e.g. Mac Sierra, Windows 10, Linux Ubuntu 16.04): Ubuntu 20.04
TensorTrade version: v1.0.3
TensorFlow version: 2.5.0
Python version: 3.6 inside docker (3.7 on machine)

You can obtain the TensorFlow version with:
python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.version)"
Describe the current behavior
Fails to run last cell on train_and_evaluate.ipynb
Describe the expected behavior
It should train without failing
Code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate the problem.
train_and_evaluate.ipynb
Other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

Thanks for the tip!


Always call resolveErrorCall on overload failure to ensure subexpressions are checked

Fixes a break in rwc and user tests.

Ahhh rip, #36744 got there first


Smart-On-FHIR Inferno Compliance

These branches have already been deleted, but adding them here for reference #1130, #1109, #1108, #972

This is complete

https://www.medplum.com/docs/compliance/onc


PHP 7 Support

From what I could see, PHP 7 support is currently blocked by third party libraries, whoops and php-debugbar.
For whoops, it's just needed to upgrade to v2, for php-debugbar we need to wait they add support for PHP 7: https://github.com/maximebf/php-debugbar/issues/264

@snowair are you willing to accept a pull request for this issue?

yes


array items $ref - not supported or bugged - compile fails

I've seen similar reports but none quite fit my case.
What version of Ajv are you using? Does the issue happen if you use the latest version?
8.12.0 (latest)
Ajv options object
{
  allowUnionTypes: true,
  verbose: true,
  allErrors: true,
}

JSON Schema
{
  "$schema": "https://json-schema.org/draft/2020-12/schema",
  "type": "object",
  "properties": {
    "buildingDetails": {
      "type": "array",
      "items": [
        {
          "$ref": "#/$defs/A"
        }
      ]
    }
  },
  "$defs": {
    "A": {
      "type": "object",
      "properties": {
        "name": {
          "anyOf": [
            {
              "type": "string"
            },
            {
              "type": "string",
              "enum": [
                "unkown"
              ]
            }
          ]
        },
        "details": {
          "type": "array",
          "items": {
            "anyOf": [
              {
                "type": "string"
              },
              {
                "$ref": "#/$defs/B"
              }
            ]
          }
        }
      },
      "required": [
        "name"
      ]
    },
    "B": {
      "type": "string",
      "enum": [
        "xyz"
      ]
    }
  }
}

Your code
Use the demo repo - I'll provide the code here as well for documentation and reference.
import Ajv from "ajv/dist/2020";
import addFormats from "ajv-formats";
import xSchema from "../schema/x.json"; // the above

const ajv = new Ajv({
  allowUnionTypes: true,
  verbose: true,
  allErrors: true,
});
addFormats(ajv);

const v = ajv.compile<X>({ $async: true, ...xSchema });

where the generated typings are:
export interface X {
  buildingDetails?: [] | [A];
  [k: string]: unknown;
}
export interface A {
  name: string | "unkown";
  details?: (string | B)[];
  [k: string]: unknown;
}
export type B = "xyz";

Validation result, data AFTER validation, error messages
Test suite failed to run

    schema is invalid: data/properties/buildingDetails/items must be object,boolean

    > 14 | const v= ajv.compile<X>({ $async: true, ...xSchema });
         |                        ^
      15 |
      16 | console.warn("test")
      17 |

      at Ajv2020.validateSchema (node_modules/ajv/lib/core.ts:519:18)
      at Ajv2020._addSchema (node_modules/ajv/lib/core.ts:721:30)
      at Ajv2020.compile (node_modules/ajv/lib/core.ts:384:22)
      at Object.<anonymous> (src/x.ts:14:24)
      at Object.<anonymous> (src/x.spec.ts:3:1)

What results did you expect?
No error during compile.
Are you going to resolve the issue?
I don't think so.

You probably need
"items": {
"$ref": "#/$defs/A"
}
The way your schema is written would only validate the first item, and it does not match the type you have.

Thanks @epoberezkin
why would it only validate the first item? In this example it'd be fine as B is just a subset of string. In a case where the first one doesn't match it should validate against the second one.


defaultProps for a Component do not show up in validate function props argument

Are you submitting a bug report or a feature request?
Bug report.

What is the current behavior?

When a Component is given defaultProps, they do not show up in props argument of validate function.
What is the expected behavior?
defaultProps should show up validate function's props argument. (see SimpleForm component in codesandbox example)
Sandbox Link
https://codesandbox.io/s/mm3xx3pvmp

What's your environment?

All redux form versions
Other information

logs included in codesandbox example

You're adding default props to your SimpleForm component. You need to instead pass them to your redux form component, otherwise redux-form has no way of knowing they exist. e.g.
const ReduxSimpleForm = reduxForm({ ...config }, SimpleForm)

ReduxSimpleForm.defaultProps = { extra: "extra" }

See https://codesandbox.io/s/21z150n640


Adding argv parameters in the examples

Currently bots in the examples are created with
var bot = mineflayer.createBot({
  username: "chatterbox",
  viewDistance: "tiny",
});

Which means that in order to start them, it's needed to add the host and post in the code. Git is then telling us that the file change and all, not practical.
I think it would be better to have something like this in all the examples :
if(process.argv.length!=4)
{
	console.log("Usage : node example.js <host> <port> <name>");
	process.exit(1);
}
var bot = mineflayer.createBot({
	username: process.argv[4],
	verbose: true,
	port:parseInt(process.argv[3]),
	host:process.argv[2]});

What do you think ?

I approve, but maybe also add [password]?

:+1:

Ok I'll do a PR for this. Yeah I'm ok with [password].


replace-newlines Not Replacing Windows New Lines on Unix

NLog version: 1.6.8
Platform: .Net 3.5 / .Net 4.5 / .NET Core 2 / .NET Core 3
Current NLog config
var fileTarget = new FileTarget
{
    FileName = targetPath,
    Encoding = Encoding.UTF8,
    ArchiveAboveSize = 104_857_600, // 100mb
    ArchiveFileName = archivePath,
    KeepFileOpen = true,
    ConcurrentWrites = false,
    OpenFileCacheTimeout = 30,
    EnableFileDelete = true,
    MaxArchiveFiles = 10,
    WriteBom = false,
    Layout = "${longdate} \t"
             + "${level:uppercase=true} \t"
             + "${processid}:${threadid} \t"
             + "${logger} \t"
             + @"${replace-newlines:replacement=\\r\\n:${message}}${onexception:inner= Exception\:${exception:format=ToString}}"
};

What is the current result?

Given the following message logged under Linux (using Windows line endings, per spec):
<!DOCTYPE html>
<html lang="en">
<head>
    <meta charset="utf-8" />
    <me...

The following is logged:
<!DOCTYPE html>
\r\n<html lang="en">
\r\n<head>
\r\n    <meta charset="utf-8" />
\r\n    <me...

It appears the \r character is never replaced.

What is the expected result?

The replace-newlines wrapper should replace all line endings.

Are there any workarounds? Regex, the solution to all.
Is there a version in which it did work? Unknown
Can you help us by writing an unit test? Yep, could also make a pr, but since this behavior depends on Environment.NewLine, I'm unsure how we want to make that test.

I can make a pr, I just wanted to check on how you want the unit tests done. I can abstract out Environment.NewLine? If these tests run on both Linux and Windows, I could make the new test only execute on Linux?

PR would be great. We run on Windows and Linux. Another (maybe better) option would be to mock the newlines values so we could run them everywhere

There is already support for Unix/Mac is, but I guess it's buggy then
https://github.com/NLog/NLog/blob/dev/src/NLog/LayoutRenderers/Wrappers/String transformations/ReplaceNewLinesLayoutRendererWrapper.cs#L78

The support is for handling unix-newlines on the windows-platform.
Now it is the other way around. Windows-newlines on the linux-platform (currently unsupported).

Closing as resolved by  #3949


Notice: Use of undefined constant LC_MESSAGES - assumed 'LC_MESSAGES'

my code:
<?php

require __DIR__.'/vendor/autoload.php';

setlocale(LC_ALL, 'en-US');

$t = new \Gettext\GettextTranslator();

//Set the language and load the domain
$t->setLanguage('gl');
$t->loadDomain('messages', 'Locale');

And there are errors following...

Deprecated: setlocale(): Passing locale category name as string is deprecated. Use the LC_* -constants instead

Warning: setlocale(): Invalid locale category name LC_MESSAGES, must be one of LC_ALL, LC_COLLATE, LC_CTYPE, LC_MONETARY, LC_NUMERIC, or LC_TIME

It seems that LC_MESSAGES is not available. According with the documentation, this constant is available only if PHP was compiled with libintl. So Translator::setLanguage() should check whether the constant exists before use them, and if not, use LC_ALL instead.
Thanks for the report, I'll fix this ASAP.

Released v3.5.7


Datastore have issue saving AWSJSON type

Describe the bug
I have a field content: AWSJSON! in graphql api.
When I generate models for Datastore then its converted to content: string.
The output from draft-js is converted to string with
content: JSON.stringify(convertToRaw(editorState.getCurrentContent()))
the actual content is
"{"blocks":[{"key":"872eg","text":"this is a test message","type":"unstyled","depth":0,"inlineStyleRanges":[],"entityRanges":[],"data":{}}],"entityMap":{}}"
I get following error from Datastore/API
Supplied AttributeValue is empty, must contain exactly one of the supported datatypes
Amplify CLI Version
4.18.0
To Reproduce

npx create-react-app test --use-npm --template typescript
amplify init with dev env
amplify add auth default everything
amplify add api use following schema

type Comment @model
  @auth(rules: [{ allow: private }]) {
  id: ID!
  owner: String!
  content: AWSJSON!
}

then save with button like this
async function createComment() {

    try {

      const results = await DataStore.save(new Comment({
        owner: 'd346afec-1d50-4aeb-bfea-e70f67adc0f9',
        content: JSON.stringify(convertToRaw(editorState.getCurrentContent()))
      }))

      console.log(results)

    } catch (err) {

      console.error(err)
    }
  }

Then you get following error
Supplied AttributeValue is empty, must contain exactly one of the supported datatypes
Expected behavior
I should be able to save JSON in dynamodb with datastore. Without error.
Desktop (please complete the following information):

OS: Ubuntu 18.04 on Windows WSL
Node Version. 12.16.1

Additional context
It works in 2 conditions

when content: AWSJSON! I save it with content: JSON.stringify(JSON.stringify(convertToRaw(editorState.getCurrentContent())))
if I change  content: AWSJSON! to content: String! and save with JSON.stringify(convertToRaw(editorState.getCurrentContent()))

Well my question is changed

Should I save JSON as AWSJSON and stringify twice and parse twice?
Or should I have JSON as String and only stringify once and parse once?

Which one would cost less?


Add license

Explicit better than implicit, and by default all repls are under the MIT license, so I thought we should add it to make it more clear.

reviewing

maybe add a quick explanation:
You can do whatever you want with the code as long as you include the original copyright and license notice in any copy of this code.

merge time