Stage 0 scripts and config

Hi there,
Great work! Could you please provide us with the pretrain_stage0.sh or the config file (except the log file). We would like to reproduce some experiments! Thank You!

I used something similar to this (if you find anything here inconsistent with the log, please feel free to replace it). The stage-0 was trained on an other server at Northwestern with 3x RTX-A6000, which I only kept the log and pre-trained weights.
model:
  arch: pformer_opt
  model_type: pformer_opt2.7b
  load_pretrained: False
  # intialize stage 2 pretraining from stage 1 pretrained model
  # pretrained: "https://storage.googleapis.com/sfr-vision-language-research/LAVIS/models/BLIP2/blip2_pretrained.pth"
  freeze_vit: True

datasets:
  sentence_dataset:
    text_processor:
        train:
          name: "blip_caption"

run:
  task: image_text_pretrain   ### no need to change
  # runner: runner_iter
  # optimizer
  lr_sched: "linear_warmup_cosine_lr"
  init_lr: 1e-4
  min_lr: 1e-5
  warmup_lr: 1e-6

  weight_decay: 0.05
  max_epoch: 5
  # max_iters: 60000
  # iters_per_inner_epoch: 6000
  batch_size_train: 128
  batch_size_eval: 64
  num_workers: 4
  warmup_steps: 2000

  seed: 42
  output_dir: "output/BLIP-T/Pretrain_stage0"

  amp: True
  resume_ckpt_path: null

  evaluate: False
  train_splits: ["train"]

  device: "cuda"
  world_size: 3
  dist_url: "env://"
  distributed: True

I got the error below when using the yaml file your provided above.
(less4) root@ddab717805f9:/home/tiger/maas/BLIText# bash run_scripts/blip-T/train/pretrain_stage0.sh                                                                      [125/399]
WARNING:main:

Setting OMP_NUM_THREADS environment variable for each process to be 1 in default, to avoid your system being overloaded, please further tune the variable for optimal performance i
n your application as needed.

