Deletion not detected by VarDict

Dear all,
I am using VarDict in a sample with known genotype. The sample was sequenced with an Illumina TruSight custom amplicon for BRCA1/2 on a MiSeq sequencer.
We know it has a deletion of 70bp on BRCA1 gene but VarDict does not detect it (while HaplotypeCaller yes).  I generated the 8 columns bed file as suggested starting from the Target section of Illumina Manifest, here some lines as example
chr13   32890519        32890777        BRCA2   0       +       32890547        32890714
chr13   32893133        32893406        BRCA2   0       +       32893163        32893512
chr13   32893349        32893611        BRCA2   0       -       32893163        32893512
chr13   32899118        32899337        BRCA2   0       +       32899162        32899371
chr13   32899280        32899538        BRCA2   0       -       32899162        32899371
chr13   32900061        32900308        BRCA2   0       +       32900187        32900800
chr13   32900253        32900530        BRCA2   0       -       32900187        32900800
chr13   32900475        32900751        BRCA2   0       +       32900187        32900800
chr13   32900697        32900969        BRCA2   0       -       32900187        32900800
chr13   32903497        32903767        BRCA2   0       +       32903529        32903679

Then I used vardict as suggested in the manual page, here my line:
perl ../VarDict-master/vardict.pl -G hg19.fa -N 14-0101 -b 14-101.bam -f 0.01 -c 1 -S 2 -E 3 -g 4 -I 1000 file_for_vardic.sorted.bed | ../VarDict-master/teststrandbias.R | ../VarDict-master/var2vcf_valid.pl -N 14-0101 -E  > 14-101_vardict.vcf
But the deletion is not presente in the final VCF, I tried using different values of -f but nothing changed..Moreover VarDict does not detect a SNV (known to be there) in the same gene.
Is there something that I am doing wrong? Am I missing something?
Thanks for the help,
Stefania

Hi Stefania,
Thanks for trying out VarDict and sorry about the issues. When using VarDict in amplicon aware mode the last two columns must define the "insert" and be subsets of those values in columns 2 and 3.
Your bed file doesn't look quite right for this purpose as the last two columns keep repeating and are not subsets of columns 2 and 3. If you are OK to run VarDict without amplicon awareness, just drop the last two columns and let me know if you see the variant?

Hi @mjafin,
thanks for the reply!!
You are completely right, looking at manifest file I verified that this happens when there is more than one probe for the same target. Moreover they are in opposite strand.
From the manifest file is impossible to identify the exact portion of the target that is detected from the probe, so do you think that in these cases I can use the first coordinate of the first probe and the last coordinate of the second probe for the bed file?
For example, these are the original coordinates as declared in the manifest file:
chr13   32893133        32893406        BRCA2   0       +       32893163        32893512
chr13   32893349        32893611        BRCA2   0       -       32893163        32893512

And this is the new line:
chr13        32893133        32893611        BRCA2   0       +       32893163        32893512
In this way the target is fully included in the probe section. I used the plus strand even if the second coordinate is declared in the - strand. Is this correct or the strand is an important information?
I want to use the amplicon mode because I am sure that this is an amplicon based panel. But if you think that the result is not affected by this choice I can try using the software in the other way.
Many thanks for the support!!
Stefania

Hi Stefania,
I wouldn't recommend combining amplicons in the bed file into one pseudo amplicon as the algorithm was really designed to pick up individual amplicons, which may or may not overlap. In your case I suspect the problem would be that there would be reads coming from two slightly different amplicons and you're representing them as one in the bed file
The assumption in amplicon VarDict is that for each bed line file the majority of reads coming from given amplicon will have reads starting exactly at the coordinate defined in column 2 and ending exactly at the coordinate in column 3. The algorithm then works out, based on columns 7 and 8, how many primer bases to ignore at the beginning and end of the fragment.
Unfortunately if you don't have the exact amplicon and insert coordinates the algorithm is not going to perform as designed and you'll have false negatives most likely.

Dear @mjafin
Sorry for my late answer and thank you so much for the kind help.
Considering your answer I tried to correct my bed file. I considered the manifest given by the vendor. It is divided in two section, one for the target region and one for the probes.
The part for the probes has these informations:
Target ID       Species Build ID        Chromosome      Start Position  End Position    Strand  ULSO Sequence   DLSO Sequence   Probe Strand
And the part of the target has these informations:
TargetA TargetB Target Number   Species Build ID        Chromosome      Start Position  End Position    Probe Strand    Sequence
Because I have the sequence of the specific target region and of the primers, I decided to blast all the sequences and I retrieved all the start and end points.
In the end I generated the new file for VarDict and it looks like this:
chr13   32890519        32890777        BRCA2   0       .       32890545        32890750
chr13   32893133        32893406        BRCA2   0       .       32893161        32893378
chr13   32893349        32893611        BRCA2   0       .       32893377        32893584
chr13   32899118        32899337        BRCA2   0       .       32899148        32899309
chr13   32899280        32899538        BRCA2   0       .       32899307        32899511
chr13   32900061        32900308        BRCA2   0       .       32900090        32900280
chr13   32900253        32900530        BRCA2   0       .       32900280        32900502
chr13   32900475        32900751        BRCA2   0       .       32900502        32900724
chr13   32900697        32900969        BRCA2   0       .       32900724        32900942
chr13   32903497        32903767        BRCA2   0       .       32903524        32903740
chr13   32904967        32905187        BRCA2   0       .       32904993        32905160
chr13   32905133        32905363        BRCA2   0       .       32905160        32905336
chr13   32906230        32906449        BRCA2   0       .       32906258        32906419
chr13   32906382        32906601        BRCA2   0       .       32906410        32906571
chr13   32906500        32906719        BRCA2   0       .       32906529        32906691
chr13   32906662        32906883        BRCA2   0       .       32906691        32906855
chr13   32906780        32906999        BRCA2   0       .       32906807        32906969
chr13   32906892        32907112        BRCA2   0       .       32906920        32907085
chr13   32907058        32907337        BRCA2   0       .       32907085        32907311
chr13   32907284        32907522        BRCA2   0       .       32907311        32907495

I then run again the command:
perl ../VarDict-master/vardict.pl -G hg19.fa -N 14-0101 -b 14-101_S1_RGSorted.bam -f 0.01 -c 1 -S 2 -E 3 -g 4 -I 15 new_vardict.sorted.bed | ../VarDict-master/teststrandbias.R | ../VarDict-master/var2vcf_valid.pl -N 14-0101 -E > 14-101_vardict.vcf
But as before both the deletion and the SNV are not detected.
So I suppose that my bed file is still wrong even if this time looks correct..
At this point, do you think that is not possible to use VardDict in this kind of amplicon panel?
Do you have any experience in using VarDict with pre-designed commercial amplicon panels?
In particular I am using Illumina TruSight Custom Amplicon for BRCA1/2 genes.
Thank you so much for the help and support!!
Stefania

Hi Stefania,
Sorry about the problems you're seeing. Here are a few things you could do to help debug this problem:

Paste a screenshot of the reads in the regions of the variants and include your current bed file as a track (IGV, NGB or similar browser) so that we can see if the variant is e.g. in only one of overlapping amplicons, close to amplicon boundary or similar
Try to replace vardict.pl with the Java version of the VarDict algorithm: https://github.com/AstraZeneca-NGS/VarDictJava - try the most recent release. It's a drop-in replacement for the perl script.
If still no luck you could also provide us a slice of the bam file with the two variants and the bed file lines for the associated regions.

We have primarily used VarDict with Qiagen GeneRead v2 panels where Qiagen provided the bed files. We haven't used TSCA much as it doesn't work with the material we have (FFPE, ctDNA) as the amplicons tend to be too long but that's an aside.

@smerella, hello!
I've checked this issue and after trimming the adapters from initial BAM this deletion was detected by VarDict. The adapters were determined as parts of softclips in the realignment and they influenced on detection of such variants. I guess I can close this issue.


Bugfix for SVD Recommender SGD and RMSE calculation

fix partial derivatives of user factor.  delta_u = err * i_f - self.regularization * u_f
current rmse formula is sqrt(sum(error)**2 / num. of training instances)
It should be RMSE = sqrt(sum(error ** 2) / num. of training instances).

Thanks

Thanks Alex Lin! :D Pull request accepted!


Remove provisioner dependencies from e2e provisioning test

Description
Changes proposed in this pull request:

Remove provisioner dependencies
Bump other dependencies

Related issue(s)

/kind enhancement


Channels are no longer working.

Hello,
It looks like the channels are no longer working. The website and mobile app were recently updated. Would it be possible to update the add-on?
Thank you

Sorry, I posted my issue for the wrong project. I just noticed the new version has a different name. Will close this issue and open a new one on the correct project.


operations that select fields from extend type Query fail to validate

Description
I'm getting incorrect diagnostics when trying to select fields defined on a type extension.
Steps to reproduce
let mut compiler = ApolloCompiler::new();
compiler.add_type_system(
    r#"
type Query {
  foo: String
}

extend type Query {
  bar: String
}
"#,
    "schema.graphql",
);

compiler.add_executable("query Query { foo bar }", "op.graphql");
let diagnostics = compiler.validate();
for diagnostic in &diagnostics {
    println!("{}", diagnostic);
}

Expected result
No diagnostics.
Actual result
  × Cannot query `bar` field
   ╭────
 1 │ query Query { foo bar }
   ·                   ──┬─
   ·                     ╰── `bar` field is not in scope
   ╰────
  help: `bar` is not defined on `Query` type

Environment

Crate version: 0.6.0

yep, this one is legit - we are still add extensions to our validation rules! coming super soon though

this one is fixed with some of the work in #482 and #484, and is published as part of the new releases. cargo add<EMAIL_ADDRESS>


Unable to specify path for config files.

I have used following config file (YML)
storage:  ~/verdaccio/storage
auth:
    htpasswd:
         file: ~/verdaccio/htpasswd

But , it is creating verdaccio folder in current directory of config file but not the actual home directory.

Some info relevant how we treat those paths (https://verdaccio.org/docs/en/cli#default-config-file-location)

@juanpicado - I am following following structure.

package.json
{
  "name": "verdaccio-repo",
  "version": "1.0.0",
  "description": "",
  "main": "index.js",
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1",
    "start": "verdaccio --config ./config.yaml"
  },
  "repository": {
    "type": "git",
    "url"<EMAIL_ADDRESS>  },
  "keywords": [],
  "author": "",
  "license": "ISC",
  "homepage": "https://bitbucket.org/rivigotech/verdaccio#readme",
  "dependencies": {
    "verdaccio": "^3.8.6"
  }
}

config.yml
storage: ~/verdaccio/storage
# path to a directory with plugins to include
plugins: ./plugins

listen: <IP_ADDRESS>:4000

web:
  # WebUI is enabled as default, if you want disable it, just uncomment this line
  #enable: false
  title: Rivigo NPM Registry

auth:
  htpasswd:
    file: ~/verdaccio/htpasswd
    # Maximum amount of users allowed to register, defaults to "+inf".
    # You can set this to -1 to disable registration.
    #max_users: 1000

