bug(?): tab completing [ causes an error to be logged

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
Given the full and only command line:
[
Pressing tab at the end of the buffer (right after the bracket) causes an error to be logged to $Error.
Expected behavior
No error to be logged to `$Error`, even if no completion is produced.

Actual behavior
An error is logged to `$Error` right after pressing tab.

Error details
Type           : System.Management.Automation.CmdletInvocationException
ErrorRecord    :
    Exception             :
        Type        : System.Management.Automation.WildcardPatternException
        ErrorRecord :
            Exception             :
                Type    : System.Management.Automation.ParentContainsErrorRecordException
                Message : The specified wildcard character pattern is not valid: \[*
                HResult : -2146233087
            CategoryInfo          : NotSpecified: (:) [], ParentContainsErrorRecordException
            FullyQualifiedErrorId : RuntimeException
        TargetSite  :
            Name          : Parse
            DeclaringType : System.Management.Automation.WildcardPatternParser, System.Management.Automation, Version=<IP_ADDRESS>0, Culture=neutral, PublicKeyToken=31bf3856ad364e35
            MemberType    : Method
            Module        : System.Management.Automation.dll
        Message     : The specified wildcard character pattern is not valid: \[*
        Source      : System.Management.Automation
        HResult     : -2146233087
        StackTrace  :
   at System.Management.Automation.WildcardPatternParser.Parse(WildcardPattern pattern, WildcardPatternParser parser)
   at System.Management.Automation.WildcardPatternMatcher.MyWildcardPatternParser.Parse(WildcardPattern pattern, CharacterNormalizer characterNormalizer)
   at System.Management.Automation.WildcardPatternMatcher..ctor(WildcardPattern wildcardPattern)
   at System.Management.Automation.WildcardPattern.Init()
   at System.Management.Automation.LocationGlobber.IsChildNameAMatch(PSObject childObject, WildcardPattern stringMatcher, Collection`1 includeMatcher, Collection`1 excludeMatcher, String& childName)
   at System.Management.Automation.LocationGlobber.GenerateNewPSPathsWithGlobLeaf(List`1 currentDirs, PSDriveInfo drive, String leafElement, Boolean isLastLeaf, ContainerCmdletProvider provider, CmdletProviderContext context)
   at System.Management.Automation.LocationGlobber.ExpandMshGlobPath(String path, Boolean allowNonexistingPaths, PSDriveInfo drive, ContainerCmdletProvider provider, CmdletProviderContext context)
   at System.Management.Automation.LocationGlobber.ResolveDriveQualifiedPath(String path, CmdletProviderContext context, Boolean allowNonexistingPaths, CmdletProvider& providerInstance)
   at System.Management.Automation.LocationGlobber.GetGlobbedMonadPathsFromMonadPath(String path, Boolean allowNonexistingPaths, CmdletProviderContext context, CmdletProvider& providerInstance)
   at System.Management.Automation.PathIntrinsics.GetResolvedPSPathFromPSPath(String path, CmdletProviderContext context)
   at Microsoft.PowerShell.Commands.ResolvePathCommand.ProcessRecord()
   at System.Management.Automation.Cmdlet.DoProcessRecord()
   at System.Management.Automation.CommandProcessor.ProcessRecord()
    CategoryInfo          : NotSpecified: (:) [Resolve-Path], WildcardPatternException
    FullyQualifiedErrorId : RuntimeException,Microsoft.PowerShell.Commands.ResolvePathCommand
    InvocationInfo        :
        MyCommand       : Resolve-Path
        HistoryId       : 2
        InvocationName  : Microsoft.PowerShell.Management\Resolve-Path
        CommandOrigin   : Internal
    ScriptStackTrace      : at <ScriptBlock><End>, <No file>: line 38
    PipelineIterationInfo :

TargetSite     :
    Name          : Invoke
    DeclaringType : System.Management.Automation.Runspaces.PipelineBase, System.Management.Automation, Version=<IP_ADDRESS>0, Culture=neutral, PublicKeyToken=31bf3856ad364e35
    MemberType    : Method
    Module        : System.Management.Automation.dll
Message        : The specified wildcard character pattern is not valid: \[*
InnerException :
    Type        : System.Management.Automation.WildcardPatternException
    ErrorRecord :
        Exception             :
            Type    : System.Management.Automation.ParentContainsErrorRecordException
            Message : The specified wildcard character pattern is not valid: \[*
            HResult : -2146233087
        CategoryInfo          : NotSpecified: (:) [], ParentContainsErrorRecordException
        FullyQualifiedErrorId : RuntimeException
    TargetSite  :
        Name          : Parse
        DeclaringType : System.Management.Automation.WildcardPatternParser, System.Management.Automation, Version=<IP_ADDRESS>0, Culture=neutral, PublicKeyToken=31bf3856ad364e35
        MemberType    : Method
        Module        : System.Management.Automation.dll
    Message     : The specified wildcard character pattern is not valid: \[*
    Source      : System.Management.Automation
    HResult     : -2146233087
    StackTrace  :
   at System.Management.Automation.WildcardPatternParser.Parse(WildcardPattern pattern, WildcardPatternParser parser)
   at System.Management.Automation.WildcardPatternMatcher.MyWildcardPatternParser.Parse(WildcardPattern pattern, CharacterNormalizer characterNormalizer)
   at System.Management.Automation.WildcardPatternMatcher..ctor(WildcardPattern wildcardPattern)
   at System.Management.Automation.WildcardPattern.Init()
   at System.Management.Automation.LocationGlobber.IsChildNameAMatch(PSObject childObject, WildcardPattern stringMatcher, Collection`1 includeMatcher, Collection`1 excludeMatcher, String& childName)
   at System.Management.Automation.LocationGlobber.GenerateNewPSPathsWithGlobLeaf(List`1 currentDirs, PSDriveInfo drive, String leafElement, Boolean isLastLeaf, ContainerCmdletProvider provider, CmdletProviderContext context)
   at System.Management.Automation.LocationGlobber.ExpandMshGlobPath(String path, Boolean allowNonexistingPaths, PSDriveInfo drive, ContainerCmdletProvider provider, CmdletProviderContext context)
   at System.Management.Automation.LocationGlobber.ResolveDriveQualifiedPath(String path, CmdletProviderContext context, Boolean allowNonexistingPaths, CmdletProvider& providerInstance)
   at System.Management.Automation.LocationGlobber.GetGlobbedMonadPathsFromMonadPath(String path, Boolean allowNonexistingPaths, CmdletProviderContext context, CmdletProvider& providerInstance)
   at System.Management.Automation.PathIntrinsics.GetResolvedPSPathFromPSPath(String path, CmdletProviderContext context)
   at Microsoft.PowerShell.Commands.ResolvePathCommand.ProcessRecord()
   at System.Management.Automation.Cmdlet.DoProcessRecord()
   at System.Management.Automation.CommandProcessor.ProcessRecord()
Source         : System.Management.Automation
HResult        : -2146233087
StackTrace     :
   at System.Management.Automation.Runspaces.PipelineBase.Invoke(IEnumerable input)
   at System.Management.Automation.PowerShell.Worker.ConstructPipelineAndDoWork(Runspace rs, Boolean performSyncInvoke)
   at System.Management.Automation.PowerShell.CoreInvokeHelper[TInput,TOutput](PSDataCollection`1 input, PSDataCollection`1 output, PSInvocationSettings settings)
   at System.Management.Automation.PowerShell.CoreInvoke[TInput,TOutput](PSDataCollection`1 input, PSDataCollection`1 output, PSInvocationSettings settings)
   at System.Management.Automation.PowerShell.CoreInvoke[TOutput](IEnumerable input, PSDataCollection`1 output, PSInvocationSettings settings)
   at System.Management.Automation.PowerShell.Invoke(IEnumerable input, PSInvocationSettings settings)
   at System.Management.Automation.PowerShellExecutionHelper.ExecuteCurrentPowerShell(Exception& exceptionThrown, IEnumerable input)

### Environment data

```powershell
Name                           Value
----                           -----
PSVersion                      7.2.4
PSEdition                      Core
GitCommitId                    7.2.4
OS                             Microsoft Windows 10.0.19044
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

### Visuals

_No response_

No repro in the latest versions.

@insomnimus - Thank you for reporting this issue. The Working Group has been able to reproduce this issue. We will look into this bug.

I could reproduce this in the latest LTS (7.2.13) & Stable (7.3.6) and these returned the same error as provided by the user. Attempts in v7.4.0-preview4 and the latest daily, as @sdwheeler mentioned for me did not return this error so I think that this was recently fixed and therefore needs to be looked into which PR fixed this & whether this can be backported at all to both LTS and Stable versions.
I also then started another instance of each with pwsh -noprofile -noninteractive in the above to rule out it just being within PSReadline & got the same error in LTS & Stable and no error in either 7.4.0-preview4 or the latest daily.
All tested on  Microsoft Windows 10.0.25905

@kilasuit It was probably fixed by this PR: https://github.com/PowerShell/PowerShell/pull/19489 which was pretty much a complete rewrite of the file completion logic. Backporting a huge change like that to LTS seems unrealistic, even if it is a very nice change (I would love if all my changes got backported to 5.1 as well 😁)

Seeing as 7.4 will be the next LTS - I don't think it's too unrealistic to say that if that was the fix @MartinGC94 that it's not planned for backporting - however that's up to the team to decide.


Bringing back case classes...

Introduction
Some time ago in the ancient past, I banished case classes from the Indigo code base. I cannot now remember why I did this, other than a vague idea that I thought I was helping performance. I'm confident that I had no concrete evidence to base that decision on.
I've been wondering about the (lack of?) wisdom of that decision ever since, and after recent discussions with @drsimonmiles I have decided to get to the bottom of the matter in a slightly but hardly conclusively scientific way.
Hypothesis

In the worst case, case classes will be no less performant than classes. The additional methods case classes generate, and the persisted fields they have will have barely any impact on a fully optimised Indigo game.

In the best case, the implementation of case classes and their use of shallow cloning may have a positive impact on memory usage and garbage collection.

Method
We will test three versions of Indigo against the Cursed Pirate Demo.
The cursed pirate demo is the only Indigo game that does any real work every frame without requiring user input, which could affect the results because I'd need to do the same kinds of actions over and over again fairly consistently.
The three version we will test are:

The current public release 0.3.0
The "master" branch, which has no case classes on hot code paths.
The "case-classes" branch, where most of the public API has been converted to case classes - as you can see for yourself in this Pull Request.

The public release is our baseline, but it's important to test the master branch since some changes have already been made there, and they may affect performance.
We will test each version three times, by:

Rebuilding the demo with sbt clean compile fullOptJS indigoBuildFull
Opening a Firefox window and its performance window.
Running the game for 30 seconds to let the JS engine warm up.
Taking a memory snapshot.
Recording the performance for 30 seconds, and then stopping.
Where we need to look at time series values, we will only sample from the second 10 seconds in recorded 30 second window, to reduce the effects of starting and stopping the profiler. I also picked frames at random from periods where the game seemed to be running nominally (i.e. not right next to a big GC pause...)

This was difficult to measure, particularly since the profiler has an observable effect on performance, but I looked at:

File size
Memory footprint
Average frame duration
Observed GC pattern

Results
I ran each three times as described above, but actually found them to be reassuringly consistent between builds/runs. The values below are averages, and all taken from the same computer.
All of the big GC blocks followed a near identical pattern at regular intervals. Mostly NonIncremental GC, some CC graph reduction, and one-off Minor GC's and Cycle collections. The only thing that varied was the interval.
Published version
Least stable frame rate, periodically bouncing between 59/60 FPS. Others were stable at 60.

File size: 834KB
Memory: 93.88MB
Average frame duration: 8ms
GC interval: 5 seconds

Master
Smallest file size, slightly better performance than master.

File size: 820KB
Memory: 93.36MB
Average frame duration: 7ms
GC interval: 5 seconds

Case classes
Weirdly same file size as the published version. My hunch is that this has nothing to do with the code (which I checked was definitely the right version) and more to do with the closure compiler hitting some sort of complexity cut off. But I'm just guessing.

File size: 834KB
Memory: 93.46MB
Average frame duration: 7ms
GC interval: 7 seconds

Conclusion
There's very little in it.
There appears to be no apparent performance difference between the current "master" and "case class" branches, and improvements over the published version seem due to commits that live on both branches.
The only value that has moved an any interesting way is the GC interval, where GC pauses are consistently further apart in the new version. This could actually be a bad thing, since in games the ideal would be to have a very regular, frequent, predictable, and small GC penalty. However it's hardly conclusive because it could have been doing the amount of clean up but simply not needing to do it as often. Certainly there was no observable frame duration penalty to suggest it was struggling.
What does this all mean?
Since there is no obvious performance reason not to use case classes, even if there's no performance improvement; Moving to case classes is still worth doing since it's cleans up the code base and presents more usable objects to the end user.
So if we're happy with the PR, we should go ahead and merge it.

There are some other changes I made along the way here, fixing / making terminology consistent, syntax corrections, adding missing methods (again for consistency), renaming a few ugly things, mostly around Flip.


Will not install

I cannot get the app to install on my system. I run the command and it returns the following error:
C:\WINDOWS\system32>dotnet tool install --global haveibeenpwned-downloader
C:\Users\someuser\AppData\Local\Temp\jpq2nno2.b2t\restore.csproj : error NU1100: Unable to resolve 'haveibeenpwned-down
loader (>= 0.0.0)' for 'net6.0'.
C:\Users\someuser\AppData\Local\Temp\jpq2nno2.b2t\restore.csproj : error NU1100: Unable to resolve 'haveibeenpwned-down
loader (>= 0.0.0)' for 'net6.0/any'.
The tool package could not be restored.
Tool 'haveibeenpwned-downloader' failed to install. This failure may have been caused by:

You are attempting to install a preview release and did not use the --version option to specify the version.
A package by this name was found, but it was not a .NET tool.
The required NuGet feed cannot be accessed, perhaps because of an Internet connection problem.
You mistyped the name of the tool.

For more reasons, including package naming enforcement, visit https://aka.ms/failure-installing-tool
Any idea what the issue with the install command is?

This command was required to be run prior to running the install command
dotnet nuget add source https://api.nuget.org/v3/index.json -n nuget.org
Then the install will run correctly. Could be helpful for others in the readme file.
Thanks!

Normally, adding the standard NuGet source should not be required. Do you have an idea what's causing dotnet to not know about it? @JVorbeck @Barbarrosa

@stebet Not sure about the case I found, but I looked it up & found people talking about several causes e.g. certain VSCode updates or package manager installation scripts.


Revert "Move orchestrator.def to configdefinititions/"

Reverts vespa-engine/vespa#3354
Broke config server, need to look into it

👍


Create JupyterNotebook.gitignore

Reasons for making this change:
Making gitignore support for jupyter notebooks' projects i.e. not adding checkpoints and its directory to git
Links to documentation supporting these rule changes:
http://gitignore.io/api/jupyternotebooks

@gvvynplaine apologies for the delay in replying to this.
We have an existing template here if you'd like to make any changes here: https://github.com/github/gitignore/blob/master/community/Python/JupyterNotebooks.gitignore

@shiftkey Thank you for your answer.


Multiple addDocuments triggered on non-existent index fails

Describe the bug
⚠ this might be solved with the v0.25.0 but just in case it is still a bug
When running addDocuments almost simultaneously on an index that does not exists, all addDocuments except the first one throw an index already exists error.
I think this is why:

To Reproduce
Steps to reproduce the behavior:

Make multiple async calls that you do now await  to POST /documents
Errors are thown

Expected behavior
Well no throws
MeiliSearch version: 0.24.0

Wonderful schema!
@bidoubiwa Can you check it's fixed with v0.25.0?

It's fixes with v0.25.0 !


.NET Core tools telemetry must be opt in

We must avoid unwanted data sharing giving users the ability to consciously agree to collect and share usage data.

There's already a discussion about this here: https://github.com/dotnet/cli/issues/3093.

@svick Should I close this one?

@gguimaraesbr Yeah, I think there is no reason to have two separate issues.

Closing because there is another issue regarding this theme: https://github.com/dotnet/cli/issues/3093


Add NPM link to Readme.md

Added link to NPM package page(https://www.npmjs.com/package/analytics.js).

Instead of the npm link, I'd add "Installation" section like:
### Installation

$ npm install analytics.js

Yeah I'd prefer this to be in an installation section.


Alert counts seem out

Go to /environments and /alerts/count API URLs in your browser and paste the output below, please.

Sorry can't seem to replicate now as mongo housekeeping has happened but I feel it could be due to expired alerts perhaps being excluded.  Was only just aware this morning when I raised this that housekeeping was needed and implemented sadly losing the state reported earlier

No problem. That's given me enough of a clue as to where to look. Thanks.

No problem. Thanks too this is sorting out our alerts view which was mostly in email until this this month :)

I can't reproduce this so can't confirm that it's a problem. If it happens again grab the output from /environments and /alerts/count and we can reopen this issue. Thanks.


Getting the message "Error the build was canceled" immediately after setup

Describe the bug
I get multiple errors on build - right after running npm create tutorial@latest both on windows and wsl
noam@Win11:~/try/t01/online-remult-tutorial$ npm run build

><EMAIL_ADDRESS>build
> astro check && astro build

✘ [ERROR] The build was canceled

08:03:39 [types] Generated 1.41s
08:03:39 [check] Getting diagnostics for Astro files in /home/noam/try/t01/online-remult-tutorial...
Result (8 files):
- 0 errors
- 0 warnings
- 0 hints

✘ [ERROR] The build was canceled

08:03:45 [types] Generated 397ms
08:03:45 [build] output: "static"
08:03:45 [build] directory: /home/noam/try/t01/online-remult-tutorial/dist/
08:03:45 [build] Collecting build info...
08:03:45 [build] ✓ Completed in 1.45s.
08:03:45 [build] Building static entrypoints...
 Some chunks are larger than 500 kB after minification. Consider:
- Using dynamic import() to code-split the application
- Use build.rollupOptions.output.manualChunks to improve chunking: https://rollupjs.org/configuration-options/#output-manualchunks
- Adjust chunk size limit for this warning via build.chunkSizeWarningLimit.
08:03:59 [vite] ✓ built in 4.99s

 generating static routes
08:03:59 ▶ @tutorialkit/astro/dist/default/pages/index.astro
08:03:59   └─ /index.html (+14ms)
08:04:00 ▶ @tutorialkit/astro/dist/default/pages/[...slug].astro
08:04:00   └─ /1-basics/1-introduction/1-welcome/index.htmlTrying to compute an absolute file URL but Astro.site is not set.
 (+27ms)
08:04:00 ✓ Completed in 314ms.

08:04:00 [@tutorialkit/astro] 1-basics-1-introduction-1-welcome-solution.json
08:04:00 [@tutorialkit/astro] 1-basics-1-introduction-1-welcome-files.json
08:04:00 [@tutorialkit/astro] template-default.json
08:04:00 [build] 2 page(s) built in 16.48s
08:04:00 [build] Complete!
noam@Win11:~/try/t01/online-remult-tutorial$

Link to a StackBlitz project which shows the error
No response
Steps to reproduce

Run 'npm create tutorialkit@latest`
Run npm run build
Get these errors

Expected behavior
Expected to look like there are no errors out of the box :)
Screenshots
No response
Platform

TutorialKit version: [e.g. 1.0.1]
OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: [e.g. 91.1]

Additional context
The backstory is that I'm trying to upgrade a project created with 0.0.2 to latest and greatest and it doesn't build - and I'm trying to figure out what needs to be done, by comparing it to a project just after install.

I think errors have always been there. @Nemikolh any ideas where they come from?

and it doesn't build

It does build successfully. In the end of your log output there is:

08:04:00 [build] 2 page(s) built in 16.48s
08:04:00 [build] Complete!

@AriPerkkio I see that it eventually produces a result - just thinking of it as a newbie - red error notices are scary :)

just thinking of it as a newbie - red error notices are scary

I agree! We should get rid of these errors. The build command seems to run astro check && astro build. If you remove astro check, do the errors still appear?

@AriPerkkio not completely
C:\try\t12\online-remult-tutorial>npm run build

><EMAIL_ADDRESS>build
> astro build

09:33:27 [vite] Re-optimizing dependencies because vite config has changed
X [ERROR] The build was canceled

09:33:28 [types] Generated 1.36s
09:33:28 [build] output: "static"
09:33:28 [build] directory: C:\try\t12\online-remult-tutorial\dist\
09:33:28 [build] Collecting build info...
09:33:28 [build] ✓ Completed in 2.80s.
09:33:28 [build] Building static entrypoints...
 Some chunks are larger than 500 kB after minification. Consider:
- Using dynamic import() to code-split the application
- Use build.rollupOptions.output.manualChunks to improve chunking: https://rollupjs.org/configuration-options/#output-manualchunks
- Adjust chunk size limit for this warning via build.chunkSizeWarningLimit.
09:33:44 [vite] ✓ built in 5.12s

 generating static routes
09:33:44 ▶ @tutorialkit/astro/dist/default/pages/index.astro
09:33:44   └─ /index.html (+20ms)
09:33:45 ▶ @tutorialkit/astro/dist/default/pages/[...slug].astro
09:33:45   └─ /1-basics/1-introduction/1-welcome/index.htmlTrying to compute an absolute file URL but Astro.site is not set.
 (+31ms)
09:33:45 ✓ Completed in 510ms.

09:33:45 [@tutorialkit/astro] 1-basics-1-introduction-1-welcome-files.json
09:33:45 [@tutorialkit/astro] 1-basics-1-introduction-1-welcome-solution.json
09:33:45 [@tutorialkit/astro] template-default.json
09:33:45 [build] 2 page(s) built in 19.57s
09:33:45 [build] Complete!

C:\try\t12\online-remult-tutorial>

and it looks like this:

Draws attention

There's a lot of similar reports on Astro about this too. We should go through their reports and look for solutions there 🤔

https://github.com/withastro/astro/issues/11390
https://github.com/withastro/astro/issues/6631


Update job input types handling based on its type

Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
Prevent path validation for input types: INTEGER, STRING, NUMBER AND boolean.
Because they don't have "path" property and it was incorrect to try to validate the path
All SDK Contribution checklist:

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

General Guidelines and Best Practices

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

Testing Guidelines

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

API change check
API changes are not detected in this pull request.


Getting undefined method component error

Hey guys, not sure if this is the best place to ask, but here it goes:
I keep on getting this error on my local:

I can load http://localhost:3000/lookbook just fine, but when I want to preview one of the components, it throws that error. I can also view the component just fine from http://localhost:3000/lookbook/example/with_default_title (one of them)
Here are my code:
# app/components/example_component.rb

class ExampleComponent < ViewComponent::Base
  def initialize(title:)
    @title = title
  end
end

<span title="<%= @title %>"><%= content %>
<%= @title %>

# test/components/previews/example_component_preview.rb
class ExampleComponentPreview < ViewComponent::Preview
  # 🎉 Button playground! 🎉
  # -----------------------
  # You can use the controls in the 'Params' section
  # to set button property values on the fly.
  #
  # @param title text
  def with_default_title(title: "hello")
    render_with_template(locals: {
      title: title
    })
  end

  def with_long_title
    render(ExampleComponent.new(title: "This is a really long title to see how the component renders this"))
  end

  def with_content_block
  end
end

<%=
  render(ExampleComponent.new(title: "This component accepts a block of content")) do
    tag.div do
      content_tag(:span, "Hello")
    end
  end
%>

<%= render(ExampleComponent.new(title: title)) %>

My current environment:

Rails v5.2.5
view_component v2.51.0
lookbook v0.7.1

Let me know if you need more information. Thanks in advance.

Hey @amree - this is definitely the best place to ask :-) Sorry you are having issues and thanks for the detailed report -  I can't see anything obviously wrong with any of those files so it may well be a bug in the latest release.
If possible could you try switching to v 0.6.1 and letting me know if you see the same issue? And I'll try and dig a bit deeper into this as soon as I can.

Hey again @amree,
So I've tried recreating your issue and I can't seem to do it. I've created a Rails v5.2.5 test app (with Lookbook v0.7.1 and ViewComponent v2.5.1) and added the example components that you outlined and it all seems to work fine for me.
I've pushed the app up to a repo here - if you have any time to pull that down and give it a spin it would be great to know if you see the same issue on that repo. If not then I will have try to think of some other ways to investigate it.
Many thanks :-)

I haven't heard back from you @amree about this issue and I can't replicate it so I'm going to go ahead and close this. If you are still having problems feel free to reopen!

@allmarkedup Sorry about this. But yeah, let's close it first as I'm not sure when I can explore this again. I will update here if there's any new changes.
Thanks for looking into this, really appreciate it.

@amree no worries at all! Do let me know if you try again and run into issues or if there is anything else I can help out with 👍


FlowScrollTo bug

Hi there,
First of all great work on your control - its awesome!
Just wanted to let you know that I've found bug in your FlowScrollTo method in
DLToolkit.Forms.Controls/FlowListView/DLToolkit.Forms.Controls.FlowListView/FlowListView.cs
You are casting ItemsSoucre to IEnumerable:
var castedItemsSource = ItemsSource as IEnumerable<IEnumerable>;
var internalItem = castedItemsSource?.FirstOrDefault(v => v.Contains(item));
ScrollTo(internalItem, position, animated);\
'As' in this case will always return null cause of the fact that ObservableCollection does not inherit  IEnumerable.
Hope this will be helpful.
Again great job man :)
Cheers!

