Extensive testing of subworkflows

Please create sub-issues for each subworkflow.

dup_to_ins parameter does not exist

sample parameter into truth-id

It is written nextflow run nf-core/benchmark -profile test_liftover,<docker/singularity> --outdir <OUTDIR> in test_liftover.config file.
It should be nextflow run nf-core/variantbenchmarking ....

add info about truth samples, link GIAB truth ftp directories.

I added normalization and deduplication preprocess into test_somatic.config. When I run nextflow run . -profile test_somatic,docker --outdir test_results, I got this error:
Command error:
  [W::vcf_parse_info] INFO 'STRANDS' is not defined in the header, assuming Type=String
  REF_MISMATCH  chr1    669501  N       A
  REF_MISMATCH  chr1    964661  N       G
  [E::vcf_format] Invalid BCF, the INFO tag id=13 is too large at chr1:669501
  [flush_buffer] Error: cannot write to 13059_2022_2816_MOESM4_ESM.rh.dedup.vcf.gz

I added filter_contig into the test_stub.config. And then run nextflow run . -profile test_stub,docker --outdir test_results -stub
Error message:
Execution cancelled -- Finishing pending tasks before exit
-[nf-core/variantbenchmarking] Pipeline completed with errors-
WARN: Killing running tasks (9)
ERROR ~ Error executing process > 'NFCORE_VARIANTBENCHMARKING:VARIANTBENCHMARKING:SMALL_GERMLINE_BENCHMARK:HAPPY_HAPPY (freebayes)'

Caused by:
  Process `NFCORE_VARIANTBENCHMARKING:VARIANTBENCHMARKING:SMALL_GERMLINE_BENCHMARK:HAPPY_HAPPY (freebayes)` terminated with an error exit status (1)

Command executed:

  echo "" | gzip > freebayes.HG002.small.freebayes.roc.all.csv.gz
  echo "" | gzip > freebayes.HG002.small.freebayes.roc.Locations.INDEL.csv.gz
  echo "" | gzip > freebayes.HG002.small.freebayes.roc.Locations.INDEL.PASS.csv.gz
  echo "" | gzip > freebayes.HG002.small.freebayes.roc.Locations.SNP.csv.gz
  echo "" | gzip > freebayes.HG002.small.freebayes.roc.Locations.SNP.PASS.csv.gz
  echo "" | gzip > freebayes.HG002.small.freebayes.metrics.json.gz
  echo "" | gzip > freebayes.HG002.small.freebayes.vcf.gz
  touch freebayes.HG002.small.freebayes.vcf.gz.tbi
  touch freebayes.HG002.small.freebayes.summary.csv
  touch freebayes.HG002.small.freebayes.extended.csv
  touch freebayes.HG002.small.freebayes.runinfo.json

  cat <<-END_VERSIONS > versions.yml
  "NFCORE_VARIANTBENCHMARKING:VARIANTBENCHMARKING:SMALL_GERMLINE_BENCHMARK:HAPPY_HAPPY":
      hap.py: 0.3.14
  END_VERSIONS

Command exit status:
  1

Command output:
  (empty)

Command error:
  .command.sh: line 14: freebayes.HG002.small.freebayes.vcf.gz: cannot overwrite existing file

Work dir:
  /workspace/variantbenchmarking/work/41/4240508390b8f5c64ca437a44466ea

Container:
  quay.io/biocontainers/hap.py:0.3.14--py27h5c5a3ab_0

Tip: when you have fixed the problem you can continue the execution adding the option `-resume` to the run command line

 -- Check '.nextflow.log' file for details
ERROR ~ Pipeline failed. Please refer to troubleshooting docs: https://nf-co.re/docs/usage/troubleshooting

 -- Check '.nextflow.log' file for details
Join mismatch for the following entries: key=[id:tiddit2, caller:tiddit, vartype:sv, subsample:HCC1395_HCC1395T, normshift:null, normdist:null, normsizediff:null, maxdist:null, pctsize:null, pctseq:null, pctovl:null, refdist:null, chunksize:null, dup_to_ins:null, typeignore:null, bpDistance:null, percentThreshold:null, absoluteThreshold:null, maxMatches:null, evaluationmode:null] values=

Join mismatch for the following entries: key=[id:freebayes, caller:freebayes, vartype:small, subsample:null, normshift:null, normdist:null, normsizediff:null, maxdist:null, pctsize:null, pctseq:null, pctovl:null, refdist:null, chunksize:null, dup_to_ins:null, typeignore:null, bpDistance:null, percentThreshold:null, absoluteThreshold:null, maxMatches:null, evaluationmode:null] values=

ERROR ~ Pipeline failed. Please refer to troubleshooting docs: https://nf-co.re/docs/usage/troubleshooting

 -- Check '.nextflow.log' file for details

dup_to_ins parameter does not exist

It exists as a truvari parameter being processed form ext.args, should be defined in samplesheet.

sample parameter into truth-id

that is resolved in #100

It is written nextflow run nf-core/benchmark -profile test_liftover,<docker/singularity> --outdir <OUTDIR> in test_liftover.config file.
It should be nextflow run nf-core/variantbenchmarking ....

this is fixed

add info about truth samples, link GIAB truth ftp directories.

a spesfic .md file is created for truth files

I added normalization and deduplication preprocess into test_somatic.config. When I run nextflow run . -profile test_somatic,docker --outdir test_results, I got this error:
Command error:
  [W::vcf_parse_info] INFO 'STRANDS' is not defined in the header, assuming Type=String
  REF_MISMATCH  chr1    669501  N       A
  REF_MISMATCH  chr1    964661  N       G
  [E::vcf_format] Invalid BCF, the INFO tag id=13 is too large at chr1:669501
  [flush_buffer] Error: cannot write to 13059_2022_2816_MOESM4_ESM.rh.dedup.vcf.gz

Similar issue: samtools/bcftools#1720

I will skip this error since it is originated through the truth vcf misformatting

I added filter_contig into the test_stub.config. And then run nextflow run . -profile test_stub,docker --outdir test_results -stub
Error message:
Execution cancelled -- Finishing pending tasks before exit
-[nf-core/variantbenchmarking] Pipeline completed with errors-
WARN: Killing running tasks (9)
ERROR ~ Error executing process > 'NFCORE_VARIANTBENCHMARKING:VARIANTBENCHMARKING:SMALL_GERMLINE_BENCHMARK:HAPPY_HAPPY (freebayes)'

Caused by:
  Process `NFCORE_VARIANTBENCHMARKING:VARIANTBENCHMARKING:SMALL_GERMLINE_BENCHMARK:HAPPY_HAPPY (freebayes)` terminated with an error exit status (1)

Command executed:

  echo "" | gzip > freebayes.HG002.small.freebayes.roc.all.csv.gz
  echo "" | gzip > freebayes.HG002.small.freebayes.roc.Locations.INDEL.csv.gz
  echo "" | gzip > freebayes.HG002.small.freebayes.roc.Locations.INDEL.PASS.csv.gz
  echo "" | gzip > freebayes.HG002.small.freebayes.roc.Locations.SNP.csv.gz
  echo "" | gzip > freebayes.HG002.small.freebayes.roc.Locations.SNP.PASS.csv.gz
  echo "" | gzip > freebayes.HG002.small.freebayes.metrics.json.gz
  echo "" | gzip > freebayes.HG002.small.freebayes.vcf.gz
  touch freebayes.HG002.small.freebayes.vcf.gz.tbi
  touch freebayes.HG002.small.freebayes.summary.csv
  touch freebayes.HG002.small.freebayes.extended.csv
  touch freebayes.HG002.small.freebayes.runinfo.json

  cat <<-END_VERSIONS > versions.yml
  "NFCORE_VARIANTBENCHMARKING:VARIANTBENCHMARKING:SMALL_GERMLINE_BENCHMARK:HAPPY_HAPPY":
      hap.py: 0.3.14
  END_VERSIONS

Command exit status:
  1

Command output:
  (empty)

Command error:
  .command.sh: line 14: freebayes.HG002.small.freebayes.vcf.gz: cannot overwrite existing file

Work dir:
  /workspace/variantbenchmarking/work/41/4240508390b8f5c64ca437a44466ea

Container:
  quay.io/biocontainers/hap.py:0.3.14--py27h5c5a3ab_0

Tip: when you have fixed the problem you can continue the execution adding the option `-resume` to the run command line

 -- Check '.nextflow.log' file for details
ERROR ~ Pipeline failed. Please refer to troubleshooting docs: https://nf-co.re/docs/usage/troubleshooting

 -- Check '.nextflow.log' file for details
Join mismatch for the following entries: key=[id:tiddit2, caller:tiddit, vartype:sv, subsample:HCC1395_HCC1395T, normshift:null, normdist:null, normsizediff:null, maxdist:null, pctsize:null, pctseq:null, pctovl:null, refdist:null, chunksize:null, dup_to_ins:null, typeignore:null, bpDistance:null, percentThreshold:null, absoluteThreshold:null, maxMatches:null, evaluationmode:null] values=

Join mismatch for the following entries: key=[id:freebayes, caller:freebayes, vartype:small, subsample:null, normshift:null, normdist:null, normsizediff:null, maxdist:null, pctsize:null, pctseq:null, pctovl:null, refdist:null, chunksize:null, dup_to_ins:null, typeignore:null, bpDistance:null, percentThreshold:null, absoluteThreshold:null, maxMatches:null, evaluationmode:null] values=

ERROR ~ Pipeline failed. Please refer to troubleshooting docs: https://nf-co.re/docs/usage/troubleshooting

 -- Check '.nextflow.log' file for details

this was resolved in #98


Cannot create an order because of the cors

Component(s) affected:  CORS, GRAPHQL MUTATION
Description
When I execute queries to retrieve information from the GraphQL API everything works fine. But when I have to execute Mutations there is a CORS error. According to what I read it should indeed be configured on the server side. I use Nuxtjs + Nuxt GraphQL Client and here is my configuration:
How to reproduce
nuxt.config.js

order request

mutation

request header

console log

Possible Solution
I think that the servers could be configured to accept requests that could come from Localhost.  I've already encountered this problem several times when I develop with my own backend locally and I always had to configure the server correctly.
Additional context
If you want to take a closer look at the code, you can find it in this repo: https://github.com/michael-mb/E-Com-Crystallize
The .env is naturally ignored

https://github.com/Diizzayy/nuxt-graphql-client/discussions/260 also look at this discussion, the problem it seems is with the 'nuxt-graphql-plugin'

"X-Crystallize-Access-Token-Id and X-Crystallize-Access-Token-Secret headers must both be set."
I think it's actually done but I have the same problem

@michael-mb Can you attempt removing the same-origin condition via corsOptions and useGqlCors? I had attempted to investigate this issue but was unable to properly setup my crystalize account to gain access to the /orders api

Maybe the problem is there ?


Educational Codeforces Round 96 (Rated for Div. 2) | A-G |

Educational Codeforces Round 96  -
**Solutions of  Educational Codeforces Round 96 **
Type of Issue -
Please add/delete options that are not relevant.

[ ] Adding New Code
[ ] Improving Code
[ ] Improving Documentation
[ ] Bug Fix

