Intellisense not working with Unity (5.5.0f3) project (MacOSX)

Environment data
Product Information:
Version:            1.0.3
Commit SHA-1 hash:  37224c9917
Runtime Environment:
OS Name:     Mac OS X
OS Version:  10.12
OS Platform: Darwin
RID:         osx.10.12-x64
Base Path:   /usr/local/share/dotnet/sdk/1.0.3
Mac OSX Version: 10.12.4 (Sierra)
Unity Version: 5.5.0f3
Unity Extension Version: 1.2.0
VS Code version: 1.12.1
C# Extension version: 1.9.0
Steps to reproduce
Open the C# project via Unity (See log below)
Expected  behaviour
Omnisharp loads and the intellisense works
Actual behaviour
Omnisharp throws an error and intellisense does not work (See logs below)
Notes:

Output of which mono in bash: /usr/local/bin/mono
I uninstalled mono and reinstalled via brew
I uninstalled VSCode plugins and reinstalled (C# and Unity)
Mono 3.5 API is located at /usr/local/Cellar/mono/<IP_ADDRESS>/lib/mono/3.5-api
I unlinked and relinked mono via brew
I restarted my computer :)
This was working up until just recently.

Log Output:
Starting OmniSharp server at 5/8/2017, 8:11:34 PM
Target: /Users/user/Projects/unity/pong-3d/pong-3d.sln
OmniSharp server started wth Mono
Path: /Users/user/.vscode/extensions/ms-vscode.csharp-1.9.0/bin/run
PID: 14677
OmniSharp: -s /Users/user/Projects/unity/pong-3d/pong-3d.sln --hostPID 14386 --stdio DotNet:enablePackageRestore=false --encoding utf-8 --loglevel information formattingOptions:useTabs=false formattingOptions:tabSize=4 formattingOptions:indentationSize=4
{"Event":"log","Body":{"LogLevel":"INFORMATION","Name":"OmniSharp.Startup","Message":"Omnisharp server running using Stdio at location '/Users/user/Projects/unity/pong-3d' on host 14386."},"Seq":1,"Type":"event"}
info: OmniSharp.DotNet.DotNetProjectSystem
Initializing in /Users/user/Projects/unity/pong-3d
info: OmniSharp.DotNet.DotNetProjectSystem
Auto package restore: False
info: OmniSharp.DotNet.DotNetProjectSystem
Update workspace context
info: OmniSharp.DotNet.DotNetProjectSystem
Resolving projects references
info: OmniSharp.MSBuild.MSBuildProjectSystem
MSBUILD_EXE_PATH environment variable set to /Users/user/.vscode/extensions/ms-vscode.csharp-1.9.0/bin/omnisharp/msbuild/MSBuild.dll
info: OmniSharp.MSBuild.MSBuildProjectSystem
MSBuildExtensionsPath environment variable set to /Users/user/.vscode/extensions/ms-vscode.csharp-1.9.0/bin/omnisharp/msbuild
info: OmniSharp.MSBuild.MSBuildProjectSystem
MSBuildSDKsPath environment variable set to /Users/user/.vscode/extensions/ms-vscode.csharp-1.9.0/bin/omnisharp/msbuild/Sdks
info: OmniSharp.MSBuild.MSBuildProjectSystem
MSBuild will use local OmniSharp installation.
info: OmniSharp.MSBuild.MSBuildProjectSystem
Detecting projects in '/Users/user/Projects/unity/pong-3d/pong-3d.sln'.
info: OmniSharp.MSBuild.MSBuildProjectSystem
Loading project from '/Users/user/Projects/unity/pong-3d/Assembly-CSharp-firstpass.csproj'.
fail: OmniSharp.MSBuild.ProjectFile.ProjectFileInfo
The reference assemblies for framework ".NETFramework,Version=v3.5" were not found. To resolve this, install the SDK or Targeting Pack for this framework version or retarget your application to a version of the framework for which you have the SDK or Targeting Pack installed. Note that assemblies will be resolved from the Global Assembly Cache (GAC) and will be used in place of reference assemblies. Therefore your assembly may not be correctly targeted for the framework you intend.
warn: OmniSharp.MSBuild.MSBuildProjectSystem
Failed to process project file '/Users/user/Projects/unity/pong-3d/Assembly-CSharp-firstpass.csproj'.
/Users/user/Projects/unity/pong-3d/Assembly-CSharp-firstpass.csproj
    Loading project from '/Users/user/Projects/unity/pong-3d/Assembly-CSharp.csproj'.

    The reference assemblies for framework ".NETFramework,Version=v3.5" were not found. To resolve this, install the SDK or Targeting Pack for this framework version or retarget your application to a version of the framework for which you have the SDK or Targeting Pack installed. Note that assemblies will be resolved from the Global Assembly Cache (GAC) and will be used in place of reference assemblies. Therefore your assembly may not be correctly targeted for the framework you intend.

    Failed to process project file '/Users/user/Projects/unity/pong-3d/Assembly-CSharp.csproj'.

/Users/user/Projects/unity/pong-3d/Assembly-CSharp.csproj
    Detecting CSX files in '/Users/user/Projects/unity/pong-3d'.

    Could not find any CSX files

    Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.CSharpWorkspaceOptionsProvider

    Configuration finished.

I figured out how to get it working for now. If I load the project via Unity the OmniSharp extension fails to load and intellisense does not work. If I navigate to the root of the project via terminal and start VSCode from there the OmniSharp extension loads and intellisense works fine. Not sure what is actually causing the problem though.

FYI I got this fixed by updating monoframework to the latest version.


Add check_random_states to __init__.py in utils submodule

This function can be useful at time, yet is not made easy to access. It is defined in mlresearch.utils._check_pipelines.py.

So the issue is to just move check_random_states function to init?

Yes, so importing the function in the init file and adding it to the __all__ list. In addition, the corresponding entry in the API documentation page (https://github.com/joaopfonseca/ml-research/blob/master/docs/api.rst) should be added.

Alright. Created a pull request

Merged PR #38. Thank you!


dart/appjit_cha_deopt_test times out in Dart 2 mode

optimization-counter-threshold may be interacting badly with the DFE

I think this is no longer the case.


MAINTAINERS: update file and generate patterns for CODEOWNER file

next step is to actually use the github handle instead of email, not sure how much we can catch use the emails...

Lets's split out the CODEOWNER change from the fixing of maintainers in this PR.

What do you want to split exactly? There are 2 commits, one for maintainers file, the other for codeowners, what is there to split?

I feel like we should convert MAINTAINERS to CODEOWNERS once and remove MAINTAINERS file after that conversion.

We could, however I find the MAINTAINERS file better structured and easier to read, i have a script that converts to CODEOWNERS.

I was just saying that the cleanup to MAINTAINERS can get merged now, while we figure out how to use CODEOWNERS and if we replace MAINTAINERS, use a script to update from it, etc..

I was just saying that the cleanup to MAINTAINERS can get merged now, while we figure out how to use CODEOWNERS and if we replace MAINTAINERS, use a script to update from it, etc..

I did cleanup MAINTAINERS to be able to produce a clean and a working CODEOWNERS. cleaning it on its own right now does not give us anything. CODEOWNERS is being generated from it, I want to see how things work out before removing MAINTAINERS if we decide to this, but nothing prevents us from merging both now, I do not expect any negative side effects right now only positive (I am hoping).

Do you want to submit the script as part of this?

Do you want to submit the script as part of this?

sure, it is a quick hack though, and do not think it should be added into the tree, I will just put it in the commit message.

How about tweaking the results of the script to sort the results alphabetically.
done


MatFormField documentation should prefer outline/fill style

In the most recent Material Design update, the "standard" / "legacy" style is now omitted from the spec with only the outline and fill styles remaining. We should update our own documentation and examples to reflect this update.

So is that the final decision to keep the "standard" and "legacy" style in the doc ? And if that's the case is there any point in keeping the issue open :) ?

@Orodan yes there is still a point as the examples need to be updated to use filled or outline by default. The legacy/standard should only be used in the section describing the 4 different appearances. Currently only the appearance example includes fill or outline appearances. All other examples use legacy and need to be updated to fill or outline.
@mmalerba what will be the default in the future (maybe v10?) fill?

I'm fine with either fill or outline being the new default. I don't think its important which one, but its nice to have the concept of a default since that can be overridden globally with a provider to change the whole app at once.


Allow for customization of project property source

Currently a Gradle project property can be used as a source for a buildParameter but the property name is identical to the buildParameter path <group>.<property>.
This causes problems when overriding properties from the environment (via ORG_GRADLE_PROJECT_<propName>) as *nix systems cannot use . in environment variable names. Much like the fromEnvironment method accepts a source variable name, it would be great if we could override a "source property name" as well.
It could be via an propertyAlias function that would fall back on the alias property name if the original property was unset.
The primary use case for overriding properties via environment variables is in unified CICD systems where abstracting on top of Gradle lifecycle tasks may still require individual projects to override their own pipeline/environment variables.

@mcwhitak why doesn't fromEnvironment work for you? Can you give a more comprehensive example for your use case?

fromEnvironment implies something about the source of the property and Gradle properties can be set from a variety of sources (environment variables, CLI, project local gradle.properties files, gradle properties files in gradle user home, etc).
Hardcoding fromEnvironment would mean the build now had an opinion on the possible sources and we'd have to track two mechanisms to set that were not analogous. (ex: With Alias you can set the property via any existing Gradle means, with fromEnvironment now you have a different property name vs environment variable despite gradle properties being able to be set via environment variables). fromEnvironment is useful when the input is not Gradle aware (like say, getting the commit sha from a CICD pipeline).
An alternative to this that would solve the same issue for me would be the ability to customize the group delimiter
group("myGroup") {
  delimiter("_")
  string("myString")
}

Though I feel that's a bit less intuitive as then the parameter path buildParameters.myGroup.myString vs the property path myGroup_myString will feel more at odds. With the "alias" mechanism the original property is still valid there's just a second (or more) fallback entrypoint.

@mcwhitak thank you for providing more context! I still struggle to understand your concrete use case. Can you give a very concrete example of how you're injecting configuration into builds? Something like
"We use maven-publish to publish to our internal repository. Our developers set interalRepoUser and internalRepoPassword in their gradle.properties files in the user home. On CI we want to inject this values but we don't want to put them on the command line..."
This makes it much easier for us to reason about a good general solution.
Thank you so much! 🙏🏻

Team decision
We understand the use case for setting a project property via ORG_GRADLE_PROJECT environment varibables and we see that this is currently not possible with build-parameters. We don't want to allow abitrary aliases for build parameters. So in order to address the ORG_GRADLE_PROJECT environment varibables use case, we are going to make build parameters also look up the lower_snake_base version of the property. This should cover your use case @mcwhitak. Can you please confirm?

Ok I think that makes sense, I'll close my existing PR and make a new one if you are all open to outside contributions.
I still believe positioning buildParameters as a compile-safe reference to multiple property resolution mechanisms will eventually require the feature I've outlined here but I can totally understand not wanting to open that door (as it'd be impossible to close).

