openvswitch: 2.15.1 -> 2.16.1

Motivation for this change
A number of bug fixes were included in this release
P.S.: I also have updates ready for openvswitch-lts, but due to #98200 I have not included them in this commit. It might be helpful to decide whether openvswitch-lts will be actually removed or not as a package.
Things done

Built on platform(s)

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

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

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

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.


Add support for the remaining product options on Checkout

Adding support for some of the missing product_options on the Checkout

@abishekrsrikaanth will need a bit more time for this one.

@abishekrsrikaanth thanks for all the help 😊


Prediction length for Monash benchmark

Hi, may I check how the baseline results for the Monash benchmark (Figure 4, e.g. Wavenet, Transform., DeepAR, etc.) were obtained? From my understanding of the codebase, it is using the huggingface monash_tsf dataset repository to obtain the Monash time series. The prediction length is based on this:
https://github.com/ngruver/llmtime/blob/37d0a33ac528e726cf05e6e3996c38a107520fe1/data/monash.py#L43
My concern is that the prediction lengths from the huggingface dataset are different from the default prediction length in the Monash dataset. For example, solar 10 minutes from the hf dataset has a prediction length of 60 while the Monash baseline results have a prediction length of 1008. Please correct me if I am mistaking anything here. Thank you!

Hi Geral,
Thanks so much for bringing this to our attention. The monash baseline numbers are from the original paper, and it is possible there is a mismatch in our evaluation. I will be on vacation this upcoming week, but I will take a close look the day I get back.
Nate

Hi @ngruver, any updates on this and plans to release updated results figure/table?

Hi Gerald, thanks for following up. We've updated the results in the NeurIPS camera ready (https://openreview.net/forum?id=md68e8iZK1). The monash numbers now include 19 datasets:
covid deaths,
solar weekly,
tourism yearly,
tourism quarterly,
tourism monthly,
australian electricity demand,
pedestrian counts,
hospital,
fred md,
us births,
nn5 weekly,
nn5 daily,
traffic weekly,
traffic hourly,
saugeenday,
cif 2016,
bitcoin,
sunspot
As you pointed out, solar 10 minutes has a much longer prediction horizon than original represented in the huggingface datasets and therefore we dropped that one from consideration. We corrected the horizons in the other datasets that were inconsistent.
I'm in the process of further expanding to 30 of the datasets by adding the following ones to the analysis:
weather,
kdd cup,
electricity hourly,
electricity weekly,
m1 yearly,
m1 quarterly,
m1 monthly,
m3 yearly,
m3 quarterly,
m3 monthly,
m3 other
After I finalize those results, I will update the arxiv.


Description editing impact assessment

We need to have a comprehensive view of the impact of changing from editing descriptions in HTML to using Markdown
Output -
an approach to handling this in 3++ implementation
Report of expressiveness lost (if any)
Data Migration
Editing
DITA generation (or Markdown export?)
Publication

Think embedded WYSIWYG html editor has been incorporated as part of editing epic?

Think embedded WYSIWYG html editor has been incorporated as part of editing epic?

Burned - as new editor is using HTML.


attempt to perform arithmetic on local 'idx'

Hi,
Not sure if this is an error / issue but when I run some command like  "map", "digraphs" or anything bringing lots of data, and I press "q" to quite .. here I always receive this message:

"...re/nvim/lazy/fine-cmdline.nvim/lua/fine-cmdline/init.lua:101: attempt to perform arithmetic on local 'idx' "

I noticed that if I nagivate all the way to the end of the data and only then press "q", then no message / error shows up.

This should be fixed now.


Add instructions for Windows users - bash kernel

[ ] test python scripts with bash kernel from cmd
[ ] test python scripts with bash kernel from powershell
[ ] last resort: install wsl2 and test again
[ ] if needed: instructions on how to install wsl2
[ ] if cmd or powershell work, add the requirement to the readme to help Windows users

tutorial for wsl install

closed by #9


[SPARK-28306][SQL] Make NormalizeFloatingNumbers rule idempotent

What changes were proposed in this pull request?
The optimizer rule NormalizeFloatingNumbers is not idempotent. It will generate multiple NormalizeNaNAndZero and ArrayTransform expression nodes for multiple runs. This patch fixed this non-idempotence by adding a marking tag above normalized expressions. It also adds missing UTs for NormalizeFloatingNumbers.
How was this patch tested?
New UTs.

Can one of the admins verify this patch?

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/12514/
Test PASSed.

Test build #107376 has started for PR 25080 at commit 8b914cc.

Test build #107378 has started for PR 25080 at commit 58ec2a2.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/12516/
Test PASSed.

ping @cloud-fan due to NormalizeFloatingNumbers

Test build #107376 has finished for PR 25080 at commit 8b914cc.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class KnownFloatingPointNormalized(child: Expression) extends UnaryExpression

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/SparkPullRequestBuilder/107376/
Test FAILed.

Test build #107378 has finished for PR 25080 at commit 58ec2a2.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
trait TaggingExpression extends UnaryExpression
case class KnownNotNull(child: Expression) extends TaggingExpression
case class KnownFloatingPointNormalized(child: Expression) extends TaggingExpression

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/SparkPullRequestBuilder/107378/
Test FAILed.

Why this is reported as a bug? If this is a bug, please make a regression test with the title prefix SPARK-28306.

Test build #107382 has started for PR 25080 at commit 201b287.

Changed to improvement.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/12520/
Test PASSed.

Thanks, @yeshengm .

Test build #107382 has finished for PR 25080 at commit 201b287.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
trait TaggingExpression extends UnaryExpression
case class KnownNotNull(child: Expression) extends TaggingExpression
case class KnownFloatingPointNormalized(child: Expression) extends TaggingExpression

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/107382/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/12532/
Test PASSed.

Test build #107394 has started for PR 25080 at commit 1f6773c.

Test build #107394 has finished for PR 25080 at commit 1f6773c.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/107394/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/12534/
Test PASSed.

Test build #107396 has started for PR 25080 at commit f255c8e.

Test build #107396 has finished for PR 25080 at commit f255c8e.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/107396/
Test FAILed.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/12539/
Test PASSed.

Test build #107400 has started for PR 25080 at commit f255c8e.

Test build #107400 has finished for PR 25080 at commit f255c8e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/107400/
Test PASSed.

I'm okie with it too

thanks, merging to master!


KernelGateway script failing

Hello,
I created (via AWS console) a KGW lifecycle with the following script and attached to domain :
# This script installs a single pip package on a SageMaker Studio Kernel Application
#!/bin/bash

set -eux

pip install -e /root/.apitextract/lib-app0337-api-textract/utils

It fails with the following error :
/opt/ml/lifecycleconfig/lifecycle_script.sh: 3: /opt/ml/lifecycleconfig/lifecycle_script.sh: : not found

Any ideas ? I did restart studio after attaching the lifecycle config to domain.

Closing - There was an error in my script. The folder was not found.


Add composable operator to community-operators

Thanks submitting your Operator. Please check below list before you create your Pull Request.

Flat operator directory structure is obsolete from 23-rd of October 2019, only nested directory structure will be accepted.

New Submissions

[*] Has you operator nested directory structure?
[ ] Have you selected the Project Community Operator Submissions in your PR on the right-hand menu bar?
[*] Are you familiar with our contribution guidelines?
[*] Have you packaged and deployed your Operator for Operator Framework?

[*] Is your submission signed?

Updates to existing Operators

[ ] Is your new CSV pointing to the previous version with the replaces property?
[ ] Is your new CSV referenced in the appropriate channel defined in the package.yaml ?
[ ] Have you tested an update to your Operator when deployed via OLM?
[ ] Is your submission signed?

Your submission should not

[*] Modify more than one operator
[*] Modify an Operator you don't own
[*] Rename an operator - please remove and add with a different name instead
[*] Submit operators to both upstream-community-operators and community-operators at once
[*] Modify any files outside the above mentioned folders
[*] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[*] Description about the managed Application and where to find more information
[*] Features and capabilities of your Operator and how to use it
[*] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[*] Human readable name and 1-liner description about your Operator
[*] Valid category name1
[*] One of the pre-defined capability levels2
[*] Links to the maintainer, source code and documentation
[*] Example templates for all Custom Resource Definitions intended to be used
[*] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file a PR against this repo and explain your need
2 For more information see here

You need to remove the Kubernetes YAML multi-document syntax from your CRD - that's why it's not parsed correctly and the test to build a catalog with all the bundle component fails.


Link to Register

I try to create an account I notice the link to it is not functioning, I will like to help

Actually registration is working but trying to fix some stuff before merging to main. But you can still go ahead


add an achievement that is unlocked when playing a certain week with botplay

Describe your problem here.
as it says on the issue title, i'm trying to do that if the game detects a week is completed with botplay (and also practice mode cuz why not), it unlocks an achievement. i already added it into Achievements.hx and PlayState.hx.
this is how i wrote the code to unlock it, but it won't work because the weekData is an unknown identifier.
case 'botplay_achievement': 						unlock = (ClientPrefs.getGameplaySetting('botplay') && usedPractice && weekData.getCurrentWeek('weekplaceholder'));
any kind of help is appreciated.
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
mostly modified the PlayState.hx and the Achievements.hx files.