Hi @gr3chu Thanks! Scrolling issues should be fixed with: https://github.com/daniel-luberda/DLToolkit.Forms.Controls/commit/167f34f98a8018e8eaf8e645186a7d806dbf399d
Cheers :)


unable to load scripts for hubot

Your Rocket.Chat version: 0.35.0
I put scripts into the scripts folder of home/danny/scripts but I don't know what I am missing.
docker run -it -e ROCKETCHAT_URL=<IP_ADDRESS>:3000 \
-e ROCKETCHAT_ROOM='GENERAL' \
-e ROCKETCHAT_ROOM='diceroom' \
-e LISTEN_ON_ALL_PUBLIC=true \
-e ROCKETCHAT_USER=testbot \
-e ROCKETCHAT_PASSWORD=password \
-e ROCKETCHAT_AUTH=password \
-e BOT_NAME=testbot \
-e EXTERNAL_SCRIPTS=hubot-pugme,hubot-help,basic \
-v $PWD/scripts:/home/hubot/scripts \
rocketchat/hubot-rocketchat:v0.1.4

`# hubot greeting.

(hi|hello) - say hi to your butler
module.exports = (robot) ->
robot.respond /hi|hello/i, (msg) ->
msg.send "Howdy!"`

This issue is a bit dated. I feel sorry it didn't get any attention. But now we are active again! Are you still having this problem or did you find a solution?