@mcwhitak we're definitely open to outside contributions 👍🏻


Inputs: Hide Clear button when readonly

Description

Closes #4275 and closes #4277
How Has This Been Tested?

unit
Type of Changes

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

Checklist

[X] The PR is submitted to the correct branch (dev).
[X] My code follows the code style of this project.
[X] I've added relevant tests.

@ScarletKuro Do you know why the tests are failing? I'm missing something here

@ScarletKuro Do you know why the tests are failing? I'm missing something here

No idea. It seems like you broke something, since not only the new tests fail but some existing ones as well like SelectClearableTest and TimePicker_Should_Clear.

@ScarletKuro The usage of "readonly" is super inconsistent. Sometimes it means not-editable, sometimes not.  Take a look at a default MudSelect with Clearable=true and you'll see that the input has a readonly attribute. Read only doesn't always mean read only. It has different meanings. @henon Any ideas on how to escape this hole?

Actually, I don't see a problem here. Unlike MudAutocomplete, MudSelect never allowed to edit the input text. It just looks like a text input, but is none. We can change this in the new MudComboBox

Actually, I don't see a problem here. Unlike MudAutocomplete, MudSelect never allowed to edit the input text. It just looks like a text input, but is none. We can change this in the new MudComboBox

@henon I'm actually having this issue with several controls. Could you take a look at the failing tests

I'm actually having this issue with several controls.

We'll have to look at them on a case-by-case basis.

Could you take a look at the failing tests

OK

@danielchalmers I am pushing a fix that works.

This fixes some tests but I don't know what is the problem with MudMask and the TimePicker.

hope we will get rid of that State suffix

@ScarletKuro @henon Any insight or commits you guys can make would be really helpful because I'm not as familiar with the long history of the components and this would let me work on CSS/layout plans I have for several components. I didn't expect this small tweak to be so involved

@danielchalmers I will help you but it is very time consuming to get this working as it is even hard for me to get back into the complicated logic of these components. I also thought I might fix it in an hour then had to give up for the day after two.

@ScarletKuro @henon Any insight or commits you guys can make would be really helpful because I'm not as familiar with the long history of the components and this would let me work on CSS/layout plans I have for several components. I didn't expect this small tweak to be so involved

Typically, I don’t usually get involved with input components. The experts here are @mckaragoz and @Mr-Technician, who have worked on the disabled and read-only states, with @henon, who has been part of the project from the start.

even hard for me to get back into the complicated logic of these components

Yes, looking at all this, I really don’t like what I’m seeing. I might even suggest that we abandon it for now and take some time to rethink things.
I recommend writing down all the input components and going through them one by one to describe how each component should behave when it is in the disabled or read-only state. Should users be able to write in the input? Are clearable buttons visible, etc.? Then we can analyze what is currently implemented and come up with ideas on how to standardize this.
I think we could create an internal interface, IEditable, that would include ReadOnly, Disabled, ParentReadOnly, ParentDisabled, etc. (we might even split it into two interfaces if not all components require a parent). Would help us avoid duplication everywhere:
private bool GetClearable()
{
    if (SubscribeToParentForm)
        return Clearable && !GetReadOnlyState() && !GetDisabledState();
    return Clearable && !GetDisabledState();
}

protected bool GetReadOnlyState() => ReadOnly || ParentReadOnly;

protected bool GetDisabledState() => Disabled || ParentDisabled;

We could also use extension methods, and potentially have variations of them if some inputs require special cases.

While working on this I discovered that MudInput on its own doesn't even have any test cases ! I am adding the first one now for the readonly / clearable functionality

OK, I even manged to pull the ShowClearButton() logic up into MudBaseInput to avoid duplication. It was the same in every input component anyway. Except for Mask, which is a special case.

@danielchalmers you can take it from here.


Regression tests for age_global_graph

Waiting for the Travis CI checks to complete before reviewing

Looks good to me although I feel like the revision to this comment:
-- should return true (succeed) because previous command should not delete the 1st graph's context
should be restored because the replacement comment gives less information regarding this test case.

Updated the comments. Thanks @dehowef.

@PragyanD great work pragyan. Thank you

If possible though, do you mind squashing all the commits down to one commit so it appears more clean? Thanks. @PragyanD

@dehowef I think the commits are squashed now, thanks for the feedback.

@PragyanD Can you fix the conflicts, which must be be pretty simple?

@PragyanD The regression test fails -
test cypher_merge                 ... ok
test age_global_graph             ... FAILED
test age_load                     ... ok

The comments for an SQL file are -- not --- and we don't need a whole line of ----------....
Is this missing something?
--- delete unitialized graph context
--- should throw exception graph "ag_graph_4" does not exist
SELECT * FROM cypher('ag_graph_4', $$ RETURN delete_global_graphs('ag_graph_4') $$) AS (result agtype);

-- delete_GRAPH_global_contexts
--

-- load contexts again

Additionally, the following shouldn't be this way -
LOAD 'age';
SET search_path TO ag_catalog;

--

-- delete_specific_GRAPH_global_contexts
--

--

It should be more like -
LOAD 'age';
SET search_path TO ag_catalog;

--
-- delete_specific_GRAPH_global_contexts
--

@jrgemignani Thanks John, I must've erased some of the lines while resolving the conflicts with master.

The build failed. You should run your PR locally to fix the regression test that failed.

The build failed. You should run your PR locally to fix the regression test that failed.

working on this right now. I will submit a patch as soon as possible.

It still fails on -
test age_global_graph             ... FAILED
Due to -
[jgemignani@cent7nuc7i5bnk incubator-age]$ diff regress/results/age_global_graph.out regress/expected/age_global_graph.out
1,19d0
< /*
<  * Licensed to the Apache Software Foundation (ASF) under one
<  * or more contributor license agreements.  See the NOTICE file
<  * distributed with this work for additional information
<  * regarding copyright ownership.  The ASF licenses this file
<  * to you under the Apache License, Version 2.0 (the
<  * "License"); you may not use this file except in compliance
<  * with the License.  You may obtain a copy of the License at
<  *
<  * http://www.apache.org/licenses/LICENSE-2.0
<  *
<  * Unless required by applicable law or agreed to in writing,
<  * software distributed under the License is distributed on an
<  * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
<  * KIND, either express or implied.  See the License for the
<  * specific language governing permissions and limitations
<  * under the License.
<  */
<
[jgemignani@cent7nuc7i5bnk incubator-age]$

Additionally, you need to rebase it off the current master -
src/backend/parser/cypher_transform_entity.c: In function ‘find_transform_entity’:
src/backend/parser/cypher_transform_entity.c:68:5: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
     ListCell *lc;
     ^

Thanks John, I'm having some issues with the AGE installation in my end so I'm not able to test the files.

Good job @PragyanD! Thanks @aked21!


Vagrant osx ImportError: No module named splash.server

Following the installation guide http://portia.readthedocs.org/en/latest/installation.html I cannot access localhost:8000. /var/log/upstart/slyd.log says:
Traceback (most recent call last):
  File "bin/slyd", line 3, in <module>
    import splash.server
ImportError: No module named splash.server

Hi, the build of splash docker/vagrant images in the master/develop branches is a bit broken, could you try the qt5 branch instead?

Thanks for the reply. I don't see a branch qt5, where should I look for it?

Err, sorry, qt5 branch is only in my fork for the moment

Yes, I figured out this was a PR.
git fetch origin pull/330/head:pr-330
git checkout pr-330

does the trick. However I still cannot run slyd with the same reason. I tried localhost:8000 and localhost:9001 (since you changed that in the PR). I ran vagrant destroy and vagrant up and I am waiting to see if it would work.

Finally it seems to work with one caveat:
PYTHONPATH=/vagrant/slyd:/vagrant/slybot bin/slyd -p 9002 --root ./dist had to be changed to 9001 because this is the port which we forward.
Thanks a lot for the help.


Simplified light_client_bootstrap header

The light client bootstrap function is recreating the block header stored in state.latest_block_header.
Cleaned up the definition so that it's less confusing.
This should be no functional change, just make the spec clearer when trying to implement.
Thanks to @robzajac for noting this so that we can raise this PR.

Ok so that makes sense, the state_root isn't correct in the object in the state.. I think I'll just close this PR because it's not going to work.


Switch the native port to cppwinrt 2.0 and latest windows SDK.

The goal here is to update the native port to use cppwinrt 2.0 instead of C++/CX.

won't merge?


Dunfell RB5 support

run tests

hey @lumag , can you please record the cherry pick with '-x'. i think it really helps when looking back at history.
thanks

@ndechesne done


issues with properties

There are issues with properties / boolean properties in Inferno.  It's basic knowledge that a boolean can only be either true or false.
Checked is also a boolean, but not counted for on this line:
https://github.com/trueadm/inferno/blob/dev/src/DOM/patching.js#L124
Shouldn't be set to empty string. That is not a boolean value.
This line: https://github.com/trueadm/inferno/blob/dev/src/DOM/patching.js#L127
If it's a boolean, it can be either true or false. Nothing else. Not counted for.
There are no unset of this properties in the diff either. And when you unset a boolean property it has to have the value false. Not true or somethng else.
@trueadm Why isn't this fixed? I repeaed this many times on slack.
Quoted from Google to avoid discussions:
"In computer science, the Boolean data type is a data type, having two values (usually denoted true
and false), intended to represent the truth values of logic and Boolean algebra. It is named after
George Boole, who first defined an algebraic system of logic in the mid 19th century."

@Kflash Thanks – truthy values resovle to true and falsey to false. Fixed and closed.


Intrinsic for custom images

Hello I love what you have built and would like to get camera intrinsically for my own images. Would you kindly tell me how to generate them for custom images?

Please take a look at Colmap. Colmap has a "sparse bundle adjustment" mode, where it recovers camera extrinsics and intrinsics. Please also check out the Readme of DeepVoxels, where I briefly discuss Colmap.


Rename meta search to external sources

Hi @vj27111997, this work was already completed in #1868. Please look at our good first issues if you'd like something else to work on.


Design System > Components > User table > Default > Sometimes icon is not display proper.

Describe the bug
Sometimes icon is not display proper.
E.g. User Table
Datatable
Steps to reproduce bug

Go to 'https://stage-react.raaghu.ai/'
Navigate to 'Design System > Components > User table > Default'

Actual result-
Sometimes icon is not display proper.
E.g. User Table
Datatable
Screenshots

Expected result-
Sometimes icon should be display proper.
E.g. User Table
Datatable
Desktop (please complete the following information):
-OS: [Windows 10 Enterprise]
-Browser [ edge]
-Version [ 125.0.2535.92]

Fixed on dev

Fixed on staging


Unify Data Model

Instead of duplicating the metrics, use the simplified MonthlyServiceMetrics.

With some corresponding, supporting changes here: https://github.com/alphagov/gsd-view-data/pull/83


add context to logging of supervisor work queue interaction

This includes the changes from #602 to enable builds


