Error executing action restart on resource 'service[iptables]'

Cookbook version
2.6.2
Chef-client version
13.2.20
Platform Details
CentOS release 6.9 (Final)
# iptables --version
iptables v1.4.7

cookbook
firewall 'default' do
    action :install
end
firewall_rule 'mysql' do
    port      3306
    protocol  :tcp
    position  0
    command   :allow
end

# cat /etc/sysconfig/iptables
# position 0
-A INPUT -p tcp -m tcp -m multiport --dports 22 -m comment --comment "ssh" -j ACCEPT
-A INPUT -p tcp -m tcp -m multiport --dports 3306 -m comment --comment "mysql" -j ACCEPT
# position 1
*filter
# position 2
:INPUT DROP
# position 3
:FORWARD DROP
# position 4
:OUTPUT ACCEPT
# position 100
COMMIT

# service iptables restart
iptables: Applying firewall rules: iptables-restore: line 2 failed
                                                           [FAILED]

Chef Output
--- /etc/sysconfig/ip6tables	2017-08-01 21:48:03.091009003 -0400
==> mysql:
==> mysql: +++ /etc/sysconfig/.chef-ip6tables20170801-5056-m44out	2017-08-01 21:48:57.155009002 -0400
==> mysql:
==> mysql: @@ -1,2 +1,14 @@
==> mysql:
==> mysql: -# created by chef to allow service to start
==> mysql:
==> mysql: +# position 0
==> mysql:
==> mysql: +-A INPUT -p tcp -m tcp -m multiport --dports 22 -m comment --comment "ssh" -j ACCEPT
==> mysql:
==> mysql:       +-A INPUT -p tcp -m tcp -m multiport --dports 3306 -m comment --comment "mysql" -j ACCEPT
==> mysql:       +# position 1
==> mysql:       +*filter
==> mysql:       +# position 2
==> mysql:       +:INPUT DROP
==> mysql:       +# position 3
==> mysql:       +:FORWARD DROP
==> mysql:       +# position 4
==> mysql:       +:OUTPUT ACCEPT
==> mysql:       +# position 100
==> mysql:       +COMMIT
==> mysql:
==> mysql:       - restore selinux security context
==> mysql:
==> mysql: [2017-08-01T21:48:57-04:00] INFO: firewall[default] sending restart action to service[iptables] (delayed)
==> mysql:   * service[iptables] action restart
==> mysql:
==> mysql:
==> mysql:
==> mysql: ================================================================================
==> mysql:
==> mysql: Error executing action `restart` on resource 'service[iptables]'
==> mysql:
==> mysql: ================================================================================
==> mysql:
==> mysql:
==> mysql:
==> mysql:
==> mysql: Mixlib::ShellOut::ShellCommandFailed
==> mysql:
==> mysql: ------------------------------------
==> mysql:
==> mysql:     Expected process to exit with [0], but received '1'
==> mysql:     ---- Begin output of /sbin/service iptables start ----
==> mysql:     STDOUT: iptables: Applying firewall rules: [FAILED]
==> mysql:     STDERR: iptables-restore: line 2 failed
==> mysql:     ---- End output of /sbin/service iptables start ----
==> mysql:     Ran /sbin/service iptables start returned 1
==> mysql:
==> mysql:     Resource Declaration:
==> mysql:     ---------------------
==> mysql:     # In /var/chef/cache/cookbooks/firewall/libraries/provider_firewall_iptables.rb
==> mysql:
==> mysql:     153:         iptables_service = service name do
==> mysql:     154:           action :nothing
==> mysql:     155:         end
==> mysql:     156:       end
==> mysql:
==> mysql:     Compiled Resource:
==> mysql:     ------------------
==> mysql:     # Declared in /var/chef/cache/cookbooks/firewall/libraries/provider_firewall_iptables.rb:153:in `rescue in lookup_or_create_service'
==> mysql:
==> mysql:     service("iptables") do
==> mysql:       action [:nothing]
==> mysql:       default_guard_interpreter :default
==> mysql:       service_name "iptables"
==> mysql:       enabled nil
==> mysql:       running nil
==> mysql:       masked nil
==> mysql:       pattern "iptables"
==> mysql:       declared_type :service
==> mysql:       cookbook_name "mysql-drupal"
==> mysql:     end
==> mysql:
==> mysql:     System Info:
==> mysql:     ------------
==> mysql:     chef_version=13.2.20
==> mysql:     platform=centos
==> mysql:     platform_version=6.9
==> mysql:     ruby=ruby 2.4.1p111 (2017-03-22 revision 58053) [x86_64-linux]
==> mysql:     program_name=chef-client worker: ppid=5051;start=21:47:50;
==> mysql:     executable=/opt/chef/bin/chef-client
==> mysql:
==> mysql: [2017-08-01T21:48:58-04:00] INFO: firewall[default] sending restart action to service[ip6tables] (delayed)
==> mysql:   * service[ip6tables] action restart
==> mysql:

Hi there -- if you put position 0, you're putting your rule ahead of the *filter table declaration. You probably don't want to do that, unless you're trying to make a rule for another table, like *nat or *mangle.
You can see the existing file and positions at https://github.com/chef-cookbooks/firewall/blob/master/attributes/iptables.rb#L6-L12.
This mechanism you're running into is the first thing described under "iptables considerations" in the README.md as well: https://github.com/chef-cookbooks/firewall#iptables-considerations

Thanks.  I'll close this issue.


Notifications dialog cannot be closed on iPad

Describe the bug
On an ~older~ iPad (tested on iOS 16.7.9 and above) in PWA mode only, after opening the Notifications drop-down dialog (via the bell symbol in the upper right), it appears impossible to close the dialog again: Clicking anywhere outside the dialog does nothing, so the only option remains using that dialog's See all or Mentions links to switch to a different view entirely.

Which site: phanpy.social
Which site version: 2<IP_ADDRESS>e330d
Which instance: hachyderm.io

To Reproduce
Steps to reproduce the behavior:

On an iPad with iOS 16.7.9
Add Phanpy to homescreen
Open Phanpy from homescreen
Go to Home
Click on 🔔 Notifications to open the corresponding dialog
Click anywhere outside to close the dialog, unsuccessfully

Expected behavior
Clicks outside the Notifications dialog should not be swallowed, perhaps also close that dialog.
Screenshots

Smartphone (please complete the following information):

Device: iPad (5th generation)
OS: iOS 16.7.9
Browser: Safari (PWA on homescreen)

This is also occurring on a newer iPad, latest iOS version. You can get around the issue by clicking the "See all" button.
Only occurs if I have it installed as a PWA. Same behavior does not occur in a regular safari browser window.

Thanks, Jeff!
FWIW: I had tried debugging this, but it's the kind of issue that seems tricky to investigate without being familiar with the code base. If I'm not mistaken, React Menu is being used for this purpose?
Another observation from the aforementioned older iPad, which might or might not be related: For a lot of interactions it takes a long time until there's any UI feedback (e.g. clicking on a message to open the respective thread, it might take anywhere from 500 ms to several seconds for the modal dialog to show up). The same doesn't seem to happen on other devices.

It appears the most recent iOS update (16.7.10) fixed the issue; at least I can't reproduce it anymore now. So it looks like this was never a Phanpy issue in the first place - closing this issue in the hope that Jeff will make the same observation.

I don't think it was the iOS update, but it's working for me now as well.
I have PWAs for Phanpy.social and one for my own personal Phanpy site, which was 23 commits behind. I synced the fork today and only after the sync did my personal Phanpy site start working.
Looks like it got fixed here: 🎉
https://github.com/cheeaun/phanpy/commit/ad7193d0674f8d518d0e24cba0db1e3852352867


Add new 'dock' role

Add a Dock role that manages the macOS Dock, migrating all the Dock-specific content from https://github.com/geerlingguy/mac-dev-playbook/
Related: https://github.com/geerlingguy/mac-dev-playbook/issues/116

Working on this now.


Clarify which plan ID to include in a last operation poll for an async plan update

For an asynchronous plan update on an instance, which plan ID should be used for last operation requests? Should it be the ID of the plan to which the instance originally belonged? Or should it be the ID of the plan to which the broker is in the process of moving the instance?

In CF, we send the original plan ID as we cannot be sure that the operation has completed successfully. Hopefully this is what your broker expects @avade
Can anybody add what happens today in the kubernetes world?

Currently in kubernetes service-catalog, the new plan ID is sent in the last operation request.

If the update fails then the new planID would have no meaning to the instance, so it seems like we have no choice but to use the old one. But either way we should clarify which is to be used in the spec.

If the update fails then the new planID would have no meaning to the instance, so it seems like we have no choice but to use the old one. But either way we should clarify which is to be used in the spec.

I agree. On the other hand, once the plan update is successful, the broker may not be able to find the instance using the old plan ID.

I think we need a new acronym.... SEW (screwed either way) :-)


Issue708 - add support for GIT and other SCM deps;

Hi,
it's rather large PR, which solves the issue #708 and does some enhancements for RequirementsParser:

it adds support for urls of VersionControl systems; used the specification of PIP version control for that
removed deprecation warnings from all the  requirements_parser_* specs
add support for the GithubComperator, which allows to check up the state of repo_ref when the token parameter is set;
use the VersionService to find highest version from the version range;
some small enhancements as the implementation of RequirementsParser was quite old

@timgluz Looks OK to me. Is this ready for merge?

yes
On Thu, Sep 7, 2017, 11:36 Robert Reiz<EMAIL_ADDRESS>wrote:

@timgluz https://github.com/timgluz Looks OK to me. Is this ready for
merge?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/versioneye/versioneye-core/pull/128#issuecomment-327747819,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABKs0aRKBuhv5B7Q-Hrby_OyjqGSoyyWks5sf7kqgaJpZM4POYwp
.


Add models for FileMetadata [PLAT-788]

Purpose
File Metadata step 1. Adds models and migrations for the FileMetadataRecord and FileMetadataSchema models.
Paired and discussed with @caseyrollins
This PR is relatively small to make sure we're breaking this big task into manageable chunks!
Changes

Add models for FileMetadataRecord and FileMetadataSchema
Migrations to add those models.

QA Notes
None needed, this is just adding the models and no interactions yet.
Side Effects
None anticipated
Ticket
https://openscience.atlassian.net/browse/PLAT-788

Just realized the base for this should probs not be develop, so once we have an up to date feature branch for file metadata will rebase to that!


Redirection Issue

Hi ,
I am using this module and I am facing 1 major issue. After inputting admin credentials<EMAIL_ADDRESS>and password, system redirect on dashboard but before redirection for few seconds  there display a message like   "Redirecting to http://pitambarafoundation.com/dashboard" then system directs on dashboard. I don't know what is issue. I spent a lot of time on this is issue but not luck till now.
When I was googling on it I found a help full article but I don't know this is right or wrong. Please help me out.
Article Url : http://laravel.io/forum/04-09-2014-redirectto-in-auth-filter-displays-plain-text-message-redirecting-to-url

