edt - error

Hi @uschmidt83 , Hi @maweigert ,
I just discovered the new optional tool "You might experience improved performance during training if you additionally install the Multi-Label Anisotropic 3D Euclidean Distance Transform (MLAEDT-3D)" .
The installation with pip went smoothly :
Installing collected packages: edt
Successfully installed edt-2.1.0

But I got the error below when starting the training:
Using Augmentation
WARNING:tensorflow:From /opt/conda/lib/python3.7/site-packages/tensorflow_core/python/ops/nn_impl.py:183: where (from tensorflow.python.ops.array_ops) is deprecated and will be removed in a future version.
Instructions for updating:
Use tf.where in 2.0, which has the same broadcast rule as np.where
WARNING:tensorflow | From /opt/conda/lib/python3.7/site-packages/tensorflow_core/python/ops/nn_impl.py:183: where (from tensorflow.python.ops.array_ops) is deprecated and will be removed in a future version.
Instructions for updating:
Use tf.where in 2.0, which has the same broadcast rule as np.where
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-8-572a4cf9baea> in <module>
      2 if (is_augment) :
      3     print( "Using Augmentation" )
----> 4     model.train( X_trn, Y_trn, validation_data=(X_val,Y_val), augmenter=augmenter )
      5 else:
      6     print( "No Augmentation" )

/opt/conda/lib/python3.7/site-packages/stardist/models/model2d.py in train(self, X, Y, validation_data, augmenter, seed, epochs, steps_per_epoch)
    354         n_take = self.config.train_n_val_patches if self.config.train_n_val_patches is not None else n_data_val
    355         _data_val = StarDistData2D(*validation_data, batch_size=n_take, length=1, **data_kwargs)
--> 356         data_val = _data_val[0]
    357
    358         data_train = StarDistData2D(X, Y, batch_size=self.config.train_batch_size, augmenter=augmenter, length=epochs*steps_per_epoch, **data_kwargs)

/opt/conda/lib/python3.7/site-packages/stardist/models/model2d.py in __getitem__(self, i)
     56         X, Y = tuple(zip(*tuple(self.augmenter(_x, _y) for _x, _y in zip(X,Y))))
     57
---> 58         prob = np.stack([edt_prob(lbl[self.b]) for lbl in Y])
     59
     60         if self.shape_completion:

/opt/conda/lib/python3.7/site-packages/stardist/models/model2d.py in <listcomp>(.0)
     56         X, Y = tuple(zip(*tuple(self.augmenter(_x, _y) for _x, _y in zip(X,Y))))
     57
---> 58         prob = np.stack([edt_prob(lbl[self.b]) for lbl in Y])
     59
     60         if self.shape_completion:

/opt/conda/lib/python3.7/site-packages/stardist/utils.py in edt_prob(lbl_img, anisotropy)
     84         lbl_img = np.pad(lbl_img, ((1,1),)*lbl_img.ndim, mode='constant')
     85         warnings.warn("EDT of constant label image is ill-defined. (Assuming background around it.)")
---> 86     dist_func = _edt_dist_func(anisotropy)
     87     objects = find_objects(lbl_img)
     88     prob = np.zeros(lbl_img.shape,np.float32)

/opt/conda/lib/python3.7/site-packages/stardist/utils.py in _edt_dist_func(anisotropy)
     50 def _edt_dist_func(anisotropy):
     51     try:
---> 52         from edt import edt as edt_func
     53         # raise ImportError()
     54         dist_func = lambda img: edt_func(np.ascontiguousarray(img>0), anisotropy=anisotropy)

edt.pyx in init edt()

ValueError: numpy.ndarray size changed, may indicate binary incompatibility. Expected 88 from C header, got 80 from PyObject

First, I though it was due to the augmenter, which is defined as below :
from albumentations import (
                            Compose, RandomBrightness, RandomContrast, HorizontalFlip, VerticalFlip, RandomRotate90, RandomScale, ElasticTransform, GaussianBlur, RandomScale
                            )
is_augment = True
# Define the dafult transformations we are allowed to use.
transforms = Compose([
                RandomBrightness(limit=0.1),
                RandomContrast(limit=0.2, p=0.5),
                HorizontalFlip(),
                VerticalFlip(),
                RandomRotate90(),
                #GaussianBlur(blur_limit=[3,7]),
                #ElasticTransform(alpha=0.5, sigma=50, alpha_affine=50),
                #RandomScale(scale_limit=[0.5,2.0], p=1)
    ])

def augmenter(x,y):
    t = transforms(image=x, mask=y)
    x = t['image']
    y = t['mask']
    return x, y

but without augmenter I also got :
No Augmentation
WARNING:tensorflow:From /opt/conda/lib/python3.7/site-packages/tensorflow_core/python/ops/nn_impl.py:183: where (from tensorflow.python.ops.array_ops) is deprecated and will be removed in a future version.
Instructions for updating:
Use tf.where in 2.0, which has the same broadcast rule as np.where
WARNING:tensorflow | From /opt/conda/lib/python3.7/site-packages/tensorflow_core/python/ops/nn_impl.py:183: where (from tensorflow.python.ops.array_ops) is deprecated and will be removed in a future version.
Instructions for updating:
Use tf.where in 2.0, which has the same broadcast rule as np.where
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-6-572a4cf9baea> in <module>
      5 else:
      6     print( "No Augmentation" )
----> 7     model.train(X_trn, Y_trn, validation_data=(X_val,Y_val), augmenter=None)
      8
      9 trainingNiceEnd = True

/opt/conda/lib/python3.7/site-packages/stardist/models/model2d.py in train(self, X, Y, validation_data, augmenter, seed, epochs, steps_per_epoch)
    354         n_take = self.config.train_n_val_patches if self.config.train_n_val_patches is not None else n_data_val
    355         _data_val = StarDistData2D(*validation_data, batch_size=n_take, length=1, **data_kwargs)
--> 356         data_val = _data_val[0]
    357
    358         data_train = StarDistData2D(X, Y, batch_size=self.config.train_batch_size, augmenter=augmenter, length=epochs*steps_per_epoch, **data_kwargs)

/opt/conda/lib/python3.7/site-packages/stardist/models/model2d.py in __getitem__(self, i)
     56         X, Y = tuple(zip(*tuple(self.augmenter(_x, _y) for _x, _y in zip(X,Y))))
     57
---> 58         prob = np.stack([edt_prob(lbl[self.b]) for lbl in Y])
     59
     60         if self.shape_completion:

/opt/conda/lib/python3.7/site-packages/stardist/models/model2d.py in <listcomp>(.0)
     56         X, Y = tuple(zip(*tuple(self.augmenter(_x, _y) for _x, _y in zip(X,Y))))
     57
---> 58         prob = np.stack([edt_prob(lbl[self.b]) for lbl in Y])
     59
     60         if self.shape_completion:

/opt/conda/lib/python3.7/site-packages/stardist/utils.py in edt_prob(lbl_img, anisotropy)
     84         lbl_img = np.pad(lbl_img, ((1,1),)*lbl_img.ndim, mode='constant')
     85         warnings.warn("EDT of constant label image is ill-defined. (Assuming background around it.)")
---> 86     dist_func = _edt_dist_func(anisotropy)
     87     objects = find_objects(lbl_img)
     88     prob = np.zeros(lbl_img.shape,np.float32)

/opt/conda/lib/python3.7/site-packages/stardist/utils.py in _edt_dist_func(anisotropy)
     50 def _edt_dist_func(anisotropy):
     51     try:
---> 52         from edt import edt as edt_func
     53         # raise ImportError()
     54         dist_func = lambda img: edt_func(np.ascontiguousarray(img>0), anisotropy=anisotropy)

edt.pyx in init edt()

ValueError: numpy.ndarray size changed, may indicate binary incompatibility. Expected 88 from C header, got 80 from PyObject

Do you have any hint?
Thank you again for the great tool!
Cheers,
Romain

Hi @romainGuiet ,
this seems to be a known issue  when using edt with numpy versions older than 1.20. On my machine, updating numpy to 1.20 does resolve the issue.
Hope that helps!
M

HI @maweigert ,
Thank you for the hint !
$ pip install numpy --upgrade
Collecting numpy
  Downloading numpy-1.20.2-cp37-cp37m-manylinux2010_x86_64.whl (15.3 MB)
     |████████████████████████████████| 15.3 MB 5.5 MB/s
Installing collected packages: numpy
  Attempting uninstall: numpy
    Found existing installation: numpy 1.19.2
    Uninstalling numpy-1.19.2:
      Successfully uninstalled numpy-1.19.2
ERROR: After October 2020 you may experience errors when installing or updating packages. This is because pip will change the way that it resolves dependency conflicts.

We recommend you use --use-feature=2020-resolver to test your packages with the new resolver before it becomes the default.

scikit-tensor-py3 0.4.1 requires numpy==1.16.*, but you'll have numpy 1.20.2 which is incompatible.
scikit-tensor-py3 0.4.1 requires scipy==1.3.*, but you'll have scipy 1.6.2 which is incompatible.
Successfully installed numpy-1.20.2

$ pip install edt
Collecting edt
  Using cached edt-2.1.0-cp37-cp37m-manylinux2014_x86_64.whl (1.9 MB)
Requirement already satisfied: numpy in /opt/conda/lib/python3.7/site-packages (from edt) (1.20.2)
Installing collected packages: edt
Successfully installed edt-2.1.0

I tried and it works (even if scikit-tensor-py3 0.4.1 claims incompatibility)!
Cheers,
Romain


mention that it is wise to run a sync when writing any files to /data

@shaunmulligan is this still necessary with named volumes?