Add linux aarch64 wheel support

Added linux aarch64 wheel support.
Related to #53. @bstellato Could you please review this PR..

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

@bstellato, could you please review this PR? Thanks.

@bstellato, could you please review this PR? Thanks.

@bstellato, could you please have a look at this PR? Thanks in advance.

@bstellato, could you please have a look at this PR? Thanks in advance.

@bstellato, could you please have a look at this PR?

@vineetbansal, could you please review this PR? Thanks in advance.

I'm curious why the commit been reverted? I couldn't find any detail in the revert commit message either.


Request failed with status code 500

Hello,
I am getting requested failed with status code 500 error. I was wondering if there is a way to debug this. The verbose output does not give much details.
destreamer version: latest master
NodeJs: 12.18.3
ffmpeg: 4.2.2, 4.3.1 and latest master
Chromium: 80.0.3987 64 bit
I tried to open Chromium manually from node_modules/puppeteer/.local-chromium/win64-72234/chrome-win/Chrome.exe and I am able to successfully watch the video using Chromium.
Note: I am using an HTTP proxy to access the internet.

You could add console.dir(err) right before this line here and rebuild.
https://github.com/snobu/destreamer/blob/7bfc565a05ffc95e462afa56cff3419f9b3592bb/src/ApiClient.ts#L30
That should print out the body of that 500 error as well.

Thank you @snobu the error is very clear now, it was proxy related so I am closing this issue.

Reopening this since we should return response body in the final throw. Thanks for pointing out verbose doesn't do that, @kostapsimoulis.

@snobu per axios documentation you were right about err.response.data (see here)


BXMSPROD-17 change kie-docs PR verification job to standalone.

Signed-off-by: Piotr Szubiakowski<EMAIL_ADDRESS>

Jenkins please retest this.

Jenkins please retest this.


TouchableOpacity don't call onPress callback when hitSlop is set (on some devices)

Environment
we also built over expo.
React Native Environment Info:
System:
OS: macOS 10.14.2
CPU: (4) x64 Intel(R) Core(TM) i5-5350U CPU @ 1.80GHz
Memory: 40.01 MB / 8.00 GB
Shell: 3.2.57 - /bin/bash
Binaries:
Node: 10.15.0 - /usr/local/bin/node
npm: 6.4.1 - /usr/local/bin/npm
SDKs:
iOS SDK:
Platforms: iOS 12.1, macOS 10.14, tvOS 12.1, watchOS 5.1
Android SDK:
API Levels: 25, 27, 28
Build Tools: 27.0.3, 28.0.3
System Images: android-21 | Google APIs Intel x86 Atom_64, android-27 | Intelx86 Atom_64
IDEs:
Android Studio: 3.2 AI-181.55<IP_ADDRESS>56338
Xcode: 10.1/10B61 - /usr/bin/xcodebuild
npmPackages:
react: 16.5.0 => 16.5.0
react-native: https://github.com/expo/react-native/archive/sdk-31.0.1.tar.gz =>0.57.1
npmGlobalPackages:
react-native-cli: 2.0.1
Description
TouchableOpacity don't call onPress callback when hitSlop is set, (atleast on samsung note8).
It works on emulator.
Reproducible Demo
<EMAIL_ADDRESS>

So I encounter the same problem again today on samsung note8, this time hitSlop was not defined and it was calling onPressIn and then immediately calls onPressOut without calling onPress


When I use atomic operations, does cache consistency cause some threads to endless loop?

What version of Go are you using (go version)?

$ go version
go version go1.13.5 darwin/amd64

Does this issue reproduce with the latest release?
don't try
What operating system and processor architecture are you using (go env)?
go env Output
$ go env

What did you do?
When I test the following code, some threads will appear dead loop, and the threshold of I * J is about 60000. But when I add print (I), " os.Creat (test), or os.Open("test"), this will not happen.

What did you expect to see?
What did you see instead?

Thank you for raising this issue. Unlike many projects, the Go project does not use GitHub Issues for general discussion or asking questions. GitHub Issues are used for tracking bugs and proposals only.
For asking questions, see:

Stack Overflow with questions tagged "go"

The Go Forum, a web-based forum

Gophers Slack, use the invite app for access. The #general channel is a good starting point.

Go Community on Hashnode with questions and posts tagged with "go"

The golang-nuts mailing list

IRC channel #go-nuts on Freenode

Thank you for raising this issue. Unlike many projects, the Go project does not use GitHub Issues for general discussion or asking questions. GitHub Issues are used for tracking bugs and proposals only.
For asking questions, see:

Stack Overflow with questions tagged "go"

The Go Forum, a web-based forum

Gophers Slack, use the invite app for access. The #general channel is a good starting point.

Go Community on Hashnode with questions and posts tagged with "go"

The golang-nuts mailing list

IRC channel #go-nuts on Freenode


"|" symbol display error in .yml

The"|" symbol  display to a "/".

Other IDE display normally：

What theme are you using? It seems to me that this is rendered italics.
To inspect this, set the cursor before the | and invoke the command Inspect editor token and scopes. If you can paste a screenshot of the inspection hover that would be great.

theme: Dart+
xx.yml:

xx.java:

gif:

As for the theme, do you mean the Dart+ or Dark+? The gif shows that you select the built-in Dark+.
The built-in Dark+ does not define 'italics for keyword.control.
Also, I can't reproduce the issue on my machine.
Did you customize the theming rules in the settings? (editor.tokenColorCustomizations)

I got a setting:
    "editor.fontFamily": "Monaco, 'Courier New', monospace",
    "editor.tokenColorCustomizations": {
        "comments": {},
        "keywords": {
            "fontStyle": "italic",
            // "foreground": "#ffee00"
        },
        "functions": {
            "fontStyle": "bold",
            // "foreground": "#15ff00"
        },
    },

But, "|" is not a keywords.

