image size changed

Hello I am using imgaug do image augmentation on my image set and my code is like this
`for char in path:
subdir = root+'/'+ data +'/'+char
childpath = os.listdir(subdir)
for child in childpath:
subpath = data+'/' +char + '/'+child
d = ' %s' %(i)
t = subpath + d
img_pil = Image.open(subpath)
images = np.array(img_pil)
seq = iaa.Sequential([iaa.GaussianBlur((0, 0.5)),
iaa.ContrastNormalization((0.75, 1.5)),
iaa.AverageBlur(k=(2, 7))
])
    images_aug = seq.augment_images([images])[0]
    new_img = np.array(images_aug)
    scipy.misc.imsave(subpath +'au.jpg', new_img)
    print new_img.shape

`
my input image is 32by32by3 but the size of output image became 33by32by3??
anyone know what happened here?

Can you check whether your input images are really all 32x32x3 and maybe write a minimal working example? I tried the below (similar) code and didn't get any such problem:
from __future__ import print_function
from imgaug import augmenters as iaa
import imgaug as ia

image = ia.quokka(size=(32, 32))

seq = iaa.Sequential([
    iaa.GaussianBlur((0, 0.5)),
    iaa.ContrastNormalization((0.75, 1.5)),
    iaa.AverageBlur(k=(2, 7))
])

for _ in xrange(100):
    image_aug = seq.augment_images([image])[0]
    print(image.shape, image_aug.shape, "changed" if image.shape != image_aug.shape else "OK")

Output:
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK
(32, 32, 3) (32, 32, 3) OK

hello! thank you for reminding me to check out my images. Just found the reason is the image generate code has some problem so some images became (32,33,3). The augmentation part worked fine... sorry

If it is just a single image you can do something like
image_aug = seq.augment_image(image)
scipy.misc.imsave("save/to/directory/image.jpg", image_aug)

or
image_aug = seq.augment_images([image])[0]
scipy.misc.imsave("save/to/directory/image.jpg", image_aug)

If it is multiple images you can do something like
images_aug = seq.augment_images(images)
for i, image_aug in enumerate(images_aug):
    scipy.misc.imsave("save/to/directory/%06d" % (i,), image_aug)

thank you so much, that helped a lot. I was wondering for the shear and rotate in Affine . Can I fill the background with 255(white pixel)? I read the Docs but cannot find the parameter about that in those function

That should work with Affine(shear=..., rotate=..., mode="constant", cval=255) . Fills all newly created pixels with RGB (255, 255, 255).


I do /d creeper... and my server reboots.

I do /d ... and the server crashes.
Hmm....

What version are you using?

the last one posted.i jsut download it today
On Mar 19, 2017 9:56 PM, "Lazerplayz"<EMAIL_ADDRESS>wrote:

What version are you using?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Falkirks/DisguiseMe/issues/57#issuecomment-287665962,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQkQCOknzG18SZC7qdCqHyYnsT81XQHxks5rndzQgaJpZM4Mh6nx
.

Any errors or crashdumps and what software please provide detail :)

no crash dumps, just reboots the server when I go to switch from a mob to
the next, Tesseract... latest.
On Mar 19, 2017 10:01 PM, "Lazerplayz"<EMAIL_ADDRESS>wrote:

Any errors or crashdumps and what software please provide detail :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Falkirks/DisguiseMe/issues/57#issuecomment-287666283,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQkQCF_Nn9iwDLljF1J4CJzjGudZscZ9ks5rnd3igaJpZM4Mh6nx
.

Are you using the latest GitHub release or the code in the repo?

latest github
On Mar 22, 2017 3:02 PM, "Falkirks"<EMAIL_ADDRESS>wrote:

Are you using the latest GitHub release or the code in the repo?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Falkirks/DisguiseMe/issues/57#issuecomment-288505537,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQkQCODh8oXYsX7eKBQSohtMaK_-6QqHks5roXBPgaJpZM4Mh6nx
.

confirmed on the latest build of tesseract (mcpe 1.0.5) and the latest version of DisguiseMe (not a github release). just do /d (anymob) then do again /d (anymob) and ur server will crash .

also . when u do /d (anymob) then stay disguised for a while 1~5 minutes the server crashes with no crashdumb

Weird this happens on Pocketmine?

I found this problem.


Update scala example neural-style

Follows the changes in python example. #1964

Nice work! Please update the branch then I'll merge.

BTW, mind telling how you finally fix the CI build? https://github.com/travis-ci/travis-ci/issues/6019

@javelinjs  Actually I am not sure how the problem is solved.
I just separated the old pr into two( one has changed the scala core and the other is this) and push them respectively.
And I also noticed the pr #2086 recently, not sure if it is related to the problem.


FT: Add PartNumber support for getObject

Depends on https://github.com/scality/Arsenal/pull/309.

@ironman-machine try

Hello @bennettbuchanan
Starting end to end procedure using the following payload:
{"REPO_NAME": "S3", "DEFAULT_BRANCH": "master", "SCALITY_S3_BRANCH": "ft/S3C-635/support-mpu-for-big-files"}
Please follow http://ci.ironmann.io/gh/scality/Integration/14112 for CI status.

@ironman-machine try

Hello @bennettbuchanan
Starting end to end procedure using the following payload:
{"REPO_NAME": "S3", "DEFAULT_BRANCH": "master", "SCALITY_S3_BRANCH": "ft/S3C-635/support-mpu-for-big-files"}
Please follow http://ci.ironmann.io/gh/scality/Integration/14130 for CI status.

@ploki I have run these tests successfully with AWS_ON_AIR=true.

@ironman-machine try

Hello @bennettbuchanan
Starting end to end procedure using the following payload:
{"REPO_NAME": "S3", "DEFAULT_BRANCH": "master", "SCALITY_S3_BRANCH": "ft/S3C-635/support-mpu-for-big-files"}
Please follow http://ci.ironmann.io/gh/scality/Integration/14131 for CI status.

Great! thanks!

Hello @bennettbuchanan
Starting end to end procedure using the following payload:
{"REPO_NAME": "S3", "DEFAULT_BRANCH": "master", "SCALITY_S3_BRANCH": "ft/S3C-635/support-mpu-for-big-files"}
Please follow http://ci.ironmann.io/gh/scality/Integration/14132 for CI status.

@ironman-machine try

Hello @bennettbuchanan
Starting end to end procedure using the following payload:
{"REPO_NAME": "S3", "DEFAULT_BRANCH": "master", "SCALITY_S3_BRANCH": "ft/S3C-635/support-mpu-for-big-files"}
Please follow http://ci.ironmann.io/gh/scality/Integration/14133 for CI status.


Tenant UI for non-Indy exchanges

TBD, see https://github.com/bcgov/traction/pull/1280#pullrequestreview-2205556919

Would we want Tenant UI functionality for VC-API endpoints?
https://github.com/hyperledger/aries-cloudagent-python/blob/main/docs/features/JsonLdCredentials.md#vc-api


Fix page not found error in CONTRIBUTION.md

Fixes # .
Description
Fix the "page not found" error in CONTRIBUTION.md.
Status
Ready
Please ensure all the checkboxes:

[x] Codeformat tests passed locally by running ./runtests.sh --codeformat.

/build


Seperate BWC tests into parallel workflow

Description
BWC tests were being run after normal tests for each JDK.  This was
redundant and we can shorten the test runtime by running them in
parallel
Check List

[ ] ~New functionality includes testing~
[ ] ~New functionality has been documented~
[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #1706 (14a4fec) into main (782b042) will decrease coverage by 0.05%.
The diff coverage is n/a.

:exclamation: Current head 14a4fec differs from pull request most recent head 86af36c. Consider uploading reports for the commit 86af36c to get more accurate results

@@             Coverage Diff              @@
##               main    #1706      +/-   ##
============================================
- Coverage     62.85%   62.79%   -0.06%
+ Complexity     3259     3258       -1
============================================
  Files           253      253
  Lines         18082    18082
  Branches       3242     3242
============================================
- Hits          11365    11355      -10
- Misses         5063     5073      +10
  Partials       1654     1654

Impacted Files
Coverage Δ

...security/auditlog/sink/InternalOpenSearchSink.java
69.23% <0.00%> (-11.54%)
:arrow_down:

...org/opensearch/security/rest/TenantInfoAction.java
77.94% <0.00%> (-10.30%)
:arrow_down:

Continue to review full report at Codecov.

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


Dependcy analysis not working in v0.5.1

Caused by adding object type to object path

fixed with 0.5.2


Is this still maintained?

I was trying to use sphinx-licenseinfo and sphinx-toolbox, but first of all sphinx-toolbox on conda-forge seems to be outdated, and sphinx-licenseinfo seems to force a downgrade of sphinx 7.2.6 to an older version 5 (even when pip installed).

pypi.org page clearly shows this package is unmaintained. Needs to be fork'ed, renamed, and the requirements issue fixed
Use this Sphinx extension for logging-strict and just learned that the sphinx-licenseinfo requirements needs to be fixed.
Have my eyes on this package


Could I establish my PNN on simulation platform such as PSCAD/EMTDC rather than a real circuit?

Hello!
Thank you for your contributions in PNN.
I am very excited to read this paper, which may be applied in my subject for fault diagnosis in power system.
I want to establish a similar PNN on the simulation software like PSCAD/EMTDC first, or have ever implement PNN in some simulation software before?

Hi Jianhong-Gao,
if the simulation software has a Python API, you can use our code to test the performance of PNNs using the simulation software. However, you always additionally need an auto-differentiable model of your simulation written in pytorch to use our code.
from pat import make_pat_func

def f_forward(inputs, parameters):
    # use API to simulation software
    # return outputs from simulation software

def f_backward(inputs, parameters):
    # pytorch (!) code with differential model of f
    # return outputs from differentiable model

pat_func = make_pat_func(f_forward, f_backward)

You cannot use other simulation software than pytorch in f_backward. During the work on our paper, we used non-pytorch simulation software in f_forward and built a blacbox differentiable model of these simulations using deep neural networks for f_backward. These models were conceptially much simpler than the full forward simulation. Please see the section on 'Differentiable digital models' in the 'Methods' section of our paper.
Let me know if you have further questions on this.

Dear Authors：
     Thank you for your response.
     I am the PHD student in Fuzhou University, China.
     We want to apply your work on the fault diagnosis in real-time to detect the wave-head of traveling wave in power system to change the current paradigms of fault diagnosis.
     The further work would be shared with you.
     By the way, if possible, I wonder if there is any chance to cooperate in writing papers after we applied your work in my research successfully? :)

