HADOOP-17643 WASB : Make metadata checks case insensitive

NOTICE
Please create an issue in ASF JIRA before opening a pull request,
and you need to set the title of the pull request which starts with
the corresponding JIRA issue number. (e.g. HADOOP-XXXXX. Fix a typo in YYY.)
For more details, please see https://cwiki.apache.org/confluence/display/HADOOP/How+To+Contribute

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 37s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
35m 20s

trunk passed

+1 :green_heart:
compile
0m 38s

trunk passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
compile
0m 33s

trunk passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
checkstyle
0m 26s

trunk passed

+1 :green_heart:
mvnsite
0m 38s

trunk passed

+1 :green_heart:
javadoc
0m 30s

trunk passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javadoc
0m 27s

trunk passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
spotbugs
1m  3s

trunk passed

+1 :green_heart:
shadedclient
14m 28s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 29s

the patch passed

+1 :green_heart:
compile
0m 31s

the patch passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javac
0m 31s

the patch passed

+1 :green_heart:
compile
0m 26s

the patch passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
javac
0m 26s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 18s

the patch passed

+1 :green_heart:
mvnsite
0m 30s

the patch passed

+1 :green_heart:
javadoc
0m 24s

the patch passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javadoc
0m 19s

the patch passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
spotbugs
1m  3s

the patch passed

+1 :green_heart:
shadedclient
13m 59s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
2m  4s

hadoop-azure in the patch passed.

+1 :green_heart:
asflicense
0m 34s

The patch does not generate ASF License warnings.

76m 22s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2972/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/2972

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell

uname
Linux 16864d42b9cd 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / af0c24352c82400399edb11479769704a89a132f

Default Java
Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2972/1/testReport/

Max. process+thread count
720 (vs. ulimit of 5500)

modules
C: hadoop-tools/hadoop-azure U: hadoop-tools/hadoop-azure

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2972/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

+1

Thanks @anoopsjohn.
Can you raise backport PR for 3.2 and 3.3 branch ?

Thanks.  Raised backport PRs.


Illuminate/queue missing dependency (composer)

Illuminate/queue requires an implementation of the Illuminate\Contracts\Bus\Dispatcher class, I think it should either require or suggest the illuminate/bus package.

No need. We require the contract. It's up to the user to make sure an implementation exists.

We should really be using virtual packages on composer I guess, but that's not caught on. Maybe for Laravel 6. ;)


src/utils/eventLevels.js : findIndex is not supported on ie11

Since this commit, an error is thrown by ie11, complaining about findIndex support.
support

We corrected the problem by adding the polyfill when initializing our project.

This should be fixed since https://github.com/intljusticemission/react-big-calendar/pull/671 by including lodash's findIndex method.


[HUDI-7664] Remove Hadoop dependency from hudi-io module

Change Logs
This PR removes usage of Hadoop classes and the Hadoop dependency from hudi-io module.
Impact
Makes hudi-io module Hadoop indepedent.
Risk level
none
Documentation Update
none
Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

@hudi-bot run azure


Update app script for new flow

will likely have some conflicts with https://github.com/axiom-crypto/axiom-quickstart/pull/4
main content in this PR is the app/ folder:

significantly streamlines client side app code (app/index.ts)
user first runs npx axiom circuit compile app/axiom/average.circuit.ts to compile the circuit
user then runs npm run start to run the script to send a Query with the compiled circuits and exampleInputs

updates in the other folders can be overridden by https://github.com/axiom-crypto/axiom-quickstart/pull/4

Closing as it has been merged into #4


Source controller support for Instance Metadata Service v2 (IMDSv2) on AWS

I get the following error message in source-controller when Flux is configured to use S3 bucket source,:
{
  "level": "error",
  "ts": "2022-06-03T13:29:32.250Z",
  "logger": "controller.bucket",
  "msg": "Reconciler error",
  "reconciler group": "source.toolkit.fluxcd.io",
  "reconciler kind": "Bucket",
  "name": "deploy-bucket",
  "namespace": "flux-system",
  "error": "failed to confirm existence of 'retracted-flux-1ejashffo6o0p' bucket: 401 Unauthorized"
}

I have created the bucket source with the following command:
flux create source bucket deploy-bucket \
  --bucket-name=retracted-flux-1ejashffo6o0p \
  --provider=aws \
  --endpoint=s3.amazonaws.com \
  --region=eu-central-1 \
  --interval=5m

.. and receive the following error message:
✚ generating Bucket source
► applying Bucket source
✔ Bucket source created
◎ waiting for Bucket source reconciliation
✗ failed to confirm existence of 'retracted-flux-1ejashffo6o0p' bucket: 401 Unauthorized

It appears that minio-go library uses Instance Metadata Service (IMDS) at <IP_ADDRESS> to create temporary credentials to access the S3 bucket (from captured traffic):
GET /latest/meta-data/iam/security-credentials/ HTTP/1.1
Host: <IP_ADDRESS>
User-Agent: Go-http-client/1.1
Accept-Encoding: gzip

This is not permitted, because my configuration requires Instance Metadata Service v2 (IMDSv2) since doing so avoids some vulnerabilities:
HTTP/1.1 401 Unauthorized
Content-Length: 0
Date: Fri, 03 Jun 2022 13:29:32 GMT
Server: EC2ws
Connection: close
Content-Type: text/plain

The fix would be to use Instance Metadata Service v2 (IMDSv2) compatible client with the minio-go library. This adds a session token to the requests.
I was able to workaround by creating an IAM user with an access key, then creating the source with--access-key and --secret-key args.
I am running Flux v0.30.2 on EKS with the Kubernetes version v1.21. I have created the IAM policy to access the bucket as instructed by documentation. Instances receive this policy from the instance profile. The access works when tested with AWS CLI aws s3 ls command.
Resources:

AWS: Use IMDSv2
aws/containers-roadmap issue EKS: Support for IMDSv2 #930

Have you found the solution to this problem?


Do pics

Now, GWAS ingestion, LD expansion and PICS works. Tests are passing. I would merge this branch at this point and make updates when required.

Not sure how the coverage has changed though...

Codecov Report

Merging #57 (19148c0) into main (a7210a7) will increase coverage by 0.83%.
The diff coverage is 52.25%.

@@            Coverage Diff             @@
##             main      #57      +/-   ##
==========================================
+ Coverage   53.89%   54.72%   +0.83%
==========================================
  Files          35       41       +6
  Lines         590      740     +150
==========================================
+ Hits          318      405      +87
- Misses        272      335      +63

Impacted Files
Coverage Δ

src/etl/common/utils.py
71.42% <ø> (+4.76%)
:arrow_up:

src/etl/v2g/intervals/Liftover.py
35.71% <ø> (-0.96%)
:arrow_down:

src/run_gwas_ingest.py
44.00% <25.00%> (-16.00%)
:arrow_down:

src/etl/gwas_ingest/study_ingestion.py
37.50% <30.43%> (ø)

src/etl/gwas_ingest/ld.py
38.70% <38.70%> (ø)

src/etl/gwas_ingest/pics.py
45.28% <45.28%> (ø)

src/etl/gwas_ingest/process_associations.py
47.82% <46.42%> (-10.51%)
:arrow_down:

src/pics_experiment.py
56.25% <56.25%> (ø)

src/run_precompute_ld_indexes.py
58.33% <58.33%> (ø)

src/etl/gwas_ingest/association_mapping.py
81.48% <81.48%> (ø)

... and 28 more

@DSuveges Tests are failing cause unfortunately we cannot simply port the doctest to get_record_with_maximum_value.
First of all, this function depends on another one and the doctest is not able to resolve that context. And then the functions are not identical, yours created a new column of booleans to identify the top one within the window, mine applies the filter on the df. If the usecase is still valid for you, we can leave the test outside of the docs. These functions are however tested already inside tests/test_spark_helpers.py.

Yes, that's right, I just wanted to push before I leave.


Pre-training on GPUs seems to be stuck

