Unclear example

Because it adds too many new components, this code just made me create my own sample to understand
  Dim i As String
  For Each i In [Enum].GetNames(GetType(flavorEnum))
      MsgBox(i)
  Next

Better would be
debug.print( [Enum].GetNames(GetType(flavorEnum))(2))

returns
"sour"
Edit by @billwagner: code formatting
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: bffd60e8-8973-d39c-887f-7685572c9610
Version Independent ID: 490bdc0e-3557-02fe-37e3-9e16eca2d4b9
Content: How to: Determine the String Associated with an Enumeration Value - Visual Basic
Content Source: docs/visual-basic/programming-guide/language-features/constants-enums/how-to-determine-the-string-associated-with-an-enumeration-value.md
Product: dotnet-visualbasic
GitHub Login: @KathleenDollard
Microsoft Alias: kdollard

Thanks for writing this @ErnestCook
I've added it to our backlog, and added the "up for grabs" label, if community members want to address this before we get to it.


Ruby 2.6.0-preview2 does not support new Psych.safe_load interface.

:wave: Hey folks! I wanted to flag an issue I noticed with #6396. It implements a new interface for Psych.safe_load that's landed in Ruby trunk, checking that the active Ruby version is at least 2.6. Unfortunately, the last 2.6 preview passes the version check but does not include the new Psych code; running Rubocop under Ruby 2.6.0-preview2 yields an error like this:
/usr/local/rvm/rubies/ruby-2.6.0-preview2/lib/ruby/2.6.0/psych.rb:313:in `safe_load': unknown keywords: whitelist_classes, whitelist_symbols, aliases, filename (ArgumentError)
	from /usr/local/rvm/gems/ruby-2.6.0-preview2@structur/gems/rubocop-0.60.0/lib/rubocop/config_loader.rb:185:in `yaml_safe_load'
	from /usr/local/rvm/gems/ruby-2.6.0-preview2@structur/gems/rubocop-0.60.0/lib/rubocop/config_loader.rb:159:in `load_yaml_configuration'
	from /usr/local/rvm/gems/ruby-2.6.0-preview2@structur/gems/rubocop-rspec-1.30.0/lib/rubocop/rspec/inject.rb:8:in `defaults!'
	from /usr/local/rvm/gems/ruby-2.6.0-preview2@structur/gems/rubocop-rspec-1.30.0/lib/rubocop-rspec.rb:24:in `<top (required)>'
	from /usr/local/rvm/rubies/ruby-2.6.0-preview2/lib/ruby/gems/2.6.0/gems/bundler-1.17.1/lib/bundler/runtime.rb:81:in `require'
	from /usr/local/rvm/rubies/ruby-2.6.0-preview2/lib/ruby/gems/2.6.0/gems/bundler-1.17.1/lib/bundler/runtime.rb:81:in `block (2 levels) in require'
	from /usr/local/rvm/rubies/ruby-2.6.0-preview2/lib/ruby/gems/2.6.0/gems/bundler-1.17.1/lib/bundler/runtime.rb:76:in `each'
	from /usr/local/rvm/rubies/ruby-2.6.0-preview2/lib/ruby/gems/2.6.0/gems/bundler-1.17.1/lib/bundler/runtime.rb:76:in `block in require'
	from /usr/local/rvm/rubies/ruby-2.6.0-preview2/lib/ruby/gems/2.6.0/gems/bundler-1.17.1/lib/bundler/runtime.rb:65:in `each'
	from /usr/local/rvm/rubies/ruby-2.6.0-preview2/lib/ruby/gems/2.6.0/gems/bundler-1.17.1/lib/bundler/runtime.rb:65:in `require'
	from /usr/local/rvm/rubies/ruby-2.6.0-preview2/lib/ruby/gems/2.6.0/gems/bundler-1.17.1/lib/bundler.rb:114:in `require'
	from /Users/daniel/Repos/Ignota/structur-api/config/application.rb:21:in `<top (required)>'
	from /usr/local/rvm/gems/ruby-2.6.0-preview2@structur/gems/spring-2.0.2/lib/spring/application.rb:92:in `require'
	from /usr/local/rvm/gems/ruby-2.6.0-preview2@structur/gems/spring-2.0.2/lib/spring/application.rb:92:in `preload'
	from /usr/local/rvm/gems/ruby-2.6.0-preview2@structur/gems/spring-2.0.2/lib/spring/application.rb:153:in `serve'
	from /usr/local/rvm/gems/ruby-2.6.0-preview2@structur/gems/spring-2.0.2/lib/spring/application.rb:141:in `block in run'
	from /usr/local/rvm/gems/ruby-2.6.0-preview2@structur/gems/spring-2.0.2/lib/spring/application.rb:135:in `loop'
	from /usr/local/rvm/gems/ruby-2.6.0-preview2@structur/gems/spring-2.0.2/lib/spring/application.rb:135:in `run'
	from /usr/local/rvm/gems/ruby-2.6.0-preview2@structur/gems/spring-2.0.2/lib/spring/application/boot.rb:19:in `<top (required)>'
	from /usr/local/rvm/rubies/ruby-2.6.0-preview2/lib/ruby/2.6.0/rubygems/core_ext/kernel_require.rb:54:in `require'
	from /usr/local/rvm/rubies/ruby-2.6.0-preview2/lib/ruby/2.6.0/rubygems/core_ext/kernel_require.rb:54:in `require'
	from -e:1:in `<main>'

I've just installed ruby-head for the time being, so this is a pretty low-pressure complaint, but I imagine it's not uncommon for folks to be using the preview rather than a trunk build. Maybe the version check could be stricter?
Thanks for looking into it, and for all your work maintaining Rubocop!

Expected behavior
Rubocop should load successfully under Ruby 2.6.0-preview2.
Actual behavior
Rubocop crashes with an error about the keyword arguments passed to Psych.safe_load.
Steps to reproduce the problem

Install Ruby 2.6.0-preview2.
Attempt to run Rubocop with a custom config file.
:boom:

RuboCop version
~ ❯❯❯ rubocop -V
0.60.0 (using Parser <IP_ADDRESS>, running on ruby 2.6.0 x86_64-darwin18)

Also seeing this for 11 of my repos after upgrading to rubocop  0.60.0 gem:
ArgumentError: unknown keywords: whitelist_classes, whitelist_symbols, aliases, filename
/Users/.../.rvm/gems/ruby-2.6.0-preview2@ruby2.6-rails4.2/gems/rubocop-0.60.0/lib/rubocop/config_loader.rb:185:in `yaml_safe_load'
/Users/.../.rvm/gems/ruby-2.6.0-preview2@ruby2.6-rails4.2/gems/rubocop-0.60.0/lib/rubocop/config_loader.rb:159:in `load_yaml_configuration'

Thanks for the feedback. I opened a PR #6413 that fixed this issue.

Neat, thanks for the speedy response!

Any idea when a new version of rubocop gem will be released? Thanks....

Note: I found that docker's official ruby image v2.6rc uses older bundler and rubygems. and caused this problem.
unknown keywords: permitted_classes, permitted_symbols
/usr/local/lib/ruby/2.6.0/psych.rb:328:in `safe_load'
/home/circleci/project/vendor/bundle/ruby/2.6.0/gems/rubocop-0.61.0/lib/rubocop/config_loader.rb:188:in `yaml_safe_load'
/home/circleci/project/vendor/bundle/ruby/2.6.0/gems/rubocop-0.61.0/lib/rubocop/config_loader.rb:161:in `load_yaml_configuration'

It may be fixed after https://github.com/docker-library/ruby/pull/251 merged


macos: Do not set IPv6 address if not found.

The default 0 address gets encoded to :: and therefore fails the empty
check, causing us to try and initalize IPv6 when its not available.

Fixes the MacOS CI failure :+1:

Fixes the MacOS CI failure :+1:

bump :+1:

bump :+1:

Hi Ian,
I am really sorry I still could not give this issue the necessary attention. Since I had some work done on it, I would like to analyse it with a little more care. Maybe I can do it this weekend.
Best regards,
Marcelo.

Hi Ian,
I am really sorry I still could not give this issue the necessary attention. Since I had some work done on it, I would like to analyse it with a little more care. Maybe I can do it this weekend.
Best regards,
Marcelo.

Hi @mrjimenez
Happy to wait for you review properly, just wanted to make sure it was not lost in the void.
Cheers!

Hi @mrjimenez
Happy to wait for you review properly, just wanted to make sure it was not lost in the void.
Cheers!

Hi Ian,
Again, sorry for the delay. I just wanted to have the details of the problem in my head before applying your patch.
I understand it addresses the first point of #193, which we both agreed.
Since only you and @G10h4ck have answered, and you have a good point about removing the configure time --enable-ipv6, maybe we can go for it now, what do you think?
Regards,
Marcelo.

Hi Ian,
Again, sorry for the delay. I just wanted to have the details of the problem in my head before applying your patch.
I understand it addresses the first point of #193, which we both agreed.
Since only you and @G10h4ck have answered, and you have a good point about removing the configure time --enable-ipv6, maybe we can go for it now, what do you think?
Regards,
Marcelo.