With Best Regards
Jian-Hong Gao (高健鸿)
College of Electrical Engineering and Automation, Fuzhou University
Address: 2, Xue-Yuan Rd., Minhou, Fuzhou, China
Mobile: +86-13685012269
WeChat:<PHONE_NUMBER>9
E-mail: @.***


API doesn't work on Google App Engine for Business projects

These are projects with IDs of the form "example.com:project-id".

Fixed in Cloud Datastore v1beta3. More information: https://cloudplatform.googleblog.com/2016/04/Google-Cloud-Datastore-gets-faster-cross-platform-API.html


feat: create podmonitor cr in helm chart

Ⅰ. Describe what this PR did

create a template function named gateway.podTemplate to rendering pod template spec, so we can keep daemonset.yaml and deployment.yaml DRY
support create a podmonitor/vmpodscrape customresource for monitoring purpose

Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
try helm template --set gateway.metrics.enabled=true xxxx helm/core/ to verify.
Ⅴ. Special notes for reviews

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@fengxsong Please resolve the conflicts

@johnlanni PTAL

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

Project coverage is 44.31%. Comparing base (ef31e09) to head (0da389a).
Report is 108 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1157      +/-   ##
==========================================
+ Coverage   35.91%   44.31%   +8.40%
==========================================
  Files          69       75       +6
  Lines       11576     9823    -1753
==========================================
+ Hits         4157     4353     +196
+ Misses       7104     5142    -1962
- Partials      315      328      +13

see 90 files with indirect coverage changes


Add limitations section to README

In particular point out #12.

Not really relevant now that #12 is implemented.


Migrate existing extractors to CLI

We can start with the Freesurfer one and check implementation difficulty for the other ones (functional networks, etc.).
Still need to figure out some things regarding how to manage software dependencies and containers, etc.
One idea would be for extractors to be scripts that are not Nipoppy-aware, so the inputs are just paths to files/directories that are pipeline outputs. But the outputs would have to follow some standard (e.g. subjects/session labelling). Then we could have a Boutiques descriptor (e.g. freesurfer-extractor.json) for it, and the extractor workflow would loop over subjects/sessions who completed the pipeline, run the extractor script for all of them, then potentially aggregate them into a single file (or not).

Some pending questions:

Does it make sense to think of pipelines like this:

Anything that Nipoppy executes with Boutiques is considered a pipeline
Input/output directories for each type of pipeline

BIDS conversion pipelines: sourcedata --> bids
Processing pipelines: bids --> derivatives
Extractor pipelines: derivatives --> derivatives (potentially: derivatives/output --> derivatives/idps)

How do we version extractor scripts that we develop/maintain?
Where do users put extractor scripts if they are not containerized?
How do we configure extractors in the global config file? Within a pipeline config or parallel to BIDS_PIPELINES and PROC_PIPELINES?

@nikhil153 thoughts?

From a chat with @mathdugre:

Upload the descriptors on Zenodo so that they work with normal Boutiques
Put the sample invocation and pipeline config in the docs for people to copy (possibly with an automated way to copy them into their own Nipoppy datasets)

Summary of chat with @nikhil153 today:
We want extractor outputs to be in the idps subdirectory of a processing pipelines:
<ROOT>/
└── derivatives/
    └── <PIPELINE_NAME>/
        └── <PIPELINE_VERSION>/
            ├── output
            └── idps

Does it make sense to think of pipelines like this:

Anything that Nipoppy executes with Boutiques is considered a pipeline
Input/output directories for each type of pipeline

BIDS conversion pipelines: sourcedata --> bids
Processing pipelines: bids --> derivatives
Extractor pipelines: derivatives --> derivatives (potentially: derivatives/output --> derivatives/idps)

Yes

How do we version extractor scripts that we develop/maintain?

Version them with the same version as the processing pipeline they are for, with extra patch version for bug fixes

Where in the Nipoppy dataset do users put extractor scripts if they are not containerized?

Inside the code/scripts directory (need the new spec change)

How do we configure extractors in the global config file? Within a pipeline config or parallel to BIDS_PIPELINES and PROC_PIPELINES?

Parallel, with extra fields for linking with the associated with processing pipelines
Feedback welcome! @bcmcpher


is the faq for question "How to check if the migration is finished" not updated?

for version develop, "cc226f841e2e244c48246ebfcfd5a50396b59710,go1.15.7,2021-09-03_10:06:55", no metrics called "PullCommandTotal, BypassCommandTotal, PushCommandTotal" exist, and should it be "PullCmdCountTotal, BypassCmdCountTotal, PushCmdCountTotal" instead?

Yes, the old wiki is not used anymore, please use the new version of redis-shake: https://github.com/alibaba/RedisShake/tree/v3


tsdb/engine/tsm1: Check os.Open error before using file.

[x] Rebased/mergable
[x] Tests pass
[ ] CHANGELOG.md updated
[x] Sign CLA (if not already signed)

os.Open is documented as:

Open opens the named file for reading. If successful, methods on the returned file can be used for reading;

That suggests the file's methods should only be called if opening was successful. The original code would defer f.Close() right after os.Open, before ensuring that err is nil, so f.Close() would run even if os.Open did not return successfully.
Apply https://github.com/golang/go/wiki/CodeReviewComments#indent-error-flow suggestion to keep the normal path at minimal indentation, and indent the error handling code instead. This improves code readability.
(This is my first PR to InfluxDB, and I welcome any feedback/criticism for how I can do better.)

@shurcooL The pull request looks great. Thanks! The error handling looks good and the indented error flow also works much better than before. :+1:

👍 Thanks @shurcooL!


Small changes to the installation docs

I'm fairly new to running in docker containers and this had me confused for the best part of an evening.

Thanks!


jetty: 11.0.16 -> 12.0.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/jetty/versions.
meta.description for jetty is: A Web server and javax.servlet container
meta.homepage for jetty is: https://www.eclipse.org/jetty/
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 12.0.1 in filename of file in /nix/store/drai5a9vrq41d6hn3xadag53lcylrvdq-jetty-12.0.1

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
jetty

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/drai5a9vrq41d6hn3xadag53lcylrvdq-jetty-12.0.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A jetty https://github.com/r-ryantm/nixpkgs/archive/7e549d52e4f521a55088915ddd66bb6bbf86e87f.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/7e549d52e4f521a55088915ddd66bb6bbf86e87f#jetty

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/drai5a9vrq41d6hn3xadag53lcylrvdq-jetty-12.0.1
ls -la /nix/store/drai5a9vrq41d6hn3xadag53lcylrvdq-jetty-12.0.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

jetty

Maintainer pings
cc @emmanuelrosa for testing.

The Jetty 12 upgrade come with some breaking changes leading plantuml-server to fail. https://github.com/NixOS/nixpkgs/issues/262893
I don't know how to fix these problems other than downgrading Jetty, would it be possible to add a Jetty 11 package?

@ryantm This Jetty package is updated by nixpkgs-update, which appears to get version information from repology.
If I were to add a top-level attribute for a secondary version of jetty (to mitigate API incompatabilities) my understanding is that nixpkgs-update would update both packages to the same version; The latest version. Is this correct? If so, it looks like my best bet is to add update scripts to both packages.


Add missing install step

Fixes #3

ping @lonnieking ;)

Sorry for the delay! Thanks for adding the correction!


sechud is night vision

Description

job icons are visible in the dark
Reproduction

wear sechud and have nothing in your active hand to see job icons
Screenshots

Additional context

additionally, secglasses arent hud, which seems strange to me as they did in ss13

Talked about it with emo but i'm like 80% it's another overlay not clearing its shader so

IMO it makes sense for sechud icons to be visible in the dark. My interpretation was always that its an actual display built into a pair of smart glasses that scan for ID cards. The glasses don't require light to read ID cards, and the smartglasses can show you the information even in the dark.
However status icons should only be unshaded if it makes sense (i.e., HUD glasses), so the shading should probably be configurable per icon type. That way rev/cult icons can be obscured when someone is lurking in a dark room.

its an unfair advantage to be able to see icons in pitch black

do we want this to be a common occurence?

do we want this to be a common occurence?

Given headrevs have flashes and wearing a sechud doesn’t give you flash immunity, I think this is honestly fine. Gives sechud more of a purpose over the sunglasses (as a tradeoff) and gives an actual reason to wear a sechud during rev rounds

Given headrevs have flashes and wearing a sechud doesn’t give you flash immunity, I think this is honestly fine. Gives sechud more of a purpose over the sunglasses (as a tradeoff) and gives an actual reason to wear a sechud during rev rounds

#21218

icons should still be shaded like outlines are