I have tried to perform pre-training from scratch on GPUs using the following command:
python run_pretraining.py --albert_config_file=albert_config.json --do_train --input_files=/somewhere/*/tf_examples.*.tfrecord --meta_data_file_path=/somewhere/train_meta_data --output_dir=/somewhere --strategy_type=mirror --train_batch_size=128 --num_train_epochs=2
But it seems to be stuck as follows:
...
INFO:tensorflow:Reduce to /job:localhost/replica:0/task:0/device:CPU:0 then broadcast to ('/job:localhost/replica:0/task:0/device:CPU:0',).
I1209 00:48:14.076103<PHONE_NUMBER>37952 cross_device_ops.py:427] Reduce to /job:localhost/replica:0/task:0/device:CPU:0 then broadcast to ('/job:localhost/replica:0/task:0/device:CPU:0',).
INFO:tensorflow:batch_all_reduce: 32 all-reduces with algorithm = nccl, num_packs = 1, agg_small_grads_max_bytes = 0 and agg_small_grads_max_group = 10
I1209 00:48:24.566839<PHONE_NUMBER>37952 cross_device_ops.py:748] batch_all_reduce: 32 all-reduces with algorithm = nccl, num_packs = 1, agg_small_grads_max_bytes = 0 and agg_small_grads_max_group = 10
INFO:tensorflow:batch_all_reduce: 32 all-reduces with algorithm = nccl, num_packs = 1, agg_small_grads_max_bytes = 0 and agg_small_grads_max_group = 10
I1209 00:48:45.377745<PHONE_NUMBER>37952 cross_device_ops.py:748] batch_all_reduce: 32 all-reduces with algorithm = nccl, num_packs = 1, agg_small_grads_max_bytes = 0 and agg_small_grads_max_group = 10
2019-12-09 00:49:16.104345: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcublas.so.10.0

GPUs are running, but no outputs are found.
The core of pre-training code is similar to the following tensorflow BERT code, and I have succeeded in running the following pre-training code.
https://github.com/tensorflow/models/tree/master/official/nlp/bert
My environment is as follows:

tensorflow-gpu==2.0.0
CUDA 10.0

Thanks in advance.

After 20 hours later from starting, a log line was outputted, and so it wasn't stuck.
When I used the above tensorflow BERT code, log lines were outputted frequently. I am not sure why these are different.

@tomohideshibata I am facing the same issue, can you suggest what changes I need to solve this issue?

@008karan Hi. As in my above comment, a log line was outputted after 20 hours later from starting. (I changed nothing)
I think there is something strange in the pre-training codes.


add reorder selection property when order in selectize has been changed

Closes #92
I'm not quite sure if the tests are enough and would appreciate some guidance :-)

Did some refactoring to get the tests green but I couldn't find a better solution for this:
https://github.com/mszoernyi/ember-cli-selectize/commit/6ce8e6d8014efc42135aa2c7e1aa4406b41e0d7f#diff-d0312fe215da74aeb01ef78212c49b42R356
Any suggestions for this?

Any updates on this?

Still didn't have the time to review this, @mszoernyi. Sorry.

actually I tried it and it stuck in the endless loop with addItem, onChange events.

seems like doesn't work with the latest version. Will check when I get some time. I'm using this code with 0.3.x.

@mszoernyi then there is no need to this PR, I guess? It is a bit misleading, though, it gave me a couple of ideas.

is there any other solution for selection ordering when drag_drop plugin is used?


[fix][pass] Save the function when it is used as a call arg

Currently the args of the call nodes are not checked when removing functions. It causes function undefined problem when a global function is invoked by a call as its argument because the definition will be removed.
cc @wweic @icemelon9

Thanks @zhiics @wweic


python3Packages.fastapi: 0.67.0 -> 0.68.0

Motivation for this change
Update to latest upstream release 0.68.0
Change log: https://fastapi.tiangolo.com/release-notes/#0680
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

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

1 package failed to build:

theharvester

9 packages built:

dyndnsc
python38Packages.dependency-injector
python38Packages.fastapi
python38Packages.json-logging
python38Packages.slowapi
python39Packages.dependency-injector
python39Packages.fastapi
python39Packages.json-logging
python39Packages.slowapi

Result of nixpkgs-review pr 133412 at 60348662 run on x86_64-linux 1

1 package failed to build:

theharvester

9 packages built successfully:

dyndnsc
python38Packages.dependency-injector
python38Packages.fastapi
python38Packages.json-logging
python38Packages.slowapi
python39Packages.dependency-injector
python39Packages.fastapi
python39Packages.json-logging
python39Packages.slowapi

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement. Does something look off? Please file an issue or reach out on IRC.


Overflow of transaction row / collapse of transaction rows  on Safari not working

Describe the bug
Display of transactions does not work as expected on Safari. Content in rows overflow to next rows. Same applies to account detail page (same component?)
On transaction detail page, collapse feature does not work either. Adds scroll bar to transaction row instead.
To Reproduce
Open OBE and view transactions
Click on transaction hash and open transaction details page
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
Homepage:

Transaction Details:

Account page:

Desktop (please complete the following information):

OS: MacOS 12.2.1
Browser Safari
Version 15.3

@Kersten-TCD I haven't been able to reproduce this bug. Would you mind confirming it so we can close this?


Error after fresh installation

ℹ Computer information

Platform OS (e.g Windows, Mac, Linux etc):  Windows
Python Version: 3.7
Brain Interface Used (e.g Muse, OpenBCI, Notion etc): Muse2

📝 Provide detailed reproduction steps (if any)

Did fresh installation, new environment, clone, pip install -e .
Started quality check

✔ Expected result
Channel quality check as per @JohnGriffiths NTCS videos
❌ Actual result
  File "C:\Users\daniele\miniconda3\envs\eeg-notebooks\Scripts\eegnb-script.py", line 33, in <module>
    sys.exit(load_entry_point('eeg-notebooks', 'console_scripts', 'eegnb')())
  File "C:\Users\daniele\miniconda3\envs\eeg-notebooks\Scripts\eegnb-script.py", line 25, in importlib_load_entry_point
    return next(matches).load()
  File "C:\Users\daniele\miniconda3\envs\eeg-notebooks\lib\site-packages\importlib_metadata\__init__.py", line 203, in load
    module = import_module(match.group('module'))
  File "C:\Users\daniele\miniconda3\envs\eeg-notebooks\lib\importlib\__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1006, in _gcd_import
  File "<frozen importlib._bootstrap>", line 983, in _find_and_load
  File "<frozen importlib._bootstrap>", line 967, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 677, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 728, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "c:\users\daniele\dropbox\code\eeg-notebooks\eegnb\cli\__main__.py", line 9, in <module>
    from .introprompt import intro_prompt
  File "c:\users\daniele\dropbox\code\eeg-notebooks\eegnb\cli\introprompt.py", line 6, in <module>
    from eegnb.devices.eeg import EEG
  File "c:\users\daniele\dropbox\code\eeg-notebooks\eegnb\devices\eeg.py", line 17, in <module>
    from brainflow import BoardShim, BoardIds, BrainFlowInputParams
ImportError: cannot import name 'BoardShim' from 'brainflow' (C:\Users\daniele\miniconda3\envs\eeg-notebooks\lib\site-packages\brainflow\__init__.py)

This happens whichever command I input, like
eegnb checksigqual -ed muse2 or
eegnb runexp -ip
the muse is streaming ok with bluemuse
📷 Screenshots

Fixed in https://github.com/NeuroTechX/eeg-notebooks/commit/75e42ef09181a976eac4531a75e9282590a6345e, just cherrypicked onto master.

( FYI this appeared to be due to a small brainflow API change . @danielemarinazzo please try out a re-install with @ErikBjare 's fix. )

thanks, can I just redo pip install -e . or shall I undo something?

checked and I also had the error before the fix; and pip install now working ok again on my side.

good catch @ErikBjare and @danielemarinazzo , thanks for flagging.

Fixed in https://github.com/NeuroTechX/eeg-notebooks/commit/f208e086a7302ffec069ed1475faabbf6b95f9d9
There may be follow-up errors, but they deserve separate issues.

works now, thanks!


Add logo-use page linking to SMART guidelines

This is a step toward resolving #475. The linked content at smarthealthit.org is not yet available, but is a work in progress. It would be good to incorporate this general guidance into the official docs, linking out to the more detailed guidelines.

Wouldn't it make more sense to wait until https://smarthealthit.org/cds-hooks-logo exists before merging? Why put a broken link on cds-hooks.org? Broken links simply degrade a visitor's trust in the site.

We're working on getting content up, indeed. BCH counsel is drafting an updated, simplified set of guidelines. I think having the notice is strictly better than not -- but we can commit to getting a placeholder up to avoid a broken link ASAP, even before the guidelines are compete.

Hey Josh,
First - great work getting progress from BCH counsel! This is important stuff.
I recommend three changes:

Don't publish until there's actually an accurate legal description of how the logo can be used. Otherwise, the cds-hooks.org page specifically does not contain allowed uses of the logo.
Move the "Logo Use" link from the main menu into the footer with the rest of our legalese.
Since the cds-hooks.org/logo-use page is explanatory text and not the actual legal use license, I think that we should soften the "The Children's Medical Center Corporation" term, which is not a recognizable name and sounds extremely corporate. Preferably, we'd simply call it the "Boston Children's Hospital" (like we do in the footer). If this isn't legally advisable, I'd suggest explaining, like:

The CDS Hooks logos are owned by The Children's Medical Center Corporation (the parent company of Boston Children's Hospital) ...

Apologies for the tardiness of this feedback.
Isaac

To be clear, I merged this after @brynrhodes's review yesterday. I'm happy to move the link to the bottom of the page, and to update the language to read "Boston Children's Hospital" per your suggestion (will do this in a follow-on PR). It's very helpful though to have this content in place as we get detailed guidance onto the smarthealthit.org site, so I hope it's okay that we have a pointer to content-in-progress. (To be fair, the terms are always in-progress in the sense that they can be changed, so I don't see this as a huge limitation, and in any case we should have more specifics shortly.)

Updated https://github.com/cds-hooks/docs/pull/480 with your suggestions @isaacvetter

Is https://smarthealthit.org/cds-hooks-logo/ complete?!?!

I think so -- is there something surprising about it (given the double interrobang)??


Exception when trying to return DMChannel from User.dart

Exception has occurred.
StateError (Bad state: No element)

is thrown when return (client.channels.findOne((Channel c) => c is DMChannel && c.recipient.id == this.id) as DMChannel) ?? await downloadChannel(); is called at row 59, User.dart
Just tried to do a simple try-catch which makes is return null and run downloadChannel() correct. Which works fine for me atm but would ofc be better to solve it properly.
Basically I'm just trying to store the dm channel like this;
var dmChan = await ctx.member.dmChannel;
Cheers!

Are you using rewrite_modular branch?

Nope, this is 0.30.0. Should I switch?

Yeah, 0.30.0 is pretty much outdated. Join https://discord.gg/discord-api and head to #dart_nyxx for more info.
You will need that code to get nyxx from git, and dart dev sdk.
nyxx:
    git:
      url: git://github.com/l7ssha/nyxx.git
      ref: rewrite_modular
      path: nyxx

To run code you will need to specify additonal parameters --enable-experiment=non-nullable --no-null-safety. Ask in #dart_nyxx in discord api guild if you have further questions

Thank's man, will check it out! Closing it now and re-opening if I see it when I've changed branch. Cheers!


Crash while executing /pex promote s

The server crashes while executing "/pex promote s"
config: https://pastebin.com/eqg0QHBm
The server log does not contain any information

Unfortunately PermissionsEx is currently unmaintained and no further updates are planned. Because of this, all pull requests are being closed. If you are looking for an actively maintained permissions plugin, take a look at LuckPerms. It is compatible with major server platforms and can import data from PEX.


SIGSEGV: Illegal storage access. (Attempt to read from nil?)

sorry, I've got a new bug with the binary you sent me  :-)
( the conditions are the same as previously : https://github.com/brentp/duphold/issues/22 )
+ /ccc/work/cont007/fg0019/lindenbp/packages/duphold/duphold --snp /ccc/scratch/cont007/fg0113/lindenbp/20190428.TROUCARD.MANTA/work/bd/c70293fb517483436faf17dd3b2e61/B00I4BY.snv.bcf --vcf jeter.bcf --bam /ccc/genostore/cont007/fg0073/fg0073/analyse/projet_TROUCARD_605/ANALYSE/ANALYSE_T605_B00I4BY_HJWJJALXX_2_hs37d5/MAPPING_B00I4BY/reliable.realign/T605_DA_B00I4BY_HJWJJALXX_hs37d5_MERGE_PE_2.reliable.realign.bam --fasta /ccc/work/cont007/fg/fg/biobank/by-taxonid/9606/hs37d5/hs37d5_all_chr.fasta --threads 4 -o jeter.duphold.vcf
I, [2019-05-01T15:41:17] -- duphold: starting to read snps for chrom: chr1
I, [2019-05-01T15:41:17] -- duphold: done reading 4143 bi-allelic snps for chrom: chr1
SIGSEGV: Illegal storage access. (Attempt to read from nil?)

trying to run with gdb:
(...)
I, [2019-05-01T15:45:31] -- duphold: starting to read snps for chrom: chr1
I, [2019-05-01T15:45:31] -- duphold: done reading 4143 bi-allelic snps for chrom: chr1
Program received signal SIGSEGV, Segmentation fault.
0x00002aaaaaab8039 in collectZCT_EN6T32AMm3va9bsrdxtF0cg ()
(gdb) bt
#0  0x00002aaaaaab8039 in collectZCT_EN6T32AMm3va9bsrdxtF0cg ()
#1  0x00002aaaaaab8b9b in collectCTBody_XHio9cMpnLoH7GyCj1Z9besg_2 ()
#2  0x00002aaaaaab8db6 in rawNewObj_ehkAaLROrd0Hc9aLROWt1nQ.constprop.33 ()
#3  0x00002aaaaaab9609 in newObj ()
#4  0x00002aaaaaaf54aa in annotate_5vOmVhlMmJnT4dAxyYRfyg ()
#5  0x00002aaaaaaf7235 in main_KwKibQAZqlBKGrljFq9agmQ ()
#6  0x00002aaaaaaf9477 in NimMain ()
#7  0x00002aaaaaaaf1dd in main ()

P.

does it run successfully without the SNP vcf?
and any chance you can share the files to recreate? looks like it would just need chr

the program is currently running without any problem without --snps.
Would it be useful if i just send the manta.vcf and the snps.vcf file ? (not the bam ).  Or do you have an instance of duphold with debug/log message to trace where is the error ?
16H00. I'll be away from git until tomorrow.

here is a debug version that will give more info.
if you could send both vcfs and a bam, that would help. maybe it happens right away so I'd only need the first part of bam (not sure if that's the case).
duphold_debug.gz

one last for today with your debug-version:
+ /ccc/cont007/home/fg0019/lindenbp/duphold_debug --snp /ccc/scratch/cont007/fg0113/lindenbp/20190428.TROUCARD.MANTA/work/bd/c70293fb517483436faf17dd3b2e61/B00I4BY.snv.bcf --vcf jeter.bcf --bam /ccc/genostore/cont007/fg0073/fg0073/analyse/projet_TROUCARD_605/ANALYSE/ANALYSE_T605_B00I4BY_HJWJJALXX_2_hs37d5/MAPPING_B00I4BY/reliable.realign/T605_DA_B00I4BY_HJWJJALXX_hs37d5_MERGE_PE_2.reliable.realign.bam --fasta /ccc/work/cont007/fg/fg/biobank/by-taxonid/9606/hs37d5/hs37d5_all_chr.fasta --threads 1 -o jeter.duphold.vcf
I, [2019-05-01T16:16:24] -- duphold_debug: starting to read snps for chrom: chr1
I, [2019-05-01T16:16:25] -- duphold_debug: done reading 4143 bi-allelic snps for chrom: chr1
duphold.nim(659)         duphold
duphold.nim(647)         main
duphold.nim(525)         annotate
system.nim(3059)         sysFatal
Error: unhandled exception: index 4143 not in 0 .. 4142 [IndexError]

OK. Thanks very much for persisting. As you're finding, the SNP stuff is less battle-tested than the depth stuff (mostly because it doesn't add much value in our hands).
This build should address that problem:
duphold_debug2.gz

@brentp  that worked, thanks Brent


Generate osbuild stages documentation

Now that the json schemas exist outside the Python code, we should be able to just generate the documentation for each stage and publish it on the website.
The target should be: https://github.com/osbuild/osbuild.github.io/tree/main/docs/developer-guide/02-projects/osbuild
And a script similar to: https://github.com/osbuild/osbuild.github.io/blob/main/scripts/pull_readmes.py

See: #77


Different bleu scores in Google colab and Jupyter notebook

I am trying get Bleu score for a list of 1000 English sentences and their references.
sample sentence is as below,
prediction_en = 'A man in an orange hat presenting something'
reference_en= 'A man in an orange hat starring at something'

Used code,
from nltk.translate.bleu_score import sentence_bleu
from nltk.translate.bleu_score import SmoothingFunction
smoother = SmoothingFunction()

sentence_bleu(reference_en, prediction_en , smoothing_function=smoother.method4)

For this I am getting a Bleu score of  0.768521 in google colab but I am getting  1.3767075064676063e-231 score in jupyter notebook without smoothing and with smoothing  0.3157039
Can you please help me what and where I am going wrong?

This was due to different versions of nltk

Beyond what is implemented and whether it's the "canonical" version of BLEU that stood the "test of time" and everyone should follow. Out of curiosity, given your data, which score are you expecting?


Feedback to clarify the destination directory..

[Enter feedback here]

For example, if AZURE_TOMCAT85_HOME had a value of D:\Program Files (x86)\apache-tomcat-8.5.37, the full path of the copied configuration directory would be D:\home\tomcat\conf.

I think we should rename D:\home\tomcat\conf to D:\home\apache-tomcat-8.5.37
It would be a good idea to highlight that apache-tomcat-8.5.37 should be copied as is to D:\home folder. Users get the impression that they need to create a directory named "tomcat" under D:\home and copy the conf folder. But it should be D:\home\apache-tomcat-8.5.37\
Then only the changes made to the server.xml will be taken effect. When my customer struggled last week, I was able to solve their problem by copyg the source folder as is to D:\home directory.

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

ID: df68e904-7b05-38ba-4203-50eb8491946f
Version Independent ID: 921377f7-e428-143d-70e4-c9efc25e41d1
Content: Configure Windows Java apps - Azure App Service
Content Source: articles/app-service/configure-language-java.md
Service: app-service
GitHub Login: @JasonFreeberg
Microsoft Alias: jafreebe

@kumarallamraju Thank you for your feedback. We will investigate and update the thread further.

@shashishailaj I merged a PR to fix this. Please close.

@kumarallamraju Thanks for bringing this to our attention. We will now close this issue.


live stops within in a minute

General Question
i  a using live to start unfortunately it is getting stopped with in a minute. I am using obs to stream the video. what is the issue

What's your code looking like? Boone can help you otherwise.

const { broadcast_id, upload_url } = await ig.live.create({       // create a stream in 720x1280 (9:16)       previewWidth: 720,       previewHeight: 1280,       // this message is not necessary, because it doesn't show up in the notification       message: 'Hi all',       });       // (optional) get the key and url for programs such as OBS       const { stream_key, stream_url } = Api.LiveEntity.getUrlAndKey({ broadcast_id, upload_url });      console.log(Start your stream on ${stream_url}\n
Your key is: ${stream_key}`.cyan);
console.log('Your broadcast id: '+ broadcast_id);
/**
* make sure you are streaming to the url
* the next step will send a notification / start your stream for everyone to see
*/
const startInfo = await ig.live.start(broadcast_id);
// status should be 'ok'
console.log("Status:" + startInfo.status);
  await ig.live.unmuteComment(broadcast_id);

  await Bluebird.delay(2000);
  let finishStream = false;
  let info = await ig.live.info(broadcast_id);
  //console.log(info.dash_playback_url);
  console.log("Watch the preview here: https://iglivepreview.glitch.me/?liveUrl="+info.dash_playback_url);
  console.log();

  process.on('SIGINT', function() {
      finishStream = true;
  });