That explains what you see. The YAML grammar gives the | the keyword.control scope, which is typically used for keywords like if, switch.
You can file an issue against the YAML grammar (https://github.com/textmate/yaml.tmbundle) and argue that the '|' should not be classified as a keyword.


include m2m values in detail view

Implements #309. Feel free to discard it if dont like it.
Example: the staff row:

I'm afraid this could cause big issues with a number of queries and related objects in the general case.
Generally, I don't see a big value of making complex generic default detail view, b/c in real cases the detail view would be customized in a unique way.


do not have the irt_sheet.tsv

Dear authors,
Thanks for developing this tools for DIA_MS data analysis. I want to know if I can analyze the DIA without irt_sheet.tsv. Also can I use my own fasta to do the alignment?

Hi,
Thanks for checking out this workflow, its been a while that I was working on this but let me try to help here - were you using any kind of irts in your experiment? or do you have a library of cirts that you could use that are typically found in proteomics samples? Just from a fasta its not possible but there is an alternative option to generate pseudo_irts using the flag "--generate_pseudo_irts". It uses very confident Ids from your library across the RT range in order to do the alignment.

Thanks for your reply. There are no any kind of arts in the experiment. It is said that we can use the housekeeping gene to as this.Also the Diann use deep learning method to generate one library for this. Is this like generate pseudo_irts?  I just wonder if we do not give the fasta, how to do the alignment. How does the software know the species of the samples?

No problem, DIA-NN is an entirely different approach that can do library-free search. This workflow here uses the "classical" library-based approach. You could give the "generate_pseudo_irts" option a try for your samples, but maybe youre better off using DIA-NN or other workflows. In any case, as this is not a bug but rather a special application I would prefer to close the issue again.

Thanks for your reply. I will try 'generate_pseudo_irt'.  I still do not understand the alignment.I just wonder if we do not give the fasta, how to do the alignment. How does the software know the species of the samples?

There is no need to know the organism. Please check the paper again - the alignment should be explained in there.


styles-legacy.min doesn't exist

Issue description:
Noticed styling issue in the customizer on all the switches and radio buttons. A css file appears to be missing: controls/css/styles-legacy.min.css?ver=3.0.17
Version used:
This is in 3.0.17

And /controls/css/styles.min.css?ver=3.0.17

This has already fixed in the wordpress.org version, please delete your copy and re-download. It was a problem with the SVN upload.


TLS examples should load certs from memory

There is a pretty big issue with the tls examples currently: they rely on relative paths to find the correct certs. We could move these into global constexpr string views and load directly from memory to fix this quite easily.
The client-websocket-secure example is broken currently, unless you run it from four directories down relative to the malloy route (the bindir is currently 2 directories down, presumably this is from before we moved everything in bin), and all the other (tls) ones are broken unless run from <malloy root>/*/*.
Ideally we would also load the static content direct from memory, but that might cause issue with demonstrating the file routing.
Yet another alternative could be to bake the absolute paths of the files into the examples via defines, which would be ok I think because they're just examples and not designed to be used on their own, some dependence on malloy files at runtime is ok (just not to the point of dependence on where the binary is invoked from or located lol), but I don't know how useful it would be from a learning perspective to just have a bunch of macros instead of file paths

We could move these into global constexpr string views and load directly from memory to fix this quite easily.

+1

Ideally we would also load the static content direct from memory, but that might cause issue with demonstrating the file routing.

Yep, exactly.

Yet another alternative could be to bake the absolute paths of the files into the examples via defines, [...]

Wait for it....

[...], but I don't know how useful it would be from a learning perspective to just have a bunch of macros instead of file paths

Exactly my reasoning too!
I'd say lets do this: TLS certs/keys for the example(s) are loaded directly from memory. The remaining parts I'd leave as it is for now to keep the examples as readable as possible.
I don't really like that the examples have so much code duplication going on but I really don't want to do that thing where you create infrastructure specific to maintaining examples / lowering code duplication as it makes it harder for someone looking at it to understand "what it takes" to use this library and ideally "how easy it is to use it".

I don't really like that the examples have so much code duplication going on but I really don't want to do that thing where you create infrastructure specific to maintaining examples / lowering code duplication as it makes it harder for someone looking at it to understand "what it takes" to use this library and ideally "how easy it is to use it".

Yeah I was originally going to include ideas for tackling this in the issue too, but looking at the examples they are mostly just a few lines of setup and one line of teardown (at least after #102), really quite minimal. I don't really see how we could merge that in a way which was actually shorter while still being flexible enough for all the slight variations in setup needed. One thing I suppose could be moving the base config creation to a function which just took thread count and using that for anything but the basic examples (by which point you're assumed to know how to init malloy already), it would only save a handful of lines but its a least a little less noise. Could also have a function which did the more advanced setup of the server config with default values, and introduce it to the user somehow before we started using it, that would also remove some setup which I think is helpful for the more advanced examples where the focus should be on the new stuff.
Like I said though its already pretty good, especially for most c++ libraries "basic" examples, and most especially for a networking library; serving a file with load balanced across multiple threads and async in <50 lines is damn impressive (its one of the reasons I started using and contributing to malloy, the router and client interfaces are really nice well done :p)

Thank you - I really appreciate your feedback!
As you know there are some rough edges based on the fact that this was initially meant to be a playground but your contributions are excellent - I appreciate that a lot. I actually picked up a few tricks from you which made their way into other software projects by now :p
I really hope that this library can get some traction in the (near) future :)
I do agree that we might justify using some setup function(s) for the more advanced examples.


fix(search): add search for the docs

Also fix google indexing by adding meta noindex
Signed-off-by: Charly Molter<EMAIL_ADDRESS>

Netlify preview <3


Godot 3.1.1 android screen flickering

Godot version:
3.1.1
OS/device including version:
version : android 9
Issue description:
When I start a game on Android, the screen flickers and nothing happens
A few months ago there was nothing wrong with exporting to Android
But I've had a problem since I changed my developing pc
The newly created project will run fine without any problems.
I changed the path of the files again and downloaded and set the Android custom template again.
I am using a separate splash image
Image with white background, GODOT logo is not set as splash image
Steps to reproduce:
Connect your Android device to your pc
Click One-Click Deployment
The screen flickers when the game is run on an Android device after all deployments
Minimal reproduction project:
I don't know what's mean

Seizure warning (flickering screen)

@MungMoongYi I recommend trying Godot in GLES 2 mode. See this comment If that does not fix the issue then report back with more details like adb logcat output and more phone specs.

I remember having similar issues quite some time ago, turns out I was using the wrong export template (different godot versions for template and editor).
Just something you could double check given that you say you are using a custom export template

@MungMoongYi I recommend trying Godot in GLES 2 mode. See this comment If that does not fix the issue then report back with more details like adb logcat output and more phone specs.

I've been using GLES2 since the beginning.

I remember having similar issues quite some time ago, turns out I was using the wrong export template (different godot versions for template and editor).
Just something you could double check given that you say you are using a custom export template

Thank you!
As you said, the version of the custom package was a problem.
Now I can run the game successfully
There's still a problem with so laggy

I'm surprised that the splash screen on your custom template is a Godot 2.1 logo. Is that on purpose, or do we actually still have an occurrence of the old logo somewhere that is used on Android?


Remove nptyping as a dependency

This package is noisy with warnings and is no longer worth it. We can use np.ndarray as the type for all of these instances

There is also numpy.typing.  We might even decide to enable its mypy plugin.
https://numpy.org/devdocs/reference/typing.html


Prevent invalid value for batch node property

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

Proposed changes
Because the minimum value for batch node property is not set, we can input invalid values, such as a negative number.
I fixed to set minimum values of properties and spinners to prevent invalid value.
Checklist

[x] I have read the contribution guidelines
[ ] For non-bugfix PRs, I have discussed this change on the mailing list/slack team.
[x] I have run grunt to verify the unit tests pass
[ ] I have added suitable unit tests to cover the new/changed functionality

Coverage remained the same at 76.506% when pulling 211eeea05d9d92ccc33d49e6c95e05c3206713ed on node-red-hitachi:master-batch into 5cf9c07b7332869a108fb1a3d39ae29b16580d68 on node-red:master.


decomp: finish vol | helix-water | minecart

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
5 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.005%) to 54.8%

Files with Coverage Reduction
New Missed Lines
%

goalc/debugger/Debugger.cpp
5
72.58%

Totals

Change from base Build<PHONE_NUMBER>:
-0.005%

Covered Lines:
40803

Relevant Lines:
74458

💛  - Coveralls


⚠ Bofrost.de has degraded performance

In 17d6d5a, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 2028 ms

Resolved: Bofrost.de performance has improved in e75e0bc after 19 minutes.


Fix presto memory configs, update to latest release.

Instead of using fixed defaults, configure query-per-node memory,
distributed total memory, and system reserved memory with help
from the cluster's spark executor configs.

Update to version 0.144.1.

Seems reasonable overall.

Re-opened in https://github.com/GoogleCloudPlatform/dataproc-initialization-actions/pull/50, I can't seem to fix this branch's commit logs.


chore: make benchmarks runnable in CI

Need to go drop off some servers so I'll probably miss when you mark it ready, but feel free to ping me on discord (@bdraco)


Incorrect Position of 3d Bounding Boxes

I am trying to visualize 3d bounding boxes using open3d. The bounding boxes show up, but in the wrong place. In the video, the bounding boxes (shown in red) show up way above the vehicle.
https://user-images.githubusercontent.com/49762976/232761382-45169887-6bb7-4a53-a607-8f31be4c096f.mp4
My lidar callback function looks as follows
def lidar_callback(point_cloud, point_list, world, vehicle, lidar):
    """Prepares a point cloud with intensity
    colors ready to be consumed by Open3D"""
    data = np.copy(np.frombuffer(point_cloud.raw_data, dtype=np.dtype('f4')))
    data = np.reshape(data, (int(data.shape[0] / 4), 4))

    # Isolate the intensity and compute a color for it
    intensity = data[:, -1]
    intensity_col = 1.0 - np.log(intensity) / np.log(np.exp(-0.004 * 100))
    int_color = np.c_[
        np.interp(intensity_col, VID_RANGE, VIRIDIS[:, 0]),
        np.interp(intensity_col, VID_RANGE, VIRIDIS[:, 1]),
        np.interp(intensity_col, VID_RANGE, VIRIDIS[:, 2])]

    # Isolate the 3D data
    points = data[:, :-1]

    # We're negating the y to correclty visualize a world that matches
    # what we see in Unreal since Open3D uses a right-handed coordinate system
    points[:, :1] = -points[:, :1]

    int_color *= np.array([0, 255, 0])
    world_2_lidar = lidar.get_transform().get_inverse_matrix()
    for npc in list(world.get_actors().filter('*vehicle*')) + list(world.get_actors().filter('*pedestrian*')):
    # Filter out the ego vehicle
        if npc.id != vehicle.id:
            bb = npc.bounding_box
            dist = npc.get_transform().location.distance(vehicle.get_transform().location)
            # Filter for the vehicles within 50m
            if dist < 50:

            # Calculate the dot product between the forward vector
            # of the vehicle and the vector between the vehicle
            # and the other vehicle. We threshold this dot product
            # to limit to drawing bounding boxes IN FRONT OF THE CAMERA
                forward_vec = vehicle.get_transform().get_forward_vector()
                ray = npc.get_transform().location - vehicle.get_transform().location
                verts = [[] for _ in range(3)]
                if forward_vec.dot(ray) > 1 or True:
                    verts = [v for v in bb.get_world_vertices(npc.get_transform())]
                    verts = [[v.x, v.y, v.z] for v in verts]
                verts = np.array(verts)
                if verts.shape[1] > 0:
                    colors = np.ones((verts.shape[0], 3))*np.array([255, 0, 0])
                    points = np.concatenate((points, verts), 0)
                    int_color = np.concatenate((int_color, colors), 0)
    point_list.points = o3d.utility.Vector3dVector(points)
    point_list.colors = o3d.utility.Vector3dVector(int_color)

Do you know why does it happen and how can I circumvent the problem?
CARLA version: 0.9.14
Platform/OS: Ubuntu 20.02

@JosenJiang Thank you for your reply. I have made sure that the frame numbers are the same by adding the following line to my lidar_callback:
print(f"\nworld.get_snapshot().frame: {world.get_snapshot().frame}\tpoint_cloud.frame: {point_cloud.frame}")

And this is the result that I've got
world.get_snapshot().frame: 82289	point_cloud.frame: 82289
world.get_snapshot().frame: 82290	point_cloud.frame: 82290
world.get_snapshot().frame: 82291	point_cloud.frame: 82291
world.get_snapshot().frame: 82292	point_cloud.frame: 82292
world.get_snapshot().frame: 82293	point_cloud.frame: 82293
world.get_snapshot().frame: 82294	point_cloud.frame: 82294
world.get_snapshot().frame: 82295	point_cloud.frame: 82295
world.get_snapshot().frame: 82296	point_cloud.frame: 82296
world.get_snapshot().frame: 82297	point_cloud.frame: 82297
world.get_snapshot().frame: 82298	point_cloud.frame: 82298

So they are indeed the same.
I believe that the reason is rather in incorrect geometric transformation: as you can see in the video, the lidar data and the bounding boxes are synchronized, but they show up in the different places.
Do you know how to fix that?

@RostyslavUA I believe that the transformation is incorrect. I've used the following code to create the 8-point bounding box visualization.
assert self.sensor is not None, f"{self.name} has not been added to a sensor yet"
        location = self.sensor.location()
        forward_direction = self.sensor.forward_vector(normalized=True)
        w2c = self.sensor.w2s()[0:4, 0:4]
        for bbox in self.bounding_box_set:
            # Filter bounding boxes that are too far away
            if bbox.location.distance(location) > self.sensor_range:
                continue
            ray = bbox.location - location
            ray_direction = ray/ray.length()

            # Filter bounding boxes that are not in the field of view
            cos_sim = forward_direction.dot(ray_direction)
            if cos_sim < self.cos_threshold:
                continue
            verts = np.array([carla_vec2np_array(v) for v in bbox.get_world_vertices(carla.Transform())])
            verts = np.hstack((verts, np.ones((verts.shape[0], 1))))
            verts = np.dot(w2c, verts.T).T
            # Negate the y-axis to match the camera coordinate frame
            verts[:, 1] *= -1
            # Rescale the points to the world coordinate frame
            verts[:, 0] /= verts[:, 3]
            verts[:, 1] /= verts[:, 3]
            verts[:, 2] /= verts[:, 3]
            # Remove the fourth component
            verts = verts[:, 0:3]

            scene.points.extend(o3d.utility.Vector3dVector(verts))
            scene.colors.extend(o3d.utility.Vector3dVector(np.tile([255, 0, 0], (verts.shape[0], 1))))
        self.sensor.pcd = scene
        return scene

def w2s(self): # function in my sensor wrapper class.
        """
        Get the world-to-sensor transformation matrix.

        Returns
        -------
        np.ndarray
            The world-to-sensor transformation matrix.
        """
        return np.array(self.transform().get_inverse_matrix())

@RostyslavUA have you resolved the issue yet? I tried integrating the code provided by @Tottowich, but unfortunately it doesn't work for me.


README_en.md may contain link to VGA -> SCART cable that may be unsuitable

Hi,
While reading through the README_en.md, I noticed a potential issue/inconsistency with the spanish version (README.md).
Specifically, in the section "What do you need to use it?" :
After the explanation about VGA -> SCART, which mentions the recommendation for a particular cable because :

It is recommended to use this cable with the Analogizer because it is verified that it takes the +5V from pin 9 of the VGA connector and passes it to pin 16 of the SCART

There is a list of links, 2 of them for VGA -> SCART cables :

https://manuferhi.com/p/cable-rgb-scart-for-mister-mist-sidi-and-n-go
https://ultimatemister.com/product/rgb-scart-cable/
(more links)

The second link (https://ultimatemister.com/product/rgb-scart-cable/) contains a cable which seems unsuitable based on the following information in the linked page :

VGA pin 14 connects to SCART pin 16 directly (1,6v or less, no attenuation required)
Our cable does not use VGA PIN 9, works with MiSTer I/O set in 3.3v

When looking at the spanish README, that link in question is not present :

https://manuferhi.com/p/cable-rgb-scart-for-mister-mist-sidi-and-n-go
https://antoniovillena.com/product/splitter-for-official-mister/
https://manuferhi.com/p/snac-adapter-for-mister
https://ultimatemister.com/product/snac-db15-neogeo/

The link in question was added exclusively to the english readme in the following changeset :
https://github.com/RndMnkIII/Analogizer/commit/a6d653f981da6fd88093d54297811fb106a0b36a

Based on this, I believe this may have been a typo. I'm bringing it to your attention just in case it is indeed problematic.
Thank you very much for the work you are doing! Please accept my apologies if I misunderstood and am mistaken.
Best Regards,

VGA pin 14(VSync) is tied to high state in the Analogizer interface for RGBS video output or 3.3V. I think this should work with this cable.


go: properly handle iterator for nested messages

Codecov Report

Merging #2005 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #2005      +/-   ##
============================================
+ Coverage     85.19%   85.19%   +<.01%
- Complexity     4982     4983       +1
============================================
  Files           457      457
  Lines         20056    20058       +2
  Branches       2104     2105       +1
============================================
+ Hits          17086    17088       +2
  Misses         2269     2269
  Partials        701      701

Impacted Files
Coverage Δ
Complexity Δ

...gle/api/codegen/transformer/go/GoSurfaceNamer.java
90.84% <100%> (+0.13%)
59 <0> (+1)
:arrow_up:

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 4358726...b47f940. Read the comment docs.


Add RAV4_TSS2 and RAV4H_TSS2 to Improved long tune

Add RAV4_TSS2 and RAV4H_TSS2 to Improved long tune
P.S. those who tried my long tune on non-TSS2 cars - please DM me your experience on Discord.

People with good experience please also comment on the PR.

I hope it's not too late, but @briskspirit longitudinal tune is very good, and I would use it.

@C0mpl3t3N00b Are you driving a Rav4/Rav4 hybrid with TSS2?

Looks like it https://github.com/commaai/openpilot/issues/20054

@C0mpl3t3N00b Are you driving a Rav4/Rav4 hybrid with TSS2?

I guess I need to increase the amount of issues I post, but yes 2020 Rav4 Hybrid. :)


AbstractMirrorModel.getPositionFromOffset: Model is disposed

Issue Id: 70b18577-7bef-134a-009f-883d9af52536Versions - 0.10.6-release-  dfc08dc9edf6607483b193bf934d3badc3fa2e11Stack Error: AbstractMirrorModel.getPositionFromOffset: Model is disposed/vs/editor/common/model/mirrorModel.ts#L130:9 (AbstractMirrorModel.getPositionFromOffset)/vs/languages/json/common/jsonWorker.ts#L174:37 (JSONWorker.doValidate)    at Array.forEach (native)/vs/languages/json/common/jsonWorker.ts#L169:41 (JSONWorker.doValidate)vs/base/common/worker/vs/base/common/winjs.base.raw.js#L1440:0 (onComplete)vs/base/common/worker/vs/base/common/winjs.base.raw.js#L1127:0 (_notify)vs/base/common/worker/vs/base/common/winjs.base.raw.js#L1343:0 (enter)vs/base/common/worker/vs/base/common/winjs.base.raw.js#L1311:0 (_run)vs/base/common/worker/vs/base/common/winjs.base.raw.js#L1440:0 (onComplete)vs/base/common/worker/vs/base/common/winjs.base.raw.js#L1127:0 (_notify)

Now with the JSON language in an extension this won't happen in VS code anymore. However, other legacy modes Monaco might be impacted
The JSON worker gets the mirrorModel from the resourceService. It always gets it new on every 'validate', so it looks like a bug in the resourceService that it would hand over a disposed model.

We don't have that code anymore


RTN17a

To be revisited after #385 is merged.

👍

PTAL

👍

The fix used to pass but now it crashes, I guess because of some conflict with what got merged to master in the meantime. I'll merge this and #388, #390, #444 and #406 just with pending tests, without the fixes, and then do a separate PR fixing all of them together, to avoid conflicting fixes between them.


Add support for IPV6 in dns resolvers

fixes https://github.com/kubernetes/contrib/issues/2188
fixes https://github.com/kubernetes/contrib/issues/2193
fixes https://github.com/kubernetes/ingress/issues/74

This change is

I am hitting the issue in my current setup, wondering if there will be a release once this patch is merged or should I for now remove ipv6 from resolv.conf ?

@gurvindersingh for now remove ipv6 from resolv.conf

yeah that's what I did to have a workaround, but wondering about possible 0.8.4 or 0.9.0 with the fix.

@gurvindersingh we are still migrating and changing the code. I thin we need a couple of weeks before a release. Apologies for the delay

@aledbf thanks the update. Will look forward to the next release.

Coverage increased (+0.08%) to 39.847% when pulling 77608bf82ab85149b6223b852fbc5979936fc74f on aledbf:dns-resolver into 8e90fc029025f5af82121c9b16fdbab222eef1e0 on kubernetes:master.

Using
core@localhost ~ $ cat /etc/resolv.conf
nameserver <IP_ADDRESS>
nameserver 2001:4860:4860::8888

setting dnsPolicy: Default the generated nginx.conf is:
+    access_log /var/log/nginx/access.log upstreaminfo if=$loggable;
+    error_log  /var/log/nginx/error.log notice;
+
+    resolver <IP_ADDRESS> [2001:4860:4860::8888] valid=30s;
+
+    # Retain the default nginx handling of requests without a "Connection" header
+    map $http_upgrade $connection_upgrade {
+        default          upgrade;
+        ''               close;
+    }

Coverage increased (+0.08%) to 39.847% when pulling 61dad19f8e94ee873c50f1b6085b0abeaed72f55 on aledbf:dns-resolver into 8e90fc029025f5af82121c9b16fdbab222eef1e0 on kubernetes:master.

Thanks, LGTM


[RESOLVED] Interface eth0(): High error rate (>2 for 5m)

Problem started at 09:12:29 on 2023.05.31
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Operational data: errors in: 1, errors out: 0
Original problem ID: 22991
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23483&eventid=22991

Problem has been resolved in 3m 0s at 09:15:29 on 2023.05.31
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Original problem ID: 22991


[WIP] Add http to copied url and move function to componentWillReceiveProps

Issue:

http was returned in url shortener, some editor like jira cannot parse url directly as a link
after CopyQueryTabUrl was changed to a pure component, componentWillMount was not called when props update, share-url always takes the old sql
Before

Done:

added http to returned url in shortner
add a getText function with callback in CopyToClipboard component, once clicked, getText function is called to retrieve text to be copied to clipboard

After:

needs-review @ascott @mistercrunch @bkyryliuk

LGTM, wait for the build.

still LGTM


Dockerfile: Use ubi as base image

I'm trying to use butane in a Dockerfile as part of a multi-stage
build, and not having anything except the binary in the container
breaks that since Dockerfile wants to invoke things via /bin/sh.
Use ubi8 as a base.

Hmm, yeah, that makes sense.  Let's use registry.fedoraproject.org/fedora:35 or registry.fedoraproject.org/fedora-minimal:35 as a base though.  We usually don't use the UBI in our upstreams.
Also, is the path change important for your use case?  I'm wondering if that risks a compat break somewhere.

We usually don't use the UBI in our upstreams.

There's no restrictions on UBI by design; a specific advantage over fedora as a host here is we don't need to churn this as often.
But we are unlikely to be consistent on this, and it doesn't matter much to me so...switched to fedora.

Also, is the path change important for your use case?

Nah, I just thought it was cleaner.  Reverted.

There's no restrictions on UBI by design; a specific advantage over fedora as a host here is we don't need to churn this as often.

Yeah, I know.  But I'd feel weird about a community project using a non-community base image.  Thanks for making the switch.


Unhandled FileNotFound error when loading in directory without venv

$ xonsh
Traceback (Most recent call last):
71 /Users/ryan/.local/pipx/venvs/xonsh/lib/python3.10/site-packages/xontrib/avox_poetry/venv.py on_post_init --> activate_venv(P
ath.cwd())
46 /Users/ryan/.local/pipx/venvs/xonsh/lib/python3.10/site-packages/xontrib/avox_poetry/venv.py activate_venv --> venv = get_ven
v_path(path)
28 /Users/ryan/.local/pipx/venvs/xonsh/lib/python3.10/site-packages/xontrib/avox_poetry/venv.py get_venv_path --> venv = has_ven
v_with_same_name(path.name)
19 /Users/ryan/.local/pipx/venvs/xonsh/lib/python3.10/site-packages/xontrib/avox_poetry/venv.py has_venv_with_same_name --> for
env in iter_venvs():
40 /Users/ryan/.local/pipx/venvs/xonsh/lib/python3.10/site-packages/xontrib/avox_poetry/venv_poetry.py iter_venvs --> yield from
 (
1015 /Users/ryan/.asdf/installs/python/3.10.0/lib/python3.10/pathlib.py iterdir --> for name in self._accessor.listdir(self):
FileNotFoundError: [Errno 2] No such file or directory: '/Users/ryan/.virtualenvs'
Exception raised in event handler; ignored.
$ xontrib list
abbrevs             installed      loaded
argcomplete         installed      loaded
autovox             installed      loaded
avox_poetry         installed      loaded
bashisms            installed      not-loaded
coreutils           installed      not-loaded
direnv              installed      loaded
distributed         installed      not-loaded
free_cwd            installed      not-loaded
history_encrypt     installed      not-loaded
homebrew            installed      loaded
jedi                installed      not-loaded
kitty               installed      loaded
mpl                 installed      not-loaded
mplhooks            installed      not-loaded
pdb                 installed      not-loaded
prompt_ret_code     installed      not-loaded
readable-traceback  installed      loaded
thefuck             installed      loaded
vox                 installed      not-loaded
voxapi              installed      loaded
whole_word_jumping  installed      loaded
xog                 installed      not-loaded

I create my virtualenvs in-project, using the virtualenvs.in-project option in my poetry config. Does this xontrib support that workflow?

Working now. Thanks so much for this! I can finally convert to xonsh fully I think :D


Active inline breakpoint isn't highlighted

re #81175

set breakpoints as show in this file https://github.com/microsoft/vscode-references-view/blob/a48315c11bfeec50fc0d018f555097f61c888ce4/src/extension.ts#L26
hide the references viewlet
the 'editorHighlight.hide()' inline breakpoint is active
:bug: the inline breakpoint doesn't get the highlight (yellow arrow enclosing dot), instead the gutter breakpoints is highlighted, step in, step out and the yellow arrow does appear inline.

Good catch. We currently do not support this.
And yes it makes perfect sense. So assigning first to September though I might move it out to next week.

We now support this as shown by this gif:
Verifier: make sure that when the inline breakpoint is at the same location as the top stack frame indicator that you get one decoration and not two. The one decroation should be the yellow arrow with a red dot inside (as shown in the .gif)

I found an issue with this change and opened a PR for it: https://github.com/microsoft/vscode/pull/83851


Bug: apiToken missing from a forked PR

same issue as the official plugin, it's missing apiToken because in forked repo it doesn't have access to the secret.
I was expecting this action to work because it mentioned ✅ GitHub Deployments on PRs from forks on readme.

relevant workfile

name: Publish www

on:
  push:
    branches:
      - dev
    paths:
      - 'apps/www/**'
  pull_request:
    branches:
      - dev
    paths:
      - 'apps/www/**'

jobs:
  publish:
    runs-on: ubuntu-latest
    permissions:
      contents: read
      deployments: write
    name: Publish to Cloudflare Pages

    steps:
      - name: Checkout
        uses: actions/checkout@v3

      # Run a build step here
      - name: Setup Node.js environment
        uses: actions/setup-node@v2
        with:
          node-version: 18

      - uses: pnpm/action-setup@v2
        name: Install pnpm
        with:
          version: 8
          run_install: false

      - name: Get pnpm store directory
        shell: bash
        run: |
          echo "STORE_PATH=$(pnpm store path --silent)" >> $GITHUB_ENV
      - uses: actions/cache@v3
        name: Setup pnpm cache
        with:
          path: ${{ env.STORE_PATH }}
          key: ${{ runner.os }}-pnpm-store-${{ hashFiles('**/pnpm-lock.yaml') }}
          restore-keys: |
            ${{ runner.os }}-pnpm-store-
      - name: Install dependencies
        run: pnpm i --frozen-lockfile

      - name: Build www
        run: pnpm build

      # Run a action to publish docs
      - name: Publish to Cloudflare Pages
        uses: AdrianGonz97/refined-cf-pages-action@v1
        with:
          apiToken: ${{ secrets.CLOUDFLARE_API_TOKEN }}
          accountId: ${{ secrets.CLOUDFLARE_ACCOUNT_ID }}
          projectName: shadcn-vue
          directory: .vitepress/dist
          # Optional: Enable this if you want to have GitHub Deployments triggered
          gitHubToken: ${{ secrets.GITHUB_TOKEN }}
          # Optional: Switch what branch you are publishing to.
          # By default this will be the branch which triggered this workflow
          # branch: main
          # Optional: Change the working directory
          workingDirectory: apps/www
          wranglerVersion: '3'

Hey @zernonia, thanks for raising an issue! Sorry for the delay, I hadn't realized that notifications weren't enabled on this repo. That should be fixed now.
The action does support GitHub Deployments from PR forks, however, it's not (yet) documented as to how to get that particular feature working.
Here's an example workflow of how we do it in Melt UI: https://github.com/melt-ui/melt-ui/blob/develop/.github/workflows/preview.yml
The reason as to why it's not documented yet is because it requires the use of the pull_request_target event in the action, which has security concerns that need to be considered before plopping it into a repo. I'm still trying to find the best way to document this clearly without encouraging people to unknowingly expose secrets during preview deployments.
However, if your site does not use/have any secrets during the build step or in your preview environment on CF, it should be alright to use then. If you're using it for shadcn-vue, which looks to be just a static site without needing any secrets, then it's probably a good fit.
Here's an example of this:
Preview Deployment with NO SECRETS

name: Preview Deployment
on:
  pull_request_target:
    paths:
      - "apps/www/**"

jobs:
  deploy-preview:
    runs-on: ubuntu-latest
    permissions:
      contents: read
      pull-requests: write
      deployments: write
    name: Deploy www Preview to Cloudflare Pages
    steps:
      - uses: actions/checkout@v3
        with:
          ref: ${{ github.event.pull_request.head.ref }}
          repository: ${{ github.event.pull_request.head.repo.full_name }}

      # Run a build step here
      - name: Setup Node.js environment
        uses: actions/setup-node@v2
        with:
          node-version: 18

      - uses: pnpm/action-setup@v2
        name: Install pnpm
        with:
          version: 8
          run_install: false

      # PNPM Store cache setup
      - name: Get pnpm store directory
        id: pnpm-cache
        run: |
          echo "pnpm_cache_dir=$(pnpm store path)" >> $GITHUB_OUTPUT
      - name: Setup pnpm cache
        uses: actions/cache@v3
        with:
          path: ${{ steps.pnpm-cache.outputs.pnpm_cache_dir }}
          key: ${{ runner.os }}-pnpm-store-${{ hashFiles('**/pnpm-lock.yaml') }}
          restore-keys: |
            ${{ runner.os }}-pnpm-store-

      - name: Install dependencies
        run: pnpm i --frozen-lockfile

      - name: Build site
        run: pnpm build

      - name: Deploy to Cloudflare Pages
        uses: AdrianGonz97/refined-cf-pages-action@v1
        with:
          apiToken: ${{ secrets.CF_API_TOKEN }}
          accountId: ${{ secrets.CF_ACCOUNT_ID }}
          projectName: shadcn-vue
          directory: .vitepress/dist
          githubToken: ${{ secrets.GITHUB_TOKEN }}
          workingDirectory: apps/www
          deploymentName: Preview
          wranglerVersion: "3"

In cases that it does use secrets, then the deployment job can be fitted with an environment field that requires manual approval before each deployment (like it does in the Melt UI workflow).
Manual approval needs to be setup at the repo level, as described in this PR under the "Make the Preview environment protected" step: https://github.com/melt-ui/melt-ui/pull/899
Preview Deployment WITH SECRETS

name: Preview Deployment
on:
  pull_request_target:
    paths:
      - "apps/www/**"

jobs:
  deploy-preview:
    environment: Preview # Requires manual approval before each deployment!
    runs-on: ubuntu-latest
    permissions:
      contents: read
      pull-requests: write
      deployments: write
    name: Deploy www Preview to Cloudflare Pages
    steps:
      - uses: actions/checkout@v3
        with:
          ref: ${{ github.event.pull_request.head.ref }}
          repository: ${{ github.event.pull_request.head.repo.full_name }}

      # Run a build step here
      - name: Setup Node.js environment
        uses: actions/setup-node@v2
        with:
          node-version: 18

      - uses: pnpm/action-setup@v2
        name: Install pnpm
        with:
          version: 8
          run_install: false

      # PNPM Store cache setup
      - name: Get pnpm store directory
        id: pnpm-cache
        run: |
          echo "pnpm_cache_dir=$(pnpm store path)" >> $GITHUB_OUTPUT
      - name: Setup pnpm cache
        uses: actions/cache@v3
        with:
          path: ${{ steps.pnpm-cache.outputs.pnpm_cache_dir }}
          key: ${{ runner.os }}-pnpm-store-${{ hashFiles('**/pnpm-lock.yaml') }}
          restore-keys: |
            ${{ runner.os }}-pnpm-store-

      - name: Install dependencies
        run: pnpm i --frozen-lockfile

      - name: Build site
        run: pnpm build
        env:
          SOME_SECRET: "foo" # Uses some secret during build!

      - name: Deploy to Cloudflare Pages
        uses: AdrianGonz97/refined-cf-pages-action@v1
        with:
          apiToken: ${{ secrets.CF_API_TOKEN }}
          accountId: ${{ secrets.CF_ACCOUNT_ID }}
          projectName: shadcn-vue
          directory: .vitepress/dist
          githubToken: ${{ secrets.GITHUB_TOKEN }}
          workingDirectory: apps/www
          deploymentName: Preview
          wranglerVersion: "3"

Hope this helps!


docs(formatting): added back-ticks for tables

added backtiks to cli table docs

cc @montogeek

Thanks!


Uncaught TypeError: this.$(...).editable is not a function

Hey guys, this seems to be the same problem mentioned here for Ember.
I had this problem using froala with rails_admin.

.editable was used in V1. Starting with V2, you should be using .froalaEditor


conflict: unable to remove repository reference on manual update from 1.7 to 1.8

Hello!
When updateing manually from lndg 1.7 to 1.8 i get the following error. this used to work without any problem with previous versions via terminal: "sudo ~/umbrel/scripts/app update lndg"
But now i get the folliwng error and cannot find any solution to get around it.. can somebody provide some help for me here?
Error response from daemon: conflict: unable to remove repository reference "ghcr.io/cryptosharks131/lndg:v1.7.0@sha256:xxx" (must force) - container 59ba6aded2c7 is using its referenced image 91d8e4fae533

It seems like the container using the LNDg image is still running. See if you can check on the container status and stop it if its still running, then try to update again.


-sish.http and -sish.https not used in terminal output

The following output doesn't seem to honour the -sish.https=:8443 and -sish.http=:8080 parameters.
HTTP: http://sish-host.com:80
HTTPS: https://sish-host.com:443

I was actually looking for a way to remove the port if it uses the default 80/443.

I'm gonna close this issue in favor of consolidating it for #47!


gitlab-runner: 11.4.0 -> 11.5.0

Motivation for this change
Update to latest version
Things done

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

[x] 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 nox --run "nox-review wip"
[x] 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)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Added the bump into the ongoing 11.5.0 PR for 18.09.


