Handling/Recovery when Gecko units are disconnected (RF Error, tub powered off for maintanence)

Version of the custom_component
Version 0.0.5 - from manifest.json
Version 0.0.1 - from const.py
With local modification to line 45 of iniy.py from issue #
Original
            facade = await locator.get_spa_from_identifier(spa_identifier).get_facade(False)

Changed to
            spa = await hass.async_add_executor_job(locator.get_spa_from_identifier, spa_identifier)
            facade = await hass.async_add_executor_job(spa.get_facade, False)

Configuration
NA
Describe the bug
The custom component generates an error on startup if the Gecko units aren't connected (RF error).  (My tub was temporarily powered down for maintenance.)
Since no other errors have been generated, and things haven't changed, it doesn't look like it is going to poll at some point and recover. I've only waited 15 minutes so far.
So I'm guessing I'm going to have to restart Home Assistant to get the integration to reconnect.
Debug log

2021-06-12 11:04:13 WARNING (Thread-6) [geckolib.driver.udp_socket] Couldn't find new handler for b'RFERR'
2021-06-12 11:04:15 WARNING (Thread-6) [geckolib.driver.udp_socket] Couldn't find new handler for b'RFERR'
2021-06-12 11:04:17 WARNING (Thread-6) [geckolib.driver.udp_socket] Couldn't find new handler for b'RFERR'
2021-06-12 11:04:19 WARNING (Thread-6) [geckolib.driver.udp_socket] Couldn't find new handler for b'RFERR'
2021-06-12 11:04:21 WARNING (Thread-6) [geckolib.driver.udp_socket] Couldn't find new handler for b'RFERR'
2021-06-12 11:04:23 WARNING (Thread-6) [geckolib.driver.udp_socket] Couldn't find new handler for b'RFERR'
2021-06-12 11:04:25 WARNING (Thread-6) [geckolib.driver.udp_socket] Couldn't find new handler for b'RFERR'
2021-06-12 11:04:27 WARNING (Thread-6) [geckolib.driver.udp_socket] Couldn't find new handler for b'RFERR'
2021-06-12 11:04:29 WARNING (Thread-6) [geckolib.driver.udp_socket] Couldn't find new handler for b'RFERR'
2021-06-12 11:04:39 ERROR (Thread-5) [root] Uncaught thread exception
Traceback (most recent call last):
  File "/usr/local/lib/python3.8/threading.py", line 932, in _bootstrap_inner
    self.run()
  File "/usr/local/lib/python3.8/threading.py", line 870, in run
    self._target(*self._args, **self._kwargs)
  File "/usr/local/lib/python3.8/site-packages/geckolib/spa.py", line 282, in _ping_thread_func
    self.refresh()
  File "/usr/local/lib/python3.8/site-packages/geckolib/spa.py", line 311, in refresh
    if not self.is_connected:
  File "/usr/local/lib/python3.8/site-packages/geckolib/spa.py", line 306, in is_connected
    raise RuntimeError("Spa took too long to connect ...")
RuntimeError: Spa took too long to connect ...
2021-06-12 11:04:39 ERROR (MainThread) [custom_components.gecko] Exception during entry setup
Traceback (most recent call last):
  File "/config/custom_components/gecko/__init__.py", line 49, in async_setup_entry
    while not facade.is_connected:
  File "/usr/local/lib/python3.8/site-packages/geckolib/automation/facade.py", line 68, in is_connected
    if not self._spa.is_connected:
  File "/usr/local/lib/python3.8/site-packages/geckolib/spa.py", line 306, in is_connected
    raise RuntimeError("Spa took too long to connect ...")
RuntimeError: Spa took too long to connect ...

@scope666 -    My understanding, based on my own observations, is RFERR is an error reported by the Gecko unit connected to the network when doesn't have a radio connection (RF) to the unit in the spa.  For example that's the error that shows up if the spa is temporarily powered down (the spa side unit no longer has power so no RF connection.)
I wouldn't expect you to have a solid blue light at that point. I saw you mentioned in the other issue that you did.  Perhaps your RF connection is marginal?     (If that's the case repositioning the gecko unit might help.

Yeah, I think something must have introduced new RF interference and it was blocking communications.   I grabbed an ethernet bridge and got the transmitter as close to the tub as possible, this seemed to fix it.

@Scope666 - Did you have to restart Home Assistant to get it to reconnect with the Gecko units? In my limited experience it seemed the integration never recovered from the RF error.

Yeah, I think something must have introduced new RF interference and it was blocking communications.

Besides distance and things that are in the way, one of the less obvious factors that can affect reception is proximity to other devices that generate RF noise. The noise, (RF interference or RFI), can drown out the other device and prevent it from being heard. For example, Raspberry Pis (especially in plastic cases) are pretty noise, so it's good to keep them away from receivers.

@Scope666 - Did you have to restart Home Assistant to get it to reconnect with the Gecko units? In my limited experience it seemed the integration never recovered from the RF error.

Yeah, I think something must have introduced new RF interference and it was blocking communications.

Besides distance and things that are in the way, one of the less obvious factors that can affect reception is proximity to other devices that generate RF noise. The noise, (RF interference or RFI), can drown out the other device and prevent it from being heard. For example, Raspberry Pis (especially in plastic cases) are pretty noise, so it's good to keep them away from receivers.

Yes, once I moved the transmitter, I had to restart HA to get the integration working again.  Another thing I noticed, if you set a filter schedule on the tub itself, the temp sensor won't work.  I had to go back in an select one of my profiles in the mobile app to get that to work.

@Scope666 interesting, that gives me some pointers to work on, TBH I've not handled any tub errors at present but I think it's important for the integration to recover cleanly from faults etc. Well I mean the underlying library actually as we don't do protocol in HA.

v0.1.0 ought to handle this situation now, please re-open if there are other issues


OpenGL compatibility via metaclass: Matrix, DecimalNumber, Variable

Changelog / Overview

This PR is a continuation of #1572, and adds OpenGL compatibility for Matrix, DecimalNumber, Variable

Documentation Reference

Testing Status

Tested two scenes that contain the modified mobjects with both renderers, and they worked fine.
Further Comments

Matrix mobject needs the modified VGroup from #1572 to work.
Checklist

[x] I have read the Contributing Guidelines
[x] I have written a descriptive PR title (see top of PR template for examples)
[x] I have written a changelog entry for the PR or deem it unnecessary
[ ] My new functions/classes either have a docstring or are private
[x] My new functions/classes have tests added and (optional) examples in the docs
[x] My new documentation builds, looks correctly formatted, and adds no additional build warnings

Reviewer Checklist

[ ] The PR title is descriptive enough
[ ] The PR is labeled correctly
[ ] The changelog entry is completed if necessary
[ ] Newly added functions/classes either have a docstring or are private
[ ] Newly added functions/classes have tests added and (optional) examples in the docs
[ ] Newly added documentation builds, looks correctly formatted, and adds no additional build warnings

I am not so sure about changing (in matrix.py)
return np.vectorize(self.element_to_mobject)(
    matrix, **self.element_to_mobject_config
)

To a pure array, since such a change potentially breaks users' code who assume mob_matrix to be an np array. I found that
return np.vectorize(self.element_to_mobject, otypes=[object])(
    matrix, **self.element_to_mobject_config
)

Prevented an error while still allowing for an np array.
Also, I wonder if it is necessary to have an OpenGLValueTracker. Can we not just do the same as with VMobject and have a metaclass MetaMobject for classes that extend Mobject? I have implemented this and can create a PR if this seems like a good idea.

I am not so sure about changing (in matrix.py)
return np.vectorize(self.element_to_mobject)(
    matrix, **self.element_to_mobject_config
)

To a pure array, since such a change potentially breaks users' code who assume mob_matrix to be an np array. I found that
return np.vectorize(self.element_to_mobject, otypes=[object])(
    matrix, **self.element_to_mobject_config
)

Prevented an error while still allowing for an np array.

I think the matrix_to_mob_matrix function is intended to be internal. It is used only once in the Matrix mboject with an intermediate variable. However, since you provided a solution, I will go with it. Thank you!

Also, I wonder if it is necessary to have an OpenGLValueTracker. Can we not just do the same as with VMobject and have a metaclass MetaMobject for classes that extend Mobject? I have implemented this and can create a PR if this seems like a good idea.

I see that you created a PR. Once the PR is approved, we can remove the OpenGLValueTracker and the changes made here.


There is an issue with nuxt module that imports files (css, plugins) when using yarn link or npm link.

Environment

Operating System: Windows_NT
Node Version:     v18.7.0
Nuxt Version:     3.2.2
Nitro Version:    2.2.3
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      modules
Runtime Modules:  my-nuxt-module
Build Modules:    -

Reproduction
How to reproduce:
Create a nuxt module with the example of injecting plugins within nuxt documentation:
Execute npm link or yarn link on newly created nuxt module.
Create a nuxt app project importing this nuxt module via npm link  or yarn link  - add in nuxt.config → modules
Start nuxt app
Describe the bug
There is an issue with nuxt module that imports files (css, plugins) when using yarn link or npm link.
Additional context
No response
Logs
ERROR  Failed to load url /@fs/D:/my-nuxt-module/runtime/plugin.js (resolved id: D:/my-nuxt-module/runtime/plugin.js) in virtual:nuxt:D:/nuxt-app/.nuxt/plugins/client.mjs. Does the file exist?

Would you provide a reproduction? 🙏

@danielroe We couldn't provide a reproduction due to yarn link not working on stackblitz.
As a workaround we are editing the nuxt module directly in node-modules.

@danielroe Here is a simple reproduction:
https://github.com/denys119/reproduction

Install dependencies for nuxttestmodule and nuxtapp.
Go to nuxttestmodule and use yarn link.
Go to nuxtapp and use yarn link "nuxttestmodule" and run yarn again.
Start the nuxt app using yarn dev.
The error from logs should be displayed.

Any news about this issue?

would appreciate to be able to use yarn link too!