Programming Language
Please add/delete options that are not relevant.

[ ] Python
[ ] C++
[ ] Java
[ ] Other Language

Self Check

Ask for issue assignment and wait to get assigned to it before making Pull Request.
Add your file in the proper folder
Clean Code and Documentation for better readability
Add Title and Description of the program in the file

:star2: Star it :fork_and_knife:Fork it :handshake: Contribute to it!
Happy Coding,

Please assign to me

@sarthak0318  you can now start working
Happy Coding!

I would like to work on this!

@error404-afk  happy coding!


Expected to find nested object within data[0].attributes.xxxx that matches \data$\

{
	"data": [{
			"id": "359296",
			"type": "someType",
			"attributes": {
				"productionGenres": [{
						"id": "DR",
						"name": "Drama"
					}]
			     }
		     }]
}

public class MyResourceClass {
        public string Type { get; set;}
        public string Id { get; set; }
        [JsonProperty("productionGenres")]
        public ReferenceData[] ProductionGenres { get; set; }
}

    public class ReferenceData
    {
        [JsonProperty("id")] public string Id { get; set; }
        [JsonProperty("name")] public string Name { get; set; }
    }

I am getting below error when I try to serialize above json.

Expected to find nested object within data[0].attributes.productionGenres that matches \data$\

It works fine if I comment id property in Reference class. I can understand that id and type should not be in attributes according to JsonApi specification but inner objects can have id or type properties.
any suggestion?

It was a bug in the old version, when I updated to latest 1.7.0 it fixed it.


CoverBench

New benchmark for complex claim verification from Google.
Paper: https://arxiv.org/pdf/2408.03325

@ysjprojects by anychance are you able to reproduce the results from the original paper?


Display name for multiple SIP Accounts

Hi, It seems There can only be 1 single Display name for every single SIP account. Change the Display Name for Account B, and both the Display names for account A and B are changed into the new name.

The actual DISPLAY_NAME property is only applied to the account being edited. The account SIP registration wizard however is somehow being reused as a singleton and doesn't load the properties correctly into the UI.


Can't deploy Next.js SSR app

Describe the bug
I can't for the life of me figure out how to deploy Next.js SSR app. Tried every possible combination, with standalone output or not:
      #- script: yarn build
      #  displayName: 'Build'

      - task: AzureStaticWebApp@0
        displayName: 'Deploy to Azure Static Web Apps'
        inputs:
          azure_static_web_apps_api_token: $(AZURE_STATIC_WEB_APPS_API_TOKEN)
          app_location: '/'
          api_location: ''
          output_location: ''
          app_build_command: yarn build && npm pkg set<EMAIL_ADDRESS>        env:
          PRE_BUILD_COMMAND: corepack enable

Not only this uses Yarn 3 for build and Yarn 1 for deploy parts (completely fails to detect .yarnrc.yml which is supposed to detect modern version of Yarn), but ends up with an error:
The content server has rejected the request with: BadRequest
Reason: The size of the function content was too large. The limit for this Static Web App is 104857600 bytes.

Tried this with custom build:
      - script: yarn build
        displayName: 'Build'

      - task: AzureStaticWebApp@0
        displayName: 'Deploy to Azure Static Web Apps'
        inputs:
          azure_static_web_apps_api_token: $(AZURE_STATIC_WEB_APPS_API_TOKEN)
          skip_app_build: true
          app_location: ''
          output_location: ''

and:
      - script: yarn build
        displayName: 'Build'

      - task: AzureStaticWebApp@0
        displayName: 'Deploy to Azure Static Web Apps'
        inputs:
          azure_static_web_apps_api_token: $(AZURE_STATIC_WEB_APPS_API_TOKEN)
          skip_app_build: true
          app_location: '.next'
          output_location: ''

and also:
      - script: yarn build
        displayName: 'Build'

      - task: AzureStaticWebApp@0
        displayName: 'Deploy to Azure Static Web Apps'
        inputs:
          azure_static_web_apps_api_token: $(AZURE_STATIC_WEB_APPS_API_TOKEN)
          skip_app_build: true
          app_location: '.next/standalone'
          output_location: ''

and got:
App Directory Location: '.next' is invalid. Could not detect this directory. Please verify your deployment configuration file reflects your repository structure.

or:
Failed to find a default file in the app artifacts folder (.next). Valid default files: index.html,Index.html.
If your application contains purely static content, please verify that the variable 'app_location' in your deployment configuration file points to the root of your application.
If your application requires build steps, please validate that a default file exists in the build output directory.

Expected behavior
Next.js application to deploy
Device info (if applicable):

OS: [e.g. iOS] n/a
Browser [e.g. chrome, safari] n/a
Version [e.g. 22] n/a

try 'npx vercel' deploy if ur trying to deploy there. otherwise if you're using something like netlify, you could try just connecting the github repo to the project.
it seems like it's saying that you're basically doing too much at once, you could try swapping to npm, code-splitting, or install @next/bundle-analyzer the running 'ANALYZE=true yarn build' that should help determine an exact cause a bit easier. also 'yarn lint' can be a good tool.


How to render JS to generate fingerprint for cookie?

This website uses JS to set cookie.
How could I run the JS to mock as browser to avoid 429 error?
from requests_html import HTMLSession

with HTMLSession() as s:
  url = 'https://www.realestate.com.au/auction-results/nsw'
  r = s.get(url)
  print(r.status_code)
  print(r.text)

  r.html.render()
  print(r.text)

The requests were captured as following,

request nsw (status 429)
request three JS
request fingerprint XHR which will return new key for the cookie
request nsw again (status 200)


Migrated CTL package

coverage: 78.631% (-0.05%) from 78.683%
when pulling 85c932c4600ea54b35e7043932e5771032ab6ebf on dissoupov
into 92c8d151d13919309e275985417ad6167852e9f0 on main.


I've finished checking and all implementation matchs with the claims in PR message.


Unable to run brew install android-platform-tools

Description
Hey ! 👋
The current homebrew instance used into the latest macos runner contains a bug which block the installation of the formulae android-platform-tools from homebrew.
Homebrew already fixed the issue from his side here => https://github.com/orgs/Homebrew/discussions/4501 but the actual runner macos-latest doesn't use the fixed homebrew instance so my builds cannot succeed until the runner is updated.
Platforms affected

[ ] Azure DevOps
[X] GitHub Actions - Standard Runners
[ ] GitHub Actions - Larger Runners

Runner images affected

[ ] Ubuntu 20.04
[ ] Ubuntu 22.04
[ ] macOS 11
[ ] macOS 12
[X] macOS 13
[ ] Windows Server 2019
[ ] Windows Server 2022

Image version and build link
https://github.com/VincentMasselis/TPMS-advanced/actions/runs/4957595681/jobs/8869490137
Is it regression?
https://github.com/VincentMasselis/TPMS-advanced/actions/runs/4917194908/jobs/8781980089
Expected behavior
I except brew install android-platform-tools to succeed.
Actual behavior
Downloading https://dl.google.com/android/repository/platform-tools_r34.0.1-darwin.zip
Error: SHA256 mismatch
Expected: ef3dd374ebd2dddf5d8ab6179feb8016462ad25473ccc44555a98fb37adc9d63
  Actual: 38e4abc642250e7090041e569d9de8d3f1dee2908fa30daf5ebe45a3089ec54b
    File: /Users/runner/Library/Caches/Homebrew/downloads/005288f91f3bb87ab8eb3f0e01977da70607cb4cb9658cb193bad69c07582827--platform-tools_r34.0.1-darwin.zip
To retry an incomplete download, remove the file above.
Error: Process completed with exit code 1.

Repro steps

Create a runner which uses macos-latest
Add a step which run bash scripts
Add brew install android-platform-tools to the script

Hello! it will be fixed only after new image rollout

For those who come here with the same issue, you have to put the instruction brew update before using brew install into your script to fix the error

@VincentMasselis , updated image should have been deployed, can you please check ?

I can confirm every works has expect now :)
https://github.com/VincentMasselis/TPMS-advanced/actions/runs/5155470734/jobs/9285187505
Thanks a lot !


feat(linter) eslint plugin unicorn: prefer event target

Current dependencies on/for this PR:

main

PR #1463

PR #1464

PR #1465   👈

PR #1466

This stack of pull requests is managed by Graphite.

Current dependencies on/for this PR:

main

PR #1463

PR #1464

PR #1465   👈

PR #1466

This stack of pull requests is managed by Graphite.

Merge activity

Nov 20, 9:02 PM: @Boshen started a stack merge that includes this pull request via Graphite.


No issue tracker for linenoise-ng

Quite a few people use https://github.com/arangodb/linenoise-ng independently of arangodb.
However, the linenoise-ng repo doesn't have an issue tracker. Does linenoise issues go here as well, or should someone add issues to the linenoise repo?
Thanks.

That was a mistake, sorry. I added the issue tracker functionality to the linenoise-ng repository. Please, feel open to add the issue there.

please note that meanwhile there also is https://github.com/AmokHuginnsson/replxx


Unmaintained?

@mitsuhiko is this tool still maintained?
It is a great usability benefit for command-line users, so it would be sad to see it die slowly. I am mainly wondering, as I was thinking about packaging it for NixOS.

I fix it for my own issues only atm.
On Tue, Sep 5, 2017 at 6:49 PM, Eduard Bopp<EMAIL_ADDRESS>wrote:

@mitsuhiko https://github.com/mitsuhiko is this tool still maintained?
It is a great usability benefit for command-line users, so it would be sad
to see it die slowly. I am mainly wondering, as I was thinking about
packaging it for NixOS.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mitsuhiko/unp/issues/15, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAc5PE3IgLaYBV8kAWDrj07HCE3GQpiks5sfVFtgaJpZM4PM-hx
.

There are alternatives like unp from Debian, dtrx, atool, and I think I saw one written in go, but can't find it.

@Moredread, thanks for the hint.

As well as the aforementioned aunpack (part of atool) there's also a new version of this tool by the same author written in Rust: unbox.


life.py: Would be nice to determine console size on launch, and launch at that size.

Maybe still allow a user-specified size via command-line args.
If we're going to do this, a --help would be nice too.

Separating out the sub issues into their own issues.
Keeping this for the topic, which will become the default.


interface color

the colors on the interface are too dark to see anything properly.

Let's take a specific view and I will modify it to make it more readable, and then we will use it as guiding for further improvements. What view is the most "unreadable" to you?
Note: the dark theme correlates to the upcoming theme in hoi4 and will remain.

that would be the research view.

Less contrast for NWO fonts - need if ok, do this for all fonts. Tech window - black text -background made brighter to better see the text.


android nougat

We are having issues with android nougat where users can't sometimes see the conversations, the messages are never sent/received to the device (the push notifications are). We are using the below versions:
compile ('com.layer.sdk:layer-sdk:0.21.3')
compile ('com.layer.atlas:layer-atlas:0.2.14')

Is there something missing here?

Is it fixed ?

Same problem is there any fix?

