details of merge options

Hi,
I want to use SURVIVOR merge to merge calls from several VCFs. I've downloaded the latest release version and the help message for merge command reports these 2 options (among others) that are not clear to me:
1. max distance between breakpoints (0-1 percent of length, 1- number of bp)
2. estimate distance based on the SV size (1==yes, else no

If I set 1000 for example for option 1, I expect that SV events with both start and end breakpoints closer than 1000 bp will be merged. Instead, if I set 0.1 for option 1, then I expect that 2 SVs are merged if their start and end points are within a distance equal 10% of their respective lengths. What happens in case in the case of SVs with different lengths? The maximum allowed distance is determined by the shorter event since its fraction will be the shortest?
Given the above, which already seem to take SV length into account, which is the effect of option 2?
Thanks!

So point 2 is currently inactivated. I left it in there to not mess with existing pipelines.
We usually use option 1 with 1000 bp. This works very well for different data sets. The output holds the information about which SV where merged.
thanks
Fritz

Thanks!
This clarifies the conflicting options.
Additional question: in your experience which is the maximum number of VCF that can be merged by survivor? I will possibly like to scale to ~50k samples. Any advice to merge such a large cohort?

not a huge problem. We had run it on Topmed...
You might want to split by chr... memory consumption is not bad.. but the output file can get large.
Also please make sure input VCF files are sorted. That will ease the job.
Cheers
Fritz

and @edg1983 just ping me if you encounter an issue. 50k gets you in my gold member club ;)

I'm trying now to merge the big dataset. Few question / suggestions:

How long can it take to merge a ~50k sample dataset without splitting by chrom? My medium queue runs for 24h max, is this enough, or you advise allowing a longer run time?
Can the tools read compressed VCFs? It seems not working when the input files are bgzip compressed
It would be great to have a region option to restrict merge by chromosome. As you suggested, merging a large cohort can be done by chromosome, but this can be painful if I have to prepare by chromosome VCFs from ~50k individual VCFs.

I'll keep you updated! Thanks!

Thats hard to predict as it depends also on the number of SV per sample.
@2: No sadly... I have still not included that.
@3 agree. I previously just did that by splitting the original VCF files..
thx
Fritz

Just to clarify the longest part is actually the writing out of the VCF file. :)

Just an update on this. After several tries, I was not able to merge the whole cohort (79530 samples) in a single run, after ~1 week of running time it crashed with a bab_alloc error. What I've noticed across my various attempts is that survivor merge get slower as it progress and above ~25000 samples the time taken for each sample become huge.

strange.. we have merged 80k WGs. We split it up per chr .. but I guess that you have done the same ?
And you are saying that the memory is the issue not the wring out.?
Can you confirm the version you are using ?

Unfortunately, I cannot split by chromosomes, since this would generate more files than my inode quota on the system can allow. So I merged PASS calls across all genome for each sample. Memory usage is not linear apparently, since I didn't see a large difference when processing 25-30k samples or 50-70k for example (it goes up to 180Gb max).
I used survivor version 1.0.7 with the following command
survivor merge filelist.txt 1000 1 1 1 0 50 output_vcf.vcf

What typically take a long time is the writing out..
Sorry for the issues.
Can you do it interatively. E.g. take chr22 and test it there ?
I usually run it without the strand as this flag is sometimes hard to parse:
survivor merge filelist.txt 1000 1 1 0 0 50 output_vcf.vcf
thx
Fritz

I would do some more tries to find the best approach for merging, but right now I'm satisfied with 3 chunks. I will now test if the output VCF obtained is compatible with the downstream tools we use for SVs analysis.
I found survivor the only tool (among the few I tested) able to merge such a large cohort so it's great! Adding the ability to read compressed/indexed VCF and thus process just a region of interest would make it perfect!

Ok great. Otherwise we would have needed to get creative :)


The testing results of the whole dataset is empty

I'm very glad to receive your reply.
but
After the modification, there is still this error when running the test command. Is there anything else to modify. My config_file is cascade_mask_rcnn_swin_tiny_patch4_window7_mstrain_480-800_giou_4conv1f_adamw_3x_coco.py

No description provided.

@sure7018    Hi，Have you solved this problem? I  have the same with you! if yes, please help me out! thank you!!!


pull request for e2e test at 12/11/2016 11:56:03

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Support for Hummingbird

Reported by maximo.piva Jul 1, 2013
http://hummingbird.me/
Seems like MAL, but with trending, recommendations system, facebook links, chat, discussion, etc. like MAL but NEXT GEN. it's beta, but the forum/chat/discussion seems to be populated, so lots of people are using it.
Jul 3, 2013 werndlymob
Have joined the site and it looks pretty nice.
But it doesn't look like it has any sort of API, which is the main reason I haven't put any major effort into MAL.
I'll look at this later to see if they plan to support it
Jul 3, 2013 Are.Torrentbytes
they will make the api public when its done apparently. so will you guys look into it? they are apparently moving away from MAL (since right now you can import your MAL collection into the hummingbird collection with all the MAL faults as well >____< )
but they are moving away with that so that's good :D
apparently noone have heard of the JMM server system you made. so i made a little shout out about it and asking for alittle support to come our way.
http://hummingbird.me/community/forums/site-suggestions/topics/add-support-for-the-jmm-server-system?page=1
sorry that i really seem over excited and such about the subject but it really looks like it can benefit you both with more users both ways.
Jul 4, 2013 werndlymob
Yes, if the api is pretty good then I will support it
The most important thing is being able to associate an anime on AniDB with an anime on Hummingbird
Jul 4, 2013 Are.Torrentbytes
well they arnt using AniDB at all ATM, they do use an import thing from MAL tho.
but they are saying that they want to be for them self without using anyone else.
they are apparently not that big with people that download their Anime and such.
most of them are streaming and they are working on a onsite linking system to many streaming sites like Netflix and Hulu and such.
well payed streaming isn't a possibility for me so that isn't happening for me and free streaming is too crappy in quality for my OCD about quality. so for me i stick with real blu-rays in my shelf or blu-ray rips and such from fan subs :D

Humming bird is now https://kitsu.io/
Can read the post about the move here: https://medium.com/heykitsu/hummingbird-is-dead-long-live-kitsu-bda6ccfbbcce#.4c15cznd2
"Kitsu is a rewrite and redesign of Hummingbird from the ground up."
it also seems that we missed they went open source on Github aswell!
https://github.com/hummingbird-me/hummingbird
@ElementalCrisis Kitsu/Hummingbird would also fit perfectly into the V2 Client in terms of looks!
title of OP should be edited to fit the new name <3

This is something that probably won't make it in for a long time if ever unfourantely, we've got a lot of work to do even before Client V2 and that is itself a huge undertaking.
The community sites will be treated like plugins in V2 so they can have separate updates, should make adding new ones easier for other developers.

its a nice step forward with a plugin based system. i like that alot.
im just updating info that was out of date on this page <3
their site still got long to go anyway.

I'll keep this issue open, but I don't think this will evert happen. AniList is next to be added and that's taken a long time.


Fix fetch_episodes_by_season

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] I have read the CONTRIBUTING document.

All Submissions:

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Fix #215

@pinoelefante Can you fix those 3 whitespace issues found by Codeclimate? (see https://codeclimate.com/github/asciidisco/plugin.video.netflix/pull/218)
Then I´ll be able to merge & make a new release.


G1 and G2 equivalence

This includes proving equivalence between gallina and fiat-crypto with fiat-crypto point equality for both G1 and G2.
There is also a proof for equivalence between gallina and hacspec for G1.

Looks good to me. What do you think @RasmusHoldsbjergCSAU ?
Aside: The easy and now tactic are sometimes helpful and they check that subgoals are actually completed.

I've refactored the proof in EquivToBedrock to make use of the equivalence proof over abstract fields.

This mainly includes a proof that hacspec point multiplication is equivalent to fiat-crypto point multiplication.
There is also some refactoring of the addition equivalence proof between hacspec and fiat-crypto, as well as an established connection between hacspec and gallina for G2 points.

Looks nice! Maybe you can add a bit more documentation?
@RasmusHoldsbjergCSAU What do you think?


[BACKLOG-22916] Set error handling to true by default

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -fn -f 'pom.xml' -pl 'engine' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  engine/src/main/java/org/pentaho/di/trans/steps/uniquerows/UniqueRowsMeta.java
  engine/src/main/java/org/pentaho/di/trans/steps/uniquerowsbyhashset/UniqueRowsByHashSetMeta.java

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes (click to expand)
org.pentaho.di.trans.steps.ldapoutput.LDAPOutputMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change: -1.1% :small_red_triangle_down:
Complexity Change: -1.3% :small_red_triangle_down:

org.pentaho.di.trans.steps.setvalueconstant.SetValueConstantMeta

Instruction Change: -0.2% :small_red_triangle_down:

org.pentaho.di.trans.steps.sortedmerge.SortedMergeMeta

Instruction Change: -0.2% :small_red_triangle_down:

org.pentaho.di.trans.steps.tableoutput.TableOutputMeta

Branch Change: -0.9% :small_red_triangle_down:
Complexity Change: -0.8% :small_red_triangle_down:

org.pentaho.di.trans.steps.webservices.WebServiceMeta

Branch Change: -1.7% :small_red_triangle_down:
Complexity Change: -1.2% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:


TMM isnt starting

What TMM version are you using?
newest pre-release, i tried also the standard release
release, pre-release, nightly, or directly from GitHub/branch?
What is the actual behaviour?
start cancel (Window: Java/TM) Platform SE binary funktioniert nicht mehr)
What is the expected behaviour?
Steps to reproduce:

first
second
...

Additional
Have you attached the Logfile from the day it happened?
Your Operating system? (win/mac/linux? version?)
Windows 10, Java Version 8 Update 121
launcher.log.txt
tmm.log.txt

your are the 5th today, reporting that issue :|
There seems to be a problem on W10 with latest Java....
Please see http://forum.kodi.tv/showthread.php?tid=304981

oh I have a high place, so im proud!!! 📦
I searched the forum today and I found nothing, so I created this thread
P.S.: installing old version of Java didnt fix the prob!

update in the forums: http://forum.kodi.tv/showthread.php?tid=304981&pid=2509292#pid2509292
short: a new NVIDIA driver causes crashes to java/tmm.
adding -Dsun.java2d.d3d=false to the file extra.txt in the tmm folder let tmm start again


feat: add zap support

add https://github.com/uber-go/zap support

Hi, since #12, your pr seems need to rebase.

OK, I'll fix this later

Please note that by default, only logr and klog is enabled right now (latest base)

Should we enable zap by default ?
zap logger seem popular enough, and is widely used

rebased to master

I see. it is just doc change. I've updated README.md


Cannot read json policy file

Hello,
I'm trying to combine two or more json file policies into one but i can't make it work, this is what I have for now.
provider "aws" {
  profile = "staging"
  region  = "us-east-1"
}

resource "aws_iam_policy" "AmazonSSMManagedInstanceCore" {
policy = file("AmazonSSMManagedInstanceCore.json")
}

resource "aws_iam_policy" "ReadOnlyKMSAccess" {
  policy = file("ReadOnlyKMSAccess.json")
}

module "aggregated_policy" {
  source = "git::https://github.com/cloudposse/terraform-aws-iam-policy-document-aggregator.git?ref=tags/0.2.0"

