netIOB/COB visualization to replace basal-only rendering

Per https://github.com/openaps/oref0/issues/1008#issuecomment-390044904
With SMB, people often get frustrated/concerned that they're not getting enough insulin when they see zero temps in the NS basal rendering, as it's difficult to mentally combine the SMB bolus bubble visualization with the NS basal rendering one into a holistic mental picture of how much insulin is (and should be) given. As an alternative, we might want to build in real-time netIOB and COB visualizations that can replace or augment the NS basal rendering.  These could be similar to the ones already generated by NS reports.  When both netIOB and COB renderings are overlaid, they can be scaled by the user's carb ratio to provide a quick visual check of whether enough insulin is on board to cover the carbs on board.
Let's use this issue to discuss what kind of visualization we'd like to make available for SMB use, and then we can open a Nightscout cgm-remote-monitor issue when we're ready to build it.

Hi Scott
I agree with this, it would be good for this to built in to either xdrip or nightscout, I'm on SMB_always which works well for me but makes the nightscout basal profile unrepresentative as you say. I turned off SMB for three weeks before my hospital appointment so that I could better demonstrate my graphs to my consultant.
Sorry to dig up an old issue, but I wondered if there had been any work in this area? Unfortunately I wouldn't know where to start...

No, this hasn’t gotten much interest yet.

A shame. We could have an additional render option for the insulin-on-board perhaps, which I think I've seen on androidAPS.


Fix gateway get dns targets

This PR includes the changes from #161 and adds a fix to detect istio's incorrect reporting of AWS ELB hosnames as Type: IPAddress in the status of istio managed Gateways.
Supersedes #161 and fixes #160 .
/kind bug
/cc @mikenairn
/cc @maleck13

The DNS for an AWS load balancer is now correctly generated, see this example https://argocd-hcg-argocd.apps.ocp-c1.prod.psi.redhat.com/applications/mctc-tenant-unstable?node=kuadrant.io%2FDNSRecord%2Fmctc-tenant-unstable%2Fexample-gw.hcg-stage.rhcloud.com%2F0


Support for changes in ConnectionMonitor API

ConnectionMonitor won't be supported beyond API version 2019-06-01 (there will be a V2 version added later), this PR is to address that and avoid introducing breaking changes down the line. What it does:

Separates ConnectionMonitor specs from NewtorkWatcher into it's own JSON file
The package for 2019-06-01 will be basically the same
The package for 2019-07-01 will reference ConnectionMonitor specs from 2019-06-01 so the future SDK packages won't break

Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[x] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[x] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[x] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Automation for azure-sdk-for-python

Can one of the admins verify this patch?

Automation for azure-sdk-for-go

SDK java failed with timeout (as usual)...

/azp run

Rebased PR to include fixes for CI that were merged into network-august-branch recently

/azp run


fix: npm package relative paths on the browser

Fixed npm package relative paths on the browser

Note: the code assumes the assets are under /aries-framework-go.

Also added a basic Vue.js sample application.

Signed-off-by: George Aristy<EMAIL_ADDRESS>

@llorllale @troyronda Just wondering if the sample app to test the JS wrapper should be inside cmd folder. May be inside test or a new example folder at higher level would seem better.

@troyronda @rolsonquadras I'm going to merge this to move faster. Feel free to suggest an appropriate path for vue-aries-framework-go. Note that the rest api stuff also resides in cmd.

@troyronda @rolsonquadras I'm going to merge this to move faster. Feel free to suggest an appropriate path for vue-aries-framework-go. Note that the rest api stuff also resides in cmd.
Edit: chucks, I can't approve my own PR

Do we have an issue to track this ? We can discuss on the path later.


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-1.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 805

Questions ?
Please refer to the Backport tool documentation


Provide a command to easily install and update OmniSharp Roslyn

Improve usability by adding a simple way of installing the OmniSharp Roslyn server.
This command currently only works on Linux, macOS, Cygwin and WSL. Windows support could be added in the future through the use of a simple PowerShell script.
For the installation to take place the user will also have to have Curl installed on their machine. This is intentional to avoid omnisharp-vim needing to know where it is installed to on the user's machine.
Running the command again will update the existing server, after stopping all running instances.
You may want to move the script used somewhere else (e.g. the OmniSharp GitHub organisation), if you are unhappy with it being in my personal repo. I have licenced the script under the Unlicence, so feel free to do so. The script could also be used in other OmniSharp editor plugins.
Potential enhancements could include:

Automatically setting g:OmniSharp_server_path if server is found under the path which the installer installs it to.
Auto-install the server if it cannot be found.

I am open to all suggestions and improvements :)

Great stuff @axvr, thanks!
I do think it would be good to have the script locally in this repo, so future devs can easily modify it themselves if things change - and if anyone wants to write a Windows version then the two will be in the same place which will be helpful.
As far as I know other O# editors already have their own versions of this - VSCode has some kind of install system and Emacs has also has a process for getting the latest roslyn. However if we add the script to this repo, we can always migrate it to a more generic OmniSharp repo later (perhaps even add it directly to OmniSharp-roslyn?). How does that sound to you?
I think we should also make the installed server path ~/.omnisharp/vim/run the default, when the user doesn't have a g:OmniSharp_server_path setting. Currently there is some old and unused code in autoload/OmniSharp/util.vim which sets the default server path. This is a leftover from when the 2 servers server (the original OmniSharp server) and roslyn were both managed as git submodules of this repo, and needed to be compiled, which was problematic and slow on many systems. So we could replace that whole if block with a quick check to see whether ~/.omnisharp/vim/run exists, and if not, run :OmniSharpInstall - perhaps after prompting the user?

Yeah, Emacs implemented something along those lines a little while ago:
https://github.com/OmniSharp/omnisharp-emacs/issues/275
If it helps, here's their PR that added Windows support: https://github.com/OmniSharp/omnisharp-emacs/pull/330

Thanks, this is really going to help people get up and running easily!

@nickspoons, thanks! If I have time over the next couple of days I'll try and get around to:

Updating the README,
Setting a default server path,
Creating a prompt to install the server if is wasn't found,
Starting work on the Windows install script.

@nickspoons, great idea. I had originally used the location ~/.omnisharp/vim in case another editor was using the ~/.omnisharp directory, but your way actually makes a lot more sense.

@stevearc, to be honest a PowerShell install script shouldn't be too difficult to create (I believe that Microsoft have implemented all of the required commands for it).
I had originally attempted to write the install script in vim script, but I found that it was too difficult to debug, and detect all of the required system information needed for the install.
Personally I prefer having many smaller programs joined together into a single program than one monolithic program which does anything, I think it helps with "Separation of Concerns" having

@stevearc, to be honest, I don't think it will take too long to write a PowerShell version of the install script because I believe that Microsoft have already implemented all of commands that are required for the installation.
The script I created is was designed to be general purpose, it is capable of installing the mono version of the server, the standard version of the server, the HTTP version, and any previous releases of the server. This was done to allow it to be used in other OmniSharp editor plugins, especially as I wrote it in a language native to the *nix family of OSs, and was planning on writing a PowerShell variant for MS Windows. At the moment the script may not be of much use, but if editor plugins are created for other editors it may be useful.
I had originally attempted to write the installer in vim script but I found that it was difficult to debug and made the code more complex than it needed to be (lots of system() calls and conditional statements). Personally I prefer the approach of having small individual programs working together to achieve a single goal.
But if you wan't to try writing it in Python instead, be my guest, I'll happily take an unbiased look at it. :slightly_smiling_face:

Since you think the PS script won't be difficult, I'll leave you to it. Thanks again for adding this!


Splashscreen doesn´t work: solution found

When I try to change the splashscreen in a existing Retropie image, the player can not show the video.
I checked the script /home/pi/RetroPie-Setup/scriptmodules/supplementary/splashscreen.sh
And at line 240, tries to execute "mpv", but by default, is not installed. You can install
#sudo apt-get install mpv
If the following message appears, you can just send the command:
#sudo apt-get install -f
pi@retropie:/home/pi/RetroPie/splashscreens# sudo apt-get install mpv
Reading package lists... Done
Building dependency tree
Reading state information... Done
You might want to run 'apt-get -f install' to correct these:
The following packages have unmet dependencies:
 libsdl2-dbg : Depends: libsdl2-2.0-0 (= 2.0.5+5mali) but 2.0.6+1mali is to be installed
 mpv : Depends: libavdevice55 (>= 6:11~beta1) but it is not going to be installed
	   Depends: libavfilter5 (>= 6:11~beta1) but it is not going to be installed
	   Depends: libegl1-mesa (>= 7.8.1) but it is not going to be installed or
				libegl1-x11
	   Depends: libguess1 but it is not going to be installed
	   Depends: libva-glx1 (>= 1.0.3) but it is not going to be installed
	   Depends: libva-x11-1 (>= 1.0.3) but it is not going to be installed
	   Depends: libwayland-egl1-mesa (>= 10.0.2) but it is not going to be installed or
				libwayland-egl1
 ppsspp-odroid : Depends: libsnappy1 but it is not going to be installed
				 Depends: libzip2 (>= 0.10) but it is not going to be installed
E: Unmet dependencies. Try 'apt-get -f install' with no packages (or specify a solution).

And try again:
#sudo apt-get install mpv
After the install, is possible to execute the video.
Now you can just restart your system and the splashscreen with video will work fine :-)

by default , MPV is installed in all images, except one, OrangePi Zero slim image. We can consider it's gonna become a bug since we're adding slim images to all boards soon.

Noted.

I have a orangepi pc, the older version of all hehehe.
I can install in another card to try to reproduce the error and check if this happen with my image or with a fresh new installation to.
Best regards

on a closer look, you probably tried to enable splashscreen in an old retropie build. A lot has changed, RetrOrangePi is based on Armbian for at least a year and includes mpv by default.

Flash a RetrOrangePi v3.0.1 image, update to v4 and help troubleshooting . Anyway, a bug was discovered before it was born, :p

:-)
Em 6 de nov de 2017 21:46, "RetrOrangePi"<EMAIL_ADDRESS>escreveu:
Flash a RetrOrangePi v3.0.1 image, update to v4 and help troubleshooting .
Anyway, a bug was discovered before it was born, :p
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/retr0rangepi/RetrOrangePi/issues/7#issuecomment-342326562,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEoDbaUHa0nYGj-CttUvQw-y4MldEyvfks5sz5pkgaJpZM4QT_UA
.

Added to slim v4.2.


fixed position div modal for map div

how to fix map position in fixed position parents ?  https://jsfiddle.net/hpfe07yy/
if you a little drag result panel (panel of fiddle) and you see map will be correct but without trick not correct

