(No need to review) Testing grpc 1.45.4 and google-auth-library-java 1.6.1 on previous version 2.40.0

We are testing compatibility with gRPC 1.45.4 with Beam 2.40.0
This PR is just to check the CI status. This PR does not need to be merged.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run Java_Spark3_Versions PreCommit

@kennknowles : As mentioned in yesterday's email thread, could you trigger the additional workflows for this PR? Thank you!

Did they succeed? (From Kenn's email)
    :sdks:java:io:google-cloud-platform:postCommit
    :runners:flink:1.13:job-server:validatesPortableRunnerBatch
    :runners:spark:2:job-server:validatesPortableRunnerBatch
    :runners:spark:3:job-server:validatesPortableRunnerBatch
    :runners:google-cloud-dataflow-java:validatesRunnerV2
    :runners:google-cloud-dataflow-java:validatesRunnerV2Streaming

Run Java PostCommit

Run Java PostCommit

Run PostCommit_Java_Dataflow

Run PostCommit_Java_DataflowV2

Run Java Spark PortableValidatesRunner Batch

Run Java Dataflow V2 ValidatesRunner Streaming

Dataflow V2 Java Post Commit Tests => success
Google Cloud Dataflow Runner V2 Java ValidatesRunner Tests (streaming)  => success
Java Spark PortableValidatesRunner Batch Tests => success

Run Java PostCommit

Run Java_PVR_Flink_Batch PreCommit

Java SDK Post Commit Tests => success
Java_PVR_Flink_Batch ("Run Java_PVR_Flink_Batch PreCommit") => success

Run Java PreCommit

This appears to be failing in a PreCommit that does look a little mildly concerning:
00:10:05 org.apache.beam.vendor.grpc.v1p43p2.io.grpc.StatusRuntimeException: CANCELLED: Multiplexer hanging up
00:10:05 	at org.apache.beam.vendor.grpc.v1p43p2.io.grpc.Status.asRuntimeException(Status.java:535)
00:10:05 	at org.apache.beam.vendor.grpc.v1p43p2.io.grpc.stub.ClientCalls$StreamObserverToCallListenerAdapter.onClose(ClientCalls.java:479)
00:10:05 	at org.apache.beam.vendor.grpc.v1p43p2.io.grpc.internal.DelayedClientCall$DelayedListener$3.run(DelayedClientCall.java:463)
00:10:05 	at org.apache.beam.vendor.grpc.v1p43p2.io.grpc.internal.DelayedClientCall$DelayedListener.delayOrExecute(DelayedClientCall.java:427)
00:10:05 	at org.apache.beam.vendor.grpc.v1p43p2.io.grpc.internal.DelayedClientCall$DelayedListener.onClose(DelayedClientCall.java:460)
00:10:05 	at org.apache.beam.vendor.grpc.v1p43p2.io.grpc.internal.ClientCallImpl.closeObserver(ClientCallImpl.java:562)
00:10:05 	at org.apache.beam.vendor.grpc.v1p43p2.io.grpc.internal.ClientCallImpl.access$300(ClientCallImpl.java:70)
00:10:05 	at org.apache.beam.vendor.grpc.v1p43p2.io.grpc.internal.ClientCallImpl$ClientStreamListenerImpl$1StreamClosed.runInternal(ClientCallImpl.java:743)
00:10:05 	at org.apache.beam.vendor.grpc.v1p43p2.io.grpc.internal.ClientCallImpl$ClientStreamListenerImpl$1StreamClosed.runInContext(ClientCallImpl.java:722)
00:10:05 	at org.apache.beam.vendor.grpc.v1p43p2.io.grpc.internal.ContextRunnable.run(ContextRunnable.java:37)
00:10:05 	at org.apache.beam.vendor.grpc.v1p43p2.io.grpc.internal.SerializingExecutor.run(SerializingExecutor.java:133)
00:10:05 	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
00:10:05 	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
00:10:05 	at java.lang.Thread.run(Thread.java:750)

There was a version change to grpc and it looks like the errors is being thrown in grpc.


Stratos Helm Chart: Fix Upgrade

Fixes https://github.com/SUSE/stratos/issues/83
To test:
Install old version
$ helm install stratos/console --version 2.1.1 --namespace console --name console

Upgrade with the new chart
$ cd deploy/kubernetes
$ helm upgrade console ./ --set consoleVersion=2.2.0-592c239 --recreate-pods

LGTM


bug: running cargo in readme.md fails

environment:
Mac m3
k3d/ks3
ran:
# Install Rust
curl --proto '=https' --tlsv1.2 -sSf https://sh.rustup.rs | sh
source $HOME/.cargo/env
# Install WebAssembly target for Rust
rustup target add wasm32-wasi

# Install WasmEdge
curl -sSf https://raw.githubusercontent.com/WasmEdge/WasmEdge/master/utils/install.sh | bash -s -- -e all
source $HOME/.wasmedge/env

git clone https://github.com/second-state/microservice-rust-mysql.git

cd microservice-rust-mysql

cargo build --target wasm32-wasi --release

expected:
no errors on build
received:
   Compiling order_demo_service v0.1.0 (/Users/andyanderson/projects/microservice-rust-mysql)
warning: private item shadows public glob re-export
 --> src/main.rs:7:5
  |
7 | use std::result::Result;
  |     ^^^^^^^^^^^^^^^^^^^
  |
note: the name `Result` in the type namespace is supposed to be publicly re-exported here
 --> src/main.rs:4:9
  |
4 | pub use mysql_async::*;
  |         ^^^^^^^^^^^^^^
note: but the private item here shadows it
 --> src/main.rs:7:5
  |
7 | use std::result::Result;
  |     ^^^^^^^^^^^^^^^^^^^
  = note: `#[warn(hidden_glob_reexports)]` on by default

warning: `order_demo_service` (bin "order_demo_service") generated 1 warning
    Finished release [optimized] target(s) in 20.14s

Hmm, this is a warning not an error. We still need to fix it. But I think you can ignore it for now. :)

ok, will proceed.  thank you


pkg/readerutil: see what can be moved to go4.org/readerutil

Parts of pkg/readerutil can probably be replaced with, or moved to go4.org/readerutil

https://review.gerrithub.io/302648

https://camlistore-review.googlesource.com/9887


Aggregations prototype

PoC for aggregations that work either at the top level or per-bucket for time series histograms.

Work-in-progress rebased and merged into branch dhermann_aggs_prototype


Don't add icon separator if there is no icon for a status module

When rendering status modules, there is a separator between the icon and the module text. Before, it was redundantly shown even when the module icon was disabled by the user.

Reimplemented, please have a look

It could possibly fix #85


[WIP] Aspect to monitor turnaround time of healthchecks

See https://github.com/Netflix/genie/pull/632


WeChat

Is anyone working on WeChat support?

well.
making psa support wechat is also my target..
if i can do something for it, then i'm in.


Sound problem in iOS, Chrome

hello, I have problem with sound when I run my phaser game on iOS in Chrome browser. Problems are:

when I lock device, music keeps playing
when I minimize browser, music keeps playing.

it happens on iOS7 and iOS8.
I tried events game.onPause.add(....); and game.onResume.add(....);. It works on android, but doesnt work on iOS.
How can I make this work?  Thank you

Just adding that I can confirm that audio correctly pauses if you 'home screen' while Safari is running in Phaser 2.4-dev (and Phaser 2.3 actually, but we've made a few significant changes to the way 2.4 audio works so it's worth using that as the benchmark now). Feel free to re-open with a specific test case if you can prove otherwise though.


How to insert image from URL?

I am new to Github and this is my first question.
Is there any way to insert image into Presentation from file URL instead of filepath in my computer? If so how?
I am trying this and getting an error: [Errno 22] Invalid argument: 'https://img-trendyol.mncdn.com/mnresize/415/622/Assets/ProductImages/oa/66/6741004/1/4646032011667_1_org_zoom.jpg'
Thank you very much.

No. You'll need to arrange download of the image yourself. Commonly this would be done using a BytesIO object (in-memory file) such that you don't leave old files hanging around on the filesystem. Search on "python bytesio from url" to find out more about that.

thank you scanny. python pptx is an amazing tool.


Fast textual inversion?

Is it possible to speed it up like dreambooth? The most used colab is from here
https://github.com/nicolai256/Stable-textual-inversion_win

I'll take a look, if possible, I'll add a colab for it