  source_documents = [
    "aws_iam_policy.AmazonSSMManagedInstanceCore",
    "aws_iam_policy.ReadOnlyKMSAccess"
  ]
}

output "result_document" {
  value       = module.aggregated_policy.result_document
  description = "Aggregated policy document"
}

Both json files are in the current same folder as the .tf file and I get the following error
module.aggregated_policy.data.aws_iam_policy_document.empty: Refreshing state...
module.aggregated_policy.data.aws_iam_policy_document.zero: Refreshing state...

Error: invalid character 'a' looking for beginning of value

  on .terraform/modules/aggregated_policy/main.tf line 22, in data "aws_iam_policy_document" "zero":
  22: data "aws_iam_policy_document" "zero" {

Terraform v0.12.28
+ provider.aws v2.68.0

The policy files are standard json policies
Just an example
{
    "Version": "2012-10-17",
    "Statement": [
        {
            "Sid": "VisualEditor0",
            "Effect": "Allow",
            "Action": [
                "kms:GetParametersForImport",
                "kms:DescribeCustomKeyStores",
                "kms:ListKeys",
                "kms:GetPublicKey",
                "kms:ListKeyPolicies",
                "kms:ListRetirableGrants",
                "kms:GetKeyRotationStatus",
                "kms:ListAliases",
                "kms:GetKeyPolicy",
                "kms:DescribeKey",
                "kms:ListResourceTags",
                "kms:ListGrants"
            ],
            "Resource": "*"
        }
    ]
}

Thank you a lot in advance.

This looks like the incorrect usage.
See https://github.com/cloudposse/terraform-aws-iam-policy-document-aggregator#usage
This needs to be a list of JSON documents.
  source_documents = [
    "aws_iam_policy.AmazonSSMManagedInstanceCore",
    "aws_iam_policy.ReadOnlyKMSAccess"
  ]


Kafka sink ignores neo4j.batch.timeout.msecs when neo4j.batch.parallelize = false

Source
Reported from Neo4j Startup user
Expected Behavior (Mandatory)
Kafka Sink has a neo4j.batch.timeout.msecs to timeout the query and uses it.
Actual Behavior (Mandatory)
The Kafka Sink has a neo4j.batch.timeout.msecs to timeout the query, but when neo4j.batch.parallelize = false, the SINK doesnt care about the timeout at all. The Sink code doesnt use the timeout.msecs setting at all in parallelize false.
Versions

Neo4j: 4.2.x EE
Neo4j-Streams: 4.0.4

Validated -- we are working on this and have spotted the problem.

Closed by recent release


Update fff module tests

Remove the module tests (they're no longer needed since upstream FFF runs better CI now)
Update the header to match the pending PR at https://github.com/meekrosoft/fff/pull/113

Failed compliance check is a false positive, the () cannot be moved closed to RETURN otherwise the macro expansion will treat RETURN as a macro function

@carlescufi what do you think of this? The CI on fff has been updated to use github actions so we also will be running those on our fork (once that PR merges) but we have a chicken & egg issue that I can't merge those changes without rolling back the additional commit that allowed us to run the fff tests on our CI. What I'd like to do:

Merge this commit to remove FFF from the CI of this repo
Roll back https://github.com/zephyrproject-rtos/fff/pull/1
Pull all the upstream changes from https://github.com/meekrosoft/fff
Update the CI to just run a diff between the fff.h in our fork vs this repo (they should always match)

@carlescufi what do you think of this? The CI on fff has been updated to use github actions so we also will be running those on our fork (once that PR merges) but we have a chicken & egg issue that I can't merge those changes without rolling back the additional commit that allowed us to run the fff tests on our CI. What I'd like to do:

Merge this commit to remove FFF from the CI of this repo
Roll back Add support for Zephyr fff#1
Pull all the upstream changes from https://github.com/meekrosoft/fff
Update the CI to just run a diff between the fff.h in our fork vs this repo (they should always match)

I have no objections with the proposed plan, but I'd like @stephanosio to weigh in on it as well.

Merge this commit to remove FFF from the CI of this repo
Roll back Add support for Zephyr fff#1
Pull all the upstream changes from https://github.com/meekrosoft/fff
Update the CI to just run a diff between the fff.h in our fork vs this repo (they should always match)

As long as the "roll back" and the "pull" are done in a non-destructive way (i.e. no force push), no objections. It is important that the commit(s) that are referenced by previous Zephyr releases are valid.

Merge this commit to remove FFF from the CI of this repo
Roll back Add support for Zephyr fff#1
Pull all the upstream changes from https://github.com/meekrosoft/fff
Update the CI to just run a diff between the fff.h in our fork vs this repo (they should always match)

As long as the "roll back" and the "pull" are done in a non-destructive way (i.e. no force push), no objections. It is important that the commit(s) that are referenced by previous Zephyr releases are valid.

Yup, that's exactly what I had in mind, the PR is ready to go at https://github.com/zephyrproject-rtos/fff/pull/2/commits. You can see there that the first commit is a revert then a fetch from upstream.


Broken link to release archive

Azure Data Studio Version: 1.21.0

Steps to Reproduce:

https://github.com/microsoft/azuredatastudio/releases/tag/1.22.0

Linux TAR.GZ (https://go.microsoft.com/fwlink/?linkid=2142212)

curl -I "https://go.microsoft.com/fwlink/?linkid=2142212"
HTTP/1.1 302 Moved Temporarily
Location: https://azuredatastudio-update.azurewebsites.net/1.22.0/linux-x64-archive/stable

curl -I "https://azuredatastudio-update.azurewebsites.net/1.22.0/linux-x64-archive/stable"
HTTP/1.1 404 Not Found

@abist can you assist please?

Fixed, apologies for that @MiRacLe-RPZ! Let us know if you're still seeing issues.
@abist Sync up with me so we can make sure this is documented correctly.


I want to try use helm v3 but met some problems

Hi @andyzheung, as the error suggests, it would appear that your kubernetes cluster is unreachable. Your Kubernetes instance could be down, or your network setup is incorrect, or your local setup (your kube config) is misconfigured. What is the output of kubectl cluster-info?

https://github.com/helm/helm/blob/4b462c39022993a8687bfc44df308534cf1cc306/pkg/kube/client.go#L75-L83

https://github.com/helm/helm/blob/4b462c39022993a8687bfc44df308534cf1cc306/pkg/kube/client.go#L75-L83

![image](https://user-image@bacongobbler .githubusercontent.com/22912347/68191591-d572a000-ffea-11e9-9812-65d279faeec7.png)
It seem to running at 6443 port.
@bacongobbler

It seem to running at 6443 port.
@bacongobbler

And my other k8s cluster is 16443 port..
Can I change the port of helm connect?

no, you can't
helm use kubeconfig to access the cluster, if helm is install on the same node of your second cluster, it should be working.
If helm is installed on other hosts, you need to get a working kubeconfig file first, make sure you can use kubectl to access the cluster

My cluster can use kubectl to access.
my kubeconfig.yaml

It seems I use the kubeconfig file which my kubelet use that still don't work.

If you can provide a set of steps on how to reproduce this issue, that would be helpful to diagnose the issue. Currently if I boot up a cluster with minikube start, Helm will work right out of the box.

I just use two way to run k8s:
1、k3s:
https://github.com/rancher/k3s
2、microk8s:
https://microk8s.io/docs/
and them just install helmv3:
wget https://cloudnativeapphub.oss-cn-hangzhou.aliyuncs.com/helm-v3.0.0-alpha.1-linux-amd64.tar.gz
tar -xvf helm-v3.0.0-alpha.1-linux-amd64.tar.gz
mv linux-amd64 helm3
mv helm3/helm helm3/helm3
chown root.root helm3 -R
cat > /etc/profile.d/helm3.sh << EOF
export PATH=$PATH:/home/andy/helminstall/helm3
EOF
source /etc/profile.d/helm3.sh
helm repo add apphub https://apphub.aliyuncs.com
helm search repo guestbook
helm install guestbook apphub/guestbook --set service.type=NodePort

@andyzheung It seems that you are using v3.0.0-alpha.1 which is a very early version of Helm 3 unreleased. Do you mind using the latest which is Helm 3 RC-2 and test again?

I am also having exactly the same issue with a new k3os instance.
I used Helm 3 RC-2.
The k3os instance is working perfectly, I can run machines on it and have had no connection problems. The kubeconfig file is also perfectly fine.

Oh, I found how to make it work! I have to specify manually the kubeconfig file using the flag, like this:
helm install runner gitlab/gitlab-runner -f /home/rancher/.gitlab/values.yaml --kubeconfig /etc/rancher/k3s/k3s.yaml

After 8 hours trying to make it work... finally it succeeds...

Closing as an issue on the user's end with how the cluster is configured. Nothing actionable on our end here.

@andyzheung It seems that you are using v3.0.0-alpha.1 which is a very early version of Helm 3 unreleased. Do you mind using the latest which is Helm 3 RC-2 and test again?

Thank you so much, I use  Helm 3 RC-2 and test like this:

It works .
@KinoshitaShimizu @hickeyma
But Helm 3 RC-1 still not work.. may be some bug in this version.  @hickeyma
Any way , I will use RC-2 to test.


Custom font not loading on internet explorer

The custom font on the website does not load when viewing the website using internet explorer. It does work with Chrome, Safari and Firefox.

So I read a little bit about this in fondeck’s docs. It should not be a problem with this on IE so I think ive got something configured wrong. One thought is that our link to the css never included the domain name properly so I added that to our configuration
Se this file (line 15)
https://github.com/eCitizen/gamr-web/blob/master/config.json#L5
I added what i presume is the correct domain into the link (but i wasnt sure if the subdomain needed to be there)
I think we should try building this, and if it doesn't work we should try it without the subdomain. If that doesn't work I will keep digging.

I'll redeploy later today...
On Oct 12, 2015 22:08, "Greg Skiano"<EMAIL_ADDRESS>wrote:

So I read a little bit about this in fondeck’s docs. It should not be a
problem with this on IE so I think ive got something configured wrong. One
thought is that our link to the css never included the domain name properly
so I added that to our configuration
Se this file (line 15)
https://github.com/eCitizen/gamr-web/blob/master/config.json#L5
I added what i presume is the correct domain into the link (but i wasnt
sure if the subdomain needed to be there)
I think we should try building this, and if it doesn't work we should try
it without the subdomain. If that doesn't work I will keep digging.
—
Reply to this email directly or view it on GitHub
https://github.com/HumanDynamics/MassScaleSocialSystems/issues/103#issuecomment-147574111
.

The font did not load on IE once, but then it correctly loaded a number of times after and on different machines.


handle embed_one/many in the JSON extension

Currently, the JSON extension doesn't handle structs declared as embeds_one or embeds_many fields.
This checks whether the module being parsed has embeds, and if so recursively encodes the values for that embed depending on cardinality.
Need to flesh this out a bit but wanted to see if this made sense conceptually.

Hey, thanks for the PR!
Yes, good catch. It doesn't currently apply recursively to embeds.
Your approach makes sense to me.
What all do you think needs more polishing?

cool, yea actually this works as is.


feat(ci): Add initial CI with flake checks and derivation caching

Declarative cache usage looks like:
  nixConfig = {
    extra-substituters = ["https://shanesveller-nix-beam-flakes.cachix.org"];
    extra-trusted-public-keys = [
      "shanesveller-nix-beam-flakes.cachix.org-1:DMWI87/57GNone8wN7dXcqlgdIk36qHfvhXJ/esq5hk="
    ];
  };

Confirm the pubkey at https://app.cachix.org/cache/shanesveller-nix-beam-flakes#pull .


How to show the value of variables in debug mode?

Describe the bug
This is a question rather than a bug. How to show the value of variables (e.g. the r-averaged concentration c_s_av as below) in debug mode (in VS code)? This is convenient to check the definition of equations and to dig out the issues as a developer.
Another question is how to get the r-averaged concentration for all electrode particles in a P2D model? It only has x-average concentration of c_s_n_xav and c_s_p_xav in standard_variables.py
To Reproduce
Steps to reproduce the behaviour:

Set a breakpoint to Line 37 in base_particle.py
Debug SPMe.py, then it will stop at the breakpoint in the last step.
Watch the variable c_s_av, then it shows a list of features but not the values.

Expected behaviour
To show a matrix/array of values for the average concentration for electrode particles.
Screenshots

When the model is being built, everything is purely symbolic so the value of the concentration doesn't exist. Typically what I do is solve the model and then look at the values of the variables. What is the reason for needing to see the value before solving?
It looks like we never compute the r-average, I guess we never needed it until now. It should be added to _get_standard_concentration_variables in base_particle.py, like this:
"R-averaged "
+ self.domain.lower()
+ " particle concentration": pybamm.r_average(c_s)

Thanks for your reply. For some intermediate variables defined in functions but not for outputs, can I still see the values in the solution? I was trying to figure out where was the issue when getting a wrong result, by checking the values of intermediate variables.
Following your explanation on the r-average, is it applicable to many particles?

You would need to define the intermediate variables and add them to the list of variables. If it doesn't make sense to add them to the dictionary of variables, you could do it in your main script e.g.
c = model.variables["Electrolyte concentration"]
model.variables["Two times c"] = 2*c

then they will appear in the final solution.
Many-particle models aren't implemented yet, but r_average should work when c_s depends on both x and r

I have tried to get the r_average concentration, but it fails to broadcast the variable to the domain negative electrode. Besides, is it applicable to the DFN, as I'm currently using the SPM? Thanks.
I used the commands:
        c_s_n = variables["Negative particle concentration"]
        c_s_n_avg_p = pybamm.r_average(c_s_n)  # average concentration for particles
        c_s_n_avg = pybamm.SecondaryBroadcast(c_s_n_avg_p,["negative electrode"])
And when I printed their domain, I got
c_s_n.domain
['negative particle']
c_s_n_avg_p.domain
['negative particle']
c_s_n_avg.domain
['negative particle']

In theory, you shouldn't need to broadcast it again as c_s_n already depends on x and r, so just taking the r_average would make it depend only on x. But I have just noticed there is a bug in r_average in the lines
# If symbol is a Broadcast, its average value is its child
elif isinstance(symbol, pybamm.Broadcast):
    return symbol.orphans[0]

It does this even if the symbol is broadcast in the x-direction, which will be the case if you are using SPM. So r_average should be edited to deal with that case. More specifically:

c_s_n is pybamm.PrimaryBroadcast(c_s_n_xav, "negative electrode") where c_s_n_xav depends only on r.
in this case, r_average should return pybamm.PrimaryBroadcast(r_average(c_s_n_xav), "negative electrode")

Thanks for your reply. I need to report another bug (I think) below, which you may correct at the same time.
The children of  c_s_n = variables["Negative particle concentration"] is Variable(0x2c798945b2921b7, Negative particle surface concentration, children=[], domain=['negative electrode'], auxiliary_domains={'secondary': "['current collector']"}).
So pybamm.r_average(c_s_n) turns out to be Negative particle surface concentration following the if condition above. I'm testing with DFN this time.

I can't reproduce this. When running the normal DFN, the variable c_s_n like you defined gives
Variable(-0x6f6b862f45774429, Negative particle concentration, children=[], domain=['negative particle'], auxiliary_domains={'secondary': "['negative electrode']", 'tertiary': "['current collector']"})

with no children.
The way I can get it to do what you reported is if I run the DFN with options {"particle": "fast diffusion"}, in which case that is expected behaviour (solve an ODE for average particle concentration which is the same as surface particle concentration). Were you doing this, or something different?

Yes, I was using the fast particle submodel, which is the default option from a notebook example. The Fickian type model should be used instead. Thanks for pointing this out.


GraphMap ONT parameters

Hi,
I am comparing few nanopore aligners on ONT 1D and ONT 2D data, so I would like to verify if the general command below is correct for those types of reads
graphmap align -r ref.fasta -d input.fasta -o output.sam
or I need to specify some other parameters as well.
I would highly appreciate your input on this.
Thank you,
Natasha

Hi Natasha,
You are right. This is the correct command. Other parameters depend on specific use cases such as circular genomes, etc.
Best
Mile

Thank you!


Custom title controls on Windows

Removing native menu and title bar since it's to contrast and spoils whole app appearance in Windows
Moving context menu under sandwich icon
Adding custom buttons for window minimize, maximize and close

macOS view stays untouched.
Note. This PR does not include custom window controls in Preferences, Add cluster, Settings and other layouts which uses full-screen components. Will be handled in ongoing contributions.
Fixes #1437

https://user-images.githubusercontent.com/9607060/145201266-c6b96171-344a-4de6-9df8-63f5d274f95f.mov

The menu bar disappears if the window is closed and re-opened (on linux):
https://user-images.githubusercontent.com/40840436/145250595-756f4178-1c82-481d-ba25-046bb39e28a6.mov

I reverted Linux view to use native controls. So current implementation looks like this:
MacOS

Windows

Ubuntu
https://user-images.githubusercontent.com/9607060/145383960-e34f6873-8513-42c9-b68f-011381da6989.mov

@aleksfront That ubuntu view looks strange. Why hide the menu until a click?

The menu is hidden by default and gets activated (showed) if you press Alt key. If you then just click outside the menu, it disappears.

That seems suboptimal because you have to know to push ALT. There is #1734 for Linux, so I think we should do better.

The menu is hidden by default and gets activated (showed) if you press Alt key. If you then just click outside the menu, it disappears.

That seems suboptimal because you have to know to push ALT. There is #1734 for Linux, so I think we should do better.

I agree, there needs to be some visual cue for the hidden menus on linux. I'd prefer it to be like windows, with the "sandwich", or keep the linux menu bar always visible.

The menu is hidden by default and gets activated (showed) if you press Alt key. If you then just click outside the menu, it disappears.

That seems suboptimal because you have to know to push ALT. There is #1734 for Linux, so I think we should do better.

I agree, there needs to be some visual cue for the hidden menus on linux. I'd prefer it to be like windows, with the "sandwich", or keep the linux menu bar always visible.

I'd also prefer Linux to be like Windows. Is there some deeper issue with Linux support?

@jim-docker I've changed linux view to have custom window controls also. Here's current state on Win and Ubuntu 20.04. Please check linux view on your machine.

Looks great, works well on linux. Though it's a little odd that there are no window controls (min/max/close) on the Preferences, Extensions, etc. pages:

I've set menu position to be x: 20, y: 20 explicitly, it will show context menu always on icon center:
https://user-images.githubusercontent.com/9607060/145984393-2af1b32b-6027-4942-b1af-6b6ea4085775.mov
@jim-docker @Nokel81 please review this again.


refactor(context): use dataplane fetch approach

This PR highlights the problem with existing approach and explores a new approach to do minimal calls for service commands.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation change
[ ] Other (please specify)

The approach to use one call is good here just had a few other questions ^


[2624] Update Course level -> Education phase in filters

Context
https://trello.com/c/6wci02Mt/2624-s-change-label-of-course-level-filter-to-education-phase
Changes proposed in this pull request
What it says on the tin.
Guidance to review

Check the filter heading and the 'selected filters' label say 'Education phase'

 [S] Change label of 'Course level' filter to 'Education phase'


fix: allow circular references in DTOs

Hi, thanks a lot for your effort spent building this library.
Describe the bug
I started to declare my existing DTOs (some of them are already used in my OpenAPI specs) and unfortunately i encountered the following issue when running the server and generating AsyncAPI specs :
RangeError: Maximum call stack size exceeded
...

It happens when we use circular reference in our data models, it seems we can work around that by providing the option { dereference: { circular: 'ignore' } } in asyncapi-generator.ts like mentioned in this issue.
To Reproduce

Create a first UserDto

import { ApiProperty, getSchemaPath } from '@nestjs/swagger';

export class UserDto {
  @ApiProperty({
    description: "User's name",
    required: true,
  })
  username: string;

  @ApiProperty({
    description: "User's companies",
    required: false,
    type: 'array',
    items: {
      oneOf: [{ $ref: getSchemaPath('CompanyDto') }, { type: 'string', format: 'uuid' }],
    },
  })
  companies?: CompanyDto[] | string[];

  @ApiProperty({
    description: "User's assigned companies",
    required: false,
    type: 'array',
    items: {
      oneOf: [{ $ref: getSchemaPath('CompanyDto') }, { type: 'string', format: 'uuid' }],
    },
  })
  assignedCompanies?: CompanyDto[]  | string[];
}

Create a CompanyDto

import { ApiProperty, getSchemaPath } from '@nestjs/swagger';

export class CompanyDto {
  @ApiProperty({
    description: "Company's name",
    required: true,
  })
  name: string;

  @ApiProperty({
    description: "Company's owner",
    required: true,
    oneOf: [{ $ref: getSchemaPath('UserDto') }, { type: 'string', format: 'uuid' }],
  })
  owner: UserDto  | string;

  @ApiProperty({
    description: "Company's members",
    required: true,
    default: [],
    type: 'array',
    items: {
      oneOf: [{ $ref: getSchemaPath('UserDto') }, { type: 'string', format: 'uuid' }],
    },
  })
  members: UserDto[]  | string[];
}

Declare UserDto in a service

  @AsyncApiPub({
    channel: 'READ/USER',
    message: {
      name: 'User',
      payload: {
        type: () => UserDto,
      },
    },
  })

Expected behavior
We should either be able  :

to provide circular refs
to provide a custom configuration to generateFromString method

Edit
After digging a bit more, it seems the issue comes from asyncapi/parser as handling reference inside oneOf/allOf... is not yet supported. See https://github.com/asyncapi/parser-js/issues/293.

@getlarge thanks for the issue report,
checked the asyncapi/parser-js#293 issue, looks like it's resolved in the latest release (1.12.0)
Will check out if it's still an issue on weekends and release new version is it solves the issue.

@getlarge @flamewow Little update from AsyncAPI team: Yeah, we resolved problem with circular refs in parser-js side, but we still need some work in the react-component -> html-template, and hopefully we will merge all PRs in this week. You can check progress here https://github.com/asyncapi/parser-js/pull/402 Please wait and sorry for problems. Support for circular references when we need to stringify spec took a while 😅

@flamewow @magicmatatjahu thanks for the reply and the news.
I look forward to use this feature!

https://github.com/asyncapi/html-template/pull/286

@getlarge just release new version (with updated html-template).
Please check if you still have an issue.

If there will be any issues, please ping me in this issue :)


developing_omero_clients_print

Fix print statements for py3 in AdvancedClientDevelopment.rst
Also removed components/tools/OmeroPy/build/lib from build-system table.

Thanks @jburel for catching those other prints.

Looks good 👍


[2pts] User authentication

Create API endpoint to user authentication.
Create actions to user sessions.

DONE


add tests

I already have build-success testing. Anything else would be getting pretty crazy

http://zachholman.com/posts/how-github-writes-blog-posts/

Done!


Errors in Action Cable log after update to 0.11.0

Rails 5.0.2, development mode. After update lograge to 0.11.0 I got errors in log:
[ActionCable] [Admin] [lvh.me] There was an exception - NoMethodError(undefined method `config' for nil:NilClass) [ActionCable] [Admin] [lvh.me] /Users/alabama/.rbenv/versions/2.4.4/lib/ruby/gems/2.4.0/gems/lograge-0.11.0/lib/lograge.rb:228:in `lograge_config' /Users/alabama/.rbenv/versions/2.4.4/lib/ruby/gems/2.4.0/gems/lograge-0.11.0/lib/lograge/rails_ext/action_cable/connection/base.rb:24:in `block in handle_close' /Users/alabama/.rbenv/versions/2.4.4/lib/ruby/gems/2.4.0/gems/activesupport-<IP_ADDRESS>/lib/active_support/notifications.rb:166:in `instrument' ...
...
If add to development.rb config.lograge.enabled = true - errors disappear, but if no need "lograge" in development mode and add config.lograge.enabled = false - errors still going on.
Sorry for my english.

@alabamaair, thanks for reporting this. Will post a PR with a fix today


desired version is being ignored

Hey,
I am tryiing to use a specific version of node, which is exactly what this package is used for.
steps:
      - uses: actions/setup-node@v2
        with:
            node-version: '12.8.1'
      - uses: actions/checkout@v2
      - uses: bahmutov/npm-install@v1
      - run: npm run build
      - uses<EMAIL_ADDRESS>
but then, still getting the error:
npm WARN EBADENGINE Unsupported engine {
npm WARN EBADENGINE   package<EMAIL_ADDRESS>npm WARN EBADENGINE   required: { node: '>=12 < 15' },
npm WARN EBADENGINE   current: { node: 'v15.11.0', npm: '7.6.0' }
npm WARN EBADENGINE }

so it seems like, the desired version is getting ignored completely. is this a bug, or am i missing something?
workflow

If you look at the output from setup-node, I suspect you'll see that the correct version is reported.
I suspect bahmutov/npm-install is the culprit.


[Feature] Add search ability

Instead of having just an add field, it could be a add/search field so that when a user enters text, if it's found, results are displayed so the user can mark it as complete or incomplete. If it's not found, it gets added to the list.

Might be a good idea for a future release :)

Agreed, right now it's too easy to add duplicate items to long lists. Apps with similar UI/UX that have this feature

https://github.com/dsoulayrol/android-sholi
https://github.com/openintents/shoppinglist


cleanup python env var checks examples

Description of change
Used a list of env vars to iterate through instead of checking them individually
Links to any relevant issues
fixes #1358
Type of change

Bug fix (a non-breaking change which fixes an issue)
Documentation Fix

How the change has been tested
Ran the unit tests and ran some of the examples in the terminal
Change checklist
Tick the boxes that are relevant to your changes, and delete any items that are not.

[x] I have followed the contribution guidelines for this project
[x] I have performed a self-review of my own code
[x] I have checked that new and existing unit tests pass locally with my changes

Thanks :)


🚧 wip

Story
⚠ THIS WORK WILL LIKELY BE DONE IN HYKU NOW that we've contributed back OER/ETD

https://github.com/samvera/hyku/pull/2237

https://github.com/samvera/hyku/pull/2237


Script failed when 'runner-results' folder not exist or if not empty

Hi,
in the start script, there is a line code that remove the 'runner-results' folder but the script throw an exception in 2 cases:
-> when this folder not exist
-> when this folder exist and it contains files
So before run the cmd, i must clean the folder or create a empty 'runner-results' folder. Could you handle this exception?

cypress-parallel -s cy:run -t 4 -d cypress/integration/ -p cypress-report-config.json

(node:30920) UnhandledPromiseRejectionWarning: Error: ENOENT: no such file or directory, rmdir 'C:\Personal_Unsaved\Projet\front-end\runner-results'
at Object.rmdirSync (fs.js:684:3)
at cleanResultsPath (C:\Personal_Unsaved\Projet\front-end\node_modules\cypress-parallel\cli.js:19:6)
at start (C:\Personal_Unsaved\Projet\front-end\node_modules\cypress-parallel\cli.js:23:3)
at Object. (C:\Personal_Unsaved\Projet\front-end\node_modules\cypress-parallel\cli.js:124:1)
at Module._compile (internal/modules/cjs/loader.js:778:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:789:10)
at Module.load (internal/modules/cjs/loader.js:653:32)
at tryModuleLoad (internal/modules/cjs/loader.js:593:12)
at Function.Module._load (internal/modules/cjs/loader.js:585:3)
at Function.Module.runMain (internal/modules/cjs/loader.js:831:12)
(node:30920) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:30920) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Hey @PBIIBP . I'm having the same problem and actually even if I create the empty runner-results folder in the project's root dir, it's being deleted and then this error occurs.
Did you managed to overcome this issue somehow?

changing the cleanResultsPath method in node_modules/cypress-parallel/cli.js to the following did the trick for me:
function cleanResultsPath() {   if(!fs.existsSync(resultsPath)) {   	fs.mkdirSync(resultsPath, { recursive: true })   } else {     fs.readdir(resultsPath, (err, files) => {             if (err) console.log(err);             for (const file of files) {                 fs.unlink(path.join('path here', file), err => {                     if (err) console.log(err);                 });             }         });   } }

@PBIIBP  May be issue is with your cypress-report-config.json file. Can you share the json file here ?

I encountered the same issue with version cypress-parallel 0.9.0 and cypress 9.4.1
project structure: project/cypress/integration/folder/test.spec.ts
command: npx cypress-parallel -s dev -t 2 -d **/user.spec.ts
Would be great if you can have a look @ISanjeevKumar
`node:fs:1390
handleErrorFromBinding(ctx);
^
Error: ENOENT: no such file or directory, scandir 'C:\Users\user\dev\project\runner-results'
at Object.readdirSync (node:fs:1390:3)
at collectResults (C:\Users\user\dev\project\node_modules\cypress-parallel\utility.js:35:26)
at start (C:\Users\user\dev\project\node_modules\cypress-parallel\cli.js:32:23)
at processTicksAndRejections (node:internal/process/task_queues:96:5) {
errno: -4058,
syscall: 'scandir',
code: 'ENOENT',
path: 'C:\Users\user\dev\project\runner-results'
}`

Looks like issue with the way you are executing the script . If you can share  what is inside dev script , then may be I can help.

I understood the problem now. it is getting failed because :
When none of the test case has got executed then report folder will not get created by 'json-stream.reporter', Hence you are getting this error.
Command which you are using looks not correct to me.  Please use something like this '"node_modules/.bin/cypress-parallel -s dev  -t 2-d '**/user.spec.ts'.

HI @ISanjeevKumar, sorry for the late response to your suggestion.

When none of the test case has got executed then report folder will not get created by 'json-stream.reporter', Hence you are getting this error.

I can confirm if none test cases are found this error will occour. In our case tests where executed but still the error will occour.

Command which you are using looks not correct to me. Please use something like this '"node_modules/.bin/cypress-parallel -s dev -t 2-d '**/user.spec.ts'.

This is exactly what npx does from my understanding. https://www.npmjs.com/package/npx
I also double checked and with both execution ways the error occours.

@nils-hoyer nils-hoyer
you only copy'n'pasted the runner config to a new file?
{     "reporterOptions": {         "mochaFile": "tests/junitresults_integration-[hash].xml",         "toConsole": true,         "attachments": true     } }
i tried but no luck. thats the script in package.json:
cypress-parallel -s cy:run -t 2 -d cypress/e2e/one/* -a '\"--browser electron --headed  --reporter junit --config video=false\"' -r cypress-multi-reporters -o 'configFile=reporter-config.json

Sorry, doesn't seems to work for me. Even if I remove the reporter config complete.
ENOENT: no such file or directory, scandir  ClientApp\node_modules\cypress-parallel\utility.js:35:26)

if you upload an example repo reproducing the error I can have a quick look on it.

I have this somewhat similar issue. Unfortunately i cannot share the repo is a customer project, but i run the build without parallel it works all fine on jenkins but everytime i try it out with parallel i get this error.
node:fs:1405
  handleErrorFromBinding(ctx);
  ^

Error: ENOENT: no such file or directory, scandir '/home/jenkins/slave/workspace/some-project-folder/runner-results'
    at Object.readdirSync (node:fs:1405:3)
    at collectResults (/home/jenkins/slave/workspace/some-project-folder/node_modules/cypress-parallel/utility.js:35:26)
    at start (/home/jenkins/slave/workspace/some-project-folder/node_modules/cypress-parallel/cli.js:32:23)
    at processTicksAndRejections (node:internal/process/task_queues:96:5) {
  errno: -2,
  syscall: 'scandir',
  code: 'ENOENT',

this started to happen after cypress 10 upgrade, but works well with cypress 9

I am facing the same problem

Same problem. Will check up on the PR linked above.


Scoped styles not working on SSR

Version
2.4.1
Reproduction link
https://github.com/vuejs/vue-hackernews-2.0/issues/229
Steps to reproduce

What is expected?
the h1 in red color
What is actually happening?
You wont see the h1 in red when loading from server, insted if you load the page throught the router you will see

Can you provide an actual reproduction? The hackernews repo is running on 2.4.1 and using scoped styles, and it renders with styles from the server, so what's the eel scenario that is not working for you? Please fork the repository and edit it so it demonstrates the issue.

This is caused by a bug in vue-loader and should be resolved after you upgrade vue-loader to 12.2.2 or 13.0.2.


[solarized][markdown] bold text in Markdown looks faint in Solarized theme.

Looks caused by
https://github.com/codemirror/CodeMirror/blob/master/theme/solarized.css#L90

Do I understand correctly that this provides too low contrast in the light variant of the theme? Would simply removing that line be an acceptable solution?

Ah... yes, this was in regard to the solarized light theme. Interesting; in the dark version, it seems that removing the line could theoretically make it hard to tell the difference between bold and not bold.
But it's worth noting that the dark theme example in http://ethanschoonover.com/solarized for markdown (http://ethanschoonover.com/solarized/img/screen-pandoc-dark.png) does seem to suggest that it should be possible to make a dark variant that doesn't require a color change. IIUC, the philosophy for solarized is that the foreground colors shouldn't need to change when switching between light and dark variants. It doesn't look like #eee is part of the solarized palette, though.

Actually, after playing with it more I'd argue it should be fine to simply remove the line.


Number of elastic pools per logical server/region/subscription

I am told through internal communication that there are limits on the number of elastic pools that can be created per logical server, per region, and per subscription. I don't see those documented anywhere - is that something that can be added to this document? Thanks!

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

ID: ece92a7d-3250-d109-97e9-20a731dc4b20
Version Independent ID: 29a97076-fc64-0ae9-4574-de1aeda237d8
Content: Azure SQL Database vCore-based resource limits - elastic pools
Content Source: articles/sql-database/sql-database-vcore-resource-limits-elastic-pools.md
Service: sql-database
GitHub Login: @CarlRabeler
Microsoft Alias: carlrab

@dcbrown16 Thanks for bringing this to our attention. Your feedback has been shared with the content owner for further review.

@dcbrown16 merged - https://github.com/MicrosoftDocs/azure-docs-pr/pull/47960
#please-close


Cant do google login if 2fa is enabled.

Creating Google session for XXXXXXX
Starting new HTTPS connection (1): android.clients.google.com
Starting new HTTPS connection (1): android.clients.google.com
Access token not generated
Session not created successfully

The solution is to generate a app password here. https://security.google.com/settings/security/apppasswords


Update Showcase item Haltscam, including rename to Profiler

Checklist

[x] I have read the How to Contribute.
[x] I have run yarn build after adding my changes without getting any errors.

Updating documentation or Bugfix
We have rebranded our project from HaltScam to Profiler to better align with our evolving mission and ambitions. As our project expands, Profiler now offers more than just crowd-sourcing risky Cardano addresses. As an advanced Cardano explorer centered on wallets, Profiler empowers users to thoroughly investigate their wallets, providing a comprehensive overview of their transactions, tokens, and NFTs, accompanied by engaging and interactive data visualizations.

Thanks for your detailed feedback

Regarding haltscam.com domain is not in use anymore our new app moved to https://www.profiler.biz
There is no blacklisting the web app should be available everywhere
There are already some measurements in place to prevent the website from being scraped or probed

Did you test profiler.biz?
I will update the description and thumbnails image for the showcase

ugh, you're right: I read the diff wrong & was using the old domain. 🤪  I'm not seeing any page load or asynchronous problems on the new domain... therefore it looks like only items 3 & 4 above are valid and I'll look forward to checking again after your updates. 😎

😀, ok I pushed a new commit, I hope the thumbnails is fine now.

thanks @rdlrt - I'm still working on crippled Internet half the day so maybe my Refresh button on Files page didn't work... 🤔
Can you please describe what you did above where it says "force-pushed"?  It's different than what I recommended to people here and I would like to understand any other options.

Thanks @rdlrt but this is still not explicit enough for me to understand what you're doing.  Even after all this time, some of this language and procedure about git means nothing to me.  For instance I can't see any way to remove yarn.lock from any commit without removing it from the branch.
Maybe just this once please you could run through it all from the beginning: what you're doing locally in order to 1) modify someone else's PR and then 2) delete a file from that local branch without deleting it from the repository?  I've asked both these questions on Discord and GitHub a few times each and honestly I'm still improvising when it comes to a few of these things.

Is any action needed from my side?


mega_gallery has many analyzer errors

I happened to be analyzing the entire flutter repo when I ran the tests and it noticed a bunch of errors in the mega_gallery. Is that expected?

It's not. I'll validate that it's still happy. You may have an old copy; you may want to rm -rf dev/benchmarks/mega_gallery/ or dart dev/tools/mega_gallery.dart.

A new copy shows as clean for me:
Analyzing 835 files...
No analyzer warnings! (ran in 23.1s; 56 public members lack documentation)

so you may want to just delete your copy.

Odd, I had cleaned my whole repo and the errors appeared when I ran the tests. I'll close this and file a new one if I notice this again and have more data.


How do I get the URL of the upload?

Hello.
I'm using your library and I uploaded the image. I want to get the URL in my local, how do I do that?

@DanielSalazar1
I'm using URL.createObjectURL - see code below
onDrop(picture) {     if (picture[0]) {       const newPicURL = URL.createObjectURL(picture[0]);       this.setState({         picture: picture, // This is the object         newPicURL: newPicURL, // This is the local URL       }, () => this.props.callbackHandler("SELECT_IMAGE", picture));     }   }
You can then use newPicURL as the source of the image within render; I'm doing this
const pic = this.state.picture.length > 0 ? this.state.newPicURL : this.props.picture;


⚠ Bofrost.be has degraded performance

In 63910db, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1030 ms

Resolved: Bofrost.be performance has improved in 3b3f04a.


Cannot find release

I am unable to find a release on the plugin website. Are you sure it is there?

Hi! Sorry about this, it took a while for the plugin to go through the review process. It's now available at https://grafana.com/grafana/plugins/bsull-console-datasource/, but currently only works with Grafana 8.4 - I need to issue a new release (and wait for review) for it to work with Grafana 9.


Refactor into vba-test

vba-test is a new approach to VBA-TDD that simplifies the API and allows for future work on more powerful assertion messages and integration with vba-blocks.
You can view a mockup of the new API here: vba-test/readme
The underlying changes are a complete rename and refactor, although the approach is generally already in use in v2.0.0-beta (with an evented suite). It may be a burdensome upgrade to map .Expect(A).ToEqual B to .IsEqual A, B, but I think it will be a welcome simplification and opens up new avenues in the future.

Ok, this is ready to merge, here is a short upgrade guide:

Dim Specs As New SpecSuite to Dim Tests As New TestSuite
Specs.It("...") to Tests.Test("...")
.Expect(A).ToEqual B to .IsEqual A, B
.Expect(A).ToNotEqual B to .NotEqual A, B
.Expect(A).ToBeUndefined to .IsUndefined A
.Expect(A).ToNotBeUndefined to .NotUndefined A
.Expect(A).ToBeNothing to .IsOk A Is Nothing
.Expect(A).ToNotBeNothing to .NotOk A Is Nothing
.Expect(A).ToBeEmpty to .IsOk IsEmpty(A)
.Expect(A).ToNotBeEmpty to .NotOk IsEmpty(A)
.Expect(A).ToBeNull to .IsOk IsNull(A)
.Expect(A).ToNotBeNull to .NotOk IsNull(A)
.Expect(A).ToBeMissing to .IsOk IsMissing(A)
.Expect(A).ToNotBeMissing to .NotOk IsMissing(A)
.Expect(A).ToBeLT B to .IsOk A < B
.Expect(A).ToBeLTE B to .IsOk A <= B
.Expect(A).ToBeGT B to .IsOk A > B
.Expect(A).ToBeGTE B to .IsOk A >= B
.Expect(A).ToBeCloseTo B, C to .IsApproximate A, B, C
.Expect(A).ToNotBeCloseTo B, C to .NotApproximate A, B, C
.Expect(A).ToContain B to .Includes A, B
.Expect(A).ToNotContain B to .NotIncludes A, B
.Expect(A).ToMatch B to .IsOk InStr(B, A)
.Expect(A).ToNotMatch B to .NotOk InStr(B, A)

Some of the previous matchers will most likely be added back at some point in the future, but for now I'm happy with the simplified API.

Closes #11
Closes #12
Closes #13
Closes #16
Closes #17
Closes #22


post about GraphStage, the Ultimate

Post by @drewhk
Illustrations by @ktoso

Merging as per agreement with Endre yesterday :)

Happy weekend reading hakkers!


Ignore certain files

[x] This feature is not on the latest version

Request
I want to be able to ignore sertin files from the asset pipeline.
Examples
For expample I have a folder where images are located together with some JSON metadata. The metadata ends up in the asset dir. I want to be able to ignore certain file by specifying a settting like: ignore: "*.json" or `ignore: "samples/*.json"

We don't compile, or process, or even check for anything you do not explicitly request via our helpers. So there is no need to ignore any files, because even if stuff you do wish to include is inside of folders full of stuff you do not wish to include, we will not notice, or care about the files you don't ask us to process.  Can you further elaborate your use case or why?

Yes, I'm building a "scafolding" (or a framework or whatever you want to call it) for people to do step by step instruction. The instruction details are specified in json files and later put together by jekyll. An instruction can have an image associated with it. For the purpouse of the framework I require users to add an image with the same name as an instruction step. This way they can easily track that everything is ok. (As soon as I will move images into another dir someone will 100% misstype one lettsr and start complaing). jekyll-assets is super nice because I don't have to care about the format of the image or whether it is present. But as the JSON files are used to construct the website they are also move to the assets folder (which is not totaly bad, but it is definetely unneeded).

Another use case ... consider a git repo which contains the _config.yml file, _posts/ directory, and so forth ... in the root of the repo. I'd like to have a README.md file in the root of the repo, so that when somebody looks at that repo in Github or some other web interface, I'll have a way to explain how to set up Jekyll on their workstation and what the normal workflow looks like, without that README.md becoming part of the generated site.
Unless the "standard" way to use Jekyll is to have the "Jekyll root" (where the _config.yml file is) be a sub-directory within the git repo, and that's just not clearly explained anywhere?

Update: after leaving the comment aboive I found that, if I use the word "exclude" instead of "ignore", the search results are a LOT more helpful, and they pointed me to the documentation.
I just added the following to my site's _config.yml file:
exclude:
  - README.md

... and now when it generates the site, it doesn't include the README.md file in the output.
This solved the problem I ran into, maybe it can solve this issue as well?
Also, it might be useful to find a way to add the word "ignore" in the documentation when talking about the exclude list, to make it easier for people who happen to choose the wrong word when searching for information about this.


Recipe: Create Cloud Native Example/Tutorial

Create a recipe/example/tutorial for adding telemetry to an application

I can take this.

@narmathabala and @mahomedalid - last comment on this was that mahomedalid would take this issue and work on it (on Feb 2020) -- is this still an active task or was this already solved but the task was accidentally left open?

Closing for now, will be added as needed


LSNBLDR-499; backout and reimplement LEAP

Please do not commit yet. It needs a change to portal

need an update


change build to check

Related issue(s)

Resolves https://github.com/noir-lang/noir/issues/657
Description
Summary of changes
change the nargo build cli command to nargo check

Dependency additions / changes

Test additions / changes

Checklist

[x] I have tested the changes locally.
[x] I have formatted the changes with Prettier and/or cargo fmt with default settings.
[x] I have linked this PR to the issue(s) that it resolves.
[x] I have reviewed the changes on GitHub, line by line.
[x] I have ensured all changes are covered in the description.

Additional context

Will we need to update any commands in the book when this change lands? cc @Globallager

Thanks for the review sir 🙂

Thanks for the review sir 🙂

@jfecher hmm should the driver function name to be changed to check as well or leave it as it build https://github.com/noir-lang/noir/blob/8c1e101a7a6e50d0a70bea5ba3ab15e53e006a78/crates/nargo/src/cli/check_cmd.rs#L28
https://github.com/noir-lang/noir/blob/348ca5dd3f1cca39ca4c89ac7520ad61bbebb940/crates/noirc_driver/src/lib.rs#L132

It can be updated but there is no rush


url_for() doesn't return a working URI for a blueprint route with host

Describe the bug
When i use blueprints with host argument and try to get url with url_for i get an incorrect url - blueprint's host is used as a path, and not as a hostname
Code snippet
from sanic import Sanic, Blueprint
from sanic.response import text

bp = Blueprint('bp_app', host='bp.example.com')

@bp.route('/', name='index')
async def bp_index(request):
    url = request.app.url_for('bp_app.index')
    return text(url)

@bp.route('/internal', name='internal')
async def bp_index(request):
    url = request.app.url_for('bp_app.internal')
    return text(url)

@bp.route('/external', name='external')
async def bp_index(request):
    url = request.app.url_for('bp_app.external', _external=True)
    return text(url)

app = Sanic('app_name')
app.blueprint(bp)

if __name__ == '__main__':
    app.run(port=8000)

Output
# 1
$ curl -H "Host:bp.example.com" http://<IP_ADDRESS>:8000/
bp.example.com

# 2
$ curl -H "Host:bp.example.com" http://<IP_ADDRESS>:8000/internal
bp.example.com/internal

# 3
$ curl -H "Host:bp.example.com" http://<IP_ADDRESS>:8000/external
http:///bp.example.com/external

1 example returns hostname by like a path
2 example returns hostname and path but it's like a path
3 example return full url, but with no hostname (3 slashes)
Expected behavior
I'm expecting correct urls.
For internal urls - only path returned as stated in route.
For external urls - fully qualified domain name with full path
1 example - /
2 example - /internal
3 example - http://bp.example.com/external
Environment (please complete the following information):

OS: macOS
Version: 19.12.2

Additional context
Can't get correct place, but found some places, where it can be
app.py#L829-L832,
app.py#L848 (perhaps, blueprint's host should be used, and config's server name as default)

Thanks for reporting and triaging this!


Review relation between providers and RouteDelegate

Review relation between [RiverpodNavigator] <=> [TypedPathNotifier]<=> [RiverpodRouterDelegate] <=> [ExampleApp]
see appFlutter.dart
...
  final navigator = ref.read(exampleRiverpodNavigatorProvider);
  final delegate = RiverpodRouterDelegate(navigator, pageBuilder: _pageBuilder, initPath: [HomeSegment()]);
  ref.listen(typedPathNotifierProvider, (_, __) => delegate.notifyListeners());
...

could help:
 final navigator = ref.read(exampleRiverpodNavigatorProvider);

waiting for https://github.com/rrousselGit/river_pod/issues/1113

solved


powershell-linux binaries support for openssl 3.0

Package used: powershell-linux-7.0.0
powershell-linux$ grep -r "ERR_put_error" .
Binary file ./System.Security.Cryptography.Native.OpenSsl.so matches
Binary file ./System.Security.Cryptography.Native.OpenSsl.a matches
ERR_put_error which has references in powershell-linux binaries is deprecated in openssl 3.0
Reference: https://www.openssl.org/news/changelog.html

I believe this should be logged with .Net itself (it probably already is). I’m not aware of any direct OpenSSL deps that PowerShell has so there’s not much they can do here.

I believe this should be logged with .Net itself (it probably already is). I’m not aware of any direct OpenSSL deps that PowerShell has so there’s not much they can do here.

This has and i was thinking this is a powershell release, please confirm:
https://github.com/PowerShell/PowerShell/releases/download/v7.0.0/powershell-7.0.0-linux-x64.tar.gz

Please re-check with latest 7.0.3 version.
If the issue persists - we can do nothing in the repo - open issue in .Net Runtime repository.
PowerShell 7.0 is based on .Net 3.1.


Datapack issue with JAOPCA

There is a datapack incompatibility between the latest version of Occultism and JAOPCA. I reported it both here and on the other mod's issue tracker. When creating or loading a world the "Datapack loading error"  screen appears. I've narrowed it down to these two mods.

Mincraft Version: 1.16.5
Forge Version: 36.1.61
Occultism Version: 1.7.0
JOAPCA Version: <IP_ADDRESS>

Can confirm, and JAOPCA doesn't need to be there for it to happen. The "Datapack Loading Error" happens with just Occultism. v1.7.0 and Forge version doesn't matter. I tried several different versions between 36.1.32 and 36.2.0.

@Rensik thank you for reporting - can you provide the log files?

In the interest of getting this figured out more quickly, I did a bit of testing in a standalone instance.
I was unable to reproduce the datapack error screen with nothing but Occultism and its dependencies installed, but as soon as I added in JAOPCA the error began to happen upon trying to create a world.
I would hazard a guess that another mod @Rensik has installed is somehow causing the issue to occur for a similar reason to JAOPCA, but can only speculate on that since I don't know if they have other mods installed/what they are.
Here is the latest.log from just booting up the game and trying to create a new world:
https://gist.github.com/Febilian/3d86f120b1935ebd1336f229b675343e
Forge 36.2.0
Occultism 1.7.0
Curios 1.16.5-<IP_ADDRESS>
Patchouli 1.16.5-53.1
JAOPCA <IP_ADDRESS>

I work nights and just checked my email, so I do apologize for not getting you the log. If you still need one from me I can get it uploaded in the morning as soon as I get home.

Thanks guys! Based on the crashlog JAOPCA's occultism compatibility layer has a problem - most likely due to a new setting in the crushing recipes (that's just a guess, but the crash happens in the crushing compat, and there was a change in 1.7.0)

Will be fixed in the next JAOPCA release, see https://github.com/TheLMiffy1111/JAOPCA/issues/204#issuecomment-886240476


[#1967, #1964, #1960, #1959, #1956, #1952, #1951, #1943, #1939, #1936, #1934, #1925, #1927, #1915, #1897, #1892, #1890, #1886, #1885, #1869 ] feat: Missing private constructor

What changes were proposed in this pull request?
Utility classes were updated with private constructors.
Why are the changes needed?
Utility classes contain only static members that we group together around a specific topic. Thus, the classes themselves are stateless, while their members contain code meant to be reused across several layers.

Resolves: #1967, #1964, #1960, #1959, #1956, #1952, #1951, #1943, #1939, #1936, #1934, #1925, #1927, #1915, #1897, #1892, #1890, #1886, #1885, #1869

cool, maybe you can change the title 'feat' to 'improvement' ?

cool, maybe you can change the title 'feat' to 'improvement' ?

Just changed...
Should I also run ./gradlew :common:spotlessApply ? I've seen this being referenced in another PR.

Thanks for your contribution. There is no need for the comment "// private constructor" as it doesn't add anything. Also, there's no need to throw an exception, as a private constructor can't usually be called from outside the class.
You should run  ./gradlew build -x test, and if there are any issues with the code formatting you may need to run spotlessApply.

Can you please change the PR tile to like "[#xxx][#xxx][#xxx]..." to follow our convention. Also please add the Fix: #xxx in the PR description one by one line, which will close the related issues when this PR is closed. You can follow what other PR did.

@codeHysteria28
Can you format your code (./gradlew :api:spotlessApply) to fix CI failure?

@codeHysteria28 Can you format your code (./gradlew :api:spotlessApply) to fix CI failure?

yes will do !

Tests all build successfully and also formatting should be fixed. Pushing the changes to this PR.

LGTM

Thanks @codeHysteria28 for your contributions.


fixUpdate external Brown test case out file.

<--Describe your changes.-->
Update external Brown test case out file to match the new precision.
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


E220-400T22S Having Error on Serial1

Hi Dear,
I used this library for working on module Ebyte E220-40022S. I am getting compilation error.

Arduino: 1.8.19 (Windows Store <IP_ADDRESS>) (Windows 10), Board: "Arduino Uno"
C:\Users\Krishan Kumar\Documents\Arduino\libraries\E220_900T22S_JP-1.0.0\example\test1\test1.ino: In function 'void setup()':
test1:14:5: error: 'Serial1' was not declared in this scope
     Serial1.begin(9600);
     ^~~~~~~
C:\Users\Krishan Kumar\Documents\Arduino\libraries\E220_900T22S_JP-1.0.0\example\test1\test1.ino:14:5: note: suggested alternative: 'Serial'
     Serial1.begin(9600);
     ^~~~~~~
     Serial
exit status 1
'Serial1' was not declared in this scope
This report would have more information with
"Show verbose output during compilation"
option enabled in File -> Preferences.

Please help me to solve the issue. Thanks

Hi,krishank652,
Thank you for using this library. You use arduino UNO R3 , right? Arduino UNO, atmega328 has only one UART, that is talking with  USB-devices. Then "Serial1" is not avairable. It is good to use software serial.

See below if you know about serial.

https://www.arduino.cc/reference/en/language/functions/communication/serial/

See below if you know about software serial.

https://docs.arduino.cc/learn/built-in-libraries/software-serial/

Thanks for a quick reply!
I am using Arduino UNO, I am a beginer in this. Can you please tell me how can i implement in the code?

Dear @nihinihikun ,
Please help me How can I implement software serial. Give me idea to implement it. I read supportive documents as your shared. But I don't know How can I do it.
Can you tell me please, In library where I need to update it.
Thanks

Dear @krishank652
Sorry for the late reply. The previous version of this library supported only Serial1. I've updated the library to support both SoftwareSerial and HardwareSerial. Please install version 2.0.0. You can now use the E220 with an Arduino UNO by following the example in softwareserialuse/softwareserialuse.ino.
Thanks

Thanks for the updation of library.
I tried it and updated pin no.: SoftwareSerial mySerial(2, 3); // RX, TX . Found some issue.
Error occur while compilation:
Arduino: 1.8.19 (Windows Store <IP_ADDRESS>) (Windows 10), Board: "Arduino Uno"
C:\Users\Krishan Kumar\Documents\Arduino\libraries\E220-900T22S-JP_Arduino-main\src\config_E220.cpp: In member function 'int config_E220::ReadResister(byte, byte, byte*)':
C:\Users\Krishan Kumar\Documents\Arduino\libraries\E220-900T22S-JP_Arduino-main\src\config_E220.cpp:29:13: error: 'Serial1' was not declared in this scope
     _length=Serial1.write(set_data, 3);
             ^~~~~~~
C:\Users\Krishan Kumar\Documents\Arduino\libraries\E220-900T22S-JP_Arduino-main\src\config_E220.cpp:29:13: note: suggested alternative: 'Serial'
     _length=Serial1.write(set_data, 3);
             ^~~~~~~
             Serial
C:\Users\Krishan Kumar\Documents\Arduino\libraries\E220-900T22S-JP_Arduino-main\src\config_E220.cpp: In member function 'int config_E220::WriteResister(byte, byte, byte*)':
C:\Users\Krishan Kumar\Documents\Arduino\libraries\E220-900T22S-JP_Arduino-main\src\config_E220.cpp:57:5: error: 'Serial1' was not declared in this scope
     Serial1.write(set_data, _length+3);
     ^~~~~~~
C:\Users\Krishan Kumar\Documents\Arduino\libraries\E220-900T22S-JP_Arduino-main\src\config_E220.cpp:57:5: note: suggested alternative: 'Serial'
     Serial1.write(set_data, _length+3);
     ^~~~~~~
     Serial
C:\Users\Krishan Kumar\Documents\Arduino\libraries\E220-900T22S-JP_Arduino-main\src\config_E220.cpp: In member function 'int config_E220::SetTemporaryResister(byte, byte, byte*)':
C:\Users\Krishan Kumar\Documents\Arduino\libraries\E220-900T22S-JP_Arduino-main\src\config_E220.cpp:83:5: error: 'Serial1' was not declared in this scope
     Serial1.write(set_data, _length+3);
     ^~~~~~~
C:\Users\Krishan Kumar\Documents\Arduino\libraries\E220-900T22S-JP_Arduino-main\src\config_E220.cpp:83:5: note: suggested alternative: 'Serial'
     Serial1.write(set_data, _length+3);
     ^~~~~~~
     Serial
C:\Users\Krishan Kumar\Documents\Arduino\libraries\E220-900T22S-JP_Arduino-main\src\config_E220.cpp: In member function 'void config_E220::SetDefault()':
C:\Users\Krishan Kumar\Documents\Arduino\libraries\E220-900T22S-JP_Arduino-main\src\config_E220.cpp:100:5: error: 'Serial1' was not declared in this scope
     Serial1.write(default_config_data, 11);
     ^~~~~~~
C:\Users\Krishan Kumar\Documents\Arduino\libraries\E220-900T22S-JP_Arduino-main\src\config_E220.cpp:100:5: note: suggested alternative: 'Serial'
     Serial1.write(default_config_data, 11);
     ^~~~~~~
     Serial
exit status 1
Error compiling for board Arduino Uno.

This report would have more information with
"Show verbose output during compilation"
option enabled in File -> Preferences.

Sorry, I overlooked the fact that the Arduino IDE compiles all library files regardless of their relation. I fixed config_E220.cpp. As a result, you should now be able to compile successfully with version 2.0.2."

Thank you, I successfully compiled the SoftwareSerial example. Much appreciated.
My objective is to send data to a receiver device with a unique address, ensuring that only that specific device can receive the message. Could you please advise on how to implement addressing in the code for both the transmitter and receiver devices?

For a setup with two devices (1 transmitter and 1 receiver), should I assign the same address to both sides, or is it necessary to define different addresses for the transmitter and receiver?
In the scenario of having two or more devices, how can I address and establish communication among them? (e.g., 2 transmitters and 1 receiver)
Do I need to configure key parameters using the Ebyte software, as shown in the screenshot below?  I confused because I saw in the program there is a addressing.

Your assistance is greatly appreciated. Thank you.

I don't know about E220-400T22S/D,but I think generally, radio module should have own id.  (Please see the datasheet)

I believe you've created a dedicated library for the E220-900T22S module. Both my module and the library module are identical, except for the frequency difference. Each radio module has its own ID or key, which we can configure using the software screenshot I provided.
My only question is, how can I instruct the transmitter device to send data only to that specific device?


Text bleed in notifications

Description
Text should wrap; not bleed/overflow outside of the notification box.

@bciar Thx!


Add vscode-rust fork of RustyCode

It has a 35 commits after RustyCode so I think it is fair to include it.

Thank you

Thank you


Enumerate Icon API

I'd like an API for enumerating icons so that I can build better property validation.
Example of what I'd like to do:
// assuming Item is a react component:

Item.propTypes = {
  icon: PropTypes.oneOf(['icon1', 'icon2'])
}

<Item icon="random-icon" />

// Will log an error to the console because random-icon is not icon1 or icon2.

Possible solution: Expose Icons in public API
import { Icons } from 'hig.web';

Object.keys(Icons);

This would let me do the following: PropTypes.oneOf(Object.keys(Icons)
@recyclerobot thoughts?

Yes should do


Default to transparent title bar (--title-bar-style hidden / hiddenInset) on macOS

[x] I looked at nativefier --help and https://github.com/jiahaog/nativefier/blob/master/docs/api.md , no existing option fits my needs.
[x] I searched existing issues, open & closed. Yes, my feature request is new.
[x] I'm using the latest version available at https://github.com/jiahaog/nativefier/releases

Problem statement
After Build, the App has the default OS Titlebar which is not bad, but It would be much better to be transparent & merge with whole window.
Suggested solution
The is an electron app built in Flotato app which transparents the default Titlebar.

Context

Nativefier: 10.1.0
Node.js: v14.5.0
Npm: 6.14.5
OS: macOS X Catalina 10.15.4

@sirpooya from the initial title of your issue, I'm not sure if you know this is already supported (see api / --title-bar-style), or if you do and what you're asking for is to change the defaults.
I like the idea, but I don't have a Mac to fiddle with it and see how it behaves. I'll happily merge a PR checking these three boxes:

Should we use hidden or hiddenInset or hiddenInset
How do we make sure the window is always draggable? Our API (written by the implementer of --title-bar-style) notes you need custom, site-specific CSS for that. Do we really?

And once these are clear,

Identify a dozen Nativefier Mac active users (look at macos-tagged issues and see commenters) and @mention them asking for their feedback and potential undesirable side-effects

Hi @sirpooya did you get it to work? If so how?
Cheers

I think this can be cloes due to:
nativefier --name "Google"  https://google.com --title-bar-style 'hiddenInset' --inject ./style.css
style.css content:
`
body {
-webkit-app-region: drag;
}
/* make class elements draggable  */
.aside-brand-wrapper {
-webkit-app-region: drag;
padding-top: 5px;
}
div#kt_header {
-webkit-app-region: drag;
}
/* make element with ID not draggable  */
div#kt_aside_menu_wrapper {
-webkit-app-region: no-drag!important;
}
div#kt_content {
-webkit-app-region: no-drag!important;
}
`


Parameter routes routing issue

Hey,
I am facing an issue with parameter based routes on latest version of uWebsockets.js (v20.40.0),
To illustrate the problem, I have created an example app below:
const uWS = require('uWebSockets.js');

const app = uWS.App()
  .get('/something/:id/sync', (res, req) => {
    res.end('sync endpoint');
  })
  .get('/something/:somethingId/pin', (res, req) => {
    res.end('pin endpoint');
  })
  .get('/something/:id/:attribute', (res, req) => {
    res.end('attribute endpoint');
  })
  .listen(9000, (token) => {
    if (token) {
      console.log('Listening to port ' + 9000);
    } else {
      console.log('Failed to listen to port ' + port);
    }
  });

In this application, Request GET /something/1234/pin returns attribute endpoint as result while expected response is pin endpoint.
I went through the routing strategy of uWebsockets which mentions that routes are matched in the order of specificity which implies that pin endpoint should be the above response.
Also, if :somethingId is changed to :id, routing works as expected. Would like to understand if this is a bug with uWebsockets.
Thank you!

Stellar bug report, it is fixed and tested in https://github.com/uNetworking/uWebSockets/commit/fc2b7be765983fbce8a3839bc3ff16b51c81b44b

Try https://github.com/uNetworking/uWebSockets.js/releases/tag/v20.41.0


Version 5.0.1 cannot display https requests, 4.04 can display https requests

Problem Description
Version 5.0.1 cannot display https requests, 4.04 can display https requests
v5.01 I tried for a long time，v5.0.1 need additional configuration?
Steps to reproduce the behavior:
System Information
v5.0.1 linux

Thanks for getting in touch. We're generally happy to help out, but can I please ask you to use StackOverflow for all support and how-to questions? You can post the link to your question here so that people can follow up.
Thanks!


Failed to Export Identity

codesigndoc successfully found the required Identities/Certificates and Provisioning Profiles, but the Identities/Certificates export failed.
Error:
INFO[19:17:04]  * Exporting Identity: iPhone Developer: XYZ (XYZ)
FATA[19:17:04] Failed to Export Identity: FindIdentity: failed to get 'labl' property: getCFDictValueUTF8String: failed to convert value to string

Fixed in https://github.com/bitrise-tools/codesigndoc/pull/2


build: Make experimental packages compatible with moduleResolution: node16

Updates the packages in experimental/ to build ESM using tsc-multi, create valid ESM types, and adds an exports field and testing using arethetypeswrong.

The patch to sorted-btree has the changes shown in this PR.


Migrate developments from jar2018 branch

[x] ~Combined refinements in the tools (min/max prune)~
[x] Generic error location-based search
[x] ~Combined search strategies in tools~
[x] Multi-counterexample refiner
[x] Extra check in assumption evaluation
[x] ~Solverless option for explicit transfer function~

Extra check in assumption evaluation: 488d16f47859b0334855f8939ca99bd0ff2decec
Multi-counterexample refiner: 7e58e6d9ab2504446a532c91afe44675cfb87754
Generic error location-based search: b65fcce8b3399855a8747108da45003ec0277bd0

The rest did not perform well, so not planning to migrate.


Use CronJob object instead of internal Cron lib.

https://kubernetes.io/docs/concepts/workloads/controllers/cron-jobs/

This issue was moved to kubedb/project#22


Artist birth and death dates

How we describe artists birth-death now:
<http://data.doremus.org/artist/6963af5e-b126-3d40-a84b-97e0b78f5452>
        ecrm:P98i_was_born	"1770"^^xsd:gYear ;
	ecrm:P100i_died_in	"1827"^^xsd:gYear .

Problem: P98i and P100i should have as range an event (it is an ObjectProperty and not a DataProperty). So the connection between the artist and its birth date should be Artist > Birth Event > Time Span > Date literal. Very long.
Different proposals:

bio:birth it is a ObjectProperty (used with errors by BnF)
schema:birthDate
dbo:birthDate
... others?

True, the birth event is a bit longer, but it allows for richer descriptions : date of birth, place of birth, from parents X & Y, notes about birth, etc.
I made a schema about it a while ago, although it is simplified (no Time-Span step which should definitely be there) it gives a good idea of possible ramifications.
And, since we are in a dynamic model, why not use event-based descriptions rather than regressing to static, attribute-like descriptions ?

And, since we are in a dynamic model, why not use event-based descriptions rather than regressing to static, attribute-like descriptions ?

What if put both for allowing both simpler and richer queries? :)

What if put both for allowing both simpler and richer queries? :)

what would be the schema in this case ?

@pasqLisena
Btw, when I did this schema I was interested in one additional point, which relates to our discussion: the possibility to infer the nationality of a person depending on informations about birth and death places. In this (fake) case, the person was born Russian but died Ukrainian after Ukraine was independent. So maybe this is one more argument in favor of the Birth and Death events ? ;)

what would be the schema in this case ?

Considering also #56 :
<http://data.doremus.org/artist/6963af5e-b126-3d40-a84b-97e0b78f5452>
        schema:birthDate	"1770"^^xsd:gYear ;
	schema:birthDate	"1827"^^xsd:gYear ;
        ecrm:P98i_was_born <http://data.doremus.org/artist/6963af5e-b126-3d40-a84b-97e0b78f5452/birth> ;
	ecrm:P100i_died_in	[ #similar ] .

<http://data.doremus.org/artist/6963af5e-b126-3d40-a84b-97e0b78f5452/birth>
        a ecrm:E67_Birth ;
        ecrm:P7_took_place_at <http://sws.geonames.org/524901/> ;
        ecrm:P4_has_time-span <http://data.doremus.org/artist/6963af5e-b126-3d40-a84b-97e0b78f5452/birth/time> .

<http://data.doremus.org/artist/6963af5e-b126-3d40-a84b-97e0b78f5452/birth/time>
    a ecrm:E52_Time-Span, tl:Interval;
    rdfs:label "1770" ;
    tl:start "1770"^^xsd:date ;
    tl:end "1770"^^xsd:date .

+1 for Pasquale example
Jean
Jean Delahousse
Independent Consultant
Semantic technologies, Linked Open Data, Semantic Web, Knowledge
engineering
<EMAIL_ADDRESS><EMAIL_ADDRESS>mob +33 6 01 22 48 55
skype jean.delahousse
2017-07-26 10:21 GMT+02:00 Pasquale Lisena<EMAIL_ADDRESS>
what would be the schema in this case ?
Considering also #56 https://github.com/DOREMUS-ANR/marc2rdf/issues/56 :
http://data.doremus.org/artist/6963af5e-b126-3d40-a84b-97e0b78f5452
schema:birthDate	"1770"^^xsd:gYear ;
schema:birthDate	"1827"^^xsd:gYear ;
ecrm:P98i_was_born http://data.doremus.org/artist/6963af5e-b126-3d40-a84b-97e0b78f5452/birth ;
ecrm:P100i_died_in	[ #similar ] .
http://data.doremus.org/artist/6963af5e-b126-3d40-a84b-97e0b78f5452/birth
a ecrm:E67_Birth ;
ecrm:P7_took_place_at http://sws.geonames.org/524901/ ;
ecrm:P4_has_time-span http://data.doremus.org/artist/6963af5e-b126-3d40-a84b-97e0b78f5452/birth/time .
http://data.doremus.org/artist/6963af5e-b126-3d40-a84b-97e0b78f5452/birth/time
a ecrm:E52_Time-Span, tl:Interval;
rdfs:label "1770" ;
tl:start "1770"^^xsd:date ;
tl:end "1770"^^xsd:date .
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/DOREMUS-ANR/marc2rdf/issues/57#issuecomment-317983963,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AD8Ze8TafoRh7-k2ydQsrhb1PLCr5YCZks5sRvb-gaJpZM4Oinr8
.

Hum, 2 small observations:

The URI you're minting for interval should rather be <http://data.doremus.org/artist/6963af5e-b126-3d40-a84b-97e0b78f5452/birth/interval> rather than <http://data.doremus.org/artist/6963af5e-b126-3d40-a84b-97e0b78f5452/birth/time> and then, follow the modeling proposal described in issue #56 (i.e. use the OWL Time ontology rather than the Timeline ontology)
You want to introduce twice the same information, using the schema:birthDate and schema:deathDate properties (take care, you have a typo in your example). The risk is that if you modify one value and not the other, the two values become inconsistent. This risk can be mitigated by the fact that we generate the values of those properties and that, a priori, the converter will not do stupid things (but this is fragile in the case of the conversion made by MeaningEngines).

Your rationale is that it makes simpler queries, but with SPARQL 1.1 property paths, this is not quite right.
With your shortcut:
select ?p ?b
where {
  ?p schema:birthDate ?b
}

Without your shortcut:
select ?p ?b
where {
  ?p ecrm:P98i_was_born/ecrm:P4_has_time-span/time:hasBeginning/time:inXSDDate ?b
}

In practice, given that there are not many options after each node, the SPARQL query evaluation plan should roughly give the same thing.

The difference between these 2 queries is that the former is much more readable :) I am also thinking about other people that would like to use our data.
Moreover, we sometimes duplicate the information, like with U70_has_title and rdfs:label. And it is instantiated at the same time by the converter, so there will be not incoherence. I am still for putting them both if this is the only point against.

Right, but people are not supposed to see queries, nor raw data :-)
I'm fine though that we also make an exception of duplicating those values between the schema:birthDate (resp. schema:deathDate) and the longer ecrm path as we do between rdfs:label and mus:U70_has_title.
@nicolasGuillouet Do you watch this issue? This has consequences on your converter too!
@pasqLisena Can you implement those changes and close the issue afterwards?


No query with id "XX" found propably expired session

OrientDB Version: 3.0.19
Java Version: jdk1.8
OS: linux
Expected behavior

Actual behavior
Using the orientdb-client to connect to a clustered database for queries, occasionally encounters this error
Steps to reproduce

Hi,
this sounds like the client switched to a new server that do not have the cursor of the query active, this should not really happen, do you set a connection strategy when you create the client OrientDB instance ?  This could happen anyway if a node shutdown while someone is running a query.
Regards

Hi,
Also consider to update to a more recent version 3.0.x is out of support for a while now.
Regards

I also encountered the issue of setting connection policies when creating a client? What type of strategy should be set up?
@tglman


Update Ulm National Ideas & Add Mission Tree

Change National Ideas to something befitting the true final boss of eu4
Add missions for Ulm that explicitly require you to stay a free city opm. scaling modifiers as you progress the mission, giving yearly base tax income, flat province FL, defensiveness, dev cost etc. idea would be a 50-60 dev province with enough buffs that if can compete even mid to late game, but it is all lost if they lose free city status.

Combining "reach reform tier 5" with "establish M.I.L.F." and replacing "reach reform tier 5" another step of "develop the city to the moon"


DROP TABLE IF EXISTS logged as error if the table doesn't exist

Describe the bug
When running a DROP TABLE IF EXISTS statement and the referenced table doesn't exist, a log entry with ERROR severity is displayed.
To Reproduce
package ae.pax.subscriber;

import com.klarna.hiverunner.HiveShell;
import com.klarna.hiverunner.StandaloneHiveRunner;
import com.klarna.hiverunner.annotations.HiveSQL;
import org.junit.Test;
import org.junit.runner.RunWith;

@RunWith(StandaloneHiveRunner.class)
public class DropTableIfExistsTest {

    @HiveSQL(files = {})
    private HiveShell shell;

    @Test
    public void shouldDropTableIfExists() {

        shell.executeQuery("create database my_db");
        shell.executeQuery("use my_db");
        shell.executeQuery("drop table if exists my_db.my_table");
        shell.executeStatement("drop table if exists my_db.my_table");

    }
}

Expected behavior
An INFO log reporting the table does not exist.
Logs
OK
OK
OK
11:38:08.188 [52bd8d48-a712-4e9f-9df0-79cce13a9eb2 Thread-1] ERROR hive.ql.metadata.Hive - Table muy_table not found: my_db.my_table table not found
OK
11:38:08.313 [52bd8d48-a712-4e9f-9df0-79cce13a9eb2 Thread-1] ERROR hive.ql.metadata.Hive - Table muy_table not found: my_db.my_table table not found
OK
OK

Versions (please complete the following information):

HiveRunner Version: 4.1.0
Hive Versions: 2.1.1

This doesn't fail the test but is just a log message right? That message is coming from Hive itself (https://github.com/apache/hive/blob/rel/release-2.3.4/ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java#L1277) so there's not really anything we can do about it from HiveRunner as we're just executing Hive's code at that point and can't intervene. If it's really bothering you then you could probably update the logging configuration for your tests to exclude ERROR messages from the above class.

Correct! The test doesn't fail; it is just the ERROR message in the log that looks very strange as there is no actual error. I was checking the Hive code, and to change the severity it won't be so easy as the getTable method is used in several places, and in some of them maybe makes sense to keep the log as an error (although I still don't fully agree with the severity).
Let's close this issue as it is not related to HiveRunner and I will just tell our team to get used to that ERROR message in the logs. 😉

Sounds good!


Update TextMate grammar to properly represent Razor content.

Right now we're re-using the built-in Razor TextMate grammar but it has its own flaws. One of which is the following end </p> tag is not colored correctly<EMAIL_ADDRESS>

This new grammar should be in its own repo a lot like https://github.com/Microsoft/TypeScript-TmLanguage
That way VSCode itself will ship our grammar.

https://github.com/aspnet/Razor.VSCode/issues/383#issue-473664717 has some very specific examples on when this falters. @danroth27 this is by far our number 1 area for improvement in Razor VSCode, issues piling up as dups 😄. I also haven't linked every dup issue so there's more than just mentioned here, i'll get better at that!.


CursorMut::get() should return a mutable reference?

There doesn't seem to be a way to modify the contents of an item in the RBTree.

This is by design: you can't get a &mut T because there may be multiple references to it (e.g. an item has multiple links and is inserted into more than one collection). You should use a Cell/RefCell/UnsafeCell if you need to mutate the contents of an item.

Hi, @Amanieu
I appreciate and understand your design approach. It's a good point to limit accessing &mut T to ensure soundness. However, I believe that providing an unsafe get_mut() method may be necessary for developers. In most situations, a linked list has only one link, and CursorMut can guarantee that there are no multiple mutable references to the list node. Additionally, the rust standard crate CursorMut provides similar methods for accessing mutable references and the intrusive-rs crate can maintain consistency with the standard crate's API.
An proposed code like this:
    /// Returns a mut reference to the object that the cursor is currently
    /// pointing to.
    ///
    /// This returns None if the cursor is currently pointing to the null
    /// object.
    ///
    /// # Safety
    /// The pointing object has and merely has one link.
    #[inline]
    pub unsafe fn get_mut(&mut self) -> Option<&mut <A::PointerOps as PointerOps>::Value> {
        Some(unsafe {
            &mut *(self.list.adapter.get_value(self.current?)
                as *mut <A::PointerOps as PointerOps>::Value)
        })
    }

If you need unsafe mutable access then you can always do that yourself by wrapping your type inside an UnsafeCell.
The standard library CursorMut can provide get_mut because it doesn't expose the links in the returned reference. This would be unsound in this crate because mutable access allows you to unsafely modify the links while the cursor is still iterating it.


Use WebRTCStar for communication

Using webrtcstar as a means of transmission.
Bye the way, Supports Safari and iframe embedding.

Hey, thanks for this! I'd be willing to accept this is if passes @latenssi or @JaniAnttonen 's review, and the commits were squashed!

By the way, If we want to use WebRTC transport, We cannot use WebWorker API, so I removed WebWorker API.

Hold on merging this. This is a massive PR with big consequences.
@skysbird just to clarify: it seems to me that this PR "re-brands" OrbitDB for Chinese audience or some other use. Did you mean to PR all this to upstream (here), or was that by accident? (texts, server addresses, etc.)
Generally though, I don't think we should move to use WebRTC by default, especially given the trade off that we wouldn't be able to use web workers.
@skysbird, is there another way for you to achieve what you're trying to do, without having to change the defaults here in the main repo?

Hold on merging this. This is a massive PR with big consequences.
@skysbird just to clarify: it seems to me that this PR "re-brands" OrbitDB for Chinese audience or some other use. Did you mean to PR all this to upstream (here), or was that by accident? (texts, server addresses, etc.)
Generally though, I don't think we should move to use WebRTC by default, especially given the trade off that we wouldn't be able to use web workers.
@skysbird, is there another way for you to achieve what you're trying to do, without having to change the defaults here in the main repo?

Ok, sorry for that, I forget this is a special version for my website. I will make a pure version later.


[playground] Adding workflow to Update K8s Cleanup Script

Notes for Reviewers
This PR fixes #7818
Signed commits

[ ] Yes, I signed my commits.

@Freedisch

@Freedisch

Just the above requested change and everything LGTM


V4 features

This implements v4 of the repo, as discussed in #13

@shreeya correct; not specifying options (the second+ option parameters) causes the role to use it's defaults:
ruleName: 0, // rule disabled

ruleName: 1, // rule enabled as warning with defaults

ruleName: 2, // rule enabled as errors with defaults

ruleName: [0, ...], // rule disabled (with non-default options)

ruleName: [1, ...], // rule enabled as warning with non-default options

ruleName: [2, ...], // rule enabled as error with non-default options


config: use <IP_ADDRESS> for listen address for dns multiaddrs

When constructing node configs for validators, if a dns address was provided use Ipv4Addr::UNSPECIFIED as the listen address instead of panicing.

crates/sui-config/src/utils.rs