rabbitmq_plugin[rabbitmq_management] erlexec: HOME must be set

I tried to setup rabbitmq using the sensu-chef cookbook on this machine:
ubuntu-trusty-14.04-amd64-server-20151218 (ami-581cbe2b)
In order to use the ubuntu version I have configured this in my wrapper cookbook:
# Rabbitmq settings
normal['rabbitmq']['use_distro_version'] = true

[2016-01-05T12:42:25+00:00] INFO: Processing group[rabbitmq] action create (sensu::rabbitmq line 22)
[2016-01-05T12:42:26+00:00] INFO: group[rabbitmq] created
[2016-01-05T12:42:26+00:00] INFO: Processing directory[/etc/rabbitmq/ssl] action create (sensu::rabbitmq line 32)
[2016-01-05T12:42:26+00:00] INFO: directory[/etc/rabbitmq/ssl] created directory /etc/rabbitmq/ssl
[2016-01-05T12:42:26+00:00] INFO: Processing file[/etc/rabbitmq/ssl/cacert.pem] action create (sensu::rabbitmq line 45)
[2016-01-05T12:42:26+00:00] INFO: file[/etc/rabbitmq/ssl/cacert.pem] created file /etc/rabbitmq/ssl/cacert.pem
[2016-01-05T12:42:26+00:00] INFO: file[/etc/rabbitmq/ssl/cacert.pem] updated file contents /etc/rabbitmq/ssl/cacert.pem
[2016-01-05T12:42:26+00:00] INFO: file[/etc/rabbitmq/ssl/cacert.pem] group changed to 1001
[2016-01-05T12:42:26+00:00] INFO: file[/etc/rabbitmq/ssl/cacert.pem] mode changed to 640
[2016-01-05T12:42:26+00:00] INFO: Processing file[/etc/rabbitmq/ssl/cert.pem] action create (sensu::rabbitmq line 45)
[2016-01-05T12:42:26+00:00] INFO: file[/etc/rabbitmq/ssl/cert.pem] created file /etc/rabbitmq/ssl/cert.pem
[2016-01-05T12:42:26+00:00] INFO: file[/etc/rabbitmq/ssl/cert.pem] updated file contents /etc/rabbitmq/ssl/cert.pem
[2016-01-05T12:42:26+00:00] INFO: file[/etc/rabbitmq/ssl/cert.pem] group changed to 1001
[2016-01-05T12:42:26+00:00] INFO: file[/etc/rabbitmq/ssl/cert.pem] mode changed to 640
[2016-01-05T12:42:26+00:00] INFO: Processing file[/etc/rabbitmq/ssl/key.pem] action create (sensu::rabbitmq line 45)
[2016-01-05T12:42:26+00:00] INFO: file[/etc/rabbitmq/ssl/key.pem] created file /etc/rabbitmq/ssl/key.pem
[2016-01-05T12:42:26+00:00] INFO: file[/etc/rabbitmq/ssl/key.pem] updated file contents /etc/rabbitmq/ssl/key.pem
[2016-01-05T12:42:26+00:00] INFO: file[/etc/rabbitmq/ssl/key.pem] group changed to 1001
[2016-01-05T12:42:26+00:00] INFO: file[/etc/rabbitmq/ssl/key.pem] mode changed to 640
[2016-01-05T12:42:26+00:00] INFO: Processing directory[/etc/rabbitmq/ssl/client] action create (sensu::rabbitmq line 54)
[2016-01-05T12:42:26+00:00] INFO: directory[/etc/rabbitmq/ssl/client] created directory /etc/rabbitmq/ssl/client
[2016-01-05T12:42:26+00:00] INFO: Processing file[/etc/rabbitmq/ssl/client/cert.pem] action create (sensu::rabbitmq line 61)
[2016-01-05T12:42:26+00:00] INFO: file[/etc/rabbitmq/ssl/client/cert.pem] created file /etc/rabbitmq/ssl/client/cert.pem
[2016-01-05T12:42:26+00:00] INFO: file[/etc/rabbitmq/ssl/client/cert.pem] updated file contents /etc/rabbitmq/ssl/client/cert.pem
[2016-01-05T12:42:26+00:00] INFO: file[/etc/rabbitmq/ssl/client/cert.pem] group changed to 1001
[2016-01-05T12:42:26+00:00] INFO: file[/etc/rabbitmq/ssl/client/cert.pem] mode changed to 640
[2016-01-05T12:42:26+00:00] INFO: Processing file[/etc/rabbitmq/ssl/client/key.pem] action create (sensu::rabbitmq line 61)
[2016-01-05T12:42:26+00:00] INFO: file[/etc/rabbitmq/ssl/client/key.pem] created file /etc/rabbitmq/ssl/client/key.pem
[2016-01-05T12:42:26+00:00] INFO: file[/etc/rabbitmq/ssl/client/key.pem] updated file contents /etc/rabbitmq/ssl/client/key.pem
[2016-01-05T12:42:26+00:00] INFO: file[/etc/rabbitmq/ssl/client/key.pem] group changed to 1001
[2016-01-05T12:42:26+00:00] INFO: file[/etc/rabbitmq/ssl/client/key.pem] mode changed to 640
[2016-01-05T12:42:26+00:00] INFO: Processing apt_package[erlang-nox] action install (erlang::package line 27)
[2016-01-05T12:42:36+00:00] INFO: apt_package[erlang-nox] installed erlang-nox at 1:16.b.3-dfsg-1ubuntu2.1
[2016-01-05T12:42:36+00:00] INFO: Processing apt_package[erlang-dev] action install (erlang::package line 28)
[2016-01-05T12:42:38+00:00] INFO: apt_package[erlang-dev] installed erlang-dev at 1:16.b.3-dfsg-1ubuntu2.1
[2016-01-05T12:42:38+00:00] INFO: Processing apt_package[logrotate] action install (rabbitmq::default line 33)
[2016-01-05T12:42:38+00:00] INFO: Processing apt_package[rabbitmq-server] action install (rabbitmq::default line 36)
[2016-01-05T12:42:44+00:00] INFO: apt_package[rabbitmq-server] installed rabbitmq-server at 3.2.4-1
[2016-01-05T12:42:44+00:00] INFO: Processing service[rabbitmq-server] action enable (rabbitmq::default line 84)
[2016-01-05T12:42:44+00:00] INFO: Processing service[rabbitmq-server] action start (rabbitmq::default line 84)
[2016-01-05T12:42:44+00:00] INFO: Processing directory[/var/lib/rabbitmq/mnesia] action create (rabbitmq::default line 149)
[2016-01-05T12:42:44+00:00] INFO: directory[/var/lib/rabbitmq/mnesia] mode changed to 775
[2016-01-05T12:42:44+00:00] INFO: Processing template[/etc/rabbitmq/rabbitmq-env.conf] action create (rabbitmq::default line 156)
[2016-01-05T12:42:44+00:00] INFO: template[/etc/rabbitmq/rabbitmq-env.conf] created file /etc/rabbitmq/rabbitmq-env.conf
[2016-01-05T12:42:44+00:00] INFO: template[/etc/rabbitmq/rabbitmq-env.conf] updated file contents /etc/rabbitmq/rabbitmq-env.conf
[2016-01-05T12:42:44+00:00] INFO: template[/etc/rabbitmq/rabbitmq-env.conf] owner changed to 0
[2016-01-05T12:42:44+00:00] INFO: template[/etc/rabbitmq/rabbitmq-env.conf] group changed to 0
[2016-01-05T12:42:44+00:00] INFO: template[/etc/rabbitmq/rabbitmq-env.conf] mode changed to 644
[2016-01-05T12:42:44+00:00] INFO: template[/etc/rabbitmq/rabbitmq-env.conf] sending restart action to service[rabbitmq-server] (immediate)
[2016-01-05T12:42:44+00:00] INFO: Processing service[rabbitmq-server] action restart (rabbitmq::default line 205)
[2016-01-05T12:42:50+00:00] INFO: service[rabbitmq-server] restarted
[2016-01-05T12:42:50+00:00] INFO: Processing template[/etc/rabbitmq/rabbitmq.config] action create (rabbitmq::default line 164)
[2016-01-05T12:42:50+00:00] INFO: template[/etc/rabbitmq/rabbitmq.config] created file /etc/rabbitmq/rabbitmq.config
[2016-01-05T12:42:50+00:00] INFO: template[/etc/rabbitmq/rabbitmq.config] updated file contents /etc/rabbitmq/rabbitmq.config
[2016-01-05T12:42:50+00:00] INFO: template[/etc/rabbitmq/rabbitmq.config] owner changed to 0
[2016-01-05T12:42:50+00:00] INFO: template[/etc/rabbitmq/rabbitmq.config] group changed to 0
[2016-01-05T12:42:50+00:00] INFO: template[/etc/rabbitmq/rabbitmq.config] mode changed to 644
[2016-01-05T12:42:50+00:00] INFO: template[/etc/rabbitmq/rabbitmq.config] sending restart action to service[rabbitmq-server] (immediate)
[2016-01-05T12:42:50+00:00] INFO: Processing service[rabbitmq-server] action restart (rabbitmq::default line 205)
[2016-01-05T12:42:56+00:00] INFO: service[rabbitmq-server] restarted
[2016-01-05T12:42:56+00:00] INFO: template[/etc/rabbitmq/rabbitmq.config] sending restart action to service[restart rabbitmq-server] (immediate)
[2016-01-05T12:42:56+00:00] INFO: Processing service[restart rabbitmq-server] action restart (sensu::rabbitmq line 79)
[2016-01-05T12:43:02+00:00] INFO: service[restart rabbitmq-server] restarted
[2016-01-05T12:43:02+00:00] INFO: Processing service[rabbitmq-server] action enable (rabbitmq::default line 205)
[2016-01-05T12:43:03+00:00] INFO: Processing service[rabbitmq-server] action start (rabbitmq::default line 205)
[2016-01-05T12:43:03+00:00] INFO: Processing rabbitmq_plugin[rabbitmq_management] action enable (rabbitmq::mgmt_console line 27)
[2016-01-05T12:43:04+00:00] INFO: Enabling RabbitMQ plugin 'rabbitmq_management'.
[2016-01-05T12:43:04+00:00] INFO: Processing execute[rabbitmq-plugins enable rabbitmq_management] action run (/var/chef/cache/cookbooks/rabbitmq/providers/plugin.rb line 35)
[2016-01-05T12:43:04+00:00] INFO: Running queued delayed notifications before re-raising exception
[2016-01-05T12:43:04+00:00] INFO: Running queued delayed notifications before re-raising exception
[2016-01-05T12:43:04+00:00] INFO: template[/etc/init.d/chef-client] sending restart action to service[chef-client] (delayed)
[2016-01-05T12:43:04+00:00] INFO: Processing service[chef-client] action restart (chef-client::init_service line 32)
[2016-01-05T12:43:04+00:00] FATAL: SIGTERM received, stopping
[2016-01-05T12:43:05+00:00] INFO: Daemonizing..
[2016-01-05T12:43:05+00:00] INFO: Forked, in 14564. Privileges: 0 0
[2016-01-05T12:43:06+00:00] INFO: service[chef-client] restarted
[2016-01-05T12:43:06+00:00] INFO: cookbook_file[/etc/ntp.leapseconds] sending restart action to service[ntp] (delayed)
[2016-01-05T12:43:06+00:00] INFO: Processing service[ntp] action restart (ntp::default line 112)
[2016-01-05T12:43:08+00:00] INFO: service[ntp] restarted
[2016-01-05T12:43:08+00:00] INFO: cookbook_file[/etc/apparmor.d/usr.sbin.ntpd] sending restart action to service[apparmor] (delayed)
[2016-01-05T12:43:08+00:00] INFO: Processing service[apparmor] action restart (ntp::apparmor line 20)
[2016-01-05T12:43:10+00:00] INFO: service[apparmor] restarted
[2016-01-05T12:43:10+00:00] ERROR: Running exception handlers
[2016-01-05T12:43:10+00:00] ERROR: Exception handlers complete
[2016-01-05T12:43:10+00:00] FATAL: Stacktrace dumped to /var/chef/cache/chef-stacktrace.out
[2016-01-05T12:43:10+00:00] FATAL: Please provide the contents of the stacktrace.out file if you file a bug report
[2016-01-05T12:43:10+00:00] ERROR: rabbitmq_plugin[rabbitmq_management] (rabbitmq::mgmt_console line 27) had an error: Mixlib::ShellOut::ShellCommandFailed: execute[rabbitmq-plugins enable rabbitmq_management] (/var/chef/cache/cookbooks/rabbitmq/providers/plugin.rb line 35) had an error: Mixlib::ShellOut::ShellCommandFailed: Expected process to exit with [0], but received '1'
---- Begin output of rabbitmq-plugins enable rabbitmq_management ----
STDOUT:
STDERR: erlexec: HOME must be set
---- End output of rabbitmq-plugins enable rabbitmq_management ----
Ran rabbitmq-plugins enable rabbitmq_management returned 1
[2016-01-05T12:43:10+00:00] FATAL: Chef::Exceptions::ChildConvergeError: Chef run process exited unsuccessfully (exit code 1)

