CSV-284: Formalize PerformanceTest

There is a class org.apache.commons.csv.perf.PerformanceTest which is not documented and does not work without some hand-edits.
Re-enable this test, clean it up, and document it.

https://issues.apache.org/jira/browse/CSV-284

Coverage remained the same at 98.31% when pulling 77896a74c4dace7deabfb056b01cc0f0f9bdc858 on belugabehr:CSV-284 into a2ba9b52886162d7f046ff7fe8019c1ba5334a9a on apache:master.


[SPARK-9919] [MLlib] Implement hashCode for Dense and Sparse Matrices

Implement hashCode for Dense and Sparse Matrices

Merged build triggered.

Merged build started.

Test build #41129 has started for   PR 8278 at commit 67f0610.

@jkbradley @mengxr
I am facing some issues with hashing when isTransposed is not the same.
This is all right with the case of Dense Matrices because hashing the same 64 rowIndices, colIndices and values is not difficult (https://github.com/apache/spark/pull/8278/files#diff-3c938d5dd2a742291f0a17a87f4f9b58R291)
However while hashing SparseMatrices for equality, it is not that easy. For example, since we need to hash the same 64 rowindices, colIndices and values for the same SparseMatrices (both with isTransposed set True and False), the pseudocode when isTransposed set True goes like this.
val sm = new SparseMatrix(3, 2, Array(0, 1, 3, 5), Array(1, 0, 1, 0, 1), Array(3.0, 1.0, 4.0, 2.0, 5.0), true)
# Set colInd to zero.
# Search for all indices of colInd in rowIndices (which is (1, 3), when colInd is 0)
# For ptr in these Indices ((1, 3))
       # Scan colPtrs for the two numbers in between which ptr lies
       # The lowest of the two numbers gives the rowInd
       # Hash rowInd
       # Hash colInd
# Increment colInd

Only this would verify that the hash for the transposed and non-transposed case is equal.
Do you have any other suggestions?

@jkbradley @mengxr
Actually let me try a few more things. I'll ping you again when I think this is ready.

Test build #41129 has finished for   PR 8278 at commit 67f0610.

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

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

Merged build finished. Test PASSed.

I'll take a look now

Sorry, I was delayed by stuff...

Taking another look, my initial comment is that we should probably follow the example of Vector and define a default implementation with defined behavior, which can then be overridden by the subclasses.

I agree it'll be expensive to compute the hash for transposed matrices.  But correctness is the higher priority for now.

might not be able to work on this. feel free to cherry pick and complete.


LEDs in serial wont work

Hello,
I am trying to make a blink example of two LEDs wired in serial, but it does not work.

It makes no difference, if I insert a resistor or not.
Is there something I am missing, or does this not yet work?
Thanks for the amazing program, highly appreciate the work!
Nick

Hi Nick, thanks for reporting and for the feedback!
Yes, this does not yet work. The simulation core is digital, so it does not know how to handle this type of circuit.
What is your use case?

Thanks for the quick reaction!
I am actually developing a course for teaching uCs to kids.
Our project is to create a small electronic dice that they can use to 'roll' a dice and see the results on LEDs.
You can imagine it (simplified of cause) like this:

The original circuit is this:

It would have been nice to do serial to have it exactly like the schematic, but it works like shown above as well.
 Teaching uCs to kids without a simulation wouldn't be possible at all.
Cheers


Turing: Add additional events to telemetry API

button clicks and additional events
@compulim to talk with Dominic from Turing team.

They would like to differentiate a message sent from the user, is it due to an imBack card action, or the user type the message?
The imBack feature is "sending a message back". It is designed to "pretend the user type the message, with a convenience of clicking a button."
This is interesting. In order to make this work, someone need to put some in-band data into the activity. It's either production code in Web Chat, or customization code (depends on popularity and legitimacy of this feature).
But on the other side, as the service team want to clamp down the usage of channelData, we are encouraged not to use channelData for anything.
They are currently unblock by some customization code done on their side.

Should we make this a production feature?
If not going to production, should we build an official sample to educate more devs if they need to do something similar?

Moving to R11 as we gathered business requirement.

Punting due to time constraints.


Add NV scrape

We need a NV warn scraper with PDF scraping functionality similar to FL. See here for NV WARN data: https://detr.nv.gov/Page/WARN

Every PDF prior to 2021 is nicely parseable. 2021 is low-res image of a table, would need to be OCRed, I believe.


Fix pointing _keys and _values to the same object after clear

Array is not a primitive so assigning the same object of type 'Array' to _keys and _values is a bug. It causes 'set' to push both keys and values to the same array.

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

When could I expect for this fix to be merged and released?

@Irmiz As soon as it passes the tests on CI :-)


Add SENSU_LOADED_TEMPFILE_DIR to sensu.default.erb

Sensu stores sensu_server_loaded_files in /tmp directory by default and it causes silent failures of handlers. An environmental variable SENSU_LOADED_TEMPFILE_DIR was added to Sensu to provide a way to change the directory in order to fix the problem.
Below is the related topics.
https://github.com/sensu/sensu-settings/issues/21
https://github.com/sensu/sensu-settings/pull/20
For example, adding the following line to /etc/default/sensu will fix the problem in Ubuntu 14.04.
export SENSU_LOADED_TEMPFILE_DIR=/var/run/sensu
My environment:
Ubuntu 14.04 LTS
Sensu 0.25.3-1
Chef 12.7.2-1

Closed by #487


359-session-data-comes-back-encoded

What kind of change does this PR introduce?
Fixes the unicode problem by converting the string to percent-encoding before being decoded
Note: the escape function is obsolete, which is why i went with this fix
What is the current behavior?
The atob function cannot decode chars outside the latin1 range
What is the new behavior?
characters outside the latin1 range can be decoded
fixes #359

@supastore thanks so much for looking into this. Could you outline the sources you used to come to this solution? It works, but I'd love to understand the background. Thanks :)

Ah, I think I found it: https://stackoverflow.com/a/30106551/17622044

Great, thanks.


Secure Practice MailRisk data connector publisherId

Required items, please complete
Change(s):

Updated publisherId

Reason for Change(s):

Updated publisherId to match publisherId in partner center

Testing Completed:

Yes

Checked that the validations are passing and have addressed any issues that are present:

Yes

Hi @ingin97, please discard this change from the PR Tools/Create-Azure-Sentinel-Solution/V2/input/Solution_SecurePracticeMailRisk.json.
Add this file from this path \Solutions\MailRisk\Data\Solution_SecurePracticeMailRisk.json

@v-rbajaj Is it okay now?


Recommendations

adding recommendation modal on front page. Has commented out code for similar applications in quickview
@mleeBoeing @blynch11p

If changes need to be made to this code feel free to checkout the branch and make them

Also whoever does the second :+1:, please merge the branch

:+1:

:+1:


Update grub command line with inst.stage2 label CDROM

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[x] The toolchain has been rebuilt successfully (or no changes were made to it)
[x] The toolchain/worker package manifests are up-to-date
[x] Any updated packages successfully build (or no packages were changed)
[x] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/tools/cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[x] All source files have up-to-date hashes in the *.signatures.json files
[x] sudo make go-tidy-all and sudo make go-test-coverage pass
[ ] Documentation has been updated to match any changes to the build system
[x] Ready to merge

Summary

Update grub command line with inst.stage2 label CDROM. This is required to boot when ISO laydown to USB and boot on Ampere, Cavium SoCs
Change Log

Change
Change
Change

Does this affect the toolchain?

YES
NO
Associated issues

#xxxx

Links to CVEs

https://nvd.nist.gov/...

Test Methodology

Built ISO for Arm64 and X64. Verified Arm64 when laydown on USB successfully booted on Arm64 Ampere SoC, HyperV VM and X64 VM.

I am not sure why it is only working when I use this flag "inst.stage2". I am using rufus tool to laydown ISO to USB and not sure if rufus is doing something here. will look into it further,

The use of mariner.media=CDROM is a workaround that our installer does not require, but is observed to be needed to boot on some hardware/SoCs.


*: add watch with client keepalive test

Address https://github.com/coreos/etcd/issues/8022.

lgtm


Can you specify a license?

Can you specify a license MIT/GPL/LGPL/Research-only etc.
Thanks,

Thanks for pointing out the missing license.


ansis:SoilSite - update purpose def

update the definition - purpose of the site is why it was established

"@id": "ansis:purpose",
"range@type": "ansis:Classification",
"title": "purpose",
"description": "Purpose of or reason for activity.",
"description": "Purpose of or reason for a site or activity. For example, why a Land Survey or Soil Site was established; why a profile was created; or why a site was visited.",
"type": "object",
"$ref": "./entities.json#Classification"

ontology -
anais:purpose definition can be updated to as per schema -
from -
"description": "Purpose of or reason for activity.",
to-
"description": "Purpose of or reason for a site or activity. For example, why a Land Survey or Soil Site was established; why a profile was created; or why a site was visited.",


[Kotlin] Table null needs to define at least one column

I get the following error while building my app (it is written in Kotlin). But when I delete the build folder in Android Studio before building everything is fine.
When I build the next time, I get the error again.
:myapp:compileReleaseJavaWithJavac
Error:*==========*
  Table null needs to define at least one column
  *==========*
Error:*==========*
  Table null needs to define at least one primary key
  *==========*
Error:*==========*
  Table null needs to define at least one column
  *==========*
Error:*==========*
  Table null needs to define at least one primary key
  *==========*