Could you try adding the following to your nuxt.config.js (seems to work on my end) @denys119 :
export default defineNuxtConfig({
	vite: {
		resolve: {
			preserveSymlinks: true
		}
	},
	...

@grexen Thank you, I will try


Add a simple evaluator

The following snippets of mylang code should all return 42.
return.my
return 42;

let_and_call.my
let f = func (a) { return a; };
return f(42);

Closed by #58


Changes to enable support for deploying Kubernetes pods with Sysbox.

These changes were already reviewed by @rmolina on the sysbox enterprise repo.


Add crawler functionality for identifying sites' usage of GPP 1.0 vs 1.1 and write to database

GPP 1.0 is no longer supported. If a site is broadcasting a GPP 1.0 signal, other entities on the page (eg Prebid.js or Google Ad Manager) generally will not understand it. You should just fail any site providing an API that no one understands. At Prebid, we're removing support for reading GPP 1.0 signals entirely and GAM already has.

Thanks, @patmmccann!
@katehausladen already looked into this issue. We will further evolve our code to reflect people's move from GPP v1.0 to v1.1.
@franciscawijaya, can you take the lead on this issue and implement the functionality @katehausladen described and as outlined below for our June crawl? @Mattm27, can you work with @franciscawijaya as needed to bounce off ideas and discussion? And, @katehausladen, it would be great if you were available for any questions that @franciscawijaya and @Mattm27 still have remaining and general observations to make sure we are not making any mistake here 😄.
What we need before the June crawl is logic for:

Identifying whether a site implements GPP v1.0 or v1.1
Unpacking and storing both GPP v1.0 and v1.1 values in our database (let's write a new column to our crawl data after the z column "gpp_version")

@katehausladen prepared this move already in the analysis.js. So, @franciscawijaya, that file is a good starting point together with @katehausladen's description. Please go ahead and create a new issue-110 branch to start the implementation ...
Once we have a record of which site is using which version, we can interpret the results in our analysis after the crawl accordingly.

You can see if GAM understood the gpp string by looking in the payload of the network requests it makes to itself. This is an example of a success on the call (filter to gampad in network tab)

However, you might often see errors in this location for sites using gpp 1.0, which GAM and their recipients are treating as opt in (same as no signal)

Identifying whether a site implements GPP v1.0 or v1.1

After reading more onto GPP string and CMP API, it seems that the one with an update to 1.1 version is the CMP API which captures the information of the GPP string

I have also confirmed with Kate that our current code looks for version 1.1 first and then 1.0. It then proceeds to store only one string value. While the string value would be the same, the difference would lie in how we access the value (i.e getGPPdata function in v1.0 or just the ping function in v 1.1. (Reference: https://github.com/privacy-tech-lab/gpc-web-crawler/issues/60#issuecomment-1664621751)

You should just fail any site providing an API that no one understands. At Prebid, we're removing support for reading GPP 1.0 signals entirely and GAM already has.

Hypothesis as of now:
I think the problem with this is then these entities are looking for the newly merged ping function from the ver 1.1 CMP API, instead of the getGPPdata value from the v1.0

from https://github.com/privacy-tech-lab/gpc-web-crawler/issues/60#issuecomment-1664621751

@patmmccann Could we clarify if, by changes in the GPP versions, you were referring to the changes in the CMP API versions? Currently, according to IAB, there is only one GPP version (1.0) but there are 2 CMP API versions (1.0 and 1.1) [CMP API captures the information of the GPP]

Identifying whether a site implements GPP v1.0 or v1.1
Unpacking and storing both GPP v1.0 and v1.1 values in our database (let's write a new column to our crawl data after the z column "gpp_version")

Action plan on our end:

[ ] We will be adding a new column in the crawl data that indicates whether the site uses CMP API v1.0 (one that has a getGPPdata function) or v1.1 (one that only uses ping function)
[ ] Training data (10-20 sites)

@patmmccann Could we clarify if, by changes in the GPP versions, you were referring to the changes in the CMP API versions? Currently, according to IAB, there is only one GPP version (1.0) but there are 2 CMP API versions (1.0 and 1.1) [CMP API captures the information of the GPP]

Yes the CMP API version 1.1, which we probably should have called 2.0 but oh well. https://github.com/InteractiveAdvertisingBureau/Global-Privacy-Platform/pull/70
cc @iamrowena

We will be adding a new column in the crawl data that indicates whether the site uses CMP API v1.0 (one that has a getGPPdata function) or v1.1 (one that only uses ping function)

I suggest you get the version out of the ping response instead of testing for the absense of getgppdata. Some commercial vendors, eg @janwinkler, have backported getgppdata to assist in transitions yet still conform to the 1.1 spec and would have a signal recognized by platforms gathering the signal with the newly formatted eventlistener model.

Thank you @patmmccann! Your insight was very helpful in guiding the steps that I need to take to enhance the crawler functionality.
A note to self:
I've confirmed that our current code do not test the version based on the absence of getgppdata. Instead, our injection script is modeled upon the update to the version 1.1 (ie. callback takes precedence over a return value since v 1.1 removed return values in favor of callback functions).
This would prioritize the 1.1 spec since all default gpp functions (including ping and getGPPData) used return values in v1.0 now have callback functions in v1.1. On the other hand, v1.0 would return values as expected, with some executing callback functions and some don't. Hence, sites would fall into 3 categories:
v1.1 : callback only
v1.0: executes callback and returns value
v1.0: returns value only

In order to add the column in the crawler data, I believe these are the steps I should take:

adding new column in the rest-api (in the app.post in index.js)

explicitly collect the data of the different versions (which I believe has been analyzed under the function runAnalysis -- it is actually posted to debug)

log/store the data in the extension while a site is being analyzed (this is what analysis_userend[domain] is for)

LogData is a function in analysis.js that parses the incoming data and puts it into an object (namely analysis_userend, which has an entry for each domain).
I can model it like this:

populate the database with the collected information.

This would be done automatically as long as the data is in the analysis_userend[domain] because after analysis has finished for a particular site, the data in analysis_userend[domain] is posted to the database.

Logs/Update on adding the new column:

I added a new variable for gpp_version before and after GPC on index.js (which should add a new column on the Crawl Data)
logData the gpp_version under the runAnalysis and haltAnalysis (which should collect the data for the gpp_version)
Added the new two variables under the analysisUserendSkeleton and analysisDataSkeletonFirstParties functions
Under logData function, wrote an if statement for GPP_version to parse and put into objects gpp_version_before_gpc and gpp_version_after_gpc

A side note: while figuring out the code for the addition of the gpp_version column, I also encountered some questions about some functions in analysis.js that I need to clarify and am currently asking Kate about it.
Next step: I will be repackaging the gpc-analysis-extension into xpi file and test the extension locally before making a commit.

Excellent!

Update: After successfully repackaging it to xpi-file, I ran the analysis. Unfortunately, it gave me null values for the gpp version. I also tried to debug using the debug column and the code actually managed to analyze which gpp version the site is using, however it still fails in storing and printing it in the analysis column.

I am currently taking another approach in the logic: instead of checking the version both before and after gpc signal is detected, I'm trying to write the code for collecting just one gpp version (regardless of whether it is after or before gpc signal is detected).

I've successfully added the code to identify the gpp version that a site is using, collecting the data of the gpp version and store that data in the new column (gpp_version). The result of the crawler on a site will be as attached below.

Next step:
I have tested for 2 sites while writing and testing the code. I will begin testing for a slightly bigger sample size (10-20 sites) to ensure that the gpp version that the sites are using are recorded properly.

Excellent! Well done, @franciscawijaya!

Using the April Crawl Data, I tested the crawl for sites that output GPP strings (as tested in April) to check the gpp-version. Out of the 20 sites that I picked from the data, it seems that all of them used v1.1 and that data is reflected in the column gpp-version accurately. I also tested on sites that do not output GPP strings before and after the gpc signal is sent and as expected the column would reflect a 'null' value for gpc_version, since their gpp_before_gpc and gpp_after_gpc would also output a 'null' value.
In my testing and debugging of 20 sites, I have yet to encounter a site (that was crawled and identified to have a GPP string in April Crawl) that uses the v1.0. I'm not sure if this indicates and confirms that most sites have switched to the v1.1.
While I'm thinking of continuing my manual testing of other sites from the site list that had gpp strings in April Crawl to make sure of this switch, I wonder if there is a way for me to get a hold of sites that are still using v1.0 right now and test those site instead of going through our site list.

I wonder if there is a way for me to get a hold of sites that are still using v1.0 right now and test those sites out, instead of going through our site list.

I tried searching BuiltWith to find sites with GPP. But it does not detect GPP. Maybe, there are similar lead generation sites like BuiltWith that do, though.
Another option may be to try the Internet Archive and Archive.today to see if they store sites with all their third parties.
It is also possible to create your own site with GPP v1.0. But let's not go there unless it is absolutely necessary.

Other than that, Google search for GPP v1.0 code snippets may get some relevant search results.

@patmmccann Hello! Would you mind sharing sites that still use the v1.0 when you came across this issue? My sample sets of sites seemed to have switched to 1.1 but I'm currently still looking to test our crawler for sites that still use the v1.0 version. I would greatly appreciate any help. Thank you in advance!

I've tried using web.archive to check sites before the release of GPP v1.1 to see if I can test the gpp version (which should be v1.0). However, I was not able to do this as web.archive does not seem to store sites with their third parties. I confirmed this by comparing the current site and the archive version on the web console which showed that the current sites stores a GPP string while the web.archive version does not.

I have also tried testing it on some other sites on our crawl list but I have yet to encounter v1.0. For now, I think we can just go ahead to try this code for the June crawl.
Next step: I will soon request to merge this branch with main and ask Matt to hep confirm reviews the code on his end via the local test and then close this issue. I will also be preparing for the crawl and began by this weekend.

Sounds all good, @franciscawijaya!

I am having trouble tracking down some of the old gpp implementations at the moment. Perhaps other outreach has been quite successful!

Merged to the main branch!

GPP 1.0 không còn được hỗ trợ. Nếu một trang web đang phát tín hiệu GPP 1.0 thì các thực thể khác trên trang (ví dụ: Prebid.js hoặc Google Ad Manager) thường sẽ không hiểu được tín hiệu đó. Bạn sẽ không thành công với bất kỳ trang web nào cung cấp API mà không ai hiểu được. Tại Prebid, chúng tôi sẽ loại bỏ hoàn toàn hỗ trợ đọc tín hiệu GPP 1.0 và GAM đã có.

Không mở được


segfault in toml::v2::impl::ex::parser::parse_value

Whilst experimenting with the example toml_to_json_transcoder built from commit dca694, we discovered a file which triggers a segfault:
0x0000555555594f54 in toml::v2::impl::ex::parser::parse_value (this=0x7fffffffbdf0)
    at /root/tomlplusplus/include/toml++/toml_utf8.h:892
892                     return codepoint <= U'\u001F' || codepoint == U'\u007F';

crash.zip

Wow, that file is nothing if not pathological. The parser uses recursive descent; input like that will cause a stack overflow. How did you come to use this input? Was it real, or from something like a fuzzer?

Oh, never mind, I just did a quick scan of your profile; I'm going to guess it was from fuzzing :)
Welp, guess I better put some pathological case detection into the parser's descent logic...

You're welcome, the input was derived from an actual file, but after about 10 million iterations it's no longer recognizable. We found a few bugs in similar parsers, so we wanted to cover all of the more popular packages like yours too.

Awesome work!
Also, turns out this actually exposed two bugs:

Fixed in master. Thanks again @geeknik!

Fix confirmed. Will let you know if we encounter anything more serious. 👍🏻


Allow users to add  tags to site pages

Currently, a single page may have multiple URLs:

www.example.com/test
www.example.com/index.php/test
www.example.com/?cID=123
www.example.com/index.php?cID=123

That's not good for search engines: they say duplicated contents at different URLs.
Let's tell them that all these URLs are for the same page (www.example.com/test)

This looks good to me, @aembler what do you think?

@mlocati resolve conflicts and I think this is good to merge.

resolve conflicts

Done

@mlocati Thank you for this feature. We're facing a different issue.
For example, we have the following URL.
https://example.com/action/param
But when there is a reference link from facebook/twitter/google it adds some parameters like-
https://example.com/action/param?utm_content=buffer539cd3&utm_medium=social&utm_source=twitter.com&utm_campaign=buffer
So, the canonical URL also get the same value. Our client wants to remove this get parameters from the canonical URL. I know you already have added an option excluded_querystring_parameters. But I think it would be a long list for our client. What could be the best solution for it then?

The only solution I see is to add utm_content, utm_medium, utm_source, utm_campaign and so on to the value of the seo.canonical_tag.excluded_querystring_parameters site configuration key...

Thank you.

The only solution I see is to add utm_content, utm_medium, utm_source, utm_campaign and so on to the value of the seo.canonical_tag.excluded_querystring_parameters site configuration key...

Thank you @mlocati , but if so, we should add some get parameters from popular search engines and social networking sites into the default config file, didn't it?

@hissy what about creating a file located at /application/config/site.php with this contents?
<?php

return [
    'sites' => [
        'default' => [
            'seo' => [
                'canonical_tag' => [
                    'excluded_querystring_parameters' => [
                        'cID',
                        'ccm_token',
                        'utm_content',
                        'utm_medium',
                        'utm_source',
                        'utm_campaign',
                    ],
                ],
            ],
        ],
    ],
];

@mlocati Yeah, I can do that because I know about concrete5 config system well, but most of the beginners are not.

I agree.
I'd collect the various querystring parameters defined by the different search engines/ads systems (Google, Bing, Facebook Pixel, ...) and add them to the default list.

We should ignore query and search_paths. Search block uses these parameters, so it will be attached to the canonical URL on the search result. It's annoying.

I'd collect the various querystring parameters defined by the different search engines/ads systems (Google, Bing, Facebook Pixel, ...) and add them to the default list.

@mlocati Any updates on this issue? Should I create a new open issue about this?

Should I create a new open issue about this?

I think it's a good idea: comments in merged pull requests aren't a great place to write about new issues, since they are somehow "hidden"


[google_sign_in_web] Remove deprecated signIn method.

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Use case
In order to facilitate the migration from the old Google Sign In SDK to the Google Identity Services SDK, the signIn method of the google_sign_in_web plugin attempted to both authenticate and authorize users and return both tokens.
The way this worked was by using the oauth2 authorization process, and then doing a request to the People API to retrieve information about me (the currently signed-in user). This didn't return a proper idToken (authentication) so the method wasn't really usable by users.
Proposal
Remove the signIn method from the web implementation of google_sign_in, so it starts throwing an UnimplementedException, now that users have migrated to the new Card / SDK Button method.
Remove all the accessory code that was used to attempt to retrieve the information about the currently signed-in user, like _computeUserDataForLastToken, _requestedUserData and any others.
Update the documentation to note that the plugin does not require access to the "People API" any longer.
Bump a major version on the plugin, so we need to update the core plugin (and its documentation) to release this deprecation to the public.

@ditman I want to start this task, how can I help? Where can I code?

Thanks @BerkSpar!
If you haven't touched code in flutter/packages yet, please take a look at the specific contributing guide, readme, etc, here:

https://github.com/flutter/packages/blob/main/CONTRIBUTING.md

The code for the web implementation of the package:google_sign_in package (package:google_sign_in_web) lives here:

https://github.com/flutter/packages/tree/main/packages/google_sign_in/google_sign_in_web


feat(README): Improving the install README

Issue #, if available:
Description of changes:
Update install readme
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@tomjuraska let me know if you are fine with it

Looks good, thanks.


QOL API methods

While we're contributing, we were wondering if you had any interest in any of the following quality of life methods we've added to our wrapper for ClassicLevel:

compactFull - Convenience function that discovers the first and last key and calls compactRange
size - Convenience function that discovers the first and last key and calls approximateSize
has - Definitely a feature that would be better if it existed upstream, but iterates keys until specified key is discovered (if at all)

I'm +-0 on the first two, as discovering first and last key is cheap, and it's convenient indeed. However it's also simple enough that it could live in userland.
-1 to the third method, because it means a full table scan and in many scenarios where you need something like leveldb (have a lot of data) isn't actually what you want to do

Agree with @juliangruber on the first two.
Not sure I understand the third method. Can you give an example?

I imagine it's something like https://github.com/juliangruber/level-exists/blob/master/index.js

Let's assume that's the case (shouldn't require iteration though). That's covered by https://github.com/Level/community/issues/106 so I'll close this.


Code-insiders crashes on launch

VS Code Version: 1.54.0-insider
OS Version: Ubuntu 20.04

Steps to Reproduce:

Launch vscode ( I used code-insiders -n --disable-extensions --sync=off --verbose)
wait 3-5 seconds...

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

I ran code-insiders --crash-reporter-directory <dir>, and zipped the resulting file: dmpfile.zip. Also, here's the log from the console:
vscode-crash.log.

I downgraded to version 1.52.0-1607352923, and it still crashes. I was previously using code-insiders via snap, and then I removed the snap and installed the .deb package. Crashes happened soon after that.

Can you please follow the steps in https://github.com/Microsoft/vscode/wiki/Native-Crash-Issues to get at more details around the crash and attach the result here? Thanks!

Thanks for the dump, crash is from the vscode-encrypt module. Merging to https://github.com/microsoft/vscode/issues/116690


fix eip projcet source & snapshot bugfix

这个 PR 实现什么功能/修复什么问题:

UCloud snapshot bugfix & disk project src bugfix

是否需要 backport 到之前的 release 分支:

release/2.10.0

Codecov Report

Merging #1435 into master will decrease coverage by 0.01%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master   #1435      +/-   ##
=========================================
- Coverage    7.72%   7.71%   -0.02%
=========================================
  Files         409     408       -1
  Lines       62607   62390     -217
=========================================
- Hits         4838    4811      -27
+ Misses      57362   57175     -187
+ Partials      407     404       -3

Flag
Coverage Δ

#aFlag
7.71% <0%> (-0.02%)
:arrow_down:

Impacted Files
Coverage Δ

pkg/util/ucloud/storagecache.go
0% <0%> (ø)
:arrow_up:

pkg/util/ucloud/disk.go
0% <0%> (ø)
:arrow_up:

pkg/util/ucloud/snapshot.go
0% <0%> (ø)
:arrow_up:

pkg/compute/guestdrivers/ucloud.go
14.28% <0%> (-2.96%)
:arrow_down:

pkg/compute/models/elasticips.go
1.4% <0%> (-0.01%)
:arrow_down:

pkg/util/ucloud/region.go
0% <0%> (ø)
:arrow_up:

pkg/httperrors/errors.go
26.28% <0%> (-6.88%)
:arrow_down:

pkg/keystone/models/users.go
2.22% <0%> (-0.22%)
:arrow_down:

pkg/keystone/models/groups.go
6.47% <0%> (-0.2%)
:arrow_down:

pkg/mcclient/options/loadbalanceragents.go
0% <0%> (ø)
:arrow_up:

... and 17 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 64ea691...675d764. Read the comment docs.

/area region

/lgtm
/approve


Invalid addon metadata causes spurious warning

This library contains two in-repo addons that are intended to support its development. They don't ship in the published npm package. But the package.json file refers to both of them even in the published case.
This means that if you try to build an ember app that has a transitive dependency on simple-html-tokenizer, you get a spurious warning:
WARNING:
WARNING: /Users/edward/hacking/css-blocks/node_modules/simple-html-tokenizer/commands:
WARNING:    Missing package directory

(Normally ember apps don't depend on simple-html-tokenizer because it's consumed as a build step when publishing glimmer-vm, but this is coming up for css-blocks because its glimmer template analyzer uses simple-html-tokenizer.)

Unfortunately, there isn't an easy way to have "dev deps" via ember-addon paths.  ember-engines has a similar issue and works around it via:

adding these as dev deps with file: relative paths
a prepublish script to symlink those into node_modules (otherwise local development sucks since npm/yarn "vendor" them into node_modules and changes aren't live)

I think these should work here too...


Input download not supported by storage

I am trying to test NCBI PGAP pipeline with cwl-test, and I got this from funnel
{"attempt":0,"error":"Input download not supported by storage: could not find matching storage system for: file:///panfs/pan1.be-md.ncbi.nlm.nih.gov/gpipe/home/badrazat/github/repos/pgap/MG37/ASM2732v1.annotation.nucleotide.1.fa\nlocalStorage: can't access file, path is not in allowed directories: file:///panfs/pan1.be-md.ncbi.nlm.nih.gov/gpipe/home/badrazat/github/repos/pgap/MG37/ASM2732v1.annotation.nucleotide.1.fa","index":0,"level":"error","msg":"System error","ns":"local","taskID":"bcr7t15bhkg3snik2pn0","time":"2018-06-29T18:46:28Z","timestamp":"2018-06-29T18:46:28.893110876Z"}
How do I tell funnel to recognize local files?

You need to configure the local storage module in funnel. Check out the docs here: https://ohsu-comp-bio.github.io/funnel/docs/storage/local/

@azat-badretdin were you able to get Funnel working?

Closing due to inactivity and since this is a Funnel config issue rather than a cwl-tes issue.


[MRG] GUI refactoring

Try to refactor the GUI code in an OP manner.

see how the Makefile is used to make the test commands consistent across different CIs and developers: https://github.com/tqdm/tqdm/blob/4f208e72552c4d916aa4fe6a955349ee8b2ed353/Makefile#L65

@chenghuzi I was looking at tqdm and it seems they also have a notebook for the tests of the notebook: https://github.com/tqdm/tqdm/blob/4591083884c8ca079ccd1192c079113c7baeadc8/tests_notebook.ipynb. I think that might be the simplest solution for us to avoid complicated frameworks ...

I think probably we can directly call this in a . py file. I'll submit a commit about this soon

How do you plan to use the class and the methods in the documentation? Let's say you are describing the part where drives must be added? You would want to:
a) activate the drive tab
b) click "add drive"
c) select drive parameters
d) show the current state of GUI (i.e., re-paint the layout/GUI)
is that something that can be done with the current interface?

@chenghuzi you need to update api.rst

We have dynamic connectivity tab now in 8c920a2b19e5d44a902ea9d618800d861d943ae3

@chenghuzi I really like how the tests are shaping up. It looks really nice. I just tried running them and noticed that plots start popping up. Do we need to set the matplotlib "Agg" backend in the test? You can see how it's done in the other tests

@chenghuzi I really like how the tests are shaping up. It looks really nice. I just tried running them and noticed that plots start popping up. Do we need to set the matplotlib "Agg" backend in the test? You can see how it's done in the other tests

Oh I didn't notice this as my local tests are on the server. I think you're right and I'll add it soon.

Also the tests could be faster if you use a smaller network? Maybe the class HNNGUI could have an option of passing a params dictionary? Then you can do:
params['N_Pyr_x'] = 3
params['N_Pyr_y'] = 3

HNNGUI(params=params)

Also the tests could be faster if you use a smaller network? Maybe the class HNNGUI could have an option of passing a params dictionary? Then you can do:
params['N_Pyr_x'] = 3
params['N_Pyr_y'] = 3

HNNGUI(params=params)

For tests we can do that manually

@chenghuzi you need to update api.rst

Updated in c39810a254fd568a32b0a113b99608c025fb7099

should the code logic for finding possible connectivity pairs between cell types be in a separate function? I can imagine this logic could be useful outside of the GUI as well.

should the code logic for finding possible connectivity pairs between cell types be in a separate function? I can imagine this logic could be useful outside of the GUI as well.

Agreed. But we may need another PR for this because the current way we use to extract connections is hierarchical while a general method (possibly in the form of a method in Network) should be plain. And then we can further simplify the existing code for extracting possible connectivity boxes using that method.

variables is now simulation_data.  We keep that because it allows modifying the item in the dict easily, otherwise we need to return the value to modify them

Great. I'm doing a pass over the code right now. Report back in an hour :-)

okay back to you. Finish this PR: https://github.com/chenghuzi/hnn-core/pull/2/files
and I think we're good to go here.
I also had some trouble following the logic of loading the param files. But let's discuss that later.

okay back to you. Finish this PR: https://github.com/chenghuzi/hnn-core/pull/2/files
and I think we're good to go here.
I also had some trouble following the logic of loading the param files. But let's discuss that later.

All format issues fixed

Never merge your own PR :) But I think it's time to move on to the next steps! Thanks @chenghuzi . This is a great improvement 🥳 🥳 🥳 . Next step is to try and figure out automatic building of documentation using html2canvas and jupyter-require ?

okay back to you. Finish this PR: https://github.com/chenghuzi/hnn-core/pull/2/files
and I think we're good to go here.
I also had some trouble following the logic of loading the param files. But let's discuss that later.

All format issues fixed

Never merge your own PR :) But I think it's time to move on to the next steps! Thanks @chenghuzi . This is a great improvement 🥳 🥳 🥳 . Next step is to try and figure out automatic building of documentation using html2canvas and jupyter-require ?

I figured out a way to do that, though not every elegant. Will open a PR for that soon.


Various Tweaks

Fix unique constraint check in Oracle
Fix reading from undefined when no PK
Check extra as number (ref #30)
Quote ident in PG
Order PG columnInfo output ordinally
Rename type -> data_type
Rename precision -> numeric_precision
Rename scale -> numeric_scale
Fixes https://github.com/knex/knex-schema-inspector/issues/27

The renames are technically a breaking change, and seeing that the rest of the API hasn't changed at all, combined with the fact that now all 5 vendors are supported, this is as good a time as any to call this 1.0.0


dialog: init at 1.2-20150225

Create ncurses dialogs from shell scripts.

@garbas why did you merge before this typo was fixed?

@jagajaga please check the commit. i've fixed it there.
i dont think it is nice to ask for every little change. especially if it takes 2 seconds to fix it yourself. that is after commenting what can be approved. this way pull requests are also used as a teaching tool and are not a bottleneck of accepting new commit.

Fixing it by yourself is not good because

people will send us not properly formed commits and we will have to fix them everytime.
if you fix and push, the author of the commit will be the same, but that can't be seen in the profile of that person. Some people don't like that, @matthiasbeyer for example.

@jagajaga i don't think you've really looked at the commit

author of the commit is the same (i'm just the commiter) and github shows this respectively. just check on @spacefrogg timeline (https://github.com/NixOS/nixpkgs/commits?author=spacefrogg)
people learn, give them a chance.

PR is not marked as "merged", that's what I wanted to say.
People learn by their mistakes, and the best way is to solve these
mistakes, it's not good to just look at final result :)
On 22 June 2015 19:33:11 Rok Garbas<EMAIL_ADDRESS>wrote:

@jagajaga i don't think you've really looked at the commit

author of the commit is the same (i'm just the commiter) and github shows
this respectively. just check on @spacefrogg timeline
(https://github.com/NixOS/nixpkgs/commits?author=spacefrogg)
people learn, give them a chance.

Reply to this email directly or view it on GitHub:
https://github.com/NixOS/nixpkgs/pull/8457#issuecomment-114172559


Encoding in QGIS is set to "system"

I know, this might be better issued in QGIS, but maybe that's a shorter way here.
When loading a FlatGeobuf file in QGIS , the encoding gets set to "system" by default, which is never UTF-8. So there are encoding problems.
Could you please advertise the layer as UTF-8 only (or whatever the layer is in)? See https://github.com/OSGeo/gdal/issues/2254

Never UTF-8? On the contrary, all sane systems use UTF-8 by default. 🙂
Seriously though, seems like something I want to fix but is either a GDAL or QGIS issue, not sure yet. I'll begin by investigating the GDAL side of this.


Increase jimdo prometheus client version.

Codecov Report

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

@@            Coverage Diff            @@
##             master       #8   +/-   ##
=========================================
  Coverage     94.24%   94.24%
  Complexity       39       39
=========================================
  Files             5        5
  Lines           139      139
=========================================
  Hits            131      131
  Misses            8        8

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 6a712f6...a11cd18. Read the comment docs.

Pull Request Test Coverage Report for Build 93

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.3%) to 94.074%

Totals

Change from base Build 90:
-1.3%

Covered Lines:
127

Relevant Lines:
135

💛  - Coveralls

Pull Request Test Coverage Report for Build 93

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.3%) to 94.074%

Totals

Change from base Build 90:
-1.3%

Covered Lines:
127

Relevant Lines:
135

💛  - Coveralls

Pull Request Test Coverage Report for Build 93

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.3%) to 94.074%

Totals

Change from base Build 90:
-1.3%

Covered Lines:
127

Relevant Lines:
135

💛  - Coveralls


Issue #6 fixed routing error caused by homepage in package json

When opening this project via the npm start command it defaulted to localhost:3000/BookCheff, but there was no routing for this endpoint. The default should have been just localhost:3000/ but due to the homepage section in your package.json it was going to the other address. All I've done to fix this was remove the section of your homepage with that endpoint.

Thank you for pointing out the issue and suggesting a solution. I apologize for any confusion caused by this issue and will make sure to update the documentation accordingly to avoid this problem in the future. Thank you again for your contribution!


Measure-FleetCoreWorkload - Method invocation failed

Method invocation failed because [System.Management.ManagementObject] does not contain a method named
'RefreshRegParams'.
At C:\Program Files\WindowsPowerShell\Modules\VMFleet\<IP_ADDRESS>\VMFleet.psm1:8197 char:13

        $r = Invoke-Command -session $s {

        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidOperation: (RefreshRegParams:String) [], RuntimeException
FullyQualifiedErrorId : MethodNotFound
PSComputerName        : XXXXX

This is understood. I am reverifying Server 2019 and should have a fix for this up in the next few days.
In the meantime you can comment out the calls to SetCacheBehavior in VMFleet.psm1. The method we use for accelerating read cache warmup only applies to Server 2022+.

@dl2n Thanks!

This is fixed in 2.0.2 - RS5 is now supported and confirmed to work.


Maximum Stack Depth Reached

Scenario:

am implementing this for a client in an existing implementation and am forced to run this as an after save.
have this running in a scheduled flow, batches of 10 leads, 1 mins after lead insert

When inserting 200 leads through data loader, regardless of batch size (except 1) and time delay (up to 2 mins) the first 50 leads are successfully inserted, after which I receive a message: Maximum Stack Depth Reached: 1001
Thanks for your help!

Do you have the full stack trace for the error message? Do you have any recursion in your flows or triggers?

Sadly I only have the error message provided in the flow:
Error element Assign_Via_Round_Robin (FlowActionCall).

Amazing thank you!!
On 14 Mar 2023 at 18:43 +0000, James Simone @.***>, wrote:

OK, I will take a look and get back to you!
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

@gpuviani I haven't been able to reproduce this issue locally - however, I am creating a new package version #9 which may help with some of these issues. I think a screenshot of your Flow input setup, and a log if you are able to attach that (the log that fails) would be helpful.
My gut is saying that you have a recursive process which is updating the leads you're inserting more times than is necessary. With the package version I am about to release, I've included some updates which may help with that, but if that doesn't address the issue we'll have to troubleshoot more with what I've requested above.

Hi James, thanks for your support on this. Sadly I am now also getting an error : Unable to lock row on the round robin records themselves. Would setting up some time help? Thanks again.

@gpuviani I'm afraid I don't have the bandwidth at the moment to do something like a call. The next two months will be quite busy for me. I am committed to helping asynchronously on this issue.
This morning, I was testing an update to seventy five opportunities where the Flow Action to round robin was being called without issue, but I'll try to take another look when I have the chance to see if I can reproduce the issues you've been having.

Closing, haven't been able to repro


Splitting bindings from high-level API

There are advantages in having both the bindings generation and the high-level API in one single crate.
But for adoption it seems like a lot of people want a the "raw" bindings first, and maybe they might want to
opt-in for the high-level rust api for flecs.
Thus I've made a PR that achieves this split. I've purposefully tried to keep it as minimal as possible.
If this is a decision to be taken forward, I'd happy to help with other tasks that comes along it.
Please take this as a point of discussion, and not necessarily a complete and closed PR.

If we were to do this change, I would prefer that we leave the high level api as is, just flecs. And just split off the new flecs-sys crate. This is generally the approach most binding+api libs take.

I don't know why the diffs are now the entire crate. I'm sorry for that, but I had to git pull and fix some stuff.

Closed in favour of the simpler #17. Feel free to request things from here to that PR if needed.


[CUK-73] Implement chromatic testing

LGTM, but it will be better if you prepare a short video with explanations.

LGTM, but it will be better if you prepare a short video with explanations.

Added a task for that: CUK-77


large configs, such as xo should be below dependencies

Given the comment at https://github.com/keithamus/sort-package-json/issues/30#issuecomment-378197067 , I'm curious about why xo was placed above dependencies, given that I have a very large xo config (> 100 lines. I'm not proud of it, but that's besides the point).
In particular, the block of
    'browserify',
    'babel',
    'xo',
    'prettier',
    'eslintConfig',
    'eslintIgnore',
    'stylelint',
    'jest',

seem like they are all configs that can become very long, and should always be below the dependencies.

Thanks for the issue @forivall.
I mostly added these blocks above dependencies because they are more often hand-edited than the dependencies/devDependencies fields, and so having them closer to the top means less navigating within the file to edit them.
I am not massively concerned either way. Perhaps itd be good to hear what others think?

Yeah, my preference in general is that all of the "official" and generic entries should be above sections that are specific to a third party tool, and aren't related to the core parts of a package.json. Like, it makes sense to put betterScripts right next to scripts, but linter and test config aren't really core.
It definitely is a fuzzy classification system though.

my intuition was that it made sense to put these configs next to scripts also - as script invocation can imply parts of the config.
Let's get others involved here, @raphinesse and @jdcrensh may have opinions on this, as well as past contributors like perhaps @bluelovers @selbekk or @simonkberg

Thanks for the ping. Personally I prefer the ordering as it is today, for the same reasons you already brought up:

because they are more often hand-edited than the dependencies/devDependencies fields, and so having them closer to the top means less navigating within the file to edit them.

and

my intuition was that it made sense to put these configs next to scripts also - as script invocation can imply parts of the config

Other than that, always having the dependencies defined at the end also means it's easier to navigate to them when needed, instead of trying to find them somewhere in the middle of the file.

i will like put they low then dependencies & resolutions

I'm of the mind to put all config keys between scripts and dependencies. My reasoning is that dependencies should always appear at the bottom, which is consistent with nearly all projects I've worked with. It's not that hard to scroll to the end of the file.

I put the xo key next to similar keys. There's nothing more to it. With >100 Lines config, I would probably prefer to have a separate config file though :sweat_smile:
There's not one size that fits all here. Given that config is out of the picture (https://github.com/keithamus/sort-package-json/issues/30#issuecomment-380425399), I would put everything non-standard at the end. Alphabetically sorted.

So it seems as though we have the following:

@forivall, @bluelovers, @raphinesse would vote for these keys below dependencies
myself, @simonkberg, @jdcrensh would vote for these keys between scripts and dependencies

I'm not the greatest at maths, but that looks like a tie to me. Nothing so far said has convinced me to move the keys below dependencies, although @jdcrensh's comments about consistently having dependencies at the bottom certainly sways me to put unrecognised non-standard keys higher up.

oh make this as option let user choose it?

I am not sure this will be resolved any time soon. I'll close this for now. If people have opinions on this please feel free to file a new PR or issue! Thanks


Bug Report:  No proper margin on the right side of home page sections

Reproduction steps

Expected behavior
it should have some margin for creative designs. or else it may lead to user interface problems
Actual Behavior
it actually should have some margin for creative designs
👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

please assign this to me


Make plugin loading order predictable

1. Why is this change necessary?
When loading the plugins, the DirectoryIterator which scans for plugins is not working
in a predictable order, because it uses the ctime of the directories as sorting order.
2. What does this change do, exactly?
It makes the loading order of the plugins predictable (simply alphabetically).
This is just a workaround, there should be a meaningful plugin loading order according
to their dependencies (either composer or requiredPlugins from XML).
3. Describe each step to reproduce the issue or behaviour.
If you try to override a symfony parameter or service of another plugin, you can't be sure
that it works consistently due to deviations of the ctime values of the plugin directories.
This can be caused by deployment, etc.
4. Please link to the relevant issues (if any).
https://issues.shopware.com/issues/SW-22399

Why don't you sort them in the query?

I could, but it wouldn't make that much difference. The really necessary part is sorting after the DirectoryIterator scanned for directories. I just used it in both places for consistency.

Hey @hhoechtl,
thanks for your contribution.
While this is a nice change, this could break previous plugins, so we would have to implement it with the next major version.
Since this is only a workaround, we'd rather like to fix it properly for a major version.
If you're still interested in this change, make sure to ping any of us (@soebbing, @shyim) in Gitter, so we can talk about a more thorough solution.


Vimeo support?

I assume this has Vimeo support? And if so, will it download videos I have rented?

yes, both Vimeo and Vimeo on demand are supported, if it doesn't work for then you can fill a new issue with needed information.


Customize Admin UI?

Is it possible to customize the UI for the admin panel? I'm wanting to make the admin panel more user friendly for my admins, but I don't see anything in the documentation for such a thing. I'm coming from Sanity, although being a CMS, I really enjoy the customizability of the front end. Is this possible in PocketBase?

No, at the moment the Admin UI can't be customized. Eventually this might be handled with https://github.com/pocketbase/pocketbase/issues/898.

The Admin UI is near perfect craft. I'm really in awe with this project.

The Admin UI is near perfect craft. I'm really in awe with this project.

It's amazing for developers, however I'm wanting to customize the UI to be more straightforward for my less tech-savvy counterparts. What I'm asking for is probably out of scope for now, but I really enjoy the backend. I could roll out my own Admin Panel, but that's far more work than just customizing the existing panel.

I guess cloning the project and modify this UI part of the code will do?
https://github.com/pocketbase/pocketbase/tree/master/ui


Issue 2394

Provide test case for issue #2394

Confirmed that this is expected behaviour, see #2394


PEP8 plus optimizations.

I've built a websocket server based on gevent-websocket and another one using ws4py. The server based on ws4py is quite slow compared to gevent-websocket, so I looked trough ws4py code to see what can be optimized.
I guess one of the reasons is that ws4py targets python 3 too having additional code paths.
The main reason for performance difference should be some architectural decisions, but I am not that good to fix it.

Hello,
Well a big thanks for your hard work. I don't have the time going through all of the changes right now but will do my best to do so asap.
Have you witnessed performance gains then?

You decide if you want to merge them.
No I don't have precise becnhmarks, so I believe the speed if improved did it insignificantly.
Also I not sure I am doing the benchmarks or interpreting the results correctly.
https://github.com/warvariuc/python-websocket-channels/blob/202897fa61068111757ff99e449d047334ed207f/test.py
https://github.com/warvariuc/python-websocket-channels/blob/202897fa61068111757ff99e449d047334ed207f/echo_gevent_server.py

I don't want to keep the gevent-websocket version, because though it's fast, you cannot plug in control of the handshake -- for example check cookies, session, etc.
https://github.com/warvariuc/python-websocket-channels/blob/202897fa61068111757ff99e449d047334ed207f/websocket_channels.py#L20

I will see, there might be a few aspects I dont merge (I'm usually not a big fan of local imports for instance, a matter of consistancy with my usual code pratices).
With that said, I don't really know what i'm going to do with ws4py anyway. I feel there are much better alternatives with bigger communities out there. Is there a future for ws4py worth heavily investing in?
At least, I want the existing code to be as bug-free as possible and I do appreciate it's simple enough for many people.

With that said, I don't really know what i'm going to do with ws4py anyway. I feel there are much better alternatives with bigger communities out there. Is there a future for ws4py worth heavily investing in?

What alternatives would you suggest?
I like ws4py because it's quite simple to understand and plug in, because I want to use Redis's pub/sub to implement channels.

Autobahn (http://crossbar.io/) and Tornado come to mind.

 I like Gevent :).

I understand but, to be fair, Autobahn does support asyncio as well these days. Anyway, if you're happy with ws4py, I will not complain. I just wanted to be fair before you invest anymore time. This is a small project with a tiny community :)

Ok, then myabe my third try will be with Autobahn + asyncio + Python3 :) But I hate camelCase they use everywhere.

@warvariuc FYI, I've released https://pypi.python.org/pypi/wsaccel to accelerate frame masking.

I hear you :)
Remember that ws4py does support Python3 and asyncio. It's just bare websocket whereas autobahn has much more to offer (pubsub and all).

Indeed, wsaccel does improve performances a lot under some cases. See http://www.defuze.org/oss/ws4py/testreports/servers/0.3.4/

@methane Yes I pip installed it.
@Lawouach Would you make a new release ti ws4py? This commit https://github.com/Lawouach/WebSocket-for-Python/commit/1f206bc0f361fab3c74d94052f56137228bc471a is not in PyPi

Indeed, it's long overdue. I will but not before the end of the week I'm afraid.

It's not urgent

@Lawouach please encourage people to contribute and grow the community! I recently switched to ws4py from gevent-websocket. Of the non-autobahn websocket implementations, I think this is the most actively maintained and I've found it easy to use. The thing about using asyncio is that it's not really well used with any popular frameworks yet, afaict. So, unless I want my websocket server to be separate from the rest of my pyramid application, I can't use asyncio. This is the best gevent solution I found that makes it easy for me to have everything in one place.

Hi, thanks for the kind words regarding ws4py. Its community is small and quiet so it's hard to understand whether or not it's actually useful to anyone :)

Hi. Thanks you for your work, but i can't merge it like that. It is a little bit outdated, and such changes (like pep8) will also break others PR for minor adjustements.
Also, few changes (like removing the format_address) may also broke for people that use it right now.
Consider rebase your branch if possible :)


Replace GetDirectories in CimDscParser

Avoid array allocations from GetDirectories/GetFiles.
Contributes to #14318.

CodeFactor issues are false positives.

2 years - time to merge 😄

PowerShell-CI-linux failure seems unrelated:
Invoke-WebRequest tests.Denial of service.Charset Parsing

Expected the actual value to be greater than 5, but got 4.98730809053562.

@SteveL-MSFT Can you please re-review.


ARROW-3272: [Java][Docs] Add documentation about Java code style

The Java code style is based off of the Google style guide with some small differences. This adds documentation with a link to the Google style guide and a list of what is different.

cc @icexelloss @jacques-n


UX: Designs for change log as it appears on the individual park page

Currently we have a placeholder under "recent changes" on the View Park Record page. This ticket is to cover the design of what is currently in that placeholder.
Suggestions/context:

Business only wants to see the major, legal changes, which include adding a new legal name/moving a name from current to historical, repealing a park
Design should determine what the important info is to display here (business did not have any requirements)
We should limit the number of changes displayed - either by total number, recency (e.g., up to 5 years old), or some other mechanism
Once the change log section is complete, we could link to the full change log to allow users to see older changes
Since it is tied to legal changes, the typical park will only have a couple of entries at most - unlikely that this log will get very long

looking good - @meredithom can you add these ones to implementation ticket #225
moving to done


client runs: export all node reports for a single node

User Story
As an Automate user, I would like to retrieve all the client run reports (within a given timeframe) for a single node.
Definition of Done
API call for retrieving (downloading) all client run reports for a single node, given a timeframe (start and end time). Should support CSV and JSON download.
https://github.com/chef/automate/issues/178

yay! curl --insecure -H "api-token: token-val" https://a2-local-inplace-upgrade-dev.cd.chef.co/api/v0/cfgmgmt/reports/export -d '{"output_type":"json","node_id":"68656f04-c3d6-4fd7-ab9d-cf46b55587ae"}'


PDFViewer Control based on PDFjs

Is there an existing issue for this?

[X] I have searched the existing issues

Is your feature request related to a problem? Please describe the problem.
I would like to know if we could have PDFJs based PDFViewer Control for BitPlatform
Reference:
https://mozilla.github.io/pdf.js/
Describe the solution you'd like
BitPdfViewer Control
Additional context
No response

Thanks for contacting us. We're investigating this issue.
We'll let you know if it's possible to work on this issue.

Resolving this issue has started. It will be announced when this issue is resolved.

This issue has been resolved and it will be available in the next pre-release version. It will be announced when the pre-release becomes ready.


Multiple self-closing tags render only the first one

Version
3.0.11
Reproduction link
https://codepen.io/ds2013/pen/LYxLQWa
Steps to reproduce
None
What is expected?
Multiple self-closing tags render all
What is actually happening?
None

For the browser, <user-name> is a custom tag, and the custom tag requires a closing tag. Only certain tags in HTML are allowed to be self-closing: https://html.spec.whatwg.org/multipage/syntax.html#void-elements, and a non-void element must have an end tag, also see https://github.com/WICG/webcomponents/issues/624


TypeError when processing playlist "Your Time Capsule"

First - thanks for this great script! I run it from a cronjob regularly, as my playlists are very dear to me and I can not afford to let Spotify own them. Thanks!
So for the issue:
The Spotify meta playlist "Your Time Capsule" causes the following error:
$ spotify-backup.py --format txt playlist.txt
[08:14:09] Loading playlist: Your Time Capsule (55 songs)
Traceback (most recent call last):
  File "/Users/erikw/src/github.com/caseychu/spotify-backup/spotify-backup.py", line 170, in <module>
    main()
  File "/Users/erikw/src/github.com/caseychu/spotify-backup/spotify-backup.py", line 161, in main
    uri=track['track']['uri'],
TypeError: 'NoneType' object is not subscriptable
2 tries left.

got the same with:
[08:07:42] Loading playlist: 16 Horsepower – Yours Truly (25 songs)
Traceback (most recent call last):
File "/home/steffen/bin/spotify-backup.py", line 170, in
main()
File "/home/steffen/bin/spotify-backup.py", line 161, in main
uri=track['track']['uri'],
TypeError: 'NoneType' object is not subscriptable

I don't get this error anymore. It somehow disappeared.


Why would webpack-cli require html-webpack-include-assets-plugin

I am getting a strange error and don't  understand why its happening via npm.
I have the following npm packages installed:
"devDependencies": {
    "@fortawesome/fontawesome": "^1.1.8",
    "@fortawesome/fontawesome-free-solid": "^5.0.13",
    "bootstrap": "^4.1.1",
    "css-loader": "^0.28.11",
    "exports-loader": "^0.7.0",
    "html-webpack-plugin": "^3.2.0",
    "jquery": "^3.3.1",
    "json-loader": "^0.5.7",
    "mini-css-extract-plugin": "^0.4.0",
    "node-sass": "^4.9.0",
    "popper.js": "^1.14.3",
    "postcss-loader": "^2.1.5",
    "sass-loader": "^7.0.3",
    "webpack": "^4.12.0",
    "webpack-cli": "^3.2.1",
    "webpack-dev-server": "^3.1.4"
  },

when I run npm start from the terminal, I get the following error message:
C:\Users\...\node_modules\webpack-cli\bin\cli.js:235
                                throw err;
                                ^
Error: Cannot find module 'html-webpack-include-assets-plugin'

Why is this happening to me?

Sounds like you have it in your webpack config. Try doing a text search across all files maybe?

Yeah I did that.  Nothing comes up.  So strange.  My Webpack environment runs great with it.  I don't mind holding on to it in development, but I wish I knew why npm is complaining about it when its gone...

Very strange, it must be loading the plugin from somewhere, in some kind of webpack.config.js file would be my bet.
I don't think there's much I can do to help, since the issue is actually outside of this plugin somewhere.
Gonna close it, good luck tracking down where config is that's causing your error!


Makes 'Tensor.gathering' generic over 'Int32' and 'Int64'.

Fixes #259.

I’m not sure this is the safest solution, because Int8 and Int16 also conforms to TensorFlowInteger.

Oh I see. I can make a new protocol then, as you suggested.

It's weird though that the raw ops generation script assigns TensorFlowScalar & BinaryInteger as the signature of the gather ops.

In either case, I created a separate protocol and local tests pass. Currently running CI tests.

Yeah, I think the wrapper generation script doesn’t pay attention to the specific integer types. We should change the script to use the new protocol for APIs that only take index scalar types.

Sure, I can do that in a separate PR after we're done with this one.

Thanks!


PropertyChanged gives exception for enum columns

Setup:
TreeDataGrid with a column that contains an enum.
Action:
The model raises a PropertyChanged event for the enum.
Expected behavior:
The TreeDataGrid updates the cell to the new enum value.
Observed behavior:
A System.InvalidCastException is triggered.
Workaround tried:
I used an IValueConverter for the conversion between the enum and strings. Did not change anything.
Apparently the way the data is displayed normally and the way the data is updated are different.
Example for this issue: https://github.com/blackmole/AvaloniaComboBoxEnum

So what happens is this chain: TextCell.Value is changed --> TreeDataGridTextCell.Value is changed which sets TextCell.Text which tries to convert the text back to the enum. And which is unnecessary because the TextCell.Value is already correct.
Not sure what the clean solution is, but I guess it might work to change the TextCell.Text setter to:
set
{
    if (value != _value?.ToString())
         Value = (T?)Convert.ChangeType(value, typeof(T));
}


Us14 event notify

Coverage decreased (-2.3%) to 67.299% when pulling b839966eb81a64ed37b68457c861ea3ecac1ee28 on US14_event_notify into fa950d9167345b10413b793fe8de81d404d7c702 on development.

Coverage decreased (-2.3%) to 67.235% when pulling 451f6c0dc505c7da17d90e723e97ef566f7424d1 on US14_event_notify into fa950d9167345b10413b793fe8de81d404d7c702 on development.

Coverage decreased (-2.3%) to 67.235% when pulling c4cc9673da771c2fb9d8abb9728a2f20632fa819 on US14_event_notify into fa950d9167345b10413b793fe8de81d404d7c702 on development.

Coverage decreased (-2.4%) to 67.139% when pulling e2cc6f5909c46427935ce2be6b885923f91d35da on US14_event_notify into fa950d9167345b10413b793fe8de81d404d7c702 on development.

Coverage decreased (-2.4%) to 67.139% when pulling eb14ed68b86ba68cee66b2403766dca073436cf8 on US14_event_notify into fa950d9167345b10413b793fe8de81d404d7c702 on development.

Coverage remained the same at 74.225% when pulling 53e71ba77c2e3d49f04c74a862312652052a8894 on US14_event_notify into 1fcd83c29b607fd720182ef9f6565db87a9d5fbd on development.


Non-AlphaFold colors

In some of my predicted structures created from AlphaFold 3, PyMOL shows colors such as purple or green. Is it meant to be there, or is this an error of some sort?
Attaching an image to show an example.

Hope to hear from someone soon!

Found another example for green coloration.

The original version of @cbalbin-bio excludes values that match exactly 90, 70, and 50. It's unfortunate when this happens, as sometimes the pLDDT values match these exact numbers.
I have submitted a pull request with code modifications to address this issue and update the script. You can find the details of the changes in the pull request.
Best


Document to setup multiple domain with application

Can you give a link to Document to setup multiple domain with application on production.
How the domain redirection works?
Please explain if you get time, i want to host blogs of my application with multi domain
like blog.codecrux.com, blog.exploreout.in, blog.goos.in etc.

Although I’m not the author, in an effort to help him out I will see if I can answer this to some degree.
First steps are the same for a single site installation; pick a domain or subdomain you want to be the default site for Camaleon and install (I'm using the latest version, not sure if that matters) – for this example we will use blog.codecrux.com as our default site. It’s important to remember which site is the default site as it’s this site’s admin you will go to later to add the additional sites.  Make sure your other domains/subdomains are pointed to the same webroot as this domain/subdomain so they all share the same Camaleon install.
After installation is complete, log in to the Admin at blog.codecrux.com/admin.  Upon successful login, you should find “Sites” located under “Settings” in the side nav. NOTE: You can only find this by logging into the default site’s admin.  Navigate to “Sites” and click “Add Site”.  For the slug type in one of the other domains/subdomains, for example, blog.exploreout.in, give it a name and click Submit. You can now log in to the admin at blog.exploreout.in/admin and edit that sites content.
Hope this helps and good luck!

Good answer Charles!
Additionally don't forget to configure your domain configuration, sample this is my configuration for camaleon:
Main Site:
server {
      listen 80;
      server_name tuzitio.com;
      root /var/www/tuzitio2/public;   # <--- be sure to point to 'public'!
      passenger_enabled on;

      rails_app_spawner_idle_time 0; # rufus scheduler both lines
      passenger_min_instances 1;

      client_max_body_size 200M;
}

Children Sites:
server {
      listen 80;
      server_name *.tuzitio.com;
      root /var/www/tuzitio2/public;   # <--- be sure to point to 'public'!
      passenger_enabled on;
      client_max_body_size 200M;
}

Note: If you have several domains and not subdomain, you need to register as a main domain with the same root directory.
I am sorry, I was busy with some improvements!
Thank you for your interest in Camaleon CMS.


Sort on save

This is a great extension, it's the only one I've found that lets me easily configure the sorting rules just as I want them. But, it would be even better if it had an option to sort imports when a file is saved.
Please consider adding this feature. Thanks!

Sure. Will do it in the next feature

Awesome, thanks!

done in v1.1.1. You will need to enable importSorter.generalConfiguration.sortOnBeforeSave setting to true.


Transforming columnNames back to attribute names

Waterline version: 0.13.0-rc9
Node version: v7.9.0
NPM version: 4.2.0
Operating system: Windows 10 pro

Warning: After transforming columnNames back to attribute names, a record
in the result has a value with an unexpected data type for property created.
The corresponding attribute declares type: 'string' but instead
of that, the actual value is:
2017-05-17T11:08:24.000Z

You are seeing this warning because there are records in your database that don't
match up with your models.  This is often the result of a model definition being
changed without also migrating leftover data.  But it could also be because records
were added or modified in your database from somewhere outside of Sails/Waterline
(e.g. phpmyadmin, or another app).  In either case, to make this warning go away,
you have a few options.  First of all, you could change your model definition so
that it matches the existing records in your database.  Or you could update/destroy
the old records in your database; either by hand, or using a migration script.
(For example, to wipe all data, you might just use migrate: drop.)
More rarely, this warning could mean there is a bug in the adapter itself.  If you
believe that is the case, then please contact the maintainer of this adapter by opening
an issue, or visit http://sailsjs.com/support for help.

Here is my model:
module.exports = {    attributes: {     someDate: {       type: 'string',        columnType: 'datetime'     }   } };

Hey @Alessy, the attribute 'datetime' does not exist anymore on Sails 1.0
You should use the type: 'number' + columnType: 'bigint' to store an unix timestamp (same as createdAt and updatedAt fields)
However, if you are using sails-postgresql, there is a known issue here https://github.com/balderdashy/waterline/issues/1487 that you should take into account.
Best,

Hey @billyshena, I have the same issue with default model attributes for createdAt and updatedAt though I have been following your upgrade guide: https://github.com/balderdashy/sails-docs/blob/1.0/upgrading/To1.0.md#the-autoupdatedat-and-autocreatedat-model-settings-are-now-attribute-level-properties
As I have been using Sails since version 0.9 I cannot change all my database table from datetime to int.
attributes: {
    createdAt: { type: 'string', autoCreatedAt: true },
    updatedAt: { type: 'string', autoUpdatedAt: true },
   ...
}

Warning: After transforming columnNames back to attribute names, a record
in the result has a value with an unexpected data type for property `updatedAt`.
The model's `updatedAt` attribute declares itself an auto timestamp with
`type: 'string'`, but instead of a valid timestamp, the actual value
in the record is:
2017-05-26T12:42:46.000Z

What would you recommend to do?

@markBerlin

For type: 'string', these timestamps are stored in the same way as they were in Sails 0.12: as timezone-agnostic ISO 8601 JSON timestamp strings (e.g. '2017-12-30T12:51:10Z'). So if any of your front-end code is relying on the timestamps as strings it's important to set this to string.
For type: 'number', these timestamps are stored as JS timestamps (the number of milliseconds since Jan 1, 1970 at midnight UTC).

To move to v1.0 you need a migration to pull in all your datetimes, convert them to ISO 8601 timestamp strings with momentjs or whatever your preferred formatting tool is. I'd create an object cache associate row id to timestamp, then execute an alter table that drops the column, then add the column again but with the new datatype, then insert. Then your model can still be string.
You should look into if your database adapter exposes a migration cli tool like this one for sequelize.
Or you shouldn't upgrade to v1.0,

Is there any way to turn off these warnings?


when i submit the solution in leetcode here in github its not updating

Please review previous closed issues before filling out a new one! Duplicate issues will be closed without comment.
Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

this is an issue with the new UI of LeetCode. A solution is pushed here https://github.com/QasimWani/LeetHub/pull/464 and waiting for being merged


Tab key support is missing for menu options.

VSCode Version: 1.2.0-alpha
OS Version: Windows10

I observed tab key is not working as expected for menu options. When user click on File menu and try to switch to other menu options using tab key then nothing is happening.
Expecting it should behave similar to Visual studio:

https://github.com/Microsoft/vscode/issues/17278


docs: fix breadcrumb import from

📑 Description

Fix breadcrumb import from

✅ Checks

[x] My pull request adheres to the code style of this project
[x] My code requires changes to the documentation
[x] I have updated the documentation as required
[x] All the tests have passed
[x] My pull request is based on the latest version

ℹ Additional Information

From
  import { Breadcrumb } from '$lib/index'

To
  import { Breadcrumb } from 'flowbite-svelte'

Thanks for the catch.
I will merge it soon.

Thanks for the catch. I will merge it soon.

No problem, glad I can help. Cheers 🍻

Since I merged your last PR, my main is one commit ahead than this PR.
Could you pull upstream.
Can you try this and let me know.
git checkout your-branch
git pull upstream main

@shinokada Successfully merge to latest main


feat: utilize keycloak for authing to LFAI things

Authorizing calls to the API by validating the credentials provided by keycloak and sending that information in the 'session token' passed to the backend.
NOTE: Using authservice might change this approach.

CLOSING AS DUPLICATE OF #483

Addressed by https://github.com/defenseunicorns/leapfrogai/pull/533


i want to integrate flask in this project

can someone make this work with flask

@Rahwik I need more information to handle this issue. Do you mean that you want to create a website, or an API, or just a back-end for the bot with Flask?

How to make it work on Windows?


bug(): Modal 组件使用Ctl+F5刷新后 出现错误

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
当我在页面中添加

</ModalDialog>

组件，

Expected Behavior
No response
Steps To Reproduce
No response
Exceptions (if any)
No response
.NET Version
No response
Anything else?
No response

@mk7710 更新到最新试一下可以吗？

@mk7710 无反馈自动关闭


Hide rwg

The rwg command is showing up for regular players tab complete can you please resolve that i.e. using plugin.yml permission or via code

This should be fixed in commit abf185b93db77f0583bcf40ffbca895c3d70c7a2 now. It will get rolled out to the plugins when they next get updated to it.


update to postcss-6.0.1

I only checked that three test are failing when updating the dependencies to postcss-6.0.1:
  tokens
Node#before is deprecated. Use Node#raws.before
    1) compose-node-module
    2) multiple-dependencies
    3) multiple-sources

  1) tokens compose-node-module:
     postcss-modules-scope: /home/marcin/repos/postcss/css-modules-require-hook/test/tokens/cases/compose-node-module/source.css:3:3: referenced class name "i__imported_paragraph_0" in composes not found
  CssSyntaxError: /home/marcin/repos/postcss/css-modules-require-hook/test/tokens/cases/compose-node-module/source.css:3:3: referenced class name "i__imported_paragraph_0" in composes not found

    1 | .foo
    2 | {
  > 3 |   composes: paragraph from 'awesome-theme/common.css';
      |   ^
    4 | }
    5 |

  2) tokens multiple-dependencies:
     postcss-modules-scope: /home/marcin/repos/postcss/css-modules-require-hook/test/tokens/cases/multiple-dependencies/source.css:3:3: referenced class name "i__imported_text_0" in composes not found
  CssSyntaxError: /home/marcin/repos/postcss/css-modules-require-hook/test/tokens/cases/multiple-dependencies/source.css:3:3: referenced class name "i__imported_text_0" in composes not found

    1 | .article
    2 | {
  > 3 |   composes: text from './colors.css';
      |   ^
    4 |   composes: medium from './fonts.css';
    5 |   composes: mediumBottom mediumTop from './margins.css';

  3) tokens multiple-sources:
     postcss-modules-scope: /home/marcin/repos/postcss/css-modules-require-hook/test/tokens/cases/multiple-sources/source.css:3:3: referenced class name "i__imported_oceanic_0" in composes not found
  CssSyntaxError: /home/marcin/repos/postcss/css-modules-require-hook/test/tokens/cases/multiple-sources/source.css:3:3: referenced class name "i__imported_oceanic_0" in composes not found

    1 | .article
    2 | {
  > 3 |   composes: oceanic from './colors.css';
      |   ^
    4 | }
    5 |