`


Membership page Discount Information Component

User Story
As a website visitor, I want to be able to view the different merchant discounts offered by CSSA membership card on the Membership page so I can see the details of each discount.
As a member of the IT department, I want to be able to update the content of the component by changing the content in the database so that I can avoid modifying the frontend source code and redeploying the website with content changes.
Acceptance Criteria

The component is located under the banner component and takes a prominent size on the page.
The component is dynamic and fetches the discount information from the backend.
The discount information is automatically formatted and organized in a comprehensible layout by a frontend algorithm.
The component displays all the discount information with their respective merchant in a comprehensible way, with the details and the conditions of the discount.

Priority
High
Story Points
8

No longer relevant, refer to issue #15 and issue #16.


[BUG] Theme does not work with Breadcrumbs plugin grids

Describe the bug
This theme does not seem to work well with the Breadcrumbs plugin, which is used to create navigational breadcrumbs on top of the preview view.
In other themes, its grid view can display multiple "paths" from the current note to the index note through this grid-like visualization. But in Spectrum it will just stack all elements vertically, without being able to see each paths.
To Reproduce
Steps to reproduce the behavior:

Use breadcrumbs plugin
Activate "grid" view
Have some hierarchy with multiple paths

Expected behavior
It should look like in other themes, indicating path of all breadcrumbs, instead of just stacking all boxes.
Screenshots
Example 1:

Example 2

Desktop (please complete the following information):

OS: macOS
Obsidian Version: 0.12.16

I will be looking into this, thanks for the issue, sorry for the delay.

fixed in e3c72f3


remove double negatives from config.

Several comments were made in #pump.io about how confusing it is to require double negatives in config files, e.g. "I don't want to use the CDN, so I need to set this CDN related parameter to false" so I've switched them over to more sensible ones.

@tekktonic if you want to submit this again I'd be interested in taking it (not sure when though since it's semver-major)


Images always resized along with columns

Greetings and Happy Friday! This time I have a real bug for you, hopefully you can help me out once more. Using the provided example file and code snippet you should be able to see the problem I'm having.
If the source spreadsheet contains an image, and then the widths of the columns it overlaps are manipulated, the image will be scaled as well.  However, there are options within Excel that allow users to explicitly determine the behavior of image resizing with respect to column manipulation (right-click image -> "Size and Properties" -> "Properties" section).  In this case, I have set the image properties to "Don't move or size with cells", so I'm hoping that there is a way for that setting to be taken into account.
Hopefully this is enough to get started. Please let me know if I can be of any more help.
Thanks!
example-with-image.xlsx
const xlsxPopulate = require('xlsx-populate');

xlsxPopulate.fromFileAsync('./example-with-image.xlsx')
            .then(workbook => {
                const sheet = [
                    [1, 2, 3],
                    [4, 5, 6],
                    [7, 8, 9]
                ];

                const worksheet = workbook.sheet(0);

                // Insert sheet data
                const rowsCount = (sheet.length - 1);
                const columnCount = (sheet[0].length - 1);
                const cellStartInsert = workbook.find("${data}")[0];
                const cellEndInsert = cellStartInsert.relativeCell(rowsCount, columnCount);
                const addrStartCell = cellStartInsert.address();
                const addrEndCell = cellEndInsert.address();
                const rangeForSheet = worksheet.range(`${addrStartCell}:${addrEndCell}`);
                rangeForSheet.value(sheet);

                // Auto-size column widths and set heights to rows
                const addrHeaderEnd = cellStartInsert.relativeCell(0, columnCount).address();
                const rangeHeader = worksheet.range(`${addrStartCell}:${addrHeaderEnd}`);
                rangeHeader.forEach((cell, ri, ci, range) => {
                    const cellColumnLast = cell.relativeCell(rowsCount, 0);
                    const addrColumnFirst = cell.address();
                    const addrColumnLast = cellColumnLast.address();

                    const columnWidth = 25;

                    const addrColumn = cell.column().address().split(':')[0];
                    worksheet.column(addrColumn).width(columnWidth);
                });

                workbook.toFileAsync('./out.xlsx');
            });

First off, thanks for the extremely helpful error report! Having the sample code and template demonstrating the issue is extremely helpful!
Second, just a couple tips. The Sheet.range method also accepts start cell and end cell. See the API docs here. So you could use that instead of concatenating the cell addresses into a range address. And instead of using Workbook.find, you could use Defined Names to identify a cell. It is more performant than searching for all matches of a string.
Now to your issue. I do see the issue you are having, but the news isn't very good. It looks like the image is referenced in the XLSX zip in xl/drawings/drawing1.xml. There is a block of XML that seems to be responsible for positioning:
<xdr:from>
	<xdr:col>1</xdr:col>
	<xdr:colOff>0</xdr:colOff>
	<xdr:row>1</xdr:row>
	<xdr:rowOff>1</xdr:rowOff>
</xdr:from>
<xdr:to>
	<xdr:col>9</xdr:col>
	<xdr:colOff>0</xdr:colOff>
	<xdr:row>8</xdr:row>
	<xdr:rowOff>129541</xdr:rowOff>
</xdr:to>

Looks like references to starting/ending row/columns with some offset (not sure of the units).
When you resize the columns with xlsx-populate, it doesn't touch this drawing.xml file at all. So the 'to' xml node still refers to column 9, which is why the image gets resized.
If you resize the column in Excel, those XML nodes change to something like:
<xdr:from>
	<xdr:col>1</xdr:col>
	<xdr:colOff>0</xdr:colOff>
	<xdr:row>1</xdr:row>
	<xdr:rowOff>1</xdr:rowOff>
</xdr:from>
<xdr:to>
	<xdr:col>7</xdr:col>
	<xdr:colOff>142875</xdr:colOff>
	<xdr:row>8</xdr:row>
	<xdr:rowOff>129541</xdr:rowOff>
</xdr:to>

Notice that Excel updated the 'to' node in such a way to keep the image size the same.
This is unfortunately a non-trivial issue to fix as there is no current drawing support in xlsx-populate. We'd have to support parsing/writing the drawing.xml files. Not opposed to doing it at some point, but it doesn't look very simple.
The only hack I could think of for you to try is to change the size of another column in the range covered by the image to offset the change. So if you increase column A from 10 to 15, reduce column B from 10 to 5. Not ideal, but I don't have any more clever ideas.

My pleasure! I'm glad it was enough information to perform an investigation.
Thank you very much for the thorough explanation on the issue from your perspective.  It's too bad that there isn't a fix for this possible in the short term but I kinda figured that would be the case.  Your "hack" would definitely fix the issue but, sadly, my actual resize logic determines the column widths based on the size of the content within them, so offsetting the overall delta wouldn't work out in the end.  Oh well, I'll figure something out.
Also, I really appreciate the suggestions, I'll be implementing those shortly, should help clean up my code a bit.
Thanks for all your support.  Really appreciate it.


First draft of combination indicator documentation

Based on material from Pratik and Addison.
Dev preview: http://rosmarus.refsmmat.com/delphi-epidata/api/covidcast-signals/indicator-combination.html

Bootstrap section LGTM.


Add ability to prevent broker from taking leadership for any partition

Additional tests to validate PartitionLeaderSelector and PartitionStateMachine changes would be nice.


JsonConvert.SerializeObject is crashing in winodws xp sp3

I am using below code to serialize the object using Json .
jsonObject = JsonConvert.SerializeObject(NodeConfigurator.Instance, Formatting.None, new JsonSerializerSettings()
{
ReferenceLoopHandling = ReferenceLoopHandling.Ignore
});
But in windows xp sp3 it is crashing & giving below exception.
"An unhandled exception of type 'System.NotSupportedException' occurred in Newtonsoft.Json.dll". The exception message is "'CommandConverter' is unable to convert 'abc.RelayCommand' to 'System.String'". The InnerException is null.
But in windows 7 & above it is working correctly. If the structure is simple it is working good but if it is complected it is crashing. My json in windows 7 is like below & in windows xp it is crashing
{"LastSavedPath":"C:\Program Files (x86)\abc\uOPC Server\Lastclosed.uopf","Channels":[{"NewDeviceCommand":{},"DeleteChannelCommand":{},"ChannelDoubleClickCommand":{},"Device":[{"NewTagCommand":{},"DeleteDevice":{},"IsNewTagCommandVisible":true,"IsDeleteDeviceVisible":true,"IsNewDeviceCommandVisible":true,"Tags":[{"IsEnableApplyBtn":false,"TagName":"Tag1","TagAddress":"102364","TagNameimg":null,"TagRightImg":null,"TagDescription":"New","DataType":"Bool","ClientAccess":"Read Only","ScanRate":0,"NewTag":null,"NewGrpTag":null,"DeleteCurGrpTag":null,"MultiTagGen":null,"ScalingNone":true,"ScalingLinear":false,"ScalingSquareRoot":false,"RVRHigh":0,"RVRLow":0,"SVRData":"Double","SVRHigh":0,"SVRHighClamp":false,"SVRLow":0,"SVRLowClamp":true,"SVRUnits":null,"NegateScalVal":true,"Location":"DiscreteInputs","Address":"02364","Length":1},{"IsEnableApplyBtn":false,"TagName":"Tag2","TagAddress":"102365","TagNameimg":null,"TagRightImg":null,"TagDescription":"dfdfdf","DataType":"Bool","ClientAccess":"Read Only","ScanRate":0,"NewTag":null,"NewGrpTag":null,"DeleteCurGrpTag":null,"MultiTagGen":null,"ScalingNone":true,"ScalingLinear":false,"ScalingSquareRoot":false,"RVRHigh":0,"RVRLow":0,"SVRData":"Double","SVRHigh":0,"SVRHighClamp":false,"SVRLow":0,"SVRLowClamp":true,"SVRUnits":null,"NegateScalVal":true,"Location":"DiscreteInputs","Address":"02365","Length":1}],"DeviceModel":"Modbus","DeviceName":"Device1","AsciiDeviceID":"5","AsciiDeviceIDType":"Decimal","EtherNetDeviceID":null,"ScanMode":"","ConnectionTimeout":3,"ScanRate":0,"IsAddedFirstDevice":false,"RequestTimeout":1000,"FailAfterSuccTimeout":3,"InterReqDelay":0,"EnableAutoDeviceDemotion":false,"DemotAftSuccFailure":0,"DemotForMS":0,"DiscardWriteReq":true,"StartUp":"Do not generate on startup","Action":"Do not overwrite, log error","PortNumber":"502","IPProtocol":"","AddtoGroup":0,"AllowAutoGenSubGrp":true,"UseZeroBasAdd":true,"UseZeroBitAddReg":false,"UseHoldRegBitMask":false,"UseModFun06RegWrite":false,"SingleRegisterWrites":"0","UseModFun05CoilWrite":false,"singleCoilWrites":"0","ClientPrivilege":"Read/Write","ModbusDeviceBaseAddress":"1","DefaultModByteOrd":true,"FirstWrdLowin32BitDT":true,"firstWordLowIn32Bit":"1","ByteOrder":"Big-endian","FirstDWrdLowin64bitDT":false,"firstDwordLowIn64Bit":"0","ModiconbitOrd":true,"LongasDblePrsUD":false,"CoilOutput":32,"CoilInput":32,"RegInternal":32,"RegHolding":32,"PerBlockRead":false,"CloseTCPSocTimeout":false,"VariableImport":null,"VariableImportBTN":null,"IncludeDescription":true,"DeactivateTagIllAddrExe":true,"RejectRepMsg":false,"UseModTCPFrame":true,"LeadingByte":0,"TrailingByte":0,"ImportFile":null,"ImportFileBTN":null,"SelectObjs":null,"FilterOptProp":false,"CreateTagasRWAllowed":false,"USeObjNameTagGrpName":false,"BitsperRegister":"16","IsModBusTcpIpSelected":false,"IsScanModeTextBoxEnable":true,"IsRTUSelected":true,"IsSelected":true}],"PageVisite":false,"ChannelName":"Channel1","DeviceDriver":"Modbus ASCII Serial","EnableDiagnostic":true,"VirtualNetwork":"None","TransactionsPerCycle":0,"NetworkMode":"Load Balanced","PhysicalMedium":"COM Port","ReadProcessing":"Ignore","ComID":"6","BaudRate":"9600","Databit":"7","Parity":"Even","Stopbit":"1","FlowControl":"None","NetworkAdapter":"Default","ReportConnErr":true,"reportConnectionErrors":"1","CloseConnWnIdle":true,"closeConnectionWhenIdle":"1","SecofIdleTime":0,"WriteAllTags":false,"LatesValNonboolTags":false,"LatesvalAllTags":true,"DutyCycle":0,"NonNormalizedValue":"","SingleSocket":true,"MoreSocket":false,"MaxSocketsperDevice":0,"PortNumber":"","IPProtocol":"","SerializePortsetting":true,"OperationCommunications":false,"EtherNetSettings":false,"IsTagListViewVisible":true,"IsChannelListViewVisible":true,"ChannelSelectedItem":null,"IsChannelSelected":false}]}

Ask on stackoverflow


One action in two sections with same route values.

I my project, i have structure like this:
<mvcSiteMapNode controller="Home" action="Index">
    <mvcSiteMapNode controller="Home" action="Table1">
        <mvcSiteMapNode controller="Process" action="Show" typeCode="Type1" preservedRouteParameters="id" />
    </mvcSiteMapNode>

    <mvcSiteMapNode controller="Home" action="Table2">
        <mvcSiteMapNode controller="Process" action="Show" typeCode="Type2" preservedRouteParameters="id" />
    </mvcSiteMapNode>
</mvcSiteMapNode>

But the problem is that there is no route value named 'typeCode', this code must be loaded from database. How to make breadcrumbs work?

Sorry for the late reply.
If there is no route value named 'typeCode' configured, the URLs will be produced with typeCode appended to the URL as a query string.
/Process/Show/123?typeCode=Type1
/Process/Show/123?typeCode=Type2

However, creating nodes in XML similar to crating ActionLinks in a view. They are, for the most part, static.
If you want your nodes to have database driven values, then you will need to use a dynamic node provider. Have a look at this article.

No, i have no typeCode variable neither in route values dictionary nor in query string parameters.
I have urls like /Process/Show/123 - and i need to make tree like this:
/Home/Table1
/Process/Show/123
/Process/Show/125
/Home/Table2
/Process/Show/124
I cannot use dynamic node provider because process list changes too frequently.

Well, if what you are doing is trying to track some CRUD operations, it is better to use preservedRouteParameters anyway.
But in no way does that mean you can't use dynamic node provider if you wanted to keep up with fast changing data. You would just need to use [SiteMapCacheRelease] attribute on every action method that changes the data (assuming your MVC application has full control over the changes).
You have configured it correctly in MvcSiteMapProvider (you don't have 2 nodes that are exactly the same), but keep in mind that MvcSiteMapProvider cannot create the URLs when using preservedRouteParameters - they need to be built from your views. Typically, when browsing database driven items, you would already have them displayed in a list or table, anyway.
But you need to account for the extra parameter when building the hyperlinks to get to the pages. If typeCode has no meaning to you, I would recommend something more meaningful, like parentAction. Or stick with typeCode - it will work the same way.
<mvcSiteMapNode title="Home" controller="Home" action="Index">
    <mvcSiteMapNode title="Table1" controller="Home" action="Table1">
        <mvcSiteMapNode title="dynamic" controller="Process" action="Show" parentAction="Table1" preservedRouteParameters="id" />
    </mvcSiteMapNode>

    <mvcSiteMapNode title="Table2" controller="Home" action="Table2">
        <mvcSiteMapNode title="dynamic" controller="Process" action="Show" parentAction="Table2" preservedRouteParameters="id" />
    </mvcSiteMapNode>
</mvcSiteMapNode>

parentAction doesn't have to mean anything to your application. Its only purpose is to make the URL unique so it will know to match the node below "Table2" when the parentAction is "Table2". It might be better if you adjusted your routes to make it part of the URL instead of a query string, though.
Then when you build the URLs in your view, you need to add the extra parameter to the ActionLink (or RouteLink) when it is displayed in your view.
@Html.ActionLink("123", "Show", "Process", new { id=123, parentAction="Table1" }, null)
@Html.ActionLink("125", "Show", "Process", new { id=125, parentAction="Table1" }, null)

@Html.ActionLink("124", "Show", "Process", new { id=124, parentAction="Table2" }, null)

You need one more thing, though - to fix the title of the node. Since you are using the same node for every "Show" node, you need to update it dynamically according to the current page being viewed. This can be accomplished with the SiteMapTitle attribute. This example just sets the title to the value of "id", but you could set it to anything that makes sense - the value could even come from your database.
public class ProcessController
{
    [SiteMapTitle("id")]
    public ActionResult Show(int id)
    {
        return View();
    }
}

When clicking the first link, your breadcrumb will be:
Home > Table1 > 123

And the last link will be:
Home > Table2 > 124

I found another way (node keys and explicit node selection logic) - but i want do it right way, without any hacks or extra URL parts.

PS It is not a question, it is a feature request

It is a core limitation of Microsoft's design (which this is based on) that the request must be unique in order to identify which node it matches. The nodes are compared from the top of the SiteMap down to the bottom and the first match always wins.
Since your URLs are unique without using preservedRouteParameters, you can use a dynamic node provider (or ISiteMapNodeProvider) to get exactly the URL scheme you want with absolutely no hacks (assuming you don't put the same node under both Table1 and Table2, anyway). I really don't understand your objection to this unless you have a SiteMap that changes every few seconds.
Using preservedRouteParameters is another option that I once viewed pretty harshly, but is actually a more efficient way to get the job done, especially if you have an enormous number of records or they change rapidly. The problem you have is that when you combine this technique with nesting the same controller and action under separate parent nodes, you fall into a situation where your route combinations are not unique and you have to follow the procedure from Mutiple Navigation Paths to a Single Page.
Do note there are many different ways to make a URL unique. I just use the query string example because it is the simplest to implement. You could also potentially return the results of one action method from another:
public class ProcessController
{
    public ActionResult ShowTable1(int id)
    {
        return View();
    }

    public ActionResult ShowTable2(int id)
    {
        return ShowTable1(id);
    }
}

Or, as I mentioned, you can add a route to make the URLs more eye appealing and search engine friendly:
public class RouteConfig
{
	public static void RegisterRoutes(RouteCollection routes)
	{
		routes.IgnoreRoute("{resource}.axd/{*pathInfo}");

		routes.MapRoute(
			name: "ProcessWithParent",
			url: "Home/{parentAction}/Process/{action}/{id}/",
			defaults: new { controller = "Process", action = "Index", id = UrlParameter.Optional }
		);

		routes.MapRoute(
			name: "Default",
			url: "{controller}/{action}/{id}",
			defaults: new { controller = "Home", action = "Index", id = UrlParameter.Optional }
		);
	}
}

Which would make your URL structure:

/Home/Table1
/Home/Table1/Process/Show/123
/Home/Table1/Process/Show/125
/Home/Table2
/Home/Table2/Process/Show/124

This also fits in line with the canonical tag that the search engines use, because even they recognize that it is not possible to always host content on a single URL (likely for this very reason).
I spent a lot of time and research contemplating how to add this feature because I required it personally, including Microsoft connect answers to this very problem, and the consensus is that making the URL unique is far preferable to the alternative, which would be to use session state or a cookie to track which parent node the current user navigated from. There are at least 2 problems with that approach:

It requires session state or a cookie to function.
Navigation falls apart if the user doesn't follow a path that you expect, such as navigating directly to the URL without first going to the parent page.

It might work sometimes, but not 100% of the time. It can't work 100% of the time regardless of how good you make it because there is no way to completely control how a user navigates a website.
I am not seeing any other options to accommodate your feature request. If you have any ideas about how this could be implemented, or if you are fine with a feature that will work most of the time but not all of the time, please let me know.


Missing how to tackle a large code base

I am suggesting to add a section about how to tackle a large code base in an open source project .
I see that is pretty common that someone doesn't know how to tackle a large code base. Besides reading the documentation (which might be outdated sometimes), they might not be aware of some tools to use (like grep or source explorer software), aren't like an user expert of the software before hand or maybe don't try to create some tests to see if they can grasp how it works.

Sorry, we're not adding new guides at this time.


Gray (and no red) could shown in Warning section, if no warnings

Is your feature request related to a problem? Please describe.
Now red color on the report output, even if no warnings or errors
Describe the solution you'd like
When no warnings, the colors in that section could be green or gray

这是来自foxmail邮箱的自动回复邮件。      您好,您的邮件已经收到，我查阅后，尽快给您回复。

Hey Repo owner,
I want to work on this issue, with your permission.
Hoping to hear soon!

@rajSaheel, sure you're welcome contribute a pr : )

closing this as won't fix.

this is subjective, but having the warning-section colored gray or green, feels "weird" to me

i purposely want users to be aware of the big red warning-section, even when their code is free of jslint-warnings.


docs: add links in Configuring TypeScript section

This is good, but what I wanted to do is link to sections in this page from other parts of the documentations, so for example, if someone is using useNavigation, they have a link to check how to setup TypeScript for that.


Samsung Galaxy A23 wrong scan area

Mobile scanner 3.2.0
Device Samsung Galaxy A23 (tested about 4 device with same result)

It seems that the detection area is located at the upper right screen edge and that it is very small, infact to detect codes i have to put the camera backward enough to reduce the framed code size and move it to the upper right screen edge.
InkedScreenshot_20230422_095609_DropPay Stage

Just tested using the scanWindow attribute but seems to be ignored.

Fixed uncommenting this line in MobileScanner.kt inside method start():
analysisBuilder.setTargetResolution(Size(1440, 1920))

I've tried out many resolutions and it always do the job, so i guess, for certain devices, target resolution for analysis builder must be set.

This one solves the problem without any aspect ratio issue, no need to set target resolution anymore:
            val camProfile = CamcorderProfile.get(CamcorderProfile.QUALITY_1080P)
            analysisBuilder.setDefaultResolution(Size(camProfile.videoFrameWidth,
                camProfile.videoFrameHeight))

I don't know if this is the right solution but in my case i've tested many devices and everything seems working as aspected.

Did you tested on ios device?

Thank you bro, you save the day

Hello,
any news on this? When this issue will be resolved? Having the same symptoms reading QR code on A23.

I am working on an updated version of #612 which should fix this issue

Any one test version 3.5.1

Tested on 4.0.0 and the issue is still there

Issue is easily resolved setting cameraResolution in MobileScannerController to  some reasonable value , for example 1024, 768

Yes, but... still an issue. Also setting the resolution has other implications

Hi,
Any update here?

The author of this plugin already managed to solve this issue, just set a 4:3 cameraResolution when constructing MobileScannerController.
    final _scannerController = MobileScannerController(
      // 24MPix with an aspect of 4:3
      cameraResolution: const Size(6080, 8096),
    );

Feel free to set a higher resolution, if not available it will be set using the nearest one.
Hope this will be useful!


automation: Fix syntax of 'release_branches'

'release_branches' were not properly specified, so builds were not
properly pushed into the 4.2 change-queue.

@sandrobonazzola , @dron1 please note, we need this merged to properly release builds to 4.2

This issue is also in other ansible repos, I will fix them.

@ifireball is there a way the DSL parser can detect it and fail the patch if the syntax isn't correct?

@eedri no, there is no way to detect this since this is essentially specifying the same key twice in a dictionary, the YAML parser just picks one at random and the code has no way to tell that  this had happened.


Add hgemm functions for half precision data

Complication: the hgemm functions are wrapped in #if defined(__cplusplus), and bindgen struggles to generate bindings if i manually define that


PARQUET-2115: [C++] Parquet dictionary bit widths are limited to 32 bits

Specification says that maximum bit width is 32bits:
https://github.com/apache/parquet-format/blob/master/Encodings.md#dictionary-encoding-plain_dictionary--2-and-rle_dictionary--8
Fuzzer created a file with a bit width of 35bits, which causes CHECK to
fail during RLE decode.

@tachyonwill Very nice! Which fuzzer is this, by the way?

@tachyonwill Very nice! Which fuzzer is this, by the way?

It's more or less the same stack that powers OSS-Fuzz but run internally. One difference is that we are not going through the arrow table interface(parquet::arrow::internal::FuzzReader) but we have our own wrapper around ParquetFileReader/TypedColumnReader::ReadBatch. We are also investigating on whether we can apply other fuzzing techniques, like structured fuzzing, but that is still in the early stages.

Maybe I am not understanding submodules workflows correctly, but do we need to merge the PR in the submodule https://github.com/apache/arrow-testing/pull/73 ?

Maybe I am not understanding submodules workflows correctly, but do we need to merge the PR in the submodule apache/arrow-testing#73 ?

Wow. Well, I must admit I'm not sure what Github is doing here... but I definitely get the new files in the submodule here, I can only assume that somehow the changeset reference exists regardless of whether the PR is merged or not?


Add hints on how to configure the components of zeebe-full-helm

Awesome :heart: I had a similar thought yesterday that it is not really visible what to put in their. I often check then https://github.com/camunda-community-hub/zeebe-helm-profiles/blob/master/zeebe-dev-profile.yaml
TIL: Wasn't aware of the show values command cool :+1:


TODO: Update build image functionality to show some kind of progress

Use SSE or long-polling or something to check if the image build is finished or running

I think I'll not add this for now


Implement CLI for segment nuclei and generate annotations for them

@cooperlab I want to start working on this issue. Can you point me and @slee172 to the function to use for nuclei segmentation?
Also which of the two color deconvolution (standard/sparse) was working better so i could use that?

@cdeepakroy use the standard color deconvolution.
The function is really a sequence: normalize color -> deconvolve to hematoxylin (H) & eosin (E) -> mask -> cLoG response -> max clustering -> feature extraction.
I will write an example script if I get some free time tomorrow.

This is fixed by PRs #94 and #102.


open to a PR to de-crustify this?

@subprotocol would you accept a PR that makes this thing easier to run out-of-the-box? When I npm install and build things locally I get some errors and the examples are all non-functional.
Happy to simplify things and get the default state running again if you're open to that.

Hey @mreinstein  I started working on a typescript version of this library since this one hasn't received a commit in 10 years and seems a little abandoned.
In case you are interested:
https://github.com/JonasGruenwald/verlet-ts
It's incomplete as of now though.
Really appreciate how straightforward this original library is though, very easy to understand, I enjoyed adapting it.


[WIP] DBZ-2879 Redis Streams target in Debezium Server

Could anyone please help me to finalize this PR? Thanks.

Could anyone please help me to finalize this PR? Thanks.

Thanks a lot, @sazzad16! Going to look into this PR asap.

Thanks a lot, @sazzad16! Going to look into this PR asap.

@gunnarmorling thanks!

Ha, let's rather thank @sazzad16 for this contribution!
@sazzad16, as you mark this as PoC, what's missing from your perspective to complete it?

@gunnarmorling testing!

@gunnarmorling

From debezium-server README, it seems that tests are executed against a remote instance. I wasn't sure if automated tests are also done in such way. If this is the case, we can set up a free Redis Enterprise account for this purpose.

Docker images are also available for Redis. If you prefer, we can do the tests against a docker image as well.

I wasn't sure about the contract of BaseChangeConsumer, e.g. whether the implementation should be thread-safe or not. Current implementation of RedisStreamChangeConsumer is not thread-safe.
Current PR supports only standalone Redis instance. But Redis also has a cluster mode. If we finalize the implementation and testing for standalone instance, I can then extend this PR to support both version.

@sazzad16, my hope would be this could be tested locally, i.e. no need for any sort of account. Can't Redis Streams be started up via Docker / Testcontainers, as we do for Apache Pulsar for instance?

@gunnarmorling (Though I don't know how Apache Pulsar works) Redis Streams is readily available in any Redis 5.0+. If Redis instance is running, you don't have to start Stream separately.

https://github.com/debezium/debezium/blob/master/debezium-server/debezium-server-pulsar/src/test/java/io/debezium/server/pulsar/PulsarIT.java#L38

Do you mean this?
https://github.com/debezium/debezium/blob/7e34c918e205f75d926a76dc885f84ab6ed829be/debezium-server/debezium-server-pulsar/src/test/java/io/debezium/server/pulsar/PulsarTestResourceLifecycleManager.java#L26-L31
Starting the docker image from code?

Yes, exactly.

I haven't done something like this before. But you're here right? :)

@gunnarmorling PTAL

Hey, sorry for the relative radio silence here. We're planning to revisit this and properly review for Debezium 1.6. In the meantime, could you rebase this PR and address the merge conflict in the one pom.xml.

@gunnarmorling

could you rebase this PR and address the merge conflict in the one pom.xml.

Done.

@jpechane

I've left one comment related to topinc name mapping.

Done.

Also in the meantime we've introduced BOM for Debezium Server st the pom.xml change will need some updates.

Done.

you can start working on the documentation.

Where do I add these?
I can only find https://github.com/debezium/debezium/blob/master/debezium-server/README.md. But this PR uses docker image similar to Pulsar server which has nothing to setup. Please guide me where and what to add.

The file https://github.com/debezium/debezium/blob/master/documentation/modules/ROOT/pages/operations/debezium-server.adoc needs to be updated.
Please check https://github.com/debezium/debezium.github.io/blob/develop/CONTRIBUTING.md#build-and-make-changes for how to build and verify the docs locally.

@sazzad16 And applied, thanks a lot, nice work!

@jpechane Thank you very much for all the guidelines and finally merging this PR.


Bug with .crca and global custom reaction

Navi King - 03/06/2018

There's a bug wherein if .crca is enabled for a global custom reaction you can't overwrite it at the server level.

This may not be a bug so much as it is an intended behavior, according to Nadeko Issue 1886
If you know .crca is enabled for the global custom reaction, temporarily disable it, make your change, and then re-enable it.

The problem was that as soon as I reenabled it, it stopped being blocked.
So it was like

Disable .crca
Now the custom reaction can be overwritten at the server level
Reenable .crca
The server custom reaction will still overwrite the global one if the
global one is used by itself, but when used as part of a larger message
still triggers the original global custom reaction

Sincerely,
Sean McNulty
On Wed, Mar 14, 2018 at 5:20 PM, Natalie Diebold<EMAIL_ADDRESS>wrote:

Navi King - 03/06/2018
There's a bug wherein if .crca is enabled for a global custom reaction you
can't overwrite it at the server level.
This may not be a bug so much as it is an intended behavior, according to Nadeko
Issue 1886 https://github.com/Kwoth/NadekoBot/issues/1886
If you know .crca is enabled for the global custom reaction, temporarily
disable it, make your change, and then re-enable it.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/NetworkFrontier/LinBot/issues/17, or mute the thread
https://github.com/notifications/unsubscribe-auth/ARCTRfangygMBy8f3SbEsDOw8jaBaJheks5teYm7gaJpZM4SrMaQ
.

I see, so with an example:

Create global custom reaction MyReaction Original (1)
Enable .crca 1 (MyReaction Original)
Disable .crca 1 (MyReaction Original)
Overwrite MyReaction Original to MyReaction Overwritten at server level (2)
Enable .crca 1 (MyReaction Original)

What you described as being a problem is the intended behavior. The Original (1) will appear in sentences (according to .crca) while the Overwritten (2) will appear in the source server's custom reactions.
If you want MyReaction Overwritten to appear in sentences in the source server, wouldn't you just need to do .crca 2?
NOTE: I say "overwritten" but in reality the Original (1) and Overwritten (2) are separate records in the database and do not interfere with each other. Overwritten (2) will only be used in its source server, with its own .crca settings. Original (1) will be used globally according to its own .crca settings.

Unless you're saying that you want neither Original (1) nor Overwritten (2) to appear in sentences in 2's source server, in which case we'd some way of telling Nadeko to ignore a specific global reaction if its trigger matches a server reaction.

So I guess the crux of the problem is that, as far as I know, no one can
overwrite a .crca reaction at server level. Essentially they just create a
new custom reaction with the message specific trigger that would overwrite
a global message specific trigger, but will never overwrite a crca trigger.
What I would like to happen then is a way for users to disable a .crca
enabled reaction at the server level, either by being able to toggle it
with .crca (or a custom equivalent so we don't interfere with Kwoth's) or
being able to overwrite it with .acr somehow.
Sincerely,
Sean McNulty
On Wed, Mar 14, 2018 at 6:10 PM, Natalie Diebold<EMAIL_ADDRESS>wrote:

I see, so with an example:

Create global custom reaction MyReaction Original (1)
Enable .crca 1 (MyReaction Original)
Disable .crca 1 (MyReaction Original)
Overwrite MyReaction Original to MyReaction Overwritten at server
level (2)
Enable .crca 1 (MyReaction Original)

What you described as being a problem is the intended behavior. The
Original (1) will appear in sentences (according to .crca) while the
Overwritten (2) will appear in the source server's custom reactions.
If you want MyReaction Overwritten to appear in sentences in the source
server, wouldn't you just need to do .crca 2?
NOTE: I say "overwritten" but in reality the Original (1) and Overwritten
(2) are separate records in the database and do not interfere with each
other. Overwritten (2) will only be used in its source server, with its own
.crca settings. Original (1) will be used globally according to its own
.crca settings.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/NetworkFrontier/LinBot/issues/17#issuecomment-373191854,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARCTRQ3AlcNCO3pcxz7-Xb-kiN_KbpyOks5teZVIgaJpZM4SrMaQ
.

Well you can half-overwrite a .crca trigger. You can overwrite by turning the server reaction's .crca on, but you can't turn global .crca off at server level.
It looks like whatever Kwoth did in Nadeko Issue 1549 might have affected your expected behavior.

Yeah, turning global .crca off at a server level was something I was
wanting/expecting to happen.
Sincerely,
Sean McNulty
On Wed, Mar 14, 2018 at 6:27 PM, Natalie Diebold<EMAIL_ADDRESS>wrote:

Well you can half-overwrite a .crca trigger. You can overwrite by turning
the server reaction's .crca on, but you can't turn global .crca off at
server level.
It looks like whatever Kwoth did in Nadeko Issue 1549
https://github.com/Kwoth/NadekoBot/issues/1549 might have affected your
expected behavior.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/NetworkFrontier/LinBot/issues/17#issuecomment-373196022,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARCTRWmmrkZGPq6cccdohQh6snkrXHseks5teZlggaJpZM4SrMaQ
.

Actually it seems this behavior was never supported. The CustomReactionService first checks for a match in server reactions, and if none match, it checks for a match in global reactions. There is no true blocking of a global reaction, just priority for messages matching server reactions.
I will submit this as an issue to Nadeko for now.

Now being tracked as Nadeko Issue 2172


Upgrade OpenVPN to 2.5.x for ChaCha20 support

OpenVPN 2.5 has been released officially and includes support for the ChaCha20 cipher. This is great for low power devices who don't have AES acceleration built in. As an example, my raspberry pi zero can encrypt 45MB/s with chacha20 and 10 MB/s with aes-128-gcm. Sadly, on things like my asus routers, wireguard is not an option, only openvpn so this is quite a boost in speed for them. openssl is already shipping in opnsense with chacha support, so it should be a fairly simple update.

it's in fresh ports now: https://www.freshports.org/security/openvpn

@beren12 2.5.3 does not list ChaCha20 in supported ciphers with LibreSSL, other than the new OpenVPN will be ready with 21.1.8.

Great as long as it works with open SSL

Looks like a larger issue: https://github.com/libressl-portable/portable/issues/483
There is no reason we should keep this open since we don't know how long it will take.


Ruby on OSX: trivial RpcServer shutdown leaks tcp-server-listener

OS: Mac OS X (10.11.5)
Ruby version: 2.2.5
gRPC version: 0.15.0
Note: the repro intentionally fails to register a service with the RPC server simply for the sake of a simplified repro, but even if a service is registered, the same wait-delay and leak is experienced when SIGINT is received by the process.
Repro:
require 'grpc'

puts "gRPC version: #{GRPC::VERSION}"
server = GRPC::RpcServer.new
server.add_http2_port('<IP_ADDRESS>:3000', :this_port_is_insecure)
server.run_till_terminated

Output (stdout and stderr):
gRPC version: 0.15.0
D0701 15:37:37.892394000<PHONE_NUMBER>22336 ev_posix.c:101] Using polling engine: poll
/Users/peterhuene/.rbenv/versions/2.2.5/lib/ruby/gems/2.2.0/gems/grpc-0.15.0-universal-darwin/src/ruby/lib/grpc/generic/rpc_server.rb:340:in `block in run': cannot run without registering services (RuntimeError)
	from /Users/peterhuene/.rbenv/versions/2.2.5/lib/ruby/gems/2.2.0/gems/grpc-0.15.0-universal-darwin/src/ruby/lib/grpc/generic/rpc_server.rb:339:in `synchronize'
	from /Users/peterhuene/.rbenv/versions/2.2.5/lib/ruby/gems/2.2.0/gems/grpc-0.15.0-universal-darwin/src/ruby/lib/grpc/generic/rpc_server.rb:339:in `run'
	from repro.rb:6:in `<main>'
D0701 15:37:38.932565000<PHONE_NUMBER>22336 iomgr.c:99] Waiting for 1 iomgr objects to be destroyed
D0701 15:37:39.952944000<PHONE_NUMBER>22336 iomgr.c:99] Waiting for 1 iomgr objects to be destroyed
D0701 15:37:40.983342000<PHONE_NUMBER>22336 iomgr.c:99] Waiting for 1 iomgr objects to be destroyed
D0701 15:37:42.010067000<PHONE_NUMBER>22336 iomgr.c:99] Waiting for 1 iomgr objects to be destroyed
D0701 15:37:43.045996000<PHONE_NUMBER>22336 iomgr.c:99] Waiting for 1 iomgr objects to be destroyed
D0701 15:37:44.074138000<PHONE_NUMBER>22336 iomgr.c:99] Waiting for 1 iomgr objects to be destroyed
D0701 15:37:45.096166000<PHONE_NUMBER>22336 iomgr.c:99] Waiting for 1 iomgr objects to be destroyed
D0701 15:37:46.124101000<PHONE_NUMBER>22336 iomgr.c:99] Waiting for 1 iomgr objects to be destroyed
D0701 15:37:47.147566000<PHONE_NUMBER>22336 iomgr.c:99] Waiting for 1 iomgr objects to be destroyed
D0701 15:37:47.974508000<PHONE_NUMBER>22336 iomgr.c:118] Failed to free 1 iomgr objects before shutdown deadline: memory leaks are likely
D0701 15:37:47.974544000<PHONE_NUMBER>22336 iomgr.c:80] LEAKED OBJECT: tcp-server-listener:(sockaddr family=157) fd=9 0x7fd35ad43670

This appears fixed in 1.0.0.  Closing for now.


[Question] How can I build a desktop app instead of UMP

I want a portable executable file, an ordinary Win32 program.

Off topic. Consider taking a look at https://docs.microsoft.com/en-us/windows/apps/desktop/ for a list of options for building a Win32 app.

I want a portable executable file, an ordinary Win32 program.

You can't build an UWP app as a portable executable file.


Incompatibility with Hot Chocolate v11.0.8

I updated Hot Chocolate to version the newest version today "11.0.8", and Fairy Bread seems to have problems with this version.
I have the following 3 packages installed, alongside Fairy Bread 3.0.0 of course:

HotChocolate.AspNetCore (v11.0.8)
HotChocolate.AspNetCore.Authorization (v11.0.8)
HotChocolate.Data (v11.0.8)

The exception that's thrown:

Method not found: 'System.Collections.Generic.IReadOnlyDictionary`2<System.String,System.Object> HotChocolate.Types.IField.get_ContextData()'.