Fix mamba env create with nodefaults

get_index prepended channels that might not be in the channel_urls list and thus also not in the _channel_priority_map list. Not sure how to write a unit test for this.
This fixes the following error:
# >>>>>>>>>>>>>>>>>>>>>> ERROR REPORT <<<<<<<<<<<<<<<<<<<<<<

    Traceback (most recent call last):
      File "/opt/conda/lib/python3.7/site-packages/conda/exceptions.py", line 1079, in __call__
        return func(*args, **kwargs)
      File "/opt/conda/lib/python3.7/site-packages/conda_env/cli/main.py", line 80, in do_call
        exit_code = getattr(module, func_name)(args, parser)
      File "/opt/conda/lib/python3.7/site-packages/conda_env/cli/main_create.py", line 111, in execute
        result[installer_type] = installer.install(prefix, pkg_specs, args, env)
      File "/opt/conda/lib/python3.7/site-packages/mamba/mamba_env.py", line 48, in mamba_install
        priority = len(_channel_priority_map) - _channel_priority_map[chan.url(with_credentials=True)][1]
    KeyError: '***/other-channel/linux-64'

thanks :pray:


Contrib followup

WIP

can you check your formatting stp?

ok sweet !

very WIP

Alright it's finally ready for review. Sorry for all the push and the email you must have received. Configuring CI can be annoying since it's not reproducible locally easily.