you forgot to uppercase the W in weekData

it works now. i also modified the WeekData thing to WeekData.getCurrentWeek().weekName == 'weekplaceholder' since i placed too many arguments in getCurrentWeek. thanks.


National view: give circles same hover & click behavior as their backing state (S)

Mousing over any circle should emphasize the backing state in the same way as mousing over the state under the circles (for circles that extend beyond their state boundary, does "backing state" mean the one directly behind the mouse or the one represented by the circle?) (S)
Clicking either a dot or state poly zooms you to the state view (S)

For circles extending beyond the edge of a state, “backing state” means the state the dot/county belongs to.

As of #106 + #109, i think really what we should do is remove mouse events from circles entirely. i think this was done in #106 but accidentally removed/overwritten in #109.

I had it working in #106 where you only moused over counties and it didn't matter how big the circle was because I turned off pointer events

@lindsaycarr can you explain how that works to me? Just curious about the implementation. If the counties are below the circles (lower z-index), how does the pointer get recognized by the polygon under the dot?

Because the dot doesn't have any pointer event, so the pointer event passes right through it. This probably explains it much better - https://developer.mozilla.org/en-US/docs/Web/CSS/pointer-events

none
The element is never the target of mouse events; however, mouse events may target its descendant elements if those descendants have pointer-events set to some other value. In these circumstances, mouse events will trigger event listeners on this parent element as appropriate on their way to/from the descendant during the event capture/bubble phases.

THAT IS AWESOME! thank lcarr


Print java and mvn version

Signed-off-by: Francesco Guardiani<EMAIL_ADDRESS>
Changes

Print java and maven version (followup of https://github.com/knative/test-infra/pull/2753)

/cc @chizhg

I think the pull-knative-hack-kind-tests failure is not related to this PR (given the other test pass)


False Positive: Unexpected Duplicate Selector

Code
const DisclosureButton = styled.button`
  padding: 0;
  margin: 0;
  background: transparent;
  font-weight: bold;
  outline: none;
  border: none;
`

const Layout = styled.div`
  padding: 1.25rem;

  ${DisclosureButton} {
    margin-left: 0.5rem;
  }
`

Error
 17:31  ✖  Unexpected duplicate selector ".selector10 ", first used at line 10   no-duplicate-selectors
Versions

Node 8.11.3
"stylelint": "^9.5.0",
"stylelint-config-recommended": "^2.1.0",
"stylelint-config-styled-components": "^0.1.1",
"stylelint-processor-styled-components": "^1.3.2"

Similar with #193. Before someone made the substitution logic more clever, interpolation-tagging is one of workarounds.

@chinesedfan Thanks. The following worked:
const DisclosureButton = styled.button`
  padding: 0;
  margin: 0;
  background: transparent;
  font-weight: bold;
  outline: none;
  border: none;
`

const Layout = styled.div`
  padding: 1.25rem;

  ${/* sc-sel */ DisclosureButton} {
    margin-left: 0.5rem;
  }
`

@Undistraction OK. I will close this issue. If #110 was implemented, things will be better.


DueDate field on creating/editing milestones modify date by -1 day

Issue found on https://try.gogs.io (v<IP_ADDRESS>5).
On the right side of the screen we can see two fields. Shorter one have label "Due Date (optional)" and it allow us to set end date for given milestone. There is a problem with field on saving changes. After save date is modified by 1 day backward.
This issue can be connected somehow with #2564

Thanks your feedback!
Does the server runs on the different timezone that you're located?

If viewdns.info is correct and http://try.gogs.io IP is <IP_ADDRESS> (located in Palatine, Illinois, USA), than yes. I'm testing it from Poland, Europe (my timezone is GMT+1).

OK... I might know the problem, which I've noticed... but no solution yet. :sweat:

Found the problem, I'll open a PR for it.

@tkausl looking forward!

Added pr #2563, however regulateTimeZone is just a bad hack to fix a bug in the orm-engine. We should either get the orm-owner to fix it or fork the orm and fix it ourselfs. See https://github.com/go-xorm/xorm/pull/226 (its a pr for mssql but mysql has the exact same problem)

@tkausl you're absolutely right.


Just added
if exist "build-win.log" (del build-win.log)

No big deal.

Thanks! Got this in #474


Fix for var init

Hey, here is a potential fix for var init.
I have tried your addon and it seems to be the first error i encountered.
=)
But maybe you already work on this ^^'

Thanks, I haven't seen this error (because the variable already existed it seems).


Noise in documentation

Thank you for you an amazing piece of software for DDM models.
Should the get_noise method of the Noise class return the standard deviation or the variance of the noise? In places in the documentation it sounds like it should return the variance, but in the methods section of your preprint it sounds like the standard deviation.
Thanks for any clarification, and for your time.

Thank you for pointing this out!  The documentation has been updated to clarify that it is standard deviation, not variance.


Disable markdown for a chunk of code

Hi there,
I was wondering if Marked had a feature where I could write:
<div markdown="0">
...
</div>

Where anything inside the "div" is left completely untouched.
I need this as I am trying to allow a chain of filter, where first I apply markdown, and then I apply EJS to the file. However, doing so it's impossible since Markdown escapes things.
Thank you!
Merc.

Sorry to bother... just checking if this was being considered! Thanks...

#983


Limit deploy-to-pypi to single python version

Fix for #138
We want to avoid that each item in the test matrix triggers a deployment
to pypi. This limits the deploy job to when the python version is 3.7,
hence it should only run once.

@johnraz @James1345  what do you think? I m not familiar with travis config. But reading this https://docs.travis-ci.com/user/deployment/#conditional-releases-with-on seems promising.
I wouldnt know how we could test it easily. On the other hand we know we cannot deploy twice anway, so it cant make the situation worse.

Hey @belugame sorry for the delay here.
I googled a bit as I'm not used to travis either and found this: https://docs.travis-ci.com/user/build-stages from this discussion https://github.com/travis-ci/travis-ci/issues/929.
I believe your solution is good enough as a temporary fix but it would be safer to only deploy once the all test matrix passes.
What do you think ?

Yes, that sounds like the proper way of doing it!


[wenet/squeezeformer] compat with websocket server

Great job! test for chunk size -1 works fine, works both for cpu and gpu.


Report has no project data but created erroneous record on CH ETL

https://extractives.companieshouse.gov.uk/company/00245717