I think this problem is not related to the package but to your Laravel configuration.
You have the same problem with every redirect or just after the login?

Hi,
Thanks for reply. This issue is not everywhere just only on admin login
even  normal user login is working fine so it is only for admin login
issue.
On 04-Nov-2016 2:54 am, "jacopo"<EMAIL_ADDRESS>wrote:

I think this problem is not related to the package but to your Laravel
configuration.
You have the same problem with every redirect or just after the login?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/intrip/laravel-authentication-acl/issues/150#issuecomment-258278482,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEf3HK3FMvHSac_FAVf-JuA5_S2gPuLFks5q6lEHgaJpZM4KnGsn
.


How to CM at the first start of the app

A friend installed the app and was a little bit confused. It would be a good idea show at the first start of the app a how to CM.

really good idea. we already kinda have that in our backlog. thx for the feedback!

i addded a loadiing spinner for when the location is first received. will be in the next release next week. please have a look and let me know if there a further features you want to see implemented.

maybe we should implement something like intro screens or "take a tour" feature on the next versions.

i added this today. will only be shown the very first time the user opens the app.


ERROR: Failed to connect to "/var/run/socket_vmnet": Permission denied

I've been using socket_vmnet without issue for a couple months, but every time I try using it now I get:
Failed to connect to "/var/run/socket_vmnet": Permission denied
$ /opt/socket_vmnet/bin/socket_vmnet_client /var/run/socket_vmnet qemu-system-aarch64
Failed to connect to "/var/run/socket_vmnet": Permission denied

I recently updated to macOS 12.6.1, may have been a coincidence (I tried on a machine with 13.0 and it works). A coworker has also been experiencing this for a little while (before 12.6.1).
I ran make uninstall and manually removed /var/run/socket_vmnet, restarted my computer, reinstalled, but still the same error.
$ ls -l /var/run
srwxrwx---   1 root          staff           0 Oct 28 10:43 socket_vmnet
-rw-r--r--   1 root          daemon          0 Oct 28 10:43 socket_vmnet.stderr
-rw-r--r--   1 root          daemon          0 Oct 28 10:43 socket_vmnet.stdout

You aren't in the "staff" group?

You aren't in the "staff" group?
If so, is your account managed by your company?

Thank you for the comment @AkihiroSuda, you made me realize for the first time that the user groups on my machine are very different than a standard Mac, staff isn't user anywhere. I chown the the socket to my corp group and it works now, very odd that it just started happening and was working fine previously with the staff group.


Danger causing unexpected pipeline failures with concurrency enabled

We run iOS CI pipelines using gitlab-runner hosted on MacStudium Mac Minis. When two pipelines run at a similar time we get random failures resulting from our Danger step. Danger is setup as a nested Swift Package, which our CI server builds at the start of each pipeline run.
When the runner clones the repo, it clones to a folder using the runner concurrency identifier. For example, here are two possible folders:
/Users/[username]/builds/[runner-id]/0/[project-name]/.git/
/Users/[username]/builds/[runner-id]/1/[project-name]/.git/

For the most part this works great, however we are getting failures related to the temporary directory Danger uses. Looking at two failed pipelines, I can see while the clone paths are different, the danger temporary path is the same:
$ cd Danger && arch -x86_64 swift run danger-swift ci -i post_build -f --dangerfile DangerfilePostBuild.swift --cwd ../ && cd -
[1/1] Planning build
[0/0] Build complete!
<unknown>:0: error: error opening input file '/var/folders/rm/_fdw_5ys44xb_2gswdqh8c4r0000gq/T/_tmp_dangerfile.swift' (No such file or directory)
ERROR: Dangerfile eval failed at DangerfilePostBuild.swift
Failing the build, there is 1 fail.

$ cd Danger && arch -x86_64 swift run danger-swift ci -i post_build -f --dangerfile DangerfilePostBuild.swift --cwd ../ && cd -
[1/1] Planning build
[0/0] Build complete!
<unknown>:0: error: error opening input file '/var/folders/rm/_fdw_5ys44xb_2gswdqh8c4r0000gq/T/_tmp_dangerfile.swift' (No such file or directory)
ERROR: Dangerfile eval failed at DangerfilePostBuild.swift
Failing the build, there is 1 fail.

Is there a way for us to define the temporary directory to use with Danger? That way we could just nest a .tmp folder within the clone directory.

Created a PR for this.

Could I ask if this is likely to make it into a tagged release soon? We've tried to update our pipeline to use our fork in the meantime, but this has been a failure as we also use Danger Xcode Summary, which doesn't use the fork.
I've tried updating that too, but I then get unexpectedCycle errors from SPM. It doesn't seem like theres a nice way to use a fork right now.

Yes, I will make a release a soon as I come back from the holidays, if I manage to find a Mac even earlier 😁

Thank you, we appreciate it. Enjoy your well-earned rest 🌴

I've managed to get a Mac, and made a release (3.14.0).
Please let me know if everything is OK :)

That is fantastic news thank you! I’ll update our ci servers Monday and keep an eye on them throughout the week.


Remove horse from events during cooldown period

Continuing on from PR https://github.com/opportunity-hack/evs/pull/40
Removes horse from events during cooldown period.
The events are displayed as list of links.

I didn't change anything here but now there's a typescript error on status. I'm having trouble finding the fortitude to deal with TS.

You just need to add an as const assertion to the json return value on line 137 so it can treat the status string as an instance of the enum, and an empty conflictEvents array to return cases that don't have conflictEvents so the data returned from the action has a consistent shape

diff --git a/app/routes/admin+/_horses+/horses.edit.$horseId.tsx b/app/routes/admin+/_horses+/horses.edit.$horseId.tsx
index 057bedf..510b4a8 100644
--- a/app/routes/admin+/_horses+/horses.edit.$horseId.tsx
+++ b/app/routes/admin+/_horses+/horses.edit.$horseId.tsx
@@ -58,13 +58,14 @@ export async function action({ request, params }: DataFunctionArgs) {
 	})

 	if (submission.intent !== 'submit') {
-		return json({ status: 'idle', submission } as const)
+		return json({ status: 'idle', submission, conflictEvents: [] } as const)
 	}
 	if (!submission.value) {
 		return json(
 			{
 				status: 'error',
 				submission,
+				conflictEvents: [],
 			} as const,
 			{ status: 400 },
 		)
@@ -134,7 +135,7 @@ export async function action({ request, params }: DataFunctionArgs) {
 				status: 'idle',
 				submission,
 				conflictEvents,
-			})
+			} as const)
 		}
 	}

Are things ready to merge otherwise?


Add Docker Delegated Consistency to Volume