Downloading a deck returns a "500 Internal server error"

Oops! An Error Occurred
The server returned a "500 Internal Server Error".
Something is broken. Please let us know what you were doing when this error occurred. We will fix it as soon as possible. Sorry for any inconvenience caused.

Which deck are you trying to download, and in which format (text, octgn)?


Support for faces via shared links

This may end up being negated by #188, but currently guests using a shared link to access the library cannot use the Faces tab.
Is it possible to enable this? Or is this a limitation of how shared folders are implemented?
I imagine we are bound by the same permissions of #188.
Ultimately I'm looking for a way to allow users to browse a subset of the library and also use the faces functionality. If user permissions is the best way to achieve that I can close this issue.

Hi,
This feature is very far from what the gallery is capable of today.
There are these tables (plus some more):

faces (containing all names) <-- this is shown on the faces tab
photos (listing all photos, with a foreign key to a directory)
faces <-> photos switching table
directories <-- this is what you can share

Not sure what would happen if the faces tab would be rendered based on the directory that one can look at. I assume a really O(minutes) loading time.
Sorry I only have limited time for playing our with this app, and this feature does not align with my plans. I can give some pointers if you would like to give it a try.

Totally reasonable. A lot of complexity for this feature definitely. Just wanted to get some idea of scope.
I may take a look at it as I familiarize myself a bit more with the app.
Not sure if this is a possible route but could you generate a custom {share}faces table at link creation?

I would rather try and check how slow would it be to list the faces on the fly from the available photos in: https://github.com/bpatrik/pigallery2/blob/90a6c4b6b1d2a2e22411288ad5e2613fc62de38a/src/backend/model/database/sql/PersonManager.ts#L34-L49
It is not possible to enable faces tab for shared link (limited guest), but not for guest (practically a non existing role), because that would mess up the user rights: https://github.com/bpatrik/pigallery2/blob/master/USERRIGHTS.md
A user should have all the rights that the other with lower Id have.

Thanks for your insight, I'll check that out!


Expose devtools plugin permissions through the metadata endpoint

Hey, I just made a Pull Request!
Add missing permissions to be exposed in metadata endpoint for devtools plugin

:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Feel like I totally missed that was needed now

@awanlin - The only place I found it referenced was in the "diff" of code changes required when adding permissions to a plugin.


feat: add  kubectl-plugin for risingwave-operator

We can add a kubectl-plugin for risingwave-operator just like kubectl-rw. Users can use it by kubectl rw init to deploy the risingwave-operator && use kubectl rw create instance --params.. to create a risingwave db.
Ref: https://kubernetes.io/docs/tasks/extend-kubectl/kubectl-plugins/

Good idea! We can even integrate it into the risedev tool for fast provisioning.

I think there's a tracking issue #107 so I'm closing this.


Spare Linear Matrix Solver

Sorry if I'm overlooking something obvious here, but does this library include a way to solve sparse matrix equations? E.g. I have a matrix A and a vector v and I want to find the vector x`` that satisfies the equation Ax=v. I looked in the documentation and could not find any functions under "inverse" or "solve" other than the triangular solvers, which seem like they are meant only for a specialized case. Is there, or is there planned to be, a method of solving such equations using this library? Thank you for any help or guidance.

There is the sprs_ldl package for solving a hermitian positive definite matrix. Have a look at suitesparse and we could see if a wrapper for some function might fit your usecase