Add mod unmark command

Student is unable to unmark a taken mod as not taken for a batchmate, so any accidental marking of mod previously could not be undone.
Adding a mod unmark function could allow student to change and update the status of mods of a batchmate more easily.
Let's

implement the mod unmark command
add tests for mod unmark command
update UG for mod mark and mod unmark to change some phrasing

Fixes #88 .

Codecov Report

Merging #96 (e89904f) into master (74d282c) will increase coverage by 0.45%.
The diff coverage is 84.09%.

@@             Coverage Diff              @@
##             master      #96      +/-   ##
============================================
+ Coverage     71.58%   72.04%   +0.45%
- Complexity      499      511      +12
============================================
  Files            83       84       +1
  Lines          1605     1649      +44
  Branches        194      202       +8
============================================
+ Hits           1149     1188      +39
  Misses          380      380
- Partials         76       81       +5

Impacted Files
Coverage Δ

...seedu/address/logic/commands/ModUnmarkCommand.java
73.91% <73.91%> (ø)

...c/main/java/seedu/address/model/person/Person.java
93.82% <88.88%> (-0.62%)
:arrow_down:

...a/seedu/address/logic/parser/ModCommandParser.java
94.73% <100.00%> (+0.98%)
:arrow_up:

src/main/java/seedu/address/model/person/Mod.java
81.48% <0.00%> (+7.40%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Add optional  and  elements

I'm thinking we can distribute optional elements/apollo-query.js and elements/apollo-mutation.js that would basically follow the pattern set out in the README
@LasaleFamine @FluorescentHallucinogen What do you think about that? I think we can start experimenting with the design for <apollo-mutation> set out in https://github.com/bennypowers/lit-apollo/issues/2#issuecomment-406976682 and see where it takes us.

Although I won't like to use custom elements for my queries I think this could be useful for some.
I'm not sure, but how will these elements share the apollo-client? I'll should pass it down?

Could take it as a property. Might mean needing to reevaluate the order of
operations when setting the client e.g. first setting query, afterwards
setting client.
On Tue, Jul 24, 2018, 4:55 PM Alessio Occhipinti<EMAIL_ADDRESS>wrote:

Although I won't like to use custom elements for my queries I think this
could be useful for some.
I'm not sure, but how will these elements share the apollo-client? I'll
should pass it down?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bennypowers/lit-apollo/issues/11#issuecomment-407415306,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABZgNIs3XJrkEBOTSKgnB3xet45cInH7ks5uJydfgaJpZM4VcaKA
.

--
Benny Powers
Licensed Israel Tour Guide
In Israel<PHONE_NUMBER>
In Canada<PHONE_NUMBER>
In the USA<PHONE_NUMBER>
bennypowers.com

I'm interested in <apollo-query>, <apollo-mutation> and <apollo-subscription> custom elements.
They will help users migrate from polymer-apollo-client.

Just found that on https://webcomponents.org/element/lit-apollo page ApolloMutation is listed in the "Elements" section, but ApolloQuery is not, ApolloQuery is listed in the "Classes" section, but ApolloMutation is not:

It looks strange, because I haven't found customElements.define('apollo-mutation', ApolloMutation); anywhere in the code. It was in the previous versions (https://github.com/bennypowers/lit-apollo/blob/v0.6.4/apollo-mutation.js#L186), but now it's removed.
ApolloQuery is actually a class (https://github.com/bennypowers/lit-apollo/blob/master/apollo-query.js#L64).
@bennypowers How is that even possible? :thinking:

Probably some mismatched JSDoc comments

Closed by d056b63
If problem persists after next release, feel free to reopen.


Added "Import" button

Issue Title: Adding "Import" button
resolve issue: #87
Type of change ☑
What sort of change have you made:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist: ☑

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly wherever it was hard to understand.
[ ] I have made corresponding changes to the documentation.
[ ] My changes generate no new warnings.
[x] I have added things that prove my fix is effective or that my feature works.
[ ] Any dependent changes have been merged and published in downstream modules.

How Has This Been Tested? ⚙
Sample -
https://github.com/singodiyashubham87/Draw-it-out/assets/134587780/92fc7298-b4dd-46b2-afcf-5ee729eb50a9
Please check it.

@StephenStolk The importing is not working as expected, it's disturbing the whole canvas & app layout too.

I will fix the ui.
Can you please tell the functionalities you are expecting ?
My fix can allow users-

to import image,
draw/color on the image

@StephenStolk We want here a feature which will help the user to import any file & put it in canvas along with functionalities like dragging that image across the canvas & resizing.

@StephenStolk Any updates?

I will pull up the request by this weekend. I am working on fixing the ui

@StephenStolk Is there any update?

Closing due to inactivity.


vis2.x: CSS Font & Text is not rendered in editor

Describe the bug
widget section CSS Font & Text is not rendered in editor. It is only applied, if runtime in a new window is opened.
To Reproduce
import widget
[
  {
    "tpl": "tplHtml",
    "data": {
      "refreshInterval": "0",
      "g_common": true,
      "html": "{0_userdata.0.test1}",
      "g_css_font_text": true
    },
    "style": {
      "left": 172,
      "top": 105,
      "color": "rgba(58,27,203,1)",
      "text-align": "right",
      "font-family": "Courier",
      "font-style": "italic",
      "font-weight": "bold",
      "font-size": "25px"
    },
    "_id": "i000001"
  }
]

Screenshots & Logfiles

Versions:

Adapter version: ioBroker.vis v2.0.6
JS-Controller version: v4.0.23
Node version: v16.18.1
Operating system: ioBroker.docker v7.2.1

@ice987987 I imported your widget and with vis 2.0.23 it is ok


does not start./wgd.sh start

I have:
Alpine Linux v3.16
Python 3.10.5
pip 22.2.2 from /usr/lib/python3.10/site-packages/pip (python 3.10)
following the instructions, an error occurs at the startup stage:

/wgdashboard/src# ./wgd.sh start
------------------------------------------------------------
| Starting WGDashboard with Gunicorn in the background.    |
Failed to read config file: /root/wgdashboard/src/gunicorn.conf.py
Traceback (most recent call last):
  File "/usr/lib/python3.10/site-packages/gunicorn/app/base.py", line 111, in get_config_from_filename
    spec.loader.exec_module(mod)
  File "<frozen importlib._bootstrap_external>", line 883, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/root/wgdashboard/src/gunicorn.conf.py", line 4, in <module>
    app_host, app_port = dashboard.get_host_bind()
  File "/root/wgdashboard/src/dashboard.py", line 1702, in get_host_bind
    init_dashboard()
  File "/root/wgdashboard/src/dashboard.py", line 1641, in init_dashboard
    config['Peers']['remote_endpoint'] = ifcfg.default_interface()['inet']
  File "/usr/lib/python3.10/site-packages/ifcfg/__init__.py", line 81, in default_interface
    return Parser(ifconfig=ifconfig)._default_interface(route_output=route_output)
  File "/usr/lib/python3.10/site-packages/ifcfg/parser.py", line 34, in __init__
    self.parse(self.ifconfig_data)
  File "/usr/lib/python3.10/site-packages/ifcfg/parser.py", line 91, in parse
    raise RuntimeError(
RuntimeError: Tried to add prefixlen=128 multiple times to eth0, it was already: 64
| Log files is under log/                                  |
------------------------------------------------------------

how to solve the problem?

i have same problem

@evgn88 @lokidv
https://github.com/donaldzou/WGDashboard/issues/191#issuecomment-1510433790

Closing due to inactivity :-) resubmit an issue if you still encounter it.


Min Width/Height not working and no examples

I would like to set min width on items but I have tried supplying minW (as searched and found in the js file) but cannot get it to work.
{"x":0,"y":0,"w":2,"h":2,"i":"0","minW":2} on an item seems to have no affect on resizing.

Nevermind, finally after some digging found that it has to be written as min-w in the javascript or html templates.

I am trying to use min-w and min-h but it doesnt work
{"x": 6.5, "y": 122, "w": 4, "h": 9, "min-w": 3.5, "min-h": 6.5},
what am I doing wrong?

Check #101 for an example


Thread Safety

Today's C# code is typically multi-threaded and async virtually everything. The Dispose pattern snippet that Visual Studio uses and you have copied for this blog post is not thread safe. Please update it with a version that is.

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

ID: 572ef6c6-5066-4a14-76a0-c3fb4b62e604
Version Independent ID: 9c754da1-c234-fb12-580b-0fcf9ffe2c20
Content: Implement a Dispose method
Content Source: docs/standard/garbage-collection/implementing-dispose.md
Product: dotnet-fundamentals
GitHub Login: @gewarren
Microsoft Alias: gewarren

Associated WorkItem - 44841

@IEvangelist do you want to take this one? Maybe you can make yourself the author of this article?

A piece of code is thread-safe if it functions correctly during simultaneous execution by multiple threads. The Dispose method should be idempotent, thus no matter what thread is calling it, or no matter how many times you execute them, you achieve the same result. This is a non-issue.


Set rating to empty

Great package! Idea:
Currently it is not possible to clear the number of stars (i.e. to 0). Perhaps it is an idea that if the current rating is 1 and 1 is chosen again, the rating will become 0 :)

Hi!
By default minRating is set to 0.5. Have you tried setting it to 0?
Now that I think about it this prop is probably not needed since it can be easily implemented in onChange if needed. I think it makes sense to remove it and not have a minRating by default.


Install with pip >=20 always triggers build

Downgrading to < 20 uses wheel.
$ pip --version
pip 20.0.1 from /home/juho/py3/lib/python3.5/site-packages/pip (python 3.5)
(py3) juho@ub:~$ pip install cmsis-pack-manager
Collecting cmsis-pack-manager
  Using cached cmsis-pack-manager-0.2.9.tar.gz (42 kB)
Collecting appdirs>=1.4
  Using cached appdirs-1.4.3-py2.py3-none-any.whl (12 kB)
Collecting milksnake>=0.1.2
  Using cached milksnake-0.1.5-py2.py3-none-any.whl (9.6 kB)
Processing ./.cache/pip/wheels/e4/76/4d/a95b8dd7b452b69e8ed4f68b69e1b55e12c9c9624dd962b191/PyYAML-5.3-cp35-cp35m-linux_x86_64.whl
Collecting cffi>=1.6.0
  Using cached cffi-1.13.2-cp35-cp35m-manylinux1_x86_64.whl (397 kB)
Processing ./.cache/pip/wheels/f2/9a/90/de94f8556265ddc9d9c8b271b0f63e57b26fb1d67a45564511/pycparser-2.19-py2.py3-none-any.whl
Building wheels for collected packages: cmsis-pack-manager
  Building wheel for cmsis-pack-manager (setup.py) ... error
  ERROR: Command errored out with exit status 1:
   command: /home/juho/py3/bin/python3 -u -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-aa67zdj0/cmsis-pack-manager/setup.py'"'"'; __file__='"'"'/tmp/pip-install-aa67zdj0/cmsis-pack-manager/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' bdist_wheel -d /tmp/pip-wheel-tar7vy6_
       cwd: /tmp/pip-install-aa67zdj0/cmsis-pack-manager/
  Complete output (9 lines):
  running bdist_wheel
  running build
  running build_py
  creating build/lib
  creating build/lib/cmsis_pack_manager
  copying cmsis_pack_manager/__init__.py -> build/lib/cmsis_pack_manager
  copying cmsis_pack_manager/_version.py -> build/lib/cmsis_pack_manager
  copying cmsis_pack_manager/pack_manager.py -> build/lib/cmsis_pack_manager
  error: [Errno 2] No such file or directory: 'cargo'
  ----------------------------------------
  ERROR: Failed building wheel for cmsis-pack-manager
  Running setup.py clean for cmsis-pack-manager
Failed to build cmsis-pack-manager
Installing collected packages: appdirs, pycparser, cffi, milksnake, pyyaml, cmsis-pack-manager
    Running setup.py install for cmsis-pack-manager ... error
    ERROR: Command errored out with exit status 1:
     command: /home/juho/py3/bin/python3 -u -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-aa67zdj0/cmsis-pack-manager/setup.py'"'"'; __file__='"'"'/tmp/pip-install-aa67zdj0/cmsis-pack-manager/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' install --record /tmp/pip-record-jt_ysbz4/install-record.txt --single-version-externally-managed --compile --install-headers /home/juho/py3/include/site/python3.5/cmsis-pack-manager
         cwd: /tmp/pip-install-aa67zdj0/cmsis-pack-manager/
    Complete output (9 lines):
    running install
    running build
    running build_py
    creating build/lib
    creating build/lib/cmsis_pack_manager
    copying cmsis_pack_manager/__init__.py -> build/lib/cmsis_pack_manager
    copying cmsis_pack_manager/_version.py -> build/lib/cmsis_pack_manager
    copying cmsis_pack_manager/pack_manager.py -> build/lib/cmsis_pack_manager
    error: [Errno 2] No such file or directory: 'cargo'
    ----------------------------------------
ERROR: Command errored out with exit status 1: /home/juho/py3/bin/python3 -u -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-aa67zdj0/cmsis-pack-manager/setup.py'"'"'; __file__='"'"'/tmp/pip-install-aa67zdj0/cmsis-pack-manager/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' install --record /tmp/pip-record-jt_ysbz4/install-record.txt --single-version-externally-managed --compile --install-headers /home/juho/py3/include/site/python3.5/cmsis-pack-manager Check the logs for full command output.
(py3) juho@ub:~$ pip install pip==19.3.1
Collecting pip==19.3.1
  Using cached pip-19.3.1-py2.py3-none-any.whl (1.4 MB)
Installing collected packages: pip
  Attempting uninstall: pip
    Found existing installation: pip 20.0.1
    Uninstalling pip-20.0.1:
      Successfully uninstalled pip-20.0.1
Successfully installed pip-19.3.1
(py3) juho@ub:~$ pip install cmsis-pack-manager
Collecting cmsis-pack-manager
  Using cached https://files.pythonhosted.org/packages/fd/6b/b1f8f4117b3a4efafbdb04eaa7e7685a395cda565b3b221860d04f2a827c/cmsis_pack_manager-0.2.9-py2.py3-none-manylinux1_x86_64.whl
Requirement already satisfied: pyyaml>=3.12 in ./py3/lib/python3.5/site-packages (from cmsis-pack-manager) (5.3)
Requirement already satisfied: milksnake>=0.1.2 in ./py3/lib/python3.5/site-packages (from cmsis-pack-manager) (0.1.5)
Requirement already satisfied: appdirs>=1.4 in ./py3/lib/python3.5/site-packages (from cmsis-pack-manager) (1.4.3)
Requirement already satisfied: cffi>=1.6.0 in ./py3/lib/python3.5/site-packages (from milksnake>=0.1.2->cmsis-pack-manager) (1.13.2)
Requirement already satisfied: pycparser in ./py3/lib/python3.5/site-packages (from cffi>=1.6.0->milksnake>=0.1.2->cmsis-pack-manager) (2.19)
Installing collected packages: cmsis-pack-manager
Successfully installed cmsis-pack-manager-0.2.9
WARNING: You are using pip version 19.3.1; however, version 20.0 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.

I still do experience an error with pip 20.3.4
Collecting cmsis-pack-manager<0.3.0,>=0.2.3
Using cached cmsis-pack-manager-0.2.10.tar.gz (45 kB)
ERROR: Command errored out with exit status 1:
command: /home/marcel/.pyenv/versions/2.7.18/bin/python2.7 -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-t6pg_T/cmsis-pack-manager/setup.py'"'"'; file='"'"'/tmp/pip-install-t6pg_T/cmsis-pack-manager/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(file);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, file, '"'"'exec'"'"'))' egg_info --egg-base /tmp/pip-pip-egg-info-y36jvN
cwd: /tmp/pip-install-t6pg_T/cmsis-pack-manager/
Complete output (60 lines):
/tmp/easy_install-TGE31H/setuptools_scm-6.0.1/src
<pkg_resources.WorkingSet object at 0xffffa6fc9250>
Traceback (most recent call last):
File "", line 1, in
File "/tmp/pip-install-t6pg_T/cmsis-pack-manager/setup.py", line 73, in
test_suite="tests"
File "/home/marcel/.pyenv/versions/2.7.18/lib/python2.7/site-packages/setuptools/init.py", line 144, in setup
_install_setup_requires(attrs)
File "/home/marcel/.pyenv/versions/2.7.18/lib/python2.7/site-packages/setuptools/init.py", line 139, in _install_setup_requires
dist.fetch_build_eggs(dist.setup_requires)
File "/home/marcel/.pyenv/versions/2.7.18/lib/python2.7/site-packages/setuptools/dist.py", line 719, in fetch_build_eggs
replace_conflicting=True,
File "/home/marcel/.pyenv/versions/2.7.18/lib/python2.7/site-packages/pkg_resources/init.py", line 782, in resolve
replace_conflicting=replace_conflicting
File "/home/marcel/.pyenv/versions/2.7.18/lib/python2.7/site-packages/pkg_resources/init.py", line 1065, in best_match
return self.obtain(req, installer)
File "/home/marcel/.pyenv/versions/2.7.18/lib/python2.7/site-packages/pkg_resources/init.py", line 1077, in obtain
return installer(requirement)
File "/home/marcel/.pyenv/versions/2.7.18/lib/python2.7/site-packages/setuptools/dist.py", line 786, in fetch_build_egg
return cmd.easy_install(req)
File "/home/marcel/.pyenv/versions/2.7.18/lib/python2.7/site-packages/setuptools/command/easy_install.py", line 679, in easy_install
return self.install_item(spec, dist.location, tmpdir, deps)
File "/home/marcel/.pyenv/versions/2.7.18/lib/python2.7/site-packages/setuptools/command/easy_install.py", line 705, in install_item
dists = self.install_eggs(spec, download, tmpdir)
File "/home/marcel/.pyenv/versions/2.7.18/lib/python2.7/site-packages/setuptools/command/easy_install.py", line 890, in install_eggs
return self.build_and_install(setup_script, setup_base)
File "/home/marcel/.pyenv/versions/2.7.18/lib/python2.7/site-packages/setuptools/command/easy_install.py", line 1158, in build_and_install
self.run_setup(setup_script, setup_base, args)
File "/home/marcel/.pyenv/versions/2.7.18/lib/python2.7/site-packages/setuptools/command/easy_install.py", line 1144, in run_setup
run_setup(setup_script, args)
File "/home/marcel/.pyenv/versions/2.7.18/lib/python2.7/site-packages/setuptools/sandbox.py", line 253, in run_setup
raise
File "/home/marcel/.pyenv/versions/2.7.18/lib/python2.7/contextlib.py", line 35, in exit
self.gen.throw(type, value, traceback)
File "/home/marcel/.pyenv/versions/2.7.18/lib/python2.7/site-packages/setuptools/sandbox.py", line 195, in setup_context
yield
File "/home/marcel/.pyenv/versions/2.7.18/lib/python2.7/contextlib.py", line 35, in exit
self.gen.throw(type, value, traceback)
File "/home/marcel/.pyenv/versions/2.7.18/lib/python2.7/site-packages/setuptools/sandbox.py", line 166, in save_modules
saved_exc.resume()
File "/home/marcel/.pyenv/versions/2.7.18/lib/python2.7/site-packages/setuptools/sandbox.py", line 141, in resume
six.reraise(type, exc, self._tb)
File "/home/marcel/.pyenv/versions/2.7.18/lib/python2.7/site-packages/setuptools/sandbox.py", line 154, in save_modules
yield saved
File "/home/marcel/.pyenv/versions/2.7.18/lib/python2.7/site-packages/setuptools/sandbox.py", line 195, in setup_context
yield
File "/home/marcel/.pyenv/versions/2.7.18/lib/python2.7/site-packages/setuptools/sandbox.py", line 250, in run_setup
_execfile(setup_script, ns)
File "/home/marcel/.pyenv/versions/2.7.18/lib/python2.7/site-packages/setuptools/sandbox.py", line 45, in _execfile
exec(code, globals, locals)
File "/tmp/easy_install-TGE31H/setuptools_scm-6.0.1/setup.py", line 52, in
with open("test_requirements.txt") as testreq:
File "/tmp/easy_install-TGE31H/setuptools_scm-6.0.1/setup.py", line 29, in scm_config
spec.add_cffi_module(
File "/tmp/easy_install-TGE31H/setuptools_scm-6.0.1/src/setuptools_scm/init.py", line 8, in
File "/tmp/easy_install-TGE31H/setuptools_scm-6.0.1/src/setuptools_scm/config.py", line 6, in
File "/tmp/easy_install-TGE31H/setuptools_scm-6.0.1/src/setuptools_scm/utils.py", line 41
print(*k)
^
SyntaxError: invalid syntax
----------------------------------------
ERROR: Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.

downgrading to pip 20.0.2 did not help unfortunately

@marcel-bluestone It looks like you are using Python 2, which is not supported by cmsis-pack-manager any more. (I'm surprised pip even tried to install that version of setuptools_scm, which also does not support Python 2.)

@flit Thanks - I am on Pip 21.3, but still wasn't working.
But installing Rust worked a treat.
Appreciate the quick response.  :)

Ok, thanks for the update! Not sure what the problem is then, we'll have to investigate. I'll create a new issue to track it, since it seems to be a different cause.

@flit Let me know if I can help.  I'm in the process of getting my Nucleo-H743ZI2 up and running with mbed, and so far it's been fairly easy - but pyocd doesn't work yet in Window or Linux.  I'm sure it's mostly pebcak as I'm very new to all this, but if I can help test or shoot logs over to you I'll be more than happy to do so.

Fyi, the issue with Python 3.10 is simple: we didn't release wheels for Python 3.10 since it wasn't final. Working on getting a wheel out. 😄


AnimationTree - Object was deleted while awaiting a callback

The error "Object was deleted while awaiting a callback" appears when I try to edit a blend tree graph node inside an AnimationTree:
I use Godot 3.2.2
I have an AnimationTree node in my scene. The root of the AnimationTree node is an AnimationNodeBlendTree. In the graph editor, there are OneShot nodes, Transition nodes, Blend2 and Blend3 nodes, a StateMachine, TimeScales - and two BlendTree nodes.
Each of those two BlendTree nodes inside the AnimationTree (which has an AnimationNodeBlendTree root) freezes the engine when I want to edit them. Only once in a while I am able to open those two BlendTree nodes when I press the edit button, but the rest of the time the engine hangs.

Duplicate of #35653 (the underlying issue is the same).


Master to Live - 4/25/2019

Docs Build status updates of commit d5deab5:
:warning: Validation status: warnings

File
Status
Preview URL
Details

mixed-reality-docs/openxr.md
:warning:Warning
View
Details

mixed-reality-docs/holoLens-demo-guide.md
:white_check_mark:Succeeded
View

mixed-reality-docs/images/models_icon_voicecommanding_100x100.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/models_icon_voicecommanding_200x200.png
:white_check_mark:Succeeded
View

mixed-reality-docs/mr-dev-days-sessions.md
:white_check_mark:Succeeded
View

mixed-reality-docs/openxr.md

[Warning] Metadata validation unexpected error: System.Exception: Unexpected response when checking ruleset: Forbidden - Method: GET, RequestUri: 'https://docs.microsoft.com/api/metadata/rules', Version: 1.1, Content: <null>, Headers:
{
X-Metadata-FileName: openxr.md
X-Metadata-RepositoryUrl: https://github.com/MicrosoftDocs/mixed-reality
X-Metadata-RepositoryBranch: live
} -
<TITLE>Access Denied</TITLE>

Access Denied

You don't have permission to access "http://docs.microsoft.com/api/metadata/rules" on this server.

Reference #18.daafedcc.1556223908.218fbe

at Apex.Validation.Logic.ValidationManager.<>c__DisplayClass9_0.<<ValidateMetadata>b__0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Apex.Validation.Logic.ValidationManager.<DoWithRetryAsync>d__11.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at Apex.Validation.Logic.ValidationManager.<DoWithRetryAsync>d__11.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Apex.Validation.Logic.ValidationManager.<ValidateMetadata>d__9.MoveNext()
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

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

File
Status
Preview URL
Details

mixed-reality-docs/holoLens-demo-guide.md
:white_check_mark:Succeeded
View

mixed-reality-docs/images/models_icon_voicecommanding_100x100.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/models_icon_voicecommanding_200x200.png
:white_check_mark:Succeeded
View

mixed-reality-docs/mr-dev-days-sessions.md
:white_check_mark:Succeeded
View

mixed-reality-docs/openxr.md
:white_check_mark:Succeeded
View

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


Setting up azure pipeline

1. Setup azure pipeline
2. Wrote a minimalistic azure-pipeline.yml by taking inspiration from pandas repo
3. used Makefile from #51 to add doc building pipeline

@sara-02 Thanks for the suggestions. I am kind of done with the first iteration. I am wondering why the build is not triggered in this PR because when I tried doing it for #106 it triggered for that branch alone.
The hard-coded values are from the pandas repo. I just replicated the method followed there

Also, one more thing, more of a question about how does pandas-dev perform adding commits, and what strategy we follow. Do we retain all the commit history or squash it into a single commit before merging?
@datapythonista @dujm

@sara-02 When I worked on this issue for pandas they didn't ask me to squash the commits. Not sure if that's the standard
https://github.com/pandas-dev/pandas/pull/26876

@sara-02 Thanks for the suggestions. I am kind of done with the first iteration. I am wondering why the build is not triggered in this PR because when I tried doing it for #106 it triggered for that branch alone.
The hard-coded values are from the pandas repo. I just replicated the method followed there

Hey not an expert here, but in my last org, unless you were a public member of the group, the CI would not trigger. If you want to give it a shot, maybe try making your membership in python-sprint public instead of private. Go to https://github.com/orgs/python-sprints/people and modifiy your status.

Great job, looks really good. Couple of comments:
The condition in the job is saying that this build will just run for commits to master, and not PRs. I don't think the CI is not running because of this, but the build wouldn't happen. The reason we have this in pandas is that PR builds are not able to access the secret variables for security reasons, so we can't push to the repo since we can't read the token.
Unrelated, my preference would be to set up the CI with a hello world build as you did, and then add the build to the docs. Gradually if possible. Obviously you learned a lot about how it works @bhavaniravi, but would be nice if the rest of the people is able to follow step by step. And all this in a single PR is too much I think.
About the squashing, GitHub has the option to do the squashing when merging a PR. For me there is no value in the local commits people do, so I've been squashing here too, so in the history of the repo we just have one commit per PR. Happy to have a discussion about this (this PR doesn't seem to be the best place, but an issue can be created).