Amusingly I had forgotten entirely about #193 until you mentioned it, and rediscovered this fix by my own investigations.
I think the library should use whichever IP stack is functionally able to support it, be it IPv4 or IPv6 or any combination. We have had for example a report recently that the library fails to start on an "accidentally" pure IPv6 host (https://github.com/gerbera/gerbera/issues/1090 we didnt handle the failure and segfaulted, whoops!) but I would like to enable pure v6 in libupnp at some point.
As for removing the --enable-ipv6 I think it is a good idea, I would imagine most distributions will have been shipping with that option as Gerbera has required it (due to UpnpInit2) for a number of years now.
That said, it is probably sensible to add at the API layer, rather than build time to enable/disable IPv4/IPv6 support.

Amusingly I had forgotten entirely about #193 until you mentioned it, and rediscovered this fix by my own investigations.
I think the library should use whichever IP stack is functionally able to support it, be it IPv4 or IPv6 or any combination. We have had for example a report recently that the library fails to start on an "accidentally" pure IPv6 host (https://github.com/gerbera/gerbera/issues/1090 we didnt handle the failure and segfaulted, whoops!) but I would like to enable pure v6 in libupnp at some point.
As for removing the --enable-ipv6 I think it is a good idea, I would imagine most distributions will have been shipping with that option as Gerbera has required it (due to UpnpInit2) for a number of years now.
That said, it is probably sensible to add at the API layer, rather than build time to enable/disable IPv4/IPv6 support.


UTF-8 support in .env

Hi, first of all, thanks for nice library! It's awesome software 👍
I need to use UTF-8 in .env and want to use python-decouple for this, but I suppose it won't work with UTF-8, because it doesn't pass encoding param to open function:
https://github.com/henriquebastos/python-decouple/blob/24cc51c7880b79d81b5a75c2d441a41428792aa6/decouple.py#L125
So it will fallback to locale.getpreferredencoding(), as specified in Python docs, which is platform dependent. On my Windows 10 its cp1251, but I want to write UTF-8 in .env and be sure it will be read correctly by python-decouple.
It would be nice, if we could pass encoding kwarg to config() and it will be used in open() (or default if this kwarg is not specified)
Feel free to notify me, if I can help with something

This seems nice. Can you propose a PR adding a default argument to config to do the trick?

Sure, I'll do it in few days

Hi Guys,
Since you're working on "Encoding", I've a concern on retrieving data from a UTF-8 .env file containing some "Persian/Farsi" string/text. When I get it back by "config()' it turns to another "charset/codpage", I tried to type the text in different editors, but the result is the same. FYI hereby is the original and retrieved values.It would be nice to let me know what's going on. Tnx
Code:config('COMPANY_NAME', default='ایده پردازان ارتباطات نوین')
.env:COMPANY_NAME=ایده پردازان ارتباطات نوین
Original:ایده پردازان ارتباطات نوین
Retrieved by config():'Ø§ÛŒØ¯Ù‡ Ù¾Ø±Ø¯Ø§Ø²Ø§Ù† Ø§Ø±ØªØ¨Ø§Ø·Ø§Øª Ù†ÙˆÛŒÙ†'

Hi, @farzbood, look like this issue is quite stale. I proposed PR more than year ago (now I see it's completely not ideal and should be updated), and there're no more updates on this

Yeah, I visit that. I will try to contact the owner too.
Thank you for response though.
good Luck

Great! It works perfectly nice.
Thank You.
Peace V


Concepts should inherit from hana::integral_constant

Right now, concepts only provide a nested ::value static member, which is not idiomatic. Instead, they should inherit from hana::integral_constant, but that might raise some circular dependencies issues.
Another take on this is that on the contrary, we don't want the concepts to inherit from an integral constant class, since we'll eventually move to C++17 concepts, and the migration will be easier if we only provide a ::value member.

I recently did something like using type = hana::bool_c<value>; in a concept so I could use it in hana::trait<Concept>.

Ok, I'll implement this. But not in 1.0, because I'm freezing master until Boost 1.61 is out.

Having the concepts be integral constants would help them play nicer with the rest of hana, if the user would want to, say for example, use them in a predicate to filter.


Weird order-dependency when transpiling Foo.Bar = class extends Foo

repro
The following code should work, but causes a type error:
class Foo {}
Foo.Bar = class extends Foo {};
Foo.Baz = class {};

input0:3: WARNING - Expected constructor name but found Foo.Baz.
Foo.Baz = class {};
          ^^^^^^^^

This seems to have to do with how ES6 classes are transpiled to explicitly add static fields from superclasses onto subclasses (omitting most jsdoc):
var Foo = function() {};
Foo.Bar = function(var_args) { Foo.apply(this, arguments); };
$jscomp.inherits(Foo.Bar, Foo);
/** @struct @constructor @suppress {visibility} */
Foo.Bar.Baz = Foo.Baz;
Foo.Bar.Bar = Foo.Bar;
Foo.Baz = function() {};

I'm assuming that the error is actually coming from the line Foo.Bar.Baz = Foo.Baz since the latter has not yet been defined but is being assigned to an @constructor.
Interestingly, if Foo.Baz is not a class but rather a simple primitive then there is no error (presumably since there's no @constructor annotation), but the output is still broken:
Foo.Bar = function(var_args) { /* ... */ };
$jscomp.inherits(Foo.Bar, Foo);
/** @suppress {visibility} */
Foo.Bar.baz = Foo.baz;
Foo.Bar.Bar = Foo.Bar;
Foo.baz = 42;

I think the way to fix this is that we need to add the declaration after the later of the two classes.

I looked into fixing this in Es6ToEs3ClassSideInheritance but apparently this pass does not actually run in transpile-only modes, so my runtime regression tests fail in IE8.
Potentially a better fix would be to simply insert the $jscomp.inherits call after the last property on the base class in the event that it occurs after the subclass is declared.
I'm not sure how this interacts with modifying classes across modules.  It's possible that this could move the $jscomp.inherits call from the base chunk into a later module, which would be bad. @concavelenz

There is no way to handle this in an IE10-compatible way when doing single-file transpilation.  The best we can hope for is a check during normal compilation that late-added static properties aren't used via inheritance.
An easy way to get this now would be to simply not copy later properties.  This would cause type checking to fail, so that the runtime tests' failures will at least be consistent with a build error.  But it will not carry over very well to a future where we transpile classes after a class-side-inheritance-aware type checker.  For that to work, we'll need to add an explicit check to Es6RewriteClasses that no late statics are used in an IE10-incompatible way.


content-type application/json header removed by proxy

What is your Test Scenario?
The client side of my application is sending a request to the server with a Content-Type application/json header but this header seems to be getting removed by the proxy.
What is the Current behavior?
content-type application/json header is being removed by the proxy when the client side of my application adds it to a POST.
What is the Expected behavior?
The content-type header should not be removed by the proxy.
Your Environment details:

testcafe version:                   1.1.2
node.js version:                    8.11.1
command-line arguments:  testcafe chrome test.js
browser name and version: Chrome 73
platform and version:          macOS 10.14.4

@bryanh84
I don't see any content-type replacement in the testcafe-hammerhead proxy code and the information you provided is not sufficient to reproduce the issue. Please provide us a working project which demonstrates the issue and your test code.

@Farfurix
Please see my repo containing a sample app and test script for reproducing the issue. https://github.com/bryanh84/testcafe-bug

Hi @bryanh84
Thank you for the provided details. I've reproduced the problem.
For team
The problem is in FetchSandox during processing the fetch headers.
Need to debug the fetch request creation - https://github.com/bryanh84/testcafe-bug/blob/a5e74666412c374047b8fc4a7060242ecaafddbf/src/App.js#L15


[ISSUE #6708] Use GitHub cache for CodeQL

Which Issue(s) This PR Fixes

Fixes #6708
Brief Description

Enable GitHub cache for CodeQL to ensure the success of maven dependencies' download
How Did You Test This Change?

I have tested it in my fork.

Codecov Report

Merging #6709 (7c498bf) into develop (2da6e9b) will decrease coverage by 0.03%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             develop    #6709      +/-   ##
=============================================
- Coverage      42.88%   42.86%   -0.03%
- Complexity      8999     9003       +4
=============================================
  Files           1110     1110
  Lines          78643    78643
  Branches       10233    10233
=============================================
- Hits           33728    33707      -21
- Misses         40692    40716      +24
+ Partials        4223     4220       -3

see 19 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Fix the home page

when visiting the homepage, it should be redirected to https://open-bus-map-search.hasadna.org.il/dashboard.
the text "click to navigate to dashboard." should never appear

thanks @rupeshghosh10 ! :1st_place_medal:

Thanks for the review


Instantiating Plyr, Acquiring Instance

There's little documentation about acquiring a Plyr instance, despite all the documentation describing a simple player variable.  How is this player element acquired?  document.getElementById?  Why is Plyr seemingly auto-binding to certain elements, but not others?  How can I specifically target an element for Plyr instantiation?

Suggestion: var player = plyr.setup('.some-css-selector');

Well the instance is stored against the original selector. I agree it could be made more clear but it's essentially covered here:
https://github.com/Selz/plyr#api
e.g.
var player = document.querySelectorAll(".player")[0].plyr;

Also setup will return the element similar to your second example, however if you have multiple on the page, it will return them all, which might be less than useful.
var player = plyr.setup(config);

However currently setup only accepts a full config as an argument, I could easily change setup() to allow either the config, the selector or the config and selector if you think that's easier to understand?
var player = plyr.setup('.js-player');
var player = plyr.setup('.js-player', config);
var player = plyr.setup(config);

In my opinion, I would prefer:
var player = plyr.setup('.js-player');
var player = plyr.setup('.js-player', config);
var player = plyr.setup(config);

What do you think @martindale ?

OK. I've made it like so:
Passing a NodeList:
plyr.setup(document.querySelectorAll('.js-player'), options);

Passing a HTMLElement:
plyr.setup(document.querySelector('.js-player'), options);

Passing a string selector:
plyr.setup('.js-player', options);

Passing just the options object:
plyr.setup(options);

This will go in v1.4 in the next couple of days

Added docs here in the develop branch:
https://github.com/selz/plyr/tree/develop#quick-setup
and
https://github.com/selz/plyr/tree/develop#api
Will add more on the
Warning classnames and the default hook will change in 1.4 to plyr instead of player to be consistent. Easy enough to find/replace.

I like the new method, it's much more in line with expectations.  I'd go so far as to not instantiate unless explicitly called; that is to say, don't automatically instantiate on anything (well, definitely not a class name).  Barring that, it looks good! :+1:
Ah, and the other requirement of having a nested <video> or <audio> tag -- just let it bind to anything, and automatically create the required child elements.

Yeah, you're probably right. I guess my thinking was just to make it easier for the less technically advanced users out there. I agree with you though, it'd be better to not do it automatically.
My issue with not having an initial <audio> and <video> element on the page is that there's no non JS fallback in place then. I've tried to keep this a progressive enhancement thing.

Fixed in v1.5.0

Unfortunately I'm still not clear how to get current instance?
My setup is
const players = Plyr.setup('.plyr', options)
in this page I've 10 video now how do I can get current object/instance? so I can do something using their events...

Why did you give all my replies thumbs down? I'd have replied to help but don't want to risk another thumbs down! Good luck with it.

@sampotts sorry for that. Actually it's doesn't clarify how to get an instance or current (playing) instance. Like in many videos (in a page) how do you get the instance(s) that's are actually playing?
Btw... I managed to get it in a very different way, & its working fine. I also added feature (on my app only):

Stop other player (thats are playing) as soon as last one starts to play.
Play only one player at a time, if one already playing then stop that & start new one.
Pop out the player (like fb) when user scrolls down.
Resizing & moving of the poped player..

btw thanks for your effort.

@apudiu Can you please tell the procedure how to get the current instance

sure @jahid32, I passed an additional property in the config (in this case an ID) then I initialized all player in a variable players. while initializing I added several event handlers (like, end, playing, paused etc..) to all player. whenever any instance starts playing I can check the id of that (my passed id) instance then I can do anything with it. you also can loop the players & check existing getters to check which one is playing.


Missing Kong command in the docker run merged documentation (Enterprise and OSS)

Where is the problem?
https://docs.konghq.com/gateway/2.6.x/install-and-run/docker/
What happened?
The instruction to execute the Db migration is incorrect :
docker run --rm --network=kong-net \
  -e "KONG_DATABASE=postgres" \
  -e "KONG_PG_HOST=kong-database" \
  -e "KONG_PG_PASSWORD=kong" \
  -e "KONG_PASSWORD={PASSWORD}" \
  kong migrations bootstrap

return /docker-entrypoint.sh: line 57: exec: migrations: not found
What did you expect to happen?
The right command should be :
docker run --rm --network=kong-net \
  -e "KONG_DATABASE=postgres" \
  -e "KONG_PG_HOST=kong-database" \
  -e "KONG_PG_PASSWORD=kong" \
  -e "KONG_PASSWORD={PASSWORD}" \
  kong kong migrations bootstrap

and same for the Entreprise command
Code of Conduct and Community Expectations

[X] I agree to follow this project's Code of Conduct
[X] I agree to abide by the Community Expectations

@jdevillard thank you for opening this issue! I'll look into this and update.

I have created an internal task for this. I was able to replicate this issue.
I'm going to add a text input for the second kong in kong kong migrations bootstrap.

Closing out as this has been resolved. Thank you again, @jdevillard!


Expose the event loop as an optional parameter to app.run(..)

Since the event loop is created in server.serve it could be beneficial to expose passing a pre-allocated event loop that could be used before the server starts to enable database connections and such.

Strongly +1 this, I think it could make nice designs much more straightforward.

Merged, closing :)


Resized window gets reset when enabling transparency.

I'm not sure if I'm doing anything wrong here but when I trigger  AppWindowUtility.Transparent = true; my window gets resized after I changed its size.
Even if I resize my window via Screen.SetResolution or using my mouse, if I enable transparency it will be reset.
It always seems to be the resolution that the program launched and I can't seem to find a way to change this value anywhere.
Is there a way to have re-sizable windows using this feature?

Hi, i'm updating this issue because I have a similar problem, when I use Screen.SetResolution and then change the Transperency the windows get resetted to default size


device_tracker support

is it possible to add the device_tracker support? I can see this as fixed type - just like binary_sensor.
I've got quite a few devices that I track with network presence, and they all have corresponding device_tracker sensors [home = device is on, not_home = device is off]. right now I started to add template binary_sensors to be able to add them to powercalc integration, but it would be great to avoid that sensor duplication.

Yes will add support for device trackers soon. This is a small change.

thank you!

We can leave this issue open, as this is also my todo list ;-). Otherwise I will forget about this.

We can leave this issue open, as this is also my todo list ;-). Otherwise I will forget about this.

oh ok, my bad! :)

new beta is released

ok, I've tested new beta.
looks like device_tracker works - sensor is created correctly. I'll have to wait some time to see the readouts, but I guess it will be ok :)
however - some issues with entity_id naming are back: kodi, tv and desktop pc again named like mentioned in https://github.com/bramstroker/homeassistant-powercalc/issues/108#issue-976866025 & https://github.com/bramstroker/homeassistant-powercalc/issues/108#issuecomment-904485806 and

Very strange that you entity naming issues are back. The code for that is still in the beta branch. I think it has to do with the fact that you switched back to the master branch (which created the wrongly named entities). Did you remove those entities first?

Very strange that you entity naming issues are back. The code for that is still in the beta branch. I think it has to do with the fact that you switched back to the master branch (which created the wrongly named entities). Did you remove those entities first?

hm, I thought that earlier installations could mess something up, so before testing I did those steps:

removed [commented] all powercalc configurations from my yamls + restarted homeassistant
uninstalled powercalc + restarted homeassistant
checked the entities list in dashboard, removed all that were still there + restarted homeassistant
installed new beta + restarted homeassistant
brought back powercalc configurations [uncommented all] in my yamls + restarted homeassistant

then I've noticed that my template sensors built as a sum of few powercalc sensors - are down, so I've checked entities list & noticed, that the "old" names are back.
I can redo all above steps just to be sure it's reset, but are there any other places/ways I could/should clean those entities?
btw, device_tracker-based sensor's readouts are ok! :)

Did you see in your (debug) logs that these entities were created by Powercalc again? If this is not the case Home Assistant magically created these entities again. Not sure what can cause this, maybe because there are still references somewhere in the database state table. Have a look here how to clear that: https://community.home-assistant.io/t/delete-history-of-one-sensor/87203.

And by "old" names back you mean each power sensor is duplicated (with old and new name), or did the new names disappear?

And by "old" names back you mean each power sensor is duplicated (with old and new name), or did the new names disappear?

the new names didn't showed up at all, only "old" ones.
but I tried your earlier hint now - I went all my steps I wrote about, but after 3rd step I additionally deleted all instances of those entities from the history database, then made one more restart and went to step no. 4 and 5 [and because in the meantime third beta showed up - I installed that one, so I'm on latest&greatest ;)]
right now I can see that all wanted entities are created, with the names I hoped for [built from entity_ids]. so looks like all is cleared for now and works as intended :)

Thanks for testing and the feedback. Think we have green lights for a stable 0.5.0 release now. Will do that later today.

no problem, thank you for adding new entity support & making changes to entity_id name building!

Feature is released


Adds test module to user role guide - #230

Adds a test module to the guide for user role plug per #230 that can be used as a base for users working to understand one way to implement roles in Pow.
It may also be worth mentioning other authorization libraries or strategies, such as bodyguard.

It may also be worth mentioning other authorization libraries or strategies, such as bodyguard.
Perhaps then briefly explain pros and cons? Can I achieve the same using Pow, or are there situations where an extra library is needed?

Thanks a lot! I just refactored the code a bit, and made the tests more about testing various users 😄

Perhaps then briefly explain pros and cons? Can I achieve the same using Pow, or are there situations where an extra library is needed?

Yeah, I think that would be great! However from my own experience, using third party for authorization only really makes sense when you got complex permission setup, and then anyway most would outsource that to IdP/IAM services. Otherwise plugs makes it super easy to do it yourself.

Totally fair. Probably outside the scope of this guide to get into more involved authorization schemes, rather than just giving a basic roles example.


v-model on radio input should be string

https://github.com/johnsoncodehk/volar/blob/1421e48ca374b14de5c30b5c0d4251bff88d2d4e/packages/vscode-vue-languageservice/src/generators/template.ts#L1358-L1359
code modified from https://v3.vuejs.org/guide/forms.html#radio
<template>

    <input type="radio" id="one" value="One" v-model="picked" />
    One

    <input type="radio" id="two" value="Two" v-model="picked" />
    Two

    Picked: {{ picked }}

</template>

npx vue-tsc --noEmit
Radio.vue:3:46 - error TS2322: Type 'string' is not assignable to type 'Booleanish'.

3     <input type="radio" id="one" value="One" v-model="picked" />
                                               ~~~~~~~

Radio.vue:6:46 - error TS2322: Type 'string' is not assignable to type 'Booleanish'.

6     <input type="radio" id="two" value="Two" v-model="picked" />
                                               ~~~~~~~

Found 2 errors.

This is upstream problem, see https://github.com/johnsoncodehk/vue-tsc/issues/58.

I still don't know how to fix this after read  johnsoncodehk/vue-tsc#58 .
It seems talking about value, but this issue is about v-model.

You can use 0.26.10 for now.


How should the Athenz jwk config be distributed?

The AuthZpeClient supports both the "traditional athenz.conf" and the "jwk format of athenz.conf". While the traditional athenz.conf can be obtained using the athenz-conf CLI, there seems to be no CLI available for obtaining the jwk format of athenz.conf.
In light of this, I have two questions:

How are Athenz users expected to obtain the jwk format of athenz.conf?
We are considering implementing HttpPublicKeyStore.java to retrieve jwk from ZTS, but we are concerned that this implementation is not currently provided. Is there any reason why either the traditional athenz.conf or the jwk format of athenz.conf must be placed as a file before the application starts?

Is it assumed that the jwt format of athenz conf will be distributed by sia?
https://github.com/AthenZ/athenz/blob/f9f1240a770df867206aa94311e091d7ac49115b/libs/go/sia/agent/agent.go#L342-L347

I understand that sia acquires the information via postInstanceRegisterInformation and postInstanceRefreshInformation.
Would it be feasible to expose the jwkConfig utilized by the two methods mentioned above as an API of ZTS? https://github.com/AthenZ/athenz/blob/f9f1240a770df867206aa94311e091d7ac49115b/servers/zts/src/main/java/com/yahoo/athenz/zts/ZTSImpl.java#L411-L414

I'm slowly moving away from athenz.conf since that was a proprietary format and instead I prefer to use the jwk format as defined in rfc7517.
the initial implementation relies on SIA to automatically fetch the conf file and keep it updated periodically. This eliminates the need for the client to worry about how to refresh and whenever it sees an unknown key, all it needs to do is to reload the latest file from disk which should be updated.
So I would strongly recommend if you're implement your own SIA, then make this part of the agent so it automatically gets updated daily. The server logic only returns the data if there are changes so you don't waste network resources by sending the data every time.
As far as separate APIs, we already have /oauth2/keys API which returns the ZTS keys in this format. If you want to get the zms keys, we can extend this api to specify an optional query element to specify the service name (e.g. /oauth2/keys?rfc=true is default and returns ZTS keys but if you specify /oauth2/keys?rfc=true&service=zms then it can return the ZMS keys - this way we don't need yet another API

Thank you for your proposal. We will consider how we can implement this in our environment.
By the way, regarding the use of JWS policy, I believe that the verification key for ZMS is not inherently necessary. However, is it correct to understand that the current ZTS provides the ZMS verification key in postInstanceRegisterInformation and postInstanceRefreshInformation for backward compatibility?
Specifically, the PublicKeyStore interface used by ZPE has a method named getZmsKey. While the verification of JWS Policy only requires the ZTS verification key, ensuring the principle of Java interface contracts mandates the implementation of this method. https://github.com/AthenZ/athenz/blob/b68e84687542a639c33e49cd67af23950807ca11/clients/java/zpe/src/main/java/com/yahoo/athenz/zpe/pkey/PublicKeyStore.java#L29-L34
I believe that in the future, the ZMS verification key will become unnecessary for ZPE's PublicKeyStore. Is there any difference in understanding?

ZTS does not just cater ZPE use cases. It's possible that you have an application that fetches full domain jws files from ZMS  and needs the public keys for verification of the signature. So we provide both ZMS and ZTS public keys for all clients.
So while ZPE will eventually no longer need ZMS public keys (most likely we'll remove it from the 1.12.x release scheduled sometime in Q3), ZTS will continue to provide the ZMS and ZTS keys.


AB#5169

Proposed changes

Related issues

Checklist

[x] I consider the submitted work as finished
[x] I tested the code for its functionality
[ ] I wrote test cases for the relevant uses case
[ ] I added/update the relevant documentation (either on github or on notion)
[ ] Where necessary I refactored code to improve the overall quality

Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

This looks good with the local build. @r-joyce please review and merge.

@r-joyce sorry, forgot you were on PTO! Ignore these today.


Power Pages data model migration issue with non-English sites

After running the command to migrate the data model for our Power Pages site: pac powerpages migrate-datamodel --webSiteId <siteId> –-mode all
I checked the migration status with pac powerpages migrate-datamodel --webSiteId <siteId> --checkMigrationStatus, and saw that it was set to "Fullført", the Norwegian word for Completed. So I ran the migration:
pac powerpages migrate-datamodel --webSiteId <siteId> --updateDatamodelVersion --portalId <portalId>
And just got the response about the current migration status, without the actual migration being run. Since I had successfully done this on another Power Pages site that was in English, I noticed that there the status was listed as "Completed" and it worked as it should. In the Norwegian environment, I went to the table adx_websitemigrationtracker and edited the Choice column adx_migrationstatus so that the label for the completed status read "Completed" rather than "Fullført". After doing this, I ran the final migration step again and the data model migration finished successfully.
It seems like there is a bug where the migration status gets checked against the English word "Completed" rather than being language agnostic as was expected.

This is a known issue - https://admin.powerplatform.microsoft.com/knownissues/4305713
The above link has a workaround to the problem. We are working towards fixing this.

Same for me still showing Running Status after 3 hrs.

Hey @varun6288, if your base language is English, then I would suggest you to create a separate issue. As the issue being discussed in this thread is only around - https://admin.powerplatform.microsoft.com/knownissues/4305713


python312Packages.aioqsw: 0.4.0 -> 0.4.1

Diff: https://github.com/Noltari/aioqsw/compare/refs/tags/0.4.0...0.4.1
Changelog: https://github.com/Noltari/aioqsw/releases/tag/0.4.1
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 334792 run on x86_64-linux 1

5 packages built:

home-assistant-component-tests.qnap_qsw
python311Packages.aioqsw
python311Packages.aioqsw.dist
python312Packages.aioqsw
python312Packages.aioqsw.dist


Add gitter chat for discussion

Resolves: https://github.com/scalatest/scalatest/issues/626

Failure unrelated.


Fix toLowerCase hook

PR to fix #74  issue

Thanks!  Can you fix the one on line 7, too?
https://github.com/feathersjs/feathers-authentication/pull/75/files#diff-5c3a88f39327f1c9056093a30b1638aaR7

done!

Cool.  One more thing I missed, can you revert the version number bump?  Our npm release scripts will take care of that.

done too!
but 0.3.1 it's still into README changelog. have you a script to bump changelog?

Updating the changelog is fine.
@marshallswain It looks like there are no tests for this hook correct?

Correct.  I just checked.  That one is untested.

Ya I'll add some tests today but that is a good fix. Thanks @enten! It's been on my list in auth repo but I haven't had time to get around to it. Honestly, wasn't sure if anyone would use that hook.

Thanks!
I thinks that is it a great helper for fields with an unicity constraint (like username or email).


How to integrate to Azure Active Directory

Greetings,
Just wondering if we can integrate Elastic Stack to Azure Active Directory, so that we can configure permissions on Azure Active Directory.  Thanks!
All the best,
Daniel

Hey @DanielWangZhanggui - It's possible to do with X-Pack and I'm working on a blog post to share details on how to do this.

Cool! Looking forward it. :-D
All the best,
Daniel

@DanielWangZhanggui the blog post is now live:

Hey @russcam, great blog post, currently trying to configure SAML SSO with Azure AD. We have an existing elastic stack deployed on azure (we used the template from the marketplace), the only place I'm confused on in your blog post is setting samlMetadataUri and samlServiceProviderUri, can these be configured after it's been deployed, if so, how do I got about configuring these values?
Thank you!
Alex

Hey @alexknipfer, samlMetadataUri and samlServiceProviderUri are input parameters to the ARM template that are not exposed in the Azure Marketplace offering - this is a deliberate decision, to keep the Marketplace getting started experience as simple as possible.
If you've already deployed Elasticsearch, you'll need to edit the elasticsearch.yml config on each node in the cluster for SAML integration. Perhaps the most straightforward way to understand what is required would be to check out the elasticsearch installation script does:
https://github.com/elastic/azure-marketplace/blob/59afd796c21c05188e50b3aea6dfd261c3147636/src/scripts/elasticsearch-ubuntu-install.sh#L1081-L1105
as well as the kibana install script
https://github.com/elastic/azure-marketplace/blob/59afd796c21c05188e50b3aea6dfd261c3147636/src/scripts/kibana-install.sh#L285-L302
Note, all the prerequisites listed in the blog post apply e.g. Kibana must be installed, license applied must be one that enables the SAML realm, TLS for Elasticsearch HTTP layer, Elasticsearch version must be equal or greater than 6.2.0, etc.
Another option may be to deploy a new cluster, snapshot data from the existing cluster, and restore to the new cluster. We're working on documentation for the ARM template, including a section on SAML.
Hope that helps!


Remove Dynamic*ClientLibraryServlet and breaks out TouchUI widgets into discrete Client Libraries

Im not 100% sure about the role of these to client lib categories:

acs-commons.granite.ui.coral2.foundation
granite.ui.coral.foundation.addon.coral2

Would be great if you can review their use (in the actual CL and in the Wrapper CL definitions)
/cc @kwin

Pull Request Test Coverage Report for Build 3952

0 of 0   changed or added relevant lines in 0 files are covered.
27 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.1%) to 49.335%

Files with Coverage Reduction
New Missed Lines
%

bundle/src/main/java/com/adobe/acs/commons/wcm/impl/AbstractDynamicClientLibraryServlet.java
27
0.0%

Totals

Change from base Build 3950:
-0.1%

Covered Lines:
11868

Relevant Lines:
24056

💛  - Coveralls


assemble preserves circuit name with a single parameter_binds parameter

I think bind_parameters should preserve circuit.name in the returned copy of the circuit. However, that was not not decision when https://github.com/Qiskit/qiskit-terra/issues/5185 was fixed. Therefore, fixing this at assemble time.
Fixes #6200

Closing this since, as discussed, the current one is the expected behaviour and the textbook should be patched.


Update esbuild version. 14.14 wasn't available anymore.

Default version 14.14 is not available anymore, too old, put the latest in the readme !
<3

:green_heart: :blue_heart: :purple_heart: :yellow_heart: :heart:


[FLINK-22617][Hive]Add log for hive source that use native or mapred reader

What is the purpose of the change
Hive table sink  has some log that tells us whether to use native or mapred .
LOG.info("Hive streaming sink: Use MapReduce RecordWriter writer.");
LOG.info("Hive streaming sink: Use native parquet&orc writer.");
LOG.info(
 "Hive streaming sink: Use MapReduce RecordWriter writer because BulkWriter Factory not available.");

I have some ideas we can add log  to make it more  obvious when read hive for createBulkFormatForSplit.

@luoyuxia Hi , please look at this.

@luoyuxia  Hi ,please review.

Thanks @hehuiyuan for working on this. Since a reader is created for every split, will it be more useful if we also print the split in the logs? Besides, maybe it's better to print at DEBUG level. Big jobs can easily have thousands of splits. Printing a message for each of them at INFO level seems too verbose to me.

Thanks @hehuiyuan for working on this. Since a reader is created for every split, will it be more useful if we also print the split in the logs? Besides, maybe it's better to print at DEBUG level. Big jobs can easily have thousands of splits. Printing a message for each of them at INFO level seems too verbose to me.

Hi @lirui-apache ,  yes, it is many unuseful log.
Originally, I was going to use debug,  but I'll revise it after you confirm and reply.

Thanks @hehuiyuan for updating. LGTM overall. Just left a minor comment.


Yarn can't find the package

Looking forward to using this! Has this been tagged for release? It's not coming through with Yarn 1.22.0:
yarn add @twotentytwo/jscroll
yarn add v1.22.0
[1/4] 🔍  Resolving packages...
error An unexpected error occurred: "https://registry.yarnpkg.com/@twotentytwo%2fjscroll: Not found".
info If you think this is a bug, please open a bug report with the information provided in "{path}/yarn-error.log".
info Visit https://yarnpkg.com/en/docs/cli/add for documentation about this command.

@thelucre hi. A major update will come within a week, along with the release:)

Oooh baby! Can't wait ✨

Is there any news on the release?

@tomrutgers I just used src stuff for this. Still can't install it though

Hi there,
I´m really interested in your package.
But yarn add @twotwentytwo/jscroll still leads to this error:
error An unexpected error occurred: "https://registry.yarnpkg.com/@twotentytwo%2fjscroll: Not found".
info If you think this is a bug, please open a bug report with the information provided in "{path}/yarn-error.log".
info Visit https://yarnpkg.com/en/docs/cli/add for documentation about this command.

Then I tried npm install git+https://github.com/jesperlandberg/JScroll.git with my project (I'm using vite btw).
But I get this The requested module […] does not provide an export named 'default' " error in my console.
Can you give me a hint how to fix this, or workaround this?
Thank you!


Add mongoDB plugin report db instance info

If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes https://github.com/apache/skywalking/issues/11252.
Update the CHANGES log.

Please format the PR, this seems no clear description.

Please format the PR, this seems no clear description.

I have enriched the description information, please check the PR

For others reading this PR, the context happens at https://github.com/apache/skywalking-java/pull/597


chore: fix & fmt

Fixes RVT-3904

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1343  👈
#1342
#1341
#1340
#1339
#1338
#1337
#1336
#1335
#1334
#1333
#1332
#1329
#1328
#1327
#1322
#1321
#1320
#1319
#1318
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @NathanFlurry and the rest of your teammates on  Graphite


Elite Rank badges

Is there any problem with me create 3d models of your rank badges. These are aimed at being able to be 3d printed? I will probably upload them to thingiverse when finished.

It's probably not a problem, as long as it's not commercialized I don't think you would break Frontier's community license.
From my side you're allowed to take the assets and make 3D models, but keep in mind you have to follow Frontier's community license as well


Notebooks: Combine and rewrite data use case

Add another use case to the notebooks documentation for combining data from multiple buckets, apply a simply multiplication, and rewrite.
Excerpt from user research explaining the use case (only accessible to InfluxData employees):
https://drive.google.com/file/d/1kD9b5AKgRiFvFN2ku6YKEaqSIKz1TnKl/view?usp=sharing
Relevant URLs
https://docs.influxdata.com/influxdb/cloud/notebooks/
What products and version are you using?
InfluxDB Cloud

Closing this per discussion with @russorat


(optimization) update cli to use local sdk

Codecov Report

Merging #614 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #614   +/-   ##
=======================================
  Coverage   63.98%   63.98%
=======================================
  Files          33       33
  Lines        1158     1158
=======================================
  Hits          741      741
  Misses        406      406
  Partials       11       11

Impacted Files
Coverage Δ

cli/util/cliparamsatisfier/cliParamSatisfier.go
43.08% <ø> (ø)
:arrow_up:

cli/core/op_install.go
100% <ø> (ø)
:arrow_up:

cli/core/dataResolver.go
86.15% <ø> (ø)
:arrow_up:

cli/cmds/node/create/httpListener.go
0% <ø> (ø)
:arrow_up:

cli/util/cliparamsatisfier/inputSrc_cliPrompt.go
8.92% <ø> (ø)
:arrow_up:

cli/core/core.go
0% <ø> (ø)
:arrow_up:

cli/util/clioutput/cliOutput.go
100% <ø> (ø)
:arrow_up:

cli/core/op_kill.go
100% <ø> (ø)
:arrow_up:

cli/core/run.go
80.18% <ø> (ø)
:arrow_up:

cli/core/events.go
96.15% <ø> (ø)
:arrow_up:

... and 4 more

Continue to review full report at Codecov.

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


Add GToolkit4Famix3 to the main baseline

We need to have support for code analysis directly in the main distribution.


Add compiler api & lang test suites

Add compiler api & lang test suites
Added compiler api test suites;
Added compiler lang sub-test suites ARR, BINC, DASG, EXCP, EXEC, FP, ICLS, INFR, INTF, MOD, NAME, PKGS, THRD, TYPE;
Note: a few other sub-tests to be added later such as lang/ANNOT, lang/CLSS, lang/CONV, lang/EXPR, lang/LEX, lang/LMBD, lang/STMT.
Reviewer: @ShelleyLambert
FYI: @DanHeidinga @pshipton
Signed-off-by: Jason Feng<EMAIL_ADDRESS>

Just for the record, I did verification run locally.
Test suite api
STF 18:16:52.702 - =====================   R E S U L T S   =====================
STF 18:16:52.702 - Stage results:
STF 18:16:52.702 -   setUp:     pass
STF 18:16:52.702 -   execute:   pass
STF 18:16:52.702 -   teardown:  pass
STF 18:16:52.702 -
STF 18:16:52.702 - Overall result: PASSED

jck-compiler-api_0_PASSED

sub-test suites lang/ARR
STF 14:37:44.500 - =====================   R E S U L T S   =====================
STF 14:37:44.500 - Stage results:
STF 14:37:44.500 -   setUp:     pass
STF 14:37:44.500 -   execute:   pass
STF 14:37:44.501 -   teardown:  pass
STF 14:37:44.501 -
STF 14:37:44.501 - Overall result: PASSED

jck-compiler-lang-ARR_0_PASSED

sub-test suites lang/TYPE
STF 16:24:24.138 - =====================   R E S U L T S   =====================
STF 16:24:24.138 - Stage results:
STF 16:24:24.138 -   setUp:     pass
STF 16:24:24.138 -   execute:   pass
STF 16:24:24.138 -   teardown:  pass
STF 16:24:24.138 -
STF 16:24:24.138 - Overall result: PASSED

jck-compiler-lang-TYPE_0_PASSED


add pyspark hive support

This is the hive support of spark on ray, the related issue is :raydp not support read hive ,I found that the original implementation does not support hive read and write by default, this is my contribution,If there is any problem, welcome to correct and discuss

It's not my code bug, what should I do @ConeyLiu @kira-lin

@kira-lin I didn't change the pytest file, why am I getting this errorerror

@hezhaozhao-git Yes, it is not related to your pr. It's ok because linux CI passed.

@kira-lin我没有更改 pytest 文件，为什么会出现此错误错误

So I don't have to make any changes now, just wait for your next release right?

Can you please address the Optional issue mentioned above, and you can use our raydp-nightly release tomorrow. Thanks again

I don't know why this error occurs, so I don't know how to fix it，can you see more details? @kira-lin

Thanks, @hezhaozhao-git , I will fix the CI in other CIs, it's not related to your PR.

You can use raydp-nightly now, it includes this commit:)