I have a pr out (https://github.com/sparsemat/sprs/pull/327) to add bindings to umfpack, which handles more general square systems - hopefully that helps

@kesleta ,  #327 is merged, so you should be able to use the methods in sprs_suitesparse_umfpack to do arbitrary square Ax=b solves with a dense right-hand-side.


Issue #2269: self detection.

Please have a look at this PR.  I chose to remove any references to self in the parent list when creating the parent_record at the time parent.config is read.  Uses information known to traffic server at startup, Machine.hostname and Machine.ip_string.

[approve ci autest]

@zwoop - no it doesn't look like it will work on multi-homed hosts.  Do you use multi-homed hosts?  All ours are bonded so we just have the bonded interface.  With that said, there may be others that have multi-homed caches and I can certainly look to add support for that.

Yeah, we have multi-homed boxes, and it's unclear which of those interfaces becomes the Machine interface. In our case, we have one interface that is the bonded NIC, but we have backplane and management interfaces that are internal only.
I can also totally see a common use case to be that you have one interface (be it bonded or not) for the front-side (users), and a second interface (possibly bonded) for the back-side for the proxying. In this scenario, you would use this back-side interface for the "cluster" (cache peer communication), so that's the one you have to identify as self.
I hope this makes sense? I don't think relying on the current Machine object is sufficient for almost any use case, and I suspect the Machine object isn't reliably using the "correct" interface.

Ok, got it.  I’ll work on updating the Machine class to identify all up interfaces to exclude their hostnames and ip’s.
From: Leif Hedstrom<EMAIL_ADDRESS>Reply-To: apache/trafficserver<EMAIL_ADDRESS>Date: Wednesday, July 19, 2017 at 7:02 AM
To: apache/trafficserver<EMAIL_ADDRESS>Cc: John Rushford<EMAIL_ADDRESS>Assign<EMAIL_ADDRESS>Subject: Re: [apache/trafficserver] Issue #2269: self detection. (#2277)
Yeah, we have multi-homed boxes, and it's unclear which of those interfaces becomes the Machine interface. In our case, we have one interface that is the bonded NIC, but we have backplane and management interfaces that are internal only.
I can also totally see a common use case to be that you have one interface (be it bonded or not) for the front-side (users), and a second interface (possibly bonded) for the back-side for the proxying. In this scenario, you would use this back-side interface for the "cluster" (cache peer communication), so that's the one you have to identify as self.
I hope this makes sense? I don't think relying on the current Machine object is sufficient for almost any use case, and I suspect the Machine object isn't reliably using the "correct" interface.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHubhttps://github.com/apache/trafficserver/pull/2277#issuecomment-316378687, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AEVyboIagl7LmJWtJRab7lr6_4M8Favyks5sPf5BgaJpZM4Oao9a.

@jrushford Cool! One thought / idea: Would it make sense to add an API to the machine object for "self" identification? Such that it loops over all interfaces, rather than exposing the array of strings? So, something like
machine->is_self(ip);

and
machine->is_self(hostname);

Or some such.

@zwoop that’s an excellent idea!  I’ll work on that.
From: Leif Hedstrom<EMAIL_ADDRESS>Reply-To: apache/trafficserver<EMAIL_ADDRESS>Date: Wednesday, July 19, 2017 at 7:37 AM
To: apache/trafficserver<EMAIL_ADDRESS>Cc: John Rushford<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [apache/trafficserver] Issue #2269: self detection. (#2277)
@jrushfordhttps://github.com/jrushford Cool! One thought / idea: Would it make sense to add an API to the machine object for "self" identification? Such that it loops over all interfaces, rather than exposing the array of strings? So, something like
machine->is_self(ip);
and
machine->is_self(hostname);
Or some such.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/apache/trafficserver/pull/2277#issuecomment-316389796, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AEVybk_gWkMYVEGLd1uyLcAhlL3_u21_ks5sPgaGgaJpZM4Oao9a.

@zwoop I modifiied the Machine objec st that there is now an is_self() function and updated ParentSelection to use it.

@SolidWallOfCode suggested changes have been made.

Note that machine_ids would currently be a set of 1 element.
   Machine::init(nullptr, &machine_addr.sa);
For the case of mutli-homed, we need to insert the other IPs and host names.
Thanks. This will be useful in parsing lists or peers as well.

@a-canary have a look again, for each interface with an ipv4 or ipv6 address, the ip and hostname is added when resolvable.  i have non-multi homed machines and got the main hostname/ip and the localhost name/ip.  it should work just fine for multi-home hosts now

@zwoop, @SolidWallOfCode, @vmamidi I am now using ink_hash_table.

@dragon512 the autest is a good idea.  Are you okay with an issue for myself to add an autest later?  I'll have to familiarize myself with autest.

@zwoop, @SolidWallOfCode, and @vmamidi are you +1 on this pr?

yikes, I screwed the pooch on this. I'll look at this again tomorrow.

No problem, thanks @zwoop
From: Leif Hedstrom<EMAIL_ADDRESS>Reply-To: apache/trafficserver<EMAIL_ADDRESS>Date: Thursday, August 10, 2017 at 9:42 PM
To: apache/trafficserver<EMAIL_ADDRESS>Cc: John Rushford<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [apache/trafficserver] Add next-hop self detection (Issue #2269) (#2277)
yikes, I screwed the pooch on this. I'll look at this again tomorrow.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/apache/trafficserver/pull/2277#issuecomment-321726269, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AEVybt7oe0Y0KgF40BcU6b58XghXlxPOks5sW82PgaJpZM4Oao9a.

Thanks Leif, I'll merge it today.


deploy test

Does this occur consistently?
Repro steps:

Action: appService.Deploy
Error type: Error
Error Message: "RemojuWeb" is connected to a GitHub repository. Push to GitHub repository to deploy.
Version: 0.16.3
OS: darwin
OS Release: 19.3.0
Product: Visual Studio Code
Product Version: 1.42.1
Language: ja

Call Stack
Object.<anonymous> extension.bundle.js:218:3756621
extension.bundle.js:218:3754488
i extension.bundle.js:218:3754174
extension.bundle.js:218:3756338
t.ExtHostProgress._withProgress extensionHostProcess.js:495:724
t.ExtHostProgress.withProgress extensionHostProcess.js:495:478
Object.withProgress extensionHostProcess.js:787:930
Object.<anonymous> extension.bundle.js:218:3756270
fulfilled extension.bundle.js:218:3754242
processTicksAndRejections task_queues.js:89:5

@remoju it seems like you configured your web app for GitHub deploy, meaning the deploy button in our extension cannot be used. Instead, just push a commit to GitHub


Introduced application.rb variable for setting digest class (FIPS compliance)

#8163 Setting of Digest::MD5 as the default implementation will prevent clients from breaking if this PR is merged.  There is some work that will need to follow to update methods named 'md5' and 'generateMD5' (among potentially others) to be renamed into more generic methods such as 'generateDigest'

Some of these may not be doable.  If the target provider uses md5, and we are required to send an md5 string, we can't change it

This PR is really about FIPS compliance so that ManageIQ could be used in places where FIPS is required.  FIPS does not allow md5 at all (it's even blocked at the OS level, from what I can tell), so we'd have to replace all md5 usage with something else.  In most cases it's just used for corruption checks, so anything will do.  Even so, there are one or two that I think just aren't doable at all.  In those cases, I'm not sure what we can do.  We might have to wait until ManageIQ switches to have truly pluggable providers, and those providers would be excluded from FIPS-compliant distributions.
@woot4moo I've pinged the various people involved, so hopefully they can chime in on their bits.  However, I'd rather not make it configurable.  Let's just pick a sensible default in all cases...probably SHA256, and not extend configuration.

@woot4moo Also, are you planning for this to be backportable to darga?  If so, it might need a database migration to update any existing md5 values in the binary_blobs and binary_blob_parts tables.

@Fryguy  That is a fair point, however we know from a similar PR that certain CPUs will be dramatically affected by using a slower hash such as SHA256.

@woot4moo You would need the backport to darga if you want this to make it to CloudForms 4.1.  Not sure what product release you were aiming for, if at all.
The DB migration is about existing users who have md5 values already in their database.  If you change the code to use a new digest, then nothing will match in their database, and lines like https://github.com/ManageIQ/manageiq/pull/8207/files#diff-6672a8b4ee6c71e5fd77d12c59e2e825L18 will fail.

however we know from a similar PR that certain CPUs will be dramatically affected by using a slower hash such as SHA256.

Define dramatically.  I don't think we are doing that many md5s in a given period of time for it to make any difference at all, but perhaps I am wrong.  The only one that might really be affected is the Azure disk one, because that md5 is called on every disk read, but since I think we have no alternatives there, the only other option is to just not do it.

I recall the number being tossed around as a 30% performance impact.  However, I do completely agree that the way we do hashing is not going to be the limiting factor in ManageIQ's performance.   In regards to the Azure piece, I think it may be a non-issue given the citation above, will wait to see what the due diligence says.

I was not targeting any specific release.

is there anything extra that is needed or up for discussion?

@woot4moo I thought I had replied but apparently not :(
This PR needs

To hardcode usage of SHA256.  I don't think we should go the configurable route.
A bunch of changes with respect to the various discussions above, e.g. updating to SHA256 where it can, or just removing the unnecessary md5 checks entirely.
A data migration to fix any existing md5 entries.  I would even go so far as having a second migration that renames columns named md5 to something else like checksum or digest.

@Fryguy  OK I want to make sure I am reading your comments correctly.  Close out this PR, make the changes as prescribed above, make a new PR?

@woot4moo It's up to you...I think you can just amend this PR and force push and I'd be fine with that.

@woot4moo  sorry, the removal of amazon requires you to PR against two repos :(
But you can cherry-pick. Configure your repo as a remote and cherry-pick with Xsubtree
~/src/manageiq-providers-amazon master
❯ git remote -v
manageiq_upstream       git@github.com:ManageIQ/manageiq.git (fetch)
manageiq_upstream       git@github.com:ManageIQ/manageiq.git (push)
❯ git cherry-pick -Xsubtree=gems/manageiq-providers-amazon <sha>

@durandom  its OK.  I am working this on another PR since there was a lot more under the covers by the time I got to dig into this.

Closing this PR as it being worked separately.  We can always reopen if necessary.


Fix tests/unit/test_config.py::test_find_project_config in pytest 7.0

Hey, I just made a Pull Request!
Description
pytest 7.0 uses pathlib more itself and this patching ends up
interfering with it in some way. So scope the patching to the test
itself which avoids it affecting pytest itself.
Motivation and Context
pytest upstream testing pytest 7.0.0rc1: https://github.com/pytest-dev/pytest/discussions/9415
Have you tested this? If so, how?
Ran tox with pytest 7.0.0rc1.
Checklist for PR author(s)

[x] Changes are covered by unit tests (no major decrease in code coverage %).
[x] All tests pass.
[x] Docstring coverage is 100% via tox -e docs or interrogate -c pyproject.toml (I mean, we should set a good example :smile:).
[x] Updates to documentation:

[x] Document any relevant additions/changes in README.rst.
[x] Manually update both the README.rst and docs/index.rst for any new/changed CLI flags.
[x] Any changed/added classes/methods/functions have appropriate versionadded, versionchanged, or deprecated directives.  Find the appropriate next version in the project's __init__.py file.

Release note

The test suite now runs cleanly under pytest 7.

Thank you so much @bluetech !


Overhaul style sheet deprecation view and show deprecations also in non-dev mode

This pull request stops using the atom-selector-linter module, and relies on atom.styles instead to show style sheet deprecations.
Also, this enables showing the deprecation count in the status-bar again: up until now this feature was enabled only in dev mode, but especially since we are going to remove the shadow DOM boundary from atom-text-editor elements, it feels reasonable to let users see package deprecations so that they can report them to the respective package authors.
/cc: @atom/core

Glad to see this moving so quickly!


Closes #270 | Create dataset loader for OpenViVQA #270

Closes #270
Checkbox

[x] Confirm that this PR is linked to the dataset issue.
[x] Create the dataloader script seacrowd/sea_datasets/my_dataset/my_dataset.py (please use only lowercase and underscore for dataset naming).
[x] Provide values for the _CITATION, _DATASETNAME, _DESCRIPTION, _HOMEPAGE, _LICENSE, _URLs, _SUPPORTED_TASKS, _SOURCE_VERSION, and _SEACROWD_VERSION variables.
[x] Implement _info(), _split_generators() and _generate_examples() in dataloader script.
[x] Make sure that the BUILDER_CONFIGS class attribute is a list with at least one SEACrowdConfig for the source schema and one for a seacrowd schema.
[x] Confirm dataloader script works with datasets.load_dataset function.
[x] Confirm that your dataloader script passes the test suite run with python -m tests.test_seacrowd seacrowd/sea_datasets/<my_dataset>/<my_dataset>.py.
[ ] If my dataset is local, I have provided an output of the unit-tests in the PR (please copy paste). This is OPTIONAL for public datasets, as we can test these without access to the data files.

Scripts Passed:
1.```python -m tests.test_seacrowd seacrowd/sea_datasets/openvivqa/openvivqa.py
2. ```make check_file=seacrowd/sea_datasets/openvivqa/openvivqa.py

A friendly reminder for @faridlazuarda to review.

Hi, please help close this PR together with the #270


fix: The elements of the Content felt behind the elements of host website with high z-index

Details

[x] closes #156

Code of Conduct

[x] I agree to follow this project's Code of Conduct
[x] I checked the current PR for duplication.

Tho let's not use important, as we do want to allow developer to override it as needed.


name-setter-run-at-start

Change-Id: I8628ab16952a394e8fd2f19becbec8e8051ee3e3

Tested ✅
Updated this job with https://jenkins-staging.devaws.dataxu.net/job/jenkins-jobs_update_single_job/1284/
Job showing change: https://jenkins-staging.devaws.dataxu.net/job/jenkins-jobs_update_single_job/1285/

LGTM - I have this feeling that we're using an old branch of JJB that I pinned us to back when Ryan was trying to get his stuff merged upstream

yep, still pinned here: https://github.com/dataxu/jenkins-jobs/blob/master/bin/setup_virtualenv.sh#L10

https://github.com/dataxu/jenkins-jobs/pull/1379 fixes that


[HUDI-5153] Remove the optional wildcard from FSUtils#LOG_FILE_PATTERN

Change Logs
The change is introduced in about March of 2019, for supporting spark task retry. While recently we want to add new pattern to the log for cdc changelog, see https://github.com/apache/hudi/pull/7042 for details,
we would propose a new pattern like this:
// the .cdc part is optional
.fe58c163-7d3d-4413-9aa1-f6431379ec02_20220805130456743.log.2_1-0-1.cdc

the cdc suffix is optional, there are conflicts for multiple ()? sub-exprs for regex expression.
So we need to remove it now, we actually always add the write token to the file name though ~
Impact
No impact for versions after 2019/March, may break the compatibility before that time.
Risk level (write none, low medium or high below)
low
Documentation Update
Describe any necessary documentation update if there is any new feature, config, or user-facing change

The config description must be updated if new configs are added or the default value of the configs are changed
Any new feature or user-facing change requires updating the Hudi website. Please create a Jira ticket, attach the
ticket number here and follow the instruction to make
changes to the website.

Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

basically, It's not related to cdc. that https://github.com/apache/hudi/pull/7042 can work without any other changes.
This pr should just think about whether the ? optional wildcard can be removed.

Looks good, but we need to consider compatibility to all cases.


nixosTests.installer.*: increase RAM for the VM

They started to fail too often; in particular the .swraid test:
https://hydra.nixos.org/build/205689097#tabs-buildsteps
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] ~x86_64-darwin~
[ ] ~aarch64-darwin~

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Installed NixOS contain the manual by default, I believe.

I thought the usual problem is that configuration of a machine affects some details around what the manual contains.

Sounds OK to merge that.  I expect we sufficiently test the manual elsewhere, and speeding up the installer tests will be a nice bonus.


Add cover letter and interview fodder

Achievements/incidents which could feed into answers to interview questions

[ ] Highlighted need for AIF project run-down strategy (which projects, and which parts of projects should we focus on finishing, and how should the work be split up?)


How to reuse the pretrained model with different datasets?

I trained a model on a small datasets.
However, I have a much bigger and better datasets now.
Does anyone know how to reuse the pretrained model with the new datasets?
I replaced the former .tfrecord file with the new one, and use
$ tensorboard --logdir checkpoints/20180410-1445
to reload the model, but it didn't work.
It still shows the previous imgs in the tensorboard
Does anyone know how to reuse the pretrained model with different datasets?
Thank you, plz

OK， I solved the problem.....replacing the tfrecords file is working，maybe I made some mistake last time .......hahahah


Choose License

Tool to help choose an OSS license

Do you think adding tools to sidebar and opening it as iframe would be a good step? or we should stick to the current approach?

I think we should stick with the current approach. I think the goal of this project is more than just making a collection of tools, you can do that with a static page of links. The goal is to make those tools better by integrating them into one collection and offering better user experience.
It's important to give credit to the original author of those tools, and you're already doing that.
It's also important to pick a license that doesn't violate the license of those other tools, or pick tools with permissive licenses.


Hello there,
Is there a port of of flutter_localizations/flutter_localizations.dart for the flutter_web available? If yes, where?
Thank you,
JasMich

This will be auto-fixed when we are in the Flutter SDK.

When about will that be?

Can't promise dates, but we're actively working on it.


Update NAME to use a separate builder struct and value enum

For my first addition to the project I started on something simple, improving the name struct.
I did some small touch ups, most notably, added a separate builder and removed the NameFieldValue  in favor of adding the values directly onto the enum.

Question, What is the purpose of having separate short_identity_number and extended_identity_number?

Are setters needed on the NAME itself?
After name creation using a NameBuilder, there should be no need to change the NAME values, right?

Are setters needed on the NAME itself? After name creation using a NameBuilder, there should be no need to change the NAME values, right?

Probably just depends on how we want to handle cases where a CF changes NAME via the NAME management protocol or other means - if it makes more sense to construct a new NAME (it probably does?) then there's no need for it to be changeable.

Question, What is the purpose of having separate short_identity_number and extended_identity_number?

Without going into too much detail, this is something we use at Raven that is a little non-standard. I would be OK with removing them to keep us doing only what is in 11783-3

Probably just depends on how we want to handle cases where a CF changes NAME via the NAME management protocol or other means - if it makes more sense to construct a new NAME (it probably does?) then there's no need for it to be changeable.

In that case it would be usefull to keep them and use them in NameBuilder.build() as mentioned by @GwnDaan.

Without going into too much detail, this is something we use at Raven that is a little non-standard. I would be OK with removing them to keep us doing only what is in 11783-3.

No problem with keeping them as an addition.

I will follow up with an improved PR.
Thank you all for your feedback :smile:

I will follow up with an improved PR.
Thank you all for your feedback 😄

@Thom-de-Jong Ohh I didn't mean to close this PR. My thought was we can merge it, and make additional changes in another PR. So it is easier for reviewing, because these changes looked good!
One thing is that the pipeline was failing due to improper formatting, would you be able to fix that? Then we can go ahead and merge this PR https://github.com/Open-Agriculture/AgIsoStack-rs/actions/runs/5821122136/job/15782941156

O, misunderstanding.
Updated PR will come next week.

I pushed a little fix for the formatting so we can at least merge this in as it is now to get the nice things like NAME::default


[home page] make the tab/pills sticky

I take the time to set my favorite dashboards and charts, but the home page won't remember that I prefer "Favorites" over "Mine". Let's make that sticky. Let's also create a nice simple abstraction for things to be sticky. Maybe that becomes a sticky React prop for our Tab/Pills component in our design system (?)

As in save the preference to local storage or in metadata db?

local storage @nytai

I would be nice to create a framework where this is saved in the metadata db instead of local storage

my first reaction is local storage, that's how we set in Explore, but after thinking through, it might make more sense to save it in metadata db. @nytai

Let's just work with a good abstraction that should allow us to switch over to a different storage mechanism later on. As the UI gets more complex and we create more of these sticky options, it would be a shame for all of that to be lost once someone upgrades to a new computer which is fairly common in corporate settings.

+1 on good abstraction. Another benefit of saving things like this to db is cross-platform/browser consistency.


libunwind 1.5.0 Upgrade

libunwind is used by C++ components to maintain function call history (details). An upgrade is required to support newer Linux kernels.
Changes to this library are non-breaking and the changelog can be found here. Tests are all passing locally on Ubuntu 18.04.

lgtm
On Fri, Jan 21, 2022 at 12:58 PM Josh Fischer @.***>
wrote:

@.**** approved this pull request.
—
Reply to this email directly, view it on GitHub
https://github.com/apache/incubator-heron/pull/3760#pullrequestreview-860020199,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABQPBHZK5U5Y6NCA2HLQZL3UXHCHRANCNFSM5MKYFDIA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because your review was requested.Message ID:
@.***>


@conda-forge-admin, please rerender

Additional comment:
Letting the bot open a PR for Py3.11 migration

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I rerendered the recipe in conda-forge/pymatgen-feedstock#111.


[Tooltip] Use core positioning utils

Description
this does three things:

refactor(popover): move positioning stuff to core lib
refactor(popover): make arrow pos depends on itself dimensions change
fix(tooltip): use core positioning utils - this one fixes the issue

@mrfrac I slightly adjusted the code to the popover code. Check if it's ok for you.
DW @pimenovoleg

@mrfrac I slightly adjusted the code to the popover code. Check if it's ok for you. DW @pimenovoleg I've moved the positioning stuff from the popover code to the core code. Ok?

Thanks for pointing this out. Moving the positioning logic to core sounds good to me—makes it reusable for other components as well.

@mrfrac I slightly adjusted the code to the popover code. Check if it's ok for you. DW @pimenovoleg I've moved the positioning stuff from the popover code to the core code. Ok?

Hi! I'll definitely watch it today! Thanks :)

@pimenovoleg @mrfrac I've also replaced all the window and inject(DOCUMENT) with the appropirate injectWindow() and injectDocument() within popover and tooltip

@pimenovoleg @mrfrac I've also replaced all the window and inject(DOCUMENT) with the appropirate injectWindow() and injectDocument() within popover and tooltip


IntelliSense not working when using absolute import paths

VSCode Version: 1.45.0
OS Version: Deepin 15.11 Desktop 64-bit Linux

Steps to Reproduce:

Added the following to jsconfig.js

{
  "compilerOptions": {
    "baseUrl": "src",
    "module": "es6"
  }
}

IntelliScence is not working

Does this issue occur when all extensions are disabled?: Yes

Does this reproduce in the latest VS Code insiders build?
Also, please provide a small example project to reproduce the problem

@mjbvz
The update of 1.45.1 solved the issue already :) Thanks! It's working now.


Reset doesn't work if machine-controller isn't deployed or there are no MachineDeployments

What happened:
The reset command always tries to delete all MachineDeployment objects. This doesn't work if machine-controller is not deployed or there are no MachineDeployment objects.
What is the expected behavior:
The reset command works if machine-controller is not deployed or there are no MachineDeployment objects.
How to reproduce the issue:

Create a cluster using KubeOne with machine-controller disabled,
Run kubeone reset.

Duplicate of #447


tenant_route domain or subdomain

Bug description
tenant_route only works for full domains and not when subdomains are passed.
I am trying to get the tenant_route() from central context to a tenant context. Some of my tenants use subdomain others have their own domain. Right now the tenant_route function assumes all passed domains are full domains and not subdomains. So when a subdomain is passed into tenant_route (for example using tenant()->domains->first()->domain ) the url that is created becomes something like https://sudomain/route/path instead of https://sudomain.central_domain/route/path.
possible solution would be something like adding
  if (!strstr($domain, '.')) {
      $domain = $domain.'.'.config('tenancy.central_domains')[0];
  }

Steps to reproduce
$domain = tenant()->domains->first()->domain;
$url = tenant_route($subdomain, 'app.dashboard');
// url becomes https://sudomain/route/path

Expected behavior
https://sudomain.central_domain/route/path
Laravel version
11
stancl/tenancy version
3.8.4

I don't understand how this is a bug.


Switch to use zend apis for array iteration

Fixes #218

Hey @joehoyle thanks for the PR!
By any chance, did you try your code against the example in the associated issue?


Question: Release of Jest v17.0

Hi,
I would like to know if there is a release date in mind for version v17.0. Badge on Readme.md shows as if it would be already in NPM but it doesn't seem to be the case.
Thanks for the awesome work,
Leo

It is on npm. Just install 17.0.0. Their website is slow to update.

Thanks for the rapid response. I think it would have helped as well to have seen it under releases in github.


feat: move the widget creation CTA off the Entity explorer

Pull Request Template
Use this template to quickly create a well written pull request. Delete all quotes before creating the pull request.

Description

Please include a summary of the changes and which issue has been fixed. Please also include relevant motivation
and context. List any dependencies that are required for this change
Links to Notion, Figma or any other documents that might be relevant to the PR

Add a TL;DR when description is extra long (helps content team)

Fixes https://github.com/appsmithorg/appsmith/issues/17511

if no issue exists, please create an issue and ask the maintainers about this first

Media

A video or a GIF is preferred. when using Loom, don’t embed because it looks like it’s a GIF. instead, just link to the video

Type of change

Please delete options that are not relevant.

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
Chore (housekeeping or task changes that don't impact user perception)
This change requires a documentation update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions, so we can reproduce.
Please also list any relevant details for your test configuration.
Delete anything that is not important

Manual
Jest
Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

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

Checklist:
Dev activity

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

QA activity:

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

/ok-to-test sha=3e5fd1e

[ ] Widget selection box doesn't goes off in 1st click on welcome tutorial page on 6th step.
i) Button icon is getting highlighted only for the 1st time when the widget tab open is it expected?
ii)The widget box lines are not aligned properly as well. Please refer to the attached video:- https://jiju8jbmwa.vmaker.com/record/OSaCRbF5Tvju6P7F

/ok-to-test sha=5e59dba

/ok-to-test sha=b073d85

/ok-to-test sha=418881d

/ok-to-test sha=c0c0197

/ok-to-test sha=35c8b80

/ok-to-test sha=6613205

/ok-to-test sha=e889cbe

/ok-to-test sha=69f2f13

/ok-to-test sha=184aea7

@akash-codemonk

[x]  Widget selection box doesn't goes off in 1st click on welcome tutorial page on 6th step.
i) Button icon is getting highlighted only for the 1st time when the widget tab open is it expected?
ii)The widget box lines are not aligned properly as well. Please refer to the attached video:- https://jiju8jbmwa.vmaker.com/record/OSaCRbF5Tvju6P7F