# a list of other known repositories we can talk to
uplinks:
  npmjs:
    url: https://registry.npmjs.org/

packages:
  '@*/*':
    # scoped packages
    access: $authenticated
    publish: $authenticated
    proxy: npmjs

  '**':
    # allow all users (including non-authenticated users) to read and
    # publish all packages
    #
    # you can specify usernames/groupnames (depending on your auth plugin)
    # and three keywords: "$all", "$anonymous", "$authenticated"
    access: $authenticated

    # allow all known users to publish packages
    # (anyone can register by default, remember?)
    publish: $authenticated

    # if package is not available locally, proxy requests to 'npmjs' registry
    proxy: npmjs

# To use `npm audit` uncomment the following section
middlewares:
  audit:
    enabled: true

# log settings
logs:
  - {type: stdout, format: pretty, level: http}
  #- {type: file, path: verdaccio.log, level: info}

and then starting the verdaccio with following command.
npm run start

====
The above start the verdaccio but it create a directory inside the git repo project "~". Like this

I actually, want to point the home location or custom location relative to home.
cc - @ayusharma

why storage: ~/verdaccio/storage ? Any specific reason? It takes user home folder by default.

@juanpicado
storage: ./storage

The default value is ./storage
If I use this default value then it will create storage folder in the current git repo path.
storage: ./storage
plugins: ./plugins
listen: <IP_ADDRESS>:8000
web:
  title: Rivigo NPM Registry
auth:
  htpasswd:
    file: ./htpasswd

then these files are created directly inside the git repo like this.

Am I able to explain my problem?

I am not using npm install -g verdaccio. Instead, I am committing my config.yaml file inside a git repo. in that git repo, verdaccio is a local package. and I am starting verdaccio using npm start command. Since, my config.yaml is located in the git repo path, it is generating storage and other files inside the git repo. In the main server, these files will be replaced at next deployments.
So, I want to specify some folder like /opt/verdaccio. I guess, these absolute path will work but I am trying to specify path relative to home directory but that is not working.
@juanpicado

To locate the home directory, we rely on $XDG_CONFIG_HOME as a first choice and Windows environment we look for APPDATA environment variable.  (it is mis-spelled in docs. I will fix it)
export XDG_CONFIG_HOME="<path-which-you-want-to-set>"
e.g.
export XDG_CONFIG_HOME="$HOME/github/verdaccio/conf"

@ayusharma - I have tried setting up XDG_CONFIG_HOME variable too. not worked.

@ayusharma @juanpicado
Please fork this repo - https://github.com/nsisodiya/npm-repo
Clone the repo and run npm start, This will start verdaccio using local config file.
I am unable to set the desired path location for storage. I want to create htpasswd and storage files in the <HOME_DIR>/verdaccio_conf directory.

@nsisodiya
Now, I took a look and solved your issue:
Please use absolute path for storage
# path to a directory with all packages
storage: /Users/ayushsharma/Berlin
# path to a directory with plugins to include
plugins: ./plugins
listen: <IP_ADDRESS>:9000

@ayusharma -
That is not a solution to my problem. I know from the beginning that absolute path will work. But I want to specify a particular path relative to the HOME directory of the user.
If this is not possible then please accept as feature/bug in the current system.

Please go through this https://stackoverflow.com/questions/21077670/expanding-resolving-in-node-js
As far as I know, It is not possible to resolve home path (~) in node.