Hello @kalvish21 @kraghu @kenzo23312
Can you please follow the steps here (https://docs.layer.com/sdk/android/support) to provide logs, this is an sdk issue and we use a different system to track the ticket. After submitting the log, you will get a ticket id to reference.
Thanks.


Enhancements ideas for the Sound Generator webapp

I think that this "Sound Generator" can bring value and interest from a larger community. so improving it is of great value, but this is low prior and only intended as side path as main priority is for the paper and what is agreed during the meetings.

My PC is so fast that the generation is complete before I have time to move the mouse to play, I can repeat almost instantly, after some time it becomes annoying to press "generate" every time, so if an option would allow to "generate on mouse up" or release event of the slider that would make it more user friendly.
Adding spectrogram and other measures for generated samples
Putting all history items in a list
reloading the sliders config when selecting an element from the history

Thanks for the input @wassfila
I often find myself changing multiple parameter values, so if we regenerate audio after every slider change, then overall the process becomes slower for me since I have to then wait half a second for every change to take effect. So, I feel like keeping the generate button is still better. But I'm keen to hear more on this from other users and we can address it accordingly
I agree on keeping temporary history and showing spectrograms - those would be cool additions


ofRoute: adds support for RegExp

fixes #1, also enhances readme a bit
cheers! 🍻

awesome stuff! thanks

Planning to update after work :)

:wink:

bump! :shipit:


Environment Variable KERB_TICKET_CACHE_PATH is unnecessary

https://github.com/tensorflow/tensorflow/commit/83559ee1d8cce32dd80cb017cd8bb76880790b23 have provided a commit to add kerberos ticket cache file.
In com.sun.security.auth.module.Krb5LoginModule#initialize(), it default set ticketCacheName from ticketCache of options, ticketCache was set in org.apache.hadoop.security.HadoopConfiguration by System.getenv("KRB5CCNAME");.
So, according to https://www.tensorflow.org/deploy/hadoop, set ticket cache by KRB5CCNAME instead of KERB_TICKET_CACHE_PATH, the program still working properly.
Therefore, https://github.com/tensorflow/tensorflow/commit/83559ee1d8cce32dd80cb017cd8bb76880790b23 is unnecessary, just remind users set KRB5CCNAME in https://www.tensorflow.org/deploy/hadoop

/CC @jhseu.

@llhe thoughts?

@weberxie @jhseu
According to kerberos document, KRB5CCNAME is the built in environment variable. So we can revert this PR and change the document to set the environment variable it in kerberos way.

Checked that PR, it also fixes the tests. For backward compatibility, should we just change the document and logging deprecation warning?

Checked that PR, it also fixes the tests. For backward compatibility, should we just change the document and logging deprecation warning?

@llhe This seems to be the best choice.

It has been 14 days with no activity and the awaiting tensorflower label was assigned. Please update the label and/or status accordingly.


[RFC] Multi OS support

Based on the pod config annotations, we change the underlying
hypervisor configuration. Pod config annotations can carry a
kernel path, a guest image path or both.
Those annotations, if valid, will overwrite the hypervisor
configuration paths. If not valid, virtcontainers will error
out.
Optionally, asset SHA-512 hashes can be passed through pod
annotations as well. In that case virtcontainers will verify
that the asset binary matches the hash annotation and error
out when it does not.
More details on the architecture in #439
Fixes #439
TODO:

[X] Unit tests
[X] Update pkg/oci/utils.go to convert an OCI annotations into a PodConfig one for assets

cc @mcastelino @sboeuf @jodh-intel @grahamwhaley

Coverage decreased (-1.6%) to 63.211% when pulling 23f15d626de612760fe7c1a5c23630b867d09d14 on sameo/topic/multios into b05726778f1263c36547de7fd93da9221767397d on master.

Coverage decreased (-0.7%) to 62.777% when pulling 84f191b1c4d77931b132d4136beef8bf8e87b9d1 on sameo/topic/multios into 24d2e47187feab0a90ceea9294042c48a2312aaf on master.

Coverage increased (+0.2%) to 63.621% when pulling 3952ec7d8b3608bd16a1d68728a7ed80bef96c37 on sameo/topic/multios into 45fe7975442461611854ddbf576f07a20a0b68ee on master.

@sboeuf

Did I miss something explaining that we MUST implement this in virtcontainers ?

No you did not miss anything and this could be done by only modifying utils/oci.go. However, I did not want to do something OCI specific and I wanted the code to be generic and specification agnostic. I also realized it would be good to have a generic asset handling API for virtcontainers.
So it makes the code a little complex but I think it's cleaner.
Also, one thing I was considering is adding private fields to HypervisorConfig for not overwriting the public paths passed from above. I think it would be cleaner.

Well, I think the problem is to have two ways to setup the same thing from virtcontainers perspective. We should not give the choice between either the Pod annotations, or the HypervisorConfig structure to the consumer of virtcontainers, otherwise we're just bringing confusion IMO.
Now, if you say we want to modify HypervisorConfig so that all its fields are private, then the annotations make sense since this would be the only way to setup the kernel/image. But there one issue about that, we cannot make those fields private because this structure is stored to our filesystem through a json marshalling and it needs to fields to be public.

Now, if you say we want to modify HypervisorConfig so that all its fields are private, then the annotations make sense since this would be the only way to setup the kernel/image. But there's one issue about that, we cannot make those fields private because this structure is stored to our filesystem through a json marshalling and it needs those fields to be public.

Oh, don't get me wrong: I don't want to have all HypervisorConfig fields private, only the ones we'd use to set assets not defined by the caller directly.
So I fully agree things could be done from the oci utils package or from the runtime directly, and it would make sense if the definition of alternative assets would be part of the OCI spec (Which may eventually become reality). In that case, we'd just use the OCI spec and translate that into an HypervisorConfig setting.
But here I'm looking at the problem from a k8s point of view where the only thing we're going to be able to pass in to the runtime or virtcontainers are pod annotations. We can decide to translate those into HypervisorConfig fields in several places. In generalization order:

From the runtime: This becomes CC specific
From virtcontainers/pkg/oci: This becomes OCI specific
From virtcontainers/assets.go: This becomes virtcontainers specific

So by adding a customAssets []asset field to HypervisorConfig or even to Pod, we have a generic implementation (not OCI or CC specific) and it is transparent to the user: There is still only one way to pass a configured set of assets (through HypervisorConfig) and Annotations would only allow the higher layers (Docker CLI, k8s) to overwrite the runtime configured assets.

Coverage increased (+0.2%) to 63.621% when pulling b28666746cd3c9edab8fe16c372b7f2e812b97df on sameo/topic/multios into adc504e4132d10e0099e06239d92aa1f9ce36941 on master.

Coverage increased (+0.2%) to 63.683% when pulling 1892fc007640e87c6989c9ab7849b8fe553e1031 on sameo/topic/multios into adc504e4132d10e0099e06239d92aa1f9ce36941 on master.

@sameo so you're saying that you want something generic for those annotations, but as of today, k8s relies on CRI which relies on OCI. That's why I consider this has to be handled at the OCI level and not at the virtcontainers level. But I think I understand that you want to be able to have any orchestrator using those annotations in order to give specific info to virtcontainers. The problem that I see here is that all orchestrations use a runtime (and hence a specification file). This means that we are creating a generic thing, but in practice, we will have to translate those annotations to PodConfig annotations  in every runtime implementation.

You have some golint errors.
Let's also wait for @jodh-intel review now that you have addressed his comments.
LGTM

I'll merge this one once the 17.04 CI is fixed.


OAuth2RestTemplate should check token validity from the Autorization Server !

Hello,
In the implementation of the method getAccessToken() in OAuth2RestTemplate org.springframework.security.oauth2.client.OAuth2RestTemplate, you always check the token is expired by doing : if (accessToken == null || accessToken.isExpired())
But you never check if the token is still valid in the Authorization server which generated this token.
Here is the case where the accessToken is NOT EXPIRED but NOT VALID  (revoked by the authorization server for a reason or another)

Isn't OAuth2RestTemplate used by a client for the purposes of making a call to a OAuth2 protected resource server? If so, why would it be the clients responsibility to check with the server if its token is valid? This would be akin to a user asking Twitter if their account is valid before attempting to log in.
Checking token validity should be the responsibility of the resource server receiving the token which i believe the Spring Security OAuth resource servers do. If using JWT, it verifies validity by checking the RSA signature on the token. If not using JWT, i believe it calls the authorization servers user info endpoint to verify validity.


Added functionality to buy now button

Functionality for Buy now has been added. When user clicks on buy now, only the current product will be bought and it will not affect the cart.
Fixes #288
Type of change
How Has This Been Tested?
Added a product to cart and used buy now for another product.
Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my code
[X] I have commented my code, particularly in hard-to-understand areas
[X] My changes generate no new warnings
[X] New and existing unit tests pass locally with my changes

why did you comment out the entire file?

Just some problem sorry for that.

But the code runs perfectly.

Ok mentors will review and merge if everything's fine.

why did you comment out the entire file?

Should I make another commit to remove the commented part?


Fix to permit closing of incidents.

Handle presumed data structure change of incidents; permit proper matching of past incidents in order to update or close them.

Automated tests error is on this line:
E:526,24: Non-iterable value self.incidents is used in an iterating context (not-an-iterable)
def find_incident(self):

...
for incident in self.incidents:

[test]


Fixes to the spec about the whatsapp test

Closes: https://github.com/TheTorProject/ooni-spec/issues/101

@joelanders @bassosimone can this be merged?


updated

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #XXXXX

"Added HTML, CSS, and JavaScript to style and format text on the Empty index.html page. added fonts for headings and body text. Also added a script to handle user interactions and dynamically update the text content."

"Added HTML, CSS, and JavaScript to style and format text on the empty index.html page. added custom fonts for headings and body text. Also added a script to handle user interactions and dynamically update the text content."

Hey @GajendraRao007
Thank you for opening this pull request.
This is a standard message notifying you that we've reviewed your pull request and have decided not to merge it. We would welcome future pull requests from you.
Thank you and happy coding.


fix: remove unexpected whitespaces in CSS bundle

Motivation
Bumping version of clean-css because there was bug where spaces and line breaks were not removed around var calls (if any) in CSS code. This was fixed in 5.1.5, but removeWhitespace option in level 1 also needs to be enabled for correct work.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Preview

Before
After

Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

thanks 👍