I need to add the volumes command as below and create a folder called scripts in my working directory
volumes:
  - ./scripts:/home/hubot/scripts

Oh superb. Thanks for the solution!


Connection broker

This PR adds support for a new module, the "connection broker", that allows origins to callback to the cache.

Merge blocked by #765 due to failures (Ubuntu needs an updated container).

@haoming29 --
That looks a bit like the issue that was solved with xrdcl-pelican v0.9.3: the token isn't passed through from the cache to the origin.  I wasn't able to reproduce the issue with the latest.


Fix the protected branch panic issue

Fixes #3326

Codecov Report

Merging #3567 into master will increase coverage by <.01%.
The diff coverage is 42.85%.

@@            Coverage Diff             @@
##           master    #3567      +/-   ##
==========================================
+ Coverage   35.78%   35.78%   +<.01%
==========================================
  Files         285      285
  Lines       40850    40852       +2
==========================================
+ Hits        14617    14619       +2
  Misses      24065    24065
  Partials     2168     2168

Impacted Files
Coverage Δ

routers/repo/branch.go
57.73% <42.85%> (+0.5%)
: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 19bf4dd...db8ae5d. Read the comment docs.

I think this should be backported to release/v1.4 branch


Create request methods enumerator

Isn't it a good idea to create a uniform enumerator with the names of the request methods? Similar to RequestOptions.
final class RequestMethods
{
    public const POST = 'POST';

    public const GET = 'GET';
    //etc
}

This approach minimizes the risk of error and standardizes the code.

No. I don’t think so. It is a matter of taste. Feel free to create a separate class/package with this.


community list auth fix and join button

Still some bugs but the community page should work without being logged in now

Works flawlessly, an absolute example of pure engineering prowess. Truly the greatest button functionality man has ever seen.


Can't See Transparent Objects in Normal Buffer in CompositorEffect

Tested versions
v4.3.beta1.official [a4f2ea91a]
System information
Godot v4.3.beta1 - Windows 10 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 3050 Ti Laptop GPU - AMD Ryzen 7 5800H with Radeon Graphics (16 Threads)
Issue description
I am trying out the new CompositorEffect in 4.3. I am trying to create a post-transparent effect that uses both the depth buffer and the normal buffer. (The ultimate goal of this is to create an outline shader that can outline transparent objects.) However, it seems like transparent objects aren't showing up in the normal buffer.
Here is the regular color buffer:

Here is the colors inverted using the CompositorEffect:

Here is the depth buffer:

And here is the normal buffer (the transparent sphere and cube are missing):

Steps to reproduce
I've included an MRP. There's a single 3D scene with a camera, light, world environment (where the CompositorEffect is attached), and a few meshes, as shown in the images above. You can find the code for the CompositorEffect and the shader in the MRP.
One thing worth noting in particular: On transparent materials, depth draw mode is set to "Always". It seems that transparent materials won't show up in the depth buffer otherwise. I couldn't find an equivalent option related to the normal buffer. This makes me wonder, is it just not possible in Godot to have transparent objects show up in the normal buffer? If not, would it be possible to add this as an option, just like with the depth buffer?
I know none of the particulars about how any of this works, so I'm very sorry if I'm missing something here or if this belongs as a proposal rather than a bug.
Minimal reproduction project (MRP)
compositor-test.zip

im jroking my peanots


Add tasks to create icingadb-redis log directory

Icinga's packages no longer create /var/log/icingadb-redis/ since logging moved to the journal by default. If logging to a file is needed, the according directory has to be created beforehand.
Fixes #298

If I understand correctly we will "revert" the default logging method from icingadb-redis because we have a default set and the variable icingadb_redis_logfile isn't empty.
In the config template I recognised the following, I guess within the next release we need to provide a switch to choose between these two options?
#syslog-enabled no

Regarding the PR, this looks very good and I approve the changes.


Approximate comparison (re: issue 38 - https://github.com/facebook/ios-snapshot-test-case/issues/38)

Hey.
I've added in an option to pass a maxumim number of pixels that can differ for an image to be counted as equal.
This solves my particular use case (we have an errant pixel in a gradient that's throwing our tests off) but if people want more control this could be extended to include different metrics of differenceness.
Let me know what you think!

PS I don't know of any standards for pull requests - I've made tests for my changes, the travis build script passes and the pod lints. I didn't update the podspec version number because I don't know what else goes into releases etc. I also didn't squash my commits, but I can if you need me to. S

Thanks @nap-sam-dean! Will take a closer look once we get a CLA on file. Here's some initial feedback:

can we express max difference as a percent?
instead of adding a new comparison method, can we just add a parameter to the old?
looks like the project file and podspec got churned a bit because of the new dependency, but since that dependency isn't necessary those files don't need to be changed in this PR.

This would be such a great addition.

Yes I agree with @dstnbrkr, we shouldn't have a new comparison method but this will make it a breaking change though.
Don't forget to have the same changes for FBSnapshotVerifyLayer macros

If you don't mind breaking changes then that would certainly be cleaner :)
@dstnbrkr When you say 'add a parameter to the old' do you mean from the #define onwards?
And I'm waiting for net-a-porter to get back to me about the cla, sigh. I should have done this in my own time :)

Hi.
I've updated the pull request to include

percentage difference instead of absolute difference
removed annoying changes in the project file
just updated the original macro / methods instead of adding additional ones
updated the ..Layer macros, sorry @nscoding !
reverted the Podfile.lock - this was changed because of the pod install step in build.sh - looks like I'm using a slightly different version of the cocoapods gem!

More feedback welcome, I'm still waiting on the HR team here so I can sign the SLA :(

Overall looks awesome, it will be such a great addition to the framework.

Hey.
Latest changes:

Remove both silly spacing mistakes in FBSnapshoptTestCase.m
Remove the tolerance parameter for the basic macro; t now assumes a tolerance of 0

Looks like FBSnapshotTestCaseDemoTests still needs an update (still passes third arg to FBSnapshotVerifyView).

Well, that's pretty embarrassing :) Could have sworn I ran ./build.sh before I pushed. Oh well, it should pass now!

Hi everyone.
I've signed the CLA and pushed fixes to the conflicts. However, I've got a few questions:

For some reason my tests fail if we scale the context on retina devices. Looking at the pixel memory without scaling I get the correct values but with scaling I just see 0xFF indicating that the image isn't being drawn correctly. However, I don't know why we need to scale the context so I don't know what effect removing the scaling code has (I suspect something subtle will break). Instead of using the scaling value for the screen should we be using the scale property of the incoming image? (And should we put a check in place so that images with different scale values aren't equal?)

The CI is failing because it can't find an example image - should I run the failing test in recode mode and commit the reference image for this failing test? I can do that no problem but I'm surprised that it's been passing up until now :)

Thanks everyone :)

Thanks @nap-sam-dean! The most recent merge changed the reference image directory configuration from a preprocessor macro to an environment variable. I see in the Travis CI logs that the reference image directory isn't set correctly:
FBReferenceImageFilePathKey=P‹<EMAIL_ADDRESS>That "P‹" before the "_32" looks like a bug - but no idea how that bug would be first appearing in this PR. I'll investigate as well, but if you want to take a look - the problem might be in how the path to the reference image is constructed.

We don't have a CLA on file yet @nap-sam-dean, can you submit once more? @facebook-github-bot will comment on this PR once it receives the CLA.

ping! couple minor test fixes and CLA resubmit and this PR is all set

Hi,
Sorry for the small pause :)
I've signed the CLA again (my mistake, I'd signed as my private github account, not my net-a-porter one, sigh)
I've fixed the reference images path, removed the commented out scaling code and removed the spaces before the ++s. The CI build is passing again now :)
PS The issue was a bad merge by me to resolve the conflict from pull request 89 :(

Thanks for this great contribution @nap-sam-dean! I'll defer to @nscoding for the merge.

Thank you so much @nap-sam-dean

No problem :)

:clap: :clap:
Is it possible, now that this is merged, to get a release that includes this feature?
Thanks!

@kerrmarin Yes of course, we will let it sit on master for few days and then we will tag it as 2.0


Design a logo for this project.

Is your feature request related to a problem? Please describe.
Since this project has no logo. A logo has a significant impact on a project's public perception. Logo grabs attention and makes a strong first impression and a foundation of the brand/project identity.
Describe the solution you'd like
Design a good, formal, attractive and elegant logo for this project that stands for our objective and relates to the project identity.

I would like to work on this

Hello @PragatiVerma18  I've added few suggestion of logo here #21
Please help me with feedback.

I would really like to work on this issue. Kindly assign me this issue.

@PragatiVerma18 Please assign this to me. I can design a good logo for this project