Broadcast message from root@ip-172-31-21-124
	(unknown) at 12:43 ...

Funny enough this issue only occurs if chef-client is started from cfn/cloud init. If start chef-client manually in a ssh session everything works fine...


Closes #1697 add keep vars source arg@devel

…ed relevent unit tests and updated NEWS.md.
Thank you for your Pull Request! We have developed this task checklist from the Development Process Guide to help with the final steps of the process. Completing the below tasks helps to ensure our reviewers can maximize their time on your code as well as making sure the admiral codebase remains robust and consistent.
Please check off each taskbox as an acknowledgment that you completed the task or check off that it is not relevant to your Pull Request. This checklist is part of the Github Action workflows and the Pull Request will not be merged into the devel branch until you have checked off each task.

[x] Place Closes #<insert_issue_number> into the beginning of your Pull Request Title (Use Edit button in top-right if you need to update)
[x] Code is formatted according to the tidyverse style guide. Run styler::style_file() to style R and Rmd files
[x] Updated relevant unit tests or have written new unit tests, which should consider realistic data scenarios and edge cases, e.g. empty datasets, errors, boundary cases etc. - See Unit Test Guide
[ ] If you removed/replaced any function and/or function parameters, did you fully follow the deprecation guidance?
[x] Update to all relevant roxygen headers and examples, including keywords and families. Refer to the categorization of functions to tag appropriate keyword/family.
[x] Run devtools::document() so all .Rd files in the man folder and the NAMESPACE file in the project root are updated appropriately
[x] Address any updates needed for vignettes and/or templates
[x] Update NEWS.md if the changes pertain to a user-facing function (i.e. it has an @export tag) or documentation aimed at users (rather than developers)
[x] Build admiral site pkgdown::build_site() and check that all affected examples are displayed correctly and that all new functions occur on the "Reference" page.
[x] Address or fix all lintr warnings and errors - lintr::lint_package()
[x] Run R CMD check locally and address all errors and warnings - devtools::check()
[x] Link the issue in the Development Section on the right hand side.
[ ] Address all merge conflicts and resolve appropriately
[ ] Pat yourself on the back for a job well done! Much love to your accomplishment!

@pharmaverse/admiral , should we move the issue to next release or should we update only derive_extreme_records() in this release?

Given the release is on 5th june, it might be more sensible to push to the next release?

@gg106046 This one has opened some discussion that we need to resolve post-release.  Hopefully next Wednesday we will have a resolution and can get this PR in for 0.12.0

@gg106046  we want to use keep_source_vars rather than keep_vars_source.
Let's close this PR and branch and restart as it will be easier dealing with merge conflicts and recent admiral updates.


Core module pr patch

Description
Removed workspaceEnabled$ from workspace service module, a followup PR will create to implement the flag within capabilities service
Issues Resolved

Screenshot

Testing the changes

Check List

[ ] All tests pass

[ ] yarn test:jest
[ ] yarn test:jest_integration
[ ] yarn test:ftr

[ ] New functionality includes testing.
[ ] New functionality has been documented.
[ ] Update CHANGELOG.md
[ ] Commits are signed per the DCO using --signoff

Codecov Report

Merging #215 (d0413bd) into workspace-pr-integr (c05b434) will decrease coverage by 0.01%.
The diff coverage is n/a.

:exclamation: Current head d0413bd differs from pull request most recent head 3d5a042. Consider uploading reports for the commit 3d5a042 to get more accurate results

@@                   Coverage Diff                   @@
##           workspace-pr-integr     #215      +/-   ##
=======================================================
- Coverage                66.46%   66.46%   -0.01%
=======================================================
  Files                     3408     3408
  Lines                    65222    65219       -3
  Branches                 10457    10457
=======================================================
- Hits                     43353    43349       -4
  Misses                   19265    19265
- Partials                  2604     2605       +1

Flag
Coverage Δ

Linux_
?

_1
34.82% <ø> (+<0.01%)
:arrow_up:

_2
55.45% <ø> (-0.01%)
:arrow_down:

_3
44.49% <ø> (+<0.01%)
:arrow_up:

_4
34.81% <ø> (-0.01%)
:arrow_down:

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

Files
Coverage Δ

...lication/capabilities/capabilities_service.mock.ts
100.00% <ø> (ø)

...c/core/public/workspace/workspaces_service.mock.ts
100.00% <ø> (ø)

src/core/public/workspace/workspaces_service.ts
88.88% <ø> (-1.12%)
:arrow_down:

...e/server/capabilities/capabilities_service.mock.ts
75.00% <ø> (ø)

...c/core/server/capabilities/capabilities_service.ts
100.00% <ø> (ø)

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

@ruanyl Is there any examples about how to use this flag in the capabilities service?


Send/verify signature of ssl certificate for snode 2 snode

Seems good to me after conflicts resolved
Not sure how to test


Remove unused keycloak console integration

Remove all code related to setting up Keycloak identity provides, console oauth settings and admin users.
Update CSV with permissions needed for console service
Update CHANGELOG.md with console change
Remove some entries from npm-shrinkwrap

@enmasse-ci run tests

@enmasse-ci run tests profile=upgrade

@k-wall How about removing docs and systemtests but keep type in api for backwards compatibility?

@enmasse-ci run tests profile=smoke

@enmasse-ci run tests

@enmasse-ci run tests testcase=UserApiTest

@lulf so test for federated user in user api test should be removed as well

@kornys it is done


Suunto Ambit Support

I'm trying to process a file that looks to be from a Suunto Ambit that I can view by uploading to https://www.fitfileviewer.com/ If I try to process using this library I get Undefined offset error @ $this->types = $this->endianness[$architecture];
$architecture is showing as 14. Obviously, the code expects zero or one. Any idea? Fit file attached.
68270818303.zip

Hi, can you try commenting-out line 1343
https://github.com/adriangibbons/php-fit-file-analysis/blob/master/src/phpFITFileAnalysis.php#L1343
Appears to be a swimming fil with a Garmin 920XT. Were there any sensors attached e.g. HR strap?

Hmm yep looks like's a Garmin. Commenting out 1343 worked in allowing me to parse the file (and other similar ones I've had issue with). It's not my file so I'll try and find out whether the user had any sensors attached.

No worries. As you can read from the comment on the line before it was a bit of a fudge to ensure all data is processed. Otherwise "buffered messages" were not being parsed. I haven't kept up with changes to the FIT protocol standard for some time. Out of interest, what are you using the PHP class for? Is there a URL of a site I could visit? To be honest, I thought this project had gone stale and is quite old / on the verge of being made read-only as I don't really maintain it any longer.

I'm using it to process FIT files from the Garmin API. The project I'm involved with does our own processing and analysis on the files (some of which Garmin doesn't provide hence why we're processing ourselves). Public/marketing site is https://athletica.ai/ private site in beta at the moment.

Nice! Would you mind dropping me a line at<EMAIL_ADDRESS>? I'm interested in knowing about the Garmin API. I plug into the Strava API for my main project. After access to the Strava API, I didn't really need to process FIT files myself, hence the drop-off in maintaining this project.


TIGRESS addback changed to include Mult1 events and clover addback, smal...

...l changes to TTip
Added multiplicity one events to addback. Added clover addback (more suitable for running in suppressed mode). Made TTip timing sensible (Zc, not timestamp) and added PID function (PID itself not included yet).

Right now, tigress is hardcoded to be in the close configuration, until we
change this (see Griffin) the positions for the suppressed mode will not be
correct.    Once the positions are correct, addback should work as normal
... as proposed we are adding diagonal crystals, which I am not sure we
want to do.
On Mon, May 4, 2015 at 2:52 PM, jhenderson88<EMAIL_ADDRESS>wrote:

jhenderson88 wants to merge 1 commit into GRIFFINCollaboration:master from
jhenderson88:master:
...l changes to TTip
Added multiplicity one events to addback. Added clover addback (more
suitable for running in suppressed mode). Made TTip timing sensible (Zc,
not timestamp) and added PID function (PID itself not included yet).
You can view, comment on, or merge this pull request online at:
https://github.com/GRIFFINCollaboration/GRSISort/pull/304
Commit Summary

TIGRESS addback changed to include Mult1 events and clover addback,
small changes to TTip

File Changes

M include/TTigress.h
https://github.com/GRIFFINCollaboration/GRSISort/pull/304/files#diff-0
(7)
M include/TTipData.h
https://github.com/GRIFFINCollaboration/GRSISort/pull/304/files#diff-1
(25)
M libraries/TGRSIAnalysis/TTigress/TTigress.cxx
https://github.com/GRIFFINCollaboration/GRSISort/pull/304/files#diff-2
(109)
M libraries/TGRSIAnalysis/TTip/TTip.cxx
https://github.com/GRIFFINCollaboration/GRSISort/pull/304/files#diff-3
(2)
M libraries/TGRSIAnalysis/TTip/TTipData.cxx
https://github.com/GRIFFINCollaboration/GRSISort/pull/304/files#diff-4
(4)

Patch Links:

https://github.com/GRIFFINCollaboration/GRSISort/pull/304.patch
https://github.com/GRIFFINCollaboration/GRSISort/pull/304.diff

—
Reply to this email directly or view it on GitHub
https://github.com/GRIFFINCollaboration/GRSISort/pull/304.

Sure, we should remove the diagonals. As it is we needed a simple clover addback for the current configuration - diagonals might introduce some noise but considering using clover addback significantly improves the signal/background over the regular addback mode.

