bam_clean error: Bad Input

Hi,
I have two bam files(normal-tumor). I would like to call variants across these samples. Since I dont require alignment I preferred bam clean and bam sort as it is suggested in the bcbio documentation. As I looked for the error, I dont know in which point I piped the wrong input. These bam files are the bam files from ICGC consortium so I dont know what I can do? Pass bam sort and clean ?
During the run I came across with the following error;
[2016-04-17T12:27Z] Timing: organize samples
[2016-04-17T12:27Z] multiprocessing: organize_samples
[2016-04-17T12:27Z] Using input YAML configuration: /mnt/work/bcbio_sample-forvm.yaml
[2016-04-17T12:44Z] Checking sample YAML configuration: /mnt/work/bcbio_sample-forvm.yaml
[2016-04-17T12:44Z] Downloading GRCh37 samtools from AWS
[2016-04-17T12:46Z] Downloading GRCh37 samtools from AWS
[2016-04-17T12:46Z] Downloading GRCh37 samtools from AWS
[2016-04-17T12:46Z] Testing minimum versions of installed programs
[2016-04-17T12:46Z] Timing: alignment preparation
[2016-04-17T12:46Z] multiprocessing: prep_align_inputs
[2016-04-17T12:46Z] multiprocessing: disambiguate_split
[2016-04-17T12:46Z] Timing: alignment
[2016-04-17T12:46Z] multiprocessing: process_alignment
[2016-04-17T22:07Z] Uncaught exception occurred
Traceback (most recent call last):
  File "/usr/local/share/bcbio-nextgen/anaconda/lib/python2.7/site-packages/bcbio/provenance/do.py", line 21, in run
    _do_run(cmd, checks, log_stdout)
  File "/usr/local/share/bcbio-nextgen/anaconda/lib/python2.7/site-packages/bcbio/provenance/do.py", line 95, in _do_run
    raise subprocess.CalledProcessError(exitcode, error_msg)
CalledProcessError: Command '/usr/local/share/bcbio-nextgen/anaconda/bin/gatk-framework -Xms750m -Xmx1600m -XX:+UseSerialGC -Djava.io.tmpdir=/mnt/work/tx/tmpQp9Xj2 -U LENIENT_VCF_PROCESSING --read_filter BadCigar --read_filter NotPrimaryAlignment -T PrintReads -R /mnt/work/inputs/data/genomes/GRCh37/seq/GRCh37.fa -I /mnt/work/bamclean/DO51159-Normal/normal-reorder-fixrgs.bam --out /mnt/work/bamclean/DO51159-Normal/tx/tmpbrspAF/normal-reorder-fixrgs-gatkfilter.bam --filter_mismatching_base_and_quals --filter_bases_not_stored --filter_reads_with_N_cigar --fix_misencoded_quality_scores
/usr/local/share/bcbio-nextgen/anaconda/bin/gatk-framework: line 7: warning: setlocale: LC_ALL: cannot change locale (en_US.UTF-8): No such file or directory
INFO  22:07:02,266 HelpFormatter - ---------------------------------------------------------------------------------
INFO  22:07:02,276 HelpFormatter - The Genome Analysis Toolkit (GATK) v3.4-65-g2434e49, Compiled 2015/10/09 18:46:40
INFO  22:07:02,276 HelpFormatter - Copyright (c) 2010 The Broad Institute
INFO  22:07:02,276 HelpFormatter - For support and documentation go to http://www.broadinstitute.org/gatk
INFO  22:07:02,279 HelpFormatter - Program Args: -U LENIENT_VCF_PROCESSING --read_filter BadCigar --read_filter NotPrimaryAlignment -T PrintReads -R /mnt/work/inputs/data/genomes/GRCh37/seq/GRCh37.fa -I /mnt/work/bamclean/DO51159-Normal/normal-reorder-fixrgs.bam --out /mnt/work/bamclean/DO51159-Normal/tx/tmpbrspAF/normal-reorder-fixrgs-gatkfilter.bam --filter_mismatching_base_and_quals --filter_bases_not_stored --filter_reads_with_N_cigar --fix_misencoded_quality_scores
INFO  22:07:02,349 HelpFormatter - Executing as ubuntu@frontend001 on Linux 3.13.0-85-generic amd64; OpenJDK 64-Bit Server VM 1.7.0_95-b00.
INFO  22:07:02,349 HelpFormatter - Date/Time: 2016/04/17 22:07:02
INFO  22:07:02,350 HelpFormatter - ---------------------------------------------------------------------------------
INFO  22:07:02,350 HelpFormatter - ---------------------------------------------------------------------------------
INFO  22:07:02,803 GenomeAnalysisEngine - Strictness is SILENT
INFO  22:07:02,930 GenomeAnalysisEngine - Downsampling Settings: No downsampling
INFO  22:07:02,937 SAMDataSource$SAMReaders - Initializing SAMRecords in serial
INFO  22:07:02,989 SAMDataSource$SAMReaders - Done initializing BAM readers: total time 0.04
INFO  22:07:03,166 GenomeAnalysisEngine - Preparing for traversal over 1 BAM files
INFO  22:07:03,169 GenomeAnalysisEngine - Done preparing for traversal
INFO  22:07:03,169 ProgressMeter - [INITIALIZATION COMPLETE; STARTING PROCESSING]
INFO  22:07:03,170 ProgressMeter -                 | processed |    time |    per 1M |           |   total | remaining
INFO  22:07:03,170 ProgressMeter -        Location |     reads | elapsed |     reads | completed | runtime |   runtime
INFO  22:07:03,177 ReadShardBalancer$1 - Loading BAM index data
INFO  22:07:03,179 ReadShardBalancer$1 - Done loading BAM index data
##### ERROR ------------------------------------------------------------------------------------------
##### ERROR A USER ERROR has occurred (version 3.4-65-g2434e49):
##### ERROR
##### ERROR This means that one or more arguments or inputs in your command are incorrect.
##### ERROR The error message below tells you what is the problem.
##### ERROR
##### ERROR If the problem is an invalid argument, please check the online documentation guide
##### ERROR (or rerun your command with --help) to view allowable command-line arguments for this tool.
##### ERROR
##### ERROR Visit our website and forum for extensive documentation and answers to
##### ERROR commonly asked questions http://www.broadinstitute.org/gatk
##### ERROR
##### ERROR Please do NOT post this error to the GATK forum unless you have really tried to fix it yourself.
##### ERROR
##### ERROR MESSAGE: Bad input: while fixing mis-encoded base qualities we encountered a read that was correctly encoded; we cannot handle such a mixture of reads so unfortunately the BAM must be fixed with some other tool
##### ERROR ------------------------------------------------------------------------------------------
' returned non-zero exit status 1
Uncaught exception occurred
Traceback (most recent call last):
  File "/home/ubuntu/install/bcbio-vm/data/anaconda/lib/python2.7/site-packages/bcbio/provenance/do.py", line 21, in run
    _do_run(cmd, checks, log_stdout)
  File "/home/ubuntu/install/bcbio-vm/data/anaconda/lib/python2.7/site-packages/bcbio/provenance/do.py", line 95, in _do_run
    raise subprocess.CalledProcessError(exitcode, error_msg)
CalledProcessError: Command 'docker attach --no-stdin 4b75acae24049a12503db425d9fe7b1ce72a9ac7bf9b143fc002cc73407c7376
INFO  22:07:03,169 ProgressMeter - [INITIALIZATION COMPLETE; STARTING PROCESSING]
INFO  22:07:03,170 ProgressMeter -                 | processed |    time |    per 1M |           |   total | remaining
INFO  22:07:03,170 ProgressMeter -        Location |     reads | elapsed |     reads | completed | runtime |   runtime
INFO  22:07:03,177 ReadShardBalancer$1 - Loading BAM index data
INFO  22:07:03,179 ReadShardBalancer$1 - Done loading BAM index data
##### ERROR ------------------------------------------------------------------------------------------
##### ERROR A USER ERROR has occurred (version 3.4-65-g2434e49):
##### ERROR
##### ERROR This means that one or more arguments or inputs in your command are incorrect.
##### ERROR The error message below tells you what is the problem.
##### ERROR
##### ERROR If the problem is an invalid argument, please check the online documentation guide
##### ERROR (or rerun your command with --help) to view allowable command-line arguments for this tool.
##### ERROR
##### ERROR Visit our website and forum for extensive documentation and answers to
##### ERROR commonly asked questions http://www.broadinstitute.org/gatk
##### ERROR
##### ERROR Please do NOT post this error to the GATK forum unless you have really tried to fix it yourself.
##### ERROR
##### ERROR MESSAGE: Bad input: while fixing mis-encoded base qualities we encountered a read that was correctly encoded; we cannot handle such a mixture of reads so unfortunately the BAM must be fixed with some other tool
##### ERROR ------------------------------------------------------------------------------------------
' returned non-zero exit status 1
Traceback (most recent call last):
  File "/usr/local/bin/bcbio_nextgen.py", line 226, in <module>
    main(**kwargs)
  File "/usr/local/bin/bcbio_nextgen.py", line 43, in main
    run_main(**kwargs)
  File "/usr/local/share/bcbio-nextgen/anaconda/lib/python2.7/site-packages/bcbio/pipeline/main.py", line 39, in run_main
    fc_dir, run_info_yaml)
  File "/usr/local/share/bcbio-nextgen/anaconda/lib/python2.7/site-packages/bcbio/pipeline/main.py", line 82, in _run_toplevel
    for xs in pipeline(config, run_info_yaml, parallel, dirs, samples):
  File "/usr/local/share/bcbio-nextgen/anaconda/lib/python2.7/site-packages/bcbio/pipeline/main.py", line 123, in variant2pipeline
    samples = run_parallel("process_alignment", samples)
  File "/usr/local/share/bcbio-nextgen/anaconda/lib/python2.7/site-packages/bcbio/distributed/multi.py", line 28, in run_parallel
    return run_multicore(fn, items, config, parallel=parallel)
  File "/usr/local/share/bcbio-nextgen/anaconda/lib/python2.7/site-packages/bcbio/distributed/multi.py", line 86, in run_multicore
    for data in joblib.Parallel(parallel["num_jobs"])(joblib.delayed(fn)(x) for x in items):
  File "/usr/local/share/bcbio-nextgen/anaconda/lib/python2.7/site-packages/joblib/parallel.py", line 800, in __call__
    while self.dispatch_one_batch(iterator):
  File "/usr/local/share/bcbio-nextgen/anaconda/lib/python2.7/site-packages/joblib/parallel.py", line 658, in dispatch_one_batch
    self._dispatch(tasks)
  File "/usr/local/share/bcbio-nextgen/anaconda/lib/python2.7/site-packages/joblib/parallel.py", line 566, in _dispatch
    job = ImmediateComputeBatch(batch)
  File "/usr/local/share/bcbio-nextgen/anaconda/lib/python2.7/site-packages/joblib/parallel.py", line 180, in __init__
    self.results = batch()
  File "/usr/local/share/bcbio-nextgen/anaconda/lib/python2.7/site-packages/joblib/parallel.py", line 72, in __call__
    return [func(*args, **kwargs) for func, args, kwargs in self.items]
  File "/usr/local/share/bcbio-nextgen/anaconda/lib/python2.7/site-packages/bcbio/utils.py", line 51, in wrapper
    return apply(f, *args, **kwargs)
  File "/usr/local/share/bcbio-nextgen/anaconda/lib/python2.7/site-packages/bcbio/distributed/multitasks.py", line 67, in process_alignment
    return sample.process_alignment(*args)
  File "/usr/local/share/bcbio-nextgen/anaconda/lib/python2.7/site-packages/bcbio/pipeline/sample.py", line 120, in process_alignment
    data)
  File "/usr/local/share/bcbio-nextgen/anaconda/lib/python2.7/site-packages/bcbio/pipeline/cleanbam.py", line 30, in picard_prep
    return _filter_bad_reads(rg_bam, ref_file, data)
  File "/usr/local/share/bcbio-nextgen/anaconda/lib/python2.7/site-packages/bcbio/pipeline/cleanbam.py", line 51, in _filter_bad_reads
    do.run(cmd, "Filter problem reads")
  File "/usr/local/share/bcbio-nextgen/anaconda/lib/python2.7/site-packages/bcbio/provenance/do.py", line 21, in run
    _do_run(cmd, checks, log_stdout)
  File "/usr/local/share/bcbio-nextgen/anaconda/lib/python2.7/site-packages/bcbio/provenance/do.py", line 95, in _do_run
    raise subprocess.CalledProcessError(exitcode, error_msg)