Hi @PragatiVerma18 I would like to work on this. Please assign me on this issue.

@iBruceW please start working on this issue and keep us updated regarding the progress. All the best!!

Ok ma'am. Already working on that.

@iBruceW please start working on this issue and keep us updated regarding the progress. All the best!!
Hi @PragatiVerma18, I have created a PR for this issue. Please review my PR.

Hi @PragatiVerma18, I have created a PR for this issue. Please review the my PR.

Hi @PragatiVerma18, I have created the PR for this issue, please review it.

Hi @PragatiVerma18, I have created the PR for this issue, please review it.

Please refer the PR to this issue.

@PragatiVerma18 Can I work on this issue?

@PragatiVerma18 Can I work on this issue?

Yeah sure, this issue can have multiple instances, so please contribute your logo design.

Hi @PragatiVerma18 , I created a PR #241. Please review and merge.

will you please assign it to me? @iBruceW

Can you assign me this issue

@deepdarshan21 please contribute your logo designs.

it is showing that permission denied to my id....can you help me in this

@deepdarshan21 please share a screenshot of your problem.

it was resolved now
and I also made a pull request for the Logo design

is you saw my pull request

@deepdarshan21 please refer your PR here for a quick review.

@PragatiVerma18  #288

Is you seen it

@PragatiVerma18

@deepdarshan21 Please do the required changes, I reviewed it.

@PragatiVerma18  I made a new pull request in feature/frontend branch, according to your review request in the form of new pull request #308 . So please review it


add git info to changelog

Changelog entries should be able to reference and add in the git references and / or the PR information (e.g. PR number in Github) for each change.

This was added and merged in #90.


add plug LIDL Silvercrest HG06337 / SAPZ 1 A1 (Tuya TS011F)

ability to change the behavior of the led on the plug and the power on state on the plug LIDL Silvercrest HG06337 / SAPZ 1 A1 (based on Tuya TS011F)
for information jeedom format parameters associated
"config" : [
  {"endpoint":11,"cluster":6,"attribute":32769,"name":"Light mode","type":"select","values":[
    {"value":0,"name":"Led disabled"},
    {"value":1,"name":"Led enabled"},
    {"value":2,"name":"Led enabled but inverted"}
  ]},
  {"endpoint":11,"cluster":6,"attribute":32770,"name":"Power on state","type":"select","values":[
    {"value":0,"name":"Off"},
    {"value":1,"name":"On"},
    {"value":2,"name":"Last state"}
  ]}
],

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

27 of 27 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.09%) to 80.509%

Totals

Change from base Build<PHONE_NUMBER>:
0.09%

Covered Lines:
4585

Relevant Lines:
5695

💛  - Coveralls

@dmulcahey Is it good making 2 different quirks for tuya TS011F devices ?
I think its better using the tuya quirks and not splitting and making problem finding the quirk one device shall uing.
This is using  MODELS_INFO and tuya is using MODEL so the device shall having it already if using one up to date ZHA installation.

@dmulcahey Is it good making 2 different quirks for tuya TS011F devices ? I think its better using the tuya quirks and not splitting and making problem finding the quirk one device shall uing.
This is using MODELS_INFO and tuya is using MODEL so the device shall having it already if using one up to date ZHA installation.

unfortunately it is not a Tuya product but a white labelled Lidl product made by Tuya. Features are not exactly the same. That's why I move the quirks in the Lidl folder. Seems better no?
I dont know why but the checks are not successful...

Codecov Report

Merging #1237 (6d672ae) into dev (d5a7e9f) will increase coverage by 0.09%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##              dev    #1237      +/-   ##
==========================================
+ Coverage   80.41%   80.50%   +0.09%
==========================================
  Files         207      209       +2
  Lines        5668     5695      +27
==========================================
+ Hits         4558     4585      +27
  Misses       1110     1110

Impacted Files
Coverage Δ

zhaquirks/lidl/__init__.py
100.00% <100.00%> (ø)

zhaquirks/lidl/ts011f_plug.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

Can you plea posting the device signature without your new quirk so its possible see how the device is configured in real.
And what is not working OK with the tuya quirk ?
The failing lines:
-
-from zhaquirks.lidl import (
-    TuyaZBOnOffAttributeCluster,
-)
+from zhaquirks.lidl import TuyaZBOnOffAttributeCluster

Use https://black.vercel.app/ and its fixing the splitted lines and little more good things.

The signature is the following
SizePrefixedSimpleDescriptor(endpoint=11, profile=260, device_type=266, device_version=1, input_clusters=[0, 3, 4, 5, 6], output_clusters=[25, 10])
SizePrefixedSimpleDescriptor(endpoint=242, profile=41440, device_type=97, device_version=0, input_clusters=[], output_clusters=[33])
Read model 'TS011F' and manufacturer '_TZ3000_wamqdr3f' from <Endpoint id=11 in=[basic:0x0000, identify:0x0003, groups:0x0004, scenes:0x0005, on_off:0x0006] out=[ota:0x0019, time:0x000A] status=<Status.ZDO_INIT: 1>>

All tuya TS011F is having electrical meating cluster but all extra attributes is implanted so only making one more device class and its working.
Z2M is having Moes and many more splitted tuya devices and its one great PITA and make no sin then its so many nonamed devices coming on the market and its not making any sense splitting tuya quirks by brands its only making the code base larger and not easy maintaining in the long run.

All tuya TS011F is having electrical meating cluster but all extra attributes is implanted so only making one more device class and its working.
Z2M is having Moes and many more splitted tuya devices and its one great PITA and make no sin then its so many nonamed devices coming on the market and its not making any sense splitting tuya quirks by brands its only making the code base larger and not easy maintaining in the long run.

Agree but the plug doesn't have the same signature, the endpoint is not the same 1 in tuya, 11 in lidl. so how we can have the tuya quirks working for the other brand ?

One extra line to much and all looks OK !!
Only making one new device class in the tuya quirk and its working then its matching in the TS011F and the cluster on the endpoints.
And very likely its coming more plugs with the same endpoints and clusters from different brands.
The LIDL 3 plugs with 4 USB is already in the quirk with other tuya devices.

Thanks for feedbacks and help ! New PR submitted https://github.com/zigpy/zha-device-handlers/pull/1241


Support for --freshen in ec track tekton-task

Adds support for --freshen when invoking ec track tekton-task née ec track bundle.
Reference: EC-487

/retest

Merging now, ping me with any followups...


SQLSTATE[HY000] [2002] Connection refused

Hi, I am running laradock on Windows 10 Pro,  can't get it two work as laravel throws PDO exception in the browser but works fine in tinker. I tried changing .env to localhost and <IP_ADDRESS> get the same result, as well in config database file.
This is the cmd I used to compose my container.
docker-compose up -d nginx mariadb redis php-fpm

@asolopovas
change your db host to mysql
DB_CONNECTION=mysql
DB_HOST=mysql

@manshoor it has been changed since I hose using vagrant with homestead.

@manshoor if I change DB_HOST to mysql thats what happens
SQLSTATE[HY000] [2002] php_network_getaddresses: getaddrinfo failed: Name or service not known

@asolopovas
can you share your whole mysql settings in .env file.
because i'm on branch 4.12.0 and my config is this.
try root for username and password.
DB_CONNECTION=mysql
DB_HOST=mysql
DB_PORT=3306
DB_DATABASE=mydbname
DB_USERNAME=root
DB_PASSWORD=root

well, i'm not sure about your side but default settings are working fine for me, i'm just using root credentials because i'm not using homestead db and too lazy to create a new user every time ;)
well your application is working 💯 now, so you're good to go.

@philtrep we can close this issue.

@manshoor Good job 👍

How do you change Host to mysql.I did that and a big funny error appears in my konsole
[PDOException]
SQLSTATE[HY000] [2002] php_network_getaddresses: getaddrinfo failed: Name or service not known
[PDOException]
PDO::__construct(): php_network_getaddresses: getaddrinfo failed: Name or service not known

By the way all solutions i''ve tried none works for php artisan migrate.Over six hours of search and try nothing.If anyone got a solution sugestion please help me out

I figured out what was the problem. Simply you need edit your local hosts file to redirect host  that you specify in docker-compose.yml to <IP_ADDRESS>, for example
    // in this case the host is mariadb
    mariadb:
        build: ./mariadb
        volumes:
            - mariadb:/var/lib/mysql
            - ./mariadb/sql:/docker-entrypoint-initdb.d
            - ./mariadb/hosts:/var/tmp/hosts
        environment:
            MYSQL_USER: homestead
            MYSQL_PASSWORD: secret
            MYSQL_ROOT_PASSWORD: root
        ports:
            - "3306:3306"
        networks:
            - lemp-network

So your local hosts file will have this line.
<IP_ADDRESS> mariadb
Then in your .env file put DB_HOST=mariadb
After that your artisan and webapp will be capable of successfully connecting to your docker db...

I think the issue php-fpm is capable of accessing db inside docker container, but when you run php artisan you may be running local  php which for some reason it can't connect to local db port. Maybe it has something to do with mariadb firewall setup which blocks connection to port from outside the docker containers.

This does not seems like an ideal solution even though it works. Has anyone got a better way of doing it (not changing hosts file)?
CC:@asolopovas @leomask101193

The hostname in docker-compose is the name of your container, so in his case it's mysql, if he would have named his container database, then the hostname would be database. Can you connect through MySQL terminal tool, if no you have a connection issue, try this in your container mysql -u root -h mysql

@asolopovas your solution worked! I could not for the life figure out what the problem was! thanks!

I agree with @bzbislawski that editing /etc/hosts doesn't seem like the right approach, especially since that's not mentioned in the Laradock setup documentation.
What I'm frustrated with is that when I run cd /apps/laradock && docker-compose up -d nginx mysql redis php-worker memcached proxy, everything works for a number of minutes... and then I start seeing SQLSTATE[HY000] [2002] php_network_getaddresses: getaddrinfo failed: Name or service not known.
I think I temporarily fix it by running cd /apps/laradock && docker-compose restart mysql, but I don't know why it ever starts failing in the first place.

I wanted to add an additional note for anyone dealing with this and trying to use the editing your hosts file solution - if you changed the port for your mysql container, be sure to update the port it's connecting to in your hosts file as well.

The hostname in docker-compose is the name of your container, so in his case it's mysql, if he would have named his container database, then the hostname would be database. Can you connect through MySQL terminal tool, if no you have a connection issue, try this in your container mysql -u root -h mysql

Thank you for this!

This solution works for me.

I am using these services in Laradock: docker-compose up -d mariadb nginx redis workspace php-fpm
So I link the workspace with mariadb by edit docker-compose.yml file. Line 140 (service workspace > links section)

workspace:
  ...
  links:
   - docker-in-docker
   - mariadb

Then update DB_HOST=mariadb. Restart docker-compose, clear artisan cache and run migrate.

This works for me (I am also using laravel, docker and laradock):