Stacktrace:

at FairyBread.DefaultFairyBreadOptions.DefaultImplementations.ShouldValidateBasedOnValidateDescriptor(IMiddlewareContext context, IInputField argument)\r\n   at FairyBread.DefaultFairyBreadOptions.DefaultImplementations.ShouldValidate(IMiddlewareContext context, IInputField argument)\r\n   at FairyBread.InputValidationMiddleware.InvokeAsync(IMiddlewareContext context)\r\n   at HotChocolate.Utilities.MiddlewareCompiler`1.ExpressionHelper.AwaitTaskHelper(Task task)\r\n   at HotChocolate.Execution.Processing.ResolverTask.ExecuteResolverPipelineAsync(CancellationToken cancellationToken)\r\n   at HotChocolate.Execution.Processing.ResolverTask.TryExecuteAsync(CancellationToken cancellationToken)

I had to go back to v11.0.0 temporarily, I'd appreciate it if this is addressed pretty soon. Thanks!

Hey @AradAral , thanks for reporting. I'm having a hard time reproducing this. It would seem from the stack trace that it's failing when it tries to do argument.ContextData, but after bumping to 11.0.8, that property still exists and all tests still pass.
Are you able to produce a small test that reproduces it? Thanks!

Hey @AradAral , thanks for reporting. I'm having a hard time reproducing this. It would seem from the stack trace that it's failing when it tries to do argument.ContextData, but after bumping to 11.0.8, that property still exists and all tests still pass.
Are you able to produce a small test that reproduces it? Thanks!

Tried 11.0.9-preview.3 too, seems ok.

Tried 11.0.9-preview.3 too, seems ok.

Tried 11.1.0-preview.8 too, seems ok.

Tried 11.1.0-preview.8 too, seems ok.

I can't seem to reproduce this again either! 😄
I have no idea what was going on at the time. Seems to be working well right now.
Thank you.

I can't seem to reproduce this again either! 😄
I have no idea what was going on at the time. Seems to be working well right now.
Thank you.

I get this error when I try to run a QUERY and not a mutation. When running a mutation it seems to work fine.
Using Hot Chocholate 11.0.9 and latest Fairy Bread 4.1.0.
Query:
{
  controller(id:"iotEdge-A"){
    chargePoints{clusterId}
  }
}

Response
[
  {
    "message": "System.MissingMethodException: Method not found: 'System.Collections.Generic.IReadOnlyDictionary`2<System.String,System.Object> HotChocolate.Types.IField.get_ContextData()'.\r\n   at FairyBread.DefaultFairyBreadOptions.DefaultImplementations.ShouldValidateBasedOnValidateDescriptor(IMiddlewareContext context, IInputField argument)\r\n   at FairyBread.DefaultFairyBreadOptions.DefaultImplementations.ShouldValidate(IMiddlewareContext context, IInputField argument)\r\n   at FairyBread.InputValidationMiddleware.InvokeAsync(IMiddlewareContext context)\r\n   at HotChocolate.Utilities.MiddlewareCompiler`1.ExpressionHelper.AwaitTaskHelper(Task task)\r\n   at HotChocolate.Execution.Processing.ResolverTask.ExecuteResolverPipelineAsync(CancellationToken cancellationToken)\r\n   at HotChocolate.Execution.Processing.ResolverTask.TryExecuteAsync(CancellationToken cancellationToken)",
    "locations": [
      {
        "line": 2,
        "column": 3
      }
    ],
    "path": [
      "controller"
    ],
    "extensions": {
      "message": "Method not found: 'System.Collections.Generic.IReadOnlyDictionary`2<System.String,System.Object> HotChocolate.Types.IField.get_ContextData()'.",
      "stackTrace": "   at FairyBread.DefaultFairyBreadOptions.DefaultImplementations.ShouldValidateBasedOnValidateDescriptor(IMiddlewareContext context, IInputField argument)\r\n   at FairyBread.DefaultFairyBreadOptions.DefaultImplementations.ShouldValidate(IMiddlewareContext context, IInputField argument)\r\n   at FairyBread.InputValidationMiddleware.InvokeAsync(IMiddlewareContext context)\r\n   at HotChocolate.Utilities.MiddlewareCompiler`1.ExpressionHelper.AwaitTaskHelper(Task task)\r\n   at HotChocolate.Execution.Processing.ResolverTask.ExecuteResolverPipelineAsync(CancellationToken cancellationToken)\r\n   at HotChocolate.Execution.Processing.ResolverTask.TryExecuteAsync(CancellationToken cancellationToken)"
    }
  }
]