subprocess.CalledProcessError: Command '/usr/local/share/bcbio-nextgen/anaconda/bin/gatk-framework -Xms750m -Xmx1600m -XX:+UseSerialGC -Djava.io.tmpdir=/mnt/work/tx/tmpQp9Xj2 -U LENIENT_VCF_PROCESSING --read_filter BadCigar --read_filter NotPrimaryAlignment -T PrintReads -R /mnt/work/inputs/data/genomes/GRCh37/seq/GRCh37.fa -I /mnt/work/bamclean/DO51159-Normal/normal-reorder-fixrgs.bam --out /mnt/work/bamclean/DO51159-Normal/tx/tmpbrspAF/normal-reorder-fixrgs-gatkfilter.bam --filter_mismatching_base_and_quals --filter_bases_not_stored --filter_reads_with_N_cigar --fix_misencoded_quality_scores
/usr/local/share/bcbio-nextgen/anaconda/bin/gatk-framework: line 7: warning: setlocale: LC_ALL: cannot change locale (en_US.UTF-8): No such file or directory
INFO  22:07:02,266 HelpFormatter - ---------------------------------------------------------------------------------
INFO  22:07:02,276 HelpFormatter - The Genome Analysis Toolkit (GATK) v3.4-65-g2434e49, Compiled 2015/10/09 18:46:40
INFO  22:07:02,276 HelpFormatter - Copyright (c) 2010 The Broad Institute
INFO  22:07:02,276 HelpFormatter - For support and documentation go to http://www.broadinstitute.org/gatk
INFO  22:07:02,279 HelpFormatter - Program Args: -U LENIENT_VCF_PROCESSING --read_filter BadCigar --read_filter NotPrimaryAlignment -T PrintReads -R /mnt/work/inputs/data/genomes/GRCh37/seq/GRCh37.fa -I /mnt/work/bamclean/DO51159-Normal/normal-reorder-fixrgs.bam --out /mnt/work/bamclean/DO51159-Normal/tx/tmpbrspAF/normal-reorder-fixrgs-gatkfilter.bam --filter_mismatching_base_and_quals --filter_bases_not_stored --filter_reads_with_N_cigar --fix_misencoded_quality_scores
INFO  22:07:02,349 HelpFormatter - Executing as ubuntu@frontend001 on Linux 3.13.0-85-generic amd64; OpenJDK 64-Bit Server VM 1.7.0_95-b00.
INFO  22:07:02,349 HelpFormatter - Date/Time: 2016/04/17 22:07:02
INFO  22:07:02,350 HelpFormatter - ---------------------------------------------------------------------------------
INFO  22:07:02,350 HelpFormatter - ---------------------------------------------------------------------------------
INFO  22:07:02,803 GenomeAnalysisEngine - Strictness is SILENT
INFO  22:07:02,930 GenomeAnalysisEngine - Downsampling Settings: No downsampling
INFO  22:07:02,937 SAMDataSource$SAMReaders - Initializing SAMRecords in serial
INFO  22:07:02,989 SAMDataSource$SAMReaders - Done initializing BAM readers: total time 0.04
INFO  22:07:03,166 GenomeAnalysisEngine - Preparing for traversal over 1 BAM files
INFO  22:07:03,169 GenomeAnalysisEngine - Done preparing for traversal
INFO  22:07:03,169 ProgressMeter - [INITIALIZATION COMPLETE; STARTING PROCESSING]
INFO  22:07:03,170 ProgressMeter -                 | processed |    time |    per 1M |           |   total | remaining
INFO  22:07:03,170 ProgressMeter -        Location |     reads | elapsed |     reads | completed | runtime |   runtime
INFO  22:07:03,177 ReadShardBalancer$1 - Loading BAM index data
INFO  22:07:03,179 ReadShardBalancer$1 - Done loading BAM index data
##### ERROR ------------------------------------------------------------------------------------------
##### ERROR A USER ERROR has occurred (version 3.4-65-g2434e49):
##### ERROR
##### ERROR This means that one or more arguments or inputs in your command are incorrect.
##### ERROR The error message below tells you what is the problem.
##### ERROR
##### ERROR If the problem is an invalid argument, please check the online documentation guide
##### ERROR (or rerun your command with --help) to view allowable command-line arguments for this tool.
##### ERROR
##### ERROR Visit our website and forum for extensive documentation and answers to
##### ERROR commonly asked questions http://www.broadinstitute.org/gatk
##### ERROR
##### ERROR Please do NOT post this error to the GATK forum unless you have really tried to fix it yourself.
##### ERROR
##### ERROR MESSAGE: Bad input: while fixing mis-encoded base qualities we encountered a read that was correctly encoded; we cannot handle such a mixture of reads so unfortunately the BAM must be fixed with some other tool
##### ERROR ------------------------------------------------------------------------------------------
' returned non-zero exit status 1
' returned non-zero exit status 1

Also I would like to share my config file, maybe i did something wrong
ubuntu@frontend001:/encrypted/project9/work/config$ cat deneme7.yaml
details:
- algorithm:
    aligner: false
    bam_clean: picard
    bam_sort: coordinate
    ensemble:
      numpass: 2
    indelcaller: scalpel
    platform: illumina
    quality_format: illumina
    realign: false
    recalibrate: false
    remove_lcr: true
    variantcaller:
    - mutect
    - vardict
    - varscan
    - freebayes
  analysis: variant2
  description: DO51159-Normal
  files:
  - s3://tuncproject/icgcrun/input/normal.bam
  genome_build: GRCh37
  metadata:
    batch: ICGC
    phenotype: normal
- algorithm:
    aligner: false
    bam_clean: picard
    bam_sort: coordinate
    ensemble:
      numpass: 2
    indelcaller: scalpel
    platform: illumina
    quality_format: illumina
    realign: false
    recalibrate: false
    remove_lcr: true
    variantcaller:
    - mutect
    - vardict
    - varscan
    - freebayes
  analysis: variant2
  description: DO51159-Tumor
  files:
  - s3://tuncproject/icgcrun/input/tumor.bam
  genome_build: GRCh37
  metadata:
    batch: ICGC
    phenotype: tumor
fc_date: '2015-04-14'
fc_name: ICGC-trials
resources:
  gatk:
    jar: s3://tuncproject/gatktools/GenomeAnalysisTK.jar
  mutect:
    jar: s3://tuncproject/gatktools/mutect-1.1.7.jar
upload:
  bucket: tuncproject
  dir: ../final
  folder: icgcrun/input/final
  method: s3
  region: us-east-1

Tunc;
You're exactly right in your diagnosis -- there is something problematic about the ICGC BAM files that GATK does not like. My suggestion would be to align with bwa through bcbio so you get clean inputs:
aligner: bwa

If you don't want to do that you could also try cleaning them as you suggest by adding:
bam_clean: picard

This may take some manual work on your side to clean up the quality scores in the input files if neither of these work cleanly and the original file is a mix of multiple quality encoding types, but fingers crossed one of these will work. Hope this helps.