expected behaviour

[x]  Widget tool-tip and widget search box are overlapping with each other. Please refer to the attached SS:-

Fixed

[x]  The border of the canvas seems truncated. Please refer to the attached SS:-

fixed

[x]  Widget box fails to disappear from the canvas after the widget selection, it stays on the canvas until and unless clicked again. Is this expected behavior? Please refer to the video:- https://jiju8jbmwa.vmaker.com/record/8L6asZ6nXPh2qcSr

expected.

[ ]  Copy the widget from the top nav bar and paste it below the viewport. Now delete it and undo the same widget. Observe that the correct widget is not getting highlighted. Please refer to the attached video:- https://jiju8jbmwa.vmaker.com/record/jDR8SVVRGLxHlvRx

Unable to repro

[x]  Widget save tool-tip stays on the canvas for a longer period of time than expected. Please refer the attached video:- https://jiju8jbmwa.vmaker.com/record/rFvxxBnGj9T3MQQW

Doesn't seem like a bug. Happens on release as well.

[x]  Widget menu box opens after every drag and drop of the widget from the top 5 widget list. Please refer the video:- https://jiju8jbmwa.vmaker.com/record/PXzj9wfaw7UtL9Zf

Fixed

[x]  widget tool-tip is getting truncated on the Page Settings page. Please refer the SS:-