That fiddle doesn't load Leaflet, because of https issues. Try Leaflet Playground, it's a bit easier to work with for this reason among others.
From the looks of it, you need to call map.invalidateSize() after the map becomes visible, since its size has changed.
Also, please note that this is not a support forum, you should only report issues and feature requests here. You're better off asking questions of Stackoverflow or gis.stackexchange.com.


Running conditional_expected_number_of_purchases_up_to_time on ParetoNBD model throws a pandas NotImplementedError

I tried this on several different standard transactional datasets, and all gave the same error, which seems to be due to pandas/scipy incompatibility. The traceback is below, but it's a pandas NotImplementedError, found when implementing conditional_expected_number_of_purchases_up_to_time on the ParetoNBD model. The method works fine for BetaGeo with this data, and more weirdly, even when I transform the data into exactly the same format as the load_cdnow_summary dataset, the load_cdnow_summary works and my data doesn't.
My code, and a sample of the data, is attached below (f is the summary dataset)
nbd = ParetoNBDFitter(penalizer_coef=0.01)
nbd.fit(f['frequency'], f['recency'], f['T'])

nbd.conditional_expected_number_of_purchases_up_to_time(1, f['frequency'], f['recency'], f['T'])
---------------------------------------------------------------------------
NotImplementedError                       Traceback (most recent call last)
/var/folders/z7/h1rmk7f11kg5yg15rzg3xk8h0000gn/T/ipykernel_868/191803226.py in <module>
----> 1 nbd.conditional_expected_number_of_purchases_up_to_time(1, f['frequency'], f['recency'], f['T'])

~/Environments/ltv/lib/python3.8/site-packages/lifetimes/fitters/pareto_nbd_fitter.py in conditional_expected_number_of_purchases_up_to_time(self, t, frequency, recency, T)
    277         r, alpha, s, beta = params
    278
--> 279         likelihood = self._conditional_log_likelihood(params, x, t_x, T)
    280         first_term = (
    281             gammaln(r + x) - gammaln(r) + r * log(alpha) + s * log(beta) - (r + x) * log(alpha + T) - s * log(beta + T)

~/Environments/ltv/lib/python3.8/site-packages/lifetimes/fitters/pareto_nbd_fitter.py in _conditional_log_likelihood(params, freq, rec, T)
    212
    213         A_1 = gammaln(r + x) - gammaln(r) + r * log(alpha) + s * log(beta)
--> 214         log_A_0 = ParetoNBDFitter._log_A_0(params, x, rec, T)
    215
    216         A_2 = logaddexp(-(r + x) * log(alpha + T) - s * log(beta + T), log(s) + log_A_0 - log(r_s_x))

~/Environments/ltv/lib/python3.8/site-packages/lifetimes/fitters/pareto_nbd_fitter.py in _log_A_0(params, freq, recency, age)
    179
    180         rsf = r + s + freq
--> 181         p_1 = hyp2f1(rsf, t, rsf + 1.0, abs_alpha_beta / (max_of_alpha_beta + recency))
    182         q_1 = max_of_alpha_beta + recency
    183         p_2 = hyp2f1(rsf, t, rsf + 1.0, abs_alpha_beta / (max_of_alpha_beta + age))

~/Environments/ltv/lib/python3.8/site-packages/pandas/core/generic.py in __array_ufunc__(self, ufunc, method, *inputs, **kwargs)
   2027         self, ufunc: np.ufunc, method: str, *inputs: Any, **kwargs: Any
   2028     ):
-> 2029         return arraylike.array_ufunc(self, ufunc, method, *inputs, **kwargs)
   2030
   2031     # ideally we would define this to avoid the getattr checks, but

~/Environments/ltv/lib/python3.8/site-packages/pandas/core/arraylike.py in array_ufunc(self, ufunc, method, *inputs, **kwargs)
    290             # well. Previously this raised an internal ValueError. We might
    291             # support it someday, so raise a NotImplementedError.
--> 292             raise NotImplementedError(
    293                 "Cannot apply ufunc {} to mixed DataFrame and Series "
    294                 "inputs.".format(ufunc)

NotImplementedError: Cannot apply ufunc <ufunc 'hyp2f1'> to mixed DataFrame and Series inputs.

Here's the .describe() of the dataset, in case you spot any patterns. Any help would be appreciated!
Here are the relevant versions:
pandas 1.3.0
scipy 1.7.0
lifetimes 0.11.3

Update: fixed this in the end by passing in ndarrays rather than pandas Series (which is as simple as using, for instance, f['frequency'].values rather than f['frequency'] in my code above).
Ideally the function would work fine with pandas datatypes (like the rest of the package), but this is a workaround for now!

@sam-lupton, were you able to get the ParetoNBDFitter model to work with the customer_lifetime_value function provided by the GammaGammaFitter model? I can only get it to work with the BetaGeoFitter model and I think it's related to the above.


Add handling for independent encoding in Report batches to Mixer

Initial stab at adding DELTA v INDEPENDENT encoding handling to Mixer.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: douglas-reid
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

mixer/OWNERS [douglas-reid]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@douglas-reid: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
783fbf7860b9fd4494808f651190439c237a2c75
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Updated according to comments. PTAL.

/lgtm
/approve

@kyessenov: changing LGTM is restricted to assignees, and assigning you to the PR failed.

In response to this:

/lgtm
/approve

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: douglas-reid, kyessenov
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

mixer/OWNERS [douglas-reid,kyessenov]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@douglas-reid: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
783fbf7860b9fd4494808f651190439c237a2c75
link
/test istio-pilot-multicluster-e2e

prow/istio-integ-k8s-tests.sh
c7ddb73bf7d58e9bb517f9742ad734810216fa22
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@douglas-reid: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
c7ddb73bf7d58e9bb517f9742ad734810216fa22
link
/test istio-integ-k8s-tests

prow/istio-pilot-multicluster-e2e.sh
c7ddb73bf7d58e9bb517f9742ad734810216fa22
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@wenchenglu @duderino can you merge this into the 1.1 release. it is meant to (partially) address #11151 .

/lgtm
/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: douglas-reid, kyessenov
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

mixer/OWNERS [douglas-reid,kyessenov]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


[2.0.2] zh: add an alerting rule

What is changed, added, or deleted? (Required)

align with https://github.com/pingcap/dm/pull/1438
Which DM version(s) do your changes apply to? (Required)

[x] master (the latest development version, including v3.0 changes)
[x] v2.0 (TiDB DM 2.0 versions)
[ ] v1.0 (TiDB DM 1.0 versions)

What is the related PR or file link(s)?

This PR is translated from:
Other reference link(s):

Do your changes match any of the following descriptions?

[ ] Delete files
[ ] Change aliases
[ ] Need modification after applied to another branch
[ ] Might cause conflicts after applied to another branch

/lgtm


Doctest errors don't fail the build

There are doctest errors right now but the build doesn't fail as a result of them.  I've left a trivial error in (some inconsistent leading whitespace in the README.md around line 197) and as you can see from this build it still passes.
cc: @gsarma @travs

Right, and there is also an error in ConfigureTest that doesn't fail the build. Checking into this now.

I got the ConfigureTest one in my latest commit :)
ᐧ
On Mon, Jun 15, 2015 at 7:05 AM, Travis Jacobs<EMAIL_ADDRESS>wrote:

Right, and there is also an error in ConfigureTest that doesn't fail the
build. Checking into this now.
—
Reply to this email directly or view it on GitHub
https://github.com/openworm/PyOpenWorm/issues/144#issuecomment-112081643
.

Closed by #145


[Doc] update show create table and optimize show partitions

from 2.5.7, show create table does not show bucket number if user does not set bucket number when creating a table. show partitions can be run to show this info.
Signed-off-by: evelynzhaojie<EMAIL_ADDRESS>Fixes #issue
What type of PR is this:

[ ] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.1
[x] 3.0
[x] 2.5
[ ] 2.4

@Mergifyio backport branch-3.1

@Mergifyio backport branch-3.0

@Mergifyio backport branch-2.5


It is only possible to make the payment using the command: node assetsMassTransfer.js

The command fails, gives an error message:
node assetsTransfer.js
6 transfers were found, feeAssetId was overriden with 'WAVES' and fee is: 0.001...
Got error status 400 during transfer: failed to parse json message
Got error status 400 during transfer: failed to parse json message
Got error status 400 during transfer: failed to parse json message
Got error status 400 during transfer: failed to parse json message
Got error status 400 during transfer: failed to parse json message
Got error status 400 during transfer: failed to parse json message
6 transfer transactions were submitted!
0 transfer transactions were accepted!

The node assetsMassTransfer.js command already worked perfectly

Not able to reproduce.


Add a link on the course run page to easily go back to the related course page

