refactor!: Reduce MessageBus config to have a single host

BREAKING CHANGE: Configuration now only needs  single broker host info
closes #196
Signed-off-by: Leonard Goodell<EMAIL_ADDRESS>

If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/go-mod-messaging/blob/main/.github/Contributing.md
PR Checklist
Please check if your PR fulfills the following requirements:

[ ] I am not introducing a breaking change (if you are, flag in conventional commit message with BREAKING CHANGE: describing the break)
[x] I am not introducing a new dependency (add notes below if you are)
[x] I have added unit tests for the new feature or bug fix (if not, why?)
[ ] I have fully tested (add details below) this the new feature or bug fix (if not, why?) Unit tests suffice
[ ] I have opened a PR for the related docs change (if not, why?) N/A

Testing Instructions

New Dependency Instructions (If applicable)

Codecov Report

Merging #200 (78d6cd1) into main (3ead656) will decrease coverage by 0.15%.
The diff coverage is 83.33%.

@@            Coverage Diff             @@
##             main     #200      +/-   ##
==========================================
- Coverage   75.38%   75.22%   -0.16%
==========================================
  Files          20       20
  Lines         914      896      -18
==========================================
- Hits          689      674      -15
+ Misses        173      172       -1
+ Partials       52       50       -2

Impacted Files
Coverage Δ

pkg/types/host_info.go
71.42% <ø> (ø)

internal/pkg/nats/options.go
86.88% <66.66%> (-0.42%)
:arrow_down:

internal/pkg/redis/client.go
87.96% <81.25%> (-1.39%)
:arrow_down:

internal/pkg/mqtt/client_options.go
94.28% <100.00%> (+7.79%)
:arrow_up:

messaging/factory.go
85.71% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Windows: Build fails after update to react-native 0.75.x and react-native-windows 0.75,x

For react-native version 0.75.x and react-native-windows version 0.75.x this library does not compile anymore on windows. There are two issues: the TargetPlatformMinVersion is too low and there is an error when compiling the sources.
The error is

× Build failed with message C:\blobutiltestproject\node_modules\react-native\ReactCommon\react\bridging\Convert.h(81,1): error C2938: 'std::enable_if_t<false,int>' : Failed to specialize alias template (compiling source file C:\blobutiltestproject\node_modules\react-native-windows\Microsoft.ReactNative.Cxx\TurboModuleProvider.cpp) [C:\blobutiltestproject\node_modules\react-native-blob-util\windows\ReactNativeBlobUtil\ReactNativeBlobUtil.vcxproj]. Check your build configuration.
Command failed. Re-run the command with --logging for more information.

Steps to reproduce:

npx --yes @react-native-community/cli@next init blobutiltestproject --version "latest"
cd blobutiltestproject
yarn add<EMAIL_ADDRESS>npx react-native init-windows --overwrite
yarn add react-native-blob-util
change WindowsTargetPlatformMinVersion in node_modules/react-nativve-blob-util/windows/ReactNativeBlobUtil/ReactNativeBlobUtil.vcxproj to 10.0.17763.0
yarn windows
get the error messge mentioned above

Package versions (resolutions):
react-native: 0.75.3
react-native-windows: 0.75,0
react-native-blob-util: 0.19.11
OS Version: Windows 10
Visual Studio: Visual Studio Community 2022, Version 17.11.35303.130
To summarize, there are two issues:

WindowsTargetPlatformMinVersion needs to be increased
Windows project does not compile (I don't know how to fix it)

@RonRadtke do we have any updates on this? :)

@RonRadtke do we have any updates on this? :)

None for now 🤷🏼♂

@RonRadtke do we have any updates on this?

Facing the same issue

We were now forced to replace react-native-blob-util with this one: https://github.com/birdofpreyru/react-native-fs

If anyone has a fix for it - would be great.
Will take a while till I have the time to set up and look into it since it requires the windows setup

Ok thanks for the reply! Unfortunately, the windows setup is really annoying. Now that we found a different solution we are of course much less inclined to work out a solution ourselves.

Ok thanks for the reply! Unfortunately, the windows setup is really annoying. Now that we found a different solution we are of course much less inclined to work out a solution ourselves.

Of course - I can understand that very much. Still would be great though, but I can understand that you switch and don't want to invest in this lib then, no worries :)

Can someone confirm that the following patch also works for them?
If so, I'll make a PR :)
react-native-blob-util+0.19.11.patch


feature: Refactor color selections to support 8-bit colors...

...since many terminals do not support 24-bit 'true color'. There's also no surefire way to detect color support, see https://github.com/termstandard/colors. Theoretically should fix #3
Screenshot of output from Terminal.app on MacOS for reference.

Thank you for the fix, it works as expected. This closes #3


Set header with all HTTP requests

We need to set an HTTP header for HTTP requests to the app since we need to be able to see which products are actually scanned.

@teolemon  Can I start working on this issue ?

Yup :)

Please add the User-agent header, and set it To something like OFF Android + the app version number

Hello, i don't really understand what is expected, isnt the PR https://github.com/openfoodfacts/openfoodfacts-androidapp/pull/2035 sufficient to add a header in the http get request ?


Right Click Menu: Page element menu v1

Context
Right Click Menu work
Summary
Adds the initial right click menu for page/background. Right clicking on the page/background should show this menu.
Relevant Technical Choices
~Opted to use the clipboard api (navigator) when copy/pasting. Was having trouble creating a ClipboardEvent that would be propagated and add data to the clipboard directly. Instead, I chose to use the newer Clipboard api and add the copied element to the navigator serialized as text.~
~We already had functions that we could leverage for pasting data from the clipboard that was serialized to text. There are already comments there saying we should upgrade to the clipboard api when it's ready too, but that is not in scope of this PR.~
Copy/Paste/Delete work on the entire page, so we don't need the clipboard api.
To-do

[x] ~Implement a way to override paste permissions for puppeteer tests (different PR)~ unnecessary

User-facing changes

Action

Copy + Paste

Delete

Duplicate Page

Delete Page

Rtl

Testing Instructions
First:

Enable the 'right click menus' experiment
Go to editor

Copy + paste

add whatever you want to the page
right click the page background or anything on the canvas
click Copy
add new page
right click page background
click Paste (may have to allow pasting permissions in your browser)
should see content pasted

Delete