Dear Brad;
If you may look at my configuration files, I have already settled bam_clean: picard . Did you mean leaving bam_clean only there and remove ```bam_sort```` or trying that option. Because I have already included them in my configuration file and got this option.
Thank you,
Tunc

Tunc;
My mistake, sorry. I guess the cleanup we have is not handling this input file. I don't believe bam_sort will help with this problem since it looks like it's due to misencoded quality scores. Hopefully realigning with bwa will resolve the issue.


Charcoal\Relation

Adds support for creating and working with relationships between models, using an intermediate object. A source model has one or many relationships with a target model. These relationships are stored in an intermediate pivot model.

Coverage decreased (-0.007%) to 10.503% when pulling d033896efd7b58742c85cdf4ae3555f63e2dd821 on dlord-relation into 625304f3bdd60dc41e8c3159cc4f473189513a64 on master.

Coverage decreased (-0.007%) to 10.503% when pulling 713421c5958d76bc640416315694eaec2465710e on dlord-relation into 625304f3bdd60dc41e8c3159cc4f473189513a64 on master.

Coverage decreased (-0.007%) to 10.503% when pulling 888609241d1cf374ed86cb5881812d33d9dbc4b6 on dlord-relation into 625304f3bdd60dc41e8c3159cc4f473189513a64 on master.

Coverage decreased (-10.5%) to 0.0% when pulling c14f1d32ff64b94e6c20aa74e0029c5aaabbf7c7 on dlord-relation into 625304f3bdd60dc41e8c3159cc4f473189513a64 on master.

Coverage decreased (-10.5%) to 0.0% when pulling c14f1d32ff64b94e6c20aa74e0029c5aaabbf7c7 on dlord-relation into 625304f3bdd60dc41e8c3159cc4f473189513a64 on master.

@mcaskill I agree with CMS not being the right place. However, we have to start somewhere, and like many Charcoal components that have found their place in the proper packages, we can migrate parts of Relation to wherever it makes more sense as we start using it.

Coverage decreased (-1.07%) to 9.968% when pulling c681275bda14802c04d8d820e3dc80f0e981629b on dlord-relation into ae53375363119d3657fba7ec9bb46e667bfec670 on master.

Coverage decreased (-1.07%) to 9.968% when pulling c681275bda14802c04d8d820e3dc80f0e981629b on dlord-relation into ae53375363119d3657fba7ec9bb46e667bfec670 on master.


downtime and event...

Hello @ravener , @erickang21 :
I would like to have a countdown timer, downtime...
in which the time for the countdown could be configured and when reaching the countdown it would give us an event that it has reached time 0 of the countdown?

Can it be obtained with this library?
Or would it be an improvement to this library?
thanks you, Víctor.

Hi,
Sorry but a stopwatch is only supposed to count the elapsed time for basic benchmarking purposes and so a countdown is out of the scope of this project, since you also want to know when the timer reaches zero that will be opinionated regarding how events are handled and it'd require an event loop or similar patterns to handle events.

Hi,
Sorry but a stopwatch is only supposed to count the elapsed time for basic benchmarking purposes and so a countdown is out of the scope of this project, since you also want to know when the timer reaches zero that will be opinionated regarding how events are handled and it'd require an event loop or similar patterns to handle events.

Hi @ravener ,
I found this library, do you think it will help me?
What I don't know is how to know how much time I have left? in this other library...
https://github.com/Saadmairaj/date-time-event


Add a show-star command

Shows who starred a message.
Also fixes terrible code :^)

wdym terrible code pls.

still better than no code @Crawl

too bad i mentioned the wrong Crawl 🤷♀


Arguments from environment

In k8s, it is much easier to template with env rather than CLI arguments.

No context, not entirely sure what you mean.

Right now: go run . --issuer example.com
Request: TESTER_ISSUER=example.com go run .

Yeah no worries, can implement with koanf.


Swift 4 Migration Issue

2 errors occur after migrating to swift 4.

First error
Initializer for conditional binding must have Optional type, not 'AVCaptureVideoPreviewLayer'

Second Error
Cannot convert value of type '[String : AnyObject]' to expected argument type '[NSAttributedStringKey : Any]?'

I'm also facing same issue.

Resolved with Swift 4 Update


Is there any plan to support openstack ?

Is there any plan to support for Openstack and Eucalyptus [ aws compatible cloud ] ?

@shamimgeek
No plans for Eucalyptus, but OpenStack is queued up for the next release. See these existing issues and PR:
#530, codedellemc/libstorage#182, codedellemc/libstorage#243,

Thanks for your reply.
all the best for your hard work towards making rexray most trusted tool for managing persistent volumes in cloud.
Regards
Shamim

Hi, Is this released already? From the release notes, it looks like these changes went in? Looking forward to using this on openstack in docker swarm for persistant storage.

Yep, it's available today.

Thanks. I have got it working on my swarm on openstack. Really appreciate all the amazing work!


View buttons crashing

Describe the bug
When I am cycling between the html viewer and the page viewer (the i and the eye) so that I can look at my project while I'm coding, I get stuck on the html viewer. The icons light up white when selected normally. In this case the "eye" icon will not light up white and will not display the page that I'm working on, making it impossible to see my changes without closing and reopening the app. After closing and reopening the app I can then select the "eye" icon and view my page until I make a change to the html and then it all happens over again.
To Reproduce
Steps to reproduce the behavior:

Open the app
Click on 'eye' icon
Click on the html
Make any change
Now try and click on the 'eye' icon

Expected behavior
It should not light up the "eye" icon. You should be unable to see the page view. Clicking back and forth between icons for html and page view should crash page viewer so that it becomes unresponsive and you are unable to open page viewer.
Screenshots
If applicable, add screenshots to help explain your problem.
Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
I do the majority of my coding on iPad M2 pro that I bought in Nov, 2023. And I've been doing all my FCC app on my iPad as well.

Hey @xJosHubx,
Thank you for reporting this. Could you share a video with us? The instructions are somewhat hard to follow.

I'm not sure if someone worked on this issue but I'm not able to reproduce it anymore. Will keep you posted. Thank you so much for your response!

Hey @sembauke here is the video of this issue https://youtu.be/ID_cKUDgnco?si=ZJzcrdCEziGcW0N5. Thank you so much for working on this! I really want to have a good UX while using the app. 😊

Thank you for the update and the video. It seems that there are many issues on large devices.

Thank you, Sem. I'll do my best.


Not able to create Dynamic sub keyPrefix

Not able create Dynamic sub keyPrefix while uploading Images???

did you find any solution for dynamic keyPrefix.. If not please give me some suggestions

Hi mahesh.. Did you find any solution in this issue??


Localise Horizontal Scaling page in hindi

The Hindi version of Horizontal Scaling is not available.
Link: https://glossary.cncf.io/horizontal-scaling/

The Hindi version of Horizontal Scaling is not available. Link: https://glossary.cncf.io/horizontal-scaling/

I would like to work on this @VamshiReddy02 Please assign me.

Hi @SaxenaAnushka102, Are you still interested in this issue?

Hello @anubha-v-ardhan! Sure I can take it up👍

Hello @anubha-v-ardhan! I've created a PR for this issue, please check it out & let me know if any changes have to be made. Thanks!

Closing as completed by #1988


Group is not enabling/disabling feature

I’ve a user model and trying to enable feature flag for only for specific users.
# Registered a group
# allowed? is a custom method in User(ActiveRecord) class
Flipper.register(:normal_users) do |actor|
  actor.is_a?(User) && actor.allowed?
end

# Enabled the feature for this group
feature = Flipper[:stats]
feature.enable_group :normal_users

But when check on users to see if it got enabled for users it doesn’t work:
feature.enabled?(current_user) => false

Also, when placing binding.pry inside allowed? method it never gets called.
On the other hand actors work just fine.
And when deleting this group from UI it says:
"The group named "normal_users" has not been registered."

Can you tell me more about your setup? Is the flipper ui mounted in the same app? Where are you defining the group in code? Any and all setup or flipper specific code you had in a gist or comment here would help me but I don’t have enough to go on at this point.

On Jul 24, 2020, at 4:50 AM, Talha Meh<EMAIL_ADDRESS>wrote:

I’ve a user model and trying to enable feature flag for only for specific users.
Registered a group
allowed? is a custom method in User(ActiveRecord) class
Flipper.register(:normal_users) do |actor|
actor.is_a?(User) && actor.allowed?
end
Enabled the feature for this group
feature = Flipper[:stats]
feature.enable_group :normal_users
But when check on users to see if it got enabled for users it doesn’t work:
feature.enabled?(current_user) => false
Also, when placing binding.pry inside allowed? method it never gets called.
On the other hand actors work just fine.
And when deleting this group from UI it says:
"The group named "normal_users" has not been registered."
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Did simple setup of installing gem and then mounting UI via routes.rb
Just trying it out in rails console. I'll put up a sample application for you but its just all basic installation stuff.
When creating groups, are they being persisted in database? As when i try to re-register group within a same rails console session, it gives me Flipper::DuplicateGroup: Group :admins has already been registered which is fine but when i quit rails console and start it up again and re-register group with the same name, it creates the group again.
Also, sometimes groups created from rails console dont get displayed in flipper UI groups dropdown.

@jnunemaker here's a test repo, checkout the readme for info of the the issue and setup.

Groups definitely work. I've never had this reported before and I use groups on a weekly basis without any problems. There has to be something unique about your setup. I'm not sure why you have the FlipperFeature model. That isn't necessary as it is in the AR adapter. The only thing you need is the migration.
The Flipper.register call should be somewhere that runs on boot. Do you have a config/initializers/flipper.rb that defines the group? If that is undefined then enabled would return false. That is likely your problem. Can you try that if you haven't and let me know?

I ran into this problem. In my case, the issue was that I was calling actor.is_a?(User), like OP. The problem is that actor is a Flipper::Types::Actor, so my check always returned false.
I fixed the issue by changing actor.is_a?(User) to actor.thing.is_a?(User), which checks the wrapped object's type instead of the wrapper's type.

Ah. Interesting. I usually check respond to instead of is a which is why I haven’t hit that issue. Thanks for sharing.

On Aug 26, 2020, at 5:00 PM, Ersin Akinci<EMAIL_ADDRESS>wrote:

I ran into this problem. In my case, the issue was that I was calling actor.is_a?(User), like OP. The problem is that actor is a Flipper::Types::Actor, so my check always returned false.
I fixed the issue by changing actor.is_a?(User) to actor.thing.is_a?(User), which checks the wrapped object's type instead of the wrapper's type.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or unsubscribe.


Change URL of ajax dynamically

Hi there,
great plugin! Works nice!
Is there a way to change the url of ajax dynamically?
I have got a task navigation and in order to reuse the autosave.js I need to change the ajax url according to the current opened task. (the task nav hides and shows the forms, so there is no page reload).

@patrickf0 you can pass in dynamic data to the ajax save handler but using a function instead of an object -- but that won't work for the url property. You could, alternatively, write your own ajax handler and use it instead of the default one.

@patrickf0 What I ended up doing was updating the <form> tag's action attribute dynamically and in the save callback I'd re-read the form's action attribute:
$("#my-form").autosave({
  callbacks: {
    save: {
      method: "ajax",
      options: {
        url: $("#my-form").attr('action')
      }
    }
  }
});


The link to Deploy-WindowsAutoPilot.pdf is not correct, the correct one is https://github.com/MicrosoftDocs/windows-itpro-docs/blob/master/windows/media/ModernSecureDeployment/Deploy-WindowsAutoPilot.pdf

[Enter feedback here]

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

ID: 4dd609f7-fa8a-a9ce-fe89-96b7c368ba1d
Version Independent ID: 6e372641-cb87-b500-5d10-20d68385aa1b
Content: Deploy Windows 10 - architectural posters
Content Source: windows/deployment/windows-10-architecture-posters.md
Product: w10
GitHub Login: @lizap
Microsoft Alias: elizapo

Yes it is fixed, I'm closing the issue, thank you greg-lindsay for noting it.


PWG 5100.18-2015: IPP Shared Infrastructure Extensions (INFRA)

Develop test files for PWG 5100.18-2015: IPP Shared Infrastructure Extensions (INFRA), to include:

[ ] Syntax and values/ranges of all attributes
[ ] Existence and support for required attributes and operations
[ ] Individual tests for submitting print jobs using different Job Template attributes
[ ] Individual tests for new and amended operations

Looks like this was done.


Ensure that a path can be found to a directive.

Pull request summary
Description:
Added:

None

Changed:

None

Removed:

None

Fixed:

None

Testing checklist:

[x] Changes are backward-compatible OR version migration code is included
[x] Codebase compiles with current tsconfig configuration
[x] Tested changes on {choose PUBLIC/PRIVATE} server OR changes are trivial (e.g. typos)

This is unnecessary -- useFindRoute is automatically determined if it is unspecified in this line: https://github.com/bencbartlett/Overmind/blob/master/src/movement/Pathing.ts#L80 Manually setting useFindRoute to true for short distances can actually yield less efficient paths. If you have a specific case where a directive is not finding a colony and it should, please create an issue and provide replay links and/or screenshots.

I assume this is worth opening an issue about? While this isn't the optimal solution, there should be one eventually?


Add package crud

#89  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @jespert and the rest of your teammates on  Graphite

Merge activity

Jun 11, 5:58 AM EDT: @jespert merged this pull request with Graphite.


Question about build

Environment：
OS = Ubuntu 20.04
ROS = Noetic
Python = 3.8.10

Hello. I have a question about the build.
I execute the following command.
sudo apt install libnlopt-dev libgoogle-glog-dev
git clone --recursive https://github.com/whoenig/uav_trajectories.git
mkdir uav_trajectories/build
cd uav_trajectories/build
cmake ..
make

but I get the following error and cannot proceed.
Is the uav_trajectory only available for Ubuntu 16.04?
Also, if you don't mind, could you please point out what I am doing wrong?

Error message 1
I got this when I first ran make.

Error message 2
I build nlopt.hpp which the cause of the error, separately and add uav_trajectories/mav_trajectory_generation/mav_trajectory_generation/include/mav_trajectory_generation/ directory.
After that, I ran make and the following message came up.

It's not finding nlopt.hpp on your system. What do you mean by "I build nlopt.hpp"?

Sorry for the confusing explanation.
In the first error, I got a message that nlopt.hpp was not found,
so I downloaded it from the official nlopt page and built it separately.
(from [https://github.com/stevengj/nlopt])
Then I moved the generated nlopt.hpp to the following directory.
uav_trajectories/mav_trajectory_generation/mav_trajectory_generation/include/mav_trajectory_generation/
In addition, changed the description in line 25 of polynomial_optimization_nonlinear.h from #include <nlopt.hpp> to #include "nlopt.hpp".
Then, I ran make again.

I solved the problem myself.
I was able to install uav-trajectory correctly by re-installing libnlopt-dev, libgoogle-glog-dev again, and rebuilding the nlopt.hpp file and placing it in bin/include.
Thanks for helping me.

Thanks for the feedback~


Create a new Color dialog for routes

I use the built-in colour picker to give a range of route colours.
Chuck wants a limited range of colours to display.
Therefore I need to create a customized colour selection component.

done


Implemented Themed help text-Part2 - Review Comment fix

Moved logic implemented in the main.go file to setup_cli/help_helpers.go and we changed CLI tests in the  integration_test_runner.go to unit tests(setup_cli/help_helpers_test.go) as suggested by Davidwadden. https://github.com/cloudfoundry-incubator/lattice/pull/119
In the current implementation when unknown command is invoked then its exiting with 0. To handle this we have added the following code.
https://github.com/HuaweiTech/lattice/blob/hwcf-issue-12/ltc/cli_app_factory/cli_app_factory.go#L98
Is this OK or this scenario have to be handled using panic?
PS:Original contributor @ShwethaKumbla

Coverage increased (+3.06%) to 97.59% when pulling ee66b4cc4059a00a7d1510feba72569cfeb4575c on HuaweiTech:hwcf-issue-12 into a9300f5a53b9c098142a9cb0098ecc226d612a5c on cloudfoundry-incubator:develop.

@ShwethaKumbla - thank you for salvaging our coverage :smile:
i took some notes after reviewing it on #95623780.  if you want to take another pass at it (no pressure), you're welcome to try and tackle some of these ideas.
i think the biggest one would be gained simply by setting cliApp.Writer = cliStdout in cli_app_factory.MakeCliApp,  then injecting a "fake writer"  i.e. gbytes.Buffer from test, then you can assert super easy on expected output from the cli.App, and we don't have that mutating state thing; this probably means asking you to shuffle all the code you just moved from main into setup_cli over into cli_app_factory.   what's your thought?
again, we appreciate the contribution
david

re: your question about putting os.Exit in the app.CommandNotFound
i think this is the right behavior.  error message and non-zero error code.  i don't think this scenario ought to panic
but, you can test that behavior by wiring it into the exit_handler.ExitHandler  and calling exitHandler.Exit(<exit_code>), similar to ltc target   (test).
note:  right now, sometime the actual implementation of the ExitHandler exits the app before it has a chance to hook into the shutdown, since it's slightly asynchronous.  we'll probably be committing a fix for this tomorrow.  #87638412
see:
$ ltc target wrong
Error verifying target: Get http://receptor.wrong/v1/desired_lrps: dial tcp: lookup receptor.wrong: no such host
✔ ~/workspace/diego-release/src/github.com/cloudfoundry-incubator/lattice/ltc [develop|✚ 1⚑ 1]
$ echo $?
0

Hi @davidwadden,
Its looks like pretty much done. I can see your commit #95623780.  If you want me to do any changes I ll be happy to do.
Thanks
@ShwethaKumbla

Yeah I took care of a bunch of it that was in cli_app_factory.  I mostly didn't touch the help_helpers.go under setup_cli/ though, aside from pulling up shared logic into the BeforeEach's.
I still think those static methods under setup_cli/help_helpers.go belong somewhere else, they're all only called from main package, right?  If so, let's move the help_helpers.go and help_helpers_test.go back into the main package, but still in their separate files.

@davidwadden
As per your suggstion, I will raise a PR for moving help_helper related code to main package tomorrow.
Thanks
@ShwethaKumbla


[Enhance Search filter]

I want to enhance the input search box
i have implmenteted and enhanced the search filters for our input box, i just need your permisson so i can push changes

Users can choose from our given list of keywords to find images, the search filter also suggest the possible keywords and also be able to show no options, if no keyword that user has typed matched to our list of keywords.

Room for improvement for asynchronous actions, can also get our list of keywords from our database and if they are predefined by us and have them on db , we can dynamically populate our keywords inside  search filter as the user start typing in the search box.

Output:

please allow me this task

Yeah that's a good thing
but does the suggestions come for every item searched?
also i would like to improve the UI of suggestion a little bit more

yes the suggestion popups for user, and also user can type inside this input and can also choose to open the dropdown and see possible keywords

Ok cool. Go for it 🚀


AlluxioJniFuseFileSystemTest#create fail

Alluxio Version:
2.8.1
Describe the bug
mvn clean test -Phadoop-2 -Dhadoop.version=2.7.3 -pl 'integration/fuse' -Dtest=alluxio.fuse.AlluxioJniFuseFileSystemTest#create
    when(mFileSystem.getStatus(any(AlluxioURI.class))).thenReturn(mock(URIStatus.class));

      return Optional.of(CommonUtils.waitForResult("file completed", () -> {
        try {
          return fileSystem.getStatus(uri);
        } catch (Exception e) {
          throw new RuntimeException(
              String.format("Unexpected error while getting backup status: %s", e));
        }
      }, URIStatus::isCompleted,
          WaitForOptions.defaults().setTimeoutMs(MAX_ASYNC_RELEASE_WAITTIME_MS)));`

