Aid Effect not Triggering Modifiers Matter

The Aid effect isn't getting highlighted when it's the effect that tips a check into the next degree of success. I know this used to work, as I'd configured a specific animation for it. I'm not sure what happened that caused it to stop being picked up. I've confirmed other bonuses are highlighting green as expected within this world. Screenshots included. Thanks for your work on this module!

Example of Modifiers Matter working:

The error wasn't with Aid, it was with targeting AC.  Same issue as in #23.  Fixed for next release, v1.7.6


Improves readability

You do it everywhere - stop using c code in cpp pls.
Use the iostream instead of stdio.h.
I don't even look at the code anymore - I just indent it, so my coworker can read it better for practice.

Reverting code to former coding standards. "{" are not on new lines.

i know it blows up the code with some more lines, but the readability gets way better, i would take a try

btw - use tab as spaces (2 or 4) - it will help with foramtting the code with commentations ect.


REPL driven development

Currently there is no way how user can change rules while maintaining same state. Think about it. Maybe we can use what-if function once it's ready?

Too soon to create issues


Feature Request:  Magic Item stat block

Would be awesome if you could have magic item stat blocks similar to the monster statblocks that allows customization as well.  Love what you have done so far!!!

Magic Items, and any Item Statblocks, are now fully possible by creating a custom layout and enabling the fields to render markdown styling.


Uncaught Error: Class 'Language' not found

Hello I have some problems with latest version.
Get leadeboards is not working because:
Uncaught Error: Class 'Platform' not found
news and store not working because
Uncaught Error: Class 'Language' not found
Full error:
FastCGI sent in stderr: "PHP message: PHP Fatal error: Uncaught Error: Class 'Language' not found in /domain/file.php:5
Stack trace:
#0 {main}
thrown in /domain/file.php on line 5" while reading response header from upstream

My code:
<?php
require_once 'vendor/autoload.php';
use Fortnite\Auth;
$auth = Auth::login('x@x.x','x');
var_dump($auth->store->getStore(Language::ENGLISH));

<?php require_once 'vendor/autoload.php'; use Fortnite\Auth; use Fortnite\Platform; use Fortnite\Language; $auth = Auth::login('x@x.x','x'); var_dump($auth->store->getStore(Language::ENGLISH));
Should work by adding "use Fortnite\Platform" or "use Fortnite\Language".
Also this repo does not have the modifications you're using so i suggest adding any further issues you have to the fork which does.


Migrate to bevy 0.8

Issue tracking the migration possibility.
Core crates:

[x] bevy_asset_loader
[x] iyes_loopless
[ ] bevy_ecs_tilemap
[x] bevy_framepace

Crates used for debugging:

[ ] bevy_mod_debugdump
[ ] bevy_inspector_egui

IyesGames/iyes_loopless@3c5ebad4ae24fb3f30f03acb607e80d934fa7ad7 -- iyes_loopless migrated to 0.8

aevyrie/bevy_framepace@bf67f3350ad8da76966de29264fb4b057b2560e6 -- bevy_framepace migrated to 0.8

jakobhellermann/bevy-inspector-egui@2b189ee26cd97f761a0c94513f0cb8e4f029fb67 -- bevy_inspector_egui migrated to 0.8

jakobhellermann/bevy_mod_debugdump@e52dbf6e0e4260aa898b2c3903721156a0f9e2f7 -- bevy_mod_debugdump migrated to 0.8

StarArawn/bevy_ecs_tilemap@f69316a781f8537b6cd0dd91e36f412931c2254d -- bevy_ecs_tilemap migrated to 0.8

Current migration roadmap:

[ ] Basic level spawning
[ ] Moveable tech
[ ] Player systems
[ ] Tile animations (loading)
[ ] Tile tagging
[ ] Tile activation
[ ] Tile animation (using it in the logic)

Code cleaning:

[ ] Split Level into LevelTiles and LevelGraphics -- the first carries the level geometry, the second carries around tile indexing
[ ] Split ActivatableTag into ActivationCondition and ActivatableAnimation
[ ] Push tiled level parsing into a submodule and seal the parsing behind a trait


Please make it php5.3.* compatible

Composer installs only 0.3 version of this library, because all version after that require php5.4.0 or later.
But that version has problems with lower/uppercase name.
php cli 5.3.3
php web 5.3.28
My workaround is
cp AuthorizationCode.php  Authorizationcode.php

But that is no solution for a new Flyspray release we plan.

One of the League requirements is that the packages be PHP 5.4+. Additionally, the class name is correctly AuthorizationCode. Why do you need to make the "C" lowercase?

On Mar 4, 2015, at 17:12, peterdd<EMAIL_ADDRESS>wrote:
Composer installs only 0.3 version of this library, because all version after that require php5.4.0 or later.
But that version has problems with lower/uppercase name.
php cli 5.3.3
php web 5.3.28
My workaround is
cp AuthorizationCode.php  Authorizationcode.php
But that is no solution for a new Flyspray release we plan.

Reply to this email directly or view it on GitHub.

The bug he's referring to is one that was around before you took over @ramsey. There was an open bug where it didn't properly uppercase the C in authorization code for the default grant type.
@peterdd since 5.4+ is the minimum for the package now, your only solution is the one you presented, or you can fork the project and make changes to AbstractProvider. You want to change the getAccessToken method to include this line
$className = str_replace(' ', '', ucwords(str_replace(['-', '_'], ' ', $grant)));

instead of the old, incorrect casing rules.
But those are your only two solutions unless you upgrade PHP.

Why not simply fix classname capitalization and tag as 0.3.1?

The current active devs at https://github.com/Flyspray/flyspray aren't experienced with that composer stuff. php5.2.* and php5.3.* is still wide spread on hostings and we want Flyspray be very easy to install and userfriendly.
This oauth2-client seems so far the only showstopper on php5.2.* or php5.3.* environments, so if you know a solution that we can until the release we plan around end this month we would be very glad.
Maybe some composer.json modification? Well, we just don't know to less about composer.
@rakeev: If that is possible and the composer can handle that automatically that would be great!

I'm not sure if @ramsey wants to tag, but I went ahead and fixed this over here: https://github.com/tpavlek/oauth2-client.
@peterdd here is how you can fix this for your projects. First, go to the above link, I've already performed the capitalization fix and tagged a 0.3.1 release. In the upper right hand corner, click fork, and create a new fork of that project in your account.
You should now be on https://github.com/peterdd/oauth2-client or something similar.
You can now instruct any applications that use oauth2-client to use your fork instead of the official PHP League one. All you have to do is in the composer.json of the project requiring PHP 5.3 support add a section:
"repositories": [
    {
        "type": "vcs",
        "url": "https://github.com/peterdd/oauth2-client"
    }
]

Now, when you run composer update, it will try to install oauth2-client from that repository instead. That way you can still maintain PHP 5.3 support for your projects, regardless of what this project officially supports.
I hope this helps.

Yeah, best way would be a official 0.3.1 version and composer chooses this version when the user with php <5.4.0 version install Flyspray.
Users with newer php versions should be on the offical channel, maintaining additional an own repository of oath2-client sounds a bit painful.

I tried to make a branch "php5.3 compatible" .
But now this dumps Travis CI: https://travis-ci.org/thephpleague/oauth2-client/builds/53698862
PHP Fatal error:  Using $this when not in object context in /home/travis/build/thephpleague/oauth2-client/test/src/Provider/AbstractProviderTest.php on line 90

Could someone please dig into this and fix it for working all versions php5.3 to current tested php versions?

I'm just going to put it out there that you're asking for support for an unsupported version. The entire purpose of not supporting old version of PHP is so that the maintainers of the project don't have to support it. It's rather cyclical.
Ramsey might do that. I wouldn't if I were him. Since you want to use the package in the long-haul with an unsupported version of PHP, maintaining your own fork would likely be the best option for you.
Just my 2 cents. Someone else might chime in and help. Maybe.

As I said earlier, PHP 5.4+ is our minimum version. League packages right now are setting that as their baseline, and we will support PHP 5.4+, but not earlier. Sorry.
As @tpavlek suggested, this is a good case for maintaining your own fork of the project. You may still use your fork with Composer.

:-(


Create a db pool

We have an async mysql connection https://github.com/rryqszq4/ngx_php7/blob/master/t/lib/mysql.php
But the biggest cost is the time connecting via TCP to the db.
Any persistent app use a pool of connections, so they don't have the overhead of connect and disconnect.
Example in swoole:
https://github.com/TechEmpower/FrameworkBenchmarks/blob/master/frameworks/PHP/swoole/swoole-server.php#L188-L247

@joanhey
The underlying layer uses the nginx connection pool. In php, the ngx_socket_* related function is just a layer shell. The implementation reference ngx_lua.
Currently, only the nginx directive "php_keepalive" supports the configuration of the connection pool capacity, the configuration method in php has not been implemented.
You can see that.
https://github.com/openresty/lua-nginx-module#lua_socket_pool_size
or
https://github.com/openresty/lua-nginx-module#tcpsocksetkeepalive

I am rethinking, maybe necessary an object pool in ngx_php?!

Check the results in the last run:
https://www.techempower.com/benchmarks/#section=test&runid=92383925-3ba7-40fd-88cf-19f55751f01c&hw=ph&test=fortune&l=zik073-v&p=zik0zj-zik0zj-zik0zj-zdk8an-v
We are losing more time connecting, reconnecting and disconnect, that receiving data of the queries.
If we can have more async connections will be better.

@joanhey
In the v0.0.22 version, some performance optimizations have been done. My local test results are doubled in performance. Let's wait for the test results to come out.


[feature request] add multi field editor

Hi, something else: I would like to have a multi field editor. Just multiple Input fields (if possible in more than one column, to save line-space) which can be cycled by arrow up/arrow down and tabulator. What do you think about such a feature?

Can you please explain more on how this would work etc..?

I imagine something like this: multiple normal inputs, organized row by row in a multicol design which makes clever use of the available terminal width. Focus can be cycled through the fields by tab and arrow up/arrow down.
However, I realized that this might come closer to a full TUI lib and less to a dialoguer. Currently I'm investigating whether it is the right step to ditch dialoguer for cursive, as this feature becomes out of scope for dialoguer eventually. That however is not my decision to make :smile:
Is it more clear now?

Yeah, it is out of scope for this library.


Add missing next import

🚀 Deployed on https://6757d4f4ccf4f2a8d906e175--cozy-fox-0defec.netlify.app


Discuss: Transfer maintenance of this project into the libass org?

I noticed that this repo hasn't had any commits in over a year now; as the project has gained fairly decent adoption, it would probably benefit from a slightly larger maintainer community. @Dador, would you like to move this package into the libass org as an official libass-js binding? If you'd like to discuss in more detail, we're available at #libass on the Freenode IRC network.
If we don't hear back on this for a while, we'll probably assume this repo is abandoned and figure out future maintenance on our own, but I'd much rather work with the original authors on it if possible.

@Dador, would you like to move this package into the libass org as an official libass-js binding?
Sounds good. I'll write to IRC channel.

Lately this project was maintained mostly by @TFSThiagoBR98 and @JustAMan.
I didn't worked on this project in a while but if you'll have any questions I'm happy to help.

I see we missed you on IRC, but the transfer procedure is documented here: https://docs.github.com/en/github/administering-a-repository/transferring-a-repository

@rcombs, I couldn't send transfer request directly to the libass org, as:

To transfer a repository that you own to an organization, you must have permission to create a repository in the target organization.

So I sent transfer request to you, so you can then transfer it to the libass org.

I think is better to also transfer the npm package as well, @rcombs can you create an org in npm?

Created the libass org; you should have access.

Closing this out, as the bulk of the transfer is complete.

@rcombs can you give me permission to create packages in the org to make the transfer?

Believe you should have it now.

Ok, transfered

Sorry for the necro, but I just discovered this and wanted to say "Congratulations" :)