@juanpicado @ayusharma - But there should be some way to specify relative path to home. Because, for this repo - I cannot specify the absolute path. Because, I really don't know the directory of deployment server.
Don't you think, it will be great if we able to do it by passing arguments to it.
Right now, verdaccio take arguments like --config`` and --listen`. So it will be great if we add this via one extra parameter.
Thanks.

@nsisodiya
I need some time to discuss this with @juanpicado and where can be ship this (in v3 or v4), if we decide to include this feat.

I am having a similar issue as @nsisodiya  except, I am using Windows and trying to save on a different drive.
# path to a directory with all packages
# storage: ./storage
storage: /d/Verdaccio
# path to a directory with plugins to include
plugins: ./plugins

creates the path C:/d/Verdaccio when obviously I want to files saved to D:\Verdaccio.

I think that's not possible. Verdaccio relies on local environment variables to define the root of the application. https://github.com/verdaccio/verdaccio/blob/master/src/lib/config-path.js#L77

@juanpicado If an absolute path is given, shouldn't that be used instead of any environmental factors?

All is possible, I'm just describing how it works now. The key is here https://github.com/verdaccio/local-storage/blob/master/src/local-database.ts#L276 where it joins the storage with self_path https://github.com/verdaccio/verdaccio/blob/master/src/lib/cli.js#L55 .
If someone manage to make it work with an absolute path while does not break current installations, I think I can push it to master. It seems trivial, but, be aware functional/unit testing might be affected.
Side note: any development should be done on top of 4.x branch and master on local-storage.


Sorry for asking but...

Is this a datapack or this mod is not compiled yet? I can't manage to make this mod work, i put in /mods folder and nothing happen... But in downloads on modrinth is only on .zip and not .jar... Sry i'm new on this and i don't know other way to ask you for help

First of all, did you put the mod into your server files or player files? It's server-side, meaning it will only work on servers.
Second: this mod uses something called Polymer library - maybe the mod works, but the textures dont load in? Try /polymer generate-pack - this will generate the resource pack with the textures which you can use locally (send it to players/set up an auto-download when players join your server). You can also check with /give @p fishing101:sushi if the mod compiled and if the textures work.
Zip is basically the same as .jar so this shouldnt be the issue.
Let me know if any of this worked and have a nice day!

@Macck209 I'm not sure why you liked my post.  What I'm saying is that you need to compile your mod, and upload the compiled jar to Modrinth/Curse and here as a release, instead of expecting people who want to use your mod to compile it themselves.

@Ricoware Yeah, sorry about that. I've got what you were saying and I'm really thankful for pointing out my mistake (imagine if nobody told me this - How many more people would get confused!). Haven't had the time to fix this yet tho. Will probably do it later today. Thanks again and sorry @lAlexiel for providing an unfinished project :C

@lAlexiel Should work now, just download the new version from modrinth.

Nice, it's good to have everyone on the same page.  :)  I've downloaded the jar and I'm currently testing it on my private server.  So far, so good!

@Macck209 Tysm ♥

All right! I'm gonna close this issue. If anything else comes up, just open another one. Thanks!


Invalid elliptic curve point encoding in proof

This happens with this test
---- autogen_aggregation_verifier stdout ----
thread 'autogen_aggregation_verifier' panicked at 'called `Result::unwrap()` on an `Err` value: Transcript(Other, "Invalid elliptic curve point encoding in proof")', /app/prover/src/aggregation_circuit.rs:213:26

Hey @pinkiebell, would you like to try to patch halo2_proofs with this branch https://github.com/han0110/halo2/tree/feature/configurable-instance-query?
I think in halo2@v2022_09_10 the instance commitments are still written to transcript, so plonk-verifier gets confused by some extra commitments there.
(hopefully these patching issues will never show again after both https://github.com/privacy-scaling-explorations/halo2/pull/96 and https://github.com/privacy-scaling-explorations/plonk-verifier/pull/5 are merged...)


gomendracollege.edu.np added

gomendracollege.edu.np domain added

@saileshrijal First stage of verification passed, official domain is gomendracollege.edu.np as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

is this domain (gomendracollege.edu.np) verified?

@saileshrijal Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Add screenshot

It's a bit hard to understand that it's something in Windows' UI.
https://visualstudiomagazine.com/articles/2016/10/01/universal-windows-platform-media-app.aspx

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

ID: c756203f-fa1e-c530-7827-96f1325f91a7
Version Independent ID: 86dc190a-3c44-e3bf-6b3c-3207dca7c371
Content: Integrate with the System Media Transport Controls - UWP applications
Content Source: windows-apps-src/audio-video-camera/integrate-with-systemmediatransportcontrols.md
Product: uwp
Technology: audio-video-and-camera
GitHub Login: @drewbatgit
Microsoft Alias: drewbat

and this also could apply to https://docs.microsoft.com/en-us/uwp/api/windows.media.systemmediatransportcontrols

I've added a screenshot to this page and updated the description of the SMTC to make it more clear that the SMTC is system UI: "The system transport controls enable media application developers integrate with the built-in system UI to display media metadata such as artist, album title, or chapter title. The system transport control also allows a user to control the playback of a media app using the built-in system UI, such as pausing playback and skipping forward and backward in a playlist."


Compiler Refactor

⚠ This PR takes as base gsoc20/build-aab, so the new build system can be tested in a real case with different build types. It should get soon changes from Android SDK 29 branch.

@ewpatton everything seems to be working, including AAB. I will start testing with an appengine server, to make sure progress is sent properly.
But you can start reviewing it whenever you want.


fix: handling of parameterized values that contain dt entity ids

What this PR does / Why we need it:
This is a funny one (also the solution).
Some configurations, e.g. synthetic monitors, support some special syntax to subistitute values, for example setting a header using the value like "Bearer:  {SOME_ENTITY_ID | token}" and things like that.
For quite some time, monaco inspects each payloads and searches for entity ids and replaces them with monaco references.
This process breaks go templating once we come across such values like above (e.g., In the example above something like {{{ externalIDs.SOME_ENTITY_ID }}} is created)
The solution: as soon as we download configuration, parse the json tree and replace each opening curly brace ("{") with a go template containing a function call, currently just called "curly": So each "{" gets replaced with {{ curly }} in the JSON string nodes. I did this because all other replacements strategies causes some error somewhere, e.g. replacing with {{'{'}} causes the go template engine to substitute { with 123 (the ascii code) which wherease {{"{"}} would correctly escape to just { but is no valid JSON any more and causes problems so it would need to be escaped to {{\"{/"}} which again is rejected by go templating, and so an and so forth....
Special notes for your reviewer:
If you come up with something better, feel free to discuss. I know that this (draft) solution is not perfect :)
Btw, the security scan issue can be ignored i think, pls confirm :)
Does this PR introduce a user-facing change?

I do not have a better idea on how to solve this, but I would actually go with {{ "{" }} instead of {{ curly }}.
The reason is that we're already escaping workflows (jinja templates) like that, and I would rather reuse something that's already there. Not reuse from an internal perspective but from a customer/template perspective.

Please add some tests test that {SOME_ENTITY_ID | something} gets replaced correctly.

One more question: Does this change also change other existing configurations? E.g., if I download an environment before this change (with valid configs oc), will I get the same template after this change?

I do not have a better idea on how to solve this, but I would actually go with {{ "{" }} instead of {{ curly }}. The reason is that we're already escaping workflows (jinja templates) like that, and I would rather reuse something that's already there. Not reuse from an internal perspective but from a customer/template perspective.

I agree with @Laubi , IMO we should generally behave the same when templating out these and the workflow curlies.
Adding a 'curly' function seems even harder to describe to users than the reason for "templating" curly brackets - basic value templating is a concept we expect users to know. Functions in templates are not something we so far expect a normal user to be aware off.
re: invalid JSON - we already often produce invalid JSON with templating - e.g. templating integer values will result in a "string" for the template var which is not enclosed by quotes. I think it's fine to produce invalid JSON here as well.

@UnseenWizzard @Laubi If we don't care about the validity of the JSON template, we can certainly do it with the "regular" escaping strategy. And if are already in a situtation where we break JSON i think it's fine...
I will change it accordingly


XOR validation question

[ x] I have looked at the documentation here first?
[ x] I have looked at the examples provided that may showcase my question here?

Package version eg. v9, v10: v10
Issue, Question or Enhancement:
I'm attempting to do XOR validations with structs.  Works fine with strings as shown in this code example:
https://go.dev/play/p/l67xEMBnKfM
Code sample, to showcase or reproduce:
// Essentially this does not work:
type A struct {
	FieldA *string
}

type B struct {
	FieldB *string
}
type StructTest struct {
	StructA *A `validate:"required_without=StructB,excluded_with=StructB"`
	StructB *B `validate:"required_without=StructA,excluded_with=StructA"`
}

// But this does work:
type StringTest struct {
	A *string `validate:"required_without=B,excluded_with=B"`
	B *string `validate:"required_without=A,excluded_with=A"`
}

I'm at a loss on how to get this to work for StructTest, what am I missing?
Thanks in advance!

@khawkinson Thanks for the issue. This seems to be some kind of bug. I will try to write a test case for this and check what can I do.

This might be eligible for closing, I'm unable to reproduce the issue with the latest version of the package.
I believe this was fixed by https://github.com/go-playground/validator/pull/1122.

@zacharyarnaise  yeah it looks like my test is now failing when it should, thanks!


⚠ ESPN has degraded performance

In 080776d, ESPN (https://espn.com) experienced degraded performance:

HTTP code: 200
Response time: 651 ms

Resolved: ESPN performance has improved in bf013ed.


Add run interval support for table ttl settings

https://github.com/ydb-platform/ydb/blob/main/ydb/public/api/protos/ydb_table.proto#L391

Hi,I want to contribute to this issue.Can you just guide me through it?I do not have much experience with it but I am willing to learn and contribute:smile:.

You are welcome!)
For beginning, you can took at repository java-genproto, it contains java classes generated from proto specs. You can just update usage of TtlSettings in classes CreateTableSettings, AlterTableSettings and TableDescription

Hi,I read the proto specs and CreateTableSettings,AlterTableSetting.But I didn't understand how exactly are these classes using TtlSettings because it just returns ttlSetting object without performing any action/modifications.Then I looked up in BaseSession class where objects of  CreateTableSettings, AlterTableSettings class set the expirySeconds and ColumnName for object of YdbTable.DateTypeColumnModeSettings.Builder class generated by proto.Can you help me understand how exactly do I need to   update CreateTableSettings, AlterTableSettings ,what role do these classes play,and where does run_interval come into play.Thanks!!

Classes CreateTableSettings and AlterTableSettings are helpers for proto classes in YdbTable. Because direct usage of YdbTable is very unusual we made helpers for every proto message. Current implementation of TtlSettings and BaseSession supports only  DataTypeColumnModeSettings and doesn't support ValueSinceUnixEpochModeSettings and run_interval_seconds. You can see it in this code which constructs CreateTableRequest message.
For resolving this issue we have to add new field to class TtlSettings and also add correct constructing of CreateTableRequest and AlterTableRequest in BaseSession's methods createTable and alterTable


ACM-13002-Policy-overview-with-lots-of-clusters

clusters with violations are sorted to the top, then cutoff with "Show more" to expand the list

also the backend can now emulate lots of clusters by changing this value:

Signed-off-by: John Swanke<EMAIL_ADDRESS>

/test unit-tests-sonarcloud

/test images

/test images

/cc @mprahl

/cc @showeimer

@jeswanke the merge commits seem to have messed up the diff. Are you able rebase instead?


Fuller usage context in readme.

This is my suggestion for more "general users context".

Thanks @lbdreyer @corinnebosley .
Hoping I've now fixed those things.

Looks good!
I'm going to merge this in...


Validate URL, report correct error

I had the following set in my .env:
WEBHOOK_PROXY_URL=https:/smee.io/CHANNEL

It took me a while to find the problem, but there's only one / in https://. This resulted in a stream of errors from Probot, spamming my terminal as it tried to reconnect:
03:13:42.044Z ERROR probot:  (type=error)

This should be caught earlier by the smee-client.
Originally opened as https://github.com/probot/probot/issues/439

@JasonEtco I am working on this issue but on running locally when I am trying to make changes in serve/components/Blank.js it is not reflecting in my localhost. I was verifying URL but not able to proceed due to this error. Please help?

@knrt10 this logic should be implemented in the client package, not the server. Your work should be contained within the client directory of this repository, so you shouldn't need to change with the front-end React components.

@JasonEtco thanks for info will do according that.

@JasonEtco Is there any way that while running smee on localhost I can receive webhooks?

@knrt10 you can use localtunnel.me, but because this error should appear when the Smee starts (not when it receives webhooks), this shouldn't be necessary.

Ok. Thanks for info. Let me try and implement that.

@JasonEtco Please check my PR. Thanks in advance.

@JasonEtco please close this.


Added use.md file and linked from install.md per issue-14

@Rene-Ch1 could you please review for spelling, grammar, tone/voice, and whatever else you feel appropriate.
Signed-off-by: Joe Pearson<EMAIL_ADDRESS>This will close issue #14

Some best practices comments....directional content (before, previous, after, above, and so on) are discouraged because vision impaired readers have challenges with directional content. The same goes for any visual cues like colors or descriptions of icons, shapes, arrows, and so on. Same goes for Latin-based content like i.e and etc. Slashes are also discouraged as in and/or. These things create computations for visually-impaired and translation issues. same goes for jargon or Americanisms. Just saying...:-)

Thanks!  I'll open a separate ticket to fix that.

PR merged, closing


[Admin] Publish

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

File
Status
Preview URL
Details

docs/develop/dialog-api-in-office-add-ins.md
:white_check_mark:Succeeded
View

docs/excel/excel-add-ins-pivottables.md
:white_check_mark:Succeeded
View

docs/outlook/add-mobile-support.md
:white_check_mark:Succeeded
View

docs/reference/requirement-sets/excel-preview-apis.md
:white_check_mark:Succeeded
View

docs/reference/requirement-sets/powerpoint-api-1-2-requirement-set.md
:white_check_mark:Succeeded
View

docs/reference/requirement-sets/powerpoint-preview-apis.md
:white_check_mark:Succeeded
View

docs/testing/clear-cache.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

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

File
Status
Preview URL
Details

docs/develop/dialog-api-in-office-add-ins.md
:white_check_mark:Succeeded
View

docs/excel/excel-add-ins-pivottables.md
:white_check_mark:Succeeded
View

docs/outlook/add-mobile-support.md
:white_check_mark:Succeeded
View

docs/reference/requirement-sets/excel-preview-apis.md
:white_check_mark:Succeeded
View

docs/reference/requirement-sets/powerpoint-api-1-2-requirement-set.md
:white_check_mark:Succeeded
View

docs/reference/requirement-sets/powerpoint-preview-apis.md
:white_check_mark:Succeeded
View

docs/testing/clear-cache.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Error code assert

Assert should take an error code, this can be defined as a table in the abi for printing human readable messages.
Two versions of assert will exist:

assert(uint64_t)
asserts(const char*, size_t)

The asserts intrinsic is the same as the old assert, but will now also take a length.

There will actually be three versions of the assert intrinsics:
void eosio_assert( bool condition, null_terminated_ptr msg );
void eosio_assert_message( bool condition, array_ptr<const char> msg, size_t msg_len );
void eosio_assert_code( bool condition, uint64_t error_code );

The eosio_assert function name will remain for backwards compatibility. It could in theory be implemented on the WASM side using the eosio_assert_message intrinsic (and strlen). But just like the prints and prints_l variations, both versions are provided because it is faster to do strlen on the native side than on the WASM side (particularly for binaryen).
The platform will accept any uint64_t number as the error code in the eosio_assert_code intrinsic, whether that error code is associated with a message string in the ABI or not. We do not want the ABI to become part of consensus.
User clients (like cleos) can then look up the error code in the ABI. If they find it, they can also print the human-friendly message in the error log. Otherwise, they will only print the error code.


docs: Fix yarn tab storage key name

Update the tabs to use matching keys to keep in sync.

Original message!

Super minor thing, but those two tabs had different key for yarn and they didn't sync with others (syncing is nice subtle feature, thanks) — @O4epegb


Explicit behavior for key

Would be great if the key has an interface with a Byte() func instead of interface{} type
What do you think? Implicit is better than explicit :), btw this would introduce a huge BC...
cfr: https://github.com/gin-gonic/contrib/pull/19

See #50


Version is not populated in success box in PoW section.

The currently reads:
Success! During the last 100 Blocks we have reached the minimum of 75% Blocks which are on the latest Block Version vX
That "vX" should contain the correct version.
Also, it should be looking for v4, not v3, which means it should not be active yet.

Fixed in incoming PR


Remove Babel plugin add-module-exports to support TypeScript

See issue #193.
The plugin should works in JavaScript and TypeScript with this change.

Anything further I have to do for merging? :)

Anything further I have to do for merging, @CookPete? :)

84d6c35ef444b1aef826f06a0728f288e151a35a should take care of this. Thanks for the PR but I didn't want to remove the plugin completely, just move it to the bower build. Committing dist files is also unnecessary.

Works, thanks man!


WifiControl: SSID is not properly decoded

If SSID contains non-ascii characters, they are converted to utf-8 and escaped to hexadecimal escape sequences.
In old RDK wifi-hal, this is unescaped by printf_decode provided by wpa_supplicant.

A lot of changes have been made around encoding and escaping of JSON information. Expectation is that this resolved the issue at hand here. If not let us know the use case (a name of the SSID) that we can double check and fix it.


Expose public key in RegistrationResponse

I hope this is correct. This would help transmitting the public key during registration, removing the need to send it separately.

Replaced by #193.


Feature request: option to convert it back to the SVG path string

As described. That'd be very handy!

Sorry @benkeen, I don't have enough time to spare to prioritise this :') My suggestion would be to write an independent package that does this, and shares the same format as this one. Cheers!

@benkeen Here's a function to do what you want. It's not a good fit for this library (which is currently JUST a compiled PEG grammar).
function pathData(data){
  var params = ['rx','ry','xAxisRotation','largeArc','sweep','x1','y1','x2','y2','x','y'];
  var lastCode;
  return data.map(function(cmd){
    var a = [];
    params.forEach(function(param){
      if (param in cmd) {
        var val = cmd[param]*1;
        if (a.length && val>=0) a.push(',');
        a.push(val);
      }
    });
    var result = (lastCode===cmd.code?(a[0]<0?'':','):cmd.code) + a.join('');
    lastCode=cmd.code;
    return result;
  }).join('');
}

In Action: https://jsfiddle.net/js7zo38o/
The function does a bit of work to try to make the path as short as possible:
Original Path String:
M3,7 5-6 L1,7 1e2-.4 m-10,10 l10,0
V27 89 H23           v10 h10
C33,43 38,47 43,47   c0,5 5,10 10,10
S63,67 63,67         s-10,10 10,10
Q50,50 73,57         q20,-5 0,-10
T70,40               t0,-15
A5,5 45 1,0 40,20    a5,5 20 0,1 -10-10  Z

Parsed and Recreated:
M3,7L5-6,1,7-100-0.4m-10,10l10,0V27,89H23v10h10C33,43,38,47,43,47
c0,5,5,10,10,10S63,67,63,67s-10,10,10,10Q50,50,73,57q20-5,0-10T70,40
t0-15A5,5,45,1,0,40,20a5,5,20,0,1-10-10Z

Optimizations that it does not do:

It makes no attempt to constrain decimal precision (usually your biggest problem)
A moveto followed by an absolute lineto will still include the L (which is not needed).
No attempt is made to convert an absolute or relative command into a shorter equivalent.
No attempt is made to convert an explicit Bézier command (C/c/Q/q) to a shorthand equivalent, when possible.

Wow, thanks @Phrogz - and @hughsk for responding.
Terrific stuff, I'll give it a go.


Deploy error: No space left on device

I've tried running this action a few times and it always fails after ~8 minutes with the same error:

I'm fairly certain I've filled out the workflow file correctly. I wasn't able to get the cookie the same way as provided in the documentation, so I assumed that was the problem and ignored that step. Even without the cookie it still doesn't work.

Nevermind I'm new to Git so I didn't know where the problem was originating from. Running a separate action to free up disk space solved it.

I think this may be a problem on GitHub's end as it seems like you've used up all the space the GitHub Action can handle... thus the lack of space error occurs.


fix(docs): style headings more specifically

Headings for the docs have been styled in a way that affected headings inside components. To prevent this, a more specific selector is being used now.

Preview environments are getting posted here, as soon as they are ready!


Configure: also break up the OPTIONS line

c00d9311 ("Configure: break long lines in build files", 2019-05-23)
introduced wrapping of long Makefile lines, but it forgot to wrap the
configuration options, which are generally also longer than one screen
line.
Signed-off-by: Beat Bolli<EMAIL_ADDRESS>

This one worries me, 'cause if there's anything quoted amongst the options, that string will be corrupted.
The options may cover more than one terminal line, but not too many. It's not like the huge list of files you get for some targets...

All right, that's a usage I didn't think of. I just looked at the
generated Makefile and the OPTIONS line really jarred.
Thanks anyway for the quick review!


Vault - List vesting vaults

optimize useGetVaults hook
list vesting vaults on Vault page
house keeping on Vault type and mock data

We should be able to achieve better responsiveness

About the responsiveness issue: yes definitely. I've sent a message to Pavel about this last week but he's currently busy with the Isolated Lending feature. I'll ask him again

I spoke with Natasha who proposed to use a wider endpoint to make the switch between 2 to 1 column. This fixes the issue and I've pushed the change.


Request-for-Comments (RFC) Discussions

I have left a number of Request-for-Comments (RFC) questions on various APIs and other places
in the code as there are some cases where I'm not sure what the desirable behavior is.
This issue is a place to collect responses and discussions for these questions.
Please quote the RFC-statement (and/or link to its source line) and provide your feedback here.

I'm looking at multipart libraries for implementing some middleware that will intercept, process requests, and write them back out before forwarding the potentially modified raw body onto the application. The multipart parser and serializer implementation here looks pretty good but the API is incredibly painful to work with because it's setup to expect full request structs that provide a bunch of methods. Would you be open to exposing some lower level functions to handle just the parsing/serialization and operating on a stream/vector/list of parts?

Is Multipart::with_body() not sufficient? That just takes the boundary and a Read impl.

It looks like Multipart::with_body will handling parsing out the individual parts but from the client side it doesn't look like I can easily just pass the parts back into the client side and re-serialize them. I also necessarily am buffering the full body and looking to avoid having to deal with saving out the files to disk during read and then reading them back out during serialization. I've already paid the memory cost and don't really have the ability to avoid that.

The default client side API lets you generate request bodies in a streaming fashion, though I don't have it set up for working with a raw Write impl I could make that change pretty easily, if that's all you need from the client side.

Some API, on the client side, where I'm able to pass in a Vec or Iterator with a boundary and it serializes it out to either a [u8], Write, or some other type of buffer would be awesome.
Ideally being able to get an Iterator from Multipart in server would be awesome but I understand that's an issue with the streaming implementation. In my case, since I already have the full buffer copied, it would be cool if there was just a zero copy parser exposed that returns an iterator of them that I can use as Cow so I can only modify parts that I need to modify and just map over the Iterator to get the new set of parts I need to write out.

On a side-note, have you looked at the mock module at all?

I just took a peek and it could help some with doing the deserialization of the body, but it looks like it doesn't offer me anything that I don't already get with Multipart::with_body and the Client mocking doesn't take in the parsed parts. The real piece that would make things easier at this point is being able to operate on all parts and serialize them back out to a buffer of some sort easier.
Also, I'm not sure if this it the correct place to be discussing this and if you'd like I'm happy to move it elsewhere. It just seemed somewhat fitting since it was looking for feedback.

Yeah a new issue would be preferable. You would use mock::ClientRequest with client::Client, I dunno if that was very clear. That would allow you to serialize the body out to a Vec<u8> and get the boundary as well.


safe: Apache APISIX security configuration

check the current nginx.conf if it has any security problem.

fixed by https://github.com/apache/incubator-apisix/pull/1759


JFXSnackbar non default constuctor causes  NPE

Hi,
I've got an NPE while trying to use the latest jfoenix.jar (which you provided 2 hours ago). The exception occurs when I try to create a jfxsnackbar using this command:
new JFXSnackbar(somePane);
If I use the default constructor and then call .registerSnackbarContainer it works fine too.
Note that this works fine with jfoenix version 'jfoenix-0.0.0-20160321.094207',
It seems the listener is null:
Caused by: java.lang.NullPointerException 	at com.sun.javafx.binding.ExpressionHelper.addListener(ExpressionHelper.java:67) 	at javafx.beans.property.ReadOnlyDoubleWrapper$ReadOnlyPropertyImpl.addListener(ReadOnlyDoubleWrapper.java:168) 	at com.jfoenix.controls.JFXSnackbar.registerSnackbarContainer(JFXSnackbar.java:163) 	at com.jfoenix.controls.JFXSnackbar.<init>(JFXSnackbar.java:126)

it's been fixed, thanks for pointing out the issue.

So, did you update the latest jfoenix.jar on jfonix.com?


Test case for $.once

Looking into the source code I've found an undocumented method once and I've tested it: in this example I want to move one div twice, but I want to execute the transitionend handler just once.
Expecting result: the number inside the foobardiv should be increased one time in all browsers
Actual result: the number inside the div increases twice on Chrome 47.0.2526.80 (64-bit) and Safari 9.0.2 (95<IP_ADDRESS>.1) and increases properly once on Firefox 42.0 on a  MacOSX 10.9.5 (see screenshot below about the comparison between Chrome and Firefox)
Hint: it seems the problem is related to 'webkitTransitionEnd transitionend' event type string: if I specify only 'transitionend' the demo works as expected in all the tested browsers.

This is my test case

0

It seems like Chrome understands both webkitTransitionEnd and transitionend events and executes once for each event?

@zdfs In this case I suppose that on chrome and safari the counter would be immediately 2 after the first animation (since they would be both executed once)

Oh, $.once() is undocumented indeed! That was not on purpose, it was just forgotten. We/I need to document it :)
Nice test too. Easy to fix as well, I think (it just needs a loop where it removes the event). If nobody else fixes it by tomorrow, I'll push a fix (I have a deadline tomorrow, so can't do much right now).
Thanks for both catches @fcalderan!

Does it pass now?

The test case works fine both in Chrome and Safari.

Awesome, closing this then! Thanks @fcalderan!


Extensions: Optimize build to remove most exports via SIDE_MODULE=2

Improvement depends a lot on the extensions, can be 0-30%. For example, icu.duckdb_extension.wasm goes from 8.0MB to 6.3MB (uncompressed size)

This is the linked PR in spatial: https://github.com/duckdb/duckdb_spatial/pull/234

Also adding here, to be addressed in a future PR or at the Emscripten or even standardization level:
duckdb-wasm extensions do not expose their symbol at the JavaScript level, so the exports do not need to have the actual mangled names as export names, but they could have unnamed export names.


Problem with metaV2 on MDX

What version of Remix are you using?
1.16.1
Are all your remix dependencies & dev-dependencies using the same version?

[X] Yes

Steps to Reproduce
On any MDX
---
meta:
  - title: My Blog
  - og:type: website
  - og:title:  My Blog
  - og:url: https://My.Blog/
  - description: Blablabla
  - og:description: blablabla
  - keywords: bla,bla,bla
  - robots: index,follow
  - googlebot: index,follow
---

# Content

Expected Behavior
Output of <Meta /> on root file
<title>My Blog</title>
<meta name="og:type" content="website">
<meta name="og:title" content="My Blog">
<meta name="og:url" content="https://My.Blog/">
<meta name="description" content="Blablabla">
<meta name="og:description" content="blablabla">
<meta name="keywords" content="bla,bla,bla">
<meta name="robots" content="index,follow">
<meta name="googlebot" content="index,follow">

Actual Behavior
<title>My Blog</title>
<meta og:type="website">
<meta og:title="My Blog">
<meta og:url="https://My.Blog/">
<meta description="Blablabla">
<meta og:description="blablabla">
<meta keywords="bla,bla,bla">
<meta robots="index,follow">
<meta googlebot="index,follow">

I've had to change the default return after all the if's statements of the <Meta /> component based on the @remix-run/react to render it correctly.
Here's what I've done.
return (
            <meta
              key={JSON.stringify(metaProps)}
              name={`${Object.entries(metaProps)[0][0]}`}
              content={`${Object.entries(metaProps)[0][1]}`}
            />
          );

if anyone wants to use it, just download from here
I've grabbed the dist files from the @remix-run/react and deleted almost everything not related to the Meta component and changed the return of it.

The YML is the example doesn't seem to follow the V2 convention. The YML should be equivalent to the JSON required for the API.
meta:
  - title: My Blog # Should still work as is
  - name: "og:type" # You need to specify the attribute name and the value for every meta tag
    content: website

This way is counter intuitive, check docs.

I agree it's counter intuitive but the intuitive approach blocks valid use cases, which was the whole motivation behind meta v2 🫣 We tend to prefer power > convenience


Imported types are not resolving

TypeScript Version:  4.0.2

Search Terms:
type inference
reference types
Code
// A *self-contained* demonstration of the problem follows...
// Test this by running `tsc` on the command-line, rather than through another build tool such as Gulp, Webpack, etc.

/// <reference types="node-fetch" />
import { request, IncomingHttpHeaders, RequestOptions } from "http";
import { URL } from "url";

interface NodeOptions {
  url: URL;
  options: RequestOptions;
}

function mapFetchParameters(
  urlInfo: RequestInfo,
  init?: RequestInit
): NodeOptions {
  const reqUrl =
    typeof urlInfo == "string" ? new URL(urlInfo) : new URL(urlInfo.url);
  let headers = {};
  let method: string = "";
  let port = "80";
  if (urlInfo instanceof Request) {
    Object.assign(headers, urlInfo.headers);
    method = urlInfo.method;
    port = urlInfo.port.toString();  // ERROR??
  }
  if (init) {
    method = init.method ? init.method : "";
  }
  const options = {
    host: "localhost",
    port: port,
    path: "/posts",
    method: method,
    headers: headers,
  };
  const res = {
    url: reqUrl,
    options,
  };
  return res;
}

Expected behavior:
Compiler should find the port class property on the Request class.
The @types/node-fetch index.d.ts exports
export type RequestInfo = string | URLLike | Request;
export class Request extends Body {
method: string;
...
port?: number;
}
Actual behavior:
error TS2339: Property 'port' does not exist on type 'Request'.
71     port = urlInfo.port.toString();
Playground Link:
https://www.typescriptlang.org/play?#code/PQgEB4CcFMDNpgOwMbVAFwJ4AdoGcBeAIkQHsATaAWnnWQAsjRgA+AKAEsBbbUydUAG9QAJWgBHAK750AeWzoOpRHlABfULEikuoIvXTpsRANycefAcICqIgDLrN23UUmQANqbadE6BLABDVFAAOQpoeUVlVUE2UFA3dwAuUFs7M3jSBSUVFLEpGUicvDM1b1hJFCjEUC4A7AAxaDp6AAUAyACuZoQ8AAo4hI8ASURYUjyJaTx0UfGAGkGORA50AH5JgpnR1bYAShSwyiLooUHkaIEYcWsPUAJB+KxcUlgh9znSe4I9GchlgDmTDWoEQ0AA7ql7H1Ep89qAUmDIWkYSMxqQAHSJPYZUDuZqgejQAKUSCqH6CNS4-ECbroegUFJ-QH3PReeI00C8fisogADgADOzQBw3qiPuiRSp0AEUNBXqIpjJ4bF4vFZAAjABW0GQ6AxATweA4AMQfSJJN683enwxFtJeBxj1qzQZ5FZsPRGLpbtx8W5Ah+nvGGIDGPQpAAyuh-ogAX0nfEyvFRaA+stVirnT6KKyM-qc+6Qfnva7cykiMLk6ALtLQFlquSzmrCaQZhX3KRkAF3AyZkRFi2AykA4O1dgAvSK8BeDM8APs2XyClC2P4vbeikN2Sx1TzpdQDAm6q1YkUtdbu41-XstFd7iYOg3DUj6U2EA
Related Issues:
GitHub Link
https://github.com/paboulos/http-types

From the documentation:

Use these directives only when you’re authoring a d.ts file by hand.

It doesn't seem to me that you're writing a d.ts file. You should just use an import statement in your regular code.

The type is imported through package.json and tsconfig. How do I import types without installing node-fetch module? I want to reuse only the types and publish to npm, without node-fectch implementation module as a dependeny.
This documentation here https://github.com/definitelytyped/definitelytyped/#definitelytyped says:
"The types should then be automatically included by the compiler. You may need to add a types reference if you're not using modules:
/// "

I  updated the type reference directive in a local index.d.ts and changed typeRoots in tsconfig. Getting additional errors from compiler:
lib/index.ts:5:8 - error TS2304: Cannot find name 'HttpRequest'.
5   api: HttpRequest,
~~~~~~~~~~~
lib/index.ts:7:12 - error TS2304: Cannot find name 'HttpResponse'.
7 ): Promise<HttpResponse> {
~~~~~~~~~~~~
lib/index.ts:8:15 - error TS2304: Cannot find name 'HttpResponse'.
8   const resp: HttpResponse = await api(request);
~~~~~~~~~~~~
lib/index.ts:61:20 - error TS2339: Property 'port' does not exist on type 'Request'.
61     port = urlInfo.port.toString();
~~~~

This looks like a misconfiguration of some sort, not a bug. We can investigate if there's something clear pointing to a defect, but I don't see evidence of that here.

You explanation should include a reference to spec or doc saying so. Technical answers shouldn't be based on intuitive feelings.

That's fine either way, because this is a bug tracker, not a forum for answers.

If something doesn't work it's a bug until you can prove it otherwise. I haven't seen anything like that yet.

I looked at the repo and I can't tell why you think this should work

HttpRequest is declared in a module file (index.d.ts), not a global, so it's not going to be visible in index.ts without an import. This behavior is documented
Same for HttpResponse
A file in @types/index.d.ts doesn't belong to any package because it's not in a folder. This isn't going to get picked up automatically; it doesn't match any existing pattern so it seems to have been put there on accident?
You've tested for instanceof Request where Request is the built-in DOM Request object, not the node-fetch Request object. This is because you are using the default lib option which includes dom, and because you didn't import Request from the node-fetch library

The problem is with node-fetch type Request.port? Are you saying the only way TypeScript lets you import types without using the module is to make it a global namespace?


Update logical handling of ChEBI biological roles.

Changed several logical definitions to reference biological roles using the has role property (fixes #2698). Here is a readable diff:
https://stars-app.renci.org/owldiff/diff?left=https%3A%2F%2Fgithub.com%2Fobophenotype%2Fmammalian-phenotype-ontology%2Fraw%2F1f0ba32e38116bd94c2ffad31a2fb6d3b9548bb8%2Fsrc%2Fontology%2Fmp-edit.owl&right=https%3A%2F%2Fgithub.com%2Fobophenotype%2Fmammalian-phenotype-ontology%2Fraw%2Fissue-2698%2Fsrc%2Fontology%2Fmp-edit.owl&loadimports=true

Ignore the Travis failures—there doesn't seem to be any actual Travis job configured for this repo (so I disabled the integration for the time being).


Support TypeScript generics

Fixes https://github.com/egoist/vite-plugin-compile-time/issues/22

Bump?

Thanks for updating this!


Migrate from "g2" to "sz"

See https://github.com/senzing-garage/sz-sdk-go/issues/177

Pull requests:

https://github.com/senzing-garage/validate/pull/95


API usage

Thanks for creating a great stuff.
After following all steps, I could run <IP_ADDRESS>:8000 and API explorer screen.
I also created the passport client called 'admin'.
What should I do next?
I get Unauthorized bugs when I run http://<IP_ADDRESS>:8000/api/users

Thank, you should use passport's endpoints to create a new access token.
https://laravel.com/docs/5.7/passport#issuing-access-tokens
The API is secured with Laravel passport so you should be able to generate tokens and use them according to Passport's documentation
https://laravel.com/docs/5.7/passport


bower not working yet

So I'm just getting used to creating bower packages and so far it seems like there isn't a way to add built files yet without checking them into git. Just seeing if there are any nice alternatives before I commit this solution.

Looks like its all working :-)


Upgrade resin-sdk

This should means that the CLI depends only on a single version of the SDK (and thus of resin-token), which will solve https://github.com/resin-io/resin-sdk/issues/286.

Is it a dub-dep of the CLI?

@emirotin ...what's a dub-dep?
It is a dep here that's been conflicting with the dep in the CLI, which is why we get that peer dependencies error.

Meant to be sub-dep(endency) :)
Got it.


Application hangs if rsync cannot be found

I have the rsync binary installed at:
/usr/local/bin/
and tried to direct RsyncUI to use that path but upon running or estimating in RsyncUI, I get the message:
"The file "rsync" doesn't exist.
The file doesn't exist."
Then RsyncUI hangs for ever and need to be forcefully terminated.
RsyncUI crashlog.txt
rsync  version 3.3.0pre1-7-g2f9b963a  protocol version 31
RsyncUI 2.0.0

I will check it out :-) long time ago since I used other paths than default for rsync. I will check the code now and make an update to solve the issue...

I have discovered why it hangs... I will make an update later this evening, thx for reporting the issue..

Great to hear.
Thanks for such swift correction.

I will make an update for you to test in some hours. I am just about to leave my cabin in the mountains.. It is some hours driving home and I have to verify the changes as well before making a new version for you to test...

Just updated the version 2.0.0 rc. There is a temporary fix for the issue. To reset any wrong values to default values, toggle version of rsync. RsyncUI now starts with no valid rsync in path, and it also shows no valid rsync where rsync version is presented. Thx very much for reporting the issue.

Hello
Do the 2.0.0 b101 build contain the fix?
I'm still getting hangs when trying to open the preferences to change the paths.
The preferences pane opens behind the main screen but the notice about "The file "rsync" doesn't exist." is placed ontop and then the applications hangs.

Hello Do the 2.0.0 b101 build contain the fix?
I'm still getting hangs when trying to open the preferences to change the paths. The preferences pane opens behind the main screen but the notice about "The file "rsync" doesn't exist." is placed ontop and then the applications hangs.

That is kind of weird, it should not hang. The reason for hang (previous rc) was because RsyncUI executes a rsync --version at startup. I suggest you just delete the settings file, see here where it is: https://rsyncui.netlify.app/post/configfiles/
and restart RsyncUI. RsyncUI then starts with default configs. I have tested the new build for rc 2.0.0 and if I enter a not valid path, it notifies me and starts.

And by toggling the Rsync ver3 in settings, just resets any user settings..


GQL returns posted transactions without mining the block - with block set to null

I've noticed that simply posting (arweave.transactions.post) a transaction to arlocal (without mining the block) makes it available through a GQL search,
e.g. I'm posting a SmartWeave interaction for a contract, load the contract interactions (without calling the arweave.api.get('mine')) - the interaction is already there, but with block set to null, e.g.
[
  {
    "node":{
      "id":"VVsN7duY7w9q5mMyRytYM7dZSb2_T4ADkAlnH5g9Jh8",
      "owner":{
        "address":"owSxQ1wSTAY8ZEo4oX5iVdDKjIz7icgBHJYVrT4TwFg"
      },
      "recipient":"",
      "tags":[
        {
          "name":"App-Name",
          "value":"SmartWeaveAction"
        },
        {
          "name":"App-Version",
          "value":"0.3.0"
        },
        {
          "name":"SDK",
          "value":"RedStone"
        },
        {
          "name":"Contract",
          "value":"enYRcGrdFIdU-KXp0MQ-KKgca0aP6j1tIn8qJW_4NXc"
        },
        {
          "name":"Input",
          "value":"{\"function\":\"add\"}"
        }
      ],
      "block":null,
      "fee":{
        "winston":"7860528"
      },
      "quantity":{
        "winston":"0"
      },
      "parent":null,
      "bundledIn":null
    },
    "cursor":"WyIyMDIyLTA2LTA1VDE5OjM0OjI0LjYxMVoiLDFd"
  }
]

Tested on ArLocal 1.1.30 and 1.1.38
Query variables:
{
  "tags":[
    {
      "name":"App-Name",
      "values":[
        "SmartWeaveAction"
      ]
    },
    {
      "name":"Contract",
      "values":[
        "WUKD-tJkaPVsFtuQKZjUR3WCE1_90pthjPqKJZwhvxE"
      ]
    }
  ],
  "blockFilter":{
    "min":null,
    "max":null
  },
  "first":100
}

Query:
query Transactions($tags: [TagFilter!]!, $blockFilter: BlockFilter!, $first: Int!, $after: String) {
    transactions(tags: $tags, block: $blockFilter, first: $first, sort: HEIGHT_ASC, after: $after) {
      pageInfo {
        hasNextPage
      }
      edges {
        node {
          id
          owner { address }
          recipient
          tags {
            name
            value
          }
          block {
            height
            id
            timestamp
          }
          fee { winston }
          quantity { winston }
          parent { id }
          bundledIn { id }
        }
        cursor
      }
    }
  }

This happens when blockFilter.max is set to null. It works fine (i.e. does not return the non-mined transaction) if the "max" is set to current network height.
I believe that original arweave.net GQL works fine also when "max" is set to null.

Hey there @ppedziwiatr , I don't think this is a bug. On arweave.net we can receive non-mined txs too and they should return null on the block height too. Please correct me if I'm wrong and we will look into it further, and update this.

Hey there @ppedziwiatr , I don't think this is a bug. On arweave.net we can receive non-mined txs too and they should return null on the block height too. Please correct me if I'm wrong and we will look into it further, and update this.

Oh, I didn't know that :-) I was testing it before creating this issue on arweave.net and it didn't seem to return transactions without block...I'll ask the ar.io team what is the behaviour :-)


Post-processing effect doc

For #2962

Background

Change List

new doc

Coverage remained the same at 62.945% when pulling 2311aadd8bf0e87be4715f1e674e77fa0675a84a on post-process-doc into c773601276fcc1f7bcc1617f58bc2e9a3b246062 on master.


start page issue

I found there was a format issue in start page when I add tiles with js.
start - Copy.zip

Sorry, I'm not familiar with codepen, can you please help to create? I already upload the code.

I don't have time to download and deploy user code, for all users an equal rules: https://github.com/olton/Metro-UI-CSS/blob/master/CONTRIBUTING.md#issues-guidelines

https://codepen.io/FelixHo/pen/xyQVwr
the tiles are smaller than not involve start.css and start.js

remove style:
[class*=tile-] {
  transform: scale(0.8);
}


Get products

I have some trouble in getting products from akami api using nodejs AkamaiOPEN-edgegrid-node.
Here is my code:
eg.auth({
path: '/billing-usage/v1/products',
method: 'POST',
headers: {'reportSources': '[{'id':'123','type':'reportGroup'},{'id':'123','type':'contract'},{'id':'123','type':'reportGroup'}]',
'startDate': '{'month':1, 'year': 2015}',
'endDate': '{'month':11, 'year': 2015}'}
});
eg.send(function(error, response, body) {
    console.log(error);
    //console.log(response);
    console.log(body);
});

like in example.
Also i have tried change headers key name to body, still getting result 400 Bad request.

What is the response you are receiving? Also, could you post the full code?

This works for me:
var param = "reportSources=" + encodeURIComponent('[{"id":"' + contractId + '","type":"contract"}]') +
    "&startDate=" + encodeURIComponent('{"month":4, "year": 2016}') +
    "&endDate=" + encodeURIComponent('{"month":4, "year": 2016}');

eg.auth({
    path: '/billing-usage/v1/products',
    method: 'POST',
    headers: { 'Content-Type': 'application/x-www-form-urlencoded' },
    body: param
});

eg.send(console.log);

@fholzer Thanks for tagging in on this! Appreciate the help =]
@farkhad-alizade If you're still having issues I'd recommend posting to the Akamai OPEN Community site (there is much better coverage on there for helping to answer these types of questions). Also, there is a great knowledge base there and it may already contain the info you're looking for.
https://community.akamai.com/community/developer


Benchmark FileChannel vs BufferedOutputStream for FileDataWriter

Inspiration : http://java.dzone.com/articles/java-sequential-io-performance

No significant gain observed for the volume we write on disk. --> won't change the DataFileWriter
Reading from a Channel or a RandomAccessFile should be faster than a BufferedIS but real life tests are not significant --> won't change DataLoader.
Removing from 1.1.0 and closing.


factor::rho: Refactor for readability and fewer allocations

Follow-up on #1549

Rebased so Github only displays the changes relative to master

I’m really happy with all the work you’re doing on this BTW

Thanks @Arcterus <3


Fix duplicate variable ids

@pavel-kirienko Sorry for breaking master! Should have checked that before requesting to to pull my PR.

Thanks for fixing this. It's a serious problem that we don't check this automatically. I should implement this check soon.

I will try and add a travis check for this. So that this can be avoided.

Awesome.

I think simply running dsdl compiler on it should do. Do you agree.

Yes, it should be enough. We have a ticket for this: https://github.com/UAVCAN/dsdl/issues/18


Feature reuqest: Use configuration folder

It would be beneficial to be able to specify a folder in the config.yaml file that contains the configuration. The expected behavior would be that Nebula imports and combines all files present in the configuration folder to compose the configuration.
The benefit of this feature is to allow simpler automation of the configuration

Lucky for you, this has been supported since Nebula v1.0.0! Just point nebula -config at a folder and it will load every .yaml and .yml file in there in alphabetical order. Lists are combined, so you can spread firewall rules across multiple files.

You can also see this documented with nebula -help

very interesting, thanks a lot :), I was not aware of it!


Fix mobile views

The mobile version of the site is not great. It'd be nice if things scaled properly. I'd suggest moving certain things to use flex boxes and just editing some CSS and markup where necessary.

Fixed the top bar mobile view here: https://github.com/jrhackett/dnd-5e-spells/commit/57cb3abcbe888800fb1ecc72a4612c5d7e2f4b54


show "Access is denied" error when unable to inspect elements

when running a process like notepad as admin, capturing will not find any UIA Element Properties of notepad.
the same applies to inspect.exe so it's not a bug, but inspect shows an error message:
How found:	Mouse move (1402,689)
hwnd=0x000000000010076A ??bit class="Edit" style=0x50200104 ex=0x0
Cannot get object from point:
[Error: FromPoint: hr=0xFFFFFFFF80070005 - Access is denied.]

It will be fixed in the next version.


Publish the new version to crates.io

@ryjones @mikelodder7 Could you please publish the new version to crates.io? It would help us a lot.

should I wait for #190 to get merged?

Yes please

@ryjones Could you please ask someone to approve it?

@askolesov I asked in chat

@askolesov is #186 also needed?

We've already bumped k256 here. p256 doesn't introduce conflicts for us so I suggest publishing the new version now.

@ryjones  Thanks for creating a GitHub release! Can this be also pushed in crates.io?

@askolesov I have tried to set up an action to publish, but it doesn't work like I expect. Could you help me over here? https://github.com/ryjones/ursa/commit/e90c24b9f1769defb752dbaefdf76449ab66d949 once the action will do a --dry-run successfully then I will move it over here

Sure I tried to build a package with @Artemkaaas today but it fails with strange errors. We think they were in the previous release as well. @mikelodder7 could you please help us?

@askolesov what are the errors? Could you paste them here?

@brentzundel check out over here: https://github.com/ryjones/ursa/commit/2eaa468f92c8505f8472ed5f36dfbc120302e425#diff-9efc738000935f7d55626940aa06fd013cffe8779f5169b16b50e98331612aa0R3

@askolesov I think Mike Lodder published it two days ago - confirm?

Yes, thanks a lot!


Fairground explorer is configured to point to a node that is unhappy

In a private window / separate browser, visit: https://explorer.fairground.wtf/

The explorer will fail to connect to the node it is configured to point to, and open up the 'Connect to a node' dialog
Tasks

[ ] Update teh configuration for fairground explorer to point to a node that is currently available
[ ] Ensure that stagnet3 nodes do not appear as an optional alternative node for a Fairground explorer

This is the current netlify configuration:

Here is the configuration file for the environment:
https://github.com/vegaprotocol/frontend-monorepo/blob/7fb3a23a5afdbaaf641e4b5174b025e42bca9970/apps/explorer/.env.stagnet3#L1-L9


Edit Table in Pin Mode

You cannot select any cell of a table who is being pinned. You can edit them, but you can't select and use the arrow keys in the table.

Huh, played around a little bit more, somehow I got multiple text cursors, by clicking in the pinned table and then using arrow keys to move along text in the expression list on a folder

I can no longer reproduce this. Assuming it's been fixed.


Document Prometheus labels needed

The adapter makes some assumptions about the Prometheus labels attached to metrics. We had problems originally when it couldn't find a Pod metric because it was missing the pod label.
To scrape Pod metrics correctly we ended up with something like this in our Prometheus config:
scrape_configs:
  - job_name: 'kubernetes-pods'
    kubernetes_sd_configs:
    - role: pod
      relabel_configs:
        - source_labels: [__meta_kubernetes_pod_name]
          action: replace
          target_label: pod
        - source_labels: [__meta_kubernetes_namespace]
          action: replace
          target_label: namespace

I've not looked into collecting metrics for other types of object (Ingress, Service etc.) but I'm guessing they probably also depend on certain labels. It'd be great to have a reference that says which labels must be set in Prometheus for it to succeed.
Thanks!

There's information in the README now about this.  I'm going to close it.  Feel free to re-open if you think that's insufficient.


How to use data from other frames?

I have 2 frames called ABC1 and ABC2, how to select or use data from ABC2? meanwhile i'm also using ABC1 too

@anauwal If you have multiple frames returned, the panel allows choosing which one to display, similar to the table.
If you need to work with both of them, you can merge using Transformations:

Let me know if it helps.


Word: Table "units statement" outside of table and in funny paragraph alignment, header not repeating properly

From metanorma/iso-2533#25
Now:

Should probably look like this:

I put the units statement inside the table's first header row, and changed alignment to right. We probably need some "style" for this units statement because it's meant to be informational and therefore small in font size.
Which makes it look more reasonable as with the original:

Word does support repeating headers but it seems that the full header is not being repeated, only the first header row:

P.S. I'm actually impressed that Word loaded the file so fast, and smoothly scrollable, compared to Safari.

P.S. I'm actually impressed that Word loaded the file so fast, and smoothly scrollable, compared to Safari.

It's hung my Word instance, so I don't share the sentiment...

Note currently looks like this:

Boldface has just been removed. Need to see if there is a built-in style for keeping right alignment and/or small font. Will not be moving note inside table, that's not how ISO define it.


Formatting of bar text

Dear dexplo,
First, thanks for the great work! I have one question. In your basic example in README.md, you show a COVID chart with the following parameters:
bar_textposition='inside',
bar_texttemplate='{x:,.0f}',
bar_label_font=7
To my understanding this would allow us to format the text displayed together with the bar chart. However, in the most recent version in Python 3.10 (using on Ubuntu 22), with pandas and matplotlib up-to-date,there are none of these options apparently? I get the following error:
TypeError: bar_chart_race() got an unexpected keyword argument 'bar_textposition'

By opening _make_chart.py, function plot_bars(...), lines 376 and 382 display a hardcoded text formatting:
text = f'{y1:,.0f}'

Are these missing options new, not yet published in the official pip version?
Thank you very much,
Cristiano

Ok so apparently the version here on github is very different from the version I get when I install the package with pip... I wonder why?

Resolved by installing the package as:
pip install git+https://github.com/dexplo/bar_chart_race.git


Conventional commits

dd9cabf pom changes could be in a different commit (build)? ⚠
Awaiting resolution of FlowingCode/DevelopmentConventions/issues/12

9bf1751 assembly.xml changes could be in a different commit (ci?) ✔
Done. As mentioned in https://github.com/FlowingCode/DevelopmentConventions/issues/12#issuecomment-742718469 I understand it belongs to the same commit that updates the pom. Regarding ci versus build, that needs to be addressed in the guidelines (https://github.com/FlowingCode/DevelopmentConventions/issues/10)

dd9cabf pom changes could be in a different commit (build)?
No (because the changes in POM are required in order to run the SerializationTest)

@ngonzalezpazFC Please review

@ngonzalezpazFC Please review

LGTM


How to set wandb offline？

Hello!
Can I know how to set wandb offline？ Because my school doesn't support me to use it online.
But when I set it offline according to someone else's method, it will report the following error.
wandb: Network error (ReadTimeout), entering retry loop.
Thank you very much!

Hi @jjjsyyy,
Please set it to be "disabled" in here.
Cheers,
Yuyuan


Any hope for a v10 support?

Just thought I would ask the question, hoping to get a response or I may ask for the league to take over development as this module is required for the terrain module to actually function!

Yes, the module will be ported to v10. However I've got quite a long list of modules that broke with v10, so the port may take a while.

@manuelVo Thanks for your work, do you know a timeline for the update ?

If everything goes according to plan, the release should be out some time this week.

I've just released a v10 compatible version

Amazing thanks :)


[QUESTION]  DS engine-image not Ready

Question
After fresh install pods of DS engine-image are not Ready. Readness probe failed. But if we run
k exec -n longhorn-system engine-image-ei... -ti -- /bin/bash

and then run
sh -c ls /data/longhorn && /data/longhorn version --client-only

and recieve json output.
Environment

Longhorn version: 1.3.2
Kubernetes version: 1.24.5 (Cri-O)
Node config

OS type and version Ubuntu 22.04
CPU per node: 6
Memory per node: 8Gb
Disk type SSD
Network bandwidth and latency between the nodes: 10Gb

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal): Baremetal

Additional context
Add any other context about the problem here.

it seems that something is wrong with the readiness probe for that pod. Can killing (and restarting) the pod resolve this?

Thanks. But restarting pods not work. Pods are in the same status

Can you provide the kubelet log of the nodes the NotReady pods reside on? Need to check the error message of the readiness probe first and see what happened.

Besides, Longhorn team needs to investigate if the engine image daemonset needs both the liveness probe and readiness probe. They seem to be redundant.


[FEATURE] show pvc name in UI

PVC Name:miop-zs-dev-elasticsearch-data-volv-pvc
PV Name:pvc-18545714-5d23-40ba-9efc-f7a99e53fcf3
it only show PV Name, no  PVC Name currently.

The PVC name, Workload info, and the namespace of the volume are already shown in the UI volume detail page as long as they are using the volume.

@shuo-wu
But show PVC name in list view page is more convenient to users.
thanks

There is a tooltip/hover presenting both PV and PVC name in the volume list page.

@yasker Do we still need to add one more column to explicitly present the info in the volume list page?

The reason we're using tooltip is that there are too many columns in the list already. Maybe in the future we can show the PVC name separately if we completely let user define what fields they want to see in the list. But it will be hard for us to show it in the list page directly now - especially PVC names are tend to be very long.


Feature/users controller tests

Coverage increased (+1.6%) to 53.718% when pulling 31a99c88e4f981a637f7114bb447acb44aec1f92 on feature/users-controller-tests into 9b99af4b9ca62605c29fa305d2226c2ba1a561e1 on staging.

@cromulus, this is ready for review. tests still failing on travis. not sure why .. any ideas?

@cromulus, this is ready for review. tests still failing on travis. not sure why .. any ideas?

Will fix. I'm pretty sure I know why.

FYI, it was the build cache.
Something is still failing, will look into that.

Timecop is weird on travis, for some odd reason.


fix typescript nodenext resolution

thanks for this amazing library! tiny fix here: while your ESModule build is working totally fine, when downstream consumers are set up to use nodenext for their module & moduleResolution settings, typescript reads your package.json and sees that the exports.import points to the dist-esm folder and tries to also look there for the typescript typings
as per the typescript docs this just lets typescript know to still keep looking in the dist folder for the types :)

Wow, @rayzr522 , thank you so much for your help!

🚀 released in v7.6.3, @rayzr522

omg tysm for the quick turnaround time!! i was just spinning up a new backend service using this and i was just using pnpm patch for now, thx for getting this out in record time!!! 💜


Adds Paket syntax coloring tag

This PR:

Updates FSharp.Formatting version to 2.13.4 which contains tpetricek/FSharp.Formatting#379
Adds the [lang=paket] command to all code blocks that contain paket code to use the new Paket syntax coloring.

The documentation will look like this:

this is so amazing!

this is really great stuff.
noticed a small bug with file::// at http://fsprojects.github.io/Paket/git-dependencies.html#Referencing-a-Git-repository

:astonished: I also noticed that I missed the "HTTP" keyword

I will fix asap


Error when using rdt to configure existing image

Image is the Raspi3 Betav5 based image
Error when
Babs:rov-image-resin brian$ sudo rdt configure /dev/disk2
? Network SSID mysid
? Network Key mykey
? Do you want to set advanced settings? Yes
? Device Hostname briandev
? Do you want to enable persistent logging? No
EBUSY: EBUSY: resource busy or locked, open '/dev/disk2'

If you need help, or just want to say hi, don't hesitate in reaching out at:

	GitHub: https://github.com/resin-os/resin-device-toolbox/issues/new
	Gitter: https://gitter.im/resin-io/chat

Hey @BrianAdams ,
Thanks for reporting this. The specific issue is related to https://github.com/resin-os/resin-device-toolbox/issues/32 and it looks like the issue can be traced back to an internal module we are using. I've opened an issue there as well (https://github.com/resin-io/reconfix/issues/7) so feel free to subscribe to it to get real time updates on our progress.

Hey @BrianAdams ,
We have released<EMAIL_ADDRESS>which should address this issue. Can you please reinstall rdt with

npm install -g resin-device-toolbox

and try reconfiguring the SDCard again?

Till breaking on 0.0.8
>rdt version
0.0.8

>sudo rdt configure /dev/disk2
? Network SSID XXXXXXX
? Network Key YYYYYYYYY
? Do you want to set advanced settings? Yes
? Device Hostname ZZZZZZ
? Do you want to enable persistent logging? No
EBUSY: EBUSY: resource busy or locked, open '/dev/disk2'

If you need help, or just want to say hi, don't hesitate in reaching out at:

	GitHub: https://github.com/resin-os/resin-device-toolbox/issues/new
	Gitter: https://gitter.im/resin-io/chat

@BrianAdams This is strange - can you please try cleaning the cache and reinstalling with

npm cache clean && npm install -g resin-device-toolbox

Let us know if you still reproduce the issue. Also, what OS are you using?


update(Emoji): Added remaining Emoji

Nice update, thanks. 1 small thing, could you make the Property names consistent though please? All CamelCase would be my preference, thanks.

Yeah no problem!


Updated tests and added format option

Codecov Report

:exclamation: No coverage uploaded for pull request base (dev@d4c7baf). Click here to learn what that means.
The diff coverage is 100%.

@@         Coverage Diff          @@
##             dev     #1   +/-   ##
====================================
  Coverage       ?   100%
====================================
  Files          ?      1
  Lines          ?     41
  Branches       ?      0
====================================
  Hits           ?     41
  Misses         ?      0
  Partials       ?      0

Impacted Files
Coverage Δ

src/core/core.go
100% <100%> (ø)

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 d4c7baf...733dc89. Read the comment docs.


Allow a list of algorithms for decode

Hard-coding only one algorithms makes rolling out changes harder. Both ends need to be deployed simultaneously or retry logic needs to be implemented in app.
Allowing the algorithms to be a list containing potentially multiple entries enables flexibility when changing algorithms. The default value is left unchanged and giving :algorithm option instead of  :algorithms still works.

@lautis Thanks for the PR. I left you a comment to improve the code.

@excpt should that be visible somewhere?

@lautis Now it should be visible. I need to press the "submit review"-button in order to make my review visible.

I added one more change: removed the combination of option because it could lead into the default algorithms array being used in addition to the value given with algorithm option.

Thanks again for your contribution. 🎉

Thank you for maintaining JWT! 🙏


Update template mapping

New

Update template mapping data

Link of the new template: https://docs.google.com/spreadsheets/d/1DrA0NCO9h-EjvQ-RobZJjwiB0qPCwd5P8IC1uMkeMoA/edit#gid=1841555907

seems like there is a remaining Regional tab from the Template that stay in the Generated sheet

note that contrary to the branch name the isue is IA-666


Supressing axes (decorations) on heatmaps

Is there any intuitive/effective way to achieve axes suppression, completely, when plotting heatmaps? I have used AbstractPlotting. Much appreciated

Scene(show_axis = false) should work. (If you call heatmap() you can pass it there. If you're using MakieLayout you need to pass it as scene_kw=(; show_axis = false) to the LAxis or LScene iirc)

For LAxis it would be
hidedecorations!(ax)
hidespines!(ax)

Thanks! show_axis = false worked smoothly. I tried with LAxis, but that did not work.


ApplicationName parameter

lines 386-389 have
if ($ApplicationName) {
    $CSBuilder["Application Name"] = $ApplicationName
}

but I don't see a parameter defined for $ApplicationName.
Should there be one like this?:
[Parameter(Position = 11,
	Mandatory = $false,
	ValueFromPipelineByPropertyName = $true,
	ValueFromRemainingArguments = $false)]
[string]$ApplicationName

Using the ConnectionStringBuilder was a change I started almost a year ago and things got muddy when I attempted to merge/PR my changes several months after I made the changes. I'll check into this, I probably missed something in copying my changes over.

@potatoqualitee or @RamblingCookieMonster can you please assign this to me?


feat: Support 'fullWidth' prop on all text fields

Adds support for a PresentationFieldProps.fullWidth property. The following components will respect the property:

FormLines
TextField
TextAreaField
RichTextField
SelectField
MultiSelectField
DateField
NumberField
TreeSelectField

Updates the FormLines component to default to the "lg" size instead of "full". This will hopefully minimize the impact on existing layouts as the previous default width for each field within FormLines was 550px. By using the lg size, we'll continue to default the fields within FormLines to 550px.

Is the idea that text fields were just getting ridiculous too wide, so 550px is a better default max size? That makes sense...

No, 550px is currently the default max-width. With this PR we are allowing the option to remove the max-width now. It sounds like in some instances we want to allow them to be wider.


DB clean up: obsolete sessions

Created by: @indyboox Mar 01 2022 at 14:04 CET
Last updated: Jul 19 2022 at 10:56 CET
ClickUp ID: IK-2347
Actuate what found and wrote in
Acceptance criteria:
Whenever user logs in, we create a session node, but don’t clean up. Run cypher that deletes authentication session longer than 2-3 months, free up a few 100K nodes on prod DB
Older sessions have no create_time attribute, be sure they are deleted as well

Older sessions without create_time attribute
77787 found
MATCH (s:Session) WHERE s.create_time IS NULL
DETACH DELETE s
Older than 2 months sessions
8056 found
WITH datetime({timezone:"UTC"}) AS cDate
WITH cDate, cDate-duration('P2M') AS sDate
MATCH (s:Session)
WHERE sDate>=s.create_time
DETACH DELETE s
Remaining sessions [15/03/2022]
42539 total
5001 revoked (inactive but usable for AI purpose)
NOTE: Before running the ttl step below, we should ensure that the newly created sessions are ttl-enabled.
I would suggest to edit the cypher(s) regarding session creation and management to include ttl, so that then the ttl query below would be run just once to assure all sessions are and will have ttl enabled on them.
TTL on all other sessions
params
:params
{delta:60,unit:'days'}
set ttl on all sessions
//no apoc iterate - the limit set was good enough to run it
MATCH (s:Session) WITH s, apoc.date.convert(toInteger( $ delta),  $ unit, 'seconds') AS secs WITH s, apoc.date.add(s.create_time.epochSeconds, 's', secs, 's') AS exp LIMIT 500000
CALL apoc.ttl.expire(s,exp,'s')
RETURN COUNT(s)
X found
this query below is the alternative one that raises permission errors, thus it's not to be used currently:
//with apoc iterate, not working
CALL apoc.periodic.iterate(
"MATCH (s:Session) WITH s, apoc.date.convert(toInteger( $ delta),  $ unit, 'seconds') AS secs RETURN s, apoc.date.add(s.create_time.epochSeconds, 's', secs, 's') AS exp",
"CALL apoc.ttl.expire(s,exp,'s')",
{batchSize:50000,params:{delta: $ delta,unit: $ unit}})
YIELD batches,total,timeTaken,committedOperations,failedOperations,failedBatches,retries,errorMessages,batch,operations,wasTerminated,failedParams,updateStatistics RETURN batches,total,timeTaken,committedOperations,failedOperations,failedBatches,retries,errorMessages,batch,operations,wasTerminated,failedParams,updateStatistics

Attachment(s):
https://github.com/indykite/clickup-attachments/blob/master/82230dd1-9aff-41e2-b484-6ad059c4c42d.jpg

Commented by: @marvinkite Mar 05 2022 at 09:48 CET
I found on miro board a note how the password credential was changed from one node to node use property node.
Attachment(s):
https://github.com/indykite/clickup-attachments/blob/master/82230dd1-9aff-41e2-b484-6ad059c4c42d.jpg

Commented by: @indyboox Mar 14 2022 at 09:55 CET
still cannot run cyphers as of today because of permission related issues..this below is the (dev) error trace, very similar between prod and dev db, even though I'm assigned read-only in the former, while admin role in the latter.
"errors": {
"org.neo4j.graphdb.QueryExecutionException: Set property for property 'ttl' on database 'neo4j' is not allowed for user 'andrea' with roles [PUBLIC, admin] overridden by TOKEN_WRITE overridden by READ.": 16,
"org.neo4j.graphdb.QueryExecutionException: Set label for label 'TTL' on database 'neo4j' is not allowed for user 'andrea' with roles [PUBLIC, admin] overridden by TOKEN_WRITE overridden by READ.": 14
  }

Commented by: @indyboox Mar 15 2022 at 09:33 CET
the permission related errors are constrained to the use of apoc.periodic.iterate to batch transactions and the Neo4j server instance. When running locally and/or outside of the batching, the ttl works fine.
The sessions older than 2 months (with and without the create timestamp) were deleted before setting up the TTL (Step 1 and 2).
Further steps in the task description.

Commented by: @indyboox Mar 15 2022 at 13:44 CET
@bayegan @yetizer
Because sessions have been cleaned up anyway and the apoc slowed down a lot the task on ttl, I would open a new task for taking care of ttl, in which there are the two sub-tasks:

edit session management cyphers in BE to include ttl
set ttl on remaining sessions in db

this would also allow @maprokes  to enable ttl in prod and have his waiting task 27h8yay released, since we understood at least where the problem is and how to work around it, so it is not necessary to hold the task anymore after enabling ttl in the prod db.
What do you think?


TypeError: RequestInit: duplex option is required when sending a body.

Run npx chrome-webstore-upload-cli@3 upload --source chrome-extension-cfd726002d89d231ebfd214435baf76305827fca.zip --auto-publish
npm WARN exec The following package was not found and will be installed<EMAIL_ADDRESS>- Fetching token
- Uploading chrome-extension-cfd726002d8[9](https://github.com/alexandredepape/yazyk/actions/runs/7360389303/job/20036472601#step:4:10)d231ebfd214435baf76305827fca.zip
TypeError: RequestInit: duplex option is required when sending a body.
    at Object.fetch (node:internal/deps/undici/undici:[11](https://github.com/alexandredepape/yazyk/actions/runs/7360389303/job/20036472601#step:4:12)730:11)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async APIClient.uploadExisting (file:///home/runner/.npm/_npx/da388c0a4eab99e8/node_modules/chrome-webstore-upload/index.js:34:25)
    at async doAutoPublish (file:///home/runner/.npm/_npx/da388c0a4eab99e8/node_modules/chrome-webstore-upload-cli/cli.js:74:28)
    at async init (file:///home/runner/.npm/_npx/da388c0a4eab99e8/node_modules/chrome-webstore-upload-cli/cli.js:[16](https://github.com/alexandredepape/yazyk/actions/runs/7360389303/job/20036472601#step:4:17)7:9)
    at async file:///home/runner/.npm/_npx/da388c0a4eab99e8/node_modules/chrome-webstore-upload-cli/cli.js:[17](https://github.com/alexandredepape/yazyk/actions/runs/7360389303/job/20036472601#step:4:18)6:5
Error: Process completed with exit code 1.

Ever since https://github.com/fregante/chrome-webstore-upload-cli/pull/74
Any idea ?

https://github.com/nodejs/node/issues/46221

I'll fix it soon

This has been fixed: https://github.com/fregante/chrome-webstore-upload/releases/tag/v3.0.1
Can you test again?

Thanks, will test


include imageMaps into counted figures

currently imageMaps are not numbered in the figure-numbering scheme...
it should be possible to add the appropriate html useMap directive to the asciidoctor image:: tag?!
need to investigate

could try:

include image with image-adoc-statement,  add "useMap" directive
create an independend counter

or simply ignore


exploitdb: 2023-05-25 -> 2023-05-27

Description of changes
update to 2023-05-27
commit tagged as 2023-05-27: https://gitlab.com/exploit-database/exploitdb/-/commit/9e36596021d19dadc97fcc4441fbfda0096284cb

Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[X] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

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

[X] Fits CONTRIBUTING.md.

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

1 package built:

exploitdb


asciidoc support

I'd love to see AsciiDoc support in peach.
Markdown is fine for simple use-cases documentation. AsciiDoc is far more richer than Markdown and is a necessity in serious documentation work.
AsciiDoc also has high quality tools for producing a wide range of output formats: html5, xhtml, docbook, pdf, epub, dvi, latex, roff, ps.
Peach can then provide a "export as pdf" kind of feature !
(hint: in the absence of plain go library that supports asciidoc, external ruby/python helpers can be used as done in case of hugo - https://github.com/spf13/hugo/issues/470)

Thanks your info!

:+1:
https://www.github.com/VonC/asciidocgo


Magenta Colors Fix

Hey bro sorry to bother you again, I have found the real cause behind the screen brightness and low graphic quality on my HP Elitebook 8470p Big Sur. Colors on my laptop screen or not accurate blue color is like indigo and grey color has changed to pink color. Same exact to the issue in this video: FIX Mojave Catalina MAGENTA MONITOR
This fix in video suggest to modify System>Display.  Please help.

you have already issue for display fix , and you can fix yourself


hsetnx missing

Heyo
HSETNX seems to be missing.
https://redis.io/commands/hsetnx/
Cheers

Thanks, that must have slipped through
I'm on vacation starting tomorrow, so unfortunately this has to wait until ~22nd-25th of august.
But I'll add it right after

Aces... have a great holiday

hmm, looks like it's already implemented, what made you think it's not available?

@magicspon

Oh yeah... weird, my code editor must have been having a funny 5.


Azure Kubernetes Service operator implementation

General feedback: errors should usually be check and returned. If there is a reason not to that should be made clear.

General feedback: errors should usually be check and returned. If there is a reason not to that should be made clear.

I fixed all comments and re-tested aks-operator. It can be merged now

nit: I noticed many functions accept the whole config as an argument but only use 1 or 2 fields off of it. The params to these can be changed.


chore: upgrade to js-libp2p 2.0

Description

bump libp2p and related deps to latest version
PeerId no longer contains a private key, anything that needs a private key must pass a PrivateKey instead

just deployed to feat3
ready for review

there is 1 mainnet block received very late on feat3, I'm not sure if it's related to the new libp2p or not

@wemeetagain is this IDONTWANT feature ON for this branch? https://github.com/ChainSafe/js-libp2p-gossipsub/pull/498/
what's the consequence of that and do we need to tweak params for them? GossipsubIdontwantMinDataSize seems low as it's around Attestation message

Yes IDONTWANT is now enabled by default.

what's the consequence of that and do we need to tweak params for them?

When we receive new (non-duplicate) messages greater than idontwantMinDataSize (default 512), we will immediately send an IDONTWANT to our mesh peers.
We likely want to avoid having to send "a lot" of IDONTWANTs and make sure we're just sending them for messages that are "larger". Avoid sending for attestations, ensure sending for blocks and blobs.
We can bump idontwantMinDataSize to 16829 to avoid the largest Attestations in electra.
node -e "import('@lodestar/types').then(x => console.log(x.ssz.electra.SignedAggregateAndProof.maxSize))"

in the last 2 days peer counts are around ~130 on the mainnet node and ~120 on the 1k node, not sure if that's an issue of new libp2p but let's see if it happens after the restart

we received too many IDONTWANT messages with js-libp2p 2.0 @wemeetagain

:tada: This PR is included in v1.23.0 :tada:


Convert/reconnect from unauthenticated chat to authenticated chat

Is it possible to convert an unauthenticated chat session to an authenticated chat session using the chat reconnect functionality? And do it in such a way, that the chat history of the unauthenticated session is retained and visible to the human agent in the authenticated session? So basically:

Start an unauthenticated chat session
Get the reconnect context and reconnectId
Start a new authenticated chat session with the reconnectId of the unauthenticated session

Currently "elevating" an unauthenticated session to an authenticated session results in the agent having to deal two separate chat sessions. If this is not possible is it a limitation of the chat SDK or the Omnichannel service?

It is not a supported scenario with Omnichannel Service. Here's the list of scenarios we support:

Unauthenticated Chat

Chat Reconnect

Authenticated Chat

Chat Reconnect
Persistent Chat