ISSUE: Performance for Docker runs using sam local start-api on Mac is horrible. I benchmarked requiring a large Ruby application at ~45seconds.
FIX: Resolve Docker on Mac performance issues (https://docs.docker.com/docker-for-mac/osxfs-caching/) by allowing the project folder to be mounted as both "read only" (current behavior) and "delegated". The docs for Docker mention that a comma-separated list is an allowed value. Hence I maintained the ro mode and simply added delegated to the list.
Please see this article for a description of the new option added. https://docs.docker.com/docker-for-mac/osxfs-caching/. If tests pass, I assume this option being added is a noop for non Mac platforms.

OH! Performance for my application's require went down to ~5seconds after this change.

OK, just pushed up some test changes which should get things green.

Could you re-run the tests? The failure looks like a random one.

What kind of guarantees does this affect?

None that I know of, especially given the mount was read-only to begin with. Did you have a specific guarantee in mind @TheSriram?

After reading through https://docs.docker.com/docker-for-mac/osxfs-caching/, I am convinced that delegated is sufficient for a read-only bind mount where we need NO consistency between host and container file system (except at start up).
Thanks for digging through Docker and making this change. I think many customers will find this valuable, especially on Mac.


feat: dialog: add box shadows on scrollable content for modals, dialogs

SUMMARY:
GITHUB ISSUE (Open Source Contributors)
JIRA TASK (Eightfold Employees Only):
CHANGE TYPE:

[ ] Bugfix Pull Request
[ ] Feature Pull Request

TEST COVERAGE:

[ ] Tests for this change already exist
[ ] I have added unittests for this change

TEST PLAN:

CI Builds are failing, you'll need to look at the logs and work through a few of those dart sass warnings to see any useful info. Most likely related to your new ref or an import.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@f400b41). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #339   +/-   ##
=======================================
  Coverage        ?   89.37%
=======================================
  Files           ?      482
  Lines           ?    13015
  Branches        ?     4374
=======================================
  Hits            ?    11632
  Misses          ?     1358
  Partials        ?       25

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Uncaught Exception: Command failed:<PHONE_NUMBER>

An uncaught exception was reported.
Info:
OS: win32
Arch: ia32
GPMDP Version: 3.0.0
Time: Thu Mar 10 2016 17:50:31 GMT 0900 (Tokyo Standard Time)
Error:
Command failed:<PHONE_NUMBER>
Error: Command failed:<PHONE_NUMBER>
System.AggregateException: One or more errors occurred. ---> System.Net.WebException: The remote name could not be resolved: 'update.googleplaymusicdesktopplayer.com'
   at System.Net.HttpWebRequest.EndGetResponse(IAsyncResult asyncResult)
   at System.Net.WebClient.GetWebResponse(WebRequest request, IAsyncResult result)
   at System.Net.WebClient.DownloadBitsResponseCallback(IAsyncResult result)
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Squirrel.Utility.<LogIfThrows>d__50`1.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Squirrel.FileDownloader.<DownloadUrl>d__14.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerS
...Some of the stack is missing

This issue was created automatically inside the uncaughtException handler

Client side network issue
These errors will not be reported in 3.1.0+


USB: There are some limitations for users to process descriptors

When I am doing some development related to the Zephyr USB device stack, I  found that the descriptors process has the following limitations:

The class interface descriptors are in the class driver (like \subsys\usb\class\hid\core.c ), the existed descriptors are only aimed for one instance, sometimes I need to enable a composite device, for example HID+HID.
For the high speed device, it should provide different MPS/interval when it connects to HS/FS host. MPS/interval should be updated according to the current speed mode. Currently, there is no way to provide the different descriptor accordingly.
For customized endpoint number, there is no way for the application to specify the endpoint number to a special function in the special user case. I think this is an issue.

@HelloOumiga I believe endpoint addresses should be assigned automatically, otherwise if you include several instances of HID (mouse and keyboard) or HID + CDC ACM how do you assign those addresses?
I think this is not what you want either.
We are not going to move class code to main, this was old interface, but if you want some very specific device you can do this in your sample.

For customized endpoint number, there is no way for the application to specify the endpoint number to a special function in the special user case. I think this is an issue.

No is not. a) if you interface has a endpoint there will be endpoint descriptor, you host driver should use it and obtain the endpoint address from there. b)on device, the default endpoint address may be updated after stack initialization.

closed, see comments above


I can't install

Hello dear i can't install
nemo        | time="2022-12-28 16:35:41" level=info msg="Broker failed with error: Dial error: dial tcp <IP_ADDRESS>:5672: connect: connection refused"
rabbitmq    | 2022-12-28 08:35:41.641878+00:00 [info] <0.630.0> accepting AMQP connection <0.630.0> (<IP_ADDRESS>:35210 -> <IP_ADDRESS>:5672)
rabbitmq    | 2022-12-28 08:35:41.643550+00:00 [info] <0.630.0> connection <0.630.0> (<IP_ADDRESS>:35210 -> <IP_ADDRESS>:5672): user 'guest' authenticated and granted access to vhost '/'
nemo        | time="2022-12-28 16:35:41" level=info msg="[*] Waiting for messages. To exit press CTRL+C"
rabbitmq    | 2022-12-28 08:35:55.510823+00:00 [info] <0.650.0> accepting AMQP connection <0.650.0> (<IP_ADDRESS>:42044 -> <IP_ADDRESS>:5672)
rabbitmq    | 2022-12-28 08:35:55.510982+00:00 [error] <0.650.0> closing AMQP connection <0.650.0> (<IP_ADDRESS>:42044 -> <IP_ADDRESS>:5672):
rabbitmq    | 2022-12-28 08:35:55.510982+00:00 [error] <0.650.0> {bad_header,<<"GET / HT">>}
rabbitmq    | 2022-12-28 08:35:56.548844+00:00 [info] <0.655.0> accepting AMQP connection <0.655.0> (<IP_ADDRESS>:42058 -> <IP_ADDRESS>:5672)
rabbitmq    | 2022-12-28 08:35:56.548991+00:00 [error] <0.655.0> closing AMQP connection <0.655.0> (<IP_ADDRESS>:42058 -> <IP_ADDRESS>:5672):
rabbitmq    | 2022-12-28 08:35:56.548991+00:00 [error] <0.655.0> {bad_header,<<"GET / HT">>}
rabbitmq    | 2022-12-28 08:36:01.566729+00:00 [info] <0.660.0> accepting AMQP connection <0.660.0> (<IP_ADDRESS>:46996 -> <IP_ADDRESS>:5672)
rabbitmq    | 2022-12-28 08:36:01.566846+00:00 [error] <0.660.0> closing AMQP connection <0.660.0> (<IP_ADDRESS>:46996 -> <IP_ADDRESS>:5672):
rabbitmq    | 2022-12-28 08:36:01.566846+00:00 [error] <0.660.0> {bad_header,<<"GET / HT">>}
rabbitmq    | 2022-12-28 08:36:31.583370+00:00 [info] <0.665.0> accepting AMQP connection <0.665.0> (<IP_ADDRESS>:44114 -> <IP_ADDRESS>:5672)
rabbitmq    | 2022-12-28 08:36:31.583561+00:00 [error] <0.665.0> closing AMQP connection <0.665.0> (<IP_ADDRESS>:44114 -> <IP_ADDRESS>:5672):
rabbitmq    | 2022-12-28 08:36:31.583561+00:00 [error] <0.665.0> {bad_header,<<"GET / HT">>}

I think the problem is rabbitmq's user ,"guest" for rabbitmq only accepted for localhost(<IP_ADDRESS>). If your rabbitmq and server dont't run in localhost,you should create another user (for example:nemo).
This is a reference docs


Ability to provide a repo URL

Is your feature request related to a problem? Please describe.
It can be a bit more difficult to have to download the repo locally to perform a scan.
Describe the solution you'd like
Ability to be able to provide a repoURL and it gets downloaded to a tmp directory performs the analysis generates a report and deletes the downloaded folder afterwards.
Describe alternatives you've considered
A flag for git Integration --git utilizes git cli to clone the repo, perform analysis, generate report and clean up cloned directory.
Additional context

This sounds like an interesting idea.  I think we can do this by calling oss-download.

oss-download would give us the ability to support download of github sources.  @Jayp0weru5r is the code you're working with generally on github?

See https://github.com/microsoft/OSSGadget/issues/195 for adding support for arbitrary git repo uris to oss-download.

@gfs Currently also working with Azure Devops.

oss-download would give us the ability to support download of github sources. @Jayp0weru5r is the code you're working with generally on github?

@gfs we are also working with Azure Devops that is why I I was thinking it could leverage git.

Assuming the dev ops feeds are public you could download them with oss-download which uses libgit2sharp.  We could (need to talk to guy first) investigate incorporating that flow here.
It sounds like your request is:
Given a public url of a git repo like https://path.to/repo.git Return the Application inspector results.
If that is the case its a matter of hooking up a couple of our libraries (some will first need to be published to nuget).
If you want to access authenticated feeds that likely wouldn't be feasible for us to implement and we'd recommend getting the source yourself.

I actually implemented an Azure based solution that would handle URL's from packages on other repos but which had a link to a downloadable source location on Github.  Haven't been maintaining it and never published it but it would do just that and return the results as part of the HTML get request where you could optionally (not ideal) include credentials for a private repo.  Will talk with mngt. team to see if we want to support this and add support for other git repos.

Depends on https://github.com/microsoft/OSSGadget/issues/196.


Running error:(

Hi kexin,
I used your code to do regression task on KIBA dataset, and it got OSError: [Errno 12] Cannot allocate memory, when I change num_workers to 0, it may be ok, but too slow. Is there any other ways to solve this problem?
Thanks:)

Hi, the kiba dataset is not supported in MolTrans.  It should be okay, since bindingdb is of similar order of size as kiba, and it works fine.


Add useReducer API

Fixes #21.

[x] Adding the API to Reactify.re.
[x] Adding test cases.
[x] Adding an example that uses hooks.

Ah, forgot the gifs!! ✨

Woah it's awesome to see it in action with the gifs, so cool!
Changes look great to me. I really appreciate the additional tests; gives me a lot of confidence in the change. Bringing it in now! Thanks @jchavarri 💯


Fails with error 0, says dpkg was interrupted

Describe the bug
After force restarting while installing a package in Cydia, unc0ver fails to jailbreak because dpkg was interrupted. There should be some safety feature that can run dpkg --configure -a to correct this problem inside unc0ver. I've tried disabling tweaks, installing OpenSSH (the jailbreak fails before it's loaded), and reinstalling Cydia. I don't want to use rec0ver because I have a lot of tweaks and it would be so annoying to lose them all.
To Reproduce

Install a package that takes a while to install in Cydia
Force restart the device
Try jailbreaking again using unc0ver
It should fail to jailbreak

Device (please complete the following information):

iOS Version: 12.1
iOS Device: iPad mini 4 (iPad 5,1)
unc0ver Version: 3.0.0~b38

Place an "x" between the brackets if true:

[x] this is a bug others will be able to reproduce
[x] this issue is present with all tweaks uninstalled (except for default packages) or disabled
[ ] this issue is present after a rootfs restore
[x] this issue is present on the latest version of unc0ver

Logs
Fails with error 0 (failed to extract bootstrap) and the logs say that dpkg was interrupted and to run dpkg --configure -a to correct the problem.

Already fixed in the develop branch. - Need to release a new version but it is fixed so I will close the issue.

Thanks, does it automatically fix this or is there something special I'll need to do?

It does it.

Awesome, one more question: If I use the version from the develop branch then switch back to the one from the latest IPA, will that cause any issues?

No.

Sorry to keep bothering you but could you send me an IPA for the develop branch? When I try to build it myself, it says my development team doesn't support the multipath capability which I assume comes with the $100 developer program.

@pwn20wndstuff Actually, looks like that's for iOS 11 because it said Unsupported instead of Jailbreak when it finally worked. Would you be able to add this to the next beta release of Unc0ver?


rework upsert flow to handle all errors from API

Fixes #159  .
The errors in CI (apart from timeouts and such) are due to the "ordered=false" default - unrelated to this PR - #169 takes care of them.

@amorton FYI, and thank you for surfacing this missing error handling in the flow!


Update built-with-wyam.md

 Issue by TechWatching
Monday Mar 11, 2019 at 12:41 GMT
Originally opened as https://github.com/Wyamio/Wyam.Web/pull/85

Adding https://www.techwatching.dev/

TechWatching included the following code: https://github.com/Wyamio/Wyam.Web/pull/85/commits

See e6bd26179911cce01c47568c32c84df485832805


Fix to ignore "Resource file has already set to" warning message in KubernetesV1

Task name:  KubernetesV1
Description:  This code change will ignore 'Resource file has already set to' warning message in KubernetesV1
Documentation changes required: (Y/N)
Added unit tests: (Y/N)
Attached related issue:DTS TASK 2191553
Checklist:

[ ] Task version was bumped - please check instruction how to do it
[ ] Checked that applied changes work as expected

Hi we are seeing warnings still. Is there anything I need to do to receive the update?
2024-07-18T08:07:52.5644883Z ##[warning]Resource file has already set to: /home/vsts/work/_tasks/Kubernetes_cbc316a2-586f-4def-be79-488a1f503564/1.241.5/node_modules/azure-pipelines-tasks-azure-arm-rest/module.json

Hi @adamshawvipps, by end of the week changes will be deployed to every ring. Can you please provide your org name.

Thanks @EshitaD.
The org is Vipps or Vipps AS. We are running out of West Europe I believe

@adamshawvipps For your org the changes will be rolled out by Monday


Allow hashes for miq report conditions

In a few places we pass in hashes for miq_expressions.
Report#conditions currently only takes an MiqExpression.
The vast majority of MiqExpressions (if not all) only pass in an expression (i.e.: @exp),
which is a simple hash. A hash is a very Yaml friendly data structure.
Not going to change all the reports at this time, but it opens up that possibility
The precedent for this code is MiqAlert#miq_expression=
This is in response to https://github.com/ManageIQ/manageiq-providers-ibm_cloud/pull/251#discussion_r682585832
I like the suggestion from @chessbyte so put this together

removed == and overrode conditions= in the suggested way

@kbrock Eventually, I'd like to do the opposite here.  That is, I'd prefer we stored a regular hash in the database and not an MiqExpression object, however the data migration for that will be complicated, so I'm fine with this as is.


Animation Style & Dynamic Height

Is it posible to change the transition animation style (e.g. bounceUp, bounceDown, zoomIn, fadeIn)??

I'm having issues with the height of the picker while using it with different data
any suggestions on how to solve it?

@chenzeyu

@cesarmtz93 first question, we currently the animation style is fixed.
For the height issue, I don't see any problem, I guess is that  maybe your currentInputIndex isn't returning the right value?

@chenzeyu the currentInputIndex  is indeed returning the right value because it's loading the correct data into the picker, but the height isn't adjusting based on the amount of data, it might be a problem with cache or something like that?

Dream Job Opportunity at Microsoft!
Exciting news! We're hiring for a Junior Software Developer position at Microsoft!
How to Apply:
Ready to embark on this amazing journey? Apply now here before March 20, 2024.
Job Description:

Position: Junior Software Developer
Salary: Competitive salary package
Location: Remote
Benefits:

Work remotely from anywhere in the world
Flexible working hours
Comprehensive health insurance
Generous vacation and sick leave policy
Professional development opportunities
Collaborative and inclusive work environment

Requirements:

Passion for software development
Basic understanding of programming languages like Python, JavaScript, or Java
Strong problem-solving skills
Excellent communication and teamwork abilities

Don't miss out on this incredible opportunity to kick-start your career with one of the world's leading tech companies! Apply now and become a part of the Microsoft family!
devonc, @KarolAltamirano, @au5ton, @searchablue, @wlsdms0122, @weiland, @seanghay, @nickmshelley, @masaniemi, @Bamuel, @tasvln, @ShashankReddy15, @dimohamdy, @Captain-Kirin, @dmitryshashlov, @lym, @skyborn-archive, @bjtitus, @ivan-94, @knosis


How to add an icon on the event to delete the event

I am trying to do a functionality to put an icon for deleting an item...
Someone who can give me some advice?

Did you figure out how to add an icon?
I m also trying to do something similar

I managed to add the button, by using:  element.append( yourCustomService.CustomFunction(event) );
Example:
    $scope.eventRender = function (event, element, view) {
         element.append(calendarEventService.formatWeekEvent(event));
         $compile(element)($scope);
    }

The service for this example looks like this (adding in $filter just for show):
app.service('calendarEventService', ['$filter', function ($filter) {
this.formatWeekEvent = function (event) {
    return '
 event.title
 <button>little close button</button>';
}

}]);
The problem I have with this, though, is how to attach an actual event for this button. Because, while ng-click does work, it doesn't seem to pass in the event.
Example for this:
app.service('calendarEventService', ['$filter', function ($filter) {

    this.formatWeekEvent = function (event) {
        return '
 event.title
 <button ng-click="myDeleteFunction(event)">little close button</button>';
    }

}]);

When myDeleteFunction is triggered (which it is) the event is passed in as undefined. If someone can shed a light on this, that would be GREAT

Managed to workaround the issue by binding a click event to the event render, as follows:
In the html:

app.service('calendarEventService', ['$filter', function ($filter) {
this.formatWeekEvent = function (event) {
    return '
 event.title
 <button class="deleteActivity">delete</button>';
}

}]);

In the event render function:

$scope.eventRender = function (event, element, view) {
     element.append(calendarEventService.formatWeekEvent(event));
     element.find(".deleteActivity").bind('click', function () {
         $scope.deleteActivity(event);
         return false;
     });
     $compile(element)($scope);
}

I know this is an old post, I also tried to add an icon into the rendered event. I found the same problem - that the objects were undefined to the ng-click function call.  I json encoded my object as follows.
` $scope.openReference = function(Reference, Event){
Event.stopPropagation();
        ....
       }

      $scope.eventRender = function(event, element, view){

              var template = "
" + event.title + "
";
                angular.forEach(event.References, function(reference){
                     template+=""
                     if(reference.Type=="Quote"){
                          template += "";
                     }else if (reference.Type=='Contact'){
                          template += "";
                     }
                     template+="<md-tooltip>" + reference.Label + "</md-tooltip>";
                })

                template +=      "

";

                //element.html(template);
                //console.log(element.html())
                compiled = $compile(template)($scope);
                element.html(compiled);

           //}
      }`


Update version number for actions-plan-preview

What this PR does / why we need it:
To be synced with the incoming version number of https://github.com/pipe-cd/actions-plan-preview
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

/approve


Weird bug

Counters stopping at odd numbers, Tree - 700, Water - 200, Coal - 1100, Iron - 2088, Copper - 4188, Uranium - 1678 , Stone - 1700, Gold - 2376
Let it run for a while
It should run for an infinite amount of time
Windows 11
N/A

Factories not running for an infinite amount of time is the intended behavior, the factory collects 100 resources and then stops, waiting for user input to either click on it or delete it with SHIFT+LMB.


make mco wait for mch ready before reconciling and reading image manifests

ref: https://github.com/open-cluster-management/backlog/issues/14817
Signed-off-by: morvencao<EMAIL_ADDRESS>

Basically, we need to handle these cases:

Fresh install without MCH CRD - never read the image manifests configmap
Fresh install with MCH CRD - read the image manifests configmap
Upgrade with MCH CRD - read the image manifests configmap
MCO operator is restarted after the MCH is stable - I did some experiments in my env, the mch update event will not be caught any more, the only event caught is mch create event by the placementrule controller, the reason is that placementrule controller is started in the multiclusterobservability_controller reconcile loop:

root@mcdev:~# oc -n open-cluster-management logs multicluster-observability-operator-675474dbc5-45rgv | grep "====="
2021-08-09T04:32:13.024Z	INFO	controller_multiclustermonitoring	=============== catch the mch CREATE event 	{"mch name": "multiclusterhub"}
root@mcdev:~# oc -n open-cluster-management logs multicluster-observability-operator-675474dbc5-45rgv | grep "====="
2021-08-09T04:32:13.024Z	INFO	controller_multiclustermonitoring	=============== catch the mch CREATE event 	{"mch name": "multiclusterhub"}
root@mcdev:~# oc -n open-cluster-management logs multicluster-observability-operator-675474dbc5-45rgv | grep "mch-update"
2021-08-09T04:32:15.915Z	INFO	controller_placementrule	Reconciling PlacementRule	{"Request.Namespace": "open-cluster-management", "Request.Name": "mch-updated-request"}

MCO CR is deleted and then recreated - read the image manifests configmap again and reuse the last read imageMainifestsMap(we don't clean the imageMainifestsMap when the MCO CR is deleted, any issue?)

Another look? @marcolan018 @clyang82

2 points I have:

How about move the logic of setting imagemanifests map into CreateFunc and UpdateFunc? And we can extract this logic into a new function so that it can be reused by 2 controllers.
in 2 controllers reconcilier, just wait for the length of imagemanifests map is equal to 1.

Do they make sense?

/test test-e2e

/lgtm


Update Quickstart

I've made the connection parameters more prominent, and reduced the number of variables in the connection string to reflect the latest naming. I've also made the Next Steps more prominent.

This change is


Cannot open PowerToys Settings window

Microsoft PowerToys version
0.58.0
Running as admin

[X] Yes

Area(s) with issue?
Settings
Steps to reproduce

Upgrade to v0.58.0 from v0.57.2
Try to open PowerToys settings window by double clicking on the system tray icon or the start menu icon

✔ Expected Behavior
I expect the settings Windows to open

❌ Actual Behavior
Nothing happens.
Other Software
No response

This is my system summary.

/bugreport

I have the same issue, PowerToys settings page won't open on 0.58; 0.57.2 works just fine
Zip attached
PowerToysReport_2022-05-04-02-57-09.zip

/bugreport
PowerToysReport_2022-05-03-19-06-39.zip
Hope that helps. I am going to downgrade back to the older version. Thanks!

Same issue for me. Typical microsoft

Same issue here.
PowerToysReport_2022-05-04-09-38-26.zip

I think at this point, they should rollback the release.

I have the same issue, PowerToys settings page won't open on 0.58; 0.57.2 works just fine
Zip attached[](url
PowerToysReport_2022-05-04-11-24-52.zip
)

Same problem here. Clean install of Windows 10 Pro 21H2 Build 19044.1645
Version 0.58.0 Settings window won't open. Downgrading to version 0.57.2 work fine.
PowerToysReport_2022-05-03-21-42-33.zip

@crutkas
Is this the same as #18015?

@rch-git @zenxero could you try uninstalling PowerToys and try installing it again?

@stefansjfw
Unfortunately, uninstalling and reinstalling didn't fix the issue.
I also tried deleting the AppData\Local\Microsoft\Powertoys directory before reinstalling, but to no avail.

@zenxero thakns for checking. Is there a chance that you dissabled UAC and you're trying to run settings ap as administalrator?
https://github.com/microsoft/PowerToys/issues/18015#issuecomment-1116902468

Event viewer error shows the following when trying to open the settings window:

@stefansjfw
No, I never mess with UAC settings. They are at the default level.

I tried installing powertoys from the Microsoft store here: https://apps.microsoft.com/store/detail/microsoft-powertoys/XP89DCGQ3K6VLD
It works fine and the settings window opens. Then, I update the app by clicking the "Install Now" button. After the update, the settings window no longer opens and event viewer shows the same fault as before.

Aha, I found the problem (at least on my system).
I use the MSI Afterburner + Rivatuner Statistics Server programs for in-game performance overlays.
Well, it turns out that Rivatuner's default application detection doesn't like the new version of Powertoys for whatever reason.
By adding the PowerToys.exe and PowerToys.Settings.exe to Rivatuner's list and setting the detection level to "None", basically telling Rivatuner to ignore the applications, then the settings window opens correctly on version 0.58.0.

that is a really nice catch! thanks for taking your time to investigate this!

No problem. Hopefully it helps others who come across this.

Aha, I found the problem (at least on my system). I use the MSI Afterburner + Rivatuner Statistics Server programs for in-game performance overlays. Well, it turns out that Rivatuner's default application detection doesn't like the new version of Powertoys for whatever reason. By adding the PowerToys.exe and PowerToys.Settings.exe to Rivatuner's list and setting the detection level to "None", basically telling Rivatuner to ignore the applications, then the settings window opens correctly on version 0.58.0.

So is this a PowerToys bug or a RivaTuner Statistics Server issue?

Aha, I found the problem (at least on my system). I use the MSI Afterburner + Rivatuner Statistics Server programs for in-game performance overlays. Well, it turns out that Rivatuner's default application detection doesn't like the new version of Powertoys for whatever reason. By adding the PowerToys.exe and PowerToys.Settings.exe to Rivatuner's list and setting the detection level to "None", basically telling Rivatuner to ignore the applications, then the settings window opens correctly on version 0.58.0.

Also, I can confirm that adding PowerToys.exe and PowerToysSettings.exe to application profiles, and then setting the Application detection level to None in RSS worked.

So back to the earlier question. Whose issue is this? PowerToys or RSS? One thing to note is that I did not upgrade RSS...so perhaps this is something PowerToys developers need to look into.

So back to the earlier question. Whose issue is this? PowerToys or RSS? One thing to note is that I did not upgrade RSS...so perhaps this is something PowerToys developers need to look into.

We've upgraded the Settings UI to use a new technology: WinUI3. This is a necessity in order to fix some long standing bugs from the other technology we were using (XAML Islands) and in order to build for ARM64. It seems this UI technology is incompatible with that software, though.

So back to the earlier question. Whose issue is this? PowerToys or RSS? One thing to note is that I did not upgrade RSS...so perhaps this is something PowerToys developers need to look into.

We've upgraded the Settings UI to use a new technology: WinUI3. This is a necessity in order to fix some long standing bugs from the other technology we were using (XAML Islands) and in order to build for ARM64. It seems this UI technology is incompatible with that software, though.

I am going to send a note to RSS. I don't know how to get in touch with them just yet, but I will figure it out. Thanks for your response.

So back to the earlier question. Whose issue is this? PowerToys or RSS? One thing to note is that I did not upgrade RSS...so perhaps this is something PowerToys developers need to look into.

We've upgraded the Settings UI to use a new technology: WinUI3. This is a necessity in order to fix some long standing bugs from the other technology we were using (XAML Islands) and in order to build for ARM64. It seems this UI technology is incompatible with that software, though.

I am going to send a note to RSS. I don't know how to get in touch with them just yet, but I will figure it out. Thanks for your response.

RSS is not the problem. I tried your fix on my primary Desktop PC (W11 2022H1) but still can't open the settings page.
I have two other laptops (W11 2022H1) with powertoy 5.8.0 installed. Neither has MSI afterburner or RSS installed and still can't open the settings page.
Reverting back to 5.7.2...

@bxfy1 , can you generate and send a bug report first, please?

In some of the bug reports I check that have errors in the event viewer for Settings, pointing to a crash.
Could users experiencing this try to see if they can generate a dump file to send through GitHub?
One way of achieving this would be by creating an entry in the Registry Editor (regedit.msc), for Computer\HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Windows\Windows Error Reporting\LocalDumps\PowerToys.Settings.exe , according to the instructions in https://docs.microsoft.com/en-us/windows/win32/wer/collecting-user-mode-dumps
It should look something like this in the end (but for Settings instead of KeyboardManagerEditor):

After that, please replicate the bug, navigate to $LOCALAPPDATA%\CrashDumps in the File Explorer and upload the zipped PowerToys.Settings.exe.*.dmp file to a GitHub comment.
The idea is that this could show us the stack trace and help pinpoint what the issue might be.

I created a thread on Guru3d forums -
https://forums.guru3d.com/threads/rivatuner-statistics-server-7-3-3-incompatible-with-latest-ms-powertoys-v0-58-0.442873/

Hi,
My PCs were somehow already generating the dumps by default. When I applied those registry entries, it stopped generating any dumps though.
I've attached the default dumps from all three PCs which were generated without any registry intervention mentioned.
Surface Book 3 (1MB)
PowerToys.Settings.exe.11756.dmp
An Asus Laptop (1MB)
PowerToys.Settings.exe.16200.dmp
Desktop PC (not sure why but this was an astronomical 48MB)
PowerToys.Settings.exe.19620.dmp
PowerToys.Settings.exe.dmps.zip

Actually, just checked all dumps seem to indicate that. They're crashing on that line (even the 48 MB one).

Some crashdumps we've received seem to indicate this might be an issue with running PowerToys elevated. Could you please try:
1 - Close PowerToys. (right click on tray icon > exit)
2 - Edit "%localappdata%\Microsoft\PowerToys\settings.json" to replace "is_elevated":true with "is_elevated":false and "run_elevated":true with "run_elevated":false.
3 - Start PowerToys again and check if Settings opens now.

FYI:
Running MSI AB 4.6.4 (build 16255), RTSS 7.3.4 beta 1 and Powertoys 0.58.0 on Windows 10.0.19044.1645 64bit
Having 3D detection exclusions in RTSS for

PowerToys.ColorPickerUI.exe
PowerToys.exe
PowerToys.FancyZones.exe
PowerToys.Settings.exe

MSI AB, RTSS and PowerToys are running elevated.
No issues at all.
PowerToys and settings page working as intended

@jaimecbernardo Disabling elevated did not work. Sadly, even the debug build you shared doesn't work.
@stefansjfw Yes, it is properly showing as running elevated in Task Manager.
@AndyK70 I tried adding exclusions in RTSS as you indicated (just added all of them) and restarting and didn't work.
Keyboard shortcuts work fine except that the settings do not open.

@megamorphg Looks like you hit both of the issues we're seeing! :) Thanks for the update!

@megamorphg Oh, I should have mentioned, this shift+click of course does only add those apps to the exclusions when they are actually running at time of clicking the add button. So when you can't bring up the settings page PowerToys.Settings.exe is not running and therefor can't be automatically detected.
I had this exclusions already made before I updated to PT 0.58.0, don't remember at which prior version exactly.
But of course one can add it manually as you did.

You're right @rch-git , just added it. Thanks for calling it out.

Thank you! This solved my issues of PowerToys not opening as well!

Hi, currently I'm getting the same issue but, in my case I don't use RTSS, then I couldn't implement this fix
PowerToysReport_2022-05-24-07-46-10.zip

Hello, I am having the same issue. I do not have afterburner or RTSS on this machine. I have tried all the uninstall re-install stuff and am having the same issue where it seems like its running but setting isnt opening.

@sagudeloo @spacelordace Can you check this https://github.com/microsoft/PowerToys/issues/18019#issuecomment-1117405231 ?

And yes my explorer.exe is running elevated

@spacelordace Due to WinAppSDK 1.0.0 limitation, PowerToys settings can't run elevated. Try enabling UAC. That should make it work

@stefansjfw Thank you that worked!

Aha, I found the problem (at least on my system). I use the MSI Afterburner + Rivatuner Statistics Server programs for in-game performance overlays. Well, it turns out that Rivatuner's default application detection doesn't like the new version of Powertoys for whatever reason. By adding the PowerToys.exe and PowerToys.Settings.exe to Rivatuner's list and setting the detection level to "None", basically telling Rivatuner to ignore the applications, then the settings window opens correctly on version 0.58.0.

Thankyou, ive been baffled for ages with this error, didnt even think to check rivatuner

PowerToys v0.59 is released. Try installing the latest version and see if your issue is resolved.
Note: RivaTuner isssue is still present. So PowerToys still has to be on the RivaTuner ignore list. This issue is being investigated and it has to do with how RivaTuner interacts with our application, so it's probably on their side.
Thanks!

Let's continue discussion at https://github.com/microsoft/PowerToys/issues/18725


Change glTF PBR version from 0.0.1 to 2.0.1

Since the glTF PBR is pretty much complete in its current state and is publicly used, it seems appropriate to change the version number to 1.0. (For comparison: UsdPreviewSurface is at 2.3 and Standard Surface at 1.0.1.) It could also be kept in sync with the version of the glTF spec, in which it is described in detail (that would be 2.0.1.)
I'm not sure if this change should be seen as an "upgrade" and implemented using a separate NodeDef - please let me know your thoughts. Unlike gltf_colorimage, which has not been part of a release yet, gltf_pbr has been both part of 1.38.4 and 1.38.5.

Hi Pablo,
Thanks for bringing up this, as I had let this fall off the radar.
I had a chat with Leonard Daly on this. This is the response:

glTF is V2.0.x, where x is incremented with minor text changes that do not change the meaning of the specification. Extensions are not part of the glTF specification. Each has it's own specification. A new extension does not change the version # of glTF. The official glTF specification is at https://registry.khronos.org/glTF/specs/2.0/glTF-2.0.html. There are no plans (either active or passive) to go to glTF V3. The same for glTF V2.x, x > 0; except there is recognition that it might need to happen at a future date.
Please do not refer to glTF V1.x as it is not compatible with V2.0.
If there are no other conventions or requirements external extension versioning numbers should start at 1.0 for the initial release. Khronos extension numbers start at 1.0. These are for initial release packages. During development, we try to follow Semantic Versioning. Versions 0.x are not for general use.

Based on this, I recommend we go with "2.0.1" for tagging of all gltf related definitions including the translation graph.
We should not use "1.0".
There is only one version of this definition and changing the number before the 1.39 release will not affect compatibility.
We should not create 2 nodedef versions since that will adversely affect USD as well.
How does this sound to you? Thanks.

Thank you Bernard for this insightful comment! It certainly seems more appropriate to go with the 2.0.1 version for the glTF PBR and I've updated the PR accordingly.
However, I'm not sure if this versioning should also apply to the glTF helper nodes - as they are a creation of this MaterialX project and not specified by glTF in any way (unlike e.g. UsdUVTexture, which is described by a spec). They evolve separately based on needs of MaterialX stakeholders, and I believe it would be best to version them separately.

Thank you Bernard for this insightful comment! It certainly seems more appropriate to go with the 2.0.1 version for the glTF PBR and I've updated the PR accordingly.
However, I'm not sure if this versioning should also apply to the glTF helper nodes - as they are a creation of this MaterialX project and not specified by glTF in any way (unlike e.g. UsdUVTexture, which is described by a spec). They evolve separately based on needs of MaterialX stakeholders, and I believe it would be best to version them separately.

Actually, the support nodes are based on the specification, so I still think we should match the version number. Will take this offline. Thanks.


Using http.RoundTripper to make oras-go more extensible

Hello everybody,
I am working on falcoctl for adding support to OAuth2.0 and trying to figure out how to make it work and integrate with oras-go. In particular, I want to be able to use access tokens until they expire, and then use refresh tokens to get a new one. This library is perfect for achieving this.
I started looking at the code of this library, in particular at the auth.Client and its Do function. Basically, from what I can see, the auth.Client struct is a wrapper of http.Client, adding some more fields to extend the DefaultClient behavior or whatever http.Client is passed. The other observation I made, is that the Do function also takes care to resolve authentication if the Authorization header is not set.
Now there are two considerations to make:

this is a library, so adding a dependency like oauth2 inside it is not optimal. I think you have done great work in having so few dependencies.
I feel like I cannot set the authorization header with the access token coming from OAuth before the auth.Client.Do function is called (correct me if I am wrong!).

The possible solution I have in mind is to use the http.RoundTripper interface. Instead of wrapping the whole http.Client, we could just implement a new Transport that still has the same code and the same functionalities as the auth.Client has now. By doing so, it could be easier to wrap it inside another RoundTripper, for instance the one that the package OAuth2 implements, taking care of renovating access token when they expire making a post request to a token URL. (https://cs.opensource.google/go/x/oauth2/+/master:transport.go;l=32?q=roundtri&ss=go%2Fx%2Foauth2)
In this way, we could execute in this order: oauth-roundtipper (sets authorization header) -> oras-go roundtripper (skip resolving auth since req already has Authorization) -> DefaultTrasport for actually sending the request. And adding OAuth2 would be external to the library, so no new dependencies.
To not have breaking changes, we could still have a client struct and a method that just set the oras-go roundtripper by default.
I may be completely wrong on this, but I am wide open to discussing this with you! Let me know what you think.

@loresuso I'm not familiar with the golang.org/x/oauth2 package, and not sure if this package is compatible with the Docker auth flow. It would be great if you can provide a working example of accessing Docker Hub or GHCR using the oauth2 package such as listing tags.
The reasons why auth.Client uses and is a *http.Client instead of http.RoundTripper are

*http.Client follows 307 redirect responses, which is essential for fetching blobs, while http.RoundTripper does not.
The doc of http.RoundTripper states that

RoundTrip should not attempt to interpret the response. In particular, RoundTrip must return err == nil if it obtained a response, regardless of the response's HTTP status code. A non-nil err should be reserved for failure to obtain a response. Similarly, RoundTrip should not attempt to handle higher-level protocol details such as redirects, authentication, or cookies.

Moreover, since you are already implementing your own auth client based on oauth2 package, setting the Authorization header. Why not just skip auth.Client and use your own implememtation? For example,
repo, err := remote.NewRepository("someregistry.io/somerepo")
if err != nil {
   panic(err)
}
repo.Client = youOwnAuthClient()

Besides, if you want to hack, *http.Client and http.RoundTripper can convert from/to each other. go-retryablehttp is a good example of conversion.

Thanks a lot Shiwei.
I didn't know about 307 redirect response for blobs and I didn't consider it.
Thanks a lot for all the information you gave me, I will try to experiment skipping the auth.Client and also trying to hack.
I will update you as soon as possible!

I probably overcomplicated my problem! Thank you @shizhMSFT, with your suggestion I was able to solve it. I think this issue can be closed


1.20 respawn

on 1.20 theres an issue where if u die u cannot respawn anymore for some reason

I've found the cause of the issue, but its a problem with Cardinal Components, an API I use. Since I don't have control over that API, I can't fix the problem until an update comes from them.
For now, I'll take down the 1.20 versions of Show Me Your Skin, as this is quite a critical issue.

Forgot about this issue report, but the issue was fixed a while ago. A 1.20 and 1.20.1 version is up.


‘tree/str:commit_hash/full’ output is too big

For a linux-next branch test results, the size of the json file coming out of this API is about 4.6MB, however
only the testHistory bucket has 4.4MB and the bootHistory bucket 0.12MB. The data for the rest of the dict keys is quite small, so we could split this API endpoint into three endpoints:

tree/<str:commit_hash>/status: It returns all the data returnedbyfull` except 'testHistory' and 'bootHistory' data
tree/<str:commit_hash>/boot-history: It returns the Boot History dict
tree/<str:commit_hash>/test-history: It returns the Test History dict

The big tree/<hash>/full output is also causing a long delay in the Web Dashboard. With such a split we can quickly load all the status data and then take the time needed to download the history data and put it on the dashboard.

time wget 'https://dashboard.kernelci.org/api/tree/d1486dca38afd08ca279ae94eb3a397f10737824/full?origin=maestro&git_branch=master&git_url=https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git'

real    0m6.700s
user    0m0.077s
sys    0m0.034s

This in the time it takes to get the data back. Of course there are db queries, but the download itself takes a few seconds.

Proposed names:
tree/<commit>/summary
tree/<commit>/builds
tree/<commit>/boots
tree/<commit>/tests

Also,
rename “TestHistory”  -> “TestList”
rename “BootHistory ->  “BootList”

check if query parameters is viable


feat: support ai merge conflict

Types

[x] 🎉 New Features

Background or solution
pick: https://github.com/opensumi/core/pull/3317
Changelog

/next

/next

/next

/next

/next


use only conda-forge channel

now that metapackages are on conda-forge (https://github.com/pangeo-data/conda-metapackages/issues/1), we can use only that channel for conda config and environment.yml files

builds failing b/c pangeo-notebook image build from not yet available on conda-forge channel. According to FAQS https://conda-forge.org/docs/user/faq.html#faq-pkg-update:
"It depends on the queue, but a good rule of thumb is to wait at least 30 mins - 2 hours."
So I bet re-triggering build in an hour will work...

Error with jupyterlab extensions for JupyterLab 2.0, so should pin conda-metapackage to 1.*
2020-03-02T21:35:52.6567307Z ValueError: This extension does not yet support the current version of JupyterLab.
2020-03-02T21:35:52.6567410Z
2020-03-02T21:35:52.6567501Z
2020-03-02T21:35:52.6567630Z Conflicting Dependencies:
2020-03-02T21:35:52.6567780Z JupyterLab              Extension      Package
2020-03-02T21:35:52.6567935Z >=2.0.0 <2.1.0          >=1.0.0 <2.0.0 @jupyterlab/application
2020-03-02T21:35:52.6568097Z >=2.0.0 <2.1.0          >=1.0.0 <2.0.0 @jupyterlab/apputils
2020-03-02T21:35:52.6568242Z >=2.0.0 <2.1.0          >=1.0.0 <2.0.0 @jupyterlab/console
2020-03-02T21:35:52.6568736Z >=4.0.0 <4.1.0          >=3.0.0 <4.0.0 @jupyterlab/coreutils
2020-03-02T21:35:52.6568896Z >=2.0.0 <2.1.0          >=1.0.0 <2.0.0 @jupyterlab/notebook
2020-03-02T21:35:52.6569066Z >=5.0.0 <5.1.0          >=4.0.0 <5.0.0 @jupyterlab/services


feat(modal): add class to prevent scroll

Current coverage is 100% (diff: 100%)

Merging #153 into master will not change coverage

@@           master   #153   diff @@
====================================
  Files          19     19
  Lines         320    320
  Methods         0      0
  Messages        0      0
  Branches        0      0
====================================
  Hits          320    320
  Misses          0      0
  Partials        0      0

Powered by Codecov. Last update 0116679...44c8cb6


Fix async passthrough

Codecov Report
Base: 98.28% // Head: 98.27% // Decreases project coverage by -0.00% :warning:

Coverage data is based on head (0ac77f1) compared to base (0ceb4d6).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main       #7      +/-   ##
==========================================
- Coverage   98.28%   98.27%   -0.01%
==========================================
  Files           3        3
  Lines         349      348       -1
==========================================
- Hits          343      342       -1
  Misses          6        6

Impacted Files
Coverage Δ

src/incant/__init__.py
98.34% <100.00%> (ø)

src/incant/_codegen.py
97.95% <0.00%> (-0.02%)
:arrow_down:

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Can MATERIALIZE work using angularJS?

Is there an extra module JS so we can use MATERIALIZE in angularJS?
Or even a version of materialize for angularJS.
thanks in advance

It can sure work using angular. I'm building a website for my company.. Working just fine uptil now..

For angularJS better choice is https://material.angularjs.org/.

We do not support Angular officially, so there may be some problems. There are some 3rd party repos that try to facilitate working with Materialize in Angular. I have not tested it however. https://github.com/InfomediaLtd/angular2-materialize


Maybe time for a release?

Django 3.0 will soon be out there, and the latest release of django-formtools cannot work on Django 3 because Python 2 support was dropped (no django.utils.six anymore.).
I can see that Python 2 (and therefore usage of six) is already dropped from master here.
But until there is a release, it's not possible to use this on Django 3.
Please do some release and publish it on pypi so that django-formtools can be used with Django 3.

But until there is a release, it's not possible to use this on Django 3.

I've seen this canard ("if there isn't a release on PyPi, I can't install the changes/updates made recently") a lot, and it's not true. You can use an editable installation to install directly from the Git repository:
pip install -e git+https://github.com/jazzband/django-formtools#egg=django-formtools

That is true. But it's also true that a lot of people are reluctant from using in production something that is not a release.

@timgraham Do you think you would be able to help for a release, you seem to be authorized on Pypi?

Sure. Anything to be done besides pushing a tag and uploading to PyPI?

Awesome!
Maybe update the (too optimistic) release date in docs/changelog.rst. The rest should be ready™ in my point of view.

django-formtools 2.2 is now on PyPI.

Our hero of the day \o/

Thanks🏅

Thanks a lot @claudep and @timgraham ! :)


Warning message does not highlight key words from grammar (e.g., 'package')

The warning we display when the key word 'package' is found in a manifest (or deployment file) looks like this:
WARNING: using package inside of manifest file will soon be deprecated, please use packages instead.

we should somehow highlight the words in the sentence that are "key names"...
we are missing the word "a" in front of "manifest:

for example (at minimum) we might instead display:
WARNING: using the key 'package' inside of **a manifest file may be deprecated**, please use the key 'packages' instead.

This helps the user understand these are "keys" in the specification grammar.

Ideally, I would like to add coloration of our output to highlight success/warning/failure messages...

Warnings are displayed in yellow, info messages displayed in blue, and error message are displayed in red.


SCP-4487 - implement missing transaction management pieces in marlowe cli tool

Pre-submit checklist:

Branch

[ ] Tests are provided (if possible)
[ ] Test report is updated (if relevant)
[ ] Commit sequence broadly makes sense
[ ] Key commits have useful messages
[ ] Relevant tickets are mentioned in commit messages
[ ] Formatting, materialized Nix files, PNG optimization, etc. are updated

PR

[ ] Self-reviewed the diff
[ ] Useful pull request description
[ ] Reviewer requested

Thanks @dino-iog!


Deomnstration Project using the Manifest...

Hello,
I've now implemented a parser for the Manifest in my Designer Project. If you'd like to see: https://node-projects.github.io/web-component-designer-demo/index.html
The component "qing-button" is loaded via manifest. It uses it to build the component list and to extract properties and types.

The demo project could now load NPM packages directly. see: https://node-projects.github.io/web-component-designer-demo/index.html


Need an Expansion Tile Builder

Use case
I want to build a dynamic size expansion tile like listViewBuilder. But couldn't find one.
Proposal

Hi @abinvs-2019, I believe you can already achieve that with the existing APIs, Can you share a code sample that doesn't work? This will also help me understand your use case.
Thanks

Yes, I have achieve it. But the issue was there is a need of Expansion tile builder.

@abinvs-2019 I do understand that you are proposing to add ExpansionTile builder, But What I don't understand is, if it can be achieved with the existing APIs then why would we need this?

@maheshmnj yes, I think this is effecting the perfomance. Any ways I think this is not needed. Closing the issue.


The text area is at risk of XSS injection

ref: https://forum.casbin.com/t/82
When I review the old version code. I think the <textarea>  is at risk of XSS injection. because it's doesn't deal with the text data encode and decode, here is source code:
https://github.com/casbin/casbin-forum/blob/9f21cfaa6c7d9518dcc87eb7cb34f3896549cf23/web/src/main/NewNodeTopicBox.js#L300

@oceanlvr great found! Let's fix it!

I think we should restrict the rich text area, or try to mount it with other editors，If you still use markdown syntax, you can limit the script and iframe tags, and use the loader for secondary processing of the video


bug: Connectors in package even if not needed

Is there an existing issue for this?

[X] I have searched the existing issues

Package Version
1.4.2
Current Behavior

Expected Behavior
I was expecting to install connectors as an extra dependency, not automatically when installing core
Steps To Reproduce
No response
Link to Minimal Reproducible Example (StackBlitz, CodeSandbox, GitHub repo etc.)
No response
Anything else?
No response

Intentional so you don't need to worry about bumping versions.

Intentional so you don't need to worry about bumping versions.

Ok, but is there a way to exclude them?
This makes my node_modulesexplode, which is a problem for some running environments where the size of this folder is limited...


Publish 04/09/2021 10:30AM PT

Docs Build status updates of commit 9a0447d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/New-LabelPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-LabelPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Removed the word "Finally"

Removed the word "Finally", which could have caused people to stop reading and not notice the last paragraph.

Merged! Thank you.


docs(readme): Fix command package:win

Add right command for Windows package
Fixes #78

Thank you very much for spotting this!


mgmt, fix litegen test proxy start failure issue

Description
Encounter issue when running mgmt lite generation CI, while local and  java - confidentialledger - ci does not have error. After debugging the code and compare with java - confidentialledger - ci, I found there is a step to start testproxy server in CI.
This pr added the start testproxy server step in to mgmt lite generation CI, referring to the step in ci.tests.yml.
Hi @weidongxu-microsoft, could you please suggest how can I test the CI before merging the code?
java.lang.RuntimeException: Test proxy did not initialize.
	at com.azure.core.test.utils.TestProxyManager.startProxy(TestProxyManager.java:67)
	at com.azure.core.test.TestProxyTestBase.setupTestProxy(TestProxyTestBase.java:37)

All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

On the other hand, if there is api-version upgrade on the litegen, recorded tests likely will certainly fail (regardless of whether test proxy is started or not).
We may choose either

always use LIVE test for lite
or
in litegen, skip test (though it still means dev need to fix it in PR)

On the other hand, if there is api-version upgrade on the litegen, recorded tests likely will certainly fail (regardless of whether test proxy is started or not).
We may choose either

always use LIVE test for lite

or

in litegen, skip test (though it still means dev need to fix it in PR); not sure if there is option to only skip PLAYBACK test

My personal preference would be LIVE test only.
(but given confidentialledger is our only test proxy mgmt case, it likely need to live till we had PoC on premium mgmt)

Thanks @weidongxu-microsoft  for bringing up. I also plan to do the change, about to confirm if it is acceptable to enable LIVE for all Lite SDK, or we should only enable for specific RPs like confidential ledger.
I tend to always use LIVE tests for lite as well. Let me update in a seperate pr. Got one question, if we use LIVE tests, how do we get environment variables like client_id, subscription_id, secret? It would be great if there are existing examples.
BTW, do you know how can I test the CI before merging the code?

For test the CI, you need to push it to upstream (Azure org), and run litegen on this branch, with property settings (readme etc.)
For LIVE test, our doc (for service) here https://github.com/Azure/azure-sdk-for-java/blob/main/sdk/resourcemanager/docs/HOW_TO_ADD_LIVE_TESTS.md
The env variable is set by some step in CI (last time I checked, script that deploys the ARM template will get them, and set to env variable).

For test the CI, you need to push it to upstream (Azure org), and run litegen on this branch, with property settings (readme etc.)
Some one need to write the LIVE test before one enable it.
For LIVE test, our doc (for service) here https://github.com/Azure/azure-sdk-for-java/blob/main/sdk/resourcemanager/docs/HOW_TO_ADD_LIVE_TESTS.md
The env variable is set by some step in CI (last time I checked, script that deploys the ARM template will get them after deploy, and set to env variable).

Thanks. I have tested the CI in upstream branh, it passes, I will merge the pr first. I created an issue to track enabling LIVE tests for all lite gen. https://github.com/Azure/azure-sdk-for-java/issues/34995
One thing is if we change to live test,  do we need to update litegen CI to run LIVE tests, or only runLIVE tests in  java - confidentialledger - ci?  If we need to run LIVE in litegen CI, from my understanding, we need to add one step to [generation.yml](https://github.com/Azure/azure-sdk-for-java/blob/main/eng/mgmt/automation/generation.yml#L33 like this, but currently 'Generation' step in litegen CI will call generate.py (it will call mvn verify which triggers tests). we may need to skipTests in 'Generation' step.

/check-enforcer override


fix build error on windows

#1 Makes it work on windows

Bump. Can we get this merged?

bump. I agree, we should get this merged.

Because this is unmaintained I published this fix to better-graceful


docker build -t mymicroservice .

ERROR: error during connect: this error may indicate that the docker daemon is not running: Get "http://%2F%2F.%2Fpipe%2Fdocker_engine/_ping": open //./pipe/docker_engine: The system cannot find the file specified.
how i can solve it?

Please make sure you have docker installed/running, that should fix the issue.


musixmatch failing

I'm getting Error: Could not find lyrics. on luca turilli's rhapsody - yggdrasil that indeed is available on musixmatch. I see you have musixmatch in the backend code but i checked with a sniffer and no request is ever made to it. Edit: there is a http connection attempt (tunnel) but no https request is ever sent. Is proxying toggleable, if you are doing it (i saw some proxy class or property in the code)?

Solved minutes ago with #47!


Update Korean translations

Category:
Other (update translation)
Overview
Updated Korean translations to fix some awkward expressions.
@Bogyie as the original author of the translation, please feel free to review these changes.
Note to Korean users

'구성'을 더 널리 쓰이는 용어인 '설정'으로 변경했습니다.
일부 어색한 표현을 수정했습니다.

Code Quality Checklist (Please complete)

[ ] All changes are backward compatible
[ ] All lint checks and tests are passing
[ ] There are no (new) build warnings or errors

@Lissy93 any chance to get this merged?

So sorry @bskim45 for the delay. Looks awesome, merging now :)


Home Assistant is up-to-date (core-2021-11-5) and HACS too (1.17.2).
I've just updated the deployed code with the latest content pulled minutes ago.
I disabled many customization (Nanoleaf, Google Assistant.etc.) to see if another one causes a side-effect on the hilo plugin without any success...
Any clue?

Hey Dom,
Can you enable debug by any chance?
This is explained here:
https://github.com/francispoisson/hilo#contributing
Thanks

HI David,
Debug mode is enabled.
As the plugin setup fails, its code is not reached and hence no log related to it are produced...
I'll continue looking for hint a bit later and will share progresses.

Merci de vous référer à ce repo pour les développements futurs:
https://github.com/dvd-dev/hilo/


Add locations endpoint test

Signed-off-by: Kevin Ruiz<EMAIL_ADDRESS>Contributes to # (add issue number here. i.e. #302)
What did you do?
Add test for locations endpoint
Why did you do it?
Missing test
How have you tested it?
Local test run
Were docs updated if needed?

[X] No
[ ] Yes
[ ] N/A

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[ ] My changes generate no new console logs
[ ] Fixes entire issue
[ ] Partial fix for issue

I'm sure there are a couple more test i'm missing, would appreciate some other suggestions on what to test

@upkarlidder @tonypearson


errors when running the code

Hi !!
I was playing with your cyclegan code but I got the following error message:
File "./mnist_test.py", line 254, in
test_cyclegan()
File "./mnist_test.py", line 182, in test_cyclegan
cat_out = generator_cats.predict(mnist_in)
File "/home/USERS/chouy/Applications/anaconda2/lib/python2.7/site-packages/keras/engine/training.py", line 1713, in predict
verbose=verbose, steps=steps)
File "/home/USERS/chouy/Applications/anaconda2/lib/python2.7/site-packages/keras/engine/training.py", line 1269, in _predict_loop
batch_outs = f(ins_batch)
File "/home/USERS/chouy/Applications/anaconda2/lib/python2.7/site-packages/keras/backend/tensorflow_backend.py", line 2273, in call
**self.session_kwargs)
File "/home/USERS/chouy/Applications/anaconda2/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 895, in run
run_metadata_ptr)
File "/home/USERS/chouy/Applications/anaconda2/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 1124, in _run
feed_dict_tensor, options, run_metadata)
File "/home/USERS/chouy/Applications/anaconda2/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 1321, in _do_run
options, run_metadata)
File "/home/USERS/chouy/Applications/anaconda2/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 1340, in _do_call
raise type(e)(node_def, op, message)
tensorflow.python.framework.errors_impl.InvalidArgumentError: image must have at least one channel
[[Node: up_sampling2d_1/ResizeNearestNeighbor = ResizeNearestNeighbor[T=DT_FLOAT, align_corners=false, _device="/job:localhost/replica:0/task:0/gpu:0"](leaky_re_lu_23/sub, up_sampling2d_1/mul)]]
[[Node: conv2d_27/Sigmoid/_59 = _Recvclient_terminated=false, recv_device="/job:localhost/replica:0/task:0/cpu:0", send_device="/job:localhost/replica:0/task:0/gpu:0", send_device_incarnation=1, tensor_name="edge_4095_conv2d_27/Sigmoid", tensor_type=DT_FLOAT, _device="/job:localhost/replica:0/task:0/cpu:0"]]
Caused by op u'up_sampling2d_1/ResizeNearestNeighbor', defined at:
File "./mnist_test.py", line 254, in
test_cyclegan()
File "./mnist_test.py", line 111, in test_cyclegan
generator_cats = mnist_generator(mnist_shape)
File "/ISFILE3/USERS/chouy/Projects/DeepLearning/cycleGAN/example2/cyclegan/models.py", line 62, in mnist_generator
x = UpSampling2D(size=(2, 2))(x)
File "/home/USERS/chouy/Applications/anaconda2/lib/python2.7/site-packages/keras/engine/topology.py", line 602, in call
output = self.call(inputs, **kwargs)
File "/home/USERS/chouy/Applications/anaconda2/lib/python2.7/site-packages/keras/layers/convolutional.py", line 1361, in call
self.data_format)
File "/home/USERS/chouy/Applications/anaconda2/lib/python2.7/site-packages/keras/backend/tensorflow_backend.py", line 1768, in resize_images
x = tf.image.resize_nearest_neighbor(x, new_shape)
File "/home/USERS/chouy/Applications/anaconda2/lib/python2.7/site-packages/tensorflow/python/ops/gen_image_ops.py", line 927, in resize_nearest_neighbor
name=name)
File "/home/USERS/chouy/Applications/anaconda2/lib/python2.7/site-packages/tensorflow/python/framework/op_def_library.py", line 767, in apply_op
op_def=op_def)
File "/home/USERS/chouy/Applications/anaconda2/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 2630, in create_op
original_op=self._default_original_op, op_def=op_def)
File "/home/USERS/chouy/Applications/anaconda2/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 1204, in init
self._traceback = self._graph._extract_stack()  # pylint: disable=protected-access
InvalidArgumentError (see above for traceback): image must have at least one channel
[[Node: up_sampling2d_1/ResizeNearestNeighbor = ResizeNearestNeighbor[T=DT_FLOAT, align_corners=false, _device="/job:localhost/replica:0/task:0/gpu:0"](leaky_re_lu_23/sub, up_sampling2d_1/mul)]]
[[Node: conv2d_27/Sigmoid/_59 = _Recvclient_terminated=false, recv_device="/job:localhost/replica:0/task:0/cpu:0", send_device="/job:localhost/replica:0/task:0/gpu:0", send_device_incarnation=1, tensor_name="edge_4095_conv2d_27/Sigmoid", tensor_type=DT_FLOAT, _device="/job:localhost/replica:0/task:0/cpu:0"]]