fix(vue-app): router registerModule memory leak (#7915)

Prevents reallocating a new function on every request for the registerModule override, as reported in #7915.

Codecov Report

Merging #7916 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #7916   +/-   ##
=======================================
  Coverage   68.92%   68.92%
=======================================
  Files          91       91
  Lines        3846     3846
  Branches     1041     1041
=======================================
  Hits         2651     2651
  Misses        971      971
  Partials      224      224

Flag
Coverage Δ

#unittests
68.92% <ø> (ø)

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

Continue to review full report at Codecov.

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


_to_be_replaced is not replaced

originally reported by @er1c
steps
Install sbt 1.3.2
$ cat /usr/local/Cellar/sbt/1.3.2/libexec/bin/sbt | grep "to_be"
declare init_sbt_version=_to_be_replaced

problem
_to_be_replaced is a placeholder that was not replaced.
expectation
That should use 1.3.2.
notes
This regressed likely when two bash scripts were merged. See https://github.com/sbt/sbt-launcher-package/blob/v1.2.8/build.sbt#L156-L160

Fixed by sbt/sbt-launcher-package#289


Hidden Column

Hey @linmasahiro is it possible to have a hidden column?
A column that is not visible in the table but whose data can be used in a v-slot for example.
This will prevent the need for hitting the server api again just to get details for example which
I want in a hover over my summary field. Details is not shown in the Table columns but that data should be available to be used.

Hey @linmasahiro is it possible to have a hidden column? A column that is not visible in the table but whose data can be used in a v-slot for example. This will prevent the need for hitting the server api again just to get details for example which I want in a hover over my summary field. Details is not shown in the Table columns but that data should be available to be used.

Yes, and very easy, don't add to table.columns only.
  <table-lite
    :is-static-mode="true"
    ...
  >
    <template v-slot:summary="data">
      <span>{{ data.value.aaa }}</span>
    </template>
  </table-lite>

......

    // This is your data from API
    const data = reactive([]);
    for (let i = 0; i < 126; i++) {
      data.push({
        id: i,
        summary: "TEST" + i,
        aaa: "Hidden" + i
      });
    }

    // Table config
    const table = reactive({
      columns: [
        {
          label: "ID",
          field: "id",
          width: "3%",
          sortable: true,
          isKey: true,
        },
        {
          label: "Summary",
          field: "summary",
          width: "10%",
          sortable: true,
        },
      ],
      ......
    });

......


helm repo update tries to access wrong windows profile folder

Hi I just installed helm on windows 10 into minikube (after downgrading minikube kubernetes version as helm is not yet compatible with kubernetes 1.16) and "helm init" worked well and created in the correct user profile which is C:\ElserM.helm it's profile.
But when I try to do a "helm repo update" it tries the store it in C:\marc.elser.helm which is an old inactive profile on this windows machine that wasn't used for months. Here's the complete output
$ helm repo update
Hang tight while we grab the latest from your chart repositories...
...Skip local chart repository
...Unable to get an update from the "stable" chart repository (https://kubernetes-charts.storage.googleapis.com):
        open C:\Users\marc.elser\.helm\repository\cache\stable-index.yaml: Das System kann den angegebenen Pfad nicht finden.
Update Complete. ⎈ Happy Helming!⎈

my %USERPROFILE% directory is set to:
C:\Users\elserm
I also grepped my whole 'env' for "marc.elser" and didn't found any reference. So where does helm even get this information from and how to fix this
Even when I try to set HELM_HOME to 'C:\Users\elserm.helm' I get exactly the same error, it still want's access .helm on a wrong old user profile for whatever reason I don't know and I also don't know how to resolve this issue.
Output of `helm version`:
Client: &version.Version{SemVer:"v2.9.1", GitCommit:"20adb27c7c5868466912eebdf6664e7390ebe710", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.9.1", GitCommit:"20adb27c7c5868466912eebdf6664e7390ebe710", GitTreeState:"clean"}

Output of `kubectl version`:
Client Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.4", GitCommit:"67d2fcf276fcd9cf743ad4be9a9ef5828adc082f", GitTreeState:"clean", BuildDate:"2019-09-18T14:51:13Z", GoVersion:"go1.12.9", Compiler:"gc", Platform:"windows/386"}
Server Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.4", GitCommit:"67d2fcf276fcd9cf743ad4be9a9ef5828adc082f", GitTreeState:"clean", BuildDate:"2019-09-18T14:41:55Z", GoVersion:"go1.12.9", Compiler:"gc", Platform:"linux/amd64"}

Cloud Provider/Platform (AKS, GKE, Minikube etc.):
minikube

@marcelser It might be worth first upgrading to the latest version of Helm 2 which is v2.15.1.  There would have been a lot of fixes and capability added since 2.9.1.
To try and troubleshoot this: Could you also . try setting HELM_HOME to something like 'C:\Users\elserm.helm' and see if that works? Also maybe create the path  'C:\Users\elserm beforehand.

In the end I noticed that I can fix it in 2.9.1 by just editing the paths in repositories.yaml file. But the question remains how helm constructed this paths with wrong user profile dir. But I agress I should upgrade to helm 2.15.1 I guess, it's just that "choco install kubernetes-helm" is very outdated then because it install 2.9.1 which is not good as it's mentioned as one of the install methods on the official docs. I would expect that all installation methods are kept up to date.

But the question remains how helm constructed this paths with wrong user profile dir.

Difficult to tell with version that old. If you can try and upgrade, we can better troubleshoot this.

it's just that "choco install kubernetes-helm" is very outdated then because it install 2.9.1 which is not good as it's mentioned as one of the install methods on the official docs.

As far as I can see it currently supports v2.15.1: https://chocolatey.org/packages/kubernetes-helm

Ok, this is one of the classical cases of user too stupid. I didn't know that chocolatey keeps a local repo so I was assuming that it always installs newest version. I turned out I was very wrong, you have to update chocolatey repo and voila 2.15.1 shows up. So the only person to blame in this case is myself. So I will close this now. Hope I didn't bother you too much.

@marcelser Not a bother.


The recording Only supports the landscape

The recording Only supports the landscape?

Currently only support PORTRAIT, and later perfect

Can you  please suggest me how to add the sticker in KSYRecordKit?

Sorry, temporarily not support, you can contact our products, later consider support

Is there any solution for add the .gif image instead of .png or .jpg into the video?
Thank in adavace.

Sorry, temporarily not support


Cfg condition

Final part of #180
Example for config:
[[object]]
name = "Gtk.PaperSize"
status = "generate"
cfg_condition = "tst"
[[object.function]]
name = "is_ipp"
version = "3.16"
cfg_condition = "unix"

No cfg condition applied to sys mode

Something like this?

Rebased to master.

More like just get rid of cfg and cfg_1 moving the duplication down where it originates from
for cfg in &cfgs {
    contents.push(cfg.clone());
}
contents.push(format!("mod {};", module_name));

and so on

Then it IMHO become bad as we need push cfgs 3 times:
    contents.push(format!(""));
    for cfg in &cfgs {
        contents.push(cfg.clone());
    }
    contents.push(format!("mod {};", module_name));
    for cfg in &cfgs {
        contents.push(cfg.clone());
    }
    contents.push(format!("pub use self::{}::{};", module_name, analysis.name));
    if generate_trait(analysis) {
        for cfg in &cfgs {
            contents.push(cfg.clone());
        }
        contents.push(format!("pub use self::{}::{}Ext;", module_name, analysis.name));
        for cfg in &cfgs {
            traits.push(format!("\t{}", cfg));
        }
        traits.push(format!("\tpub use super::{}Ext;", analysis.name));
    }

This has a chance of becoming nicer with Vec::extend_from_slice becoming available in 1.6:
contents.extend_from_slice(&cfgs);

Fixed and squashed.

Thanks.
@homu r+

:pushpin: Commit a6fddc4 has been approved by gkoz

:hourglass: Testing commit a6fddc4 with merge 2b2d75b...

:sunny: Test successful - status


Changes in editor to DynamicFontData are not reflected in 2D view.

Godot version:
master-branch, recent (775e74e0d41daa9cf3679c2e215f4ef46eee9cbc)
Issue description:
When making changes to a DynamicFontData sub-resource in the inspector in the editor, the changes are not reflected visually in the 2D view, unless the editor is restarted or changes are made to unrelated properties in DynamicFont. The bug is AFAICT editor-only.
Steps to reproduce:

Create a Label.
Enter some text for the Label
In the inspector for the Label, create in "Custom fonts" a DynamicFont.
For the DynamicFont, create in "Font -> Font data" a DynamicFontData.
For the DynamicFontData, select in "Font Path" a font resource (for instance "FreeSans.ttf" as in the reproduction project).
Note that the label text is buggily not drawn in the 2D view of the editor (it is drawn when running the game).
Change an unrelated property, for instance size, in DynamicFont, and check that the text is now drawn in the 2D view of the editor. You can also restart the project instead to get the text to be drawn.
Note that the text drawn in the 2D view is drawn antialiased as per the default.
Change the antialiased setting in DynamicFontData to not be antialiased.
Note that the text in the 2D view buggily does not reflect this change in antialiasing.
Change an unrelated property, for instance size, in DynamicFont, and check that the text in the 2D view now correctly is not antialiased.

Minimal reproduction project:
A simple reproduction project with a font resource that makes it easier to follow the reproduction steps:
dynamicfontdata_bug.zip
Other comments:
The bug seems to occur due to changes being made directly to the DynamicFontData resource through the editor being not indicated to the DynamicFont resource in the editor that uses the DynamicFontData. I do not know whether similar bugs might occur or have occurred for similar cases.

Still reproducible in 3.2.2 RC 2 with the above steps to reproduce the bug.

I can confirm that the issue is still valid in 3.3, and fixed in 4.0. It's unlikely that the fix could be backported as it's probably related to Complex Text Layout/Font changes and/or the refactor of the Inspector and property change signalling.


chore: appsmith ai v0.1

Description
This PR only contains interfaces for the EE AI feat. These are temporary changes
Type of change

Chore (housekeeping or task changes that don't impact user perception)

How Has This Been Tested?

Manual

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

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

QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

/ok-to-test sha=8aa7534

/ok-to-test sha=70c7612

/ok-to-test sha=f559475

/ok-to-test sha=15261b0

/ok-to-test sha=2b70419

/ok-to-test sha=fbe8c19

/ok-to-test sha=34581c1

/build-deploy-preview env=release


MeshPickingSettings.require_markers = true doesn't ignore UI nodes without RayCastPickable

Bevy version
0.15.0
Relevant system information
- cargo 1.83.0 (5ffbef321 2024-10-29)
- MacOS Sonoma 14.1.1 (23B81)
- SystemInfo { os: "MacOS 14.1.1 ", kernel: "23.1.0", cpu: "Apple M1 Max", core_count: "10", memory: "64.0 GiB" }
- AdapterInfo { name: "Apple M1 Max", vendor: 0, device: 0, device_type: IntegratedGpu, driver: "", driver_info: "", backend: Metal }

What you did
I'm using bevy::picking to pick meshes from the main camera. I also have a different camera with some UI overlay that provides context information (it is shown over the objects). In my case I have only few types of entities that can be picked, so instead of marking everything with PickingBehavior::IGNORE I'm just marking which entities I want to be pickable with RayCastPickable (also MeshPickingSettings.require_markers = true).
What went wrong
UI entity always steals all pointer events:

All this happens despite both entity and its camera having no RayCastPickable marker:

Additional information
The only workaround so far is to manually mark every UI node with PickingBehavior::IGNORE but this becomes very tedious and error prone with ever increasing number of different entities.
Example code:
use bevy::prelude::*;
use bevy::render::view::RenderLayers;
use bevy_inspector_egui::quick::WorldInspectorPlugin;

fn main() {
    let app = &mut App::new();
    app
        .add_plugins((
            DefaultPlugins,
            MeshPickingPlugin, WorldInspectorPlugin::default(),
        ))
        .add_systems(Startup, startup);

    app.world_mut().resource_mut::<MeshPickingSettings>().require_markers = true;
    app.run();
}

fn startup(
    mut commands: Commands,
    mut meshes: ResMut<Assets<Mesh>>,
    mut materials: ResMut<Assets<StandardMaterial>>,
) {
    commands.spawn((
        Camera3d::default(),
        Transform::from_xyz(2.0, 1.0, 2.0).looking_at(Vec3::ZERO, Vec3::Y),
        RayCastPickable,
    ));

    commands.spawn((
        PointLight {
            shadows_enabled: true,
            ..default()
        },
        Transform::from_xyz(4.0, 8.0, 4.0),
    ));

    commands.spawn((
        Mesh3d(meshes.add(Cuboid::from_length(1.0))),
        MeshMaterial3d(materials.add(Color::srgb(1.0, 0.0, 0.0))),
        RayCastPickable,
    ))
        .observe(on_pointer_enter)
        .observe(on_pointer_leave);

    let ui_camera = commands.spawn((
        Name::new("UI Camera"),
        Camera2d,
        Camera {
            clear_color: ClearColorConfig::None,
            order: 1,
            ..default()
        },
        RenderLayers::layer(1),
    )).id();

    commands.spawn((
        Name::new("UI Node"),
        Node {
            position_type: PositionType::Absolute,
            top: Val::Percent(33.0),
            width: Val::Percent(100.0),
            height: Val::Percent(33.0),
            ..default()
        },
        TargetCamera(ui_camera),
        BackgroundColor(Color::WHITE.with_alpha(0.1)),
    ));
}

fn on_pointer_enter(
    event: Trigger<Pointer<Over>>,
    query: Query<&MeshMaterial3d<StandardMaterial>>,
    mut materials: ResMut<Assets<StandardMaterial>>,
) {
    if let Ok(material) = query.get(event.target) {
        if let Some(material) = materials.get_mut(material) {
            material.base_color = Color::srgb(0.0, 0.0, 1.0);
        }
    }
}

fn on_pointer_leave(
    event: Trigger<Pointer<Out>>,
    query: Query<&MeshMaterial3d<StandardMaterial>>,
    mut materials: ResMut<Assets<StandardMaterial>>,
) {
    if let Ok(material) = query.get(event.target) {
        if let Some(material) = materials.get_mut(material) {
            material.base_color = Color::srgb(1.0, 0.0, 0.0);
        }
    }
}

Just found that PickingBehavior::should_block_lower fixes this issue...


how to use layoutlm in Chinese?

Thx, we want to use layoutlm model In Chinese.How will we use this model?Whether to retrain the model?

We are working on multilingual pre-trained LayoutLM models. Please stay tuned.


Added the implementation of DotCode

Simplified also some base functions in zint_helper by using more Delphi RTL functions.
Should theorically also be more efficient for some functions.
I hope I didn't mess the things with older Delphi versions.
Just wondering, should not the TArrayOfChar be TArrayOfAnsiChar instead ? Because in C a char is not unicode...
Doesn't seem to be a big deal to change this. But who will check this completely (I'm using only a small range of barcode types).

I don't think we have to support all delphi versions. I'll take a look at TArrayOfChar .


Question regarding Training of the

Thank you for publishing UDOP, a real step towards unified document processing!
Can you give Information about the training time (on the used equipment) for the self-supervised learning and the curriculum learning parts?
thank you in advance!

We trained on our cluster with 32 A100.
It will take around 1 week for three curriculum stages.
Each stage takes 2-3 days.
Let me know if you have any questions

Thank you!


No module named 'db'

(yourenvname) C:\Users\User>english2cypher-master\e2c\predict.py
Traceback (most recent call last):
File "C:\Users\User\english2cypher-master\e2c\predict.py", line 23, in
from db import *
ModuleNotFoundError: No module named 'db'

Hello!
What version of python are you using? It sounds like maybe you’re using
Python < 3 and the system is not seeing the /db/ folder as a module.
On October 8, 2018 at 2:58:42 AM, raspberian1234<EMAIL_ADDRESS>wrote:
(yourenvname) C:\Users\User>english2cypher-master\e2c\predict.py
Traceback (most recent call last):
File "C:\Users\User\english2cypher-master\e2c\predict.py", line 23, in
from db import *
ModuleNotFoundError: No module named 'db'
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Octavian-ai/english2cypher/issues/6, or mute the thread
https://github.com/notifications/unsubscribe-auth/AOUFIsyK6xtsJ85ssfEKVShag-NQGt59ks5uiyHRgaJpZM4XMm2U
.

i am using python 3.6.4

Weird - what command line are you executing the app with?
python -m e2c.predict I /believe/ is the right incarnation
On October 8, 2018 at 8:44:05 AM, raspberian1234<EMAIL_ADDRESS>wrote:
i am using python 3.6.4
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Octavian-ai/english2cypher/issues/6#issuecomment-427883698,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOUFIk84iuArFJ4rXNxMYN6xp750iLjSks5ui3LFgaJpZM4XMm2U
.

i am using anaconda and doing in virtual environment
i downloaded file and copied in home directory
C:\Users\User>english2cypher-master\english2cypher-master\e2c\predict.py
Traceback (most recent call last):
File "english2cypher-master\english2cypher-master\e2c\predict.py", line 23, in
from db import *
ModuleNotFoundError: No module named 'db'
got this problem

hi
(english2cypher) C:\Users\User>git clone https://github.com/Octavian-ai/english2cypher.git
Cloning into 'english2cypher'...
remote: Enumerating objects: 437, done.
remote: Total 437 (delta 0), reused 0 (delta 0), pack-reused 437
Receiving objects: 100% (437/437), 10.70 MiB | 591.00 KiB/s, done.
Resolving deltas: 100% (300/300), done.
(english2cypher) C:\Users\User>python -m e2c.predict
C:\Users\User\Anaconda3\envs\english2cypher\python.exe: Error while finding module specification for 'e2c.predict' (ModuleNotFoundError: No module named 'e2c')
(english2cypher) C:\Users\User>
this is the problem i am facing

Try cd into “english2cypher” directory, then run the python command
On October 9, 2018 at 12:19:48 AM, raspberian1234<EMAIL_ADDRESS>wrote:
hi
(english2cypher) C:\Users\User>git clone
https://github.com/Octavian-ai/english2cypher.git
Cloning into 'english2cypher'...
remote: Enumerating objects: 437, done.
remote: Total 437 (delta 0), reused 0 (delta 0), pack-reused 437
Receiving objects: 100% (437/437), 10.70 MiB | 591.00 KiB/s, done.
Resolving deltas: 100% (300/300), done.
(english2cypher) C:\Users\User>python -m e2c.predict
C:\Users\User\Anaconda3\envs\english2cypher\python.exe: Error while finding
module specification for 'e2c.predict' (ModuleNotFoundError: No module
named 'e2c')
(english2cypher) C:\Users\User>
this is the problem i am facing
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Octavian-ai/english2cypher/issues/6#issuecomment-428087008,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOUFIkQyOIzr5vBl-AncmZzSsnJeLcJ_ks5ujE4DgaJpZM4XMm2U
.

Weird, I’m not familiar with running this on Windows, so sorry to not be
more helpful.
Here’s my osx (e.g. Linux) workflow:
I’m running Python 3.6.2
On October 9, 2018 at 6:25:49 AM, raspberian1234<EMAIL_ADDRESS>wrote:
C:\Users\User>cd C:\Users\User\english2cypher
C:\Users\User\english2cypher>python -m e2c.predict
C:\Users\User\Anaconda3\python.exe: No module named e2c.predict
i am not sure python -m e2c.predict is not python command
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Octavian-ai/english2cypher/issues/6#issuecomment-428190628,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOUFIgo0QlsnwTORftkkiEwIQWhu8UqAks5ujKPYgaJpZM4XMm2U
.

i had set of questions and answers in a csv file
where i should put that file in your code
to generate english2cypher and give answer  as output.

Make sure you're running the right version of tensorflow (check the pipfile)
Either follow the YAML format in /data or write some code to load your CSV directly :)


getStaticProps is not called

I just tried to migrate from getInitialProps to getStaticProps but the new method is never called.
Just try with the documentation example to reproduce:
import fetch from 'node-fetch'

function Blog({ posts }) {
  return (

      {posts.map(post => (

{post.title}

      ))}

  )
}

export async function getStaticProps() {
  console.log('Call getStaticProps')
  const res = await fetch('https://.../posts')
  const posts = await res.json()

  return {
    props: {
      posts
    }
  }
}

export default Blog

@MLerasle should be fixed with: https://github.com/vinissimus/next-translate/pull/75 . Just update to 0.10.6 and it should work :)

@BjoernRave wow 7 minutes to solve the issue and reply, thank you very much :)
This is working like a charm!


Overeager module search

Juno thinks that the bottom half of this code is in module file.
function show_for_analyze(method_instance::MethodInstance)
  original = string(method_instance)
  shortened = replace(original, "MethodInstance for ", "")
  mod = method_instance.def.module
  file = method_instance.def.file
  line = method_instance.def.line
  "$shortened in $module at $file:$line"
end

Similarly, this is in module bar:
sym = :module
bar = 1

Maybe amend the regex so that module can't be preceded by a dot or colon? I can't think of any other legal syntax that would include module.

The module detection part of this is fixed with https://github.com/JunoLab/CodeTools.jl/commit/835b10ba557b7ed045d6828ea06b12754939adb4, the syntax highlighting bug is fixed with https://github.com/JuliaEditorSupport/atom-language-julia/commit/9fafea41b210eddb434fc339deb6506b9eab0943.

❤


Need to update to V5 API

Track this project:  https://github.com/arraylabs/pymyq/issues/22

Complete


Visual Studio Code is crashing time and again. Upon opening, it bounces for a few seconds into dock, and then stops. I am on Macbook air (intel chip)

Hello VS Code Team.
I know it may not be the right place to highlight application related issue, as it is a bug-report section. But, I beleive this place is the one where my query is expected to get resolved quickly, as I don't want to migrate to another IDE, and apparently I am into the middle of nowhere at this point.
So,  I am a front-end developer who is building a major enterprise level project in react.js and certain other stacks. I have been using visual studio code, for like past 2.5yrs.
The way it is behaving from past 2 days, it never ever had before.
Issue -

Upon Opening, it bounces for like 2 seconds into mac dock area as if it is struggling to open, and then it stops.
Tried opening several times, it bounces into dock-area and stops. Therefore never opening at all.

What I did when VS-code was working before -

Well, I am having mac-air with 128gb variant. And most of the time , I ran out of space. So, I decided to clean up the "caches" in mac.
So, I cleaned-up all the caches in "~/Library/Caches" folder, which immediately granted my mac with 10gb of free space.
But, later that day, VS Code begin to crash, as I mentioend into "Issue".

Well, as far as I know, deleting Caches do not impact the functioning of the application, right?
Idk whether I lost any important file which was helping with VSCode's stability, accidently!.
Steps I tried later -

I completely removed the vs-code from my macbook, using following terminal commands -

rm -rf ~/Library/Application\ Support/Code
rm -rf ~/Library/Caches/com.microsoft.VSCode
rm -rf ~/Library/Caches/com.microsoft.VSCode.ShipIt
rm -rf ~/Library/Preferences/com.microsoft.VSCode.plist
rm -rf ~/Library/Preferences/com.microsoft.VSCode.helper.plist
rm -rf ~/Library/Saved\ Application\ State/com.microsoft.VSCode.savedState
rm -rf ~/.vscode

Even I deleted the VSCode from "Application" folder too,
Restared my mac, and installed the newer (this time a primitive version then the one I had been using for long).
Well, for like 1hour, it worked, but again, it starting behaving similarly, and afterwards,  did not open at all.
I later tried uninstalling, and re-installing the fresh-new instances of VSCode from official VSCode webpage. In all of my attempts, VsCode tends to run for like 30 mins to 1 hours (or till I log out from my system).
When I again login to mac, and try opening VSCODe, it bounces and stops.
So, I need your help with this issue. Is this the VSCOde specific bug?
Or, is there anything that is supposed to be in "Chaches" folder, but is not present now!
How could I resolve this isse, and permanently fix vs-code, such that It always run , when I clicks to open it!!
I need all of your suggestions, and steps to mitigate the issue.

The new version 1.91 has a bug that it cannot be started on the old MacBook system. Roll back to the previous version. I used 1.88 and it can be opened.

Thanks god, you save my life

Thanks!!! You save me a lot. I firstly have issue of how to revert back but I later used the old setup that I have which is 1.68 and it works.

I think this issue occured as a result of automatic update which may be updated to the latest version of 1.9

/duplicate of https://github.com/microsoft/vscode/issues/219893 which is currently pinned at the top of the Issue section.

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


Does this work with JSForce v3 ?

yes, we use it in e2e browser tests:
https://github.com/jsforce/jsforce/blob/c83cbd4b0aa4c1706cbe1edaf1eb73ecf64fdf4d/package.json#L152


Graphite documentation on Stack Overflow

Hi All,
We reached 5 votes on Stack Overflow documentation site, so, now we have Graphite section there - https://stackoverflow.com/documentation/graphite
Please share your ideas what we need to put there.
/cc @DanCech @iksaif @gwaldo @graphite-project/committers

I hadn't heard of SO-Docs, so thanks for bringing it up!
I can start a "getting started" page off the Overview or FAQ sections that
basically points to Synthesize, and strongly points out that this isn't for
prod.
On Sun, Mar 26, 2017 at 1:14 PM, Denis Zhdanov<EMAIL_ADDRESS>wrote:

Hi All,
We reach 5 votes on Stack Overflow documentation site, so, now we have
Graphite section there - https://stackoverflow.com/documentation/graphite
Please share your ideas what we need to put there.
/cc @DanCech https://github.com/DanCech @iksaif
https://github.com/iksaif @gwaldo https://github.com/gwaldo
@graphite-project/committers
https://github.com/orgs/graphite-project/teams/committers
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/graphite-project/graphite-web/issues/1873, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AANKc_MK-ZzhlGSX2DNvPbZJpeW4pPgoks5rpp0EgaJpZM4Mpfkr
.

Some of this might be duplication of content at http://graphiteapp.org/ (https://github.com/graphite-project/graphite-project.github.io), which is fine as long as we keep stuff in sync.

IMO more documentation is better than less documentation. :)
But more burden to make all in sync ofc.