Verify continuous power status via XCAT

Verify continuous power status via XCAT

Test is already present in xcat (i.e. tag - Continuos_Node_Power_Status)


Fix evidence links

Some (all?) of the evidence code links aren't working from annotation sections.  eg.
http://geneontology.org/page/imp-inferred-mutant-phenotype
http://geneontology.org/page/igi-inferred-genetic-interaction
Does anyone know what the new links are?
(I thought we fixed this a while ago when the pages moved?)

Maybe these wiki pages?:
http://wiki.geneontology.org/index.php/Inferred_from_Mutant_Phenotype_(IMP)
http://wiki.geneontology.org/index.php/Inferred_from_Genetic_Interaction_(IGI)

I found the old issue: https://github.com/pombase/website/issues/911
It sounds like, for a while, there were redirects from the old URLs to the wiki pages:
https://github.com/geneontology/go-site/issues/582#issuecomment-425378208
That GO issue has a nice table of old versus new so I've fixed the links.


[GSK-854] Disparate impact test doesn't work in case of numeric classification_labels

to reproduce:

Take "Credit scoring classification_model" demo notebook
add credit.default = credit.default.apply(lambda x: 1 if x=='Default' else 0)  after the credit dataset is created
follow the rest of the notebook until the upload_model_and_df
create a test suite automatically
run disparate impact test

From SyncLinear.com | GSK-854

Thanks Andrey, I'll have a look asap!

fixed here: https://github.com/Giskard-AI/giskard/pull/878


support db and password config for redis connection

parse REDIS_CONN with redis.ParseURL in order to support db and password config for redis connection

 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.Junfeng Ye seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Can anyone make some explaination on GFW identification

Connection speed is one of the key factors we care. Another key factor is safety. From your point of view, does GFW have the ability to analyse the data usage or packet structure to identify its proxy behavior?

does GFW have the ability to analyse the data usage or packet structure to identify its proxy behavior?

It doesn't, for now, obviously. Otherwise it would be unusable for users in China already.
But you are more than welcome to look into potential detection methods for the current protocol, and I will try my best to fix them.


Added method to set page output cache

Added method to set the page output cache to improve the page load time, based on pre-existing cache profiles.

Thx Nanddeep. I'll be looking into this.


length arg error of SE2.plot() function (trplot2)

I found an error in trplot2()
The code below reproduces the error
a = SE2.Rand()
a.plot(length=0.1)

Sorry. I hadn't checked the code, but found it was also wrong. I'll reupload.


pacman?

I'm not sure if it fits with your design goals or not, but have you considered including pacman?  I think that would make it a lot easier to add other tools.

A primary goal of w64devkit is support for fully offline development, an
area I feel has been neglected. An internet connected package manager that
depends on external services doesn't fit with this goal. That domain is
already well-covered by MSYS2, so I'll leave it to them.
Besides, I have a rustic vision of a team choosing w64devkit as the
toolchain for their project. It will never change nor surprise them with
"updates", and everything they need is packed away in a .zip that they can
archive alongside their project. Unlike a lot of software, they would
never lose the ability to compile from source. Introducing a package
manager undermines this idea and creates the temptation to introduce
unnecessary dependencies.

OK, thanks for the explanation.  I was looking for an easy way to add clang/llvm.


Feature request: transition guards

Does SCION-CORE support transition guards? If not, are there any plans to implement them?

Nevermind, just found out guards are indeed implemented and called "cond", presumably based on SCXML.


Other Vtuber Groups

Wondering if you had any plans to add any other Vtuber groups. I watch a lot of Nijisanji as well and would love to be able to use Holodex for finding their content as well.

At the moment, Holodex will probably be hololive only. I have given it some thought, and the two options is to either make a separate website similar to Holodex but called Nijisanjidex, or merge everything together (but might involve a few challenges that needs to be worked out).
TLDR: Needs a little bit more thought.

A solution I was thinking of regarding this issue is, can you add an option to manually add in our own channels to keep track of in Holodex? A very useful feature on Hololive.jetri.co for example is that you can add any youtube video/stream to play alongside Holo members by adding the URL, and I'm wondering if you can do the same for channels on Holodex.
The list of added channels could then be saved to cookies and an individual could have their favorite Hololive channels alongside any other Nijisanji/Indie vtubers they want to watch.

A solution I was thinking of regarding this issue is, can you add an option to manually add in our own channels to keep track of in Holodex? A very useful feature on Hololive.jetri.co for example is that you can add any youtube video/stream to play alongside Holo members by adding the URL, and I'm wondering if you can do the same for channels on Holodex.
The list of added channels could then be saved to cookies and an individual could have their favorite Hololive channels alongside any other Nijisanji/Indie vtubers they want to watch.

Our server does a lot of the tagging and tracking for each channel and video, so cookies by itself would not be able to power some of the features that we have.
However, we are currently working on support for many of your favorite vtubers in the next major upgrade. Stay tuned!

Our server does a lot of the tagging and tracking for each channel and video, so cookies by itself would not be able to power some of the features that we have.
However, we are currently working on support for many of your favorite vtubers in the next major upgrade. Stay tuned!

Major update has been released, and we now track over 400+ vtuber channels

Major update has been released, and we now track over 400+ vtuber channels


Unable to import module '[module_name]': /var/task/Crypto/Cipher/_AES.so: invalid ELF header

I'm trying to use credstash to retrieve credentials in an aws lambda function e.g.
from credstash import getSecret
password = getSecret(USERNAME)

But I'm getting the following error when the function runs:
Unable to import module '[module_name]': /var/task/Crypto/Cipher/_AES.so: invalid ELF header

I'm not sure what the cause is. My lambda function lives in a virtualenv, and when I deploy I copy the contents of lib/python2.7/site-packages/ into the zip. Do you have any ideas?

Did you build your virtualenv on a linux box? The Invalid ELF header error makes me think that the openssl shared library is messed up

Whoops. I'll bet that's what it was. I'll check right now.

Yep, that was all it was. Sorry to trouble you!

Sorry to bring this one back up, but I am relatively new to python.  I am trying to use credstash in lambda function and am getting this error as well.  How do I un mess up my openssl?  I am working with python 2.7.10 on OS X, by the way.
Thanks

Similarly, also have the exact same issue.

The python openssl library brings a shared library along with it. You want to make sure you build your virtualenv on a linux box. If you build it on OSX, the shared library will be build for OSX, not for linux.

Thanks, @alex-luminal.  I wound up doing that but am taking it a step further and using a docker container for this.  That gets me a linux os for packaging with the shared lib but still keeps the development local.

Thanks a million @alex-luminal saved my day!


No module named 'pretrain_trfm'

Thank you so much for your implementation.
When I ran your experiments/ablation1.jpynb, I got the following error message:
ModuleNotFoundError                       Traceback (most recent call last)
 in
1 import torch
----> 2 from pretrain_trfm import TrfmSeq2seq
3 from pretrain_rnn import RNNSeq2Seq
4 from bert import BERT
5 from build_vocab import WordVocab
ModuleNotFoundError: No module named 'pretrain_trfm'

The error occurs because "pretrain_trfm" is not seen from .ipynb file.
Import by from ..smiles_transformer.pretrain_trfm import TrfmSeq2seq or set path by
import sys
sys.path.append("../smiles_transformer/")

c.f. https://stackoverflow.com/a/50011984

Thank you. The second option works.
I am not sure if you missed init.py file at the folder /smiles_transformer


livecheck: add support for casks

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew tests with your changes locally?

This allows brew livecheck to work on casks and adds a new livecheck stanza to the Cask DSL. Since many casks use methods such as version.before_comma, a new version field is added to the Livecheck class that accepts either a String or a valid Symbol (before_comma, after_comma, before_colon, after_colon)
Example livecheck for ansible-dk:
livecheck do
  url :homepage
  version :before_comma
end

Great work.
Would the url in the livecheck block accept :appcast?

Would the url in the livecheck block accept :appcast?

also implicitly using the :appcast if nothing is specified would make sense.

I don't often find myself in homebrew-cask but I happened to be looking at a cask the other day and thinking about adding support for casks to livecheck. Good timing!
I'm in the process of reviewing and testing this and I'll have some feedback and requested changes once I'm finished.

Would the url in the livecheck block accept :appcast?

@miccal Yes

also implicitly using the :appcast if nothing is specified would make sense.

we might want to have the appcast URL take highest priority (and be the first addition to the urls array).