@JardarM let's continue this on #37 , seems a tricky one to figure out.


TypeScript type declaration file

If you want I can write a TypeScript type declaration file for this package so that TS developers can utilise the type-checking feature of the language as they use this package. Let me know if you are interested.

I for one would appreciate this!


Use ENV key=value in Dockerfile

This will fix workflow failure
#21 ERROR: failed to push ***/bskyweb:ac9d910e1e77c559eff8b32cd8412335f41074f1: failed commit on ref "manifest-sha256:fd35dcfb18dd12d130322733b0d01dbc209a292dfb4941f2e90a8074c24e774f": unexpected status from PUT request to https://***/v2/bskyweb/manifests/sha256:fd35dcfb18dd12d130322733b0d01dbc209a292dfb4941f2e90a8074c24e774f: 403 Forbidden
------
 > exporting to image:
------

 1 warning found (use docker --debug to expand):
 - LegacyKeyValueFormat: "ENV key=value" should be used instead of legacy "ENV key value" format (line 20)
ERROR: failed to solve: failed to push ***/bskyweb:ac9d910e1e77c559eff8b32cd8412335f41074f1: failed commit on ref "manifest-sha256:fd35dcfb18dd12d130322733b0d01dbc209a292dfb4941f2e90a8074c24e774f": unexpected status from PUT request to https://***/v2/bskyweb/manifests/sha256:fd35dcfb18dd12d130322733b0d01dbc209a292dfb4941f2e90a8074c24e774f: 403 Forbidden
Error: buildx failed with: ERROR: failed to solve: failed to push ***/bskyweb:ac9d910e1e77c559eff8b32cd8412335f41074f1: failed commit on ref "manifest-sha256:fd35dcfb18dd12d130322733b0d01dbc209a292dfb4941f2e90a8074c24e774f": unexpected status from PUT request to https://***/v2/bskyweb/manifests/sha256:fd35dcfb18dd12d130322733b0d01dbc209a292dfb4941f2e90a8074c24e774f: 403 Forbidden