Fixed

[ ]  When clicked on the + icon on the settings page the widget box doesn't open is this expected? Please refer the video:- https://jiju8jbmwa.vmaker.com/record/tXCLwHicNnMjc3l2

Unable to repro

[x]  Page settings pane completely disappears and the edit dialog still persists on the canvas. Please refer the attached video:- https://jiju8jbmwa.vmaker.com/record/8kAiwRvBWyIsSDyj

Existing issue on release. Out of scope for this PR.

[ ]  collapse the entity explorer and observe behavior of the plus button; refresh the page and check behavior. Please refer the video:- https://www.loom.com/share/1ca37de3e228476b9a9e1ac55d87ae69

Didn't understand the issue. Incorrect recording?
@Richarex
Updated dp with fixes https://appsmith-m17a897n8-get-appsmith.vercel.app/

Verified these bugs on the latest DP, and they are working fine:

Widget selection box doesn't goes off in 1st click on welcome tutorial page on 6th step - Fixed
Widget tool-tip and widget search box are overlapping with each other - Fixed
The border of the canvas seems truncated - Fixed
widget tool-tip is getting truncated on the Page Settings page - Fixed
Widget save tool-tip stays on the canvas for a longer period of time than expected - Not a bug
Widget box fails to disappear from the canvas after the widget selection, it stays on the canvas until and unless clicked again - Not a bug (expected behavior)
Page settings pane completely disappears and the edit dialog still persists on the canvas - Out of scope (existing on release)
When clicked on the + icon on the settings page the widget box doesn't open is this expected - fixed
Copy the widget from the top nav bar and paste it below the viewport. Now delete it and undo the same widget. Observe that the correct widget is not getting highlighted - Fixed
Widget menu box opens after every drag and drop of the widget from the top 5 widget list - Fixed

/ok-to-test sha=53551e2

/ok-to-test sha=53551e2

/ok-to-test sha=52d214f


ci: add auto-submit workflow for snapshot/timestamp

Signed-off-by: Asra Ali<EMAIL_ADDRESS>This PR adds an auto-submit bot for the snapshot/timestamp updates. The workflow will still trigger, and then this will run every 6 hours to (1) Check for any outstanding snapshot/timestamp updates (2) Start a review and approve and (3) Attempt to merge.
Fixes https://github.com/sigstore/root-signing/issues/553
cc @haydentherapper

Summary

Release Note

Documentation

i wll merge and see if it will work, if not we can follow up on that

i will trigger that manually instead of wait for 6 hours :)

https://github.com/sigstore/root-signing/actions/runs/3600760731/jobs/6065777450


Linkedin link gives broken response even when it isn't

Describe the bug
While searching for broken links, the LinkedIn link always gives a false-positive result. Even if it is not broken, it always gives a BROKEN result.
To Reproduce
Run the tool on a site that has the LinkedIn link and see that it gives 404 even if it is valid.
Expected behavior
Should not be giving error when it is not broken.
Environment:
All latest and up to date.

duplicate of https://github.com/stevenvachon/broken-link-checker/issues/91


Variable default value merge issue with v0.31.0

Environment
Device and OS: Ubuntu
App version: v0.31.0
Kubernetes distro being used: K3d
Other:
Steps to reproduce
mkdir -p reproduce-zarf-issue/{base,overlay}

cat << EOF > reproduce-zarf-issue/base/zarf.yaml
kind: ZarfPackageConfig
metadata:
  name: base

variables:
  - name: VAR
    default: foo
    prompt: false

components:
  - name: test
    required: true
    actions:
      onDeploy:
        before:
          - cmd: |
              echo "\${ZARF_VAR_VAR}"
EOF

cat << EOF > reproduce-zarf-issue/overlay/zarf.yaml
kind: ZarfPackageConfig
metadata:
  name: overlay

variables:
  - name: VAR
    default: bar
    prompt: false

components:
  - name: test
    required: true
    import:
      path: ../base
EOF

zarf package create reproduce-zarf-issue/overlay --confirm
zarf package deploy zarf-package-*.tar.zst --confirm

Expected result
bar
Actual Result
foo
Visual Proof (screenshots, videos, text, etc)
Severity/Priority
Additional Context
Add any other context or screenshots about the technical debt here.