Any suggestions ??
Thank you !!
Yiyu

Not exactly sure what it is, but you should try an earlier version of tensorflow. It sounds like the upsample operation is expecting different arguments.
I am not sure exactly what version of tensorflow/keras I built this for. I intended it as more of a proof-of-concept.


use lax.cond operator for gradient accumulation optimizer

Use lax.cond operator for gradient accumulation optimizer.
Original while operator will introduce large memory overhead, we can save great memory costs by using lax.cond. For an example PAXML config (TP=2, DP=4, Scan=On, 24 layers), this change can reduce peak memory costs from 85GB to about 52GB.
For the issue described by previous comments:
    # PAX makes use of vectorized map for repeated layers. XLA currently doesn't
    # handle conds with-in vmap well and thus calls into both branches with a
    # select. Here we rewrite a lax.cond as while_loop to get around this issue
    # and get faster step time.

Needs to verify whether the issue still exist, and there is, should fix it instead of using while operator to work around.

Why this PR isn't merged after approval?

What is the perf impact that was seen internally?
In theory, there shouldn't be one.
Do you have any idea where it come from? How hard it would be to optimize it?

@sherhut

A way out here to unblock this would be to make the insertion of cond vs. while a params. Something like use_cond_instead_of_while and switch between the two. The default should be False for now.