Evictor fix for patching consumergroup when removing evicted pod placement

Signed-off-by: aavarghese<EMAIL_ADDRESS>Part of #1537
Fixes issue documented in https://github.com/knative-sandbox/eventing-kafka-broker/issues/1537#issuecomment-1041891419

Proposed Changes

Modify consumer group status placement to remove evicted pod via Patch rather than Update

/cc @pierDipi


link-to-changelog-file : add link to changelog on tag and release UI cards

Description
The link-to-changelog-file currently adds a link at the top of the pages which list all releases and all tags. It currently links to the changelog file on the main/master branch.
When one manages multiple release branches, one needs a link to the changelog file of each release.
I suggest you add a changelog link to each UI car for an individual release of a tag, at the top of the card next to the release or tag number.
This link would point to the changelog file in the branch of the release or tag.
Nice to have : display a disabled and crossed out changelog button if the branch has been deleted.
Example URLs
https://github.com/d3/d3/releases

Raga/releases are global on GitHub, so we follow that


Add workflowhub to cross .* workflow list page

And update the metadata file weekly.
Not sure if you have feelings about this @shiltemann, half a megabyte file, do we keep it as part of the build process, or bake it into the GTN. It takes about 1.25 minutes to collect the data. It would be fresher, but, it's already a slow deploy process.