Example: https://github.com/bluesky-social/social-app/actions/runs/11653248662/job/32445429993#step:7:1252

The workflow failure is no longer happening because of this specific error
https://github.com/bluesky-social/social-app/actions/workflows/build-and-push-bskyweb-aws.yaml
But it's'= better to fix it, in case it happens again.


Enrage, Adrenaline Rush, Guardian Shout not showing in cooldown bar

Pretty much what title said, I dont like the class window  so Id like to keep it off while keeping ALL the cooldowns in same place in UI. Until now ive been using older version because Ive been having performance issues with new versions and 4.8 was just perfect for me, it had everything I needed without anything "extra" that would anoy me on screen.
Is there any way to make this work that if you hide the class window the cooldowns of skills I listed above will show in the second row like they used to?

Ok, I'll add a check for that.

Thank you sooo much for the superfast fix, much appreciated!
---------- Původní e-mail ----------
Od: Foglio<EMAIL_ADDRESS>Komu: Foglio1024/Tera-custom-cooldowns <Tera-custom-cooldowns@noreply.
github.com>
Datum: 6. 6. 2017 15:17:13
Předmět: Re: [Foglio1024/Tera-custom-cooldowns] Enrage, Adrenaline Rush,
Guardian Shout not showing in cooldown bar (#51)
"
Ok, I'll add a check for that.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
(https://github.com/Foglio1024/Tera-custom-cooldowns/issues/51#issuecomment-306482896)
, or mute the thread
(https://github.com/notifications/unsubscribe-auth/Ab4DsxPqHNdgL_p2n38SUpwPej1h0zpkks5sBVFWgaJpZM4NxTmd)
.
"


delegate macro supports multiple methods

The title is pretty explicit i guess.
I think we can save some typing by allowing to delegate multiple methods to the same object in one call.

Yeah this would be time-saving :+1:

This seem like it would be convenient! A :+1: from me, too.

This would be a perfect fit for https://github.com/sdogruyol/kemal/blob/master/src/kemal/context.cr#L26-L28

Looks good! :-)
I think it works too if you do macro delegate(*names, to_object). In any case the macro just became "arg-unsafe": I can do delegate :foo and I won't get any compile error and that line won't do anything. So maybe doing something like: macro delegate(first_name, *other_names, to_object) so we make sure you at least call it with two arguments. Or just {{ raise "..." if names.size < 1 }} for the delegate(*names, to_object) variant.

Thanks for the tip :+1: ! Update on the way

Thanks!!

Yeah, my first crystal contribution !


Pr/v1.11 backport 2022 11 30

#22361 -- fqdn: dnsproxy: fix forwardng of the original security identity (@aspsk)

Once this PR is merged, you can update the PR labels via:
$ for pr in 22361; do contrib/backporting/set-labels.py $pr done 1.11; done

/test-backport-1.11

/test-gke

/test-backport-1.11

/test-1.20-4.9

/test-1.20-4.9

/test-gke

/test

/test-backport-1.11

/test-backport-1.11

It looks like this PR was also impacted by the issue that was recently fixed in https://github.com/cilium/cilium/pull/22724 . All other tests are passing and the other backports for this are in. I'll make an exception to CI failures and merge this to ensure that the fix lands in all trees in the upcoming patch release.


Update Romanian translation, use pt-BR for Portugese

Tasklist

[x] Update Romanian translation
[x] Use pt-BR for pt localization
[ ] Add changelog entry
[ ] Test with osrm-frontend (?)
[ ] Review

Requirements / Relations
Link any requirements here. Other pull requests this PR is based on?

I asked to join the pt-PT team at Transifex to finish the European Portuguese translation. In fact I already have it done. Translated the Json on my own for my use. Can submit a pull request here, or can submit it at Transifex. BTW, Indeed there are diferences. The grammar is mostly the same. The difference is only in a few expressions that Portuguese people don't use. Ex. "Pegue a rampa" becomes "Apanhe a rampa"(Take the ramp), "rotatória" becomes "rotunda"(roundabout).

@1ec5 Well, I would resend the request if I could. Portuguese (Portugal) is not on the list anymore when I press the Join Team button. My Transifex username is psykickruhyn

Just a quick question, are these only written instructions or are they also used for verbal instructions on turn-by-turn navigation apps/devices?

Both. The Mapbox Navigation SDK primarily uses them for verbal instructions, but other clients including Web-based clients typically display them visually.

Ok, then I'm doing it well, using language which sounds more natural on spoken form. Was afraid I was rewriting too many of the verbal times of the previous translation. Still need to go back and change the structure of some sentences, also. I will open a new Issue because I have some doubts on othet stuff.

@Rui-Santos has kindly created an actual European Portuguese localization that we’re adding in #229. @ggl, unfortunately, this PR has become outdated, so please open a new PR whenever you’re ready to update the Romanian localization again. Thanks!

@mdomnita is my colleague, his desk is in front of mine. I based my translation upon his initial work.
Thanx for adding me to the team, the remaining strings have been translated on Transifex.

I wound up including these translations in #231. Please take a look at that PR’s test fixtures to ensure that the generated sentences make sense. Thanks for your help, @ggl @mdomnita!


Option to include comments in json output

One use case is for documentation. For example, at the top of each CSS file I might include a title and a description. The JSON could then be used to output that meta information along with the actual CSS.

Thanks for the request! I agree to your opinion.
I also wanted supporting comments, however, I was troubled because there is something difficult to solve. As we know, comments blocks can be located at anywhere in CSS. So, sometimes it's difficult to put comments informations into JSON:
/* field decorator */
.my-field {
    padding-left: 1em; /* left padding */
    /* background color */
    background-color: /* don't fix this */ #fff /* white */
}

I know this is ugly example. but, it's possible. :/
If you have any idea to handle these cases, please let me know. :)
Thanks! :)