@core-code @nandahkrishna Done (I'm assuming all casks at least have url since a cask would be useless without it)

Also, would --formulae-only and --casks-only flags for the brew livecheck developer command be useful?

Yes, probably

"version"  => @version,

Would we want to omit this from the hash unless @formula_or_cask is a cask?

Maybe, but the doc comment says "Returns a Hash of all instance variable values"

-      @livecheck ||= Livecheck.new(self)
-      return @livecheck unless block_given?
-      raise CaskInvalidError.new(cask, "'livecheck' stanza may only appear once.") if @livecheckable
-      @livecheckable = true
-      @livecheck.instance_eval(&block)
+      set_unique_stanza(:livecheck, !block_given?) do
+        livecheck = LiveCheck.new(self)
+        livecheck.instance_eval(&block)
+        livecheck
+      end

@reitermarkus I actually did this initially but the Livecheck utility requires livecheck method to return a non-nil value

One thing that sticks out to me right now is the relative verbosity of the various formula_or_cask language. It would be nice if we could find a generic way to refer to the concept of a formula or cask that's shorter but maybe this is just me.

Yes, I agree — what about recipe or item?

Hopefully we can look through the URLs and identify patterns that can be handled with additional strategies.

I'll try my best

I just now saw Nanda's suggestion about the same flags, ahah.
😄

As for the flags, it seems brew list has --formula/--formulae and --cask/--casks so we could go with those for consistency.

List of casks without an automatic livecheck strategy (2,319 total): https://gist.github.com/SeekingMeaning/6c7a6d072a562c6ac474d5c0bb558076

One thing that sticks out to me right now is the relative verbosity of the various formula_or_cask language. It would be nice if we could find a generic way to refer to the concept of a formula or cask that's shorter but maybe this is just me.

I'm fine with this, personally. Would definitely prefer over anything like fc being used more widely.

List of casks without an automatic livecheck strategy (2,319 total): https://gist.github.com/SeekingMeaning/6c7a6d072a562c6ac474d5c0bb558076

if you are looking for text patterns to find new versions of casks - i can contribute a few dozen of those.
patterns.txt

@SeekingMeaning, anything still missing here?

@SeekingMeaning Would love to get this in in the nearish future. We can iterate on this in future PRs; I don't think this needs to be perfect to be merged.

Review period ended.

With this having been rebased, I'll take another pass to review and test this as soon as I can. Thanks for helping to get this into shape, @reitermarkus.

I have removed version completely for now, since I don't think it is really that useful for casks to know whether the version actually increased.

I don't think it is really that useful for casks to know whether the version actually increased

I can't speak for casks but we use it with formulae to identify when a livecheck block may need to be fixed or if a formula has been updated to a version it shouldn't be using.
For example, it's sometimes necessary to check a URL other than stable, head, or homepage and those livecheck blocks can become out of sync when a formula is updated to use an archive from a different source. In those cases, it becomes apparent when the formula version is newer than the version that livecheck is reporting as newest.

I think whether a livecheck block should be fixed should be determined by brew audit. Basically, check during audit whether the version matches the current version.

Oh, nevermind, ignore that comment. I hadn't looked at this recently and I only skimmed the change (missing the context) and thought that version was referring to something else.
The purpose of the version addition to the livecheck DSL is to ensure that livecheck's version comparison works correctly. Picking a random cask out of a hat, a check for appcode could look like this without version:
livecheck do
  url :appcast
  regex(%r{/AppCode[._-]v?(\d+(?:\.\d+)+)\.dmg}i)
end

The version variable for appcode is currently 2020.3,203.5981.213, so by default livecheck would return appcode : 2020.3,203.5981.213 ==> 2020.3 (where the cask version appears newer than upstream). The appcode cask instead uses version.before_comma in url, so livecheck needs to follow suit.
The goal of the version addition to the livecheck DSL is to be able to tell livecheck to use version.before_comma, like:
livecheck do
  url :appcast
  regex(%r{/AppCode[._-]v?(\d+(?:\.\d+)+)\.dmg}i)
  version :before_comma
end

Though this may not be exactly the shape this takes in the end, the previous code in this PR would produce correct livecheck output: appcode : 2020.3 ==> 2020.3.
That said, I'm good with removing the version DSL addition from this PR, as I think it should be handled in a follow-up PR. I believe there's still some discussion needed around what shape it should take and this PR is already lengthy enough, so it makes sense to review it separately.

The appcode cask instead uses version.before_comma in url, so livecheck needs to follow suit.

If the cask only uses version.before_comma, we can just change to cask to only that part.
Having livecheck return a version which is not equal to the format of the cask's version is useless in my opinion. We want livecheck so we can automatically update casks ideally, so livecheck should return the full version, including the comma, if anything.
In any case, I think we should merge this as-is now and improve on it. From my tests this is working well enough for simple x.y.z versions, everything else can be tweaked when we actually get to it in actual casks.

Having livecheck return a version which is not equal to the format of the cask's version is useless in my opinion. We want livecheck so we can automatically update casks ideally, so livecheck should return the full version, including the comma, if anything.

Ah, I was only thinking about the version used in the url, as I'm not very familiar with casks (what their versions are like, how they're updated, etc.). The version as it appears in the cask (2020.3,203.5981.213) isn't available in the same format upstream and livecheck would have to combine the "version" (2020.3) and "build" (203.5981.213) fields from the appcast JSON data for that to work.
There are different ways to approach this but it's not currently possible in livecheck. There's a lot to discuss about how to approach this, so we can create a separate issue sometime after this PR is done. I'm planning to create a PR for a related feature in the livecheck DSL (once I finish the livecheck docs PR) and it could theoretically be leveraged in this context as well. I'll save the explanation for a later discussion, though.

In any case, I think we should merge this as-is now and improve on it. From my tests this is working well enough for simple x.y.z versions, everything else can be tweaked when we actually get to it in actual casks.

I'm in the process of reviewing and testing this to ensure we aren't breaking any existing behavior with regard to formulae. I'm aiming to get through it by the end of the day (EST) but please don't merge this until then.

please don't merge this until then

Please feel free to merge it once you have reviewed it.

I'm about 90% of the way through this but I've run out of time/energy for today. There are some areas that I'll address by pushing commits tomorrow and others that warrant some brief discussion (I'll add review comments and, once we decide how to proceed, I'll push commits to address those as well).
These changes won't be time-consuming but there's at least one functional issue that we definitely need to address before merging, so I'm going to label this do not merge for the moment. I'll return to this when I wake up in the morning but it's been a long day and I need some sleep.

These latest commits take care of some little things here and there. I'm still in the process of testing the functionality to make sure I didn't break anything but I figured I would push this for review in the interim time.
The remaining functional issue that I believe should be addressed before merging is the behavior of brew livecheck --cask and brew livecheck --formula. Without other arguments that determine which formulae/casks to check, brew livecheck checks the watchlist file. I would expect the --cask flag to make livecheck only check the casks in the watchlist and --formula to make livecheck only check the formulae in the watchlist.
However, livecheck doesn't distinguish between casks and formulae in the watchlist file. Instead --cask treats all the names as casks and --formula treats all the names as formulae. With a watchlist file that only contains formulae, brew livecheck --cask will give an error like Error: Cask 'activemq' is unavailable: No Cask with this name exists. and exit. With a watchlist file that only contains casks, brew livecheck --formula will give an error like Error: No available formula with the name "1password". and exit.
I'm still doing some final testing to confirm that everything's working properly but that's the main issue that needs to be addressed in the interim time. I haven't worked on resolving this issue yet and I'll be away for the next few hours. If anyone tackles this while I'm away, I'll review when I'm available again.
After we've addressed that issue and I've finished testing this (will be done by end of day EST), I'll remove the do not merge label and merge this.

However, livecheck doesn't distinguish between casks and formulae in the watchlist file

Given that livecheck only worked with formulae before, I don't think this issue is a blocker and can be resolved in a  follow-up PR.

It turned out to be easy to fix (only took me a few minutes once I was back at my computer), so I pushed a commit to fix it. brew livecheck --cask and brew livecheck --formula now work as you would expect with regard to formula/cask names in the watchlist file.
I did the customary comparison of verbose JSON output of full livecheck runs across homebrew/core, on master and on this PR branch, and everything looks good. I imagine there's still some work ahead of us to improve support of homebrew/cask in livecheck but I'm good with where this is now and we can continue this in follow-up PRs. I'll merge this once CI is green.


Error while rendering "page": template: theme/_default/single.html:10:7

Hi !
I found this error when I want to use this template :

Error while rendering "page": template: theme/_default/single.html:10:7: executing "theme/_default/single.html" at <partial "article_met...>: error calling partial: template: theme/partials/article_metadata.html:10:27: executing "theme/partials/article_metadata.html" at <$.Site.Params.date_f...>: invalid value; expected string

Note that I use blogdown rstudio project with these command lines :
blogdown::new_site()
blogdown::install_theme('gcushen/hugo-academic')

@pzhaonet I saw you used this hugo template with blogdown, do you already have same issue ?

Academic is a framework, not just a theme. You need to follow Academic's
installation guide on our website as you appear to be missing Academic's
config file.
On Mon, 11 Sep 2017, 22:24 Jérôme Buisine<EMAIL_ADDRESS>wrote:

Hi !
I found this error when I want to use this template :
Error while rendering "page": template: theme/_default/single.html:10:7:
executing "theme/_default/single.html" at <partial "article_met...>: error
calling partial: template: theme/partials/article_metadata.html:10:27:
executing "theme/partials/article_metadata.html" at
<$.Site.Params.date_f...>: invalid value; expected string
Note that I use blogdown https://github.com/rstudio/blogdown rstudio
project with these command lines :
blogdown::new_site()
blogdown::install_theme('gcushen/hugo-academic')
@pzhaonet https://github.com/pzhaonet I saw you used this hugo template
with blogdown, do you already have same issue ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/gcushen/hugo-academic/issues/270, or mute the thread
https://github.com/notifications/unsubscribe-auth/AHMD5waQrfyLlnAzhkheC8ww_IPHR2wRks5shaUDgaJpZM4PTymn
.

Is it the config.toml file ? If it is, I have this file into my project.
Below, a quick view of this configuration file :

