Symfony 5 compatibility

tested by running phpunit locally

This should close #922 and I don't expect any issue with Symfony 5 anyway, but let's wait until we can really test it in the CI because right now the highest jobs are using the components at version 4.4

Symfony 5 got released (live on stage at SymfonyCon, I've seen it! :smile: )
I'll restart the CI job...

@Jean85 That's not a blocker for this package. If I don't use php-cs-fixer, I could already use these

The problem is not that it's blocking the installation for someone but that we cannot test the compatibility with Symfony 5 as the test suite cannot be executed to prove that it works, even though as I said I'm sure that it would

Yes, those are 2 different problems.

Ref https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/4608

What about removing CS Fixer as dependency, and start using https://github.com/jakzal/phpqa or a Github action?

@Jean85 PHP-CS-Fixer updated, and now we can restart checkers?

@rvanlaak can you please force-push? I can restart the Travis CI build but not AppVeyor's one

Force push ✔ checks are still green

It seems that CS-Fixer has the compat on the 2.15.4, but Composer is preferring 2.16.0, which isn't compatible. Also, PHPStan is getting in the way too.
PHPStan will be released as a PHAR from 0.12 so it will auto-resolve, CS-Fixer needs just a tag (the compat is already merged in the 2.16 branch).

There is no need to share dependencies with an independent tool like CS-Fixer. What about removing CS-Fixer and running it via the phpqa docker image?
What do you think about merging this PR together with the Symfonyt 5 changes? https://github.com/scribbr/sentry-php/pull/1

What about removing CS-Fixer and running it via the phpqa docker image?

There is no reason to use Docker to run in the CI such tools as using the PHARs should be enough. PHPStan 0.12 will be released December 3rd, so let's wait until that date and then let's see what's the status of the dependencies and act accordingly

Yeah, tools like php-cs-fixer shouldn't install via composer as dev dependency. You can download it in the CI as phar archive and use that. Otherwise, you will always have the problem that the dependencies do not fit. Either the lib can not be tested with the latest version or the tools can not be updated.

Yeah, tools like php-cs-fixer shouldn't install via composer as dev dependency

Sometimes it's convenient to install it that way as you don't have to deal with downloading the PHARs, and sometimes it's necessary as you have to load extensions and it wouldn't be possible by using PHARs: one such example is PHPStan, that's way I'm curious of understanding how 0.12 solves the issue

For now it is CS Fixer that is blocking this PR 👍  ... that is why somehow removing it as dependency would be great.
$ composer why-not symfony/options-resolver:^5
friendsofphp/php-cs-fixer  v2.16.0     requires  symfony/options-resolver (^3.0 || ^4.0)

https://github.com/getsentry/sentry-php/pull/925#pullrequestreview-321716954
If the phar is the way to go, I will make another PR for that.

You can just require "friendsofphp/php-cs-fixer": "^2.17@dev"until a new stable tag will be released.
I tried locally and it works, installing Symfony 5 components (and with green tests)

If the phar is the way to go, I will make another PR for that.

No need for another PR, we can use this one :wink: but again, I want to wait PHPStan 0.12 so that we don't have to make one more PR to update how we fetch its PHAR

You can just require "friendsofphp/php-cs-fixer": "^2.17@dev"until a new stable tag will be released.

It's not a great idea to use a non-stable dependency, isn't it?

You can just require "friendsofphp/php-cs-fixer": "^2.17@dev"until a new stable tag will be released.

It's not a great idea to use a non-stable dependency, isn't it?

You can do it only on your non-stable release, then everything matches

You can do it only on your non-stable release, then everything matches

That's not a solution, we want to release a patch and declare compat with SF5. Surely making us indipendent from dev tooling is the best approach, but I agree that a Docker container is a bit overkill here; also, we need to take into account local DX, otherwise noone will fix CS locally and we will get a lot of errors in future PRs.

PHP-CS-Fixer has been released, @rvanlaak please can you trigger the build one more time?

LGTM then :+1:

Did force-push to rerun CI ✅

Thank you, even though it was not needed as @Jean85 closed and reopened the PR to forcefully trigger the CI. Everything's fine, LGTM


Using TCK Tested JDK builds of OpenJDK  in GH actions

The AdoptOpenJDK has been discontinued since July 2021. When using Zulu you get all the latest updated (TCK Tested) builds for all versions of OpenJDK.

Thanks for the update 👍

Thanks for the update 👍

No problem. Glad to help contribute!


Optionally remove top level headings

I'm wondering if there might be a good way to optionally remove the top level heading from the content to be excerpted. Two fairly common patterns for markdown files is to insert either a
# Heading at the Top of the File

or a
# Heading at the Top of the File

## Followed By a Second Heading

For an except it doesn't make much sense to grab headings, so to be able to at least optionally ignore all headings would be great.

Indeed, will be added to the next release. Thanks for your suggestion.
Stay tuned. 😌


Completion of new features

It’s been quite a while since anything new has been added. What’s going on?

https://forum.unity.com/threads/released-realtime-csg-level-design-for-unity.427601/page-8#post-3514790

Oh Ok, I really understand why you would have been really stressed out with working on Realtime-CSG while working at Unity and well I thank you for that. You have done a tremendous job with it! And I am happy for your decision to work with unity now and put aside Realtime-CSG. You have really helped me out with Level Design and possibly many others is well! Don't want to sound like a asshole or anything but maybe it would be better to maybe integrate some of Realtime-CSG features in with SabreCSG, as your still getting your name out there and your helping others., it could also be better used as more and more people are using it now

Thanks :)
I'm still going to make sure that Realtime-CSG will keep working with the latest version of Unity etc.
I Just can't work on it as before.
I still haven't decided what to do with Realtime-CSG long term


Code Summary Table

Description
Added code summary table with badges.
Issues Resolved

Deleted old badges
Updated badges for code cov
Updated documentation link

Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #314 (f45bda4) into main (b1ff77c) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main     #314   +/-   ##
=========================================
  Coverage     99.90%   99.90%
  Complexity     2691     2691
=========================================
  Files           256      256
  Lines          6533     6533
  Branches        422      422
=========================================
  Hits           6527     6527
  Misses            5        5
  Partials          1        1

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 b1ff77c...f45bda4. Read the comment docs.

@davidcui1225

LGTM- please re-run the Link Checker workflow before merging ✌
Link checker is fixed in https://github.com/opensearch-project/sql/pull/313


Added Gulp to generate minified js

cli version is much simpler. And don't need gulp

Oh. I didnt notice that. Will fix it asap. :)

We still need command. Use npm scripts please.

:+1: nice one guys, thanks


Comparisons fail with nested arrays retrieved from JSON.parse

Godot version:
3.1 / 3.1.1
Issue description:
After retrieving an array of arrays from JSON.parse, inner arrays are not comparable with normal arrays in any way. Any comparison will return false.
Steps to reproduce:
var not_from_json = [[0, 2]]
var from_json = JSON.parse('[[0, 2]]').result

print(not_from_json)				# [[0, 2]]
print(from_json)				# [[0, 2]]

print(typeof(not_from_json) == TYPE_ARRAY)	# True
print([0, 2] in not_from_json)			# True
print(not_from_json.has([0, 2]))		# True
print(not_from_json[0] == [0, 2])		# True

print(typeof(from_json) == TYPE_ARRAY)		# True
print([0, 2] in from_json)			# False
print(from_json.has([0, 2]))			# False
print(from_json[0] == [0, 2])			# False

This is likely related to mismatched types, since JSON parses all numbers as floats/reals.
Doing this works:
print(from_json[0] == [0.0, 2.0])		# True

Oh, right. In JS, 2 === 2.0 though (because there's only one number type so they're exactly the same).
Would parsing numbers without a floating point into integers be possible? That's what other languages (PHP, for instance) do. Can be very counter-intuitive otherwise.
I can create another more specific issue or rename this one if need be, but I think that'd be a decent improvement.

I would argue the opposite actually.
extends Node

var example_dict = [{"value":3.2},{"value":3.0}]

func _ready():
	var printed_json = JSON.print(example_dict)
	print(printed_json)

Godot output:
[{"value":3.2},{"value":3}]
so after parsing this JSON again the first value would be an float while the second one would be an integer.
Certainty that the output will always be a float is a good thing and you can always easily parse it to an integer if you so require.

so after parsing this JSON again the first value would be an float while the second one would be an integer which could mess things up.

Why? That JSON.print line would accordingly need to return "[{"value": 3.2}, {"value": 3.0}]" then.
Here's how it works in Python for instance:
https://onlinegdb.com/ByQ0IBXh4
I really don't see a good reason not to go this way.

Previous discussion regarding the same thing:
https://github.com/godotengine/godot/issues/9499

Thanks, exactly the same topic indeed. After reading it though, I still fail to understand the reasoning behind not wanting to handle it more "naturally", as @punto- also mentioned there.
Again, pretty much all languages seem to do it (or at least, be able to do it) that way.
Guess moderators/devs can decide whether this deserves rediscussing or not (despite @reduz sharing his view on the other issue).

Closing in favor of https://github.com/godotengine/godot-proposals/issues/8830
Also #47502 might clear up the confusion.


Add primal_internal.h

primal_internal.h contains all headers for the C interface.
It includes prima.h such that we can only write the exported routines in primal.h and the unexported ones in primal_internal.h.

Thank you @amontoison . The errors complain that some function is deprecated and suggest a replacement. It seems easy to fix, right? Thanks.

For the issue with deprecated routine, we can use the preprocessed macro WIN32_ to check if we are on Windows.
For the alignment error, I don't know what we should do.

Hi @amontoison ,

For the issue with deprecated routine, we can use the preprocessed macro _WIN32 to check if we are on Windows.

// Thread-safe version of localtime
#ifdef _WIN32
#define localtime_safe(a, b) localtime_s(a, b)
#else
#define localtime_safe(a, b) localtime_r(b, a)
#endif