Do you you mean it would be difficult to construct JSON from the example above or that its difficult to parse?
In any case what I wanted to do was use the comments for documentation, where the source would look like:
/*
# My Thing

This component does x, y, and z.
*/

.my-thing {
  padding-left: 1em;
}

Here only the comments at the root level would be captured, though I could see that may be too specific a use case for this script.

Okay. I'll include this feature in the next release. :)


Add PAT Authentication for http git repos

Because:

Security
For clients where no ssh/git private cert is available

Signed-off-by: Florian Hopfensperger<EMAIL_ADDRESS>

Pull Request Test Coverage Report for Build 64a269c3d6d26a0b2f667532701e4cdb3e242065-PR-15

21 of 21 (100.0%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.5%) to 91.111%

Totals

Change from base Build 264e411a13021ec1895b44ae4f9a3f98dedc4617:
0.5%

Covered Lines:
205

Relevant Lines:
225

💛  - Coveralls


Update Mattermost version

Hello,
I've downloaded current docker version via git clone https://github.com/mattermost/docker, setup and run Mattermost server.
However when I run the server it says it's version 8.1.9. But the current version is 10.1.2.
Can you please update docker installer so it has current version?
Thank you very much.

Version depends on your .env file. edit it and use docker to pull new image. db backups first.