I agree, however with the segmentation information of tigress, we can do a
better job at adding back the clover as opposed to just adding the crystal
together...  There is is some history to this fight,  we put in
AddBackClover into griffin mostly to prove to Carl S. that our addback
scheme worked well, which it does.
Now it has become some issue to convince some people which addback to use,
where the short answer is "we should use the one that takes there
positions into account".  Adding this option to tigress, I think will only
introduce this same confusion into another array.  On the other hand, if
we set the array positions correctly depending on the configuration of
tigress; we can accomplish the same effect of not adding neighboring
crystals with out the confusion we introduced in griffin.
The only reason I hadn't done this already is because, when running a
radioactive beam the advantage to running eff over suppressed is huge -
however Kris S. does things a bit differently.
All that being said, if you still want to go through with this, we should
fix the loop error Jenna pointed out and re-push the update.
On Mon, May 4, 2015 at 3:09 PM, jhenderson88<EMAIL_ADDRESS>wrote:

Sure, we should remove the diagonals. As it is we needed a simple clover
addback for the current configuration - diagonals might introduce some
noise but considering using clover addback significantly improves the
signal/background over the regular addback mode.
—
Reply to this email directly or view it on GitHub
https://github.com/GRIFFINCollaboration/GRSISort/pull/304#issuecomment-98819282
.

Alright. I'll leave it in for now (making the change Jenna mentioned) because it gives a very significant improvement in signal/background over the in-built addback mode. Once the positions can be properly determined in the suppressed mode we can remove it.

yeah, I understand why it is good.  It is hard to remove things once they
have been added but we will tackle that one when we have too.
On Mon, May 4, 2015 at 3:34 PM, jhenderson88<EMAIL_ADDRESS>wrote:

Alright. I'll leave it in for now (making the change Jenna mentioned)
because it gives a very significant improvement in signal/background over
the in-built addback mode. Once the positions can be properly determined in
the suppressed mode we can remove it.
—
Reply to this email directly or view it on GitHub
https://github.com/GRIFFINCollaboration/GRSISort/pull/304#issuecomment-98826500
.


Consolidation of the fallback storage.

This PR combines the three separate types of the fallback storage implementation into a single, simpler one.

This looks great, thank you!


First code testing App Engine with Node.js

This is inicial code for testing App Engine with Node.js for the project

@andreban & Michael PTAL

Just a small nit on the format of the copyright comment, which we can change later.

LGTM, let's merge and get hacking!


fix(chat): remove empty chat regenerate button if was an error in previous chat (Issue #1044)

Description:
Remove empty chat regenerate button if was an error in previous chat
Issues:

https://github.com/epam/ai-dial-chat/issues/1044
https://github.com/epam/ai-dial-chat/issues/949

PRs:

https://github.com/epam/ai-dial-chat/pull/1057

Checklist:

[x] the pull request name complies with Conventional Commits
[x] the pull request name starts with fix(<scope>):, feat(<scope>):, feature(<scope>):, chore(<scope>):, hotfix(<scope>): or e2e(<scope>):. If contains breaking changes then the pull request name must start with fix(<scope>)!:, feat(<scope>)!:, feature(<scope>)!:, chore(<scope>)!:, hotfix(<scope>)!: or e2e(<scope>)!: where <scope> is name of affected project: chat, chat-e2e, overlay, shared, sandbox-overlay, etc.
[x] the pull request name ends with (Issue #<TICKET_ID>) (comma-separated list of issues)
[x] I confirm that do not share any confidential information like API keys or any other secrets and private URLs

/deploy-review

/deploy-review


[FIX] m.SelectList: browser navigation with alt + arrow keys

Browsers allow page navigation with the following keyboard shortcuts: https://en.wikipedia.org/wiki/Table_of_keyboard_shortcuts#Browsers_/_Go_menu
SelectList was preventing users from doing so as long as one of its items had the focus.
Now users can navigate with keyboard shortcuts.

Fixes: https://github.com/SAP/openui5/issues/1414
Specifically: https://github.com/SAP/openui5/issues/1414#issuecomment-447364113

Hello @boghyon ,
Your pull request was accepted and merged.
Fix will be available in next version (1.63).
Regards,
Nayden

Hi @nnaydenow
Just curious; why couldn't my commits be merged directly? Sometimes they get merged directly, sometimes not. What's the decision criteria? How is it internally handled?

For the sake of discoverability:

https://github.com/SAP/openui5/commit/f1c8b1e161a4f710d88de82dcadd6d412cb5dfcd
Merge: https://github.com/SAP/openui5/commit/b79790997ac4d56a4c8bd18a35966f4d7897b044

Hello @boghyon,
The reason that your change is not merged directly is that I permitted myself to format your commit messege. I apologise about the confusion.
Regards,
Nayden

I see. Thanks for the clarification


Add caching support with pluggable storage classes

Our application relies heavily on the caching support of httplib2. We cannot consider switching to requests until requests has basic caching support with pluggable storage backends (we use a BSDDB backend for a multi-process shared cache).
Glancing over the code, I did not see any cache support. Let's sprint on this some day. I maintain an http cache handler for urllib2 and I've also worked on the httplib2 code base (though I recognize that might be as much a detriment as a compliment ;).

What's the status of caching support? Has anyone considered it yet? Would pull requests be considered? Are there any other considerations that should be made before going down the path of an implementation?

Excellent! It's absolutely something that I'd like to support eventually, as long as the API is right, and it doesn't make requests difficult to maintain.
If it is, it may serve best as a "sister module". We'll have to see. I'd like to see it in core if it fits, though.

I'm interested in middleware that does storing, caching, purging and additionally protects against API failures and downtime. This is the reality of most API's, so why not bake that in since we can use values from the cache while the API is down.
The logic for such a module:

Cache on path and parameter dictionary variations
On default only cache responses for GET requests
If API response is erroneous or non responsive, use cached value, reset TTL so the service get's a little time to breath

For an example of a CachedResponse class check out this tumblr.py module I'm using in production: https://gist.github.com/1455583

@jokull Something much like this is done in the HTTP cache Varnish by setting a grace period where Varnish will return stale values if the back-end is not responding.
The only note I would like to add is that cache headers should be respected and not to just blindly cache all GET requests.
Also other nice properties for a cache would be:

Ordering of GET string variables before caching so things like http://example.com?a=1&b=2 is the same as http://example.com?b=2&a=1
Excluding GET string variables that don't affect content like Google Analytics variables

I love to have cache to file support :)

I love to have cache to file support :)


I/Q Signals

Hi there
Do you know whether it is possible to get the I/Q Signals?
Kind regards!

I don't know.
The radio chip has a code upload function, which I have only ever seen used for their spectrum analyzer gimmick, but I am sure in theory could be used to interface more directly with the radio. I have yet to find any documentation for that though.

AFAIK there is a 8051 in the radio (i believe from BlackFin?).
There are also other firmware patches available. It should be possible to reverse them as others already done this...


get rid of use of refs and use onKeyDown

topic picker was failing in some apps with error
discussed here: https://gist.github.com/jimfb/4faa6cbfb1ef476bd105
after numerous dead ends, turned out that the refs inside
TopicPicker were causing this issue. Easily addressed by getting
rid of refs.

lgtm :+1:


Saving and Loading a Dataset returns a different elements

Click to expand!
Issue Type
Bug
Source
binary
Tensorflow Version
2.9.1
Custom Code
No
OS Platform and Distribution
MacOS Monterey
Mobile device
No response
Python version
3.7
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
I am trying to save and subsequently load a dataset. I am observing that after loading the dataset the elements of it are many times different from the saved one.
In order to run the code I have two text files in the following structure with contents:
test
 |-neg->file.txt: Once again Mr. Costner has dragged out a movie
 |-pos->file.txt: I went and saw this movie last

I am expecting that the two loops should print an identical Tensor.
Standalone code to reproduce the issue
import tensorflow as tf
from tensorflow import keras
from tensorflow.keras import layers

import pickle

def create_int_ds(test_ds):
    max_sequence_length = 10
    max_tokens = 20000
    text_vectorization = layers.TextVectorization(
        max_tokens=max_tokens,
        output_mode="int",
        output_sequence_length=max_sequence_length,
    )

    text_only_train_ds = test_ds.map(lambda x, y: x)
    text_vectorization.adapt(text_only_train_ds)
    int_test_ds = test_ds.map(
        lambda x, y: (text_vectorization(x), y), num_parallel_calls=4)
    return int_test_ds

batch_size = 1
test_ds = keras.utils.text_dataset_from_directory("test", batch_size=batch_size)
int_test_ds = create_int_ds(test_ds)

take_test_ds = int_test_ds.take(1)
print(f"type of dataset: {type(take_test_ds)}")
print(f"spec: {take_test_ds.element_spec}")

for e in take_test_ds:
    print(e)

# Save
tf.data.experimental.save(take_test_ds, "int_test_ds")
with open('element_spec', 'wb') as out_:  # also save the element_spec to disk for future loading
    pickle.dump(take_test_ds.element_spec, out_)

# Load
with open('element_spec', 'rb') as in_:
    es = pickle.load(in_)
new_take_test_ds = tf.data.experimental.load("int_test_ds", element_spec=es)

print(type(new_take_test_ds))
for e in new_take_test_ds:
    print(e) # Shoud print the same as the previous for loop

Relevant log output
type of dataset: <class 'tensorflow.python.data.ops.dataset_ops.TakeDataset'>
spec: (TensorSpec(shape=(None, 10), dtype=tf.int64, name=None), TensorSpec(shape=(None,), dtype=tf.int32, name=None))
(<tf.Tensor: shape=(1, 10), dtype=int64, numpy=array([[105, 183, 111,  16, 138, 159,  41,  10,   8, 147]])>, <tf.Tensor: shape=(1,), dtype=int32, numpy=array([0], dtype=int32)>)
<class 'tensorflow.python.data.experimental.ops.io._LoadDataset'>
(<tf.Tensor: shape=(1, 10), dtype=int64, numpy=array([[  4,  64,   7,  39,  11,   8, 121, 108, 184, 175]])>, <tf.Tensor: shape=(1,), dtype=int32, numpy=array([1], dtype=int32)>)

There is no bug. text_dataset_from_directory shuffles the samples by default and that is why I would see different samples.


Dynamically initialize distributor_dict.

This pull request changes distributor_dict initialization from static to dynamic.
Dynamic initialization allows to recreate the distributor dict with its default values at runtime. This is necessary after some values were popped or local distributor instances were added.
Some global variables were refactored into distributors.global_vars to break cyclic imports.
Dynamic initialization is handled by the new static member function dist_init_distributor_dict in the individual distributor modules.
Moved kicost_gui_notdependences function to top of __main__.py to solve a "function not found" error if wxPython is not installed.
Fixes #186.

This fix #186


Feat/#502 consul integrate test

Enhance the registry test by integrate with https://www.baeldung.com/docker-test-containers, so a real consul can be established automatically when testing.
Since this integrate test depends on docker so it should be disabled all the time unless it is used.
Related issue: https://github.com/networknt/light-4j/issues/502

Codecov Report

Merging #503 into master will increase coverage by 0.12%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #503      +/-   ##
============================================
+ Coverage     49.66%   49.79%   +0.12%
+ Complexity     1822     1816       -6
============================================
  Files           260      260
  Lines         11162    11098      -64
  Branches       1574     1559      -15
============================================
- Hits           5544     5526      -18
+ Misses         4970     4928      -42
+ Partials        648      644       -4

Impacted Files
Coverage Δ
Complexity Δ

...tworknt/client/oauth/ClientCredentialsRequest.java
73.68% <0%> (-1.32%)
2% <0%> (ø)