Warning:The following options were not recognized by any processor: '[kapt.kotlin.generated]'
Error:Execution failed for task ':myapp:compileReleaseJavaWithJavac'.
> Compilation failed; see the compiler error output for details.```

Same issue here. Haven't found a workaround yet besides using java for the database classes

this is a Kotlin issue, that apparently is not solved. (http://blog.jetbrains.com/kotlin/2015/06/better-annotation-processing-supporting-stubs-in-kapt/). I tested this and found writing it in Kotlin appears to work in the IDE, but when you reference those generated java files from DBFlow in Kotlin, the Kotlin compiler cannot find the generated classes.

Is it one of these Kotlin issues causing the problems? KT-10352, KT-8733, KT-7277
Some of them are already closed, although the issues still persist. Doesn't seem they are actively working on that at the moment.  The model classes looked so clean when defined in Kotlin. Back to verbose Java.

@ubuntudroid no this is a different kind of issue (https://youtrack.jetbrains.com/issue/KT-11978). closing this b/c this is not dbflow issue.


Add support for YouTube embedding in markdown

The JamStack blog post is one of the places we need that

You can see an embedded youtube clip in this blog post: https://freesewing.org/blog/freesewing-goes-jamstack#this-is-a-starting-point
On the new beta site, this doesn't work: https://beta.freesewing.org/en/blog/freesewing-goes-jamstack
For this to work, we need to add a Gatsby plugin to embed YouTube clips in MarkDown, this one for example: https://www.gatsbyjs.org/packages/gatsby-remark-embed-youtube/

Closing as we switched to MDX


(sysinternals) Latest version failing testing

Seems like the latest version of sysinternals is failing the automated testing,
I'm not in a position to check out why right now, so I'm opening this for tracking purposes.
https://gist.github.com/choco-bot/bb69df515725513a33ca862f261ee03e

@pascalberger thanks, just noticed it failed. didn't look for the reason.


Retrieving the log file's name provided in a Configuration file

I'm trying to retrieve the log file's name provided in a Configuration file. For example, if you have the following configuration file:
* GLOBAL:
   FORMAT               =  "%datetime %msg"
   FILENAME             =  "/tmp/logs/my.log"
   ENABLED              =  true
   TO_FILE              =  true
   TO_STANDARD_OUTPUT   =  true
   MILLISECONDS_WIDTH   =  6
   PERFORMANCE_TRACKING =  true
   MAX_LOG_FILE_SIZE    =  2097152 ## 2MB - Comment starts with two hashes (##)
   LOG_FLUSH_THRESHOLD  =  100 ## Flush after every 100 logs
* DEBUG:
   FORMAT               = "%datetime{%d/%M} %func %msg"

and you want to retrieve the FILENAME field, i.e. "/tmp/logs/my.log". I managed to do that as the following:
el::Logger* logger = el::Loggers::getLogger("default");
std::string log_filename = logger->typedConfigurations()->filename( el::Level::Global );
LOG(DEBUG)<<"filename is "<<log_filename;

Is this approach correct?
As it was a bit hard to find out how to do that I was thinking, can I add a small sample demonstrating that?

This is part of documentation under https://github.com/easylogging/easyloggingpp/blob/master/README.md#reading-configurations but yes i think having sample for it is also good but make sure you add this sample under develop branch and push it to develop branch.
I will be doing another release 9.81 soon


Pretrained GoogLeNet without batch norm

🚀 Feature
Looking at the commit history for torchvision.models.googlenet, I noticed that the squashed commit messages for the initial PR #678 mention a plain and batch norm version of GoogLeNet. Currently, there is only one googlenet defined in that module's __all__. Is there a reason we can't offer both googlenet and googlenet_bn like before?
Motivation
I am trying to use PyTorch to replicate a custom Caffe model which used GoogLeNet as one of its components. In both the Caffe and ONNX implementation GoogLeNet, batch norm is not used. In order to have the most direct comparison, I would like to have access to a PyTorch model which also does not use batch norm.
Pitch
Use the commit history of the linked PR to supply two GoogLeNet implementations - one with batch norm and one without
Alternatives
As mentioned in the forums, I could use nn.modules.Linear.Identity to delete all batch norm layers from the current implementation. However, this would render the pre-trained weights ineffective and therefore require many more epochs to get an optimized backbone. With a smaller dataset, I am also not confident that the quality of the backbone would ever reach that of the pre-trained ImageNet model.
Additional context
The official ONNX GoogLeNet can be downloaded from here. Since I am using, PyTorch 1.4.0, I chose the ONNX opset 6. Following your conversion docs, I wrote the following script to save PyTorch's GoogLeNet as an ONNX file
from numpy.testing import assert_allclose
import onnx
import onnxruntime
import torch
from torchvision.models import googlenet
import torch.onnx

def to_numpy(tensor):
    return tensor.detach().cpu().numpy() if tensor.requires_grad else tensor.cpu().numpy()

# Trace inputs through the PyTorch model to export
model = googlenet(pretrained=True)
model.eval()
dummy = (torch.randn(1, 3, 224, 224), )
torch_outs = model(*dummy)
path = 'googlenet-pytorch.onnx'
torch.onnx.export(
    model=model,
    args=dummy,
    f=path,
    do_constant_folding=True,
    input_names=['images'],
    output_names=['scores'],
    dynamic_axes={
        'images': {0: 'batch_size'},
        'scores': {0: 'batch_size'}})

# Load/check the ONNX model and run through Python API
onnx_model = onnx.load(path)
onnx.checker.check_model(onnx_model)
ort_session = onnxruntime.InferenceSession(path)
ort_inputs = {inp.name: to_numpy(dummy[i]) for i, inp in enumerate(ort_session.get_inputs())}
ort_outs = ort_session.run(None, ort_inputs)

# Confirm outputs match
assert_allclose(to_numpy(torch_outs), ort_outs[0], rtol=1e-03, atol=1e-05)

Loading both implementations into Netron for visualization, you can clearly see how the first inception block differs in regard to its use of batch norm
PyTorch

ONNX

@addisonklinke thanks for FR.
@TheCodez could you please shed some light why googlenet_bn was not retained ?

The original model I send was completely based off the paper and used the same initialization that inception_v3 uses, which is the same the tensorflow implementation uses.
The tensorflow implementation uses batch norm. I'm guessing the original googlenet doesnt work with this initialization without batchnorm.
After those failed training attemps I added the Xavier initialization as that worked for the BVLC Googlenet. Sadly this version was never trained by @fmassa, as we decided to use the pretrained weights from tensorflow instead, which network structure is a bit different from what the paper describes https://github.com/pytorch/vision/issues/906

@TheCodez So if I understand correctly, your initial work did fully implement the architecture for a non-batch norm GoogeLeNet, but we never had pre-trained weights for it? Since the weights are from Tensorflow, we had to use their batch norm based architecture

@addisonklinke Correct. Given that training Googlenet from scratch is known for being hard, using the Tensorflow weights and architecture was the easier choice.

@vfdev-5 Since training is the issue, perhaps we could port pre-trained weights for the non-batch norm architecture from the official ONNX model I linked?

@TheCodez could you please add a link on the tensorflow implementation ?
@addisonklinke there are also the weights from BAIR caffe : https://github.com/BVLC/caffe/tree/master/models/bvlc_googlenet which can be also ported. This one is without BN and has conv kernel=5 for branch3 (#906).
I also wonder how would we call this "original" googlenet here keeping current implementation with its name for backward compatibility.

@vfdev-5 https://github.com/tensorflow/models/blob/master/research/slim/nets/inception_v1.py
That's a good question, ideally the current googlenet should be called inception_v1 but this makes backwards compatibility hard.

I think it worth to update the docs of https://github.com/pytorch/vision/blob/195c41ee7ae3fff8f64a4fda67632259dd4f7454/torchvision/models/googlenet.py#L26 saying that implementation and weights are from tf-slim.
Maybe, "original" model can be called as bvlc_googlenet and implementation's doc should also contain a link on caffe repository and bothbvlc_googlenet and googlenet should note the difference between each other.
@addisonklinke would you be interested in implementing bvlc_googlenet and porting caffe weights to pytorch ?
cc @fmassa for visibility and other thoughts

@vfdev-5 I'm on a rather time-sensitive project, so will likely pursue some alternatives like using a different torchvision.models class for the backbone rather than diving down the Caffe to PyTorch rabbit hole. If that doesn't pan out, then I may have more time and effort to invest into porting from another framework


We are seeing the action fail again in some situations when the first commit reference doesn't exist (due to the ^ reference).

Was reverting from ... back to ^.. directly related to the fix for #64? It's hard to tell because there were lots of changes in the regression commit and I wasn't involved in that one.

@lawndoc oh shoot, my bad - do not why restored it.


Which version of cuda?

When I type python webcam_inference.py I get
2020-12-29 19:08:07.664029: W tensorflow/stream_executor/platform/default/dso_loader.cc:60] Could not load dynamic library 'libcudart.so.11.0'; dlerror: libcudart.so.11.0: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /root/distro/install/lib:/usr/local/cuda-10.0/lib64/:/root/distro/install/lib
2020-12-29 19:08:07.664062: I tensorflow/stream_executor/cuda/cudart_stub.cc:29] Ignore above cudart dlerror if you do not have a GPU set up on your machine.
2020-12-29 19:08:09.472208: I tensorflow/compiler/jit/xla_cpu_device.cc:41] Not creating XLA devices, tf_xla_enable_xla_devices not set
2020-12-29 19:08:09.472893: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library libcuda.so.1
2020-12-29 19:08:09.507898: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:941] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-12-29 19:08:09.508338: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1720] Found device 0 with properties:
pciBusID: 0000:01:00.0 name: GeForce GTX 1050 computeCapability: 6.1
coreClock: 1.493GHz coreCount: 5 deviceMemorySize: 1.95GiB deviceMemoryBandwidth: 104.43GiB/s
2020-12-29 19:08:09.508442: W tensorflow/stream_executor/platform/default/dso_loader.cc:60] Could not load dynamic library 'libcudart.so.11.0'; dlerror: libcudart.so.11.0: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /root/distro/install/lib:/usr/local/cuda-10.0/lib64/:/root/distro/install/lib
2020-12-29 19:08:09.508514: W tensorflow/stream_executor/platform/default/dso_loader.cc:60] Could not load dynamic library 'libcublas.so.11'; dlerror: libcublas.so.11: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /root/distro/install/lib:/usr/local/cuda-10.0/lib64/:/root/distro/install/lib
2020-12-29 19:08:09.508578: W tensorflow/stream_executor/platform/default/dso_loader.cc:60] Could not load dynamic library 'libcublasLt.so.11'; dlerror: libcublasLt.so.11: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /root/distro/install/lib:/usr/local/cuda-10.0/lib64/:/root/distro/install/lib
2020-12-29 19:08:09.508649: W tensorflow/stream_executor/platform/default/dso_loader.cc:60] Could not load dynamic library 'libcufft.so.10'; dlerror: libcufft.so.10: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /root/distro/install/lib:/usr/local/cuda-10.0/lib64/:/root/distro/install/lib
2020-12-29 19:08:09.508724: W tensorflow/stream_executor/platform/default/dso_loader.cc:60] Could not load dynamic library 'libcurand.so.10'; dlerror: libcurand.so.10: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /root/distro/install/lib:/usr/local/cuda-10.0/lib64/:/root/distro/install/lib
2020-12-29 19:08:09.508805: W tensorflow/stream_executor/platform/default/dso_loader.cc:60] Could not load dynamic library 'libcusolver.so.10'; dlerror: libcusolver.so.10: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /root/distro/install/lib:/usr/local/cuda-10.0/lib64/:/root/distro/install/lib
2020-12-29 19:08:09.508883: W tensorflow/stream_executor/platform/default/dso_loader.cc:60] Could not load dynamic library 'libcusparse.so.11'; dlerror: libcusparse.so.11: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /root/distro/install/lib:/usr/local/cuda-10.0/lib64/:/root/distro/install/lib
2020-12-29 19:08:09.509066: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library libcudnn.so.8
2020-12-29 19:08:09.509080: W tensorflow/core/common_runtime/gpu/gpu_device.cc:1757] Cannot dlopen some GPU libraries. Please make sure the missing libraries mentioned above are installed properly if you would like to use GPU. Follow the guide at https://www.tensorflow.org/install/gpu for how to download and setup the required libraries for your platform.
Skipping registering GPU devices...
Traceback (most recent call last):
File "webcam_inference.py", line 49, in
tf.config.experimental.set_memory_growth(physical_devices[0], True)
IndexError: list index out of range
My cuda is release 10.0, V10.0.130
Which CUDA and CUDNN I should use?

Hiello,
Thanks for the message,
The code has been tested under Tensorflow 2.3. As recommended by Tensorflow we're using cuDNN 7.6 and CUDA 10.1 for this version. Additionally looking at your error, it seems that the inference code assumes one GPU is available. I just pushed a new commit https://github.com/Visual-Behavior/detr-tensorflow/commit/363bd73c27db83eb6788ec322bcc464f46324746 to fix this issue. However, without the GPU, inference time is very slow.
Best,
Thibault


Update use-blocklist.md

combined into https://github.com/MicrosoftDocs/azure-docs-pr/pull/250606
#please-close


Fix time units

What does this PR do?
Fixes #50
Before submitting

[x] Was this discussed/approved via a Github issue? (no need for typos, doc improvements)

[ ] N/A

[ ] Did you write any new necessary tests?

[x] N/A

[x] Did you make sure to update the docs?

[ ] N/A

[ ] Did you update the changelog?

[x] N/A

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (86f0cdc) 90.23% compared to head (3b969ed) 90.23%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #51   +/-   ##
=======================================
  Coverage   90.23%   90.23%
=======================================
  Files          27       27
  Lines        2038     2038
=======================================
  Hits         1839     1839
  Misses        199      199

Impacted Files
Coverage Δ

hta/analyzers/breakdown_analysis.py
98.83% <100.00%> (ø)

hta/analyzers/cuda_kernel_analysis.py
90.90% <100.00%> (ø)

tests/test_trace_analysis.py
100.00% <100.00%> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


GitHub Workflows security hardening

...redirected from https://github.com/WeblateOrg/weblate/pull/8064
This PR adds explicit permissions section to workflows. This is a security best practice because by default workflows run with extended set of permissions (except from on: pull_request from external forks). By specifying any permission explicitly all others are set to none. By using the principle of least privilege the damage a compromised workflow can do (because of an injection or compromised third party tool or action) is restricted.
It is recommended to have most strict permissions on the top level and grant write permissions on job level case by case.

@nijel

Merged, thanks for your contribution!


[Feature]: Select the SKU version of the system image

Is your proposal related to a problem?
no
Describe the solution you'd like.
How can I choose my own SKU version? I would like to keep the installation fully automated, but I would like to select the system version manually, e.g. Windows Server 2025 Datacenter
Describe alternatives you've considered.
Fully manual installation, or download the specified version of the image via uupdump.net
Additional context
No response

@kroese It's so cool. This program is like a treasure. It has a lot of undiscovered features. If you have the time, document these unlisted features, I have a feeling there are many more fun features waiting for me to experience.


feat: add etag for objectMeta

Which issue does this PR close?

Closes #2240
Rationale for this change
Add Etag for objectMeta as a unique identifier, which is a hash value by last_modified.

What changes are included in this PR?

Are there any user-facing changes?

For the actual stores, like S3, the etag is part of the response. Using a manual hash is just for in memory and filesystem where this data isn't stored

The emulator tests appear to be failing for this?

The emulator tests appear to be failing for this?

---- http::tests::http_test stdout ----
thread 'http::tests::http_test' panicked at 'called `Result::unwrap()` on an `Err` value: Generic { store: "HTTP", source: InvalidPropFind { source: Custom("missing field `getetag`") } }', object_store/src/lib.rs:1203:62

getetag is meant to work, but it failed.
[13.6](https://datatracker.ietf.org/doc/html/rfc2518#section-13.6) getetag Property

   Name:       getetag
   Namespace:  DAV:
   Purpose:    Contains the ETag header returned by a GET without
   accept headers.
   Description: The getetag property MUST be defined on any DAV
   compliant resource that returns the Etag header.
   Value:      entity-tag  ; defined in [section 3.11 of [RFC2068]](https://datatracker.ietf.org/doc/html/rfc2068#section-3.11)

   <!ELEMENT getetag (#PCDATA) >

It may not be supported, we may need to make the ETag optional 🤔


[QA] Testing Session | Avatars and backpacks stop loading with many participants

Explorer Alpha build versions:

v0.21.0

Issue Description:
During a testing session with around 25 participants, avatars stopped loading in the backpack and other users' avatars were missing when opening their passports. Initially, avatars and passports loaded correctly, but as more participants joined, the issue worsened, leading to more missing avatars and backpacks.
STR:

Launch the explorer and join a session with around 25 participants.
Observe the avatars in the backpack and other users' passports.
Note if avatars stop loading in the backpack and if other users' avatars are missing in the passports as the number of participants increases.

Expected behavior:
Avatars should load correctly in the backpack and passports consistently, even with a high number of participants, without needing to restart the application.
Current behavior:

https://github.com/user-attachments/assets/27463899-b729-4ad7-88e0-3b529bbf3395
Notes

The problem worsens as more participants join the session.
The issue is mitigated after relaunching the client.

Hoping lazy loading improves the situation.

This issue is potentially fix on this pr: https://github.com/decentraland/unity-explorer/pull/2399
We will wait until the next group testing session to re-validate it.


Update cards on homepage

This PR adds 2 two new cards to the homepage of the Unipept website (one for UMGAP and one for the desktop app) and reorders the other cards.
@ninewise If you have some spare time, could you provide me with a suitable image for the UMGAP that we can use on the homepage?
Current screenshots:

I would reverse the order of the tiles on the top row.

Quickly made a logo for the UMGAP; The top left and top middle are the unipept colours (but I don't really like them close together). I prefer the bottem left or top right for light/dark environments. Maybe @bmesuere can give some advice.
I couldn't upload an SVG image here, but I could commit one if you tell me where to put it.

@ninewise Oooh, I like them! Especially the top right one is one of my favourites! If (after approval from @bmesuere) you would like to add the images, you can add them to the "assets/images" folder (name it something like "home_feature_umgap.svg"). Thanks!

I prefer the ones using white as one of the two colors.


Is there any way to make a bulleted list?

It seems as though the typical markdown for a bulleted list will isntead turn into a 3 column table.

Unfortunately at this time I think you'll have to do it with custom css.


UnReferenced irrelevant section causing htmlhelp to fail compiling

Expected Behavior
HTML Help should compile.
There should be no warnings or errors thrown from content that is not referenced by the root document.
Actual Behavior
Help CHM does not compile correctly. Internal html files are missing.
Possible Solution
Topic reader should ignore unreferenced sections.
Steps to Reproduce
testFiles.zip

Use test content provided in zip
Correct paths for your setup
Run makeCHM.bat

Copy of the error message, log file or stack trace
log.log in attached zip.
[topic-reader] file:/C:/Users/admin/Desktop/Output/test/ReferencedTopic.dita:14:59: [DOTJ036W][WARN] The file "file:/C:/Users/admin/Desktop/Output/Irrelevant.dita" is outside the scope of the input dita/map directory.
Environment

DITA-OT version: 3.1.1
Operating system and version: Win10x64
How did you run DITA-OT? makeCHM.bat included
Transformation type: htmlhelp

Notes
ReferencedTopic.dita contains a section "Irrelevant section", which is not referenced by the publishing map. Content in this irrelevant section should not affect anything in this publishing scenario.
The specific problem in this case seems to be that the irrelevant section contains an out of scope link to another dita file (../Irrelevant.dita). If you remove the preceding "../", then the compile still throws an error saying Irrelevant.dita' does not exist, but the CHM is produced correctly.
The use case for this is that the repository does not have file access compatible with the DOTK, so for publishing we are exporting the relevant content to a temporary file system location in order to run the DOTK, however, we do not want to export the entire repository; Only the content necessary for publishing as referenced by the selected map!

The processing shouldn't fail in this case but the markup as provided is very bad practice. All resources used by topics should be reflected in the map, so the ReferencedTopic.dita, which is the conref target of the section in MainTopic.dita should be referenced from the map using a resource-only topicref. Likewise, having an xref to a topic that is not used from the map is a broken link and you should have no expectation that the link will work. In PDF or similar monolithic output the link cannot work (because the referenced topic would never be part of the publication.
With DITA 1.2, if you use a key reference, you can have the link redirect to the appropriate target in the context of this map, which might be the topic as published on a Web site or a different topic that is in the map.
With DITA 1.3 you could make the link be a cross-publication link if that's what it is.

The link is not relevant, it is irrelevant content in the "Irrelevant section" ;-)
None of the content of the "Irrelevant section" in ReferencedTopic.dita is used in this map. So the fact that the link is broken should be irrelevant. Only section ReferencedTopic.dita#rts2 is used from MainTopic.dita, which is used by the map. Also, it's not the fact that the link is broken that makes the parser fail, it's only if the link is out of scope. You can leave the broken link in and it will compile the html help just fine, if the broken link is in-scope (i.e. remove the preceding ../ in the path).
map.ditamap -  topicref href="MainTopic.dita"
MainTopic.dita - section conref="ReferencedTopic.dita#rts2"
The "Irrelevant section" ReferencedTopic.dita#rts1 is never used and any errors in it's content should be ignored by the parser as they are irrelevant to the output.

Ah, the link is in a section that is not used from MainTopic.dita.
But there is still the problem that because ReferencedTopic.dita is not explicitly marked as resource-only by a topicref from the map the processor has no way to know for sure what its relevance is or isn't. It is referenced by a conref from MainTopic.dita, which makes the entire topic relevant. The processor may not know at the time it evaluates the links in that topic that it is (as far as the map author is concerned) a resource-only topic because it was never told that. The fact that the section with the link in it is in fact never used is not really something the processor would necessarily know at that point, or it may never bother to know it (that is, the processor is under no obligation to track which things that contain links are or are not actually reflected in the final result so that it can either avoid doing things that would ultimately not need doing (such as checking the resolvability of a link that is never actually rendered) or provide better diagnostic messages.
But note that if you turned on the "only topics in map" flag, the conref would not be resolvable at all.
So again, the failure is bad but your content reflects bad practice that cannot produce reliable results (or desired results) with all transform types. It would be worth the experiment to see if referencing ReferencedTopic.dita with a resource-only topicref changes the failure behavior.

I'm guessing this comes down to the fact that DITA-OT is very greedy when evaluating links -- it will follow every link it finds, without regard to whether it needs the topic or not. This was a design choice back in DITA-OT -- I don't like it, but it's proven nearly impossible to change without re-architecting pretty much everything. So ... this is an issue that has existed since DITA-OT 1.0. The "only topics in map" flag was a band-aid that tried to make things better, but it is really intended to help control what generates output (it does not control whether links are followed).
With map-first processing introduced in 3.0 it should now be possible to do better, but we're still not there.
Basically if you have a reference to ReferencedTopic.dita, and that file has a link to another local topic, DITA-OT is going to try and follow that link. It doesn't matter if you get there by evaluating @conref in a topic, or by a normal <topicref>, or by a resource-only <topicref>.

I can't say for sure -- but HTML Help has long had problems with topics that are outside of the scope of the map directory. Because HTML Help is a zip, where the project files need to be at the root directory, files that end up outside of that directory are missing; moving the zip up to get everything results in a broken CHM.
I haven't had time to test with these specific files, so I can't say what is causing the compile to fail, but that's my first guess -- that simply trying to follow the link throws off directory detection, and results in compilation running in the wrong spot.

outer.control=quiet gets rid of the warning, but it still fails with the same HHC5003: Error: Compilation failed while compiling MainTopic.html. .. so I'm not really sure where the issue is.

That makes sense -- setting outer.control=quiet doesn't actually change any of the processing, all it does is avoid generating the error message. So, nothing about the actual process that led to the failure will change with that. The issue is going to come down to a build variable somewhere getting thrown off between (temp directory) or (directory inside of temp that holds the map) -- those values do not match when there is a ../ link outside of the map dir, and that's where this sort of processing has historically had problems.


[Issue-2033] Update chainlist (w43)

Related issue(s)

#2033

Is your feature request related to a problem(s)? Please describe.

Update chain-list

Describe the solution you'd like

Update chain-list

Describe alternatives you've considered

No

Additional context

No

🚀 Deployed on https://pr-2070--sw-web-runner.netlify.app


[SPARK-22068][CORE]Reduce the duplicate code between putIteratorAsValues and putIteratorAsBytes

What changes were proposed in this pull request?
The code logic between MemoryStore.putIteratorAsValues and Memory.putIteratorAsBytes are almost same, so we should reduce the duplicate code between them.
How was this patch tested?
Existing UT.

Hi @cloud-fan @jiangxb1987 , would you mind take a look ? Thanks a lot.

Can one of the admins verify this patch?

ok to test.

Test build #81961 has started for PR 19285 at commit d2b8ccd.

Test build #81961 has finished for PR 19285 at commit d2b8ccd.

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

Test build #81971 has started for PR 19285 at commit 9ea8f49.

Test build #81971 has finished for PR 19285 at commit 9ea8f49.

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

Merged build finished. Test FAILed.

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

retest this please

Test build #81981 has started for PR 19285 at commit 9ea8f49.

Test build #81981 has finished for PR 19285 at commit 9ea8f49.

This patch fails RAT 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/81981/
Test FAILed.

Test build #82084 has started for PR 19285 at commit d0fcf4f.

@ConeyLiu Could you rebase this with the latest master so we can continue review it? Thanks!

It's updated. Thanks a lot.

Test build #83527 has started for PR 19285 at commit 3a90ad1.

Test build #83527 has finished for PR 19285 at commit 3a90ad1.

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

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

Merged build finished. Test PASSed.

Test build #83575 has finished for PR 19285 at commit bc3ad4e.

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

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

Merged build finished. Test PASSed.

Can one of the admins verify this patch?


Executors use the default service account instead of the Driver's service account

Currently, if I want executors to pull using a secret, the secret needs to be part of the default service account under the namespace of the Spark application even if we specify serviceAccount on the DriverSpec.
Either the executors should use the Driver's serviceAccount (DriverSpec) or there should be a way of specifying it on the ExecutorSpec.
Any thoughts?
Thanks

Have you tried spec.imagePullSecrets? Secrets specified here get applied to both the driver and executor pods.

Yes, that's how I set it. When I do this, my driver starts but the
executers block on trying to pull the image.
On Fri, Aug 30, 2019 at 12:28 PM Yinan Li<EMAIL_ADDRESS>wrote:

Have you tried spec.imagePullSecrets? Secrets specified here get applied
to both the driver and executor pods.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/spark-on-k8s-operator/issues/596?email_source=notifications&email_token=AA3OUNH3DXUEXJXNSOA4IPLQHFDDJA5CNFSM4ISO22M2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5SEKZY#issuecomment-526665063,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA3OUNAG2LS6F5DZGKDDFY3QHFDDJANCNFSM4ISO22MQ
.

That's interesting. Have you checked the executor pods to see if they do have the image pull secrets?

Can't check this now, but as soon as I add the secret to default in the
app's namespace the pull works.
On Fri, Aug 30, 2019 at 3:01 PM Yinan Li<EMAIL_ADDRESS>wrote:

That's interesting. Have you checked the executor pods to see if they do
have the image pull secrets?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/spark-on-k8s-operator/issues/596?email_source=notifications&email_token=AA3OUNHHD3V3PDUGELM3WZLQHFVBDA5CNFSM4ISO22M2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5SQA4I#issuecomment-526712945,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA3OUNANRW32TY666LUB7ETQHFVBDANCNFSM4ISO22MQ
.

When you get a chance, can you also paste here an example of how you set imagePullSecrets?

I can't seem to reproduce it now even if I remove the imagePullSecret from
the default sa under spark-operator. I tried publishing a new image and the
executors can still pull it. We're running a bare metal cluster which
wasn't allowed to pull until I added the secret to the default sa for the
executors.
Any idea why it works now as I've seen it consistently behave the way I
described (all sa under the spark-operator namespace have an empty
imagePullSecrets)?
Here's the yaml for my sample app which only worked when I added mysecret
to default sa under spark-operator:
apiVersion: sparkoperator.k8s.io/v1beta1
kind: SparkApplication
metadata:
  name: myapp
  namespace: spark-operator
spec:
  type: Scala
  mode: cluster
  image: myreg.com/myns/myimage:v0.0.1
  imagePullPolicy: Always
  imagePullSecrets:
  - mysecret
  mainClass: myclass
  mainApplicationFile: local:///my.jar
  sparkVersion: 2.4.0
  driver:
    cores: 1
    memory: 10m
    labels:
      version: 2.4.0
    serviceAccount: spark-operator-spark
  executor:
    cores: 1
    instances: 3
    memory: 10m
    labels:
      version: 2.4.0

On Fri, Aug 30, 2019 at 3:10 PM Yinan Li<EMAIL_ADDRESS>wrote:

When you get a chance, can you also paste here an example of how you set
imagePullSecrets?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/spark-on-k8s-operator/issues/596?email_source=notifications&email_token=AA3OUNHBVV7R7NHZXH4E7ELQHFWCNA5CNFSM4ISO22M2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5SQV4Y#issuecomment-526715635,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA3OUNCWFZEEGCDCU27VBXDQHFWCNANCNFSM4ISO22MQ
.

This is fixed here: https://issues.apache.org/jira/browse/SPARK-27872 coming with Spark 3.0.0.

@doctapp @liyinan926 This is fixed here: https://issues.apache.org/jira/browse/SPARK-27872 coming with Spark 3.0.0.

Thanks @skonto I knew I wasn't crazy ;)
Now, if this currently works on our cluster, does this imply a potential security issue with Kubernetes? I find it odd that the nodes weren't able to pull the image without the secret, once added to default the pull succeeded, and now can pull anything even if the secret is absent in default. I tested with a different images and the pull still works.
Any thoughts on this?
Thanks

Somehow the pull secrets are cached?

Stale and closing.


Host name and password.

Question 1
Hello, I am writing to know why it is necessary to change the host to> http: // hassio / homeasistant?
How could I change it?
Currently my host name is hassio.local: 8123
and I am working correctly with homekit and the iOS application.
Will you have a problem in the future?
My Config.json is default... only changed       "default_visibility"  to "visible".
Question 2
In the password they recommend leaving it empty, but I would like to insert a password ... there I have to place the same password that I have put in the configuration file.yaml?
http:
   # Secrets are defined in the file secrets.yaml
   #api_password: ??????
The password here is MD5 encrypt?????
  "platform": "HomeAssistant",
  "name": "HomeAssistant",
  "host": "http://hassio/homeassistant",
  "password": "0aeb6d9f99924f3fbd12998ac03d76c5",

thanks

Hi @fradducho,
Answer to question 1:
The URL given (http://hassio/homeassistant) is an internal Hass.io network. Which is far more efficient to use then connecting to your own URL's (which would go via outside the host).
Answer to question 2:
With the URL advised in answer 1, the connection would go over a private connected network. There is not ouside access, so the need for password is less important. Secondly, the add-on will replace the password with an Hass.io API token, which is an MD5 based token (NOT A PASSWORD!).
Hopefully this answers your questions.
Please be sure to use the forums for questions next time 😉


experiment/canvas-set-pins-2

Canvas Experiment
How to try it?
Select a pinned element and move it to the edge of the containingblock to change a style.left to style.right.  The timer is set to 1.5 seconds. You should see the containing block highlighted before the property change happens.
Please keep width/height in style props, it's a bit more limited than the inspector pin controls.
🍕 https://utopia.pizza/p/2d4da2ed-sleepy-reminder/?branch_name=experiment/canvas-set-pins-2

interesting! I feel like this feature is very hidden though. in Framer the snapping to right AFAIK happens if you cross to the right half of the frame, instead of only when you are snapping to the very right edge of the parent,.

that sounds very aggressive to me, that's why I added only the edge

I agree with both of you -  It's basically undiscoverable, but it's also delightfully unaggressive :) As an experiment I feel it needs more thought, but I think the pattern we have here is a winner, because it's still faster than the inspector, and doesn't require you to context switch.
If we put a bit of thought into how to design it (eg make it more discoverable, and give you a sense of what will happen after the delay) we can use it in a bunch of situations.


DFS of Graph

❗Write Code in Only One Language ( CPP , C , Java , Python )
Give File Name :  DFS of Graph  ✅ In respective language folder
📑 DESCRIPTION
DFS of Graph

You are given a connected undirected graph. Perform a Depth First Traversal of the graph.
Note: Use the recursive approach to find the DFS traversal of the graph starting from the 0th vertex from left to right according to the graph.
Example 1:
Input: V = 5 , adj = [[2,3,1] , [0], [0,4], [0], [2]]
Output: 0 2 4 3 1
Explanation:
0 is connected to 2, 3, 1.
1 is connected to 0.
2 is connected to 0 and 4.
3 is connected to 0.
4 is connected to 2.
so starting from 0, it will go to 2 then 4,
and then 3 and 1.
Thus dfs will be 0 2 4 3 1.

Follow-up: Can you come up with an algorithm that is less than O(n^2) time complexity?

@Harsh76200 assign this to me for java

@Harsh76200 Can you assign this to me for python?

@Shreya111111 solve

I can do it for C++. Can you please assign it to me?


[feat] sqrt() implementation

sqrt(x) returns the square root of largeInt x

Hi. This is about making a function or a method from a class?

Hi. This is about making a function or a method from a class?

it will be a method on the class largeInt

ok. Then I would like to take this one

okay @dabc312GitHub, go ahead


Backend inconsistency with torch.load

🐛 Bug
sox and soundfile backends use the parameter normalization, whereas sox_io uses normalize.
Code defaulting on one and falling back on another therefore will throw errors due to a wrong keyword argument.
Similarly, sox_io uses frame_offset instead of offset.
To Reproduce
Steps to reproduce the behavior:

torchaudio.set_audio_backend("sox_io")
torchaudio.load(..., normalization=False)

Expected behavior
Consistent keyword naming across backends.
Environment
torchaudio 0.6.0
Additional context
I saw some typos in the new documentation and may make a pull request for the lot if I get the time soon.
While I actually prefer normalize, normalization is probably preferred for backwards compatibility.

I also found that sox_io does not currently support pathlib Paths for loading, where other backends do.

And I reread the warning note and find that it does advise the changed signature :)
I would still suggest accepting pathlib Paths for the filepath argument.

I'll have a look on Sunday, and will probably submit a PR then (I don't think it should be a lot of work?)!

@timlod

I'll have a look on Sunday, and will probably submit a PR then (I don't think it should be a lot of work?)!

Thanks.
Yes, I think it's just changing the type annotation and adding path=str(path) to convert it to string internally before passing it to C++ code.


Error on Meteor

Hi,
I added a screencastify plugins for Meteor (https://github.com/groblochon/meteor-screencastify) but an error occured on startup :
Uncaught TypeError: Cannot read property 'registerModule' of undefined
(anonymous function) @ castify-api.js:3

Have you some idea ?

Hi,
I just digged into it. castify-api.js currently bundles traceur-runtime, which assumes to be loaded globally (it writes to window.System and assumes that this can be accessed globally as System). Meteor, however auto-scopes global variables, effectively breaking traceur-runtime.
Loading castify-api.js in the outer scope like this should work:
var loader = document.createElement('script');
loader.src = "https://www.screencastify.com/libs-api/castify-api.js";
document.getElementsByTagName('head')[0].appendChild(loader);

I'll try to provide castify-api.js alternatively as an ES6 module in the future.

Closing, please re-open If you should still experience this issue.


EmitterService - Issue Sending Payload

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[X] Bug report
[ ] Performance issue
[ ] Feature request
[ ] Documentation issue or request
[ ] Support request => https://github.com/ngxs/store/blob/master/CONTRIBUTING.md
[ ] Other... Please describe:

Current behavior
I'm not able to send payloads in the actions of the EmitterService.
Expected behavior
Send payloads to the actions of the EmitterService without getting any problem
Minimal reproduction of the problem with instructions
https://stackblitz.com/edit/ngxs-labs-emitter-aetrwj?file=src%2Fapp%2Fapp.component.ts
here... inline 25 of app.component.ts you will see the error.
"Argument of type 'number' is not assignable to parameter of type 'void'."
the changes in the emitter service in the last commit are causing this bug.
What is the motivation/use case for changing the behaviour?
Fix a bug currently in version 2.0.0 (It use to work in 1.9.0)
Environment

Libs:
- @angular/core version: ~9.1.6
- @ngxs/store version: ^3.6.2
- @ngxs-labs/emitter: ^2.0.0

Browser:
- [x] Chrome (desktop) version XX
- [ ] Chrome (Android) version XX
- [ ] Chrome (iOS) version XX
- [ ] Firefox version XX
- [ ] Safari (desktop) version XX
- [ ] Safari (iOS) version XX
- [ ] IE version XX
- [ ] Edge version XX

For Tooling issues:
- Node version: v12.16.3
- Platform:   Mac

Others:

I also created a PR solving it... but I'm not able to push the changes...
So I'll show you where is the problem...
https://github.com/ngxs-labs/emitter/commit/912e5f1817e71f974b4f0e427f746f661ebca6e3
just search for my comments... thank you!

Nevermind. A colleague realized that to be able to send payloads in version 2.0.0 is required to specify the generic type in the action and in this way is able to send whatever payload you need.
`
export interface IChangeLoading {
isLoading: boolean;
}
// in your method using the service...
this.emitterService.action(AppState.changeLoadingState).emit({ isLoading: false });
`
Issue closed.


x/tools/gopls: missing unimported package candidates

Repro (assuming completeUnimported=true is configured):

Create a new module with contents:

package main

func main() {
}

Complete rand.. You should see suggestions from crypto/rand and math/rand.
Select rand.Int from "crypto/rand"
Delete line and save ("crypto/rand" import should disappear).
Wait a second to make sure "crypto/rand" package handle gets GC'd 😉
Complete rand. again. Now you only see untyped completions from "crypto/rand". I expected to see typed completions from "crypto/rand" and completions from "math/rand".

See https://youtu.be/VaOs1crVI4g
/cc @stamblerre @heschik

When I look at logs, I see a ton of "no file for" errors, which are coming from findFileInPackage in pkg.go. Will look tomorrow.

...I looked a little now. The problem is that findFileInPackage can't find anything for math/rand/rand.go because it's not in the package's dependencies any more. Annoying, since I had the package to start with when I called packageMembers.


BUG: very irritating in ionic, cordova while building app in Linux.

1- when in run " ionic info" i get:
Cordova CLI: Not installed
Gulp version:  CLI version 3.9.0
Gulp local:
Ionic CLI Version: 1.7.8
Ionic App Lib Version: 0.6.4
OS: Distributor ID:	Ubuntu Description:	Ubuntu 14.10
Node Version: v0.12.7

Dependency warning - for the CLI to run correctly,
it is highly suggested to install/upgrade the following:
Please install your Cordova CLI to version  >=4.2.0 npm install -g cordova

2- when i run: sudo cordova --version i get:
5.4.0
everthing is working fine but when in run the command :
sudo cordova build android or sudo ionic build android
i get :
Running command: /user/hello/platforms/android/cordova/build
[Error: Failed to find 'ANDROID_HOME' environment variable. Try setting setting it manually.
Failed to find 'android' command in your 'PATH'. Try update your 'PATH' to include path to valid SDK directory.]
ERROR building one of the platforms: Error: /user/hello/platforms/android/cordova/build: Command failed with exit code 2
You may not have the required environment or OS to build this project
Error: /user/hello/platforms/android/cordova/build: Command failed with exit code 2
This is highly irritating. i have setted all the android paths and its working as well.
for example when i run:
adb it works and when i run android it works.
But im unable to build the app anyhow.
Pls helpout.

You have several issues here.

ionic can't find cordova
This is because your user doesn't have the correct privileges to run cordova. IE, you should not have to run "sudo cordova". See this:
https://issues.apache.org/jira/browse/CB-10062

The ANDROID_HOME env var is not set for sudo.
sudo invokes a different user - root. You will not have access to the same env vars as sudo that you do when you work as your own user. If you correct the issue above, you should no longer have to run ionic using sudo at all (since it can now find cordova).

Yes, i resolved this issue earlier, i run sudo su then other commands like build, run etc and they work, Can you tell me after loading my app on play store, how will i send updates, patches etc to users ? I'm new to mob dev.

That doesn't have anything to do with your original question, and I recommend you post to a more general site like Stack Overflow. That being said I'm confident you can figure it out yourself:
[iOS deployment] (http://lmgtfy.com/?q=how+to+deploy+ios+updates)
[Android deployment] (http://lmgtfy.com/?q=how+to+deploy+android+updates)

As a comment to future viewers, it's not a good idea to solve the original problem by simply using "sudo" or "su".  Those commands should be used for one-off things that actually SHOULD require additional privileges, not your daily work.


[SharpDX.Direct2D1 v2.6.3] CustomEffectBase PropertyBinding: performance issue due to internal exceptions

I'm using SharpDX 2.6.3 to develop a WinForms based application the uses Direct2D (DX11_1 target). While all inbuilt Effects are working well, the example "RippleEffect" (and my own shader effects) causes huge performance drops when Properties are accessed (see https://github.com/sharpdx/SharpDX-Samples/blob/master/WindowsAppStore81/D2DCustomPixelShaderEffect/EffectRenderer.cs#L80-L97).
To illustrate the effect see following screenshots: when accessing properties in each Render call, the performance drops to ~9 fps, while when not accessing properties the result is at ~500 fps.
Calling SetValue(...):

Not calling SetValue(...):

A performance profile shows internal NullReferenceExceptions when accessing NativeGetInt (no idea why, all my properties are float and similar).

A downgrade to 2.6.2 didn't change the behavior.
Is there a fix (or workaround) for this issue? It's currently a show stopper for finishing the migration of a DX9 application to DX11/D2D1.

I have investigated this issue more in detail:
It only happens when running the application inside VS debugger (vshost). If you are running the program standalone it works correctly! Even attaching debugger later works then.
So the problem seems to be only related to VS debugging environment, but a fix would be very appreciated though.

This could be a bug in the binding process. Unfortunately, I don't have enough spare time currently to dig into this. Let me know if you can investigate this by compiling the source tagged 2.6.3 and debug from there.

I was finally able to build a custom sharpdx version and debugged this problem.
Inside Utilities.Write the destination pointer is 0. This happens for every call, so I think that probably none of my variables read the shader.
Unfortunately the stacktrace has one "external code", so I can't follow where the dataPointer gets 0.

The step before the [external code] and the local varibles:

My calling code:

Can you check this problem? If you can give me a hint, I could try some code modifications.

FYI:
adding a sanity check for null pointers does solve my problems:
        public static void Write<T>(IntPtr destination, ref T data) where T : struct
        {
            unsafe
            {
                // Sanity check for null pointers. TODO: Calling code should be checked!
                if (destination == IntPtr.Zero)
                    return;
                Interop.CopyInline((void*)destination, ref data);
            }
        }

Every 2nd call to this methods fails in my case. Could this be an indication for another issue in calling code?
This check also improves performance, as costly NREs are avoided now.

@xoofx do you think that the above fix is enough? I would supply a pull request then.
But if this is only an indicator for a deeper issue, it would be good if you could check it more in detail.


GeoTag 5.1 (MAS) on Sonoma 14.1 The operation couldn't be completed. (GeoTag.Exiftool.ExiftoolError error O.)

System Details
macOS 14.1
GeoTag 5.1 (MAS) and Downloaded
M2 MacBookPro 14"
Steps to Reproduce

Load Photos into App
Set GPS Location
Save File

Expected Result

File to be saved with geolocation added to metadata (EXIF)

Actual Result
Receive Error: The operation couldn't be completed.
(GeoTag.Exiftool.ExiftoolError error O.)

Thanks for the report.  A few questions:

Are you using a backup folder?
Are all files/folders on your internal drive?
What kind of image file are you trying to update?

Hi,
I’ve tried it with and without a backup folder. These are jpeg photos that I exported out of the photos app into my downloads folder which were originally taken with a DSLR

Thanks.  I thought there might be something special about the downloads folder, but I tried editing a jpeg file there and it worked OK.  I also exported an image from photos into downloads and was able to save changes without issue.
Can you please attach a screen shot of the sheet displaying the error message?

Hi,
Here is a screenshot of the error

/I also have that issue. Not the first time with a downloaded jpeg images, but after doing some editing, like changing the size with preview the GPS coordinates disappear and I have to insert this again. After clicking Save I get exactly the same error.  (Bash and Zhl shell) on my minimac 2023 with M2 with Sonoma 14.3
  Also this copy/paste issue with only via content menu possible is not resolved.

Exiftool is telling me there is no error (that's what error 0 means) yet the save fails.  My first guess is a permissions error.  I can force the issue by trying to edit a file that has its permissions set to read only.  Example, if I try to edit an image with these permissions

I'll see the error.  Still looking for other causes.

I tried the permission change to r/w also, but that didn't help.
Here is a timeline of what I did:
downloaded a jpeg to downloads folder. Geotag has full disk access. Checked
the exiftool metadata:
ExifTool Version Number         : 12.70
File Name                       : skybox-2.jpg
Directory                       : /Users/gideon/Downloads
File Size                       : 11 MB
File Modification Date/Time     : 2023:12:27 14:17:40+01:00
File Access Date/Time           : 2023:12:27 14:18:28+01:00
File Inode Change Date/Time     : 2023:12:27 14:18:26+01:00
File Permissions                : -rw-r--r--
File Type                       : JPEG
File Type Extension             : jpg
MIME Type                       : image/jpeg
JFIF Version                    : 1.01
Resolution Unit                 : None
X Resolution                    : 1
Y Resolution                    : 1
Image Width                     : 8192
Image Height                    : 4096
Encoding Process                : Baseline DCT, Huffman coding
Bits Per Sample                 : 8
Color Components                : 3
Y Cb Cr Sub Sampling            : YCbCr4:2:0 (2 2)
Image Size                      : 8192x4096
Megapixels                      : 33.6
Then opened the photo in GeoTag, added the GPS coordinates and could
problemless save the jpeg-file.
After that I had to add some other metadata with a script.
Results:
ExifTool Version Number         : 12.70
File Name                       : skybox-2.jpg
Directory                       : /Users/gideon/Downloads
File Size                       : 11 MB
File Modification Date/Time     : 2023:12:27 14:33:40+01:00
File Access Date/Time           : 2023:12:27 14:33:44+01:00
File Inode Change Date/Time     : 2023:12:27 14:33:42+01:00
File Permissions                : -rw-r--r--
File Type                       : JPEG
File Type Extension             : jpg
MIME Type                       : image/jpeg
JFIF Version                    : 1.01
Exif Byte Order                 : Big-endian (Motorola, MM)
X Resolution                    : 1
Y Resolution                    : 1
Resolution Unit                 : None
Y Cb Cr Positioning             : Centered
GPS Version ID                  : <IP_ADDRESS>
GPS Latitude Ref                : North
GPS Longitude Ref               : East
XMP Toolkit                     : Image::ExifTool 12.70
Cropped Area Left Pixels        : 0
Cropped Area Top Pixels         : 0
Projection Type                 : equirectangular
Use Panorama Viewer             : True
Image Width                     : 8192
Image Height                    : 4096
Encoding Process                : Baseline DCT, Huffman coding
Bits Per Sample                 : 8
Color Components                : 3
Y Cb Cr Sub Sampling            : YCbCr4:2:0 (2 2)
Image Size                      : 8192x4096
Megapixels                      : 33.6
GPS Latitude                    : 53 deg 10' 34.56" N
GPS Longitude                   : 5 deg 25' 30.75" E
GPS Position                    : 53 deg 10' 34.56" N, 5 deg 25' 30.75" E
Then I changed the size with Preview to 4096x2048
That destroyed all the changes in the metadata and gave a warning
ExifTool Version Number         : 12.70
File Name                       : skybox-2.jpg
Directory                       : /Users/gideon/Downloads
File Size                       : 3.3 MB
File Modification Date/Time     : 2023:12:27 14:37:14+01:00
File Access Date/Time           : 2023:12:27 14:37:16+01:00
File Inode Change Date/Time     : 2023:12:27 14:37:14+01:00
File Permissions                : -rw-r--r--
File Type                       : JPEG
File Type Extension             : jpg
MIME Type                       : image/jpeg
Exif Byte Order                 : Big-endian (Motorola, MM)
Orientation                     : Horizontal (normal)
X Resolution                    : 72
Y Resolution                    : 72
Resolution Unit                 : inches
Y Cb Cr Positioning             : Centered
Warning                         : Bad ExifIFD directory
Image Width                     : 4096
Image Height                    : 2048
Encoding Process                : Baseline DCT, Huffman coding
Bits Per Sample                 : 8
Color Components                : 3
Y Cb Cr Sub Sampling            : YCbCr4:2:0 (2 2)
Image Size                      : 4096x2048
Megapixels                      : 8.4
Then I tried Geotag again. Then the error in saving occurred.
ExifTool Version Number         : 12.70
File Name                       : skybox-2.jpg
Directory                       : /Users/gideon/Downloads
File Size                       : 3.3 MB
File Modification Date/Time     : 2023:12:27 14:41:33+01:00
File Access Date/Time           : 2023:12:27 14:41:35+01:00
File Inode Change Date/Time     : 2023:12:27 14:41:33+01:00
File Permissions                : -rw-r--r--
File Type                       : JPEG
File Type Extension             : jpg
MIME Type                       : image/jpeg
Exif Byte Order                 : Big-endian (Motorola, MM)
Orientation                     : Horizontal (normal)
X Resolution                    : 72
Y Resolution                    : 72
Resolution Unit                 : inches
Y Cb Cr Positioning             : Centered
Warning                         : Bad ExifIFD directory
XMP Toolkit                     : Image::ExifTool 12.70
Cropped Area Left Pixels        : 0
Cropped Area Top Pixels         : 0
Projection Type                 : equirectangular
Use Panorama Viewer             : True
Image Width                     : 4096
Image Height                    : 2048
Encoding Process                : Baseline DCT, Huffman coding
Bits Per Sample                 : 8
Color Components                : 3
Y Cb Cr Sub Sampling            : YCbCr4:2:0 (2 2)
Image Size                      : 4096x2048
Megapixels                      : 8.4
GeoTag still couldn't write the Geocordinates.
Ciao,
Gideon
~~o
<#,
()/()
Op di 26 dec 2023 om 19:05 schreef Marco S Hyman @.***>

Exiftool is telling me there is no error (that's what error 0 means) yet
the save fails. My first guess is a permissions error. I can force the
issue by trying to edit a file that has its permissions set to read only.
Example, if I try to edit an image with these permissions
Screenshot.2023-12-26.at.10.02.58.AM.png (view on web)
https://github.com/marchyman/GeoTag/assets/784627/61940a85-e63e-4e71-b4e9-3ef431ffb7b9
I'll see the error. Still looking for other causes.
—
Reply to this email directly, view it on GitHub
https://github.com/marchyman/GeoTag/issues/40#issuecomment-1869695719,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA57SROJOGGYHO22MZ4JU33YLMGWJAVCNFSM6AAAAAA6YK3N4GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNRZGY4TKNZRHE
.
You are receiving this because you commented.Message ID:
@.***>

[image: Mailtrack]
https://mailtrack.io?utm_source=gmail&utm_medium=signature&utm_campaign=signaturevirality11&
Sender
notified by
Mailtrack
https://mailtrack.io?utm_source=gmail&utm_medium=signature&utm_campaign=signaturevirality11&
27-12-23
14:42:32

Then I changed the size with Preview to 4096x2048

Exactly how did you do that?  I'd like to do the same for testing.

Opened it with Preview, went to tools > resize and filled in the with
number. Set the resolution to 72 pix/inch.
Ciao,
Gideon
~~o
<#,
()/()
Op wo 27 dec 2023 om 19:53 schreef Marco S Hyman @.***>

Then I changed the size with Preview to 4096x2048
Exactly how did you do that? I'd like to do the same for testing.
—
Reply to this email directly, view it on GitHub
https://github.com/marchyman/GeoTag/issues/40#issuecomment-1870550258,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA57SRMDNQN3XDBPOGKOL63YLRVA3AVCNFSM6AAAAAA6YK3N4GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZQGU2TAMRVHA
.
You are receiving this because you commented.Message ID:
@.***>

Thank you.  I'll give that a try.

Hmmm... I did not lose metadata after modifying a jpg image with Preview.  No warnings.  No error saving the file.  The GeoTag error does seem to be because of this:
Warning                         : Bad ExifIFD directory
Exiftool will not update the file until that error is cleared.  I don't know why exiftool reports a 0 status code when this occurs.  See https://exiftool.org/faq.html#Q20 which discusses rebuilding corrupted EXIF.
Also, I'm curious about this:
Exif Byte Order                 : Big-endian (Motorola, MM)
It wasn't in the exiftool output prior to you adding additional metadata.  I don't know if it has anything to do with anything, but do find it curious.

I used this picture. It's a 360° photo without the proper metadata.
https://storage.polycam.io/captures/F53D9E49-534A-4E01-B4BE-445C4DCE21D7/skybox.jpg
This the script I used to add them (and obviously GeoTag to add the
geocoordinates.)
exiftool -ProjectionType="equirectangular" -UsePanoramaViewer="True"
-"PoseHeadingDegrees<$exif:GPSImgDirection"
-"CroppedAreaImageWidthPixels<$ImageWidth"
-"CroppedAreaImageHeightPixels<$ImageHeight"
-"FullPanoWidthPixels<$ImageWidth" -"FullPanoHeightPixels<$ImageHeight"
-CroppedAreaLeftPixels="0" -CroppedAreaTopPixels="0" -overwrite_original
"/Users/gideon/Downloads/skybox.jpg"
Ciao,
Gideon
~~o
<#,
()/()
Op do 28 dec 2023 om 00:26 schreef Marco S Hyman @.***>

Hmmm... I did not lose metadata after modifying a jpg image with Preview.
No warnings. No error saving the file. The GeoTag error does seem to be
because of this:
Warning : Bad ExifIFD directory
Exiftool will not update the file until that error is cleared. I don't
know why exiftool reports a 0 status code when this occurs. See
https://exiftool.org/faq.html#Q20 which discusses rebuilding corrupted
EXIF.
Also, I'm curious about this:
Exif Byte Order : Big-endian (Motorola, MM)
It wasn't in the exiftool output prior to you adding additional metadata.
I don't know if it has anything to do with anything, but do find it curious.
—
Reply to this email directly, view it on GitHub
https://github.com/marchyman/GeoTag/issues/40#issuecomment-1870689394,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA57SRI4W6RAO7QRT3N4RJTYLSVDLAVCNFSM6AAAAAA6YK3N4GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZQGY4DSMZZGQ
.
You are receiving this because you commented.Message ID:
@.***>


fix zappr.yaml

The indentation of collaborators: true was wrong.
However I removed it completely, since our compliance rules state that we need approvals from at least two Zalando employees.

:+1:

Since zappr config is currently broken in master branch, we need to disable the required status check and just merge the fix.

Thanks


"when you have two labels in the same place, apparently whichever label that lldb chooses is unpredictable (but consistent)"

Derived from https://github.com/tsoding/bm/pull/423#issuecomment-855263317

Eg:
a:
b:
    nop
    nop


Avoid fetching timestamp fast field when there is no sorting on this field nor filtering

As an addendum to this, the system should also only load fast fields when absolutely needed. I believe currently they are fetched in a warmer process before the data selection occurs but on large scale queries, where many splits will produce no results, that will result in fetching a large amount of unneeded data and kill the effectiveness of the fast field cache.
The same also applies if a split only returns a single result, even if the query asks for sorting, a single result is already inherently sorted and fast fields aren't needed for that split unless required by other operations.

I believe currently they are fetched in a warmer process before the data selection occurs but on large scale queries

True. When #1745 will be merged, we will prefetch the timestamp fast field only if we need to filter values. NB: we are already pruning splits that have no overlap between the requested time range and the split time range, in this case, no prefetch is done.

The same also applies if a split only returns a single result, even if the query asks for sorting, a single result is already inherently sorted and fast fields aren't needed for that split unless required by other operations.

This may be hard to do as we don't know in advanced the result. We plan to work on fast fields codecs as we know to be more efficient (for sparse fast fields for example), looking at Parquet format can be interesting as we can apply filtering on blocks and thus reduce the number of bytes to fetch. This is still in discussion and we don't know yet when we will work on that.

Partly closed by #1749
Issue #1748 is tracking another optimization that we can make.
Closing.


Rhel 8.4 Docker 2.10.7 cluster | Cluster agent fail to update

Rancher Server Setup

Rancher version: 2.6.2 (upgraded to 2.6.3-rc4 later)
Installation option (Docker install/Helm Chart):

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc): RKE1

Proxy/Cert Details: self-signed

Information about the Cluster

Kubernetes version: v1.21.6-rancher1-2
Cluster Type (Local/Downstream): Rke1

If downstream, what type of cluster? (Custom/Imported or specify provider for Hosted/Infrastructure Provider):

rhel-8.4-docker-20.10.7-selinux-on-rpm - ami-0f3d6c27d7ba5006e (3 workers, 3 etcd, 1 control plane)

Describe the bug
Cattle-cluster-agent gets stuck updating, with a status "Deployment does not have minimum availability"
To Reproduce

Have a 2.6.2 HA setup (3 nodes)
As a standard user provision an aws cluster with the following template.

Ami: ami-0f3d6c27d7ba5006e
disk size: 40 GB
SSH user: ec2-user
docker install url: None

upgrade rancher to 2.6.3-rc4 using a helm command.

helm upgrade rancher rancher-latest/rancher --version 2.6.3-rc4 --namespace cattle-system --set hostname=server --set ingress.tls.source=secret --set bootstrapPassword=admin

Select All Namespaces in the Namespace selector
navigate to your rhel cluster --> workloads, then scroll to "Namespace: cattle-system"

Result
"cattle-cluster-agent" is stuck in "updating" status.
The pods have the following errors.
Containers with unready status: [cluster-register]
Viewing the logs of the pods you will see:
level=fatal msg="Certificate chain is not complete, please check if all needed intermediate certificates are included in the server certificate (in the correct order) and if the cacerts setting in Rancher either contains the correct CA certificate (in the case of using self signed certificates) or is empty (in the case of using a certificate signed by a recognized CA). Certificate information is displayed above. error: Get \"https:/\": x509: certificate signed by unknown authority"
Expected Result
After upgrading all resources should become active
Screenshots
Deployments

Failure

Pods

Pods recent Events

Additional context

@Auston-Ivison-Suse Can you share the full log and/or share the setup?

@superseb Looks like I had used self signed and my helm command was trying to run --set ingress.tls.source=secret as part of the command was the issue. This is a non issue.

Rancher Validation Setup

Rancher version: 2.6.2 (upgraded to 2.6.3-rc4 later)
Installation option (Docker install/Helm Chart):

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc): RKE1

Proxy/Cert Details: self-signed

Information about the Cluster

Kubernetes version: v1.21.6-rancher1-2
Cluster Type (Local/Downstream): Rke1

If downstream, what type of cluster? (Custom/Imported or specify provider for Hosted/Infrastructure Provider):

rhel-8.4-docker-20.10.7-selinux-on-rpm - ami-0f3d6c27d7ba5006e (3 workers, 3 etcd, 1 control plane)

Describe the bug
Cattle-cluster-agent gets stuck updating, with a status "Deployment does not have minimum availability"
Validation steps

Have a 2.6.2 HA setup (3 nodes)
As a standard user provision an aws cluster with the following template.

Ami: ami-0f3d6c27d7ba5006e
disk size: 40 GB
SSH user: ec2-user
docker install url: None

upgrade rancher to 2.6.3-rc4 using a helm command.

helm upgrade rancher rancher-latest/rancher --version 2.6.3-rc4 --namespace cattle-system --set hostname=server  --set bootstrapPassword=admin

Select All Namespaces in the Namespace selector
navigate to your rhel cluster --> workloads, then scroll to "Namespace: cattle-system"

Result
Cluster provisions correctly, I simply entered the wrong command for the upgrade.


rework tojupyter_download_nb_urlpath to be a global setting for the w…

…riter

Closing as not the right way to go


Fix saving and loading document

List of fixed issues (if they exist):

Drag and dropping document was not loading config options
Loading tractography, and connectomes
Saving color map labels

@cdrake , there are a few console.log lines that look like they were used for debugging. Can you convert these to use the niivue logger instead if they aren't critical logs?

Console.logs added should be removed or converted to log.error


SCEditor bug with Move down/up buttons or Property Add

Hi,
I'm using the SCEditor plugin and when I click on Move up / down or Property Add buttons the texts in SCEditor disappear.
If I switch the SCEditor to source code, I can see the sources. But the WYSIWYG text is blank.
Best,
JB

@jbmolle, have you fixed it?

Root cause is SCEditor uses iframe and json-editor moves field DOM subtrees during object property change to accommodate property ordering. Content inside iframe is lost during the move due to iframe sandbox scoping constraint.
A workaround is switching to summernote with my working fork based on pr #440, which contains some bugs perhaps due to relying on an older version of summernote api.
Related discussion on so


Move os_family mapping to yaml file

No need to specify empty os_family grain to apply defaults.
Add Gentoo support

@mgomersbach thanks for this.


It says started but nothing happens

Could you try without sudo ?

Have the same issue, with the nRF-App for Android and with the Raspberry Pico W. Nothing on a iPhone 7 and Mini 12

Have the same issue, with the nRF-App for Android and with the Raspberry Pico W. Nothing on a iPhone 7 and Mini 12

Same Here

Nethunter:

https://github.com/ECTO-1A/AppleJuice/assets/54718375/ac356869-fd6e-4faa-a0b3-d9c21417819d

Pico W

https://github.com/ECTO-1A/AppleJuice/assets/54718375/5c5389b1-8872-498d-9428-70fb8dbba487

Same problem here. Using Arch.

Got the same issue, on Arch

Got the same issue, on Manjaro (arch based)


feat: update list segment header

Description
The component ListSegmentHeader contains a styled horizontal rule, git branch feature doesn't require that rule. This PR puts horizontal rule behind a flag.
Type of change

Please delete options that are not relevant.

New feature (non-breaking change which adds functionality)

How Has This Been Tested?
Jest tests that are contained within the PR.
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:green_circle: Total coverage has increased
// Code coverage diff between base branch:release and head branch: feat-update-ListSegmentHeader

Status
File
% Stmts
% Branch
% Funcs
% Lines

:green_circle:
total
56.19 (0.01)
37.68 (-0.01)
35.69 (0.03)
56.43 (0.01)

:green_circle:
app/client/src/components/ads/ListSegmentHeader.tsx
100 (36.36)
100 (0)
100 (100)
100 (36.36)

:red_circle:
app/client/src/utils/WorkerUtil.ts
89.76 (0)
70.59 (-1.96)
100 (0)
93.33 (0)

:red_circle:
app/client/src/utils/autocomplete/TernServer.ts
52.71 (-0.23)
40.83 (-0.84)
36.21 (0)
56.74 (-0.25)

/ok-to-test sha=b72fe2c


Window render bugged on windows 11

Hello, to render my window in the code i've put that in my main function :
WidgetsFlutterBinding.ensureInitialized();

  await windowManager.ensureInitialized();

  String feedURL = 'https://update.gregsvc.fr/WinSpeakerSwitcher.xml';
  await autoUpdater.setFeedURL(feedURL);
  await autoUpdater.checkForUpdates();

  // For hot reload, `unregisterAll()` needs to be called.
  WindowOptions windowOptions = const WindowOptions(
    size: Size(800, 600),
    center: false,
    backgroundColor: Colors.transparent,
    skipTaskbar: false,
    titleBarStyle: TitleBarStyle.normal,
    title: 'WinSpeakerSwitcher',
  );
  windowManager.waitUntilReadyToShow(windowOptions, () async {
    await windowManager.show();
    await windowManager.focus();
  });

  await hotKeyManager.unregisterAll();
  runApp(const ProviderScope(child: MyApp()));

But the toolbar in the window is bugged it's render like that :

The title in the titlebar is blank only when the window is in focus state otherwise when the window is in the background the title renders correctly :

Hi there, see if this would help: #301


Add computedCssValue (other name?) Condition

Selenium has the method getCssValue which can process the actual computed values of css-styles
getCssValue("color")
getCssValue("background-color")
getCssValue("font")
getCssValue("font-size")

etc.
We should add the condition, that uses this for Selenide Condition to enable easy coding for examples like
element.shouldHave(computedCssValue("color","#00FF00"));
element.shouldHave(computedCssValue("font-size","24"));

etc. with-in typical Selenide Workflow  to automatically benefit from waiting, screenshoting, clear error messages etc.

I think that name "cssValue" will be confusing for many users, who may think, conditions looks in DOM for some css attributes....
"Computed" - gives a hint, that it is the actual value which depends on many variables (included css files, css classes, custom style attrbutes etc.).
Perhaps you could think of even better name?
computedCssStyle perhaps?

something like that?
    public static Condition cssAttribute(final String attributeName, final String expectedValue) {
        return new Condition("cssAttribute") {
            @Override
            public boolean apply(WebElement element) {
                return expectedValue.equalsIgnoreCase(element.getCssValue(attributeName));
            }

            @Override
            public String actualValue(WebElement element) {
                return element.getCssValue(attributeName);
            }
        };
    }

I think that name "cssValue" will be confusing for many users, who may think, conditions looks in DOM for some css attributes. "Computed" - gives a hint, that...

I think it's ok to call it "cssValue" as is in Selenium. It will make reference to the original Selenium method call.

I agree.
Viele Grüße
Alexei Vinogradov
On 4. Mai 2018 at 12:13:48, Boris Osipov<EMAIL_ADDRESS>wrote:

I think that name "cssValue" will be confusing for many users, who may
think, conditions looks in DOM for some css attributes. "Computed" - gives
a hint, that...
I think it's ok to call it "cssValue" as is in Selenium. It will make
reference to the original Selenium method call.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/codeborne/selenide/issues/628#issuecomment-386558840,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGQ4YAskuPkJwYwbXtDYTtTt2SF7-cLuks5tvCncgaJpZM4QFujL
.


fixed compiler warnings: updated calendar units

Thanks @Magnat12 for your PR, I merged #59 which does the same thing & a little more.


Additional RBAC role assignment required to select Key Vault during initial setup of Entra Verified ID

It seems not possible to select Key Vaults during the initial setup of Entra Verified ID unless one assign the "Verifiable Credentials Service Admin" principal the Owner role at the subscription level and deploy a new Key Vault through the below "Create new key vault" button. Even after role assignment existing Key Vaults are not showing up. The account used to set up Entra Verified ID is assigned the Global Administrator directory role and the built-in Owner role at the subscription level.

Can someone please elaborate on the required RBAC roles and scopes?

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

ID: f84b158e-eebc-7d96-0992-8864316ddf5c
Version Independent ID: 8909ba9b-bf26-fb5c-29f0-76244984e75a
Content: Tutorial - Configure your tenant for Microsoft Entra Verified ID - Microsoft Entra
Content Source: articles/active-directory/verifiable-credentials/verifiable-credentials-configure-tenant.md
Service: decentralized-identity
Sub-service: verifiable-credentials
GitHub Login: @barclayn
Microsoft Alias: barclayn

@L480
Thanks for your feedback! We will investigate and update as appropriate.

@L480 Apologies for the delay in answering this issue. Reviewed your issue statement wanted to check if you have followed the steps under Set access policies for the Verified ID Admin user and Set access policies for the Verifiable credentials service request service principal

@Givary Thanks for your reply! I did that, but it doesn't seem to be sufficient as I still couldn't select a Key Vault.

@L480 Would request you to try this step again with user having this RBAC Role "Key Vault Administrator" and let me know the results.

@Givary Same issue :/

@L480
Please send me an email to<EMAIL_ADDRESS>with Sub - Attn: Givary and following details in the email body:
Link to this thread/post
We can connect offline and discuss further on this.


Added a data copy process to serving notebook

Added a data copy process to serving notebook to make it a standalone notebook.

@kylesteckler Could you take a look at this? This is a very minor update.


fix(snackbar): remove border-radius for snackbars in handset mode

Based on the specs and on the MDC reference implementation, snack bars should not have a border-radius when being in handset mode.

Fixes #12525

If this is superseded by the 2018 spec update then we should just close it

Well, from my perspective, the specs 2018 show various types of snackbars, and a full-width snackbar seems to be still part of it.
Quote from the specs

Snackbars can span the entire width of the screen only when a UI does not use persistent navigation components like app bars or bottom navigation bars. Snackbars that span the entire width of a UI can push only FABs up when they appear.

https://material.io/design/components/snackbars.html#placement

I've recently chatted with @crisbeto about the specs here. Personally, and based on the specs image, there should be still some full-width implementation.
I agree that if we have the spec update, which then changes the default snackbar to the non full-width mode, this change isn't really needed.


docs: update youtube videos and tutorials

added tutorials.mdx; updated youtube.mdx
Rationale: the Tutorials section in the documentation is top-priority. (for example, https://pytorch.org/docs/stable/index.html) Not every project has resources to make tutorials. We have such a privilege. Community experts created several tutorials on YouTube. But the tutorial links are now hidden on the YouTube page and not easily discovered by first-time visitors.

Added new videos and tutorials that were created since the last update.
Made some reprioritization between videos on the base of the view numbers.

Who can review?

@hwchase17

@dev2049

@dev2049 Can you, please, review it? Thanks!

@dev2049 Can you, please, review it? Thanks!

@hwchase17  Can you, please, review it? Thanks!

@rlancemartin, @eyurtsev Can you, please, review it? Thanks!

@rlancemartin  Please review it. Thanks!

@hwchase17 Can you, please, review it? Thanks! I need to add a new Deeplearning AI course link but this PR holds it back.

@hwchase17 Can you, please, review it? Thanks! I need to add a new Deeplearning AI course link but this PR holds it back.

Thanks! Will do shortly. Sorry, just saw this.

Appreciate it! Feel free to assign me to PRs to review in the future. Assigned PRs I petty easily check in Github.
Email traffic from tags is challenging to follow due to very high volume of tagging :P

@rlancemartin Thanks for the approval :)
Unfortunately, I don't have permission to assign somebody to my PR


feat(preset-wind): add placeholder rule

This PR adds placeholder-<color> and placeholder-<opacity> with proper css variable.
Clean commit: https://github.com/antfu/unocss/pull/407/commits/1c97d822e5e7eab27ea6c51ac427ad9f98701d11

@chu121su12  thx for clean commit man ;)


Clarify and/or rename `highest_adjacent_block.