I have tried for a while to update the PR with a param to control whether to use cond or while operator, but I only understand the local pieces of the code, no idea on how to add the control param following praxis usual way.
Could some guy from praxis team help make this update?
@sherhut

I don't which case in what case the using while to simulate cond operator could have better perf, but if we have that case, we should triage why cond operator have bad perf, it looks wired to use while to simulate the cond operator.

Not in the Praxis team, but @sherhut asked me to take a look. I would imagine that the right way to add the flag (at HEAD) would be to add a field in ShardedStaticAccumulator and in sharded_static_accumulation (where the branching should now happen). Then, just pass it to the call to sharded_static_accumulation within ShardedStaticAccumulator, and initialize ShardedStaticAccumulator as needed in whichever model you are using :)

@akuegel  Can this PR be merged now?

@akuegel Can this PR be merged now?

I have no saying in this. This is in a code base where others need to approve.
@zhangqiaorjc Do the recent changes look ok now so that this PR can be merged?

Please resolve conflicts. I'll help with getting the PR merged once done.

Actually, I'll have to defer to @zhangqiaorjc . Looks like cannot shepperd this PR in. I didn't realize this was for google/praxis initially and I don't seem to have the required permissions here.

@zhangqiaorjc could you check if the PR can be merged now?

adding the flag to gate this behavior LGTM