...ava/com/networknt/client/oauth/SignKeyRequest.java
65.38% <0%> (-1.29%)
2% <0%> (ø)

...tworknt/client/oauth/AuthorizationCodeRequest.java
80.76% <0%> (-0.72%)
6% <0%> (ø)

...main/java/com/networknt/consul/ConsulRegistry.java
51.92% <0%> (-0.28%)
22% <0%> (ø)

...va/com/networknt/client/oauth/TokenKeyRequest.java
48.57% <0%> (-0.08%)
2% <0%> (ø)

...om/networknt/client/oauth/RefreshTokenRequest.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...ain/java/com/networknt/consul/ConsulConstants.java
81.81% <0%> (ø)
1% <0%> (ø)
:arrow_down:

...n/java/com/networknt/client/oauth/SignRequest.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...fig/src/main/java/com/networknt/config/Config.java
67.41% <0%> (+0.56%)
3% <0%> (ø)
:arrow_down:

.../java/com/networknt/client/oauth/DerefRequest.java
85.71% <0%> (+0.71%)
11% <0%> (-1%)
:arrow_down:

... and 3 more

Continue to review full report at Codecov.

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

@jiachen1120 This is a very good idea to test with a real container; however, we cannot expect all developers to use docker on their development environment. We have some repositories that depending on the external containers for testing and normally we create an extra folder called integration along with test for integration test. All unit tests will be in the test folder and they don't rely on the container and all integration test cases are in the integration folder and they rely on the container. The following is an example in light-session-4j and the integration test won't be invoked when mvn clean package is called.
https://github.com/networknt/light-session-4j/tree/master/redis-manager/src

@jiachen1120 : while I agree that it is a neat idea, please be aware that developers at my client are building/testing in several APIs on Windows machines, on prem, in order to have access to certain resources.
In those cases, they don't use Docker, as they don't have it on their machines.

@stevehu Cool, I will migrate this test to an integrate folder

@ddobrin Hey, after migrating this test to integration test folder. They would not be invoked during mvn build. They can be used just by developer to testing the framework.

I looked at the test container project and it is very promising. It is much more convenient than our docker-compose. Good job!

@jiachen1120 i'm getting failures when running the build locally.. is there something i'm missing?
[ERROR] Failures:
[ERROR]   ConsulRegistryIT.testDiscoverCaching:194 expected:<...//<IP_ADDRESS>:495[90]> but was:<...//<IP_ADDRESS>:495[89]>
[ERROR]   ConsulRegistryIT.testDiscoverWithNonEmptyStringEnvTag:169 expected:<...//<IP_ADDRESS>:495[90]> but was:<...//<IP_ADDRESS>:495[89]>

@jiachen1120 i'm getting failures when running the build locally.. is there something i'm missing?
[ERROR] Failures:
[ERROR]   ConsulRegistryIT.testDiscoverCaching:194 expected:<...//<IP_ADDRESS>:495[90]> but was:<...//<IP_ADDRESS>:495[89]>
[ERROR]   ConsulRegistryIT.testDiscoverWithNonEmptyStringEnvTag:169 expected:<...//<IP_ADDRESS>:495[90]> but was:<...//<IP_ADDRESS>:495[89]>

This passes for me now with command mvn verify -P integration-test


Genie.Cookies.set! has 2 kinds of return type

Genie.Cookies.set! has 2 kinds of return type ; Vector{HTTP.Cookies.Cookie},Dict{String,HttpCommon.Cookie}.
A set! function calls other set! function.
But their return type can't convert each other.
So it needs fix.
Which return type is that you want type ?
https://github.com/essenciary/Genie.jl/blob/d088f4676d3cc5db17c26d31f69796384f389b94/src/Cookies.jl#L50
https://github.com/essenciary/Genie.jl/blob/d088f4676d3cc5db17c26d31f69796384f389b94/src/Cookies.jl#L59

Hmmm... seems something was lost in translation from HttpServer to HTTP.
Looks like we could do with only one function:
function set!(res::HTTP.Response, key::Union{String,Symbol}, value::Any, attributes::Dict = Dict()) :: Vector{HTTP.Cookies.Cookie}
  normalized_attrs = Dict{Symbol,Any}()
  for (k,v) in attributes
    normalized_attrs[Symbol(lowercase(string(k)))] = v
  end
  HTTP.Cookie(string(key), string(value) |> Genie.Encryption.encrypt; normalized_attrs...)

  HTTP.Cookies.readsetcookies("", "")
end

Yes, set! can't set cookies. So now, I write
push!(response(@params).headers,
              "Set-Cookie" => dict2cookies([("account", account)]) * "\r\nSet-Cookie: " * dict2cookies([("uniquekey", uniquekey)]))

when get , I don't use my original get
function noencryptget(req, key::Union{String,Symbol}) :: Nullable{String}
  if haskey(Dict(req.headers), "Cookie")
    cookies = Cookies.to_dict(req)
    if haskey(cookies, string(key))
      return Nullable{String}(cookies[string(key)])
    end
  end
  Nullable{String}()
end

Maybe this problem isn't Genie's, cannot some Set-Cookie s.
I use a mad method.


FileNotFoundError at /srnade/result – [Errno 2] No such file or directory: 'de/ttest'

Hi,
I am currently trying to find a suitable platform for the analysis of a miRNA-project, and was looking forward to seeing if sRNAtoolbox would be the solution.
However, upon a test-run with 5 fa.gz-files, I cannot find a workaround to avoid this error message.
I have 5 files with 2 groups, which I have defined/ annotated after the expression profiling. Now I am trying to run the DESeq. A brief look at the results I obtained after step 1 does not reveal major problems. I have run the DESeq in R myself before, so I now the data "works". If needed, I can send the raw-files, though it does not quite look like a problem from my side?!
I have the complete error-log file attached.
FileNotFoundError at :srnade:result.pdf

it is possible that we had a temporary hiccup at this particular time, did you try rerunning your job? It should work now.
Sorry for the late reply.


add contributors list in README.md

fixes: #259
Preview

mentors:
@Mansi168 @MonalikaPatnaik

@shivam-sharma7 Good work, Thanks for Contributing :)


Support for working with raw audio

Currently, Lhotse is somewhat based on the assumption that a user will mostly work with features such as log-mel energies. It'll be useful to have PyTorch Datasets providing raw audio as well. I'm thinking we should make it a convention to put Waveform in the class name of datasets that provide raw audio (e.g. UnsupervisedWaveformDataset). I'll try to list the steps that are needed to get there.

[ ] Create Cuts directly from RecordingSet (with an empty Features field - currently, you have to extract some features to make cuts, just to ignore them later)
[x] load_audio() in Cut
[ ] load_audio() in MixedCut
[x] load_audio() in PaddingCut
[ ] Implement UnsupervisedWaveformDataset
[ ] Implement other waveform datasets

If nobody is working on this issue, I'll implement this.

Great as far as I'm concerned!
On Thu, Jul 9, 2020 at 3:34 PM Junbo Zhang<EMAIL_ADDRESS>wrote:

If nobody is working on this issue, I'll implement this.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pzelasko/lhotse/issues/44#issuecomment-655957259, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZFLO5NJ7B6RZUGOUNUSQTR2VXG3ANCNFSM4OO6RBDA
.

Awesome!
One suggestion for a starting point: it's probably the simplest if we add a recording: Optional[Recording] = None field to the Cut class, and make features optional as well. If a user wants a cut with audio loading capability, let them make sure they created it this way. This will avoid the manifests growing even bigger for users who do not need this capability.

One suggestion for a starting point: it's probably the simplest if we add a recording: Optional[Recording] = None field to the Cut class, and make features optional as well. If a user wants a cut with audio loading capability, let them make sure they created it this way. This will avoid the manifests growing even bigger for users who do not need this capability.

I wonder is it appropriate to take the Recording as a special feature:
GeneralFeatures = Union['Features', 'Recording'], and then define non-optional features: GeneralFeatures in the class Cut, instead of features: Features = None; recording: Recording = None
The motivation is that a Cut is defined as an example for training, which should contains one type of features and the corresponding label. The features here should be allowed in any form, not limited in spectrum. So it is a little redundancy to contain both raw audio and cepstral features in Cut.

There are situations where you might want to access both the features and the raw recording, for different reasons.  I think it makes sense to keep the concepts separate, especially since they have different structure.
However something that does concern me, which I don't know the right way to handle, is that there might be situations where the cut doesn't have a specific channel associated with it but all channels are relevant (e.g. array-mic).

I agree with Dan, I'd rather keep Recording and Features fields separately.
Regarding array cuts: if I understood you correctly, we can do it in at least two ways right now:

extract cuts separately for each channel and create a "MultiChannelASRDataset" (or however we name it) that sorts the CutSet by (recording-id, start-time, end-time, channel) tuple and retrieves all cuts for a given time span;
create a mixed cut out of single-channel cuts and in the Dataset iterate the mixed_cut.tracks (which is somewhat hacky)

We could extend the support though, e.g.:

add load_tracks_features() in MixedCut that returns a 3d array (n_channels, n_frames, n_features)... (and load_tracks_audio() probably)
add utility that directly creates MixedCuts from RecordingSet with multiple AudioSources