Is there any update on this?

@sullenor thanks for taking the time to merge the fix. I was super excited to see this roll out, but confused when npm did not pick up the 4.1.0 release because it was published with a beta dist-tag. I wanted to bring it up in case it was an accident.
$ npm show css-modules-require-hook

name: 'css-modules-require-hook',
description: 'A require hook to compile CSS Modules on the fly',
'dist-tags': { latest: '4.0.6', beta: '4.1.0' },


How should composition work?
If anyone has a good idea for how to get optic composition to work smoothly I'd love to hear it!
Currently all optics have the form
def Optic π A B S T := ∀ ⦃P⦄ [c : π P], P A B → P S T

where π is one of

π
optic

Profunctor
Iso

Strong
Lens

Choice
Prism

Affine
Modification (aka traversal0)

Traversing
Traversal

Mapping
Setter

Closed
Grate

Given o₁ : Optic π₁ A B S T and o₂ : Optic π₂ S T X Y, we compose these o₂ □ o₁ : Optic (π₁ ⊓  π₂) A B X Y. Where the meet operation ⊓ is the tricky part that I don't know how to define nicely.
So for example composing a l : Lens A B S T and a p : Prism S T X Y would require taking the meet of Choice P and Strong P which is Affine P. But ideally Lean would display the type as p □ l : Modification A B X Y.
I'm essentially trying to get Lean elaborator to compute the joins on this lattice:

source

It's cool to see that optics have a place in mathlib4. Could I have some context for what need they fulfil in the library?

This pull request has not seen any activity for quite a while, and it does not pass all CI checks. For these reasons, I am closing it now.
If you are still interested in getting this PR merged, please re-open the PR, update the branch, and make sure that all CI checks pass, Please label the PR with awaiting-review when ready.


Update get-networkcontrollerloadbalancerinboundnatrule.md

Added descriptions and example

@AnirbanPaul,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


[Add-AzsVMImage] "The pipeline was not run because a pipeline is already running" error

Add-AzureRmVhd : The pipeline was not run because a pipeline is already running. Pipelines cannot
be run concurrently.
At F:\AzureStack-Tools-master\ComputeAdmin\AzureStack.ComputeAdmin.psm1:183 char:9
Add-AzureRmVhd -Destination $osDiskBlobURIFromLocal -Resource ...
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
CategoryInfo          : CloseError: (:) [Add-AzureRmVhd], PSInvalidOperationException
FullyQualifiedErrorId : Microsoft.Azure.Commands.Compute.StorageServices.AddAzureVhdCommand
The environment was clean and I didn't find any concurrent jobs running Add-AzsVMImage. Can you provide more details about "the pipeline"? which pipeline?
Thanks.

Thanks for the pointer to https://github.com/Azure/azure-powershell/issues/4980 we have opened the issue in the Azure module repo. Once it is fixed there, the issue for Azure Stack should be solved too.