Feat - Add http-based authentication using passport on backend

Update backend server files to use passport for login
Update frontend to use fetch and post to the backend for the login
Add concurrently and set up a proxy so both the node and webpack-dev servers can run at the same time. (see https://github.com/rangle/rangle-starter/issues/113)

Connected to https://github.com/rangle/rangle-starter/issues/107

:shipit:


Update other language google docs to match new en.js props

They're added at the bottom with predictions. See #700.

You can now check the top of the docs to see hints about what properties might be missing!


Duplicate text label when I'm using TableView or CollectionView

The API doesn't work so well when I'm using TableView or CollectionView... sometimes the label doesn't have text and the cell is getting the old value for this label... duplicating my text and showing wrong.
To fix that... I've had to improve some part of the code... see my solution:
override public var text: String? {
didSet {
if text != nil && text != "" {
updateTextStorage()
} else {
self.textStorage.setAttributedString(NSAttributedString())
}
}
}
I hope help someone else and/or improve your code... Great API that you make... Congrats

Not able to reproduce, closing


broken link

The "Advisor Cost recommendations" link under "Next Steps" incorrectly references the same content as the "Advisor Performance recommendations" link.
The correct page to reference would be:
https://docs.microsoft.com/en-us/azure/advisor/advisor-cost-recommendations

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

ID: ac2bfa05-6828-6501-0cd4-b6e42a049726
Version Independent ID: 40f6259a-4ae3-3bfe-a3fd-a50a26980266
Content: Azure Advisor High Availability recommendations
Content Source: articles/advisor/advisor-high-availability-recommendations.md
Service: advisor
GitHub Login: @manbeenkohli
Microsoft Alias: makohli

@swing24 Thanks for bringing this to our attention. We are actively investigating and will get back to you soon.

@swing24 thanks again for bringing this to our attention. A fix has been pushed out to resolve the redundant link. the fix will be live by tomorrow a.m.
@manbeenkohli  - fyi.

Closing this issue for now. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.


fix(eof): add FrameResult if eof is in create tx

Should be merged here: https://github.com/bluealloy/revm/pull/1467
EOFCreate call now can be done from create transaction.
Add ReturnContract as success return.

Merged changes to https://github.com/bluealloy/revm/pull/1467


Translation help

i dont understand how i can translate the mods

Actually, Because it has not been updated for a long time, this mod might have quite a few issues now! I’ve been really busy recently, and due to some problems, I’ve been away from BC for quite some time. Maybe one day I’ll return…? Who knows…


[deliver] Capturing NoMethodError exception when fetching live app info

Checklist

[x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I see several green ci/circleci builds in the "All checks have passed" section of my PR (connect CircleCI to GitHub if not)
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.
[x] I've added or updated relevant unit tests.

Motivation and Context
The recent commit c4ed1a42 throws an error if a field is missing.
Description
In some cases, the app info might be missing some fields and the execution throws an error
Testing Steps
N/A

@kronenthaler Thanks for the PR! I see you checked

I see several green ci/circleci builds in the "All checks have passed" section of my PR (connect CircleCI to GitHub if not)

but I don't see that the ci/circleci builds have ran — can you please double check you're signed into CircleCI with your GitHub credentials and try pushing another commit?

@getaaron done, the build seems to report one single check, but the pipeline in circleci has multiple steps.


Image stream contains a sub-chunk size chunk

All chunks in the image stream should have an uncompressed size of chunkSize. In some instances libaff4 writes the last chunk without padding the uncompressed data to chunkSize. Suspect the problem is with flush().
This is important as decompressors (exp LZ4) are faster if they know the size of the decompressed buffer.

This does not make any sense - why would the last chunk need to be padded? It is obviously shorter than chunk size.

v1 standard @ 3.2 "Storage of uncompressed chunks is supported by the simple principle that if len(chunk) == aff4:chunk_size then it is a stored chunk."
An implication of the above is that storing uncompressed chunks of less than chunkSize is not allowed. But what if I have a last chunk that is, for example, len(chunkSize)-512, that compresses to length=chunkSize.
By the spec, I'm not allowed to store it, so I need to store the compressed version, but a reader, on seeing that length == chunkSize, will think is is stored, per the spec. We have actually struck an edge case along these lines before. This can be worked around by considering the aff4:size of the ImageStream, but makes for a complex implementation.
Obviously the ImageStream abstraction isn't nailed down tight enough in the spec given our different interpretations here. I'd argue that it is conceptually simpler, simpler to implement and more performant to treat the Image Stream as strictly a "compressed chunk stream", rather than as a "compressed chunk stream except for the last chunk which isn't a chunk stream".

I think we have considered this case during the standardization process. If the compressed size is larger than chunk size, then the code will just store the uncompressed chunk. If the last chunk is smaller than chunk size it will be stored compressed - there is no problem there.
I was never too comfortable with explicitly allowing for this edge case (interleaving compressed and uncompressed chunks in the same image) - the benefit is saving a couple of bytes for the header at best but it opens up a whole array of edge cases like this one. Why don't we always stored the same type of chunk (always compressed or always uncompressed)?

I propose updating the spec to the following:
"Chunks written to the Image Stream must be of size chunkSize. When writing sub-chunkSize  chunks, the remaining space must be padded with 0x00."

I think this is a good optimization, but I fear it's overly specific and too restrictive to be a requirement.  I think the spec could benefit from a section on implementation notes and this might fit well there.

It isn't an optimisation. The restriction has always been there: "Image streams specify the chunk_size attribute, as the number of image bytes each chunk contains (chunk size defaults to 32 kb)."[1]
Also, at p1 of the spec v1.0: "the Image Stream (a seekable, contiguous sequence of fixed sized data blocks, optionally compressed)." [2]
The above proposed spec change only serves to reinforce this.
I'd reverse the question. Why are we considering relaxing this restriction now?
[1] AFF4 (2019) paper.
[2] AFF4 Standard v1.0

I guess it just seems overly restrictive for little gain.  You can easily know the size of the last chunk via modular arithmetic and the aff4:size property.  It's not a hill I'm willing to die on, though.  Just my $0.02


[TT-1348] add fields to config representing deployed CL node details [WIP]

option to pass mockserver url, if empty, but namespace is present build it based on namespace (it has a static url). we could think about doing same for CL nodes.
Also, think whether we should support a situation, where a test is runner and in the background someone adds more nodes to the DON. As long as their URLs follow the same convention (and they all use the same url/pass) we could see if we can find them and connect to them (as another step in some test loop? or during test setup?)

Below is a summarization created by an LLM (gpt-4-0125-preview). Be mindful of hallucinations and verify accuracy.
Why
The changes introduce configuration options for defining node details and deployment preferences, allowing for more flexibility in configuring blockchain nodes. This includes the ability to specify the number of nodes, whether to use existing nodes, detailed node configurations, and namespace specifications.
What

config/node.go

Added NumberOfNodes *int to allow specifying the number of nodes to be deployed.
Added UseExisting *bool to specify whether to use existing nodes instead of creating new ones.
Added a Nodes array to contain configurations for individual nodes, including URL, internal IP, email, and password.
Introduced Namespace *string to allow setting a namespace for node deployment, providing a way to group or isolate nodes within a particular context.

no longer needed


Add gsplat.js

yet another one :)

Nice one! Looking forward to see where your project is heading. Promising!

The automatic floater removal option in that one is an absolute godsend!

@AtlasRedux Where is it? Could you point me to the code as I did not find it. I would be interested to see how it works. Thx

@AtlasRedux Where is it? Could you point me to the code as I did not find it. I would be interested to see how it works. Thx

Whops, I was talking about https://github.com/francescofugazzi/3dgsconverter, had my back/forth history on tabs messed up :P


Adding exit command to the install script

Docker setup will get stuck if there is not exit at the end of the install script.
Adding exit command at the end of the script so that the example can work correctly. There are several issues logged because of this.

@milarso81 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@milarso81 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@milarso81 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@milarso81 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@milarso81 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@milarso81 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@milarso81 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

@j-martens: I'm sorry - only the author of this article, @anandmeg, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

@milarso81, thanks for the fixes!
#sign-off