builder_cls=None, dataset_name=sentence_dataset
Traceback (most recent call last):
File "/home/tiger/maas/BLIText/train.py", line 103, in
main()
File "/home/tiger/maas/BLIText/train.py", line 81, in main
cfg = Config(parse_args())
File "/home/tiger/maas/BLIText/lavis/common/config.py", line 31, in init
dataset_config = self.build_dataset_config(config)
File "/home/tiger/maas/BLIText/lavis/common/config.py", line 102, in build_dataset_config
dataset_config_path = builder_cls.default_config_path(
AttributeError: 'NoneType' object has no attribute 'default_config_path'
builder_cls=None, dataset_name=sentence_dataset
Traceback (most recent call last):
File "/home/tiger/maas/BLIText/train.py", line 103, in
main()
File "/home/tiger/maas/BLIText/train.py", line 81, in main
cfg = Config(parse_args())
File "/home/tiger/maas/BLIText/lavis/common/config.py", line 31, in init
dataset_config = self.build_dataset_config(config)
File "/home/tiger/maas/BLIText/lavis/common/config.py", line 102, in build_dataset_config
dataset_config_path = builder_cls.default_config_path(
AttributeError: 'NoneType' object has no attribute 'default_config_path'
builder_cls=None, dataset_name=sentence_dataset
Traceback (most recent call last):
File "/home/tiger/maas/BLIText/train.py", line 103, in
main()
File "/home/tiger/maas/BLIText/train.py", line 81, in main
cfg = Config(parse_args())
File "/home/tiger/maas/BLIText/lavis/common/config.py", line 31, in init
dataset_config = self.build_dataset_config(config)
File "/home/tiger/maas/BLIText/lavis/common/config.py", line 102, in build_dataset_config
dataset_config_path = builder_cls.default_config_path(
AttributeError: 'NoneType' object has no attribute 'default_config_path'
builder_cls=None, dataset_name=sentence_dataset
Traceback (most recent call last):
File "/home/tiger/maas/BLIText/train.py", line 103, in
main()
File "/home/tiger/maas/BLIText/train.py", line 81, in main
cfg = Config(parse_args())
File "/home/tiger/maas/BLIText/lavis/common/config.py", line 31, in init
dataset_config = self.build_dataset_config(config)
File "/home/tiger/maas/BLIText/lavis/common/config.py", line 102, in build_dataset_config
dataset_config_path = builder_cls.default_config_path(
AttributeError: 'NoneType' object has no attribute 'default_config_path'

I got the error below when using the yaml file your provided above.
(less4) root@ddab717805f9:/home/tiger/maas/BLIText# bash run_scripts/blip-T/train/pretrain_stage0.sh [125/399] WARNING:main:
Setting OMP_NUM_THREADS environment variable for each process to be 1 in default, to avoid your system being overloaded, please further tune the variable for optimal performance i n your application as needed.
builder_cls=None, dataset_name=sentence_dataset Traceback (most recent call last): File "/home/tiger/maas/BLIText/train.py", line 103, in main() File "/home/tiger/maas/BLIText/train.py", line 81, in main cfg = Config(parse_args()) File "/home/tiger/maas/BLIText/lavis/common/config.py", line 31, in init dataset_config = self.build_dataset_config(config) File "/home/tiger/maas/BLIText/lavis/common/config.py", line 102, in build_dataset_config dataset_config_path = builder_cls.default_config_path( AttributeError: 'NoneType' object has no attribute 'default_config_path' builder_cls=None, dataset_name=sentence_dataset Traceback (most recent call last): File "/home/tiger/maas/BLIText/train.py", line 103, in main() File "/home/tiger/maas/BLIText/train.py", line 81, in main cfg = Config(parse_args()) File "/home/tiger/maas/BLIText/lavis/common/config.py", line 31, in init dataset_config = self.build_dataset_config(config) File "/home/tiger/maas/BLIText/lavis/common/config.py", line 102, in build_dataset_config dataset_config_path = builder_cls.default_config_path( AttributeError: 'NoneType' object has no attribute 'default_config_path' builder_cls=None, dataset_name=sentence_dataset Traceback (most recent call last): File "/home/tiger/maas/BLIText/train.py", line 103, in main() File "/home/tiger/maas/BLIText/train.py", line 81, in main cfg = Config(parse_args()) File "/home/tiger/maas/BLIText/lavis/common/config.py", line 31, in init dataset_config = self.build_dataset_config(config) File "/home/tiger/maas/BLIText/lavis/common/config.py", line 102, in build_dataset_config dataset_config_path = builder_cls.default_config_path( AttributeError: 'NoneType' object has no attribute 'default_config_path' builder_cls=None, dataset_name=sentence_dataset Traceback (most recent call last): File "/home/tiger/maas/BLIText/train.py", line 103, in main() File "/home/tiger/maas/BLIText/train.py", line 81, in main cfg = Config(parse_args()) File "/home/tiger/maas/BLIText/lavis/common/config.py", line 31, in init dataset_config = self.build_dataset_config(config) File "/home/tiger/maas/BLIText/lavis/common/config.py", line 102, in build_dataset_config dataset_config_path = builder_cls.default_config_path( AttributeError: 'NoneType' object has no attribute 'default_config_path'

The error indicates that builder_cls (for dataset) is None. sentence_dataset should be changed to laion_sentence_115m, which is to match the sentence dataset config defined here. You can also define your own sentence dataset similar to laion_sentence_115m.


feat: Parallelize the TD stage

Parallelize the TD stage.
I little bit overdid it, but it will, in parallel read all values and calculate TD (do checks), and when the reading is finished it will continue to calculate TD (do checks) and write to DB. It is even more parallelized after merging hardfork as we know the difficulty is the same as can run all checks in parallel.
There is one shot between tasks that send TD from one task to another and it is used to cancel tasks if an error happened.
Left comment inside code that explains the flow.
Why i think I overdid it is this stage is not that impactful, on first 1M block in mainnet there isn't a big difference and the times are around of 0.5s so it is not worth merging this. So it is not worth merging this, but i left it if someone is interested in code.
Will do 17M just to check, but it is probably I will just close this PR in few days.

It is faster:
New:
2023-04-21T16:36:56.176841Z  INFO Executing stage stage=TotalDifficulty
2023-04-21T16:38:23.585752Z  INFO Stage finished executing stage=TotalDifficulty checkpoint=17000000
diff: 1m 27s

Old way:
2023-04-21T16:51:57.464280Z  INFO Executing stage stage=TotalDifficulty
2023-04-21T16:53:48.599128Z  INFO Stage finished executing stage=TotalDifficulty checkpoint=17000000
diff: 1m 51s

but not worth it.

Interesting as a technique nonetheless. Where else would you do the same to accelerate things further? Senders Recovery?

Interesting as a technique nonetheless. Where else would you do the same to accelerate things further? Senders Recovery?

SenderRecovery and AccountHashing are already done and parallelized.
The most impactful place to introduce parallelization would be Execution, but this is best done after cache changes so that we don't introduce conflicts.
After execution we could do: ReceiptStage(That we need to make) AccountHashing, and indexing stages


Roll engine fde59006da24..174b73c21af8 (2 commits)

https://github.com/flutter/engine/compare/fde59006da24...174b73c21af8
git log fde59006da245dc90503919ea9091b9a56d74fb2..174b73c21af8b1f61865d0fa855a6b0ddc81c953 --no-merges --oneline
174b73c21 Roll src/third_party/dart 3e5ed47777..b53dceadaa (5 commits) b53dceadaa Update pubspec for dev_compiler and sourcemap_testing eefc18dc82 Add a hook to AnalysisDriver on the current session to be discarded. cda43dabda Use findNode.simple in migration test bd51d6f120 [vm] Build scopes for implicit getters of static const fields b876d12f03 Revert "[vm/compiler] Continued graph checker development"
2f18c3286 Lower the threshold to raster cache pictures (flutter/engine#7687)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (dnfield@google.com), and stop
the roller if necessary.

Auto-roller completed checks. About to merge.


Resume Jan 10

Current dependencies on/for this PR:

master

PR #1   👈

This stack of pull requests is managed by Graphite.


Feature: scan /etc/apt for our repo and make sure it has 'signed-by'

I just realised there may be some people who used apt-key to trust our packaging key system-wide instead of just our repo with a signed-by line in apt's sources.list or equivalent. A key-roll would break such setups, unless we correct it, because the /usr/share/keyrings/nlnetlabs-archive.keyring.gpg that we would (attempt to) replace would remain unused.
We should probably detect this, if not correct it.

It precludes the need for scanning and/or fixing. Worst case, our (old) key remains on the system keyring too.

Or maybe not, assuming we want to actively remove the trust on an old key.

As an aside: One can argue about the most optimal place for the key.

Mozilla has just decided to ship their key to /etc/apt/trusted.gpg.d/ for their new debian package repo per https://support.mozilla.org/en-US/kb/install-firefox-linux#w_install-firefox-deb-package-for-debian-based-distributions.


fix: fixed class name in breadcrumbs_#1410

Closes #1410
Description
Fixed class name in breadcrumbs

Add a brief description

Current behavior (updates)

Please describe the current behavior that you are modifying

New behavior

Please describe the behavior or changes this PR adds

Is this a breaking change (Yes/No):

Additional Information

Thanks!


Fix meta.yaml output directory

Description
A small bug was introduced when adding v1 format.
The meta.yaml file was always created in the current directory instead of recipe_dir.

Closing in favour of https://github.com/conda/grayskull/pull/552


the difference between depth image and rawdepth image?

What's the difference between depth image and rawdepth image? And what does the missingMask stand for?

If it is not too late, here is the script that was used to generate the data: https://github.com/s-gupta/nyu-hooks/blob/master/script_data.m . You can refer to the documentation of the dataset and this script to figure out the exact difference.
rawdepth is the depth before hole filling, while depth is the depth after hole filling. missing mask denotes the pixels in rawdepth that are missing values.

I am very sorry to reply to you now, thank you very much for your answer！


How is this file structured?

I don't understand how this file is structured. If I want to find the name associated with "Dicky," how would I do that aside from looking through the file manually?
And why are some names spread across multiple lines? Shouldn't each name appear on only one line? Example:
russ,russell
russell,russ,rusty
rusty,russell

Shouldn't those closely associated names be on one line?
I tried the Perl script out but all it did was display how many names had more than 5 mentions in the file.

Unfortunately, the file isn't structured very well.  I'd say that column 1 should have the given name, and columns 2+ should be the diminutive and nicknames, but as your example makes clear this is not always the case.  I think the first and third lines should be removed and line two should remain.  I'm going to try and clean up some of these.

Unfortunately, the file isn't structured very well.  I'd say that column 1 should have the given name, and columns 2+ should be the diminutive and nicknames, but as your example makes clear this is not always the case.  I think the first and third lines should be removed and line two should remain.  I'm going to try and clean up some of these.


Add styling for output layer of main plugin

Basic blue line from QGis default styles.

@Merydian What about taking the colour we also use in our maps client?
Seems a bit more aligned.

Should we add the start and endpoint markers as well?

Could you add a screenshot with start- and endpoint-markers, and the current style? That would make it easier to reason about :)
Tagging @TheGreatRefrigerator here for more input/ideas.

We should check the behaviour with default styles as well. (The default styles set by user).
Not sure if we want to override explicit settings a user made. Or give an additional option to not style the output?

We should check the behaviour with default styles as well. (The default styles set by user). Not sure if we want to override explicit settings a user made. Or give an additional option to not style the output?

We discussed this and settled on overwriting any default style for the "frontend" - people setting a default style know enough to apply their changes after the fact, and processing algorithm behaviour won't be changed.


HADOOP-18156 : Address JavaDoc warnings in classes like MarkerTool, S3ObjectAttributes, etc. https://issues.apache.org/jira/browse/HADOOP-18156

Description of PR
Debugged most of the Javadoc warnings that were there in MarkerTool.java, DirectoryPolicy.java, DirMarkerTracker.java and OperationCallbacks.java . Some of the descriptions can be improved like @return in ScanArgsBuilder class in MarkerTool.java . Some warnings are still there which I was not sure what to write description for.
How was this patch tested?
This patch was approved internally by other team member.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 47s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+0 :ok:
xmllint
0m  0s

xmllint was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

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

_ trunk Compile Tests _

+0 :ok:
mvndep
15m 17s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
26m  4s

trunk passed

+1 :green_heart:
compile
23m 17s

trunk passed with JDK Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
compile
21m 50s

trunk passed with JDK Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07

+1 :green_heart:
checkstyle
4m 41s

trunk passed

+1 :green_heart:
mvnsite
3m 21s

trunk passed

+1 :green_heart:
javadoc
2m 23s

trunk passed with JDK Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
javadoc
2m 11s

trunk passed with JDK Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07

+1 :green_heart:
spotbugs
4m 37s

trunk passed

+1 :green_heart:
shadedclient
22m 22s

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

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 23s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
1m 40s

the patch passed

+1 :green_heart:
compile
24m 10s

the patch passed with JDK Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
javac
24m 10s

the patch passed

+1 :green_heart:
compile
22m  3s

the patch passed with JDK Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07

+1 :green_heart:
javac
22m  3s

the patch passed

-1 :x:
blanks
0m  0s
/blanks-eol.txt
The patch has 26 line(s) that end in blanks. Use git apply --whitespace=fix <<patch_file>>. Refer https://git-scm.com/docs/git-apply

+1 :green_heart:
checkstyle
3m 58s

the patch passed

+1 :green_heart:
mvnsite
3m 11s

the patch passed

+1 :green_heart:
javadoc
2m 11s

the patch passed with JDK Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
javadoc
1m  4s

hadoop-common in the patch passed with JDK Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07.

+1 :green_heart:
javadoc
1m  0s

hadoop-tools_hadoop-aws-jdkPrivateBuild-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07 with JDK Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07 generated 0 new + 0 unchanged - 37 fixed = 0 total (was 37)

+1 :green_heart:
spotbugs
4m 45s

the patch passed

+1 :green_heart:
shadedclient
22m  1s

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

_ Other Tests _

+1 :green_heart:
unit
18m 43s

hadoop-common in the patch passed.

+1 :green_heart:
unit
2m 55s

hadoop-aws in the patch passed.

+1 :green_heart:
asflicense
1m 27s

The patch does not generate ASF License warnings.

241m 18s

Subsystem
Report/Notes

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

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

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

uname
Linux 0e6cadb32ce9 4.15.0-191-generic #202-Ubuntu SMP Thu Aug 4 01:49:29 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / faa81973cd696362568fc302f26364d97df7e97e

Default Java
Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07

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

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

modules
C: hadoop-common-project/hadoop-common hadoop-tools/hadoop-aws U: .

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

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

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

This message was automatically generated.


GH-39427: [GLib] Update script and documentation

Rationale for this change
Modify scripts and update documentation for building GLib and running test code in a Mac environment.

What changes are included in this PR?

Update Documentation (c_glib/README.md)
Update Script (c_glib/test/run-test.sh)

Are these changes tested?
Yes.

Are there any user-facing changes?
Yes.

Closes: #39427

Hello. I have made all the revisions based on your comments. Thank you for the review.


travis should retry on install failure

i thought travis was doing this for us, but apparently not. travis fails fairly often in install due to network failures. this isn't a bug in our code, so we should retry.

this is probably too hard for now. how do we know in what intermediate state kubeadm or openwhisk installers might've left our travis VM?


Updated steps and other info in PC Modding

Adds "THIS IS THE RECOMMENDED DOWNGRADER." in BSManager's section in Installers. Also updates the Beta Downgrading steps to include the 1.34.2_legacy option, and fixes a typo in the Downgrading Section (Oculus downgrader to OculusDowngrader)

I've edited this PR to remove ModAssistant, as it is abandoned and has bugs. If needed, I can add it again and put a deprecation notice. BSManager will now state "THIS IS CURRENTLY THE RECOMMENDED MOD INSTALLER AND DOWNGRADER."

I've edited this PR to remove ModAssistant, as it is abandoned and has bugs.

This has been discussed and voted in overflow for the record.

I would keep the full name of the 1.29.1 legacy branch, as some people will probably need to find the exact name because that's how there minds work. Unless of course, the name has been changed. And if 1.34.2 also has a longer name, I would also fix it.

I would keep the full name of the 1.29.1 legacy branch, as some people will probably need to find the exact name because that's how there minds work. Unless of course, the name has been changed. And if 1.34.2 also has a longer name, I would also fix it.

hm, 1.34.2_legacy just has a dash and another 1.34.2_legacy to go along with it, so not sure about that. But I agree with you about the exact name of 1.29.1, I will extend that.

Requests fulfilled, also added a few more changes

Meivyn found a lot more outdated info on the Wiki, so they've decided to start a pull request on my branch, and I will be in touch to merging it in once it's ready.
https://github.com/overlappingd/wiki/pull/1

Think be better merging in current changes and have the other update in a separate PR

I don't mind, however I don't see why you removed the warning about it being bugged. This might discourage users from using it because of that, which is the goal of that warning to begin with.

I've also updated the wording on my side. I agree with it being more generic though.

Thought process was more of a nudge to start transitioning to a different tool.
BMBF's notice is more direct since it was fully dropped for not working at all rather than just having bugs.
As far as Im aware, MA is still functional most of the time and the bugs have quick workarounds as more of an annoyance.

Undrafted, feel free to merge now!


Static provisioner without storage class when EKS cluster has a default one

/kind bug
What happened?
Static provisioner highlighted as an example in https://github.com/kubernetes-sigs/aws-fsx-openzfs-csi-driver/tree/main/examples/kubernetes/static-provisioning describes how to create a PV and PVC from a pre-configured FSx OpenZFS filesystem, but it assumes you don't have a default StorageClass. In my case, gp2 is my default, so when declaring the PVC with storageClassName="", it attempts to create a gp2-backed PVC.
What you expected to happen?
Documentation highlighting how to properly create a statically provisioned FSx OpenZFS.
How to reproduce it (as minimally and precisely as possible)?
Just have any default storage class and follow the example.
Anything else we need to know?:
Thanks! We've bene waiting for this driver for quite so long... =)
Environment

Kubernetes version (use kubectl version): 1.23
Driver version: 1.0.0

Hi!
Setting storageClassName="" in the PVC manifest actually prevents Kubernetes from using the default storage class to provision the volume. As stated here: https://kubernetes.io/docs/concepts/storage/storage-classes/#default-storageclass, it looks like the default storage class is only used "When a PVC does not specify a storageClassName".
This is also not a source of truth, but the AWS EBS CSI driver follows the same practice of setting storageClassName="" in the PVC when pre-provisioning a volume: https://github.com/kubernetes-sigs/aws-ebs-csi-driver/blob/master/examples/kubernetes/static-provisioning/manifests/claim.yaml.
I just ran a manual test, with a similar gp2-backed default storage class, and I could not reproduce this issue.
Could you provide a bit more context into how you reproduced this (i.e. provide commands run, contents of the manifest files)?

Thanks! We've bene waiting for this driver for quite so long... =)

This is awesome to hear!

Hi @hughdanliu !
This doesn't seem to be valid on K8s 1.23. I managed to address the issue anyway, mapping a StorageClass and then using the PV and PVC from dynamic provisioning. Here is how I did it using Terraform:
resource "kubernetes_storage_class_v1" "fsx" {
  metadata {
    name   = "aws-fsx-openzfs"
    labels = merge(var.tags, {
      Name       = "aws-fsx-openzfs"
    })
  }

  storage_provisioner = "fsx.openzfs.csi.aws.com"
  reclaim_policy      = "Retain"
  parameters          = {
    ResourceType   = "filesystem"
    DeploymentType = "SINGLE_AZ_2"
    Tags           = jsonencode(merge(var.tags, {
      Name       = "aws-fsx-openzfs"
    }))
  }
  mount_options       = [
    "nfsvers=4.1",
    "rsize=1048576",
    "wsize=1048576",
    "timeo=600",
  ]

  depends_on = [module.eks, helm_release.fsx_openzfs]
}

resource "kubernetes_persistent_volume_v1" "fsx_filesystem" {
  metadata {
    name   = "myapp-fsx-openzfs"
    labels = merge(var.tags, {
      Name       = "myapp-fsx-openzfs"
    })
  }

  spec {
    storage_class_name               = kubernetes_storage_class_v1.fsx.metadata[0].name
    access_modes                     = ["ReadWriteMany"]
    capacity                         = {
      storage = "2Ti"
    }
    persistent_volume_reclaim_policy = "Retain"

    persistent_volume_source {
      csi {
        driver            = "fsx.openzfs.csi.aws.com"
        volume_handle     = module.fsx_openzfs_filesystem.fsx_file_system_id
        volume_attributes = {
          DNSName      = module.fsx_openzfs_filesystem.fsx_file_system_dns_name
          ResourceType = "filesystem"
        }
      }
    }
  }

  depends_on = [module.eks, module.fsx_openzfs_filesystem]
}

resource "kubernetes_persistent_volume_claim_v1" "fsx_sites" {
  metadata {
    name      = "myapp-fsx-openzfs"
    namespace = kubernetes_namespace_v1.this.metadata[0].name
    labels    = merge(var.tags, {
      Name       = "myapp-fsx-openzfs"
    })
  }

  spec {
    storage_class_name = kubernetes_storage_class_v1.fsx.metadata[0].name
    access_modes       = ["ReadWriteMany"]
    volume_name        = kubernetes_persistent_volume_v1.fsx_filesystem.metadata[0].name

    resources {
      requests = {
        storage = "2Ti"
      }
    }
  }

  wait_until_bound = true

  depends_on = [kubernetes_persistent_volume_v1.fsx_filesystem]
}

Please let me know if it is ok to close this issue, or if you want to keep it open to map that as part of examples documentation.

Hmm, I just ran a manual test on an EKS cluster running on K8s 1.23 and still could not reproduce this issue. If you have time, I'd like to know more about your cluster settings to properly reproduce this issue, because I suspect this would be shared across the FSx for Lustre and EBS CSI drivers.
Given that you're out of immediate pain, feel free to close out this issue. I'll keep this in the back of my mind if any more users see this issue.


Fix typo by replacing github with gitlab in comment

Fixing a typo in the gitlab scan function from github to gitlab
Description:
Function comment incorrectly stated it scans GitHub, while it's really scanning GitLab
Checklist:

[x] Does your PR pass tests?
[x] Have you lint your code locally prior to submission?

no longer supporting gitlab/github integrations


Optimization of camera mode change for better E-M10 MKIV support

The current code does some unmanaged switching between different camera modes. This has some drawbacks especially for the E-M10 MKIV which doesn't have a "SHUTTER" mode. In order to take a picture with the E-M10 MKIV you have to stay in "RECORD" mode with live view enabled. Using the code provided by camdroid (https://github.com/joergmlpts/olympus-wifi/pull/3) it is possible to take pictures, but there are two restarts of the liveview involved. Further you have to know that you need an instance of the M10MKIV class.
What I tried to archive with the code changes is to add an approach that allows the execution of one action at a time with a mode change only if it is required. For example: If you want to set the time while you are in the PLAY mode, the mode will not be changed. If you currently execute the live view in RECORD mode, the mode will be temporarily changed during the operation. Live view will be restarted automatically.
Besides a faster operation, this allows the E-M10 MKIV to be efficiently used with the rest of the code. I added an extra path for taking pictures with the E-10 MKIV instead of the inherited class. If there are more differences with other new camera models it might be a good idea for the previous approach, but I think this should be used in combination with a factory of some kind. At this stage I think it doesn't need that kind of complexity.
I hope the code quality is sufficient as I am normally not using Python on a daily basis ;)
I tested the code using my E-M10 MKIV. I don't own any other model. It would be good if anyone is able to do some testing using an older model which supports the SHUTTER mode.

Thanks for the changes! I tried them on my TG-5.
I started the live view. The Live View window started big, in the highest resolution. I took a photo. Taking the photo worked fine, but after the photo was taken, the live view window became small. It was then displayed in the lowest resolution. The desired size in the menu remained at the highest resolution.
Do you know why this happens? I undid the changes and verified that without your changes, the window size does not change when a photo is taken.

Thank you very much for the testing. I tried to reproduce the behavior on my side. Unfortunately I wasn't able simulate the problem with the E-M10. I tried different resolutions and switched to a different mode. The code always switched back to the right resolution.
I tested with a mode switch to the "play" mode by setting the clock.

Setting the time after liveview has been opened ("File->Set clock") -> causes mode change to "play"
Switch to a different resolution via "View->Size->640x480"
Setting the time again

The network log is coherent with what I see on the screen. I also tested if a switch to a different mode implicitly stops the liveview (it does) and if the liveview is able to restart (it is).
During testing I accidentally added a mode change to "rec" right before starting the liveview, and bingo, it results in the same behavior as you described.

I would interpret this as follows: A mode change request to the same mode will be ignored by the camera. Switching to "rec" without a parameter will set the resolution to the lowest possible. This causes the liveview to be started in a low res.
Maybe the shutter mode and the rec mode are somehow "related". I'm not able to test it with the E-M10 so I added a fix which will ensure that starting liveview will always begin from "play" mode. This will fix at least the observed problem when switching from "rec" to "rec" mode.

Thanks! The window size now stays the same when I take a photo.


update to dart 2

This currently uses a forked version of package:quiver_time until https://github.com/QuiverDart/quiver_time/pull/7 is released

Thanks for the PR!
Published it as version 2.0.0 on pub.
Also removed dependence on quiver_time. The details can be seen in this commit: https://github.com/desktop-dart/duration/commit/267fae3fd6bc542d4c1f5660a72a835500416576


ImportError: No module named mpl_toolkits.mplot3d

Hi there,
When I try to load mpl_toolkits.mplot3d it gives following error:
from mpl_toolkits.mplot3d import Axes3D
Traceback (most recent call last):
File "", line 1, in
ImportError: No module named mpl_toolkits.mplot3d
I have VS 2010, PTVS 2.1, Python 2.7 and Anaconda2 4.0.0. In IPython and Spyder code is working. How can I import this library into VS environment.
Thanks,
Aziz.

It's likely that you are running using the wrong python environment.
I don't recall if PTVS 2.1 was able to auto-detect Anaconda or not.  Under Tools/Options/Python Tools there should be a page that lists all configured environments.  If Anaconda isn't detected, you'll need to configure it manually.  You can also select which one is the global default environment.
A project can use any of the globally configured environments, if none are selected then the global default is used.  That selection is done in solution explorer, under Python environments node, or via the project settings, general page.


Support Glyphs.app alternate (aka 'bracket') layers and equivalent DS rules

spun off of #62

so-called "bracket" layers, i.e. alternate layers substituted when axis conditions are met (GSUB feature-variations)
For Glyphs v3: https://glyphsapp.com/learn/switching-shapes
For Glyphs v2: https://glyphsapp.com/learn/alternating-glyph-shapes and https://glyphsapp.com/learn/alternating-glyph-shapes-with-multiple-axes
These are more complicated in that, even if Glyphs.app represent them as layers within a glyph, they actually end up becoming alternate glyphs to be substituted via GSUB in the final font. In fact when doing glyphs=>UFO that's what we do, we create new glyphs from them and add designspace  that define when to substitute which.

Designspace rules are spec'ed here:
https://fonttools.readthedocs.io/en/latest/designspaceLib/xml.html#rules-element
fontmake uses glyphsLib to turn "bracket" layers into alternate UFO glyphs and accompanying designspace 'rules' specifying the axis range or sub-space within which they are supposed to be substituted, then fontTools.varLib.featureVars turns those into OpenType FeatureVariations tables.
fonttools feaLib has experimental support for specifying conditional rules in FEA, but the effort to standardise this is somewhat stalled, see https://github.com/adobe-type-tools/afdko/issues/153

I expect I will continue to need this in every (or nearly every?) serious variable font project I do. Used by Google/Material Symbols as well as Science Gothic.

To give a concrete public repro, this appears to impact Rokkitt:
$ python resources/scripts/ttx_diff.py 'https://github.com/Fonthausen/RokkittFont#sources/Rokkitt.glyphs'
...lots of diffs, including GlyphOrder...

$ diff -u build/default/fontc.GlyphOrder.ttx build/default/fontmake.GlyphOrder.ttx
...
     <GlyphID id="733" name="verticallinemod"/>
+    <GlyphID id="734" name="naira.BRACKET.varAlt01"/>
   </GlyphOrder>

$ $ grep naira.BRACKET.varAlt01 build/default/fontmake.*.ttx
build/default/fontmake.glyf.ttx:    <TTGlyph name="naira.BRACKET.varAlt01" xMin="35" yMin="0" xMax="675" yMax="583">
build/default/fontmake.GlyphOrder.ttx:    <GlyphID id="734" name="naira.BRACKET.varAlt01"/>
build/default/fontmake.GSUB.ttx:          <Substitution in="naira" out="naira.BRACKET.varAlt01"/>
build/default/fontmake.gvar.ttx:    <glyphVariations glyph="naira.BRACKET.varAlt01">
build/default/fontmake.hmtx.ttx:    <mtx name="naira.BRACKET.varAlt01" width="706" lsb="35"/>
build/default/fontmake.HVAR.ttx:      <Map glyph="naira.BRACKET.varAlt01" outer="0" inner="112"/>
build/default/fontmake.post.ttx:      <psName name="naira.BRACKET.varAlt01"/>

For context, x.BRACKET.y appears to be from https://github.com/googlefonts/glyphsLib/blob/74c63244fdbef1da540d646b0784ae6d2c3ca834/Lib/glyphsLib/builder/constants.py#L35.
If #1156 is to be believed we have 98 fonts that differ by glyph order so potentially there are many more.


Is there a ways to make the crop area re-sizable with no ratio between x and y?

Make sure you con't have destination-height and ratio attributes set and you should be good

No further comments, closing...


Removed code cannot be selected in diff viewer when inlined; only opens context menu

When viewing PRs in vscode.dev/github.dev, code that has been deleted cannot be selected. All I can get is the context menu.

If I disable the inline view, then I can select text on either side.
I can't recall if this is new or not
Version: 1.85.2
Commit: 8b3775030ed1a69b13e4f4c628c612102e30a681
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36 Edg/<IP_ADDRESS>
Embedder: github.dev
(Forgive the lack of template; I guess the vscode.dev report issue button is not as rich as the desktop version.)

/duplicate of #8226

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

Wow, not every day I dupe a 4 digit issue.


Get data form Service web api push tree not work

public tree: TreeModel = {
value: 'All list',
children: []
};
getList() {
this.service.GeJson('Groups/GetParent').subscribe((res => {
if (res) {
res.forEach(element => {
this.service.GeJson('Groups/getSubGroup/' + element.GroupID).subscribe((su => {
let sub = su
if (sub.length == 0) {
this.tree.children.push({ value: element.GroupName, id: element.GroupID })
}
else {
var tmp = [];
tmp = new Array
sub.forEach(submb => {
tmp.push({ value: submb.GroupName, id: submb.GroupID })
});
this.tree.children.push({ value: element.GroupName, id: element.GroupID, children: tmp })
}
}))
});
}
}))
}

@duongphamit You should have a look at https://github.com/valor-software/ng2-tree#load-children-asynchronously. Please, let me know if that helped.

@mmirzaeyan You are right, we need to document this project better, especially with real-world examples

Use addChild method instead of push into the array,  since the double quotation marks will make mistakes in TreeModel .


cannot find blenderneuron in the sidebar

Hello, may I ask if this plugin is compatible with Blender version 3.2.1? I have imported and enabled the plugin in Blender, but I cannot find it in the sidebar.

thank you so much!

Hi @miaaaaaxu, it looks like you're using a newer version of Blender, currently BlenderNEURON only works with Blender 2.79. Hope that helps.

Hi @miaaaaaxu, this addon is compatible with Blender 2.79, it doesn't currently work for Blender 3+.

Hi! The current release doesn't work past Blender 2.79. If you download the version from the 3+ conversion branch, it will show up in Blender 3.1-3.3 on the bar on the upper right of the 3d view, but not all functionality works as of now. Hope that helps!

BlenderNEURON has been updated to be compatible with Blender 3 and 4. You can download the latest version of BlenderNEURON from here: https://blenderneuron.org/download/


Fix/cross dw , add load vars and enable circle ci

Replica of https://github.com/dbt-labs/dbt-project-evaluator/pull/85 which had some comments.
I am creating this new PR in order to trigger the CI check.

Finally!


Set up project board on repository

Hi, I think I have done this last week.
Link: https://github.com/users/nehCG/projects/1

Please link it to your repository!

Hi, it seems like the project is already linked with this repo because once you click items in the Todo list, it will direct you to the corresponding repository. And I did not find any method to link the whole project to a particular repo.
If my understanding is wrong or there is indeed a way to link it to this repo, please help me point out. Thank you!

Resolved!


Doc updates

Ran through chapter 3 and identified a few issues that are corrected in this PR.
Note that while MD lists in iPython are rendered in increasing order using just "1.", in HTML it is not rendered in increasing order, so you need to manually increment the counter (see my update for plumbing and serialization)

Thanks!


App\Providers\TenancyServiceProvider::class, has a bug

Describe the bug
The string type in
 public static string $controllerNamespace = '';
throws an error
syntax error, unexpected 'string' (T_STRING), expecting function (T_FUNCTION) or const
(T_CONST)
once removed everything runs well.

Remove the typehints or use PHP 7.4


MATL: sync up funcs for both games

json parsing is now packed into mostly one spot (good for replacing stuff later), and funcs are mostly identical in code and features.

thank


modified docs for 1-21

Docs for 1-21
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

You'll probably need to rebase after this change merges: https://github.com/aws/eks-distro/pull/389
Sorry! Also I stole your changelog bullet points under "patches removed", as 1.20-2 adds these. You can remove the bullet points.

We need to change default release branch too

/unhold

/hold

/unhold


Support array namespaces for module nesting

Summary
This PR addresses two issues, #15 and #34, by

replacing deep-assign and clone dependencies with lodash.merge and lodash.clonedeep
refactoring the namespace resolution utils to support array namespaces and to be more DRY
refactoring stripSlashes() in utils to support arrays and use lodash.trim instead of the regex

@jskrzypek this is beautiful.  Looks like a minor release?  Nothing breaking in here, right?

@marshallswain
Nope, I did not change any tests to make them pass. I also changed getNameFromConfig() to be getNameFromExplicit() to better match the nameStyles value, but I updated all of the references to the function, so nothing breaks.

@marshallswain I can make the release if you add me to npm.

@daffl that would be great.  I added you.  Thanks!

Released as v0.8.0. Thank you for the PR!


feat: added blog list section to blog page

Hi
I've added to blog card list to the blog page.
The following is the screenshot:

Thanks! nicely done!


Camera with Ultrasonic tests

Adding sleep to threading loop fixes #367
Also added a demo example of using multiprocessing to get frames from the camera instead of a thread, also fixes #367
For discussion only, haven't tested the implications of doing this across other use-cases

Closing since the ultrasonic problem is avoided by https://github.com/pi-top/pi-top-Python-SDK/pull/368


chore(deps): Updated client libraries to AWS SDK v3

Please check if the PR fulfills these requirements

[x ] The commit message describes your change
[ ] Tests for the changes have been added if possible (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)
[ ] Changes are mentioned in the changelog (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

Upgrades the AWS SDK libraries as used to AWS SDK v3.

What is the current behavior? (You can also link to an open issue here)

Lambda Toolbox uses the old aws-sdk v2 libraries to interact with DynamoDB and make HTTP requests against AWS AppSync GraphQL endpoints

What is the new behavior (if this is a feature change)?

Lambda Toolbox uses AWS SDK v3 to achieve the same thing

Does this PR introduce a breaking change? (What changes might users need to make in their setup due to this PR?)

Any functionality relying on behaviour or types specific to aws-sdk v2 might break and should be reevaluated. Standard use of the toolbox libraries should not be affected

Other information:

Testing is currently rudimentary, I have not yet tested the reworked HTTP request signing for AppSync against a live endpoint at all.

That looks really great.

I added some tests for the updateExistingItem/createUpdate calls, far from exhaustive, but I feel like not much is happening here anymore anyway, so I didn't want to spend too much time on there. I'd still need to test the AppSync GraphQL client, but I have no good idea on how to do that without building up an outright integration test harness. With your thumbs up, I'd mark this as ready.


deepl.usePro will not be set in initial setup

Hi,
thanks for the plugin, it's great!
I installed the plugin and tried the translation. I got a request to insert my API key and a question if I use the deepl pro plan (I'm not, so I selected No).
The translation wasn't working: It showed a notification "Translating..." but nothing happened.
I found out, that the parameter deepl.usePro wasn't added to my settings. After adding it manually ("deepl.usePro": false) the translation worked fine.
I would assume that after asking the plugin would save the answer - I can reproduce this problem on my second desktop.

Hi,
I could not reproduce the behaviour. The plugin saves the property deepl.usePro.
So I will close this issue.
Thx for using the extension. :)


[GLib] Cannot build gir files with MSVC

Describe the bug, including details regarding any error messages, version, and platform.
Trying to build gir and typelib files fails when using MSVC with linker errors related to the generated enum methods:
Arrow-1.0.obj : error LNK2001: unresolved external symbol garrow_compression_type_get_type

Component(s)
GLib

take

Issue resolved by pull request 42131
https://github.com/apache/arrow/pull/42131


1.19.3

was wondering if you plan on updating and if it already works if you can mark it on curseforge

I've marked it as 1.19.3 on Curseforge.


Usage readme section

Maybe it's just me, but i love a nice little shoutout to firing up the command palette. If this was a users first package, they may not have any idea what you even type in that gif of the readme.
:heart:

Merged Thankyou


SSR Context with store not displaying reactive value

Describe the bug
I have a context that uses a store with an object and retrieve by keys, with the value updated on mount. The displayed values are not updated when this is the first page loaded. Using console.log in the effects show the correct state.
Navigating from another page to this page works.
Doing the same without context works and ssr: false works.
// context
type Data = {
  [k: string]: {
    email: string;
  } | null;
};
const [data, setData] = createStore<Data>({
  jeff: null,
});
onMount(() => {
  setData('jeff', {
    email<EMAIL_ADDRESS>  });
});

const { data } = useData();
const state = createMemo(() => {
  if (!data['jeff']?.email) {
    return 'No email';
  }
  return 'Has email';
});
return <>

State: {state()}
 // Does not update to "Has email"

email: {data['jeff']?.email}
 // Does not update to show email
</>

Your Example Website or App
https://stackblitz.com/edit/github-afuq7e?file=src%2Froutes%2Findex.tsx
Steps to Reproduce the Bug or Issue

Go to / path
No data would be displayed
console logs in createEffects display the state correctly
Go to /links, then navigate to the problem route - it works

Expected behavior
HTML is updated to reflect state
Screenshots or Videos
No response
Platform

OS: macOS
Browser: Chrome
Version: 114.0.5735.198

Additional context
This issue occured in my project when using returning enums in state memo, no issues with normal variables. However, when making the reproduction I could not make it this far.
const state = createMemo(() => {
    if (!data['jeff']?.email) {
      return State.NO_EMAIL; // returning a string worked
    }
    return 'Has email';
});
// simply having the above also affected reading the store value, even when `state` was not being used:
return
{data['jeff']?.email}

Yeah it seems to be some sort of race condition. When I put a breakpoint in the onMount call it works. I will explore further.


<select>

Page: https://id.react.dev/reference/react-dom/components/select
Assignee: @ramadanriz

Is this issue duplicated with #427?

@aroyan @zainfathoni @resir014 This is a duplicate of #427
I suggest closing this one and using the earlier number
Also at the same time, there hasn't been any info from @ramadanriz about the progress and PR

Nice catch! I'm closing this issue in favor of #427.


Add safety disclaimer

I haven't used the app yet, so I am not positive if such a disclaimer exists already, but it would be a really good feature to have a disclaimer show at the top or something that warns the user that the results cannot be guaranteed to be fully accurate, and that the user should still use other means to identify the plant to ensure safety being eating or using it.
Here's a video talking about the issue from AtomicShrimp: https://www.youtube.com/watch?v=sjS0PgC9KSc

The app does only provide the name and images of the plant, no description or information if it is edible. So this disclaimer is not needed in my opinion.

Regarding missing description: The app displays what is available through the Pl@ntNet API, which does not include a description. So basically I am displaying everything what I can get already.
Regarding disclaimer: It might be worth adding it, and is also pretty trivial, and I think it could make sense. In the end, it wont hurt, so I'm just gonna add it.

Regarding description: I only wanted to state that there is no description and therefore no one is given information about usage or eating a plant. But thanks for confirming, that the API (sadly) does not provide a description.
If you add a disclaimer, maybe make it less intrusive because the risk is not high.
For a disclaimer or warning, I would find it useful to know if my plant-images are uploaded to the servers/website or if they stay on my device. Just so users make sure not to have people in the background of the plants or so.

Done with 27fd8df


added turkish language support

You only have to translate strings for the library (not the same app)~

Oh okay.I'll check tomorrow.

Reopen when you've only changed the two strings in the library.


[OCE] update deps 06 13 24

What

Update minor and patch deps
Why

Keep us up to date

https://skype.visualstudio.com/SPOOL/_workitems/edit/3754113
How Tested

Build locally Ci to validate

Converting to draft and starting over to get some updates in. There seems to be an issue here with libraries that are related to testing.


ReqMgr2: we should create an area in Gui for failure report

For example, if a workflow fails to get acquired by GQ (lack of input dataset, etc), we should make the error message visible somewhere. Possible on the top of the page, over the request parameters.

Alan, it should show in logdb. I am closing the issue.


Add folders with subfolders with path manager

From<EMAIL_ADDRESS>on 2014-10-26T06:02:34Z
Spyder Version:  2.3.0rc
Python Version:  2.7.7
Qt Version    :  4.8.6, PyQt4 (API v2) 4.10.4 on Windows
pyflakes >=0.5.0:  0.8.1 (OK)
pep8 >=0.6      :  1.5.6 (OK)
IPython >=0.13  :  2.1.0 (OK)
pygments >=1.6  :  1.6 (OK)
sphinx >=0.6.6  :  1.2.2 (OK)
psutil >=0.3    :  2.1.1 (OK)
rope >=0.9.2    :  0.9.4 (OK)
matplotlib >=1.0:  1.3.1 (OK)
sympy >=0.7.0   :  0.7.5 (OK)
pylint >=0.25   :  None (NOK)
What is the expected output? What do you see instead?
In Matlab, user has an option to add folders with subfolders to the search path. Does Spyder has the same functionality?
Thanks!
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=2024

From ccordoba12 on 2014-11-18T14:53:27Z
If you add a folder, you don't need to add subfolders to your path. The only thing you need to do is to add an empty init.py file to every subfolder. Then you can call modules in your subfolders just using import statements, like this
from folder.subfolder.subsubfolder import myfunction
I really don't know if it's a good for Spyder to add those files in behalf of the user.
Steven, what do you think?
Cc<EMAIL_ADDRESS>Labels: MS-v2.4 Cat-SpyderGUI

From<EMAIL_ADDRESS>on 2014-11-19T04:04:20Z
I agree with your assessment Carlos.  Adding nested folders to the Python path defeats the purpose of the Python package/subpackage system.

@ccordoba12, closing?

Yep :-)


Page not sliding down

I haven't changed any code, it's the default template that I got, but the scroll down button doesn't seem to be working... i have attached the link FYI
https://fictional-cod-6x9qp94w9gjh4j67-1234.app.github.dev/

oh nvm ig it is was never meant to be a button

https://www.canva.com/design/DAGDQrPlDCQ/wxSZe-ctRbALEeug8iaM1w/edit?utm_content=DAGDQrPlDCQ&utm_campaign=designshare&utm_medium=link2&utm_source=sharebutton


feat(tags): replace tags identifier from record urn and type to asset id

replace tags identifier from record urn and type to asset id
update docs

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

79 of 83 (95.18%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.07%) to 80.932%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

store/postgres/tag_repository.go
39
40
97.5%

tag/errors.go
0
3
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.07%

Covered Lines:
4775

Relevant Lines:
5900

💛  - Coveralls


feat(cairo_native): add batcher compiler struct

This change is

crates/blockifier/src/execution/native/contract_class_manager.rs line 33 at r2 (raw file):
Previously, Yoni-Starkware (Yoni) wrote…
Better to have this private

How do you want to call this function if it's private?
My understanding is that the PyBlockExecutor is going to create the ContractClassManager under Arc when it starts running and then give a clone of the Arc to the compilation thread and to each newly created PapyrusReader. The compilation thread just runs handle_compilation_requests_loop, and the PapyrusReader uses the caches and sends compilation requests.
If this function is private, the compilation thread cannot run it. To deny access to this function from the PapyrusReader we can create two structs: one containing the sender, the other containing the receiver (both containing the caches). Will that be better?

crates/blockifier/src/execution/native/contract_class_manager.rs line 33 at r2 (raw file):
Previously, Yoni-Starkware (Yoni) wrote…
Run the thread inside pub fn new(, and let the struct hold it. Why not?

Sounds good. I'll try that.

crates/blockifier/src/execution/native/contract_class_manager.rs line 33 at r2 (raw file):
Previously, avi-starkware wrote…
The receiver can't be part of the struct. It can only be held by one thread, so rust doesn't let me clone the arc of the struct if the struct contains the receiver.

I think this is nicer now.
To create the manager all you need is the caches struct. The compiler and channel can be created inside the method that generates the struct and starts the compilation thread.

crates/blockifier/src/execution/native/contract_class_manager.rs line 33 at r2 (raw file):
Previously, Yoni-Starkware (Yoni) wrote…
Don't you need to store the thread somewhere? what happens when new is over?
Doesn't it drop the thread?
Please add a super short test to make sure this is working

We can store the JoinHandle returned from thread::spawn, but all it does is allow us to wait for the thread to terminate, and this thread is an infinite loop.
In rust, a spawned thread stays alive as long as the main thread is alive and it did not terminate by itself.
I tried a simple example spawning a thread from inside a function and it stayed alive until it finished even after the function terminated.

crates/blockifier/src/state/contract_class_manager.rs line 69 at r4 (raw file):
Previously, Yoni-Starkware (Yoni) wrote…
I don't want it to stop the batcher. In case the queue is full, let's not propagate this error. Maybe add a log
This size should be part of the config (add a TODO)

Done.
Added TODO where the channel is created.

crates/blockifier/src/state/global_cache.rs line 55 at r12 (raw file):
Previously, meship-starkware (Meshi Peled) wrote…
Should it be changed to Compiled class as well? If so, I'll just put it here to see if it changed after a rebase.

Done.
Let me know what you think

crates/blockifier/src/execution/native/contract_class_manager.rs line 33 at r2 (raw file):
Previously, avi-starkware wrote…
Done.

The termination mechanism and drop might not be strictly necessary, because when the Sender end is dropped the channel closes (unless there are additional clones of the Sender...). When the channel closes, receiver.iter() ends without error.
See here. When a sender is dropped  the function disconnect_senders is called, which calls disconnect_receivers if there are 0 senders.
So perhaps we can assume dropping the contract class manager automatically stops the thread since it drops the sender and therefore the receiver. WDYT?

crates/blockifier/src/state/global_cache.rs line 56 at r20 (raw file):
Previously, Yoni-Starkware (Yoni) wrote…
For now, be consistent with native's type here

But where will we cache Cairo 0 contracts? Do you want to add another global cache for that?


How to access scope of target state with 1.0?

In previous version, scope of target state can be accessed through event object like:
$rootScope.$on('$stateChangeSuccess', function(event) {
  var scope = event.targetScope;
});

Now $stateChangeSuccess is deprecated and recommended to use onSuccess:
$transitions.onSuccess({}, function(transition) {
  var scope = ???
});

How can I access scope?

@seiyata have you found out how to do this?

I think you have something similar like I have, watching for $viewContentLoaded:
$rootScope.$on('$stateChangeSuccess', function (event, toState) {     event.targetScope.$watch('$viewContentLoaded', function () {
$viewContentLoaded is emitted on the viewScope, and can be captured by the viewScope, its parents up to the $rootScope.
I use the following for the moment:
$transitions.onSuccess({}, function (transition) {      $rootScope.$watch('$viewContentLoaded', function () {
@devs
Is it possible to implement a pointer to the scope created in the transition?

@fedealconada Not yet, I re-enabled $stateChangeSuccess according to https://ui-router.github.io/ng1/docs/latest/modules/ng1_state_events.html for now.

Thanks @FritzHerbers .
$rootScope.$watch('$viewContentLoaded', function () {

worked on my app. Next is how to access "scope" of the view...

Is it possible to implement a pointer to the scope created in the transition?

Sorry, no, there's no access to scope in 1.0.
What exactly are you accomplishing by accessing scope?  There is probably a different approach that would work

I'm using $stateChangeSuccess event to propagate a message from one controller to another.
// Propagate flash message from previous page
var message = FlashService.getMessage();
if (message !== undefined) {
  event.targetScope.alerts.push(message);
}

This is useful when I want to display message like "xxxxx registration completed!" after redirecting from registration page, thought not sure this is good manner...

if (message !== undefined) {
event.targetScope.alerts.push(message);
}

Why don't you create a service and push alerts through it?

any updates about this?
i need to interpolate some string with the state scope


sendFailedEmail should not go to the user but all admins

Currently, we're sending the sendFailedEmail to the individual (user) who set up the contribution.
In the context of an Organization, this is completely wrong as team members come and go.
Instead, all admins should be notified. Use notifyAdminsOfCollective.
While we're at it, that should be the same for sendArchivedCollectiveEmail, sendThankYouEmail and sendCreditCardConfirmationEmail.

🏆 This issue was completed by @snitin315
To claim the bounty, you can either:

Submit an invoice to https://opencollective.com/engineering/expenses/new (please mention the issue number in the expense's title)
Or ask for an Open Collective gift card of the same value (to donate to other collectives)


Move to bcrypt from bcrypt-nodejs

This project uses bcrypt-nodejs library for hashing and salting user passwords. That library seems very outdated, to the point there's a notice on its NPM page.
Now there's bcrypt that supports both promises and callbacks. It also looks well maintained and has typescript types available.
Can it be changed? Or is there a reason for using bcrypt-nodejs?

Hi,
It was obviously missed in the output of the default install. The NPM outdated does not report this, so it was probably just overlooked.
Yes, could be done, prefferably in the default, async version (no promises), to keep in sync with other existing code. I'd help you with a PR if you would like to do one,
Thanks!

Hey Peter, I just submitted #245 with this change.
Thank you.


simple fix to escape method names in typegen

A stopgap solution to #26.
operationId is turned into a string instead.
A more permanent solution would be to use the same type name converter that the dtsgenerator uses for operationId's in order to sanitize both the method names and type definitions.

This is no longer needed now that #34 was merged, right ?

This is no longer needed now that #34 was merged, right ?

#34 says "in conjunction with with #32", so I think this still needs to be merged.

Right. I guess this makes things safer anyways. Thanks for your input @tannerntannern !

Yyes, @tannerntannern is right. Both were branched from the same place.


StartsWith is not respecting IsCaseSensitive

Description
Turning off IsCaseSensitive does allow string equality checks to be case-insensitive. However, StartsWith is not honoring the flag.
Exception
N/A
Project
Below is full running project which reproduces and demonstrates the bug. The code follows the same pattern presented by [https://github.com/zzzprojects/EntityFramework-Effort/blob/master/Main/Source/test/Effort.Test.EF6/Features/StringFieldFixture.cs](these tests), however the tests do not specifically test StartsWith.
using System;
using System.Data.Common;
using System.Data.Entity;
using System.Linq;

namespace EffortBug
{
    public class Program
    {
        public static void Main(string[] args)
        {
            using (var effortConnection = Effort.DbConnectionFactory.CreateTransient())
            {
                using (var db = new EntityDbContext(effortConnection))
                {
                    db.Database.CreateIfNotExists();
                    effortConnection.IsCaseSensitive = false;
                    db.Tests.Add(new Test { Name = "aa" });
                    db.SaveChanges();
                    // Simple case-insensitive equality works
                    var results1 = db.Tests.Where(p => p.Name == "AA").ToList();
                    Console.WriteLine($"results1 count = {results1.Count}");
                    // However case-insensitive StartsWith does not work (works with SQL server)
                    var results2 = db.Tests.Where(p => p.Name.StartsWith("A")).ToList();
                    Console.WriteLine($"results2 count = {results2.Count}");
                    // StartsWith will work as long as it can be case-sensitive.
                    var results3 = db.Tests.Where(p => p.Name.StartsWith("a")).ToList();
                    Console.WriteLine($"results3 count = {results3.Count}");
                    Console.ReadLine();
                }
            }
        }
    }
    public class EntityDbContext : DbContext
    {
        public EntityDbContext(DbConnection dbConnection) : base(dbConnection, false) { }
        public virtual DbSet<Test> Tests { get; set; }
    }
    public class Test
    {
        public int Id { get; set; }
        public string Name { get; set; }
    }
}

Further technical details

EF version: 6.2.0
EF Effort version: 2.2.1
Database Provider: Effort (broken) / SQL Azure (working)

Looks like Contains and EndsWith also suffers from this. I discovered this bug while building unit tests against a larger project which uses both StartsWith and Contains.

Hello @Agendum ,
Thank you for reporting. We will look at it if that's possible to support.
Best Regards,
Jonathan

Performance Libraries
context.BulkInsert(list, options => options.BatchSize = 1000);
Entity Framework Extensions • Entity Framework Classic • Bulk Operations •  Dapper Plus
Runtime Evaluation
Eval.Execute("x + y", new {x = 1, y = 2}); // return 3
C# Eval Function • SQL Eval Function

Hello @Agendum ,
Just to give you an update.
We checked the issue but it looks to be harder to support then we though. Since the DbFunctions is static, we currently don't found a good way to pass the value set in the connection.
Meanwhile a better on is found, as a workaround, we could add a global variable for IsCaseSensitive. On our side, using a global variable make it very easy to support and would require us only a few minutes to code (already tested your code).
Would you like to have this solution as a fix?

The proposed solution sounds a lot like a 'DefaultIsCaseSensitive' for connections or something, which could be a way to make it forward compatible. For my situation that works just fine since I would set all connections to be case-insensitive. Thanks!

Hello @Agendum ,
The v2.2.2 has been released.
We finally take a few more days to make it the right way.
It should now work without your additional code.
Let me know if you find another issue.


zoomTo does not work consistently with animateTime > 0

Commit:
Steps to reproduce

Call zoomTo with an animateTime value greater than 0.

Observed behavior

Calling zoomTo causes the viewer to sometimes zoom to the wrong coordinates. The greater the value of animateTime, the more often that the viewer zooms to the wrong location.

Expected behavior

Calling zoomTo should consistently zoom to the same coordinates.

@chen650 Could you provide a full example. I am unable to reproduce this error. Also which version are you using.

I haven’t been able to reproduce it, and tried a variety of addresses (on and off axis), with times up to 10s. Closing.


Update signup-info

Description
update signup-info page to read content from cms.
I also removed the HTMList component in the home page and project page to use ReactMarkdown instead.
Test Instructions

run the project locally yarn dev
go to signup-info page
check if the page content displays properly
click buttons to see if navigation works properly
change language to see if French content is correct

Checklist

[ ] Strings use placeholders for translation (No hard coded strings)
[x] Unit tests have been added/updated
[x] Update CHANGELOG

Coverage Report Link for Branch https://dts-stn.github.io/Alpha-Site/refs/pull/493/merge/coverage/lcov-report

Thank you fo pointing this out, updated.


Fix typos in resources/PhoneNumberMetadata.xml

@felixonmars,
Thank you for contributing to libphonenumber!
We are not accepting pull requests for such metadata updates at this time. See our contribution guidelines.
We've accepted this issue and will fix this in an upcoming release. For your next metadata issue, we invite you to file issues in Google's new Issue Tracker, which we're currently testing.
Please note that you have submitted this information with the knowledge that we may use, modify, reproduce, publicly display, and distribute the information in accordance with our open source policies. Please let us know if you did not intend to submit the information for such use.

@felixonmars,
Fixed as part of version 8.8.8. Closing this PR as there are no further changes left.


Expanding/collapsing widgets

In addition to laying out widgets side-by-side as is currently possible, it is sometimes useful to collapse some widgets, or to expand one to the full screen. Mobile views address this in a way. Something similar should be available on the desktop.

I would love this. Along the same lines, allow manual sizing of widgets and a special widget called “button bar” to hold action buttons.

In the same vein (as I was pointed to here from this thread), it would be nice to be able to have the "menu" one gets after organizing the tables on the left folded by default.

Similar request here: https://community.getgrist.com/t/maximize-widgets-pages-sections-like-z-index/1808

Isn’t it fixed by 1dafe4bae0791eb60714fc0069180c62039ee16c ?

You're right @jperon, that specific request is now handled. I'm not 100% sure if that user is self-hosting, and this improvement hasn't reached our hosted service yet, otherwise it'd be good to tell them. In any case, we'll be publicizing maximizing in newsletter and community posts at end of month.


Is there planned BidirPT with GPU computing?

Hi. Just would like to know is there planned something like this or no?
Like this:
https://www.google.com/search?client=lightning&ie=UTF-8&oe=UTF-8&q=gpu+bidirectional+path+tracing

Duplicate of #85


Add reference to responsible rule in RULES.md

This resolves #730

No problem! If you want someone to work on adding the missing rules to RULES.md I'd be happy to help.

@joshuacolvin Yes, please take the lead on this! There's a lot of rules to document. Ideally, we'd have a justification for the rules where there's a meaningful reason.


Docs: nvidia containe toolkit and experiment docs

Add documentation about nvidia-container-toolkint installation and sorolla style experiment replicationç

Instead of mixing readme files,

Move original README.md to another file like README.orig.md
Move all new content to a file named README.org

I would pause the merge of this PR until we make docker work with CUDA without using sudo


[Feature Request] whereLike

Describe the feature
While we wait for the possibility of https://github.com/vuex-orm/plugin-search being ported to pinia-orm..
I tried doing something like this:
useRepo(Project).query().where('name', 'like', `%${this.query}%`).get();

I didn't get any errors, but it also didn't work.
It would be nice if you could add a whereLike helper.
I ended up going with this
useRepo(Project).query().where('name', (value) => {
  return value.toLowerCase().includes(this.query.toLowerCase());
}).get();

Which works well enough..
Thanks!
Additional information

[X] Would you be willing to help implement this feature?

Final checks

[X] Check existing discussions and issues.

I can try to add the keyword like to where clause. But i don't think i add a helper for it because i am orientanting on laravel syntax. And laravel doesn't have it.

As you wish. I've been using a whereLike macro in Laravel for many years now.. originally made popular through Spatie's twitter account.
For pinia-orm I ended up going for something like this:

    filteredPosts() {
      const query = useRepo(Post).query();

      let results = [];

      if (this.query.length >= this.minQuery) {
        query.where('title', (value) => {
          return value.toLowerCase().includes(this.query.toLowerCase());
        });

        results = query.get();

        this.queryResults.posts = results.length;
      }

      return results;
    },

But this clearly is nowhere near as convenient or versatile as vuex-orm's search plugin


Improve unit test coverage.

[ ] Ensure that we are not counting generated code in coverage metrics.
[ ] Add simple tests for public APIs to the specified package where they are only implemented in integration tests currently.
[ ] Add additional stories for test gaps discovered from the above.

Blocked by: fix coveralls.io 🎢


📄 Content - CSS quiz

https://sswcom-my.sharepoint.com/:w:/g/personal/jaydenalchin_ssw_com_au/EREIZVTJDx9MtZTSR-BSoxoBOa83MESn04KRnJ1F-jqN0Q?wdOrigin=TEAMS-MAGLEV.p2p_ns.rwc&wdExp=TEAMS-TREATMENT&wdhostclicktime=1704929610658&web=1
To Myself,

[ ] Add the CSS Quiz to the Admin Portal

Done - Quiz added in the portal


27 Add option to show 'goal' progress bar

https://github.com/PayButton/paybutton/issues/27

Let's change how it looks a bit:

margin-bottom: 0.6em on the top text.
Remove 'goal' text on the bottom.
Use margin-bottom: 20px instead of 25px on the progress bar.
Change the top text to Y.00 of X.00 BCH (2 decimals is probably good, bolded BCH). This matches up with what Flipstarter does.
Can we put the loading spinner/animation in place of the progress bar section while it's loading in? We want to stop the dialog/widget from jumping down when the progress bar appears.

When the amount value is 0, it's showing Send 0 BCH at the top instead of Send any amount of BCH like before.
X / Y BCH value still doesn't update on payment, only after 1 confirmation.


rename "LED display" service

As discussed, the "LED display" generally refers to a matrix of LEDs, or at least the ability to show numbers (like a clock). But, this service is really for short strips of LED, so needs a better name. Suggestions welcome:

LED small strip
???

I agree that it will be better to rename both of them.

Done.


bug

Hi, In train.py there is a problem, the class "DualNetwork" is not defined.

Hi,
We do not recommend use this code or (https://github.com/huanzhang12/CROWN-IBP/blob/master/train.py) to run CAP (https://arxiv.org/abs/1711.00851). We recommend using the original implementation (https://github.com/locuslab/convex_adversarial) for CAP.
Quick Fix:
You may comment out the lines 203 and 441 where DualNetwork is used, or delete the whole CAP parts
if kwargs["bound_type"] == "convex-adv":
    ...

We will fix the bug accordingly.
Thanks,
Sungyoon Lee


Add all folder and file names standardized to use dashes over underscore

Related Issue(s)
Resolves: https://github.com/kyverno/policies/issues/551

Description
Wrote a scripts to add all folder and file names standardized to use dashes over underscore

Checklist

[x] I have read the policy contribution guidelines.
[ ] I have added test manifests and resources covering both positive and negative tests that prove this policy works as intended.

the script code I write for this task is:

#!/bin/bash

dir=$1

find $dir -depth -name "*_*" | while read file; do
  mv -v "$file" "$(dirname "$file")/$(basename "$file" | sed 's/_/-/g')"
done

and then cd and
bash ./rename.sh /Users/haoyuguo/Desktop/policies

@chipzoller The test said this: failed to process ../policies/best-practices/add-network-policy/add_network_policy.yaml: stat ../policies/best-practices/add-network-policy/add_network_policy.yaml: no such file or directory
I think the policy test just contradicts with the code when i change all the underscore to dashes.

You will need to update all kyverno-test.yaml files to update the new names of the policies so the tests, which fire upon every PR, can run as expected. See an example of such a file and where that needs to be changed here.


Suggestion to provide a link to create an external virtual switch in Hyper-V manager

[Enter feedback here]

In Choose Network, specify the virtual switch that the VM will use. The switch needs internet connectivity to send data to Azure.

It's a good idea to provide a link to create an External Switch that provides internet connectivity
https://docs.microsoft.com/en-us/windows-server/virtualization/hyper-v/get-started/create-a-virtual-switch-for-hyper-v-virtual-machines

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

ID: 6b96edcb-267b-372c-3156-24d4320f4fc3
Version Independent ID: 7fa9f4d1-187f-ec97-fad6-ecad575b05ed
Content: Assess Hyper-V VMs for migration to Azure with Azure Migrate - Azure Migrate
Content Source: articles/migrate/tutorial-assess-hyper-v.md
Service: azure-migrate
GitHub Login: @rayne-wiselman
Microsoft Alias: raynew

Hi @kumarallamraju Thanks for your feedback. We will review and update as appropriate.

@rayne-wiselman To review and update as appropriate.

Thanks for your feedback! I've added a link - https://docs.microsoft.com/en-us/azure/migrate/tutorial-assess-hyper-v#create-the-appliance-vm
#please-close


Add game-of-life exercise

I think this should work, but I can't seem to get the tests passing.
Also, let me know if using a matrix is a good idea here, or if I should use something else

That made me think for a while!
It seems that the intersect() on line 6 isn't doing quite what we expect. On debugging, neighbor_coords is an empty list, which is obviously wrong (or at least unintended).
Now that we have tidyverse in the test runner, it is possible to add library(dplyr) at the top of the file, then replace intersect with inner_join. All tests passed when I tried this locally.

I suspect @jonmcalder would have a better solution, if he is around.

Also, using a matrix looks fine to me. Maybe my judgement is warped by long experience with Matlab and NumPy, but I don't see why it shouldn't also be idiomatic R. This data type has been in the base language for a long time.

Thanks @colinleach! I'll wait for @jonmcalder to see if he agrees too, and then we I'll continue fixing things.

Yes I agree that it makes sense to use a matrix implementation for this.
Colin is correct that you can't use intersect() in that context because it's meant to work on vectors (and expand.grid() produces a data frame). Using either dplyr::intersect() or dplyr::inner_join() instead would work here.
I don't have a different approach to suggest right now (and can't give more time to this) but I think there are some interesting approaches here if you want to explore a little more.
The implementation and tests are the main thing and in my view the example solution only really needs to demonstrate a possible way to solve the exercise (ideally as elegantly and idiomatically as possible but that can be subjective). So I'm sure I'll be happy with whatever you guys decide here as long as you both are happy.

Minor suggestion: maybe the linting should be moved to its own, separate GHA workflow. That will allow the contributor to verify that the tests are passing without having to satisfy the linter.

ideally as elegantly and idiomatically as possible but that can be subjective

I'll happily let others submit a PR to improve upon my solution :)

Minor suggestion: maybe the linting should be moved to its own, separate GHA workflow. That will allow the contributor to verify that the tests are passing without having to satisfy the linter.

Yes probably a good suggestion. Out of interest have did you consider using {styler} and/or {precommit} to help avoid linting issues as outlined in the (now very outdated I'm sure) ReadMe?

I hadn't done that. Probably should have. Sorry

@jonmcalder I've fixed CI

no offence of course @ErikSchierboom

None taken! There must be someone that can do this a ton better.

I'm merging this to have it be available, and then the updated example solution can be PR'ed later.

I hadn't previously thought about the impact on CI, after adding tidyverse to the test runner. I suspect that at some point we will want more than just dplyr available. I'm thinking particularly of stringr: a pretty major improvement on string handling in base R.
I understand the bias towards keeping CI lean, but what are the practical implications of adding more packages? Slower? More expensive?


Is there support for getting transaction history or updates using Javascript-SDK?

Hello,
I have implemented the Javascript-SDK, using which I was able to perform transfer, getAccount, getBalance etc. However, I couldn't find a way to receive transaction updates either via REST API or Websockets in the Javascript-SDK.
Can we use our own local node with the Javascript-SDK?
For example: I am creating the instance of the BnbClient using below line of code:
const bnbClient = new BnbApiClient('https://testnet-dex.binance.org/');
Regards,
Saarang Tiwari

Hello,
I have implemented the Javascript-SDK, using which I was able to perform transfer, getAccount, getBalance etc. However, I couldn't find a way to receive transaction updates either via REST API or Websockets in the Javascript-SDK.
Can we use our own local node with the Javascript-SDK?
For example: I am creating the instance of the BnbClient using below line of code:
const bnbClient = new BnbApiClient('https://testnet-dex.binance.org/');
Regards,
Saarang Tiwari

hello,
the JS sdk doesn't support websocket. but it's support in https://github.com/binance-chain/go-sdk
You need to connect to HTTP/API: https://docs.binance.org/api-reference/dex-api/paths.html.
so you cannot use your own node.
Cheers ^^


Implement Gauss-MNMF

Summary
This enables you to use Gauss-MNMF.
ref #34
TODO

Implement t-MNMF
Implement FastMNMF

Codecov Report
Base: 94.94% // Head: 95.03% // Increases project coverage by +0.09% :tada:

Coverage data is based on head (81f85ea) compared to base (c7672a9).
Patch coverage: 96.16% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #213      +/-   ##
==========================================
+ Coverage   94.94%   95.03%   +0.09%
==========================================
  Files          31       32       +1
  Lines        3798     4110     +312
==========================================
+ Hits         3606     3906     +300
- Misses        192      204      +12

Impacted Files
Coverage Δ

ssspy/bss/mnmf.py
96.14% <96.14%> (ø)

ssspy/bss/ipsdta.py
89.71% <100.00%> (+0.01%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Upgrade to latest rust-web3 version

At the moment Vibranium uses rust-web3 v0.6.0. There seem to be some problems with that version, e.g. example of the project itself don't seem to properly work.
It might make sense to upgrade to some latest HEAD and check if it solves existing problems.

Unfortunately, upgrading to latest HEAD doesn't solve the issue that transactions don't seem to resolve. This needs further investigation.


Outdated packages & download problems

binutils 2.29.1
libgcrypt 1.8.1
fribidi - has moved to github?
and please change ftp and if possible http to https download links.

Fixed.
HTTPS doesn’t really make much of a difference since the hashes are known, but I did it anyway. Can’t hurt. Now if we could get git signature verification in CMake’s ExternalProject module …


Activated signing key shows as inactive in Azure portal

I activated the signing key and the PreferredTokenSigningKeyThumbprint shows in the servicePrincipal object.
Still in Azure portal in Enterprise Apps -> SSO -> SAML -> SAML Signing Cert the certificate is shown as "Inactive".
I cannot confirm (yet) that the key works in this state.
Either this is works-as-designed visibility issue then it should be at least mentioned in the docs
OR it is a bug and should be fixed.

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

ID: aa04cf91-e1eb-5b59-f85f-1cb46a08f40b
Version Independent ID: a4aff3ba-9c3f-71c5-bf8b-a4101b94b073
Content: Use Microsoft Graph APIs to configure SAML-based single sign-on - Azure Active Directory
Content Source: articles/active-directory/manage-apps/application-saml-sso-configure-api.md
Service: active-directory
Sub-service: app-mgmt
GitHub Login: @kenwith
Microsoft Alias: kenwith

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

When the correct keytype "X509CertAndPassword" is used, the activation works.


Bug title

Which version of the app?
WinUI 3 Gallery
Description
No response
Screenshots
No response
Windows version
No response
Additional context
No response

Closing as this is a blank issue.


Fix varchar length in createDF

I fixed a logic for counting varchar length in createDF.
Current logic is only for single byte characters. If the data is multibyte  characters like Japanese, the varchar length with current logic is not correct, it is very short.
I fixed to counting byte length with utf-8 string as varchar length.

@baklarz Is it possible for you to merge this pull request? If you have any questions, please let me know. Thanks.

Actually, I have a number of updates regarding UTF-8 characters etc... We had to analyze some data that included Unicode 2+ byte characters and ended up creating an extension to the creation command:
USING df CREATE TABLE .... NCHAR x or PADDING x and CHARTYPE x
NCHAR x / PADDING x would take the size of the character string returned by Pandas and "Pad" or expand the size of the VARCHAR that we generate by this amount. So if Pandas tells the program the field is 10 characters wide, a PADDING of 2 would generate VARCHAR(20) instead of VARCHAR(10). You could also use fractions so that NCHAR 1.5 would result in VARCHAR(10) becoming VARCHAR(15).
Alternatively, you can also use the CHARTYPE x option to change the default character format from VARCHAR to whatever you want. So you could say CHARTYPE NCHAR or CHARTYPE VARGRAPHIC which would result in the program generating VARGRAPHIC(10) (without padding) or NCHAR 1.5 CHARTYPE VARGRAPHIC resulting in VARGRAPHIC(15).
I haven't uploaded this code yet but will do so shortly. You have any additional suggestions on how to handle these Unicode character sets?
The other options we were adding include INTTYPE, FLOATTYPE, CHARTYPE, CLOBTYPE, BLOBTYPE, ... so that you can specify exactly how you want the Pandas columns to be represented in Db2. Pandas causes a number of issues when trying to determine the "real" format of a column so sometimes you have to just create the table using the CREATE command (without data), modify the table definition and then use the refresh option to load the data into the table.

@baklarz Thank you for your detail comments. I understand you have a plan to fix.
My colleague had a hands-on workshop today.  He used my fixed version today because there are issues for creating table from DataFrame if it has Japanese characters. We have an alternative module db2.ipynb in my branch, so we can wait your official fix.

FYI - I did update the code with your UTF suggestion. Thanks for that! I've updated the code with additional options for changing the data types that are used for columns and an additional PADDING function in case the UTF length is insufficient for the column.


can we convert back tflite model to keras h5 format?

hi there, thanks for this powerful tool.
can we convert back tflite model to keras h5 format?
i find a solution here, but the weight in bn layer in tflite is null, how can we convert back this?

@feiwofeifeixiaowo
I think there is no way to convert tflite model to keras h5 format as some information will be lost after conversion.
Can you refer the link and see if it helps you.
This question is better asked on StackOverflow since it is not a bug or feature request. There is also a larger community that reads questions there and provide better and faster support for such issues. Thanks!

ok, thanks.


Stacks index page

After some thought, I think having an actual stacks index page is a better UX than using a dropdown for stack switching.

@sandersonet Would the "Apps & Databases" link in the nav take on to this screen?
And would this become the default landing screen when a user signs in?

@bantic  I am actually working on this issue right now.  I'll have a PR soon.
Apps & Databases is the default and will link to a new stacks.index route.


HystrixObservableCommand Emit and FallbackEmit hook

Add hooks for the 2 new events, as defined in #302

Completed in #683


ionic vue background geolocation tracking stops after 5 minute

Describe the bug
I am using plugin
https://github.com/seididieci/capacitor-backround-geolocation
to watch the user's location. then I am tracking that location with the help of a pusher. this background location works only for 5 minutes after that it just stops. I am using Capacitor's Background task. But that plugin also keeps data on phone after the user opens the app. the Background task sends data to the pusher.
Here is watch location function
  getLocation: async function () {
      BackgroundGeolocation.initialize({
        notificationText: "Your app is running, tap to open.",
        notificationTitle: "App Running",
        updateInterval: 10000,
        requestedAccuracy: BgGeolocationAccuracy.HIGH_ACCURACY,
        // Small icon has to be in 'drawable' resources of your app
        // if you does not provide one (or it is not found) a fallback icon will be used.
        smallIcon: "ic_small_icon",
        // Start getting location updates right away. You can set this to false or not set at all (se below).
        startImmediately: true,
      });
      // const geolocation = new Geolocation.Geolocation();
      BackgroundGeolocation.addListener("onLocation", (location) => {
        // console.log("Got new location", location);
        this.subscribe(location.longitude, location.latitude);
        console.log(location)
      });

      BackgroundGeolocation.addListener("onPermissions", (location) => {
        // console.log("BGLocation permissions:", location);
        this.subscribe(location.longitude, location.latitude);

        // Do something with data
      });
      BackgroundGeolocation.start();
    },

Then calling function in mounted()
mounted(){
this.getLocation
  App.addListener("appStateChange", (state) => {
       setInterval(this.getLocation, 120000);
      if (!state.isActive) {
        BackgroundTask.beforeExit(async () => {
          setInterval(this.getLocation, 120000);
          console.og('Why')
        });
      }
      if (state.isActive) {
        setInterval(this.getLocation, 120000);
        console.log('Active')
      }
    });
}

To Reproduce
Steps to reproduce the behavior:
just go in background it will work only for 5 minutes
Expected behavior
It should keep send location on server
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):
Android 10

I updated the documentation so it is more explicit.


Erronious output generated with --debug --formatting=PRETTY_PRINT --compilation_level ADVANCED_OPTIMIZATIONS set

I'm getting strange output with $s and incorrect formatting as well as syntax errors, it's supposed to simply print longer names and in a viewable format for debugging the output.

IIRC --debug is for debugging the compiler, so unless you're hacking on the compiler itself you probably don't need it.

the documentation of the compiler said it was for debugging output

and --formatting=PRETTY_PRINT isn't working

I use this all the time to debug - both flags.
Can you provide a repo case?
On Apr 15, 2015 6:18 PM, "Patrick Martin"<EMAIL_ADDRESS>wrote:

and --formatting=PRETTY_PRINT isn't working
—
Reply to this email directly or view it on GitHub
https://github.com/google/closure-compiler/issues/908#issuecomment-93595159
.

i'm uncertain as to what specifically is causing this bug to occur with my .js files, considering that the one that's been giving me the most trouble is over 950 lines long, it will take me a while to come up with a repo case, the flags i'm using are:
--debug --formatting=PRETTY_PRINT --compilation_level ADVANCED_OPTIMIZATIONS

and here is a snippet from the beginning of the output closure compiler creates from that 950 line file showing the strange abundance of dollar signs:
Object.create(window.hbmp.renderer_prototype, {$_last_time$:{value:-1, writable:!0}, $_last_events$:{value:"", writable:!0}, $_animating$:{value:!1, writable:!0}, $_scaleRatio$:{value:1, writable:!0}, $_display$:{value:null, writable:!0}, $_context$:{value:null, writable:!0},

also note that everything came out on one line instead of pretty-printed, i'm using the latest version of the compiler.

"$" in debug output is totally OK. This is how compiler denotes renamed identifiers.
Pretty print is another story. What was the source you compiled? Did you try compiling with stable version? I frequently use debug & pretty print and never had problems.

Also try disabling the type based optimizations
- - use_types_for_optimization false

ah ok i didn't know about the $ thing, as for the pretty print, i'm using the compiler from this url: http://dl.google.com/closure-compiler/compiler-latest.zip
the source I compiled is part of the javascript based video player i've been working on for the past year and a half, i've been continually testing both the un-compiled and closure compiled versions, however when i updated my closure compiler today it broke the player, so i went to go debug it and the pretty print didn't work.

The latest compiler version enables the type based optimizations by default. Turning those off will more than likely fix your problem.


Use the new auto-synced json schema files

What does this pull request do?
Fixes the test suite to use the new auto-synced json schema files. Also cleans up a lot of scaffolding around make update-json-schema
Related issues
Ref #1053

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Build Cause: Pull request #1055 opened

Start Time: 2021-02-25T17:10:07.235+0000

Duration: 23 min 1 sec

Commit: a83b800fbd585ede2c757363aad17e2f45793a91

Test stats :test_tube:

Test
Results

Failed
0

Passed
7947

Skipped
5735

Total
13682

Trends :test_tube:

:green_heart: Flaky test report
Tests succeeded.

Expand to view the summary

Test stats :test_tube:

Test
Results

Failed
0

Passed
7947

Skipped
5735

Total
13682


Check build on OTP23

Pull Request Test Coverage Report for Build 1452

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.04%) to 80.236%

Files with Coverage Reduction
New Missed Lines
%

src/brod_kafka_apis.erl
1
76.0%

Totals

Change from base Build 1447:
0.04%

Covered Lines:
2245

Relevant Lines:
2798

💛  - Coveralls


SA1009 must not be triggered when the closing parenthesis followed by '>' operator

Next code reports SA1009 warning but should not:
collection.Where(g => g.Count() > 1);

Yep, accidentally introduced this when fixing #2267.

Added tests to cover failure: #2410


unable to run requirement.txt command

OPTIONAL PACKAGE DATA
dlls: no  [skipping due to configuration]
============================================================================
                        * The following required packages can not be built:
                        * freetype, png
----------------------------------------

ERROR: Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.

This command did the trick:
apt-get install libfreetype6-dev source

@jfrery, I tried this solution, and it worked to get rid of the original error. But now I've got this one:
note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed

× Encountered error while generating package metadata.
╰─> See above for output.

Does anyone have a solution for it?
I don't know if that's a too generic information, if so, ping me so I can provide further information. Thanks in advance!!