Why do we need to differentiate _WIN32 from others? Could we just use local_time_s in both cases (I note that local_time_r is available only with C23, which is too recent). In addition, why a and b are reversed in the two cases?
Thanks.
Zaikun

https://github.com/amontoison/prima/blob/c_headers_internal/c/tests/stress.c#L10
// Make PRIMA available
#include "prima/prima_internal.h"

Shouldn't we use prima.h instead of prima_internal.h here?

https://github.com/amontoison/prima/blob/c_headers_internal/c/tests/stress.c#L10
// Make PRIMA available
#include "prima/prima_internal.h"

Shouldn't we use prima.h instead of prima_internal.h here?

You need the header of get_random_sed in stress.c nd I don't think that you want to export it.

Hi @amontoison ,

For the issue with deprecated routine, we can use the preprocessed macro _WIN32 to check if we are on Windows.

// Thread-safe version of localtime
#ifdef _WIN32
#define localtime_safe(a, b) localtime_s(a, b)
#else
#define localtime_safe(a, b) localtime_r(b, a)
#endif

Why do we need to differentiate _WIN32 from others? Could we just use localtime_s in both cases (I note that localtime_r is available only with C23, which is too recent)?
In addition, why are a and b reversed in the two cases?
Thanks.
Zaikun

Hi @zaikunzhang, I checked online and it seems that localtime_s is only available on Windows.
If localtime_s is too recent, we should keep localtime on the other platform. We don’t have warnings is a thread-safe version is not used.

Hi @amontoison ,
I wanted to try fixing other warnings but did not know how to do it cleanly on top of your PR, so I created a new one. I hope you do not mind. See https://github.com/libprima/prima/pull/196 .

Hi @zaikunzhang, I checked online and it seems that localtime_s is only available on Windows.

I checked and I agree.

You need the header of get_random_sed in stress.c and I don't think that you want to export it.

I changed its definition to static unsigned int ... and removed it from prima_internal.h.
Could you review https://github.com/libprima/prima/pull/196 ? Thank you.
Zaikun

Hi @amontoison ,
I wanted to try fixing other warnings but did not know how to do it cleanly on top of your PR, so I created a new one. I hope you do not mind. See #196 .

Hi @zaikunzhang, I checked online and it seems that localtime_s is only available on Windows.

I checked and I agree.

You need the header of get_random_sed in stress.c and I don't think that you want to export it.

I changed its definition to static unsigned int ... and removed it from prima_internal.h.
Could you review #196 ? Thank you.
Zaikun

@zaikunzhang
Do you still want to keep localtime_safe(a, b) localtime_r(b, a) for platforms that are not Windows?
You can use localtime_safe(a, b) localtime(a, b) if C23 is a constraint.

@zaikunzhang Do you still want to keep localtime_safe(a, b) localtime_r(b, a) for platforms that are not Windows? You can use localtime_safe(a, b) localtime(a, b) if C23 is a constraint.

Hi @amontoison ,
It turns out that localtime_s is not available on macOS. Since it is only for test, there is no big problem to use C23. Thank you.


Need more clarity for QnA action vs QnA Recognizer

Reference https://github.com/microsoft/BotFramework-Composer/issues/4873

covered last release.


Rename article

closes https://github.com/MicrosoftDocs/OfficeDocs-SharePoint/issues/3372

@tiburd @traya1 Please review if the redirect is correctly written. Thanks

looks good! Merging PR


Issue 1085

Replaced organisational credentials with institutional credentials for get_text messages. DMPRoadmap/roadmap#1085

looks fine


rkt: progress bar when Content-Length is unknown

This also vendors the ioprogress library with other fixes.

thanks @jzelinskie , lgtm aside from the comment nit

@jonboulle just rebased, PTAL

:+1: grazie mille


Datetime problem(#30)

Fixed by joda-time library update.

Hi @kolkhetian can you describe the problem you are seeing?

Hi @ottenhoff, the problem was related to the DST as i mentioned here (https://github.com/sakaicontrib/attendance/issues/30). i solved the problem by updating Joda.

Okay, thanks for the PR and explanation


List of Cryptocurrencies

List of Cryptocurrencies - challenge #1:
This PR includes :

Initialization of the React Native application for both iOS and Android.
Home screen listing Cryptos as in the mockup with hardcoded values.
Rendering of the hardcoded values follows API data structure for easy implementation.
Follow theme scheme.
Main navigation to "Add a Cryptocurrency" screen.
Navigation back to Home screen.
Enabling "Add" button when input field is not empty.

Check the Demo Here

Not longer needed.


Start song using double click

Is your feature request related to a problem? Please describe.
Currently songs are started using single left clicks. While this might be the default for some platforms (Mac? Not sure),
at least for me using Linux I am used to open stuff using double clicks.
In psst double clicking a song results in starting the song twice and sounds weird.
Describe the solution you'd like
A platform dependent default behaviour and an option in settings to configure the default behaviour would be nice.

I prefer single clicks, but an option sounds OK. I suppose a single click should select then, which we don't do now.

Sounds good to me :)

Now single click seem to select the track but double click still don't start it.


Feature/155

We should close #111 and #155 then, right? I would, by the way, prefer separate pull requests for each issue.

I've closed them. I think we should be grateful that someone has taken the time to contribute :)

Yes, I'm really happy that we have some additional contributors!


[BUG] Playwright is changing the terminal mode and hiding the error stream

System info

Playwright Version: [v1.XX]
Operating System: [ macOS ]
Browser: [All, Chromium, Firefox, WebKit]
Other info:

Steps
Run test cases in scala.
Expected
I am running test cases in scala. The test case run as expected. However playwright is changing the behavior of the terminal.  The up arrow start showing ^[OA^. Additionally all the error details disappear. Apparently playwright is not allowing standard error stream to work.
Playwright should not consume the data from error streams. Terminal behavior should not be changed.
Below is the output with out playwright. It displays error details and line numbers.
==> X basic.BasicSuite.renders empty elements 0.02s munit.FailException: <<masked path>>/BasicSuite.scala:51 querySelector returned null check if the query is correct
50:          val actual = dom.document.querySelector("#app > td")
51:          assert(actual != null, "querySelector returned null check if the query is correct")
52:          assertEquals(actual.outerHTML, expectedEl.outerHTML)

Actual
Below is the output with playwright it does not show the additional error message
basic.BasicSuite:
==> X basic.BasicSuite.renders empty elements 0.01s munit.FailException:

Playwright itself doesn't print any special symbols to the terminal, so it's likely something coming out of the user code. Can you provide a repro that we could run locally to reproduce the problem?

We need more information to act on this report. Please file a new one and link to this issue when you get back to it!


Using shipify/go-lua (5.2)  instead of gopher-lua(5.1)

Hello @zyedidia
This editor has turned out to be a breath of fresh air - and approachable!
However, I'd like to ask what were the reasons behind using the 5.1 version instead of 5.2 or later? Would the bundled version be changed in future?
Feel free to close the "issue", as I merely am curious about your reasons.

I don't know what zyedidia's reasons are, but from what I've seen, Lua 5.1 seems to be the most used version of Lua, and therefore the version that most developers will be familiar with. Lua 5.1 also probably has the best longevity, due to LuaJIT being based on it.
Is there anything in particular you're missing from Lua 5.2? And if switching Lua versions were on the table, why not go straight to 5.3 (unless there is no GO implementation for it yet)?

I decided to use gopher-lua because it has a nice reflection library (gopher-luar) which makes it easy to convert between lua objects and go objects.
Also gopher lua seemed like a more mature project, with many more closed issues, more documentation, more stars, and the difference between lua 5.1 and lua 5.2 really didn't seem like a big deal.
I'm also considering switching from Lua to Anko, which is a scripting language made to be used by Go (and it is similar to Go) because it can access all of Go's standard library and it is maybe a nicer language to use compared to Lua. That's just an idea though and hasn't had any attempt at implementation.
I hope I have answered your question.

Yes, if does answer my question. But, since you mention reflection, I'd like to know your thoughts on this embedded language project as well https://github.com/glycerine/zygomys ?
Clearly, it has (almost) everything you mention, be it usage of std lib,  stars etc.

Hmm that does look like an interesting project but I think using lisp would hurt the "easy-to-use" aspect of micro. I think people would be much more likely to write plugins if they used a more "normal" language. I really do see the appeal of a Lisp-like language but I don't think it really fits well into this project.

Oh, I can understand! It's like the devil offers everything except only within the parenthesis! haha
Still, out of curiosity if not for the lisp heritage, ignoring the syntax completely for zygomys - does it make for a good embedded language? I just wish to learn from your experience here :)

Yeah it looks like a good and mature project. There seems to be good documentation and very few issues. I would say if you're looking for an embedded language and don't mind the actual language itself (lisp variant), then that seems like a great project for the job.


Add a configuration file

It would be good to have some parts of the script configurable, for example proxy settings, using a simple config file.
This is slightly complicated by the fact that the 'miniperl' used by this project is quite limited in functionality and modules avaliable

Added in 35debec500e148a20c4a6f2b5531b83f7173853e. Currently very basic and does not do anything, functionality needs to be added now for specific config issues. Will start with the proxy settings.


Android - Error with linking module

After executing
npm install react-native-scanner-zebra --save
and
react-native link react-native-scanner-zebra
Building android bundle fails with the log below (react-native run-android --variant=internalDebug);
Build file '/Users/xxx/xxx/my-project/node_modules/react-native-scanner-zebra/android/build.gradle' line: 75

* What went wrong:
A problem occurred evaluating project ':react-native-scanner-zebra'.
> Project with path ':BarcodeScannerLibrary' could not be found in project ':react-native-scanner-zebra'.

RN version 0.63.3

Next version it will be fixed. Also new updates include RN library changes. Please check the documentation after next release

Next version it will be fixed. Also new updates include RN library changes. Please check the documentation after next release

I am also having this error. Any updates on the new version?

same issue here,  @cryptontech could you please let me know when can we expect the Next version.

For a quick fix you can use https://github.com/xbenjii/react-native-scanner-zebra/commit/3c2967bd8810ce0c906463e421bebb2c85fc4812

Any updated on this? Can I get your package from somewhere @xbenjii

Any updated on this? Can I get your package from somewhere @xbenjii

Should be able to just do
npm install https://github.com/xbenjii/react-native-scanner-zebra.git

Any updated on this? Can I get your package from somewhere @xbenjii

Should be able to just do
npm install https://github.com/xbenjii/react-native-scanner-zebra.git

i have same issue on Android, and i used npm install https://github.com/xbenjii/react-native-scanner-zebra.git but it can't resolves this issue.
that is on my terminal:
1: Task failed with an exception.

Where:
Build file '/Users/chubo/Work/BoTemplate_tsx_rn18/node_modules/react-native-scanner-zebra/android/build.gradle' line: 23

What went wrong:
A problem occurred evaluating project ':react-native-scanner-zebra'.

Plugin with id 'maven' not found.

Try:

Run with --stacktrace option to get the stack trace.
Run with --info or --debug option to get more log output.
Run with --scan to get full insights.
==============================================================================

2: Task failed with an exception.

What went wrong:
A problem occurred configuring project ':react-native-scanner-zebra'.

compileSdkVersion is not specified. Please add it to build.gradle

Try:

Run with --stacktrace option to get the stack trace.
Run with --info or --debug option to get more log output.
Run with --scan to get full insights.

Facing the same issue

same issue

@cryptontech any plan to update to latest SDK ?

The issue here is that react-native no longer includes the 'android' folder (it's empty). There is a README that says to add mavenCentral() to repositories.


Target qemu-x86_64

The documented build method ./bin/build qemu-x86_64 initially fails with:
 $ env -i nix-build -A all -I nixpkgs\=/home/craige/git/NixOS/mobile-nixos/nixpkgs --argstr device qemu-x86_64
warning: Nix search path entry '/home/craige/git/NixOS/mobile-nixos/nixpkgs' does not exist, ignoring
error: file 'nixpkgs/lib' was not found in the Nix search path (add it using $NIX_PATH or -I), at /home/craige/git/NixOS/mobile-nixos/lib/eval-config.nix:20:16
(use '--show-trace' to show detailed location information)

When placing a copy of nixpkgs locally, it then fails with:
 $ env -i nix-build -A all -I nixpkgs\=/home/craige/git/NixOS/mobile-nixos/nixpkgs --argstr device qemu-x86_64
error: The option `sdImage.populateBootCommands' defined in `/home/craige/git/NixOS/mobile-nixos/system-image.nix' does not exist.
(use '--show-trace' to show detailed location information)

When building with nix-build --argstr device qemu-x86_64 -A all this first boot with ./bin/boot-qemu runs out of disk space and fails.
When you reboot that image, init panics and fails due to the state of the disk.
This is reproducible via the steps taken above. Happy to provide logs and console output if it will help.
Thanks!

I got rid of the weird requirements and the wrapper script. It's now all standard nix-build commands. See the updated README.
It now should be booting to stage-2 :).
Do re-open if you are still having issues here.

This issue is certainly resolved.
Thanks!


Release/0.0.1

"Release" the final state of Phase I onto master in preparation for starting fresh for Phase II.

+1


Option to change tick and down arrow

Is your feature request related to a problem? Please describe.
Color of the tick and the dropdown is different than the rest of my layout.
Describe the solution you'd like
Option to change color of the tick or pass a custom tick component. Also the down arrow on the input.
Describe alternatives you've considered
Additional context

Hello, @SaadAhmadSaddiqui  there is prop primaryColor if you set color here that will help to change the color for both the dropdown arrow and for the tick mark.

Ah alright. Thanks!

But I was also hoping to maybe pass a custom JSX.Element down to the arrow and the tick to make it more customizable.

Ummm, I get it. I'll look into that.

Use the below props:
Prop: dropdownIcon - Change dropdown icon -> material design icon
Prop: itemSelectIcon - Change dropdown item select icon -> Custome react node or material design icon


X64 compilation

Hi
I wanted to compile the vst for x64 and I encountered few compile errors (due to assembly) here are the fixes.
I also resaved the .rc file to ansi encoding because visual studio 2017 didn't wanted to compile it...
Hope it could be merge :)