After Upgraded to Azure Signalr from Asp.Net Signalr still not working

I had followed the steps as mentioned in the below url
https://devblogs.microsoft.com/aspnet/announcing-asp-net-signalr-2-4-0-preview-1/
First message when I call the send method from client side it's working but second message on wards  it's not working.
Kindly help me to fixed this issue

Is it reproduce-able? Is it possible for you to provide a simple project that can reproduce the issue? Any detailed error messages shown when the message failed to be sent?

By mistake I closed this issue. I will get back to you on Thursday.

Hi Vicancy,
Thanks for your reply,
I am trying to upgrade from Asp.net Signalr to Azure Signalr in Asp.net MVC5 project.
By three simple steps we upgraded to Azure Signalr  using below link
https://github.com/Azure/azure-signalr/blob/dev/docs/use-signalr-service.md
we saw using the browser networking, It was connected with the new service.
It's calling to onconnected in Hub class later we didn't saw send function call. there are ajax calls to save the messages and in success we call hub to
Note : We made as default from ServerLess in AzureSignalr settings.
Please let me know if you need any information.
Correct me if I am wrong.

Is it possible for you to provide a simple project that can reproduce the issue?


Added ory hydra, keto and oathkeeper

Motivation for this change
Ory is an Open Source OAuth 2.0 & OpenID Connect solution. It could be useful for other nixos users.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Kratos was added to nixpkgs. So why not Ory Hydra as well?