Is it possible that you would do dreambooth on colabs gpu HDD not on a google drive? or at least make it optionable ? I only want the model not the data and it doesnt take that long to get weights from huggingface , the thing is sometimes setup gets messy after i rerun it and have to wipe start all over, doesnt work on T4 suddenly when it worked minutes ago , hreres the error
The following values were not passed to accelerate launch and had defaults used instead:
--num_processes was set to a value of 1
--num_machines was set to a value of 1
--mixed_precision was set to a value of 'no'
--num_cpu_threads_per_process was set to 1 to improve out-of-box performance
To avoid this warning pass in values for each of the problematic parameters or run accelerate config.
===================================BUG REPORT===================================
Welcome to bitsandbytes. For bug reports, please submit your error trace to: https://github.com/TimDettmers/bitsandbytes/issues
For effortless bug reporting copy-paste your error into this form: https://docs.google.com/forms/d/e/1FAIpQLScPB8emS3Thkp66nvqwmjTEgxp8Y9ufuWTzFyr9kJ5AoI47dQ/viewform?usp=sf_link
/usr/local/lib/python3.7/dist-packages/bitsandbytes/cuda_setup/paths.py:99: UserWarning: /usr/lib64-nvidia did not contain libcudart.so as expected! Searching further paths...
f'{candidate_env_vars["LD_LIBRARY_PATH"]} did not contain '
/usr/local/lib/python3.7/dist-packages/bitsandbytes/cuda_setup/paths.py:21: UserWarning: WARNING: The following directories listed in your path were found to be non-existent: {PosixPath('"<IP_ADDRESS>","jupyterArgs"'), PosixPath('"/usr/local/bin/dap_multiplexer","enableLsp"'), PosixPath('{"kernelManagerProxyPort"'), PosixPath('6000,"kernelManagerProxyHost"'), PosixPath('true}'), PosixPath('["--ip=<IP_ADDRESS>"],"debugAdapterMultiplexerPath"')}
"WARNING: The following directories listed in your path were found to "
/usr/local/lib/python3.7/dist-packages/bitsandbytes/cuda_setup/paths.py:21: UserWarning: WARNING: The following directories listed in your path were found to be non-existent: {PosixPath('module'), PosixPath('//ipykernel.pylab.backend_inline')}
"WARNING: The following directories listed in your path were found to "
/usr/local/lib/python3.7/dist-packages/bitsandbytes/cuda_setup/paths.py:21: UserWarning: WARNING: The following directories listed in your path were found to be non-existent: {PosixPath('/env/python')}
"WARNING: The following directories listed in your path were found to "
CUDA_SETUP: WARNING! libcudart.so not found in any environmental path. Searching /usr/local/cuda/lib64...
CUDA SETUP: CUDA runtime path found: /usr/local/cuda/lib64/libcudart.so
CUDA SETUP: Highest compute capability among GPUs detected: 7.5
CUDA SETUP: Detected CUDA version 111
CUDA SETUP: Loading binary /usr/local/lib/python3.7/dist-packages/bitsandbytes/libbitsandbytes_cuda111.so...
Steps:   0% 0/800 [00:00<?, ?it/s]Traceback (most recent call last):
File "/content/diffusers/examples/dreambooth/train_dreambooth.py", line 606, in
main()
File "/content/diffusers/examples/dreambooth/train_dreambooth.py", line 550, in main
noise_pred = unet(noisy_latents, timesteps, encoder_hidden_states).sample
File "/usr/local/lib/python3.7/dist-packages/torch/nn/modules/module.py", line 1130, in _call_impl
return forward_call(*input, **kwargs)
File "/usr/local/lib/python3.7/dist-packages/accelerate/utils/operations.py", line 507, in call
return convert_to_fp32(self.model_forward(*args, **kwargs))
File "/usr/local/lib/python3.7/dist-packages/torch/amp/autocast_mode.py", line 12, in decorate_autocast
return func(*args, **kwargs)
File "/usr/local/lib/python3.7/dist-packages/diffusers/models/unet_2d_condition.py", line 254, in forward
encoder_hidden_states=encoder_hidden_states,
File "/usr/local/lib/python3.7/dist-packages/torch/nn/modules/module.py", line 1130, in _call_impl
return forward_call(*input, **kwargs)
File "/usr/local/lib/python3.7/dist-packages/diffusers/models/unet_blocks.py", line 565, in forward
hidden_states = attn(hidden_states, context=encoder_hidden_states)
File "/usr/local/lib/python3.7/dist-packages/torch/nn/modules/module.py", line 1130, in _call_impl
return forward_call(*input, **kwargs)
File "/usr/local/lib/python3.7/dist-packages/diffusers/models/attention.py", line 167, in forward
hidden_states = block(hidden_states, context=context)
File "/usr/local/lib/python3.7/dist-packages/torch/nn/modules/module.py", line 1130, in _call_impl
return forward_call(*input, **kwargs)
File "/usr/local/lib/python3.7/dist-packages/diffusers/models/attention.py", line 217, in forward
hidden_states = self.attn1(self.norm1(hidden_states)) + hidden_states
File "/usr/local/lib/python3.7/dist-packages/torch/nn/modules/module.py", line 1130, in _call_impl
return forward_call(*input, **kwargs)
File "/usr/local/lib/python3.7/dist-packages/diffusers/models/attention.py", line 287, in forward
out = xformers.ops.memory_efficient_attention(q, k, v, attn_bias=None, op=self.attention_op)
File "/usr/local/lib/python3.7/dist-packages/xformers/ops.py", line 626, in memory_efficient_attention
return op.apply(query, key, value, attn_bias, p)
File "/usr/local/lib/python3.7/dist-packages/xformers/ops.py", line 264, in forward
causal=causal,
File "/usr/local/lib/python3.7/dist-packages/torch/_ops.py", line 143, in call
return self._op(*args, **kwargs or {})
RuntimeError: CUDA error: no kernel image is available for execution on the device
CUDA kernel errors might be asynchronously reported at some other API call,so the stacktrace below might be incorrect.
For debugging consider passing CUDA_LAUNCH_BLOCKING=1.
Steps:   0% 0/800 [00:09<?, ?it/s]
Traceback (most recent call last):
File "/usr/local/bin/accelerate", line 8, in
sys.exit(main())
File "/usr/local/lib/python3.7/dist-packages/accelerate/commands/accelerate_cli.py", line 43, in main
args.func(args)
File "/usr/local/lib/python3.7/dist-packages/accelerate/commands/launch.py", line 837, in launch_command
simple_launcher(args)
File "/usr/local/lib/python3.7/dist-packages/accelerate/commands/launch.py", line 354, in simple_launcher
raise subprocess.CalledProcessError(returncode=process.returncode, cmd=cmd)
subprocess.CalledProcessError: Command '['/usr/bin/python3', '/content/diffusers/examples/dreambooth/train_dreambooth.py', '--pretrained_model_name_or_path=/content/gdrive/MyDrive/stable-diffusion-v1-4', '--instance_data_dir=/content/data/romb', '--output_dir=/content/models/romb', '--instance_prompt=photo of romb male', '--seed=12345', '--resolution=512', '--mixed_precision=fp16', '--train_batch_size=1', '--gradient_accumulation_steps=1', '--use_8bit_adam', '--learning_rate=5e-6', '--lr_scheduler=constant', '--lr_warmup_steps=0', '--max_train_steps=800']' returned non-zero exit status 1.

disconnect the runtime and restart the colab to fix that, I cache the model on drive to avoid asking for a huggingface token everytime, there is only one folder on gdrive that's shouldn't cause a problem

I'm getting the same error as 1blackbar in the step Start DreamBooth: "RuntimeError: CUDA error: no kernel image is available for execution on the device"
It's a completely new and clean install and run.

@mauzus which gpu do you have ? run !nvidia-smi

I got a Tesla T4. See the new issue #35 for the full error and info.

Yes i did wipe whole gdrive 5 times and restared, but got same error thats why id prefere separate non gdrive version where you only use gdrive to save the model at the end with your given path , no hardcoded ones and you can edit it easily and gradio webui would use this path as well.

this error is related to your GPU, not your Gdrive, make sure it is supported, I will add the A100 shortly

you can now use any trained model with only one click in the gradio cell without running the other cells and you can choose which model.

Great, i wrote in other comment that id prefere to also have a proper path to input where my model folder is on drive instead of letting it look for its name in default save locations , but it might be just my case ,its just 15gb is not a lot to store multiple trained  subjects so i use shared drives

We need this feature too. Actually, Can i make textual inversion on the fly, why dreambooth finetune is faster than textual inversion, there are puzzles to be solved.

I will add a more customisable path system


Include actual code snippets in quickstart guides

https://github.com/kinvolk/lokomotive/blob/master/docs/quickstarts/packet.md#step-4-define-cluster-configuration

A production-ready configuration is already provided for ease of use. Copy the example configuration to the working directory and modify accordingly.

Rather than sending the user to look for sample configs in links, clone repositories or copy files, I think it's more user-friendly to provide the actual text of a sample config directly in the quickstart guide.

Maybe we can suggest wget'ing them instead? So then we avoid synchronization problem.

IMO the top priority here should be the UX. The config file is a user-facing element. The user needs to understand it and edit it, therefore I think there is no replacement to actual text directly in the guide.
Assuming we do need to have the same snippet in another place, we could use automation to keep things DRY, i.e. generate one from the other. Another thing we can think of is running a CI job on config examples in the docs. We would need to find a solution (e.g. mark cluster config samples with an "invisible" HTML tag in the markdown) to identify the code blocks and execute builds from them.


Update README to reflect where Marvin is accessible from

As of this https://github.com/zenangst/MarvinXcode/issues/35

@onmyway133


Check if we need to bump Serialization to V2

Changes: mesh triangles ushort[] to uint[]

Yes, see: https://github.com/FUSEEProjectTeam/Fusee/commit/f2582f3fe8e232ba900353f16fdce7031c3d2f0b


Create a view to post question

use model :Question
serializer: QuestionSerializer
please add success and bad request error codes

claim

Claim

2 issues are assigned to same person from 1 repo

Please correct this

Assign this to me

2 issues are assigned to same person from 1 repo

added the PR for the earlier one, it was simply merging files 😅

Still against the rules

My bad guys
I was not aware of that rule
it says you cant work on multiple issue in same repo at one time
I am assigning it @frikinomad


Update introduce-class.md

function mapStateToProps(state) {
return { todos: state.${namespace}.todos };
}
connect(mapStateToProps)(App);
I really confused that I follow you to code this sample ,but it dosen't work. Because   the param of  the function mapStateToProps  is state and  get the state number value throught it must add the namespace  , or it would take nothing but undefined.
Thanks!

Checklist

[ ] npm test passes
[ ] tests are included
[ ] documentation is changed or added
[ ] commit message follows commit guidelines

Description of change

any feature?
close https://github.com/dvajs/dva/ISSUE_URL

Coverage remained the same at 86.493% when pulling 499c248c5b27b1d985dec78345fdf73dc5340fe3 on zhanp0304:patch-1 into a5e4ca58b8d40685545cc5bff6a92736afc94d24 on dvajs:master.


Only seeing Sent.com.google.api.client.http.HttpRequest.execute as traces on Stackdriver

What version of OpenCensus are you using?
0.25.0
What JVM are you using (java -version)?
openjdk version "1.8.0_232"
OpenJDK Runtime Environment (build 1.8.0_232-b09)
OpenJDK 64-Bit Server VM (build 25.232-b09, mixed mode)
What did you do?
I have a number of GRPC services written in Kotlin which are running in a Kubernetes cluster, on Google's GKE.
I've followed various pieces of documentation relevant for setting up Stackdriver trace using OpenCensus (notably https://cloud.google.com/trace/docs/setup/java?hl=en_GB).
Based on that, I'm building a lib which is a common dependency of my services. The lib depends on
api
  io.opencensus:opencensus-api:0.25.0,
  io.opencensus:opencensus-exporter-trace-stackdriver:0.25.0

runtimeOnly
  io.opencensus:opencensus-impl:0.25.0

Then the services that depend on that lib also feature a bit of initialization that makes use of given service account, along the lines of
fun initialize(base64SAKeyJson: String) {
    with(Tracing.getTraceConfig()) {
      val traceParams = activeTraceParams.toBuilder()
        .setSampler(Samplers.alwaysSample())
        .build()

      updateActiveTraceParams(traceParams)
    }

    val credentials = getGoogleCredentialsFromBase64SAKeyJson(base64SAKeyJson)
    val configuration = StackdriverTraceConfiguration.builder()
      .setCredentials(credentials)
      .build()

    StackdriverTraceExporter.createAndRegister(configuration)
  }

(The relevant service account is correctly configured with the cloudtrace.agent role and my service logs indicate no error relevant to tracing)
The problem I'm facing is, every single trace shows up as Sent.com.google.api.client.http.HttpRequest.execute which is obviously not as expected. (The traces themselves don't even seem to line up with calls that are made to the cluster.)
Here's a screenshot of Stackdriver's UI which hopefully better demonstrates the issue:

What did you expect to see?
Traces that correspond to calls going into the cluster.
What did you see instead?
Just instances of the Sent.com.google.api.client.http.HttpRequest.execute 'trace' which does not correspond to calls going into the cluster.

Thanks for reporting this!

What version of gRPC are you using?
Was this working for you before OpenCensus v0.25?

Just so you know: since gRPC v1.27.0, OpenCensus dependency is removed from grpc-core. A grpc-census artifact now exists to provide interceptor/stream tracer factory that applies OpenCensus stats/tracing features. Users who want census integration need to add grpc-census artifact to runtime classpath.

Hi @mayurkale22. Thanks a lot for your prompt reply.
Our version of GRPC is 1.27.2. Which judging by what you've mentioned is probably the cause of the issue. (I'm relatively confident our current setup has worked in the past with OpenCensus 0.24.0 alongside GRPC 1.22.2.)
So I'll give io.grpc:grpc-census:1.27.2 a go and report back here. 👌