Could also be good to ask workflowhub people for an "embed" view or whatever you want to call it, which could be non-paginated and without the surrounding header/sidebar stuff?

yeah that could be possible. Here we're mixing the results of .* + wfh together so folks can embed a listing of "all galaxy workflows", potentially filtered on a specific search term.


http-client-java, generate mock long for BigDecimal

mock test data in mgmt was not able to handle BigDecimal type.

No changes needing a change description found.

You can try these changes here

🛝 Playground
🌐 Website
📚 Next docs
🛝 VSCode Extension


Incompatablity with Greg's SGCraft's CC SG Interface

When I have OpenPeripherals installed I get the following crash whenever I attempt connect a modem to the CC SG Interface http://openeye.openmods.info/crashes/d4bd6c3dfff959ad7a0318a5c6df78d7

Ok, I see two problems here:

Duplicate of #206, #207, #217, OpenMods/OpenPeripheral-Addons#40, OpenMods/OpenPeripheral-Addons#42, OpenMods/OpenPeripheral-Integration#1 and few others - some mod (ProjectRed again?) bundles old API and it breaks Computercraft itself
We removed SGCraft itegration in 1.7, because they have added their own. But since they do few not-so-smart things (registering peripheral in postInit and making ALL tile entities IInventory) OpenPeripheral tries to supply own methods, overriding original implementation. I have limited ability to control that, but I tried to do what I can in this snapshot. You can also try to complain to SGCraft author(s) about that partially broken design.

The new snapshot fixes any problems that there were, thanks for your time and thorough answer.

I had the same issue and it was fixed with this snapshot.  It even connects on the sides of the computer now.  Thanks so very, very much.


No project template available

After I installed the plugin, I thought there is a project template, but no. I know I have to add the SDK, but it should be possible to create a project anyway.
Regards
Chris

resolved in #126


add nodejs to docker images

Enables controller tests in https://github.com/GoogleCloudPlatform/ruby-docs-samples/pull/449 and would eventually allow for https://github.com/JustinBeckwith/linkinator to be added.

Thanks! I can't merge due to CI failure

Merged and added you as an administrator for the repo.


command to pause / resume reminders

And optionally to select the days you want reminders on

And optionally to select the days you want reminders on

whats the use case for picking the days? Is the example 'peter wants to only update on fridays so he only has reminders for friday'

some companies only have 3 standups a week i think, and varying other examples
some personal updates just don't change 7 times a week, so they wanna do a couple big ones a week instead


Enable Multi-batch processing for GSTR reports

This is new feature for India GST for generation of GSTR reports

By mistake close this so re-opening it.


Add health-check-averager-halflife flag.

Closing in favor of https://github.com/ava-labs/avash/pull/141


Expo iOS Simulator Crashes Every Time with ActionSheetIOS

Description:
I'm experiencing a consistent issue where the Expo iOS simulator crashes every time I try to use ActionSheetIOS. This issue occurs even with the simplest usage of ActionSheetIOS, following the documentation provided at https://reactnative.dev/docs/actionsheetios. Additionally, it seems this issue is present not only in this specific scenario but everywhere in the documentation where the Expo iOS simulator is used.
React Native version: [e.g., 0.74.0]

Hi @Urmin-tagline
Thanks for reporting this issue.
It seems like this issue is actually unrelated to the React Native Website. Please open this issue against this other repository:
https://github.com/expo/expo/issues
https://github.com/react-native-community/cli/issues
https://github.com/software-mansion/react-native-reanimated/issues
I'll be closing this but feel free to reopen if the other repo points back to us as a issue in the core of React Native.


Default import not working

I am importing createAuthRefreshInterceptor into one of my files using:
import createAuthRefreshInterceptor from "axios-auth-refresh";
I am using this in a npm package and calling that npm package into my own app.
While running npm run dev(vite as my build tool) I am getting this error
The requested module '/node_modules/axios-auth-refresh/dist/index.min.js?v=576ed0f7' does not provide an export named 'default'
The versions I am using are:
"axios": "^1.6.0",
"axios-auth-refresh": "^3.3.6",
node version  : v20.13.1
Please help

Same


When viewing My Datasets from dashboard produces following error:

When navigating from dashboard to my datasets the following error was displayed:

The same error occurs when trying to view your profile

This issue has been fixed


feat: hub content config support env escape as file

What this PR does / why we need it:
content config support env escape as file
Specified Reivewers:
/assign @snakorse

/approve


Multiple instances of apps

hello. i have used this app. and it's brill 👍  any way. you could make it so we can install multiple sabnzbd
1 sabnzbdtv
2 sabnzbdmovies

Is this a different type of software or are you asking to be able to install multiple instances of the software?

Having multiple instances of the same app is probably outside the remit of this installer. If more people want it though, we could consider it.
I'll leave this open for a while in order to gauge interest, just leave a thumbs up if you'd like to see this!

Is this a different type of software or are you asking to be able to install multiple instances of the software?

to be able to install multiple instances of the software. so like i could have like 2 sonarr's and 2 radarr's. and more instances of downloaders e.g SABnzbd and NZBget. sorry i wasn't very clear the 1st time

Having multi installs of SABnzbd wouldnt be a great idea Sab is very recorse intensive so running 2 would cause a maasive drop in useability

Sorry to bump this, but is it possible to add a 2nd instance of Sonarr?
E.g. if you need a 2nd Sonarr instance for 4K content.
Pretty sure that's not possible with a single Sonarr instance, if you need two separate libraries.

AFAIK this cant be done you would need a second box or a second install of the ubuntu app running

It might actually be possible without having a 2nd Sonarr, now that I think about it.. Using quality profiles for the separate 4K library/folders

I dont think that works as you can only have 1 instance of one film or tv show as far as i am aware

Hmm.. Would it be possible if I install another OS on the appbox and run the 2nd Sonarr from there??

you can install another ubuntu and do it that way

If you have different libraries it should be possible, right?


Unable to load Google Doc or MS Office viewer on LAN IP address

When trying to open a file in ArozOS connecting using LAN IP address, the following error show up on MS Office Viewer

After switching to Google Viewer, the interface get stuck here and not doing any redirection

and the Network tab return no content for the Google Viewer

I guess it would be better to find a solution to this issue, either pop up a new interface to ask if the user want to access ArozOS with the DNS returned domain or simply pop up an error message when the target return no content / error.
What do you think?

Fixed in this two commit
dbcae0ad2a62c70482ecf484558ee31c95a88fd7
6846df1de9d73f7bb89e347ce053723d492d990b


False positive for js/react/unused-or-undefined-state-property

js/react/unused-or-undefined-state-property doesn't take into account the static method getDerivedStateFromProps.
See example: https://lgtm.com/projects/g/ant-design/ant-design/snapshot/395c5ddd8d3db88e9c34cf332ec77b3c34077cfe/files/components/tag/index.tsx#xf4fd4ef83f04a4fd:1
The state property mounted is read in getDerivedStateFromProps() on line 48.

We merged https://github.com/Semmle/ql/pull/607 last week, it adds support for: getSnapshotBeforeUpdate, shouldComponentUpdate and getDerivedStateFromProps. The fix should be online soon.


Missing Parameter type AWS::SSM::Parameter::Value

Hi! I've noticed that there is a missing parameter type applicable for an AWS type parameter: AWS::SSM::Parameter::Value<String> I've solved it locally by adding the next line
"AWS::SSM::Parameter::Value": r"^(/{0,1}(?!/))[A-Za-z0-9/-_]+(.([a-zA-Z]+))?$",
to the file cloud_radar/cf/unit/_template.py to validate_aws_parameter_constraints function.
Perhaps it could be added to your package! 😊

@dhutchison you might have better context

Thanks for raising this @nowiczka!
This is the missing piece of issue 202 that I've just not had a chance to get back to. From that comment I think we can take it a little bit further than just validating the parameter type, but also tie it in to the dynamic references functionality that we already have for providing values into the rendered template.
SSM parameters to CloudFormation templates isn't something I've used before. Would it be possible @nowiczka to provide some (appropriately redacted) examples of the parameters you supply when deploying the CloudFormation template and some sample values for the SSM parameters themselves? This will help me add in some scenarios into our unit testing of the library to ensure it functions as expected.

Sure, no problem. I've just prepared an example. Please, let me know if you need another one more elaborated.
In the next template, I have defined two SSM parameters, MyBucket and MyDatabase. Then, I am referencing them in the Location and DatabaseName  properties of the resource  MyTable:
AWSTemplateFormatVersion: '2010-09-09'
Transform: AWS::Serverless-2016-10-31
Description:  A simple example of a template with SSM parameters.
Parameters:
  MyBucket:
    Type: 'AWS::SSM::Parameter::Value<String>'
    Description: The bucket name where all the data will be put into.
    Default: /my_parameters/bucket/name

  MyDatabase:
    Type: 'AWS::SSM::Parameter::Value<String>'
    Description: The name of the database where the table will be created.
    Default: /my_parameters/database/name

Resources:
  MyTable:
    Type: 'AWS::Glue::Table'
    Properties:
      DatabaseName: !Ref MyDatabase  # Reference the MyDatabase SSM parameter
      TableInput:
        Name: my_table
        StorageDescriptor:
          Columns:
            - Name: column_1
              Type: string
          Location: !Sub 's3://${MyBucket}/test  # Reference the MyBucket SSM parameter

Remember to make sure that the specified SSM parameter paths exist in your AWS Systems Manager Parameter Store with appropriate values or references before deploying this CloudFormation template.
For example, the parameter /my_parameters/bucket/name can store  a string my_custum_bucket_name
The parameter  /my_parameters/database/name can store a string  my_database
Obviously, the bucket and the database should exist in the AWS environment before deploying the template.