Very cool, looks good!
I want to do some testing to verify that it works and discuss it a bit before proper review/merge, but wanted to say thanks in the meantime :)

Thanks to you for putting the sources of your synth ! I will try to integrate it as a submodule inside my editor https://github.com/lamogui/rmeditor/ this week

@lamogui: I send #38 as a re-spin of this, which I think is a bit cleaner. I hope that's OK with you!

Superceded by #38.


maas_excluded_devices not working inside rpcd/playbooks/roles/rpc_maas/tasks/local.yml

Regardless of the configuration for maas_excluded_devices, all block devices are installed inside MAAS as a separate alarm. I provide a possible workaround since it seems it's a ansible/jinja issue

This section is completed during the issue resolution process. If you are reporting an issue, please do not modify the section below.
Which branches are impacted by this issue?
Remove branches that are not impacted.
For each pull request, update the branch line to reference it.

[ ] master
[ ] liberty-12.2
[ ] kilo

Upgrade impact. Consider the impact on moving between major, minor and patch releases.

[ ] This change does not impact upgrades
[ ] This change addresses a potential upgrade impact
[ ] The upgrade impact is being tracked by another issue.

The issue tracking the upgrade impact is:
Testing impact

[ ] An issue/pull-request has been create or exists to track the testing gap, and is referenced below
[ ] This change includes additional testing
[ ] This change is tested by the existing setup

Additional testing links:
Release note

[ ] This change includes a release note.
[ ] This change does not require a release note.

Doc Impact
Does your change require documentation?

[ ] Yes, and I submitted an issue to Docs issues
[ ] Yes, and I contributed a documentation patch to docs-rpc
[ ] No documentation required for this change

Additional documentation links (please add links if you have ticked yes):

I collapsed this for kilo 11.0 versions to
diff --git a/rpcd/playbooks/roles/rpc_maas/tasks/local.yml b/rpcd/playbooks/roles/rpc_maas/tasks/local.yml
index 9003e63..a9d4e9a 100644
--- a/rpcd/playbooks/roles/rpc_maas/tasks/local.yml
+++ b/rpcd/playbooks/roles/rpc_maas/tasks/local.yml
@@ -357,14 +357,13 @@

include: local_setup.yml
vars:
devices: "[{% for device in ansible_devices.keys()%}{% if device not in maas_excluded_devices|default([]) %}'{{ device }}',{% endif %}{% endfor %}]"
check_name: disk_utilisation
check_details: file={{ check_name }}.py
check_period: "{{ maas_check_period }}"
check_timeout: "{{ maas_check_timeout }}"
TODO (mattt): refactor this code so we can properly create dynamic alarms
without using jinja

alarms: "[{% for device in devices %}{ 'name': 'percentage_disk_utilisation_{{ device }}', 'criteria': ':set consecutiveCount={{ maas_alarm_local_consecutive_count }} if (

alarms: "[{% for device in ansible_devices.keys() if (device not in maas_excluded_devices|default([])) and (ansible_devices[device].model != 'VIRTUAL-DISK') %}  { 'name':
user: root
when: >
inventory_hostname in groups['hosts']

I need to recheck if that is still an issue for 11.1.x

if you could verify the fix let me know

Yes I have to test it again with the latest maas plugins.
It was related to https://github.com/rcbops/rpc-openstack/issues/708 where the excluded device override did not work and both are in the same alarm condition

@BjoernT do you have a PR you'd like to submit for this issue?

Yeah I had before the restructuring of MAAS, let have a quick look since I think we still have the same issue

@BjoernT ping

Having looked more into this, it seems the issue is explicitly referring to the fact that maas_excluded_devices was being ignored. maas_excluded_devices was fixed as a part of the following PRs:
kilo: https://github.com/rcbops/rpc-openstack/pull/1256
liberty: https://github.com/rcbops/rpc-openstack/pull/1250
master: https://github.com/rcbops/rpc-openstack/pull/1241
Therefore I am going to close this issue. If there is a different problem here, please open a new bug report with appropriate levels of detail for us to investigate.


test(clang-tidy): checking ci results

Description
This is not made to be merged.
PR to examine different clang-tidy results between CI and local.
Related links
Parent Issue:

Link

How was this PR tested?
Notes for reviewers
None.
Interface changes
None.

Effects on system behavior
None.

Completed research on the issue to be looked into.


Running Cartographer on Jetson TX2 board

Hi I am trying to use the Jetson TX2 board to implement online SLAM. According to the docs, it says that that 16GB of RAM is the minimum requirement, however, the Jetson TX2 board only has 8GB of RAM. Will it be possible to still use the Jetson TX2 board to efficiently run cartographer??

16GB of RAM is the minimum requirement

It is not that simple.
Memory consumption depends almost only on the map resolution and size.
2D SLAM uses much less than 3D.
I'd say just tune the resolution and filter parameters for speed as in https://google-cartographer-ros.readthedocs.io/en/latest/tuning.html#low-latency and memory also goes down.

Jetson TX2 board

You didn't specify its CPU architecture and performance.
Cartographer currently requires a 64bit x86 architecture and works best with many cores and vector math.
Without multiple cores, global SLAM (loop closure) will be slow.

Its the Nvidia Jetson TX2 board it has:

dual-core NVIDIA Denver2 + quad-core ARM Cortex-A57
256-core Pascal GPU
8GB LPDDR4, 128-bit interface
32GB eMMC
4kp60 H.264/H.265 encoder & decoder
Dual ISPs (Image Signal Processors)
1.4 gigapixel/sec MIPI CSI camera ingest