I circumvented the problem using manually installation of the theme described in your documentation and not with blogdown theme install command line.
blogdown::new_site()
git clone https://github.com/gcushen/hugo-academic.git themes/hugo-academic
cp -av themes/academic/exampleSite/* .

And then, change into config.toml the theme name used.


Expose betas for AdamW to use with LR Scheduler

I am working on building out my training using Candle. Has been good so far but I missed some of my LR schedulers I was using. Specifically I have been building out OneCycleLR from https://arxiv.org/abs/1708.07120. PyTorch has an implementation OneCycleLR
In OneCycleLR you modify both the LR and the momentum. For AdamW we can modify the beta1 value for the momentum. Would exposing the betas be invited? I can make a PR but proposing it first.
Here is the scheduler code I have so far rockerBOO/candle_scheduler.
Thank you!

Sure, sounds like an easy and useful thing to do (probably exposing the whole params field would be fairly flexible).


Music stutters in html5

There are noticable music stutters when playing the game in HTML5.

https://github.com/Poobslag/turbofat/tree/instant-puzzle-launch-time This could maybe be addressed by adopting a change like this, to where the scenes are initialized once and then added/removed from the tree.


"DatabaseError: Can't Load Database" when push() immediately follows a database creation for non-existing file

Timing issue?
Env:
Node: v4.1.1, Windows 7
node-json-db: 0.4.6
To reproduce: run
var JsonDB = require('node-json-db');
var db = new JsonDB("myDataBase", true, false);
db.push("/test1","super test");

while the file "myDatabase.json" does not exists.
DatabaseError is thrown.
Work-around is to create myDataBase.json with empty json, then it works.
Would this be a cause?
on initial database save, mkdirp is called  which is async

When you create the JsonDB object, the lib will create the file if it doesn't exists and should log: "[JsonDB] DataBase " + self.filename + " created."
In fact, I'm quite sure the program you launch that use the library doesn't have the right to write in the current directory.
Try with a path like /tmp/myDb or any other path that you're sure your program can write to.

Might be an async bug after all. I'm getting the same error as @vihanp on both a Mac OSX and a linux (Ubuntu) box.
My node / npm install is fresh and clean.
I have full rights to install and code directories, so
installed as:
npm install node-json-db --save

I just copied and pasted the first three lines of code from the first example:
var JsonDB = require('node-json-db');
var db = new JsonDB("myDataBase", true, false);
db.push("/test1","super test");

saved as foobar.js
running node foobar.js
results in
~/0-JS-tests/node_modules/node-json-db/JsonDB.js:204

            throw error;
            ^
NestedError: Can't Load Database
    at DatabaseError.Error (native)
    at DatabaseError.NestedError (~/0-JS-tests/node_modules/node-json-db/lib/Errors.js:7:25)
    at new DatabaseError (~/0-JS-tests/node_modules/node-json-db/lib/Errors.js:29:33)
    at JsonDB.load (~/0-JS-tests/node_modules/node-json-db/JsonDB.js:202:25)
    at JsonDB._getData (~/0-JS-tests/node_modules/node-json-db/JsonDB.js:74:14)
    at JsonDB._getParentData (~/0-JS-tests/node_modules/node-json-db/JsonDB.js:60:44)
    at JsonDB.push (~/0-JS-tests/node_modules/node-json-db/JsonDB.js:142:27)
    at Object.<anonymous> (~/0-JS-tests/node-json-db/foobar.js:10:4)
    at Module._compile (module.js:434:26)
    at Object.Module._extensions..js (module.js:452:10)

commenting out the third line
// db.push("/test1","super test");
and running again, creates the DB --- and then the samples work.
I also tried to create an empty DB before running the three lines uncommented, by doing merely:
 echo > myDataBase.json
but the initial push() still failed with same error.
Besides confirming the bug, I wonder if @vihanp is on to something here - that there is some kind of async error here.  The reason is that I can get the same error on the second push (), when the first push() worked.
thanks for your efforts - this is a nice idea!

H

You are both right... Sorry @vihanp
I'm using mkdirp to create directory that are set in the path of the database, for an unknown reason I didn't use the sync version of it, which lead to this bug. I reproduced, tested and corrected it.
I'll release the new version soon. Thanks.


App crashes because no animators

When trying to cancel() or finish() an animations, sometimes it gives back an error:
fatal error: unexpectedly found nil while unwrapping an Optional value
Can we have the possibility to check if animators are nil?
Thanks.

I was with 1.0.0-alpha.3.
It seems that in 1.0.0-alpha.4 it works.
Sorry.


venv is part of python, so just use it instead of virtualenv

Pull Request Description
What issue does this change request address?
Closes #2077
What are the significant changes in functionality due to this change request?
Removes installing virtualenv since venv is in all the python versions we support.

For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.

[ ] 1. Review all computer code.
[ ] 2. If any changes occur to the input syntax, there must be an accompanying change to the user manual and xsd schema. If the input syntax change deprecates existing input files, a conversion script needs to be added (see Conversion Scripts).
[ ] 3. Make sure the Python code and commenting standards are respected (camelBack, etc.) - See on the wiki for details.
[ ] 4. Automated Tests should pass, including run_tests, pylint, manual building and xsd tests. If there are changes to Simulation.py or JobHandler.py the qsub tests must pass.
[ ] 5. If significant functionality is added, there must  be tests added to check this. Tests should cover all possible options.  Multiple short tests are preferred over one large test. If new development on the internal JobHandler parallel system is performed, a cluster test must be added setting, in  XML block, the node <internalParallel> to True.
[ ] 6. If the change modifies or adds a requirement or a requirement based test case, the Change Control Board's Chair or designee also needs to approve the change.  The requirements and the requirements test shall be in sync.
[ ] 7. The merge request must reference an issue.  If the issue is closed, the issue close checklist shall be done.
[ ] 8. If an analytic test is changed/added is the the analytic documentation updated/added?
[ ] 9. If any test used as a basis for documentation examples (currently found in raven/tests/framework/user_guide and raven/docs/workshop) have been changed, the associated documentation must be reviewed and assured the text matches the example.

Job Test mac on 0197982 : invalidated by @joshua-cogliati-inl
FAILED: Diff tests/framework/ROM/TimeSeries/ARMA/InterpolatedMaxCycles

Job Test qsubs sawtooth on 0197982 : invalidated by @joshua-cogliati-inl
FAILED: Timeout tests/framework/Optimizers/SimulatedAnnealing/ExponentialSA

PR looks good.


Signature Too big by default

Tryed ratio or velocity, nothing helped. How to change the weight of a signature ?

'maxWidth' or 'minWidth' - doesn't work.

Could you create a demo of this issue e.g. on jsfiddle, codepen etc?

Its working, sorry for mistake.
On Sun, 12 Aug 2018 at 13:31, Szymon Nowak<EMAIL_ADDRESS>wrote:

Could you create a demo of this issue e.g. on jsfiddle, codepen etc?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/szimek/signature_pad/issues/383#issuecomment-412333533,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AfArMQ7ewR4f0k_wt1cd2fZSF5nACQRuks5uQAPpgaJpZM4V5Rve
.


basyx: add server with git history.

Add the content and git history of https://github.com/rwth-iat/basyx-python-sdk-http-server-docker under the folder basyx.

After researching a bit more I want to ask, are you sure it makes sense to keep all the pyproject.toml files in the same directory directly in basyx? Would it not make more sense to aim for this kind of structure for our project:
basyx-python-sdk
|
+-- common/  <-- shared files go in this project
|   |
|   +-- src/
|   +-- pyproject.toml
|
+-- basyx/
|   |
|   +-- src/
|   +-- pyproject.toml
|
+-- server/
|   |
|   +-- src/
|   +-- pyproject.toml
|
+-- .gitignore
+-- LICENSE.txt
+-- README.md

This is the structure that I had in my mind, so yes can you adapt this, if it wasn't the case. Sorry that I didn't notice before.

Can you also add the link to the specification of Part 2 to the README.md table?

I just realized, please make sure to also update the branch to main in the Dockerfile. See https://github.com/rwth-iat/basyx-python-sdk-http-server-docker/pull/5

Moved this to a different branch


Documents with the same title appear under the same category in search results

First, thanks for creating such a great library! 😄 👍
Second, we have a site with several pages with duplicate titles nested under different parents; for example:
Topic A
  - Design
  - Implementation
Topic B
  - Design
  - Implementation

When searching, if a keyword appears in multiple "Design" documents, the search results look like this:

Notice how both documents are listed under a single "Design" category, even though they are actually part of two separate documents.
It would be great if the category could include the parent titles for nested documents. For example, I'd love to see the above be displayed as:

I'm running into this same problem. I thought of a potential workaround that unfortunately still doesn't work: using front matter to specify page titles. Something like:
---
sidebar_position: 1
sidebar_label: Overview
title: First Topic Overview
---

# Overview

Unfortunately, it seems like this plugin uses the h1 as the sole source of information for the page title, preventing this approach from working.
If the maintainers here are interested in addressing this issue, perhaps the page title searching logic could change to use the front matter title property first, then the h1 if the property isn't defined.
Another approach I've thought of would be to add an option to this plugin to generate page titles with full breadcrumbs, but this seems maybe a bit wonkier vs. using front matter.


Propose a way to transition from the message type prefix of did:sov... to https://...

Please review - @TelegramSam @dhh1128 @troyronda.  Would this work as a transition plan for this change?
I noticed in looking at #225 that we really haven't settled on the precise name of the new prefix. I assume we have to settle on that before we proceed.  From my perspective, I'm OK with a change, but I would REALLY like to see the link be real, and to the spec.
In making this transition, I'm proposing that we would NOT update the RFCs to reference the new prefix until we move to Step 3 of the transition plan - e.g. when we start to change to defaulting to the new prefix.
Sam, I did not put in the line about "In step 1 you can send out a new message type if you receive one" because that directly contradicts the guidance that you "MUST only send out the old message type". How would one ever receive a new message if no one is allowed to send one out?
Also, Sam, note the new "Step 0" note that I put in about getting any RFC ducks in a row before beginning Step 1, and the use of the status values to track the steps - "Accepted", "Adopted" and "Retired" status updates trigger the start of Steps 1, 2 and 3, respectively.

I don't think we need to retain ";spec" in the new form. It would just be "https://didcomm.org/protocol/1.0". The ";spec" part was only necessary when we were planning to do DID resolution and needed an identifier "spec" for the service endpoint.

I don't think we need to retain ";spec" in the new form. It would just be "https://didcomm.org/protocol/1.0". The ";spec" part was only necessary when we were planning to do DID resolution and needed an identifier "spec" for the service endpoint.

Fixed


Reacted to descriptor changes in product code.

@ajaybhargavb and I are getting PRs out now for the descriptor changes so they can be looked in consumable pieces over as we finish the test reaction work.

Added a new RazorDiagnosticFactory abstraction to handle RazorDiagnostics and their corresponding errors/ids etc. This new API should allow for easy addition of new RazorDiagnostic errors.
Updated the DefaultTagHelperDescriptorFactory to construct TagHelperDescriptors using the new builder APIs and in the new descriptor format (1 descriptor per type).
Updated ViewComponentTagHelperDescriptorFactory to construct TagHelperDescriptors with the builder API.
With both factory implementations code was duplicated because the ViewComponent work will be moving outside of Razor once we have the proper hooks.
Updated TagHelper binding bits to capture a binding result in order to query which rules appy to a given tag name.
Still need to handle deserialization of TagHelperDescriptor types within the Razor extension.

#973

🆙 📅


Feature: Add deactivate as action to healtcheck failure

Use Case
We have services that interact with hardware, unfortunately the hardware can sometimes become unavailable or not work properly resulting in many many different types of internal errors. Rancher is very good at ensuring the containers continue to run, which isn't always desired.
Desired
If a service's health check becomes unheathly, allow for an option to deactivate (stop) the service vs re-create or do-nothing.

Any thoughts on this?

With the release of Rancher 2.0, development on v1.6 is only limited to critical bug fixes and security patches.


[5.1 Early] [Frontend] Default the prebuilt cache to //prebuilt-modules

By default, prebuilt modules will live in
//prebuilt-modules, so make this explicit if it wasn't
passed in explicitly.
rdar://49631050
(Cherry-pick of #24295 to 5.1-early)
Original patch was reviewed by @jrose-apple.

@swift-ci please test


implement sorted (on interfaces)

https://docs.python.org/2/library/functions.html#sorted

Can this be implemented using generics? the underlying building of generics is an interface{}. If not, then I can submit a PR on interfaces, that support []int, []float64, []string

I’d love it if you submitted a PR
Sent via Superhuman iOS @.***>
On Sun, Sep 17 2023 at 6:21 AM, e. alvarez @.***>
wrote:

Can this be implemented using generics? the underlying building of
generics is an interface{}. If not, then I can submit a PR on interfaces,
that support []int, []float64, []string
—
Reply to this email directly, view it on GitHub
https://github.com/seiflotfy/do/issues/10#issuecomment-1722387864, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAA4YTINF7QDF7RGMJ5IV2LX2Z3C3ANCNFSM4BF5YTHA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Migrate Elevator Plugin

Since the list of missing plugins is still long, I thought I would take on migrating the Elevator plugin. Are there any new requirements for it?

Thanks!
One difference I can think of is that now we can use an ignition::msgs::Int32 message instead of using a string message and converting to an int.
Also, you can probably leverage the JointPositionController system to do the PID control.

I started by creating a new, more feature-rich, model.
I configured all non-moving links as kinematic. It behaves as expected in Gazebo 9, but not in Ignition Dome.

The model can be found here.
@chapulina is this a bug, or is the kinematic property not implemented yet?

is this a bug, or is the kinematic property not implemented yet?

The kinematic flag isn't supported yet, I ticketed an issue: https://github.com/ignitionrobotics/ign-gazebo/issues/437


"'mat-form-field' is not a known element" even though MatFormFieldModule is being imported

Bug, feature request, or proposal:
Bug
What is the expected behavior?
I should be seeing the Material-styled <input> element.
What is the current behavior?
I'm seeing the following error message in the browser console:

'mat-form-field' is not a known element:

What are the steps to reproduce?
(I seem to be unable to reproduce this bug on StackBlitz – possibly because it uses Angular 6?)

Create a new Angular project using ng new
Change app.module.ts to:

import { BrowserModule } from '@angular/platform-browser';
import { NgModule } from '@angular/core';
import {
  MatFormFieldModule,
  MatInputModule,
} from '@angular/material';

import { AppComponent } from './app.component';

@NgModule({
  declarations: [
    AppComponent
  ],
  imports: [
    BrowserModule,
    // Angular Material modules (make sure to import after BrowserModule)
    MatInputModule,
    MatFormFieldModule,
  ],
  providers: [],
  bootstrap: [AppComponent]
})
export class AppModule { }

Change app.component.html to:

<mat-form-field><input matInput placeholder="Foo"></mat-form-field>

Which versions of Angular, Material, OS, TypeScript, browsers are affected?
$ ng --version
[ --- Angular CLI ASCII logo --- ]
Angular CLI: 1.7.4
Node: 9.8.0
OS: linux x64
Angular: 5.2.10
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router

@angular/cdk: 5.2.5
@angular/cli: 1.7.4
@angular/material: 5.2.5
@angular-devkit/build-optimizer: 0.3.2
@angular-devkit/core: 0.3.2
@angular-devkit/schematics: 0.3.2
@ngtools/json-schema: 1.2.0
@ngtools/webpack: 1.10.2
@schematics/angular: 0.3.2
@schematics/package-update: 0.3.2
typescript: 2.5.3
webpack: 3.11.0

Is there anything else we should know?
Other Material components / modules work fine: For instance, if I import the MatButtonModule and use <button mat-button> in app.component.html, I'll see a Material-styled button. As far as I can see, only <mat-form-field> is causing the above issues.

I have to apologize, it looks like the error was caused by the fact that the template of some of my other components still contained a  <mat-form-field> tag and the module containing this component didn't import the Material modules, while I was doing my tests in my app.component.html and was wondering what the heck was going on. I should have read the error message more carefully.


Break out project priorities by category

Categories:

Expansion
New feature
Processes we are supporting

Does this do a good job of capturing what we discussed @erose @kevinrobinson?

@alexsoble looks great to me 👍


envtest: Stick to a specific setup-envtest version

Using @latest for tools dependecies produces unrepeatable builds which might break at any point in time.
This is particularly important when maintaining released versions which need a backport fix.
Ref:

https://github.com/openshift/sriov-network-operator/pull/849

Note: At the moment is not possible to reference the package
sigs.k8s.io/controller-runtime/tools/setup-envtest with a specific
version:
go<EMAIL_ADDRESS>module<EMAIL_ADDRESS>found, but does not contain package sigs.k8s.io/controller-runtime/tools/setup-envtest

See

https://github.com/kubernetes-sigs/kubebuilder/issues/2480

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
We've detected an issue with your CI configuration that might affect the accuracy of this pull request's coverage report.
To ensure accuracy in future PRs, please see these guidelines.
A quick fix for this PR: rebase it; your next report should be accurate.

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 25.583%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2348

Relevant Lines:
9178

💛  - Coveralls

LGTM


docs: Move react-email to dev dependencies in install guides

react-email can be installed as a dev dependency, instead of a dependency that we need to ship at runtime, thus saving bundle space. Adjust the documentation for all package managers to reflect this.
This has been mentioned initially in this issue thread: https://github.com/resend/react-email/issues/1101
Side by side comparison

@gabrielmfern Sorry for the delay, I've changed all the dev flags to their shorthand version.
I would keep the -D and -E flags separate instead of chaining them -DE, since it seems rerasier to read and research those flags in case someone is not aware what they do.
Also, do we need to generate a chengeste for this small change?

Thanks for making the changes!

Also, do we need to generate a changeset for this small change?

Nope, changeset is only meant for when changing something in a package that would require a subsequent versioning change.


User-reported unhelpful error message

Program: https://code.world/#PmwSAJwDIIAkufkejuKi8uA
Message:
program.hs:29:24-26: error:
    • Couldn't match expected type Shiptype with actual type Number

Seems about as helpful as can be expected to me.


Add sysMultiHostCpuUsageRatio to BIG-IP profile

What does this PR do?
Add sysMultiHostCpuUsageRatio to BIG-IP profile.
Motivation
I think Getting CPU usage metric is common use case,
so adding sysMultiHostCpuUsageRatio to the example would be useful.
Additional Notes
https://oidref.com/<IP_ADDRESS>.4.1.33<IP_ADDRESS>.<IP_ADDRESS>
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

@gmarabout @FlorianVeaux
Thank you for your review!
Fixed https://github.com/DataDog/integrations-core/pull/10924/commits/3e6767bcf7495cb8396bea3510acf5dfc2ce290a

Removing  [snmp] from the title since we usually rely on label integration/snmp instead.

@AlexandreYang
I appreciate your kind support.
Could you merge this pr instead of me because I can't merge it?
If you have to add something, I wait for it.

/azp run PR Changes


Changes to the example code

https://livenorthumbriaac-my.sharepoint.com/:u:/g/personal/w23019992_northumbria_ac_uk/ETFYOlJQth5GptN2RyiBqFEBsVxNOCkswlQagb7h9uFCbQ?e=Jhk3ua
I've made the x^2 so it is actually x^2 and for partial differential, changed the input function to f_xy (instead of f_x) since it is a function of two variables.

Thank you so much for spotting these two errors @connorwatson128, and then for making the corrections yourself. Excellent work 👍 . I will incorporate now.

Hello @lucydot, yes, I agree this solves it. Thanks.


The text for the internal link in the page list appears below the internal link icon

Q
A

Bug?
yes

New Feature?
no

Sulu Version
1.3.0

Browser Version
The Bug appears in all browser except in Edge, even though the behaviour in Firefox is slightly different

Actual Behavior
How does Sulu behave at the moment?
The elements in the page list which represent internal links are displayed as follows:
The Text for the internal link is displayed below the internal link icon. If the internal links were hovered the then the text and the icon appear at the same height. After the hover the text of the internal link appears again below the internal link icon. In Firefox the internal link element is displayed correct after the hover.
Steps to Reproduce

Open a page list with internal links
Hover the internal links

I can't reproduce that issue, but maybe I simply don't know what to look out for... Can you maybe post a screenshot?

Ok, I have checked the internal links content type. But I also can't reproduce the issue you are showing here. Have tested it with Sulu 1.3.0 and 1.4.1 on Mac OSX with Chrome, Firefox and Safari and with IE11 on Windows 8.1.
Closing the issue as non-reproducable, please add more information if available.


Some devices gives crashes after showing interstatial ads

Hi
Some devices are giving crash after showing interstatial or after closing the ad.
Usually android version 4.2
I have 20 crash reports same as this and they are all 4.2
java.lang.Error: FATAL EXCEPTION [UnityMain]
Unity version : 5.0.0f4
Device model : Digma Optima 10.2 3G TT1040MG
Device fingerprint: alps/mr900/mr900:4.2.2/JDQ39/1403577075:user/release-keys
Caused by: java.lang.Error: signal 11 (SIGSEGV), code 1 (SEGV_MAPERR), fault addr 00000020
Build fingerprint: 'alps/mr900/mr900:4.2.2/JDQ39/1403577075:user/release-keys'
Revision: '0'
pid: 24534, tid: 24550, name: UnityMain >>> com.xxx.xxx<<<
r0 00000000 r1 00000000 r2 ffffffff r3 600159f8
r4 00000000 r5 6015e7a0 r6 00000000 r7 00000000
r8 00000001 r9 63be9001 sl 00000002 fp 00000001
ip 00000002 sp 5f1499d8 lr 405df705 pc 405e1636 cpsr 3a353730
at libEGL.00001636(Native Method)
at Unknown.00003009(Unknown Source)

What devices are you seeing this crash on and what version of the Unity plugin are they using? Are you able to replicate this crash? If so would you be able to provide logs?

Ram

Unfortunately, i don't have one of these devices. Here the list of devices.

Using an Alcatel One Touch on Android 4.2.2 I was unable to replicate this issue. Without access to any of the other listed devices or more information, I won't be able to troubleshoot any further.

Ram


Print values using the Python API

Hello developers. Firstly, congratulations and thank you very much for this wonderful project. I have just started exploring this for a personal project of mine. My question is weirdly the same as #12712 with a small difference. I want to be able to do the same thing i.e. print values xmin(), ymin(), xmax() , ymax() defined under class BBox2D( ) using the Python API. Any pointers will be appreciated. Thank you very much once again.

python protobuf api default implements the __str__ method,  the print function/statement will give you a human-readable representation of the message.
ref

Closed due to inactivity. If the problem persists, pls feel free to reopen it or create a new one and refer to it.


⚠ Bofrost.nl has degraded performance

In 1375ab4, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 6029 ms

Resolved: Bofrost.nl performance has improved in 87993cd after 6 minutes.


Verify CAI export file exists in GCS

Story
When an inventory is created with CAI enabled, Forseti saves the CAI export to GCS.
Proposed Solution
Use the output from the forseti inventory create command to check if the file exists in the GCS bucket. The file name will be gs://forseti-cai-export-{{SUFFIX}}/organizations/{{ORG_ID}}/resource/{{INVENTORY_ID}}.dump.
Acceptance Criteria
GIVEN: CAI is enabled
WHEN: Inventory is created
THEN: Test verifies the dump file was saved to GCS

Started this branch: https://github.com/forseti-security/forseti-security/tree/feature/issues-3347

Git Branch:
https://github.com/forseti-security/forseti-security/tree/feature/issues-3347

@gkowalski-google Unable to confirm passing test until without test harness.

@gkowalski-google Created PR here: https://github.com/forseti-security/forseti-security/pull/3407


Adds support for swiftmailer >= v6.0

Starting with version 6.0 Swiftmailer removed newInstance() methods everywhere.
This change reflects this change to support Swiftmailer >= v6.

Thanks a lot for the quick correction! Looks good for me.
Congrats on your first contribution :)


Options not working

I have tried calling panZoom with some of the options described in the README, and they are not working.
I have looked in the code and they are not even being evaluated...
The only options that are checked are zoomFactor, zoomMin and zoomMax.
Did you forget to add the evaluation of the remaining options?
Thanks

Ok I downloaded a version that apparently is not the current one.
I installed svg.panzoom.js instead of @svgdotjs/svg.panzoom.js ...
My bad. Will also close my other issue.


updates to more correctly obey the style guide

Ref issue: #7030

This change is

/assign @steveperry-53
/assign @Bradamant3

@steveperry-53 does this do what you’re after in terms of more correctly obeying the style guide?

@tengqm thanks, good catch - definitely more correct. Updated

/lgtm

@heckj Can you reopen this? I don't understand why it got merged. Thanks.

I know this is a weird one, but I think "If there are more than one node in your cluster" should be "If there is more than one node in your cluster".

each of the nodes are connected -> the nodes are connected

Regardless of how many nodes, all Kubernetes clusters will generally have -> Regardless of the number of nodes, all Kubernetes clusters generally have

a Kubernetes cluster is by setting them up -> a Kubernetes cluster is by setting it up
Alternative: setting one up

Are you sure that the Katacoda Playground uses Minikube?

to play with in on the web -> to play with on the web

"Be familiar with how to Organizing Cluster Access" doesn't work. Maybe this: Be familiar with Organizing Cluster Access by using configuration files.

Need a period after, "these resources at Understanding Kubernetes Objects".

Our style guide has a section named Use simple and direct language. One of the examples we give is to say "To" instead of "In order to". Personally, I would like to see another example added to this section in the guide: Use "in" instead of "within". Let's see what the others think.
So I would prefer to see "in the Kubernetes documentation" rather than "within" the Kubernetes documentation. I would prefer "in the cluster" rather than "within" the cluster.

This sentence needs a couple of commas.
As a cluster operator you may not need to use all these resources although you should be familiar with them to understand how the cluster is working and being used.

3 Get information about my cluster -> 3 Get information about your cluster
Why do I think it should be "your"? The rest of the page has a "you" feel to it. And the first sentence in this section starts with "You can".

access clusters using the cluster API -> access clusters using the Kubernetes API

These two sentences don't work for me: "You can access clusters using the cluster API. If you are not already familiar, run through the tutorial to view your cluster’s Pods and Nodes."
If you are not already familiar with what? The Kubernetes API? I don't see how I can help my unfamiliarity with the Kubernetes API by doing this particular tutorial.

There are three instances of "kubernetes". Need upper-case K.

typically discuss simpler stateless applications -> typically discuss simple stateless applications

As you move into fuller dev, testing, and production environments"
The word "fuller" sounds strange to me. Let's check with the others.

These links should have title casing:
Managing Compute Resources for Containers
Configuration Best Practices

Use sentence casing, not title casing for this heading: Work with ingress, networking, storage, workloads
Possibly capitalize Ingress because it's an API resource.

Remove "TODO: identify major items".

Don't say "currently".
Role-based access control (RBAC) is currently the recommended mechanism

Comma after structure.
Kubernetes provides some basic logging structure and you may want to use additional tools to help aggregate and analyze log data.

There is an overview at of tools at -> There is an overview of tools at

several options to installing Prometheus -> several options for installing Prometheus

and CoreOS provides a prometheus operator and kube-prometheus which adds on ->
and CoreOS provides prometheus-operator and kube-prometheus, which adds on

Capitalize Minikube.

This sentence has gotten off track: A common configuration on minikube and some kubernetes clusters used Heapster along with InfluxDB and Grafana.
Maybe this? A common configuration on Minikube and some Kubernetes clusters is to use Heapster along with InfluxDB and Grafana.

It got merged because @tengqm put an /lgtm On it, and by default approvers come in with “approved” already in place - so it had the tags it was looking for, and nothing else was stopping it (such as a hold tag).
I love these edits, thank you - I’ll make another PR with the content of them as updates. It would have been fantastic if you could have provided them while it was still on a feature branch and prior to our merging of this content, but I’ll take it!

opened a full-on PR #7217 to address the additional feedback. Thanks Steve! Assigned you as reviewer and put a hold on the PR so that it won't merge prior to your full review


Remove merchantId from defaultGooglePay

merchantId is not specified in googles docs https://developers.google.com/pay/api/android/reference/request-objects#MerchantInfo

Better late than never 🤣


Docker vs Ephemeral Stacks

Possible Title: The case against docker?

Ephemeral Stacks vs Localstack & Docker, Pros and Cons
Show example for review apps for apis (so far we only have an article for frontend review apps)
Review Apps in AWS can easily be forwarded to PMs and commented
Peace of mind, simplicity: Ephemeral stacks are easy
Cons

Docker: relative maintenance effort

https://theburningmonk.com/2019/09/why-you-should-use-temporary-stacks-when-you-do-serverless/
https://theburningmonk.com/2023/02/how-to-handle-serverful-resources-when-using-ephemeral-environments/
https://theburningmonk.com/2022/05/my-testing-strategy-for-serverless-applications


[Automated] Update eventing-istio-eventing nightly (#284)

Update eventing nightly
Signed-off-by: Knative Automation<EMAIL_ADDRESS>

OCF Webhook is merging this PR


Proposal to rename ChoiceChip's onSelected method to be onTap / onChanged / onSelectionChanged

The ChoiceChip has onSelected parameter which takes a closure which is called every time the ChoiceChip is tapped! If it is to be called every time the user taps on the chip then why not rename it to something like 'onTap' or 'onChanged' or 'onSelectionChanged'?
Steps to Reproduce
Expected results:
ChoiceChip(
          onChanged: (value) {

          },
          label: Text(service),
          selected: true,
        );

Actual results:
ChoiceChip(
          onSelected: (value) {

          },
          label: Text(service),
          selected: true,
        );

While I agree that the naming could be improved, I don't think it is worth a breaking change. This would be one of those that we whish were different in Flutter.


Get credentials

Update:
@shenxn: https://github.com/shenxn/ha-dyson/issues/5#issuecomment-791887983

I've added 2FA support to libdyson (https://github.com/shenxn/libdyson). Download or clone the repo, then run python3 get_devices.py to get credentials. You may also need to install some dependencies using pip3 install -r requirements.txt.

Old
Not an actual issue, but wanted to make this available for other people
I've managed to get credentials in two ways:
1. Using libpurecool
Use this fork of libpurecool (fix_auth branch): https://github.com/bfayers/libpurecool/tree/fix_auth
python3 -m pip install https://github.com/bfayers/libpurecool/archive/fix_auth.zip

Create a new file called whatever.py
Paste this in:

    #!/usr/bin/python3
    from libpurecool.dyson import DysonAccount

    dyson_account = DysonAccount("email","password","lang")
    logged = dyson_account.login()

    if not logged:
        print('Unable to login to Dyson account')
        exit(1)
    devices = dyson_account.devices()
    for device in devices:
        print(device.serial);
        print(device.name);
        print(device.credentials);
        print(" ");

Open your DysonLink app and stare at the welcome screen :)
python3 whatever.py
Cross your fingers 🤞 and hope it works.

⚠ Don't try to loging in too fast/spam the script or you'll get IP blocked by Dyson
I've also noticed that I'm getting immediately denied login if I'm going through a VPN

2. Using an Android phone
You can probably do this with an iOS device, but I haven't tested (https://stackoverflow.com/a/8512455)

Download the Packet Capture app from Google Playstore (https://play.google.com/store/apps/details?id=app.greyshirts.sslcapture&hl=en&gl=US)
Open Dyson Link and leave it running in the background
Run the Packet Capture  app, allow it to create a VPN if you're prompted
Hit the play button with the 1 and choose the Dyson Link app
Switch to the Dyson Link app, it will probably do a quick refresh of the welcome page
Switch back to Packet Capture and stop capturing
Look through the captured packets for something like this:

Thx! But I am still stuck on getting the credentials. I am running Home Assistant on a PI4, and have no idea how to get them. Any suggestions? :)

@ripburger you need to run that python code snippet (quoted above) after installing the libpurecool library from bfayers (linked above).
https://adamj.eu/tech/2019/03/11/pip-install-from-a-git-repository/

@proatwork I keep getting Unable to login to Dyson account. I can use the email and password I supplied to sign in on the website. I can also curl https://appapi.cp.dyson.com/v1/userregistration/userstatus and see that my account is active with the US country.
Any tips?

Hey @soffes
Are you using the libpurecool fork above?
The credentials should be the ones you use for the Dyson mobile app
If I have some free time tomorrow I will write a step by step of how I managed to make it work

@proatwork I tried this:
$ python3 -m pip install https://github.com/bfayers/libpurecool/archive/fix_auth.zip
$ python3 dyson.py

dyson.py is your script from above. From reading the integration documentation it says US is a known working language. From reading the libpurecool code it actually passes the "language" as a country in the URL so I think US is correct.
I verified that the credentials I used work in the iOS app and can control my fan.
Any help would be awesome. Super appreciate your time on this! :)

@soffes
pi@raspberrypi:[~]$ python3.7 -m pip install https://github.com/bfayers/libpurecool/archive/fix_auth.zip

Looking in indexes: https://pypi.org/simple, https://www.piwheels.org/simple
Collecting https://github.com/bfayers/libpurecool/archive/fix_auth.zip
  Using cached https://github.com/bfayers/libpurecool/archive/fix_auth.zip
Requirement already satisfied: netifaces in /home/pi/.local/lib/python3.7/site-packages (from libpurecool==0.6.4) (0.10.9)
Requirement already satisfied: paho_mqtt in /home/pi/.local/lib/python3.7/site-packages (from libpurecool==0.6.4) (1.5.1)
Requirement already satisfied: pycryptodome in /home/pi/.local/lib/python3.7/site-packages (from libpurecool==0.6.4) (3.10.1)
Requirement already satisfied: requests<3,>=2 in /usr/lib/python3/dist-packages (from libpurecool==0.6.4) (2.21.0)
Requirement already satisfied: six in /usr/lib/python3/dist-packages (from libpurecool==0.6.4) (1.12.0)
Building wheels for collected packages: libpurecool
  Running setup.py bdist_wheel for libpurecool ... done
  Stored in directory: /tmp/pip-ephem-wheel-cache-gm2k7gt0/wheels/7f/8a/a5/8fa837fd233e6ab9b9c1fa2d26046d8db278c5279547827ba9
Successfully built libpurecool
Installing collected packages: libpurecool
Successfully installed libpurecool-0.6.4

pi@raspberrypi:[~]$ python3.7 ~/test.py

yZZpyItgEttmmWqzUxY5HzC+y66j4K51u/O****Kpt5n6DrbrDDBtilqbImu7BcUJNPyBBy6ZIxqd+TTZsI3w==

@soffes
pi@raspberrypi:[~]$ python3.7 -m pip install https://github.com/bfayers/libpurecool/archive/fix_auth.zip

Looking in indexes: https://pypi.org/simple, https://www.piwheels.org/simple
Collecting https://github.com/bfayers/libpurecool/archive/fix_auth.zip
  Using cached https://github.com/bfayers/libpurecool/archive/fix_auth.zip
Requirement already satisfied: netifaces in /home/pi/.local/lib/python3.7/site-packages (from libpurecool==0.6.4) (0.10.9)
Requirement already satisfied: paho_mqtt in /home/pi/.local/lib/python3.7/site-packages (from libpurecool==0.6.4) (1.5.1)
Requirement already satisfied: pycryptodome in /home/pi/.local/lib/python3.7/site-packages (from libpurecool==0.6.4) (3.10.1)
Requirement already satisfied: requests<3,>=2 in /usr/lib/python3/dist-packages (from libpurecool==0.6.4) (2.21.0)
Requirement already satisfied: six in /usr/lib/python3/dist-packages (from libpurecool==0.6.4) (1.12.0)
Building wheels for collected packages: libpurecool
  Running setup.py bdist_wheel for libpurecool ... done
  Stored in directory: /tmp/pip-ephem-wheel-cache-gm2k7gt0/wheels/7f/8a/a5/8fa837fd233e6ab9b9c1fa2d26046d8db278c5279547827ba9
Successfully built libpurecool
Installing collected packages: libpurecool
Successfully installed libpurecool-0.6.4

pi@raspberrypi:[~]$ python3.7 ~/test.py

yZZpyItgEttmmWqzUxY5HzC+y66j4K51u/O****Kpt5n6DrbrDDBtilqbImu7BcUJNPyBBy6ZIxqd+TTZsI3w==

Hello, I have the same problem,
I reproduced what you described ☝  and I get "Unable to login to Dyson account".
I still can logout and login to the mobile app with those credentials. I also tried  with a  new account without success.
Are there any other step to take into account ?
Thank you.

@krazar
I created my account through https://www.dyson.com/your-dyson, if that makes any difference 🤷

That's really strange.
I can connect with my account on the dyson website (from my region)
And, to be sure, I uninstalled libpurecool and reinstall the version from bfayers from the snippet above.
Nothing changes.

Same here @krazar. The bfayers fork was working for me until the most recent HA update :/

@krazar @soffes
I just re-tested the script again and got
Unable to login to Dyson account
I opened the Dyson app on my phone, and retried the script and it provided me with the credentials...
See if that makes it work for you 🤷
pi@raspberrypi:[~]$ python3.7 ~/test.py
Unable to login to Dyson account
[ opened the Dyson app / Android ]
pi@raspberrypi:[~]$ python3.7 ~/test.py
yZZpyItgEttmmWqzUxY5HzC+y66j4K51u/OBgva7Kpt5****rDDBtilqbImu7BcUJNPyBBy6ZIxqd+TTZsI3w==

Not able to get credentials here, either - I finally managed to get get_devices.py to run :-)
I have tried the mobile login/logout process already.
bash-5.0# python get_devices.py
Please choose your account region
1: Mainland China
2: Rest of the World
Region [1/2]: 2
Region code: US
Email<EMAIL_ADDRESS>Password:
Traceback (most recent call last):
  File "/config/custom_components/libdyson-main/libdyson/cloud/account.py", line 127, in login_email_password
    response = self.request(
  File "/config/custom_components/libdyson-main/libdyson/cloud/account.py", line 109, in request
    raise DysonInvalidAuth
libdyson.exceptions.DysonInvalidAuth

During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "get_devices.py", line 23, in
account.login_email_password(email, password, region)
File "/config/custom_components/libdyson-main/libdyson/cloud/account.py", line 138, in login_email_password
raise DysonLoginFailure
libdyson.exceptions.DysonLoginFailure

ok, i'm new to this don't shoot me, but when i enter into the bash session inside my home assistant session, i've tried to execute the bfayers python code, except i get the following:
bash-5.0# python3 dyson.py
Traceback (most recent call last):
  File "dyson.py", line 12, in <module>
    from .dyson_pure_cool import DysonPureCool
ImportError: attempted relative import with no known parent package

is the path for the package correct:
/usr/local/lib/python3.8/site-packages/libpurecool
I think I am missing something here to get the credential info, any help is appreciated!

@kintaroju I managed to get get_devices.py from https://github.com/shenxn/libdyson/ to run.
Still having a problem logging in, but that's a different problem.

@eiddor i am guessing it's related to the new 2FA issue? lol

@eiddor i am guessing it's related to the new 2FA issue? lol

Yeah, that's what I suspect.

@eiddor totally hear you. I think if the developer can somehow trigger the 2FA email and integrate the 2FA value I think we can be in the clear.

@eiddor totally hear you. I think if the developer can somehow trigger the 2FA email and integrate the 2FA value I think we can be in the clear.

That may work ok for Dyson Local and grabbing the credentials once, but I don't know how well it would scale for the native HA Dyson implementation or @shenxn's Dyson Cloud.
I'm not a dev, so maybe I'm missing something - I think it's more a matter of Dyson catching their API up with their mobile app.

@eiddor , you are right, but let's see what developer can do.
if the developer can figure this out, big thank you in advance!

I manually copied the github folder and when I go into the libepurcool folder and run dyson.py I get this.
C:\Python\libpurecool\libpurecool>python dyson.py
Traceback (most recent call last):
File "dyson.py", line 12, in
from .dyson_pure_cool import DysonPureCool
ImportError: attempted relative import with no known parent package

@monsieurlatte you're not supposed to run that :)
create a new .py file and paste the snippet from the first comment then run that

Ah gotcha, well I made that file and put it in the libcoolpure folder I downloaded from the fix link and when I run I get this.
C:\Python\libpurecool-fix_auth>python jrdyson.py
Traceback (most recent call last):
File "jrdyson.py", line 1, in
from libpurecool.dyson import DysonAccount
File "C:\Python\libpurecool-fix_auth\libpurecool\dyson.py", line 12, in
from .dyson_pure_cool import DysonPureCool
File "C:\Python\libpurecool-fix_auth\libpurecool\dyson_pure_cool.py", line 8, in
from .dyson_pure_cool_link import DysonPureCoolLink
File "C:\Python\libpurecool-fix_auth\libpurecool\dyson_pure_cool_link.py", line 13, in
import paho.mqtt.client as mqtt
ModuleNotFoundError: No module named 'paho'

Thanks, that helps a ton, figured it out, now I just get the same as everyone else with the whole unable to log into dyson account.

I also managed to get the credentials by using my android phone.
https://play.google.com/store/apps/details?id=app.greyshirts.sslcapture&hl=en&gl=US
I used this to monitor the Dyson app for a couple of seconds (by pressing the triangle icon with the 1, and selecting Dyson Link), and lo and behold, the credentials.

I also managed to get the credentials by using my android phone.
https://play.google.com/store/apps/details?id=app.greyshirts.sslcapture&hl=en&gl=US
I used this to monitor the Dyson app for a couple of seconds (by pressing the triangle icon with the 1, and selecting Dyson Link), and lo and behold, the credentials.
The serial and credentials are separated by an X, at least in my case

that's awesome, dont see anything for iOS in my first look through the app store.

Haven't tested, but maybe you can make it work with Bluestacks

@monsieurlatte this might help https://stackoverflow.com/a/8512455

I'll work on the new 2FA over the weekend. Hopefully that will fix the problem.

@monsieurlatte this might help if you've used Wireshark before https://stackoverflow.com/a/8512455
Thanks, I'll look into that!

So here's something, I updated the App on my phone, opened the dyson link app on the phone made sure it connected and all that. Then I just added in the HACS Dyson Cloud integration and now it works? Seems like you need the new Dyson app to trigger something to allow it to work.

Confirmed! Just opening the app and then rerunning the script with no changes fixed it for me! Thanks so much ❤

I've added 2FA support to libdyson (https://github.com/shenxn/libdyson). Download or clone the repo, then run python3 get_devices.py to get credentials. You may also need to install some dependencies using pip3 install -r requirements.txt.

Now, I can get the credentials.
Thank you !

Just tried your libdyson and it works like hot cakes. Do you plan to integrate it with your Dyson local implementation @shenxn to simplify the process?

@kintaroju Use ha-dyson-cloud (https://github.com/shenxn/ha-dyson-cloud). Set up Dyson Cloud with your account first, then all your devices will be shown as discovered entities.

hey, nice managed to get my dysons fans connected using the packet readrer on my Android phone. Question, for some, I have:
Dyson Purifier Hot+Cool Formaldehyde HP09
Dyson Pure Hot+Cool™ HP04

What is the difference between "Dyson Pure Hot+Cool (new)" and "Dyson Pure Hot+Cool"?
Also, I manged to add the HP09 as a "Dyson Pure Hot+Cool (new)" seems to work...but definitly not seeing the formaldehyde numbers nor the catalyst life for the formaldehyde..I guess a future update?

managed to get my dysons fans connected using the packet readrer on my Android phone

How did you do this?  I have been unable to get credentials using libdyson after months of trying.
I have credentials for Homebridge, but there doesn't seem to be a way to convert them to work in Homeassistant.

managed to get my dysons fans connected using the packet readrer on my Android phone

How did you do this? I have been unable to get credentials using libdyson after months of trying.
I have credentials for Homebridge, but there doesn't seem to be a way to convert them to work in Homeassistant.

Scroll up to the instructions from monsierlatte, and that's what I followed. Once I got the username and credentials, sometimes I had to try a couple of times before it actually submitted successfully probably due to Wi-Fi connection to the fan.

cheers.
I am on iOS so was looking for it's equivalent, but Apple won't let you do it (security).

cheers.
I am on iOS so was looking for it's equivalent, but Apple won't let you do it (security).

Probably, You may wish to make friends with Android people LOL

You may wish to make friends with Android people
Ha, yes! 👍

2. Using an Android phone
You can probably do this with an iOS device, but I haven't tested (https://stackoverflow.com/a/8512455)

Download the Packet Capture app from Google Playstore (https://play.google.com/store/apps/details?id=app.greyshirts.sslcapture&hl=en&gl=US)
Open Dyson Link and leave it running in the background
Run the Packet Capture  app, allow it to create a VPN if you're prompted
Hit the play button with the 1 and choose the Dyson Link app
Switch to the Dyson Link app, it will probably do a quick refresh of the welcome page
Switch back to Packet Capture and stop capturing
Look through the captured packets for something like this:


Can I run the model on the custom dataset?

Hello Vukasin,
Thank you for this excellent open source project.
I wish to run inference on my own dataset using the provided weight. Is there a function to load and infer any custom dataset? Thank you for your time!

Hello @jaehongk ,
In case you already have a trained YOLOX-X model, it is very simple. You need to make only few changes.

Add a branch after line 44 in main.py.
Modify function get_detector_path_and_im_size() (it is called from main.py in line 65).
Modify function get_mot_loader() from dataset.py folder.

Note 1: all these modifications are simply adding if/elif branches and setting the right path and/or dataset name.
Note 2: after modifying the functions I mentioned, you will be able to switch between different datasets by simply specifying dataset argument in command line.
Best regards!
Vukašin

Thank you so much for your kind response!
Sincerely,
Jaehong


Add a guide on how to use the Ruby Octokit SDK

Why: Added a guide on how to use the Ruby Octokit SDK
Closes: #23844

What's being changed (if available, include any code snippets, screenshots, or gifs):
Added a file

Check off the following:

[x] I have reviewed my changes in staging, available via the View deployment link in this PR's timeline.

For content changes, you will also see an automatically generated comment with links directly to pages you've modified. The comment won't appear if your PR only edits files in the data directory.

[x] For content changes, I have completed the self-review checklist.

Thanks for the contribution @Manas-Kenge! I'll get this triaged for review. ⚡

Thank you for this PR 🚀 ✨
We pushed some changes to get the tests passing and render a preview. You can see a preview of your new article here: https://docs-29083-2e2021.preview.ghdocs.com/en/rest/guides/scripting-with-the-rest-api-and-ruby
At first glance this looks great! I'm going to request a technical review from the SDK team, then we'll have a writer review it.


chore: fix typo in DOMImplementation.js

What is it?

[ ] Feature / enhancement
[ ] Bug
[x] Docs / tests

Description
propertly -> property
Checklist:

[x] My code follows the developer guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] Added new tests to cover the fix / functionality

@zanettin Thank you!
I think you're right!


Add Support for Nested Function Use In WHERE Clause

Description
Syntax: nested( [field] | [field,path] )
Add support for use of the nested function in the WHERE clause as a predicate expression. Supports nested function on left of operator, and literal on right. When using nested function in WHERE clause, the inner hits of the query are not added to the nested query DSL. To do this you must use the nested function in conjunction with a SELECT clause nested function call. See documentation for this nested implementation HERE (WIP).
Example Queries
SELECT message.info FROM nested_objects WHERE nested(message.info, message) = 'a';
SELECT nested(message.info) FROM nested_objects WHERE nested(message.info, message) = 'a';
SELECT message.info FROM nested_objects WHERE nested(message.info) = 'a' OR nested(comment.data) = 'b' AND nested(message.dayOfWeek) = 4;
Issues Resolved
Issue: 1111
Check List

[x] New functionality includes testing.

[x] All tests pass, including unit test, integration test and doctest

[x] New functionality has been documented.

[x] New functionality has javadoc added
[x] New functionality has user manual doc added

[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.

Have you considered implementing nested similarly to relevance search functions?
In particular it's unexpected that generic classes like LuceneQuery and OpenSearchRequestBuilder needed changes.
I'd like to understand this before approving this PR.

The main reason why I haven't implemented this similarly to relevance search functions is the syntax. If we choose the syntax option nested(path, condition) we can follow a similar strategy of adding BuildintFunctionName.NESTED to luceneeQueries map and have a build() function in NestedQuery. This is due to there being only one function in the WHERE clause.
With the syntax option nested(field, field | path) Operator Literal when the code execution gets to FilterQueryBuilder.java#L100-L102 the query could look like
Func(>) with arguments (nested, literal). Therefore we can't override the build function because we are now supporting a predicate expression of nested function on left and literal on right and the build function will go to RangeQuery.doBuild after LuceneQuery.build in this example.
I believe with the alternate syntax nested(path, condition) we may need a different build function to override with functionality to visit the inner expression, but this may be a better trade-off if you want to keep code modular to their own query builders. This may be a good argument to support one syntax over the other.


Added Loading Callbacks

Description
Added support for onLoad and onLoadingComplete callback functions in the CldImage component.
Issue Ticket Number
#155
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Fix or improve the documentation
[x] This change requires a documentation update

Checklist

[x] I have followed the contributing guidelines of this project as mentioned in CONTRIBUTING.md
[x] I have created an issue ticket for this PR
[x] I have checked to ensure there aren't other open Pull Requests for the same update/change?
[x] I have performed a self-review of my own code
[x] I have run tests locally to ensure they all pass
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes needed to the documentation

hey thanks @jamesqquick!! a few things

made a quick note. would you mind also adding it to the documentation?
https://github.com/colbyfayock/next-cloudinary/blob/main/docs/pages/components/cldimage/configuration.mdx?plain=1

if you want, could be a good opportunity to add the same functionality for onError as well, after the existing logic in handleOnError

finally it doesnt look like there were any types added, which is curious. i wonder if it's because they may exist already on ImageProps from the Next.js library? just want to make sure we wont run into any TS issues

@jamesqquick given our chat on discord, that some of these props work without any additional code, is anything here still worth getting in?


Scale bar

May not be possible for some / most SEGY's ... need reliable trace position information. Unlikely we can scour trace spacing from the file header reliably.

But I do think that we could come up we some kind of average and do a decent scale bar
Things to try out:
https://gist.github.com/dmeliza/3251476
https://pypi.python.org/pypi/matplotlib-scalebar


Fix compatibility with API level 17

CellInfoWcdma is from API level 18.

I've also updated the provider to not scan the cell list every second. Instead, it only requests cell list on GSM events. This is not enough, see https://github.com/microg/android_packages_apps_UnifiedNlp/issues/23.

Hi,
I don't have a device to test this right now :-(
I've ordered a new ZTE Open C and will try this with a new ROM.

Ping?


microk8s docker image pull error

we are using microk8s on ubuntu 18.04 .
we have a private docker registry hosted on remote machine which is tls secured.
all the images are going in imagepullBackoff as images are not getting pulled from the registry
Events:
Type    Reason   Age                    From     Message

Normal  BackOff  2m22s (x344 over 80m)  kubelet  Back-off pulling image "docker-registry.com:5000/*******"
We have configured microk8s to take docker as a container runtime in below file location
--->/var/snap/microk8s/current/args/kubelet
Change --container-runtime=docker
after exploring we have added below config as mentioned but still we are seeing imagePullBackOff issue.
/var/snap/microk8s/current/args/certs.d/:5000/hosts.toml
server = "https://:5000"
[host."https://****:5000"]
capabilities = ["pull", "resolve"]
Please let us know if we need to update any other configuration.

Hi @lramachandr
Is there any specific error message you are seeing in containerd? Is there anything in the output of microk8s kubectl describe pod $yourpod or sudo journalctl -u snap.microk8s.daemon-containerd that looks suspicious?
Is the registry perhaps using a self-signed certificate? Finally, can you try pulling an image directly like this:
microk8s ctr image pull --hosts-dir /var/snap/microk8s/current/args/certs.d $yourimage

Also, which microk8s version is this? Does the registry require credentials to work?

1)we have self signed certificate

microk8s ctr image pull --hosts-dir /var/snap/microk8s/current/args/certs.d docker-registry.com:5000/nginx/nginx-ingress:1.10.1
INFO[0000] trying next host                              error="failed to do request: Head "https://docker-registry.com:5000/v2/nginx/nginx-ingress/manifests/1.10.1\": x509: certificate relies on legacy Common Name field, use SANs instead" host="docker-registry.com:5000"
INFO[0000] trying next host                              error="failed to do request: Head "https://docker-registry.com:5000/v2/nginx/nginx-ingress/manifests/1.10.1\": x509: certificate relies on legacy Common Name field, use SANs instead" host="docker-registry.com:5000"
ctr: failed to resolve reference "docker-registry.com:5000/nginx/nginx-ingress:1.10.1": failed to do request: Head "https://docker-registry.com:5000/v2/nginx/nginx-ingress/manifests/1.10.1": x509: certificate relies on legacy Common Name field, use SANs instead

Looks like containerd is having issues with the self-signed CA then. Can you test if the documentation for configuring the CA solves your issue?

yes i have followed the same steps but its not accepting CA

OK, in the issue description I now saw (sorry, formatting is a bit confusing) that you have changed the --container-runtime to docker? If so, then any containerd configs you change would be irrelevant.
Further, which MicroK8s version are you using?

vmauser@ubuntu18:~$ microk8s version
MicroK8s v1.25.3 revision 4128


[Bug] <Footer: Add some margin from bottom >

Description

Footer actually not looking properly on the website. It's feel like hiding in bottom side, So add some margin from bottom side.

Screenshots
No response
Additional information
No response

Solved


nz-carousel should resume auto play after blur

What problem does this feature solve?
Currently  nz-carousel  stops auto play after being clicked, but does not resume auto play after blur. This may be a bug, but the issue-helper cannot choose the version when reporting the bug, so I had to submit to the feature request .
This is the link (https://stackblitz.com/edit/angular-m6on4r?file=src/app/app.component.ts), steps to reproduce: click the content of nz-carousel, you will find it stops auto play Up
What does the proposed API look like?
None

Unfortuantely If you cannot reproduce it using the latest version, we will be probably short of hands to make a fix and a release. Could you reproduce it using the latest ng-zorro-antd? Thanks!

Unfortuantely If you cannot reproduce it using the latest version, we will be probably short of hands to make a fix and a release. Could you reproduce it using the latest ng-zorro-antd? Thanks!

I have provided a link to reproduce and it is the latest version. You only need to click on the view of the component to find that nzAutoPlay no longer works.
reproduce link


The stated cost in the Announce dialog can be NaN

Current Behavior
While running hostd on the zen test net, clicking the announce button presents a dialogue that says:

Announcing will cost NaN TS.

Expected Behavior
I would expect NaN to be a number--the cost of the announcement.
Steps to Reproduce

Run terminal HostD (zen network)
Open the localhost UI
Configure your settings
Click the Announce button in the upper right

Version
v1.1.2
What operating system did the problem occur on (e.g. Ubuntu 22.04, macOS 12.0, Windows 11)?
macOS Sonoma 14.5
Anything else?
Chrome Version 126.0.6478.183

Seems like this got fixed at some point


I have created a how to use binary search in a matrix

previous  issues  were resolved which were there like duplicate file issue
📥 Pull Request
Description
Added a feature in binary search and created a file and codes

And fixed the issues

Fixes #123
Type of change

[x ] Bug fix (non-breaking change which fixes an issue)
[ x] New feature (non-breaking change which adds functionality)

Checklist:

[x ] My code follows the style guidelines of this project
[x ] I have performed a self-review of my code

can u review this pls