add whatever you want to the page
right click the page background
click Delete
page should be deleted (unless it's the first page)

Duplicate Page

add whatever you want to the page
right click the page background
click Duplicate page
new page should exist that is identical to the one you created

Delete Page

add whatever you want to the page
right click the page background
click Delete page
nothing should happen if there's only one page
Add a new page
right click page background
click Delete page
Page should delete

QA

[x] This is a non-user-facing change and requires no QA (behind feature flag)

Reviews
Does this PR have a security-related impact?
~yes - the navigator api requires users to accept new browser permissions to allow our application to write from their clipboard.~ no, this was removed.
Does this PR change what data or activity we track or use?
no
Does this PR have a legal-related impact?
no
Checklist

[x] This PR addresses an existing issue and I have linked this PR to it in ZenHub
[x] I have tested this code to the best of my abilities
[x] I have verified accessibility to the best of my abilities (docs)
[x] I have verified i18n and l10n (translation, right-to-left layout) to the best of my abilities
[x] This PR contains automated tests (unit, integration, and/or e2e) to verify the code works as intended (docs)
[x] I have added documentation where necessary
[x] I have added a matching Type: XYZ label to the PR

Fixes #6138

Do we have a ticket reference for exploring the navigator paste permissions?

Do we have a ticket reference for exploring the navigator paste permissions?

Right here! https://github.com/google/web-stories-wp/issues/7973

The color contrast of the keyboard shortcut for "Copy + Paste" doesn't seem very good. What contrast ratio is that?

The color contrast of the keyboard shortcut for "Copy + Paste" doesn't seem very good. What contrast ratio is that?

Looks like the main text has a good contrast ration but the shortcut has a poor contrast ratio

Main text
shortcut

The color contrast of the keyboard shortcut for "Copy + Paste" doesn't seem very good. What contrast ratio is that?

So the colors are this screenshot. We are putting a pin in the right click menus for this release cycle (1.9) and coming back so the colors aren't perfect yet.
that said, there's definitely a failing contrast level, but since we are pausing this work behind a feature flag now didn't really feel like the appropriate time to get into the details on it: https://webaim.org/resources/contrastchecker/?fcolor=5E6668&bcolor=131516

In that case, if you could create a ticket to keep track of this potential color contrast issue, that would be great. It should be an easy one to fix & add tests for, as well as to inform the design team about so they know about this kind of stuff in the future.

In that case, if you could create a ticket to keep track of this potential color contrast issue, that would be great. It should be an easy one to fix & add tests for, as well as to inform the design team about so they know about this kind of stuff in the future.

sure! https://github.com/google/web-stories-wp/issues/8044


Remove special coloring for message selection box on unread messages

At present, the message selection box on unread messages is green:

This comes up in views that do not mark messages as read, such as search views, and views where messages have just been marked as unread.
As pointed out on CZO, the result can be visually jarring. As there is already an unread message marker on the left, there is no need to additionally highlight that these messages are unread.
We should therefore make the message selection box always be the (default) blue color, regardless of whether or not the selected message is unread.

@zulipbot claim

This issue does not affect main. It’s part of a chat.zulip.org experiment (#21477).


Made correction in the output of println function when it prints an i…

…nteger string.

[ ] I have read freeCodeCamp's contribution guidelines.
[ ] My pull request has a descriptive title (not a vague title like Update index.md)
[ ] My pull request targets the master branch of freeCodeCamp.
[ ] None of my changes are plagiarized from another source without proper attribution.
[ ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Closed and Reopened this PR to attempt to resolve a specific Travis build failure.

Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 📝


Alerts not being sent, even if alerting condition is maintained

Hi,
I'm working with the frequency rule type of ElastAlert and observe that even if alerting condition is
maintained, the matches is 0 even if hits is non-zero.
I'm sharing the yaml file for the rules. I'm checking that at least 1 document arrives in the index in the last 2 mins, I should be alerted. My elastic search is 7.x and elastalert uses buffer time as 5 mins.
When the rule runs, the elastalert_status_status contains
endtime: the time when rule ran
starttime: endtime - 5 mins (buffer time)
hits: number of documents matching the rule between starttime and the endtime
matches: 1 (only for the first occurence of the alert)
name: elastalert-test
type: frequency
index: alert-test
buffer_time:
  minutes: 5
num_events: 1
timeframe:
  minutes: 2
realert:
  seconds: 0
run_every:
  seconds: 5
filter: []
alert:
- "victorops"

victorops:
victorops_api_key: "xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx"
victorops_routing_key: "elastalert"
victorops_message_type: "CRITICAL"
victorops_entity_display_name: "Test Alert"

I'm sharing my observation and the difference from the expectation.

When I index a doc in the index alert-test, I get an alert. This is expected.
Since, I'm not aggregating or silencing the alert (realert is set to 0), I expect an alert to be sent every 5 seconds (run_every is set to 5s) for the next 2 mins (the timeframe of frequency rule type). This is because there will be at least one doc in the index in the last 2 mins. But I don't get any further alert. On checking the metadata index elastalert_status_status, the hits is 1 for the next 5 mins (buffer_time is 5m), but matches is 0

I observe that alert is sent only when 1 document is indexed since the last run of the rule checker.
My motivation is to know when does an alert gets resolved and I was trying to get this info from len(rule['type'].matches) in elastalert.py
I'll appreciate any pointer or suggestion.

Yelp/elastalert is no longer maintained. Please use jertel elastalert. Questions to the discussion below
https://github.com/jertel/elastalert/discussions


feat: add some filesystem storage

Forgive my ignorance, but why do we need this?

Forgive my ignorance, but why do we need this?

Wouldn't we want to store the content of the SBOMs and advisories we get asked to stored? Assuming we manage SBOMs, I think a use case would be to retrieve that document again at some point.
Additionally, if we decide to make changes in the way we extract information, we would be able to re-process those documents.

Storing a lot of stuff in the database isn't the best idea IMO. For one, you'd consume RDS instance resources, vs S3 (which is much cheaper).
Second, I know that "can" store BLOBs. However, I also know that it's not performing well, compared to a filesystem.

I still don't like the retrieval part (besides that I still need to implement it for SBOMs).
Right now, it uses the primary key of the advisory (the id). I think that's ok. However, that also requires that the user can find that ID of an advisory. I think there should be a second function: fetch advisory by "document ID" and "source".

Closing in light of #114, which includes these commits and resolves the conflicts resulting from #116


[Webhook] sesion invanate

have the webhook cog have a option to run sesion forever in a channel not just have a 2 min timeout

This has been added in the latest update to the cog. Sessions will now persist until the cog is reloaded or bot restarts.


Pipfile touched even on pipenv run

I have a watch command tracking my Pipfile but on every call of pipenv run the command is executed, even if the file wasn't changed. I tracked this down to the call of Project.touch_pipfile. I'm not sure if it required, for Pipfile to be touched every time, probably it is just a side effect of calling  cli.py@ensure_project.
Would it be reasonable to only touch the Pipfile if it either doesn't exist or if there is actually a change to its contents?

I noticed this too. In my Makefiles, I run pipenv install when Pipfile* is newer than the installed dependencies, but now this runs all the time.

Is this just annoying, or a real problem?

landed a fix in master

IIUC this was supposed to have been fixed with 7.1.0, but the issue persists for me.
$ pipenv --version
pipenv, version 7.1.0
$ ls -l --time-style=full-iso Pipfile
-rw-rw-r-- 1 mbarkhau mbarkhau 416 2017-09-15 09:46:34.611467318 +0200 Pipfile
$ pipenv run python -m os
$ ls -l --time-style=full-iso Pipfile
-rw-rw-r-- 1 mbarkhau mbarkhau 416 2017-09-15 09:46:56.526411866 +0200 Pipfile

To answer the earlier question, mostly this is just very confusing, as opposed to being a real problem.

Same for me. It's not a huge deal but it breaks all my build caching. It could cause issues if pipenv intends to run in a container with a read-only filesystem.

Hi @kennethreitz I think this may have fallen off your radar, or is this a non issue in your view?

the file should only be touched when it doesn't exist. I can look into it, but i don't see it as a big issue, no.

The Pipfile is still updated on every run with 7.6.7.
For reference, this is the Makefile pattern that this causes problems with:

.PHONY: install
install: .venv/installed
.venv/installed: Pipfile*
	pipenv install --dev
	@ touch $@

.PHONY: run
run: install
	pipenv run honcho start

You should be checking the lockfile for changes.

It's what's being used for install anyway.

@jacebrowning i'd be happy to accept a pull request that doesn't touch the makefile on every run — there's code in place to only touch it if it doesn't exist, so i'm not sure why this is happening, so you'd have to dig in a little bit.

Cool. I'll look into it.

I would recommend keying off of the lockfile though — it's what's important, not the Pipfile.

and the lockfile doesn't get touched constantly.

pipenv automatically re-locks whenever anything gets installed/uninstalled, so it should take care of all your problems.

Yeah. It's still handy to be able to manually add packages to the Pipfile or update version specs and have these changes get picked up automatically by Make tasks.

Gotcha. The intention behind pipenv is that you never have to edit the pipfile yourself unless you're doing fancy stuff (like markers, etc)


feat: add enable read operation

Discussion
As we discussed before, supporting replica when using bloom filter.

Hi @proost,
The docker used in Github Actions seems to be updated and the docker-compose command has gone. Please rebase this PR to the latest main branch.

Merged. But I forgot that the parseRedisMajorVersion is still there. Could you send another PR to remove it?


Change all settings to disabled by default and require ModMenu and GlassConfigAPI

The nature of this mod is like BetaTweaks, enable what you want. Don't touch what you don't want

In fact the inventory crafting is actually an old Alpha bug and could be in BetaTweaks instead...
That would only leave two recipes in this mod.

More stairs recipe
And soon to be shapeless jack'o'lanterns.

idk, gotta think on this one

The whole identity of this mod is switching to MiscTweaks. Stairs will be default 4, shapless jack'o'lanterns will be default true


per test sharding?

Instead of creating one shard per test case Flank can create shards that contain tests that add up to a given duration

Is it not possible to have one shard per test case? In terms of fault isolation/video/logs, there are some use cases where this is desirable.

It is possible through a properties file. To enable one test per shard set shard-duration to -1. Please see the Readme for more information.

Ok, thanks!


Dan branch

creating this just for easier viewing of diffs and potentially commenting

Converted to real PR, since the test is useful :)

@danthegoodman1 is it really testing something new? (In terms of jellyscript features)

tbh nah lets kill this


Feature/seperate docker files

Removes dockerfiles as there are equal to the default images used by the MicroShed modules. The documentation provides background info on how to specify a Dockerfile.
Fixes https://github.com/MicroShed/microshed-testing/issues/308

@KyleAure rebased!


⚠ Jenkins Plugins Artifactory has degraded performance

In a47d850, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1234 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 1d8f518 after 33 minutes.


How to identify DND event in AVS SDK v1.7?

We test DND event in echo dot, we can see purple when enable Don't disturb me.
Then we test it in our device, and use sdk v1.7.  From log message, it is not easy to identify which message can point out "Don't disturb me' or not.
I saw close tickets #675 , #508 , all point out SDK does not support it.
But why echo do can recognize it?
Could you help to provide some idea?

Hi! Echo Dot has a proprietary software inside and but are working hard to provide the same capabilities to SDK users as well. DND functionality is not available in our SDK, though.

Hi @mavamazon
Why list DND item in certification if AVS SDK cannot identify DND?
Have I misunderstand about test item 247.
From functional self-test , it has certification about DND function.

I already get response from Amazon FAE , so I close it, thanks

@jackeyu  : hey hi .
Why list DND item in certification if AVS SDK cannot identify DND?
Have I misunderstand about test item 247.  <<<---- what is the solution for this.


Dart experiment to remove (or hide) dart:html and friends from included libraries

It would be super cool to have an experiment (that can be used with --enable-experiment) that removes (or hides) dart:html and its friends from the supported/surfaced SDK libraries.
This would allow the analyzer and compilers to catch usages of the old libraries in your package and dependencies (sure, can be partially done by compiling with dart2wasm).
Perhaps more importantly though, the analyzer could avoid surfacing those libraries in completion suggestions, greatly improving the developer experience of those using modern interop functionality.
It would be a useful step towards migration that could be enabled relatively soon due to the lack of guarantees around experiments.

For what it's worth, we have a dart2js flag --server-mode  that does this for dart2js.
I'll start the triage here first by asking a few questions:
For the @dart-lang/language-team: is it possible to use experiment flags and language versioning to hide a library?
For the analyzer team: if not, are there mechanisms to enable/disable an SDK library from being available in the analysis server?

I didn't think about --server-mode, thanks for calling that out as another option! Yeah I think standard support for this from the web compilers is nice to have, but I feel that's less important since it can be surfaced with dart2wasm and the server mode.
The big win I see for developers is mostly in analysis errors and less noisy completion suggestions :)

You can also use the import_path package to check for imports to any library (such as dart:html):
pub global activate import_path
import_path web/main.dart dart:html

/cc @bwilkerson , @scheglov

that removes (or hides) dart:html and its friends from the supported/surfaced SDK libraries.

It might be nice to make it more general than this - why can't I pick which specific dart: libraries I do/don't want (like I can with other dependencies)? There are quite a few libraries you get "for free" from the SDK and even some non-web ones you might just never want to use.

I've previously also wondered whether the Dart SDK could include package: libraries rather than them having to be dart: so that they are opt-in, similar to how you can reference packages from the Flutter SDK:
dependencyes:
  flutter:
    sdk: flutter
  mirrors:
    sdk: dart // package:mirrors but comes from the SDK

That could allow things to be shipped in the Dart SDK without being automatically available to projects (for example allowing other packages to use the same names without completion lists being full of duplicates). For example IIRC in flutter_tools it's forbidden to use dart:io instead of going through an abstraction - but it's very easy to accidentally get all the types from dart:io with fixes/completion.

Closing in favor of https://github.com/dart-lang/sdk/issues/54610 which implemented not surfacing dart:html and friends when not already used.
For determining outdated usages during compile, you can use dart compile wasm.
Thanks all.


Use the file location as a part of the namespace

OpenUI5 version: every version
This is a feature request.
I would like to see that ui5 takes the file location as a part of the namespace and the namespace mapped to '/' as a default namespace.
Benefits:

This would end redundant namespace declarations.
Would make it possible, to copy things fast between projects
Namespace changes can be done at one location

Other frameworks like steal do the same.
Assuming i have this namespace:

ui5 expect already everything under root tagged with this namespace. Given this structure:

I want to name the controller like:

anywhere else within this component, define should allow these two ways of require
sap.ui.define([ 'de/my/namespace/controller/application/Detail'...
sap.ui.define([ 'controller/application/Detail'...
other component need the full namespace
sap.ui.define([ 'de/my/namespace/controller/application/Detail'...
the same would apply to extend
myController.extend('controller.application.master')

Did you know that for sap.ui.define you can do this already using the "./" or "../" notation which is aligned with the way AMD works? Using shortened names without such a marker IMHO wouldn't be a good idea as it makes resolving names quite ambiguous, esp. when multiple components from different locations are combined in an app.
For the other usages of qualified names, esp. for extend, we considered similar notations, but the context is not as well defined as it is for sap.ui.define. A more likely improvement would be to make the name in the extend call obsolete for top level calls (calls that are executed while a module is executed) and to derive the name from the ID of the current module. But there are use cases where more than one extend call is executed in a single module, so some syntax for relative names still would be helpful. And, whatever we do there, potentially needs to be transferable to other module loaders.
For other APIs that also use names (e.g. sap.ui.xmlview) the problem of the missing context is even worse. The framework simply doesn't know at runtime in which module a sap.ui.xmlview call is executed. So it can't take that context into account when resolving a name.
So all in all, I understand that such implicit namespaces would be helpful, but for anything other than module dependencies and - maybe - extend calls, it is not very likely to be implemented soon.

Hi @bkrrrr,
thanks a lot for your enhancement request. We've discussed this topic in our development team and decided to not implement this enhancement. We understand that this might be helpful but it introduces also many confusion which option is the best and why we would offer two options. Also as @codeworrior wrote many scenarios will not work. Thanks a lot and we're looking forward to your next enhancement request.


ITT-1098 - Minor addition to the login validation

This change prevents submitting the login form when one or both fields is/are empty.
I also changed the digest for the license hint - it wouldn't show on page load and instead show up in the next digestion cycle.

Nope, I'll try this again. Too many commits.


Editor audio response

Actually you can open it. It mirrors the state of the advanced audio response menu for the slideshow.
-> considering they are named identically they probably share the same  ImGui Id, easy fix!


Invalid css after $typi, Line 2

Hi, What program are you using to process the SCSS?
Compass is throwing an error:
Invalid css after $typi: (null:
Expected ")"
_typi.scss; Line 2

Which version of Compass are you running? As far as I know, support for Compass has been discontinued, so, I'm not sure if it uses the newer versions of Sass (with support for Sass maps).


[Feature] Support git commit message in fluxcd addon

Is your feature request related to a problem? Please describe.
When I use fluxcd addon in 'pull' pattern，my sre gitlab project looks like this:

Well, I can't figure out what a commit does quickly, util I click inside the commit mesage, it's not so convenient.
Describe the solution you'd like
If we can specify the commit mesage as we want, it should be better.
For example:

feature: allow shopping [automatically]

fix: login failed [automatically]

...
Describe alternatives you've considered
I would like the commit message comes from the origin git repository.
Additional context

Thanks for your report!
/assign @FogDong

@FogDong Have this issue supported now?


Accelerate to Mouse in Python via pyglet

Accelerate to Mouse in Python via pyglet module

Community Contribution
Link to live project, video, or image:

Sharing

[x] I would be happy to have my project shared on The Coding Train's social media!

Instagram:
Twitter:

https://thecodingtrain.com/learning/nature-of-code/1.6-acceleration-vector.html


Publish snapshot to Sonatype?

The current samples for Mortar and Flow are based on 0.17-SNAPSHOT and 0.9-SNAPSHOT, respectively. I am assuming those snapshots are on your internal or local maven repositories (I cannot find them on https://oss.sonatype.org/content/repositories/snapshots/).
Would you consider publishing those snapshots to the Sonatype snapshot repo?

You bet.
On Fri Jan 02 2015 at 10:04:26 AM Richard Cirerol<EMAIL_ADDRESS>wrote:

The current samples for Mortar and Flow are based on 0.17-SNAPSHOT
https://github.com/square/mortar/blob/master/mortar-sample/pom.xml#L27
and 0.9-SNAPSHOT
https://github.com/square/mortar/blob/master/mortar-sample/pom.xml#L62,
respectively. I am assuming those snapshots are on your internal or local
maven repositories (I cannot find them on
https://oss.sonatype.org/content/repositories/snapshots/).
Would you consider publishing those snapshots to the Sonotype snapshot
repo?
—
Reply to this email directly or view it on GitHub
https://github.com/square/mortar/issues/119.

Done, I think. Can you confirm that they're working for you in a few hours
(given Sonatype lag times)?
On Sat Jan 10 2015 at 7:06:08 PM Ray Ryan<EMAIL_ADDRESS>wrote:

You bet.
On Fri Jan 02 2015 at 10:04:26 AM Richard Cirerol <
<EMAIL_ADDRESS>wrote:

The current samples for Mortar and Flow are based on 0.17-SNAPSHOT
https://github.com/square/mortar/blob/master/mortar-sample/pom.xml#L27
and 0.9-SNAPSHOT
https://github.com/square/mortar/blob/master/mortar-sample/pom.xml#L62,
respectively. I am assuming those snapshots are on your internal or local
maven repositories (I cannot find them on https://oss.sonatype.org/
content/repositories/snapshots/).
Would you consider publishing those snapshots to the Sonotype snapshot
repo?
—
Reply to this email directly or view it on GitHub
https://github.com/square/mortar/issues/119.

Working for me. Thanks much!


Failed to preview

successfully see the editor
but when I tried to preview, maybe I missing something due to my poor js skills
<textarea name="content" cols="40" rows="10" class="markdown" required id="id_content"></textarea>

Not Found: /api/markdown/
[05/Aug/2018 16:31:49] "POST /api/markdown/ HTTP/1.1" 404 3628
I cant preview, always loading
By the way, I am using django with mongodb, so, maybe I have a lots of job to do

can you share the whole snippet of js


Can't update visualization options after the first render

Once a chart is rendered once, you can't change a visualization's options prior to re-rendering.
The options get overwritten in contour.js line 360
var opt = _.merge({}, this.options, visualization.options);

which is designed to allow the visualization-specific options (ie. .line(data, options)) override the Contour options (new Contour(options)).
As an example, render a line chart
var chart = new Contour({
    el: '#vis'
})
.cartesian()
.line(data)
.render();

then change the animationDirection of the line
chart.options.line.animationDirection = 'bottom-to-top';

and re-render with updated data
chart.setData(data).render();

The animation is still left-to-right.
It works if you set
chart._visualizations[0].options.line.animationDirection = 'bottom-to-top';

but that's a temporary workaround.

Slightly better workaround is
chart.select(0).options.line.animationDirection = 'bottom-to-top';

More relevant conversations for context:

naren shouldn't
          this.contourInstance.options.chart.animations.enable = false;
          this.contourInstance.render();```
update the title and not re-animate the line series? (edited)
or am i mis-interpreting how that works?
(i see the title change but it still animates)

drosen I think the option has already been copied to the line visualization instance by this point, so unfortunately you have to do a hack

naren what's the hack?
the line doesn't take an animate property does it?

drosen well I looked and the line visualization reads the chart’s options, so it should work

naren doing 'this.contourInstance.options.chart.animations.duration = 0' kinda sorta fixed it
it still animates the markers however :-/

drosen maybe give this a try:
```this.contourInstance._visualizations[0].options.animations.enable = false;

naren i don't know at this point how many visualizations are there though -- is just looping through _visualizations the same thing?
or does _visualizations also include axes etc?
drosen it’ll include line and tooltip but I don’t think axes are in there
naren markers?
drosen markers are part of line
naren that should be fine then, thanks
drosen it’s a hacky solution though
naren yeah, i'll file an issue on the contour github abt it
maybe a contourinstance.options({..} ) is called for
drosen that might be a good solution. Ideally the options passed into the visualization wouldn’t be a deep clone, but if I remember correctly, Jaime had a reason for it
drosen well, there’s Contour#select(index) which returns _visualizations[index], but you’d still need to get the length of _visualizations or keep incrementing index until it returns undefined
naren the lack of the leading underscore makes me feel better already
drosen or, there’s Contour#composeOptions() which appears to copy the options to the visualizations, but possibly might overwrite things you don’t want to overwrite

Oh, silly me, there's VisInstanceContainer#setOptions(options), which you would use as:
chart.select(0).setOptions({ line: { animationDirection: 'bottom-to-top' } });

This does does not seem to be documented anywhere. It should probably be documented near http://forio.com/contour/documentation.html#contour -> Contour#select().

@drosen0 added to the list for next docs updates, thanks


Think about to fix the issues found by adapter checker

I am an automatic service that looks for possible errors in ioBroker and creates an issue for it. The link below leads directly to the test:
https://adapter-check.iobroker.in/?q=https://raw.githubusercontent.com/Schnup89/ioBroker.openwrt

[x] [E153] common.dependencies must contain {"js-controller": ">=1.5.8"} or {"js-controller": ">=2.0.0"} or {"js-controller": ">=3.0.0"}
[x] [E506] Word "text_settings" is not translated to russian in admin/words.js. You can use https://translator.iobroker.in/ for translations
[x] [E605] No actual year found in copyright. Please add "Copyright (c) 2022 Schnup89<EMAIL_ADDRESS>at the end of README.md
[x] [E701] No actual year found in LICENSE. Please add "Copyright (c) 2022 Schnup89<EMAIL_ADDRESS>at the start of LICENSE

I have also found warnings that may be fixed if possible.

[x] [W400] Cannot find "openwrt" in latest repository
[x] [W505] setTimeout found in "main.js", but no clearTimeout detected

Thanks,
your automatic adapter checker.
P.S.: There is a community in Github, which supports the maintenance and further development of adapters. There you will find many experienced developers who are always ready to assist anyone. New developers are always welcome there. For more informations visit: https://github.com/iobroker-community-adapters/info

Solved with V0.0.3


Fix errors related to IDEA

This is my warmup PR. :smiley:
It adds IDEA project files to .gitignore and fixes some error messages in IDEA, so that IDEA instead shows unchecked assignment warnings.

Note that I had three assertions fails unrelated to this change. Strangely they were gone after a second test run.

The error message appeared in IDEA 14.1.4 and 142.4083.2. I first thought, this might be a EAP regression.

Thanks for the PR. I'm not a big fan of unchecked assignments, because bugs can hide in them. Is there another way to make IDEA happy? For example, would the following make it happy:
Function<T, ObservableValue<U>> g = t -> f.apply(t);
return new FlatMappedVal<>(src, g);

Plus, to prevent someone else coming along this code and realizing the cast/extra variable is unnecessary and removing it, it should be documented that it is a workaround for a bug in IDEA compiler, with a link to their bug tracking system (so that the workaround can eventually be removed once the bug is resolved).

Regarding the failing tests, it is a problem with the tests themselves. All those tests have to do with timing and break if scheduled events don't occur when within an expected time window, typically if the system is under heavy load. I used to see those tests failing occasionally on my old laptop. The right thing to do would be to make the tests robust.


Fix program name in zsh

In zsh, $0 becomes the function name in the function.
$ sh tetris -a
tetris: unrecognized option '-a'

$ zsh tetris -a
die_usage: unrecognized option '-a'

oops


Switch from strings.Title to golang.org/x/text/cases.Title

Fixes #29

Codecov Report

Merging #30 (a9094ca) into main (3ba4103) will increase coverage by 0.00%.
The diff coverage is 87.50%.

@@           Coverage Diff           @@
##             main      #30   +/-   ##
=======================================
  Coverage   93.07%   93.07%
=======================================
  Files          10       10
  Lines        2367     2369    +2
=======================================
+ Hits         2203     2205    +2
  Misses        127      127
  Partials       37       37

Impacted Files
Coverage Δ

generator/testmoqs/types.go
0.00% <0.00%> (ø)

generator/converter.go
97.16% <100.00%> (+<0.01%)
:arrow_up:


Is --cleanWorkdir=onSuccess available for cwltoil?

'CWLWorkflow' C/q/jobQn0554    INFO:toil.worker:Worker log can be found at /home/005/rg3930/dev/test_bcbio_cwl/cwltoil_work/toil-631282a7-9f7e-428f-aa0f-ac769d6dbb27/tmpcgS1HA.

Set --cleanWorkDir to retain this log

Shouldn't this flag be called --keepWorkDir if the intended effect is to keep the logs?

There are multiple options for this parameter: --cleanWorkDir=always, never, onSuccess and onFailure. The default is always and you can use the other options so only logs you want will be retained. onSuccess means that the work directory will only be removed when the job is successful and is typically the most useful debugging option.

Thanks @cket for the hints! Is it possible that cwltoil does not support it or am I passing it wrong on the cmdline?:
cwltoil: error: unrecognized arguments: --cleanWorkdir=onSuccess

Currently using the following cmdline from test_bcbio_cwl:
bcbio_vm.py cwlrun toil --no-container run_info-cwl-workflow -- --cleanWorkdir=onSuccess
--batchSystem torque --logLevel DEBUG

@brainstorm what happens if you try --cleanWorkDir onSuccess instead?

Thanks for the feedback @jessebrennan! For each run above I've rm -rf cwltoil_work *.log, so all intermediate files and jobstore should be gone. I'm giving it a try with that flag now!

That seems to work, unfortunately I'm stuck with issue https://github.com/BD2KGenomics/toil/issues/1772 now.
Closing this one though.


Compatibility with Node 14

Hello, I am trying to build Theia with the current LTS version of Node.js - 14.15.4, but the problem seems that dugite-extra is not compatible with Node 13+.
Do you plan to be compatible with Node 14?
Best regards,

even more problematic electron-builder requires node 14+
see https://github.com/evolvedbinary/fusion-studio/issues/91

@msujew @thegecko do you know if there are any plans to maintain and uplift dugite-extra? As far as I understand it is only used for applications which consume @theia/git (which I believe is true for both your cases), and it might be good to uplift as to support newer versions of theia.

@vince-fugnitto Yes, we should definitely uplift the dependency (for the time being). I've seen ideas of deprecating @theia/git in favor of the appropriate vscode extensions in the future. What's the plan on that?

@vince-fugnitto Yes, we should definitely uplift the dependency (for the time being). I've seen ideas of deprecating @theia/git in favor of the appropriate vscode extensions in the future. What's the plan on that?

@msujew that is what we are currently doing, it allows us to use git-related extensions such as gitlens. As for removing it as part of the framework (or deprecating it) there is the following issue https://github.com/eclipse-theia/theia/issues/7482. I believe Arm was planning on moving functionality they required out of it: https://github.com/eclipse-theia/theia/issues/7152.

I've updated dugite-extra to 0.1.15 removing the node constraint. I'll close this.


Support SystemTextJson

The SDK uses Newtonsoft by default, but what about SystemTextJson ?
Should SystemTextJson be the default?
How can I implement IJsonSerializer based on SystemTextJson?

Should SystemTextJson be the default?
AFAIK it's only available in more recent .NET Core versions (3.0 onwards?), and this SDK has to support a wide range of target versions. Could be doable with preprocessor directives though

How can I implement IJsonSerializer based on SystemTextJson?
I think it depends on which version of .NET you are running. In 6 and 7 there are Serialize and Deserialize overloads that take streams, so a shallow proxy on top could do. 5 and down have overloads that passes strings around, so for the serializer you could read the incoming stream into a string and pass that along, and for the deserializer take the string received and put it in a memorystream that you return perhaps?

@daveleek, I get it. What confused me was this part of the documentation:
The server api needs camel cased json, but not for certain dictionary keys. CamelCaseExceptDictionaryKeysResolver

But the implementation doesnt need that, right?
I'm using .NET6.0, so will implement that and post here! Thank you for the response.


⚠ Cioos Pacific CKAN has degraded performance

In 6207655, Cioos Pacific CKAN (https://catalogue.cioospacific.ca) experienced degraded performance:

HTTP code: 200
Response time: 5444 ms

Resolved: Cioos Pacific CKAN performance has improved in 352b299 after 5 minutes.


REP-004 - Internal Documentation Improvements

There's been a some discussions around Rez's documentation. Both in terms of "what can we document better" as well as "how can we improve what we have documented, already"
This REP will focus on the latter portion. From group discussion, these key points have been identified

[ ] Versioned Sphinx documentation

[ ] The majority choice is to build + integrate Rez documentation using readthedocs.io

[ ] Better / Fully automated Rez API documentation
[ ] Unity between Rez's wiki pages and the Rez Sphinx documentation

[ ] Eventually deprecate GitHub Pages, following a separate audit once contributors are happy with the Sphinx results

[ ] Documentation previews In PRs
[ ] Ensure clean Sphinx builds
[ ] Changelog as documentation

REP-004.1 - Versioned Sphinx documentation
The desire is to have documentation per-Rez-version as well as a "latest" / "stable" location which is consistent with Rez's latest release. Leveraging readthedocs.io to handle this will be a key point of this work.
REP-004.2 - Better / Fully automated Rez API documentation
Rez API documentation is currently out of date and manually generated. Use autosummary, sphinx-apidoc, or another, similar technology to make the API pages on the fly, so they're always kept up to date.
REP-004.3 - Unity between Rez's wiki pages and the Rez Sphinx documentation
Rez Sphinx documentation and Rez GitHub pages (wiki markdown) are currently separate. Making them as a single source will encourage more people to make documentation as well as make the documentation more meaningful, because hand-written documentation and now directly link to API documentation and vice versa.
REP-004.4 - Documentation Previews In PRs
Currently, hand-written documentation or API documentation changes must be reviewed locally in order to see the "rendered" result, which is not ideal. Assuming REP-004.1 is successfully implemented with readthedocs.io, we could leverage readthedocs.io's  PR documentation preview service. (See https://docs.readthedocs.io/en/stable/pull-requests.html for details)
REP-004.5 - Ensure clean Sphinx builds
Requires: REP-004.1
Rez's Sphinx documentation currently builds with about 90 warnings and a few error. A general clean-up would be greatly appreciated. Also, once Sphinx is better supported, we'll be able to go back in and improve our use of Sphinx roles (:ref:, :doc:, :mod:, :func:, etc) as a separate, parallel effort.

Other points to potentially add as part of REP-004

Changelog as part of documentation (example: https://ftrack-python-api.readthedocs.io/en/stable/release/release_notes.html)
Markdown / ReST

Those points may need more discussion before we add them to the REP.

I created https://github.com/nerdvegas/rez/discussions/1293 to help with discussions. It should hopefully make it eaiser to have sub-threads.

Added a PR for REP-004.1 and REP-004.2 over at https://github.com/nerdvegas/rez/pull/1295


Add support for LoongArch

boost 子模块 tools/build , please review,  thanks! @loongarch64/dev-team

LGTM


cant roll down more users

cant roll down more users
see only few user? what problems?

IG:dm last version

same problem here, makes it unusable :(


KAFKA-5422: Handle multiple transitions to ABORTABLE_ERROR correctly

cc @hachikuji

Merging to trunk and 0.11.0.


Rename middleware to filters

Middleware is an overloaded term and often used to describe larger, encompasses ESB products etc. A better term to describe this Dapr capability of injecting processing into the API pipeline is filter.
This change is to rename middleware to filter
Also related https://github.com/dapr/components-contrib/issues/231
Once a PR is submitted for this, raise a doc issue in the docs repo.

Nack on the proposal
Middleware is a industry standard term that refers to software that's assembled into an app pipeline
to handle requests and responses
ASP.NETCore uses the term: https://docs.microsoft.com/en-us/aspnet/core/fundamentals/middleware/?view=aspnetcore-3.1
ASPNetCore Google auth middleware: https://docs.microsoft.com/en-us/dotnet/api/microsoft.aspnetcore.authentication.google.googlemiddleware?view=aspnetcore-1.1
gRPC uses the term: https://godoc.org/github.com/grpc-ecosystem/go-grpc-middleware

Having review the term and with the references to other recent technologies also using middleware, we are going to keep this terminology.


Immediate notification for initial data should be optional for BriteContentResolver

For many situations getting the initial data is exactly what you want. However, there are other situations where you only want to be notified when the content changed.
For instance, when an SMS is sent or received. I don't want an immediate notification for that because that SMS may have just been sent or sent a long time ago.
It seems the startWith operator is what provides this functionality
https://github.com/square/sqlbrite/blob/master/sqlbrite/src/main/java/com/squareup/sqlbrite/BriteContentResolver.java#L105
For the time being I can use the skip(1) operator to ignore the first result cleanly.
It would be nice if there were another API to skip the immediate notification…
 public QueryObservable createQuery(
        @NonNull final Uri uri,
        @Nullable final String[] projection,
        @Nullable final String selection,
        @Nullable final String[] selectionArgs,
        @Nullable final String sortOrder,
        final boolean notifyForDescendents,
        boolean skipFirst)

The existing API could call this one with true passed in as the last argument.

I don't see what this offers over skip(1). We could put skipInitial() on QueryObservable which would just be a type-safe alias for skip(1), but if you put an operator between the two then you'd lose the ability to use it (already a problem with QueryObservable, I'll note).

True.
Maybe all that needs to be "fixed" is an update in the documentation to indicate that one should use skip(1) if one does not want the immediate notification. I'm relatively new to RxJava but I'm also comfortable digging into the open source to see how it all works. Other's may not initially realize that skip(1) is an legitimate work-around.

Yeah. At the very minimum that is what we should do.
In our code we usually put skip(1) // Ignore initial value. to further indicate the intention.

:+1:


Refactor write_y4m_frame() call into muxer.rs.

Coverage decreased (-0.01%) to 80.294% when pulling 23d51ab1f0ddf95cf3a07d18aadb2bf919c17521 on negge:repl into 3d5389903a5c3166d6b3dbc5ec4b4b9a0e0a57fe on xiph:master.


is it possible to not create immediately widgets and not save state?

Hi, thank for this library. I have a question, is it possible not to immediately create the initial routes at app startup? I need them to be created only when the tab is selected.
Thanks

have you found the fix for this? Building all tabs initially degrades performance of the app alot.

have you found the fix for this? Building all tabs initially degrades performance of the app alot.

Hi, thank for this library. I have a question, is it possible not to immediately create the initial routes at app startup? I need them to be created only when the tab is selected.
Thanks
have you found the fix for this? Building all tabs initially degrades performance of the app alot.

Hi, thank for this library. I have a question, is it possible not to immediately create the initial routes at app startup? I need them to be created only when the tab is selected.
Thanks
have you found the fix for this? Building all tabs initially degrades performance of the app alot.


Not getting auto generation

Hi Team,
I'm a new bee to visual studio and trying to do HTML. I installed plugin even after I couldn't get auto-detection? my boilerplate is enabled in visual studio file extension is also ends with .html.

You likely have to download an auto-complete extension too.  E.g.: "auto complete tag"

Please make sure the file extension in the bottom right corner says 'HTML'. If not, you can click on it and see it to HTML and then it should work.


fix: Slow rendering of JSON and RAW response

Description
API response JSON/RAW rendering is too slow.
Codemirror editor was not rendering the required lines of code on increasing the viewport. (video).
Codeeditor upon the first render is set to 100% of container height. And editor also renders x extra lines above and below the viewport. (this is can be set by updating viewportMargin parameter in CodeEditor/index.tsx - now set to 10). Codemirror is using Observers to achieve this.
Solution 1

Can increase viewportMargin to 50/100. Won't have much impact performance.
But the approach is not an ideal solution. Because what if the screen is large and 100 is also fewer lines of code??

Solution 2

Codemirror has viewportChange event, but the problem here is that the viewport of the container is resizing but not the editor itself.

Solution 3

Refresh editor on resize of the container. This is currently implemented method. (edtior.refresh() is the function used to refresh)

Fixes #16705
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Checklist:

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

/ok-to-test sha=58e465b

/ok-to-test sha=de4a111

/ok-to-test sha=fb35b71

/ok-to-test sha=57222ee


Made Feedback Form-2 responsive

Fixes Issue🛠
Closes #1063
Description👨💻

Made responsive across all devices.
Changed the background color and added a gradient to the form card and added green border colors to placeholders.
Added keyframe animation to the stars

Type of Change📄

[ ] Bug fix (non-breaking change which fixes a bug)
[ ] Style (non-breaking change which improves website style or formatting)
[ ] Responsiveness (non-breaking change which improves UI/UX on different screen sizes)

Checklist✅

[ ] I am an Open Source contributor
[ ] I have performed a self-review of my code
[ ] My code follows the style guidelines of this project

Screenshots/GIF📷

@abhirajadhikary06
Few things to note:

Always update the branch of your forked repo to avoid the conflicts, I have resolved this time.
Write [x] in the checkboxes of the PR description to check it.
Keep all your files with indent of 4 spaces always.

@Rakesh9100 Is any more changes required to get the file to a mergeable state.

Edit the PR description and check the checkboxes @abhirajadhikary06

Edit the PR description and check the checkboxes @abhirajadhikary06

Done...


Start RP.Flash implementation

I think the API could be improved:

Maybe not use the C types here but HAL types
Have dedicated types for number of pages and number of sectors rather than a precondition.

I've started working on integrating this with the HAL.Block_Drivers interface, tests not yet passing.
https://github.com/JeremyGrosser/rp2040_hal/tree/flash

I would really prefer to have the basic Erase and Program procedures available to the users.
The HAL.Block_Drivers interface can be useful but I want to be able to have my own access to the flash.
So I would move the block stuff to RP.Flash.Block.

Yep, I'm planning to put those back in, just haven't had a chance yet.

Ok, I've removed the HAL.Block_Drivers implementation for now, just to focus on getting the flashing right. I've defined a Flash_Offset type and added some conversion functions and preconditions to ensure you can't use flash erase/program on memory areas that aren't actually in flash.
The FLASH_BLOCK_SIZE that pico-sdk defines as 64k is probably a bug... It means all erases must be a multiple of 64k, so erasing anything smaller than that will have unexpected consequences. Therefore, I've made Block_Size an argument to Erase with a precondition that ensures that it's a multiple of 4k (Sector_Size). The flash needs to be erased before programming, so even though you can program pages as small as 256 bytes, an entire 4k sector needs to be erased first.
I wrote unit tests for Erase and Program, but it's commented out of the test suite right now, because GNATcov_RTS Witness functions try to execute from flash during coverage runs. I tried adding the Linker_Section and No_Inline aspects to those functions, but that didn't fix it. I think there might be some weird interaction with pragma Pure that's used by those packages, but I can't remove that pragma without patching the code gnatcov generates.
https://github.com/JeremyGrosser/rp2040_hal/blob/flash/src/drivers/rp-flash.ads

The FLASH_BLOCK_SIZE that pico-sdk defines as 64k is probably a bug... It means all erases must be a multiple of 64k, so erasing anything smaller than that will have unexpected consequences. Therefore, I've made Block_Size an argument to Erase with a precondition that ensures that it's a multiple of 4k (Sector_Size).

I don't understand the Block_Size argument. In the SDK it's a fixed constant that's passed to the rom function.

   Flash_Size  : constant := 16#0100_0000#;
   --  XIP maps up to 16 MB of flash. Actual flash chip may be smaller.
   Page_Size   : constant := 16#100#;
   Sector_Size : constant := 16#1000#;

I am not a huge fan of sizes in hexadecimal as my brain is not able to convert them to decimal :sweat_smile:

Ok, I definitely was confused about the meaning of Count. I've changed Block_Size to have a default value of 65536 and renamed Count to Length, to match common Ada naming.

Just for some more context, the bootrom has this comment near the erase function
// block_size must be a power of 2.
// Generally block_size > 4k, and block_cmd is some command which erases a block
// of this size. This accelerates erase speed.
// To use sector-erase only, set block_size to some value larger than flash,
// e.g. 1ul << 31.
// To override the default 20h erase cmd, set block_size == 4k.

I'm just going back to having Block_Size be a constant... I can see now that they did that to avoid this confusion. I'm not a fan of the way they've overloaded the meaning of block_size in flash_range_erase, but I'm sure it had something to do with reducing code size.


bigInteger not parsed with parseInt by default?

I'm using the pg adapter and it seems biginteger columns are parsed as strings in my bookshelf model.
Shouldn't bookshelf automatically parse those with parseInt or it should be done manually in my parse() model method?

That's a feature of the node-postgres module due to the known precision problems of Javascript when dealing with bigints: https://github.com/brianc/node-postgres/pull/353.
In any case, that would be something to discuss in knex, not Bookshelf. See also this issue for further info: https://github.com/tgriesser/knex/issues/387.


Negative SHAP expected value

Hi Scott,
Thank you for this wonderful library. I have a question regarding expected value property of explainer.
In your notebook Census income classification with LightGBM

What does this array mean for expected value , how can value be negative here

The negative is probably coming from log loss.

Yes. It should be equal to model.predict(X, raw_score = True).mean()


Remove "timestamps: 0" from .formatter.exs

As discussed in https://github.com/phoenixframework/phoenix/pull/2866#discussion_r184858272

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


Parabolic Sar - feature-request

First: GREAT LIBRARY!!, @bukosabino!
(better than all the others!!)
And because of that I would be happy to find the parabolic sar from you in it. It's very complicated. I will try to code it for myself. If I'm ready with a good solution I will send it to you. But maybe you're faster! ;)

Does anybody know how to code an exact parabolic SAR?
Because I found examples leading to wrong results...

Hi @TravelTrader ,
When I have time, I will try to implement the indicator based on this explication: https://school.stockcharts.com/doku.php?id=technical_indicators:parabolic_sar
Best,
Dario

Thanks @bukosabino!
The reason I am asking the second time is that you're library is the best I found. Your ATR for example is the most exact version I ever found. And for Parabolic SAR I tried all examples and libraries I found online over the last 7 months and in addition to that I tried to code it for myself.
But:
All examples were totally different from the Parabolic SAR of TOS, Yahoo Finance, IB, TradingView, ...
So, I am sure, your version would be the first function I can use for calculations, backtests and charting with Python!
I would be very happy!! :smiley:

@bukosabino, I just opened a pull request for you for parabolic SAR

Hi all,
@mprivat has pushed a PSAR implementation which was added to the library in the last version (v0.5.9). To install the last ta version:
pip install -U ta

Thank you so much @mprivat


Add @andreyvelich to kf-kcc members

I added myself to kf-kcc members from @kubeflow/automl-leads side to help maintaining CI/CD for Training and AutoML WG.
Related: https://github.com/kubeflow/community-infra/issues/14.
/assign @jlewi
/cc @gaocegege @johnugeorge

/lgtm
/approve


kafka samples

Note:

When rebalance happens - All the group consumer instances will stop receiving messages until the rebalance ends!

/lgtm

/retest


not open in browser after answer the question from search result

problem
not open in browser
after answer the question
from search result
expected result
automatically open in browser (new tab or new brower window)
after answer the question
from search result
current work around
user must click the url shown
on url column
from search result
info

env where vidsrc-search installed : google colab
os on google colab : ubuntu jammy x86_64
brower used when open google colab : google chrome
user os : win 11

trial 1st just hit enter on last question
 > Choose an index to open in browser: 1

 [Warning] The content of the movie is hosted on a third party site. The
           site is not endorsed by the author or checked for its quality,
           content, or authenticity. The author of vidsrc-search disclaims
           any responsibility, express or implied, of the consequences
           as a result your usage or dependence on the website provided
           through this tool.
 [Warning] The following window shown will be the contents of vidsrc.to

 > I have read and understood the conditions above (Y/n)

trial 2nd
> Choose an index to open in browser: 1

[Warning] The content of the movie is hosted on a third party site. The
          site is not endorsed by the author or checked for its quality,
          content, or authenticity. The author of vidsrc-search disclaims
          any responsibility, express or implied, of the consequences
          as a result your usage or dependence on the website provided
          through this tool.
[Warning] The following window shown will be the contents of vidsrc.to

> I have read and understood the conditions above (Y/n) Y

[Info] Verifying internet connection
[Info] Testing connection by pinging google.com
[Info] Testing connection by pinging vidsrc.to

[Info] Checking for local cache
[Info] Local cache not found: Getting remote html
[Info] Remote html request status code is 200
[Info] Writing remote html content to local cache
[Info] Opening #1 'Steinheist (2022)' in new browser window

best regards

Are you running vidsrc-search on Google Colab?
If so, not much can be done about this issue. Because Python is running on a remote server, it can only access the services and apis that the server provides and allows. So unless Google Colab has an api that allows Python to remotely control the connected browser, getting a local browser tab open while running on Google Colab is not possible.
What probably happened after you opened an entry is that Python attempted to open the default browser on the Google Colab server. This obviously resulted in nothing, since there is probably no browser install on the Google Colab server, which is why nothing happened.
If you would like vidsrc-search automatically opening a browser after selecting an entry, try running it on the machine that you'd like to see the browser opened on (e.g. locally).

Are you running vidsrc-search on Google Colab?

yes, loud and clear explanation,
thank you


Add ciao-deploy create-bat-workloads and use in SingleVM

This PR introduces a new tool, ciao-deploy with commands called "create-bat-workloads" and "create-cnci" which will create the BAT workload on a running cluster and setup the CNCI respectively.
For CNCI this command downloads the image and customises it and then uploads it to the cluster
For BAT this encompasses all steps from downloading the image from the remote location, uploading the image to controller and creating the workload derived from it.
This PR should be ready for review immediately.
It introduces a dependency on Cobra (and vendors it) which is the command line tool used by k8s and provides a good framework for managing commands & subcommands, etc. This dependency was discussed with @markdryan in person and it seems like this would also be a good solution for ciao-cli in the future.

Coverage decreased (-0.02%) to 65.717% when pulling 77943f5331f9cd4b7bbd0813060c111000b31be4 on rbradford:deployment-2 into 3b6ebc4aca4eaf49861519222bfb9b5fee5c95b0 on 01org:master.

Semaphore passed first time \o/

@mcastelino  @kaccardi  Now with CNCI setup too. I had thought you might want to test and land this separately before the full cluster setup is in place. However that should be happening within the next week so i'm easy either way. Unfortunately the PR is only going get bigger however it doesn't touch very much outside ciao-deploy.

Coverage increased (+0.02%) to 65.695% when pulling f8a99b6e74357e1c69233a494e0d18c627a70638 on rbradford:deployment-2 into 6ee7e569ba87af2cf8f97749b44baa94efaae332 on 01org:master.

Coverage increased (+0.02%) to 65.699% when pulling 3cfbdd1054213a59fe447e95b9b2713e8aa1e652 on rbradford:deployment-2 into 6ee7e569ba87af2cf8f97749b44baa94efaae332 on 01org:master.


Bumped angular version to ^5.2.3

Coverage remained the same at 100.0% when pulling 6e84ad1f995b575eaa64011637af58a24684ca80 on atishin:bump-anguar into 5752b5f6d04846de59b72f779bf1a0bb2bf8d54c on fknop:master.

I don't think it is really necessary to make a new release for this as angular is a dev dependency. And it's highly unlikely that they will break pipes syntax.

This is done as part of #37


Messages not sent when destroying scripts

When you press the button in the hand will become stuck in "primed" mode because the grip point never unhighlights the object.
This similarly happens when you pull the bubbler away from your hand when you're triggering it. The old grip never gets the chance to tell it to stop and you can drop it while it's still firing.

This still happens but I switched to supporting the .enabled flag instead of destroying the script, which I think is a better design. While this is still technically an issue, I think it's not unexpected behaviour for Destroy() and there are better alternatives that do work.
I also have plans to remove two grips attaching to the same object in issue #9. That will fix that half of the problem.


Pause the game if everyone connected has it paused

Might be better to make sort of simple poll with a question like "should we pause the game?" and press y/n for everyone. and similar to unpause or maybe even auto start unpause poll


Implement an api

Move the reusable logic from the CLI into the CFLint class

Is this completed for 1.3.0?

I think so.  It is minimally documented on the readme.


iOS 启动报错 Null check operator used on a null value

[VERBOSE-2:ui_dart_state.cc(209)] Unhandled Exception: Null check operator used on a null value
#0      MethodChannel.binaryMessenger (package:flutter/src/services/platform_channel.dart:121:86)
#1      MethodChannel._invokeMethod (package:flutter/src/services/platform_channel.dart:146:36)
#2      MethodChannel.invokeMethod (package:flutter/src/services/platform_channel.dart:329:12)
#3      MethodChannelNetworkInfo.getWifiIP (package:network_info_plus_platform_interface/method_channel_network_info.dart:38:26)
#4      NetworkInfo.getWifiIP (package:network_info_plus/network_info_plus.dart:60:22)
#5      AnyInspect._getServerAddress (package:anyinspect/src/anyinspect.dart:69:38)
#6      AnyInspect.start (package:anyinspect/src/anyinspect.dart:92:34)
#7      main (package:fstore/main.dart:39:14)

版本
  anyinspect: ^0.1.1
  anyinspect_plugin_network: ^0.1.0
  anyinspect_plugin_shared_preferences: ^0.1.0

flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 2.5.3, on macOS 12.0.1 21A559 darwin-arm, locale zh-Hans-CN)
[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
[✓] Xcode - develop for iOS and macOS
[✓] Chrome - develop for the web
[✓] Android Studio (version 2020.3)
[✓] IntelliJ IDEA Ultimate Edition (version 2021.1.3)
[✓] VS Code (version 1.62.3)
[✓] Proxy Configuration
[✓] Connected device (2 available)

• No issues found!

需要在 main 开始处调用 WidgetsFlutterBinding.ensureInitialized();
import 'package:anyinspect/anyinspect.dart';
import 'package:anyinspect_plugin_network/anyinspect_plugin_network.dart';
import 'package:anyinspect_plugin_shared_preferences/anyinspect_plugin_shared_preferences.dart';

Future<void> main(List<String> args) async {
  // Must add this line.
  WidgetsFlutterBinding.ensureInitialized();

  AnyInspect anyInspect = AnyInspect.instance;
  anyInspect.addPlugin(AnyInspectPluginNetwork());
  anyInspect.addPlugin(AnyInspectPluginSharedPreferences());
  anyInspect.start();

  // ...
}


Adding delta PCC to AMALGKIT output

I added delta PCC to the AMALGKIT output as shown on the paper figure, like this:
uncorrected
Malus_domestica.2.correlation_cutoff.pdf
corrected
Malus_domestica.2.correlation_cutoff.sva.pdf
I also added a table to the output, which contains mean PCC, median PCC and variance for each boxplot category (bwbw, bwwi, wiwi, wibw) and each round. This is the data I used for the cross-species delta PCC plots of the paper figure. Here's how the table looks like:
Malus_domestica.sva.correlation_statistics.zip
Should I add these plots to csca output?
If it is OK like this, I'll push the update!

The plot looks good, and the table would be useful. Please proceed with pushing the update. I also look forward to the csca update as well.

https://github.com/kfuku52/amalgkit/commit/36428a5f2245e41cbb92ad2402d4e83c1e52ece5
I added the PCC plots to csca as well! This is how they look like:
csca_delta_pcc_boxplot.pdf
The style is a bit different from the figure I made for the paper since those plots were done in Python, but now the styles are consistent between the PCC and delta PCC plots.

Thank you! If "Group" refers to the sample_group column, using "Sample group" will make its meaning clearer. Some of the curate plots may still display legacy labels like "Organ", but they should now be updated to "Sample group" as well. Could you please make these corrections?

Good points! I'll update the PCC plots and legacy labels!


[Popup] Wrong positioning in a scrolling modal

Steps to Reproduce

Open testcase
Click on the blue buttons (you can scroll the modal)
Increase the width of the preview frame (maximize it) and click again

Expected
The Popup should position next to its button.
Result
The Popup does not position correctly and goes to the right, this only happens when the modal is in "scrolling mode" (when bigger than the window).
Testcase
http://jsfiddle.net/6mt4kj5e/2/

Have anyone know how to fix this issue ?. I meet the similar issue when the popup is show in scrolling modal content


Custom error type per middleware

Collect them all up and turn them into an enum so it's still typesafe.

Handled in the new syntax.


Added all inf imb functions to tutorial

But now the notebook is long and confusing. Any ideas, e.g. titles, to structure it more?

I have a few comments.
Regarding the length of the notebook, I believe this is not a problem; as long as everything is clear and well-labeled, length should not affect the usability.
I would probably include titles to the sections that refer to the functionality presented rather than the type of data employed. For example I would rename:
3D Gaussian with small variance along z -> Coordinate information in asymmetric data
4D isotropic Gaussian -> Coordinate information in symmetric data
10D isotropic Gaussian dataset -> Coordinate information and feature optimization in high-dimensional symmetric data
Sinusoidal function -> Coordinate information in asymmetric nonlinear data
Moreover, I would add two section headings to the last part (sinusoidal function) separating the 2D, the 3D, and the 16D cases.

Also, some things to fix:
The cell after the second plot in the 10D Gaussian section is a bit weird:
"numpy not add two arrays element wise". I believe there is an error here?
Adding plt.plot() at the end of cells that create plots in the notebook will eliminate the annoying messages like "Text(0, 0.5, '$x2$')"

Thanks Romina! Great work. I agree with Claudio that the nootebook is not too long. But I disagree with the renaming of the section, I think naming the sections with the data type is more intuitive and practical for the average reader of the notebook. Romina what is your opinion on this?
In addition to Claudio's minor corrections I have the following ones:

there are some cells with the content "Type Markdown and LaTeX:  𝛼2". These should probably be removed.

When you list the "included attributes", you should probably change the original text

#### Included attributes:
#### Prediction of two feature subsets vs. each other: return_inf_imb_two_selected_coords

into something like
### Included methods:
- Prediction of two feature subsets vs. each other: return_inf_imb_two_selected_coords

since we are talking about methods of the class, and since the hyphens "-" are used for bullet points while the hash symbol "#" creates sections and subsections.

Thank you very much Romina! merging this now

Yes I think that a code snippet in the neighbourhood overlap would be definitely needed


Add library info

What is logf4j version is supported? I can not compile Logger.getRootLogger().addAppender(cloudwatchAppender); with log4j 2.7
What is your library version??? pom.xml says 0.0.6-SNAPSHOT, maven repository tells 0.0.2. What is last or where are 3,4 and 5 versions?

Log4j 2.x is not supported. Latest supported version is 1.2.17 as per  pom.xml
Release of versions 3 and 4 failed because of artifact signature... but not sure about what happened with version 5... it should be on maven central by now... :-(

version 0.0.3 is now out.


docs: add json-schema

Adds JSON schema at https://flagd.dev/schema/v0/flagd-definitions.json, which can be used to add validation to flagd config files.
Note, to accomplish this cleanly, I have cloned the schemas module twice (at different versions) one is used for json releases (and tagged to something like json/json-schema-v0.1.2) while the other is for the protobuf generation and tagged to a release of that. I don't love this, but I think it's needed.
See schema here: https://deploy-preview-1115--polite-licorice-3db33c.netlify.app/schema/v0/flagd-definitions.json
Once this is available, I intend to update a bunch of docs and examples to leverage this schema, and and an blurb in the docs about it.

@thisthat the fixes for your issues are included here.


Add Missing live_component Handler

Hello 👋🏼
After the introduction of live_component events to the automatic telemetry attachment, I noticed that the handler was being detached at runtime due to a function clause error. The current handle_event_start/4 function does not accept :live_component events.
This PR adds another function clause (which is the same as the existing function clause except it does not track session data, as session data is not available in live components). It also adds copies of the relevant tests that exercise the function.

Original error:
21:15:01.209 [error] Handler {Appsignal.Phoenix.LiveView, [:phoenix, :live_component, :handle_event, :start]} has failed and has been detached. Class=:error
Reason=:function_clause
Stacktrace=[
  {Appsignal.Phoenix.LiveView, :handle_event_start,
   [
     [:phoenix, :live_component, :handle_event, :start],
     %{monotonic_time: -576460747140818208, system_time:<PHONE_NUMBER>209609917},
     %{
       component: MyAppWeb.MyLive,
       event: "my_event",
       params: %{"my_value" => ""},
       socket: #Phoenix.LiveView.Socket<
         assigns: %{...} (truncated)

@aj-foster The build is failing on CI. Looks like there are some version specific issues. Looks like some calls are made multiple times?

I’ve continued work on this in https://github.com/appsignal/appsignal-elixir-phoenix/pull/50, which includes a fix for the tests and a patch that removes the need for a separate clause. Let’s move discussion about this pull request there.


Game crash when hovering over a bundle.

I'm having an issue where my game will crash whenever I move my mouse over a bundle
Steps to reproduce the behavior:

Open the inventory
Move the mouse over an empty bundle (haven't tested it with anything in them)
Have your game crash

1.16.3
forge-1.16.3-34.1.23
This happens in both JEI and the creative inventory.
Crash report attached
crash-2020-10-23_16.48.39-client.txt
P.S. looks like a great mod!

That's strange, I just tested with that exact version of Forge and got no problem hovering an empty bundle in creative inventory :/ Have you tried updating the Forge version?

I just did, the issue still persists, here's the crash report
crash-2020-10-24_15.28.28-client.txt

From the crash log I assume there is something really strange going on here. Essentially the fornt render method is probably deobfuscated in the "release" version of Forge but not in the dev build... Maybe the two version uses different mappings, and that's why you have this error. However even stranger is that the 1.16.3-34.1.23 version I installed doesn't have this difference, which is really really weird... I will see if I can find this difference and patch it out, but in this situation is really really hard for me to even think about something, because I can't call a method or another based on what the build is (the build won't even compile)

same crash here. 1.16.4 forge ver 35.1.7
crash report mentioned fonts

downgraded to 1.1, and seems to work just fine. What changes were made between 1.1 and 1.3?

Just somme bug fixing. As I said this bug is related to the Forge mappings in release, which although they are the same for some reason there are some differences. I'll see when I'll update the mod if there is a 100% way of avoiding this problem

I had the same error, then when i tried to download the 1.1 like Skjolbir, i saw that the version that curseforge downloads as latest is for 1.15.2 then goes on to the 1.16.3 so I tried  to download the 1.3 for the 1.16.3 and it worked without a problem

just to note that bundles 1.3 also crashes on 1.16.5. same stack trace as above.
i hovered over the item in jei recipes (right click the leather item) and the game crashed.

Fixed in version 1.4
https://www.curseforge.com/minecraft/mc-mods/bundles-mod/files/3903683


Fail FV tests on core dumps and races.

Description

We've had detection for races and core dumps for a long time but we weren't failing tests as a result.  This made the detection not very useful: when we did get a race/core dump there was no associated log because logs are hidden unless the test fails.

Make tests fail in AfterEach() if a race/corefile is detected.  This causes Ginkgo to dump logs, making it much easier to debug the race/panic.
Make it clear what the result of the test was to avoid confusion between "test failed" and "a race was hit during the test".
Stop dumping core for test-workload and other test binaries.  Some panic on failure so we were getting lots of core files for no reason!
Fix race caused by configuring logrus to disable its lock.  Struggled to find a better fix.  The problem is that Felix starts some threads before it does that configuration, and then can log before the lock gets disabled.  It's only safe to disable the lock once our logging configuration is in place.

Related issues/PRs

CORE-10585
Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

TBD

Reminder for the reviewer
Make sure that this PR has the correct labels and milestone set.
Every PR needs one docs-* label.

docs-pr-required: This change requires a change to the documentation that has not been completed yet.
docs-completed: This change has all necessary documentation completed.
docs-not-required: This change has no user-facing impact and requires no docs.

Every PR needs one release-note-* label.

release-note-required: This PR has user-facing changes. Most PRs should have this label.
release-note-not-required: This PR has no user-facing changes.

Other optional labels:

cherry-pick-candidate: This PR should be cherry-picked to an earlier release. For bug fixes only.
needs-operator-pr: This PR is related to install and requires a corresponding change to the operator.

/merge-when-ready


still warning on old formula names for some commands

Please always follow these steps:

[x] Confirmed this is a problem with running a brew command and just brew installing one or more formulae? If it's a formulae-specific problem please file this issue at https://github.com/Homebrew/homebrew-core/issues/new
[x] Ran brew update and retried your prior step?
[x] Ran brew doctor, fixed all issues and retried your prior step?
[x] Ran brew config and brew doctor and included their output with your issue?

Re: https://github.com/Homebrew/brew/pull/1485, I am still having this issue, specifically I noticed it with brew uses and formulae from homebrew-dupes. For example:
> % brew uses rarian
Warning: Use tcl-tk instead of deprecated homebrew/dupes/tcl-tk
gnumeric                                                                           zenity
> % brew uses scrollkeeper
Warning: Use tcl-tk instead of deprecated homebrew/dupes/tcl-tk
> % brew uses --include-build --include-optional scrollkeeper
Warning: Use make instead of deprecated homebrew/dupes/make
Warning: Use tcl-tk instead of deprecated homebrew/dupes/tcl-tk

troubleshooting output:
> % brew config
HOMEBREW_VERSION: 1.2.1-123-gd1f802c37
ORIGIN: https://github.com/Homebrew/brew.git
HEAD: d1f802c372563e36fe656c4662cae9e5b4138cdd
Last commit: 7 hours ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: 8787b257fc04919bf327fdcad55f074d4323eb9b
Core tap last commit: 2 hours ago
HOMEBREW_PREFIX: /usr/local
HOMEBREW_REPOSITORY: /usr/local/Homebrew
HOMEBREW_CELLAR: /usr/local/Cellar
HOMEBREW_BOTTLE_DOMAIN: https://homebrew.bintray.com
CPU: quad-core 64-bit haswell
Homebrew Ruby: 2.0.0-p648
Clang: 8.1 build 802
Git: 2.13.0 => /usr/local/bin/git
Perl: /usr/local/bin/perl => /usr/local/Cellar/perl/5.24.1/bin/perl
Python: /Users/alex/anaconda/bin/python => /Users/alex/anaconda/bin/python2.7
Ruby: /usr/local/bin/ruby => /usr/local/Cellar/ruby/2.4.1_1/bin/ruby
Java: 1.8.0_131, 1.8.0_121, 1.8.0_112
macOS: 10.12.5-x86_64
Xcode: 8.3.2
CLT: <IP_ADDRESS>.1.1492020469
X11: 2.7.11 => /opt/X11
MacPorts/Fink: /opt/local/bin/port

> % brew doctor
Please note that these warnings are just used to help the Homebrew maintainers
with debugging if you file an issue. If everything you use Homebrew for is
working fine: please don't worry and just ignore them. Thanks!

Warning: Anaconda is known to frequently break Homebrew builds, including Vim and
MacVim, due to bundling many duplicates of system and Homebrew-available
tools.

If you encounter a build failure please temporarily remove Anaconda
from your $PATH and attempt the build again prior to reporting the
failure to us. Thanks!

Warning: "config" scripts exist outside your system or Homebrew directories.
`./configure` scripts often look for *-config scripts to determine if
software packages are installed, and what additional flags to use when
compiling and linking.

Having additional scripts in your path can confuse software installed via
Homebrew if the config script overrides a system or Homebrew provided
script of the same name. We found the following "config" scripts:
  /Users/alex/anaconda/bin/curl-config
  /Users/alex/anaconda/bin/freetype-config
  /Users/alex/anaconda/bin/icu-config
  /Users/alex/anaconda/bin/libdynd-config
  /Users/alex/anaconda/bin/libpng-config
  /Users/alex/anaconda/bin/libpng16-config
  /Users/alex/anaconda/bin/python-config
  /Users/alex/anaconda/bin/python2-config
  /Users/alex/anaconda/bin/python2.7-config
  /Users/alex/anaconda/bin/xml2-config
  /Users/alex/anaconda/bin/xslt-config
  /usr/local/pspdev/bin/psp-config

Warning: You have MacPorts or Fink installed:
  /opt/local/bin/port

This can cause trouble. You don't have to uninstall them, but you may want to
temporarily move them out of the way, e.g.

  sudo mv /opt/local ~/macports

Warning: You have unlinked kegs in your Cellar
Leaving kegs unlinked can lead to build-trouble and cause brews that depend on
those kegs to fail to run properly once built. Run `brew link` on these:
  root

Warning: You have a curlrc file
If you have trouble downloading packages with Homebrew, then maybe this
is the problem? If the following command doesn't work, then try removing
your curlrc:
  curl https://github.com

Warning: Homebrew's sbin was not found in your PATH but you have installed
formulae that put executables in /usr/local/sbin.
Consider setting the PATH for example like so
  echo 'export PATH="/usr/local/sbin:$PATH"' >> ~/.zshrc

What does brew tap output?

> % brew tap
caskroom/cask
caskroom/drivers
caskroom/versions
homebrew/bundle
homebrew/core
homebrew/dupes
homebrew/games
homebrew/gui
homebrew/science

Just FYI you can brew untap homebrew/dupes, homebrew/games, homebrew/gui as they are empty.
What's the output of grep -r homebrew/dupes/tcl-tk /usr/local/Homebrew/Library/Taps?

> % grep -r homebrew/dupes/tcl-tk /usr/local/Homebrew/Library/Taps
/usr/local/Homebrew/Library/Taps/homebrew/homebrew-science/pymol.rb:  depends_on "homebrew/dupes/tcl-tk" => ["with-threads", "with-x11"]

This needs to be fixed/reported in homebrew/science in that case.

There is also an issue with a homebrew/core formula, see edited comments. Will followup separately.

For reference there is already an issue filed for homebrew/science problem at https://github.com/Homebrew/homebrew-science/pull/5483. I see also that there are some unmerged PRs for crosstool-ng that do not adjust the homebrew/dupes/make dependency, e.g. https://github.com/Homebrew/homebrew-core/pull/11999. Is there some known issue or should I file an issue?

One more update for completeness, here's a grep for all references to homebrew/dupes.
> % grep -r homebrew/dupes /usr/local/Homebrew/Library/Taps
/usr/local/Homebrew/Library/Taps/homebrew/homebrew-core/Formula/crosstool-ng.rb:  depends_on "homebrew/dupes/make" => :optional
/usr/local/Homebrew/Library/Taps/homebrew/homebrew-science/circle.yml:        && brew tap homebrew/dupes
/usr/local/Homebrew/Library/Taps/homebrew/homebrew-science/pymol.rb:  depends_on "homebrew/dupes/tcl-tk" => ["with-threads", "with-x11"]

Fixed in https://github.com/Homebrew/homebrew-core/commit/ba4f908be0c757764d6ecf4811ed9d04077136ea, thanks.


"key_padding_mask" in attention mechanism not be implemented?

Hi, arghosh ! The idea of the paper is so amazing , and the code is so beautiful. Here I want to confirm some details about your code . I found that the sequence length of a student is 200 in your setting, and use 0 as padding number. In your implemention of attention mechanism I just found the upper triangular matrix as mask to ignore the influence of the time steps after current time step, but the padding values of the sequence may also should be ignored, that means these values should not be used to compute attention score. Well , thanks for you if you could solve my problem.    : )

Hi huajian123. Thank you for your interest. In L204, i defined an upper triangular matrix of ones, in L205 mask is inverted (so lower triangular matrix of ones). in L305, I set values to -inf to all future items (including padded indices). However, for computing prediction at those padded time indices, it computed attention over possibly padded indices. But it does not matter since, prediction loss does not flow through those padded timesteps. Hope my explanation helps.

Okay, arghosh, thank you for your detailed explanation. It may because that the padded values is in the right of the sequence, and use the upper triangular matrix could ignore the padded index. Well, thanks again.  : )


Test issue

Migrate the split-repo https://github.com/googleapis/python-api-gateway to https://github.com/googleapis/google-cloud-python.
The migration readiness criteria are the following, which we track via GitHub labels on this issue. These criteria apply to the split repo we are migrating from:

No open issues
No open PRs
No handwritten samples
No system tests
No client-specific customizationsYESITSME

Testing what happens when issue is closed.

Reopening


maint: Remove references to old files that were removed.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 76.59%. Comparing base (6360c7d) to head (0f34cd9).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4332      +/-   ##
==========================================
- Coverage   81.53%   76.59%   -4.95%
==========================================
  Files         107      107
  Lines       52271    52271
==========================================
- Hits        42621    40038    -2583
- Misses       9650    12233    +2583


SocketException

I am not sure if this is at all relevant, but I am starting to see a lot of SocketExceptions for a service oriented system where all services (+20) go through the same service to log.
This stacktrace which is saying nothing about my code, but is pointing to a direction of perhaps request exhaustion of my requests to ApplicationInsights.
System.Net.Http.HttpRequestException:
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw (mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089)
at System.Net.Http.HttpClient+d__58.MoveNext (System.Net.Http, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a)
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw (mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089)
at Microsoft.ServiceProfiler.Agent.FrontendClient.StampFrontendClient+<>c__DisplayClass9_01+<<HttpGetAsync>b__0>d.MoveNext (Microsoft.ServiceProfiler.Agent.FrontendClient, Version=2.6.1909.2701, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a)    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw (mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089)    at Microsoft.ServiceProfiler.Agent.FrontendClient.StampFrontendClient+<HttpRequestAsync>d__121.MoveNext (Microsoft.ServiceProfiler.Agent.FrontendClient, Version=2.6.1909.2701, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a)
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw (mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089)
at Microsoft.ServiceProfiler.Agent.FrontendClient.StampFrontendClient+d__9`1.MoveNext (Microsoft.ServiceProfiler.Agent.FrontendClient, Version=2.6.1909.2701, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a)
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw (mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089)
at Microsoft.ServiceProfiler.Agent.FrontendClient.ProfilerFrontendClient+d__3.MoveNext (Microsoft.ServiceProfiler.Agent.FrontendClient, Version=2.6.1909.2701, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a)
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw (mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089)
at Microsoft.ServiceProfiler.Uploaders.StampEtlUploader+d__1.MoveNext (ApplicationInsightsProfiler, Version=2.6.1909.2701, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a)
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw (mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089)
at Microsoft.ServiceProfiler.Collectors.DetailedTraceCollector+d__41.MoveNext (ApplicationInsightsProfiler, Version=2.6.1909.2701, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a)
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw (mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089)
at Microsoft.ServiceProfiler.Collectors.DetailedTraceCollector+d__38.MoveNext (ApplicationInsightsProfiler, Version=2.6.1909.2701, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a)
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw (mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089)
at Microsoft.ServiceProfiler.Engine+<>c__DisplayClass1_1+<b__2>d.MoveNext (ApplicationInsightsProfiler, Version=2.6.1909.2701, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a)
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw (mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089)
at Microsoft.ServiceProfiler.Agent.Orchestration.Orchestrator+d__18.MoveNext (Microsoft.ServiceProfiler.Agent.Orchestration, Version=2.6.1909.2701, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a)
Inner exception System.Net.WebException handled at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw:
at System.Net.HttpWebRequest.EndGetResponse (System, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089)
at System.Net.Http.HttpClientHandler.GetResponseCallback (System.Net.Http, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a)
Inner exception System.Net.Sockets.SocketException handled at System.Net.HttpWebRequest.EndGetResponse:
at System.Net.Sockets.Socket.InternalEndConnect (System, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089)
at System.Net.Sockets.Socket.EndConnect (System, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089)
at System.Net.ServicePoint.ConnectSocketInternal (System, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089)

@Danielovich Thanks for filing the issue with detailed call stack. There's some info that I want to confirm with you so that we know it's the right direction for the investigation:

This is about the traditional App Insights Profiler, right? (Microsoft.ServiceProfiler.Agent.FrontendClient, Version=2.6.1909.2701).
Are you seeing this issue only when the application is deployed? Are applications all deployed to Azure App Service?
What's your application's Framework?
Do you have multiple applications deployed to the same app service plan? If yes, how many? Do you experience the same when you have only 1 app running on 1 plan?
All those info will be helpful for the troubleshooting. Thanks!

@xiaomi7732 Thanks for replying!

Not sure, we are simple utilizing Microsoft.ApplicationInsights and Microsoft.ApplicationInsights.AspNetCore, both version 2.14.0.
We are seeing this when the app is deployed yes and all services run under app services.
.NET core 3.1
We run 20 app services under 1 service plan (P3V2). I haven't tried to run a service under a different plan, not sure I can do that sorry.

@Danielovich Thanks for post back. Having 20 app services running on 1 service plan is going to cause trouble for the profiler to run.
Do I understand correct that you are trying to use Application Insights but no necessary to having profiler up & running? In that case, do you mind to try disable the profiler for all your apps like this (Note: go to the app service resource, not app insights resource to access the page) and set the value to Off:

@Danielovich, did you get a chance to try it?

Close this due to no actionable at this moment. Feel free to open new issues.


ZF3 - Using Identity view helper requires that 'Zend\Authentication\AuthenticationService' be registered

When using zend-authentication with ZF3, it seems that identity view helper requires that you register Zend\Authentication\AuthenticationService (literally) in your config file:
'service_manager' => [
        'factories' => [
            'Zend\Authentication\AuthenticationService' => Service\Factory\AuthenticationServiceFactory::class,
        ],
    ],

But, I expect that I should register this service with the ::class keyword, as follows:
'service_manager' => [
        'factories' => [
            Zend\Authentication\AuthenticationService::class => Service\Factory\AuthenticationServiceFactory::class,
        ],
    ],

However, this doesn't work. Is this a bug?

@olegkrivtsov It looks like you are declaring a namespace in your configuration file (based on the fact that you are declaring your factory using Service\Factory\AuthenticatinoServiceFactory::class. As such, try globally qualifying the AuthenticationService class:
'service_manager' => [
    'factories' => [
        // Note the "\" preceding the "Z"!
        \Zend\Authentication\AuthenticationService::class => Service\Factory\AuthenticationServiceFactory::class,
    ],
],

@weierophinney it works. Thanks. This can be closed.


[Feature] Add support for cannon Spring feature

Looks like its partially implemented.  There's an API under constraints, but its not a constraint.  There appears to be message support.
Looks like a little more is needed to support this feature.
I'm trying to finish converting cannon spring example

The following example shows how to use the new Spring feature
@Component({
  selector: 'spring-example',
  templateUrl: './spring-example.component.html',
  providers: [NgtPhysicBody, NgtPhysicSpring],
})
export class SpringExample {
  size = 1;
  boxsize = [this.size * 0.3, this.size, this.size] as NgtTriple;

  sphereProps = this.physicBody.useSphere(() => ({
    mass: 0,
    args: [0.1],
  }));

  boxProps = this.physicBody.useBox(() => ({
    mass: 5,
    args: this.boxsize,
    position: [0, -this.size, 0]
  }));

  spring = this.physicSpring.useSpring(this.sphereProps.ref, this.boxProps.ref,
    {
      localAnchorA: [0, -this.size, 0],
      //  localAnchorB: [0, 0, 0],
      //  restLength: 0,
      //  stiffness: 50,
      //  damping: 1,
    }
  );

  constructor(
    private physicBody: NgtPhysicBody,
    private physicSpring: NgtPhysicSpring,
  ) { }
}