Original question posed:

The name Igniter.Code.Common.highest_adjacent_block/1 was a bit confusing to me (perhaps because the code nests left<->right while AST nests up<->down, and it's unclear which mode of thinking was necessary). How's something like outer_containing_block/1? (Separately, wouldn't it always return defmodule's block? Or maybe I'm misunderstanding the functionality.)

My thoughts:
I think a bette name is definitely in order. It only goes to the "outermost" block that is only a {:__block__, and it only goes up the tree while it is findings {:__block__. So if it gets to something like {:def, ....} it would not continue going up. That is the adjacent part of that. I will see if I can think of a better name for it.

@ibarakaiev I can't really think of a better name yet. It's a private function so there isn't really a rush to get it figured out. I think I'll actually close this for now since it's not a public function....yeah, gunna not worry about it for now.


refLength was not always updating

The refLength update wasn't always done. This had the impact of eventually breaking the clipplane effect which depends on it.

Thanks @martinRenou

Bot please update snapshots

Some visual changes are not ok, putting as draft

Bot please update snapshots

Could this be the reason why the pointer "ball" was sometimes oversized / undersized?

Could this be the reason why the pointer "ball" was sometimes oversized / undersized?

Very likely yes. The pointer ball diameter should now stay roughly a tenth of the reference length.


New Sample: Hybrid Work Employee Check-in

Q
A

Bug fix?
no

New feature?
no

New sample?
yes

Related issues?

What's in this Pull Request?

This adaptive card extension sample provide option for employees to check-in in hybrid work environment about their preference of work place and self declaration related to Covid. Data is managed in a SharePoint list.

Thank you @ArijitCloud! We will be processing the sample soon! 🧡💜

@ArijitCloud Thanks for submitting the sample. I merged the PR. David will reach out to see if you would like to schedule a demo.


[2.7] Track status of metrics-proxy daemonset

Submariner 0.14.0 added a new Daemonset, submariner-metrics-proxy which is used to allow access to submariner metrices for gateway and globalnet. Addon should track status of this Daemonset and report errors if any.
Refer https://github.com/submariner-io/submariner/issues/1944

/assign @vthapar


fetchgit with leaveDotGit: increase determinism

Motivation for this change
#46739 without controversial changing of number of threads
-f means --no-reuse-deltas
-F means --no-reuse-deltas --no-reuse-objects

Any risk that this breaks existing hashes?

Any risk that this breaks existing hashes?

yes, probably the two mentioned in #46739 and one #56451
I did not expect this PR to be accepted (at least it the original form) so did not fix the caches. It should be done before the merge, if it will happen


Workaround for IOExceptions when generating forms-related code

@pefernan or @romartin could you please quickly check this?

@psiroky good point! Thanks for fixing that. +1


N plus 1 queries

🚩New Implementations: Fix N+1 Problems
Hi there! @Luffytaro22 👋
Here I detail a summary of the new features that have been implemented in this branch:

[x] Install the bullet gem to analyze the code and detect N+1 problems
[x] Fix N+1 problems in the controllers, views
[x] Fix some issues in some of the test cases
[x] Improve the CSS styles of the app
[x] Improve the views, controllers and models logic
[x] Update the README file
[x] Update the LICENSE file

⭐To the code reviewer 👨💻
Please, I kindly ask you to review the entire code and verify that all requirements have been met correctly.
if there is any issue 🦯 in this PR, please do list 📃 them in a descriptive 💡 manner and give your best suggestions 🎁 if needed.

@Luffytaro22 Thank you so much for reviewing the last changes added to this branch!