Nativefier breaking sudo

Homework

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

Bug description
I recently updated nativefier on my Raspberry Pi running TwisterOS (based on RaspberryPi OS), then I did something else that required a reboot. after the reboot I got errors from sudo and this error from dbus: org.freedesktop.DBus.Error.Spawn.ExecFailed: Failed to execute program org.blueman.Mechanism: permission denied. I changed the permission of the broken sudo file by logging in to the root account, but still got that error and wifi wasn't working.
I didn't suspect nativefier at all, I thought I had done something wrong, so I reflashed my ssd and after installing nativefier and rebooting, I got the same errors! I still didn't suspect nativefier.
this time I reflashed the ssd with regular plain RaspberryPiOS, and it worked perfectly fine untill I installed nativeier, then suddenly sudo stopped working, I changed its permissions again and so far its still working ok (I didn't reboot yet).
Right now I'm 100% sure nativefier is causing this.
could this happen because I'm installing nativefier with sudo (sudo npm install -g nativefier) because otherwise it fails?
Steps to reproduce

Flash a RPiOS image to a sd card/SSD/USB stick.
boot the pi.
install node.js (I installed 15.x).
run sudo npm install -g nativefier.
try to run sudo, it will fail.

Expected behavior
I expected sudo to work fine after installing nativefier.
Actual behavior
sudo broke and caused some other things like wifi to break.
Context

Node.js: 15.9.0
Npm: 7.5.3
OS: RaspberryPiOS and TwisterOS - RPiOS is based on Debian and TwisterOS is based on RPiOS.
Machine: RaspberryPi 4b 4gb model.

@Itai-Nelken uuuuuuh that's a weird one. I don't see how the npm install -g could have barfed your install so hard. Can you still reproduce this bug, and are you 101% sure it's reliably attributable to Nativefier?

Also, which permissions does your broken sudo have?

Might need codereview sounds more like NSA backdoor or malware

@Itai-Nelken uuuuuuh that's a weird one. I don't see how the npm install -g could have barfed your install so hard. Can you still reproduce this bug, and are you 101% sure it's reliably attributable to Nativefier?

I'll try to reproduce this again next week, but it happened 3 times in a row after installing nativefier. so I'm pretty sure is caused by it.

@ronjouch I think I found the issue: installing node.js from apt (using the nodesource repo) breaks the permissions, I installed node.js using nvm and I didn't even need sudo to install nativefier.

This doesn't have anything to do with nativefier.
I have been using it with node.js installed uwing nvm for a few weeks and didn't get any issues so far.


Adjust T1 doc strings to automatic template

Summary
Addressing #254, mimicking #255.
Details and comments
Currently failing (at least locally, for me, let's see how the CI reacts) with:
reading sources... [ 69%] stubs/qiskit_experiments.library.characterization.ResonanceAnalysis
Exception occurred:
  File "/home/hrlquantum/work/experiments/NaokiDoc/docs/_ext/custom_styles/styles.py", line 251, in _extra_sections
    defaults=self._target_cls._default_transpile_options().__dict__,
AttributeError: type object 'ResonanceAnalysis' has no attribute '_default_transpile_options'

Please fix it (probably @eggerdj), then I guess that this PR will be ready to go.

Yes, CI agrees with my local run

@nkanazawa1989 Thanks for the review, I fixed according to your comments :smile:

I have the same issue like Merav, of seeing #section.
Why does it say that there are no experiment and analysis options, and how to fix it?

FYI
https://github.com/Qiskit/qiskit-experiments/blob/main/qiskit_experiments/library/calibration/drag.py
https://github.com/Qiskit/qiskit-experiments/blob/main/qiskit_experiments/library/calibration/analysis/drag_analysis.py

Current status:

I still see "#section".
In the T1 page there's "reference to analysis class: T1Analysis" without a link to the doc of this class.


Warning: Illegal string offset 'stats' in /root/Dan/vendor/illuminate/support/Arr.php on line 410

Warning: Illegal string offset 'stats' in /root/Dan/vendor/illuminate/support/Arr.php on line 410
Notice: Array to string conversion in /root/Dan/vendor/illuminate/support/Arr.php on line 410
Warning: Illegal string offset 'stats' in /root/Dan/vendor/illuminate/support/Arr.php on line 413
[01:23:38] [ERROR] Exception was thrown.
[01:23:38] [ERROR] Cannot create references to/from string offsets nor overloaded objects
[01:23:38] [ERROR] On line 413
[01:23:38] [ERROR] Of file /root/Dan/vendor/illuminate/support/Arr.php

Fixed via https://github.com/UclCommander/Dan/commit/ab5f16d5644a9bcde44c24243e728034dc592cd8


win32_window : Respect floating.

From my other closed PR. Fullscreen should not be topmost unless floating is set.
This is important for borderless fullscreen windows, which are expected to behave like normal windows.
If this is too drastic of a change. A new window option should be added to control this behavior, as it makes true borderless fullscreen impossible.

I believe you can achieve this behavior with a non-floating, non-fullscreen borderless window that covers the desired monitor.

Hi @elmindreda, that's what I've been trying to do :) But when creating the borderless fullscreen window, it goes through this code-path. I'm using the instructions in the window guide, though I might be missing something.
Basically,
		if (GLFWmonitor* monitor = get_monitor(_position)) {
			if (const GLFWvidmode* mode = glfwGetVideoMode(monitor)) {
				glfwSetWindowMonitor(_window_ptr.get(), monitor, 0, 0,
						mode->width, mode->height, mode->refreshRate);

				on_resize(mode->width, mode->height);
			}
		}

I mean leave it as a non-fullscreen window (i.e. no monitor set for the window) which will avoid the GLFW fullscreen path.
if (GLFWmonitor* monitor = get_monitor(_position))
{
    int x, y;
    glfwGetMonitorPos(monitor, &x, &y);
    const GLFWvidmode* mode = glfwGetVideoMode(monitor);
    glfwSetWindowAttrib(GLFW_DECORATED, 0);
    glfwSetWindowMonitor(window, NULL, x, y, mode->width, mode->height, 0);
    on_resize(mode->width, mode->height);
}

The call to glfwSetWindowMonitor isn't doing anything fullscreen-related here, it just sets the size and position of the window.

I think that will work, thx.


Reset main to remove fixup commit

#93 included a fixup commit a804e5620626d7ac32c4f35a0fbd45ab579cd7cc which should have been squashed before merging. I would like to rebase main to squash the fixup commit with its parent commit.

This is fine. Thanks.

Done
To github.com:spruceid/ssi.git
 + 0e16f72...99eefba main -> main (forced update)


Generate and mirror map data

There is an interest in downloading and self-hosting map data: https://www.reddit.com/r/openstreetmap/comments/v9yoed/comment/j8lvfyr/?utm_source=share&utm_medium=web2x&context=3
If we find about 10 independent entities which would contribute mirrors in all geographic areas, then we could build a sustainable network.
Note that we need at least 2 entities which generate map data. The more the better.

How does IPFS work by the way? I think I saw Hidenori (https://www.linkedin.com/in/hidenori-fujimura-7a273711/) experiment with that in Japan.

Or we can have the pmtiles file on Cloudflare R2. There you don't pay for outbound traffic.

Pmtiles and cloudflare is very simple to set up and requires basically no maintenance. If we should ever be unhappy with cloudflare, we can just go to any other solution that supports http range requests...

Sure, one of the mirrors can use cloudflare. But we need at least a few mirrors nonetheless. Having multiple organsiations/people who support the project makes it last.
e.g. if at some point we do not want to continue hosting then there are still other people out there who supply users with the data.
This is more a people-problem than a technical problem.
I'm not 100% certain which exact problem IPFS solves. E.g. is it an alternative to the torrent protocol? But I guess it's definitely more complex than running 10 Cloudflare R2 stores or 10 nginx instances.

You are right, this is a people problem. Yeah let's build a small network of mirrors.

I have a 1 gbps hetzner in Germany we can use...


add query runner support for query file resampling

Description
We add support for resampling of the query input file to the pinot-tools QueryRunner. This enables us to easily expand a small set of queries proportionally to run performance benchmarks at scale. All changes are fully backwards-compatible.
Upgrade Notes
Does this PR prevent a zero down-time upgrade? (Assume upgrade order: Controller, Broker, Server, Minion)
No
Does this PR fix a zero-downtime upgrade introduced earlier?
No
Does this PR otherwise need attention when creating release notes? Things to consider:
No

Codecov Report

Merging #6136 into master will decrease coverage by 21.20%.
The diff coverage is 48.75%.

@@             Coverage Diff             @@
##           master    #6136       +/-   ##
===========================================
- Coverage   66.44%   45.24%   -21.21%
===========================================
  Files        1075     1223      +148
  Lines       54773    57744     +2971
  Branches     8168     8522      +354
===========================================
- Hits        36396    26126    -10270
- Misses      15700    29477    +13777
+ Partials     2677     2141      -536

Flag
Coverage Δ

#integration
45.24% <48.75%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...ot/broker/broker/AllowAllAccessControlFactory.java
100.00% <ø> (ø)

.../helix/BrokerUserDefinedMessageHandlerFactory.java
52.83% <0.00%> (-13.84%)
:arrow_down:

...org/apache/pinot/broker/queryquota/HitCounter.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...che/pinot/broker/queryquota/MaxHitRateTracker.java
0.00% <0.00%> (ø)

...ache/pinot/broker/queryquota/QueryQuotaEntity.java
0.00% <0.00%> (-50.00%)
:arrow_down:

...ava/org/apache/pinot/client/AbstractResultSet.java
26.66% <0.00%> (-30.48%)
:arrow_down:

.../main/java/org/apache/pinot/client/Connection.java
22.22% <0.00%> (-26.62%)
:arrow_down:

.../org/apache/pinot/client/ResultTableResultSet.java
24.00% <0.00%> (-10.29%)
:arrow_down:

...not/common/assignment/InstancePartitionsUtils.java
64.28% <ø> (-8.89%)
:arrow_down:

.../apache/pinot/common/exception/QueryException.java
90.27% <ø> (+5.55%)
:arrow_up:

... and 1202 more

Continue to review full report at Codecov.

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


Pinpoint-web - To use PinPoint, enable cookies and JavaScript

Description

Environment

Pinpoint version: 1.6.x ,1.8.0-SNAPSHOT
OS, WAS, affected library, etc: Win10
Additional details (e.g. how your collector/web is set up):
Browsers: Google Chrome、IE

Additional Info

hi jacobsy
You could get such an error message by setting the was.
I recommend you to check was message and was configuration.
thanks.

hi koo-taejin
I encountered such an error when running the source code in the eclipse tomcat server. When the package was deployed to the tomcat run, there was no problem.
I found that the generated js file is empty in the eclipse deployment path, See the picture below,so I close this question first, I will check the ide setting!
thanks for your reminder.

hi koo-taejin
I encountered such an error when running the source code in the eclipse tomcat server. When the package was deployed to the tomcat run, there was no problem.
I found that the generated js file is empty in the eclipse deployment path, See the picture below,so I close this question first, I will check the ide setting!
thanks for your reminder.

Hello! I also encountered the same problem after packaging in version 1.8.0. Have you solved it? Can you ask me how to solve it?

解决了吗？

Hello! I also encountered the same problem after packaging in version 1.7.3. Have you solved it? Can you ask me how to solve it? @jacobsy


Add wallet backup

Right now we don't really have a way to properly backup the wallet. It's possible to get the seed phrase from the settings menu, but this is not a proper backup for a silent payments wallet.
For a proper backup, we need the following:

the birthday
list of used labels (though labels are not yet implemented)
the transaction history

Perhaps also other stuff that is not strictly silent payments related, but more wallet related, e.g. contacts.
We should probably allow the user to export this data to a (password protected) file. Only if users don't have access to the file, should we allow seed phrase restoration as a last resort.

I made a milestone for this: Backups.


Add homepage link in login.blade.php

My Christmas present, a link to homepage!
Yes, he sorely lacking me when I disconnect from Voyager.
I don't know about you, but for me it's very helpful.

I can barely tell what you changed since you also made all those whitespace changes.
Having said that, Voyager should not presume your site is available at /.  So I don't think this we'll be able to add this into Voyager.  I'd like to get @emptynick's opinion though.

The addition of the link is from line 93 to 96
The rest is only small syntax corrections

I'm going to decline this per my previous comment.
Also, in the future, please refrain from committing those whitespace/formatting changes.  All they do is pollute the pull request.

OK it's noted. Thank's


Log whenever the application detects tampering of sources

Confirmed that we do track the integrity of sources today and surface a notification to users via https://github.com/microsoft/vscode/blob/d73fa8b14a6c873958d00a7d7ad13fcb540a052c/src/vs/workbench/services/integrity/electron-sandbox/integrityService.ts#L85-L120. It is also added to issues created from our builtin issue reporter. We can add this data to trace logs as well for cases were the issues are created outside the application.

Originally posted by @deepak1556 in https://github.com/microsoft/vscode/issues/206522#issuecomment-1978225514

This is a great idea:

Previously, extensions that patched VSCode like Background would occasionally result in a notification only after VSCode was updated.
After this change, this banner appears every time VSCode is opened, even if it was previously dismissed for the same version. This feels very obtrusive.
Was that an intended part of this change?

Banner will be removed again in https://github.com/microsoft/vscode/pull/207050, I think it will cause more harm than being useful.


VK issue

VK login not working?

Hi, I have some suggestions to help solving this issue!
Here is a list of similar issues:

Update vk.rst

VK API workflow fix if error happens on vk-side

fix session expiration in vk backend

Hope it's useful!
Cheers,
Cinnabot.


OpenLane Notebooks: "::env(FP_PDN_LOWER_LAYER)": no such variable

Hello,
I've been trying to use the notebooks, and I came across this strange error at Step 6 of the OpenLane flow:
[INFO]: Reading ODB at '/content/runs/RUN_2023.06.17_19.13.55/tmp/floorplan/5-tapcell.odb'...
define_corners Typical
read_liberty -corner Typical /content/conda-env/share/pdk/sky130A/libs.ref/sky130_fd_sc_hd/lib/sky130_fd_sc_hd__tt_025C_1v80.lib
Error: pdn_cfg.tcl, 92 can't read "::env(FP_PDN_LOWER_LAYER)": no such variable
child process exited abnormally

[ERROR]: Creating issue reproducible...

I'm not sure if this belongs on OpenLane instead, but I've seen this error on both the XLS Adder and  Digital Inverter.
Thanks,
Thomas Pluck

related https://github.com/The-OpenROAD-Project/OpenLane/issues/1877

This seems to be fixed with:
openlane            2023.06.26_0_g3bc9d02d  20230606_125334       litex-hub      12MB

Related https://github.com/hdl/conda-eda/issues/343


data verse and package release matching

@atrisovic
how can we download from dataverse with a specific version?

Update on this: pyDataverse that we now use does not support retrieving specific file of specific version (and neither does easyDataverse). This is however possible in the R implementation: https://github.com/IQSS/dataverse-client-r#data-download
Relevant alternative for Python: https://github.com/fatiando/pooch
Ref: Dataverse chat thread: https://dataverse.zulipchat.com/#narrow/stream/377090-python/topic/downloading.20a.20file.20by.20dataset.20DOI.20and.20filename

I got some input on this and will give it another shot in implementation.

Hi All,
For ideas to implement the data retrieval with specific version, view the conversation here: https://dataverse.zulipchat.com/#narrow/stream/377090-python/topic/downloading.20a.20file.20by.20dataset.20DOI.20and.20filename


Azure cloud instances fail with error 'Operation failed with status: 'Bad Request'

Failure Information (for bugs)
https://app.circleci.com/pipelines/github/networkservicemesh/networkservicemesh/9060/workflows/2fa11b49-38f7-4542-8e6c-203fe999f52d/jobs/122183/tests
Steps to Reproduce
Trigger CI

@edwarnicke Can this problem be related to the subscription problem?

@haiodo I haven't access to AKS, so if you'll have a time please take a look at this issue.

I think we already resolved this by fixing kubernetes Version, @denis-tingajkin  Please close if so.

@haiodo yes, sure


fix: bitwise_memcpy keeps untouched bits

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#586  👈 (View in Graphite)
#569
#585
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

add a test here in a follow-up!

Merge activity

Dec 5, 3:42 PM EST: A user started a stack merge that includes this pull request via Graphite.


Sdk branch now based on v0.35.0

👍
Does this add Xcode 8/iOS 10 support?

I think so  - I haven't tried yet though.


OS X binary does not work on Snow Leopard and Yosemite

Download the OS X binary of PhantomJS 2.0
Run it on Snow Leopard
PhantomJS refuses to run

The OS X binary does not work in Yosemite (10.10.1) either, refuses to run and outputs Killed: 9.

The OS X binary for PhantomJS 1.9.8 seems to work on Yosemite (10.10.1). The 2.0.0 binary does not work.

@Vokkim @petermichaux  I had the same "Killed: 9" issue while starting Phantom2 on Yosemite, so I applied a small patch as recommended elsewhere (https://github.com/ariya/phantomjs/issues/12630#issuecomment-69337830) and recompiled working V2 binaries for OSX/Ubuntu/CentOS - they are available at https://github.com/eugene1g/phantomjs/releases if you want them

same issue +1

+1 here on Yosemite

From https://github.com/ariya/phantomjs/issues/12974:
brew install upx
upx -d bin/phantomjs

Then it works.

Broken here with any binaries I try on Yosemite.

Exact same issue on OSX Yosemite.
[1]    57910 killed     ./phantomjs

This is the only binary I could get to run on Yosemite: https://bitbucket.org/ariya/phantomjs/downloads/phantomjs-1.9.7-macosx.zip

The solution from @herrernst works for me on Yosemite

The solution from @herrernst works for me on Yosemite

The solution from @herrernst works for me on Yosemite

Got the same on latest OS X 10.10.2

thanks, @herrernst

I searched for 4 hours today to find this solution, this needs to be posted in a more prominent place, but thank you @herrernst !

+1

Thanks @herrernst
brew install upx
upx -d bin/phantomjs

Got the dreaded Killed: 9 with binary on Yosemite 10.10.2 but brew install phantomjs (on homebrew, natch) installed and worked no problem

+1 for
brew install upx
upx -d bin/phantomjs

This is fixed in master (3c3b47e7967ccaa1783d1329d4fa3a58ce06a00c). Let’s release v2.0.1 soon :)

Nvm

I think I found the issue.  I cd’d into the dirs for local storage and the cache, and cleared them out completely.  Once I did that no more problems.   Perhaps a file format change between versions.
In my case(OS X) they were here:
cd ~/Library/Caches/Ofi\ Labs/PhantomJS
rm -f -r data7
cd ~/Library/Application\ Support/Ofi\ Labs/PhantomJS
rm -f *.localstorage

@pyper Confirmed. Thanks for the fix!

@pyper @mathiasbynens I could not get it working.
For my case, there was no (sub)directory for PhantomJS  at ~/Library/Caches/, but ~/Library/Application\ Support/Ofi\ Labs/PhantomJS. Removed the *.localstorage, but no luck.

@soda-wy's solution works perfectly for me.

the download link https://bitbucket.org/ariya/phantomjs/downloads/phantomjs-2.0.0-macosx.zip on http://phantomjs.org/ failed with "Killed: 9".

https://github.com/eugene1g/phantomjs/releases works, thanks.

@herrernst  also working for me thanks ! :)

The solution from @herrernst worked for me.

Running it on Yosemite, but @herrernst solution doesn't works for me !

@herrernst  solution worked for me too, thanks! (Mac OS X 10.10.3)

Just for the right credits, I got that from @wallneradam (via #12974).

+1 upx installation worked on Yosemite

:+1:

Thanks, @herrernst

+1 upx installation worked on Yosemite

+1 for @herrernst's solution:
upx -d [path to phantomjs executable]

+1 from @herrernst 's solution.

Looks like current master works well with upx now.
/cc @ariya

thanks, @herrernst

@herrernst didnt worked for me
montogeek ~/Downloads/phantomjs-2.0.0-macosx/bin [05:39 PM]→ 👾  💣  ./phantomjs --webdriver=4444
[ERROR - 2015-08-12T22:39:55.228Z] GhostDriver - main.fail - {"line":85,"sourceURL":":/ghostdriver/main.js","stack":"\tat global code (:/ghostdriver/main.js:85:56)"}

  :262 in error

+1

My error was some process that didn't close correctly. After restarting the machine it worked.

+1
The solution from @herrernst worked for me.

Just to pile on:
+1 Solution from @herrernst  works

I used homebrew to update from 1.9.8 -> 2.0.0 which worked seamlessly on OSX 10.10.5.
$ brew install phantomjs (I was prompted to unlink the existing 1.9.8 -> Error: phantomjs-1.9.8 already installed followed by how-to instructions)
$ brew unlink phantomjs (successfully unlinked -> Unlinking /usr/local/Cellar/phantomjs/1.9.8... 2 symlinks removed)
$ brew install phantomjs and the install completed, version updated.
I had to mess with my required versions in my project's Gemfile and run bundler but that is to be expected.

+1 @herrernst why can't this be fixed in the source?

if this is going to keep lingering, it should be posted as a known issue on the home page.

https://github.com/ariya/phantomjs/issues/12900#issuecomment-128972972

+1 @herrernst .. took me hours to figure this out, the upx solution worked for me.

+1 @herrernst

@soda-wy's solution works perfectly for me as well.

the download link https://bitbucket.org/ariya/phantomjs/downloads/phantomjs-2.0.0-macosx.zip on http://phantomjs.org/ failed with "Killed: 9".
https://github.com/eugene1g/phantomjs/releases works, thanks.

"upx" solution worked for me on 10.10.5 Yosemite.

Why is phantomjs still broken on mac 9 months after this was reported???  :(

Please check 2.0.1 version, which should fixed issues.

Is there supposed to be a 2.0.1 git tag/release? https://github.com/ariya/phantomjs/releases shows 2.0.0 for latest.

2.0.1 is not released yet, check out the latest 2.0 branch.
That branch currently at 2.0.1-development.

I was having the same problem.
@herrernst +1 for the solution, worked for me on 10.10.2

+1 for upx!

Works now on OS X El Capitan.

I have a same issue
[1]    48389 killed     phantomjs -v


Allow Protocol to be Set for API Hosts

Allow different protocols to be used with the API hosts

PG1 447

This PR is blocking mainBluewhisk builds.
https://bluemix.pagerduty.com/incidents/PQRSGUP

PG1 447.

PG2 154. This fixes an alert.


Rendering performance of chafa is very slow

sudo apt install chafa
curl https://media.giphy.com/media/12UwsVgQCYL3H2/giphy.gif --output winanim.gif
chafa winanim.gif --font-ratio 1/3
Edit: On Ubuntu 18.04, follow directions here to get chafa sources and build 'em:
https://hpjansson.org/chafa/
Edit2: after doing a ./configure and sudo apt install loop as you realize stuff is missing, you'll get all the way through and it will whine about not being able to find the lib.
Edit3:
ubuntu
cd ~
mkdir chafa
cd chafa
curl https://hpjansson.org/chafa/releases/chafa-1.4.0.tar.xz --output chafa.tar.xz
tar xf chafa.tar.xz chafa-1.4.0/
cd chafa-1.4.0/
sudo apt install gcc pkg-config libgtk2.0-dev libmagickwand-dev
sudo ldconfig
./configure
make
sudo make install

WPR analysis:

11520ms in conhost.exe
3455ms in chafa

For conhost.exe...

5990ms on I/O thread (the likely culprit as it going slow will stop the other side from giving us data)
5183ms on Render thread (which might still be involved as it can lock the buffer when looking things up preventing I/O from writing)

On the I/O thread, hot areas include:

2007ms spent notifying accessibility eventing

Most of this (1405ms) spent inside user32.dll!NotifyWinEvent
This call causes a syscall/kernel transition which is SLOW
The best thing to do here is probably detect that no one needs the event and not transmit it OR
Transmit it less often by coalescing the accessibility events into frames much like the renderer

1196ms spent adjusting the cursor position

This is mostly attributable (823ms) to figuring out whether the cursor is sitting on top of a 2-column-wide character (so it can move 2 spaces right instead of 1). It looks like we're doing this the wrong way and wasting time here since _lookupIsWide is called below for another purpose and retrieving the same information and that is also taking a lot of time.
236ms also spent in kernelbase.dll!SetEvent to trigger the render thread (probably can't avoid, need kernel object to notify a potentially sleeping thread...)

877ms spent run-length-encoding colors

634ms spent in vector reallocation for holding the run-length-encoded colors. This could maybe be re-strategized to leave a bit of excess memory usage around in exchange for not reallocing so hard.

608ms spent looking up the narrow/wideness of characters (_lookupIsWide) during insertion

This is actually backending on gdi32full.dll!GetCharABCWidthsW versus the current font to figure out how wide it is going to be when we don't otherwise know because it's an ambiguous width character.

On the render thread, hot areas include:

4386ms in gdi32full.dll!PolyTextOutW

I don't think there's anything to be done here. I think this is just the consequence of trying to emit a ton of text really fast

I didn't do a wait chain analysis yet to see if the locking/threading was slowing things down because at this point, we have a few areas with obvious routes to improvement that might alleviate the whole deal:
Therefore, my conclusion is:

Cursor movement shouldn't be looking up the column count by the character, it should use the already known cell width value
The accessibility events need to be coalesced
We should be caching the queries to GDI for ambiguous character widths until the font changes

And I have now filed MSFT: 21167256 to do these 3 things at some point and hopefully we'll have fixed the performance issue.

Thanks Michael, that's a nice analysis. So, this is probably a moot question, but I'm assuming PolyTextOutW is called once per render frame?

Correct.

Okay, excuse the debugging by proxy ;) I'll get back to bundling up a PR for you... :)

No problem. It was a fair question to ask.  I implemented the 3rd thing above (GDI size measurement caching) quickly as I thought it was the best cost-benefit ratio and it improved things by 20-30%, but I want to make sure that I'm actually fixing your problem.
Can you possibly send me a WPR trace of your specific repro? (First Level Triage + CPU Usage Profiles? Let me know if you need help on how to do this.)

@miniksa Sure, I've got the WPR trace now. Where shall I send it?

Email me the attachment or a link to a share at Microsoft.com. My GitHub alias is unoriginal and is my work address too. Just don't sign me up for spam please.

I'm sorry, I don't understand -- your github alias is unoriginal? I don't know what you mean.

My e-mail is my Github alias @microsoft.com. Sorry for being obtuse, I'm trying to avoid spam bots picking it up if I write the real mailto:

The major time spent on the WinEvent turns out to be only if Node.js is running on your system.
If Node.js is running, it registers for the WinEvent notifications for EVENT_CONSOLE_LAYOUT to know when the window size has changed. Given WinEvents require kernel work to broadcast and tend to be registered globally, this causes a system-wide slowdown of all of your consoles when it is listening here.
https://github.com/nodejs/node/blob/0109e121d3a2f87c4bad75ac05436b56c9fd3407/deps/uv/src/win/tty.c line 2294
If you kill all node.js runtimes (including the one that Visual Studio 2017 launches), that performance drag goes away.
I need to:

Coalesce MSAA events so they don't happen so often.
Follow up with the WinEvent team to see if they can tell me Node.js is only listening for EVENT_CONSOLE_LAYOUT and not all the messages (because the expensive ones aren't the layout messages, but because MSAA/WinEvent infrastructure is very old... registering for any one registers you for all of them.)
Probably formalize this into some sort of issue/bug on the Node team to stop doing this as they're shooting the entire system's performance in the foot to get a resize notification.

I checked ReadConsoleInput queue is filled with a WINDOW_BUFFER_SIZE_RECORD at more or less the same time and circumstances as when EVENT_CONSOLE_LAYOUT is dispatched over NotifyWinEvent.
Given Node.js in the tty file is already reading through the queue with ReadConsoleInput and discarding all non-KEY_EVENT records... they could probably drop the whole MSAA hookup and just get the events there in a performant manner instead of hijacking an accessibility feature.
Of course, this also screams of #281 needing to implement a better way overall of receiving these sorts of events, but it's going to be a bit before we get to that.

That rabbit hole is getting deep :D Very interesting to read.

Yes... It is...
@DHowett-MSFT promised me he'd drive resolution with Node (or libuv upstream).

Follow up with the WinEvent team to see if they can tell me Node.js is only listening for EVENT_CONSOLE_LAYOUT and not all the messages (because the expensive ones aren't the layout messages, but because MSAA/WinEvent infrastructure is very old... registering for any one registers you for all of them.)

The answer to this is "no". We'd have to make categories for each event that we wanted to register separately and I'm not sure there are enough category flags left. Also, no one wants to touch this given it's legacy tech. We will need to drive improvement of this through the other options.

So, the upshot of this is that any time NodeJS is running (e.g. Visual Studio) then all console windows suffer an approximate 20% slowdown (or worse) due to accessibility eventing broadcasts. Urgh.

The first quick fix for this (GDI measurement caching) just went out with insider build 18932!

I wonder how this issue looks now?

Not too bad -- the slowdown at the start was when I enabled the win+g gamebar recording...
https://user-images.githubusercontent.com/1844001/112709165-3a9f8d80-8e8d-11eb-823a-3a680028bb52.mp4

@zadjii-msft Should we close this issue in favor of #10462? The rendering performance was fixed with AtlasEngine, now only the VT performance is subpar and #10462 seems more "specific".

@lhecker, if you profiled this scenario with chafa and all that's left is the VT engine being the super hot path, I'm OK with that. Chafa's just a good test and we shouldn't forget about using it in 10462 then.

Yea I dig it. Let's focus our efforts there for remaining improvements to make in the space.


⚠ ClamAV has degraded performance

In 8027252, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 660 ms

Resolved: ClamAV performance has improved in 0d5ef49.


Dark Font Palette Issue

Hello @vishanurag !!
I hope this message finds you well. I would like to address an issue I encountered on the website and propose a solution that I believe will enhance the user experience:
Issue Description:
The palette's font colour is too dark, which can cause readability issues for users. Dark fonts on certain backgrounds or low-contrast environments may lead to eye strain, making it difficult for users to engage with the content. This can negatively impact user experience, especially for those with visual impairments or when viewed in different lighting conditions.
Proposed Solution:
To resolve the issue of the font being too dark, I will adjust the palette by lightening the font colour to create better contrast with the background. This will make the text easier to read, improving the overall user experience.
Image/Video:
Adding a visual so that you can easily review and verify the issue.

If you find this proposal suitable, I would appreciate it if you could assign these tasks to me.

@vishanurag thnku for assigning it to me anurag but i have some query by any chance are u checking the pr before merging it as i clone the repo just now and checked the pallete to draw is whole gone and pallete for choosing brush and color are now twice at home page.
screen-capture.webm

@vishanurag hope you will reply as my assigned issue was to chng the dark font for the pallete and now they are clearly two palletes for choosing brush n all

@vishanurag thnku for assigning it to me anurag but i have some query by any chance are u checking the pr before merging it as i clone the repo just now and checked the pallete to draw is whole gone and pallete for choosing brush and color are now twice at home page.
screen-capture.webm

@snehhhcodes
I am extremely sorry for the inconvenience. Someone might have created this problem while resolving the merge conflicts.
Sometimes it's really hard to check every single mistake or bug created by contributors because there are so many people contributing and it's really hard to go through every problem as a PA because there is no other maintainer of this project other than me. Hope you understand.
You can still contribute to this project.
Sorry for the inconvenience happen with you. 🙏

@vishanurag thnku for assigning it to me anurag but i have some query by any chance are u checking the pr before merging it as i clone the repo just now and checked the pallete to draw is whole gone and pallete for choosing brush and color are now twice at home page.
screen-capture.webm

@snehhhcodes I am extremely sorry for the inconvenience. Someone might have created this problem while resolving the merge conflicts. Sometimes it's really hard to check every single mistake or bug created by contributors because there are so many people contributing and it's really hard to go through every problem as a PA because there is no other maintainer of this project other than me. Hope you understand.
You can still contribute to this project. Sorry for the inconvenience happen with you. 🙏

should i feix it too if you can change the label to level2 i can do it!!

Closed via #244


Update UEFIRequirements.md

duplicate up to date boot logo replacement info from os customizations page

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

windows-iotcore/commercialize-your-device/UEFIRequirements.md
:white_check_mark:Succeeded

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

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

windows-iotcore/commercialize-your-device/UEFIRequirements.md
:white_check_mark:Succeeded

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


Change video constraints on the fly

We are planning a large number of connections with video tracks. And the more connections, the smaller the size of all videos. How is it possible to change constraints media track on the fly so as not to waste user traffic?
Method MediaStreamTrack.applyConstraints() as I see it is not implemented. So I'm wondering if there is a workaround?

MediaStreamTrack in werift is just a control interface for RTP, so it does not support media control.
If you want to control the media, please control it on the client (browser or GStreamer or etc...) that is generating the media.


✨ Collection - Global

Create the Tina collection for the home page

[ ] A Global Collection is created
[ ] A Header Collection is created
[ ] #1504

Rules doesn't really have a "Home Page". But we can use a Global collection for things like SEO.

Closing as this will happen naturally once we need the collection


'Select All' and 'Deselect All' are missing in Filters 'Health status...'

Home - Hosts - Filters
Here user can select each status, SAP System, Tags, but 'Select All' and 'Deselect All' are missing in Filters 'Health status...'

Replicated in v0.6.0 (cycle 2). Given that in the Health Section only three values are possible (Passing, Warning, Critical) having Select / Deselect All doesn't help much. Low priority.,

Hey @abravosuse ,
Do we still want to implement this?
It is super easy to do from the tech standpoint

Not in my opinion @arbulu89. It's just three values to select from: select all, deselect all won't make a big difference.

Closing them


static methods of alert and dismissCurrent

This is alternative to documenting how to global dismiss which is implementation detail.
This also adds static method of alert so we dont have to do const blah = new Dialog(); .set .show, this is more inline with javascript and also RN Alert module - https://facebook.github.io/react-native/docs/alert.html
Usage is like this:
    MaterialDialog.alert({
        content: 'MaterialDialog Downloading no reference...',
        positiveText: null,
        negativeText: null,
        progress: {
            indeterminate: true,
            style: 'horizontal'
        },
        widgetColor: CONFIG.backcolor,
        cancelable: false
    });
    setTimeout(MaterialDialog.dismissCurrent, 5000);

I added these methods to a proposed overhaul of the API - https://github.com/aakashns/react-native-dialogs/pull/82
This overhaul is in line with RN APIs:

https://facebook.github.io/react-native/docs/alert.html
https://facebook.github.io/react-native/docs/datepickerandroid.html


Fix camera for simulator using modeled trajectories

Fix camera leaving scene and reenable it to work with the generated trajectories from the trained model

Solved at commit 03e6143, still no control over camera leaving scene


Adjust python dependencies

The testing code depends on scipy, but that's not mentioned in Requirements.md or requirements.txt. (The code also depends on matplotlib, which is mentioned in Requirements.md, but not requirements.txt.)
(This won't be an issue for normal people who just use Anaconda instead of miniconda...)

It's probably worth pinning packages too: https://conda.io/docs/user-guide/tasks/manage-pkgs.html#preventing-packages-from-updating-pinning
(requirements.txt specifies exact package versions, but conda update doesn't respect that.)

With having set up Travis CI, we've gone to using the environment.yml to specify the packages that are actually required (and if we don't keep it updated, the tests fail) which seems like a much more reliable way to make sure we're tracking dependencies. I also just recently removed the pinned versions from that file. Is this a good idea, or a bad idea?  Some of the pinned versions were getting significantly out of date, and everything seems to work with the newest "default" channel releases of all the packages, and I'd rather hear one at a time, gradually, as packages update and potentially break things.  But at the same time I don't want a potential user to have to deal with that. Is this normally dealt with through releases, which pin required versions, for a given release, while allowing the development to continue happening on the most recent packages?

Relying on Travis sounds great! I think the common practice is to just require the dependency, or require some version of the dependency >= something. Here's an example from dask-ml: https://github.com/dask/dask-ml/blob/master/setup.py#L13
I was wrong before, conda does allow updates that still comply with the requirements: conda env update -f environment.yml.


[Enhancement] Extraction of constants [ExporterConfiguration]

Search before asking

[X] I had searched in the issues and found no similar issues.

Enhancement Request

Describe the solution you'd like
Located at:eventmesh-trace-plugin/eventmesh-trace-api/src/main/java/org/apache/eventmesh/trace/api
We can extract the constants from the image into EventMeshTraceConstants.
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

I want to choose this issue as my first contribution to the EventMesh community.

I want to choose this issue as my first contribution to the EventMesh community.

Welcome to the eventmesh community, this pr will be merged in the next version.


Does perspective transform support source points outside the image dimensions

Hi,
I was experimenting with homography transform and one issue, I was consistently getting different results as compared to opencv. in warp perspective where the calculation of 'M' was correct.
Before I check my code or discuss more about this issue, could you please answer if the source points outside image dimensions are supported ? Because I have such "source" points which are supported in OpenCV but not in TGM ?

In theory yes, however the sampler will fill it with zeros by default but can play with the padding mode parameter from the sampler. Could you provide us a short snippet reflecting such case?

In theory yes, however the sampler will fill it with zeros by default but can play with the padding mode parameter from the sampler. Could you provide us a short snippet reflecting such case?

Okay, i have to upload a test image. Please download this npz image
https://transfer.sh/F6Kny/tgm_testimg.npz
and then this code snippet to compare the output of opencv and tgm.
import cv2
import numpy as np
import matplotlib.pyplot as plt
import torch
import torchgeometry as tgm

npz_file = np.load("testimg.npz")
gt_img = npz_file['arr_0']
points_src = np.array([[425.66718, 89.],[-3292.292,256.6667],[4270.3945 ,256.6667], [ 0.,89.]], dtype=np.float32)
points_dst = np.array([[ 852., 0.],[-426.,240.],[ 852., 240.],[-426., 0.]], dtype=np.float32)
shape = (426, 240)
M = cv2.getPerspectiveTransform(points_src, points_dst)
warped_gtimg = cv2.warpPerspective(gt_img, M, shape, cv2.INTER_LINEAR, cv2.BORDER_CONSTANT, 0)
plt.imshow(warped_gtimg)

#################### tgm #####################
gt_img_ts = torch.from_numpy(gt_img).unsqueeze(0).unsqueeze(0)
points_src_ts = torch.from_numpy(points_src).unsqueeze(0)
points_dst_ts = torch.from_numpy(points_dst).unsqueeze(0)
M = tgm.get_perspective_transform(points_src_ts, points_dst_ts)
img_warp = tgm.warp_perspective(gt_img_ts.float(), M, dsize=(240, 426))
print(img_warp.size())
np_warped_gtimg = img_warp.cpu().numpy()[0][0]
plt.imshow(np_warped_gtimg)

@Msabih thanks. I'll investigate that

@edgarriba  were you able to reproduce the issue ?

@Msabih yes, I run your script and I get an empty image.

@edgarriba  and the opencv gives lines correctly. I will also try to debug this. Any hints will be welcome. Thanks for your time.

@Msabih I'd like to reproduce the bug, but the provided .npz file is expired

@jiangwei221 thanks to help with that, I'm still working on infrastructure stuff and couldn't spend time on this. I can still download the file.

Caused by #146 .

Is it solved?

@jiangwei221 @Msabih can we close this ?

I think this is solved. Closing.


Passing in additional classes

<DatePicker
   value={this.props.date}
   onChange={this.changeDate}
   clearButtonElement={'Today'}
   calssName="form-control u-input"
 />

Not work as my mind, How to pass additional classes?

'className' not 'calssName'

@weiyuan1993  Thanks, Sorry for my Typo