Because the mock object of return is not completed, it will always be blocked in CommonUtils.waitForResult. The test case fails.
Possibly similar problems exist in write and flush test cases.
The test cases for Alluxio's pull request don't seem to include AlluxioJniFuseFileSystemTest.
To Reproduce
mvn clean test -Phadoop-2 -Dhadoop.version=2.7.3 -pl 'integration/fuse' -Dtest=alluxio.fuse.AlluxioJniFuseFileSystemTest#create

@LuQQiu Please correct me if I'm wrong.
This test suite AlluxioJniFuseFileSystemTest is known to be unstable / may fail, which is why we add the Ignore tag at the top of the file. The test cases inside are already covered by AbstractFuseIntegrationTest and JNIFuseIntegrationTest, which are indeed in the pull requests' tests.


ipv4, network tests: refactor the masquerade test table

What this PR does / why we need it:
Using a boolean parameter is usually a bad practice; as such, we now use directly the expected network CIDR, rather than a boolean which will be used to select the CIDR.
Release note:

NONE

/assign @EdDev @AlonaKaplan

/assign @EdDev @AlonaKaplan

/cc @phoracek

/cc @phoracek

I think this checks now only the custom CIDR and not the “default” one.
The fact that you explicitly define it, makes it custom.
/hold

I don' follow. This is doing exactly what it was doing before.

I think this checks now only the custom CIDR and not the “default” one.
The fact that you explicitly define it, makes it custom.
/hold

I don' follow. This is doing exactly what it was doing before.

I don' follow. This is doing exactly what it was doing before.

was it setting the cidr field if it was false?

I don' follow. This is doing exactly what it was doing before.

was it setting the cidr field if it was false?

I don' follow. This is doing exactly what it was doing before.

was it setting the cidr field if it was false?
EDIT: The masqueradeVMI was not getting any value, therefore using the default.

I see your point. Is it better now ?
I hadn't taken that into account since the tests entry names do not mention anything about default.

I don' follow. This is doing exactly what it was doing before.

was it setting the cidr field if it was false?
EDIT: The masqueradeVMI was not getting any value, therefore using the default.

I see your point. Is it better now ?
I hadn't taken that into account since the tests entry names do not mention anything about default.

I hadn't taken that into account since the tests entry names do not mention anything about default.

The flag name does mention default and custom.

I hadn't taken that into account since the tests entry names do not mention anything about default.

The flag name does mention default and custom.

I hadn't taken that into account since the tests entry names do not mention anything about default.

The flag name does mention default and custom.