in terms of its architecture It has 64 bit CPUs as well

dual-core NVIDIA Denver2 + quad-core ARM Cortex-A57

Arm 64bit is theoretically fine but may be hard to compile for, and we currently don't have any documentation for that.

oh okay thanks Gaschler I will try it and let you know how it goes.

@ekow-mensah Hi, did you succeed on 64 bit ARM?My CPU performance is worse，only 512M RAM,。I want to try it with another platform instead of ROS.

No I haven't tried it yet unfortunately as I have not had the time. Also, 512M RAM is too small. According to the minimum requirements you need 16GB of RAM. But yes feel free to try another platform instead of ROS to see if your results get any better.

@erwincoumans Did you encounter any performance degradation as time progressed. Seems like this user was having such issues with a TX1, albeit the specs are quiet different between TX2 and TX1. https://www.youtube.com/watch?v=F94sJL9vgbU
Cheers.

My experiment on my TX2 is fine with the 2D backpack demo.

@anakin0620 hello,I tried install the cartographer on my TX2,but failed.I think the reason is  I cannot install the dependency clang,Did u install clang on ur tx2? and I tried install it by apt.It failed too. can u give me some advice? thank u so much!!

@guorun198959  Hi, what Ubuntu version you are using? If it's 18.04, you should install clang6. My TX2  has 16.04 on it, I didn't have any problem with the installation of Cartographer, just follow the tutorial you can install it. If you have any questions just ask me.


Brewery.Info returns content type error

Today I started receiving an error when calling Brewery.Info
[Error] expected application/json content type, but received application/json; charset=UTF-8

I might have a fix.


update readme

We could also link to the report as well?
https://lighthouse-dot-webdotdevsite.appspot.com//lh/html?url=https%3A%2F%2Ftina-cloud-starter-orcin.vercel.app%2F


Gradient color

Would be nice if the LineStyle have a gradient color option, like the Container widget.