Forgot about the emails, usernames... All the ones in the azure pipelines config file are mostly useless. When I set up that in pandas I used the project dev list, just to have something nice. but root@localhost is as good. For the username pandas-mentoring-ci could be descriptive, even if there is not much use for it.
My preference would be to have something hardcoded that is not the actual email of anyone, but it doesn't make a difference.


Rank Zero Property Mixin

🚀 Feature
This is an alternative to the change proposed in #1408 in case the global variable approach doesn't work.
I propose to add a mixin class for the rank property, e.g., named RankZeroPropertyMixin
Motivation
There are some parts of the code base that use a rank property in combination with the @rank_zero_only decorator. Refactoring this into a mixin would avoid code duplication and it would make it straightforward to add to a new callback for example.
PR #1408 already solves the problem of code duplication.
Pitch
class RankZeroPropertyMixin:
    def __init__(self):
        self._rank = 0

    @property
    def rank(self) -> int:
        return self._rank

    @rank.setter
    def rank(self, value: int) -> None:
        self._rank = value

In the Trainer init or distributed_parts, we will check each callback, logger for the rank property and set it to the appropriate value.
Then, when we add a new callback:
class NewFancyCallback(RankZeroPropertyMixin, Callback):
    def __init__(self):
        ...

    @rank_zero_only
    def on_train_start():
        print('only on rank 0')