ginkgo is a BDD framework, as such, it should describe stuff in plain English - or whatever spoken language it's tests are describing.
Here, we are (were ? I'm not even sure this patch addresses that) not doing that, and were relying on a parameter to transmit information to the reader - i.e. if you read a test report, you have no idea that this is testing a default value. You do have that knowledge for the custom CIDR - it says right there in the test string.
My 2 cents.

I hadn't taken that into account since the tests entry names do not mention anything about default.

The flag name does mention default and custom.

ginkgo is a BDD framework, as such, it should describe stuff in plain English - or whatever spoken language it's tests are describing.
Here, we are (were ? I'm not even sure this patch addresses that) not doing that, and were relying on a parameter to transmit information to the reader - i.e. if you read a test report, you have no idea that this is testing a default value. You do have that knowledge for the custom CIDR - it says right there in the test string.
My 2 cents.

/hold cancel

/hold cancel


OPSEXP-1802 Pre-Commit Hooks Best Practices in Build Tools

@gionn I have renamed the file to .md now. What else were you explaining me to check in order to avoid white spaces issue ?

Hi @alxgomz as Gio is out, could you please review and approve accordingly?

@alxgomz please review again

As Gio is on vacation so removed his review.


Great work, Lasse!

Hi Lasse,
I might have mentioned before that we love your work :-)
Not only the german Bonnier-staff is working with your onix-codelist but many more german publishers...
As editeur announced the codelist issue 39 today I wondered if changes coming from issue 38 are online?
Thanks again for spending your time on this project,
Marion

Hi Marion,
Thanks for the feedback! I also happen to work for Bonnier here in Finland. If the service is popular in Germany, maybe I should make this 🍺 ware. As you probably are aware, EDItEUR has now launched their own Onix codelist browser. So I might shutdown my hosted service at some point. But since the hosting doesn't that cost much, I am not in a hurry.
I've updated to the latest codelist issue (39) now.


autoconfiguration of networkId from blockchain provider url

This allows us to spare a configuration line .chainId.
We can just pass a valid blockchain provider and the resolution library will make a json rpc net_version call to the provider to update itself.
Before we had to use it like this:
DomainResolution resolution = Resolution.builder()
    .chainId(NamingServiceType.CNS, Network.GOERLI)
    .providerUrl(NamingServiceType.CNS, "https://goerli-light.eth.linkpool.io/")
   .build()

Now we achieve the same result via:
DomainResolution resolution = Resolution.builder()
    .providerUrl(NamingServiceType.CNS, "https://goerli-light.eth.linkpool.io/")
    .build()

I also added a public method to get the chainID of an instance Network getChainId(NamingServiceType type); and added some tests showing how it works.
In case of a wrong blockchainProvider we return to the default that has been set before.

LGTM.


Test-network error: "the supplied identity is not valid: x509: certificate signed by unknown authority"

I am follow instraction how to setup a network form https://hyperledger-fabric.readthedocs.io/en/latest/test_network.html#creating-a-channel but when i try to run exlorer on docker i get this error: 'Default client peer is down and no channel details available database'  and on the peer site "the supplied identity is not valid: x509: certificate signed by unknown authority (possibly because of "x509: ECDSA verification failure", so it looks like the certificates are bad but i check it like 10 times today and the paths looks good and file are the same like form the network.

After 15th time it work. i'am not sure but in this time I use this comment before run explorer:
./network.sh deployCC -ccn basic -ccp ../asset-transfer-basic/chaincode-go -ccl go


(demo): Check go env in fabric2 demo

Codecov Report

Merging #529 (0ce3fd0) into release-1.2.0 (908f954) will not change coverage.
The diff coverage is n/a.

@@               Coverage Diff                @@
##             release-1.2.0     #529   +/-   ##
================================================
  Coverage            15.55%   15.55%
  Complexity             440      440
================================================
  Files                  185      185
  Lines                 8684     8684
  Branches               747      747
================================================
  Hits                  1351     1351
  Misses                7215     7215
  Partials               118      118


Adding Cobertura as an  option for Publishing Code Coverage Results

Problem: There is not support for publishing Cobertura code coverage summary files in the PublishCodeCoverageTask today. This change adds support for that.
Fix: Adding Cobertura as an option in the supported tools drop down.
Testing Done: UTs and manual testing.

shouldnt there be an update to the task.loc.json too?


ext/hal/st/stm32cube/stm32l1xx: rename SVC_IRQn -> SVCall_IRQn

SVCall_IRQn is used as enum value for SV Call Interrupt for Cortex-M
based SoCs for which CPU_CORTEX_M_HAS_BASEPRI=y. However stm32cube
for stm32l1xx uses SVC_IRQn enum value for this purpose.
This leads to this error:
  arch/arm/include/cortex_m/exc.h:101:19: error: 'SVCall_IRQn'
    undeclared (first use in this function); did you mean 'SVC_IRQn'?

    NVIC_SetPriority(SVCall_IRQn, _EXC_SVC_PRIO);
                     ^~~~~~~~~~~
                     SVC_IRQn

NB: ext/hal/st/stm32cube/stm32l1xx/soc/stm32l151xb.h file was already
fixed in 9f8260457b86 ('ext: hal: st: stm32cube: Add HAL for the STM32L1x series').

Thanks for submitting.

NB: ext/hal/st/stm32cube/stm32l1xx/soc/stm32l151xb.h file was already
fixed in 9f82604 ('ext: hal: st: stm32cube: Add HAL for the STM32L1x series').

Sorry if this was not clear but please fix them all at once in current PR.

@erwango

Add fix for stm32l151xb.h

The ext/hal/st/stm32cube/stm32l1xx/soc/stm32l151xb.h file was already
fixed in 9f8260457b86 ('ext: hal: st: stm32cube: Add HAL for the STM32L1x series').
The commit 9f8260457b86 is already merged into the master.
I can't fix it if it is already fixed!
I can make PR with two commits:

the first commit reverts fix of ext/hal/st/stm32cube/stm32l1xx/soc/stm32l151xb.h;
the second commit fix all header files in ext/hal/st/stm32cube/stm32l1xx/soc/ at once.

The problem is that the first commit will break 96b_wistrio board.

@frantony, thanks for clarification and my bad then. Twice my bad, since I reviewed and approved L1 porting.
Thanks for your proposal, but let's do it simple:
Just add the required changes to extend the fix to the whole series.
Main point is that it is documented in README as you just did.
This is only temporary as hopefully cube package will be fixed and updated in a reasonable time frame.

@frantony

Just add the required changes to extend the fix to the whole series.
Main point is that it is documented in README as you just did.

I have just done it.


Parallelize sensor search

Tech spike to increase chart loading speed through parallelization. I'm using n=4 parallel processes here, but that could become a config setting.
@Ahmad-Wahid could you do a speed benchmark on this branch (compared to main) for me? Some manual tests of mine (locally) suggested an asset chart with 4 sensors and 1 week of data loads about twice as fast. I'm interested to see whether you can reproduce that result on a cloud server, and how the speed-up factor would change with a larger time frame and with more sensors.

sure, I will do.

Docker image cannot be build, as failing in the test. Locally, I could build it but making it live causing issue.

[FLEXMEASURES][2024-01-16 14:59:19,356] ERROR: ConnectTimeout:"HTTPConnectionPool(host='*******', port=80): Max retries exceeded with url: //api/v3_0/assets?account_id=4 (Caused by ConnectTimeoutError(<urllib3.connection.HTTPConnection object at 0x7fc41d9d2200>, 'Connection to **** timed out. (connect timeout=None)'))" [occurred at /usr/local/lib/python3.10/dist-packages/requests/adapters.py(send):507,URL was: /assets/]
Traceback (most recent call last):
File "/usr/local/lib/python3.10/dist-packages/urllib3/connection.py", line 203, in _new_conn
sock = connection.create_connection(
File "/usr/local/lib/python3.10/dist-packages/urllib3/util/connection.py", line 85, in create_connection
raise err
File "/usr/local/lib/python3.10/dist-packages/urllib3/util/connection.py", line 73, in create_connection
sock.connect(sa)
TimeoutError: [Errno 110] Connection timed out
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/usr/local/lib/python3.10/dist-packages/urllib3/connectionpool.py", line 790, in urlopen
response = self._make_request(
File "/usr/local/lib/python3.10/dist-packages/urllib3/connectionpool.py", line 496, in _make_request
conn.request(
File "/usr/local/lib/python3.10/dist-packages/urllib3/connection.py", line 395, in request
self.endheaders()
File "/usr/lib/python3.10/http/client.py", line 1278, in endheaders
self._send_output(message_body, encode_chunked=encode_chunked)
File "/usr/lib/python3.10/http/client.py", line 1038, in _send_output
self.send(msg)
File "/usr/lib/python3.10/http/client.py", line 976, in send
self.connect()
File "/usr/local/lib/python3.10/dist-packages/urllib3/connection.py", line 243, in connect
self.sock = self._new_conn()
File "/usr/local/lib/python3.10/dist-packages/urllib3/connection.py", line 212, in _new_conn
raise ConnectTimeoutError(
urllib3.exceptions.ConnectTimeoutError: (<urllib3.connection.HTTPConnection object at 0x7fc41d9d2200>, 'Connection to **** timed out. (connect timeout=None)')
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/usr/local/lib/python3.10/dist-packages/requests/adapters.py", line 486, in send
resp = conn.urlopen(
File "/usr/local/lib/python3.10/dist-packages/urllib3/connectionpool.py", line 844, in urlopen
retries = retries.increment(
File "/usr/local/lib/python3.10/dist-packages/urllib3/util/retry.py", line 515, in increment
raise MaxRetryError(_pool, url, reason) from reason  # type: ignore[arg-type]
urllib3.exceptions.MaxRetryError: HTTPConnectionPool(host='****', port=80): Max retries exceeded with url: //api/v3_0/assets?account_id=4 (Caused by ConnectTimeoutError(<urllib3.connection.HTTPConnection object at 0x7fc41d9d2200>, 'Connection to **** timed out. (connect timeout=None)'))
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/lib/python3.10/dist-packages/flask/app.py", line 1823, in full_dispatch_request
rv = self.dispatch_request()
File "/usr/local/lib/python3.10/dist-packages/flask/app.py", line 1799, in dispatch_request
return self.ensure_sync(self.view_functions[rule.endpoint])(**view_args)
File "/usr/local/lib/python3.10/dist-packages/flask_classful.py", line 303, in proxy
response = view(**request.view_args)
File "/usr/local/lib/python3.10/dist-packages/flask_classful.py", line 271, in inner
return fn(*args, **kwargs)
File "/usr/local/lib/python3.10/dist-packages/flask_login/utils.py", line 290, in decorated_view
return current_app.ensure_sync(func)(*args, **kwargs)
File "/app/flexmeasures/ui/crud/assets.py", line 213, in index
assets += get_assets_by_account(account.id)
File "/app/flexmeasures/ui/crud/assets.py", line 182, in get_assets_by_account
get_assets_response = InternalApi().get(
File "/app/flexmeasures/ui/crud/api_wrapper.py", line 53, in get
response = requests.get(
File "/usr/local/lib/python3.10/dist-packages/requests/api.py", line 73, in get
return request("get", url, params=params, **kwargs)
File "/usr/local/lib/python3.10/dist-packages/requests/api.py", line 59, in request
return session.request(method=method, url=url, **kwargs)
File "/usr/local/lib/python3.10/dist-packages/requests/sessions.py", line 589, in request
resp = self.send(prep, **send_kwargs)
File "/usr/local/lib/python3.10/dist-packages/requests/sessions.py", line 703, in send
r = adapter.send(request, **kwargs)
File "/usr/local/lib/python3.10/dist-packages/requests/adapters.py", line 507, in send
raise ConnectTimeout(e, request=request)
requests.exceptions.ConnectTimeout: HTTPConnectionPool(host='****l', port=80): Max retries exceeded with url: //api/v3_0/assets?account_id=4 (Caused by ConnectTimeoutError(<urllib3.connection.HTTPConnection object at 0x7fc41d9d2200>, 'Connection to **** timed out. (connect timeout=None)'))

@Flix6x Wouldn't this PR also improve our simulation work (i.e. are we often querying multiple sensors at once there?).
Of course, we browse data there, as well, so the tooling would benefit. I'm guessing the analysis part we're doing there (using reporters to compute KPIs across sensors) would improve speed-wise with this.
I'm trying to say that this is a neglected high-value PR, which we should put in the planning for the next weeks.

Docker image cannot be build, as failing in the test. Locally, I could build it but making it live causing issue.

Maybe this is the blocking issue?

The test steps and the build step failed after 6 hours. I tried rerunning the falling steps in the pipeline on the same day, and am today retrying just one of the four test steps (for different Python versions) to see if the issue still persists.


docs: bump version, change release format

Description
Updated version because 1.0.1 doesn't support some of the types below. Shouldn't have to get updated every time release-please is bumped, but should be somewhat relevant.
Also made the release type somewhat less confusing (IMO) for people starting out.

@googlebot I signed it!

Thank you @jef,  appreciate the contribution.


Show contents of notification when phone is locked

When phone is locked it shows "Contents Hidden".
Is there any way to override the phone settings via code/payload?
Platform and Version Android 5.0
Android 5.0
Cordova CLI version and cordova platform version
cordova --version       6.0.0
cordova platform version android    4.1.1

Plugin version
cordova plugin version 1.6.1

@rohand I'm curious what is your phone set to show?
Head to Settings and click Sound & Notification. There are three options to choose from in the "When device is locked" option at the bottom of the page: show all notifications, don't' show notifications at all and hide sensitive notification information. The last option -- which will still show an incoming notification, but not any of the content -- will only be available if you have a screen lock pin, pattern or password set up.
That said there is a way to set a notification public, private or secret so I'll add this as an enhancement.

you save my time.... Thank u @rohand


Investigate possibility of not using Expression Trees

Investigate possibility of not using Expression Trees:

How does it impacts performance?
Does it result in better maintainability?

Related rexm/Handlebars.Net/issues/345

Related zjklee/Handlebars.CSharp/issues/19


[SPARK-49070][SS][SQL] TransformWithStateExec.initialState is rewritten incorrectly to produce invalid query plan

What changes were proposed in this pull request?

This patch fixes TransformWithStateExec so when its hasInitialState is false, the initialState won't be rewritten by planner incorrectly to produce invalid query plan which will cause unexpected errors for extension rules that rely on the correctness of query plan.
Why are the changes needed?

SPARK-47363 added the support for users to provide initial state for streaming query. Such query operators like TransformWithStateExec might have hasInitialState as false which means the initial state related parameters are not used. But when query planner applies rules on the query, it will still apply on the initial state query plan. When hasInitialState is false, some related parameters like initialStateGroupingAttrs are invalid and some rules will use these invalid parameters to transform the initial state query plan.
For example, EnsureRequirements may apply invalid Sort and Exchange on the initial query plan. We encountered these invalid query plan in our extension rules.
Does this PR introduce any user-facing change?

No
How was this patch tested?

Unit test
Was this patch authored or co-authored using generative AI tooling?

No

Hello, I am Mr. Jeon, a developer in Korea.
I would like to become a Committer for Apache Spark. Could you please give me some advice on how to achieve this?

Hello, I am Mr. Jeon, a developer in Korea. I would like to become a Committer for Apache Spark. Could you please give me some advice on how to achieve this?

Hi, @JeonDaehong . Here is the community guide. However, this kind of Q&A is irrelevant to this PR. So, please use the official mailing list for the general questions<EMAIL_ADDRESS>.

https://spark.apache.org/contributing.html

안녕하세요, 저는 한국에 있는 개발자 전씨입니다. Apache Spark의 커미터가 되고 싶습니다. 이것을 달성하는 방법에 대한 조언을 주시겠습니까?

안녕하세요. 다음은 커뮤니티 가이드입니다. 그러나 이러한 종류의 Q&A는 이 PR과 관련이 없습니다. 따라서 일반적인 질문은 공식 메일링 리스트를 사용하십시오<EMAIL_ADDRESS>
https://spark.apache.org/contributing.html

Thank you for your help.
I am very interested in Spark and want to become a committer, but I didn't know how, so I sought advice here. I will refer to the links you provided and work hard on it.
Thank you so much. :D

Thank you @dongjoon-hyun

Merged to master. Thanks @viirya!
Also thanks @dongjoon-hyun @anishshri-db @jingz-db for reviewing!

Thank you all!

Thank you all!


Syntax error

Hi!
I recently found your repository and would really like to use it, but there is one thing, but for some reason your code does not want to work at the same time in Visual Studio Code with plugins, it displays normally without a single error, but if I want to run it, then I teach an error with syntax!
Error:
(232) : ==> Syntax error.

Hi! I recently found your repository and would really like to use it, but there is one thing, but for some reason your code does not want to work at the same time in Visual Studio Code with plugins, it displays normally without a single error, but if I want to run it, then I teach an error with syntax!
Error: (232) : ==> Syntax error.

Thank you for reporting this issue. It has been resolved at the following commit: d24ba6b18b5ba99521e1ce748b15ff2b2a785a3e.


Dot files for better images

Hi,

Is it possible to remove the id's when printing the dots? I just want the variable name.
Nice Project btw.

Hi,
Sorry I first see this now - I have somehow set up my email notifications for this repo incorrectly. Yes, this should be easy to fix; I'll look at it tomorrow.

I have added an optional last argument include_id (type bool) which by default is false. That is, bdd_printdot(f, ..., false)/bdd_printdot(f, ...) render a result as follows

As you can see, I also made the internal nodes round; this way it follows the conventional depiction of BDDs
Using bdd_printdot(f, ..., true) behaves as before (for development debugging purposes). But, I removed the id in the print, to better align the output with the paper(s).


using 'exchange' argument is not processed

It seem that the dtc_download() function argument exchange="anything" is not used?
data = dtc_download('EURUSD', exchange='anyGarbageOk', start_date='2020.12.20', server=("<IP_ADDRESS>", 11098), debug=True)

It is translated to Sierra Chart as is in the HISTORICAL_PRICE_DATA_REQUEST. It should be seen in the debug mode. I believe it is ignored by SC for some symbols.

Optional: The exchange for the Symbol.


Make bytestring, text and vector optional dependencies

I'd like to use nothunks in test suites of bytestring and text, but currently it is impossible because of circular dependencies. It's also comparably expensive to build a non-boot library vector in such circumstances. This PR adds flags, which make bytestring, text and vector optional dependencies, thus providing a user with a greater flexibility to derive a build plan.

@edsko yes, please, a release would be much appreciated.

Done! :) https://hackage.haskell.org/package/nothunks-0.1.3

The automatic flags was a mistake. Issue #17


Docs: Edit README, add tutorial

Preview link
Adds new Usage section covering $, $N, $$.

Replaces Basic Utilities section.

Consistent formatting for all instances of "hkt-toolbelt", "TypeScript"
Import section provides full enumeration of user-facing modules.
Removed all instances of hk-type
General edits for grammar, conciseness, impact.

This is great! :D


cmd run script issue after running setup (openAI resources instead of pure azure)

Hello I'm attempting to use the setup and run .cmd files in the service folder. My configuration is using OpenAI instead of azure.
On running "run" in command prompt it outputs and error about a blank "deploymentName" which I understand as an azure only required environment variable. I looked through the appsettings.Development.json produced when running setup and don't see it there and I wouldn't expect this would be needed to run.
Build succeeded.
    0 Warning(s)
    0 Error(s)

Time Elapsed 00:00:03.94
Unhandled exception. System.ArgumentException: The value cannot be an empty string or composed entirely of whitespace. (Parameter 'deploymentName')
   at Microsoft.SemanticKernel.Diagnostics.Verify.ThrowArgumentWhiteSpaceException(String paramName)
   at Microsoft.SemanticKernel.Connectors.AI.OpenAI.AzureSdk.AzureOpenAIClientBase..ctor(String deploymentName, String endpoint, TokenCredential credential, HttpClient httpClient, ILoggerFactory loggerFactory)
   at Microsoft.SemanticKernel.Connectors.AI.OpenAI.TextEmbedding.AzureOpenAITextEmbeddingGeneration..ctor(String deploymentName, String endpoint, TokenCredential credential, String modelId, HttpClient httpClient, ILoggerFactory loggerFactory)
   at Microsoft.KernelMemory.DependencyInjection.<>c__DisplayClass0_0.<AddAzureOpenAIEmbeddingGeneration>b__0(IServiceProvider serviceProvider) in C:\Users\jcf04\Desktop\kernel-memory\service\Core\AI\AzureOpenAI\DependencyInjection.cs:line 38
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteVisitor`2.VisitCallSiteMain(ServiceCallSite callSite, TArgument argument)
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteRuntimeResolver.VisitRootCache(ServiceCallSite callSite, RuntimeResolverContext context)
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteVisitor`2.VisitCallSite(ServiceCallSite callSite, TArgument argument)
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteRuntimeResolver.Resolve(ServiceCallSite callSite, ServiceProviderEngineScope scope)
   at Microsoft.Extensions.DependencyInjection.ServiceProvider.CreateServiceAccessor(ServiceIdentifier serviceIdentifier)
   at System.Collections.Concurrent.ConcurrentDictionary`2.GetOrAdd(TKey key, Func`2 valueFactory)
   at Microsoft.Extensions.DependencyInjection.ServiceProvider.GetService(ServiceIdentifier serviceIdentifier, ServiceProviderEngineScope serviceProviderEngineScope)
   at Microsoft.Extensions.DependencyInjection.ServiceProvider.GetService(Type serviceType)
   at Microsoft.Extensions.DependencyInjection.ServiceProviderServiceExtensions.GetService[T](IServiceProvider provider)
   at Microsoft.KernelMemory.KernelMemoryBuilder.FromConfiguration(KernelMemoryConfig config, IConfiguration servicesConfiguration) in C:\Users\jcf04\Desktop\kernel-memory\service\Core\KernelMemoryBuilder.cs:line 285
   at Microsoft.KernelMemory.KernelMemoryBuilder.FromAppSettings(String settingsDirectory) in C:\Users\jcf04\Desktop\kernel-memory\service\Core\KernelMemoryBuilder.cs:line 198
   at Program.<Main>$(String[] args) in C:\Users\jcf04\Desktop\kernel-memory\service\Service\Program.cs:line 80

Just an update .. apparently only an issue when I run the run.cmd file in the Services directory.
with the follwing launch.json configuration in vscode.. it works..
{
    // Use IntelliSense to learn about possible attributes.
    // Hover to view descriptions of existing attributes.
    // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
    "version": "0.2.0",
    "configurations": [
        {
            "name": ".NET Core Launch (web)",
            "type": "coreclr",
            "request": "launch",
            "preLaunchTask": "build",
            "program": "${workspaceFolder}/service/Service/bin/Debug/net7.0/Microsoft.KernelMemory.ServiceAssembly.dll",
            "args": [],
            "cwd": "${workspaceFolder}/service/Service",
            "stopAtEntry": false,
            "serverReadyAction": {
                "action": "openExternally",
                "pattern": "\\bNow listening on:\\s+(https?://\\S+)"
            },
            "env": {
                "ASPNETCORE_ENVIRONMENT": "Development"
            },
            "sourceFileMap": {
                "/Views": "${workspaceFolder}/Views"
            }
        },
        {
            "name": ".NET Core Attach",
            "type": "coreclr",
            "request": "attach"
        }
    ]
}

hi @jcf0419 we just merged some changes touching configuration and starting options. hopefully those might help. You could also try running dotnet run setup to generate a configuration file

Please let us know if the issue is still present, we can reopen the issue. Thanks


Official pfe-band status

[x] schema
[x] theme support
[ ] framework support

[ ] Angular
[ ] React
[ ] Vue

[ ] no major outstanding bugs

T5 FED story for band layout issue
Needs bug fixed for FF layout issues (Issue 330)

This will be merged this sprint


UML model folder needed

You must push to your own repository, not to mine. Please fork the repo and work in that


data ingestion endpoint proto

batch upload data from google sheet

Each row is matched to the Data proto message based on the columns from this sheet: https://docs.google.com/spreadsheets/d/1MWVXNHXgLdTHHTVmpi9HmQ3DRV6uU3CWfHdxshUeEUU/edit?gid=0#gid=0

@zackxzr

Not much else to add after the changes @Lev1ty requested.


Support for streaming using io.Writer and io.Reader

I have the need for steaming using io.Writer and io.Reader. I am willing to provide a pull request given this feature matches the goal for this library.

The encoder/decoder core are both about 30 lines, I feel like supporting streaming operations would complicate them significantly.
Why are you encoding large amounts of data? The whole point of zbase32 is to be human-friendly; a wall of random characters is very much not human friendly.

I am working on a tool inspired by http://www.jabberwocky.com/software/paperkey/ with a few twists.

Instead of the hex encoding, I intend to use z-base-32 for obvious reasons.
Instead of being limited by to GPG keys, I intend to support what ever input the user wants to use.
I will add secret sharing using the Shamir method.

Since I do not have any control over the input and the Shamir algorithm will multiply that input data, streaming seems to be the way to go in order to keep the memory footprint on sane levels.
What I have in mind, looks more or less the same to what can be found in encoding/base32.

Surely a single cryptographic key fits in RAM at once, even after Shamir, and you don't need io.Writer.
Even on top of that, I would recommend you use zbase32 to encode short chunks, with some sort of integrity protection. This way, you can communicate to the human where exactly they have a typo/OCR flaw when re-entering the data.

Ok, agree the memory is obviously not that much of an issue as I initially thought. Let us try that from a different angle.

Why are you encoding large amounts of data? The whole point of zbase32 is to be human-friendly; a wall of random characters is very much not human friendly.

While zbase32 is meant to be human friendly, there is nithing that prohibits other use cases. If I would like to handle huge amount of data, why should I not be able to do so. I consider zbase32 as a drop in replacement for regular base32 or any other baseX encoding. Having an io.Reader or io.Writer interface allows me to try different encodings by changing one single line of code.

Even on top of that, I would recommend you use zbase32 to encode short chunks, with some sort of integrity protection. This way, you can communicate to the human where exactly where they have a typo/OCR flaw when re-entering the data

Agreed, adding integrity protection is a crucial part of this. I handle this in a reader and writer on its own where the bytes are written by or from the zbase32 reader and writer. My first approach on this was, to handle that all in one place. To be honest, it was a mess of deeply nested code and I started to refactor things and I ended using the io.Reader and io.Writer interfaces. The result is a code structure of loosely coupled and reusable components.
As for now, I have taken the liberty to fork your code. See: https://github.com/corvus-ch/zbase32.
If you are still not interested to accept this as a PR, feel free to close this issue. Otherwise, I will happily change my code to match yours and file a PR.

I'll accept addition of Reader/Writer apis if the core code manages to remain simple & maintainable enough. That is my only concern.

As it is, you seem to have not forked my code, but copy-pasted parts of it, stripping my copyright. You realize that's illegal, right?

[…] copy-pasted parts of it, stripping my copyright […]

Actually I am unsure how much needs to be done in order to be compliant. If you can point me in the right direction, I will gladly fix that.

I'll accept addition of Reader/Writer apis if the core code manages to remain simple & maintainable enough. That is my only concern.

I will see, what I can do. Maybe you can have a look at my code and point out places where it does not meet your requirement.

This seems to just add too much code and complexity into a simple library. I'll repeat what I said earlier:

Even on top of that, I would recommend you use zbase32 to encode short chunks, with some sort of integrity protection. This way, you can communicate to the human where exactly where they have a typo/OCR flaw when re-entering the data.


rekor-cli, rekor-server: init at 0.1.1

Motivation for this change
Introduce Rekor CLI and server at their latest version. Rekor is part of the sigstore project.
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 117805 at 1341c9d3 run on aarch64-linux 1

2 packages built successfully:

rekor-cli
rekor-server


Feature: Reload on Button / URDF File Change

Hey there,
thank you for this great project, it is very helpful to me! :)
I have a very small enhancement that would speed up my workflow a lot (and I would add it myself if I knew any Rust).
It would be great if there was a button/keybind to reload the URDF file from disk, so that I have a quick way to see whether my manual tinkering in the file produced the correct results in 3D.
Even better would be a file watch to automatically update the visualization, but I guess that is way more effort ;)
Thanks a lot for considering it!

As a workaround I find the opening urdf-viz from the console and pressing ctrl-c, up arrow and enter to reload urdf-viz with the same command is pretty quick when I’m making lots of changes to a Urdf.

'L' key is to reload the urdf from file. (See the README.md carefully)
(I'm sorry if i missed your point.)
It's not easy to add buttons.
It might be possible to automatically reload the file, and it seems to be easy.
Thank you for nice suggestions!


Is there any best practice of anatomy of a script like hubot?

Would botkit support for anatomy of a script like hubot? My controller is getting bigger when adding a new listener. It's not good idea to write everything in a single file. I'm thinking to make something, but if your team have any idea or suggestion, could you please share it?

Added sample code

// bot.js(main file)
var Botkit = require('botkit');
controller = Botkit.slackbot({});

controller.spawn({
  token: process.env.SLACK_TOKEN,
}).startRTM();

// I'd like to split each listener below in a file.

// ping.js
// first listener
controller.hears([/^ping$/i], ['direct_message', 'direct_mention', 'mention'],
  function(bot, message) {
    bot.reply(message, 'pong');
});

// ping2.js
// second listener
controller.hears([/^ping2$/i], ['direct_message', 'direct_mention', 'mention'],
  function(bot, message) {
    bot.reply(message, 'pong2');
});

// and more...
// controller.hears(...

hubot way
https://hubot.github.com/docs/scripting/
Thanks

Added codes to clear my question.

Now I take the following way to load files under scripts dir. The idea is from hubot's robot.coffee
https://github.com/github/hubot/blob/9fe938f35362e20f7c0e3da57370a3ec9bc5a490/src/robot.coffee#L344
The bad thing is to use global object to share the slackbot instance. I wanted to load files in CoreBot.js for making the file loading as core function, but didn't find a way. Do you have any suggestions or ideas?
const Fs = require('fs');
const Path = require('path');

global.controller = Botkit.slackbot();

controller.spawn({
  token: process.env.SLACK_TOKEN,
}).startRTM();

let loadFile = function(path, file) {
  let ext = Path.extname(file);
  let full = Path.join(path, Path.basename(file, ext));

  if(require.extensions[ext]) {
    try {
      require(full);
    } catch(_err) {
      let err = _err;
      console.log(`Unable to load ${file}: ${err.stack}`);
      return process.exit(1);
    }
  }
};

let path = Path.resolve('.', 'scripts');
Fs.stat(path, function(err, stat) {
  if(err) {
    console.log(err.message);
  } else {
    console.log(`Loading scripts from ${path}`);
    let ref = Fs.readdirSync(path).sort();

    for(let i = 0, len = ref.length; i < len; i++) {
      let file = ref[i];
      loadFile(path, file);
    }
  }
});

Closed due to inactivity, question answered.

I appreciate if any feedbacks!

You should also posit this question on dev4slack, much more developer traffic there!

oh, thanks!


noisy console output

Currently, kintone-plugin-packer command print the following logs, which are unexpected output.
schema $id ignored https://github.com/epoberezkin/ajv/blob/master/lib/definition_schema.js
schema $id ignored https://github.com/epoberezkin/ajv/blob/master/lib/definition_schema.js
schema $id ignored https://github.com/epoberezkin/ajv/blob/master/lib/definition_schema.js
schema $id ignored https://github.com/epoberezkin/ajv/blob/master/lib/definition_schema.js
schema $id ignored https://github.com/epoberezkin/ajv/blob/master/lib/definition_schema.js
schema $id ignored https://github.com/epoberezkin/ajv/blob/master/lib/definition_schema.js
schema $id ignored https://github.com/epoberezkin/ajv/blob/master/lib/definition_schema.js
schema $id ignored https://github.com/epoberezkin/ajv/blob/master/lib/definition_schema.js
schema $id ignored https://github.com/epoberezkin/ajv/blob/master/lib/definition_schema.js
schema $id ignored https://github.com/epoberezkin/ajv/blob/master/lib/definition_schema.js
Succeeded: dist/plugin.zip

That's because of kintone/plugin-manifest-validator, so I created an issue on that repository.
https://github.com/kintone/plugin-manifest-validator/issues/227

Released a fixed version as 2.0.5
https://github.com/kintone/plugin-packer/blob/master/CHANGELOG.md#205-2019-10-02


Breakpoint not shown in gutter on active line

Issue Type: Bug

Set a breakpoint in a js file
Start debugging and hit the breakpoint

Bug
Notice that the breakpoint is not shown in the left gutter:

Stepping to the next line, the breakpoint becomes visible again
VS Code version: Code - Insiders 1.28.0-insider (2558a72d6902015b142ef45c57b664ce1f09e03f, 2018-09-21T05:12:46.853Z)
OS version: Darwin x64 17.7.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4770HQ CPU @ 2.20GHz (8 x 2200)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledrasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
4, 3, 3

Memory (System)
16.00GB (0.05GB free)

Process Argv
/Applications/Visual Studio Code - Insiders.app/Contents/MacOS/Electron -psn_0_11406048

Screen Reader
no

VM
22%

Can not repro. What are you doing differently than me?
Can you repro in a simple javascript file?
Is the breakpoint red before you hit it?
If you can only repro in a specific situation please give me steps

Ok I'm not able to repo anymore either. Will let you know if I hit this again.


Confusing steps

This learn path was one of the paths recommended to me after I failed AZ-204.
I tried following these steps.  They are really confusing. There is an early reference to using a "dedicated plan" which does not appear to exist anywhere obvious - certainly not on the Function App Plan dropdown, although I did try "App Service Plan". There are then mysterious subsequent references to "App Service Editor" which doesn't appear to exist.  This in turn may explain why file editing is hard to achieve. I got round this by creating files locally and uploading them.
This met with some success.  However, the provided function.proj XML is then reported as being incorrect in some way. This causes the ApplicationInsights nuget package install to not be applied, so of course the Function App itself cannot compile.
After at least 30 minutes, I gave up and tried Visual Studio instead.  I created an empty C# function app locally, added the nuget package, pasted the Learn path code with appropriate adjustments and it worked first time. This took me 10 minutes.
Unfortunately, I no longer have the logs or original files. These all got deleted when I published from Visual Studio.
I'm glad I persisted, and I did learn something.
It may be I missed something obvious, but on the whole it was a rather discouraging and time-consuming experience.

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

ID: 6be60b15-5e41-4a0f-41a2-a592afc4ff45
Version Independent ID: aa652576-5e64-2cbf-e556-ca9c7fd91b83
Content: Create and run custom availability tests by using Azure Functions - Azure Monitor
Content Source: articles/azure-monitor/app/availability-azure-functions.md
Service: azure-monitor
Sub-service: application-insights
GitHub Login: @AaronMaxwell
Microsoft Alias: aaronmax

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

@AaronMaxwell For help!


Remarks in Contact is being hidden under NavBar

Describe the bug
The paragraph tag in Contact section is being overlapped by NavBar and is not visible.
To Reproduce
Steps to reproduce the behavior:

Go to 'Contact Section'
See error

Expected behavior
The paragraph tag should be visible. Margin needs to be fixed.
Screenshots

Desktop (please complete the following information):

OS: [Mac OS]
Browser [Google Chrome]

Thanks for raising this.


Issue while setup istio-CSR

I am trying to setup Istio-CSR. But getting following issue while deploying istio-CSR helm chart:
Command to setup from docs:
   helm install -n cert-manager cert-manager-istio-csr jetstack/cert-manager-istio-csr \
  --set "app.tls.rootCAFile=/var/run/secrets/istio-csr/ca.pem" \
  --set "volumeMounts[0].name=root-ca" \
  --set "volumeMounts[0].mountPath=/var/run/secrets/istio-csr" \
  --set "volumes[0].name=root-ca" \
  --set "volumes[0].secret.secretName=istio-root-ca"

Error: template: cert-manager-istio-csr/templates/serviceaccount.yaml:3:7: executing "cert-manager-istio-csr/templates/serviceaccount.yaml" at <ne .Values.image.pullSecretName "">: error calling ne: incompatible types for comparison
Details:
Istio: v1.13.2
cert-manager: v1.6.1
helm: v3.5.2
Is there anything I missing?

Looks like this issue is coming while deploying latest(0.5.0) helm chart of cert-manager-istio-csr. If you use cert-manager-istio-csr of version v0.4.2 this issue is not coming. I don't exact reason of this failure in latest release.
Is there any changes in deployment command?
Thanks

Hi @shinderupesh, I think this might be a problem with Helm rather than the Chart.
I have been able to reproduce the error when using the same version as yourself (v3.5.2), however the error is not present when using at least v3.8.2.
Can you update your helm installation to a higher version and test whether you get the same?

Hi @JoshVanL, yes I am able to use Istio-CSR(v0.5.0) with Helm v3.8.2.
Thanks for the response.


allow access to x labels

allow access to x labels which allows to do a lot of interesting things, for example colour each label differently or add rotation to them

thx


[feat] Improve polish prompt

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

Comparison is base (f71a0b0) 62.28% compared to head (e187116) 61.83%.
Report is 1 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #180      +/-   ##
===========================================
- Coverage    62.28%   61.83%   -0.46%
===========================================
  Files           56       56
  Lines         2678     2709      +31
===========================================
+ Hits          1668     1675       +7
- Misses        1010     1034      +24

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


Kubernetes instructions are out-of-date and no longer work

Update the kubernetes instructions to create a working reproducible build.
Add a minikube deployment folder/instructions

It'd be nice to build a helm chart for this

For clarity, the configuration in: https://github.com/microservices-demo/microservices-demo/tree/master/deploy/kubernetes is working. The documentation & instructions are the TODO here.

Same as #286

Fixed by @nustiueudinastea. See minikube PR https://github.com/microservices-demo/microservices-demo/pull/464/files
K8s on AWS was tested and works.


gomobile fixes

This PR contains changes for gomobile to build frameworks for iOS and Android. It is currently configured to build the framework using a Bitcoin Cash SPV wallet.
Additional changes are required to make the selection of the wallet configurable in \mobile\config.go.

Coverage remained the same at 26.147% when pulling e31a4462dcf1bb60191c18bf0fcdb19587e93de3 on drwasho:go_mobile_bch into 0a452254672bd47b668c1758d79b2adb4e3ef931 on OpenBazaar:master.

@cpacia changes made

@drwasho can you please address @cpacia issues?

@hoffmabc I already did, @cpacia hasn't updated his review

@drwasho can you help me understand what delta here that still needs to get into master? Or can we close?

@placer14 the current mobile folder was never configured correctly, as Chris can attest to; the only way we ever managed to get it to work was by making changes that we part of this PR.. but no one ever got around to merging them at the time. After a while it became clear that we needed to wait for multiwallet to be finished and make similar adjustments if the mobile frameworks were failing to build or function properly.
We could close this PR and prepare a fresh new one if these changes are necessary.

Archiving.


Why UDPReceiver class receive method packet.getLength() == 0 is returned ?

Description

Why UDPReceiver class  receive method  packet.getLength() == 0  is returned ?
Why not continue ?
our scenes is LB health check will be send 0 length packet
Environment

Pinpoint version: 1.6.2
OS, WAS, affected library, etc:
Additional details (e.g. how your collector/web is set up):

Additional Info

I checked the UDPReceiver. It's a bug.

Thanks for the reply
I read the code to modify to place, made a suggestion, you can take a look.


Update LostTrackingService for Unity 2020 and XRSDK/WMR

Overview
Updates the LostTrackingService to no longer call into removed legacy XR APIs (WorldManager) on Unity 2020.
Also adds support for the replacement APIs in the Windows XR Plugin for XR SDK. This required a new dependency for the LostTrackingService's asmdef on the WMR plugin, which is added via the existing WindowsMixedRealityXRSDKConfigurationChecker.
Changes

Part of #8269

/azp run

Adding DO NOT MERGE while I investigate using ARSessionSubsystem instead, which would help make this more cross-compatible.

/azp run

/azp run

Temporarily closing this out while I rethink the approach

/azp run

/azp run

:shipit:


Ft about page content

This PR adds some styling to the about page.

@andela-celisha-wigwe Could you work on the hound violations?

@andela-celisha-wigwe could you work on the issues pointed out by @andela-iamadi ☝


[SPARK-25028][SQL] Avoid NPE when analyzing partition with NULL values

What changes were proposed in this pull request?
ANALYZE TABLE ... PARTITION(...) COMPUTE STATISTICS can fail with a NPE if a partition column contains a NULL value.
The PR avoids the NPE, replacing the NULL values with the default partition placeholder.
How was this patch tested?
added UT

Merged build finished. Test PASSed.

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

Test build #94423 has started for PR 22036 at commit ee64a6b.

Test build #94423 has finished for PR 22036 at commit ee64a6b.

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

Merged build finished. Test PASSed.

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

cc @cloud-fan @gatorsmile

retest this please

Test build #94683 has started for PR 22036 at commit 0470c2d.

Test build #94686 has started for PR 22036 at commit 0470c2d.

Merged build finished. Test FAILed.

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

Merged build finished. Test PASSed.

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

Test build #94686 has finished for PR 22036 at commit 0470c2d.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

retest this please

Merged build finished. Test PASSed.

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

Test build #94693 has started for PR 22036 at commit 0470c2d.

Test build #94683 has finished for PR 22036 at commit 0470c2d.

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

Merged build finished. Test PASSed.

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

thanks, merging to master/2.3!

Test build #94693 has finished for PR 22036 at commit 0470c2d.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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


SM-362

Proposed Changes
Impacted Routes

Caveats

So with the test helpers, notice there is a file /internal/infrastructure/dbtest/test_helper.go and then also /internal/test/helpers.go that have some of the same methods - and also be careful with the db name since looks like some code was copied from other repos. It looks like the TruncateTables method you were using wasn't working because of this. If the tables don't get truncated that leads to those errors around duplicate key indexes.


cephfs/admin: Fix TestCancelClone in quincy and pacific

In order to merge quincy CI support, the test "TestCancelClone" was disabled on ceph quincy.
https://github.com/ceph/go-ceph/blob/e0369cdf88d1418846321ff6a49201dff9eb8514/cephfs/admin/clone_test.go#L147
We need to revisit this test failure and remove the test skip to have the test suite fully operational on quincy.

=== RUN   TestCancelClone
    clone_test.go:205:
        	Error Trace:	clone_test.go:205
        	Error:      	Received unexpected error:
        	            	rados: ret=-22, Invalid argument: "Traceback (most recent call last):\n  File \"/usr/share/ceph/mgr/mgr_module.py\", line 1701, in _handle_command\n    return self.handle_command(inbuf, cmd)\n  File \"/usr/share/ceph/mgr/volumes/module.py\", line 409, in handle_command\n    return handler(inbuf, cmd)\n  File \"/usr/share/ceph/mgr/volumes/module.py\", line 38, in wrap\n    return f(self, inbuf, cmd)\n  File \"/usr/share/ceph/mgr/volumes/module.py\", line 636, in _cmd_fs_clone_cancel\n    vol_name=cmd['vol_name'], clone_name=cmd['clone_name'], group_name=cmd.get('group_name', None))\n  File \"/usr/share/ceph/mgr/volumes/fs/volume.py\", line 582, in clone_cancel\n    self.cloner.cancel_job(volname, (clonename, groupname))\n  File \"/usr/share/ceph/mgr/volumes/fs/async_cloner.py\", line 389, in cancel_job\n    with open_subvol(self.fs_client.mgr, fs_handle, self.vc.volspec, group, clonename, SubvolumeOpType.CLONE_CANCEL) as clone_subvolume:\n  File \"/usr/lib64/python3.6/contextlib.py\", line 81, in __enter__\n    return next(self.gen)\n  File \"/usr/share/ceph/mgr/volumes/fs/operations/subvolume.py\", line 72, in open_subvol\n    subvolume = loaded_subvolumes.get_subvolume_object(mgr, fs, vol_spec, group, subvolname)\n  File \"/usr/share/ceph/mgr/volumes/fs/operations/versions/__init__.py\", line 95, in get_subvolume_object\n    subvolume.discover()\n  File \"/usr/share/ceph/mgr/volumes/fs/operations/versions/subvolume_base.py\", line 319, in discover\n    self.metadata_mgr.refresh()\n  File \"/usr/share/ceph/mgr/volumes/fs/operations/versions/metadata_manager.py\", line 52, in refresh\n    self.config.readfp(conf_data)\n  File \"/usr/lib64/python3.6/configparser.py\", line 763, in readfp\n    self.read_file(fp, source=filename)\n  File \"/usr/lib64/python3.6/configparser.py\", line 718, in read_file\n    self._read(f, source)\n  File \"/usr/lib64/python3.6/configparser.py\", line 1111, in _read\n    raise e\nconfigparser.ParsingError: Source contains parsing errors: '<???>'\n\t[line 13]: 'a0\\n'\n"

I filed https://tracker.ceph.com/issues/55583 to see if we can get some feedback from the cephfs team on this.

#724 must be reverted after fixing this.

adjusted issue title to reflect that it needs to be fixed(and further revert the skip) for quincy and pacific.

Backport PRs:

https://github.com/ceph/ceph/pull/47747
https://github.com/ceph/ceph/pull/47748

https://github.com/ceph/ceph/pull/47112


Drop excessive cells after task reexecution

Description
When cells become unused after recomputation of a task, drop them.
Testing Instructions

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#8687
#8170  👈
#8749
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @sokra and the rest of your teammates on  Graphite

Merge activity

Jul 24, 3:40 PM EDT: @sokra merged this pull request with Graphite.


Implement static membership feature on client side (KIP-345)

implement kip-345
add test fixtures
ts-defs

This is exciting work. It seems like this has also implemented most of (all of?) KIP-320.
Can you help me understand what parts of Nevon's earlier checklist might still be left to complete?

Implement KIP-320. This is probably gonna be an epic in and of itself. The reason we need this is because we need to support Fetch v12 in order to get the currentLeaderEpoch field, which we need in order to support OffsetFetch v6-7, which in turn we need because it includes the groupInstanceId field that's needed for KIP-345.
Add support for OffsetFetch v7
Change the behavior of a consumer leaving the group to not issue a LeaveGroup request if the instance id is set.
Expose a way of configuring the instance id on the consumer.
Update the partition assigner to take into account the instance id (the KIP has a good example of how)
I have not really mapped out the work for KIP-320, so I don't know how big or small it is, but it's definitely the first step needed. We have another issue open for that https://github.com/tulios/kafkajs/issues/818

can we advance this pr?
we can really use this ability

merge please

This feature will be very helpful in our case.
Is there any way to speed up the pr

Same here, this is really useful.

This pr will help us a lot

Agree, that this feature will help many developers

Please don't delay this pr. I need it

+1, would help a lot with a current project

@sandeepkumarcel check please comments, realy waiting for this PR

@sandeepkumarcel have you tested this PR somewhere? if yes, then let us know, at least I am ok to use your forked version in my use case.
Thanks 🙇


replace TimeScheduleConstruction model with a key/value lookup for Term

Coverage decreased (-0.008%) to 86.4% when pulling d4172f1e99ca615b4c84f78226f1179e813682ee on feature/fix-time-schedule-construction into b6f3890543bd87402dfbda94872ea020fdd155e2 on develop.


chore: remove duplicate samples

Removing duplicate samples which are present in java-docs-samples/ other split repos.

Checking for doc inclusions on this PR. Will get to it by EOW.


feat: mortgage calculator

Please describe the changes this PR makes and why it should be merged:
Added mortgage calculator
Closes #673
Status

[x] Code changes have been tested against prettier, or there are no code changes
[x] I know how to update typings and have done so, or typings don't need updating

Semantic versioning classification:

[ ] This PR changes the codebase

[ ] This PR includes breaking changes (methods removed or renamed, parameters moved or removed)
[ ] This PR changes the internal workings with no modifications to the external API (bug fixes, performance improvements)

[ ] This PR only includes non-code changes, like changes to documentation, README, etc.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1257

Relevant Lines:
5082

💛  - Coveralls