Okay, so this seems to be working fine now. Thanks very much for your advice.
I still get the odd Sent.com.google.api.client.http.HttpRequest.execute which I haven't been able to figure out but it's rare enough that I can can live with.
Feel free to close 👍 .

Thanks for the confirmation. Closing this issue now.

I still get the odd Sent.com.google.api.client.http.HttpRequest.execute which I haven't been able to figure out but it's rare enough that I can can live with.

Feel free to open a separate issue to address this.


Theme isssue

I was using "@material-ui/core": "4.9.0" for my project. I have my App.js (main parent compo) wrapped in MuiThemeProvider. I transitioned from "@material-ui/core": "4.9.0" ----> "@material-ui/core": "^4.10.2". Everything worked well. No warning , no errors. When I tried to do npm run build I was able to but when I load the acual build it was blank page and all the content was there in browser but still somehow it was blank page. I tried every thing for more than 2 days and finally went to most basic gound approach tried to go back into comments and see the issues. And to my surprise I just added "@material-ui/core": "4.9.0" back to my package.json and removed   "@material-ui/core": "^4.10.2" and boom everything works in build. I have checked this thing again 2 times. When I do "@material-ui/core": "4.9.0" it works whereas with   "@material-ui/core": "^4.10.2"  it works well but build produce blank page.
I am using  React with TS.
	<MuiThemeProvider theme={theme}>
				<CssBaseline/>
				<App />
	</MuiThemeProvider>

This is how I am using theme.

[ ] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
I am able to produce the build without any errors but the page goes blank when I host.
Expected Behavior 🤔
I should be able to get correct build which I am able to get if I do not update Material UI core. Works well with  "@material-ui/core": "4.9.0"  and show the issue when I go this "@material-ui/core": "^4.10.2"
Steps to Reproduce 🕹
It is private repo so can't share.
Steps:

create simple react app with App inside MuiThemeProvider.

Use "@material-ui/core": "4.9.0"  this version to create the app and then put "@material-ui/core": "^4.10.2" in your package.json and delete node modules and run npm i
Now everything will well. Hit npm run build. Try to host build via serve -s build or any other server you have. It will show blank page. And solution to this is put "@material-ui/core": "4.9.0" in package.json and and delete node modules and run npm i. I feel this is issue with some dependency conflict with version 4.9 and versions 4.9 above.
--------------------- OR---------------------
git clone https://github.com/innFactory/create-react-app-material-typescript-redux
go to package.json replace  "@material-ui/core": "4.9.0"  with  "@material-ui/core": "^4.10.2"
Delete node modules and run npm i
Build the project and now when you host it on server you will get the blank page. If you go back to the "@material-ui/core": "4.9.0" version the build will be as per expection. So my issue is when I try for 4.9 above version it creates blank page in build

Context 🔦
Yes it was very bad that I have been developing APP for 1 month and now because of the version issue at Materail UI core I can't do npm run  build to host my application.
Your Environment 🌎
"typescript": "^3.7.5",
tsconfig.json
{
"compilerOptions": {
"sourceMap": true,
"target": "es5",
"lib": [
"dom",
"dom.iterable",
"esnext"
],
"allowJs": true,
"skipLibCheck": true,
"esModuleInterop": true,
"allowSyntheticDefaultImports": true,
"strict": true,
"forceConsistentCasingInFileNames": true,
"module": "esnext",
"moduleResolution": "node",
"resolveJsonModule": true,
"isolatedModules": true,
"jsx": "preserve",
"noEmit": true
},
"include": [
"src"
]
}
| Tech        | Version |
| Material-UI | "4.9.0"  (works)  "^4.10.2" (not works)  |
| React       |     "^16.12.0"    |
| TypeScript  |   "^3.7.5"      |

Make sure you don't have a duplicate version of @material-ui/styles in your bundle: https://material-ui.com/getting-started/faq/#i-have-several-instances-of-styles-on-the-page.


Issue 1053 - execute all fhir-server-test tests by default

This PR introduces a pattern where-by test execution is controlled by properties in the test.properties file.  The advantage is that new tests will be added and executed automatically instead of us needing to remember to add them to the testng.xml file.

uld prefer to see that in either the maven config or the testng.xml), but I do like that it flips the d

same here... I would prefer to make everything enabled by default.


Change default controller ID to 1

Kernel 5.10 or newer added support for encoding controler ID in the
physical port name of the representor  in cases where external
controller exists (Smart-NICs).
This controller ID is by default 1 and not 0.
Signed-off-by: Adrian Chiris<EMAIL_ADDRESS>

This commit introduced the change to devlink port: https://github.com/torvalds/linux/commit/66b17082d10a3b806eec3da8fdebe8a9cd2c6612
This code determines the controller number: https://github.com/torvalds/linux/blob/9d31d2338950293ec19d9b095fbaa9030899dcb4/drivers/net/ethernet/mellanox/mlx5/core/esw/devlink_port.c#L42
when working with upstream kernel it seems the controller number is 1 and not 0

lgtm


Bugged folder rename

Folder rename bugs when trying to rename a folder while another folder with the same name, but different casing exists in the same directory.
The editor always ends up bugged when I perform mass renaming on folders, but just change their casing (not the actual names).

VSCode Version: 1.31.1
OS Version: Linux (Ubuntu 18.04)

Steps to Reproduce:

Create 2 folders with the same name (different casings).
Attempt to rename any of the created folders.

Does this issue occur when all extensions are disabled?: Yes/No

Can you reproduce this with latest vscode insiders?
https://code.visualstudio.com/insiders/


vendor code splitting

I think it would be great to put common libraries for each route such as react, react-dom, react, immutable etc in common chunk like vendor.js.
But there is one trouble npm run start doesn't start app anymore.
npm run development and npm run build works well though.

Hi @berdof
Thanks for the proposal!  It's great to see you taking so much interest in the project. :)
In regards to this piece...
In the early days of the project I did actually have a vendor bundle split.  However, I was having issues with it.  Webpack wasn't doing an optimal bundle of the vendor libs, it looks like no tree shaking etc was working.
Can you try do a build and see what the output size is for the client bundles? The current build sizes are:

The start is failing because the server isn't built to handle multi bundles.  Make sure the vendor bundle only applies for the production client build.
i.e.
    ...
    // Define our entry chunks for our bundle.
    entry: merge(
      {
        main: removeEmpty([
          ifDevClient('react-hot-loader/patch'),
          ifDevClient(`webpack-hot-middleware/client?reload=true&path=http://localhost:${process.env.CLIENT_DEVSERVER_PORT}/__webpack_hmr`),
          path.resolve(appRootPath, `./src/${target}/index.js`),
        ])
      },
      ifProdClient({
        vendor: [
          'react',
          'react-router',
          'react-dom',
          'react-helmet'
        ],
     })
    ),
    ...

I tried to do it this way and bumped into the following exception Uncaught ReferenceError: webpackJsonp is not defined. I think the problem is with the order of included js.

It's probably a misconfiguration of webpack still.  When I get some time I can try this.

Hi @berdof
I have looked into this and I have figured out why the webpackJsonp error occurs.
Basically, when a vendor chunk is included the webpackJsonp function that requires will be embedded within it as it is seen to be the "common" chunk.  Unfortunately the way that I consume the webpack-assets-plugin to embed the chunks onto the page ends up with the vendor chunk being embedded after the main chunk.  The main chunk fires first and fails because webpackJsonp does not exist.
We'll have to create an issue for this and track it.

@berdof - this will be superseded by my DLL implementation. See #105


use redisReaderGetReply(r, NULL) cause memory leak

if reply==NULL， users don't care reply，this will cause memory leak

This fix has been merged, thanks for the report!


Update dependencies

Update dependencies to resolve security vulnerabilities.

Thanks for the pull request, and welcome! The MaidSafe team is excited to review your changes, and you should hear from @krishnaIndia (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. The way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see CONTRIBUTOR.md for more information.

Styles are broken and gulp is not functioning as expected.
That's strange. They seemed okay to me. Here are the steps I followed.

Removed node_modules inside the web API playground
Ran yarn to install the packages
yarn gulp served the app on localhost:3003
I even uploaded the build dir via WHM and the published sited seemed fine too.
Am I going wrong somewhere?

@lionel1704 Most likely, is that yarn has cached safe_ux_guidelines, so it may be continuing to install for you even though the repo has been deleted.

I did a yarn cache clean and followed the same steps again. Still seems okay to me. @manavbp gave it a try on his machine and it seemed okay there too.
If it's still breaking for you, feel free to raise a PR to my branch.

@lionel1704 Okay I'll open a PR on your branch.
This is what my styles look like:

What are the contents of you build directory?

Here you go build.zip

@lionel1704 Thank you. main.css still contains scss from the no longer existing safe_ux_guidelines.
Did you delete build directory and then run yarn gulp again?

@lionel1704 Looking good! Thanks, Lionel.
Last thing is to update the gulpfile: https://github.com/lionel1704/safe_examples/pull/1

@S-Coyle can we have this merged please?


Trying to create a large Array causes a silent crash in CowData

Godot version:

v3.2.3.stable.official
OS/device including version:

OS: Arch Linux (I don't know how to give a version for this)
CPU: x86_64 AMD Ryzen 5 3600 6-Core Processor
GPU: AMD Radeon RX 580
RAM: 15 GB + 8 GB swap
Issue description:

extends Node

const ARRAY_SIZE_LIMIT = 89478485

func _ready():
	[].resize(ARRAY_SIZE_LIMIT)  # This doesn't crash
	[].resize(ARRAY_SIZE_LIMIT + 1)  # This crashes

When the game crashes, there's no error message. Even if this should have caused a crash, there should at least be an error message.
Steps to reproduce:

Create a new scene.
Attach the above script to a Node.
Run the scene.

Minimal reproduction project:

array_size_limit.zip

I ran into this too, here's my reproduction project for master:
	var INT32_MAX =<PHONE_NUMBER> # 2^31 - 1

	var byte_arr = PackedByteArray()
	byte_arr.resize(INT32_MAX) # OK
	byte_arr.resize(INT32_MAX + 1) # error

	var int32_arr = PackedInt32Array()
	#int32_arr.resize(INT32_MAX) # crash
	int32_arr.resize(INT32_MAX + 1) # error
	int32_arr.resize(INT32_MAX / 4) # OK, int32 is 4 bytes so alloc == INT32_MAX - 3 (due to truncation)
	int32_arr.resize(INT32_MAX / 4 + 1) # OK, not sure why (alloc INT32_MAX + 1)
	#int32_arr.resize(INT32_MAX / 4 + 2) # crash, alloc INT32_MAX + 5

I'm not sure #46868 is the right solution, it seems to me that we should be able to fix CowData to support allocating arrays of more than<PHONE_NUMBER> bytes.
The error mentioned in my script is:
ERROR: Condition "p_size < 0" is true. Returning: ERR_INVALID_PARAMETER
   at: resize (./core/templates/cowdata.h:260)

This is fixed in 4.0 RC 1.


Abstract time related API so time can be faked in tests

Should I squash and merge or do you want to use normal merge?

In general, it might be better for reviewers to just approve the PR and let the contributor do the merging.
There is cleanup associate with merging: delete local branch / delete remote branch / merge the changes with the other branches in flight / etc...
What do you think? Maybe to add in Contributions.md ? Or we should make it clear in the file how the approval / merging is done?
Here, yes, merge without squash would be better

In general, it might be better for reviewers to just approve the PR and let the contributor do the merging.

Do you really want to give "all" contributors that send in a PR also write rights to the repo? No idea how you wanted to "Build up the project". I know it more that there in many cases will be a "core team of devs" that have write rights and many more contributors fropm outside.
I agree with you for "contributors with write access" - there this makes sense, for others obviously not :-)
I can add that to the Contributing file.

OK, makes sense.
Or we can also not overthink this: just for me don't merge since I would prefer not and we can leave this out of contributors since this is maybe too prescriptive for not that much value.
But I will also survive if you do merge my PR each time ;-)