Alternatives
Leave as is or go with #1408. In the future, Lightning will probably add more callbacks and features that are restricted to rank 0, so it would lead to code duplication.

I think that we stay with global var for rank, right? @williamFalcon


refactor(execution, native_blockifier): move BlockExecutor logic to b…

…lockifier

This change is

Codecov Report
Attention: 250 lines in your changes are missing coverage. Please review.

Comparison is base (5a49fb9) 68.79% compared to head (78ce517) 66.90%.

Files
Patch %
Lines

...s/blockifier/src/execution/transaction_executor.rs
15.55%
148 Missing and 4 partials :warning:

crates/blockifier/src/execution/block_executor.rs
33.69%
58 Missing and 3 partials :warning:

crates/native_blockifier/src/py_block_executor.rs
50.00%
28 Missing and 7 partials :warning:

crates/blockifier/src/execution/errors.rs
0.00%
1 Missing :warning:

crates/native_blockifier/src/errors.rs
0.00%
1 Missing :warning:

Additional details and impacted files
@@                         Coverage Diff                          @@
##           barak/move_bouncer_to_blockifier    #1464      +/-   ##
====================================================================
- Coverage                             68.79%   66.90%   -1.89%
====================================================================
  Files                                    59       61       +2
  Lines                                  7884     8135     +251
  Branches                               7884     8135     +251