Thanks - looks pretty similar to how {{resolve:ssm functions so shouldn't be too complicated to get working in the rendering process (famous last words). Might have a bit of time to take a look at this over the weekend, but we'll see.

This was included in the v0.10.0 release so this issue can now be closed.
@shadycuz I noticed the release notes were empty for that release but are there for the v0.11.0 release - do you know why that is?

This was included in the v0.10.0 release so this issue can now be closed.
@shadycuz I noticed the release notes were empty for that release but are there for the v0.11.0 release - do you know why that is?

@dhutchison it's a bug in the tool that creates the release notes. I have reported it in the past, but no one knows what causes it. I suspect it has something to do with merge conflicts or rebasing.


Add configuration parameters for health check

Added module parameters to configure health check's:

path
interval
timeout
thresholds

I signed it!

Hi @kartvep! Thank you for your interest in the project. I apologize for the extreme delay in responding. As this module requires more significant updates, I am closing this pull request in favour of #20 which we will be prioritizing immediately. This issue will address your request to configure more arguments of the health check.


Updated: dbeaver 6.1.5

Newer version of dbeaver (6.2.4) is now available in master. 4aba5bee7158cf0765d467aae3d2196e3ca721c2


Fix category title overflow

Closes #1166

This PR is practically done except for fixing the font-size of the category titles on larger screens. Right now, for some reason, they are 1.25rem when they should be 1.5rem.


[FRONTEND] Create Login and Registration page

Add a UI and create both Login and Registration pages at /login and /register

I want to work on this issue under JWOC

Go ahead

I want to work on this issue under JWOC

Can i work on it under JWoC

@whiteHatpro work with @DarkFalc0n and complete it together

What's the progress? @DarkFalc0n @whiteHatpro

@BRAVO68WEB i have done with my part

Ok then make a PR

can i work on this issue under JWoc

Ok then make a PR

@BRAVO68WEB falcon has still has some work to do on his end

Ok then, both of you finish your work and then do PR

@whiteHatpro Where is your PR?

@BRAVO68WEB i have created the pr

@whiteHatpro any progress?

@whiteHatpro any progress?

@BRAVO68WEB sorry, but i am not able to link the pages

If u want to assign someone else please go ahead.

Ok fine


Third Party Button Component

Done

Added svg files that are needed for the third party buttons and other components
Third Party Button Component is done along with the test file
Passes in props for the image source, the onChange function, width and height of the image as props to the component.

To Do

Write functional test code for this component.

@sivaShilpa is this PR ready for review again?


Regex(Set)Builder: option to create regexes that match the whole string?

I'm aware this is something that can be easily worked around; I'm just creating this issue wondering whether a feature in question, if implemented, would even be useful performance-wise. Say, could something like
RegexSetBuilder::new(regexes)
    .whole_string(true)
    .build()?
    .is_match("whatever")

potentially be faster than
RegexSet::new(
    regexes.map(|r| format!("^{}$", r)
)?.is_match("whatever")

?

No, I don't think so. The cost of the latter over the former is merely a single allocation. Compared to what the regex compiler is doing behind the scenes, it's peanuts.


v4.3.0

/cc @sipayRT

Changes Unknown when pulling f49c558eac307b6aa89cb8f06481ab6e600f00bc on version/4.3.0 into ** on master**.

Coverage remained the same at 80.606% when pulling f49c558eac307b6aa89cb8f06481ab6e600f00bc on version/4.3.0 into fcbce024b478244745a563e9e56147fc523d06c9 on master.

окэй


Prepare 0.10.1 release

Pull request

Description

Related
Checklist

[x] The RFC, if required, has been submitted and approved
[x] Any user-facing impact of the changes is reflected in docs.tremor.rs
[x] The code is tested
[x] Use of unsafe code is reasoned about in a comment
[x] Update CHANGELOG.md appropriately, recording any changes, bug fixes or other observable changes in behavior

Coverage remained the same at 81.407% when pulling 8e6e37a729266d0411e6dd5b63a80da945aab2f8 on prepare-release-0.10.1 into e60ba65b9d115c2e3ceacb4a34c0de83a4afa6ad on main.


Add glogg version 1.1.4 (closes #1341)

Add glogg v1.1.4. It has been 64-bit only since version 1.0.3, so I ignored x86. It uses NSIS to install by default, but seems to merely require administrative permission to write to Program Files. I have opted to extract the contents instead since it seems quite happy running as a portable app.

Nice 👍
Please use #/dl.7z as the URL fragment just to be consistent.


Tag win_nodes roles with master

kube-proxy setup in the kubernetes master role will remove any node selectors applied to the kube-proxy daemonset.   For hybrid clusters, running just the master role may result in kube-proxy pods being scheduled on windows node.  Therefore, the linux node selector patch for kube-proxy should be reapplied anytime the master role runs.

/assign @smana

are you successfully deploying hybrid cluster linux/windows clusters ?

ci check this

are you successfully deploying hybrid cluster linux/windows clusters ?

Yup.  Kubespray for master / linux nodes, and put together ansible script based on kubernetes-for-windows for windows nodes.  Also, revamped the Kubespray testcases to work on hybrid clusters.

/approve
/lgtm


-c M1 problem

When I add -c M1 it gives me the error that says error : argument Not a valid Time frame : 1M
How should I add this to get 1 minute chart instead of tick data?

Late answer but might help someone:
instead of typing the value the way the help function says, type it in this format: {letter}{number} e.g. M15 or H4

@MShoaei Thanks very much!

It's a very strange issue. I installed duka with pip3 on two PCs, one with Ubuntu, the other with Manjaro. In both cases the help message says -c arguments are of {number}{letter} format. I checked main.py (where help message is supposed to be coming from) and /core/utils.py files and they both use {letter}{number} format. So where is this help message even coming from? I've never been so confused.

I've moved to the alternative tool:
https://github.com/Leo4815162342/dukascopy-tools


Cannot compile when minified due to +++ construct

Bug Report
Description of the problem
When minified (using Doug Crockford's jsmin), the current trunk (as of this writing) cannot compile due to...
Code sample
{value:{i:'O'+++id,w:{}}});};

Leads to:
Uncaught ReferenceError: Invalid left-hand side expression in postfix operation

Changing that ++id to (++id) fixes the problem.
Your environment

pdfkit version: trunk, as of this writing (edit: [d91b890494d5])
Node version: n/a
Browser version (if applicable): Chrome 74
Operating System: Linux Mint

jsmin does not supports es6. Use terser or new uglify

FWIW, making the necessary 2-byte change so that people who aren't using node can minify it sounds like the better approach.


Swamped with Gas Giants

Playing on a server running ATM 2.53 which is running AR 1.0.13 and... everything but Earth is a gas giant. L:una is a gas giant. The server has never been updated or anything so everything has been the same version. Singleplayer works fine.
Also on this topic, what would be the best thing to do to ungasify myself? I can't seem to get the planet generate command to work. Is there something I can delete that'll regenerate stuff?

Sounds like #436 there was an issue with the backward compat code in AR that made planets with dimids larger that 256 become gas giants.  I've fixed it for 1.0.16.  But you can reset AR's planets by deleting the /advRocketry/temp.dat file
If you want to use the latest version now, I do have a build up here http://<IP_ADDRESS>:8080/
I do plan on releasing a build either later today or tomorrow on Curse

That seems to have done it. Can't update without driving the server crazy but I changed the dimensions starting point (hopefully without causing other issues >_>) and reset temp.dat and it's working. Thank you!


Merge Feature/flint UI into Main

Description
[Describe what this change achieves]
Issues Resolved
[List any issues this PR will resolve]
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

[ ] 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.

Looks good. Minor: It would be better if we have couple of folders inside data_connections/components -> configuration_details and connection_details.

Yes good idea - I am addressing in this follow up PR: https://github.com/opensearch-project/dashboards-observability/pull/1035, it was getting quite confusing so folders should help


feat: add generating samples for any kind of object that supports openAPIV3schema

Closes https://github.com/Skarlso/crd-to-sample-yaml/issues/117

Closes https://github.com/Skarlso/crd-to-sample-yaml/issues/122 too. :)


Skip hfft related tests in HIP

These tests are flaky, and fail if other tests are run in conjunction with them. Standalone runs never fail but
for example
pytest tests/cupy_tests/sorting_tests/test_search.py tests/cupyx_tests/scipy_tests/fft_tests/test_fft.py -k 'test_cub_argmin or test_ihfft' causes the tests to fail. (Even with CUB disabled ...)
Currently I am not sure of where is the issue that causes this and if its solvable, but for the meantime I think it is better to just skip the tests to get a working CI.

/test jenkins

/test jenkins

/test jenkins

/test mini


docs: update OTW upgrade guide

The information in the OTW upgrade guide was out of date

@zwave-js-bot automerge


[SMALLFIX]Supply the variable name to Preconditions.checkNotNull

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/16189/
Test PASSed.

Merged build finished. Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/16190/
Test PASSed.


added datasource to events info table

Added database row on events info table.

Cherry-picked in https://github.com/hasura/graphql-engine/pull/5654/commits/56ac4e41ff6a85f2b253f8a6cd0c4c6b6a50386a


Review Assess functions and identify those that contain a lot of overlapping code.

@Zhongkai-Wang and @mattroumaya to review Assess functions and identify those that contain a lot of overlapping code.
Goal would be to use Assess_Binary() within several KRIs

e.g., within PD_Assess() and PFS_Assess(), might both call Assess_Binary().
Target v1.9.2 or v1.10.0

Originally posted by @mattroumaya in https://github.com/Gilead-BioStats/gsm/discussions/1480

@mattroumaya @Zhongkai-Wang Any progress on this? I'd like to implement in the next major release.


API for detecting Insiders version

I've made an extension for installing .vsix files with a right click, under the hood it calls code --install-extension, therefor it doesn't work if triggered from Code Insiders, where it should instead call code-insiders --install-extension.
In order for this to work properly I need an API for detecting if the extension is running inside Code or Code Insiders.

You should use env.appName for this. That's the only hint we  have (externally/interally) because insiders is just another distribution.

Thanks for the suggestion.


topic 53 - static RPC version

RPC version is set statically
Mismatch onHello throws error
Closes #53

@ChristopheCVB I've updated the spec/behavior.
Atm I cannot really test what message is actually being sent (that'll be addressed in #94 ), so what I test here is that if our client supports a higher version than what the server can support then it'll error out. Otherwise, the server response/support is caught in the E2E tests.


Why doesn't anyone explain how to add a context? It's not friendly to beginners.

Why doesn't anyone explain how to add a context? It's not friendly to beginners.

See Kubernetes docs for that.
Typically your cluster creation method creates contexts.


22 change the models endpoint to datasets

Addresses #22

Well, I said I would merge it but I guess I forgot to :grin:
Merged in now!


Add 120% AMI Levels

We have a request to add 120% AMI units to the Doorway Housing Portal. They can be found in this spreadsheet and are highlighted in yellow (so not new charts, just new additions to existing charts). The update for Contra Costa County is a time sensitive request for a listing, so is medium high priority. All others are low priority.
QA must be done in staging: https://partners.staging.housingbayarea.mtc.ca.gov/

Sign in
Go to a listing
Add a unit and explore the AMI options to verify that the information highlighted in the sheet above in accurately reflected in staging.

These are in staging environment: https://partners.staging.housingbayarea.mtc.ca.gov/

lgtnm


Workbench - Keycloak SSO integration.

Please merge if possible, @krisv is updated with this.

+1