I agree, each TimelineTile could have its own gradient line.
But it would be hard (can't think an easy way right now) to keep the same gradient through all the tiles, which I assume that would be the most common desired use case. Any ideias?

I did not explain so well, but the gradient option it's like you said, each TimelineTile  will have your own gradient LineStyle.
What I needed was that each TimelineTile  had your own gradient, not all the same. I think it's better in that way than have one that repeat on each tile or even one to all TimelineTile tree.
Some problems:

If you have one gradient to all TimelineTile tree.
• Maybe the user can't notice the effect on large lists

If need to repeat the gradient on all tiles
• You can create a gradient const and put it on all tiles manually

Old thread, but you can use a ShaderMask (https://api.flutter.dev/flutter/widgets/ShaderMask-class.html) to achieve all different kinds of variations. Example below creates the gradient along the entire timeline.
ShaderMask(
              blendMode: BlendMode.srcIn,
              shaderCallback: (bounds) {
                return LinearGradient(colors: [
                  Colors.yellow,
                  Colors.deepOrange
                ], tileMode: TileMode.mirror)
                    .createShader(bounds);
              },
              child: WidgetEncapsulatingTime()),


Added the cancel button

Checklist:

I have performed a self-review of my own code,
I have commented my code, particularly in hard-to-understand areas,
I have made corresponding changes to the documentation,
I have added tests that prove my fix is effective or that my feature works,
New and existing unit tests pass locally with my changes ("bundle exec rake"),
Title include "WIP" if work is in progress.

Resolves Loyola#7 - Add in the ability to cancel requests
Description
Organizations have the ability to view and then fulfill requests. The completion of issue # 7 "Add in the ability to cancel requests", will allow the ability for organizations to cancel (destroy) a request as well as view, and fulfill. The "Cancel" button will delete the request, with a confirmation dialog, and an email notification will be sent to the Partner that has requested to cancel.
Include anything else we should know about. -->  WIP
Type of change

New feature (non-breaking change which adds functionality)
- The ability to Cancel Requests

How Has This Been Tested?
The changes can be found in the DiaperBase application under "Requests."

Screenshots
Include screenshots (before / after) for style changes, highlight edited element.-->

@Skyg21 Is this feature ready for review? The PR still says it's a draft.

I just looked over the diff in more detail -- it appears to be ready, pending CI approval.
Thank you, @Skyg21 !

Just merged conflicts, ran rubocop.


feat: add python's logging and verbose mode

Hi,
This is a pull request made in order to add improvement listed in #6.
I added python's logging module, and a logger called logger.
He outputs messages on the console with a StreamHandler handler.
I changed all the print to logger.LEVEL. Please let me know if I choosed incorrect levels for some messages.
There is now a --verbose or -v option which shows debug level messages.
What do you think of these changes @gcushen ?
I have two questions.
I put the logger's declaration in the global code, is it better to put it in if __name__ == '__main__' ?
Should I add a Formatter for the messages ?
I you have suggestions, please let me know.

@FramboisePi thanks for your contribution :)

You are welcome !
Thank you for allowing me to learn about logging.


feat(dynamodb): use new Adapter API

Reasoning 💡
Update the DynamoDB adapter to use the V4 adapter API.
Checklist 🧢

[ ] Documentation
[ ] Tests
[ ] Ready to be merged

Beginner with DynamoDB so may not be perfect/correct but would be looking at using this so tried to do the update.
Diff is a little painful. Maybe better to just view the file. Functions of concern:

getUserByAccount
linkAccount
unlinkAccount

General feedback:

try not to create objects with a restrictive number of fields.

Eg. when y return a user, instead of:
dbUser // let's say you got this from the db.

return {
 id: dbUser.Item.id,
 name: dbUser.Item.name
}

just do:
return dbUser.Item

Just drop createdAt and updatedAt fields. They arent needed in the default adapters.

Doesn't work without setting session.jwt: true

I believe there are implementations for each function in the V4 adapter interface. Not sure why this would be the case.

Is it supposed to add a new record with a different id, each time I sign in (eg via GitHub)?

I believe with V3 you couldn't login with the same email using a different service but not sure if that is the case with V4.


investigate ocis issues in apiSharePublicLink2/reShareAsPublicLinkToShares

tests that fail in apiSharePublicLink2/reShareAsPublicLinkToShares are added to the expected failures list in #753 but need more details

Seems like those tests are already tagged with issues in https://github.com/owncloud/ocis/pull/784


x/tools/gopls: go doc links to symbols in the current package or non-fully-qualified packages do not render as links in tooltips

Is your feature request related to a problem? Please describe.
When hovering over a symbol in the editor, currently some go doc links do not render as links in the tooltip.
If the link is to some URL, then it will appear correctly. If the link is to a fully-qualified symbol eg [encoding/json.Marshal] then it will also appear correctly. However links to symbols imported in the current package but not fully-qualified will not appear as links.
Describe the solution you'd like
For builtins or standard library symbols that are accessible, they should link to their pkg.go.dev pages.
For symbols imported in the current package that are not rooted in the same module as the symbol being hovered on, they should also link to their pkg.go.dev pages. (Of course, they may not actually exist on pkg.go.dev, but I think this is good enough, and I believe GoLand does the same thing - it's been a while since I've used it however.)
For symbols imported in the current package that are rooted in the same module as the symbol being hovered on, I think the tooltip should change to showing the documentation for that symbol, since the full comment text should be available in the current workspace.
Describe alternatives you've considered
None
Additional context
https://tip.golang.org/doc/comment#doclinks describes the rules for links.
Given the code sample:
package sample

import "encoding/json"

// This is a sample go doc comment.
//   - [A regular link] works fine.
//   - So does a fully-qualified link to a builtin like [encoding/json.Marshal].
//   - And even non-builtins like [github.com/vmihailenco/msgpack/v5.Marshal].
//
// However...
//   - A link to a symbol accessible through an alias does not: [json.Marshal]
//   - Neither does a link to a symbol in the current package: [Bar]
//
// [A regular link]: http://example.com
type Foo struct{}
type Bar struct{}

func (Foo) F() {
	json.Marshal(nil)
}

This tooltip is shown when I hover over Foo in the editor:

A related bug: clicking on a doc link such as [fmt.Println] only works if the package fmt is imported under its usual name; for a package like "path" that is usually imported under the name pkgpath, the doc link must read [pkgpath.Join], which is not what a user would ever write in a doc comment.

The problem here is that golang.CommentToMarkdown does not provide the doc.Parser with the necessary hooks. This little hack demonstrates what the implementation must do:
func CommentToMarkdown(text string, options *settings.Options) string {
	var p comment.Parser
+	p.LookupPackage = func(name string) (importPath string, ok bool) {
+		if name == "json" {
+			return "encoding/json", true
+		}
+		return "", false
+	}
+	p.LookupSym = func(recv, name string) (ok bool) {
+		return name == "Marshal"
+	}

We already have an implementation of the necessary hooks, in PackageDocHTML (pkgdoc.go). However the parsing part needs access to a syntax package (*cache.Package), so some factoring is required to plumb the necessary information.

(If you were replying to my now-deleted comment, my bad, I actually meant to reply to a different but related issue: https://github.com/golang/go/issues/69379#issuecomment-2447554755)

In fact my comment explains the work needed to address this issue. No worries though.

I thought this might be an easy fix, but it's not. Some thoughts:

The doc.Parser needs context from the caller: it needs a mapping from PkgName.Name to PkgName.Imported.Path for each non-blank import in the particular file whose comment it is marking up. (This mapping can be derived from types.Info or from the metadata graph.) It also needs the PackagePath of the package containing the doc comment.
The doc parser may be called on fragments of doc comment from a wide variety of places in both the Hover logic and in PackageDocHTML. In each case, we need to record the import mapping for the file from which the doc comment came.
The HoverJSON would need to plumb all this context information, which is kind of a sign that it has failed as an abstraction.
It is not clear what syntax the Synopsis and FullDocumentation fields of HoverJSON use. They are (usually) produced by doc.Package.Synopsis, which returns rendered "Text", but they are (usually) consumed by CommentToMarkdown, which consumes raw doc comment form.

@adonovan you might have meant to leave this issue open with your "Updates #X" line, but since you wrote "attempt to fix #X", it still got caught as a "fixes" commit :)

Indeed--good catch!


Expand numberOfPowershellRefAssemblies list capacity

PR Summary

Increase the list capacity because .NET v5.0.100-preview.5.20278.13 has an extra assembly
Remove assert

PR Context
HEAD of master has been broken since 99da109 (#12772), when .NET was updated to 5.0.100-preview.5.20278.13
https://github.com/PowerShell/PowerShell/issues/12815#issuecomment-636132717
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[x] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[x] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[x] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and enumerated concerns in the summary. This may include:

Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode (which runs in a different PS Host).
Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.
Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).
Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

@xtqqczze Please use a more descriptive PR title.

This resolves the issue in https://github.com/PowerShell/PowerShell/issues/12815#issuecomment-636132717 but does not fix #12815 as it does not include a static test.

@TravisEz13 can you help merge to resolve the breakage described in https://github.com/PowerShell/PowerShell/issues/12815#issuecomment-636132717

@TravisEz13 and @adityapatwardhan probably best to get this merged sooner than later; folks can't currently run debug builds at all while this is an issue.

@iSazonov since you merged #12520, which this PR partially reverts, can you help merge this PR?

The capacity is now 161 as per @iSazonov suggestion.

@TravisEz13 ready to merge?


Save state between restarts

Hello,
I’ve found that at Homebridge restart, the AC devices provided by this plugin are set to off, with 20°C temperatures.
It’s not synchronized to the real state of AC, that could have been set to On, with different temperatures, or fan speed previously.
I think the plugin should save the last state at each commands, and restore it at launch. This would avoid this issue.
I don’t know how to do it, or if there are Homebridge function to do it. If you have an idea and can tell me where to search, I can try to submit a PR if you are ok with it.

Sounds like an excellent addition! I looked at #6, which I think is a very good start. I'm curious if you have a reference Homebridge plugin that also use https://www.npmjs.com/package/node-persist ?

I'm going to close this issue, since I'm confident #7 will get merged. Great work!

Sorry, but I had to revert the commit, since it broke all the tests and was using an old version of node-persist.

Hello,
Did you see the update I made on the PR ? It now uses the last version of node-persist. But I did not look at the tests for now.

Yes, I did: https://github.com/jhthorsen/homebridge-tasmota-aircon-http/pull/11#issuecomment-1139635704


Improve how expired download URLs are handled

Currently - when a OneDrive backup is initiated - we do a delta based enumeration and request download URLs for every item (this allows us to not have to call GetDriveItem for every item.
However - these URLs are time limited and if the backup operation takes too long they will expire (request will fail with a 401 error) in which case we call GetItem to get a new download URL.
This should be optimized because it's an extra round trip for every item (to get the 401 and then renew).
Possible options could be:

Do not enumerate all items up front - enumerate a batch, process it and then enumerate more - the assumption being that the download URLs will be fresher.
Track expiry time for these URLs (or trigger expiry on the first 401) and then assume we need to get new ones (i.e. save the 401 round trip)
~Is there something we can do client side to renew these URLs e.g. get a new access token and update URL?~ Update - not possible
A version of (1) will be to do a full delta re-enumeration when we hit a 401. It will be cheaper (in terms of Graph requests/tokens) vs calling GetItem for everything left

Re #2, did we also explore configurable token lifetime(CTL) to extend token lifetime from ~1hr to ~20 hrs?
Although, this may not be ideal from a security standpoint. Adjusting the lifetime of an access token is a trade-off between improving system performance and increasing the amount of time that the client retains access after the user's account is disabled.

Re option 3, did we also explore configurable token lifetime(CTL) to extend token lifetime to > 1hr ?

I don't believe that applies to these. The download URLs are generated by the backend service and their lifetime isn't configurable

Thanks! Let me explore this a bit more.

It also looks like this would be a setting in the customer environment and not something Corso would be able to control?

I found this page which seems to suggest that token lifetime policies can be attached to an application.
Also, clients can apparently request long lived access to resources - see scope field
Let me understand this a bit more and get back to you.

Cache integration into restore path is ongoing.


No voice-recognizer.pid file when running main.py

Following the HACKING.md file up to these steps:
cd ~/voice-recognizer-raspi
source env/bin/activate
python3 src/main.py

This is the traceback I get.
Traceback (most recent call last):
  File "src/main.py", line 302, in <module>
    main()
  File "src/main.py", line 139, in main
    create_pid_file(args.pid_file)
  File "src/main.py", line 105, in create_pid_file
    with open(file_name, 'w') as pid_file:
FileNotFoundError: [Errno 2] No such file or directory: '/run/user/1000/voice-recognizer.pid'

This is the result of
cd ~/voice-recognizer-raspi
ls -l
id

 ✝  ~/voice-recognizer-raspi   master  ls -l
total 64
drwxr-xr-x 2 pi pi  4096 May 10 09:11 checkpoints
drwxr-xr-x 2 pi pi  4096 May 10 09:11 config
-rw-r--r-- 1 pi pi   985 May 10 09:11 CONTRIBUTING.md
drwxr-xr-x 5 pi pi  4096 May 10 09:13 env
-rw-r--r-- 1 pi pi  1760 May 10 09:11 HACKING.md
-rw-r--r-- 1 pi pi 11358 May 10 09:11 LICENSE
-rw-r--r-- 1 pi pi   363 May 10 09:11 Makefile
drwxr-xr-x 3 pi pi  4096 May 10 09:11 po
-rw-r--r-- 1 pi pi   282 May 10 09:11 README.md
drwxr-xr-x 2 pi pi  4096 May 10 09:11 scripts
drwxr-xr-x 2 pi pi  4096 May 10 09:11 shortcuts
drwxr-xr-x 4 pi pi  4096 May 10 09:17 src
drwxr-xr-x 2 pi pi  4096 May 10 09:11 systemd
drwxr-xr-x 2 pi pi  4096 May 10 09:11 tests
 ✝  ~/voice-recognizer-raspi   master  id
uid=1000(pi) gid=1000(pi) groups=1000(pi),4(adm),20(dialout),24(cdrom),27(sudo),29(audio),44(video),46(plugdev),60(games),100(users),101(input),108(netdev),997(gpio),998(i2c),999(spi),1001(homeassistant)

Thanks for the report! @jordandrako, what operating system are you using?
@ensonic, do you think we should fall back to /tmp or somewhere else if /run/user doesn't exist? The status-monitor would need to check for both too.

I'm running a modified version of raspbian called hassbian.

There is also a dependency issue when running main.py. It asks for PySocks, and using pip install PySocks in the venv makes it go away.

@drigz yeah, probably /tmp even though it is not meant for that.
@jordandrako can you let us know how your /run dir looks like?

total 44K
drwxr-xr-x 23 root        root         800 May 10 09:18 .
drwxr-xr-x 22 root        root        4.0K Apr 21 22:14 ..
drwxr-xr-x  2 avahi       avahi         80 May 10 09:17 avahi-daemon
-rw-r--r--  1 root        root           4 May 10 09:17 crond.pid
----------  1 root        root           0 May 10 09:17 crond.reboot
drwxr-xr-x  2 root        root          60 May 10 09:17 dbus
drwxr-xr-x  3 root        root         100 May 10 09:17 dhcpcd
-rw-r--r--  1 root        root           4 May 10 09:17 dhcpcd.pid
srw-rw----  1 root        root           0 May 10 09:17 dhcpcd.sock
srw-rw-rw-  1 root        root           0 May 10 09:17 dhcpcd.unpriv.sock
drwxr-xr-x  2 root        root          60 May 10 09:17 dnsmasq
drwxr-x---  2 Debian-exim Debian-exim   60 May 10 09:18 exim4
prw-------  1 root        root           0 May 10 09:17 hogsuspend
lrwxrwxrwx  1 root        root          25 May 10 09:17 initctl -> /run/systemd/initctl/fifo
drwxrwxrwt  4 root        root         100 May 10 09:17 lock
drwxr-xr-x  3 root        root          60 May 10 09:17 log
-rw-r--r--  1 root        root           4 May 10 09:17 mosquitto.pid
drwxr-xr-x  2 root        root          60 May 10 09:17 mount
drwxr-xr-x  2 mysql       root          80 May 10 09:17 mysqld
drwxr-xr-x  2 root        netdev        80 May 10 09:17 network
-rw-r--r--  1 root        root           3 May 10 09:17 ntpd.pid
drwxr-xr-x  2 root        root          40 May 10 09:17 plymouth
drwxr-xr-x  4 root        root          80 May 10 09:17 resolvconf
-rw-r--r--  1 root        root           4 May 10 09:17 rngd.pid
-rw-r--r--  1 root        root           4 May 10 09:17 rsyslogd.pid
drwxr-xr-x  2 root        root          60 May 10 09:17 sendsigs.omit.d
lrwxrwxrwx  1 root        root           8 May 10 09:17 shm -> /dev/shm
drwxr-xr-x  2 root        root          60 May 10 09:17 snapclient
drwxr-xr-x  2 root        root          60 May 10 09:17 snapserver
drwxr-xr-x  2 root        root          40 May 10 09:17 sshd
-rw-r--r--  1 root        root           4 May 10 09:17 sshd.pid
drwxr-xr-x 15 root        root         360 May 10 09:17 systemd
-rw-r--r--  1 root        root           4 May 10 09:17 thd.pid
srwxr-xr-x  1 root        root           0 May 10 09:17 thd.socket
drwxr-xr-x  2 root        root          60 Dec 31  1969 tmpfiles.d
drwxr-xr-x  7 root        root         160 May 10 09:17 udev
drwxr-xr-x  2 root        root          40 May 10 09:17 user
-rw-rw-r--  1 root        utmp        1.5K May 10 09:18 utmp
drwxr-x---  2 root        netdev        60 May 10 09:17 wpa_supplicant
-rw-r--r--  1 root        root           4 May 10 09:17 wpa_supplicant.wlan0.pid

Ahh, so there is /run/user. Could you please also run:
ls -al  /run/user
and
id
Thanks!

Nothing there
total 0
drwxr-xr-x  2 root root  40 May 10 09:17 .
drwxr-xr-x 23 root root 800 May 10 09:18 ..

Okay, it is just missing the subdir for the user-id, which only root could create :/

I can create the directory myself to get it to run, does it need to be owned by pi?

I created the 1000 directory under /run/user and made pi the owner. Now running src/main.py gives this traceback:
Traceback (most recent call last):
  File "src/main.py", line 302, in <module>
    main()
  File "src/main.py", line 159, in main
    do_recognition(args, recorder, recognizer, player)
  File "src/main.py", line 175, in do_recognition
    import triggers.gpio
  File "/home/pi/voice-recognizer-raspi/src/triggers/gpio.py", line 19, in <module>
    import RPi.GPIO as GPIO
ImportError: No module named 'RPi'

I then ran
pip install Rpi.GPIO

Now the result of src/main.py is:
[2017-05-10 10:48:41,639] INFO:audio:started recording

This is the desired behavior, yes?

Hmm. I also had to install Rpi.GPIO outside the for the status-led.service.
sudo python3 pip install Rpi.GPIO

After that I rebooted, which I found out deleted the /run/user/1000 directory, so I had recreate, and re chown that. Then I restarted all the services and it seems to work.


DecryptingDeserializer does not support JsonDeserializer (ExtendedDeserializer)

The DecryptingDeserializer does not support JsonDeserializer as embedded deserializer. This is most likely caused by the JsonDeserializer being an implementation of ExtendedDeserializer rather than directly Deserializer.
    @Test
    public void shouldDecryptOnJsonDeserialize() {
        EncryptingSerializer<MyPayload> encryptingSerializer = new EncryptingSerializer<>(PASS, SALT, new JsonSerializer<>());
        DecryptingDeserializer<MyPayload> decryptingDeserializer = new DecryptingDeserializer<>(PASS, SALT, new JsonDeserializer<>());
        byte[] encryptedData = encryptingSerializer.serialize("my-topic", new MyPayload("my-data"));
        MyPayload decryptedData = decryptingDeserializer.deserialize("my-topic", encryptedData);
        Assertions.assertThat(decryptedData).isEqualTo(new MyPayload("my-data"));
    }

causes java.lang.IllegalArgumentException: Unrecognized Type: [null]

First of all, to fix the issue at hand, you can initialize the JsonSerializer with a target type:
JsonDeserializer<MyPayload> deserializer = new JsonDeserializer<>(MyPayload.class);
DecryptingDeserializer<MyPayload> decryptingDeserializer = new DecryptingDeserializer<>(PASS, SALT, deserializer);

Additionally, when not using shared DTOs, sending a TypeId header will most likely be problematic. In this case, you can make the deserializer ignore any type headers by using the disabling the flag useHeadersIfPresent:
JsonDeserializer<MyPayload> deserializer = new JsonDeserializer<>(MyPayload.class, false);
DecryptingDeserializer<MyPayload> decryptingDeserializer = new DecryptingDeserializer<>(PASS, SALT, deserializer);

Nevertheless, I think it would be a good idea to also add a comfortable way to disable type header sending. The consumer should have control over how they want to name their DTOs. I will add this to the wanted features list.

Thanks @roechi ! You made my day :)
We should probably think about making type header suppression the default behaviour but still allow turning them back on.


Monochrome Settings Override Behavior

Is your feature request related to a problem? Please describe.
When a theme is selected like a palette it is the default, then it later gets overridden by other options such as colors.
When a monochrome color theme is enabled it cannot be overridden by options such as colors.
Describe the solution you'd like
Monochrome theme should be overwritten with settings in the same way a palette theme is overwritten.
Describe alternatives you've considered
On change of theme setting monochrome: { enabled: false } if the colors are set. (not necessary with palettes)
Additional context
Monochrome Behavior (no effect):

Palette Behavior:

That's intentional.
The priority is this way
(highest to lowest)
colors -> palette
fill.colors -> colors
monochrome.color -> colors
So, palettes have the lowest priority because they are not defined by users. Fill colors and monochrome colors are set by the user specifically hence they a higher priority.

Thanks for the information 👍


Remove duplicacy in instructions ...

Fix https://github.com/JuliaCommunity/YouTubeVideoTimestamps/issues/175

ping @logankilpatrick

closing as not planned (stale)


Better ADRP handling in ARM

What is the feature you'd like to have?
Better ADRP handling (similar to #4650).
Is your feature request related to a problem?
Binary Ninja creates a data variable at every ADRP base address.  This leads to incorrect jump tables and bad further analysis. In the past, I have written a workflow that converted any ADRP/ADD combinations to just a single constant pointer, but it would be better if this was handled in Binary Ninja.
Additional Information:
You can see in the screenshot that the jumptable should continue, but the data variable at 0x187ec8000 (because of ADRP's) interrupts it.

Duplicate of https://github.com/Vector35/binaryninja-api/issues/4528. I believe the real fix for the issue is to relax the jump table size detection code to allow it to overwrite some data variables, for example, when the data variable is void or otherwise does not provide much value


fix: Compatible with react 18 createRoot

Use createRoot  when api exists in react-dom, otherwise leave it as is

Thanks for your contribution!


General Problem to run bundle doctor

Error Report
Questions
Please fill out answers to these questions, it'll help us figure out
why things are going wrong.

What did you do?
I ran the command /Users/spaceotter/.rbenv/versions/2.5.0/bin/bundle doctor

What did you expect to happen?
I expected Bundler to just check my Gemfile. I tried many variations. I excluded every gem out of my
Gemfile to show you that is a general problem.

What happened instead?
I get the error message.

Have you tried any solutions posted on similar issues in our issue tracker, stack overflow, or google?
I tried everything from reinstalling the hole rbenv config, switching to just using one system wide ruby
version, using rvm ... nothing helped.

Have you read our issues document, https://github.com/bundler/bundler/blob/master/doc/contributing/ISSUES.md?
Yes.

Backtrace
Errno::ENOENT: No such file or directory - /Users/spaceotter/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0/bundler
  /Users/spaceotter/.rbenv/versions/2.5.0/lib/ruby/2.5.0/find.rb:43:in `block in find'
  /Users/spaceotter/.rbenv/versions/2.5.0/lib/ruby/2.5.0/find.rb:43:in `collect!'
  /Users/spaceotter/.rbenv/versions/2.5.0/lib/ruby/2.5.0/find.rb:43:in `find'
  /Users/spaceotter/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0/gems/bundler-1.17.1/lib/bundler/cli/doctor.rb:103:in `each'
  /Users/spaceotter/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0/gems/bundler-1.17.1/lib/bundler/cli/doctor.rb:103:in `check_home_permissions'
  /Users/spaceotter/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0/gems/bundler-1.17.1/lib/bundler/cli/doctor.rb:81:in `run'
  /Users/spaceotter/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0/gems/bundler-1.17.1/lib/bundler/cli.rb:663:in `doctor'
  /Users/spaceotter/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0/gems/bundler-1.17.1/lib/bundler/vendor/thor/lib/thor/command.rb:27:in `run'
  /Users/spaceotter/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0/gems/bundler-1.17.1/lib/bundler/vendor/thor/lib/thor/invocation.rb:126:in `invoke_command'
  /Users/spaceotter/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0/gems/bundler-1.17.1/lib/bundler/vendor/thor/lib/thor.rb:387:in `dispatch'
  /Users/spaceotter/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0/gems/bundler-1.17.1/lib/bundler/cli.rb:27:in `dispatch'
  /Users/spaceotter/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0/gems/bundler-1.17.1/lib/bundler/vendor/thor/lib/thor/base.rb:466:in `start'
  /Users/spaceotter/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0/gems/bundler-1.17.1/lib/bundler/cli.rb:18:in `start'
  /Users/spaceotter/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0/gems/bundler-1.17.1/exe/bundle:30:in `block in <top (required)>'
  /Users/spaceotter/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0/gems/bundler-1.17.1/lib/bundler/friendly_errors.rb:124:in `with_friendly_errors'
  /Users/spaceotter/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0/gems/bundler-1.17.1/exe/bundle:22:in `<top (required)>'
  /Users/spaceotter/.rbenv/versions/2.5.0/bin/bundle:23:in `load'
  /Users/spaceotter/.rbenv/versions/2.5.0/bin/bundle:23:in `<main>'

Environment
Bundler       1.17.1
  Platforms   ruby, x86_64-darwin-18
Ruby          2.5.0p0 (2017-12-25 revision 61468) [x86_64-darwin18]
  Full Path   /Users/spaceotter/.rbenv/versions/2.5.0/bin/ruby
  Config Dir  /Users/spaceotter/.rbenv/versions/2.5.0/etc
RubyGems      2.7.3
  Gem Home    /Users/spaceotter/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0
  Gem Path    /Users/spaceotter/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0:/Users/spaceotter/.gem/ruby/2.5.0
  User Path   /Users/spaceotter/.gem/ruby/2.5.0
  Bin Dir     /Users/spaceotter/.rbenv/versions/2.5.0/bin
Tools
  Git         2.19.2
  RVM         not installed
  rbenv       rbenv 1.1.1
  chruby      not installed

Bundler Build Metadata
Built At          2018-10-25
Git SHA           2760d72d3
Released Version  true

Gemfile
Gemfile
source 'https://rubygems.org' do
  end

Gemfile.lock
GEM
  remote: https://rubygems.org/
  specs:

PLATFORMS
  ruby

DEPENDENCIES

BUNDLED WITH
   1.17.1

Looks like the doctor command needs to handle files not existing more gracefully, or something like that

I reprpduced the 'bug' also on a fresh new installed arch linux system. Same result.
Btw. there wasn't at any time a directory or file at ~/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0 called bundler.  ~/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0 includes the directory gems within bundler-1.17.1.


As a developer I want the AI to stick to the planet with their navmesh.

Instructions: On the main scene locate the navigation panel at the top right and when clicked you should be able to see the blue navmesh on the planet.
Notes: we decided to only have the mesh cover the sides and poles of the planet as there isnt a reason an AI would follow the player far enough to reach another navmesh.


Alignment Issues

Menu labels are not aligned with menu items

https://github.com/mubaidr/bulma-fluent/commit/0a1ec9e391703eb4aa805f8fba25742cf8bfb731

Panel-heading
Remove scroll in card heading

https://github.com/mubaidr/bulma-fluent/commit/7deef3767269e2ed93b20121e03ff9756d31bd26


Remove RustThemis wrapper changelog

Remove the CHANGELOG.md since we already have one, common for all
libraries and wrappers
Fix link to point on common changelog

BTW, noticed interesting thing, lib.rs does not parse links in .md files the same way github or crates.io do. Links to Examples and Tests on the bottom are invalid. Possible solution -- hardcoded URLs to github.
Checklist

[X] The coding guidelines are followed

Possible solution -- hardcoded URLs to github.

The downside is that the users who are browsing GitHub in non-master branches (e.g., stable) will still see links leading to the master branch. Arguably, the benefit of having working links on package listing sites is more significant.

The issue about readme rendering on lib.rs is already opened, but no progress is done. I guess the problem is within their markdown renderer.


Site request erofus.com

I dont know where to write this.But can you add support for erofus.com please

Please make any requests for new sites in this thread.  @cyian-1756 Recommend closing.


Build failing on GitHub pages

Hello,
I simply forked the repo, and in settings I'm getting the following. I tried it in my local and it's working fine, but fails when I try in GitHub pages. What could be wrong? Thanks a lot in advance.
Regards,
Hans

Please

@Hanzig how did you solve this?


Support mongodb 3.6

MongoDB 3.6 is to bind to localhost <IP_ADDRESS> by default. It cannot accept the connection from the mongo-sidecar container.
Thanks @joshorig to point out #68

Cool, thanks 😄


A bit of stack behavior in TabNavigator

Not sure if this is a bug or not but it feels a bit weird to me.
When in a TabNavigator nested in a StackNavigator (like in NavigationPlayground for example), selecting another tab than the initial tab and then executing goBack(), the TabNavigator goes back to the initial tab, I expected that goBack() would trigger the outer StackNavigator instead. If executing goBack() on the initial tab the outer StackNavigator is triggered, regardless if any other tab has been visited before the initial tab.
The DrawerNavigator has the same behavior.

This is meant to match the behavior that most Android apps have, but you can disable it with backBehavior: 'none', as described in this doc.

Ok, so it wasn't a bug then, must read the docs better.. I am a iOS user myself with very little Android experience so that's why I don't recognize the behavior. Thanks!


Added OE SDK logging mechanism

This PR implements proposal mentioned in the following issue.
https://github.com/Microsoft/openenclave/issues/1045

bors try

@anakrish @soccerGB -- Will review the global naming and more later today.

bors try

bors try

bors try

bors r+


Fixed name of dominique-unruh/qrhl-tool

(Was unruh/qrhl-tool previously which does not exist)

@dominique-unruh scala-steward failed to load the build of this repo:
2019-06-21 12:06:21,671 ERROR Check dependencies of dominique-unruh/qrhl-tool failed
java.io.IOException: Cloning into '/home/frank/scala-steward/workspace/repos/dominique-unruh/qrhl-tool'...
Submodule 'isabelle-thys/Bounded_Operators'<EMAIL_ADDRESS>registered for path 'isabelle-thys/Bounded_Operators'
Cloning into '/home/frank/scala-steward/workspace/repos/dominique-unruh/qrhl-tool/isabelle-thys/Bounded_Operators'...
remote: Repository not found.
fatal: repository<EMAIL_ADDRESS>not found
fatal: clone of<EMAIL_ADDRESS>into submodule path '/home/frank/scala-steward/workspace/repos/dominique-unruh/qrhl-tool/isabelle-thys/Bounded_Operators' failed
Failed to clone 'isabelle-thys/Bounded_Operators'. Retry scheduled
Cloning into '/home/frank/scala-steward/workspace/repos/dominique-unruh/qrhl-tool/isabelle-thys/Bounded_Operators'...
remote: Repository not found.
fatal: repository<EMAIL_ADDRESS>not found
fatal: clone of<EMAIL_ADDRESS>into submodule path '/home/frank/scala-steward/workspace/repos/dominique-unruh/qrhl-tool/isabelle-thys/Bounded_Operators' failed
Failed to clone 'isabelle-thys/Bounded_Operators' a second time, aborting

scala-steward requires that just calling sbt after cloning the repo works.

I'm removing it for now. Feel free to add it again, if the build is fixed.


replace file system with filesystem to keep consistent

Signed-off-by: Ma Shimiao<EMAIL_ADDRESS>

LGTM

LGTM


Avoid SingleTermId being null if at least one term is checked

In reference to #8661
If no term with the proper culture is found, SingleTermId is the id of the first valid checked term. This patch avoids TaxonomyViewModel.SingleTermId to be null when at least one term has been checked by the user. This was a bug happening when TaxonomyField was configured to allow the selection of a single term only.

@sebastienros I forgot to apply the last update to Orchard's dev-derived branch, which now is pushed.
I simplified the subsequent query following your suggestions too.


Add Bilingual Corpus of Arabic-English Parallel Tweets

Added Bilingual Corpus of Arabic-English Parallel Tweets. The link to the dataset can be found here and the paper can be found here

[x] Followed the instructions in CONTRIBUTING.md
[x] Ran the tests successfully
[x] Created the dummy data

IMO, the problem with this dataset is that it is not really a text/nlp dataset. These are just collections of tweet ids. So, ultimately, one needs to crawl twitter to get the actual text.

That's true.

at least it's clear in the description that one needs to collect the tweets :
This resource is a result of a generic method for collecting parallel tweets.

Looks like this is failing for other datasets. Should I rebase it and push again?
Also rebasing and pushing is reflecting changes in many other files (ultimately forcing me to open a new branch and a new PR) any way to avoid this?

No let me merge this one directly

merging since the CI is fixed on master


docs(publishLast): add publishLast operator docs

Description:
Included formal summary, description, example and return values.

Coverage increased (+0.2%) to 96.945% when pulling 05ee5bc57be73638902a20d342c181f1634d13d1 on rarmatei:publishLast-docs into f9318d898d6c017652a84202ccaebb440397016c on ReactiveX:master.

Thank you, @rarmatei!


support editing mode.

You can refer to this, maybe you can find inspiration
https://github.com/scottwillmoore/obsidian-short-internal-links-to-headings

Thanks for the suggestion, now supporting editor mode including ones in the front matter.


Add Index#search

[closes #418]

I still need to add Search::Result, after which I'll remove the don't merge label.

@blowmage This should be ready to merge, but please take a careful look at it, of course.

Looking.


Region not set to EU-WEST-1 in the '...-europe.js' examples

Some of the examples/amazon/s3 have the region set as US rather than the EU
eg.
list-objects-europe.js line 14
list-buckets-europe.js line 15
should be :
'region' : amazon.EU_WEST_1
or maybe I've missed the point of the tests in which case let me know, as am keen on these functions!

AwsSum is now deprecated in favour of the offical aws-sdk from Amazon. Thanks for opening this originally.


eCarePlanner 3.1.5 - Care Team - some names missing, showing "MISSINGGIVEN"

It was reported in eCarePlanner 3.1.5 that some Care Plan data is coming through in a way, displaying "MISSINGGIVEN undefined" as the name.

Christina investigated this a bit, who provided the following information:

Highlighted in yellow is the provider that is listed as MissingGiven. We are returning the outside provide or the provider that is via Care Everywhere. Let me know if you have any questions.

Attached is the full CareTeamResponse, in the hope that it will be useful in sorting this out:
CareTeamResponse.txt

Meld does not support  _family element -
"name": [{
"use": "usual",
"text": "Christopher Terndrup, MD",
"_family": {
"extension": [{
"valueCode": "masked",
"url": "http://hl7.org/fhir/StructureDefinition/data-absent-reason"
}
]
}
}
],

Fixed in eCarePlanner  v3.1.6


Wallet seed restored successfully displays when mismatched share sets are entered

Entering the following 3 shares gave success when in reality the python backend failed.
Must detect the mismatch ids and reject later ones different from earlier ones.
ms134uzk4k5smystn20dvgpunsu5gkf3fdx7z7vn58en975d
ms134uzkv8qsn5q0wnlg2kf2dduhe7w7jvv5va3czywq57fh
ms12sq40nux796hz9jm8stsz4pc6vzq0yevxkp0f352gdr2q

It should refuse to accept subsequent shares as correct if the header does not match the first entered share.


Failing "underlying SSH library"

I am unable to build new boxes now. I am working on setup base configs for our team. I started seeing this issue on a brand new mac it has ansible <IP_ADDRESS> and vagrant 1.81.
Since I had it working on my system I downloaded a fresh copy of Drupal VM, changed the IP and ran in to the same issues I recently upgraded to Vagrant 1.8.1 (was 1.7.X) and ansible 2.1.0
==> drupalvm: [vagrant-hostsupdater]   <IP_ADDRESS>  dashboard.drupalvm.dev  # VAGRANT: d64d789cabcb9fcfdf063adb87e96f6f (drupalvm) / d7e8c3b6-129a-475c-b7f1-02ddcdc8c6c9
==> drupalvm: [vagrant-hostsupdater] This operation requires administrative access. You may skip it by manually adding equivalent entries to the hosts file.
==> drupalvm: Setting hostname...
**An error occurred in the underlying SSH library that Vagrant uses.
The error message is shown below. In many cases, errors from this
library are caused by ssh-agent issues. Try disabling your SSH
agent or removing some keys and try again.
If the problem persists, please report a bug to the net-ssh project.
timeout during server version negotiating**

I haven't noticed this issue on either of my Macs (just brought up two new Drupal VM instances this morning, a few minutes ago, for testing); it sounds like it could be a networking issue, or even a permissions issue... can you try to restart your Mac and run vagrant up again, and failing that, also see some of the notes from this issue: https://github.com/mitchellh/vagrant/issues/6854
Other potential hints:

https://github.com/puphpet/puphpet/issues/2070
https://github.com/mitchellh/vagrant/issues/6812

Mind sharing what version of vagrant and ansible you are running?

$ vagrant --version
Vagrant 1.8.1

$ansible --version
ansible <IP_ADDRESS>

FWTW: I have VirtualBox 5.0.16; I also noticed a fresh install of vagrant gets VirtualBox 5.0.10. The issue was solved by adding "vagrant-vbguest".
vagrant plugin install vagrant-vbguest

Ah, so that fix strikes again! Annoying,,, also, I just updated all the base boxes so they have the 5.0.16 guest additions by default; you can run vagrant box update geerlingguy/ubuntu1404 to get the latest version.

FYI, I have discovered that the issues I am having have to do with the ZyWall Firewall we have at work. I can build and destroy over and over again at home, go to the office and it fails. I tested this by hardwiring to the firewall, failed; then to the Comcast Gateway, worked.
I will be looking over the settings in the firewall to see if I can get it to pass. It seems that google DNS is trying to ping back a fail on the hostname setup and the firewall is blocking it, just a guess based on the logs.

I'm currently having this same issue on my windows 10 machine.  I tried adding the vagrant-vbguest plugin as mentioned above but no luck.

@javasavvy - It seems this problem was caused in the past by network issues. Can you try again on a different network/location?

Unfortunately, I need it to work on my current network or it's not usable for me.
Your probably right though as I'm using a VMware VM as the host machine (So essentially a virtualbox vm within a vmware vm).
I got past this by setting natdnshostresolver1 to off in the Vagrantfile.  reference here:
https://github.com/mitchellh/vagrant/issues/6812
I immediately hit another issue though so It seems I won't be able to use this vagrant vm.  (My first time working with vagrant and it's been a bit ruff).  The next issue appears to be an vagarant 1.8.1 problem itself.  I'm getting the "stdin: is not a tty".  and vagarant fails..
Found a reference for a fix here:
https://github.com/mitchellh/vagrant/issues/1673
But I was not able to get it working.

Same issue here. Can't bring up any VMs :(

Same issue here too..Windows 10

Thank you @michaelpporter . I've been struggling with this issue on my iMac for the last 2 years. vagrant plugin install vagrant-vbguest
seems to have solved it.

Same issue

go have fun with your self
On Jun 22, 2016 4:18 PM, "Jeff Geerling"<EMAIL_ADDRESS>wrote:

@javasavvy https://github.com/javasavvy - It seems this problem was
caused in the past by network issues. Can you try again on a different
network/location?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/geerlingguy/drupal-vm/issues/509#issuecomment-227905813,
or mute the thread
https://github.com/notifications/unsubscribe/AFBU1sM43IWCzIgs06W_hW8Tvb1ekouJks5qOcK0gaJpZM4Ht5xY
.

Update: just checking the box 'cable connected' in network settings of VirtualBox solved the issue ;)

@Mich-b - Also consider installing the vbguest plugin; that might help make sure this kind of error doesn't happen again.

Had to modify some other things as well:

installed vbguest
checked 'cable connected' in virtualbox network settings
replaced https://download.jetbrains.com/idea/ideaIC-2016.2.4.tar.gz with https://download.jetbrains.com/idea/ideaIC-2016.2.5.tar.gz in vagrant_provisioning.sh (as the 2.4 version is not available anymore)
included branch in clone command :"git clone --branch 7.1 https://github.com/WebGoat/WebGoat.git "

Working now. Thanks for the help

Sorry about that last e-mail.  It was a mistake.
On Nov 27, 2016 12:56 AM, "Justin Fleming"<EMAIL_ADDRESS>wrote:

go have fun with your self
On Jun 22, 2016 4:18 PM, "Jeff Geerling"<EMAIL_ADDRESS>wrote:

@javasavvy https://github.com/javasavvy - It seems this problem was
caused in the past by network issues. Can you try again on a different
network/location?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/geerlingguy/drupal-vm/issues/509#issuecomment-227905813,
or mute the thread
https://github.com/notifications/unsubscribe/AFBU1sM43IWCzIgs06W_hW8Tvb1ekouJks5qOcK0gaJpZM4Ht5xY
.

:)

@michaelpporter , same issue here:works fine at home, fails at office. can you give me some suggestion?

@wjz2047 We just removed the ZyWall from part of our network; it was used so the office manager could VPN. Apologies, I can not offer more help.

@xenomedia-deploy ,thanks anyway

what worked for me was:
vagrant halt
vagrant up --provision

For me, the hanging happened because the bootloader was waiting for an input. I did the following:

Stop the VM (I did it in the VirtualBox GUI).
vagrant up
During booting, press Enter a few times. This can be done via the GUI or the command line. Just follow what this answer suggested.


Run all step shortcut

Describe the solution you'd like
is there a way to run all step with shortcut? when selecting and panning 20+ steps can be frustrating
What does your solution aim to solve?
ease of running all the steps at once
Describe alternatives you've considered
Add more shortcuts just like ctrl + alt + enter to run all.

Thanks for opening an issue. I definitely like the idea!
Duplicate of #327.

A difference is that @kingabzpro is proposing a shortcut instead of a button. I'll add that to #327 to keep the conversation focused.


Webp format

Can you make it possible for a saving in the webp format? they are smaller then the png's and would be better for the server. Also choosing a save location would be nice too.

@MrPrimate I have this working locally now if you want to assign it to me.  I'll submit a PR once I finish some more testing.

@JamzTheMan Very cool, excited to see!

Pretty simple change. I also lumped in an extra option for portrait size, but if you don't like it I can remove it.

Thanks for the great PR @JamzTheMan !


docs: explain the "robot" module in a high level way

This commit adds a README file at lib/shared/robot explaining in
detail how the "robot" mechanism works, and why it is needed.
Signed-off-by: Juan Cruz Viotti<EMAIL_ADDRESS>

@sam3d Thanks a lot for the review! Having fresh eyes take a look at this is incredibly valuable :)

Also, why are there line breaks at 80 characters? I could be wrong but I was under the impression it was a paragraph per line, feel free to ignore that one though.

Is more of a matter of convention, since its easier to edit the text this way (you don't have to deal with long lines and you can navigate the paragraph more naturally). GitHub will still render it the same way as if it didn't have text wrap.

No worries @jviotti, I know they were relatively minor changes but thank you so much for being receptive!
And vis a vis the tex wrapping, that's what I thought would be the case but just wasn't sure what convention you guys used for markdown around here.


[type:fix] Polish Code.

[type:fix] Polish Code.

Hi, please check CI

Codecov Report

Merging #4945 (071c8fd) into master (3082629) will decrease coverage by 0.12%.
Report is 1 commits behind head on master.
The diff coverage is 48.48%.

@@             Coverage Diff              @@
##             master    #4945      +/-   ##
============================================
- Coverage     63.91%   63.80%   -0.12%
+ Complexity     8524     8517       -7
============================================
  Files          1207     1207
  Lines         35895    35891       -4
  Branches       3219     3216       -3
============================================
- Hits          22943    22899      -44
- Misses        11098    11145      +47
+ Partials       1854     1847       -7

Files Changed
Coverage Δ

...nyu/admin/service/impl/TagRelationServiceImpl.java
100.00% <ø> (ø)

...ache/shenyu/admin/service/impl/TagServiceImpl.java
61.38% <ø> (ø)

...u/admin/service/manager/impl/SwaggerDocParser.java
66.16% <0.00%> (-0.51%)
:arrow_down:

...enyu/client/core/shutdown/ShutdownHookManager.java
0.00% <0.00%> (ø)

...henyu/plugin/brpc/spi/SharedThreadPoolFactory.java
0.00% <0.00%> (ø)

...enyu/plugin/sofa/cache/ApplicationConfigCache.java
69.72% <ø> (ø)

...ister/client/server/zookeeper/ZookeeperClient.java
89.74% <0.00%> (ø)

...nyu/register/client/zookeeper/ZookeeperClient.java
89.74% <0.00%> (ø)

...che/shenyu/registry/zookeeper/ZookeeperClient.java
86.41% <0.00%> (ø)

...yu/sdk/spring/proxy/ShenyuClientMethodHandler.java
64.51% <ø> (ø)

... and 12 more

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

@GOODBOY008 Hi, pls check ci


Add example of creating computed column from nested JSON

Re: Computed Columns
Issue Description
An example for creating a computed column from nested JSON would be helpful. The syntax isn't immediately intuitive.
Suggested Resolution
Add something like:
alter table vehicles add column purchase_date string as (vehicle_info->'purchase_information'->>'purchase_date') STORED;

Bumping this up since its the primary way people use indexes in production. This can be potentially downgraded if expression-based indexes lands in 21.1.

Bumping this up since its the primary way people use indexes in production. This can be potentially downgraded if expression-based indexes lands in 21.1.