Just put DB_HOST=<IP_ADDRESS> at your .env project file (thus, it isn't necessary modify /etc/hosts).

I was resisting to do that, because I have tried DB_HOST=localhost and was not working, but, for some reason, <IP_ADDRESS> works. If it isn't work, try to clear cache and use MySQL 5.7 (besides to use <IP_ADDRESS>).

This does not seems like an ideal solution even though it works. Has anyone got a better way of doing it (not changing hosts file)?
CC:@asolopovas @leomask101193

I went through problems like this and needed to install some extensions like pdo_mysl. After that, it works by both IP and HOST.
 image: php:7.4-fpm
command: docker-php-ext-install pdo_mysql mbstring zip exif pcntl


ERROR in OverlayModule is not an NgModule after upgrading to beta 10

Bug, feature request, or proposal:
Bug
What is the expected behavior?
There should be no error
What is the current behavior?
After upgrading to beta 10 I get the following error whene running ng serve:
ERROR in OverlayModule is not an NgModule
When I run it in aot mode I get:
ng serve --aot --i18n-file=src/i18n/messages.en.xlf --locale=en --i18n-format=xlf
..
ERROR in Unexpected value 'OverlayModule in<EMAIL_ADDRESS>imported by the module 'AppMaterialModule in S:/experimente/tiny-translator/src/app/app-material.module.ts'. Please add a @NgModule annotation.6a0a8c1bbb7963e348f6
What are the steps to reproduce?
What is the use-case or motivation for changing an existing behavior?

Would have expected something like a compiler error like "cannot find OverlayModule in @angular/material" with a file reference, but because it is reexported from @angular/material, there is no such error.
Even googling for "is not an NgModule" was more confusing than helpful.
Now it works as expected.
Thank you very much for your assistance.


Running into an error while unmarshalling requests

Hello,
I was putting together a simple proof of concept API using this package for encoding/decoding JSON-API endpoints but I am coming across an issue related to ID numbers.
According to the spec here the id and type keys must be present within data. So following those guidelines, I send a request like so:
PATCH /path-to-endpoint HTTP/1.1
Content-Type: application/vnd.api+json
Accept: application/vnd.api+json
{
  "data": {
  	"type": "enterprises",
  	"id": "1154833676970755108",
    "attributes": {
      "description": "This is a test enterprise",
      "name": "Test Enterprise"
    }
  }
}

My struct looks like this:
type Enterprise struct {
	ID           int64      `jsonapi:"primary,enterprises"`
	InsertedByID *int64     `jsonapi:"attr,inserted-by-id"`
	UpdatedByID  *int64     `jsonapi:"attr,updated-by-id"`
	Name         *string    `jsonapi:"attr,name"`
	Description  *string    `jsonapi:"attr,description"`
	InsertedAt   *time.Time `jsonapi:"attr,inserted-at"`
	UpdatedAt    *time.Time `jsonapi:"attr,updated-at"`
}

Below is how I'm unmarshalling the record:
req := new(model.Enterprise)

if err := jsonapi.UnmarshalPayload(r.Body, req); err != nil {
	// Handling error here...
}

The exact error is id should be either string, int or uint. This error goes away if I do not include the ID in the request body, however since it is listed as a required part of the spec, I'd prefer not to deviate. It looks like the error is stemming from here. I looked through the code to see if there were any obvious issues I could find, or issues with my code but I couldn't find anything.
Is there something I'm doing wrong?

If you look at the if part of the code you linked its checking to see if the primary field type is either "string", "int" or "uint", if its neither of those then you'll get that error. If you were to set your ID field to "int" though you'd get a parse error as well as "1154833676970755108" is too big to fit in an "int" field, strconv.Atoi() will return an error.
Assuming you can't use a smaller range of id values I'd suggest setting your ID field type to string and then parsing it yourself strconv.ParseInt(req.ID, 10, 64) if you need the actual numeric value.
As JSON API doesn't make any specific determinations on what an ID can be, and given that this package is already trying to be helpful by parsing into an Int, it would make sense for this package to also support a primary field type of int64 or uint64 as well.

I've expanded the accepted ID types in https://github.com/google/jsonapi/pull/51
That PR will allow IDs to be string, int(8,16,32,64), uint(8,16,32,64), *string, *int(8,16,32,64), *uint(8,16,32,64)

@aren55555 cool! Would love to see it merged. I just ran it locally and it seems to solve this issue.

#51 Merged!


Replace bsdtar with libarchive-tools

Just quickly tested on my existing Umbrel OS install and it doesn't look like we have bsdtar installed, only GNU tar:
umbrel@umbrel:~ $ tar --version
tar (GNU tar) 1.30
Copyright (C) 2017 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <https://gnu.org/licenses/gpl.html>.
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.

Written by John Gilmore and Jay Fenlason.
umbrel@umbrel:~ $ bsdtar --version
-bash: bsdtar: command not found

Is that expected? Do we need bsdtar if we have GNU tar? Should that binary be available in existing Umbrel OS installs?

Bsdtar is needed on the build host, not on the target.
It is only used if NOOBS images are created, so if we delete the export-noobs folder (The scripts aren't run anyway in the current Umbrel), the dependency could be removed anyway.

Ah, of course! I was thinking this was running on the device, not the host.
Thanks @AaronDewes!

For future reference, see here for reasoning: https://github.com/getumbrel/umbrel-os/pull/112#issuecomment-691474333


Implemented TreeView control

Q
A

New feature?
[x]

Related issues?
#498

What's in this Pull Request?
Added new TreeView control that allows presenting a hierarchical view of information.

Guidance
Added new TreeView control that allows presenting a hierarchical view of information.

Thank you for the great effort!


MiniAirbrakes compatible version update

Updated compatible KSP version data from "1.0.4" to "any"

Looks good; confirmed that "any" should work since it's just a part cfg. Thanks for taking care of this!


a fake PR for testing

Title
On the title describe
what you've fixed (or created) with this Pull Request (PR).
Summary
Insert a short (one or two sentence) summary here.
Fixes #Issue_Number

Note: The "Fixes #nnn" syntax in the PR description causes
GitHub to automatically close the issue when this PR is merged.
Remove that line if you don't have issues associated with this
PR. Click on the Guidelines for Contributing link above for details.

Details
Explain your changes, and why you made them. If that
information is already available in the issue referenced
above, just referencing the issue is preferred to copying
the text.
This may not be necessary depending on the scope of the PR
changes. For example, "fix typo in introduction.md" is
sufficient to describe that PR.
Suggested Reviewers
If you know who should review this, use '@' to request a review.

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

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

TTYL, DNFBOT;

Since yesterday, it seems that the OPS builds are now hooked up again. Can I close this @guangw?

@mairaw yes, please close it.


document BitsetIter functions

As much as I would like to merge this, the real solution is to fix https://github.com/GuillaumeGomez/rustdoc-stripper to handle the cases where the impl trait contains a lifetime parameter.

I'm not familiar with this tool. Can you explain a bit more about how it's used in gtk-rs?

I'm not familiar with this tool. Can you explain a bit more about how it's used in gtk-rs?

https://github.com/gtk-rs/gir is used to parse the xml files (gir files) and find the C docs for each item (function, struct & so on), convert the docs links to rust compatible ones, dump them in a markdown file and then the tool i mentioned is used to embed the docs from that markdown file in the rust files before running cargo doc in the CI.

Isn't this manually implemented though? The existing docs wouldn't be very helpful since the Rust functions have a different return type.

Isn't this manually implemented though? The existing docs wouldn't be very helpful since the Rust functions have a different return type.

Even when manually implemented, rustdoc-stripper is capable of injecting the docs back.  The different return types are not really a huge issue, we could improve gir by making it not document the return value and do only that part manually but re-copying the whole docs for every manually implemented types/function is going to be a lot of work.
Feel free to open an issue on gir side to discuss how we could improve the situation here, but I am not going to merge the docs here.


Shared seed between Trezor Model T and Ledger Nano S give different ADA wallets

I'm new to hardware wallets so not sure if this is a Daedalus problem or a problem with one of the hw wallets' firmware/app or just user error.  I use a Ledger Nano S as a backup hw wallet to my Trezor Model T.  All other cryptocurrency wallets, that I use, are in sync except for the ADA wallet.  I didn't use a passphrase on either hw wallet.  I added both hw wallets on Daedalus 3.3.1 on Ubuntu 20.04.

cc @darko-mijic @tomislavhoracek

cc @ManusMcCole

I tested this and my findings are in line with @nocmonkey. I restored the seed from my Ledger Nano S (balance of 15 ADA) on my Trezor Model T and it restores an empty wallet in Daedalus @nikolaglumac

What about restoring the same BTC wallet in both Trezor and Ledger?

Yes it seems they are standard across devices.I found this online "Ledger and Trezor support the same consensus standards and so the seeds are reversible. You can use a Ledger seed on Trezor, and you can use a Trezor Seed on Ledger, which we will go over today" at https://medium.com/@vasyavovich87/how-to-restore-ledger-wallet-with-trezor-wallet-seed-backup-ed5498d3588f. Is that sufficient confirmation ?

I tested it with my Ledger and Trezor and for BTC it restores the same wallet @nikolaglumac

OK thanks @ManusMcCole. This is very interesting and I guess we should have a broader discussion on this topic with @darko-mijic @KtorZ and @rvl.

Hi. This is "normal". Ledger and Trezor uses a different algorithm for generate root keys from seed. Trezor uses even a different method between 18-word phrases and 24-word phrases. You can find more details here: https://github.com/cardano-foundation/CIPs/blob/master/CIP-0003/CIP-0003.md
Also: Cardano is not Bitcoin :blush: ...

Thanks @KtorZ! With that being said I will now close this ticket.
@nocmonkey we suggest you to use another Ledger device as a backup for Ledgers, and another Trezor device as a backup for Trezors. Thanks!


How to keep crop window aspect ratio intact while resizing

When I drag the crop window by a corner to resize it, this also changes the aspect ratio.
I would expect that dragging a corner would only resize the window, while keeping its aspect ratio intact.
Would that be possible?
Thanks!

https://github.com/ArthurHub/Android-Image-Cropper/wiki/FAQ#how-can-i-set-crop-shape-to-squarecircle

Thanks, and sorry that I wasn't very clear.
It's not that I want to fix the aspect ratio. Users who want to change it should still be able to do so by dragging the sides of the crop window.
My request is for users who want to keep the initial aspect ratio perfectly intact while resizing. Ideally this would be possible by dragging a corner of the crop window.
What do you think?

It's a nice option, a PR would be great :)

Great!
But what is a PR?

pull request, it means you help out by adding this feature and contribute it to the library

@ArthurHub is this enhancement added to the latest version??


Task may be a callable

Task may be a callable
task('task1', function() {});
task('task2', 'functionname');
task('task3', 'NameSpace\\ClassName');
task('task4', new Object());

Nice! way to solve task/object problem.

I implemented this feature in https://github.com/oanhnn/deployer/tree/feature-task-may-be-callable
Changed:

Task may be defined from a callable
Task group be defined by function taskGroup

I will add more test cases before make a PR. :smile:

Cool!.

Merged!


Clean up Entry Points Dialog

The service catalog Item entry point dialog text is over-complicated as the text "State Machine (NS/Cls/Inst)" is implied and just adds extra clutter while messing up the formatting.
https://bugzilla.redhat.com/show_bug.cgi?id=1389437

@Fryguy I don't understand why cleaning up text would break Travis.  Is there a way to kick it to try again?

@mikedahlgren please add before/after screenshots. Text removed in this PR was to indicate to user that tree has values in Namespace/Class/Instance format and are state machine objects. @mkanoor is it ok to remove the text from screen.

I understand that is what it stands for, however it adds no value because when a person edits that path the gui brings up a hierarchy menu where the user selects the entry point.   The user CAN NOT change the order of the tree, or type it in wrong.  This is a hold over from times past.
I am also curious why the text doesn't line up exactly between the two sides, this PR helps but does not completely fix that.
Before:

After:

@mzazrivec Please review

@h-kataria euwe/yes or euwe/no please

Euwe backport details:
$ git log -1
commit 90b473d6e2db19d6fcedde69df3e26b0cdf1cccc
Author: Harpreet Kataria<EMAIL_ADDRESS>Date:   Mon Oct 31 12:03:23 2016 -0400

    Merge pull request #12247 from mikedahlgren/patch-2

    Clean up Entry Points Dialog
    (cherry picked from commit e06b9cc0c39f8855d44e31dbf3892040f13b6f73)

    https://bugzilla.redhat.com/show_bug.cgi?id=1411518


Trying to upgrade from PHP 7.3 to 7.4 permanently bricked my Docker instance

I'm trying to do some local dev with laradock. I've been using the same .env and docker-compose.yml file for 18 months without any issues. Today I tried to upgrade to PHP 7.4 by changing the PHP_VERSION in the .env file to 7.4 and this has caused my machine to basically shit the bed.
After stopping the container and then running docker-compose build --no-cache workspace suddenly threw a new error I'd never seen before. I managed to resolve it by setting WORKSPACE_INSTALL_AST=false , which was never necessary before. This allowed me to move past this error.
However, there is a persistent error I cannot resolve no matter what I try. On the build of workspace, I always get the following error on step 101/109:
executor failed running [/bin/sh -c set -xe; php -v | head -n 1 | grep -q "PHP ${LARADOCK_PHP_VERSION}."]: exit code: 1
It doesn't matter the PHP version I try (7.3, my original working version or 7.4). It doesn't matter if I fully wipe Docker Desktop and start again. It doesn't matter if I swap my custom docker-compose.yml for the default one. I always get the same error in the exact same place. If I comment out Line 1551 RUN set -xe; php -v | head -n 1 | grep -q "PHP ${LARADOCK_PHP_VERSION}." then the build finishes but the workspace container runs PHP 8.1 which breaks everything, including php artisan tinker.
Any ideas on what in the world could have happened to brick me so permanently? I can't figure it out and I really would prefer not to have to buy an entirely new computer because I tried to upgrade PHP in docker for the first time in two years.

Here is a screencap showing the error when I tried to install with 7.2 to see if anything was different

Don't really understand it, but I was able to resolve my issue by re-cloning laradock into a new directory on my local machine, building the workspace container within that directory, THEN switch back to my original directory and use docker-compose to start the containers.


runtime: enable shims for Float16 on Windows (x86)

When building for Windows x86, we do not have __i686__ defined, but do
have __i386__ defined.  Ensure that the routines are included for the
x86 Windows target.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@swift-ci please test

CC: @stephentyrone

I expect i386 is also defined for the targets using i686, so we could just use that, but this LGTM.

Yeah, I do expect that __i686__ will be defined, but the default macros that are defined can be wonky, so Im okay with a bit of duplication as a guard against some platform deciding to be more precise about the macro definitions (even though i686 is backwards compatible).


Give markdown rendering of README inside Xcode

Same contents as my merged PRs into swift-numerics, swift-crypto, swift-argument-parser and library-preview (establishing precedence 😎), giving proper markdown rendering inside Xcode.
Motivation:
Reading of markdown becomes easier. Extra important in Swift packages, where the README is the first file opened by Xcode when opening said package.
Modifications:
Just adding the .xcodesamplecode.plist file (to the root).
Result:
As seen in Apple's ARKit demo (inside /ARKitExample.xcodeproj/ directory)
Before:

After:

Drawback
⚠ Editing is now disabled in Xcode
This effectively disables editing markdown files from Xcode - or at least I have not found a way of doing so. The "workaround" is just to open markdown files with your favourite alternative text editor, e.g. emacs, VIM, VI, Sublime Text or whatever.
Meta
Since all linked to PRs above have been merged, feels like folks, in general, agree with me that this gives a much more nice README experience natively inside Xcode. Given that - maybe we ought to make this file part of swift package init command? What do you think?

I see, thanks for adding this to the recently open sourced projects 🙏
A bit annoying that it makes the readme not editable but the presentation is much nicer... Sounds okey to include, thanks 👍

@swift-server-bot test this please

thank you @sajjon
cc @neonichu

Feels almost as if we should make the integration in Xcode smarter about this:

for root packages, we would keep the current behaviour, since presumably for the packages you own you care about editing more than reading
for dependencies, we should make it so that we get the rendered version

That sounds like a good idea to solve it nicely, want me to shoot a radar or will you @neonichu ? :)


Changes for titlebar

When MMTitlebarAppearsTransparent is enabled, using window appearance (NSAppearanceNameDarkAqua, 10.10 and higher version is required) to avoid title bar font color fusion with background.
Add MMTitlebarDisableDocumentIconKey to disable file icon from titlebar

Auto-picking dark aqua color scheme: I don't think this is a good idea. What if someone has a colorscheme with a light background (e.g. delek)? Now the title text will be all white and blown out for them. This also overrides the system default of picking aqua (for light mode) vs dark aqua (for dark mode). Setting this appearance also affects things like how scrollbars look.
One option is to auto-pick light/dark theme based on the background setting (which corresponds to which colorscheme you have). I think that setting would have value beyond just transparent titlebar, however, and probably should be its own toggle. In particular, I would imagine we would want 4 possible values for that setting: 1 – default / obey OS scheme, 2 – force light mode / aqua, 3 – force dark mode / dark aqua, 4 – auto-detect based on colorscheme's background. I think this would work but would require a little bit more work to implement.
MMTitlebarDisableDocumentIconKey: I'm ok with this part, but we should probably add a line to the documentation (runtime/doc/gui_mac.txt) under the macvim-user-defaults part.

Auto-picking dark aqua color scheme: I don't think this is a good idea. What if someone has a colorscheme with a light background (e.g. delek)? Now the title text will be all white and blown out for them. This also overrides the system default of picking aqua (for light mode) vs dark aqua (for dark mode). Setting this appearance also affects things like how scrollbars look.

I missed that problem 🤦♂

If you want this feature I may add something like what I mentioned above with the four options. I may take your MMTitlebarDisableDocumentIconKey to go along with it.

If you want this feature I may add something like what I mentioned above with the four options. I may take your MMTitlebarDisableDocumentIconKey to go along with it.

That will be great. I'm not familiar for macOS app develop. Thank you !


Trouble connecting between two nodes

Hey all!
I had a weird issue where Host A could not ping Host B over Nebula. I happened to resolve it but am not sure

Why it happened in the first place
Why my "fix" worked

(For some additional context, we have thousands of nodes connecting happily over Nebula and this is the first time we've observed this issue)
Debugging
Host A: datalake-queue4-app.ca1.qprod.net, vpnIP: <IP_ADDRESS>, mainIP: <IP_ADDRESS>
Host B: secdev-canary1-app.ca1.qprod.net, vpnIP: <IP_ADDRESS>, mainIP: <IP_ADDRESS>
Host A  logs:
Oct 10 10:34:24 datalake-queue4-app.ca1.qprod.net nebula[17345] {"certName":"secdev-canary1-app.ca1.qprod.net","handshake":{"stage":1,"style":"ix_psk0"},"initiatorIndex":1059072191,"level":"info","msg":"Handshake message received","remoteIndex":0,"responderIndex":0,"time":"2020-10-10T10:34:24-06:00","udpAddr":{"ip":"<IP_ADDRESS>","port":4333},"vpnIp":"<IP_ADDRESS>"}
Oct 10 10:34:24 datalake-queue4-app.ca1.qprod.net nebula[17345]: {"certName":"secdev-canary1-app.ca1.qprod.net","handshake":{"stage":1,"style":"ix_psk0"},"initiatorIndex":1059072191,"level":"info","msg":"Prevented a handshake race","remoteIndex":0,"responderIndex":3462305623,"time":"2020-10-10T10:34:24-06:00","udpAddr":{"ip":"<IP_ADDRESS>","port":4333},"vpnIp":"<IP_ADDRESS>"}
Oct 10 10:34:24 datalake-queue4-app.ca1.qprod.net nebula[17345]: {"certName":"secdev-canary1-app.ca1.qprod.net","handshake":{"stage":1,"style":"ix_psk0"},"initiatorIndex":1059072191,"level":"info","msg":"Handshake message received","remoteIndex":0,"responderIndex":0,"time":"2020-10-10T10:34:24-06:00","udpAddr":{"ip":"<IP_ADDRESS>","port":4333},"vpnIp":"<IP_ADDRESS>"}
Oct 10 10:34:24 datalake-queue4-app.ca1.qprod.net nebula[17345]: {"certName":"secdev-canary1-app.ca1.qprod.net","handshake":{"stage":1,"style":"ix_psk0"},"initiatorIndex":1059072191,"level":"info","msg":"Prevented a handshake race","remoteIndex":0,"responderIndex":1610801854,"time":"2020-10-10T10:34:24-06:00","udpAddr":{"ip":"<IP_ADDRESS>","port":4333},"vpnIp":"<IP_ADDRESS>"}
Oct 10 10:34:24 datalake-queue4-app.ca1.qprod.net nebula[17345]: {"certName":"secdev-canary1-app.ca1.qprod.net","handshake":{"stage":1,"style":"ix_psk0"},"initiatorIndex":1059072191,"level":"info","msg":"Handshake message received","remoteIndex":0,"responderIndex":0,"time":"2020-10-10T10:34:24-06:00","udpAddr":{"ip":"<IP_ADDRESS>","port":4333},"vpnIp":"<IP_ADDRESS>"}
Oct 10 10:34:24 datalake-queue4-app.ca1.qprod.net nebula[17345]: {"certName":"secdev-canary1-app.ca1.qprod.net","handshake":{"stage":1,"style":"ix_psk0"},"initiatorIndex":1059072191,"level":"info","msg":"Prevented a handshake race","remoteIndex":0,"responderIndex":453114336,"time":"2020-10-10T10:34:24-06:00","udpAddr":{"ip":"<IP_ADDRESS>","port":4333},"vpnIp":"<IP_ADDRESS>"}
Oct 10 10:34:25 datalake-queue4-app.ca1.qprod.net nebula[17345]: {"certName":"secdev-canary1-app.ca1.qprod.net","handshake":{"stage":1,"style":"ix_psk0"},"initiatorIndex":1059072191,"level":"info","msg":"Handshake message received","remoteIndex":0,"responderIndex":0,"time":"2020-10-10T10:34:25-06:00","udpAddr":{"ip":"<IP_ADDRESS>","port":4333},"vpnIp":"<IP_ADDRESS>"}
Oct 10 10:34:25 datalake-queue4-app.ca1.qprod.net nebula[17345]: {"certName":"secdev-canary1-app.ca1.qprod.net","handshake":{"stage":1,"style":"ix_psk0"},"initiatorIndex":1059072191,"level":"info","msg":"Prevented a handshake race","remoteIndex":0,"responderIndex":1045871539,"time":"2020-10-10T10:34:25-06:00","udpAddr":{"ip":"<IP_ADDRESS>","port":4333},"vpnIp":"<IP_ADDRESS>"}
Oct 10 10:34:26 datalake-queue4-app.ca1.qprod.net nebula[17345]: {"certName":"secdev-canary1-app.ca1.qprod.net","handshake":{"stage":1,"style":"ix_psk0"},"initiatorIndex":1059072191,"level":"info","msg":"Handshake message received","remoteIndex":0,"responderIndex":0,"time":"2020-10-10T10:34:26-06:00","udpAddr":{"ip":"<IP_ADDRESS>","port":4333},"vpnIp":"<IP_ADDRESS>"}
Oct 10 10:34:26 datalake-queue4-app.ca1.qprod.net nebula[17345]: {"certName":"secdev-canary1-app.ca1.qprod.net","handshake":{"stage":1,"style":"ix_psk0"},"initiatorIndex":1059072191,"level":"info","msg":"Prevented a handshake race","remoteIndex":0,"responderIndex":611876616,"time":"2020-10-10T10:34:26-06:00","udpAddr":{"ip":"<IP_ADDRESS>","port":4333},"vpnIp":"<IP_ADDRESS>"}

However print-tunnel <IP_ADDRESS> shows a tunnel exists:
bluong@nebula > print-tunnel <IP_ADDRESS>
{"connection_state":{"certificate":{"details":{"groups":[],"ips":["<IP_ADDRESS>/11"],"isCa":false,"issuer":"redacted","name":"secdev-canary1-app.ca1.qprod.net","notAfter":"2021-03-17T02:39:58-06:00","notBefore":"2020-09-15T14:39:58-06:00","publicKey":"redated","subnets":[]},"fingerprint":"redacted","signature":"8a8e74d3dc71189f9f80bf7470ad2c85e467da2211fd9de901b211dce0491d7bb367a83d03e3dc0dffcf8f93cdf0dd105c99a003d3b373172d3c5f5753a35802"},"initiator":false,"message_counter":261376,"ready":true},"handshake_complete":true,"handshake_counter":0,"handshake_packet":{"0":"XmnnlN/LyPG1z+JEzQyEuE20l1giQ43vfCyNd+GnbnOf4K5bt/R8ZJbZhDRKukkjwwaO2PHKRWey+ZI0v4lZIAqvAQqgAQpcCiBzZWNkZXYtY2FuYXJ5MS1hcHAuY2ExLnFwcm9kLm5ldBIK7syYogaAgID/DyiezYT7BTDegMeCBkognJrJxRQfb+lZLHT3m5w1oF8k5cq0RVjM0AC0snbrVl0SQIqOdNPccRifn4C/dHCtLIXkZ9oiEf2d6QGyEdzgSR17s2eoPQPj3A3/z4+TzfDdEFyZoAPTs3MXLTxfV1OjWAIQ/Jv1mQUohabm+wU=","2":"EAAAAFM9TfwAAAAAAAAAAichwJN0bYnOll/oDTDsORznVP3B/7syN55dagKzIKd4VxIGnR7b8lZ74gTr10CK4prPZvGtfZhqEHG7JKoqUUXwcO7uYGEnu5QfyDbHJYHbY00tsJQP8d0IKcLpu2GumckOfXHKcBnSUxRQAYQ7TSo6Jrry3kQGETN1yEhmAPfVjDjruEGTJ0x66A5gGPZE1qUPHci0P8QJvr+NSuzxOQTnaomNS5NnN4yPjEredw3WhDQre7kMRYfi7aLNGaoBidsF54JE8q7Sk1Obu1crFjKUeV8ORZkwIyaZln4VACBfnjPbpROaKqdI+5WBEgUG+obcxPWxD4QCPkrsFxEQFuZZs/1vv2lzG06pVqGpt2Hx2dQXnzqnF1Oz"},"handshake_ready":false,"handshake_start":"0001-01-01T00:00:00Z","host_id":"<IP_ADDRESS>","last_roam":"0001-01-01T00:00:00Z","last_roam_remote":null,"local_index":2543601986,"packet_store":[],"promote_counter":261374,"receive_errors":0,"remote":{"ip":"<IP_ADDRESS>","port":4333},"remote_index":1396526588,"remotes":[{"active":false,"address":{"ip":"<IP_ADDRESS>","port":4333},"probe_count":0}]}

But on Host B no tunnel exists:
bluong@nebula > print-tunnel <IP_ADDRESS>
Could not find tunnel for vpn ip: <IP_ADDRESS>

Host B logs:
Oct 10 10:46:40 secdev-canary1-app.ca1.qprod.net nebula[915]: {"handshake":{"stage":1,"style":"ix_psk0"},"initiatorIndex":3767915586,"level":"info","msg":"Handshake message sent","remoteIndex":0,"time":"2020-10-10T10:46:40-06:00","udpAddr":{"ip":"<IP_ADDRESS>","port":4333},"vpnIp":"<IP_ADDRESS>"}
Oct 10 10:46:41 secdev-canary1-app.ca1.qprod.net nebula[915]: {"handshake":{"stage":1,"style":"ix_psk0"},"initiatorIndex":3767915586,"level":"info","msg":"Handshake message sent","remoteIndex":0,"time":"2020-10-10T10:46:41-06:00","udpAddr":{"ip":"<IP_ADDRESS>","port":4333},"vpnIp":"<IP_ADDRESS>"}
Oct 10 10:46:41 secdev-canary1-app.ca1.qprod.net nebula[915]: {"handshake":{"stage":1,"style":"ix_psk0"},"initiatorIndex":3767915586,"level":"info","msg":"Handshake message sent","remoteIndex":0,"time":"2020-10-10T10:46:41-06:00","udpAddr":{"ip":"<IP_ADDRESS>","port":4333},"vpnIp":"<IP_ADDRESS>"}
Oct 10 10:46:42 secdev-canary1-app.ca1.qprod.net nebula[915]: {"handshake":{"stage":1,"style":"ix_psk0"},"initiatorIndex":3767915586,"level":"info","msg":"Handshake message sent","remoteIndex":0,"time":"2020-10-10T10:46:42-06:00","udpAddr":{"ip":"<IP_ADDRESS>","port":4333},"vpnIp":"<IP_ADDRESS>"}
Oct 10 10:46:42 secdev-canary1-app.ca1.qprod.net nebula[915]: {"handshake":{"stage":1,"style":"ix_psk0"},"initiatorIndex":3767915586,"level":"info","msg":"Handshake message sent","remoteIndex":0,"time":"2020-10-10T10:46:42-06:00","udpAddr":{"ip":"<IP_ADDRESS>","port":4333},"vpnIp":"<IP_ADDRESS>"}
Oct 10 10:46:43 secdev-canary1-app.ca1.qprod.net nebula[915]: {"handshake":{"stage":1,"style":"ix_psk0"},"initiatorIndex":3767915586,"level":"info","msg":"Handshake message sent","remoteIndex":0,"time":"2020-10-10T10:46:43-06:00","udpAddr":{"ip":"<IP_ADDRESS>","port":4333},"vpnIp":"<IP_ADDRESS>"}
Oct 10 10:46:43 secdev-canary1-app.ca1.qprod.net nebula[915]: {"certName":"secdev-canary1-app.ca1.qprod.net","handshake":{"stage":1,"style":"ix_psk0"},"initiatorIndex":3767915586,"level":"info","msg":"Handshake message received","remoteIndex":0,"responderIndex":0,"time":"2020-10-10T10:46:43-06:00","udpAddr":{"ip":"<IP_ADDRESS>","port":4333},"vpnIp":"<IP_ADDRESS>"}
Oct 10 10:46:43 secdev-canary1-app.ca1.qprod.net nebula[915]: {"certName":"secdev-canary1-app.ca1.qprod.net","handshake":{"stage":2,"style":"ix_psk0"},"initiatorIndex":3767915586,"level":"info","msg":"Handshake message sent","remoteIndex":0,"responderIndex":930569426,"time":"2020-10-10T10:46:43-06:00","udpAddr":{"ip":"<IP_ADDRESS>","port":4333},"vpnIp":"<IP_ADDRESS>"}
Oct 10 10:46:43 secdev-canary1-app.ca1.qprod.net nebula[915]: {"certName":"secdev-canary1-app.ca1.qprod.net","durationNs":3267325020,"handshake":{"stage":2,"style":"ix_psk0"},"initiatorIndex":3767915586,"level":"info","msg":"Handshake message received","remoteIndex":3767915586,"responderIndex":930569426,"time":"2020-10-10T10:46:43-06:00","udpAddr":{"ip":"<IP_ADDRESS>","port":4333},"vpnIp":"<IP_ADDRESS>"}
Oct 10 10:37:05 secdev-canary1-app.ca1.qprod.net nebula[915]: {"certName":"secdev-canary1-app.ca1.qprod.net","level":"info","msg":"Tunnel status","time":"2020-10-10T10:37:05-06:00","tunnelCheck":{"method":"active","state":"dead"},"vpnIp":"<IP_ADDRESS>"}

I thought it weird that Host B's logs was showing a "handshake message received" from itself (<IP_ADDRESS>)
Running print-tunnel <IP_ADDRESS> on Host B (itself):
bluong@nebula > print-tunnel <IP_ADDRESS>
{"connection_state":{"certificate":{"details":{"groups":[],"ips":["<IP_ADDRESS>/11"],"isCa":false,"issuer":"redacted","name":"secdev-canary1-app.ca1.qprod.net","notAfter":"2021-03-17T02:39:58-06:00","notBefore":"2020-09-15T14:39:58-06:00","publicKey":"redacted","subnets":[]},"fingerprint":"redacted","signature":"redacted"},"initiator":true,"message_counter":4,"ready":true},"handshake_complete":true,"handshake_counter":6,"handshake_packet":{"0":"EAAAAAAAAAAAAAAAAAAAAXUmup3A2awVJG8QxxdDZArZttJ3k+ljYfR8VNs8ZU5Rn+CuW7f0fGSW2YQ0SrpJI8MGjtjxykVnsvmSNL+JWSAKrwEKoAEKXAogc2VjZGV2LWNhbmFyeTEtYXBwLmNhMS5xcHJvZC5uZXQSCu7MmKIGgICA/w8ons2E+wUw3oDHggZKIJyaycUUH2/pWSx095ucNaBfJOXKtEVYzNAAtLJ261ZdEkCKjnTT3HEYn5+Av3RwrSyF5GfaIhH9nekBshHc4Ekde7NnqD0D49wN/8+Pk83w3RBcmaAD07NzFy08X1dTo1gCEIXxrYEOKNTGh/wF","2":"SrjfXsLKbYaQooXhn8fA9K235ATBuLpeSu/Qecca2w0/wox/FvD1KzLvv1pEbSeLEPKBoXyqcwd8gL0ks8qjFbZPU7xos/bmMZ1Uq1xtM2sgxs1XHq6W23PUI+fGeOCNbMgHd6th8KEsszvxZCgHYOU4wZQOoumGPSBCfggHKg4jB2y4ANruYdqSdFH5ui79afvZYnBMmAxgYKKLpHKkBDGOfKx2u90JHfB8IOYkNfR2c/8SLvypFkDD8lR2wgAerLqyw1Eq4pIC82kLXXGsBrXWBZbsy4uomTxSvdw59Ae9P+nVjnAgwxZiWIoaPOe69vKHexho/TiJ4Mg//5dohq5ekNdfzacyl2mxOUlzNw+6n881omde5Q=="},"handshake_ready":true,"handshake_start":"2020-10-10T10:37:40.412234621-06:00","host_id":"<IP_ADDRESS>","last_roam":"0001-01-01T00:00:00Z","last_roam_remote":null,"local_index":3760945285,"packet_store":[],"promote_counter":1,"receive_errors":2,"remote":{"ip":"<IP_ADDRESS>","port":4333},"remote_index":1551199133,"remotes":[{"active":false,"address":{"ip":"<IP_ADDRESS>","port":4333},"probe_count":0},{"active":false,"address":{"ip":"<IP_ADDRESS>","port":4333},"probe_count":0}]}

The remote section contains <IP_ADDRESS> (Host A), which makes me think that Host B might be sending handshake msg back to itself?
I also tried this:
bluong@nebula > query-lighthouse <IP_ADDRESS>
null

Now for the fix:
I simply initiated a tunnel from another host (call it Host C) to Host A:
[prod.iad1][bluong@demo-app ~]$ ping  <IP_ADDRESS>
PING <IP_ADDRESS> (<IP_ADDRESS>) 56(84) bytes of data.
64 bytes from <IP_ADDRESS>: icmp_seq=1 ttl=64 time=236 ms
64 bytes from <IP_ADDRESS>: icmp_seq=2 ttl=64 time=16.3 ms
64 bytes from <IP_ADDRESS>: icmp_seq=3 ttl=64 time=16.4 ms

Now Host A can ping Host B without issues. print-tunnel Host A on Host B returns a tunnel, while print-tunnel Host B now returns no tunnel. query-lighthouse Host A also returns the right info: [{"ip":"<IP_ADDRESS>","port":4333},{"ip":"<IP_ADDRESS>","port":4333}]

So we saw this issue again, unfortunately it blocked connection between one of our large Nomad compute hosts and our main database and caused outages across a large number of services. The compute has no problem establishing tunnels to other hosts, likewise the database can too, but the communication between these two is completely blocked.
On the compute host we see it get duplicate handshakes from the database
Mar  6 03:40:23 compute-host nebula: {"certName":"database-host","durationNs":1440774386,"fingerprint":"f89dcdacf34268770f74e81ad3f90885573c94ae46038f6c9eac09d6eaecde97","handshake":{"stage":2,"style":"ix_psk0"},"initiatorIndex":4151608457,"level":"info","msg":"Handshake message received","remoteIndex":4151608457,"responderIndex":2791679384,"time":"2021-03-06T03:40:23Z","udpAddr":{"ip":"<IP_ADDRESS>","port":4333},"vpnIp":"<IP_ADDRESS>"}
Mar  6 03:40:23 compute-host nebula: {"handshake":{"stage":2,"style":"ix_psk0"},"header":{"messageCounter":2,"remoteIndex":4151608457,"reserved":0,"subType":"ix_psk0","type":"handshake","version":1},"level":"error","msg":"Already seen this handshake packet","time":"2021-03-06T03:40:23Z","udpAddr":{"ip":"<IP_ADDRESS>","port":4333},"vpnIp":"<IP_ADDRESS>"}
Mar  6 03:40:23 compute-host nebula: {"handshake":{"stage":2,"style":"ix_psk0"},"header":{"messageCounter":2,"remoteIndex":4151608457,"reserved":0,"subType":"ix_psk0","type":"handshake","version":1},"level":"error","msg":"Already seen this handshake packet","time":"2021-03-06T03:40:23Z","udpAddr":{"ip":"<IP_ADDRESS>","port":4333},"vpnIp":"<IP_ADDRESS>"}
After which it gets stuck in a loop, cycling through the handshake logic, receiving the handshake message from the database host, attempting to process it, finding there's a connection already established and 'Preventing' the handshake from completing.
Mar  6 03:40:23 compute-host nebula: {"certName":"database-host","fingerprint":"f89dcdacf34268770f74e81ad3f90885573c94ae46038f6c9eac09d6eaecde97","handshake":{"stage":1,"style":"ix_psk0"},"initiatorIndex":3549556135,"level":"info","msg":"Handshake message received","remoteIndex":0,"responderIndex":0,"time":"2021-03-06T03:40:23Z","udpAddr":{"ip":"<IP_ADDRESS>","port":4333},"vpnIp":"<IP_ADDRESS>"}
Mar  6 03:40:23 compute-host nebula: {"certName":"database-host","fingerprint":"f89dcdacf34268770f74e81ad3f90885573c94ae46038f6c9eac09d6eaecde97","handshake":{"stage":1,"style":"ix_psk0"},"initiatorIndex":3549556135,"level":"info","msg":"Prevented a handshake race","remoteIndex":0,"responderIndex":2336903254,"time":"2021-03-06T03:40:23Z","udpAddr":{"ip":"<IP_ADDRESS>","port":4333},"vpnIp":"<IP_ADDRESS>"}

We see many thousand of these in the logs.
Part of the race prevention loop is to send a message back to the originating host over the potentially established tunnel to test and verify it. What I suspect is that this is failing (silently) and the database host never gets to test the connection. At this stage no traffic can be sent between these hosts via the tunnel which supports this theory. Normal traffic between these hosts is fine though as they can ping one another without issue. Does this theory sound plausible to you?
I'm curious how you generally test the handshaking, are there any integration tests here. If someone can give me a few pointers about how you generally go about it I'm happy to dig further.

Hi,
I'm using two windows nodes, and similar issue with v1.7.2. I can see handshake between lighthouse and two windows nodes. But the nodes can't ping or do anything!!
Nebula subnet : <IP_ADDRESS>/27
Lighthouse : <IP_ADDRESS>
Windows client 1: <IP_ADDRESS>/27
Windows client 2: <IP_ADDRESS>/27
Lighthouse using dynamic DNS
Port 424, tcp and udp forwarded to lighthouse.
windows client 1: within the same network
windows client 2; when using same network no issues. But if isolate with a different network not pinging. tried it from a geographically different places and the result is same.
If you try to ping windows client 2 from windows client 1,
Nebula reports in console as below:
0 responderIndex=2169015535 sentCachedPackets=1 udpAddr="<IP_ADDRESS>:4242" vpnIp=<IP_ADDRESS>
time="2023-08-13T12:55:01+04:00" level=info msg="Handshake message sent" handshake="map[stage:1 style:ix_psk0]" initiatorIndex=3044435719 localIndex=3044435719 remoteIndex=0 udpAddrs="[<IP_ADDRESS>:4456 <IP_ADDRESS>:53812]" vpnIp=<IP_ADDRESS>
time="2023-08-13T12:55:24+04:00" level=info msg="Handshake timed out" durationNs=23172405400 handshake="map[stage:1 style:ix_psk0]" initiatorIndex=3044435719 localIndex=3044435719 remoteIndex=0 udpAddrs="[<IP_ADDRESS>:4456 <IP_ADDRESS>:53812]" vpnIp=<IP_ADDRESS>.
But both windows client 1 and windows client 2 with lighthouse connection:
Nebula Console report:
time="2023-08-13T12:56:39+04:00" level=info msg="Handshake message sent" handshake="map[stage:1 style:ix_psk0]" initiatorIndex=2024606593 localIndex=2024606593 remoteIndex=0 udpAddrs="[<IP_ADDRESS>:4242]" vpnIp=<IP_ADDRESS>
time="2023-08-13T12:56:39+04:00" level=info msg="Handshake message received" certName=lablighthouse01 durationNs=4074500 fingerprint=7c60cf89710798e103c212c393a0cd41255500a867301614808806b2b675bc9d handshake="map[stage:2 style:ix_psk0]" initiatorIndex=2024606593 issuer=cff8abf61e7e94edcd40a75c3dd4da8bb195e17c4af9a75cff547bdbadbf9daa remoteIndex=2024606593 responderIndex=1770724071 sentCachedPackets=1 udpAddr="<IP_ADDRESS>:4242" vpnIp=<IP_ADDRESS>

@riyas50 To avoid confusion, please file a separate ticket, thanks!
Closing the original ticket out as stale.


fix: update atomic register to u64::MAX-2 to avoid collisions

When using env and sys calls within the same method, it feels like a foot gun to have the register used for temporary operations to be 0, as most examples and usages I've seen of syscalls use this register.

How does this register work usually? I only see it being used for reads so I'm not sure if changing it has any effect besides reading from a different location. Or does the read do something special like create a lock?

How does this register work usually? I only see it being used for reads so I'm not sure if changing it has any effect besides reading from a different location. Or does the read do something special like create a lock?

Nah, this register is just used as a temporary buffer to be used within the same method call. This changes nothing functionally. The only negative is that someone could potentially be using u64::MAX-2 register with one of these calls sandwiched in between and it would break, but I think this is unreasonably unlikely, and much more of a common pattern for people to use register 0


make it run locally on windows box

for some reason, i can't run the doc site locally on my windows box.
there are some obvious reasons like some packages are missing in package.json, but there are also issues like  webpack loader test 'include' doesn't like /assets/images/ and I haven't figured out why  ( it may be a webpack bug??)
I have also hosted the doc site on azure just for my reference  http://material-ui-next.azurewebsites.net/

I'm gonna set this up on a windows machine this afternoon and make sure it's all working. It'll be some posix vs windows related stuff most likely

I've dealt with similar issues before in that path regexes doesn't match. In order to be OS independent you need to use regexes like /assets[/\\]images/ which will match both assets/images and assets\images.

@Alxandr
your are right. I've tested include: /assets[\\\/]images/ and verified it works fine on windows

I don't think it's my change that cause the regress test fail.
I didn't touch src code,and  the failed unit test seems to be a UI change on
I think maybe it's becuase somebody forgot to update  test/regressions/screenshots/baseline/textfield/textfieldrequired/chrome-xxx.png  ?

@bochen2014 can you replace quickly?

@nathanmarks  sorry was busy this morning.
have finished off rebase, please check. cheers.

/home/ubuntu/material-ui/node_modules/coveralls/bin/coveralls.js:18
        throw err;
        ^
Bad response: 422 {"message":"Couldn't find a repository matching this job.","error":true}

seems like a environment issue to me

seems like a environment issue to me

@bochen2014 Yes, it's. It should have been fixed now. You can rebase to get a green build 🍏 .


Add support for intersecting decorators

Let's say I'm applying 2 decorators to the following string: foo bar foo, the first one changes the color of the whole string to green, the second makes bar red. With the current implementation the whole string will be green.
I think a better system would take into account the order in which decorators are applied. For instance if I apply the red decorator after the green one, it should take precedence and make bar red, while still keeping the rest green.
This kind of logic can already be implemented in an extension, but it seems low-level and general enough to deserve to be implemented in VSC itself.
What do you think?

@fabiospampinato I agree. Our current recommendation is to ask everybody to use transparent colors.


Fix #1627: Loading UI improvements

Fixes #1627

If I select the 'All' period, I run into this error:

On the default predictoor selection task, chose a predictoor which has some predictions( in my case it didn't and resulted the same empty charts ), and I also get an issue when I deselect the predictoor:

can you check if the data exist on the selected predictoor and the selected feed for the 1W period?

Loading for the charts inside the modal looks good. Could you also add it to the home page charts to keep them consistent.
Another thing that would look good if the name of the chart is also displayed alongside the loading screen.

Sometimes when switching between pages and date range I run into the following error:

The initial load take too much with the latest updates. No matter on which page I am, the initial load takes around 8 seconds each time which is just too much

We use another PR for these changes
https://github.com/oceanprotocol/pdr-backend/pull/1661


Feature request: embed migration SQL files into go binary and run migrations on startup

Not sure if this is a good idea, as it would bloat the binary with something you would not need. Plus, in containerized environment, you would try to keep the startup time as low as possible I'd say, so running the migrations before a new deployment is probably best.
Is there any reason why running migrate with go run does not work for you or where you see disadvantages?

I created this more specific issue, which once implemented would then allow you to do migrations on startup by yourself. https://github.com/steebchen/prisma-client-go/issues/1134


Added new resource XSCDPMAgent

This change is

Hi @dcrreynolds, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (David Reynolds). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

Working to combine multiple DPM Agent resources into one branch/PR.


Release Pipeline - continue on error for GH release

When running the release pipeline, and when the VERSION is the same, ignore GH release step failure to continue to the deployment

Are we sure we want to overwrite releases?

@omri374  Shouldn't be the go-to scenario, GH is the only step that fails if a release exists. In case we want to redeploy if something happened, which is not the common usecase, this will allow that.


[Doubt] How do i save pane programmatically?

If I want to save a pane, how do I do it from python? (using the pane id basically). Asking wrt saving a graph (vis.line)
I checked the demo.py, didnt really find anything relevant to this.

By save a pane, do you mean save the plot content to a file?

Yes i mean that only. Should've mentioned that, sorry.

@ajabri any updates?

Visdom wasn't really designed for that kind of use case. However, each of the data structures that are sent to the server by the python client API are compatible with Plotly's python offline plotting feature. It should be easy to add an option to simply return that data structure instead of sending it to the server, e,g, here https://github.com/facebookresearch/visdom/blob/master/py/init.py#L585
Then, you can use the data and layout fields with  https://github.com/plotly/plotly.py/blob/master/plotly/offline/offline.py#L385.

Closing as duplicate of #232, which has more recent reflections on this


Internal link formatting has a few problems

So I tried out the new feature of supporting links internally within a document and it doesn't work properly in my case.  I noticed the following:
for a link that started out life as:
[[note-filename#Target Header|Visible link Text]] - in obsidian
I ended up with a rendered link, in confluence that looks like this:
https://my.domain.net//wiki/spaces/NC/pages/6767676767#Target%20Header

There are two slashes after the domain, and before wiki in the path
There is not any slash after the pageid, and before the fragment (#)
The space is rendered still as %20  and not converted to a - as per confluence norms

When I fixed these manually the link worked, but not before

Thanks. I'll have another look. I should of known better than trying to just push a quick fix without testing it.

above might be related to https://github.com/markdown-confluence/obsidian-integration/issues/2


flash-player-debugger <IP_ADDRESS>6

Thank you for the contribution. It was merged directly as 01da46b72e52f2b6a3d81599d9f131fdc864d481 to keep commit history cleaner. Your contribution is still credited to you.