;-)
I think this rule "core team contributors" merge PRs fro others, but not from other core contributors" makes sense


Added "JsonBodyParser" middleware

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
❌

Tests pass?
✔

Fixed issues
comma-separated list of tickets # fixed by the PR, if any

absorbed by https://github.com/yiisoft/request-body-parsers/pull/1


Complete code release

Hi, thanks a lot for your nice work.
As the training code has been released, when will you release the complete code (e.g., data process, evaluation, etc)?

Thanks for your interest. The other parts will be released soon once I finish cleaning the code.


Migrating from rdb:svelte-meteor-data

It may be useful to have a guide to migrate from rdb:svelte-meteor-data to zodern:melte.  Some suggestions:
No need for $var
In svelte-meteor-data, one uses:
$: players = useTracker(() => Players.find().fetch())

and then in the HTML:
{$players}

In Melte, this would become:
$m: players = Players.find().fetch()

and
{players}

No need to include anything
You do not need to include anything for $m: to work.
Remove rdb:svelte-meteor-data and svelte-compiler
$ meteor remove rdb:svelte-meteor-data
$ meteor remove svelte-compiler

You may also want to install typescript and svelte-preprocess:
$ meteor npm i typescript --save-dev
$ meteor npm i svelte-preprocess --save-dev

Some apps use both svelte-meteor-data and melte, so it isn't necessary to migrate (the app from meteor create --svelte even comes with both). Though if anyone wants to, feel free to open a PR to add a migration guide.


BasicClusteredWriteBehindTest is failing randomly

This test keeps failing on CI.
Example is https://dev.azure.com/TerracottaCI/ehcache/_build/results?buildId=339&view=logs
I don't know if it shows an actual problem or it is just unstable.

Still failing https://dev.azure.com/TerracottaCI/ehcache/_build/results?buildId=379&view=logs

Here the test seem to have deadlock: https://dev.azure.com/TerracottaCI/ehcache/_build/results?buildId=405&view=logs

@henri-tremblay I don't see any client/server logs for failing tests, is there a way to get them?

There are no failing test. The buuld got stuck on the test and then timed out

I am not asking for this particular test failure, can't find client/server logs for any failed tests. BTW how do you know this test got stuck?

No idea. I think we should add a rule to timeout and see where we are stuck. @akomakom Are the server logs somewhere?
    @Rule
    public Timeout timeout = new Timeout(1000, TimeUnit.MILLISECONDS);

@henri-tremblay nothing is currently saved.  We can, I believe, have the job attach arbitrary artifacts to the build.  Alternatively, if you can have the Timeout logic print out the logs to stdout, that would be part of the build output in azure.

Yes. That's what I will do right now.

@ramsai1729 Please merge #2528 and rebase this PR. IT should help debugging

@ramsai1729 @akomakom Have you seen it failing lately?

AFAIK no recent failures.

Let's consider it fixed.


[windows] Add 10 22H2

https://learn.microsoft.com/windows/release-health/release-information
Issue #1775

@BiNZGi thank you 💯


[FEATURE] add option for disable the format phase

What feature you'd like to add:
添加禁止format流程的选项
也即，format可以只作为环境准备的一部分，而不是csi的必要流程
Why is this feature needed:
例如，存储提供方（存储部门）和存储使用方（容器部门）属于不同的部门
存储提供方只提供已经预先做好format流程的 metaurl 即可，这样也不会暴露对象存储的秘钥等信息给使用方

Hi @CodeRusher , only "metaurl" and "name" is required in secret. If you do juicefs format before and set "metaurl" and "name" in secret, mount pod will do juicefs format <metaurl> <name> --no-update without update s3 credentials.
Did these meet your needs?

thanks @zwwhdls , it works for me!!!

Hi @CodeRusher , only "metaurl" and "name" is required in secret. If you do juicefs format before and set "metaurl" and "name" in secret, mount pod will do juicefs format <metaurl> <name> --no-update without update s3 credentials.
Did these meet your needs?

I also have this problem. I am using ceph librados, but if I don’t provide storage and bucket, will the default values be used? Why not provide a switch to turn it off?


Updated links to point to active documentation at warehouse.readthedocs.org

Very minor fix of links in text files.

Fixed.


keep socket open

Anyway to keep the socket open?
Each Time I issue this command :
./envisakit/envisakit-cli status -j
the socket gets opened, then after command is completed, socket is closed.
Anyway to keep the socket open, so I can keep sending the same command over to get up to date status.

I did not develop this. I found it on GitHub and was using it along with cron to control my alarm. The original repository was removed from so I put this one back up. I have since moved to Home Assistant and no longer use this.


Improve Aarch64 performance by removing NEON pipeline stalls.

This PR re-orders the computation in the accumulation loop of the 8x4 assembly kernel so as to remove pipeline stalls by increasing the distance between data-dependent instructions.
Single-threaded performance on a Raspberry Pi 4B improves by ~8.5%, and on my (Android) phone by ~5.0%.
What do these changes do?
Execution latency is the number of cycles that must pass before a result is available in a result register, after an instruction is issued. If a result of an instruction is an operand of the next instruction, the processor is forced to wait until the previous result is available (the pipeline is stalled).
In the Cortex A72 software optimisation guide I found the execution latencies of Neon instructions for that CPU (note: this is the CPU used in the Raspberry Pi 4B). The important ones for us are:

Instruction
Description
Execution latency

ADDP
Vector pairwise add
3

CNT
Vector byte-level popcount
3

DUP
Duplicate regular register into vector register
8

EOR
Vector exclusive or
3

LD1
Vector load (1 register, 16 bytes)
5

LD1
Vector load (2 registers, 32 bytes)
6

LD1
Vector load (4 registers, 64 bytes)
8

UADALP
Vector unsigned add and accumulate long pairwise
4

The problem
There are quite a few places where pipeline stalls occur in the existing assembly kernels. For example, there is a two-cycle stall in the BMLA instruction here:
https://github.com/larq/compute-engine/blob/936f5f0273c2e9b119ff317ea6de7f5498dcb9d5/larq_compute_engine/core/bgemm_kernels_arm64.h#L531-L532
This doesn't just happen between consecutive instructions. Here, there is a two-cycle stall at the smax instruction because the execution latency of the dup instruction is very high (8 cycles).
https://github.com/larq/compute-engine/blob/936f5f0273c2e9b119ff317ea6de7f5498dcb9d5/larq_compute_engine/core/bgemm_kernels_arm64.h#L840-L850
By far the worst place for pipeline stalls is at the end of the BMLA macro, when accumulating the byte-level popcounts. There is a two-cycle stall between each of these instructions:
https://github.com/larq/compute-engine/blob/936f5f0273c2e9b119ff317ea6de7f5498dcb9d5/larq_compute_engine/core/bgemm_kernels_arm64.h#L536-L540
The solution
At the moment, the BMLA macro quite logically performs the xor, then the cnt (the byte-level pop-count), then accumulates the result and adds it to a destination register. Unfortunately, this makes the sequence of pipeline stalls at the end of the macro unavoidable, because there are no instructions that can be inserted between the data-dependent addp/uadalp instructions.
However, in theory while accumulating the cnt results we could already be computing the xor for the next loop iteration. These xor instructions could be scheduled inbetween the data-dependent accumulate instructions of the previous iteration, removing the pipeline stalls (and making the xors effectively 'free').
This PR replaces the BMLA macro with a CNT_ACCUM_XOR alternative, which assumes that the xor of the current iteration has already been computed, then performs the CNT and accumulate in parallel with the xor of the next iteration. The new macro has no pipeline stalls (using the execution latencies from the table above), because at least three other instructions lie between every data-dependent instruction pair.
Caveat
In order to do this, the new macro needs a lot more registers than the old one did. To free up the necessary register space, I have removed the RUY_OPT_MAX_STREAMING part of the existing kernel. However, as discussed in #350 the RUY_OPT_MAX_STREAMING part of the kernel was not noticably improving performance, and even if it were this decision is justified by the benchmarks below.
How Has This Been Tested?
CI. The 'big' kernel tests that don't run on CI have been tested locally (and pass).
Benchmark Results
I have benchmarked current master (the 'Baseline') and this PR on a Raspberry Pi Model 4B and on my Android phone (a OnePlus 6). I used the LCE benchmark utility with num_runs=250 and report the average and the standard-deviation.
Raspberry Pi 4B

Model
Threads
Baseline
PR
Relative change

QuickNet
1
35.00 +- 0.06
31.98 +- 0.04
-8.6%

QuickNetLarge
1
52.81 +- 0.08
48.75 +- 0.14
-7.7%

QuickNetXL
1
92.49 +- 0.14
83.99 +- 0.14
-9.2%

QuickNet
4
17.48 +- 0.10
17.02 +- 0.05
-2.6%

QuickNetLarge
4
25.58 +- 0.09
24.39 +- 0.21
-4.7%

QuickNetXL
4
41.85 +- 0.12
39.09 +- 0.11
-6.6%

OnePlus 6

Model
Threads
Baseline
PR
Relative change

QuickNet
1
18.70 +- 0.21
17.83 +- 0.19
-4.7%

QuickNet Large
1
27.27 +- 0.22
26.02 +- 0.22
-4.6%

QuickNet XL
1
48.05 +- 0.28
45.27 +- 0.35
-5.8%

QuickNet
4
10.08 +- 0.32
9.79 +- 0.23
-2.9%

QuickNet Large
4
14.20 +- 0.30
13.84 +- 0.26
-2.5%

QuickNet XL
4
25.46 +- 0.42
24.72 +- 0.50
-2.9%

The performance improvement is greater on the Raspberry Pi (average 8.5%/4.7% performance gain for 1/4 threads) than on the OnePlus 6 (5.0%/2.8%).
Related issue number
N/A.

@lgeiger could we make a new label for performance or something like that? Although this isn't a new feature, it seems weird to describe it as an 'internal' improvement because the performance of LCE is the most important thing about it.

This is really really good. I think we should consider a special release on top op v0.3.1 to release this earlier and not wait for v0.4!

I think we should consider a special release on top of v0.3.1 to release this earlier and not wait for v0.4!

I'd be happy to push a new release based on v0.3.1, though I am not sure how cleanly this will backport since this PR is built ontop of #351 and #389. Cherry-picking #389 won't be an issue, but #351 might need some manual work.

With regards to releasing a new version, I'm happy to go either way. I definitely see the merits of getting a new version out sooner, but also if we save this for v0.4 then there'll be more to talk about when that release comes.