Oh, thank you. That helped me a lot and I was able to easily update to the most recent version :-)


K8SSAND-1754 ⁃ encryption cannot be configured but disabled in C* 4.0.5

What happened?
Attempted to start a cluster using a manifest with encryption defined, but disabled; the cluster starts correctly when these are enabled but the first pod trying to start fails with a stacktrace, and the cluster does not come up:
ERROR [main] 2022-08-26 16:01:48,145 CassandraDaemon.java:911 - Exception encountered during startup
org.apache.cassandra.exceptions.ConfigurationException: Failed to initialize SSL
        at org.apache.cassandra.config.DatabaseDescriptor.applySslContext(DatabaseDescriptor.java:1014)
        at org.apache.cassandra.config.DatabaseDescriptor.applyAll(DatabaseDescriptor.java:364)
        at org.apache.cassandra.config.DatabaseDescriptor.daemonInitialization(DatabaseDescriptor.java:178)
        at org.apache.cassandra.config.DatabaseDescriptor.daemonInitialization(DatabaseDescriptor.java:162)
        at org.apache.cassandra.service.CassandraDaemon.applyConfig(CassandraDaemon.java:818)
        at org.apache.cassandra.service.CassandraDaemon.activate(CassandraDaemon.java:754)
        at org.apache.cassandra.service.CassandraDaemon.main(CassandraDaemon.java:889)
Caused by: java.io.IOException: Failed to create SSL context using Internode messaging
        at org.apache.cassandra.security.SSLFactory.validateSslContext(SSLFactory.java:546)
        at org.apache.cassandra.config.DatabaseDescriptor.applySslContext(DatabaseDescriptor.java:1008)
        ... 6 common frames omitted
Caused by: java.io.IOException: failed to build key manager store for secure connections
        at org.apache.cassandra.security.SSLFactory.buildKeyManagerFactory(SSLFactory.java:252)
        at org.apache.cassandra.security.SSLFactory.createNettySslContext(SSLFactory.java:315)
        at org.apache.cassandra.security.SSLFactory.validateSslContext(SSLFactory.java:496)
        ... 7 common frames omitted
Caused by: java.nio.file.NoSuchFileException: conf/.keystore
        at java.base/sun.nio.fs.UnixException.translateToIOException(Unknown Source)
        at java.base/sun.nio.fs.UnixException.rethrowAsIOException(Unknown Source)
        at java.base/sun.nio.fs.UnixException.rethrowAsIOException(Unknown Source)
        at java.base/sun.nio.fs.UnixFileSystemProvider.newByteChannel(Unknown Source)
        at java.base/java.nio.file.Files.newByteChannel(Unknown Source)
        at java.base/java.nio.file.Files.newByteChannel(Unknown Source)
        at java.base/java.nio.file.spi.FileSystemProvider.newInputStream(Unknown Source)
        at java.base/java.nio.file.Files.newInputStream(Unknown Source)
        at org.apache.cassandra.security.SSLFactory.buildKeyManagerFactory(SSLFactory.java:227)
        ... 9 common frames omitted

Did you expect to see something different?
The cassandra.yaml on the pod should either have valid paths to keystore/truststore, or it should not.
client_encryption_options:
  enabled: false
  optional: true
  require_client_auth: true
  store_type: PKCS12
  keystore: conf/.keystore
  keystore_password: cassandra
  truststore_password: password
  truststore: conf/.truststore
server_encryption_options:
  internode_encryption: none
  optional: true
  require_client_auth: true
  store_type: PKCS12
  enable_legacy_ssl_storage_port: false

conf/.keystore and conf/.truststore do not exist.
How to reproduce it (as minimally and precisely as possible):

apply encryption-ready.yaml

Environment
WSL2 Ubuntu 20.04 using kind

K8ssandra Operator version:

v1.12.0

Kubernetes version information:

Client Version: version.Info{Major:"1", Minor:"24", GitVersion:"v1.24.2", GitCommit:"f66044f4361b9f1f96f0053dd46cb7dce5e990a8", GitTreeState:"clean", BuildDate:"2022-06-15T14:22:29Z", GoVersion:"go1.18.3", Compiler:"gc", Platform:"linux/amd64"}
Kustomize Version: v4.5.4
Server Version: version.Info{Major:"1", Minor:"22", GitVersion:"v1.22.7", GitCommit:"b56e432f2191419647a6a13b9f5867801850f969", GitTreeState:"clean", BuildDate:"2022-03-06T21:07:35Z", GoVersion:"go1.16.14", Compiler:"gc", Platform:"linux/amd64"}
WARNING: version difference between client (1.24) and server (1.22) exceeds the supported minor version skew of +/-1

Kubernetes cluster kind:

kind

Manifests:

encryption-ready.yaml
kind: K8ssandraCluster
metadata:
  name: demo
spec:
  cassandra:
    serverVersion: "4.0.5"
    datacenters:
      - metadata:
          name: dc1
        size: 3
        storageConfig:
          cassandraDataVolumeClaimSpec:
            storageClassName: standard
            accessModes:
              - ReadWriteOnce
            resources:
              requests:
                storage: 5Gi
        config:
          jvmOptions:
            heapSize: 512M
          cassandraYaml:
            server_encryption_options:
              internode_encryption: none
              require_client_auth: true
              store_type: PKCS12
              optional: true
            client_encryption_options:
              enabled: false
              optional: true
              require_client_auth: true
              store_type: PKCS12
        stargate:
          size: 1
          heapSize: 256M
    serverEncryptionStores:
      keystoreSecretRef:
        name: server-encryption-stores
      truststoreSecretRef:
        name: server-encryption-stores
    clientEncryptionStores:
      keystoreSecretRef:
        name: client-encryption-stores
      truststoreSecretRef:
        name: client-encryption-stores

K8ssandra Operator Logs:

n/a
Anything else we need to know?:
I was chatting with adejanovski  about this issue.
┆Issue is synchronized with this Jira Task by Unito
┆friendlyId: K8SSAND-1754
┆priority: Medium

Having the same issue. Any idea how to fix it?


Fix Replicate join schedule error with bucket/colocate join in same fragment

#1091

run starrocks_clang-format


Download JAR from Maven & fixed markerClass

Hi Philippe,
I didn't see your PR, my bad. I added the screenshot url in another commit (forgot it in the first). If you could close your PR that'd be appreciated.
Thanks a lot,


Bumping dependencies for freeze

What does this PR do?
Bumping dependencies for agent release. First Batch
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

/azp run PR Changes Comment


chore: release main

:robot: I have created a release beep boop
1.4.0
1.4.0 (2023-01-16)
Features

feature in java-eventarc (0874a69)
this is a test of release tooling (00f8d28)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/bcoe/google-cloud-java/releases/tag/v1.4.0 :sunflower:


Empty App not remembering login credentials

It might be a expected behaviour, but login page doesn't allow navigator built in function to save user credentials. I tried to enable form autocomplete at page level, and erasing autocomplete=off html attribute from user and password, but it keeps asking for user credentials.

Fixed in 0.9.6.
Thanks a lot for logging any issue you have. I find it very valuable.


User object only captures username data, not full name data

The User observable object has caption name which is described as a username (e.g. john_doe), but does not have a caption for capturing an actual name (e.g. John Doe). Perhaps name can be renamed to username; alternatively, for backwards compatibility, a new caption such as full_name can be included. While small, this change should make it one step easier to associate a username with an actual human.

I will create a PR to add a new optional attribute, full_name but not change name to username as that will have ripple effects, since all entities have name, and it will be an unnecessary breaking change.


infra: sync_wait

Doing an asynchronous wait of a coroutine is easy:
auto result = co_await task();
Doing a synchronous wait of a coroutine is more verbose:
auto result = asio::co_spawn(io_context, task(), asio::use_future).get();
also, this exposes implementation details.
Using the simple sync_wait() function of this PR, the call becomes:
auto result = sync_wait(io_context, task());
so at the call site, the intent of the code becomes clearer.
A simple getter enables us to simplify the code further when the io_context is not available in the scope and we need to get it from get_executor() method:
auto result = sync_wait(in(this), task());

I like it! I would add to this also one sync_wait overload for concurrency::TaskGroup as done in:
https://github.com/torquem-ch/silkworm/blob/master/silkworm/sync/sentry_client.cpp#L156
You can do it in next PR


Anchors exercise

I think the page needs more of an explanation of anchors and how they work, but I wasn't really sure what else to add.
Solution PR https://github.com/holochain-gym/developer-exercises/pull/35

@0phelia36 this is amazing, thanks so much :)


GitHub actions to compile and release binaries

GitHub actions to compile and release Windows binaries on every main branch push
Includes build date and commit hash

I see nothing wrong with the project (as of 24th Jan 2023) so I think if someone else approves, github can produce binaries.


Fixing the license errors

Fixing the license errors

The groovy ones are failing at the Jacoco test report & error is with the use of class name:

Since we didn't update any manifest class name, so will not touch other files to fix this.

But License test is successful now, so we can merge this PR:


Remove redundant cast

Looks like that these two Java checks failed because of this - Java Wordcount Direct Runner (macos-latest) / (windows-latest):

> Task :sdks:java:core:compileTestJava
D:\a\beam\beam\sdks\java\core\src\test\java\org\apache\beam\sdk\metrics\MetricsTest.java:450: error: incompatible types: inferred type does not conform to lower bound(s)
    assertThat(
              ^
    inferred: MetricResult<DistributionResult>
    lower bound(s): CAP#1,MetricResult<DistributionResult>
  where CAP#1 is a fresh type-variable:
    CAP#1 extends Object super: MetricResult<DistributionResult> from capture of ? super MetricResult<DistributionResult>

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

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

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

Run Java PreCommit

R: @kennknowles
PTAL, all checks passed


Clone more than we need for GitHub

This changes how we fetch the index from GitHub to ask for more than we need because they asked us to.
fixes https://rust-lang.zulipchat.com/#narrow/stream/246057-t-cargo/topic/crates.2Eio.20index.20changes
It is not yet clear if we have the request in the form that hits their happy path. Nor why they are happy path cannot be extended to include our existing fetch pattern.
I opened this PR as a statement of goodwill, if this is the best design available then we are prepared to make it happen.

r? @epage
(rust-highfive has picked a reviewer for you, use r? to override)