when I wrote the vars + consts merges looks like I had the order wrong for the merge functions:
I had helpers.MergeSlices(node.XYZ, merged, exists), when it needs to be helpers.MergeSlices(merged, node.XYZ, exists)`
But I cannot recall what the merge strat is for Vars and Consts, when I re-wrote composition I operated under the assumption it was IfNotExist and would not do deep merging.
cc: @Racer159


dataset_statistics broken

🐛 Describe the bug
the 'dataset_statistics' training parameter seems to be broken. If I set it to true I get this error
  File "/home/kazim/.local/lib/python3.8/site-packages/super_gradients/training/sg_trainer/sg_trainer.py", line 1233, in train
    self.train_loader, all_classes=self.classes, title="Train-set", anchors=unwrap_model(self.net).arch_params.anchors
AttributeError: 'Trainer' object has no attribute 'classes'

this is with supergradients 3.2.1
Versions
Collecting environment information...
PyTorch version: 2.1.0+cu118
Is debug build: False
CUDA used to build PyTorch: 11.8
ROCM used to build PyTorch: N/A
OS: Ubuntu 20.04.6 LTS (x86_64)
GCC version: (Ubuntu 9.4.0-1ubuntu1~20.04.2) 9.4.0
Clang version: 10.0.0-4ubuntu1
CMake version: version 3.27.1
Libc version: glibc-2.31
Python version: 3.8.10 (default, May 26 2023, 14:05:08)  [GCC 9.4.0] (64-bit runtime)
Python platform: Linux-5.15.0-83-generic-x86_64-with-glibc2.29
Is CUDA available: True
CUDA runtime version: 12.0.140
CUDA_MODULE_LOADING set to: LAZY
GPU models and configuration: GPU 0: NVIDIA GeForce RTX 2060
Nvidia driver version: 535.104.12
cuDNN version: Probably one of the following:
/usr/lib/x86_64-linux-gnu/libcudnn.so.8.9.4
/usr/lib/x86_64-linux-gnu/libcudnn_adv_infer.so.8.9.4
/usr/lib/x86_64-linux-gnu/libcudnn_adv_train.so.8.9.4
/usr/lib/x86_64-linux-gnu/libcudnn_cnn_infer.so.8.9.4
/usr/lib/x86_64-linux-gnu/libcudnn_cnn_train.so.8.9.4
/usr/lib/x86_64-linux-gnu/libcudnn_ops_infer.so.8.9.4
/usr/lib/x86_64-linux-gnu/libcudnn_ops_train.so.8.9.4
HIP runtime version: N/A
MIOpen runtime version: N/A
Is XNNPACK available: True
CPU:
Architecture:                       x86_64
CPU op-mode(s):                     32-bit, 64-bit
Byte Order:                         Little Endian
Address sizes:                      39 bits physical, 48 bits virtual
CPU(s):                             12
On-line CPU(s) list:                0-11
Thread(s) per core:                 2
Core(s) per socket:                 6
Socket(s):                          1
NUMA node(s):                       1
Vendor ID:                          GenuineIntel
CPU family:                         6
Model:                              165
Model name:                         Intel(R) Core(TM) i7-10750H CPU @ 2.60GHz
Stepping:                           2
CPU MHz:                            2600.000
CPU max MHz:                        5000.0000
CPU min MHz:                        800.0000
BogoMIPS:                           5199.98
Virtualisation:                     VT-x
L1d cache:                          192 KiB
L1i cache:                          192 KiB
L2 cache:                           1.5 MiB
L3 cache:                           12 MiB
NUMA node0 CPU(s):                  0-11
Vulnerability Gather data sampling: Vulnerable: No microcode
Vulnerability Itlb multihit:        KVM: Mitigation: VMX disabled
Vulnerability L1tf:                 Not affected
Vulnerability Mds:                  Not affected
Vulnerability Meltdown:             Not affected
Vulnerability Mmio stale data:      Vulnerable: Clear CPU buffers attempted, no microcode; SMT vulnerable
Vulnerability Retbleed:             Mitigation; Enhanced IBRS
Vulnerability Spec store bypass:    Mitigation; Speculative Store Bypass disabled via prctl and seccomp
Vulnerability Spectre v1:           Mitigation; usercopy/swapgs barriers and __user pointer sanitization
Vulnerability Spectre v2:           Mitigation; Enhanced IBRS, IBPB conditional, RSB filling, PBRSB-eIBRS SW sequence
Vulnerability Srbds:                Vulnerable: No microcode
Vulnerability Tsx async abort:      Not affected
Flags:                              fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush dts acpi mmx fxsr sse sse2 ss ht tm pbe syscall nx pdpe1gb rdtscp lm constant_tsc art arch_perfmon pebs bts rep_good nopl xtopology nonstop_tsc cpuid aperfmperf pni pclmulqdq dtes64 monitor ds_cpl vmx est tm2 ssse3 sdbg fma cx16 xtpr pdcm pcid sse4_1 sse4_2 x2apic movbe popcnt tsc_deadline_timer aes xsave avx f16c rdrand lahf_lm abm 3dnowprefetch cpuid_fault epb invpcid_single ssbd ibrs ibpb stibp ibrs_enhanced tpr_shadow vnmi flexpriority ept vpid ept_ad fsgsbase tsc_adjust bmi1 avx2 smep bmi2 erms invpcid mpx rdseed adx smap clflushopt intel_pt xsaveopt xsavec xgetbv1 xsaves dtherm ida arat pln pts hwp hwp_notify hwp_act_window hwp_epp pku ospke md_clear flush_l1d arch_capabilities
Versions of relevant libraries:
[pip3] facenet-pytorch==2.5.2
[pip3] numpy==1.22.4
[pip3] onnx==1.13.0
[pip3] onnx-graphsurgeon==0.3.27
[pip3] onnx-simplifier==0.4.33
[pip3] onnxruntime==1.13.1
[pip3] pytorch-lightning==1.6.1
[pip3] pytorch-metric-learning==1.2.1
[pip3] torch==2.1.0+cu118
[pip3] torchmetrics==0.8.0
[pip3] torchvision==0.16.0+cu118
[pip3] triton==2.1.0
[conda] Could not collect

Thanks for bringing this up. I beleive this option is a leftover and will not supported.
In the future we will remove it. Sorry for the inconvenience.
To extract dataset statistics & other insights you may check Data Gradients package that we built to simplify dataset analysis and extraction of important features out of it (Currently supporting classification, segmentation & detection)


grb-sync runs a lot

grb-sync runs a lot when provisioning and deleting clusters. 3x the next closest controller is not uncommon and this appears to be because of the contention of all the handlers attempting to get their finalizer on a cluster. The more clusters the more of an issue this becomes.
Is it possible to drop the finalizer and just turn this into a sync? Or perhaps an owners reference.

This is seen when attempting to create many clusters, in my case 100, in quick succession. Using the metrics endpoint hooked into grafana using the above dashboard is the best way to see this issue.

grb-sync is now a global level lifecycle which is used to enforce the deletion of cluster role bindings based off deletion of GRBs in the management cluster.
There is a cluster scoped sync that creates the initial role of admin bindings at the time of cluster creation which adds a condition to ensure the sync completes once.
A 2nd cluster scoped sync for GRBs runs for all create/updates to grbs which creates/updates the user cluster cluster role binding.

rancher/rancher:master-head 2/11
These issues re: scalability are hard to reproduce without testing at scale, so focused on regressions via automation:

Deployed Rancher
Deployed multiple custom clusters
Ran automation smoke tests and long running tests for a total of ~400 tests
Automation runs succeeded with only a few known failures


Square brackets break Emmet in TypeScript files

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.71.2
OS Version: MacOS 12.5.1

Steps to Reproduce:

Enable Emmet for TypeScript files, using the html language
Type a snippet with an attribute, like a[href]. Emmet does not expand the snippet.

Emmet behaves as expected in HTML files. This feature used to work prior to the VS Code update that came out in mid-May 2022.
Video showing unexpected behavior in a TypeScript file:
https://user-images.githubusercontent.com/883698/191588848-4f666b06-0d4e-49a1-8ac5-1fc59f34df36.mp4
Video showing expected behavior in an HTML file.
https://user-images.githubusercontent.com/883698/191589152-29a0712a-b7c7-49df-9002-0c573fc07680.mp4

This is the reverse of https://github.com/microsoft/vscode/issues/138461.
Does enabling emmet.triggerExpandOnTab and pressing tab work?

Yes, the expansion works. It doesn't show the autocomplete dropdown, but the actual expansion does work.


Istio does not install correctly on Kubernetes 1.7.4

#640:  https://github.com/istio/istio.github.io/issues/640

Jenkins job istio.github.io/presubmit passed

thanks though this was already there below
"Install or upgrade the Kubernetes CLI kubectl to match the version supported by your cluster (version 1.7 or later for CRD support)."
but maybe people will read the top one :-)


duplication glitch

If i craft 2 portable furnaces and have them together as a stack, if i then insert item to smelt and then split the 2 furnaces the items get duplicated. I presume this isnt intended?
(sorry if my grammer/sentences arent perfect, i am not an english speaker)

Fixed in dev will be fixed in version 2.2


SecureStringArgument throws exception when Console.In is redirected

When Console.In is redirected and you use SecureStringArgument, an exception is thrown.
For instance, in my application I have the following arguments defined:
public class Arguments
{
    [ArgDescription("The username")]
    [ArgShortcut("u")]
    public string Username { get; set; }

    [ArgDescription("The password.")]
    [ArgShortcut("p")]
    public SecureStringArgument Password { get; set; }
}

And my main is similar to this
private static int Main(string[] args)
{
    var arguments = Args.Parse<Arguments>(args);

    var username = arguments.Username;
    var password = arguments.Password.ConvertToNonsecureString();

    // More stuff...
}

I run the application from another process  using the following code:
var arguments = "-u myusername";
var stdIn = "mypassword";

string stdOut;
string stdErr;

using (var process = new Process())
{
    process.StartInfo.FileName = "myPowerArgsApp.exe";
    process.StartInfo.Arguments = arguments;
    process.StartInfo.RedirectStandardInput = true;
    process.StartInfo.RedirectStandardOutput = true;
    process.StartInfo.RedirectStandardError = true;
    process.StartInfo.UseShellExecute = false;
    process.StartInfo.CreateNoWindow = true;

    process.Start();

    process.StandardInput.WriteLine(stdIn);

    using (var processWaiter = Task.Run(() => process.WaitForExit((int)TimeSpan.FromSeconds(10).TotalMilliseconds)))
    using (var outputReader = Task.Run(() => process.StandardOutput.ReadToEnd()))
    using (var errorReader = Task.Run(() => process.StandardError.ReadToEnd()))
    {
        var waitResult = processWaiter.Result;
        if (!waitResult)
        {
            process.Kill();
        }

        stdOut = outputReader.Result;
        stdErr = errorReader.Result;

        if (!waitResult)
        {
            throw new TimeoutException("Process wait timeout expired");
        }
    }
}

And the line that reads the secure string throws an System.InvalidOperationException with the message

Cannot read keys when either application does not have a console or when console input has been redirected from a file. Try Console.Read.

It is because Console.ReadKey() does not work when stdin is redirected. That behavior is documented in msdn here.
Right now, I use the following workaround to have it working in my application:
private static int Main(string[] args)
{
    var arguments = Args.Parse<Arguments>(args);

    var username = arguments.Username;

    string password;
    if (Console.IsInputRedirected)
    {
        Console.Write("Enter value for Password: ");
        password = Console.ReadLine();
    }
    else
    {
        password = arguments.Password.ConvertToNonsecureString();
    }

    // More stuff...
}

I believe that having this working out of the box would be really great in order to make it possible to launch applications that uses PowerArgs and SecureStringArgument from other applications. Maybe something similar to the workaround I have implemented in my own code could be included directly in PowerArgs?

The only problem with that workaround is that it will result in the password being displayed in the console.  That sort of defeats the purpose of using a SecureString, right?

I'm not going to make any change here.  SecureString arguments are for interactive programs where you don't want the password to be shown.  If your program can get the string from either an interactive session or via redirected input then those should be two different branches in your program.

Sorry, I went to holidays after opening the issue and forgot to follow up when I came back.
In my case here, I have an application that requires a password to connect to some third party service. Sometimes, it is used by a real user and in that case, the current SecureString behavior is perfect, as the password can never be displayed in the console.
The same application is also launched by another application, and the input and output stream are redirected from and to this parent application. In that case, there is no console where the password might be shown. That's the case that is throwing the exception.
It would have been easier for me to have both scenarios working out of the box. But anyway, I'm using the workaround that I described above which works fine for me, so this is not a blocker.
Thank you for your time and this otherwise excellent library :-)


Incorrect item count in trending search

in search it shows 11 but in reality there is only one item

won't fix snek resolver


cf_units.Unit.is_time does not work for time units with format <time units> since <reference time>

🐛 Bug Report

The cf_units.Unit.is_time method returns False for units with the format <time units> since <reference time> when it should return True.
How to Reproduce
Run the following command in the python interpreter:
$ python -c 'import cf_units; u = cf_units.Unit("days since 1990-1-1 0:0:0"); print(u.is_time())'
False

Expected Behaviour

The command should output True:
$ python -c 'import cf_units; u = cf_units.Unit("days since 1990-1-1 0:0:0"); print(u.is_time())'
True

Environment

OS & Version: Rocky Linux 8.10
cf-units Version: 3.2.0

I think this is the expected behaviour.  This method returns True for units such as seconds, days, years (without the reference).  We also have the is_time_reference method which is more likely suitable for your use case.

Thanks @rcomer - my apologies for not noticing that method sooner. Closing this issue.


Hydration mismatch in hackernews examples

Go to story page. Reload. Fix  hydration again. Cry.

Just for the sake of putting things into writing that are in my brain: I'm pretty sure this is just related to changes in the way I construct elements in leptos_meta, and I have not seen it create any actual bug in examples. I'm sure I'll fix it eventually but it's a relatively low priority for now.


Dropdowns not working in Firefox and Chrome

Summary
Dropdowns are not working in Chrome and Firefox
Actual Behaviour
When clicking a dropdown to alter the view, e.g. Columns, the dropdown is not working as expected and my browser JS console drops an error.
Expected Behaviour
Clicking a dropdown shows the options
Error in Firefox
Uncaught TypeError: i.createPopper is not a function
    _createPopper dropdown.js:237
    show dropdown.js:135
    toggle dropdown.js:117
    <anonymous> dropdown.js:436
    s event-handler.js:118
    P event-handler.js:184
    on event-handler.js:217
    <anonymous> dropdown.js:434
    <anonymous> bootstrap.min.js:6
    <anonymous> bootstrap.min.js:6

Error in Chrome
Uncaught TypeError: i.createPopper is not a function
    at Mt._createPopper (dropdown.js:237:27)
    at Mt.show (dropdown.js:135:10)
    at Mt.toggle (dropdown.js:117:49)
    at HTMLAnchorElement.<anonymous> (dropdown.js:436:38)
    at HTMLDocument.s (event-handler.js:118:19)

Tested with Firefox 105.0.1 (64-Bit) and Chrome Version 106.0.5249.91 (64-Bit) on Ubuntu 22.04

Hi @theurichde sorry about this. Fix rolling out momentarily.


Fixes initial selection state of OHLC/Candle button.

Fixes #165

Still has issues, closed.

Works now.


Update to restore the page nav after it was removed for testing on IE.

@DFurnes @mikefantini Nav should be restored after this update, tee hee :)

(╯°□°）╯︵ ┻━┻

Resolves #3769


PromiseKit for android?

Hi There, first of all thank you so much for this amazing library I absolutely love it!
I have a question regarding the FAQ:
Do you want to be able to use Promises with Swift 3.x, Swift 4.x, ObjC, iOS, tvOS, watchOS, macOS, Android & Linux? Then pick PromiseKit.
As per the FAQ above, Is there a Android version of this library? i
I really want to use it for android as well. If not, can you please recommend a alternative android library that has similar patterns?

To use PromiseKit with Android use SwiftPM.

There is a kotlin port of this library that was just released by InMotion software.
https://github.com/inmotionsoftware/promisekt
@mxcl if possible we would like to work with you to try and keep parity between your swift version and the kotlin port

@huntj88 happily accept the PRs and have this be in the official repo for sure!


Remove unused .modal-closed class

The .modal-closed class appears to be unused. I've grep'd the repo and can't find any other references to this class.
P.S. Thanks for maintaining this great resource!

@rylwin thanks! You're right. Merging this.


replace all recalculate_* methods with a single method, that takes parameters as a dictionary

The number of potentially optional parameters keeps growing. I think it might be better to replace all the recalculate_* methods with a single one. It will have one mandatory parameter (the target, which could be either Int or PoolIntArray) and then a dictionary for the optional parameters. If entries are absent in the dictionary, they'd be assumed to be default values
example:
old:
recalculate_for_targets_with_costs(targets,initial_costs,max_cost,reversed)
all parameters are mandatory at the moment.
new:
recalculate(targets, {"initial costs":initial_costs, "maximum cost":max_cost,"reversed":reversed} )
All entries in the second argument (the dictionary) are optional. If absent, default values are assumed.

I also think this is of better usability

should I try to deal with that or are you on it?

Already working on it. Rust is very bureaucratic when it comes to optional/multiple-option values. It takes time.


Fixed error main.js:5 Uncaught TypeError: typedoc.Application is not a constructor

Hi there,
this is fix for https://github.com/TypeStrong/typedoc/issues/343 issue.
Also I have compiled project with TypeScript 2.1.4.

What's the change other than recompilation?

Added /// <reference path="./typedoc/Application.ts" /> to ~bootstrap.ts

Why would that fix the referred issue? The issue you linked was a runtime issue.

I guess it happen after change system from commonjs to amd.
But in case of amd you need to set dependencies to your file. ~bootstrap.ts doesn't have it and is executed the first (but there is no Application class yet)

Just look at this code. The Application instance created before it is declared (the link to the function with Application class implementation)

That has nothing to do with a reference though. You're pointing to runtime, whereas the reference doesn't change anything for runtime.

Without a reference the order of modules is the same as in master.

Great, I see now. Thanks.

Thanks!


Issue with cloudflared tunnel run --token <token_value> in Managed CF Tunnels

First of all, congratulations on creating such an excellent library! 🎉
I have a suggestion/issue: It doesn't seem to handle starting a CF managed tunnel properly with the command:
cloudflared tunnel run --token <token_value>
I’ve looked into running it as a service but haven’t found a way to pass the token parameter for startup. Am I missing something, or is there an issue with the token handling?
Any guidance or clarification would be greatly appreciated!
Thank you!

Hi, can you share your output or error?

It doesn't actually return any errors, and the tunnel is established correctly. Since this is a managed tunnel, the ingress rules are managed through Cloudflare's web interface as expected, and everything works fine.
The issue is that I need to access the properties of tunnelProcess (see code below), but the library doesn't allow parsing the response when the tunnel being established is not a quick tunnel. For managed tunnels, Cloudflare doesn't assign an ephemeral URL, so the value of the url property will not exist. Instead, the ingress configuration is returned (which is equivalent to the config.yaml file created when the tunnel is installed as a service). See the ingress returned on the log too.
Here is the startup function I'm using:

        async function startTunnel() {

            try {
                node.status({ fill: "yellow", shape: "dot", text: "starting tunnel..." });

                // Obtén el valor del token desde el nodo de configuración
                const configNode = RED.nodes.getNode(config.token); // Asumiendo que `config` está asociado al nodo de configuración
                const token = configNode ? configNode.token : "";

                node.log(`config: ${JSON.stringify(configNode)}`);
                node.log(`token: ${token}`);

                // Configura los argumentos de la conexión en función del valor del token

                const tunnelArgs = token === "test"
                    ? { "--hello-world": null }
                    : token === ""
                        ? { "--url": "http://localhost:1880" }
                        : { "run": null, "--token": token }; // Nuevo caso para manejar tokens válidos

                // Si `tunnelArgs` está vacío, lanza una advertencia
                if (Object.keys(tunnelArgs).length === 0) {
                    throw new Error(`Invalid token: ${token}`);
                }

                const { url, connections, child, stop } = await tunnel(tunnelArgs);

                // Procesar stderr del túnel
                child.stderr.on("data", (data) => {
                    const str = data.toString();
                    node.warn(`Log de error del túnel: ${str}`);
                });

                node.log('Antes de inicializar tunnelProcess...');

                tunnelProcess = {
                    url: await url,
                    connections,
                    child,
                    stop
                };

                node.log(`Después de inicializar tunnelProcess: ${JSON.stringify(tunnelProcess)}`);

                node.log(`Tunnel process inicializado: ${JSON.stringify(tunnelProcess, (key, value) =>
                    typeof value === "object" && value !== null ? "[Object]" : value
                )}`);

                const globalContext = node.context().global;
                let tunelList = globalContext.get("tunelList") || [];

                tunelList = tunelList.filter(nodeInfo => nodeInfo.id !== node.id);
                tunelList.push({
                    id: node.id,
                    url: tunnelProcess.url,
                    status: "active"
                });

                globalContext.set("tunelList", tunelList);

                globalContext.set(`tunnelProcess_${node.id}`, tunnelProcess);

                const conns = await Promise.all(connections);

                node.status({ fill: "green", shape: "dot", text: "tunnel active" });
                node.log(`Tunnel started. URL: ${tunnelProcess.url}`);
                node.send({ payload: { url: tunnelProcess.url, connections: conns } });

            } catch (err) {
                // node.status({ fill: "red", shape: "ring", text: "error starting tunnel" });
                node.error(`Error starting tunnel: ${err.message}`);
            }
        }

This is the log returned when you enter a valid token in the code above.
9 Dec 00:38:42 - [info] [cloudflared:29ee915efc9c73d0] token: eyJhIjoi...TQyNjkwOW...iOThhMjYiLCJ0IjoiNWU...1MjctMDBkMGQ5N2RmM2ZlIiwicyI6...ORGxtWmprMSJ9
9 Dec 00:38:42 - [info] [cloudflared:29ee915efc9c73d0] Antes de inicializar tunnelProcess...
9 Dec 00:38:42 - [warn] [cloudflared:29ee915efc9c73d0] Log de error del túnel: 2024-12-09T00:38:42Z INF Starting tunnel tunnelID=5e8c00b1-45e4-8527-00d0d97df3fe

9 Dec 00:38:42 - [warn] [cloudflared:29ee915efc9c73d0] Log de error del túnel: 2024-12-09T00:38:42Z INF Version 2024.11.1 (Checksum 55d789465955ccfffcd61ba72807a2a4495002f7d9b7cc5eadcaa1f93c279d25)
2024-12-09T00:38:42Z INF GOOS: linux, GOVersion: go1.22.5, GoArch: amd64
2024-12-09T00:38:42Z INF Settings: map[token:*****]

9 Dec 00:38:42 - [warn] [cloudflared:29ee915efc9c73d0] Log de error del túnel: 2024-12-09T00:38:42Z INF Autoupdate frequency is set autoupdateFreq=86400000

9 Dec 00:38:42 - [warn] [cloudflared:29ee915efc9c73d0] Log de error del túnel: 2024-12-09T00:38:42Z INF Generated Connector ID: 325ea0aa-5c98-ac73-6ccbb7eaed3a

9 Dec 00:38:42 - [warn] [cloudflared:29ee915efc9c73d0] Log de error del túnel: 2024-12-09T00:38:42Z INF Initial protocol quic

9 Dec 00:38:42 - [warn] [cloudflared:29ee915efc9c73d0] Log de error del túnel: 2024-12-09T00:38:42Z INF ICMP proxy will use <IP_ADDRESS> as source for IPv4

9 Dec 00:38:42 - [warn] [cloudflared:29ee915efc9c73d0] Log de error del túnel: 2024-12-09T00:38:42Z INF ICMP proxy will use ::1 in zone lo as source for IPv6

9 Dec 00:38:42 - [warn] [cloudflared:29ee915efc9c73d0] Log de error del túnel: 2024-12-09T00:38:42Z INF Starting metrics server on <IP_ADDRESS>:46611/metrics

9 Dec 00:38:42 - [warn] [cloudflared:29ee915efc9c73d0] Log de error del túnel: 2024/12/09 00:38:42 failed to sufficiently increase receive buffer size (was: 208 kiB, wanted: 7168 kiB, got: 416 kiB). See https://github.com/quic-go/quic-go/wiki/UDP-Buffer-Sizes for details.

9 Dec 00:38:42 - [warn] [cloudflared:29ee915efc9c73d0] Log de error del túnel: 2024-12-09T00:38:42Z INF Registered tunnel connection connIndex=0 connection=422ff6d3-f186-9060-c7d0ea6633d2 event=0 ip=<IP_ADDRESS> location=mad01 protocol=quic

9 Dec 00:38:43 - [warn] [cloudflared:29ee915efc9c73d0] Log de error del túnel: 2024-12-09T00:38:43Z INF Registered tunnel connection connIndex=1 connection=df8dbd2f-4dc7-a3dc-6dc960cd7539 event=0 ip=<IP_ADDRESS> location=mad04 protocol=quic

9 Dec 00:38:44 - [warn] [cloudflared:29ee915efc9c73d0] Log de error del túnel: 2024-12-09T00:38:44Z INF Registered tunnel connection connIndex=2 connection=942f5402-92f7-8969-a00e2ccc6cfb event=0 ip=<IP_ADDRESS> location=mad03 protocol=quic

9 Dec 00:38:44 - [warn] [cloudflared:29ee915efc9c73d0] Log de error del túnel: 2024-12-09T00:38:44Z INF Updated to new configuration config="{\"ingress\":[{\"hostname\":\"host.mydomain.com\", \"service\":\"http://localhost:1880\"}, {\"service\":\"http_status:404\"}], \"warp-routing\":{\"enabled\":false}}" version=1

9 Dec 00:38:45 - [warn] [cloudflared:29ee915efc9c73d0] Log de error del túnel: 2024-12-09T00:38:45Z INF Registered tunnel connection connIndex=3 connection=0a670d23-5051-9ffa-38ba17148282 event=0 ip=<IP_ADDRESS> location=mad02 protocol=quic

If you need a valid token for your tests, and you don't have one, happy to share one.

Hello @JacobLinCool , have you had a chance to take a look at the logs?
I was thinking that perhaps the ideal solution would be for the library to also return the "ingress" property, where the response sent by Cloudflare when a managed tunnel is deployed could be parsed. The idea is to follow the same design principle used with free tunnels, where the access URL content is retrieved and parsed. What do you think?

I'm not really sure what you mean by "ingress" here. Did you mean streaming out the raw stdout/stderr to downstream library users? I think it's a good idea.
I just tested it with the CLI (using cloudflared 2024.12.1 under the hood), and it seems to work (cloudflared tunnel run --token <token_value> works). So, I think you are referring to the library part, not the CLI part. Is that correct?

I’ve created a new PR #28  that parses the configuration file. You can check it out here and let me know if it fits your use cases. You may be interested in examples/events.mjs.

Awesome!... will try it and get back with findings!!!
Thanks!

Works great!


[Hackadoc]Updated article to include server upgrade advisor tool.

Hi @CptMonac, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

This article mixes vNext content with vCurrent content. This goes against the policy about separating content about vNext from vCurrent. Please work with your content team about the right next steps. Closing so you can resolve next week.


Showing inconsistent result when running in Parallel Mode using retry analyser

TestNG Version

When Running in non-parallel mode

When Running in parallel mode

Expected behavior
It should run consistently and should display same result in both parallel as well as non-parallel mode
Actual behavior
It is showing different result in both mode.
Is the issue reproductible on runner?

[ ] Shell
[ ] Maven
[ ] Gradle
[ ] Ant
[ ] Eclipse
[ ] IntelliJ
[ ] NetBeans

Test case sample
Test Cases.zip

@ujjwal3687 - Couple fo things

Your screenshot shows that this is from the TestNG eclipse plugin. Is that the case ? If yes, then you should be logging a bug with the TestNG eclipse project here
The attached project shows that you are using 6.14.3. Please update to using 7.0.0 which is the latest released version of TestNG. If its still reproducible with 7.0.0 via the command line or via a build tool such as gradle/maven then please create a simple standalone test project which can be used to reproduce the problem (You seem to have attached a complete project which contains a lot of other stuff including selenium)

Hi,
This is the issue not related to either TestNG or Maven. This issue is
coming to both ways if i execute the code either through build tool such as
Maven or through TestNG showing same issue.
It's not related to 6.14.3 version even i cross verified with 7.0.0,
showing same error.
Kindly provide proper solution if you have.
On Fri, Nov 1, 2019 at 12:18 PM Krishnan Mahadevan<EMAIL_ADDRESS>wrote:

@ujjwal3687 https://github.com/ujjwal3687 - Couple fo things

Your screenshot shows that this is from the TestNG eclipse plugin.
Is that the case ? If yes, then you should be logging a bug with the TestNG
eclipse project here https://github.com/cbeust/testng-eclipse
The attached project shows that you are using 6.14.3. Please update
to using 7.0.0 which is the latest released version of TestNG. If its
still reproducible with 7.0.0 via the command line or via a build tool
such as gradle/maven then please create a simple standalone test project
which can be used to reproduce the problem (You seem to have attached a
complete project which contains a lot of other stuff including selenium)

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cbeust/testng/issues/2183?email_source=notifications&email_token=ALOIS7PSARQJ65OAUYMHUPLQRPGMHA5CNFSM4JGE3F22YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEC2EPIA#issuecomment-548685728,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALOIS7NF3F6TE3IXQAEV42DQRPGMHANCNFSM4JGE3F2Q
.

@ujjwal3687 - Then please share a simple standalone test project which can be used to reproduce the problem (Please dont include any 3rd party libs apart from TestNG)

No response from user. Closing this issue.
If this is a problem please share a sample that can be used to reproduce the problem and we can get this issue re-opened again.


Added new AI personality

Foundational work for new AIs, along with one new AI that will always play a winning card if it can.

Group Game.zip


[KZG] Serialize roots of unity as hex as well

This serializes the roots_of_unity as hex as well.
Most JSON readers would fail to deserialize a bigint encoded as int and while most cryptographic libraries have a hex->internal_repr converter, many don't have a decimal->internal_repr converter, in particular BLST.

bigint encoded

In CL specs, we use SSZ to define ROOTS_OF_UNITY as a Vector[BLSFieldElement, FIELD_ELEMENTS_PER_BLOB] type. Although I support that polynomial-commitments.md should actually be independent of SSZ as it's laid in a cryptography black box, I feel it's more consistent to use SSZ serialization for preset in this case at the current stage.
^^^ However, it may be messier per #3354. 🫨

I agree with @mratsim that these should be hex-encoded. On top of that, I'm not convinced these should be a JSON file either. As was discussed at Edelweiss, most crypto libraries won't have a JSON parser, so it seems prudent to have an even flatter (text) file.

closing via #3521