OD Cost Matrix script failed (unknown reason)

Hi, I am trying to perform the "Calculate Accessibility Matrix" and I always end up with the following error message:
Validating OD Cost Matrix settings...
Validating weight field Join_Count in destinations dataset...
Inputs successfully validated.
Copying origins to output...
Precalculating network location fields for I:...\temp.gdb\CalculateAccessibilityMatrix2...
Precalculating network location fields for I:...\scratch.gdb\TNAT_TempInput5...
OD Cost Matrix script failed.
Deleting intermediate outputs...
Failed to execute (CalculateAccessibilityMatrix).
Since there is no obvious reason for the failure, I wonder if this is a known issue.
Many thanks in advance.

From this chain of messages, it seems like the failure is occurring in the parallelized OD cost matrix, but the messages aren't being passed back correctly.
Can you confirm that you have the latest version of the toolbox (TransitNetworkAnalysisTools_0.7.1)?  I know I made a change back in April to improve exactly this issue, so I would expect better messaging.
You could also open up transit-network-analysis-tools\parallel_odcm.py in a text editor and verify that lines 1110-1114 look like this:
    except Exception:  # pylint: disable=broad-except
        errs = traceback.format_exc().splitlines()
        for err in errs:
            LOGGER.error(err)
        raise

If you don't have the latest version or this code, try downloading the latest and see if that resolves the issue or at least improves the error message.