====================================================================
+ Hits                                   5424     5443      +19
- Misses                                 2005     2236     +231
- Partials                                455      456       +1

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


Where is layer.css ?

jekyll server
ERROR `/assets/js/skin/layer.css' not found.

emm... This project has no layer.css file..  Can you please provide more details?


fix(rust): bofore sending the req to delete the TCP (outlet,inlet,listener) check that the target exsits

Before sending the req to delete the TCP (outlet, inlet, listener) check that the target exist
I'm still unable to figure out how to send a bodyless request to /node/tcp/listener/ to just check if there is a TCP listener with the target address or not.  I'll appreciate it if someone can help me with this.

Closes: #6347

Current behavior
When a user runs ockam tcp-outlet delete <ALIAS> or ockam tcp-inlet delete <ALIAS> or ockam tcp-listners delete <ALIAS>  and <ALIAS> does not exist, then they get a prompt, asking them to confirm whether this command should proceed or not. Then they are shown an error stating that 404 NotFound.
Proposed changes

When the user tries to delete one (instance) of mentioned (resources) then send an get request to checks if the target are exist in the first place
If not, then mention this in an error message and exit.
If the target was exist then process the delete operation as normal

Checks

[x] All commits in this Pull Request are signed and Verified by Github.
[x] All commits in this Pull Request follow the Ockam commit message convention.
[x] There are no Merge commits in this Pull Request. Ockam repo maintains a linear commit history. We merge Pull Requests by rebasing them onto the develop branch. Rebasing to the latest develop branch and force pushing to your Pull Request branch is okay.
[x] I have read and accept the Ockam Community Code of Conduct.
[x] I have read and accepted the Ockam Contributor License Agreement by adding my Git/Github details in a row at the end of the CONTRIBUTORS.csv file in a separate pull request to the build-trust/ockam repository. done via: build-trust/ockam-contributors/pull/266

Hi @0x61nas, I wanted to apply some format+renaming changes but I couldn't push to your branch. Could you allow maintainers to push on your branch? Here is a guide on how to do it: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/allowing-changes-to-a-pull-request-branch-created-from-a-fork

I'm having trouble finding the mint option, perhaps you meant an invitation to contribute to my repo.

Still nothing, could you try checking this post? https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/allowing-changes-to-a-pull-request-branch-created-from-a-fork

could you try checking this post? https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/allowing-changes-to-a-pull-request-branch-created-from-a-fork

I still can't figure how to do this, even after following the steps. the Allow edits from maintainers option in the fourth step doesn't exist

Still nothing,

could you check this link?
https://github.com/anas-contribs/ockam/invitations

I still can't figure how to do this, even after following the steps. the Allow edits from maintainers option in the fourth step doesn't exist

I don't know exactly how to fix it, but perhaps it's related to some configuration you have or how you created the fork in the first place.
 We can close this issue in favor of the other one (the result is the same).
Thanks again for your effor you put in this PR, will get merged soon 🚀


Micropython for this board

Hi there
I'm new at this, but I understand I need to flash micropython onto the board before I can write python code for it. Where can I find that image?
Thanks!

I was able to flash micropython to this board no problems using the one available here.
https://micropython.org/download/esp32c3/
Just make sure to change -z 0x1000 to -z 0x0
Micropython works great so far on this board!

Thanks! Worked perfectly! :)


chore(superMediaPlayer):  end buffering if highbuffer larger than max…

… buffer
Signed-off-by: lifujun<EMAIL_ADDRESS>

Codecov Report

Merging #606 (11ed256) into develop (34861e8) will decrease coverage by 4.23%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop     #606      +/-   ##
===========================================
- Coverage    66.91%   62.67%   -4.24%
===========================================
  Files          230      230
  Lines        12862    12864       +2
===========================================
- Hits          8606     8063     -543
- Misses        4256     4801     +545

Impacted Files
Coverage Δ

mediaPlayer/SuperMediaPlayer.cpp
72.77% <100.00%> (-0.39%)
:arrow_down:

framework/tests/dataSource/ipList.h
0.00% <0.00%> (-100.00%)
:arrow_down:

framework/utils/errors/framework_error.h
0.00% <0.00%> (-100.00%)
:arrow_down:

framework/tests/demuxer/demuxerUnitTest.cpp
0.00% <0.00%> (-100.00%)
:arrow_down:

framework/tests/dataSource/dataSourceUnitTest.cpp
0.00% <0.00%> (-93.85%)
:arrow_down:

framework/tests/decoder/decoderUnitTest.cpp
0.00% <0.00%> (-93.75%)
:arrow_down:

framework/tests/demuxer/demuxerUtils.cpp
0.00% <0.00%> (-80.60%)
:arrow_down:

framework/data_source/ffmpeg_data_source.cpp
6.38% <0.00%> (-51.07%)
:arrow_down:

framework/demuxer/AVBSF.cpp
25.80% <0.00%> (-43.02%)
:arrow_down:

framework/data_source/curl/CURLShareInstance.cpp
69.44% <0.00%> (-27.78%)
:arrow_down:

... and 16 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #606 (11ed256) into develop (34861e8) will decrease coverage by 4.23%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop     #606      +/-   ##
===========================================
- Coverage    66.91%   62.67%   -4.24%
===========================================
  Files          230      230
  Lines        12862    12864       +2
===========================================
- Hits          8606     8063     -543
- Misses        4256     4801     +545

Impacted Files
Coverage Δ

mediaPlayer/SuperMediaPlayer.cpp
72.77% <100.00%> (-0.39%)
:arrow_down:

framework/tests/dataSource/ipList.h
0.00% <0.00%> (-100.00%)
:arrow_down:

framework/utils/errors/framework_error.h
0.00% <0.00%> (-100.00%)
:arrow_down:

framework/tests/demuxer/demuxerUnitTest.cpp
0.00% <0.00%> (-100.00%)
:arrow_down:

framework/tests/dataSource/dataSourceUnitTest.cpp
0.00% <0.00%> (-93.85%)
:arrow_down:

framework/tests/decoder/decoderUnitTest.cpp
0.00% <0.00%> (-93.75%)
:arrow_down:

framework/tests/demuxer/demuxerUtils.cpp
0.00% <0.00%> (-80.60%)
:arrow_down:

framework/data_source/ffmpeg_data_source.cpp
6.38% <0.00%> (-51.07%)
:arrow_down:

framework/demuxer/AVBSF.cpp
25.80% <0.00%> (-43.02%)
:arrow_down:

framework/data_source/curl/CURLShareInstance.cpp
69.44% <0.00%> (-27.78%)
:arrow_down:

... and 16 more

Continue to review full report at Codecov.

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


JAVA-2264: HashedWheelTimer tick duration should be >= 10 ms on Windows

Not sure how to test this other than manually verifying CPU usage.

PR updated as discussed with the team.


issue detecting all sensors (after update / installation)

2022-11-19 13:13:26.066 WARNING (MainThread) [custom_components.luxtronik2] _load_lang_from_file - file not found /config/custom_components/luxtronik2/helpers/../translations/texts.None.json
2022-11-19 13:13:26.268 ERROR (MainThread) [homeassistant.setup] Unable to prepare setup for platform luxtronik2.binary_sensor: Platform not found (No module named 'custom_components.luxtronik').
2022-11-19 13:13:26.383 ERROR (MainThread) [homeassistant.setup] Unable to prepare setup for platform luxtronik2.sensor: Platform not found (No module named 'custom_components.luxtronik').
2022-11-19 13:13:26.386 ERROR (MainThread) [homeassistant.setup] Unable to prepare setup for platform luxtronik2.switch: Platform not found (No module named 'custom_components.luxtronik').

additional note: it worked until version "2022.08.01-a.6"

Having a similar problem, when updating to either 2022.11.18 or 2022.11.19. Last version working for me was https://github.com/BenPru/luxtronik/releases/tag/2022.11.17

Having a similar problem, when updating to either 2022.11.18 or 2022.11.19. Last version working for me was https://github.com/BenPru/luxtronik/releases/tag/2022.11.17

Just tested the same - I can upgrade to version "2022.11.17" - newer versions are broken for me.

Thanks for your ticket. I have fixed the imports in the current release. Please try this.

Unfortunately this didn't do it for me.
The following integrations and platforms could not be set up:
luxtronik2.sensor ([Show logs](http://homeassistant.local:8123/config/logs?filter=luxtronik2.sensor))
luxtronik2.binary_sensor ([Show logs](http://homeassistant.local:8123/config/logs?filter=luxtronik2.binary_sensor))
luxtronik2.switch ([Show logs](http://homeassistant.local:8123/config/logs?filter=luxtronik2.switch))

Please check your config and logs.
Dieser Fehler wurde von einer benutzerdefinierten Integration verursacht
Logger: custom_components.luxtronik2
Source: custom_components/luxtronik2/helpers/helper.py:32
Integration: Luxtronik
First occurred: 14:51:53 (1 occurrences)
Last logged: 14:51:53
_load_lang_from_file - file not found /config/custom_components/luxtronik2/helpers/../translations/texts.None.json
Logger: homeassistant.setup
Source: setup.py:320
First occurred: 14:51:52 (3 occurrences)
Last logged: 14:51:53
Unable to prepare setup for platform luxtronik2.sensor: Platform not found (cannot import name 'ATTR_STATUS_TEXT' from 'custom_components.luxtronik.const' (/config/custom_components/luxtronik/const.py)).
Unable to prepare setup for platform luxtronik2.binary_sensor: Platform not found (cannot import name 'CONF_LANGUAGE_SENSOR_NAMES' from 'custom_components.luxtronik.const' (/config/custom_components/luxtronik/const.py)).
Unable to prepare setup for platform luxtronik2.switch: Platform not found (cannot import name 'CONF_LANGUAGE_SENSOR_NAMES' from 'custom_components.luxtronik.const' (/config/custom_components/luxtronik/const.py)).

custom_components.luxtronik.const

This is no more in the current code. The update on your side was not successful.
Please reinstall it.

I've already reinstalled 2022.11.18 and 2022.11.19, but haven't tried 2022.11.19 for the latest 2022.11.19a. I will retry and report back.

I've after reinstall 2022.11.19 still this issue.
2022-11-19 22:00:04.451 ERROR (MainThread) [homeassistant.setup] Unable to prepare setup for platform luxtronik2.sensor: Platform not found (No module named 'custom_components.luxtronik').
2022-11-19 22:00:04.459 ERROR (MainThread) [homeassistant.setup] Unable to prepare setup for platform luxtronik2.binary_sensor: Platform not found (No module named 'custom_components.luxtronik').
2022-11-19 22:00:05.063 ERROR (MainThread) [homeassistant.setup] Unable to prepare setup for platform luxtronik2.switch: Platform not found (No module named 'custom_components.luxtronik').

Also didn’t work for 2022.11.19a. But with 2022.11.19b everything works again. Thanks for fixing this :-)

installed 2022.11.20 and it works - Thank you!!!


Consider supporting webpack.config.mjs alongside webpack.config.js

Do you want to request a feature or report a bug?
Feature.
What is the current behavior?
webpack.config.mjs cannot be used in place of webpack.config.js.
What is the expected behavior?
webpack.config.mjs could be used instead of webpack.config.js.
If this is a feature request, what is motivation or use case for changing the behavior?
Consistency.

It's a valid request, but not possible to do without changing the config loading to be asynchronous and using import() (only available as of node 9.5.0?)

That shouldn't be an issue. We already support exporting a promise from webpack.config.js.

@webpack-bot move to webpack/webpack-cli

Is this approved? Can we define webpack.config.mjs in next versions?

@damianobarbati See the moved issue webpack/webpack-cli#282. This will probably take a good, good bit of time to be implemented and released.


Spyder dark theme for windows?

I'm not sure if the issues are for asking questions, so sorry in advance if we're not supposed to ask questions here. I wonder if I can apply the dark theme to the whole Spyder IDE rather than just the editor.

Thanks for reporting. This is a duplicate of issue #2350.


add gimme-acc tool for SA HMAC testing

Thanks @tbpg.
A bit of context here - we needed a service account pool to test a new feature in GCS where users are able to create keys against service accounts. Because the limitations on the number of keys you can have on a service account, languages that run parallels tests in the CI would run into problems if they used the same service accounts.
I think you're right - this is essentially the same thing as gimmeproj, perhaps we should just close this and reuse the same binaries but plug in a different project to -project? @frankyn

+1, thanks @tbpg! I missed that support in this tool.


Support ; separated values for juicefs.users, juicefs.group

What would you like to be added:
juicefs.users, juicefs.group requires a file.
https://juicefs.com/docs/community/hadoop_java_sdk#other-configurations
I'd like to set it as ; separated values instead of a file.
Why is this needed:
I want to let users set their juicefs.users, juicefs.group in our multi-tenant Hadoop cluster.
setting ; separated values looks easier for users.

PR: https://github.com/juicedata/juicefs/pull/4724


Add group-id filter to GET /v2/transactions

Summary
This pull request adds the functionality to search by group ID to GET /v2/transactions.
Example:
cusl -Ss 'http://<IP_ADDRESS>:8980/v2/transactions?group-id=jtNGL2E2xWqEGkaaWDgxt5HxUcXewzwgpJTFrVHv7Zg='

Test Plan
Tested in a local environment, from a light indexer snapshot.

Nice!

I need to deploy light-indexer to FNet as main indexer API  so that everyone can test this functionality.

Just noticed that we follow indexer API convention to use Base64Std instead of Base64URL encoding for URL params.

/ = are not safe for URL.

Somehow it works for other endpoints - like box search.
We need a few test cases that include + / = in group_id Base64Std encoded string and see if this works (and why)


Change "selector type" of LeaveSimilarOutSelector

Status

[x] Ready to go

Developers certificate of origin

[x] I certify that this contribution is covered by the MIT license as defined in our LICENSE and adheres to the Developer Certificate of Origin.

@ijpulidos could you approve? just fixing the fact that the "selector type" was the same as the LeaveOneOutSelector

Codecov Report

Merging #696 (8cefe00) into main (fb6251a) will increase coverage by 28.71%.
The diff coverage is n/a.

Additional details and impacted files


docs: Narrowing.md

Add a note about TypeScript's differences in typing Boolean("hello") vs double negation !!"world".
The first is typed as boolean, while the latter is typed as true.

Yeah, I agree, good work 👍🏻

One could argue this is a TypeScript bug.
It should be just as easy to infer true from Boolean("hello").

I wonder if that's been explored already (it'd probably take a few overloads on the Boolean constructor to make it work) but the docs should be accurate to what TS does today (and I went back to 3.6 to test which has the same behavior)


Base Visualization options its empty?

Hello, I have just installed Qwik Trellis and I cannot change the Base Visualization option.
What may be causing this issue?
Thank you.

Hello - For this extension to work you first need to create your master visualization and then you will be able to select your master visualization from the drop down:  https://help.qlik.com/en-US/sense/November2017/Subsystems/Hub/Content/Visualizations/create-master-visualization.htm
Also a note since I see you are using Qlik Sense Desktop, at the moment there is a bug where the charts will not render when viewing the extension within the Desktop Client, it does however work in the browser so if you go to http://localhost:4848/hub/ and then open your app from there everything should work

Thank you! I got it to do what I wanted 🙂
A super Graphic divided in several graphics to show 3 dimensions + 1 measure.
[cid:b397d15a-abdc-4b53-9afc-d6bc034c49ff]

De: rileymd88<EMAIL_ADDRESS>Enviado: 21 de fevereiro de 2019 11:31
Para: rileymd88/qwik-trellis
Cc: RitaAguiar; Author
Assunto: Re: [rileymd88/qwik-trellis] Base Visualization options its empty? (#1)
Hello - For this extension to work you first need to create your master visualization and then you will be able to select your master visualization from the drop down: https://help.qlik.com/en-US/sense/November2017/Subsystems/Hub/Content/Visualizations/create-master-visualization.htm
Also a note since I see you are using Qlik Sense Desktop, at the moment there is a bug where the charts will not render when viewing the extension within the Desktop Client, it does however work in the browser so if you go to http://localhost:4848/hub/ and then open your app from there everything should work
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/rileymd88/qwik-trellis/issues/1#issuecomment-465966074, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AqLfR6-2-eVV1SzquAJMAeaTG486GOvoks5vPoOBgaJpZM4bHNRQ.

Following the previous message. I have a suggestion: I think it would be greatly advantageous if your Trellis Chart could be a single chart, where you have 2 dimension on the x axis, a 3rd dimension stacked and 1 measure in the height (instead of splitting the 3rd dimension in several graphics). Hopping you may consider this. Thank you, your extension is very helpful!

De: rileymd88<EMAIL_ADDRESS>Enviado: 21 de fevereiro de 2019 11:31
Para: rileymd88/qwik-trellis
Cc: RitaAguiar; Author
Assunto: Re: [rileymd88/qwik-trellis] Base Visualization options its empty? (#1)
Hello - For this extension to work you first need to create your master visualization and then you will be able to select your master visualization from the drop down: https://help.qlik.com/en-US/sense/November2017/Subsystems/Hub/Content/Visualizations/create-master-visualization.htm
Also a note since I see you are using Qlik Sense Desktop, at the moment there is a bug where the charts will not render when viewing the extension within the Desktop Client, it does however work in the browser so if you go to http://localhost:4848/hub/ and then open your app from there everything should work
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/rileymd88/qwik-trellis/issues/1#issuecomment-465966074, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AqLfR6-2-eVV1SzquAJMAeaTG486GOvoks5vPoOBgaJpZM4bHNRQ.


Add autocomplete support for subcommands arguments

Tested and works for me.

why is the submodule changed for doxygen-awesome-css?

your branch is 110 commits behind, please update it, that's ancient!

Oh actually I'm not sure, thought it was an update from the submodule,
guess I'm gonna revert that
On Fri, Aug 12, 2022, 1:45 PM Craig Edwards @.***>
wrote:

why is the submodule changed for doxygen-awesome-css?
—
Reply to this email directly, view it on GitHub
https://github.com/brainboxdotcc/DPP/pull/463#issuecomment-1212783443,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATEZRM4LF4EYB4RJ2BAWIB3VYXXH7ANCNFSM56KHSE4Q
.
You are receiving this because you authored the thread.Message ID:
@.***>

Actually that's only the master branch, I'm making PR from the dev branch
On Fri, Aug 12, 2022, 1:48 PM Craig Edwards @.***>
wrote:

your branch is 110 commits behind, please update it, that's ancient!
—
Reply to this email directly, view it on GitHub
https://github.com/brainboxdotcc/DPP/pull/463#issuecomment-1212785469,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATEZRM5V4PJ3OXAXLPAX4OTVYXXVJANCNFSM56KHSE4Q
.
You are receiving this because you authored the thread.Message ID:
@.***>


Processing unwanted node_modules

esbuild-plugin-esling makes eslint process node_modules, which can get annoying if some modules depends on internal eslint linter plugins (like react-dom) or doesn't follow defined codestyle.
ignoring files with eslint makes ignoring messages which are annoyingly hard to remove.
Pull request fixes #1 by adding a glob patterns config to filter files to process before eslint call for each file.

@robinloeffel This plugin is tooo naive and I don't think it yet ready to be on the production. It seems it slows down esbuild output. Also paths are not configurable (what if I want to lint .vue or css?). I wonder why didn't you use .onEnd callback with paths configurable (or it can track paths even automatically). Anyway, I highly appreciate any thoughts..

In https://github.com/robinloeffel/esbuild-plugin-eslint/commit/e05473583ab02dba37d622ad393231f70b500cdf I've fixed a handful of issues, including the one you both are addressing. It now ignores node_modules completely, you may define your own filter for esbuild, which allows you to lint .vue files or whatever else you please, and I've moved the work that ESLint does out of the onLoad and into the onEnd hook. Thanks for you guys' feedback!


Create Expensify-Chat-Playbook-for-Conferences

Details

Fixed Issues

$
PROPOSAL:
Tests

[ ] Verify that no errors appear in the JS console

Offline tests

QA Steps

[ ] Verify that no errors appear in the JS console

PR Author Checklist

[ ] I linked the correct issue in the ### Fixed Issues section above
[ ] I wrote clear testing steps that cover the changes made in this PR

[ ] I added steps for local testing in the Tests section
[ ] I added steps for the expected offline behavior in the Offline steps section
[ ] I added steps for Staging and/or Production testing in the QA steps section
[ ] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[ ] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[ ] I included screenshots or videos for tests on all platforms
[ ] I ran the tests on all platforms & verified they passed on:

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

[ ] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[ ] I followed proper code patterns (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[ ] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[ ] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[ ] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[ ] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I followed the guidelines as stated in the Review Guidelines
[ ] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] I verified that if a function's arguments changed that all usages have also been updated correctly
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[ ] If any new file was added I verified that:

[ ] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn't already exist
[ ] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[ ] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[ ] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[ ] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

Mobile Web - Chrome

Mobile Web - Safari

Desktop

iOS

Android

Thanks @Santhosh-Sellavel ! The preview function wasn't helping me here so I'm curious if these changes look aligned with other playbooks e.g. here https://help.expensify.com/articles/playbooks/Expensify-Playbook-for-Small-to-Medium-Sized-Businesses

Let me check

Thanks @Santhosh-Sellavel ! Thats weird that most of the content seems to be missing

Do we need to remove "" from each section so the content shows?

Basically there is copy under each header that doesn't appear to be showing. Here's some of the draft for reference

Do we need to remove "< !--" and "- ->" from each section so the content shows?

That's the syntax for commenting in markdown, those comments are not visible in the preview!
Here is this is how it looks now,
https://github.com/Expensify/App/assets/85645967/694ecc5f-1f17-414d-b2fe-3e8d06ebb0cc

Nice! Ok that looks great

Here is the file content.md

Looks good to me @Santhosh-Sellavel !

Please make the changes, seems it's not done yet!

I think I made the necessary changes @Santhosh-Sellavel

What are the next steps here @Santhosh-Sellavel ?

@MitchExpensify
The next step update the PR description i.e. complete the checklist and add test steps. Here is an example PR

Or

CLA Assistant Lite bot: Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.
I have read the CLA Document and I hereby sign the CLA
You can retrigger this bot by commenting recheck in this Pull Request

@MitchExpensify You didn't sign this please do that!

I have read the CLA Document and I hereby sign the CLA

The next step update the PR description i.e. complete the checklist and add test steps

Done! Thanks

@MitchExpensify You can delete this unnecessary text

Reviewer Checklist

[x] I have verified the author checklist is complete (all boxes are checked off).
[x] I verified the correct issue is linked in the ### Fixed Issues section above
[x] I verified testing steps are clear and they cover the changes made in this PR

[x] I verified the steps for local testing are in the Tests section
[x] I verified the steps for Staging and/or Production testing are in the QA steps section
[x] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[x] I checked that screenshots or videos are included for tests on all platforms
[x] I included screenshots or videos for tests on all platforms
[x] I verified tests pass on all platforms & I tested again on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[x] I verified proper code patterns were followed (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I verified that this PR follows the guidelines as stated in the Review Guidelines
[x] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos
https://github.com/Expensify/App/assets/85645967/694ecc5f-1f17-414d-b2fe-3e8d06ebb0cc

@MitchExpensify
Need to update the routes.yml as the PR does. Then we should be good to go!

Seems like we may not need to update, as I don't see any existing article references also. So let's get an engineer to move this forward

https://help.expensify.com/hubs/playbooks
Do we need to do anything additional to get this playbook to appear just like the others in terms of being listed here and here?

No, we don't need to do anything

@MitchExpensify You have to save it with externsion .md
Expensify-Chat-Playbook-for-Conferences.md

That is the problem why your preview was not working on git

Here is the recording @MitchExpensify, that confirms this works
https://github.com/Expensify/App/assets/85645967/5e572bd9-e442-4212-b44c-8e0067a2e79f

yup what santhosh said. Also looks like you've got travis errors to address. Let me know if you need any help with it.

Ok that looks great!
@chiragsalian yes if you could help me address the Travis errors that'd be great! Thank you

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/MitchExpensify in version: 1.3.39-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/francoisl in version: 1.3.39-11 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

This hit pod but I don't see it on https://help.expensify.com/hubs/playbooks

@francoisl @Santhosh-Sellavel Any idea why this isn't showing on production?

Will issue payment to @Santhosh-Sellavel via NewDot - $500 seems fair to us both 👍

Seems Deployment failed for help docs fhttps://github.com/Expensify/App/actions/workflows/deployExpensifyHelp.yml

Cool, what do we need to do to get this live?

I haven't had time to look into this today yet, I set myself a reminder to check after the next deploy.

Thank you! We are keen to get this over to CallStack asap because its part of the plan to have them use Expensify Chat for React Europe 🙇

Done - see https://help.expensify.com/articles/playbooks/Expensify-Chat-Playbook-for-Conferences
I had to cancel this old workflow that was stuck running for 15 days, and then I manually restarted the deployExpensifyHelp.yml workflow.

Awesome!, thanks @francoisl

Approved $500 to Santhosh on ND


Instead of copying the entire installer, download drivers on demand

The current hack of copying the entire installer is fragile and skirts some normal practices, and is bound to cause some problems sooner or later.
I also figured out that the regular installer cache can be made to contain these components after all, but we can't use that technique because per-machine components require UAC to even be placed in the cache, breaking silent upgrade for people who haven't installed drivers.
@keybase/react-hackers @maxtaco

It took a long time to figure out, but at build time the local file pointed to by SourceFile has its hash baked in to the installer, and if at install time the downloaded one doesn't match, it errors out. You may notice that the build process downloads these from s3 if they aren't available locally already, but we could change that part.
Again, this is just a suggestion, we don't have to do this, but I thought it important enough to spend my 2-pointer on.

Ok, that's a good plan. But you should still download over HTTPS. And the dokanclean process doesn't seem to have this security mechanism built it. It seems to just exec whatever it downloads from S3 (over HTTP).

Dokanclean gets downloaded if not already locally copied, just as a convenience to avoid another build machine setup step. I added a hash check here but we could just skip that part of the build and do it manually if you prefer. The installer will still enforce at runtime that the one it downloads matches the one present when the installer was built.

Got it, can you point me to the place in the code where the hashes are checked on the other side?

Install time verification of hashes is in the Wix Burn source code:
https://github.com/wixtoolset/wix3/blob/c2b0b3ff94acda436ea05dce8ebc7e659cf1fc24/src/burn/engine/cache.cpp#L1825

I guess that means we can't use sha2!

Where do you specify the hashes for the various downloaded packages?

You don't specify the hash, you specify a local copy, kind of a reference copy, which it hashes itself at build time, then bakes the hashes into the installer. Note I just removed the downloading of dokanclean.exe as a build step; we'll set that up manually on each build machine.
It is true that we have little control over this in the current toolkit, but I guess we could add a different hashing mechanism and build our own WIX and/or submit a PR. Also note, our installer has been downloading .NET4 this way the whole time.

Understood, thank you for the explanation.  Can you test this by changing the image on the server after it's built and making sure the hash check fails?

yep, already checked that, but can do again

ok, thanks!

Here you go (changed the s3 file after building):

the log says which file, and what the expected + found hashes were

cool. Gotta love those windows error codes! 0x80091007 !


Use neededMakefileDependencies in the manual

Fixes #696

Thanks for the PR - looks good. Unfortunately it conflicts with the renaming to Shakefile.hs - any chance of a revised PR?

Thanks! I'll update the website sometime in the next few days.