what about lifetimes?

There are plenty of widgets and other stuff that requieres lifetime specifiying, like span, block , paragraph, text, etc. But all of it comes down to a single thing - Cow<'a, str>. I am new to rust, and i guess this was done for a higher performance. And even if so - maybe add it as a feature?
Just replace all Cow with String and that's it!

The widgets were originally created as one shot commands, which were consumed, which meant having the option to pass in borrowed or owned strings was pretty useful. We've since made it possible to render widgets usually as references, which makes that less necessary. But there's a large amount of code that uses ratatui and this would probably the sort of change that would be fairly invasive / problematic to fix (just guessing here, I may be wrong).
As a general easy workaround in your own app code, you can just make 'a 'static, which gives you basically the same thing as String. Give it a whirl, see if it performs good enough for your purposes (check out cargo-flamegraph, and dhat for some ideas or a proper profiler).
I'm going to move this over to discussions as it's less of a problem than something that may spawn some ideas...


manually instrumented traces are missing RUM details

Though AndroidTracer is configured with setBundleWithRumEnabled(true), fundamental rum details are missing in subsequently collected traces:
missing attributes: context, device, env, geo, os, and session

hi @joshskeen, we'll look into this. Could you please provide a code sample of how you are performing initialisation?

AndroidTracer.Builder()
.setSampleRate(sampleRate)
.setBundleWithRumEnabled(true)
.build()

@joshskeen one small not on what I stated above, if you use the RUM2APM feature, RUM will automatically create traces for each resource request in your application. Those traces will have by default the Geo,  Os and Device information generated by us. For all the other traces created manually with our API the above statement can be applied.

Hi John, let me chime in here .

what is the RUM2APM feature? Can you link to the documentation for this please?

Sorry about the lingo, RUM2APM is our internal way of talking about the full tracing of network requests. This is due to the fact that the tracing originate through RUM instrumenting network requests via our browser and mobile SDK, and connecting it to the APM traces tracked in the backend to give a full picture. From a RUM session, one can dig into an APM trace and e.g.: understand what caused a 500 response. You can find more information about that on this page

it appears that network traces collect context, device, env, geo, os, and session. If we omit these for performance reasons, why do we include them with these particular traces?

In the case of RUM2APM, the root span is generated from the RUM resource, meaning it uses the aggregation of all the attributes available in the backend ingesting the RUM session event, which is then duplicating all the attributes into the span, and then is forwarded to the APM processors.
When generating non-network spans in the mobile, the span doesn't go through the RUM processors, and is sent directly to the APM processors. Getting all of the informations available in RUM would require to also go through both stacks for all spans and not just the network ones.

Rather than manually passing in these parameters to each manual trace, it would be ideal to specify which rum parameters would be included in each trace. Is there any mechanism to do this?

There's no mechanism to specify which parameters should be copied from RUM as of know, but if this is something important to you, feel free to open a feature request via our support channel.

It appears that manually collected traces do not reference the RUM session they are collected within in the flame graph as a UI option, though setBundleWithRumEnabled is true. What additional details are required to enable this behavior in the trace flamegraph display?

This is an issue that we identified (we discussed this on slack) and is in our roadmap, we plan on fixing that as soon as possible

Hello @joshskeen!
Version 2.5.0 should now have a better integration of manually collected traces with RUM, you should see a link between them in Dashboard. In the meantime we are working on improving this integration for data collected by the older SDKs.


Best method for exploring and obtaining values of nested json objects when the names are not known beforehand?

Hi Niels,
Could you please advise me the best way to explore down into a list of JSON objects, without knowing the names of the nested objects beforehand.
I'm using the notation below (1 & 2) for setting and getting values and this works well, however I've not been able to find a way of quickly getting the names and values of objects when I don't know the names of those objects beforehand. I've used the iterator method (3) however it doesn't seem suited for quickly diving and exploring into nested objects.
What is the most direct way of exploring and getting values of nested objects when the names of those objects are not known beforehand?
(1)

(2)

(3)

I'm stuck at this stage.. I don't know if this is the best way of delving into nested objects.. it seems promising but I don't know how to do it.
Here is the code I'm using to get pointer to the first object ("group 1") inside the json file read into my program. From here I need to get the name (unknown) of the "group 1" nested object ("Schedule1") and obtain names for each successive sibbling object ie "Schedule1", "Schedule3", "Schedule4" etc... however I don't know beforehand their names.. they could for example be named "Morning Schedule", "Afternoon Schedule", "Rainy Day Schedule", etc or something else. ... so these names are unknown.

Debugger view below:- I have the p1 pointer.. it's pointing to "group 1" (obtained in code above)... using this p1 pointer I now need to locate position of "Schedule1", "Schedule2", etc and obtain their names (ie unknown arbitrary names) and read their values (the structure of this 'schedules' data is consistent and is known), so I can access and recreate this json data inside my program within a corresponding linked list of data.

I don't know what to do next.. your help again be much appreciated.

Explicitly showing these arbitarily named 'schedules' objects:-

How do I access these in a succinct manner without knowing their key names beforehand?
Thanks.

Hi Niels, This works below however I'm wondering, is there a more direct and succinct way of querying nested keys and obtaining their values of unknown 'key' name'?

This example should help make your for loop cleaner, but you have the basic idea.  Just make sure that you have the right types when accessing things, as in your other issue.  Use auto wherever possible so that you don't silently change the type.
https://github.com/nlohmann/json/blob/develop/doc/examples/iterator_wrapper.cpp

Can I close this issue?

yes
On Sat, Sep 23, 2017 at 10:56 PM, Niels Lohmann<EMAIL_ADDRESS>wrote:

Can I close this issue?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nlohmann/json/issues/747#issuecomment-331632458, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AeOR17naUY1KzU1JsewO0iQKfuQ-1HXeks5slQAWgaJpZM4PbjgF
.


Fixes to SYS_READ changes in PR #727 per review.

I agree with Fabien-Chouteau's review items. Thanks for catching my sloppy errors.
Here is a PR with those things fixed.

Thanks @Fabien-Chouteau and @donmr !


feat(metrics): Add stats.wasm to proxy config

Description:
This change propagates the stats.wasm module to each sidecar proxy's
config. stats.wasm is built and embedded in the osm-controller binary
and configured inline via LDS.
Follow-up from #2531