Good morning, thank you for your suggestions. I can confirm that I have the latest version (0.7.1) and the lines within the parallel_odcm.py file are identical. I have tried to run the tool locally instead of a network drive, but got exactly the same error message.
I have tried to run another tool now ("Calculate Travel Time Statistics") and interestingly it terminates with the same error message:
Validating OD Cost Matrix settings...
Inputs successfully validated.
Precalculating network location fields for I:...\scratch.gdb\TNAT_TempInput...
Precalculating network location fields for I:...\scratch.gdb\TNAT_TempInput0...
OD Cost Matrix script failed.
Deleting intermediate outputs...
Failed to execute (CalculateTravelTimeStatisticsOD).
I wonder if there is something not ok with my network (although the built in network analysis tools function perfectly). Are there any special settings I should be looking for?

I have restarted the computer and at least now I am getting an error message, however only at a later stage:
Finished OD Cost Matrix calculation 121 of 122.
Finished OD Cost Matrix calculation 122 of 122.
Calculating statistics for final output...
Traceback (most recent call last):
File "", line 900, in execute
File "C:\Temp\TransitNetworkAnalysisTools_0.7.1\CalculateODMatrixInParallel.py", line 438, in solve_large_od_cost_matrix
self._execute_solve()
File "C:\Temp\TransitNetworkAnalysisTools_0.7.1\CalculateODMatrixInParallel.py", line 613, in _execute_solve
super()._execute_solve()
File "C:\Temp\TransitNetworkAnalysisTools_0.7.1\CalculateODMatrixInParallel.py", line 388, in _execute_solve
msg_string = output.strip().decode()
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xe4 in position 81: invalid continuation byte
Failed to execute (CalculateAccessibilityMatrix).
The precalculation seems to be successful this time.

Oh!  THIS problem.  Ugh, somebody else had this problem as well, and I was never able to reproduce it on my end, so I couldn't figure out how to fix it.
I don't know why you didn't see these details (the UnicodeDecodeError) the first time until you restarted your machine. My guess is some older version of the tool was cached.  Anyway, let's ignore that for now.
Here are the technical details of what's happening: The CalculateODMatrixInParallel.py file calls the parallel_odcm.py file as a subprocess.  This is a workaround to allow parallel_odcm.py to launch parallel processes, which you can't do from a script running directly in the Pro UI.  That subprocess logs various progress messages and passes them back to CalculateODMatrixInParallel.py, which writes them to the Pro geoprocessing pane's messages.  Something bad is happening when it tries to grab one of those logged messages because it can't decode it into the utf-8 character set.  So, one of the messages logged by the subprocess in parallel_odcm.py has some unrecognized character, and CalculateODMatrixInParallel.py chokes on it.
The real question is why parallel_odcm.py is writing the problematic message in the first place.  My guess is one of your filepaths or something in your data includes a weird character with a non-standard accent mark or umlaut or something.  The other person who had this problem eventually solved it by removing all non-ascii characters.
This SHOULD work.  I want to support all utf-8 characters, so I would really like to be able to reproduce the problem on my end.  I know you can't share your data, but if you're able to share any text strings with accent marks that you're using, that might help.

Thanks once again for the detailed description. Really appreciate this support. I actually had the same thoughts about special characters (since we have some umlaut in the data). I have the created a new point layer with only the relevant information which are numeric, hence there are no strings anymore.

This is really anything that is left in the destination layer. The origin doesn't contain any special characters either. However, I still receive the same error message.
I will once again shut down the computer tonight and will try again tomorrow with a different network dataset,