Feature Request
https://github.com/openfun/richie/pull/410 is adding a specific page for each course run. We need to add a link to easily go back to the related course.
Describe the solution you'd like
I was thinking to add this link as a breadcrumb on top of the page content zone.
Describe alternatives you've considered
Add a simple button?
Discovery, Documentation, Adoption, Migration Strategy
This breadcrumb would be available on the search page, the course page and the course run page. The snapshot page which can exist between the course page and the course run page should not materialize as a step in the breadcrumb (it's content is already displayed on the course run page):

search > my course tile > course run 1

@2018D do we consider that the breadcrumb has solved this issue?

Yes it has.


Add an ability to apply all the rules

The problem
I had a problem with applying the rules providing by this package, and I think I have an idea how can it be improved.
Suppose, I have the next TSLint config:
{
  "extends": [
    "tslint:all",
    "tslint-eslint-rules",
  ]
}

This configuration means, that all the rules of preset tslint:all will be applied to my code. So I expect, that adding tslint-eslint-rules to extends section will do the same: the rules of tslint-eslint-rules preset will be applied to my code.
To my surprise, it isn't so. I spent about an hour to understand, why do I see next message:
$ node node_modules/tslint/bin/tslint --project .tsrc --config .tslint.json
No valid rules have been specified
Done in 1.38s.

And in the end, I had copied the content of https://github.com/buzinas/tslint-eslint-rules/blob/master/eslint_tslint.json into the config of my project and it worked.
Suggestion
I suggest to create a preset like tslint-eslint-rules:all, so that its use in the extends section led to the applying of all the rules provided by this package.

Thanks for the suggestion! Though, we don't want to be opinionated about which rules you should be using. We do have a json file in the repo that we use to linter our own project, which is opinionated.
Dupe of #229 (and others).

Thanks for the suggestion! Though, we don't want to be opinionated about which rules you should be using. We do have a json file in the repo that we use to linter our own project, which is opinionated.


Dynamically import template in Dart Polymer - Uncaught HierarchyRequestError: Failed to execute 'appendChild' on 'Node': Nodes of type 'HTML' may not be inserted inside nodes of type '#document'

 Issue by pjjjv
Originally opened as dart-lang/sdk#22125

What steps will reproduce the problem?
When imperatively adding a new import link (to a second html page) to the header of a index.html, and then appending the TemplateElement from the second html page as child to a node in the index.html, a HierarchyRequestError occurs when both the index.html and the second html page have a Polymer import.
The index.html is roughly:

    <link rel="import" href="packages/polymer/polymer.html">

  <app-router mode="hash">
    <app-route path="/" imp="packages/app_router_dart_test/pages/home-page.html" template></app-route>
  </app-router>

The app_router.dart does:
    document.head.append(importLink);
which effectively adds something like this in the above  tag:
    <link rel="import" href="packages/app_router_dart_test/pages/home-page.html">
This home-page.html contains no  tag, but basically only a template and inside:
<link rel="import" href="../../../packages/polymer/polymer.html">
<template>

</template>
Then app_router.dart also adds the above template into a node in index.html:
    router.activeRoute.append(element);
Effectively you get this:
    <app-route path="/" imp="packages/app_router_dart_test/pages/home-page.html" template>

    </app-route>
What is the expected output? What do you see instead?
If the polymer import (<link rel="import" href="../../../packages/polymer/polymer.html">)  at the top of home-page.html is removed, this works. If it stays there (it is needed for custom element definitions at deeper levels), then you get an error:
Uncaught HierarchyRequestError: Failed to execute 'appendChild' on 'Node': Nodes of type 'HTML' may not be inserted inside nodes of type '#document'. (http://localhost:8080/packages/polymer/src/js/polymer/polymer.min.js:13)
Code can be browsed at (but I don't suggest to try to build it yet; perhaps I can make a smaller version):
https://github.com/pjjjv/app-router-dart-test/tree/f0d362935c4754c529844e6dfe88dedde5b0ec4a
The only import I see double on the Network tab in the dev console is /packages/polymer/src/js/polymer/polymer.min.js, but I am not sure where this comes from.
There are plenty of reports surrounding this cryptic error, notably http://stackoverflow.com/questions/25918498/dynamically-import-html-template-in-dart-polymer.
I wonder if Polymer.Dart should not be more robust here, and if, really, it involves two duplicates being imported, the framework should not detect this and avoid it better.
dart 1.8.3
polymer 0.15.5

 Comment by jakemac53

Oh whoops, I misread the original post.
I presume that the primary issue here is that during the build process we inline html imports. This breaks the automatic deduplication that you usually get with html imports. For dart alone this is actually not an issue because of how dart imports work, but for javascript it is (since it just executes immediately anything that is seen).
Basically, we need some way of informing the browser (and polyfill) that we have already inlined some imports.

 Comment by pjjjv

Thanks again. I think that is what I described in the issue, right? It works, but has its limitations.
Anyway, I found #17873 through the code doc on Polymer.import() and that seems what I need. Either that or a better error message here. You may close/mark-duplicate.

 Comment by jakemac53

Sorry I wasn't clear, I meant leaving the polymer import in your index.html, but removing it everywhere else. That should work properly even though it breaks convention.

 Comment by jakemac53

Does it work if you remove the polymer import from your imported file (leaving only the one on your homepage)? This will give you a warning today, but it should work still as long as you are importing polymer somewhere.
In general though, dynamically injecting html imports is not well supported today in dart. You can't have any dart files in them (or they won't be loaded at least).

 Comment by pjjjv

By design, removing the polymer on the index.html shouldn't work, because there is that app-router Polymer custom element that is doing the dynamic importing in the first place.
It's good to know it's not well supported. I'll have a look at other methods for lazy loading then that have been demonstrated with dart.

 Comment by jonaskello

Nice to see that someone is working on a dart port of app-router.

 Comment by jakemac53

marking as duplicate

Added Duplicate label.
Marked as being merged into dart-lang/polymer-dart#149.


glsl precision mediump float

Version

Phaser Version: 3.52.0+
Operating system: Android 10
Browser: chrome

Description
'precision mediump float' in Multi-vert.js & Multi-frag.js
causes strange spots when rendering on some Android devices.
Hope to fix it as soon as possible, or provide an interface that overloads the fragShader in MultiPipeLine to help fix this issue.
Example Test Code
Additional Information
Blue Artifacts in WEBGL on Mobile #5659
spots appear when rendering #5655

Thank you for submitting this issue. We have fixed this and the fix has been pushed to the master branch. It will be part of the next release. If you get time to build and test it for yourself we would appreciate that.


Write only parameter groups that have changed rather than all parameter groups in the checkpoint

When running git theta add all of the parameter groups are saved in .git_theta/<path to model> even if only a few of the parameter groups were actually modified. Instead of writing the whole model to disk every time we run git theta add, check what has changed and only write those parameters to disk.

There was talk of using the metadata file to implement this to avoid reading both the checkpoint file and the git-theta model dir. This is a great idea. I have a suggestion on the API/implementation for this through.
Instead of something like looping through the model parameters and comparing them to the metadata file, we can have the git-theta model dir reading be lazy. Instead of returning actual parameter values it returns some object with tensor meta data (shape, dtype, hash, and maybe even a __eq__ function that knows how to hash/compare to a real tensor [I got myself excited with this lol]!) with methods to read and replace itself with the real tensor.
With this API, comparisions between the checkpoint tree and the model dir tree would look just like any tree intersection instead of being custom.
It could be overkill as this is similar to an approach we have in t5x where the main use of lazy arrays is for checkpoint manipulation for huge models which might not be a use case we care about now (but could be useful for parameter-efficient update applications in the future).

We talked in coworking and the main approach we decided on was to:

Build the model_dict mapping parameter names to real tensor values
Build a lazy_model_dict mapping parameter names to LazyArray values

This will be built from the previous metadata file, which may need to be gotten through git show HEAD:path/to/checkpoint

Iterate through the two versions together checking lazy == real (note lazy needs to be the LHS for now)
Only write values when they don't evaluate to equal.

LazyArray skeleton
class LazyArray:
  def __init__(self, shape, dtype, hash):
    self._shape = shape
    self._dtype = dtype
    self._hash = hash

  @property
  def shape(self):
    return self._shape

  @property
  def dtype(self):
    return self._dtype

  @property
  def hash(self):
    return self._hash

  def __eq__(self, other):
    if self.shape != other.shape or self.dtype != other.dtype:
      return False
    if isinstance(other, LazyArray):
      return self.hash == other.hash
    else:
      return self.hash == hash_array(other)

Eventually we plan to extend the lazy array with the ability to load the value it represents.

Note https://github.com/r-three/git-theta/pull/90 has landed. It converts the metadata file into a nested format which should make the creation of the lazy array easier

Note: The new multi-pointer approach https://github.com/r-three/git-theta/pull/114 will change how we solve this. That change also makes this important as it allows us to skip piping parameters to git-lfs


Update Seqerakit to v0.4.6

As per the esteemed Esha Joshi (Seqerakit author), v0.4.6 was released on Mar 3, 2023. Update Ansible script to bump package 0.4.5 to 0.4.6.

As per @ejseqera , 0.4.7 was released last week. Looks like my dawdling paid off. Will bump to latest and not 0.4.6.

Done.


extend the input payload forward

注意：提问前，请先看FAQ(Please read FAQ before file an issue) https://github.com/ossrs/srs/issues/2716

描述(Description)

描述你遇到了什么问题(Please description your issue here)
I want to expand the forward model to withstand more input loads. According to the publisher's description, forward cannot use cdn, so a large number of incoming streams is impossible. So is there any way to make it work? does forward take more input load.Thanks

SRS版本(Version): xxxxxx

SRS的日志如下(Log):

xxxxxxxxxxxx

SRS的配置如下(Config):

xxxxxxxxxxxx

重现(Replay)

重现Bug的步骤(How to replay bug?)

xxxxxx
xxxxxx
xxxxxx

期望行为(Expect)

描述你期望发生的事情(Please describe your expectation)

Is there any way to extend forward's input payload

For serving a large amount of publishers or streams, you should use origin cluster, please read the wiki about it.

For discussion or idea, please ask in discord.
This issue will be eliminated, see #2716


How to use Google Places API Key?

Hi, I read the documentation but cannot find a way to use a Google API Key (to increase the number of requests), is there a way to do it?
I know I could add key param inside the URL
<script type="text/javascript" src="http://maps.googleapis.com/maps/api/js?key=XXXX&sensor=false"></script>
But is that safe to embed an API Key on the client side?

Read the Google API docs ;)
<script async defer src="https://maps.googleapis.com/maps/api/js?key=YOUR_API_KEY&callback=initMap"
  type="text/javascript"></script>

You now have to use HTTPS, to ensure it is safe and the key (and your location?) is not intercepted.


Export policy violations to AWS Security Hub

Hi,
I am exploring ways to export policy violations to AWS Security Hub. Is this something you've considered as part of this project? Are you aware of any other approach to achieve that?
Thanks!

Hey, because I don't use AWS I am not aware of this tool. Do you have an API or something to consider? Then I am happy toi have a look on it and how to implement it as new target.

Hey @fjogeleit, there's this link to Security Hub API reference: https://docs.aws.amazon.com/securityhub/1.0/APIReference/Welcome.html
I am not very familiar with it either myself. I've seen Trivy does support integration with Security Hub, and what they do is generating a report with format ASFF, which then can be pushed to Security Hub
https://github.com/aquasecurity/trivy/blob/main/docs/tutorials/integrations/aws-security-hub.md

thanks, I will check it in the upcoming weeks and if there is a good way to integrate it

First draft of the AWS securityhub integration. Its a new target which pushes new results to the security hub


osx package should make available mruby-cli in user PATH

cd #10 #17

some resources that could be useful to build a osx package

http://hogliux.github.io/bomutils/
izpack.org
http://stackoverflow.com/a/8016507/831180
http://www.shivkaushal.co.uk/blog/2010/12/unpacking-an-osx-pkg-file/
https://coolaj86.com/articles/how-to-unpackage-and-repackage-pkg-osx.html
http://stackoverflow.com/a/11299907/831180


hub fork --help unexpected behaviour

I'm new to hub, and trying to fork a repo left me somewhat perplexed. You'll see what I mean.
$ git clone git@github.com:github/hub.git
Cloning into 'hub'...
// ...

$ cd hub

$ git fork nicolaiskogheim/hub
Enter your github username
nicolaiskogheim
Enter host password for user 'nicolaiskogheim':
{
  "message": "Must specify two-factor authentication OTP code.",
  "documentation_url": "https://developer.github.com/v3/auth#working-with-two-factor-authentication"
}
Cloning into 'hub'...		# <---- wat
Username for 'https://github.com': ^C

$ hub fork --help                                                                                git:(master)
Updating nicolaiskogheim
From ssh://github.com/github/hub
 * [new branch]      1.11-stable -> nicolaiskogheim/1.11-stable
 * [new branch]      1.12-stable -> nicolaiskogheim/1.12-stable
 * [new branch]      2.2-stable -> nicolaiskogheim/2.2-stable
 * [new branch]      gh-pages   -> nicolaiskogheim/gh-pages
 * [new branch]      master     -> nicolaiskogheim/master
 * [new branch]      revamp-issues -> nicolaiskogheim/revamp-issues
new remote: nicolaiskogheim

fork says it fails even though it worked

hub fork --help did not show help, but instead of erroring on an unsupported option, it did something else.

The repository was forked to my user.

Sorry for the trouble! Can you try with hub 2.2.3? I'm pretty sure those bugs have already been fixed.

I can't reproduce this with hub 2.2.3, so I'm assuming this has been fixed in a recent release. If you still have problems using hub 2.2.3, please reopen! Thanks

I'm pretty sure those bugs have already been fixed.

Ah, yes. I didn't check that I had the latest version, sorry. I only knew I used the latest one available on brew. Or, at least i thought i did, ref #1222.
I cannot reproduce with 2.2.3. Thanks for your feedback.


Fix GetAccessor() call returning 0 stride when accessor is loaded from cache

Currently if a sparse accessor is loaded a second time, it is loaded from SparseAccessorsStridesCache but the stride computation is mistakenly skipped. This commit store the stride computed into another cache and load its value on the cached load.

@cosimo-dw nice catch :) thanks