[11:33:01 AM] David Mihalyi: {"data":[{"_id":"58130262a62b295de0454988","__v":0,"proj_aliases":[],"proj_country":[],"proj_commodity":[],"proj_status":[],"_keywords":[],"proj_name":"0","proj_established_source":{"_id":"58130261a62b295de0454982","source_name":"Companies House Extractives Disclosure of Aggregate Industries UK Limited for 2015, Version 1","source_type_id":{"_id":"56e8736944442a3824141429","source_type_name":"UK Mandatory payment disclosure","source_type_id":"ukdis"
http://www.resourceprojects.org/api/projects/50/0

Done. See also https://docs.google.com/spreadsheets/d/1HMZGo3neJijUVqR7JVe9f6dC4iURvH13E70a3ipieoM/edit#gid=0


Move yq parameters

Currently make openstack fails with yq installed via pip (yq 3.2.3). This patch make it work with it again.

recheck

/lgtm

recheck

/approve

recheck

recheck

The baremetal job seems to consistently fail with this error:
 "error: must build at directory: not a valid directory: evalsymlink failure on '/home/zuul/ci-framework-data/artifacts/manifests/operator/baremetal-operator/config' : lstat /home/zuul/ci-framework-data/artifacts/manifests/operator: no such file or directory"

Studying the logs more deeply it fails with Error: unknown shorthand flag: 'y' in -y. Apparently version used by CI does not have -y option which unfortunately is required by version available in PIP. How is the job getting yq, so that I can get the same version in my lab? I guess we can give up on this PR until CI gets updated version.

I believe it gets yq from here: https://github.com/openstack-k8s-operators/install_yamls/blob/main/devsetup/roles/download_tools/tasks/main.yaml#L90

Hello,
@fao89 it is indeed [1] what causes problems. We should use yq from pip, not source. The yq from pip doesn't recognize  -y flag, but it works without it just fine.
[1]https://github.com/openstack-k8s-operators/install_yamls/blob/main/devsetup/roles/download_tools/tasks/main.yaml#L116


Whether the monaco-editor has the functionality of commond

This is just some debt... It used to be that commands were scoped per editor instance, but now all commands are global. That explains the weird addCommand method that exists on an editor instance... In order to maintain compatibility, the standalone editor has this notion of a dynamic command that it uses...
There is no API to define a global, regular command.. So you would need to go to JS
const CommandRegistry = require('vs/platform/commands/common/commands').CommandsRegistry;
CommandsRegistry.registerCommand('myId', () => alert('hello'));

Originally posted by @alexdima in https://github.com/microsoft/monaco-editor/issues/1857#issuecomment-594457013

@alexdima  node  vs platform. just monaco-editor. I want to handle js script, when click hover content.


Slack community should allow community members to join

The readme states that we can join the slack community, but the community is only available to people with google.com and apple.com corporate email addresses.
Are you able to relax the requirement so that end-users can join in?
Thanks!

The slack channel is not longer actively monitored. I removed it from the README.


Warnings in documentation for Configuration CC are on the wrong methods

What about this exception?
https://github.com/zwave-js/node-zwave-js/blob/790bd4dfaed72142cb8a5a9778189a8b11eb8303/packages/zwave-js/src/lib/commandclass/ConfigurationCC.ts#L1561-L1569


[BUG] serialized partially signed multi-sig transactions cannot be deserialized back

First of all, thank you for the effort put in this library which I use in a daily basis :)
I believe I have found a bug regarding the serialization/deserialization scheme for multi-sig transactions.
According to the following line
https://github.com/hirosystems/stacks.js/blob/bd8c847e4ac43f5f8e1fd803174a199f69d5a435/packages/transactions/src/authorization.ts#L238
it is not possible to deserialize a serialized partially signed multi-sig spending condition.
This implies that a partially signed multi-sig tx can be serialized, but it cannot be deserialized back.
I believe this breaks the following scenario:

userA creates a multi-sig tx using its public key as well as userB's public key;
userA signs the tx;
userA serializes the partially signed multi-sig tx;
userA sends the serialized tx to userB;
userB deserializes the tx;
userB signs the tx;
as the tx is now a fully signed multi-sig tx, userB broadcasts it.

Thanks for the report! We'll look into this 👍🏻

I looked into it and the scenario is valid and reproducable.
import { deserializeTransaction } from "@stacks/transactions";

const serializedUnsignedTx =
    '80800000000401a5beef0a47958d870fb90f5bd79e556b5ec7f483000000000000000400000000000000c8000000030003264290976c5bf07b0a12635f8ad57a8edff8f6e73554b30e19f6d6783ca6f4b70002688809b7fb74fb60dab30f8d23e139b2796fe112cc12d7909bdd49d356a9269700022d2426a24c70f34a05763f94ec67ac084065b23a127cb591d68c61a7e6311b5900020302000000000005159558bd2cdf99842228d266671c3f34722e803847000000000000006430000000000000000000000000000000000000000000000000000000000000000000';

try {
    const result = deserializeTransaction(serializedUnsignedTx);
    console.dir(result);
} catch (e) {
    console.log(e)
}

https://github.com/hirosystems/stacks.js/blob/bd8c847e4ac43f5f8e1fd803174a199f69d5a435/packages/transactions/src/authorization.ts#L238
To fix this need to allow numSigs <= signaturesRequired
But doing this breaks couple of test cases.  Not sure will it be acceptable changing these breaking test cases accordingly as they align with stacks-blockchain in implementation to not allow deserialization of partially signed multisig transaction.
Thinking on this further... or most welcome if you have different thought's.

@luizalbertocviana in your example, are you calling appendOrigin with user B's pubkey before user B attempts to deserialize (see pattern at https://github.com/hirosystems/stacks.js/tree/master/packages/transactions#supporting-multi-signature-transactions)?

@luizalbertocviana in your example, are you calling appendOrigin with user B's pubkey before user B attempts to deserialize (see pattern at https://github.com/hirosystems/stacks.js/tree/master/packages/transactions#supporting-multi-signature-transactions)?

Hi @diwakergupta :)
Thanks for your attention.
Maybe I have not described my test scenario as clearly as I should.
Notice that when userA sends a serialized partially signed transaction to userB, userA is in fact sending the return of the serialize method from StacksTransaction class.
That method returns a Buffer, so userA sends to userB just binary data (via https, a file transfer, etc).
This way, the first thing userB must do in order to operate on the partially signed multi-sig transaction he has received from userA is deserialize it.
Notice there is no way userB would call appendOrigin method from TransactionSigner before deserialization, as userB only has received a Buffer.

... as they align with stacks-blockchain in implementation to not allow deserialization of partially signed multisig transaction.

Hi @ahsan-javaid :)
Thanks for your quick replies to my issue.
Is there an alternative approach for sending partially signed multi-sig transactions across network?
Please correct if I am wrong, but I believe this is a critical thing to the scenario I have described, otherwise some part (userA, userB or a third part) would need to handle both userA and userB secret keys in order to properly sign and broadcast their multi-sig transaction, and this does not seem to be a decentralized approach.

Looks like there is an alternative way of bypassing this error with existing code base. Here are the steps:

UserA create the transaction with numSignatures value set to 1
UserA signs the transaction and send it to userB after serialization
UserB will be able to deserialize it as numSignatures is met now
UserB will set numSignatures = 2. just increment numSignatures++
UserB will sign the transaction and broadcast it
This approach can work for any number of users
Hack: The receiver will just need to increment numSignatures before signining.

Sample implementation:
import {
  makeUnsignedSTXTokenTransfer,
  deserializeTransaction,
  pubKeyfromPrivKey,
  publicKeyToString,
  standardPrincipalCV,
  BufferReader,
  MultiSigSpendingCondition,
  AnchorMode, TransactionSigner, createStacksPrivateKey,
} from '@stacks/transactions';

void(async () => {

  // User-A machine
  const recipient = standardPrincipalCV('SP3FGQ8Z7JY9BWYZ5WM53E0M9NK7WHJF0691NZ159');
  const amount = 2500000;
  const fee = 0;
  const nonce = 0;
  const memo = 'test memo';

  const privKeyStrings = [
    '6d430bb91222408e7706c9001cfaeb91b08c2be6d5ac95779ab52c6b431950e001',
    '2a584d899fed1d24e26b524f202763c8ab30260167429f157f1c119f550fa6af01',
    'd5200dee706ee53ae98a03fba6cf4fdcc5084c30cfa9e1b3462dcdeaa3e0f1d201',
  ];

  const privKeys = privKeyStrings.map(createStacksPrivateKey);

  const pubKeys = privKeyStrings.map(pubKeyfromPrivKey);
  const pubKeyStrings = pubKeys.map(publicKeyToString);

  const transaction = await makeUnsignedSTXTokenTransfer({
    recipient,
    amount,
    fee,
    nonce,
    memo: memo,
    numSignatures: 1, // Initially set numSignatures = 1 on userA machine
    publicKeys: pubKeyStrings,
    anchorMode: AnchorMode.Any
  });

  // Now add the required signatures in the original transactions
  const signer = new TransactionSigner(transaction);
  signer.signOrigin(privKeys[0]);

  const serializedTx = signer.transaction.serialize();

  //////////////////////////////////////////////////////////////////
  // Now give serializedTx to user B via http or file transfer etc...

  // Now on UserB machine set signaturesRequired = 2

  const bufferReader = new BufferReader(serializedTx);

  // It will successfully deserialize as numSignature condition met
  const stacksTransactionOnUserBMachine = deserializeTransaction(bufferReader);

  const sp = stacksTransactionOnUserBMachine.auth.spendingCondition as MultiSigSpendingCondition;
  // Now userB has this transaction so before sign make signaturesRequired++;
  sp.signaturesRequired++;
  const signerB = new TransactionSigner(stacksTransactionOnUserBMachine);
  // Now user B can sign the transaction
  signerB.signOrigin(privKeys[1]);
  const serializedTxUserB = signerB.transaction.serialize();
  console.log(serializedTxUserB);

  // Give it userC and so on for any number of users
  const bufferReader2 = new BufferReader(serializedTxUserB);
  const stacksTransactionOnUserCMachine = deserializeTransaction(bufferReader2);
  // On User C Machine
  console.log(stacksTransactionOnUserCMachine)
})();

cc: @luizalbertocviana

Hey @ahsan-javaid :)
Thank you so much for this reply.
I will check this on my side as soon as I can.
What are your thoughts on updating documentation?
I believe such a scenario would be much more valuable than the current one regarding multi-sig, in which a non signed tx is serialized, you know?

Hey @ahsan-javaid
Indeed your approach solves the serialization/deserialization issue.
However, I am afraid it creates a new one.
When I try to broadcast the fully signed multi-sig tx, that is, signed by both userA (with signaturesRequired == 1) and userB (after signaturesRequired++), it is rejected with the following response:
{
  error: 'transaction rejected',
  reason: 'SignatureValidation',
  reason_data: {
    message: 'Signer hash does not equal hash of public key(s): 0acc89b63de9494f481389a9f2910a3e360e3e4c != 0d2b11f6b356be4d18a5898c99a7c49ac0bd8b09'
  },
  txid: '901b3f0d5c6fc21054e1177f05c614a5c5a823340dbcc3eca793c1d7e453e59c'
}

Maybe it is worth mentioning I am creating a contract call transaction.

I was under the impression that any signing needs to "know"/include the final number of signaturesRequired, correct?
We probably need to remove the check in the serialization. I had a brief look at this w/ reed before and came to the same conclusion (without comparing to the rust implementation).


Field texture bug: Incorrect threshold being used

The incorrect threshold is being used in the final thresholding step. Instead of using the textural threshold, the diagnostic threshold is being used. This means the textural threshold was being used no where. This PR corrects this mistake.
Testing:

[x] Ran tests and they passed OK
[x] Added new tests for the new feature(s)

@fionaRust probably. I'll take a look.

@fionaRust I have parameterised over the textural threshold in a test of process. The chosen thresholds span across the ratio values returned by the test data, whilst the previously incorrectly used diagnostic threshold does not. This test demonstrates that the textural threshold is being used and has an impact on the results.

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- improver_tests/field_texture/test_field_texture.py  8

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- improver_tests/field_texture/test_FieldTexture.py  8

See the complete overview on Codacy


Add Josh to slack and the github repo

We need josh github username

Here it is: jbuxt. Thank you


java: implement common resource name

Codecov Report

Merging #1931 into master will decrease coverage by 0.01%.
The diff coverage is 61.53%.

@@             Coverage Diff              @@
##             master    #1931      +/-   ##
============================================
- Coverage     85.73%   85.71%   -0.02%
- Complexity     4792     4794       +2
============================================
  Files           441      441
  Lines         19199    19210      +11
  Branches       2009     2010       +1
============================================
+ Hits          16460    16466       +6
- Misses         2052     2057       +5
  Partials        687      687

Impacted Files
Coverage Δ
Complexity Δ

...ain/java/com/google/api/codegen/util/TypeName.java
88.63% <0%> (-11.37%)
15 <0> (ø)

...n/transformer/java/JavaModelTypeNameConverter.java
84.84% <100%> (+0.72%)
36 <4> (+2)
:arrow_up:

Continue to review full report at Codecov.

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


Test Data: Two CSVs (one clean-No PII, and another with PII)

Added dummy data & unit test
https://github.com/UBC-MDS/sanityze/pull/38


How to install?

Hbg.shop
Failed to read NSP metadata
DBI
No CNMT file in nsp
Goldleaf crashes

nx-hbloader is not a warez NSP; it is an ExeFS NSP. Atmosphère comes bundled with nx-hbloader and nx-hbmenu, so user intervention is not needed to run homebrew.
Piracy is not supported.

Thanks, but I understood nothing. I want to run HB menu right from game list (without pressing R on any game) to run not in applet mode. I can't install some nsp and do it like with RetroArch, for example?

Atmosphère's default settings will allow you to boot homebrew by holding R while loading any game you have officially installed. You can modify those settings to allow for a game to become the designated homebrew title; which you will launch (without pressing buttons) to boot homebrew. Please refer to Atmosphère's HBL settings section for more details.
Also, please do not install unsigned titles. There are numerous risks involved in their installation (including permanent banning from Nintendo's servers), and it is completely unnecessary for the purpose of running homebrew with full privileges.

I see now, thanks a lot!


Implement better loading of config file

search for it in standard locations
provide user friendly error, if not found

Fixes: https://github.com/mcurlej/joft/issues/1 https://github.com/mcurlej/joft/issues/2
With this in place we can finally release this tool to PyPI and start promoting it.

+1


Don't sum over an index for terms that lack it

Suppose I have the following case
julia> using Einsum;
julia> a = [1, 2];
julia> b = [1 2; 3 4];
julia> c = a+b*a  # what I would like to compute
2-element Array{Int64,1}:
  6
 13
julia> @einsum c[i] := a[i] + b[i,j] * a[j]  # this doesn't work
2-element Array{Int64,1}:
  7
 15
julia> @einsum c[i] := b[i,j] * a[j];  # breaking up the problem works
julia> @einsum c[i] += a[i]
2-element Array{Int64,1}:
  6
 13

It seems terms that don't include an index are still summed over that index. From my understanding of Einstein notation, this isn't supposed to happen. Is there a way to avoid the hack above? For reference, TensorOperations.jl handles the sum as expected.
julia> using TensorOperations;
julia> @tensor c[i] := a[i] + b[i,j] * a[j]
2-element Array{Int64,1}:
  6
 13

I agree that's the standard meaning of Einstein notation. But it would be far from simple to make this package do that, and if you wanted it to be efficient, you'd end up re-writing TensorOperations.jl.
Despite the name, what this package seems good at doing is other sums, or broadcast-like operations, especially with functions other than * and + on the right. Like m(pq, p, q) in the readme. (It is, sadly, quite often faster than TensorCast.jl on these things.) For these the rule of summing over all free indices is unambiguous, but again not very Einstein.
The package ArrayMeta.jl also obeyed the same rule:
julia> using ArrayMeta # Julia 0.6, or 0.7 with the right PR

julia> @arrayop c[i] := a[i] + b[i,j] * a[j]
2-element Array{Int64,1}:
  7
 15

julia> c =<EMAIL_ADDRESS>a + b * a'; dims=2)
2×1 Array{Int64,2}:
  7
 15


Language display

Microsoft PowerToys version
0.65.0
Installation method
GitHub
Running as admin
None
Area(s) with issue?
Settings
Steps to reproduce
PowerToys does not display in English but in my local language, which is not very convinient. How to change the language to English, please? Thanks.
✔ Expected Behavior
Either - choose your language on installation, or being able to adjust the language in settings.
❌ Actual Behavior
no obvious way to change ones' language choice.
Other Software
No response

/dup #7670


NEC veoffload behaviour is too cryptic

NEC veoffload behaviour isn't really transparent.
Right now, I have binary built with nc++, but when i try to load it and open context - I get an error that gives me no real information about the failure. What was wrong? Why failure happened?
Code used for veoffload init:
proc = veo_proc_create(deviceId);

handle = veo_load_library(proc, veobin);

if (proc == null)

    throw new RuntimeException("veo_proc is null");

if (handle < 1)

    throw new RuntimeException("handle is [" + handle + "]");

ctx = veo_context_open(proc);

if (proc == null || ctx == null)

    throw new RuntimeException("setDevice() failed");

i get setDevice() failed exception, which means that  ctx == null due to veo_context_open(proc) failure.
All i get in logs is this:
2020-03-24T04:15:32-0400 veos.os_module.core ERROR 30636 Assign failed exception not served PID 69596
2020-03-24T04:15:32-0400 veos.os_module.core ERROR 30636 Assign failed exception not served PID 69598
2020-03-24T04:15:32-0400 veos.os_module.core ERROR 30636 Assign failed exception not served PID 69600
2020-03-24T04:15:32-0400 veos.os_module.core ERROR 30636 Assign failed exception not served PID 69603
2020-03-24T04:15:32-0400 veos.os_module.core ERROR 30636 Assign failed exception not served PID 69605
2020-03-24T04:15:32-0400 veos.os_module.core ERROR 30636 Assign failed exception not served PID 69607

Validation tool says binary is ok:
[raver119@aurora03 nd4j-tests]$ /opt/nec/ve/bin/ve_validate_binary -f /home/raver119/.javacpp/cache/nd4j-aurora-1.0.0-SNAPSHOT-linux-x86_64.jar/org/nd4j/nativeblas/linux-x86_64/libnd4jaurora.so
**VE BINARY/SHARED LIBRARY IS VALIDATED SUCCESSFULLY!!**

It would be really nice if error messages would be actually useful for developer, because right now system says nothing about what happened and what developer should do in order to get things fixed.

cc @mikishin ^^^

Here's the code in C/C++ that reproduces issue:
#include <ve_offload.h>
#include <cstdio>
#include <cstddef>
int main()
{
  /* Load "vehello" on VE node 0 */
  struct veo_proc_handle *proc = veo_proc_create(0);
  if (proc == nullptr)
     printf("proc is nullptr\n");

  uint64_t handle = veo_load_library(proc, "/home/raver119/.javacpp/cache/nd4j-aurora-1.0.0-SNAPSHOT-linux-x86_64.jar/org/nd4j/nativeblas/linux-x86_64/libnd4jaurora.so");

  if (handle == 0)
     printf("handle is 0\n");

  struct veo_thr_ctxt *ctx = veo_context_open(proc);
  if (ctx == nullptr) {
     printf("ctx is nullptr\n");
     return -1;
  }

  veo_context_close(ctx);
  return 0;
}

Build cmd line:
g++ -o ve_binary ve_binary.cpp -I/opt/nec/ve/veos/include -L/opt/nec/ve/veos/lib64    -Wl,-rpath=/opt/nec/ve/veos/lib64 -lveo -std=c++11

Executable output:
[raver119@aurora03 ~]$ ./ve_binary
handle is 0
ctx is nullptr

So, something happens, but not quite clearly what exactly happens.

The development team says that veoffload logging will be enhanced in late May.
https://veos-sxarr-nec.github.io/veoffload/md_GettingStarted.html
By using the method described here, debug information can be output by outputting a debug log using the .log4crc file.
However, we do not know how to read the log file, so we will ask the development team.

Thank you very much!

As a result of investigation by the development team, it is an error that a module that does not include -fPIC in the compile option is linked.
I will report to the development team that it is difficult to understand the cause of the error.

Thank you, we'll check it out immediately.

Yep, thanks for help, that's it. Hope dev team will be able to improve logging for such cases.


Added unit tests, Removed c# 7 features, Created Unit Test for bug.

Sorry for the duplicate pull requests the first one was on the wrong email. I am interested in your project for work because we start up a lot of services on the fly but they do not always close down smoothly so a tool like this would be very helpful.
Here was some basic things I noticed
I have added a basic Unit Test Project for testing out the different features in the project.
At work we do not use c# 7 so I had to remove the extra features from this project which will also make it more accessible to more people.
I also added a unit test that shows a bug I noticed where a new child process does not come over on the event pipeline.
There is also a host of other things I could mention but I want to keep this initial commit brief.

Hey thanks for this contribution,
I've found the source of the bug in your test, it was being caused by bad logic in the recursion method for finding child processes.
I did notice one issue in the test itself however. You're using the Process class to get the name of a process, this will throw an exception if that process is dead. To address I will make a method on WardenProcess class so you can find a child by its id like such:
 _currentProcess.FindChildById(args.Id)

I'm going to go ahead and accept these changes and apply my changes on top.

One additional comment, if you are planning some more contributions the format style currently present should be used as to avoid needless whitespace git changes.


Test41SINITMatchesCPU allow no SINIT ACM in memory

The SINIT ACM may optionally provided by the firmware.
Don't fail in Test41SINITMatchesCPU if no SINIT ACM was provided by the firmware.

This is only the case if txt-suite is run before tboot, or on a platform without tboot, as tboot loads the SINIT ACM into memory, too.

Test changed to not required.

the corre

I think i fixed that already


SDXL Different Weights with CLIP2

There seems to be an inconsistency with the text encoder 2 weights. With the sd_xl_base_1.0.safetensors, I view conditioner.embedders.1.model.text_projection and it is:
tensor([[-0.0348, 0.0218, -0.0025, ..., -0.0034, -0.0138, -0.0075],
[-0.0037, 0.0202, -0.0014, ..., -0.0082, -0.0011, 0.0170],
[ 0.0261, -0.0221, -0.0099, ..., -0.0233, -0.0178, -0.0061],
...,
[ 0.0042, -0.0068, 0.0086, ..., 0.0008, -0.0030, -0.0042],
[-0.0236, 0.0094, 0.0040, ..., -0.0098, 0.0330, 0.0147],
[ 0.0084, -0.0021, -0.0049, ..., 0.0026, -0.0055, -0.0294]],
dtype=torch.float16)

However if I go into text_encoder_2/model.fp16.safetensors, I see text_projection.weight is:
tensor([[-0.0348, -0.0037, 0.0261, ..., 0.0042, -0.0236, 0.0084],
[ 0.0218, 0.0202, -0.0221, ..., -0.0068, 0.0094, -0.0021],
[-0.0025, -0.0014, -0.0099, ..., 0.0086, 0.0040, -0.0049],
...,
[-0.0034, -0.0082, -0.0233, ..., 0.0008, -0.0098, 0.0026],
[-0.0138, -0.0011, -0.0178, ..., -0.0030, 0.0330, -0.0055],
[-0.0075, 0.0170, -0.0061, ..., -0.0042, 0.0147, -0.0294]],
dtype=torch.float16)

Which one should be correct? They do lead to different results.

I see they seem transposed, I will check how they are loaded.

Is there any updates on this issue?


Migrate to new code style package

https://www.npmjs.com/package/@eox/eslint-config

Done in https://github.com/EOX-A/EOxElements/pull/1306


Add additional param to catch errors in cygwin process

Codecov Report

Merging #646 into master will increase coverage by 0.41%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #646      +/-   ##
==========================================
+ Coverage   95.62%   96.04%   +0.41%
==========================================
  Files          28       28
  Lines        2376     2500     +124
  Branches      199      199
==========================================
+ Hits         2272     2401     +129
+ Misses        104       99       -5

Impacted Files
Coverage Δ

browser/model/cygwin.js
100% <100%> (ø)
:arrow_up:

browser/services/platform.js
86.39% <0%> (-0.38%)
:arrow_down:

browser/pages/install/controller.js
100% <0%> (ø)
:arrow_up:

browser/pages/location/controller.js
100% <0%> (ø)
:arrow_up:

browser/services/componentLoader.js
100% <0%> (ø)
:arrow_up:

browser/directives/pathValidator.js
100% <0%> (ø)
:arrow_up:

browser/model/jdk-install.js
100% <0%> (ø)
:arrow_up:

browser/model/installable-item.js
100% <0%> (ø)
:arrow_up:

browser/pages/start/controller.js
100% <0%> (ø)
:arrow_up:

... and 16 more

Continue to review full report at Codecov.

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


Adopt a Widget - Tooltip

The Tooltip widget needs:

[x] A code / DartPad sample
[ ] More references
[ ] More explanation

For details about how to participate, see flutter.dev/adoptawidget

I would like to be assigned to this issue
Thanks

I would like to be assigned to this issue

Will start working on it right away


added "resizable" attribute to window

fixes #39
should probably re-open the issue and wait until this PR is merged to close it,
code LGTM


[question] Possible for multiple providers to organize routing per component?

Sorry for posting a question here...
Following the examples, I got redux-router working in my app.  However, we found that at some point, centralizing all routes into one place isn't ideal.  Is there a way to include multiple routers per app?   For example, if I have a route to /client.  I want to isolate all child routes inside of /client, such as /client/list, /client/add, etc...
Similar to this, basically allowing you to attach another router inside your component.
https://github.com/STRML/react-router-component
Currently, I've attempted to include another provider+router within a component.  Although it doesn't produce errors, it ignores all routing except for the root.

I just stumbled on the dynamic routing features.  Maybe this will be a solution.

Yep, dynamic routing! Also you can split up your routes on a file-basis pretty easily, if you declare them without JSX.
See: https://github.com/rackt/react-router/blob/latest/docs/guides/basics/RouteConfiguration.md#alternate-configuration

You can actually keep the JSX in separate files as well. Just use the following syntax:
import otherRoutes from './otherRoutes'

<Route path='/'>
  {otherRoutes}
</Route>

Thanks!  Will take a look at it.


DTS-UHD: Report t1cc flag in commercial name

This pull request adds the T1CC flag to DTS-UHD in the commercial name only.

@RoyFunderburk apologies, we were not able to review the last PRs before the release and we need to release, so DTS-UHD support will be in next release but without t1cc flag support. We'll handle that for the next release (it won't be long).

I tweaked a bit the output for doing the difference between the technical flag and the commercial name.


UP-01-013 uProxy Extension UI allows Contact Spoofing

During the tests with the newly working UI it was discovered, that an attacker who is already a contact with the victim can easily spoof identity and make the victim believe to be proxying via a different person. This enables MitM attacks based on weak security and trust indicators: uProxy relies on trust and relations to work properly - thus this is considered a working and trivial to exploit attack. The test was carried out as follows:

Victim (user A) and attacker (user C) are contacts
Victim frequently proxies through a trusted friend (user B)
Attacker is a contact of user B as well (optional)
Attacker notices that user A is online but user B is not (optional)
Attacker changes first- and last-name to match user B
Attacker changes avatar / profile image to match user B
Victim now sees user C looking like user B
Attacker allows victim, the user A, to proxy and can monitor and change any unencrypted traffic

While the uProxy UI is still pretty young, it should be implementing secure design from the very beginning. Showing only a user’s first- and last-name is not a good option as anyone can arbitrarily change those details at any given time. The contact list in the uProxy UI should also show the email address of the user to give more precise info as to who the contact actually is.
Further it is recommended to register name-changes and for example show a field called “Formerly called” to make sure that name changes get noticed and thereby eliminate that spoofing vector. In addition, it is recommended to for instance allow to “star” well-known and trusted users to create yet another security indicator. technically more verse users should be presented with an option to verify fingerprints or similar tokens to be sure to be communicating with the right user and not a spoof.

We think we can help guard against contact spoofing by showing a link to the user's profile along with their name and image.  This will help differentiate between 2 users with the same name and image, as each user will have a unique link which will lead to their profile page, which will have much more to help distinguish the user than just a numeric ID or a "horse rocket" type string.
@zahoriea - here is a screenshot of the new link included in the buddylist.  Any thoughts on the styling?  There will be a standard tooltip showing the link when the user hovers over (unfortunately Material Design tooltips don't work here as they don't properly truncate or word wrap and have been resulting in very bad scrollbars).

Nice! Thanks for doing this. I would love to hide this in the UI a bit more viewing someone's profile isn't a primary action for users.
Proposal: show the link-out icon only when contacts are expanded. This places the action at the same disclosure of accepting/ignoring a request or offer for feedback, or offering or asking for access (activities where it's important to verify that your contact is someone you trust.)

I like your tooltip on hover, and think that it will make the action even more clear.
Thoughts?

Looks good, thanks!  I just created pull request
https://github.com/uProxy/uproxy/pull/1229
On Tue, Apr 7, 2015 at 7:45 PM, Izzie Zahorian<EMAIL_ADDRESS>wrote:

Nice! Thanks for doing this. I would love to hide this in the UI a bit
more viewing someone's profile isn't a primary action for users.
Proposal: show the link-out icon only when contacts are expanded. This
places the action at the same disclosure of accepting/ignoring a request or
offer for feedback, or offering or asking for access (activities where it's
important to verify that your contact is someone you trust.)
[image: image]
https://cloud.githubusercontent.com/assets/945845/7035977/8d0e68ea-dd43-11e4-82cf-b5d83749d92b.png
[image: image]
https://cloud.githubusercontent.com/assets/945845/7036029/e9467936-dd43-11e4-96b8-3da3507f3766.png
I like your tooltip on hover, and think that it will make the action even
more clear.
Thoughts?
—
Reply to this email directly or view it on GitHub
https://github.com/uProxy/uproxy/issues/412#issuecomment-90762405.


Amplitude of the streaming

Hello,
Is there any way to get the amplitude of the streaming? If not then can we get the buffer data that is being sent so that we can calculate the amplitude?
Thank you for the awesome library.

Do you means audio amplitude?
In this case, this is not supported, you sneed modify library. If you want get audio buffer, you can create a callback in your base used. You can find the buffer here for camera1, if you want other case, the method name is the same in all bases.

PCM buffer:
https://github.com/pedroSG94/rtmp-rtsp-stream-client-java/blob/master/rtplibrary/src/main/java/com/pedro/rtplibrary/base/Camera1Base.java#L678
AAC buffer:
https://github.com/pedroSG94/rtmp-rtsp-stream-client-java/blob/master/rtplibrary/src/main/java/com/pedro/rtplibrary/base/Camera1Base.java#L647

Yes, audio amplitude. cool, I'll try to modify library and get the buffer.

@Override   public void inputPCMData(Frame frame) {     audioBufferListener.sendBuffer(frame.getBuffer());     audioEncoder.inputPCMData(frame);   }
is this a correct way to get audio buffer array?

Yes, but remember that you will need create a copy of buffer to avoid issues when you modify it.

sure. thanks for your support.


Inconsistent and redundant word usage in Ephemeral Volumes page

This is a Bug Report
Problem:
The sentence They are currently also not covered by the storage resource usage limits of a Pod, because **that is something that kubelet can only enforce for storage that it manages itself.** in the page https://kubernetes.io/docs/concepts/storage/ephemeral-volumes/#csi-ephemeral-volumes   has several times used same word that in single sentence. Which make sentence look differently.
Proposed Solution:
Same word cannot be repeated several tims in one sentence.
Page to Update:
https://kubernetes.io/docs/concepts/storage/ephemeral-volumes/#csi-ephemeral-volumes

/language en
/sig docs

here in this sentence  that is something that kubelet can only enforce for storage that it manages itself if we remove that is something that  then the sentence look good.

/rettile Inconsistent and redundant word usage in Ephemeral Volumes page

/retitle Inconsistent and redundant word usage in Ephemeral Volumes page

I think there is room for improvement to make the wording better.
/triage accepted
/priority backlog

/assign

This issue is invalid.

because that is something that kubelet can only enforce for storage that it manages itself.

This first "that" refers to the "storage resource usage limit";
the second "that" leads an attribute clause.
You can treat the second "that" as "which".

/remove-triage accepted
/remove-priority backlog

/retitle Inconsistent and redundant word usage in “Ephemeral Volumes” page
Avoid backticks unless referring to code / methods / etc

in reference of @tengqm  comments there is nothing to fix.
.

I am closing the issue. Marking as Nothing to fix.
/close


Allow integer inputs/outputs of linear_transpose

I don't think disallowing integers in linear_transpose should be necessary (at risk of coming across as a nerd: I think they're a special case of the transpose of a module homomorphism 🤓, so mathematically well defined, see the definition of 'transpose' here: https://en.wikipedia.org/wiki/Module_homomorphism#Operations).
The transpose rules haven't been tested on integers and there might be situations where we get silent failures ... If this change is deemed to be mergeable in principle, then I can implement such tests and add them to this PR.

My main concern was about unexpected behavior for transposing integer -> float functions:
https://github.com/google/jax/pull/3398#issuecomment-690620368
https://github.com/google/jax/issues/3402#issuecomment-643830344
So I think I'm OK with adding support for integer -> integer functions, but that needs some additional checks.

Great, I've added a refined check that ensures the function is either [float or complex] -> [float or complex] or integer -> integer. By the way, it looks like transposing an float -> integer function was allowed before. Was that intentional? If so I need to re-allow that because it's banned by the new check.
In case you're interested, I'm using this for computing a data structure 'update' function from a view-like function (the matrix representation of a view is effectively an orthogonal projection matrix, so you can get a pseudo-inverse by transposing it - kind of using a sledgehammer to squash a fly, but it works: https://github.com/j-towns/crayjax/blob/3d0a84137cf607877d4f20a19fb5ec50f59b8238/crayjax/rans8.py#L132-L138). I've been doing this trick with Autograd for a while but needed to upgrade some code to run on GPU recently.

I wonder whether we can think of a more efficient way to do view_update, because the dense tree_multimap(jnp.add, data, diff) will be O(|data|), and I guess it should be possible to do the update in O(|item|).
def view_update(data, view_fun):
    item, view_transpose = view_fun(data), linear_transpose(view_fun, data)
    item_copy = tree_map(_copy, item)
    def update(new_item):
        diff, = view_transpose(tree_multimap(jnp.subtract, new_item, item_copy))
        return tree_multimap(jnp.add, data, diff)  # <- this is inefficient
    return item, update

Great, I've added a refined check that ensures the function is either [float or complex] -> [float or complex] or integer -> integer. By the way, it looks like transposing an float -> integer function was allowed before. Was that intentional? If so I need to re-allow that because it's banned by the new check.

Interesting... I think transposing float -> integer should be OK. At least you won't see any funny truncation in the transposed computation (integer -> float).

Yeah, I see what you mean, also I guess it shouldn't technically be possible to define a linear float -> integer function (is it possible in JAX?). Shall we leave it disabled for now until/unless there's a use case for it?

The docs build failure looks to be unrelated to this change.

@shoyer I think this should be fine to merge.

This is indeed good to merge, but unfortunately it needs a rebase now after https://github.com/google/jax/pull/6422 moved the contents of api.py.

Done.

Ping @shoyer.

Thanks Jamie!
On Tue, Apr 20, 2021 at 10:23 AM copybara-service[bot] <
@.***> wrote:

Merged #6239 https://github.com/google/jax/pull/6239 into master.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/google/jax/pull/6239#event-4620581119, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJJFVUD57KAOR5ALO4D3QTTJW2BXANCNFSM4Z3G73ZA
.


Fix a possible segfault in heim::net::nic() for Windows

Hello,
After some time using heim::net::nic() on Windows (which I implemented some time ago in ad691385940babcab857b1e19ebe95af35b8d70e), I realized I introduced a potential null pointer dereferencing :fearful:
I've actually ran into this in some tests of mine, that segfaulted when an adapter had not been assigned any IP address.
This MR fixes this issue by properly checking for null pointers where I had forgotten to do so.
Luckily, this bug has never reached a tagged version nor a published version of the heim crate in crates.io

Coverage increased (+0.3%) to 46.39% when pulling 4adea5c566dfce51d55acd12a995f5c8d86c1fb4 on daladim:nic_segfault into b292f1535bb27c03800cdb7509fa81a40859fbbb on heim-rs:master.


Option for using persistent notifications

Description
First of all, thank you for creating One Second Diary ! I use it almost every day and I really like its concept.
The goal of the PR is to add an option for using persistent notifications (see the related issue).
I've added the "usePersistentNotifications" key to the English and French translation files. I don't know how to translate it into other languages. You may need to add a commit to add this key to the other translation files.
I had to add code to close the persistent notification and reschedule it when a daily entry is added, as the persistent notification cannot be cancelled by the user. I moved some methods from SwitchNotificationsComponent to NotificationService for this purpose.
For information I initially wanted to reschedule notifications to the next day when an entry is added, to make this case not possible:

A user schedules a persistent notification at 3 p.m.
The user records a video at 2pm
He receives the persistent notification at 3pm even though he has already recorded a video

I wasn't able to prevent this to happen, as the flutter_local_notification package doesn't allow to specify an initial date for "time" notifications. However, I managed to close the notification when the user opens the application if a video has already been recorded for the current day.
Checklist
Before you create this PR, please confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[X] I have added a description of the change in CHANGELOG.md.
[X] I have added my name in the CONTRIBUTORS.md file, if it wasn't already present.

LGTM!
Thanks very much for your contribution!!

I don't know how to translate it into other languages. You may need to add a commit to add this key to the other translation files.

No worries, I'll do it. I've been using ChatGPT for that 😅.


add cicero CI

Also fixed some impurities in the nix evaluation, see commit messages

I assume we need to re-configure github to require these new CI checks still.

I am going to merge even though ci/hydra-eval has (erroneously!) reporting that it has failed.


Do you need to use both simple and complex?

For example,
If I have a
1 base body (that gets used for all)
3 helmets (red/blue/green)
3 gloves (red/blue/green)
3 t-shirts (red/blue/green)
What I want to do is generate all combinations
base + red helmet + red gloves + red t-shirt
base + red helmet + red glove + blue t-shirt
etc... I am a bit comfused about the complex / simple break down of layers. I tried putting no complex, and everything in simple like this

with the config like this

but that didnt work I get

Just not exactly sure how I should organize the folders for a task like this. Cheers!

@RickWillcox complex layer is where a single layer is composed of multiple sprites. In your case you should not use complex layers. I invented the complex thingy as I wanted different tails and ears for all my cats.
For example in the layers building a body requires three parts, a body, an ear and a tail. Because ears and tails are different, colouring them means I had to create all three shapes for coloring. I know it is a bit messy, I wanted to put together something working very fast.
In theory if you leave complex_colors as an empty list in your instance_config.py, it should work with traditional single shaped designs. Feel free to suggest any solution.
By the way, which version of the artengine are you using? Recently I pushed an update.

Hi @RickWillcox , issue is fixed. Documentation is not updated, however if you create a new simple project **artengine init my-project --project-type simple" you should be fine. Documentation update and youtube video is coming.

Thank you !


upload massive data to datastore using chunked transfer

Added capability in transferring massive data in small chunks to datastore using http-chunk transfer encoding and python generators.

issue 844 in uploading massive data to datsore


Regex parsers to be ported from macros to functions

Per the 5.0 release, one of the goals was to port macros over to functions.
The regex parsers have not yet gotten ported.
I am sorry that I do not have a good test case for this, but the current test suite for regex-based parsers still use the macro versions:
  #[cfg(feature = "alloc")]
  #[test]
  fn re_capture() {
    named!(rm< &str,Vec<&str> >, re_capture!(r"([[:alpha:]]+)\s+((\d+).(\d+).(\d+))"));
    assert_eq!(
      rm("blah nom 0.3.11pouet"),
      Ok(("pouet", vec!["nom 0.3.11", "nom", "0.3.11", "0", "3", "11"]))
    );
    assert_eq!(
      rm("blah"),
      Err(Err::Error(error_position!(
        &"blah"[..],
        ErrorKind::RegexpCapture
      )))
    );
    assert_eq!(
      rm("hello nom 0.3.11 world regex 0.1.41"),
      Ok((
        " world regex 0.1.41",
        vec!["nom 0.3.11", "nom", "0.3.11", "0", "3", "11"]
      ))
    );
  }

I’ve started working on this. Am I correct in assuming that the various _static macros do not make sense as functions? A possibility for fixing this would be to provide various functions that take an already compiled regex as an argument instead of a string. One could then always use lazy_static manually to provide the regex at compile time.

Imo there shouldn't be any need to use lazy_static in the function-based versions, because even if you take a pattern string instead of a Regex, you can still do:
fn regex_parse(pattern: &str) -> impl Fn(&str) -> IResult<...> {
    let regex = Regex::new(pattern).unwrap();
    move |input| { ... }
}

However, I'd argue against a form like this; just accept a Regex or &Regex object and let the caller be responsible for its origin.


ci: use most recent OpenSSL version under Windows

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 82.76%. Comparing base (643eb35) to head (2794a4c).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #104   +/-   ##
=======================================
  Coverage   82.76%   82.76%
=======================================
  Files          11       11
  Lines         737      737
=======================================
  Hits          610      610
  Misses        127      127

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


evaporator working animation doesnt have fluid connection visable

Mod source
Github
Which mod are you having an issue with?

[ ] pyalienlife
[ ] pyalternativeenergy
[X] pycoalprocessing
[ ] pyfusionenergy
[ ] pyhightech
[ ] pyindustry
[ ] pypetroleumhandling
[ ] pypostprocessing
[ ] pyrawores

Operating system

=Windows 10

What kind of issue is this?

[ ] Compatibility
[ ] Locale (names, descriptions, unknown keys)
[X] Graphical
[ ] Crash
[ ] Progression
[ ] Balance
[ ] Pypostprocessing failure
[ ] Other

What is the problem?
sprite animation completely overlays the pipe connection point sprite so it flicks on and off as the machine works
Steps to reproduce
load mod, watch machine run
Additional context
No response
Log file
No response

duplicate of https://github.com/pyanodon/pybugreports/issues/329 ?

duplicate of #329 ?

yup. didnt see that one. didnt really look at to see it


Loss of precision on Float fields with many digits

Bug description
Hi, I found a problem similar to #2903, in this case, the Float stored on the database is 1.59283191 but the value returned is 1.5928319100000001

How to reproduce
Prisma schema:
datasource db {
  provider = "postgresql"
  url      = env("DATABASE_URL")
}

generator client {
  provider = "prisma-client-js"
}

model Transaction {
  id       Int    @id @default(autoincrement())
  amount   Float
}

Mutation:
const transaction = await prisma.transaction.create({
  data: {
    amount: 1.59283191,
  },
})
console.log({ transaction })

Returns wrong amount:
{ transaction: { id: 1, amount: 1.5928319100000001 } }

Expected behavior
{ transaction: { id: 1, amount: 1.59283191 } }

Environment & setup

OS: Debian 11 (WSL2)
Database: Postgres 12.4
Node.js version: v14.5.0
Prisma version:

@prisma/cli          : 2.6.0-dev.58
Current platform     : debian-openssl-1.1.x
Query Engine         : query-engine 650b5d0348ec38ae61e1e7db69bb54808418ede4 (at<EMAIL_ADDRESS>Migration Engine     : migration-engine-cli 650b5d0348ec38ae61e1e7db69bb54808418ede4 (at<EMAIL_ADDRESS>Introspection Engine : introspection-core 650b5d0348ec38ae61e1e7db69bb54808418ede4 (at<EMAIL_ADDRESS>Format Binary        : prisma-fmt 650b5d0348ec38ae61e1e7db69bb54808418ede4 (at<EMAIL_ADDRESS>

Thanks for the detailed reproduction, I can confirm this. This will need a fix on the engines level.

Been digging into this today, findings so far:

The values are stored with the right precision.
The values are loaded from the database with the right precision.
The JSON rendering converts the decimal into a float, which loses the precision.

We could try some different ways maybe when we jump into the JavaScript land. Here one option I tried is to serialize decimals as strings, leading to the correct precision. You see how the float is off, but the string representation is correct:

Additionally, to make it a bit nicer for the users, we could maybe use libraries such as the decimal.js to convert the floats into decimals. This would keep the precision and offer an API to handle the resulting numbers.

Ok, I found a hack that might save us here for now. What I found out from the Decimal conversion to f64 is that the library started using the mantissa algorithm, which is much faster to convert but causes a loss of precision. What we could do here is a conversion to string first and from there to float. This is very slow compared to the mantissa, but would at least save us from inaccuracies...


vjp TypeError on weakly typed cotangents

I don't know if this is actually a bug, but was suprised that TypeError is raised if I pass 1 as cotangent to vjp.
import jax
import jax.numpy as jnp

print(jax.__version__)  # 0.3.4

def f(x):
    return x.sum()

out, f_vjp = jax.vjp(f, jnp.ones(5, 'f2'))
f_vjp(1.)  # TypeError: Type of cotangent input to vjp pullback function (float16) is not the
           # expected tangent type (float32) of corresponding primal output with dtype float32.
f_vjp(jnp.array(1., 'f2'))  # this works

Thanks for the report - indeed we don't do any type promotion at this level (even for weakly-typed values) so you need to ensure the inputs to f_vjp strictly match the expected types.
This is something we may revisit in the future though.

I think this is not something we're going to change – at the level of jax.vjp, we will not do implicit type promotion for inputs.


Add client and handler implementation for cross-cluster task APIs

What changed?

Add client implementation for cross-cluster task APIs
Add handler implementation for admin GetCrossClusterTasks
Add dummy implementation for history Get/RespondCrossClusterTasks

Why?

For supporting cross cluster operations

How did you test it?

The fact that code builds means the all wiring up and type conversions are done correctly.
Unit test for GetCrossClusterTasks in history client impl will be added after the todo in the impl is resolved.

Potential risks

Release notes

Documentation Changes

Pull Request Test Coverage Report for Build 9e102460-e6cf-4502-9310-9cd8d613b7d2

6 of 296 (2.03%)  changed or added relevant lines in 18 files are covered.
180 unchanged lines in 16 files lost coverage.
Overall coverage decreased (-0.2%) to 59.814%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

service/frontend/adminGrpcHandler.go
0
2
0.0%

client/admin/grpcClient.go
0
4
0.0%

client/admin/thriftClient.go
0
4
0.0%

service/history/grpcHandler.go
0
4
0.0%

client/history/grpcClient.go
0
8
0.0%

client/history/thriftClient.go
0
8
0.0%

client/admin/client.go
0
9
0.0%

client/admin/retryableClient.go
0
9
0.0%

client/admin/metricClient.go
0
11
0.0%

client/history/retryableClient.go
0
18
0.0%

Files with Coverage Reduction
New Missed Lines
%

service/frontend/adminGrpcHandler.go
1
6.98%

common/membership/rpServiceResolver.go
2
73.58%

common/task/weightedRoundRobinTaskScheduler.go
2
89.64%

service/history/grpcHandler.go
2
52.81%

service/matching/taskListManager.go
2
74.72%

common/persistence/statsComputer.go
3
95.36%

common/persistence/serialization/parser.go
4
65.41%

common/persistence/serialization/thrift_decoder.go
4
59.6%

service/history/queue/processor_base.go
4
62.84%

common/task/fifoTaskScheduler.go
5
84.54%

Totals

Change from base Build 8385cc09-aa2d-433f-9f36-97eb9c957446:
-0.2%

Covered Lines:
90200

Relevant Lines:
150801

💛  - Coveralls


Problem removing captions with ccLoadPolicy

Hello @PierfrancescoSoffritti
I have a video that has 2 languages of captions. I am trying to hide them but they always show.
I am using the following code:

I initialize the Builder:
IFramePlayerOptions.Builder iFramePlayerOptionsBuilder = new IFramePlayerOptions.Builder();

I disabled captions (i tried with 0 too):
iFramePlayerOptionsBuilder.ccLoadPolicy(1);

I build the IFramePlayerOptions:
IFramePlayerOptions iFramePlayerOptions = iFramePlayerOptionsBuilder.build();

I set the AutomaticInitialization a False
youTubePlayerView.setEnableAutomaticInitialization(false);

I initialize my YouTubePlayerView
youTubePlayerView.initialize(new AbstractYouTubePlayerListener() { ... }, false, iFramePlayerOptions);

I don't know what I'm doing wrong, captions are always showing and I can't figure out how to hide them. On the official YouTube website I can hide the captions without problem.
Thanks...

I don't think step 4 is necessary.
Have you looked at the sample app?

Hi there for some reason ccLoadPolicy doesnt work but there is actually an alternative way and it is quite easy to implement.
So first step is you need to add this library as a module so you can edit.
Then go to res/raw/ayp_youtube_player.html
add player.unloadModule('captions'); inside startSendCurrentTimeInterval function .And thats it captions will be gone by default.


[No QA] [ESLint] allow ignoring some deprecation errors, apply to ReportUtils

Details

These deprecated warnings were being flagged in all PRs updating the ReportUtils but cannot be fixed as these lines are required. I added allowing ignoring them via // eslint-disable-next-line deprecation/deprecation
Fixed Issues

$ https://github.com/Expensify/App/issues/49781
Tests

[X] Verify that no errors appear in the JS console

Offline tests

QA Steps

[X] Verify that no errors appear in the JS console

PR Author Checklist

[X] I linked the correct issue in the ### Fixed Issues section above
[X] I wrote clear testing steps that cover the changes made in this PR

[X] I added steps for local testing in the Tests section
[X] I added steps for the expected offline behavior in the Offline steps section
[X] I added steps for Staging and/or Production testing in the QA steps section
[X] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[X] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[X] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[X] I included screenshots or videos for tests on all platforms
[X] I ran the tests on all platforms & verified they passed on:

[X] Android: Native
[X] Android: mWeb Chrome
[X] iOS: Native
[X] iOS: mWeb Safari
[X] MacOS: Chrome / Safari
[X] MacOS: Desktop

[X] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[X] I followed proper code patterns (see Reviewing the code)

[X] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[X] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[X] I verified that comments were added to code that is not self explanatory
[X] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[X] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[X] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[X] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[X] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[X] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[X] I verified the JSDocs style guidelines (in STYLE.md) were followed

[X] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[X] I followed the guidelines as stated in the Review Guidelines
[X] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[X] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[X] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[X] I verified that if a function's arguments changed that all usages have also been updated correctly
[X] If any new file was added I verified that:

[X] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[X] If a new CSS style is added I verified that:

[X] A similar style doesn't already exist
[X] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[X] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[X] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[X] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[X] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[X] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[X] I verified that all the inputs inside a form are aligned with each other.
[X] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[X] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[X] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

@ahmedGaber93 unassigning, no C+ needed

Is there a way to get these warnings to fire locally? I tried npm run lint and also npm run lint-changed (after manually changing a file) both on main and this branch, and it passed without errors...
Basically I want to make sure this is still getting the errors for usages that you aren't directly suppressing here.

Is there a way to get these warnings to fire locally? I tried npm run lint and also npm run lint-changed (after manually changing ReportActionUtils) both on main and this branch, and it passed without errors both times...
Basically I want to make sure this is still getting the errors for usages that you aren't directly suppressing here.

It's not ideal but you have to commit the changes and then run npm run lint-changed, this way you will get an error @dangrous:

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.


Update usage documentation and use cases

Since the way nutsdb is used has changed, some documents and use cases have not been updated and need to be updated now.

I see that there's a repo named nutsdb-docs in the repos list of NutsDB. So I think it can be more useful.
In this repo I found some documents but them seem old. I think we can reuse this repo to publish documents for NutsDB. I'm not quite sure what framework was used in the existing nutsdb-docs. Maybe we can perfect it or use some other framework like https://github.com/facebook/docusaurus . docusaurus has rich document functions and good supports for multiple versions, multiple languages, and also can publish blogs.
Totally, I think use a github page to present more rich documents will be better for promoting NutsDB.

@TremblingV5 Indeed, this project has not been maintained for a long time, resulting in a lot of information being out of date. Currently, it is all markdown files. It seems that the static page is generated using https://jekyllrb.com/. I think your suggestion is very good.


Any furture plans for development?

@Rainer-Lang are you looking for something in particular? There is an open issue for streetviewpanorama but I am starting to think that would be better suited to a different library. Other than that the lib is feature complete but I do intend on updating it with future releases of google maps

@sdoward No, it was only a general question. Thanks for your answer - and still maintaining the lib.


Add node event

Could you please explain, how to add a new node, dynamically

https://github.com/wbkd/react-flow/blob/master/example/src/Empty/index.js#L13-L21

https://github.com/wbkd/react-flow/issues/394


Lowering log level of "chunk bytes limit exceeds for an emitted event stream" logs to trace

Is your feature request related to a problem? Please describe.
Hey team 👋
From what I have gathered, fluentd did not split chunks if the event stream exceeds the chunk bytes limit, but this was changed in https://github.com/fluent/fluentd/commit/b5f2e9fe4b2376ce0624cc756598e90bb64970d5.
Since we are now splitting chunks and retrying the chunks, I think this log (chunk bytes limit exceeds for an emitted event stream) is now more informational (and for debugging really) more than indicative of a potential issue.
Describe the solution you'd like
What are your thoughts on lowering this log line from warn to something like trace?
Describe alternatives you've considered
Leaving it - but it does generate a lot of logs
Additional context
No response

I don't immediately get the meaning of the condition format && empty_chunk in this warning message.
I guess it is important what this condition is for.
https://github.com/fluent/fluentd/blob/0a6d706a9cee5882d751b2cc6169696709df0134/lib/fluent/plugin/buffer.rb#L678-L684


Set up stubs for Job objects to enable unit testing

can you tell me how to make test and run test?

I believe you first have to follow the install instructions for plotman so
the tests are installed (maybe you already did this)
Then you should be able to run python3 -m pytest (or python if that's
your Python 3).
On Wed, May 5, 2021 at 5:09 AM MDCCLXXVI KPT @.***>
wrote:

can you tell me how to make test and run test?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ericaltendorf/plotman/issues/13#issuecomment-832636322,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARPZIFUE7P525T7OH3DLO3DTMEYOPANCNFSM4WQ4KOOQ
.

Hopefully we can re-org to not stub out too much.  But, I haven't had a chance to dig hard into testing yet...


change network route url

I am not sure whether the original code is out of some legacy concern.
While I think the new style is enough. However this PR makes GET /networks/ behave different.  Maybe a little bit back compatibility has been broken.
- What I did
- How I did it
- How to verify it
- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)
Signed-off-by: allencloud<EMAIL_ADDRESS>

@allencloud I'll close this PR as these routes were like this for a reason, but let me know if there's a reason you wanted to change this

Oh great. @thaJeztah  Thanks for solving my confusion.


Release btpsa-1.3.0

Purpose
PR collecting the changes for release 1.3.0
Does the PR solve an issue
[X] Yes - Please add issue number - several issues see single PRs for details
[ ] No

Does this introduce a breaking change?

[ ] Yes
[X] No

Pull Request Type
What kind of change does this Pull Request introduce?
[X] Bugfix
[X] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[X] Other... Please describe: security

How to Test
Done in single PRs
What to Check
Verify that the following are valid
Done in single PRs
Other Information

Integration test was executed successfully, see https://github.com/SAP-samples/btp-setup-automator/actions/runs/3940734434/jobs/6742250050