My guess is the special character is in one of the filepaths, not in the data itself.  None of the messages the file generates should use the field values (I don't think...), but the filepaths are definitely used.

I can confirm that this was the issue! There was a special character in the file path. I have copied the whole project and data into another local folder and tada, it worked. Many thanks once again for your help.

Can you please tell me what the special character was?  I really want to fix this problem, but I can't do so until I can reproduce it reliably on my own machine.
Please paste the special character here in a GitHub comment.  Also, if you could paste a screenshot of what the character looks like on your machine, that would also help, since pasting it into GitHub might automatically alter it.

I still can't reproduce the problem locally, but I've taken an educated guess about what's causing the problem and published a branch with an attempt at a fix.
If you have the time for a little bit of experimentation, could you check out this branch locally: https://github.com/Esri/public-transit-tools/tree/fix-encoding  Try re-running the tool using the project it its original location when it caused the problem and see if this fixes it.

PS.  Before re-testing, make sure to close and reopen Pro or it won't pick up any of the code changes in the Python files.

Hello, the special character was an "ä" - so a typical german umlaut.

Unfortunately, I again cannot share the complete file path due to security issues.
I will try to re-run the tool later this week with your modified version. I am super busy for the next few days with projects but will find some time for it on thu of fri.

Another user was able to reproduce this problem and do some tests for me.  My latest change resolved it for her.  Please let me know if you experience it again and feel free to reopen this issue.


Update default git version

Tool information

Tool name: Git
Add or update? Update
Desired version: 2.25.0
Approximate size:
If this is an add request:

Brief description of tool:
URL for tool's homepage:

Area for Triage: git
Question, Bug, or Feature?: Bug
Virtual environments affected

[ ] macOS 10.15
[ ] Ubuntu 16.04 LTS
[ ] Ubuntu 18.04 LTS
[x] Windows Server 2016 R2
[x] Windows Server 2019

Can this tool be installed during the build?

Are you willing to submit a PR?
It appears that running the install script will automatically install the latest version.
My Additional Notes
The currently installed version of git (2.24) contains what I think is a pretty severe bug since it can prevent you from cloning your repository.
See the following from the Git 2.25 Release Notes:
 * An earlier update to Git for Windows declared that a tree object is
   invalid if it has a path component with backslash in it, which was
   overly strict, which has been corrected.  The only protection the
   Windows users need is to prevent such path (or any path that their
   filesystem cannot check out) from entering the index.
   (merge 224c7d70fa js/mingw-loosen-overstrict-tree-entry-checks later to maint).

Hello!
We install git on Windows images using choco so it updates automatically with each image rollout.
Currently, win images has git 2.25.0 so I am closing this issue.


Running EC2 outside of US-West-2 errors

As discussed in #497
https://github.com/Kitware/HPCCloud/pull/497#issuecomment-242533697

@TristanWright Try now I have copied the required AMI's to all the US regions. We should add a note to the docs that the PyFR and ParaView workflow are currently only supported in US regions and that they can contact us if they want to run in another regions and we can copy the AMIs over.

Note you will need an updated cumulus.

Cool, this looks all cleared up.


the speed of the demo

hello, your project works really very well, but it takes 1 second to predict one image on GPU of 12G memory,   I want to ask is it normal?  thank you.

@yuanyuanli85
thank you for your reply.  I set the predict batch_size to be 32 and it works much faster.


add time to data master so we can track most recent additions to database

good user feature?

Or we can have a secondary key that auto-increments? To discuss with Kara

I'm gonna bring this back up with kara in the meeting tomorrow.


empty compute shader resources from UE4 in GL

Hi, I'm running renderdoc on UE4 and running into a case where one particular compute shader can't be inspected in renderdoc. The calls to glDispatchCompute are in the Event Browser but when I inspect the compute state the shader id is the id from the previous dispatch and all of the resources are highlighted in red and labelled "Empty". There are several dozen dispatch calls to about five different shaders and only the calls to this specific shader don't work. Any thoughts on how I could go about diagnosing this? Happy to modify the engine if necessary. Thanks!
p.s. - Running in GL from Windows 10. For what it's worth, if I run in DX11 the dispatch calls to this shader are inspectable.

Just to get it out of the way - I guess you can't send me a capture repro'ing the problem? That'd be the easy way to investigate it :smile:.
Failing that, it sounds like something's going wrong identifying the bound program. You'd need to see what's going on with the bind and why it's not updating the current state. If you look at the dispatch draw that's broken and look in the API inspector, can you see a glUseProgram or glBindProgramPipeline? That's the one to investigate. You can identify the event ID of it and then breakpoint in Serialise_glBindProgramPipeline or Serialise_glUseProgram to see if you can spot anything wrong (conditionally breakpointing on m_CurEventID to find the right call). Also worth checking to see if there are any errors in the diagnostic log.
Then also look in GLReplay::SavePipelineState where it fetches curProg from either the current program (or if that's 0) the current pipeline binding, and see if anything's going wrong there.
If you get more detail I can tell you where to dive in more, if you don't find anything wrong I can make a branch with some diagnostic logging added to see if we can track where the problem is from there.

Can't send it, but might be able to create a simple test case.
Ah, yep, I do see glUseProgram in the API inspector, and it's got a unique ID, so I'll try what you suggested first. Thanks!

Ok, figured it out. Was due to a bug in HLSL->GLSL translation from Unreal. It was emitting a swizzle on a constant like this: "0.xxxx" which is invalid GLSL. renderdoc prints compile errors to the output log but I didn't notice them until running in VS. Thanks for the help!


[Feature Request]: Implement Role connections

Summary
Implement the upcoming  role-connection feature
What is the feature request for?
The core library
The Problem
See https://github.com/discord/discord-api-docs/commit/6a9c1b49fe9c31ceba28d48c9a26408c706de2ef
The Ideal Solution
~
The Current Solution
No response
Additional Context
No response

-> https://github.com/discord/discord-api-docs/pull/5668


pagination poorly aligned

I would think the pagination would be up a line. Taken from the docs, I see this:

add this definition to you custom css, the margin seems to come from bootstrap
.ng-table-pagination {
  margin: 0;
}
```


New Item: Small Game

please create a small game, that you can play in the lecture hall during boring lessons.
For example flipping a coin, then the item itself would be the coin, but you can decide.

@Isabelle-Gbl  Do you think it is okay to add in our mnk game with both of its classes in a simplyfied version? I will try it..

@Isabelle-Gbl @florian-huber  I got it to work, but it started the game as soon as i enterd the room or, like it is now, it doesnt start the game. I wanted to add an if statement that checks if the item (sloppy_RaspberryPi) is in the user_items list. If yes,
it should start the mnk game as soon as I enter the room. But it doesnt work for me, neither with an helping variable nor with checking the list. Do you have an idea what might be missing?

@Isabelle-Gbl check pull request #79


Bug found: No Player Voice Recorded in Video Generation Mode

Describe the bug
I attempted to generate rounds from this demo (link provided below), but the generated video no longer includes player voice as it did prior to the V3.10 update, even after using the commands tv_listen_voice_indices -1; tv_listen_voice_indices_h -1. However, the "Watch → Round" feature still includes player voice. Thank you.
Demo links
demo link:
https://www.faceit.com/en/cs2/room/1-019564cd-8bef-4a5d-ba30-2a9bd53b7ea1
Steps to reproduce

open the demo provided
right click on any players
generate → rounds
encoder I tried both, then click "Add to queue"

Expected behavior
I expected the generated video to include player voice, but I didn’t hear any, even after using the commands tv_listen_voice_indices -1; tv_listen_voice_indices_h -1. Notably, the "Watch → Round" feature still includes player voice.player's
How often does this bug happen?
Every time
Log file
No response
Environment information
Version: 3.10.1
OS: win32 x64 10.0.22631
Electron: 33.2.0
Chrome: 130.0.6723.118
Last database migrations:
v6 - 2024-11-16T12:08:13.433Z
v5 - 2024-08-20T22:59:00.285Z
v4 - 2024-08-07T11:28:35.938Z
v1 - 2024-08-07T11:27:38.129Z
Additional context
No response

Issue somehow solved


feat(docs): add markdownlint

Related Issues

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] I have read the CONTRIBUTING document.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Description
Implemented markdownlint to lint the docs folder, added the package itself & 2 scripts to the package.json to be able to lint & fix with markdownlint
Fixes #1071

Screenshots/Recordings

Added tests?

[ ] Yes
[x] No, because they aren't needed

Added to documentation?

[ ] README
[ ] Documentation
[x] No Documentation Needed

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] Any dependent changes have been merged and published in downstream modules

Your commit messages are not conventionally supported.


TS-2237: Add unit tests for escapify_url and pure_escapify_url

Unit tests for #866

FreeBSD build failed! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/600/ for details.

Linux build failed! See https://ci.trafficserver.apache.org/job/Github-Linux/496/ for details.

CI builds fail because the functions are not in the code yet.

[approve ci]

Linux build failed! See https://ci.trafficserver.apache.org/job/Github-Linux/619/ for details.

FreeBSD build failed! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/723/ for details.

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/744/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/640/ for details.

Rebased to obtain the functions from master.

Looks good to me.


Generating with Chromium is actually LibreOffice

When sending an API request to POST 'http://localhost:3000/forms/chromium/convert/url' I expected the renderer to be Chromium and not LibreOffice.
Is there a way to force usage of Chromium?

I guess you’ve requested a special format like PDF/A-1 ? It’s LibreOffice that handles such a conversion AFTER the initial PDF conversion from Chromium.

That noise you heard was me face-palming too hard. Thanks!


Guest Account with PBI Pro License

It doesn't seem like this article is accurate on the licensing side.  I used a guest account (with B2B) to access a shared report.  When logging in, I was prompted with the Pro upgrade notice, so having a Pro license in the home tenant does NOT allow access to shared content in a guest tenant.
This is the message:
This content is only available to users with a Power BI Pro license. When you upgrade, you get access to collaborate with others and distribute content. Upgrade today or try it free for 60 days.Learn more

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

ID: 7ad42c3b-e011-733d-dbdf-53944a2724a6
Version Independent ID: 05e0f67c-5852-5b10-ac94-694ec2d0d8f9
Content: Distribute Power BI content to external guest users using Azure Active Directory B2B - Power BI
Content Source: powerbi-docs/whitepaper-azure-b2b-power-bi.md
Service: powerbi
Sub-service: powerbi-service
GitHub Login: @davidiseminger
Microsoft Alias: davidi

Hi, @danmyhre -- thanks for asking! Assigning to the author to look into it.

Including the content owner of that whitepaper.

Adding additional content folks to review and comment.

Hi @danmyhre ,
I see that this Git issue is a bit old, so I'm going to close it.  I hope that you were able to get your questions answered. If not, please revisit the article since it has been updated several times since this Git Issue was submitted.  Another option is to submit your question to the Power BI community (community.powerbi.com).
All the best,
the DOCS team


fix(temp): Use fork of smoke-and-mirrors

This project uses semver, please check the scope of this pr:

[ ] #none# - documentation fixes and/or test additions
[x] #patch# - backwards-compatible bug fix
[ ] #minor# - adding functionality in a backwards-compatible manner
[ ] #major# - incompatible API change

CHANGELOG

Pull in temporary fork of s&m (pre 1.0.0)
Remove whitespace in content-container className

Changes Unknown when pulling d0b2a6fe1fbd7dfb8db8723bc7c2cda94c5d97a3 on srowhani:feature/fork-smoke-and-mirrors into ** on ciena-frost:master**.

Changes Unknown when pulling d0b2a6fe1fbd7dfb8db8723bc7c2cda94c5d97a3 on srowhani:feature/fork-smoke-and-mirrors into ** on ciena-frost:master**.

Approved


Define what constitutes an Active Effect for the purposes of the system

Active Effects are a powerful tool that can be used for many things in Foundry. We should write down a proper definition, along with examples, that we can consult in the future anytime we have doubts about whether something should be an AE or something else.

I have taken a look at several other systems, including 5e and WFRP 4e, and they appear to have dedicated lists for "permanent" / "passive" effects. Considering this was my main and basically only point of contention on how we move forward, the issue is resolved as I've come around on my opinion.


Github link

the github link could be removed and hyperlinked in your name:

Suggestion: It can be hyperlinked in a github icon maybe a complete social tab in the footer


Integration Test - Release - 2018/09/20 19:39:27

Auto create for integration test. AnotherAuthorUpdatedArticleFlow

@PRMergerTest2 : Thanks for your contribution! The author, @v-caxian, has been notified to review your proposed change.


WebXR Anchor system should provide full abstraction on top of XRAnchor

Currently in the WebXRAnchorSystem the underlying XRAnchor is exposed directly to the user, and the only way to delete an anchor is by calling anchor.xrAnchor.delete().  This goes against the typical design of WebXR features in Babylon where we create a full abstraction on the object to make it resistant to API surface area changes.
I would propose removing access to the XRAnchor object, and adding in a remove method to the WebXRAnchorSystem that allows users to explicitly remove the object.  '
eg:
IWebXRAnchorSystem.remove(anchor: IWebXRAnchor)
Additionally the promise returned by the creation functions is to the underlying XRAnchor object, which means that it is not actually yet available to be used by the consuming application as we cannot attach BabylonNodes to it or work against the anchor.
Since this API is already asynchronous I think it would make sense to have it not resolve the promise until we have a valid IWebXRAnchor.  This would allow us to more cleanly create and attach to anchors without having to track across callbacks.
eg.
const transform = new TransformNode();
const webXRAnchor = await addAnchorAtPositionAndRotationAsync(Vector3.Zero(), Quaternion.Identity());
webXRAnchor.attachedNode = transform;

@Alex-MSFT - promises will now resolved only when the anchor is tracked (great suggestion!) and are resolved with the IWebXRAnchor object, which now also has a remove() function. I preferred to keep it at the Anchor level, as I want to avoid a filter/find call every time we search by ID.
I made a few more changes here - https://github.com/BabylonJS/Babylon.js/pull/9282.
Closing this, please reopen if there is anything I didn't address.

I'd like to test this out. Which currently available device/browser combination allows testing for anchors?


Add support for Java

Using custombinding is acceptable here but would need to be documentable, may be easier to just add library support https://github.com/Azure/azure-functions-java-library

Anyone has an example to achieve in java?

@jeffhollan  / @fabiocav  - I recommend following the approach for signalr and create Java annotations in this repo.
https://github.com/Azure/azure-functions-signalrservice-extension/tree/master/binding-library/java/src/main/java/com/microsoft/azure/functions/signalr
This allows to keeping annotations and c# attributes in sync and can be released independently of azure-function-java-library

cc @cachai2

version 1.0.0-beta of azure-functions-java-library-rabbitmq is released to maven.


Test - Please delete after routing

Trying to verify that author pochiraju can receive comments on articles.

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

ID: 2469f918-a128-5375-89be-632e86af4eab
Version Independent ID: 137d87f2-a1bf-e6fb-d9c4-be438b7a234f
Content: Azure Database Migration Service Overview
Content Source: articles/dms/dms-overview.md
Service: dms
GitHub Login: @pochiraju
Microsoft Alias: jtoland

@HJToland3 Is there a specific issue we can help you with?
you can @mention individuals with their Github login to send them a notification.
We will close this issue now. If there are any further concerns, please comment and let us know.

Yes - I am trying to determine if the changes we made to the account for @pochiraju are sufficient for this item to be routed to him. We had problems with this in the past. Can you let me know if this worked?

@KalyanChanumolu-MSFT, just to ensure that you see my comment above.


SyncTrayzor still appears in Task-Manager Autostart after uninstall

Version SyncTrayzor: 1.1.24
Version Syncthing: v1.11.1
OS: Windows 10 Pro
Version OS : 10.0.19042 Build 19042
When using the unins000.exe to execute the deinstall process, it won't remove the SyncTrayzor entry in the Autostart tab in the Task-Manager. After installing and uninstalling SyncTrayzor a few times I've had serveral inactive instances of SyncTrayzor.

This is impossible to do in the general case (autostart registry keys are per-user, but the installer runs as a single particular user).
I've updated the installer to make a best-effort attempt to clean these up: it will work provided that the user who runs the uninstaller is the same user who ran SyncTrayzor, and you didn't need to elevate to a different user.


Refresh not working (again) in 1.5.2

In Fresh 1.5.2 browser refresh is not working when in development mode. When looking into Chrome devtools, I cannot see any injected scripts. Seems to be similar to #1659.

Resolved by https://github.com/denoland/fresh/pull/1956


Nesting directories/files

I have a folder/file structure like this:
controller/
  app/
    token.js
    token/
      access.js
      authorization.js
      refresh.js

It would be nice to be able to extend off of app allowing to have multiple levels. I can't seem to get it working and didn't see anything in the documentation explaining this.
Example:
controller.app.token();
controller.app.token.access(); // This doesn't work.

I'm having the same trouble...

@caio-ribeiro-pereira As a hacky solution I wound up just adding a loader function to each of my top level classes/functions. Hacky, but it works.
So token.js would have it's own express loader and load up the token.access function.

@caio-ribeiro-pereira Try my pull request.


Failed to submit a MERGE sql to SQL SERVER

When submitting a MERGE sql to the SQL server, dbeaver returns an error message complaining about the MERGE sql must end with a semicolon even through the semicolon is already there.
Tried to execute the exact query through Microsoft SQL Server Management Studio, it works fine.

你好，
你是来自中国的DBaver开发者吗？
目前，我刚开始学习RCP的开发和DBaver的代码，请多指教。
如果方便的话请加我QQ：731297616.多谢~
媛媛
2017.5.10
发件人: Jun Liu<EMAIL_ADDRESS>发送时间: 2017年5月10日 11:11
收件人: serge-rider/dbeaver
抄送: Subscribed
主题: [serge-rider/dbeaver] Failed to submit a MERGE sql to SQL SERVER (#1649)
When submitting a MERGE sql to the SQL server, dbeaver returns an error message complaining about the MERGE sql must end with a semicolon even through the semicolon is already there.
Tried to execute the exact query through Microsoft SQL Server Management Studio, it works fine.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/serge-rider/dbeaver/issues/1649 , or mute the thread https://github.com/notifications/unsubscribe-auth/AFHxbaShYMx1Od59EKg-0tBvrbbkHRY8ks5r4SqogaJpZM4NWINx .  https://github.com/notifications/beacon/AFHxbSwCT0xpbllnvsSJcIZVBxXQiqKiks5r4SqogaJpZM4NWINx.gif

Please post query sample.

MERGE INTO table1 AS T
USING (VALUES (166141, '20171229', '20171230', 'F')) as S (id, start_date, end_date, flag)
ON T.id = S.id and T.start_date = S.start_date and T.end_date = S.end_date
WHEN MATCHED
THEN UPDATE SET flag = S.flag
WHEN NOT MATCHED
THEN
INSERT (is_latest, id, start_date, end_date, create_time, create_user, update_time, update_user, flag)
VALUES (1, 166143,	'20171229', '20171230', '20170510', 'liujunj', '20170510', 'liujunj', 'G');
When this query is submitted through dbeaver to SQL Server, I got the error message listed in the issue description.

Thanks. Not sure how to fix this yet.
By default DBeaver cuts trailing semicolon because it shouldn't be there for queries/databases.
Maybe I'll add a special handler for SQL Server and MERGE commands.
Workaround: add double semicolon in the end, select query and execute with ctrl+enter.

Added extra option which leaves delimiters after queries.

I wonder if this is fixed correctly, or if there's a regression. I can execute the MERGE statement when I select it entirely, like this:

But when I just place the caret inside of a statement and hit Ctrl+Enter, then the semi colon is not sent to the server

The suggested workaround where I add a double semicolon at the end also doesn't work:

... Using DBeaver <IP_ADDRESS>003021717 on Windows 10 [Version 10.0.18362.720]

I think in release <IP_ADDRESS>311191730 this problem is "back"...
Create a table :
create table mytest(thekey nvarchar(12),thedescr nvarchar(12))
and try to merge  :
with s as (
select 'x'     thekey,'descr' thedescr
          )
merge mytest
   as target
using s as source
   on (target.thekey = source.thekey)
 when matched and target.thedescr <> source.thedescr
 then update  set target.thedescr =  source.thedescr
when not matched by target
then
insert (
       thekey
      ,thedescr
) values (
        source.thekey
      , source.thedescr
);;

Results in :
SQL Error [10713] [S0001]: A MERGE statement must be terminated by a semi-colon (;).

Error position: line: 31

Note : The double semicolon at the end was the workaround that did the trick til last release . Now 2 , 1 or no semicolon all end up in the error (  even 3 semicolon :-) )

I'm also getting this error - only when trying to execute sql that creates a stored procedure with a merge statement as a script.  The same exact code works fine if I execute it directly (not as a script).  The merge statement also executes fine by itself as a script - only fails when inside a stored procedure and running as a script.
CREATE PROCEDURE ${schema_name}.LoadWorkOrdersToImport
AS
    BEGIN
	    -- Mark WO's that have changed since last imported STALE
	    -- Unless importstatus is any value other than 'DONE' (allows for ALWAYS to stay)
        MERGE INTO ${schema_name}.iVueWorkOrdersToImport AS target
		USING iVueWorkOrderHashView AS source
		ON target.wo_id = source.wo_id
		WHEN MATCHED THEN
		    UPDATE SET
		    	target.hashkey = source.hashkey
		    	, target.importstatus = case when target.hashkey <> source.hashkey
		    									and target.importstatus = 'DONE'
							    			then 'STALE'
							    			else target.importstatus
							    			end
		WHEN NOT MATCHED BY TARGET THEN
		    INSERT (wo_id,hashkey,importstatus)
		    VALUES (source.wo_id,source.hashkey,'STALE')
		WHEN NOT MATCHED BY SOURCE THEN
			DELETE;

    END;


Diff using joins when both tables are within the same database.

Describe the solution you'd like
When data-diff sees both tables are in the same database, it may switch to an alternative, more efficient algorithm that uses joins, and therefor doesn't download any unnecessary data.

On one hand, that sounds like a good idea in the abstract.
On the other, if you have two tables in the same database containing (largely) the same data, that's indicative of some serious problems per basic relational theory.  Is it actually likely that a case like this would see much use?

It could be used, for example, to validate short-term backups, or data versioning.
 So the almost-same-data limitation no longer applies.

Plan

Make sure there are no duplicate keys or NULL keys

Split the tables into small segments, like in data-diff, so we can query from multiple threads

For each pair of segments, use FULL OUTER JOIN on the keys. Missing rows will produce NULL keys. When FULL OUTER JOIN isn't available (MySQL), we can use LEFT JOIN ..  RIGHT JOIN, which should produce the same result, as there are no duplicate keys.

Compare the columns using IS DISTINCT FROM, or equivalent. (<=> in MySQL, etc.), which treats NULL as a distinct value, like Python. If it's not supported by the db, we can fall back to the (a=b OR (a is null AND b is null)) pattern. Return 0 if equal, or 1 if different.

Count the diff for each column using SUM(is_diff_col_N). Count the total diff using sum(is_diff_col_N OR is_diff_col_N+1 OR ...),

Collect additional aggregations, such as sum of values. Open question: Do it in the same query, or in a separate query?

Algorithm draft
Here is how it might look in Python code.
(Actual code could end up looking quite different)

pk_columns = ('id1', 'id2', ...)
extra_columns = ('timestamp', ...)
a = table1
b = table2

j = outerjoin(a, b).on(a[pk_columns] == b[pk_columns])

diffed = j.select(
    '*',
    **{f"is_diff_col_{c}": is_distinct_from(a[c], b[c]) for c in extra_columns}
)

diffed.select(
    **{f"total_diff_col_{c}": sum(diffed[f"is_diff_col_{c}"] for c in extra_columns}

    total_diff = sum(or_(diffed[f"is_diff_col_{c}"] for c in extra_columns )),

    # stats - separate query?
    total_a = count(a),
    total_b = count(b),
    **{f"sum1_col_{c}": sum(a[c]) for c in extra_columns}
    **{f"sum2_col_{c}": sum(b[c]) for c in extra_columns}
)

Implemented.


WIP - Add subgraph for Comptroller BalanceDrips

@robsecord I wanted to see the changes in this branch easily so I created a PR. Obviously WIP, not to be merged in yet!

Lookin good! 👍


Revert delete before replace changes

Context:
https://github.com/pulumi/pulumi-gcp/issues/2596
Revert:

pulumi/pulumi-gcp#2586
pulumi/pulumi-gcp#2588

Affected resources:

google_compute_global_address
google_compute_instance
google_compute_vpn_tunnel
google_project_iam_member
google_service_account_iam_member

/release patch


Wait longer for the server in TLSProxy to start

In a recent PR (#3566) it seems that TLSProxy gave up trying to connect to
the server process too quickly. This meant the test failed even though the
server did eventually start. Currently we try 3 times to connect with a
0.1 second pause between each attempt. That is probably too aggressive.

Pushed. Thanks.


Set the time as UTC +1 not local.

It doesn't seem that easy, I should request an external API to find the actual time.

It doesn't seem that easy, I should request an external API to find the actual time.

This is resolved client-side on 6a7c938b008e9463f94d7fdf6a1cb91c8c2a7dda