This PR is part of several which together extend Envoy to generate custom
metrics needed to implement the SMI metrics spec (#984).
Here is a map of the PRs as they're currently planned:

(#2479) ref(injector): pass pod object to getEnvoySidecarContainerSpec
(#2488) feat(metrics): add WASM metrics source
(#2503) docs(metrics): Add docs for WASM stats
(#2517) feat(metrics): Add WASM feature flag
(YOU ARE HERE) feat(metrics): Add stats.wasm to proxy config
feat(injector): Add name, workload name, workload kind to PodMetadata
feat(metrics): Add source labels to WASM metrics
feat(metrics): Add dest labels to WASM metrics
feat(metrics): Add "unknown" for dest labels on local replies
feat(metrics): clean up WASM metrics in Prometheus config
feat(metrics): add flag to enable WASM metrics
tests(e2e): add WASM metrics test

The remaining changes can be found here: https://github.com/openservicemesh/osm/compare/main...nojnhuh:wasm-metrics

Affected area:

New Functionality      [ ]
Documentation          [ ]
Install                [ ]
Control Plane          [X]
CLI Tool               [ ]
Certificate Management [ ]
Networking             [ ]
Metrics                [X]
SMI Policy             [ ]
Security               [ ]
Tests                  [ ]
CI System              [ ]
Performance            [ ]
Other                  [ ]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project? If so, did you notify the maintainers and provide attribution?
No

Codecov Report

Merging #2546 (3ebdac3) into main (2434267) will increase coverage by 0.09%.
The diff coverage is 88.37%.

@@            Coverage Diff             @@
##             main    #2546      +/-   ##
==========================================
+ Coverage   59.29%   59.38%   +0.09%
==========================================
  Files         156      157       +1
  Lines        7115     7158      +43
==========================================
+ Hits         4219     4251      +32
- Misses       2880     2891      +11
  Partials       16       16

Flag
Coverage Δ

unittests
59.38% <88.37%> (+0.09%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/envoy/lds/connection_manager.go
93.54% <77.77%> (-2.68%)
:arrow_down:

pkg/envoy/lds/wasm.go
91.17% <91.17%> (ø)

pkg/configurator/validating_webhook.go
76.55% <0.00%> (-3.45%)
:arrow_down:

pkg/envoy/route/config.go
95.20% <0.00%> (-0.80%)
:arrow_down:

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 2434267...3ebdac3. Read the comment docs.


vscode-dts causing compile errors due to downloading vscode.d.ts into root of folder

Refs: https://github.com/microsoft/vscode/issues/74436
Steps:

be on Windows
yo code a hello world TS extension
open in VSCode
run it once from debug, should work fine
run npx vscode-dts release/1.32
run it again from debug

=> 🐛 I am seeing a lot of errors:

It looks like the npx command results in vscode.d.ts downloaded into the root of the project, is that correct?

Btw cc @joaomoreno @sandy081 for broken tree

I think that's as-designed. Do you think we should add correct @types/vscode dependency instead of downloading raw dts in this case? I thought vscode-dts as an alternative to @types/vscode, not something on top of it.

@octref I actually suggest to print the warning but also just remove the folder node_modules/@types/vscode as part of the operation.

Made a small CLI prompt and released in 0.2.0.


Fix bug in streaming_conformer_ctc egs

Fix bug in streaming_conformer_ctc egs
  File "/proj/xcdhdstaff1/bokangz/project/k2/icefall/egs/librispeech/ASR/streaming_conformer_ctc/train.py", line 648, in run
    train_dl = librispeech.train_dataloaders()
TypeError: train_dataloaders() missing 1 required positional argument: 'cuts_train'

Thanks for your contribution!


Add codemod for array-buffer-byte-length package

Codemod for array-buffer-byte-length.
~I thought about abstracting some of the ArrayBuffer stuff into a shared util, but I think there's only one other ArrayBuffer shim that needs a codemod, arraybuffer.prototype.slice, so I don't think it's worth it.~
EDIT: Actually, I think it could be worth adding utils like transformInstanceMethod and transformInstanceProperty. Would be useful for both ArrayBuffer and TypedArray codemods. I'll contribute them separately if they turn out to be useful.
I'll follow up with the slice codemod in a bit.

Oh, this has conflicts now because I merged the other PR, could you fix the conflicts and push? 🙂 otherwise LGTM!

@thepassle Fixed.

Thanks!


Add OOI doc

Overview
This PR adds initial doc for OOI datasource.

Pull Request Test Coverage Report for Build 100

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 58.0%

Totals

Change from base Build 99:
0.0%

Covered Lines:
232

Relevant Lines:
400

💛  - Coveralls

Self merging again. Just updating docs...


Zhenyi mitternachts data

Added Zhenyi; was going to add Mitternachts too, but it didn't have any damage buffing effects

Just one minor comment, otherwise looks great!

Merged and deployed


Please merge revision 8c9c54d833865201abdaadbfa25c0c17b07d9926 into dev channel

@whesse @mit-mit @kasperl
Fix for https://github.com/dart-lang/sdk/issues/29229, "Closing stdin during cleanup of isolate is crashing Flutter Tests"
Revision: https://github.com/dart-lang/sdk/commit/8c9c54d833865201abdaadbfa25c0c17b07d9926
This merged cleanly

Merged in 1.23.0-dev.11.5


Include license for Balrog

Hi,
I'm looking to use some modified code from Balrog, and I'd like to include your MIT license detailed in the BioRxiv paper. Would you be able to generate one and provide it in the Github repository if possible, please?
Many thanks,
Sam

Added

Added


First proposal for the API

fixes #4
Changes
First iteration of unzipomatic API.
I've went with functions and not classes because so far I don't see any incremental state building there at all (it is more natural for building archives, not extracting them)
Checklist

[X] Apply one of following labels; major, minor, patch or skip-release
[ ] I've updated the documentation, or no changes were necessary
[ ] I've updated the tests, or no changes were necessary

Also, I think we should follow the commitzen to commit, like: feat, chore, fix, then we can generate changelogs and releases automatically.

misclicked and didn't add feat: prefix during squash, sorry. will include prefixes in all new PRs, though


bpf: lb: have __lb*_rev_nat() take the source port from CT tuple

Instead of loading the source port from the packet, obtain it from the provided CT tuple.

/test

I want to have a quick look whether all callers really provide a fully populated tuple. And if there's any complexity surprises. But should be good for review.

I want to have a quick look whether all callers really provide a fully populated tuple. And if there's any complexity surprises. But should be good for review.

all good.


Remove build/publish process for AMD tag

This was proposed in August 2017 to be done in October 2017. v0.8.8 will be the last AMD release. This PR removes the RequireJS module tag.
Fixes #1289
Additional description
To test npm run publish:origin and check your forked origin repo. It worked for me.

Good idea!


Rewrite README

The README is a little bit...bare.
The CONTRIBUTING document is too heavy on content, much of that should be in the README, if we were to follow common practices.
I'm not going to enforce any strict structure, as there are multiple variations and they are all the time getting better and better.
Check out https://github.com/matiassingers/awesome-readme for inspiration.
For Go projects, look at https://github.com/avelino/awesome-go. There may be Go-specific things we can have on the README like pkg.go.dev badges.

Could find a few badges that will look good on the readme :)

These will require some changes in the codebase as well, for ex the go report grades are too poor and godocs dont look that useful.
For the README structure, adding installation instructions and a one-line description of the CLI along with a screenshot (like the one in gh cli) will make it more useful. What do you think?

I don't think a screenshot makes it more useful necessarily, it can go out of date quicker than high level documentation. Giving you creative control on this though :)
Look at projects other than GitHub CLI though like kubectl and gcloud which aligns more with our offering.

I don't think a screenshot makes it more useful necessarily, it can go out of date quicker than high level documentation. Giving you creative control on this though :)
Look at projects other than GitHub CLI though like kubectl and gcloud which aligns more with our offering.

Put it wrongly. screenshot is for a better look, not usefulness, not sure if we should stick to it. Let me try some combinations.

This will also close https://github.com/redhat-developer/app-services-cli/issues/622

README.md should mention install script or releases for installation.
Additionally install script can be available as part of the release files (so it is versioned properly etc.)

This can be sourced into 4-5 micro issues


Issue integrating into angular2-seed

Hi, great module, but was having trouble getting it to work with angular2-seed (https://github.com/mgechev/angular2-seed) out-of-the-box.  I was getting a 404 error when SystemJS was trying to find moment.  For anyone using angular2-seed, and for @urish, only 1 line is needed to get it to work:
In tools/config/seed.config.ts, line 87, add the line between the comments:
  protected SYSTEM_CONFIG_DEV = {
    defaultJSExtensions: true,
// ADD THIS LINE
    packageConfigPaths: [`${this.APP_BASE}node_modules/*/package.json`],
//
    paths: {
      [this.BOOTSTRAP_MODULE]: `${this.APP_BASE}${this.BOOTSTRAP_MODULE}`,
      'angular2/*': `${this.APP_BASE}angular2/*`,
      'rxjs/*': `${this.APP_BASE}rxjs/*`,
      '*': `${this.APP_BASE}node_modules/*`
    },
    packages: {
      angular2: { defaultExtension: false },
      rxjs: { defaultExtension: false }
    }
  };

I'll post this in angular2-seed as well, since other modules may encounter the same issue.

Hey there,
have you also installed the typings for angular2-moment as described in the Installation section of the README?
You do need to have typings installed as a global node module ($ npm install -g typings).
Greetings
Christian

Indeed, I did.  I have typings installed globally and moment's typings.  angular2-moment never threw a 404 or error, it was only moment itself.  I drew from this post to include the packageConfig option to get it working: https://github.com/urish/angular2-moment/issues/11#issuecomment-180819466


Theme UI derived styles are non-functional in prod

Describe the bug
Theme-UI-derived responsive CSS values are not interpolating correctly on the production site (e.g. named scales like size.icon, font.body)
To Reproduce
Steps to reproduce the behavior:

Go to https://www.zastoupil.art
Observe that the background and text colors are the default for the browser, as are the fonts.

Expected behavior
Custom theme colors and fonts should be obviously visible on the site - see screenshots for an example
Screenshots
Current production site on Chrome:

The same codebase, running locally with yarn dev:

Desktop:

OS: Mac OS
Browser: Chrome
Version: 106.0.5249.103 (Official Build) (x86_64)

Smartphone:

Device: Pixel 6 Pro
OS: Android 13
Browser: Chrome
Version:106.0.5249.118

Additional context
Add any other context about the problem here.

After manually checking out commits in the history and running yarn build -> yarn start to simulate a production build, I've isolated the commit that caused this issue to 50a4dd728ad99e31ed98de13fcddd689f315b1b1

It seems the root cause of this was the empty radii.js file, a scale imported by the main theme object. In prod, the empty file caused the theme to fail to load entirely, while in dev the empty file was simply ignored.


Fix plugin configs for aspnetcore docset

Docs Build status updates of commit b0f7ac7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Inconsistent use of - and _ in command line options

In the interest of backwards compatibility, this is probably not something that is going to to be fixed, but if future options are added you should make a choice of which one to use.
The problem is that the location commands use kebob-case but all other commands use snake_case, e.g.,
--search-location ought to be --search_location
and
--include-location ought to be --include_location.

Agreed. Changed all the arguments to use kebab-case but will still support the older arguments using underscores to preserve backward compatibility.


I need Push Notification show when my application opens like whatsapp, Viber in android(Phonegap)

Hi,
I got success setup push notification using PushPlugin in android(Phonegap).I got Notification when my application close. The problem is that I need Push Notification come when my application opens like whatsapp, Viber.
If it is possible in Phonegap?? Please help me or send me your valuable comment.

What you are looking for is probably the Cordova resume event:
https://cordova.apache.org/docs/en/4.0.0/cordova_events_events.md.html#resume
I have used this to implement something similar.  The APN or GCM handler manages a small queue using localStorage.  The on resume handler empties this queue.


Support finally tasks

Fix: #345
Demo:

Codecov Report

Merging #401 into master will increase coverage by 0.18%.
The diff coverage is 97.22%.

@@            Coverage Diff             @@
##           master     #401      +/-   ##
==========================================
+ Coverage   74.51%   74.70%   +0.18%
==========================================
  Files          58       58
  Lines        3834     3854      +20
  Branches      693      699       +6
==========================================
+ Hits         2857     2879      +22
+ Misses        977      975       -2

Impacted Files
Coverage Δ

src/yaml-support/tkn-yaml.ts
89.53% <96.29%> (+0.47%)
:arrow_up:

src/model/pipeline/pipeline-model.ts
80.65% <100.00%> (+1.17%)
:arrow_up:

src/pipeline/pipeline-graph.ts
76.10% <100.00%> (ø)

Continue to review full report at Codecov.

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

@siamaksade I'm not sure about final task in graph


Make ind9/dfs-datastores independent project

Since we've diverged a lot from upstream, I guess it's fair enough to remove the fork reference (with attribute to original work) and remove the develop -> master convention. Also change the groupId and publish it to maven central.
There are few things that I guess we could just remove from the repo which we don't use like VersionedTap and some custom schemes.
There're some work that was done in for support Pail from Spark we could move those in here as a separate module.
@sriram-r @kishorenc @vinothkr @codingnirvana Thoughts?

Update - This is no longer a fork of nathanmarz's project. I guess we can now start publishing this under com.indix groupId to maven central after few of the refactoring that was mentioned earlier in here.

This is now fixed!


No Controller Connected

Describe the bug
The website says "No Controller Connected" when the controller is connected to the phone. Am I missing a step?
To Reproduce
Steps to reproduce the behavior:

Install StadiaWireless
Connect controller to stadia app on phone
Open webpage on phone

Expected behavior
Controller connected
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Windows 11
Browser Chrome
Version 1.2.0

Smartphone (please complete the following information):

Device: Pixel 5
OS: Android 13
Browser Chrome
Version 1.2.0

Is the controller visible on your phone when you open https://gamepad-tester.com/

I got it to work by disconnecting the controller from the stadia app.


Fixing schedule logout

Fixes Schedule

@goombamaui will test at break.

Is there no way to redirect back to /schedule after logging in?

Found a way. At /login, you can set the URL parameter destination.
https://schoology.harker.org/login/ldap?&school=<school ID>&destination=schedule.

@goombamaui check my last commit. Basically it does what I proposed in my last comment.
Also, I don't think we need to click the logout button programmatically, but just redirect to that link. It's cleaner and more reliable. Wdyt?


New feature: S3 compatible driver

Dear friends at SynchroLabs,
I find Stashbox very useful for configuration file (and other) distribution in a Docker microservices environment, but I miss a S3 compatible storage driver. So that I created such a driver based on minio-js. I would like to contribute this driver to StashBox if it is OK for you (but I need the corresponding permisions to push a new branch and ask for a pull request).
I hope this can be interesting for you
Thank you very much in advance
Best
PS: I'm sorry to use an Issue creation for this but I was unable to contact you in other way

Does this provide any additional functionality to the S3 support that is already provided by StashBox via PkgCloud?

Hi Bob,
Thank you for your answer
As far as I could view, current PkgCloud does not support non-Amazon S3 compatible service (as for example Atmos from DellEMC, Swarm from Caringo or OBS form Telefonica Open Cloud).  When configuring the mount with PkgCloud I don't see how to configure a custom endpoint (you can select just the region, that points to Amazon endpoints). And this is the additional functionality: support to non Amazon, S3 compatible services.
Thanks
Best
Fernando

It seems that pkgcloud may support non-Amazon endpoints.  See: https://github.com/pkgcloud/pkgcloud/blob/b7d7218d1d1c7f9f81905f03103855ead3eb277c/lib/pkgcloud/amazon/client.js#L47-L49
It looks like you can just set "endpoint".  You might try and see if that works for you.
That being said, I have no objection to you adding your driver.

You should be able to submit a pull request without being added as a contributor.

I tried to add the endpoint to the config file and it seems to point to the custom endpoint I set, but after a lot of tests I alwais get a "NoSuchKey" error
[2017-04-22 16:56:14.078] [ERROR] console - NoSuchKey: The specified key does not exist.
(of course the objet is there, I clould use it with the "alternative" s3 driver I created).
Here is the config file I used
{
"mounts":
[
{
"mount": "/",
"provider": "amazon",
"basePath": "fimtest", <---- the bucket where I stored the file I request
"keyId": "TheAccesskey",
"key": "TheSecretKey",
"region": "eu-west-2",
"endpoint": "obs.na-mexico-1.telefonicaopencloud.com"
}
]
}
As for the submision of the s3 copatible driver, when I try to puth the branch I created for that I get an access denied
$ git push -u origin s3compatible_driver
remote: Permission to SynchroLabs/StashBox.git denied to fernandodelaiglesia.
fatal: unable to access 'https://github.com/SynchroLabs/StashBox.git/': The requested URL returned error: 403
Thank you
Best

You don't have permissions to push your changes to the repo.  You should create a pull request.  This will allow me to review your changes and merge them.  See: https://help.github.com/articles/creating-a-pull-request/
It is common on an open source project to make your first contributions as pull requests, and at some point the maintainers trust you enough based on your PRs to add you as a maintainer.

I'm sorry, it's the first time I try to submmit code to a repo that is not mine. I just did the pull request. I hope all is OK.
Thank you

Hi Bob,
Did you have the opportunity to review the proposed changes?
Thank you
Best

Solved with merge https://github.com/SynchroLabs/StashBox/pull/2
Thank you Bob
Best


SSAO improvements

improve quality by using better samples
improve performance by 30%

Build error on msvc:
D:\a\filament\filament\android\filament-android\src\main\cpp\View.cpp(243,13): error C7560: 'power': designators must appear in member declaration order of class 'filament::View::AmbientOcclusionOptions' [D:\a\filament\filament\out\cmake-mt\java\filament\filament-jni.vcxproj]


1.1.0

Fixes #58 and #59

I will re-run tests before merging, since they are failing but it's not because the code itself but a transient problem.


Feature/project api implementation

Created Project and Image models
CRUD operations for both
Linking between Projects and Images (one-to-many relationship)
Querying by {id}
Defined routes for GET/POST/PUT/DELETE for both objects
Took care of cascade update/delete

@borzamircea I am closing this in favor of performing a pull request from the integration feature branch instead.


MySQL Live View

Describe the question you are looking to answer
Analyze the statistics of a service's MySQL connection.
Should include latency of requests over time
Example scenario
Service experiences SLO violations, can't diagnose problem by looking at services that are dependent on, want to look at database interactions. Turns out database request latencies are spiking, can then switch to another live view to see the rates of database interactions across all services in the cluster and can take action appropriately.
Desired output
Output Widgets

L
E
T
(Stretch) Top-K queries sent

Desired collaboration scheme

Do you want to share this view with your team (yes/no)? yes
Do you want to persist this view in your team's git repo (yes/no)? yes
Do you want to persist this view in pixie-community's git repo (yes/no)? yes

(Optional) Draft scripts
I think we can adapt the service_info view to do this as well.
###############################################################
# The following can be edited to k8s object and name to match.
###############################################################
# The Kubernetes object to filter on.
#    Options are ['pod', 'service']
k8s_object = 'pod'

# If you want to filter the object by name, enter the partial
# or full name here.
match_name = 'sock-shop'

# Names of output columns
src_name = 'client'
dest_name = 'server'
###############################################################
ip = 'remote_addr'
df = px.DataFrame(table='mysql_events', start_time='-60s')

df[k8s_object] = df.ctx[k8s_object]
df = df[df[k8s_object] != '' and px.contains(df[k8s_object], match_name)]
#px.display(df.head(100))
df = df.groupby([k8s_object, ip]).agg(count=(ip, px.count))

df[src_name] = df[k8s_object]
df[dest_name] = df[ip]
df['pod_id'] = px.ip_to_pod_id(df[ip])
df['requestor_pod'] = 	px.pod_id_to_pod_name(df.pod_id)
# Enable if you want pod name
df['requestor_service'] = 	px.pod_id_to_service_name(df.pod_id)
px.display(df[[src_name, dest_name, 'requestor_pod', 'count']])

┆Issue is synchronized with this Jira Task by Unito

This information is now available in Pixie's px/mysql_stats. This script calculates the latency, error rate, and throughput of a pod's MySQL requests.
@philkuz do you think it would be useful for me to add Top-k queries sent?

Closing this issue because we now scripts specific to database interactions which include LET metrics: mysql_stats, pgsql_stats, cql_stats


Is there any way to fix the ordering of our videos in youtube?

https://www.youtube.com/@bacalhauproject
These start with all the oldest - can we highlight special ones or something? cc @it09 @DeveloperAlly ?

Also - this - https://www.youtube.com/@bacalhauproject/playlists - doesn't include DeSci Berlin, @it09's talk, DeSci Boston, etc etc

Yes- but I need admin permission to do these sort of things - I'm currently only an owner.

@Laurahohmann ?

@aronchick @DeveloperAlly  I can add you as admin, please send me your email (As you're currently not listed at all, not even as a manager)

<EMAIL_ADDRESS>or<EMAIL_ADDRESS>

Invited you as an owner. (There's only owners and managers)


Elementum crashing on mi box s

Elementum is crashing every time , I open it on mi box s
Here is the kodi log:
https://paste.kodi.tv/loqoxayahe

@FKOV There is no error in the log. You can try to run manually elementum from ssh (when Kodi is running).
Run this binary file: /data/data/org.xbmc.kodi/files/plugin.video.elementum/bin/android_arm/elementum

I will try it tomorrow and let you know, thanks

also, could you please set log level in elementum to debug? at least for future logs.

Closing due to no response.


[SPARK-19912][SQL][NOT FOR MERGE] String literals should be escaped for Hive metastore partition pruning

What changes were proposed in this pull request?
This is not for merging.
This shows the failures of escaping string literals in order support #17266 .
Please see #17266 .

@gatorsmile . This is the PR for showing failure.

Test build #74422 has started for PR 17271 at commit ed6207e.

Test build #74422 has finished for PR 17271 at commit ed6207e.

This patch fails Spark unit tests.
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/74422/
Test FAILed.

This fails as expected.

Retest this please

Test build #74454 has started for PR 17271 at commit 3797680.

Test build #74443 has started for PR 17271 at commit 3797680.

Test build #74454 has finished for PR 17271 at commit 3797680.

This patch fails Spark unit tests.
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/74454/
Test FAILed.

Test build #74443 has finished for PR 17271 at commit 3797680.

This patch fails Spark unit tests.
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/74443/
Test FAILed.

I see. This is a bug of Hive. See the JIRA https://issues.apache.org/jira/browse/HIVE-2943


DRAFT: Fix for grouped convolutions

This PR:

only a test PR at the moment
fix for TF mobilenet grouped convolutions issue
tensorflow.python.framework.errors_impl.InvalidArgumentError: {{function_node __wrapped__Conv2DBackpropInput_device_/job:localhost/replica:0/task:0/device:CPU:0}} Gradients for grouped convolutions are not supported on CPU. Please file a feature request if you run into this issue. Computed input depth 576 doesn't match filter input depth 1 [Op:Conv2DBackpropInput]

The big disadvantage there would be, that we would need to retrain all models which uses mobilenet as backbone and the classification models itself
Tada: references / train-text-recognition (ubuntu-latest, 3.8, tensorflow) (pull_request)  is fixed 👍🏼
The export for TF mobilenet from #1182 still works with the changes from this PR
python3 /home/felix/Desktop/doctr/references/classification/train_tensorflow.py mobilenet_v3_small --pretrained

WARNING:tensorflow:Detecting that an object or model or tf.train.Checkpoint is being deleted with unrestored values. See the following logs for the specific values in question. To silence these warnings, use `status.expect_partial()`. See https://www.tensorflow.org/api_docs/python/tf/train/Checkpoint#restorefor details about the status object returned by the restore function.
WARNING:tensorflow:Detecting that an object or model or tf.train.Checkpoint is being deleted with unrestored values. See the following logs for the specific values in question. To silence these warnings, use `status.expect_partial()`. See https://www.tensorflow.org/api_docs/python/tf/train/Checkpoint#restorefor details about the status object returned by the restore function.
WARNING:tensorflow:Value in checkpoint could not be found in the restored object: (root).layer_with_weights-1.block.layer_with_weights-0.kernel
WARNING:tensorflow:Value in checkpoint could not be found in the restored object: (root).layer_with_weights-1.block.layer_with_weights-0.kernel
WARNING:tensorflow:Value in checkpoint could not be found in the restored object: (root).layer_with_weights-2.block.layer_with_weights-2.kernel
WARNING:tensorflow:Value in checkpoint could not be found in the restored object: (root).layer_with_weights-2.block.layer_with_weights-2.kernel
WARNING:tensorflow:Value in checkpoint could not be found in the restored object: (root).layer_with_weights-3.block.layer_with_weights-2.kernel
WARNING:tensorflow:Value in checkpoint could not be found in the restored object: (root).layer_with_weights-3.block.layer_with_weights-2.kernel
WARNING:tensorflow:Value in checkpoint could not be found in the restored object: (root).layer_with_weights-4.block.layer_with_weights-2.kernel
WARNING:tensorflow:Value in checkpoint could not be found in the restored object: (root).layer_with_weights-4.block.layer_with_weights-2.kernel
WARNING:tensorflow:Value in checkpoint could not be found in the restored object: (root).layer_with_weights-5.block.layer_with_weights-2.kernel
WARNING:tensorflow:Value in checkpoint could not be found in the restored object: (root).layer_with_weights-5.block.layer_with_weights-2.kernel
WARNING:tensorflow:Value in checkpoint could not be found in the restored object: (root).layer_with_weights-6.block.layer_with_weights-2.kernel
WARNING:tensorflow:Value in checkpoint could not be found in the restored object: (root).layer_with_weights-6.block.layer_with_weights-2.kernel
WARNING:tensorflow:Value in checkpoint could not be found in the restored object: (root).layer_with_weights-7.block.layer_with_weights-2.kernel
WARNING:tensorflow:Value in checkpoint could not be found in the restored object: (root).layer_with_weights-7.block.layer_with_weights-2.kernel
WARNING:tensorflow:Value in checkpoint could not be found in the restored object: (root).layer_with_weights-8.block.layer_with_weights-2.kernel
WARNING:tensorflow:Value in checkpoint could not be found in the restored object: (root).layer_with_weights-8.block.layer_with_weights-2.kernel
WARNING:tensorflow:Value in checkpoint could not be found in the restored object: (root).layer_with_weights-9.block.layer_with_weights-2.kernel
WARNING:tensorflow:Value in checkpoint could not be found in the restored object: (root).layer_with_weights-9.block.layer_with_weights-2.kernel
WARNING:tensorflow:Value in checkpoint could not be found in the restored object: (root).layer_with_weights-10.block.layer_with_weights-2.kernel
WARNING:tensorflow:Value in checkpoint could not be found in the restored object: (root).layer_with_weights-10.block.layer_with_weights-2.kernel
WARNING:tensorflow:Value in checkpoint could not be found in the restored object: (root).layer_with_weights-11.block.layer_with_weights-2.kernel
WARNING:tensorflow:Value in checkpoint could not be found in the restored object: (root).layer_with_weights-11.block.layer_with_weights-2.kernel

failing tests infact of missing checkpoints
@frgfm What do you think makes it sense in that case of retraining all the stuff to fix this issue ?
And the more related question @olivmindee @charlesmindee
Could you retrain the models ? Especially the rotation classification model and the detection/recogition ones depends on your datasets (computation power would not be a problem on my side but the data is 😅 )

ok SeperableConv2D and DepthwiseConv2D does not support different row/col values currently which would break the rectangular pooling implementations 😩
An integer or tuple/list of 2 integers, specifying the strides of the convolution along the height and width. Can be a single integer to specify the same value for all spatial dimensions. Current implementation only supports equal length strides in the row and column dimensions. Specifying any stride value != 1 is incompatible with specifying any dilation_rate value != 1.

Maybe we can keep this as Draft and check if the next TF release brings a fix

Thanks for the suggestion Felix! Yes as you mentioned, I think this is a risky move :sweat_smile:
I used to ping the TF team regularly to fix this, but it hasn't helped so far. It's a shame, you can always expect differences of support between frameworks but grouped convolutions ... the TF team seems to have too much stuff to handle at the moment so it's complicated to get a time estimate :/

Thanks for the suggestion Felix! Yes as you mentioned, I think this is a risky move sweat_smile I used to ping the TF team regularly to fix this, but it hasn't helped so far. It's a shame, you can always expect differences of support between frameworks but grouped convolutions ... the TF team seems to have too much stuff to handle at the moment so it's complicated to get a time estimate :/

Yeah lets keep this Draft and i will try to check after each TF release if it is maybe fixed 😅


[FEAT]: Improved Project Documentation

Improved Project Documentation
Description:
I am requesting more detailed documentation for the deno_rest project. The current documentation seems [brief], and I believe that providing more detailed information on

how to create new module
how to create new api
common api response structure for frontend
all api through postman,
more detailed code explanation
one click module creation, for example it will ask some information related to new module in form submit in vscode and all files related to that module will be created.
deeper WebSocket best practice example
api for multipart-form data (file upload)
pagination best practice example

would greatly benefit the community.
Thank you for your attention to this matter.

Sure, with your guidance I will try my best.

Sure, with your guidance I will try my best.

Moving this conversation under Discussions
https://github.com/Prolifode/deno_rest/discussions/61


Is it possible to set the aspect ratio for the crop box, but still allow it to be fully resizable?

I'm submitting a question/feature request

[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report
[x] Feature request
[X] Documentation issue or request

Current behavior
Calling setAspectRatio changes the crop box dimensions, and enforces the resizing event/action to always maintain the aspect ratio.
Expected (aka. Requested) behavior
How can I change the crop box to a specific aspect ratio, but still allow a user to resize the crop box in whatever ratio/dimensions they'd like?
Environment
Cropper version: 1.4.1

Try the initialAspectRatio option of Cropper.js v1.4+

Hey @onassar, did you figure out a way around this? The initialAspectRatio is only settable on the initial cropper constructor. I am replacing the image and need to reset the initial aspect ratio when the user selects a file. I need this to always be free ratio, but default to the full image size and then reset to the aspect ratio when the image changes. I would greatly prefer not having to create a new cropper instance every time.

I unfortunately didn't. Had to adjust application logic to fit this limitation :-(

Thanks for the quick response. I've been playing around with this and I can get it to work by destroying the initial instance and creating a new one. It's ugly but works. Had to clone initial options.
Strangely that results in this (for those who may stumble on it later):
reader.onload = function (e) {
    cropper.destroy();

    cropper = new Cropper(element, initialOptions);
    cropper.replace(e.target.result);
    $el.data('cropper', cropper);

    window.setTimeout(function () {
        onCropChanged(cropper, model, viewport);
    }, 50);
};

Interesting.. yeh that could be an approach, but I was worried about destroy and re-creating multiple instances. But def a cool approach!


Feedback for “Mirror”

It should be added, since it is not obvious, that during the operation you can select a face and that face will become the mirror plane.

Added a note. Thanks.


Bug Fix: Later js's weekday syntax is different from Quartz.

Unix Cron use 0-6 as Sun-Sat, but Quartz use 1-7. Due to later.js only supporting Unix Cron, we have to make this transition.
This change transform 1-7 to 0-6 in Cron's weekday field.
Tests are verified in our experimental cluster.

I tried "update branch" button in this page. Do I still need to rebase myself? Let me know if it is good to merge.

Oops. Looks like I have to rebase locally due to suvodeep's change.

Hey Liang
Sorry about that. I had to push because my change was huge.
However, I don't think that you'll face any issues with it since they were
just file renames.
Regards
Suvodeep
On Thu, Sep 15, 2016 at 4:54 PM, Liang Tang<EMAIL_ADDRESS>wrote:

Oops. Looks like I have to rebase locally due to suvodeep's change.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/azkaban/azkaban/pull/732#issuecomment-247486453, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AGN0AWUE0yYtqFMxPr6mxc10eBByKJHJks5qqdqsgaJpZM4J9IEk
.

Please resolve the conflict and feel free to merge.
In the future I would suggest keeping the unrelated changes separate e.g. the UI change seems to be independent from the syntax conversion.

Reopened this pull request in #738


Complete the followup for pull request #1006

GitHub Issues
#1006
#968

Coverage decreased (-0.01%) to 74.241% when pulling d4c96d96a8d2e353ab0618b62d8a1aed8abe7fe0 on zwing99:master into 57ec5ff14e3c9fbc54a19d56da61dbb2a59b71e4 on Miserlou:master.

@Miserlou anything i am missing here?


Do not overwrite CSV file, instead append to it

tasty-bench overwrites the CSV file every time, whereas the behavior of gauge is to append to it. The latter is more convenient as we can keep appending measurements to the same CSV file and then the tools processing the CSV file can select or compare the measurements.
With the appending behavior as default it is easy to get the overwriting behavior by just removing the existing file before performing measurements. Alternatively, a CLI option can be provided to select the appending vs overwrite behavior. Though its always better to have fewer CLI options so I would prefer the appending behavior by default and no CLI option.

gauge does not need to parse its CSV output, so it is easy to append them without much care. But tasty-bench with --baseline reads results from a previous run to report relative comparisons. It would be impossible to do so, if the baseline contains several runs appended.
I can assign a special meaning to --csv -, dumping csv to stdout, so that a caller is able to redirect it whenever desirable. (Other output can be suppressed by --quiet)

On the second thought, dumping CSV to stdout would interfere badly with --hide-successes and probably does not buy us much: one can dump CSV to a temporary file and append it to the main one later.

I have dealt with it in the measuring script for now, by writing to a temp file and then appending the results to another file. If it poses other problems, we can close the issue and move on.


R1.15 rocm61 albm merge hipblaslt

Merge hipblaslt and update docker to rocm6.3

For the background info.
We need to build the latest hipblas for multigpu usage https://github.com/ROCm/hipBLASLt/pull/1087
To build at almalinux : https://github.com/ROCm/hipBLASLt/pull/1109


About the Faces W-Encoder

I train a e4e model on my own dataset, because i want to replace the [Faces W-Encoder] with it. But i find the model does not match, I want to know how you got the [Faces W-Encoder]？ Thanks！！

Perhaps I should ask how to obtain an initial inversion into the W latent space.


Human-readable dates need to be able to handle future dates too

Right now, if you enter a future date for "date elicited", it'll display something annoying like "-21361107 seconds ago"!

we probably have this problem too :)

@jrwdunham i added a fix to our ago dates i might be able to do the same for dative, i will take a look for "ago" in the codebase

I changed it to
timeSince = (dateObject) ->
    dateObject = asDateObject dateObject
    if type(dateObject) in ['string', 'null'] then return dateObject
    try
      date = dateObject.getTime()
    catch
      return null
    if isNaN date then return ''
    seconds = Math.floor((new Date() - date) / 1000)
    if seconds < 0 then
      prefix = "in "
      suffix = ""
    else
      prefix = ""
      suffix = " ago"

    seconds = Math.abs(seconds)
    interval = Math.floor(seconds / 31536000)
    if interval > 1 then return prefix + "#{interval} years" + suffix
    interval = Math.floor(seconds / 2592000)
    if interval > 1 then return prefix + "#{interval} months" + suffix
    interval = Math.floor(seconds / 86400)
    if interval > 1 then return prefix + "#{interval} days" + suffix
    interval = Math.floor(seconds / 3600)
    if interval > 1 then return prefix + "#{interval} hours" + suffix
    interval = Math.floor(seconds / 60)
    if interval > 1 then return prefix + "#{interval} minutes" + suffix
    return prefix + "#{Math.floor(seconds)} seconds" + suffix

I tested parts of it and its working. but i cant get rid of this error
File .tmp/scripts/utils/tooltips.js.map created (source map).
>> .tmp/scripts/utils/utils.coffee:155:5: error: unexpected if
>>     if seconds < 0 then
>>     ^^
>> In file: .tmp/scripts/utils/utils.coffee
>> On line: 154
>>     if seconds < 0 then

@jrwdunham  I need some coffeescript help. Is something strange about the syntax of seconds = Math.floor((new Date() - date) / 1000) which is making an if unexpected but an assignment fine...
I will push my branch, you can take it and maybe fix it if you would prefer to work in code..

The error is because you're using then in f seconds < 0 then. The then is only necessary when it's a one-liner "if/then" statement. When the consequent is in an indent block, the then is a syntax error.
Also, prefix + "#{interval} minutes" + suffix (and similar) should be "#{prefix}#{interval} minutes#{suffix}".


consolidate docs into single html

Currently there are 4 html files in the docs:

index.html - displays D3 v3 examples from unpkg
index-d3v4.html - displays D3 v4 examples from unpkg
index-test.html - displays D3 v3 examples from locally built files
index-d3v4-test.html - displays D3 v4 examples from locally built files

The javascript files referenced by each html is the only difference between them.  Instead of have 4 different files, create one single file and programmatically add the javascript references based on the query parameters.  Create 2 query parameters:

test

if parameter set, use local file

./dist/index.js

if parameter unset, use unpkg file

https://unpkg.com/personality-sunburst-chart/lib/charts/index.js

d3

possible values v3 (default) or v4
use value of parameter to load D3 files and set the D3 version for the sunburst chart options

If the user changes the D3 version, the page should be reloaded with the correct parameters so that the files are loaded properly.  There is no need to allow the user to change from testing to non-testing.  The npm start script should also be updated to add the query parameter test instead of loading index-test.html

@germanattanasio I also added two extra profiles for testing (the ones with _2 appended to the names).  If you click the Version 2 or Version 3 button for personality insights multiple times, it will randomly pick one of the 2 profiles to display.
This functionality can be changed to programmatically randomizing the profile each time the version buttons are clicked.  This way you can remove the 2 extra files.


add  key binding test

that have been created by

Unfortunately a fix for the backspace binding was merged with #262 (https://github.com/crate/crash/commit/3f1a2d58d47f879c26411c04c02f4d0e96f6abd6#diff-d4dfc7040c003a69ec2ba1c35f086415)