Example: BERT Question and Answer

Use a TensorFlow Lite model to answer questions based on the content of a given passage.

[x] #56

[x] Implement BasicTokenizer
[x] Implement WordpieceTokenizer
[x] Implement FullTokenizer

[x] Load the model and get results

related link: https://www.tensorflow.org/lite/examples/bert_qa/overview

Done

This should be the last task before shipping v0.4.0


fix: Announcement banner flicker (#2565)

What is the purpose of the change
Fix the Announcement banner flicker, when the Banner has already been closed, it doesn't show on refresh.
Brief Changelog

Moved Banner implementation to announcement.tsx.
show announcements only on client side.
changed the utilization of useLocalStorageState for useLocalStorage for preventing a one-frame flicker.

Testing and Verifying
This change has been tested locally , and is verified by screen recording.
https://github.com/osmosis-labs/osmosis-frontend/assets/4956754/d4e903dc-f129-41f0-bc54-b8f42ecf4739
Closes: #2565

@BrianCraig a dev doing some unrelated work ended up fixing this, but he drew inspiration from your code so I want to pay you the bounty regardless.
Can you email<EMAIL_ADDRESS> Thanks!


Removes path prefixes from admin URLS

What's this PR do?
Removes path prefixes from admin URL's. This should solve :all the things: (hopefully)
How should this be manually tested?
Can you create users now via admin interface?
Are all path prefixed url's redirected to global prefix when using admin pages?
Any background context you want to provide?
The bug being reported only occurred on path prefixed admin pages, and we dont need path prefixes on admin pages. To reconstruct the URL I took the same approach @angaither did for share links.
https://github.com/DoSomething/phoenix/pull/5646/files
What are the relevant tickets?
Fixes #5566
cc: @blisteringherb

:+1:


Centralise FalcoSidekick UI

Hello, is there any way to setup centralize Sidekick UI and configure all of deployed falco/sidekick agents across different cluster to point to single UI.
I am deploying falco to several clusters and wondering if its possible to have a central UI to review all of falco events from all clusters.

hi, i think it's already possible with change value of configmap in falco values L822
http_output:
# -- Enable http output for security notifications.
enabled: false
# -- When set, this will override an auto-generated URL which matches the falcosidekick Service.
# -- When including Falco inside a parent helm chart, you must set this since the auto-generated URL won't match (#280).
url: ""
ex :
http_output:
enabled: true
url: http://falco-falcosidekick.falcosecurity.svc.cluster.local:2801

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle rotten

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

@poiana: Closing this issue.

In response to this:

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Compiling with -fPIC on Linux

While playing around with building a package that exposes an SQLite Dart API, using the new native assets feature, I found that it's currently not possible to build SQLite with native_toolchain_c on Linux:
Unhandled exception:
ProcessException: Full command string: '/usr/lib/llvm-14/bin/clang /home/runner/work/sqlite_dart_native/sqlite_dart_native/src/sqlite/sqlite3.c --shared -o /home/runner/work/sqlite_dart_native/sqlite_dart_native/.dart_tool/native_assets_builder/fb4a407d42b5[8](https://github.com/blaugold/sqlite_dart_native/actions/runs/5906585326/job/16022913631#step:6:9)5687ec9d3b5a6c0949b/out/libsqlite.so -DRELEASE'.
Exit code: '1'.
For the output of the process check the logger output.
  Command: /usr/lib/llvm-14/bin/clang /home/runner/work/sqlite_dart_native/sqlite_dart_native/src/sqlite/sqlite3.c --shared -o /home/runner/work/sqlite_dart_native/sqlite_dart_native/.dart_tool/native_assets_builder/fb4a407d42b585687ec9d3b5a6c0949b/out/libsqlite.so -DRELEASE
#0      runProcess (package:native_toolchain_c/src/utils/run_process.dart:92:5)
<asynchronous suspension>
#1      RunCBuilder.runClangLike (package:native_toolchain_c/src/cbuilder/run_cbuilder.dart:102:5)
<asynchronous suspension>
#2      RunCBuilder.run (package:native_toolchain_c/src/cbuilder/run_cbuilder.dart:83:7)
<asynchronous suspension>
#3      CBuilder.run (package:native_toolchain_c/src/cbuilder/cbuilder.dart:116:7)
<asynchronous suspension>
#4      main (file:///home/runner/work/sqlite_dart_native/sqlite_dart_native/build.dart:16:3)
<asynchronous suspension>
Building native assets for package:sqlite_dart_native failed.
build.dart returned with exit code: 255.
To reproduce run:
/opt/hostedtoolcache/dart/3.2.0-74.0.dev/x64/bin/dart --packages=/home/runner/work/sqlite_dart_native/sqlite_dart_native/.dart_tool/package_config.json /home/runner/work/sqlite_dart_native/sqlite_dart_native/build.dart --config=/home/runner/work/sqlite_dart_native/sqlite_dart_native/.dart_tool/native_assets_builder/fb4a407d42b585687ec9d3b5a6c0949b/config.yaml
stderr:
Unhandled exception:
ProcessException: Full command string: '/usr/lib/llvm-14/bin/clang /home/runner/work/sqlite_dart_native/sqlite_dart_native/src/sqlite/sqlite3.c --shared -o /home/runner/work/sqlite_dart_native/sqlite_dart_native/.dart_tool/native_assets_builder/fb4a407d42b585687ec9d3b5a6c0949b/out/libsqlite.so -DRELEASE'.
Exit code: '1'.
For the output of the process check the logger output.
  Command: /usr/lib/llvm-14/bin/clang /home/runner/work/sqlite_dart_native/sqlite_dart_native/src/sqlite/sqlite3.c --shared -o /home/runner/work/sqlite_dart_native/sqlite_dart_native/.dart_tool/native_assets_builder/fb4a407d42b585687ec[9](https://github.com/blaugold/sqlite_dart_native/actions/runs/5906585326/job/16022913631#step:6:10)d3b5a6c0949b/out/libsqlite.so -DRELEASE
#0      runProcess (package:native_toolchain_c/src/utils/run_process.dart:92:5)
<asynchronous suspension>
#1      RunCBuilder.runClangLike (package:native_toolchain_c/src/cbuilder/run_cbuilder.dart:[10](https://github.com/blaugold/sqlite_dart_native/actions/runs/5906585326/job/16022913631#step:6:11)2:5)
<asynchronous suspension>
#2      RunCBuilder.run (package:native_toolchain_c/src/cbuilder/run_cbuilder.dart:83:7)
<asynchronous suspension>
#3      CBuilder.run (package:native_toolchain_c/src/cbuilder/cbuilder.dart:[11](https://github.com/blaugold/sqlite_dart_native/actions/runs/5906585326/job/16022913631#step:6:12)6:7)
<asynchronous suspension>
#4      main (file:///home/runner/work/sqlite_dart_native/sqlite_dart_native/build.dart:16:3)
<asynchronous suspension>

stdout:
Running `which clang`.
Running `/usr/lib/llvm-[14](https://github.com/blaugold/sqlite_dart_native/actions/runs/5906585326/job/16022913631#step:6:15)/bin/clang --version`.
Running `/usr/lib/llvm-14/bin/clang --version`.
Running `/usr/lib/llvm-14/bin/clang /home/runner/work/sqlite_dart_native/sqlite_dart_native/src/sqlite/sqlite3.c --shared -o /home/runner/work/sqlite_dart_native/sqlite_dart_native/.dart_tool/native_assets_builder/fb4a407d42b585687ec9d3b5a6c0949b/out/libsqlite.so -DRELEASE`.
/usr/bin/ld: /tmp/sqlite3-547e69.o: warning: relocation against `sqlite3_temp_directory' in read-only section `.text'
/usr/bin/ld: /tmp/sqlite3-547e69.o: relocation R_X86_64_PC[32](https://github.com/blaugold/sqlite_dart_native/actions/runs/5906585326/job/16022913631#step:6:33) against symbol `sqlite3_version' can not be used when making a shared object; recompile with -fPIC
/usr/bin/ld: final link failed: bad value
clang: error: linker command failed with exit code 1 (use -v to see invocation)

Error: Compiling native assets failed.

As far as I understand it, any non-trivial shared library needs to be compiled with -fPIC, but at least the version of Clang that comes with Ubuntu 22.04 does not enable it by default on Linux. Most distributions of GCC seem to enable it by default.
Here is a discussion regarding this difference on the LLVM forum.

  final cbuilder = CBuilder.library(
    name: packageName,
    assetId: 'package:$packageName/${packageName}.dart',
    sources: [
      'src/$packageName.c',
    ],
  );

This should take some argument for compiler flags.
PRs (and design suggestions) are welcome!

This should take some argument for compiler flags. (And probably also include paths, defines https://github.com/dart-lang/native/issues/61, and possibly all kinds of other things.)

I've started by taking a stab at macro defines (#120). Supporting compiler flags generally will definitely also be very useful. But I feel like it would be good to provide sensible defaults for PIC/PIE flags (#121 ). The cc create does that, too.


fix: guard against uni-v2 zapper assets with undefined symbols

Description

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

closes #
Risk

Testing

Engineering

Operations

Screenshots (if applicable)

Current dependencies on/for this PR:

develop

PR #4469   👈

This comment was auto-generated by Graphite.


(FACT-2596) Use file_exist? instead of test command

This would allow tests to run without the need for cygwin

jenkins please test this on debian9-64a,debian10-64a,fedora30-64a,osx1014-64a,redhat5-64a,centos7-64a,centos8-64a,redhat7-64a,redhat7-POWERa,sles12-64a,sles15-64a,solaris114-64a,ubuntu1604-64a,ubuntu1604-32a,ubuntu1804-64a,windows10ent-64a,windows2012r2-64a,windows2016-64a,windows2019-64a

Waiting for CLA signature by @florindragos
@florindragos - We require a Contributor License Agreement (CLA) for people who contribute to Puppet, but we have an easy click-through license with instructions, which is available at https://cla.puppet.com/
Note: if your contribution is trivial and you think it may be exempt from the CLA, please post a short reply to this comment with details. http://docs.puppet.com/community/trivial_patch_exemption.html

jenkins please test this on debian9-64a,debian10-64a,fedora30-64a,osx1014-64a,redhat5-64a,centos7-64a,centos8-64a,redhat7-64a,redhat7-POWERa,sles12-64a,sles15-64a,solaris114-64a,ubuntu1604-64a,ubuntu1604-32a,ubuntu1804-64a,windows10ent-64a,windows2012r2-64a,windows2016-64a,windows2019-64a


feat(cli): init with webapp dist

下载web.zip解压后得到的文件目录是dist，web_server默认的dist目录是web，这两个名称要统一下

fix issues #19

Codecov Report

Merging #35 (1c4b069) into master (b1ca356) will decrease coverage by 0.89%.
The diff coverage is 17.74%.

@@            Coverage Diff             @@
##           master      #35      +/-   ##
==========================================
- Coverage   74.43%   73.53%   -0.90%
==========================================
  Files          62       62
  Lines        4439     4496      +57
==========================================
+ Hits         3304     3306       +2
- Misses       1135     1190      +55

Impacted Files
Coverage Δ

src/main.rs
46.00% <17.74%> (-12.55%)
:arrow_down:

Continue to review full report at Codecov.

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

@Eliot00 后续改一下自动化构建那个吧，改为 web


Support channelMax and frameMax negotiation

This library can't connect if the server has limited channelMax.

Ok I pushed a fix for this, it will no honor channel max and frame max BUT it doesn't enforce and channel limits from the client.  Im not sure what the correct if you try to do something when the channel limit is reached.  Do you error immediately or wait and queue things up? I think i'm leaning towards just error quickly.
you can test it with<EMAIL_ADDRESS>heres the commit : https://github.com/dropbox/amqp-coffee/commit/bfc9fd126de51a716d611de8cdc360d566e42e0c

Thanks!


Simpler classful component check (fixes #458)

Instead of checking for the isReactComponent property (which is internal to React), it seems advantageous to check for render(). This improves Preact support (when using recompose with non-preact-compat components) but also importantly works with components that don't inherit from Component.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #463   +/-   ##
=======================================
  Coverage   88.86%   88.86%
=======================================
  Files          53       53
  Lines         386      386
=======================================
  Hits          343      343
  Misses         43       43

Impacted Files
Coverage Δ

src/packages/recompose/isClassComponent.js
100% <ø> (ø)
:arrow_up:

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 adbefc0...1c9cf7b. Read the comment docs.

To maintain compatibility with what he is already doing in isClassComponent, it would have to return true for React.Component but false for React.PureComponent and React.AsyncComponent, since React.Component is the only one that has the isReactComponent prop.  All of them would have render, but only React.Component has setState and forceUpdate.  If he doesn't care about changing it so it returns true for PureComponent then checking for render would work as well.

^ Just saw your comment about being compatible with things that are classes but don't extend React.Component.  Again goes back to what kind of contract he is going for with isClassCompoenent - is it just trying to return something that is not a functional component (in which case checking for render is perfect), or is it trying to check only that it is extending React.Component

Thank you

In case other library authors see this PR and adopt the same approach, I’m encouraging you not to.
As noted in https://github.com/acdlite/recompose/pull/538#issuecomment-334820814, this change was actually a major change (and broke people’s code in production). It can break code because render might be on the instance rather than on the prototype in some cases. It can also break code in React Native.
Here is the exact algorithm React is using:

Check for fn.prototype && fn.prototype.isReactComponent
If it is truthy

This is a class component and must be new'd

If it is falsy

Call the function without new
If the return value is an object and has a render method on it

Treat it as a class instance

Otherwise, treat it as a functional component

React plans to continue using this heuristic. In this sense, if for some reason your library needs to know whether a component is "newable" or not, fn.prototype && fn.prototype.isReactComponent is a safe way to check it.
I understand we might not be able to persuade Preact or others to adopt the same heuristic, but I hope this is helpful.


update docker image

Release new docker image for stallion server.

Use auth in dockerfile.

Stable created.


Fix wrong env var for OPENAI_MAX_TEMPERATURE in docs

@typpo honestly, should considering renaming the env var to OPENAI_TEMPERATURE cos max temperature doesn't really make sense.

  The var is now OPENAI_TEMPERATURE so we won't need to make this change - thanks for raising this in the main repo! https://github.com/typpo/promptfoo/pull/29


Added MagicaVoxel Exporter

Added Magicavoxel exporter, tested it and it seems to work (i haven't tested it with a model that has more than 255 colors though), Although i removed some stuffs from before PR cause they don't really help much.

I use linux but after running hexden wia wine, i didn't notice much difference in file analysis other than some chunks were missing (PACK,,MATT) which is ok and understandable

Can you please pull the latest develop branch in? Yours is still on 1.8.0 (Should be 1.8.02)

It should be removed if necessary


[rtl/alert_handler] Fix sec_cm related assertions

Fix one assertion about escalatoin check in FSM.
In default it should stay at idle mode, but in case of sec_cm injection,
it might go to FsmErrorSt.
Signed-off-by: Cindy Chen<EMAIL_ADDRESS>

I will disable this assertion instead.


Patch nvprof so it doesn't link outside the conda environment

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

When overlinking check is turned on, the following error message occurs:
ERROR (cuda-nvprof,bin/nvprof): runpaths ['$ORIGIN/../extras/CUPTI/lib64', '$ORIGIN/../targets/x86_64-linux/lib'] found in
/croot/cuda-nvprof_1715204971555/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_plac
ehold_placehold_placehold_placehold_place/bin/nvprof"

@conda-forge-admin, please rerender

Curious as to why overlinking checks are turned off sometimes. IMHO an important check for correct functionality of a package.


Terraform output failing, but not recorded as failed

The task reports as succeeded, but looking at the logs it is very apparent that it should've been marked as failed instead:

Reporting the proper task status is fundamental to building a stable CI pipeline.
2024-02-28T14:18:10.4624112Z ##[section]Starting: Terraform Output
2024-02-28T14:18:10.4630081Z ==============================================================================
2024-02-28T14:18:10.4630241Z Task         : Terraform
2024-02-28T14:18:10.4630303Z Description  : Execute terraform commands to manage resources on AzureRM, Amazon Web Services(AWS) and Google Cloud Platform(GCP)
2024-02-28T14:18:10.4630449Z Version      : 4.227.24
2024-02-28T14:18:10.4630538Z Author       : Microsoft Corporation
2024-02-28T14:18:10.4630614Z Help         : [Learn more about this task](https://aka.ms/AAf0uqr)
2024-02-28T14:18:10.4630706Z ==============================================================================
2024-02-28T14:18:11.1468734Z [command]/opt/hostedtoolcache/terraform/1.7.4/x64/terraform output -json
2024-02-28T14:25:42.4777063Z ╷
2024-02-28T14:25:42.4779840Z │ Error: Failed to load state: Error retrieving keys for Storage Account "XXX": autorest/Client#Do: Preparing request failed: StatusCode=0 -- Original Error: clientCredentialsToken: received HTTP status 401 with response: {"error":"invalid_client","error_description":"AADSTS700024: Client assertion is not within its valid time range. Current time: 2024-02-28T14:21:42.4623915Z, assertion valid from 2024-02-28T14:03:09.0000000Z, expiry time of assertion 2024-02-28T14:13:08.0000000Z. Review the documentation at https://docs.microsoft.com/azure/active-directory/develop/active-directory-certificate-credentials . Trace ID: 3e7d8c18-c8c8-426b-a629-7fb4ad761700 Correlation ID: 8202538a-1ca5-4ccf-be51-caf862f60b33 Timestamp: 2024-02-28 14:21:42Z","error_codes":[700024],"timestamp":"2024-02-28 14:21:42Z","trace_id":"3e7d8c18-c8c8-426b-a629-7fb4ad761700","correlation_id":"8202538a-1ca5-4ccf-be51-caf862f60b33","error_uri":"https://login.microsoftonline.com/error?code=700024"}
2024-02-28T14:25:42.4781479Z │
2024-02-28T14:25:42.4781790Z │
2024-02-28T14:25:42.4782095Z ╵
2024-02-28T14:25:42.5070749Z ##[section]Finishing: Terraform Output

Hi thanks for reporting. If you run the command locally, what exitcode do you get?
Br
Manuel

I haven't tried reproducing this issue locally. This specific error is because the task-provided auth token expired. I have no idea how I could reproduce that locally.

ok got it. I was just curious what exitcode terraform commad gets when this happens.
br
Manuel

I face the same issue. Exitcode is 1. It happens when TF can't load its state:
Pipeline continues as normal with this error:
Error: Failed to load state: Error retrieving keys for Storage Account "***": autorest/Client#Do: Preparing request failed: StatusCode=0 -- Original Error: clientCredentialsToken: received HTTP status 401 with response: {"error":"invalid_client","error_description":"AADSTS700024: Client assertion is not within its valid time range. Current time: 2024-06-30T22:04:39.8297163Z, assertion valid from 2024-06-30T21:40:32.0000000Z, expiry time of assertion 2024-06-30T21:50:32.0000000Z. Review the documentation at https://docs.microsoft.com/azure/active-directory/develop/active-directory-certificate-credentials . Trace ID: *** Correlation ID: *** Timestamp: 2024-06-30 22:04:39Z","error_codes":[700024],"timestamp":"2024-06-30 22:04:39Z","trace_id":"***","correlation_id":"***","error_uri":"[https://login.microsoftonline.com/error?code=700024"}](https://login.microsoftonline.com/error?code=700024%22})

Reproduced locally:
PS C:\...\tf> terraform output

Error: Failed to load state: blobs.Client#Get: Failure responding to request: StatusCode=403 -- Original Error: autorest/azure: Service returned an error. Status=403 Code="AuthorizationFailure" Message="This request is not authorized to perform this operation.\nRequestId:***\nTime:2024-06-30T22:39:11.4584662Z"

PS C:\...\tf> $LastExitCode
1


Clarify Threshold Fingerprint Generation

While investing the Java test failures, I noticed that some of the ThresholdCondition's fingerprint generation results don't line-up with the TestVectors published in the CryptoCondition RFC project.
For example, looking at the fingerprintContents binary in 0008_test-basic-threshold.json, I see the following:
308184800102A17FA12B8020451FE15F16299D495993FE692DB989E56A5230A90476F77392A3CD3213C0733F810302040382020308A32780204DD2EA7F85B3EACB8F19058E8360955C32E74C124392A1F44660739709C539C38103040000A4278020799239ABA8FC4FF7EABFBC4C44E69E8BDFED993324E12ED64792ABE289CF1D5F8103020000

However, in the Java code, the following call...
BaseEncoding.base16().encode(CryptoConditionReader .readFulfillment(BaseEncoding.base16().decode(testVector.getFulfillment())).getCondition().getFingerprint());

yields the only following fingerprint value:
31AF10E168BA56AB38BEBF27065413C76896A6683C53D746F15A6438FD2A6C66

I think the source of the problem is that method ThresholdSha256Condition.getFingerprintContents() only looks at the subconditions when computing the fingerprint. When CryptoConditionsReader reads the binary of the fulfillment in the test vector, it yields a ThresholdFulfillment with one subcondition and two subfulfillments (one RsaSha256Condition and then one PrefixSha256Fulfillment and a Ed25519Fulfillment).
Is it possible that the implementation is wrong, and should somehow be considering the genrated Condition of each subfulfillment, too?
As it stands, the fingerprint in the TestVector seems to have much more data than anything I can generate using the Java library.

Off the top of my head that test vector looks wrong. The fingerprint should be a constant length.

@justmoon ⬆

I created https://github.com/rfcs/crypto-conditions/issues/27 so this issue doesn't get lost in the RFC project (and for linkage).

Bump @justmoon


Extend 9 scenario

Extend 9 scenario to use both linuxbridge and ovs with vlan + vxlan neutron drivers.

Deployment was tested here


Rename 'Trash' to 'Bin' for en_GB

Well the files that I updated are just for use in the UI and have nothing to do with RFCs. The USA uses many different words and spellings to the rest of the English speaking world. See https://www.lexico.com/definition/trash and https://www.lexico.com/definition/bin for reference.

Shouldn't it be "Rubbish" then?

I've never seen any piece of software with GB language selected using "Bin" for the folder containing deleted files, emails, etc.
The name of that folder is normally "Deleted" [eg. see Thunderbird for emails, see Owncloud/Nextcloud for files, etc.] when GB language is selected, but "Trahs" is also common. As said, never seen "Bin" though.
My 2 cents..

'Bin' or 'Deleted' is not a big deal. But not 'Trash'.

I've never seen any piece of software with GB language selected using "Bin" for the folder containing deleted files, emails, etc. The name of that folder is normally "Deleted" [eg. see Thunderbird for emails, see Owncloud/Nextcloud for files, etc.] when GB language is selected, but "Trash" is also common. As said, never seen "Bin" though.

This is gmail:


Add CLI for creating mask from shapefile

Allow user to create a mask netcdf file from a template netCDF file and a shapefile.  Primary use would be for masking out pixels from other NetCDF files in other CLI commands, e.g., render_netcdf.
Same basic usage as rasterio's rio mask command.

Resolved by 36740e4e6a9e28b5f432d11e515811c58c2f02fa


Find a suitable name for our unity package

We should find a name for our unity package. This is necessary for naming the repository, namespaces etc.

3Denga


[Enhancement] mv add partition in batch

What type of PR is this：

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Moving synchronized partitions with the base table can take a long time (8-20 minutes for 2000 partitions) if too many partitions need to be added. To address this issue, I propose changing the process to add partitions in batches. However, due to the lack of support for deleting multiple partitions in the current SQL syntax, this issue cannot be handled at this time.
https://hsn4s18zbn.feishu.cn/docx/NGYWdw5VNop6F1xzJ9wcygfnnve
Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[x] 3.0
[x] 2.5
[ ] 2.4
[ ] 2.3

@Mergifyio backport branch-2.5


Help, System cannot start! Logo Flashing

This is what it's doing right now.
https://github.com/TheGammaSqueeze/GammaOS/assets/2152195/60d720df-1921-4bcb-bb34-34575df04c0c
I did connected a USB cable to copy some files into its download folder, (size in 3g) after its done in about 80%, the connection lost, and the device gets into this logo screen. Press the power button not working.
Please help 🙏

Fixed. Did adb reboot bootloader, then fastboot reboot recovery, then click power button to restart the device help to get back into the system successfully.


Add a tags column to dashboard

The tags column on the dashboard can be used to show extra information about the project of a task assignment. Users can specify the tags to the project by adding a tags field to the project_data field.

Coverage increased (+0.002%) to 93.743% when pulling 4d5d4738bd9ff3af6b7db2b92ec80a01857252a7 on add-project-tag into e98e4a72356b84372474a14c452d7f887b6ccd63 on master.


Keyboard Paste automatically trim all the contents beginning space

I've been using TagUI to do copy and pasting of website contents that contain tables of data. I would eventually copy this data into an Excel spreadsheet.
I find sometimes the table data will have a space at the beginning of the cell, and in order for the formulas to work I need to do a find [space] and replace with empty using TagUI.
I'm wondering if a new update can include this new feature, before pasting
Keyboard [cmd]v the data in the clipboard is already trim() of the extra spaces in the beginning of the cells.
Example
600.00
50000
After paste
600.00
50000
For people who are using TagUI I believe this will be a welcome feature.

is possible to share the code ?

    click secondmarker.png
    wait 3
    click secondmarker.png
    //wait 2
    wait 2
    keyboard [down]
    //temp = []
    //temp = clipboard()
    clipboard() = del_chars(clipboard(), ' ')
    keyboard [cmd]v

i just introduced clipboard() = del_chars(clipboard(), ' ')
but there is an error now
wait 2
keyboard [down]
ERROR - left side of assignment is not a reference.
but i think i need to keep the "contents" inside the [cmd]c which i assume is clipboard()
how to modify the code to keep the "contents" but remove the spaces in the contents?
example of contents could be
[[ 500, 600],[ 700, 800]]
i need
[[500,600],[700,800]]
due to need to paste inside an XLSX file sheet for the formulas to work

https://www.w3schools.com/howto/howto_js_copy_clipboard.asp
type //*[@id="myInput"] as [clear]500, 600
click //*[@id="main"]/div[3]/button
text_contents = clipboard()
echo `text_contents`
echo Removing spaces . . .
re = /\s/g
new_text = text_contents.replace(re,'')
echo `new_text`
clipboard(new_text)
type //*[@id="myInput"] as [clear]`new_text `
echo Done !

Hi @lookang you can also use the del_chars() helper function, like this:
text = ' 500'
newText = del_chars(text, ' ')
echo `newText`


Align documentation with samples

Hey,
First of, thank you for this package! It's really useful.
I just wanted to report that the documentation is somtimes a bit lacking in places. I have been trying to use the Flask server support along with the GitHub integration and I keep getting a problem with the code being expired from GH.
I thought I'd look at the playground example but realised that it was quite different from the docs. There are aspects (like caching) that aren't setup as per the docs and it's unclear what needs to actually be done.
Is the current doc up to date and authoritative or should I derive from the playground?
Thanks,

Okay so apparently, builtin cache is gone which makes the doc even less clear I'm afraid.
https://github.com/lepture/authlib/issues/23
I really wonder how to get a simplye GH client working with Flask :p

Right, so I should have probably pay attention to the version I was running. It was 0.4. After updating, and passing a Flask-Cache instance, I was able to get it to work!
For the record, I am using now:
from flask.ext.cache import Cache

cache = Cache(app, config={'CACHE_TYPE': 'simple'})
oauth = OAuth(
        app, cache=cache, fetch_token=fetch_token_info,
        update_token=update_token)

I'll let you decide if this the doc is clear enough on this subject :)

I might be wrong but it seems that the update_token is not used anywhere, is it?
Also, fetch_token never seems to be called either.
What are they used for?

I've updated the documentation so that you can understand what are fetch_token and update_token.
https://docs.authlib.org/en/latest/client/frameworks.html
BTW, don't use flask.ext.*, use flask_* instead.

Thanks. I think that helped!


merge package template from Kronos-Tools/npm-package-template-minimal

package.json

chore(devDependencies): update<EMAIL_ADDRESS>from template

Coverage remained the same at 14.543% when pulling 4d1536713621a8a515dede0d205f1a27d9d110bc on template-sync-1 into bb0b5adae14ba3129d207966ed77dd3fff2d97be on master.


Client does not detect AMD APU GPUs as OpenCL devices

The "client" test/profile application does not detect the GPU component of AMD APUs as OpenCL devices.  For example, when run on Windows 7 on an A10-6800K, only the CPU is detected as an OCL device when using the "-i" command to show CL platform information.

Feature now available #72


Run Mistify OS SDK in Docker Container under Ubuntu 14.04

 Issue by pjnorton
Tue Mar 22 18:30:05 2016
Originally opened as https://github.com/mistifyio/mistify-os/issues/170

SteveIsaacs has had issues attempting to run Mistify OS in a Docker container under Ubuntu 14.04 (3.3 kernel).  The purpose of this task is to analyze the issues encountered and resolve them.

I wrote as script to simplify the process of properly building a container which can run a build of the MistifyOS SDK. This process has been partially tested on dev-4 and started manually. I'll be using a Jenkins slave connected to the AWS Jenkins master to verify the container can be used to build MistifyOS.

Old Mistify/buildroot ticket. Closing.


Fix PED Bugs in UG for Add Command and Glossary

Specified which fields for add are compulsory.
Modified format for Glossary Section related to Command Fields.
Added hyperlinks to feature section.

Merging.


Fails in ES5 environments

String.prototype.repeat is an ES6 feature. This does not get polyfilled by the Babel ES2015 preset.
More info here: http://www.2ality.com/2015/12/babel6-helpersstandard-library.html#accessing-methods-normally
Because this is a module, you should not use babel-polyfill.
Instead, I recommend just using https://www.npmjs.com/package/repeat-string if you want ES5 support.

This is a great suggestion, thanks!


Document style guide

Write a minimal "style guide" that will probably be amendments to the Google C++ style guide located here.

Axom's are here: https://axom.readthedocs.io/en/develop/docs/sphinx/coding_guide/index.html
It's been a while since I read them in depth and coding guidelines kind of evolve so I can't remember how much is super relevant.


Switching System.Net.Http from Reference to PackageReference?

Would it be possible to change the System.Net.Http Reference to a PackageReference when the TargetFramework is 'net45'?
We are investigating an issue in our build system. It appears that when we make NuGet references to Refit and to System.Net.Http in a project, we encounter compile time collisions as the Refit reference to System.Net.Http targets the system-installed dll. We're looking to address our build system's shortcoming, but I thought I'd reach out and ask in the meantime.

We intentionally are referencing the built-in System.Net.Http for net45 to avoid issues with the package...

What issues are you referring to? I wasn't aware of any problems with the NuGet package for System.Net.Http

There were issues with early versions <4.3.1, I believe. The full issue is here: https://github.com/dotnet/corefx/issues/11100 and the postmortem https://github.com/dotnet/corefx/issues/17522


Improve Docs Homepage

Especially to include links to relevant content for specific audiences. For example:
Stellar for…

Validators
Asset issuers
On and off ramps (anchors)
Wallet builders
Infrastructure providers
Users

Can I be assigned to this issue?

Hi there! I am working on this already but if you have suggestions, I'd be happy to hear them!

@briwylde08  If i will be working on this issue here are the things I will do

Clearly define the needs and interests of each audience group (validators, asset issuers, anchors, wallet builders, infrastructure providers, users)
write on sections or panels on the homepage for each audience group.
3.Provide direct links to relevant guides, SDKs, API documentation, and resources tailored to each audience's requirements.
document on the homepage layout sections or tabs for easy navigation.

By documenting these steps, I will create a clear roadmap for implementing improvements to the documentation homepage.

Hi @Gift-Naomi! These are great ideas and I think this content would be good to have in this type of page. I'd be happy to collaborate on this with you! When I have a PR created I'll tag you in it. Just FYI, this is not part of the docs bounty. I wanted to make that clear!

@briwylde08 I just noticed it’s not part of the docs bounty, But I’ll be glad to collaborate with you on this.

Closing in favor of: https://github.com/stellar/stellar-docs/issues/905


Local Storage invalid between federation clear and restart

I believe this should only impact dev environments since reproducing this is pretty far out there for an edge cases.
We default to local storage to hydrate SetupContext so if you restart/clear a federation in the middle of setup, the UI will see that there is already SetupContext state in local storage and hydrate the context.  This will not match the actual state of the backend.
To reproduce:

startup a local federation and continue through only part of the setup flow (stopping around verify guardians or some other intermediary step)
Stop the fedimintd instances and clear the federation data.
Restart fedimintd instances.
Browse to Setup UI and refresh. You should see an error (The websocket is not opened) or similar.
Check local storage cache and observe.

This block was intended to check the current state of the server, and reset the local state if the server's status doesn't jive with our local state: https://github.com/fedimint/ui/blob/be47100bffe2804286f289c3149871ebea0ae59a/apps/guardian-ui/src/setup/SetupContext.tsx#L130-L196. This made the assumption that the fresh restart would put you into AwaitingPassword. If that assumption is no longer true, we can probably add some additional logic in here to reset for other cases.
I was also considering if we should always just prompt the user if they load the page with mid-setup state, letting them know they can either continue from when they last left off, or to start over. That might cut through a lot of these cases.

no repro after #104 . Please confirm?

I've run through the wipe to restart case a few dozen times in the past couple days with no issue, so I'm going to close. In the OP's picture, it looks like they had a websocket connection issue so the client did not yet know it was a restarted instance for it to wipe state. I believe if the websocket had properly connected, it would have received a status of AwaitingPassword and fixed itself as expected.
Please re-open with repro steps if I'm wrong!


Add a test for mypy result regex matching

A test for https://github.com/Microsoft/vscode-python/commit/bfcd2252f85c66b2b278dfa6e7814387fda8c5c1 .
Need to also check that the regex doesn't affect what mypy checks and is only for teasing out results (e.g. this won't cause mypy to check a README file suddenly).

Why is that check (target vs. results) needed relative to MyPy?  From what I can tell, the handled filenames are determined elsewhere in the extension.

rom what I can tell, the handled filenames are determined elsewhere in the extension.

That's because the file name is not part of the result. We run linters on a file by file basis, hence at the calling location we know the file that is being linted.


Revert "build(deps-dev): bump the types group with 1 update"

Reverts SciCatProject/frontend#1397

Any special reason for the revert? Or  was this triggered by the test failures?

I created this PR to check if the latest merge was the source of the e2e test failing.
Given that you found the issue somewhere else, we can close this PR.


Bug: storing validation rules in Config/Validation causes error

PHP Version
8.3
CodeIgniter4 Version
4.5.1
CodeIgniter4 Installation Method
Composer (using codeigniter4/appstarter)
Which operating systems have you tested for this bug?
Linux
Which server did you use?
apache
Database
MySQL
What happened?
Following the instructions at https://codeigniter.com/user_guide/libraries/validation.html#saving-sets-of-validation-rules-to-the-config-file I saved the following in App/Config/Validation.php (I initially tried with my own array, but then used the array from the documentation in case I'd made a typo):
public array $signup = [
        'username'     => 'required|max_length[30]',
        'password'     => 'required|max_length[255]',
        'pass_confirm' => 'required|max_length[255]|matches[password]',
        'email'        => 'required|max_length[254]|valid_email',
    ];

This immediately made all pages return this error:
Cannot assign null to property Config\Validation::$signup of type array
Steps to Reproduce
Following instrictions above
Expected Output
No error  (the validation rules were not yet being called from anywhere in the code)
Anything else?
No response

Cannot reproduce.


Changes the colors to resemble the Visual Studio Code Extension.

Changes:

Changed colors:

Added TODO and header comments:

Changed name of repository key.

Went from being tag_brackets to the more fitting: tag_params.

Fixed standalone single quote coloring rest of line as a string.

Changed coloring of [] inside of inventory containers to now be colored the same as normal text and to not be captured as a command.

If you want to try it out and look at it here is a link to test it. All you need to do is provide a link to the code you want to see highlighted.


no file name or line number in config validation error message

Home Assistant release with the issue:

0.86.4
Last working Home Assistant release (if known):
Unknown
Operating environment (Hass.io/Docker/Windows/etc.):

Docker
Component/platform:

https://www.home-assistant.io/components/camera.generic/
Description of problem:
Some (but not all) errors in my configuration lead to a validation error message with the unhelpful coordinates (See ?, line ?). For example, if I start with this valid configuration block:
camera:
  platform: generic
  still_image_url: 'https://avatars1.githubusercontent.com/u/40870121'

and then add an invalid x to platform:
camera:
  xplatform: generic
  still_image_url: 'https://avatars1.githubusercontent.com/u/40870121'

I get, in the web interface, the error message:
Invalid config for [camera]: required key not provided @ data['platform']. Got None. (See /config/configuration.yaml, line 12). Please check the docs at https://home-assistant.io/components/camera/

So far so good. But if I instead make an error in the still_image_url key:
camera:
  platform: generic
  xstill_image_url: 'https://avatars1.githubusercontent.com/u/40870121'

the coordinates are lost in the error message:
Invalid config for [camera.generic]: required key not provided @ data['still_image_url']. Got None. (See ?, line ?). Please check the docs at https://home-assistant.io/components/camera.generic/

This seems similar to #9194, but that issue was autoclosed without discussion or resolution.
Is this behavior expected/documented? I ran into it because I was working on a custom component, but found I could reproduce it with a standard component. I'm absolutely new to HA, so won't be surprised if the mistake is mine.
I saw some discussion somewhere suggesting that included files might lead to this problem, but I eliminated that possibility for test. Here's my complete configuration.yaml:
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
homeassistant:
  name: Home
  latitude: 40.1779
  longitude: -105.1009
  elevation: 0
  unit_system: imperial
  time_zone: America/Denver

frontend:
config:

camera:
  platform: generic
  still_image_url: 'https://avatars1.githubusercontent.com/u/40870121'

Traceback (if applicable):

Additional information:

Good catch. This is a bug for years. I am opening a pull request to fix it

I am now testing the 0.87.1 Docker image, and I thought this issue might have disappeared, but it persists. Does this have something to do with the fact that the fix is listed as a "beta fix" here: https://www.home-assistant.io/blog/2019/02/06/release-87/? If so, when might I expect to see it in an updated Docker image? Just curious, and apologies if I've misunderstood. Thank you for the kind support.

Somehow, it missed the 0.87 train. It is in 0.88 beta now.
See the tag in this commit https://github.com/home-assistant/home-assistant/commit/c812176e940e690a8fb1013070c77d5e15bba025

I'm on 0.91.4 (Docker image), and my reports looks like:
Log Details (WARNING)

Fri Apr 19 2019 11:36:26 GMT+0200 (Central European Summer Time)

Your configuration contains extra keys that the platform does not support.
Please remove [device].

Should this fix that, too? Cause i really miss the filename and line-number reference...  (#22528 looks simular to this..)
Could this be related to some MQTT solutions, fx? (MQTT Discovery details from my MQTT cam, zigbee2mqtt solution, or?)


Update material.css

Prevent 'jumping around' of layout elements on page load

Coverage remained the same at 100.0% when pulling a9b8c3085d2f7399b265cacca544783a9f4e40d0 on jdrmar:patch-1 into 26b35f8761cfa15c82f5cdaa99b75b770ed46f9f on tleunen:master.

Could you explain a little bit more what this fixes?
Also, this fix would need to also be in the minified file, and all the other css files as well. I'd need to regenerate all files since I already have a setup for that. (We have to generate the files in the MDL repo)

@jdrmar I'm closing this PR, I think the issue is resolved with the fix I did in https://github.com/tleunen/react-mdl/issues/280
Let me know if I'm wrong.

Yes, fixed!
:))
2016-04-10 16:58 GMT+02:00 Tommy<EMAIL_ADDRESS>
@jdrmar https://github.com/jdrmar I'm closing this PR, I think the
issue is resolved with the fix I did in #280
https://github.com/tleunen/react-mdl/issues/280
Let me know if I'm wrong.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/tleunen/react-mdl/pull/281#issuecomment-207996332


Failed to load types files -> on fresh installation

using the "Types" Menu gives me these kind of errors in the log. creating empty files doesn't help...
Why / how is it looking for files of ExpansionMod/Market or HardlineSettings.json?

Any help would be appreciated :)

Same issue, did you get it resolved?

I found a problem with this.
The file is my folders but is all lowercase. Added a copy of the file but with tha matching name the manager wants, and the types started to show in the dashboard.
Might not explained that very well ...


NPM Stops working properly if Streamed container is down.

Checklist

Have you pulled and found the error with jc21/nginx-proxy-manager:latest docker image?

Yes

Are you sure you're not using someone else's docker image?

Yes

Have you searched for similar issues (both open and closed)?

Yes

Describe the bug
When a Streamed container is stopped, the npm container enter a configuration loading loop, proxy hosts and everything else stops working.
Nginx Proxy Manager Version

To Reproduce
Steps to reproduce the behavior:

Create a new container that requires streaming ( for example, a Minecraft server )
Create a new Stream on the NPM Panel.
Stop the streamed container without first disabling the Stream.

Expected behavior
The NPM container keeps working normaly. Same as when a Proxy Host container is stopped.
Screenshots
logs of the NPM container after I stop my MC container.

Operating System

Additional context

I think a way to fix this is to do what the proxy hosts do and use proxy_pass $server; instead of proxy_pass server;. So instead of:
proxy_pass <server>;

You'd have:
set $server = "<server>";
proxy_pass = $server;

... I'm not sure why Nginx works this way, but it does, and that fixes it (for me anyway). So I think changing how NPM auto-generates files to do that for stream hosts too would solve this issue.
You can workaround this though by manually modifying /data/nginx/streams/*.conf yourself, but that breaks if you make a change in the GUI and the file gets regenerated :/


update time to cleanup sooner

For the sake of sending an EVENT 3 (stop announce), it's best to remove a destroy action up the chain of command. This ensures a quick cleanup after 'stopped' is submitted.

I'm a little confused about what this PR is supposed to accomplish. This doesn't seem necessary to accomplish the goal of waiting for the STOP message to send before destroying.


v1.10 backports 2022-08-17

#20734 -- Fix complaint about nil IP address on restore of cilium_host (@christarazi)

Only backported the first commit 08205dde0f6d ("daemon/cmd: Fix complaint about nil IP address
on restore of cilium_host") which seems to be the main fix in this PR. The second commit caused
merge conflicts because it releias on changes from #20453 which we don't backport.

#20706 -- ipcache/kvstore: fix panic when processing ip= entries (@ArthurChiao)
#20875 -- Improve CRD schema update automation during release process (@joestringer)

Skipped the last commit d9a14a007a1c ("contrib: Fix CRD schema gen when it needs updating")
because it would require additional backports (see
https://github.com/cilium/cilium/pull/20700#issuecomment-1217721443) and maintainers seem to be
using these scripts from master branch only anyway.

#20916 -- docs(bandwidth-manager): add note on per-pod limits (@raphink)
#20900 -- ipsec: Fix incorrect parsing of SPI from mark (@pchaigno)

Once this PR is merged, you can update the PR labels via:
$ for pr in 20734 20706 20875 20916 20900; do contrib/backporting/set-labels.py $pr done 1.10; done

/test-backport-1.10

/test-gke

Triage:

This backport was run before https://github.com/cilium/cilium/pull/20917 , so l4lb was likely broken. I'll rerun it.
test-1.16-netnext hit #17353

/ci-l4lb-1.10

@tklauser Similar to v1.12 backports PR, backport/pending label was missing on all of the backported PRs.


Part 6, controller methods and views in ASP.NET Core should include a pointer to the recommended way for implementing Edit

[Enter feedback here]
According to this page the scaffolder generated page is not the recommended code. Nevetheless, the MVC tutorial (Part 6, controller methods and views in ASP.NET Core) shows only this code. I suggest to add at least a pointer to the recommended code.

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

ID: 582aeaf8-770f-6861-ab47-bb6a4f2206e4
Version Independent ID: 51894773-82dc-3c71-f675-8af42cfa925f
Content: Part 6, controller methods and views in ASP.NET Core
Content Source: aspnetcore/tutorials/first-mvc-app/controller-methods-views.md
Product: aspnet-core
Technology: aspnetcore-tutorials
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

This is just an introduction. See https://docs.microsoft.com/en-us/aspnet/core/data/ef-rp/crud for optimal code.

@Rick-Anderson We are saying the same thing. In the Introduction there should be something like "The example code below is code generated by the scaffolder, which is not the recommended approach for implementing the Edit method. For the recommended approach see here"
Note, it is not about optimization: the Introduction includes (according to Microsof docs) unrecommended code.


Investigate/fix cardinality behavior in FunctionMetadataGenerator

closing as the https://github.com/Azure/azure-functions-dotnet-worker/issues/1204 is done.


Inject sidecar in PowerMax

Description
A few sentences describing the overall goals of the pull request's commits.
Issues
List the issues impacted by this PR:

Issue ID

Checklist:

[ ] I have performed a self-review of my own changes.

@medegw01: Thank you for submitting your pull request!
Please make sure you have followed our contributing guidelines. We will review it as soon as possible.

We want your feedback! If you have any questions or suggestions regarding our contributing process/workflow, please reach out to us at<EMAIL_ADDRESS>


feat: Add support for terraform insecure flag

Description of your changes
This MR will support the addition of insecure flag.
I have:

[X] Read and followed Crossplane's [contribution process].
[X] Run make submodules; make reviewable test to ensure this PR is ready for review.

Hi @schlakob,
can you generate a new release please?

Would you be so kind and test this before we release the new version?
The provider including this PR is already available at:
xpkg.upbound.io/crossplane-contrib/provider-openstack:v0.3.0-57.g2296b86

Hi @jan-di,
just tested and it worked perfectly.
By adding the option "insecure": "true" to my provider config the error message for tls: failed to verify certificate: x509: certificate signed by unknown authority disappeared and my resource was created successfully.
Thank you both for the timely response.

Thank you! We will do a release :)

Just released the v0.4.0.
https://marketplace.upbound.io/providers/crossplane-contrib/provider-openstack/v0.4.0
Thanks for your contribution.


Typo in drivers/bridge/setup_up_forwarding.go

I found typo when I explore source code. If this PR is not appropriate, feel free to close.

Sorry, I didn't know that there exists such a rule. I just added a sign.


Fix logout button

this fix does the following:
Awaits the authLogoutRequest to ensure it completes before proceeding.
Awaits signOut if the user is logged in via Google.
Sets the user's logged-in status to false only after the logout request has completed.
Redirects to the index page.
Catches any errors that occur during the logout process and logs them to the console.
closes #337

Im not sure how making proper error handling fixes this issue, but well here we are. Thanks, this was driving me insane


arangosh doesn't ignore prefixed ....> on paste

when cut'n'pasting from the documentation, the user will have ...> in front of multi-line-examples.
for convenience these were ignored - this doesn't work anymore.

Confirmed in 2.7.dev, Frank will have a look.

@dothebart can you please check if it stills compiles under windows?


TEST issue

Just a test issue - please disregard this issue.

Triggered auto assignment to @laurenreidexpensify (Internal)

Sorry for the ping @laurenreidexpensify, just testing a new feature.


[PORT] Adding datetimereference to luis v3

Port this change from botbuilder-dotnet/master branch:
https://github.com/microsoft/botbuilder-dotnet/pull/3867

Changed projects

Microsoft.Bot.Builder.AI.LUIS
Microsoft.Bot.Builder.AI.LUIS.Tests

Closing this item. It was analyzed by Southworks: No action needed, already implemented in JS.


Support viem and Hardhat toolbox

Is your feature request related to a problem? Please describe.
We've decided to switch to viem for our project, and we were previously using Smock for our tests. Unfortunately Smock does not support viem so we will need to drop it for now.
Describe the solution you'd like
Having viem support in Smock
Describe alternatives you've considered
There are no other libraries out there with the same functionality, so there aren't really any alternatives
Additional context
If you'd be open to collaboration I could have a look to understand how much work it would involve, or at least help with the effort. I'm really interested in having this :)

I'm curious what dependency you have between viem and smock. My assumption is there is no dependency that you would be able to use smock to mock/ fake contracts and still query them with viem.

Hi @coreyar,
As far as I understood Smock uses Ethers.js Contract type for instantiating the mocks/fakes. I couldn't manage to use viem Contracts with Smock, but perhaps I was doing it wrong. Have you tried this combination of Hardhat Toolbox for viem and Smock?

@robercano Smock does use ethers under the hood but you can still create an interface for interacting with the deployed smock using viem.
I made an example to show how it works. You can check out the tests rewritten with smock and viem here

I haven't but I played around with it but I dug in a bit today and I was able to get it to work here - https://github.com/coreyar/smock-viem-example/blob/main/test/SmockLock.ts#L152
I need to learn more about how smock works but it is at the evm layer hence the dependency on hardhat but not viem or ethers.


Karpenter Integration

Hello folks, Karpenter is gaining a lot of traction in the AWS EKS world, is there any plans to have an integration for it?
https://karpenter.sh/v0.18.1/tasks/metrics/

Hi! Thanks for opening this issue.
As far as I know, there's no plan to create an OOTB integration for Karpenter but I might be wrong. Pinging @shris-github and @anjaliDDog to confirm.
You can request it here. Also, since Karpenter makes some metrics available through a Prometheus endpoint, you could use our openmetrics integration to fetch them but the metrics will be considered as custom metrics, as stated here, and you'll have to create the dashboards yourself. Last note, one could also create a new integration in our integrations-extras repository and open a Pull Request.

Hi Lucas, second everything Florent has shared above. Also, happy to connect directly to learn more about your use case with Karpenter to inform our roadmap. Let us know if you would be interested in a brief chat. Thanks!

Hi Lucas, second everything Florent has shared above. Also, happy to connect directly to learn more about your use case with Karpenter to inform our roadmap. Let us know if you would be interested in a brief chat. Thanks!

Thanks Shri. I'm closing this issue but feel free to reach out if you have any questions.