Hi, I just found a potential issue in my recent experiments with raw waveform. It might not really about this PR but quite relevant so I just post it here. Please correct me if I misunderstand something.
As I notice, we are using torchaudio to extract features from raw waveform (according to here:https://github.com/pzelasko/lhotse/blob/master/lhotse/features.py#L122).
But here are two potential issues with torchaudio, especially with torchaudio.compliance.kaldi:

From my experiment, I found that these founctions can only work with CPU tensors (waveform). And in normal situations, we are most likely to put our data and model on GPUs and in that case this function will raise an error. (Haven't found any official document for this point and it would be great if someone can help me double check it).
Waveform should be in shape (c, t) where c can only be in [0, 2) (https://pytorch.org/audio/compliance.kaldi.html#mfcc). In other words, it doesn't support batch level feature extraction and can only be used before you form a batch (e.g. in __getitem__function). It would probably be OK in normal cases, but will make trouble if you'd like to do feature extraction outside dataset but in model forward instead.
As an alternative, torchaudio.transforms(torch.nn.Module) seems more flexible and we can even customize it as a normal nn.Module to fit with our user cases.
Please let me know how you think.
Thanks!

Hi, I just found a potential issue in my recent experiments with raw

waveform. It might not really about this PR but quite relevant so I just
post it here. Please correct me if I misunderstand something.
As I notice, we are using torchaudio to extract features from raw waveform
(according to here:
https://github.com/pzelasko/lhotse/blob/master/lhotse/features.py#L122).
But here are two potential issues with torchaudio, especially with
torchaudio.compliance.kaldi:

From my experiment, I found that these founctions can only work with
CPU tensors (waveform). And in normal situations, we are most likely to put
our data and model on GPUs and in that case this function will raise an
error. (Haven't found any official document for this point and it would be
great if someone can help me double check it).
Waveform should be in shape (c, t) where c can only be in [0, 2) (
https://pytorch.org/audio/compliance.kaldi.html#mfcc). In other words,
it doesn't support batch level feature extraction and can only be used
before you form a batch (e.g. in __getitem__function). It would
probably be OK in normal cases, but will make trouble if you'd like to do
feature extraction outside dataset but in model forward instead.
As an alternative, torchaudio.transforms(torch.nn.Module) seems more
flexible and we can even customize it as a normal nn.Module to fit
with our user cases.
Please let me know how you think.
Thanks!

Mm.
I wonder if it might make more sense to have the feature extraction as
python+PyTorch (or numpy?) code in lhotse itself?
The kaldi10feat project (on my github) may be something to refer to,
although it supports very little configuration.  It can't be made
compatible with current Kaldi features though, if that's a design criterion.
Dan

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/pzelasko/lhotse/issues/44#issuecomment-667619856, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZFLOYTVHXFUKG23Q3I7M3R6TLQNANCNFSM4OO6RBDA
.

... conventionally, people do compute the features on CPU, and for the most
common use case we'll probably extract features and write them (compressed)
to disk.  But of course it would be nice to be able to do it on GPU, in
general.
On Sun, Aug 2, 2020 at 1:13 PM Daniel Povey<EMAIL_ADDRESS>wrote:

Hi, I just found a potential issue in my recent experiments with raw

waveform. It might not really about this PR but quite relevant so I just
post it here. Please correct me if I misunderstand something.
As I notice, we are using torchaudio to extract features from raw
waveform (according to here:
https://github.com/pzelasko/lhotse/blob/master/lhotse/features.py#L122).
But here are two potential issues with torchaudio, especially with
torchaudio.compliance.kaldi:

From my experiment, I found that these founctions can only work
with CPU tensors (waveform). And in normal situations, we are most likely
to put our data and model on GPUs and in that case this function will raise
an error. (Haven't found any official document for this point and it would
be great if someone can help me double check it).
Waveform should be in shape (c, t) where c can only be in [0, 2) (
https://pytorch.org/audio/compliance.kaldi.html#mfcc). In other
words, it doesn't support batch level feature extraction and can only be
used before you form a batch (e.g. in __getitem__function). It would
probably be OK in normal cases, but will make trouble if you'd like to do
feature extraction outside dataset but in model forward instead.
As an alternative, torchaudio.transforms(torch.nn.Module) seems more
flexible and we can even customize it as a normal nn.Module to fit
with our user cases.
Please let me know how you think.
Thanks!

Mm.
I wonder if it might make more sense to have the feature extraction as
python+PyTorch (or numpy?) code in lhotse itself?
The kaldi10feat project (on my github) may be something to refer to,
although it supports very little configuration.  It can't be made
compatible with current Kaldi features though, if that's a design
criterion.
Dan

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/pzelasko/lhotse/issues/44#issuecomment-667619856,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZFLOYTVHXFUKG23Q3I7M3R6TLQNANCNFSM4OO6RBDA
.

Thanks Yiwen. I'll open another issue for this.


Remove HierarchyNotRespected validation error

Describe the improvement you'd like to request
According to this comment we return the HierarchyNotRespected error in some cases where we could soundly type an expression as False. This error was introduced (IIRC) to avoid having to update the Dafny type soundness proof.
Now that we've switched to using Lean, and tweaked the validation property being tested, I wonder if we could remove this error variant.
Describe alternatives you've considered
No response
Additional context
No response
Is this something that you'd be interested in working on?

[ ] 👋 I may be able to implement this internal improvement
[ ] ⚠ This feature might incur a breaking change

May depend on #539, but with that we definitely can. Haven't thought about what we can do given your tweak to the property being tested.


add privy pay with card

Current dependencies on/for this PR:

main

PR #4192   👈

This stack of pull requests is managed by Graphite.

Closing, PoC


Better arg condition and add config option for default port

@jas-chen There's three ways to set the port
Through keyboard shortcut/command - This overrides everything
Through .atom-live-server.json - This overrides config setting and is specific to the project
Through config setting [new] - only used when default command is used (startServer) and no .atom-live-server.json is found.

Will this be merged to master soon?

@Andrehatlo I still haven't gotten a response from @jas_chen, but this has been merged in my fork, richiksc/atom-live-server-plus. It's also installable in Atom.

Perfect! 👍


Dark theme

@umputun, I've created dark theme as we discussed before.
I've created this PR just as a notification, because I think that some folks want to use Remark in a “dark mode”.
Now it's possible to pass theme key to remark_config object. By default it is light, but also dark is available. If you have to, you can change theme in runtime using global function window.REMARK42.changeTheme. Read more about this in README.
/cc @Reeywhaar

@umputun, I think it's fine to merge this PR, and then test dark theme on news.radio-t.com and pirates.radio-t.com to collect feedback.
If you agree with me, merge it, please.

Coverage decreased (-0.04%) to 85.04% when pulling 53fef916ee6dc8b262ba5487ef34fc44171ceff1 on dark-theme into 126edb39f0621241aa7e3c4d1b95afc87381798a on master.


Embed magic number into model

(I'll push it just before releasing a new version)

@vbkaisetsu I fixed it not to use CARGO_PKG_VERSION.
Btw, I have not specified the minor version in MODEL_MAGIC, that is, using 0.5 instead of 0.5.0 because models for 0.5.* should always be compatible, and if broken changes arise, the major version should be updated.


Default Plural Form Support

@rmariuzzo Please consider this change
Adds a default plural form support for use with locales not listed in getPluralForm function.

Hey @aalmonte! As I could read you want the default plural form to be the first entry if none was found?


Fixed ordered list item matching

Closes #339

@wilsonowilson please set me as the reviewer for all super_editor work - I'm the only one who should be approving PRs.

Also @wilsonowilson please include a useful description with all PRs. What was the problem? How did you fix it? What might a reviewer want to know before looking at the code.

WRT tests, the higher level areas of the editor have been lax with tests, but we should start to reverse that trend.
Can you try writing appropriate tests to verify this behavior? The tests should be as narrow as possible, while still effectively validating the behavior.

@wilsonowilson do you plan to finish this work?

Hey @matthew-carroll, I'll add some tests later today

@matthew-carroll
How about this regex? ^\s*[1][.)]\s+$

It matches "1. " and "1) " but not "1 ".

Or more concisely: ^\s*1[.)]\s+$

@britannio I updated this work to use your latest regex. I can't seem to push my branch to this PR, so I'm going to close this PR and open a new one.


[5.0.0-next.1] useLingui() macro doesn't work if you don't use t

I've been trying out the latest version of the macro, the macro form of the useLingui hook works great if you include t in the destructuring, but breaks if you don't.
eg these work:
const { t, i18n } = useLingui();
const { t } = useLingui();

but this doesn't
const { i18n } = useLingui();

We want to consistently use the macro form, so this is a bit of an issue.
However, we primarily want the i18n object so we can access i18n._ for handling message descriptors constructed elsewhere, but it seems like we can just pass these objects to t without issue?

this is a bug, i will take a look.

However, we primarily want the i18n object so we can access i18n._ for handling message descriptors constructed elsewhere, but it seems like we can just pass these objects to t without issue?

It might work, but it is not how I supposed to use it, so it's better to avoid passing objects to t destructured from useLingui macro

speedy, thanks!
Is there a way to cut another release that doesn't include the swc version bump? As incorporating that would require us to upgrade to Next 15 canary, which we're not really in a position to do yet

The idea of those errors was that useLingui should be explicitly used with object destructuring, a not in a any other forms, such as
const lingui = useLingui()
lingui.t`Ola!`

But you are right, that might throw an error in your usecase as well. i'll create a tests case and fix in js repo as well.


Breaking changes in minor releases of the SDK

👋
Taking a look through the last few releases it appears there's been a few breaking changes in minor releases of the SDK:
https://github.com/Azure/azure-sdk-for-go/commit/8c472686968f1d7b29df3d4bb9435a2db9eacbe7#diff-84e58a944451296f0308ed309e92e03fL410-R438
https://github.com/Azure/azure-sdk-for-go/commit/8c472686968f1d7b29df3d4bb9435a2db9eacbe7#diff-84e58a944451296f0308ed309e92e03fL235-R263
Whilst we can adapt to these changes they are breaking changes - and as such I'd assume they'd take effect in a major version of the SDK at the end of the month as the breaking changes policy outlined here states - out of interest has this policy changed, or was this just a bug? :)
Thanks!

Hi @tombuildsstuff , in our release practice, preview packages will be considered as version 0 and always unstable, therefore breaking changes are allowed in minor version for preview packages.
See wiki page


Implement the "application signer" feature for J2EE / WAB Applications

every application deployed on a CrossWorlds server should be able tho specify the idenity (Notes FullName) to be used as "Signer" for the application.
RunAsSigner context will be running with the specified identity.
Configuration will happen with an InitParameter set in Web.xml / Context Configuration
Additional configuration options will be considered in the future

Added configuration parameter for the application context.
Implemented custom SessionFactory to get named session for the Signer (if set, otherwhise go NATIVE)


First run in VS Code fails due to welcome message

When the VS Code extension launches, we essentially run the command dotnet --version to see if we're up-to-date.  If this is done on a fresh machine with manual installs, that command won't return an expected string like 5.0.100, but will instead show a welcome message like Welcome to .NET...... and that breaks version number parsing.

N.b., the fix will likely be here where we should split on the '\n' character and only return the last line.  Will revisit this once we have a quick and easy repro to test.


Update CODEOWNERS

This aims to give the Pages team more autonomy given the recent change requiring a codeowner review.
This allows the Pages team to approve PRs which add a changeset, which touch only test files, and which make changes to fixture tests.

Should wrangler/api/pages be added too? 👀

@GregBrimble - I think most of the changes here have landed in @penalosa's PR. Shall we close this one?

Nope, I still think there's bits we need from this. Let me rebase.

Added all those spaces intentionally, @rozenmd ! Separated by groups of approvers. Figured it made it easier to see what everyone was responsible for and obvious where any new lines should go (rather than just appending to the end of the file?)


unnecessary double quotes around var

Double quotes around the ${latest} variable were preventing the install.sh script from resolving the URL

I was not able to reproduce this, but it looks cleaner anyways. Tnx for the fix.


Increase the time we wait for the aliasing

Increase the time we wait for the aliasing in the reverting tests.

Codecov Report

:exclamation: No coverage uploaded for pull request base (canary@2777ec3). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             canary    #2360   +/-   ##
=========================================
  Coverage          ?   10.91%
=========================================
  Files             ?      249
  Lines             ?     9017
  Branches          ?      986
=========================================
  Hits              ?      984
  Misses            ?     7932
  Partials          ?      101

Continue to review full report at Codecov.

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


Sudermanjr/install vault

rebase + serve vault over HTTPS + use vault dev mode (auto unsealed)

see https://github.com/KubeCrash/spring2023 instead


Added 207-Course-Schedule.kt and 417-Pacific-Atlantic-Waterflow.kt

File(s) Modified: 207-Course-Schedule.kt, 417-Pacific-Atlantic-Waterflow.kt
Language(s) Used: kotlin
Submission URL: https://leetcode.com/submissions/detail/770164214/, https://leetcode.com/submissions/detail/770164565/

Thanks, @Raven1233!


[ci] Enable writing to the bitstream cache in sival

This PR enables the earlgrey_es_sival branch to write bitstreams to the cache.
I'm not sure if this will actually work:

I haven't tracked where the GCP key is downloaded for this job just yet.
The bucket it's writing to is still bucketURI: "gs://opentitan-bitstreams/master" - will this cause issues?

Sorry for the mass review request. I changed the target branch from master to earlgrey_es_sival and it added every code owner for every commit between the two :face_exhaling:

Skipping CI so we can get a bitstream into the cache and resolve a backlog issue.


Fix pprof defaultSampleType

Fixes #415

Interprets pprof's defaultSampleType as an index into the string table as documented in the proto, not as an index into the sample types repeated-field. This allows parsing to succeed when the string-table index is not a valid sampleTypes index, which is common on allocation profiles.
Update the pprof snapshot. This is necessarily because we were previously interpreting an empty defaultSampleType as truthy-but-zero when long.js is present, ie: the first sample-type. But Speedscope-in-the-browser doesn't seem to include long.js, so our tests were disagreeing with in-browser behavior. With this PR, that should be fixed.

coverage: 42.119% (+0.09%) from 42.032% when pulling 9fe218085db2ca8704ae9481b2522b0d9c3bc8b9 on vasi-stripe:vasi-pprof-default into 0414c2f617742e7fb0cf31a66ac0f77c2f5c0540 on jlfwong:main.

Thank you!

This is now live on https://speedscope.app and published to npm as part of v1.15.2. Thanks for your contribution!


GKE Connection Test Failing

Is there another step required to get cloud build to talk to cluster nodes? In my own testing I was unable to get cloud build to talk to a node without whitelisting <IP_ADDRESS> and after running your setup I'm seeing the same results.
Step #9 - "integration tests": url: http://<IP_ADDRESS>:30332
Step #9 - "integration tests": interval: 3
Step #9 - "integration tests": retries: 20
Step #9 - "integration tests": START CONNECTION TESTi

This never seems to complete.

Hey! I just merged @murog's revised instructions -- see the updated README and apply the Firewall changes. Let us know if it works. Thanks!

fixed in #52


[ACR] Separate ContentProperties into resource-specific types

The name ContentProperties itself is a little generic.
The service team is beginning to differentiate ChangeableAttribute by resource type, e.g. ManifestChangeableAttributes have quarantine fields and RepositoryChangeableAttributes will have teleportEnabled field.  We want to allow the properties and writeable properties of each resource to evolve independently.
Ask here is:

Pending resolution of swagger file representation, design this section of the API to accomodate independent evolution of the different resource properties
Implement the changes, which may require more modification in the customization layer.
Create work items for the implementation in all languages.

Completed in https://github.com/Azure/azure-sdk-for-net/pull/21078


fix: fix get log filter with address

Which docs this PR relation:
get logs should filter with address
Which toolchain this PR adaption:
No Breaking Change
Special notes for your reviewer:

Codecov Report

Merging #119 (02c0be9) into main (7fecba2) will decrease coverage by 0.05%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #119      +/-   ##
==========================================
- Coverage   38.46%   38.40%   -0.06%
==========================================
  Files          76       76
  Lines        5195     5202       +7
==========================================
  Hits         1998     1998
- Misses       3197     3204       +7

Impacted Files
Coverage Δ

core/api/src/jsonrpc/impl.rs
0.00% <0.00%> (ø)

protocol/src/types/primitive.rs
6.42% <0.00%> (ø)


Modify the ENC28J60 MAC driver implementation to allow multicast packets (IDFGH-6151)

This patch modifies the example ENC28J60 MAC driver implementation (esp_eth_mac_enc28j60.c) to allow multicast packets to pass through the hardware's internal packet filter, rather than filtering them out.
This change makes the example driver applicable to a wider range of projects and avoids confusion from new users of the driver who want to use multicast over Ethernet. Almost no projects need multicast packets filtered out, but some certainly may want them included.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for your contribution.

Hi @andycarle, thank you for the contribution! The update you performed makes perfect sense! However, we need to prepare the PR to pass formal checks too. Have you installed the pre-commit hook for esp-idf project as specified here? Please also check you conform with other bullets. Speaking of pre-commit hook, it should ensure the licence header update. Otherwise we wouldn't pass internal tests and the change could not be merged.

Hi @kostaond, thank you very much for the pointer. You are right, I had not installed the pre-commit hook.
I'm getting that set up now. Once I have that in place and fix the license header, is the best practice to simply open a new pull request with a fixed commit? Or should I force push/squash the corrected commit into this PR?
Thank you!

I would prefer to use the same PR because of tracking the issue. Please "force push" the modified commit. Thanks! If you needed any assistance, do not hesitate to ask.

@kostaond Thank you for the recommendation. I have worked through the pre-commit fixes and force pushed the modified commit. Please take a look when you get a chance and let me know if I should make any additional changes.
(For what it's worth: I was unable to get the ESP-IDF pre-commit hooks working correctly on Windows. There seems to be a bug right now that causes false positives on the tools folder. This was also mentioned in #7547. But I got it working on macOS.)
Thank you!


Prevented save button from being activated on load even when nothing changes

Summary
Currently, the Save button in settings gets activated on load. This was because I was calling the "save needed" function in useEffect, which gets called on load as well. Converted it to a useCallback function to resolve it.

@harshilsharma63 Had a peek, no need for my review, all good


Issue 509: add autocompletion to Code editor

fix https://github.com/Altinity/clickhouse-grafana/issues/509

@lunaticusgreen could you merge with master and fix conflicts with master?

ok. i already resolve conflicts, let's merge


except AttributeError was logging incorrect error message

When I remove this catch block
except AttributeError:
     logger.error(u"invalid credentials data: path={}".format(credentials))
     result_counter.inc_fail()

The error shown suggests that it's not related to malformatted credentials, so I think the except block should be removed until it can communicate the correct error to the logger or user. Below is the stack trace that is shown with the change in the branch for this PR:
❯ sqlitebiter gs ./my-oauth-creds.json my-spreadsheet -o my-db.sqlite
Traceback (most recent call last):
  File "/usr/local/bin/sqlitebiter", line 11, in <module>
    sys.exit(cmd())
  File "/usr/local/lib/python2.7/site-packages/click/core.py", line 722, in __call__
    return self.main(*args, **kwargs)
  File "/usr/local/lib/python2.7/site-packages/click/core.py", line 697, in main
    rv = self.invoke(ctx)
  File "/usr/local/lib/python2.7/site-packages/click/core.py", line 1066, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/usr/local/lib/python2.7/site-packages/click/core.py", line 895, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/local/lib/python2.7/site-packages/click/core.py", line 535, in invoke
    return callback(*args, **kwargs)
  File "/usr/local/lib/python2.7/site-packages/click/decorators.py", line 17, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/usr/local/lib/python2.7/site-packages/sqlitebiter/sqlitebiter.py", line 453, in gs
    for logger.warnlogger.warntable_data in warndata:
  File "/usr/local/lib/python2.7/site-packages/pytablereader/spreadsheet/gsloader.py", line 119, in load
    dp_extractor=self._loader.dp_extractor)
AttributeError: 'GoogleSheetsTableLoader' object has no attribute '_loader'

Ultimately, I'm interested in knowing how to resolve AttributeError: 'GoogleSheetsTableLoader' object has no attribute '_loader' from pytablereader.

using pip install pytablereader==0.19.3, btw.

Thank you for your pull request.
I had fixed the problem (AttributeError) at pytablereader 0.20.0.
The error caused by invalid attribute reference which I failed to modify at the previous version.
I will merge your PR and publish a new version at the later.


Write a C++ program to find area of a right angled triangle

Description
Write a C++ program to find area of a right angled triangle
Area of right angled triangle = 0.5 × Base × Height
Base   : 2
Height : 3

Area : 3

How to contribute

Save the solution in FindAreaOfARightAngledTriangle.cpp file
Add FindAreaOfARightAngledTriangle.cpp file in find-area-of-a-right-angled-triangle folder

!assign


Allow typeholes in other files

My biggest feature request would be to allow type holes to push output to other files. For example, I have a directory with the following structure:

I would love to be able to create typeholes in each request that goes out to Quickbooks, and generate the types as the responses are received, and have those types go to internalTypes.ts instead of the file that the request was made in.
Loving the plugin so far! Really feels like it was developed by someone who understands that pain of not getting types from 3rd party APIs :) I could see this totally revolutionizing the creation of type files from existing Javascript libraries that have none...

Really feels like it was developed by someone who understands that pain of not getting types from 3rd party APIs

You got that right 😄
I wonder if that feature would make sense as a workspace-level configuration.. I've been meaning to add some code to typehole to resolve interface references not in the same file as your typehole. That should be relatively easy to do. When I have that feature in place, you should at least be able to create a placeholder type first to fileA.ts
export type MyPlaceholder = any

and then create a typehole with that type, so fileB.ts
const someValue: MyPlaceholder = typehole.t(someValue)

and the interface should now update in fileA

The approach I described in the previous comment should be possible with the 1.5.1 version. Give it a try and see if something like this would be enough for your use case 🙂

Perfect!

I'll try it now

This feature would be awesome! I can confirm it works with relative imports, but not with aliased imports. It would be awesome if it supported alias imports as well :)


spelling

Hi @staff0rd, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, DNFBOT;

@Tratcher can you rview and mrge?

:shpit:


uadk: adjust internal file include relationships

The file inclusion relationship in the current uadk is confusing, and there are multiple repeated inclusions, which leads to confusion in the internal inclusion relationship.
In addition, the interface to the APP application layer and the interface to the user mode driver layer are mixed together, making it inconvenient for the APP application layer to use UADK. After adjustment, when the APP application layer calls the uadk interface to execute services, it only needs to call the header file of the corresponding algorithm , and driver development only needs to include its own header file and the driver-side interface of the corresponding algorithm.

+1
Tested OK on x86 and arm.
Two warning can be fixed, others are ok
WARNING: Missing a blank line after declarations
#715: FILE: wd_util.c:152:

  struct wd_ctx_h *ctx = (struct wd_ctx_h *)h_ctx;

  if (!ctx || qfrt >= UACCE_QFRT_MAX)

WARNING: suspect code indent for conditional statements (8, 24)
#715: FILE: wd_util.c:152:

  if (!ctx || qfrt >= UACCE_QFRT_MAX)

                  return 0;

OK! Thanks you!  I'll change it

@hzhuang1 ，Please help to review

I'm fine on this.
@haofang111 What's your opinion?


[AKS] Fixed indentation

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your PR is merged into master branch, a new PR will be created to update src/index.json automatically.
The precondition is to put your code inside this repo and upgrade the version in the PR but do not modify src/index.json.

If this PR is for a new extension or change to an existing extension, use the following to try out the changes in this PR:
docker run -it microsoft/azure-cli:latest
export EXT=<NAME>
pip install --upgrade --target ~/.azure/cliextensions/$EXT "git+https://github.com/KaiqiZhang/azure-cli-extensions.git@fix-identation#subdirectory=src/$EXT&egg=$EXT"

AKS


Reduce memory by using all_gather_into_tensor

What does this PR do?
torch 1.13 introduced publicly all_gather_into_tensor (before this was _all_gather_base) which is a much more memory efficient version of gather. One thing to note mentioned in the PR here is they did not have this be the base of gather, since it handles uneven inputs automatically. Since Accelerate does this separately and checks, we can safely use this API.
For a general idea of just how much memory can be stored, I ran a small simple test:
import time
import torch
from accelerate import PartialState
from accelerate.utils import gather

def convert_bytes(size):
    "Converts `size` from bytes to the largest possible unit"
    for x in ["bytes", "KB", "MB", "GB", "TB"]:
        if size < 1024.0:
            return f"{round(size, 2)} {x}"
        size /= 1024.0

    return f"{round(size, 2)} PB"

state = PartialState()
tensor = torch.rand((64, 224, 224, 64), device=state.device)

# Using `PartialState`
start_time = time.time()
tensor = gather(tensor)
end_time = time.time()

with state.main_process_first():
    print(f"Process {state.process_index} memory allocated after: {convert_bytes(torch.cuda.max_memory_allocated(state.device))}")
    print(f"Process {state.process_index} time: {end_time - start_time}")

The results can be summarized as such:
Before:
Total CUDA memory allocated: 3.83gb
After:
Total CUDA memory allocated: 2.3gb
Fixes # (issue)
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[x] Did you read the contributor guideline,
Pull Request section?
[ ] Was this discussed/approved via a Github issue or the forum? Please add a link
to it if that's the case.
[ ] Did you make sure to update the documentation with your changes? Here are the
documentation guidelines, and
here are tips on formatting docstrings.
[ ] Did you write any new necessary tests?

Who can review?
Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.
@BenjaminBossan @LysandreJik

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Have build.rs emit an appropriate error if binaryen submodule does not exist

Right now there's just a mysterious file not found error.

done


Using data-count-to attribute instead of data-to messes up the script

Using the "data-count-to" attribute on the element seems to make the script count to the number specified + 1. In addition, the script seems to not stop updating the element(not tested but likely, because my laptop fan started whirling heavy).
I know that "data-to" should be used, I'm just pointing out a fun fact. I didn't read the docs and needed a way to dynamically pass the "to" config.

What does your code look like?  The data-count-to attribute you mentioned should be irrelevant to this plugin since it's not a supported attribute.

Gonna go ahead and close this since I haven't heard back.  Feel free to reopen if you're interested.


Support for an optional last argument, to count backward

With this PR I aim to add an optional argument to the last filter, so address the scenario where you might not wan't the last, but the second last. But without having to keep a count of children, to then use the .at(index).
I see it work like this:
const wrapper = shallow(

a

b

c

d

);

expect(wrapper.find('li').last().text()).toEqual('d');
expect(wrapper.find('li').last(-1).text()).toEqual('c');
// or
const children = wrapper.find('li')
expect(children.at(children.length - 2).text()).toEqual('c');

This already exists - it's called slice. You can do .slice(-1), for example.

Oh - in which case this is probably mute. Just thought that .at is akin to :nth-child, and .last was akin to :nth-last-child. But thanks. :100:

Since enzyme wrappers are collections, there's not actually a difference between what "last" provides (a wrapper around 1 item) - and what .slice(-2, 1) provides (a wrapper around 1 item).


[MTDSA-199] [CK] Exclude header validation check in filter for API do…

…cumentation endpoints

:+1:


children routes are not working

Hi, it seems children routes are not working at all. I tested both static and dynamic routes but no luck. Defining at top level works.
Reproduction: https://stackblitz.com/edit/vitejs-vite-ms9fna?file=src/main.tsx

It should be fixed in v1.4.1, thanks 👍 I had two lines in the wrong order here:
https://github.com/fabiospampinato/voby-simple-router/blob/9b91a59c300457f5312da4c5275e19f706c25f6d/src/hooks/use_router.ts#L23-L25

Thanks for quick fix 🙏


Add entities pool_pump.next_run_start and pool_pump.next_run_stop with format

Hello,
I added the entities pool_pump.next_run_start and pool_pump.next_run_stop
based on the comment of @mycanaletto in this issue : https://github.com/oncleben31/ha-pool_pump/issues/9
Not 100% sure what kind of object  state can store (datetime, string ? advice is welcome )
Regards,

Hello !
I couldn't test it because since I reinstalled HACS it doesn't start anymore, as if something was missing in the configuration...
Invalid config for [pool_pump]: [pool_temperature_entity_id] is an invalid option for [pool_pump]. Check: pool_pump->pool_pump->pool_temperature_entity_id. (See /config/configuration.yaml, line 99).

My code is not merged yet... But I think for this error you have to take the correct version, the one in dev, 0.0.2beta1

OK it works with the beta, I'm waiting for the merge for the new features. I'm especially interested in a winter mode...

Ok, could you descibe what you expect from a winter mode?
If you apply a passive wintering, you can set the mode to off, if you apply an active wintering, just keep the component in auto.
If I look the abacus provided by @oncleben31 it will always run 2h per day
https://www.github.com/scadinot/pool/tree/master/docs%2Fimages%2Ffiltration_courbe.png
By the way I read your article about how to migrate from jeedom to home assistant when I migrate to ha, I used it a lot, especially the climate part, nice job👍

In fact, I take care of my brother's pool.
At the moment, he runs the filtration for 5 minutes every hour from 8pm to 8am (night) and 5 minutes every 3 hours the rest of the time.
What would be nice would be to integrate a risk of frost mode (weather forecast) to increase the rate.
Under Jeedom there was a pscine plugin which seemed to be well done, but I never tested it. Maybe an inspiration if you want to achieve something complete.
Yes I wrote a lot of things, Jeedom is far away...

Hello guys. Thank you for your contribution to my integration. To be honest I didn't invest the required energy to make it more mature.
I like your proposals. So let's try to improve it.
I will review it quickly


Added emoji-cheat-sheet

Hello,
Thanks for your PR but this is literally the same content, but the first link displays emojis next to their code, so it's better I think.


docs: update for cloud docs

Description
update for cloud docs
Release required?

[ ] No release
[ ] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

/run pipeline

/ruh pipeline

/run pipeline

:tada: This PR is included in version 1.4.0 :tada:
The release is available on:

GitHub release
v1.4.0

Your semantic-release bot :package::rocket:


InvalidAuthenticationTokenTenant error when deploying Remote Monitoring Solution in Standard config

Type of issue

[X] Bug
[ ] New feature
[ ] Enhancement

Description
Issuing this command
pcs -t remotemonitoring -s standard

Results in the error shown below. I have two subscriptions on the same account, so I wonder if the problem is that a domain qualifier is required?
error:
{ Error: The access token is from the wrong issuer 'https://sts.windows.net/<removed>/'. It must match the tenant 'https://sts.windows.net/<removed>/' associated with this subscription. Please use the authority (URL) 'https://login.windows.net/<removed>' to get the token. Note, if the subscription is transferred to another tenant there is no impact to the services, but information about new tenant could take time to propagate (up to an hour). If you just transferred your subscription and see this error message, please try back later.
    at client.pipeline (C:\Users\daren\AppData\Roaming\npm\node_modules\iot-solutions\node_modules\azure-arm-authorization\lib\operations\roleAssignments.js:617:19)
    at retryCallback (C:\Users\daren\AppData\Roaming\npm\node_modules\iot-solutions\node_modules\ms-rest\lib\filters\systemErrorRetryPolicyFilter.js:89:9)
    at retryCallback (C:\Users\daren\AppData\Roaming\npm\node_modules\iot-solutions\node_modules\ms-rest\lib\filters\exponentialRetryPolicyFilter.js:140:9)
    at C:\Users\daren\AppData\Roaming\npm\node_modules\iot-solutions\node_modules\ms-rest\lib\filters\rpRegistrationFilter.js:59:14
    at handleRedirect (C:\Users\daren\AppData\Roaming\npm\node_modules\iot-solutions\node_modules\ms-rest\lib\filters\redirectFilter.js:39:9)
    at C:\Users\daren\AppData\Roaming\npm\node_modules\iot-solutions\node_modules\ms-rest\lib\filters\formDataFilter.js:23:14
    at Request.defaultRequest [as _callback] (C:\Users\daren\AppData\Roaming\npm\node_modules\iot-solutions\node_modules\ms-rest\lib\requestPipeline.js:125:16)
    at Request.self.callback (C:\Users\daren\AppData\Roaming\npm\node_modules\iot-solutions\node_modules\request\request.js:185:22)
    at emitTwo (events.js:126:13)
    at Request.emit (events.js:214:7)
    at Request.<anonymous> (C:\Users\daren\AppData\Roaming\npm\node_modules\iot-solutions\node_modules\request\request.js:1161:10)
    at emitOne (events.js:121:20)
    at Request.emit (events.js:211:7)
    at IncomingMessage.<anonymous> (C:\Users\daren\AppData\Roaming\npm\node_modules\iot-solutions\node_modules\request\request.js:1083:12)
    at Object.onceWrapper (events.js:313:30)
    at emitNone (events.js:111:20)
  statusCode: 401,
  request:
   { rawResponse: false,
     queryString: {},
     url: 'https://management.azure.com//subscriptions/<removed>/providers/Microsoft.Authorization/roleAssignments/<removed>?api-version=2015-07-01',
     method: 'PUT',
     headers:
      { 'Content-Type': 'application/json; charset=utf-8',
        'x-ms-client-request-id': '<removed>',
        'accept-language': 'en-US',
        'user-agent': 'Node/v8.9.3 (x64-Windows_NT-10.0.17758) ms-rest/2.3.4 ms-rest-azure/2.5.7 azure-arm-authorization/4.1.0 Azure-SDK-For-Node' },
     body: '{"properties":{"roleDefinitionId":"/subscriptions/<removed>/providers/Microsoft.Authorization/roleDefinitions/<removed>","principalId":"<removed>"}}' },
  response:
   { body: '{"error":{"code":"InvalidAuthenticationTokenTenant","message":"The access token is from the wrong issuer \'https://sts.windows.net/<removed>/\'. It must match the tenant \'https://sts.windows.net/<removed>/\' associated with this subscription. Please use the authority (URL) \'https://login.windows.net/<removed>\' to get the token. Note, if the subscription is transferred to another tenant there is no impact to the services, but information about new tenant could take time to propagate (up to an hour). If you just transferred your subscription and see this error message, please try back later."}}',
     headers:
      { 'cache-control': 'no-cache',
        pragma: 'no-cache',
        'content-type': 'application/json; charset=utf-8',
        expires: '-1',
        'www-authenticate': 'Bearer authorization_uri="https://login.windows.net/781de9ba-95b5-4eec-8884-6ac05b90cabb", error="invalid_token", error_description="The access token is from the wrong issuer. It must match the tenant associated with this subscription. Please use correct authority to get the token."',
        'x-ms-failure-cause': 'gateway',
        'x-ms-request-id': <removed>',
        'x-ms-correlation-request-id': '<removed>',
        'x-ms-routing-request-id': 'WESTCENTRALUS:20180913T184107Z<removed>',
        'strict-transport-security': 'max-age=31536000; includeSubDomains',
        'x-content-type-options': 'nosniff',
        date: 'Thu, 13 Sep 2018 18:41:07 GMT',
        connection: 'close',
        'content-length': '677' },
     statusCode: 401 },
  code: 'InvalidAuthenticationTokenTenant',
  body:
   { code: 'InvalidAuthenticationTokenTenant',
     message: 'The access token is from the wrong issuer \'https://sts.windows.net/<removed>/\'. It must match the tenant \'https://sts.windows.net/<removed>/\' associated with this subscription. Please use the authority (URL) \'https://login.windows.net/<removed>\' to get the token. Note, if the subscription is transferred to another tenant there is no impact to the services, but information about new tenant could take time to propagate (up to an hour). If you just transferred your subscription and see this error message, please try back later.' } }
{
  "statusCode": 401,
  "request": {
    "rawResponse": false,
    "queryString": {},
    "url": "https://management.azure.com//subscriptions/<removed>/providers/Microsoft.Authorization/roleAssignments/<removed>?api-version=2015-07-01",
    "method": "PUT",
    "headers": {
      "Content-Type": "application/json; charset=utf-8",
      "x-ms-client-request-id": "2ba25a7b-c1a1-49a4-810f-1e90c32272ac",
      "accept-language": "en-US",
      "user-agent": "Node/v8.9.3 (x64-Windows_NT-10.0.17758) ms-rest/2.3.4 ms-rest-azure/2.5.7 azure-arm-authorization/4.1.0 Azure-SDK-For-Node"
    },
    "body": "{\"properties\":{\"roleDefinitionId\":\"/subscriptions/<removed>/providers/Microsoft.Authorization/roleDefinitions/<removed>5\",\"principalId\":\"<removed>\"}}"
  },
  "response": {
    "body": "{\"error\":{\"code\":\"InvalidAuthenticationTokenTenant\",\"message\":\"The access token is from the wrong issuer 'https://sts.windows.net/<removed>/'. It must match the tenant 'https://sts.windows.net/<removed>/' associated with this subscription. Please use the authority (URL) 'https://login.windows.net/<removed>' to get the token. Note, if the subscription is transferred to another tenant there is no impact to the services, but information about new tenant could take time to propagate (up to an hour). If you just transferred your subscription and see this error message, please try back later.\"}}",
    "headers": {
      "cache-control": "no-cache",
      "pragma": "no-cache",
      "content-type": "application/json; charset=utf-8",
      "expires": "-1",
      "www-authenticate": "Bearer authorization_uri=\"https://login.windows.net/<removed>\", error=\"invalid_token\", error_description=\"The access token is from the wrong issuer. It must match the tenant associated with this subscription. Please use correct authority to get the token.\"",
      "x-ms-failure-cause": "gateway",
      "x-ms-request-id": "<removed>",
      "x-ms-correlation-request-id": "<removed>",
      "x-ms-routing-request-id": "WESTCENTRALUS:20180913T184107Z:<removed>",
      "strict-transport-security": "max-age=31536000; includeSubDomains",
      "x-content-type-options": "nosniff",
      "date": "Thu, 13 Sep 2018 18:41:07 GMT",
      "connection": "close",
      "content-length": "677"
    },
    "statusCode": 401
  },
  "code": "InvalidAuthenticationTokenTenant",
  "body": {
    "code": "InvalidAuthenticationTokenTenant",
    "message": "The access token is from the wrong issuer 'https://sts.windows.net/<removed>/'. It must match the tenant 'https://sts.windows.net/<removed>/' associated with this subscription. Please use the authority (URL) 'https://login.windows.net/<removed>' to get the token. Note, if the subscription is transferred to another tenant there is no impact to the services, but information about new tenant could take time to propagate (up to an hour). If you just transferred your subscription and see this error message, please try back later."
  }
}

...
Steps to reproduce

Have a multi-tenant azure account
pcs -t remotemonitoring -s standard
Boom!

Expected behavior
Successful deploy.
...
Current behavior <!-- The actual behavior observed --
Unable to deploy to my account.
...
Known workarounds
...
Possible solution
Detect if there are multiple tenants and prompt for a tenant qualifier?
...
Context and Environment

Operating System: Windows 10 1809 17758.1
GitHub branch: ...
Node.js Runtime: v8.9.3

Just noticed that pcs has a --domainId option - trying that...

#359 is the fix for this issue. Please let us know if you have any more questions.


Continuous Integration and deployment - Data Factory Custom Activity

Hi,
I have some custom activity in a pipeline I created in Azure Data Factory v2. I saw that there are several documents explaining how to write/do pipelines for deployment of Data Factory "apps". But none of them explains what to do when you have a custom activity (meaning you have to move a console app to a blob storage folder, etc). Since I couldn't find anything, it looks like it's a flaw? How that can be achieved in a reliable and simple way?
Thanks in advance

@breno601  Thank you for your query. We will check it with the engineering internally and let you know accordingly.

@djpmsft Can you please take a look at this ?

Hi there,
Can you elaborate on what the exact ask is? Are you trying to deploy a pipeline with a custom activity? Are you wondering on how to trigger a pipeline with a custom activity?
Thanks,
Daniel

Hey Daniel,  thanks for reaching out.
I have some pipelines in DF that uses Custom Activities. I am currently deploying it manually by moving the files to the AzureBlobStorage. I couldn't find any documentation regarding how to automate the deployment of those files/custom activities.
Thanks in advance!
Breno

Hey Breno,
By deploying, do you mean running the pipeline? On which cadence are you trying to automate the deployment of these files/custom activities?
Sorry for so many follow ups, just want to make sure I am going to give you the correct advice!
Thanks,
Daniel

Hi Daniel,
no worries, I'm glad you're helping :)
By deploying I mean deploying the pipelines of my Data Factory application. Like, let's say I modified code for a Custom Activity. Then pushed it. After merging to specific branch, I would trigger azure pipelines for deploying it to "production". But since I have those custom activities class files that must be sent to AzureBlobStorage, it makes the whole thing a bit complicate. What is the correct/recommended way to do so?
Sorry if I'm not being too clear on it. Let me know if you need more explanation.
Thanks in advance!

Hey Breno,
Apologies for the super large delay in response! This issue must have gotten lost in the mix. Overall any custom activity code needs to have a CI/CD process that parallels ADF's (this applies to any external activity that gets orchestrated). This can either be included in the same CI/CD pipeline or separate pipelines. As this is a separate Azure service that we are orchestrating, I will defer to their team for best practices.
Really sorry this took so long to respond. As there has been no activity for ~11 months I assume you have unblocked yourself and I will go ahead and close this issue. Feel free to continue commenting!
Best,
Daniel
#please-close


Sort tags on semver format to discoved latest tag

This pull request is a potential fix for #83. It uses semver::Version to turn the tags into version objects that can be sorted based on semver convention.
There is also a test included that fails on the master version and succeeds with this fix.
Please bare in mind that Rust is not my daily language and the current state of the PR should be considered a starting point rather than the fix.
I allowed edits by maintainers for this PR, please use it!

@pjvds thank you for fixing this. I will l take a  closer look tomorrow.

@pjvds thank you for fixing this. I will l take a  closer look tomorrow.

You are welcome @oknozor! Looking forward to your feedback. Take care in the meantime.

You are welcome @oknozor! Looking forward to your feedback. Take care in the meantime.

@oknozor just let me know if you need any extra help moving this one forward.

@oknozor just let me know if you need any extra help moving this one forward.

Sorry for the delay, was really busy lately. well done @pjvds ! :partying_face:

Sorry for the delay, was really busy lately. well done @pjvds ! :partying_face:


Number of total conversations doesn't much number of resources returned.

The number of conversations (14), shown in count,  is double the number of resources returned (7).
Note: these are messages I send to myself in order to replicate zooniverse/Panoptes-Front-End#3627.

That breaks paging on the frontend https://github.com/zooniverse/Panoptes-Front-End/issues/3627

Turns out messages sent to oneself are counted twice.


Delete old stale files that we don't need any longer

cc @mitchdenny @JimSuplizio

@weidongxu-microsoft  @iscai-msft @jianghaolu can you guys confirm or deny whether or not the package.json or gulpfile.js is needed in the root of this repo? I assume they are related to codegen in some way but I'm not sure how or if they are consumed.

@weshaggard
IFAIW, package.json is dependencies configure for npm/nodejs. npm will install gulp, which uses gulpfile.js as configure.
Then, gulpfile.js will consume api-specs.json which contains configuration for autorest.java source readme.md and other input parameters.
E.g.
  "advisor/resource-manager": {
    "source": "specification/advisor/resource-manager/readme.md",
    "args": "--multiapi --fluent"
  },

@weidongxu-microsoft thanks for letting know these are in use. I will remove them from my PR.
I don't know how much work it is but it would be idea if we could move these files out of the root and somewhere under the eng directory to try and clean-up the root directory further.

@weidongxu-microsoft as an aside it doesn't look the other language repo's have these files so it would be good if we can figure out how to either eliminate or move them.

@weshaggard We will have some brainstorm within our team to see whether what are the alternatives, and how other teams do for this. (one alternative would be let swagger spec raise PR automatically based on readme.md there)

@weshaggard Alternatively, could we just move them into somewhere under eng folder and leave them there? It will not cluster in project root, while we do not need to spend time to figure out another approach (though that should be relatively simple, some python script might just works).
What I heard from swagger team is that long term, the publish of mgmt SDK could be integrated into swagger CI. Basically when modification (by service team) on spec JSON file is merged, swagger CI will use README.java.md to generate mgmt SDK, and automatically put a PR into azure-sdk-for-java. Then service team could use the release request site to ask us to merge that PR. Ideally we could get that PR through our CI and merge it without touching the code at all.
This approach would probably also do away with the gulp as well. Though it will be long term and I do not get a date from swagger team.

Yes I agree ideally this should be coming from the swagger api specs repo but if it is easy to update the paths and move these scripts under eng\mgmt that would be a nice easy fix to help clean-up the root repo for now.


ProfileNotFound: The config profile (dev) could not be found

When trying to sign up for an account we get a ProfileNotFound error.
Here is an example with dummy data in place of real values:
Signing up for castLabs EVS
 - A valid e-mail address is required for account verification
>> E-mail Address<EMAIL_ADDRESS>
>> First Name [Firstname]:
>> Last Name [Lastname]:
>> Organization [Example Org]:
>> Account Name [Account]:
>> Password []:
>> Verify Password []:
ProfileNotFound: The config profile (dev) could not be found

Interestingly this appears to be because of the default behavior of boto3, which is used for account creation. If one of the environment variables AWS_PROFILE or AWS_DEFAULT_PROFILE is set to indicate a profile that does not exist in the local AWS configuration it results in ProfileNotFound.
You should be able to work around this by making sure none of the two environment variables mentioned above are set when you run the EVS tool. I'll see about making the EVS tools handle this scenario better in future versions, thanks for reporting it!

Thanks Emil - we were using a docker container to build for win32 and did not realize it had an AWS_PROFILE set (nor that it mattered). We have it working now.

I just published castlabs_evs-1.0.0 to PyPi as it is now mature enough to drop the release candidate status. It fixes a few minor things, including handling AWS related environment variables, so if they happen to be set it should not matter anymore.
Please try it out and unless you have anything else I think we can close this issue, no?


Imprpve tests: get error line and position

[x] tests and linter show no problems (npm t)
[ ] tests are added/updated for bug fixes and new features
[x] code is properly formatted (npm run fmt)
[ ] description of changes is added in CHANGELOG.md
[ ] update .d.ts typings

Landed in https://github.com/metarhia/metavm/commit/02efc2df76968252a755cd84a6b1faa8868e0375


Enable loopvar feature of go for EG

          This loopvar feature of go 1.22 requires enable env `GOEXPERIMENT=loopvar` (default closed) explicitly. Maybe we could consider enable this feature for other places as well.

Originally posted by @shawnh2 in https://github.com/envoyproxy/gateway/issues/2739#issuecomment-2068969587

The usage of this feature requires setting go env specificly, for example GOEXPERIMENT=loopvar go test.

I'm very interested in this issue, please assign it to me.

the loopvar can be detected by https://golangci-lint.run/usage/linters#copyloopvar

@shawnh2 is this issue still relevant? I would like to work on this

@Awinr are you working on this?

@shawnh2 is this issue still relevant? I would like to work on this

Yes, thinks for taking this issue, assign you.


Bump CM.VersionMajor

It seems the game version was not updated thus bookmark gives warning alert when load.

This has been done in dev already.  I've just not released it.  Sorry for the delay.