ASUS Z170M-E-D3 trouble to enable large memory.

 Make sure to confirm this in the BIOS and not with GPU-Z or similar since it can be inaccurate
[+] 4G decoding is enabled. Make sure to confirm this in the BIOS and not with GPU-Z or similar since it can be inaccurate
[+] UEFIPatch is applied (see Using UEFIPatch for more information).
[-] On some motherboards DSDT Patching is also needed
[+] I have read Common issues (and fixes)

Hello folks. Greetings from UA.
I have a problem.

Inserted ReBarDxe.ffs in my BIOS, using UEFITool.
Patched BIOS Using UEFIPatch with standart patches.txt (xCuri0/ReBarUEFI/blob/master/UEFIPatch/patches.txt)
Cheking for pad file issue (no pad`s in DXE volume at all)
Cooffee`moded BIOS with new microcodes, etc (using mutant qtj1)
Flash bios, using ch341.
After first boot turned off CSM, from moded grub changed setup_var 0x3 (setup_var 0x3 0x1).
Loading windows 11, and don`t have large memory in device manager (screenshot above)

When i choose in ReBarState.exe "1-7" - GPU-Z  says, that rebar is disabled in bios, but everything else is "yes"
When i choose in ReBarState.exe "8-16" - system freeze on boot with graphical artifacts on POST.
When i choose in ReBarState.exe "32" - system boot, but only from iGPU, Gpu-Z says "Rebar Enabled? 16gb". RTX3080 has error 12.
Trying to DSDT patching, but my bios dosnt have AmiBoardInfo, only AmiBoardInfo2, (no DSDT error, when trying to extract with AmiBoardInfoTool)
Reading all issues posts, reading hundered of pages on winraid.level1techs.com, not sleep at all 40hrs.
I realy want to solve this quest. or understand that this is impossible on my rig.
I understand that you are tired of stupid questions, but I hope for your support.
As bad luck - my MoBo not listed in "ReBar-worked mobo list" =(
Original_n_modded_bios.zip

You're most likely not enabling 4G decoding correctly. You need to set it to 0x0 and then 0x1 again for it to activate sometimes.

Thnx for answer.
But already tried.  0x0, reboot, 0x1. reboot.
Also tried with setup_var_1, setup_var_2, setup_var_vs.
No large memory in device manager =(

can you send ifr extractor results file ?

I have two "above 4g" search results. Extract two of them.
1 IFR.txt
2 IFR.txt
Thnx for help

@ssvext you missed the option above 4gb mmio assignment.
Enable and it will work


New-Item multiple files isn't supported

Summary of the new feature / enhancement
Coming from Linux, I like using touch while creating multiple files in the same directory.
In Powershell, I use New-Item or as a short-form ni like this:
ni input.tsx

But if I use 2 files, it isn't supported:
ni input.tsx select.tsx

Ideally, I'd like it to create 2 files at the same time.
Currently, I get an error:

New-Item: A positional parameter cannot be found that accepts argument 'select.tsx'.

This shouldn't happen.
Proposed technical implementation details (optional)
Just make it behave like touch so I can use touch command directly.
In anyways, I do use touch by setting an alias on Powershell:
Set-Alias touch ni

But it doesn't behave like it.
All I want is to create multiple files easily as I don't like using VSCode or File Explorer to create each file manually.
That's the entire reason I opt for Terminal but Powershell doesn't support it.

You are missing a comma to make a list
PS /tmp> ni input.tsx, select.tsx

    Directory: /tmp

UnixMode          User Group         LastWriteTime         Size Name
--------          ---- -----         -------------         ---- ----
-rw-r--r--     bythesea staff     01/20/2024 06:23            0 input.tsx
-rw-r--r--     bythesea staff     01/20/2024 06:23            0 select.tsx

I didn't get touch working at all that's why I could alias it. Probably can't alias global command, at least that's what should happen.
Anyways, here's the output I got:
PowerShell 7.4.1
PS C:\Windows\System32> which touch
which: The term 'which' is not recognized as a name of a cmdlet, function, script file, or executable program.
Check the spelling of the name, or if a path was included, verify that the path is correct and try again.

I'm not using WSL bdw.
Anyways, that answers my question but would love to know how to get touch working on PS7 :)


Duplicate autolink shortcut/shorthand sections in the docs

Two sections document the autolink shortcuts/shorthands, each with its own dfn and description.

https://tabatkins.github.io/bikeshed/#autolink-shortcuts
https://tabatkins.github.io/bikeshed/#autolinking

I suggest merging them into one section, which will reduce repetition of information (and the strong possibility of eventual contradiction).

Ah, thanks for pointing this out. It was, in fact, already diverging unacceptably.


Worked to hide the recent search from search bar

Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

There's at least one more in DomainsList.tsx - then is that everything?

Turns out someone else took care of the other cases!


HR prompt stuff not compatible with webui forks

          i know it's not this repo's responsibility to remain compatible with other forks, but i had to do the following to make the latest change compatible with 'SD Next' / vlad's fork, since a lot of these values just weren't there. maybe i'm missing some other addon update or something though.

diff --git a/sd_dynamic_prompts/dynamic_prompting.py b/sd_dynamic_prompts/dynamic_prompting.py
index 0f04096..6e76f1c 100644
--- a/sd_dynamic_prompts/dynamic_prompting.py
+++ b/sd_dynamic_prompts/dynamic_prompting.py
@@ -68,11 +68,17 @@ def get_prompts(p):

 def get_hr_prompts(p) -> tuple[str, str]:
-    hr_prompt = p.all_hr_prompts[0] if len(p.all_hr_prompts) > 0 else p.hr_prompt
+    hr_prompt = getattr(p, 'hr_prompt', p.prompt)
+    all_hr_prompts = getattr(p, 'all_hr_prompts', [])
+
+    hr_negative_prompt = getattr(p, 'hr_negative_prompt', p.negative_prompt)
+    all_hr_negative_prompts = getattr(p, 'all_hr_negative_prompts', [])
+
+    hr_prompt = all_hr_prompts[0] if len(all_hr_prompts) > 0 else hr_prompt
     hr_negative = (
-        p.all_hr_negative_prompts[0]
-        if len(p.all_hr_negative_prompts) > 0
-        else p.hr_negative_prompt
+        all_hr_negative_prompts[0]
+        if len(all_hr_negative_prompts) > 0
+        else hr_negative_prompt
     )

     return hr_prompt, hr_negative

Originally posted by @cool-bigdogs-tshirt in https://github.com/adieyal/sd-dynamic-prompts/issues/474#issuecomment-1574256067

@akx thanks!
btw, those new fields were introduced in a1111 v1.3 and actually still has some issues/regressions, that's why i haven't ported the code yet (for example, it breaks xyz scripts as cached values remain regardless of xyz values). once issues are resolved, i'll port it to my fork as well.