Chrome/Carriage Return Issue

Greetings-
I am having the same issue as was reported here:
https://github.com/volmer/bootsy/issues/78
I see that the issue was reported to bootstrap-wysihtml5, and this was the response...
https://github.com/xing/wysihtml5/issues/497
The answer is basically that this not maintained.
What I do notice is that this issue isn't present in your sample app when running on chrome? Has this been fixed in newer versions?
Ben

Good news, I just moved Bootsy to Bootstrap3-wysithml5 on master! Could you please give it a try? I'm planning to release a new version soon.

I believe this is still an issue. Installed today on the Master branch but having the same issue described in the original post. Any ideas on how to resolve?

Yes, I had to rolled back because Bootstrap3-wysihtml5 has proven to be worse than the original :cry:

Any new developments on this?


Hi,

Hi, thanks for contributing!
This project is maintained in my spare time, so in order to help me address your issue as quickly as
possible, please provide as much of the following information as you can:

Title: Please indicate the module in question, e.g. "dirPagination: X is broken when Y"
If reporting on dirPagination, please include the version you are using (can be found in the package.json / bower.json file)
If you are able to reproduce your issue on Plunker, this will vastly increase the chances of a rapid response from me.

-- Michael
=======
(Delete the above. Fill in the rest as applicable)
Description of issue:
Steps to reproduce:
Expected result:
Actual result:
Demo: (for dirPagination, fork and modify this Plunk: http://plnkr.co/edit/b37IdFFJUokaeSummETX?p=preview)
Any relevant code:

Hi,
How to get current pages values using pagination??

current-page (optional) Specify a property on your controller's $scope that will be bound to the current page of the pagination. If this is not specified, the directive will automatically create a property named __default__currentPage and use that instead.

<ANY
    dir-paginate="expression | itemsPerPage: (int|expression) [: paginationId (string literal)]"
    [current-page=""]
    [pagination-id=""]
    [total-items=""]>
    ...
</ANY>


infra/port: implement multicast l2 filtering

Add API to filter multicast addresses on ports.
Partially resolves #27

draft PR, to get some feedback on mcast filtering, before adding unicast filtering.


[github] Read the LLVM commit from environment

The workflow was set up to read the LLVM commit from action outputs, but the action was writing it to the environment instead. The workflow has been always checking out LLVM HEAD instead of the pinned version.

Please do not merge until CI passes.


Put networking on the right host

test instances are namespaced now, so we need to just use the ansible
variable for the hostname.

+1


Animations

Hi,
I have been using SUIT CSS naming convention for the last few months and I have a question related with animations.
I have some animations that I would like to reuse through different components, my first attempt was defining the animation as an utility class:
/* animation for components using angular ng-show directive */
.u-fadeAnimation.ng-hide-add {
  animation:  0.2s fadeOut ease;
}

.u-fadeAnimation.ng-hide-remove {
  animation: 0.3s fadeIn ease;
}

@keyframes fadeOut {
  0%   { opacity: 1; }
  100% { opacity: 0; }
}

@keyframes fadeIn {
  0%   { opacity: 0; }
  100% { opacity: 1; }
}

Then I just need to add the u-fadeAnimation class to any component that uses angular ng-show directive.
However, the docs says that utility classes are for "low-level structural and positional traits". I don't think animations fit in that category but I don't see other way to specify them using SUIT CSS.
Thanks.

Personally, I'm fine with how you're defining this as a utility.
You're setting one animation property, which is pretty accurately described by fadeAnimation.

Medium and Bumpers.fm (fat) uses the a- prefix similarly to u-. Maybe we can whitelist that in the linters?


ghc-8.8.3: really use ld.gold (port #80466)

Motivation for this change
Bring 8.8.3 up to date
@peti
Things done
Ported 9a5ecf1212e42f03e26d41d7b6d53853f4016af3.

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg build haskell.packages.ghc883.lens

@GrahamcOfBorg build haskell.packages.ghc883.comonad


Add Support for GDScript Syntax Highlighting

GDScript is the official scripting language of the promising Godot game engine.
It is rising in popularity and becoming a force on its own for game development.
Supporting it for syntax highlighting makes total sense imo.

We're not adding more language modes to the main distribution. The way to work with a new language is to write the mode as a separate module, and possibly publish that to NPM so that others can benefit from it.

How would I go about making a language mode into a module so someone like the people at Boostsnotes could implement it into their projects?

We usually reference the elexir mode as an example.


$ in completion list does not work

when working with puppeteer , typing page.$, completion menu shows $$eval, but upon clicking the entry, only () are inserted

It seems dup of https://github.com/hrsh7th/nvim-compe/issues/491.

.


update readme and other minor tweaks

Update the README, add a CHANGELOG, and remove the bin/juttle script.

@mstemm Once this merges I'll publish the initial version to npm.

lgtm.


Running under different users causes problems for git

When running the image with an user that does not exists in the passwd file git clone will break with fatal: unable to look up current user in the passwd file: no such user
git is used in the master for pipeline and multibranch at least AFAIK
We could use nss_wrapper to create the passwd file at runtime
http://blog.dscpl.com.au/2015/12/unknown-user-when-running-docker.html
or something like the openshift image does
https://github.com/openshift/jenkins/blob/355600974c22617f5fc4d4bf5eac1aee064e7a2f/2/contrib/jenkins/jenkins-common.sh#L13
although it does not work because /etc/passwd is not writable
This would apply for jenkins-slave image too
ping @ndeloof

@carlossg we had this discussion internally at CloudBees during march 2016 already, I expected recent git version had this issue fixed (see http://git.661346.n2.nabble.com/git-clone-fails-when-current-user-is-not-in-etc-passwd-td7643604.html)
Which version are you running ?

confirmed this is should be fixed since 2.6.5
https://github.com/git/git/commit/92bcbb9b338dd27f0fd4245525093c4bce867f3d
or a simple fix is to configure a git user identity

the openjdk image includes git 2.1.4 (does not work) and alpine includes 2.11.2 (does work)
The workaround is to set the GIT_COMMITTER_NAME and GIT_COMMITTER_EMAIL environment variables

openjdk image is based on debian:jessie, Hopefully Stretch has been published so we can expect to get some updated dependencies.

yeah, stretch has git 2.11.0


i18n(es): add google cloud guide

What kind of changes does this PR include?

New or updated content
Translated content

Description
Adds google-cloud.md in spanish

Good job @fredoist, @agustinmulet! Thank you 💜


Module cannot be found

Hi! I was trying to use your library and noticed that my compiler couldn't find the module. Then I ran the runkit test on it (https://npm.runkit.com/pusher-js-mock) and saw that it was getting the same error. Do you mind removing the use of es6 syntax for importing and exporting your files in your src directory and in your package.json put src/pusher-js-mock.js as the value for the main key?
Please and thanks! 😄

Oh shoot, I was including classes in tests directly from files, not from index.js (which doesn't exist) :)
Thanks for pointing this out, I'll fix it!

I've release 0.1.2 version, it should be fixed there, can you check? https://www.npmjs.com/package/pusher-js-mock

Thanks so much!!


incorrect

the velocity and time data for this launch is incorrect if you refer back to the official video by SpaceX

Which data did you find wrong? Note that the velocity data is in m/s while SpaceX's livestreams display km/h.
In addition, this repository is deprecated in favor of https://github.com/shahar603/Launch-Dashboard-API

but shouldn't the altitude be in km rather than m???

They are

then I would like to know how did you derive the data of x-velocity, y-velocity?

im really confused on how you analyse your data

and it is not a velocity-time graph when you are using speed?

Hey, do you have the data for the "Altitude vs Aerodynamic Pressure" Graph??? I have check excel thoroughly and I can't seem to find the data for it. I just wanted the data for a maths investigation I am completing for a year 12 subject, thanks in advance!!!


MissingFieldException

I try to make a mod with ReinDirectorCardLib and I face some problem. If I use EditMonsterCards there will be a error.
https://paste.ubuntu.com/p/KfMHfffHjH/

editMonsterCard.Add, not EditMonsterCard.Add


Spell Stack Exchange as two words

Changed via:
git grep -l StackExchange . | xargs sed -i s/StackExchange/Stack\ Exchange/

After the above replacement git grep -i stackexchange .  only found occurrences in links, and
git grep -i " stackexchange" . , i.e. with a preceding space, did not find me any hits.

ACK. Looks good. Thanks @Xekyo!


cvise: fix cannot stat 'dist': No such file or directory build failure

Without the change cvise builds on staging and staging-next as:
$ nix build --no-link -f. cvise
...
@nix { "action": "setPhase", "phase": "pythonOutputDistPhase" }
pythonOutputDistPhase
Executing pythonOutputDistPhase
mv: cannot stat 'dist': No such file or directory

The failure was exposed by https://github.com/NixOS/nixpkgs/pull/190487.
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Pushed 370cce6a3c65c026f80c4861c7c80ab60edc7f2a to staging-next earlier


[Feature Request] Add "Startup command line" option to settings panel

As per request, splitted. And thanks for the information on the other issue. :+1:
The request:
Since there are a great many switches for the "startup command line" it would be very useful to get an option added to settings (I know not recommended, but I wouldn't know where else to put it) a la CentBrowser (startup command options are stored in an .ini file which is read/called at startup), in Windows you only get so many characters to add to the "Target" field (I believe 255?) before you run out, which in Chromiums case happens very fast.
Since CentBrowser doesn't open source their build of Chromium I cannot link to anything.
These are the switches I currently use:
" --user-data-dir=".\Profile" --disable-breakpad --disable-logging --disable-sync --disable-speech-api --disable-voice-input --disable-domain-reliability"
That fills it almost up so that I for example can't add the option "--show-avatar-button never" to it.

For a tempory fix for your case (that is nor a really a beautiful trick to do, neither beautiful to see in practice):
You can create a .bat file that start chrome.exe with all the command line flags you want, then:

Create a shortcut to your batch file.
Get into shortcut property and change target to something like: cmd.exe /C "path-to-your-batch".
Simply drag your new shortcut to the taskbar. It should now be pinnable.

(from: https://superuser.com/questions/100249/how-to-pin-either-a-shortcut-or-a-batch-file-to-the-new-windows-7-8-and-10-task)
And I think you can find a way to change the icon of the shortcut to the icon of chromium in its properties.
But in practice, I think that when you're gonna click on this shorctut, Windows will open chromium but will append the chromium icon to your taskbar, like it is not pinned to it.
(And of course if you pin it thereafter, and click on it to launch chromium, it will not do it with the flags you have put in the shorcut)


Cannot install Nlog Nuget Package

Bound to TFS 2013. Other packages are installing fine.
And in the error list "Could not find a part of the path 'C:\Source2013\CyberCoders.com\LinkedInResumeGenerator\lib"
C:\Source2013\CyberCoders.com\LinkedInResumeGenerator\NLog.4.1.0.nupkg: No file matches.
The item C:\Source2013\CyberCoders.com\LinkedInResumeGenerator\packages\NLog.4.1.0\lib\net35\NLog.dll could not be found in your workspace, or you do not have permission to access it.
The item C:\Source2013\CyberCoders.com\LinkedInResumeGenerator\packages\NLog.4.1.0\lib\net35\NLog.xml could not be found in your workspace, or you do not have permission to access it.
The item C:\Source2013\CyberCoders.com\LinkedInResumeGenerator\packages\NLog.4.1.0\lib\net40\NLog.dll could not be found in your workspace, or you do not have permission to access it.
The item C:\Source2013\CyberCoders.com\LinkedInResumeGenerator\packages\NLog.4.1.0\lib\net40\NLog.xml could not be found in your workspace, or you do not have permission to access it.
The item C:\Source2013\CyberCoders.com\LinkedInResumeGenerator\packages\NLog.4.1.0\lib\net45\NLog.dll could not be found in your workspace, or you do not have permission to access it.
The item C:\Source2013\CyberCoders.com\LinkedInResumeGenerator\packages\NLog.4.1.0\lib\net45\NLog.xml could not be found in your workspace, or you do not have permission to access it.
The item C:\Source2013\CyberCoders.com\LinkedInResumeGenerator\packages\NLog.4.1.0\lib\sl4\NLog.dll could not be found in your workspace, or you do not have permission to access it.
The item C:\Source2013\CyberCoders.com\LinkedInResumeGenerator\packages\NLog.4.1.0\lib\sl4\NLog.xml could not be found in your workspace, or you do not have permission to access it.
The item C:\Source2013\CyberCoders.com\LinkedInResumeGenerator\packages\NLog.4.1.0\lib\sl5\NLog.dll could not be found in your workspace, or you do not have permission to access it.
The item C:\Source2013\CyberCoders.com\LinkedInResumeGenerator\packages\NLog.4.1.0\lib\sl5\NLog.xml could not be found in your workspace, or you do not have permission to access it.
The item C:\Source2013\CyberCoders.com\LinkedInResumeGenerator\packages\NLog.4.1.0\NLog.4.1.0.nupkg could not be found in your workspace, or you do not have permission to access it.
C:\Source2013\CyberCoders.com\LinkedInResumeGenerator\license.txt: No file matches.
C:\Source2013\CyberCoders.com\LinkedInResumeGenerator\NUnit.2.6.4.nupkg: No file matches.
C:\Source2013\CyberCoders.com\LinkedInResumeGenerator\lib\nunit.framework.dll: No file matches.
C:\Source2013\CyberCoders.com\LinkedInResumeGenerator\lib: No file matches.
C:\Source2013\CyberCoders.com\LinkedInResumeGenerator\lib\nunit.framework.xml: No file matches.
GenerateResume.csproj has been automatically checked out for editing.
C:\Source2013\CyberCoders.com\LinkedInResumeGenerator\NLog.4.1.0.nupkg: No file matches.
The item C:\Source2013\CyberCoders.com\LinkedInResumeGenerator\packages\NLog.4.1.0\lib\net35\NLog.dll could not be found in your workspace, or you do not have permission to access it.
The item C:\Source2013\CyberCoders.com\LinkedInResumeGenerator\packages\NLog.4.1.0\lib\net35\NLog.xml could not be found in your workspace, or you do not have permission to access it.
The item C:\Source2013\CyberCoders.com\LinkedInResumeGenerator\packages\NLog.4.1.0\lib\net40\NLog.dll could not be found in your workspace, or you do not have permission to access it.
The item C:\Source2013\CyberCoders.com\LinkedInResumeGenerator\packages\NLog.4.1.0\lib\net40\NLog.xml could not be found in your workspace, or you do not have permission to access it.
The item C:\Source2013\CyberCoders.com\LinkedInResumeGenerator\packages\NLog.4.1.0\lib\net45\NLog.dll could not be found in your workspace, or you do not have permission to access it.
The item C:\Source2013\CyberCoders.com\LinkedInResumeGenerator\packages\NLog.4.1.0\lib\net45\NLog.xml could not be found in your workspace, or you do not have permission to access it.
The item C:\Source2013\CyberCoders.com\LinkedInResumeGenerator\packages\NLog.4.1.0\lib\sl4\NLog.dll could not be found in your workspace, or you do not have permission to access it.
The item C:\Source2013\CyberCoders.com\LinkedInResumeGenerator\packages\NLog.4.1.0\lib\sl4\NLog.xml could not be found in your workspace, or you do not have permission to access it.
The item C:\Source2013\CyberCoders.com\LinkedInResumeGenerator\packages\NLog.4.1.0\lib\sl5\NLog.dll could not be found in your workspace, or you do not have permission to access it.
The item C:\Source2013\CyberCoders.com\LinkedInResumeGenerator\packages\NLog.4.1.0\lib\sl5\NLog.xml could not be found in your workspace, or you do not have permission to access it.
The item C:\Source2013\CyberCoders.com\LinkedInResumeGenerator\packages\NLog.4.1.0\NLog.4.1.0.nupkg could not be found in your workspace, or you do not have permission to access it.

I manually added the below to packages.config and then Visual Studio 2015 automatically restored / installed NLog for me:

I don't know why Visual Studio wouldn't automatically add that into packages.config.

Thanks for the report.
I think this is: https://github.com/NuGet/Home/issues/1164

Thanks


📃: Add Substrings Checker:

:red_circle: Title : Substrings Checker
:red_circle: Aim : It help us to check if one string is a substring of another string.
:red_circle: Tech Stack Used : HTML, CSS, JS
Screenshots 📷
N/A

:white_check_mark: To be Mentioned while taking the issue :

Full name : Anshika
What is your participant role? GSSOC

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

@Avdhesh-Varshney sir, please assign me this issue I have completed the previous issues.

@Anshika14528 go ahead.


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-main" workflow

Codecov Report

Merging #137 (3cd40b7) into main (2974dfe) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #137   +/-   ##
=======================================
  Coverage   83.33%   83.33%
=======================================
  Files           2        2
  Lines         126      126
  Branches       31       31
=======================================
  Hits          105      105
  Misses         21       21

Impacted Files
Coverage Δ

example/src/main.ts
88.46% <100.00%> (ø)

src/index.ts
82.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


ValueError: Unsupported ONNX attribute: name: "body"

I download the yolov3.onnx from onnx's model zoo, then I got this exception, here is my code:
import onnx
import onnx_coreml

if __name__ == "__main__":
    model = onnx.load('yolov3.onnx')
    cml = onnx_coreml.convert(model)

attribute body is present in ONNX op loop which is currently not supported by onnx-coreml


Block scrolling when a popup/sidebar is visible

Coverage increased (+0.003%) to 94.414% when pulling 2b7d11ed5e8d324de92fdcc7d58c9e5d6df52cc7 on cody-popup-blocks-scrolling into 5aa92ed44b221e8b6a15642497b53abf40d07b06 on main.


'Unshaky' for Windows??

I read about your Mac program in a Wall Street Journal article. I think that is a very clever idea.
I use Windows7 PRO on my desktop. I use a very old KeyPro keyboard, which is great--much better than newer ones and with lots of customizable features. But a few of the keys have begun to double print, even though the key itself returns to the up position normally. There are no parts available for this keyboard. Your incorporation of killing any second signal within a specific time is great.
Do you perhaps make a version for Windows, or do you know of anyone who does? I am not a programmer, so I do not have the capability of modifying the open source code that you have posted.
Harry

A simpler version for Windows is feasible. I don't have a Windows machine so developing a Unshaky windows counterpart is hard for me. I found a simple key map utility written in C and I think it can be modified to prevent double key presses. I could play with it when I have time.

this post might help.

The links to KeybounceFilter no longer exist. AutoHotKey still exists, and there is some code for KeybounceFilter on on the AutoHotKey site. But I am no programmer, so have no idea how to use that code. Any suggestions or alternatives?
Harry


Add Slack API token as CLI input argument

@oaaky Could you take a look at this and make the necessary changes?

Closing this issue now as the slack token has been added as a CLI argument in the merged PR.


Allow removal and editing of previous comments if feasible

Currently everything is seemingly set in stone from the user's end once a comment is submitted. Not sure if this is feasible to implement though if the goal is to have no accounts and make all commenting anonymous

If we want to limit updating and deleting to the user who posted the comment, conversation, or community, then we will need to create some concept of a user, either through managing sessions, or a modeled user that is tied to the comments, conversations and communities they create.

I'm not sure if that immediate permanence upon submitting is a bug or a feature. So many other sites have endless editing capabilities, I appreciate that we might want to make people really think about what they're sharing before publishing. Or, maybe we save their IP address for a short amount of time and they have only a few minutes to decide to edit or delete.
I could see a different approach within the groups, where users would have some regular alias & password to join / contribute. There, they might be able to edit or delete their own posts.


update purescript version

change Eff to Effect https://purescript-resources.readthedocs.io/en/latest/eff-to-effect.html

I'll be happy to accept a PR for the same.


moment().zone deprecation warning

Deprecation warning: moment().zone is deprecated, use moment().utcOffset instead. http://momentjs.com/guides/#/warnings/zone/
Arguments: -120
Error
    at Moment.zone ($HOME/oss/github.com/threefoldfoundation/tfchain-wallet-electron/node_modules/moment/moment.js:320:98)
    at Footer.render (http://localhost:1212/dist/renderer.dev.js:4735:96)
    at finishClassComponent ($HOME/oss/github.com/threefoldfoundation/tfchain-wallet-electron/node_modules/react-dom/cjs/react-dom.development.js:14741:31)
    at updateClassComponent ($HOME/oss/github.com/threefoldfoundation/tfchain-wallet-electron/node_modules/react-dom/cjs/react-dom.development.js:14696:24)
    at beginWork ($HOME/oss/github.com/threefoldfoundation/tfchain-wallet-electron/node_modules/react-dom/cjs/react-dom.development.js:15644:16)
    at performUnitOfWork ($HOME/oss/github.com/threefoldfoundation/tfchain-wallet-electron/node_modules/react-dom/cjs/react-dom.development.js:19312:12)
    at workLoop ($HOME/oss/github.com/threefoldfoundation/tfchain-wallet-electron/node_modules/react-dom/cjs/react-dom.development.js:19352:24)
    at renderRoot ($HOME/oss/github.com/threefoldfoundation/tfchain-wallet-electron/node_modules/react-dom/cjs/react-dom.development.js:19435:7)
    at performWorkOnRoot ($HOME/oss/github.com/threefoldfoundation/tfchain-wallet-electron/node_modules/react-dom/cjs/react-dom.development.js:20342:7)
    at performWork ($HOME/oss/github.com/threefoldfoundation/tfchain-wallet-electron/node_modules/react-dom/cjs/react-dom.development.js:20254:7)

closed in https://github.com/threefoldfoundation/tfchain-wallet-electron/commit/6ad0b174221ae7432954478133f2d0685abb9c27


Declaration of Illuminate\Support\Carbon::__set_state($array) should be compatible with Carbon\Carbon::__set_state(array $state)

Laravel Version: latest
PHP Version: 7.1.11

Description:
Signature of Carbon\Carbon::__set_state changed in version 1.24.0, so Illuminte\Support\Carbon::__set_state should be updated
Steps To Reproduce:
composer create-project --prefer-dist laravel/laravel blog fails in @php artisan package:discover

This commit is the source of the issue: https://github.com/briannesbitt/Carbon/commit/a05bed23de4c00040fe33cc6820c7e6be8d00c04

fixed in https://github.com/laravel/framework/pull/23459

lmao carbon guys broke it again with 1.24.1 xD

yes its broken again.

On the latest version (5.6.11) and everything seems fine. _set_state was completely removed from support\Carbon, so there's no possibility of incompatibility.


Docker Cache example incomplete and does not appear to actually work

This page references the possibility of enabling DevOps supported caching for container builds.
There are many parts of this example here that are not explained. A random referenced YAML file which then isnt used as well as no context to getting this to work with the Docker@2 task.
These docs need to be updated to clarify if this even works. I cannot seem to get the Docker@2 task to play nicely with docker save. It appears to be doing something rather strange when saving images with ***/ placed infront of the image and I cannot seem to reference it outside of the task.
Please can this documentation be updated to show an actual working example.

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

ID: 77c20289-dab2-7a6e-4f63-4b30eaff2c71
Version Independent ID: d75cc196-90c1-8291-9907-ff405bde4edf
Content: Pipeline caching - Azure Pipelines
Content Source: docs/pipelines/release/caching.md
Product: devops
Technology: devops-cicd-release
GitHub Login: @ramiMSFT
Microsoft Alias: rabououn

@GerryWilko Gerry, thanks for your feedback. Please see these resources:

Azure DevOps Services
Azure DevOps Support Bot
Azure DevOps on Stack Overflow

@ramiMSFT - Hi Rami. Please look into this issue.

@GerryWilko Thanks for the feedback! The updated content will go live shortly. Refer to Docker task, Build images and Push images for more examples and information on the Docker@2 task


java.lang.NullPointerException: Exception ticking world

A friend reported to be flying and shooting sheep at time of the crash.
http://pastebin.com/VFghJr8G

Possibly fixed next version


fix($rootScope): when adding/removing watchers during $digest

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Bug fix.
What is the current behavior? (You can also link to an open issue here)
Adding a watcher during a $digest (i.e. from within a watcher), results in the next watcher getting skipped. Similarly, removing a watcher during a $digest could result in the current watcher being run twice (if the removed watcher had not run yet in the current $digest).
What is the new behavior (if this is a feature change)?
Adding/Removing watchers during a $digest works as expected.
Does this PR introduce a breaking change?
No.
Please check if the PR fulfills these requirements

[x] The commit message follows our guidelines: https://github.com/angular/angular.js/blob/master/CONTRIBUTING.md#commit-message-format
[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

Other information:
This commit fixes both cases by keeping track of the current watcher index during a digest and properly updating it when adding/removing watchers.
There might be a tiny extra overhead due to looking up $$digestWatchIndex on scope.$$watchers during the $digest.
Fixes #15422

looks good to me, i will test (patch my 158 install) with that code tomorrow to see how it works but at first sight that should fix it
one thing: "should not skip watchers when removing a watcher during digest'"
thats not really the case, for removing its not really skipping right? its more about not executing the same watch again? i think this check:
expect(log).toEqual([1, 2, 1, 2]);
would be without your fix:
expect(log).toEqual([1, 1, 2, 1, 2]);
right?

i patched my 158 install and i don't need the evalAsync patch that i now had in my code
it seems to work fine (also going through the application i don't notice anything weird)

We should definitely test this for performance.

the only thing that is really changed for digest() is that instead of a local variable we now have a scoped variable on a object. Anybody know what that lookup/assignment cost? Is that really measurable ?
i think the code in add/remove watch shouldn't be to much to worry about because that shouldn't really be called in high volume (like the digest cycle)

Yeah, the only change that matters (performance-wise) is the switch from a local variable to a property lookup. I doubt it will have any noticeable impact (compared to all other operations typically run alongside).
I meassured the duration of the digest (without any changes) in a finished instance of the largetable/ngBind benchmark (>40.000 watchers) and I didn't see any difference. In both cases (before and after this change) the duration was ~13-14ms.

LGTM


fix adjustment factor use for pvwatts

fix #1111

Nice catch! This issue is probably more widespread than just pvwatts right? Did you try something similar with pvsamv1 for example?

@mjprilliman I modified (locally) the notebook to use pvsamv1 and didn't run into the same project with adjustment factors zeroing out AC, so they must be handled differently in some way...

See my comment on #1111 - I don't think this fixes the root cause of the issue

Do we need to do anything here for Patch 1?


When refreshing the access token, update the id_token if possible

When sending a request to refresh the access token, it is possible that
the response will contain a new id_token. If it does, update the id_token
JWT and claims in the session as well the maximum session duration if
OIDCSessionMaxDuration is 0.

in principle the id_token should represent a user authentication event and it is not supposed to be refreshed during a refresh grant since the user may no longer be present so I'm in doubt about this one spec-wise; then again, one may argue that it the OP should dictate the behavior so updating it is merely reflecting the OP policy on this

I hand thought about that myself as the wording in the spec is somewhat vague:
12.2.  Successful Refresh Response

Upon successful validation of the Refresh Token, the response body is the Token Response of Section <IP_ADDRESS> except that it might not contain an id_token.

I took this as it wasn't against the specification but I defer to you if you think it is against the spirit. I will note that in my testing both the Okta and ForgeRock AM OPs issue new id_tokens on access token refresh so it seems somewhat common.

I see; well, if the OP issues the id_token and the spec formally does not preclude it, I guess the client should support consuming it indeed


RuntimeError: Loss is diverging! Try lowering the learning rate.

Train on 1 card, already set lr to: 0.00125, gots:
Loading model from retinanet_rn50fpn.pth...
     model: RetinaNet
  backbone: ResNet50FPN
   classes: 80, anchors: 9
####### lr: 0.00125
####### warmup: 125
Selected optimization level O2:  FP16 training with FP32 batchnorm and FP32 master weights.

Defaults for this optimization level are:
enabled                : True
opt_level              : O2
cast_model_type        : torch.float16
patch_torch_functions  : False
keep_batchnorm_fp32    : True
master_weights         : True
loss_scale             : dynamic
Processing user overrides (additional kwargs that are not None)...
After processing overrides, optimization options are:
enabled                : True
opt_level              : O2
cast_model_type        : torch.float16
patch_torch_functions  : False
keep_batchnorm_fp32    : True
master_weights         : True
loss_scale             : 128.0
Warning:  multi_tensor_applier fused unscale kernel is unavailable, possibly because apex was installed without --cuda_ext --cpp_ext. Using Python fallback.  Original ImportError was: ModuleNotFoundError("No module named 'amp_C'",)
Preparing dataset...
    loader: pytorch
    resize: [640, 1024], max: 1333
    device: 1 gpu
    batch: 2, precision: mixed
Training model for 90000 iterations...
[ 1122/90000] focal loss: 1.245, box loss: 0.452, 0.492s/2-batch (fw: 0.186s, bw: 0.303s), 4.1 im/s, lr: 0.01
[ 1248/90000] focal loss: 1.186, box loss: 0.451, 0.478s/2-batch (fw: 0.176s, bw: 0.301s), 4.2 im/s, lr: 0.01
[ 1374/90000] focal loss: 1.098, box loss: 0.457, 0.479s/2-batch (fw: 0.177s, bw: 0.301s), 4.2 im/s, lr: 0.01
[ 1496/90000] focal loss: 1.065, box loss: 0.451, 0.494s/2-batch (fw: 0.184s, bw: 0.310s), 4.0 im/s, lr: 0.01
Traceback (most recent call last):
  File "/usr/local/bin/retinanet", line 8, in <module>
    sys.exit(main())
  File "/p/retinanet-examples/retinanet/main.py", line 183, in main
    worker(0, args, 1, model, state)
  File "/p/retinanet-examples/retinanet/main.py", line 129, in worker
    metrics_url=args.post_metrics, logdir=args.logdir, verbose=(rank == 0))
  File "/p/retinanet-examples/retinanet/train.py", line 106, in train
    'Try lowering the learning rate.'))
RuntimeError: Loss is diverging!
Try lowering the learning rate.

Please try different lr/warmup configurations. The original values don't necessarily need to scale with the number of GPUs.


⚠ ClamAV has degraded performance

In bf42dfd, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 306 ms

Resolved: ClamAV performance has improved in 455392f.


added simple render style

I would like to propose a simple render style in between "default" and "minimal", namely only showing (centred) title (with no icons and timestamps) and notification message: see rendering examples in the screenshots below

I have tweaked the original default.lua to centre the title and remove the spurious spaces, it should work fine with all use cases (but perhaps there are cases I have not considered - I am just slowly familiarising with the code): what do you think?

Thanks for the PR!

P. S. I forgot to add the style in help text of :h notify-render(), maybe for completeness you may want to add a small commit adding the text there too :)

Ah thanks, done :+1:


check for yield return value

Might close #35

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

Project coverage is 91.15%. Comparing base (a69cc64) to head (5c82ed0).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #38      +/-   ##
==========================================
- Coverage   91.76%   91.15%   -0.61%
==========================================
  Files          21       21
  Lines         522      667     +145
==========================================
+ Hits          479      608     +129
- Misses         25       41      +16
  Partials       18       18

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

@ksysoev I think the cause was because we were not checking yield value, so if the loops breaks out or returns, go runtime can panic.
range functions for custom collections is still pretty new and there is not enough docs/examples for it yet


Update Complete-MigrationBatch.md

Updates to NotificationEmails parameter per Issue #1201

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/move-and-migration/Complete-MigrationBatch.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Syntax seems to break at a certain point

Cheers for this neat extension. I'm having some trouble though, I wonder if you could shed some light?

I appears that {} brakets are breaking the syntax coloring

Hi @lichenophile,
I'm unable to replicate that issue here. Are you using VSCode or VSCode Server in HA? Also, do you have any other syntax highlighting plugins that may be causing a conflict?

That's very strange, I still can't replicate it on my local copy of the browser version in HA. the only thing I can think that may be causing this is Windows, I'll have to spin up a Virtual Machine and see if I can replicate the issue, I'll let you know what I find.

@lichenophile Just out of interest could you try using >- instead of > proceeding your block string? It might be Windows line breaks causing the issue. Make sure you use semicolons ;  at the end of lines strictly if you do this because it will strip out the line breaks.

@lichenophile Ok, I just revisited this after I got your message, and it still isn't an issue on the VSCode Server plugin running in HA, but this time, I was able to reproduce the issue on my local copy of VSCode running on MacOS! This makes the whole thing much more confusing considering I wasn't able to replicate it in the past! However what it does mean is that I can now troubleshoot it and figure out why. I'll mark it as a bug and work through it; it may take a little time as I haven't touched it in months and need to set up debugging, etc, again. I'll keep you posted.

@lichenophile Out of interest: On the lines/blocks that don't break, is the syntax highlighting 100% correct in those? eg. Does it use the correct colours etc?


docker-enter and maven

Is it possible to have docker-enter (nsenter) installed default on all nodes and maven installed by default on all docker images ?
docker-enter need is very obvious.
Maven - most benchmarking projects I clone end up requiring maven so be nice to just have it ?

Nsenter is not required to enter into the container: docker exec -it ambari-agent bash.
Would you like maven to be installed in the docker-ambari image which Cloudbreak launches?

Thanks for the docker exec tip.
Yes .. docker-ambari agent so I can install and run benchmarks unless there is another way to do so.
So far I ran the storm benchmark and Hibench and both needed maven.

Do you need the benchmarks to run inside the container or is it enough if it's on the host machine?

Btw we have recipes which allows you to do custom installations. It's not part of the UI, but with the Cloudbreak-shell you can do such things.


Fix the test suite to run against latest Guard.

The test suite works fine with Guard 2.1, but fails with Guard 2.13.
The failure comes from heavy refactoring in recent version of Guard,
where static initialization is expected by the runtime. The init just
does not happen when running the test suite.
guard-compat seems to address the problem, but it was not updated for a
year at time of writing, and this commit is pretty simple in the end.
The tests remain untouched and all pass.
This seems an alternative to https://github.com/yujinakayama/guard-rubocop/pull/19

:+1:


feat(Dropdown): add placement prop

Description
Added Dropdown placement prop, which allows to set the placement of the dropdown relative to the parent
Dropdown
- added `placement` prop

[IMPORTANT] PR Checklist

[x] I am aware of standards and conventions adopted in this repository, defined in the CONTRIBUTING.md file

PR conventions
Please make sure your PR complies with the following rules before submitting it.

[x] PR title follows the <type>(<scope>): <subject> structure

[x] The PR has been labeled according to the type of changes (e.g. enhancement, new component, bug).

NOTE
Some labels are used to generate release note entries: you can find the complete mapping between PR labels and release note categories here.
For a more detailed overview of PR labels, please refer to the dedicated CONTRIBUTING section.

Additional code checks
Based on your changes, some of these checks may not apply. Please make sure to check the relevant items in the list.

[ ] Changes are covered by tests
[x] Changes to components are accessible and documented in the Storybook
[x] Typings have been updated
[ ] New components are exported from the src/index.ts file
[ ] New files include the Apache 2.0 License disclaimer
[x] The browser console does not contain errors

I'd improve the prop management

Aside for the typing, and the default value, is there something else you think should be improved in props management?


Feature Request: Use log/slog instead of Zap

Now that structured logging is part of the standard libarary, are there any plans to switch to log/slog?

Since Caddy is the primary consumer of certmagic and the whole Caddy plugin ecosystem uses zap, it would be very difficult to change that at this point.
I could see a Caddy v3 causing us to change over, but I don't think we will take the effort to do that anytime soon.

I guess the ideal situation is to have a way to create a zapcore.Core that pipes output to a slog.Handler and then create a *zap.Logger from that Core that will be passed to certmagic.

I guess so. I don't know how interop between them looks like, I haven't looked into it. But that sounds reasonable for now.

Are std libs lobs zero-allocation?
If they aren't, I don't think moving will have any performance benefit.

For the user, it wouldn't be about performance but about interoperability.
For example, if I am starting a new project now, I will use slog for my structured logging. It would be nice that if I need to configure certmagic I can pass in my already existing logger.
I understand going all in on zap before slog was available, or even just in the interest of performance.
Thinking about this more, perhaps there is no need for certmagic and the rest of the Caddy ecosystem to change, there just needs to be an easy way to create a zap logger from slog.

Understood; but we can't impact the TLS server's performance like that -- logs that allocate would significantly reduce the performance of the server given how many logs we emit.
But, yeah, if what you're actually asking for is interop, then maybe a feature request for zap to integrate with slog in some way would be the best way to go. It wouldn't necessarily be as fast of course, but then CertMagic wouldn't have to change and you could still use your slog with CertMagic.

Did some quick googling...
slog recommends using the Logger.LogAttrs API for zero-allocation logging (instead of Logger.Log).
Apparently there's interop between slog can use zap as a logging backend if a Handler is provided. https://github.com/galecore/xslog is an example.

Actually, looks like zap added some compat in https://github.com/uber-go/zap/pull/1246


Make QuoteElem locatable; or configurable locatables?

Description
Evolving from a Discord chat, it would be great if QuoteElem would be also locatable. Justified reasons against it can be performance losses, but it should be checked whether this really makes a big performance loss.
Another idea I had that came up but my Rust knowledge is probably to less of a kind, is, to make all elements configurable if they should be locatable. So the user could decide which elements should be located and which won't. It's probably easier to talk about than it is to implement, but I think it could be a significant difference to the current system.
Use Case
Could allow third-party applications to work more seamlessly with the Typst compiler. If I build for example a tool to read all quotes I could allow them to be locatable, disable locating for all other types and get some perf improvements.

The document now says:

But I am curious why not make all elements locatable. (Performance related?)

Copying my response from Discord:

Making just quotes locatable is trivial by marking them as Locatable. I don't see any harm in that. The reason why not everything is locatable is performance. One more element doesn't break stuff, but allowing everything would hurt it.
The performance impact nowadays would probably be way less than back when I introduced the concept of Locatable, but I would still expect it to be quite significant.

@laurmaedje would it be an idea to make this user / dev configurable? The user / dev could then decide for himself whether the performance losses are justifiable. Forking and extending the elements needed is IMO not a good idea as you would need to roll your own Typst compiler next to the "standard" installation.

I would prefer to just lift the limitation. If that turns out to be not feasible within the near to mid-term future, we can consider an option.


analyzer test failures when switching the parser to fasta

@danrubel; I'll use this issue as something to reference in the code for tests that we mark as failing.

Are there failures to track here, or should we close this?

This is referenced in the sdk code in 28 places. We should likely improve the granularity here, but we do want to keep the issue open.

I have a question about failing quick fixes, mostly for await, but there is one for var as well.
We now report 4 errors:
AnalysisError:/project/test.dart(20..24): The built-in identifier 'await' can't be used as a type.,
AnalysisError:/project/test.dart(26..28): The value of the local variable 'foo' isn't used.,
AnalysisError:/project/test.dart(26..28): Expected to find ';'.,
AnalysisError:/project/test.dart(30..30): Expected an identifier.

In theory we could update analysis of errors in quick fixes to understand The built-in identifier 'await' can't be used as a type.. Or are there changes planned for parser to report something more specific for await and reducing the number of reported errors?

If server is using the error code to differentiate between two or more cases (which I believe is the case here), then the fasta parser needs to generate the right code for the right situation.
We also need to improve the way the parser recovers so that follow-on errors are not generated.
I don't know who will be doing the work, but I suspect that we ought to create an issue, and possibly add it to a checkbox list in the appropriate tracking issue. (Personally, I'd create a single "meta" tracking issue for all such issues, whereas I think we now have three, but I don't care enough to do the work myself.)

I'm closing this as stale. If there are actionable issues, they should be tracked more specifically.


How to get QR code

How do I get a QR code which I can scan with my phone?

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

ID: 888a6b76-3e97-a52f-85d3-c423e9664fdf
Version Independent ID: 2778814f-fba7-97f3-039c-674434e48b3f
Content: Backup and recover with Microsoft Authenticator app - Azure Active Directory
Content Source: articles/active-directory/user-help/microsoft-authenticator-app-backup-and-recovery.md
Service: active-directory
GitHub Login: @eross-msft
Microsoft Alias: lizross

Closing this issue as it is a duplicate of #17256


Fix how sudo works so that we're not going as the root user to just run a command.

More details here: https://www.reddit.com/r/commandline/comments/6gqljj/command_and_rule_over_your_shell_with_python/disg3u0/?utm_content=permalink&utm_medium=front&utm_source=reddit&utm_name=commandline

Fixed.


(chore): remove anndata.read

[x] Closes #1683
[ ] Tests added
[x] Release note added (or unnecessary)

Path forward: simply remove from docs


fcntls fix build for solaris.

close #2540

I think #2544 is a better solution for the issue, which covers all the Solaris build problems and adds the CI, so I am going to close this PR, sorry. 😶


Provide proper documentation

Great idea for a library, however by following your recomendation to read https://github.com/joshbuchea/HEAD  it's not really obvious as to how I should be structuring  the object inside head.
For instance, your example of setting the document title:
...
	head: {
		title: {
			separator: ' - ',
			inner: 'Foo',
			complement: 'Bar'
		}
	},
...

I am unable to find the words inner or compliment on https://github.com/joshbuchea/HEAD.

Yes, we do not find there the HEAD this structure, but had defined so because of the need for a more structured basis, similar to wordpress SEO.


[Bug]: SQL injection vulnerability of "/SqlInjection/assignment5a" path in Webgoat is not detected

Preflight Checklist

[X] I agree to follow the Code of Conduct that this project adheres to.
[X] I have searched the issue tracker for an issue that matches the one I want to file, without success.
[X] I am not looking for support or already pursued the available support channels without success.

Version
1.0.4
Installation Type
Official SaaS Service
Service Name
DongTai-Core
Describe the details of the bug and the steps to reproduce it

Project: WebGoat 8.2.2
uri: /WebGoat/SqlInjection/assignment5a
sqli source code:    @PostMapping("/SqlInjection/assignment5a")
@ResponseBody
public AttackResult completed(@RequestParam String account, @RequestParam String operator, @RequestParam String injection) {
    return injectableQuery(account + " " + operator + " " + injection);
}

protected AttackResult injectableQuery(String accountName) {
    String query = "";
    try (Connection connection = dataSource.getConnection()) {
        query = "SELECT * FROM user_data WHERE first_name = 'John' and last_name = '" + accountName + "'";
        try (Statement statement = connection.createStatement(ResultSet.TYPE_SCROLL_INSENSITIVE, ResultSet.CONCUR_UPDATABLE)) {
            ResultSet results = statement.executeQuery(query);

            if ((results != null) && (results.first())) {
                ResultSetMetaData resultsMetaData = results.getMetaData();
                StringBuilder output = new StringBuilder();

                output.append(writeTable(results, resultsMetaData));
                results.last();

                // If they get back more than one user they succeeded
                if (results.getRow() >= 6) {
                    return success(this).feedback("sql-injection.5a.success").output("Your query was: " + query + EXPLANATION).feedbackArgs(output.toString()).build();
                } else {
                    return failed(this).output(output.toString() + "
 Your query was: " + query).build();
                }
            } else {
                return failed(this).feedback("sql-injection.5a.no.results").output("Your query was: " + query).build();
            }
        } catch (SQLException sqle) {
            return failed(this).output(sqle.getMessage() + "
 Your query was: " + query).build();
        }
    } catch (Exception e) {
        return failed(this).output(this.getClass().getName() + " : " + e.getMessage() + "
 Your query was: " + query).build();
    }
}

Graph

Additional Information
No response
Logs
No response

The bug has been resolved by #313.


Use Cython to speed up all the statistical tests for non stationary bandit policies (CUSUM, PHT, GLR, Monitored etc)

I'm not sure if it will work, but I can (quickly) try!

[x] Actually, this notebook is/will also be a good test to do some simulations (and visualisations) of expected delay and false alarm probabilities of the change detection algorithm (as functions of T, nb of breakpoints, gap of breakpoints, parameters of the algorithm etc).
[x] Test on synthetic data, in a Jupyter notebook (with the %%cython cell magic command), if CUSUM, PHT, GLR, Monitored tests are (how much?) faster in Cython than in regular Python,
[x] Maybe also test with numba (@numba.jit) compiler!

Just forget about numba jit for kl functions, its useless:

It's so weird, how can the Cython version of Gaussian GLR is much slower than the naive Gaussian GLR ?

Demo of 2D color view of detection delay for Monitored algorithm:

Conclusion: Cython and numba are useless to speed up these functions.


filter private messages out of into the log stream

this simple change makes patchbay@6/minbay approach to private messages work much better. with out this, it loads 2.7 mb of messages on load, with this, only .5

oops, realized I had npm published but hadn't pushed to master. just pushed to 04f7023

I'm not sure how I managed to tag this with 'bug' when I was looking at it, but now it's untagged.

This is a good example of how NOT to write a pull request. The jerk (I can say that because it was me!) that made this PR implicitly expected that you'd read the code and understand the intention. It would have been much simpler to say "a 'private' option is added, and if it's set to true, only private messages are returned, but if it's false, no private messages are returned".
"this simple change" does not explain. I had forgotten about this and reading it again I couldn't figure out what this actually does. TISK TISK TISK past @dominictarr. closing


AMQP-532: Fix Pub.Conf. Nacks on close()

JIRA: https://jira.spring.io/browse/AMQP-532
Properly use the iterator to remove the entries.
cherry-pick to 1.4.x

... and cherry-picked


Remove deleted subscriptions

Just a random thought, could we script this on a cron? Or is there much benefit, depending how often we delete subscriptions.

not worth it, very rare to delete, I may have to manually intervene in this as well, will see.


All node tests are failing in interop tests

https://kokoro.corp.google.com/job/grpc/job/ubuntu/job/master/job/grpc_interop_tocloud/
Tests start to fail since build 470:
https://sponge.corp.google.com/target?id=e404c081-00e5-4033-af41-0130df6297f1&target=github/grpc&searchFor=&show=ALL&sortBy=STATUS
All tests timed out.
In this one: https://sponge.corp.google.com/target?id=6191a52e-051b-43db-aa2f-fe3320ca998d&target=github/grpc&searchFor=&show=ALL&sortBy=STATUS
Error becomes:
Failure
node: symbol lookup error: /var/local/git/grpc/src/node/extension_binary/node-v57-linux-x64/grpc_node.node: undefined symbol: grpc_timer_list_init

Does Node use CMake? It might be related to #12872

Node does not use CMake.
It looks to me like the problem is that the symbol is getting mangled by the C++ compiler, but the references to the symbol are not getting mangled. For this build, that symbol is defined in src/core/lib/iomgr/timer_uv.cc, so this specific error wouldn't be encountered in other builds.
@yashykt This seems related to your PR. Do you know what might have happened there?

I remember seeing similar errors (uv errors too) when I was creating my PR, but it was fixed by including the header which declared them as extern "C"

I see the problem. The function signature is wrong in the function declaration in timer_uv.cc, so it's not picking up the extern "C" declaration from the header file. I'll send a PR to fix it.

grpc_timer_list_init is declared in src/core/lib/iomgr/timer.h which IS included in timer_uv.cc

ah OK

Can this be closed?

Looks like it has been fixed in  #12885.


refactor(dbm-services): apigw/esb 鉴权信息放到header

关联研发信息
dev_branch: refactor/apigw_auth_info_to_headers/#471
target_branch: v1.2.0


Dc arb asset naming WIP

goal: fixes

#10

good progress:
~/projects/ag-power-tools/contract$ TRACK_TURNS=enabled DEBUG=track-turns yarn test test/test-arbAssetNames.js
...
    ℹ publish bundle arbAssetNames b1-d7fc9
    ℹ instance arbAssetName: new Promise
    ℹ instance arbAssetName settled; remaining: []
    ℹ Object @Alleged: InstanceHandle {}
    ℹ Larry launched {
        brand: Object @Alleged: BRD brand {},
        issuer: Object @Alleged: BRD issuer {},
      }
    ℹ Larry offers {
        brand: Object @Alleged: ZDEFAULT brand {},
        value: 100000000n,
      } to publish
    ℹ Larry published as board0371
    ℹ Object @Alleged: BRD brand {} at board0371


DEV UI Content - Manage locations

Describe the task

Lynn to implement content updates in DEV environment.
Content updates include business feedback, accessibility (AA+), and B.C. government content standards.

Acceptance Criteria

[x] Update Manage Locations (summary)
[x] Update Add/Edit Locations (FORM)
[ ] Update Welcome Page content to align (already done in another ticket)
[x] Polish/enhance CSS (aka UI) -- tooltips

Additional context

This work is in Content Release 2. Noting for documentation purposes only. See MOD FO Channel for release details.

In addition, this work has been split up in two tickets #1543

Done - demoed to Phil (who requested the changes)


TestCafe Server - Implement an analog of Selenium Grid

Allows to perform distributed test execution.
See more details in selenium_grid
https://testcafe-discuss.devexpress.com/t/testcafe-grid-for-ci-systems/1235/6

POC for remote workers:
https://github.com/psi-4ward/testcafe-hub

Updated the POC and added testcafe-browser-provider-hub

We decided to not implement this enhancement and focus on TestCafe core features.


Feature Request - Only show changes (within a neighborhood)

Hi James,
I'm trying to compare draft reconstructions of genome scale models for closely related strains of a species. Does sbml-diff have a functionality to only print the changes (maybe with a relative neighborhood) for the models? This would really be just a first pass, "what's in my model" kind of thing. The outputs are currently very large, which is unwieldy both in time and display.
I poked around in the help and code and didn't see anything obvious, but I thought I'd ask. If not, would you be interested in PRs? Thanks,
Erik

This is not currently implemented, but is a good idea. I'd welcome to receive a PR that implements this, if you have time to write one.
The original application for sbml-diff comparing models of genetic circuits in synthetic biology, which are typically quite small. It would be good to increase the size of models to which the tool can be usefully applied.
Implementation
The main sbml-diff.py script creates an object that will produces the output DOT code.
This has a  generate_dot function that gets called with an argument that is an object containing the union of all features (nodes or arrows) in the all input models, annotated with a list of which models contain them.
You would need to modify this function, so that it first identifies which features are in the neighborhood of a a change, and prints DOT code only for these.
A feature is changed if it is only present in some of the models (self.num_models != len(model_set)), the stoichiometry for an arrow differs between models (stoich = '?' ), or the rate law of a reaction node differs between models (rate_law == "different").

Seems reasonable - I'll take a crack at this and let you know if I have any questions.

Did you have time to try this? If so, did you get it working or encounter problems?

I don't remember why, but I added py3 compatibility (ENH_Py3 branch) and some whitespace nitpick changes that Atom did automatically, but lost progress on the project I was trying to use this for about around September and didn't have any substantial changes.
I'm definitely still interested in doing this, but I don't have the time at the moment. Are you doing a push on it now?

Also happy to rebase that since master is almost a year old. The changes were pretty small IIRC

I also added py3 compatability, and made a set of changes that look pretty similar (except that I enocuntered some problem with six.StringIO, so instead use either io.StringIO and io.BytesIO depending on whether 2 or 3 is being used ).
I think this would be a useful feature, but like you I'm fairly busy with other things at the moment.

Yeah, I saw that you'd already pulled a lot of these in. Sorry that I didn't push earlier; it would've saved some work. Out of curiosity, what problems did you find with six?
Again, I agree but it looks like it'll have to wait. When I come back to metabolic modelling again I'll check in with the issue, and push earlier.


docs(TextBox): remove old typography style prop from docs

Description

This prop was removed several weeks ago, but it is still in the documentation.
References

Testing

Automation

Checklist

[ ] all commented code has been removed
[ ] any new console issues have been resolved
[ ] code linter and formatter has been run
[ ] test coverage meets repo requirements
[ ] PR name matches the expected semantic-commit syntax

Sanity Tests Execution Passed.

:tada: This PR is included in version @lightningjs/ui-components-v2.2.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Remove automatic new untitled file in empty workbench

From #15547:  I don't see an effect of the untitled file experiment on the number of times an untitled file is used. It appears that creating an untitled file is intuitively understood by new users. Also users don't create new files that frequently, they read/work on existing files much more often it seems, that reduces the value of the automatically opened new untitled file further.
Part of the outcome of #15383.

Yes, I'm a newcomer: it's rather annoying that empty workbench does not mean empty workbench, and the setting to disable this is too well hidden.  It's this one, if anyone else is stuck like I was:
"telemetry.experiments.openUntitledFile": false

I don't see the telemetry.experiments.openUntitledFile setting in the current latest VS Code (v1.13.0), so it looks like this feature still needs to be implemented.

Will fix with #12387

@deadlydog I'm not surprised you didn't find this covert setting, but telemetry.experiments.openUntitledFile has successfully removed the untitled file annoyance for me and continues to do so in v1.13.0.


Disable analytics temporarily

Hi! Thanks first of all!
Could you add this feature https://firebase.google.com/support/guides/disable-analytics ?

Ha, ok, I forgot that it's opensource !!! :)
I'll create PR soon...

Done https://github.com/evollu/react-native-firebase-analytics/pull/3

merged and published


vscode unable to load $schema from "https://developer.microsoft.com/json-schemas/teams/v1.8/MicrosoftTeams.schema.json"

As Creating a manifest section suggests at the green-tip section, adding a "$schema" property on a json document would enable the IDE (and in particular VSCode) to hint and lint while editing.
For some reason,
{
  "$schema": "https://developer.microsoft.com/json-schemas/teams/v1.8/MicrosoftTeams.schema.json",
  ...

does not work. Attaching an image:

As workaround, I've downloaded the .json file to local, and referencing the schema as relative path does the trick:

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

ID: d0b75ec1-0ded-400e-7291-b7776d510ae4
Version Independent ID: 157f3a51-42f6-f70f-a599-6775fcb34ed1
Content: Package your app - Teams
Content Source: msteams-platform/concepts/build-and-test/apps-package.md
Product: msteams
GitHub Login: @laujan
Microsoft Alias: lajanuar

Could you please check this sample example for manifest?

yep, that's the one i was using for the screenshot:

Could you please try adding the language to the schema. Please try the link given below:
https://developer.microsoft.com/en-us/json-schemas/teams/v1.5/MicrosoftTeams.schema.json

that's it.
so maybe, i'd say to update the example at https://docs.microsoft.com/en-us/microsoftteams/platform/resources/schema/manifest-schema#sample-full-manifest
and change, from:
"$schema": "https://developer.microsoft.com/json-schemas/teams/v1.8/MicrosoftTeams.schema.json",

to:
"$schema": "https://developer.microsoft.com/en-us/json-schemas/teams/v1.8/MicrosoftTeams.schema.json",

right?

Just a minor doc change at https://docs.microsoft.com/en-us/microsoftteams/platform/resources/schema/manifest-schema#sample-full-manifest right?

@KirtiPereira - could you please update the manifest schema reference URL to following in Creating a manifest section's tip?
https://developer.microsoft.com/en-us/json-schemas/teams/v1.9/MicrosoftTeams.schema.json


Monorepo lists ambiguous "package.json" for all parallel lint-staged tasks

During a large merge operation, lint-staged is showing an unhelpfully ambiguous reference to the failing tasks as just "package.json".

It used to say which file is in violation, but something in the ecosystem changed so now that info is not included (not sure which part has that responsibility report the specific file name).
Is there a way to fix that "package.json" text to something more helpful?

Please post your debug logs.


aiohue.sensors missing?

Hey there, I followed the steps and added the hue_remote folder to custom_components
➜  homeassistant git:(master) ✗ ls custom_components/hueremote
implemented_remotes.py  __init__.py  manifest.json  __pycache__  remote.py

I am getting the following error
Platform error remote.hueremote - cannot import name 'TYPE_ZLL_ROTARY' from 'aiohue.sensors' (/usr/local/lib/python3.7/site-packages/aiohue/sensors.py)

Any ideas what I need to do to fix this?

Hi, @onlymejosh.
I solved this problem by updating home assistant.


Fix syntax issue in playwrite tests

The file had some characters that looked like colons but were not the
colons that JavaScript was looking for.

Repoman Generation Results
Repoman pushed changes to remotes for the following projects:
Project: todo-csharp-cosmos-sql
Remote: azure-samples-staging
Branch: pr/546
You can initialize this project with:
azd init -t Azure-Samples/todo-csharp-cosmos-sql -b pr/546

View Changes | Compare Changes

Project: todo-csharp-mongo
Remote: azure-samples-staging
Branch: pr/546
You can initialize this project with:
azd init -t Azure-Samples/todo-csharp-mongo -b pr/546

View Changes | Compare Changes

Project: todo-csharp-sql
Remote: azure-samples-staging
Branch: pr/546
You can initialize this project with:
azd init -t Azure-Samples/todo-csharp-sql -b pr/546

View Changes | Compare Changes

Project: todo-nodejs-mongo-aca
Remote: azure-samples-staging
Branch: pr/546
You can initialize this project with:
azd init -t Azure-Samples/todo-nodejs-mongo-aca -b pr/546

View Changes | Compare Changes

Project: todo-nodejs-mongo-swa-func
Remote: azure-samples-staging
Branch: pr/546
You can initialize this project with:
azd init -t Azure-Samples/todo-nodejs-mongo-swa-func -b pr/546

View Changes | Compare Changes

Project: todo-nodejs-mongo
Remote: azure-samples-staging
Branch: pr/546
You can initialize this project with:
azd init -t Azure-Samples/todo-nodejs-mongo -b pr/546

View Changes | Compare Changes

Project: todo-python-mongo-aca
Remote: azure-samples-staging
Branch: pr/546
You can initialize this project with:
azd init -t Azure-Samples/todo-python-mongo-aca -b pr/546

View Changes | Compare Changes

Project: todo-python-mongo-swa-func
Remote: azure-samples-staging
Branch: pr/546
You can initialize this project with:
azd init -t Azure-Samples/todo-python-mongo-swa-func -b pr/546

View Changes | Compare Changes

Project: todo-python-mongo
Remote: azure-samples-staging
Branch: pr/546
You can initialize this project with:
azd init -t Azure-Samples/todo-python-mongo -b pr/546

View Changes | Compare Changes


phabulous serve just hangs and doesn't bind on an ip address

I've followed the documentation to set up phabulous, but when I run phabulous serve, it seems to spawn three children and then just sits there, with out binding up on any IP address.
Is there a way to log what's going on to help troubleshoot this?

 Got some help compiling the binary.

:+1:


add server port in config

Use port from config instead PORT var in global env

isn't needed for Heroku to have it as an env var?

I am not an expert of Heroku, but they suggest to use env var for every kind of configuration. The PR https://github.com/lvaccaro/btctipserver/pull/18 supports env configs with also port setting.


Create hyperlink in videos

it would be nice if we could link vizzes, like the latest gage condition viz to the midwest flood viz video

Seems like several paid-for softwares can do this, but it also can be done with YouTube

Youtube, instagram & facebook how-tos:
https://lumen5.com/learn/how-to-add-clickable-links-to-your-videos/


Propogate provided dag_display_name to built dag

This PR propogates the dag_display_name value set in the yaml dag config to the built dag object.
Preview of the built dag with dag_display_name being set.

closes: #233

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

Project coverage is 93.31%. Comparing base (c171166) to head (01dde07).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #326      +/-   ##
==========================================
+ Coverage   93.29%   93.31%   +0.01%
==========================================
  Files          10       10
  Lines         776      778       +2
==========================================
+ Hits          724      726       +2
  Misses         52       52

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Adding arbitary data to sev-snp attestation report

Hi,
In SEV-SNP how we can add arbitrary data to attestation report file using vTPM NVRAM?
I'm using the following commands to extract the report from an azure VM:

 % tpm2_nvread -C o 0x01400001 > ./snp_report.bin
% dd skip=32 bs=1 count=1184 if=./snp_report.bin of=./guest_report.bin

Hi, this is relevant for us as well. Is there any update on this?

The way to do this in an SNP CVM is by including the arbitrary data in a TPM quote via "nonce" parameter, the TPM quote signing key is included in the SNP report. Follow the process here: https://github.com/Azure/confidential-computing-cvm-guest-attestation/issues/18#issuecomment-1490142453.
The SNP report is only generated once during boot and is used to attest the key in the TPM. In this architecture, allowing users to attest arbitrary data through the SNP report would allow spoofing the report for an arbitrary non-TPM key and bypass the whole security architecture. So this is not possible.


token strategy/pkg  docs

Tasks:

add docs to token strategy (godoc)
add links to the readme (api-key, bearer, x-header)

Hey, I just stumbled on this project and was going to begin using for a personal site. Are you still looking for assistance on this issue?
I realize this is a result of the changes from version 1.x to 2.x. My understanding is that the bearer type is being deprecated, and the walkthough (https://medium.com/@hajsanad/authentication-in-golang-using-go-guardian-b1cd47da47a0) isn't valid for version 2.x.
I glanced at the _example/ directory and didn't see an entry for token. I'm not sure if that is part of the work required here, or something else. Please let me know if I can help with some of the docs, examples here.
Thanks,
Brian

@briwagner
first of all, thank you for reaching out.
FYI, moved to v2 for more scalability and maintenance.
IIRC, https://github.com/shaj13/go-guardian/tree/master/_examples/basic_bearer cover the token and basic strategy may we need to rename the folder.
honestly, I will be glad if you could assist,
I would like to start from the readme so we can welcome users and navigate them in the right direction while explains the project.
let me know what you think.

Yes, I'm happy to help where you think it's needed. Let me know what is most helpful at this point, and I will try to suggest some changes.
I agree: it may be simply renaming the folder in _examples/, to "token". When I looked initially I was confused there was no entry for token. I expected the folder list for auth/strategies/ to match _examples/. Just my initial expectation.
My use case was basic and JWT token, so that's something I'm comfortable with. I don't have experience with the other strategies.
But let me know what you have in mind for expanding the README, and if I can help. Just as a comparison, I see how netlify/gotrue walks through more of the details for each method, and has more code examples. Maybe too many? It's almost a bit of info overload, but if you know what you're looking for in there, it can be helpful.

Hello, I started to use this package yesterday.
I have a little question, related to the JWT Strategy  - what if I want to change the ID to the user ID
func setupGoGuardian() {
	keeper = jwt.StaticSecret{
		ID:        "TEST", <<<<<<<<<<
		Secret:    []byte("JWTSecret"),
		Algorithm: jwt.HS256,
	}
	cache := libcache.FIFO.New(0)
	cache.SetTTL(time.Minute * 5)
	cache.RegisterOnExpired(func(key, _ interface{}) {
		cache.Peek(key)
	})
	basicStrategy := basic.NewCached(validateUser, cache)
	jwtStrategy := jwt.New(cache, keeper)
	middlewares.Strategy = union.New(jwtStrategy, basicStrategy)
}

My problem is: The ID is dynamic and based on the user from the DB.
How can I achieve this?

@briwagner
awesome let's start by re-organize the examples.
split the basic_bearer to basic and token.
the token example can use the basic similar to jwt example
regarding README maybe something similar to cobra what do you think?

@briwagner created an issue for example #104, feel free to take it.

Meant to follow up on this: I'm adding a PR for #104


feat!: Add headers in unique key computation

Description

Adds headers in unique key computation

Issues

Closes: #548

Testing

Added tests in [tests/unit/_utils/test_requests.py]

Checklist

[x] CI passed

why was this marked as breaking change?

The interface remained the same, but the computation of the extended unique key is changed. It will be released in 0.4 together with #542.


Feature driver view on bus simulation #37

What does this PR do?

Simulate the bus motion

Description of Task to be completed?
As a driver, I want to be able to command the bus to start, stop, speed up, and slow down the bus so that the bus can move.

The driver should be able to command the bus to start, move and stop.
The driver should be able to update the bus info on bus stops (those that alighted and left)
The driver should be able to command the bus to speed up and slow down
The page is responsive.

How should this be manually tested?
Clone the repo
checkout to branch ft-driverView-on-map-37
pull the changes by git pull
install all packages by running npm install
start the app by running npm run start
Any background context you want to provide?
N/A
What are the relevant pivotal tracker stories?
[Trello Board

 Simulate the bus motion


RelationshipsBinder: support binding just the ID

In case the JSON:API server does not send an "included" section that includes all referenced objects, it would be helpful to also be able to just bind the ID of an object referenced with a ToOne or ToMany relationship to a field in the Kotlin class.
This PR implements this feature by checking if the type of the field annotated with @ToOne or @ToMany is String, and if yes, it simply assigns resourceIdentifier.id to that field instead of trying to find a matching resource in the "resources" or "included" sections.

Hi, thanks for the PR and I apologize for the late reply.
Although this is an interesting approach I wouldn't go in this direction in order to keep things simple for @ToOne and ToMany annotations.
There are two different approaches to the case where you want to access the "resource identifier" (or just id strings) of the relationships that are not present in the document (e.g. no included)
1. Use of @RelationshipObject as per the following example:
  @Resource("people")
  class Person {
    // ....
    @RelationshipsObject val relationships: Relationships? = null
    // ...
  }

Then your relationships that are not included will be null (indicating that those are missing in the document) but you will be able to access the "resource identifier":
  val resourceIdentifier = author2.relationships?.get("fieldname")
  when (resourceIdentifier) {
    is Relationship.ToOne -> resourceIdentifier.data?.id
    is Relationship.ToMany -> resourceIdentifier.data.map { it.id }
  }

Upon this example you can:

create your own kotlin extension functions for easier access and/or mapping to string ids
create a simple Moshi adapter that will access and assign your custom annotated fields

2. If your API is not returning included it might include ids as a standard attribute
{
  "data": [{
    "type": "articles",
    "id": "1",
    "attributes": {
      "title": "JSON:API paints my bikeshed!",
      "authorId":"1423",
      "tagIds":["17654","234564","45675"]
    }
}

Hi, thanks for the reply!
If the main reason is that the @ToOne and @ToMany annotations should be kept simple, we could of course also use new annotations for the purpose of binding just the ID...
Your second approach is not an option for me as I'm consuming an external API and can't change what the server sends.
The first approach is possible, however it forces me to store the whole Relationships object as a field in my data class (which makes it a lot more cumbersome to e.g. make the data class implement Parcelable, see https://github.com/johan12345/EVMap/commit/7cc50d7127da7a49312a404884371bf2b2eb29d4). It seems to not be possible to just have a constructor parameter annotated with @RelationshipsObject, which is then used to extract the necessary ID(s) and store them in a field.


Anthropoworphous ver

another attempt

Press the merge button urself I won't make such a big edit for you without permission


Qbee

16 bit audio input and output with sample rate 44100 and we cleared the not used code

thanks


Reorganizes Docs and Adds HTTPRoute Resource Doc

apis/v1alpha1/gateway_types.go and apis/v1alpha1/httproute_types.go: Fixes minor godoc errors.
docs-src/httproute.md: Adds docs for detailing the HTTPRoute API type.
All other changes are related to reorganizing documentation and content generation.

/assign @robscott @bowei @hbagdi
xref: https://github.com/kubernetes-sigs/service-apis/issues/332

@robscott I addressed your comments either above or in commit bea1950, PTAL.

Thanks!
/lgtm
/approve


Create bar.txt

See title. Opened from fork, francisfuzzy/on-deployment-trigger-workflow.

Creating a deployment
Docs: https://developer.github.com/v3/repos/deployments/#create-a-deployment
curl -H "Authorization: token $TOKEN" https://api.github.com/repos/francisfuzz/on-deployment-trigger-workflow/deployments -X POST -d '{"ref":"0a981c2296fa022ad49a082fc7083fb046c19b70"}'

{
  "url": "https://api.github.com/repos/francisfuzz/on-deployment-trigger-workflow/deployments/201441501",
  "id": 201441501,
  "node_id": "MDEwOkRlcGxveW1lbnQyMDE0NDE1MDE=",
  "sha": "0a981c2296fa022ad49a082fc7083fb046c19b70",
  "ref": "0a981c2296fa022ad49a082fc7083fb046c19b70",
  "task": "deploy",
  "payload": {

  },
  "original_environment": "production",
  "environment": "production",
  "description": null,
  "creator": {
    "login": "francisfuzz",
    "id": 15894826,
    "node_id": "MDQ6VXNlcjE1ODk0ODI2",
    "avatar_url": "https://avatars0.githubusercontent.com/u/15894826?v=4",
    "gravatar_id": "",
    "url": "https://api.github.com/users/francisfuzz",
    "html_url": "https://github.com/francisfuzz",
    "followers_url": "https://api.github.com/users/francisfuzz/followers",
    "following_url": "https://api.github.com/users/francisfuzz/following{/other_user}",
    "gists_url": "https://api.github.com/users/francisfuzz/gists{/gist_id}",
    "starred_url": "https://api.github.com/users/francisfuzz/starred{/owner}{/repo}",
    "subscriptions_url": "https://api.github.com/users/francisfuzz/subscriptions",
    "organizations_url": "https://api.github.com/users/francisfuzz/orgs",
    "repos_url": "https://api.github.com/users/francisfuzz/repos",
    "events_url": "https://api.github.com/users/francisfuzz/events{/privacy}",
    "received_events_url": "https://api.github.com/users/francisfuzz/received_events",
    "type": "User",
    "site_admin": true
  },
  "created_at": "2020-02-13T02:12:34Z",
  "updated_at": "2020-02-13T02:12:34Z",
  "statuses_url": "https://api.github.com/repos/francisfuzz/on-deployment-trigger-workflow/deployments/201441501/statuses",
  "repository_url": "https://api.github.com/repos/francisfuzz/on-deployment-trigger-workflow"
}

This deployment is reflected in the UI: https://github.com/francisfuzz/on-deployment-trigger-workflow/pull/2#event-3033166016
But, no workflow runs were triggered for this workflow run 🤷♂


Update SwotTest.kt

@L1F21BSSE0127 src/test/kotlin/swot/SwotTest.kt: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.


2.4.0 Release Plan

2.4.0 Release Plan
Release Plan that outlines the focused tasks for lit-element 2.4.0.
Milestone
Notable Changes

Enhancements

[x] Add query decorator options
[x] As a developer I would like to do sync actions on property changes

Bug Fixes

[x] queryAssignedNodes doesn't correctly locate default slot

Documentation
Lots of committed document changes, already deployed. No pending documentation changes.
Release Process

[x] Run Google Internal Tests
[ ] 2.4.0 Release Notes
[ ] Verify 2.4.0 CHANGELOG

Excited for this, as I've just refactored something to use queryAssignedNodes only to find the issue that'll be fixed in this release. Is there a rough timeline that you could share?


IPython Matplotlib QT Backend RuntimeError

When starting Spyder with the QT5 graphics backend the following error is generated:
Traceback (most recent call last):

  File "<ipython-input-1-64557277dbff>", line 1, in <module>
    get_ipython().run_line_magic('matplotlib', 'qt5')

  File "C:\Anaconda3\lib\site-packages\IPython\core\interactiveshell.py", line 2095, in run_line_magic
    result = fn(*args,**kwargs)

  File "<decorator-gen-108>", line 2, in matplotlib

  File "C:\Anaconda3\lib\site-packages\IPython\core\magic.py", line 187, in <lambda>
    call = lambda f, *a, **k: f(*a, **k)

  File "C:\Anaconda3\lib\site-packages\IPython\core\magics\pylab.py", line 99, in matplotlib
    gui, backend = self.shell.enable_matplotlib(args.gui)

  File "C:\Anaconda3\lib\site-packages\IPython\core\interactiveshell.py", line 2983, in enable_matplotlib
    self.enable_gui(gui)

  File "C:\Anaconda3\lib\site-packages\ipykernel\zmqshell.py", line 482, in enable_gui
    real_enable_gui(gui)

  File "C:\Anaconda3\lib\site-packages\ipykernel\eventloops.py", line 337, in enable_gui
    raise RuntimeError("Cannot activate multiple GUI eventloops")

RuntimeError: Cannot activate multiple GUI eventloops

What steps will reproduce the problem?

Start with a clean spyder environment (delete the old .spyder* directory)
Start Spyder
Change the matplotlib backend
a. Preferences -> IPython console -> Graphics -> Graphics backend -> QT5/QT4? (No PyQt4 installed, can't check)
Restart the IPython kernel

What is the expected output? What do you see instead?
No output is expected ... plots should display in external windows.
Please provide any additional information below
This is actually not a Spyder error as it is coming from IPython, but I tried running a similar set of commands in an IPython notebook (with Jupyter) and didn't receive this error:
import matplotlib.pyplot

%matplotlib qt5
matplotlib.pyplot.plot()

%matplotlib inline
matplotlib.pyplot.plot()

%matplotlib qt5
matplotlib.pyplot.plot()

Something about the way Spyder is attaching to the eventloop with IPython seems to not be working correctly.
Workaround
I run a setup script instead of leaving some of the setup to Spyder because the options aren't available in the GUI.  I modified my script to address this by starting with an inline backend, and immediately switching to the QT5 backend.  It works, but I'd rather leave this piece to Spyder because if I switch settings (which I do from time to time) I don't want to have to remember that I hacked it into my environment setup.
from IPython import get_ipython
ipython = get_ipython()
try:
    # Some issue irrelevant commands
    ipython.magic("matplotlib inline"); ipython.magic("matplotlib qt")  # Needed to add this because pylab was giving a RuntimeError when attaching QT to the eventloop
    ipython.magic("pylab --no-import-all")
except AttributeError:  # Not in an IPython environment
    pass
del ipython

Version and main components

Spyder Version: 3.2.4
Python Version: 3.6.3
Qt Versions: 5.6.2, PyQt5 5.6 on Windows

Dependencies
pyflakes >=0.6.0 :  1.6.0 (OK)
pycodestyle >=2.3:  2.3.1 (OK)
pygments >=2.0   :  2.2.0 (OK)
pandas >=0.13.1  :  0.21.1 (OK)
numpy >=1.7      :  1.13.3 (OK)
sphinx >=0.6.6   :  1.6.3 (OK)
rope >=0.9.4     :  0.10.7 (OK)
jedi >=0.9.0     :  0.11.0 (OK)
nbconvert >=4.0  :  5.3.1 (OK)
sympy >=0.7.3    :  1.1.1 (OK)
cython >=0.21    :  0.27.3 (OK)
qtconsole >=4.2.0:  4.3.1 (OK)
IPython >=4.0    :  6.2.1 (OK)
pylint >=0.25    :  1.7.4 (OK)

Thanks for such a detailed report! Can you try this with a new instance of plain qtconsole, and see if the problem reoccurs? if so, you'll want to submit your bug there since the problem would presumably be in that codebase, not here.

I run a setup script instead of leaving some of the setup to Spyder because the options aren't available in the GUI

If you do this, you can simply turn off the option called Activate support in
Preferences > IPython console > Graphics

That way, you'll be completely in charge of setting your preferred Matplotlib backend, and Spyder won't interfere with that.

@CAM-Gerlach You're right ... it's a QtConsole issue.  If you run
get_ipython().run_line_magic('pylab', 'qt')

the same error emerges.
I will report it there.  Closing.


idleTimeout does not work with Jetty backend

When a service yields a task that takes a long time to produce a response (see /slow in the code below), the Blaze backend will drop the socket after idleTimeout has elapsed. The Jetty backend:

Keeps the connection open for 30 seconds, ignoring idleTimeout.
Then returns a 200 OK with no body.

import scala.concurrent.duration._

import scalaz.concurrent._

import org.http4s.dsl._
import org.http4s.server._

object JettyTest extends App {
  val service = HttpService {
    case GET -> Root / "slow" => for {
      _    <- Task.delay { Thread.sleep(45*1000) }
      resp <- InternalServerError("finally!")
    } yield resp

    case GET -> Root / "fail" => Task.fail(new RuntimeException)
  }

  val js = org.http4s.server.jetty.JettyBuilder
    .withIdleTimeout(1.second)
    .bindHttp(8080, "<IP_ADDRESS>")
    .mountService(service)

  val bs = org.http4s.server.blaze.BlazeBuilder
    .withIdleTimeout(1.second)
    .bindHttp(8180, "<IP_ADDRESS>")
    .mountService(service)

  js.run
  bs.run
}

Using http4s 0.8.2.

I get this same behavior mounting a raw HttpServlet that sleeps in service, so we can exonerate Http4sServlet.

This seems more like a use case for asyncTimeout, which is how long the Task has to complete.  This had a regression addressed by #341.
I can't make the idle timeout work, even removing http4s from the picture.  That setting doesn't seem to do what I'm reading it to do.

Could be. I observed idleTimeout was getting into the Jetty connector instance, but it didn't seem to affect anything. On the Blaze side, setting idleTimeout does cause the connection to be dropped if the Task hangs.
Part of the problem here is the lack of any documentation of what these parameters are meant to do, and part of the problem is the inconsistent behavior across the backends.
To be safe, should I be using asyncTimeout on Blaze?

@mossprescott, I don't think the blaze server supports the notion if an asyncTimeout: I believe that is a servlet concern when initiating an async context. I believe the idleTimeout is the only way to timeout on blaze.

The idea is that idleTimeout is for inactivity on the socket connection, and asyncTimeout for a failure to complete, even if activity is ongoing.  asyncTimeout's name is definitely of servlet origin, but I can't offhand see why Blaze couldn't support something similar.  This would probably look a lot like the timeout middleware, which races a task against the response, and returns the first completed.  (Note: I have a beef with the default status code: #345).
This is going to be a constant struggle with abstracting the backends.  The servlet specification does not have much to say about connection lifecycles, which naturally gives rise to some differences between Jetty and Tomcat.  Blaze, being under our control can mimic a servlet container's capabilities where desired.  But then which container's?  The idea is for the server builders to identify similar behaviors between backends and give them the same name.  We'll fill the holes in this abstraction the best we can, but I think it's doomed to leak at least a little.

We do need a 'took too long to initiate the response' timeout for blaze that will return a default response of some form, preferable with a good default but can be set by the user.
I don't know if that is what the asyncTimeout is technically about; I was under the impression it was for situations where you start your async context and forget to use it (or error and forget to close it) so the servlet doesn't want to just let it float in the ether indefinitely.

The asyncTimeout is about taking too long to finish the response, not initiate it.  This is good if you have a hung process, and sucks if you have a process that's taking its time but making satisfactory progress.

Actually, our requirement was just to effectively disable anything like asyncTimeout (see slamdata/slamengine#815). So whatever parameters are available, we need a way to say "let this response go for as long as x", where x is at least several minutes. And we would eventually probably like that to apply to only particular requests which we know can run for a long time, but that seems to be covered by #282.
From the http4s user's point of view, It would be helpful if there was a consistent and documented set of tuning parameters, even if they aren't all supported by all the backends.
Secondarily, the behavior of the Jetty backend returning 200 OK after timing out is confusing and wrong (see also #337). Hopefully that's something that can be addressed in http4s.


"error: memory read failed for 0x0" on iosSimulator

I tried to upgrade to 1.2.0-alpha01-dev713 and it compiles fine, but when I run it, I get an crash:
error: memory read failed for 0x0

this is my demo: https://github.com/qdsfdhvh/compose-ios-bootstrap

I tried it again, It was CenterAlignedTopAppBar and NavigationBar that caused the crash，but i don't know why.
Scaffold(
    topBar = {
        // CenterAlignedTopAppBar()  { ... }   // crash
        // Box() { ... }   // ok
    },
    bottomBar = {
        // NavigationBar() {  ... }   // crash
        // Box() {  ... } // ok
    }
)

I find that Image also crash it:
demo: https://github.com/qdsfdhvh/compose-uikit-sample

It's fix in 1.2.0-alpha01-dev716, Thx.

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.