os: ReadDir fails for UNC paths shared with Windows using Parallels

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

$ go version
go version go1.21.0 windows/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
set GO111MODULE=
set GOARCH=amd64
set GOBIN=
set GOCACHE=C:\Users\hajimehoshi\AppData\Local\go-build
set GOENV=C:\Users\hajimehoshi\AppData\Roaming\go\env
set GOEXE=.exe
set GOEXPERIMENT=
set GOFLAGS=
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOINSECURE=
set GOMODCACHE=C:\Users\hajimehoshi\go\pkg\mod
set GONOPROXY=
set GONOSUMDB=
set GOOS=windows
set GOPATH=C:\Users\hajimehoshi\go
set GOPRIVATE=
set GOPROXY=https://proxy.golang.org,direct
set GOROOT=C:\Program Files\Go
set GOSUMDB=sum.golang.org
set GOTMPDIR=
set GOTOOLCHAIN=auto
set GOTOOLDIR=C:\Program Files\Go\pkg\tool\windows_amd64
set GOVCS=
set GOVERSION=go1.21.0
set GCCGO=gccgo
set GOAMD64=v1
set AR=ar
set CC=gcc
set CXX=g++
set CGO_ENABLED=1
set GOMOD=\\Mac\Home\unctest\go.mod
set GOWORK=
set CGO_CFLAGS=-O2 -g
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-O2 -g
set CGO_FFLAGS=-O2 -g
set CGO_LDFLAGS=-O2 -g
set PKG_CONFIG=pkg-config
set GOGCCFLAGS=-m64 -mthreads -Wl,--no-gc-sections -fmessage-length=0 -ffile-prefix-map=C:\Users\HAJIME~1\AppData\Local\Temp\go-build3663691949=/tmp/go-build -gno-record-gcc-switches

What did you do?

Prepare these files in a UNC directory starting with \\ like \\Mac\Home\unctest. Here, I assume the current directory is \\Mac\Home\unctest:
\\Mac\Home\unctest\go.mod:
module foo

go 1.21

\\Mac\Home\unctest\main.go
package main

import (
        "foo/bar"
)

func main() {
        bar.Hello()
}

\\Mac\Home\unctest\bar\hello.go
package bar

func Hello() {
        println("bar")
}

And run go run . at \\Mac\Home\unctest.
What did you expect to see?
bar is printed
What did you see instead?
Running the program fails:
PS Microsoft.PowerShell.Core\FileSystem::\\Mac\Home\unctest> go run .
GetFileInformationByHandleEx \\Mac\Home\unctest: The parameter is incorrect.

I have confirmed Go 1.20 worked:
PS Microsoft.PowerShell.Core\FileSystem::\\Mac\Home\unctest> go1.20.7 run .
bar

So, this is a regression.

I could minimize the test case:
\\Mac\Home\unctest\go.mod:
module foo

go 1.21

\\Mac\Home\unctest\main.go:
package main

func main() {
        println("hello")
}

and then run go run .:
PS Microsoft.PowerShell.Core\FileSystem::\\Mac\Home\unctest> go run .
GetFileInformationByHandleEx \\Mac\Home\unctest: The parameter is incorrect.

CC @bcmills maybe?

go run ./main.go

succeeded, and
go mod tidy

failed. I think using something related to a module fails when the current directory is a UNC path.

I wasn't aware that it was even possible for the current working directory to be a UNC path. (To my knowledge it still isn't possible with cmd.exe.) 😵💫
I suspect that this may be a side effect of https://go.dev/cl/452995 (attn @qmuntal).
(CC @golang/windows)

@hajimehoshi, does os.ReadDir(`\\Mac\Home\unctest`) succeed for that directory? What kind of filesystem is serving that path?

@bcmills

does os.ReadDir(\\Mac\Home\unctest) succeed for that directory?

Apparently no:
package main

import (
        "os"
)

func main() {
        _, err := os.ReadDir(`\\Mac\Home\unctest`)
        if err != nil {
                println("os.ReadDir failed!: ", err.Error())
        }
}

PS Microsoft.PowerShell.Core\FileSystem::\\Mac\Home\unctest> go run .\main.go
os.ReadDir failed!:  GetFileInformationByHandleEx \\Mac\Home\unctest: The parameter is incorrect.

What kind of filesystem is serving that path?

I'm using Parallels. The host is macOS and the guest is Windows. \\Mac\Home is a path for my home directory of macOS from Windows.

@gopherbot, please backport to Go 1.21. This is a significant regression in ReadDir.

UNC paths are available even without Parallels. For example: https://support.apple.com/en-ae/guide/directory-utility/diru31ab8054/mac

Makes sense. Do you happen to know whether your share is using SMB or AFP?

And, another question: if you mount the UNC path as a drive letter, and refer to it by the drive letter instead of the UNC path, does ReadDir still fail?

I'm not familiar with this, but smb, maybe?

if you mount the UNC path as a drive letter, and refer to it by the drive letter instead of the UNC path, does ReadDir still fail?

I'll try tomorrow.

Z: is the drive letter for the same hoe directory.
package main

import (
        "os"
)

func main() {
        if _, err := os.ReadDir(`\\Mac\Home\unctest`); err != nil {
                println("os.ReadDir with UNC failed!: ", err.Error())
        }
        if _, err := os.ReadDir(`Z:\`); err != nil {
                println("os.ReadDir with Z: failed!: ", err.Error())
        }
}

and
PS Z:\unctest> go run .\main.go
os.ReadDir with UNC failed!:  GetFileInformationByHandleEx \\Mac\Home\unctest: The parameter is incorrect.
os.ReadDir with Z: failed!:  GetFileInformationByHandleEx Z:\: The parameter is incorrect.

So, whether the path is UNC or not doesn't matter!

Thanks for the detailed report @hajimehoshi. I suspect that your file system doesn't support traversing directories using FILE_ID_BOTH_DIR_INFO, probably because it can't generate unique 8-byte unique file IDs.
I'll prepare a CL to fallback to FILE_FULL_DIR_INFO if FILE_ID_BOTH_DIR_INFO fails, which doesn't need a file ID.

@hajimehoshi can you test if CL 452995 fixes your issue?
You need to follow these steps to run the Go toolchain at CL 452995:
go install golang.org/dl/gotip@latest
gotip download 452995
gotip run .

@qmuntal This was not fixed unfortunately:
PS Z:\unctest> gotip version
go version devel go1.21-1951857ec0 Tue Jan 24 13:26:00 2023 +0000 windows/amd64
PS Z:\unctest> gotip run .
GetFileInformationByHandleEx Z:\unctest: The parameter is incorrect.
PS Z:\unctest> gotip run ./main.go
os.ReadDir with UNC failed!:  GetFileInformationByHandleEx \\Mac\Home\unctest: The parameter is incorrect.
os.ReadDir with Z: failed!:  GetFileInformationByHandleEx Z:\: The parameter is incorrect.

PS C:\Users\hajimehoshi> gotip download 518195
This will download and execute code from golang.org/cl/518195, continue? [y/n] y
Fetching CL 518195, Patch Set 2...
From https://go.googlesource.com/go
 * branch                  refs/changes/95/518195/2 -> FETCH_HEAD
HEAD is now at e17e4a5e3f os: support file systems without file IDs when reading directories on windows
ERROR: Cannot find C:\Users\hajimehoshi\Go1.4\bin\go.exe
Set GOROOT_BOOTSTRAP to a working Go tree >= Go 1.20.6.
Success. You may now run 'gotip'!

So, should I install go at ~/Go1.4...?

@qmuntal Yep, the patch seem to work now.
PS C:\Users\hajimehoshi> gotip download 518196
This will download and execute code from golang.org/cl/518196, continue? [y/n] y
Fetching CL 518196, Patch Set 4...
remote: Finding sources: 100% (47719/47719)
Receiving objects: 100% (47719/47719), 58.93 MiB | 6.33 MiB/s, done.
remote: Total 47719 (delta 31888), reused 42826 (delta 31888)
Resolving deltas: 100% (31888/31888), completed with 675 local objects.
From https://go.googlesource.com/go
 * branch                  refs/changes/96/518196/4 -> FETCH_HEAD
Warning: you are leaving 262 commits behind, not connected to
any of your branches:

  e17e4a5e3f os: support file systems without file IDs when reading directories on windows
  d9f7e1dc73 runtime: fix asan asm on amd64
  e6637f3293 os: test that copying to append-only files doesn't fail on Linux
  6c5fc6d7ce all: update vendored dependencies
 ... and 258 more.

HEAD is now at ddd112b9f7 [release-branch.go1.21] os: support file systems without file IDs when reading directories on windows
Building Go cmd/dist using C:\Program Files\Go. (go1.21.0 windows/amd64)
Building Go toolchain1 using C:\Program Files\Go.

Building Go toolchain2 using go_bootstrap and Go toolchain1.
Building Go toolchain3 using go_bootstrap and Go toolchain2.
Building packages and commands for windows/amd64.
---
Installed Go for windows/amd64 in C:\Users\hajimehoshi\sdk\gotip
Installed commands in C:\Users\hajimehoshi\sdk\gotip\bin
Success. You may now run 'gotip'!

PS Z:\unctest> gotip run .\main.go

@hajimehoshi could you check if os.SameFile returns true for two different files under \\Mac\Home\ using go1.21? I don't have a file system without file ID support myself, and I'm wondering if os.SameFile has always been broken in that situation.

@qmuntal This seems to work as expected, even with the current Go 1.21.0.
package main

import (
        "fmt"
        "os"
)

func main() {
        // \\Mac\Home\ and Z:\ are the same path.

        f1, err := os.Stat(`\\Mac\Home\unctest\main.go`)
        if err != nil {
                fmt.Fprintf(os.Stderr, "%v\n", err)
        }
        f2, err := os.Stat(`Z:\unctest\main.go`)
        if err != nil {
                fmt.Fprintf(os.Stderr, "%v\n", err)
        }
        fmt.Printf("os.SameFile(f1, f1): %t\n", os.SameFile(f1, f1))
        fmt.Printf("os.SameFile(f2, f2): %t\n", os.SameFile(f2, f2))
        fmt.Printf("os.SameFile(f1, f2): %t\n", os.SameFile(f1, f2))
}

Not patched:
PS Microsoft.PowerShell.Core\FileSystem::\\Mac\Home\unctest> go run ./main.go
os.SameFile(f1, f1): true
os.SameFile(f2, f2): true
os.SameFile(f1, f2): true

Patched:
PS Microsoft.PowerShell.Core\FileSystem::\\Mac\Home\unctest> gotip run ./main.go
os.SameFile(f1, f1): true
os.SameFile(f2, f2): true
os.SameFile(f1, f2): true

Oh two different files? Wait a sec.

package main

import (
        "fmt"
        "os"
)

func main() {
        f1, err := os.Stat(`\\Mac\Home\unctest\main.go`)
        if err != nil {
                fmt.Fprintf(os.Stderr, "%v\n", err)
        }
        f2, err := os.Stat(`\\Mac\Home\unctest\go.mod`)
        if err != nil {
                fmt.Fprintf(os.Stderr, "%v\n", err)
        }
        fmt.Printf("os.SameFile(f1, f1): %t\n", os.SameFile(f1, f1))
        fmt.Printf("os.SameFile(f2, f2): %t\n", os.SameFile(f2, f2))
        fmt.Printf("os.SameFile(f1, f2): %t\n", os.SameFile(f1, f2))
}

Not patched:
PS Microsoft.PowerShell.Core\FileSystem::\\Mac\Home\unctest> go run ./main.go
os.SameFile(f1, f1): true
os.SameFile(f2, f2): true
os.SameFile(f1, f2): false

Patched:
PS Microsoft.PowerShell.Core\FileSystem::\\Mac\Home\unctest> gotip run ./main.go
os.SameFile(f1, f1): true
os.SameFile(f2, f2): true
os.SameFile(f1, f2): false

Is this expected?

  GetFileInformationByHandleEx \\Mac\Home\unctest: The parameter is incorrect.
panic: runtime error: index out of range [0] with length 0

goroutine 1 [running]:
main.main()
        //Mac/Home/unctest/main.go:13 +0x605
exit status 2

Go 1.21 patched:
PS Microsoft.PowerShell.Core\FileSystem::\\Mac\Home\unctest> gotip run .\main.go
os.SameFile(f1, f1): true
os.SameFile(f2, f2): true
os.SameFile(f3, f3): true
os.SameFile(f4, f4): true
os.SameFile(f1, f2): true
os.SameFile(f1, f3): false
os.SameFile(f2, f3): false
os.SameFile(f2, f4): false

Thanks! There is clearly something weird in go1.21 that hasn't been fixed in CL 518196. I'll give it another pass to see what's happening.

Done. @hajimehoshi can you updated the patched go1.21 toolchain(run gotip download 518196), and try again the previous experiment?

PS Microsoft.PowerShell.Core\FileSystem::\\Mac\Home\unctest> gotip download 518196
This will download and execute code from golang.org/cl/518196, continue? [y/n] y
Fetching CL 518196, Patch Set 5...
remote: Finding sources: 100% (47720/47720)
Receiving objects: 100% (47720/47720), 58.92 MiB | 6.34 MiB/s, done.

Resolving deltas: 100% (31893/31893), completed with 675 local objects.
From https://go.googlesource.com/go
 * branch                  refs/changes/96/518196/5 -> FETCH_HEAD
Warning: you are leaving 1 commit behind, not connected to
any of your branches:

  ddd112b9f7 [release-branch.go1.21] os: support file systems without file IDs when reading directories on windows

HEAD is now at 7ba8a3b518 [release-branch.go1.21] os: support file systems without file IDs when reading directories on windows
Building Go cmd/dist using C:\Program Files\Go. (go1.21.0 windows/amd64)
Building Go toolchain1 using C:\Program Files\Go.

Building Go toolchain2 using go_bootstrap and Go toolchain1.
Building Go toolchain3 using go_bootstrap and Go toolchain2.
Building packages and commands for windows/amd64.
---
Installed Go for windows/amd64 in C:\Users\hajimehoshi\sdk\gotip
Installed commands in C:\Users\hajimehoshi\sdk\gotip\bin
Success. You may now run 'gotip'!
PS Microsoft.PowerShell.Core\FileSystem::\\Mac\Home\unctest> gotip run .\main.go
os.SameFile(f1, f1): true
os.SameFile(f2, f2): true
os.SameFile(f3, f3): true
os.SameFile(f4, f4): true
os.SameFile(f1, f2): false
os.SameFile(f1, f3): true
os.SameFile(f2, f3): false
os.SameFile(f2, f4): true

This seems to be fixed.


[bug/feature?] Keep focus in Neogit window after certain actions (e.g. branch change)

Description
Recently, Neogit started focusing windows other than Neogit's after certain actions, e.g. checking out a branch.
In no cases is this what I want: I usually close the Neogit window after performing actions there, so to close it, now I have to focus it again and then close it.
Suggestion:

Remove that refocusing behavior, defaulting to keeping the focus in the Neogit window
Alternatively (I wouldn't use this option) make this configurable

Neovim version
NVIM v0.9.1
Build type: Release
LuaJIT 2.1.0-beta3
system vimrc file: "$VIM/sysinit.vim"
fall-back for $VIM: "/__w/neovim/neovim/build/nvim.AppDir/usr/share/nvim"
Run :checkhealth for more info
Operating system and version
Debian sid
Steps to reproduce

Open Neogit in a split
Checkout some branch

Expected behavior
Focus remains in the Neogit window
Actual behavior
Focus is given to the other window, not Neogit's
Minimal config
-

Ahh, ok, so it looks like this occurs after leaving the telescope buffer, is that right?

Good catch, that's right. I tried another action, and even exiting the telescope buffer without confirming the action lands you on the other window.

Does this sort it out for you too? https://github.com/NeogitOrg/neogit/pull/690


Kubernetes as a docker orchestration platform

Kubernetes as a docker orchestration platform
Introduction (presentation)
Minikube
Kubectl
Deployment
Service
KubeAdm multi node cluster
Secrets
Ingress (Nginx vs Traefik)
Health checks

Wouldn't it be too much for attendees coming in to learn docker, to learn "a lot" about Kubernetes?

@KamranAzeem you think it's too much ?
What would you cut ?
@meekrosoft  you also think it might be too much, and we should stick with Docker Windows and prepare Kube in case we need it?

Yes. That is how I see it. From a student's perspective, it is too much. Perhaps Docker and Kubernetes could be two separate sessions on two separate days. Kubernetes is not a single machine or a single service. It involves a lot of concepts and there are a lot of moving parts. Students will be confused trying to figure out two technologies and may end up learning neither of them. For Docker orchestration, may be the following order and technologies could be used: Docker -> Docker-Compose -> Docker Swarm . When these three Docker related technologies are covered, (a good idea), then it would be a good idea to present Kubernetes as the orchestration solution highlighting weaknesses in Docker Swarm, and highlighting strengths in Kubernetes. But this can't be done in a single day. IMHO.

I agree with Kamran in sentiment, but we have announced that the course includes kubernetes so we should try to cover that in as simple a way as possible.  I could see that we should make a 1 day course on running applications in k8s but for now we can just show how to deploy, expose, and upgrade services on a kluster.  Preferably with pre-made yaml.

I have the material for this, so that's great. Then we can see how people receive it, and just take it from there. 😉 thanks @KamranAzeem and @meekrosoft

Closing this one. Further discussion can be made in the new repo: https://github.com/praqma-training/kubernetes-katas


Update my glowing review to use my preferred title

@ecmendenhall :cough: :cough:
idk why it failed for 1.9.3, it's just a README change. I swear it's not my fault.


MSBuild Analyzer not available in all build management tools

I saw that you've recently added CLI arguments for the experimental MSBuild Project Analyzer added with #1133, however it seems the setting is missing in all the other build management integrations. Is there a reason for this, other than unlikelyhood for projects using these build tools? The later wasn't a reason for other related settings (nuspec) to be included in e.g. maven and gradle plugins.

Missing configuration options have been added and will be included in the next release.

Missing configuration options have been added and will be included in the next release.


Host Ports description at Pod Security Standards contradicts spec for that field

This is a Bug Report
Problem:
The Host Ports baseline Pod Security Standard lists 0 as one of the allowed values, but the Container spec excludes the value 0. According to container spec, the hostPort must be in the range 0 < x < 65536.
Proposed Solution:
In the Host Ports baseline policy spec, either remove 0 from the allowed list , or add an explanation why 0 should be allowed even though it is not in the valid range.
Page to Update:
https://kubernetes.io/docs/concepts/security/pod-security-standards

Relates to https://github.com/kubernetes/website/issues/36938

/language en
/sig auth

@joebowbeer I think rather that removing 0 mentioning reason for it would be great for compatibility and consistency for the space. This would even clear the issue of why the value is used.

/triage accepted


Fill parent space on double click

Hi, I noticed that fill space on click does't work.
It is from demo https://mauricius.github.io/vue-draggable-resizable
Bonus: a component constrained in parent, can automatically fill the space just by double-clicking on it.

Thanks for reporting. Actually it's just a missing prop on the demo, not a bug of the component itself.
It has been fixed with fb66967.
Also note that this functionality has been removed from version 2, which is coming out in the near future.


[Bug]: The text has not been fully visible since the 3.4.0 update, while using card_mod

Requirements

[X] I checked the troubleshooting section in the README to verify that I have the latest Mushroom version.
[X] I did a search to see if there is a similar issue or if a pull request is open.

Current Behavior
The text in the Mushroom Template Card, when used in conjunction with card_mod, is not visible across the entire card area.

Expected Behavior
Before the upgrade (and after downgrade to 3.3.0) the text is fully visible again.

Steps To Reproduce
Create a Mushroom Template Card with this config:
type: custom:mushroom-template-card
primary: '{{ now().strftime(''%H:%M'') }}'
icon: mdi:timelapse
secondary: '{{ now().strftime(''%d %B'') }}'
card_mod:
  style: |
    ha-card {
       --card-primary-font-size: 60px;
       --card-secondary-font-size: 30px;
    }
    :host {
       --mush-icon-symbol-size: 1.75em;
       --mush-icon-size: 2.75em;
    }

Context
YAML state

Environment
- Browser: Firefox, Android app
- HA Version: 2024.1.6
- Mushroom: 3.4.0
- Card-Mod: 3.4.2

Anything else?
No response

I can confirm this behavior. In 3.4.0, card size does not seem to increase with font size, like it did previously. This causes larger text to get clipped.

It looks like the line height is messed up. I worked around it by adding explicit line height to each card but this is not ideal...
When inspecting you can see the line height is not right

In the inspect code I see these are set regardless of font size
--card-primary-line-height: var(--mush-card-primary-line-height, 20px);
--card-secondary-line-height: var(--mush-card-secondary-line-height, 16px);

with --card-primary-line-height: 1;

I can confirm that adding these two lines solves the issue:
       --card-primary-line-height: 1;
       --card-secondary-line-height: 1;

This is the entire code:
type: custom:mushroom-template-card
primary: '{{ now().strftime(''%H:%M'') }}'
icon: mdi:timelapse
secondary: '{{ now().strftime(''%d %B'') }}'
card_mod:
  style: |
    ha-card {
       --card-primary-font-size: 60px;
       --card-secondary-font-size: 30px;
       --card-primary-line-height: 1;
       --card-secondary-line-height: 1;
    }
    :host {
       --mush-icon-symbol-size: 1.75em;
       --mush-icon-size: 2.75em;
    }

I can also confirm that the issue still persists in version 3.4.1

Hi 👋 Some theme variables have been changed to use the default font style of tile card (https://github.com/piitaya/lovelace-mushroom/pull/1348/files#diff-47d2527b2c38615d2cd4f5c1c06a18c0bdb9b66519e468bba99344b5d9cadd6fR19-R26)
Mushroom support with card mod is not guarantee because it's not officially supported but you can find help in the community forum
I close this issue as card mod support is not provided in this repo.

I can confirm that adding these two lines solves the issue:
       --card-primary-line-height: 1;
       --card-secondary-line-height: 1;

This is the entire code:
type: custom:mushroom-template-card
primary: '{{ now().strftime(''%H:%M'') }}'
icon: mdi:timelapse
secondary: '{{ now().strftime(''%d %B'') }}'
card_mod:
  style: |
    ha-card {
       --card-primary-font-size: 60px;
       --card-secondary-font-size: 30px;
       --card-primary-line-height: 1;
       --card-secondary-line-height: 1;
    }
    :host {
       --mush-icon-symbol-size: 1.75em;
       --mush-icon-size: 2.75em;
    }

I can also confirm that the issue still persists in version 3.4.1

Prior to v3.4.0, the value for line-height was 1.5. So if you're just trying to "undo the changes" you can add the below to your theme or card-mod-theme or however you're doing it, and you won't have to change 1000 lovelace cards.
--card-primary-line-height 1.5;
--secondary-line-height: 1.5;


Update scripts to work on Cygwin and Git Bash

The build-util scripts, when cloned in repos for Windows, don't run in Cygwin.  Need to be able to run in both.

Added line to top of scripts to allow running on Linux and Git Bash.  Close the issue.


Current % amount ain't accurate to what is actually achieved in game.

Describe the Bug
Counters+ is not accurate with current amount of % (Also if you haven't cleared a song before your "previous" score is far more accurate then the current amount)
Attach or Link Files
Config (https://pastebin.com/hnA4N4rT)
Probable Culprit
The cause of this is genuinely unknown to me because it used to work just fine and be accurate to the decimal.
Screenshots or Video

(I didn't improve my score afterward I swear it's just a really odd bug)
Talking about this counter to make it clear.

Additional context
Add any other context about the problem here.

Fixed in Counters+ 1.9.1


Shift primary setup script into VSConfig file

We should be able to shift this into a pretty quick and easy VS Config file.  By keeping this up-to-date, we can update / add dependencies and everyone will be notified versus getting getting latest and something not working all of a sudden

{
  "version": "1.0",
  "components": [
    "Microsoft.VisualStudio.Component.CoreEditor",
// .....
    "Microsoft.VisualStudio.Component.VC.Runtimes.x86.x64.Spectre"
  ]
}

This has been resolved in the 0.35 release. https://github.com/microsoft/PowerToys/releases/latest


Short solution needed: "Using location with regex" (nginx)

Please help us write most modern and shortest code solution for this issue:
Using location with regex (technology: nginx)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments to make solution explained.
Link to this issue in comments of pull request.

Using location with regex


New rich text editor#830

Fixes #830
I completely removed old mindmup/bootstrap-wysiwyg
Because this fork and original editor are not supported
Implemented new Trumbowyg -  lightweight WYSIWYG editor
That one is implemented using yarn not hard-coded, I made custom styling to match platform theme
Using editor in forms is very easy
# in place where it should be rendered
<%= render :partial => 'editor/index', :locals => {:content => @mail.content} %>

// in place where form is submitted
$("#trumbowyg-editor").cleanHtml()

Before:

After:

This is breaking button backgrounds across cv as
button:not(.trumbowyg-disable):hover,
button:not(.trumbowyg-disable):focus,
button.trumbowyg-active {
  background-color: inherit !important;
  outline: none;
}

Is applicable to every other divs,

This is breaking button backgrounds across cv as
button:not(.trumbowyg-disable):hover,
button:not(.trumbowyg-disable):focus,
button.trumbowyg-active {
  background-color: inherit !important;
  outline: none;
}

Is applicable to every other divs,

I noticed that too, will be removing the !important tag in the new pr, I don't think it will cause any problem for the text editor.

Made a hotfix here, https://github.com/CircuitVerse/CircuitVerse/commit/889095c29b5bb6229ec9db489ae9faa630287437 Need proper fix as well

@tachyons that is working perfectly fine after this hotfix, no need for further fixes, thanks!


Feature/credx issuer

This PR completes the credx anoncreds implementation by adding proper handling for issuer operations. For tests, the mixed_breed feature flag was added (with teary eyes, since we want to get rid of them not add more) so that a new profile can be constructed.
The mixed breed profile uses the credx anoncreds impl and the indy ledger and wallet impls.

Codecov Report

Merging #854 (80394ad) into main (0ef8430) will decrease coverage by 0.31%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #854      +/-   ##
==========================================
- Coverage   48.95%   48.65%   -0.31%
==========================================
  Files         415      415
  Lines       33784    33995     +211
  Branches     7493     7585      +92
==========================================
  Hits        16539    16539
- Misses      12107    12318     +211
  Partials     5138     5138

Flag
Coverage Δ

unittests-aries-vcx
48.61% <0.00%> (-0.31%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

aries_vcx_core/src/anoncreds/credx_anoncreds.rs
36.39% <0.00%> (-17.58%)
:arrow_down:

aries_vcx_core/src/indy/credentials/issuer/mod.rs
61.11% <ø> (ø)

... and 4 files with indirect coverage changes

looking good to me so far, pls tag me once Draft is removed

@gmulhearn Ping

@bobozaur I guess one more rebase needed, plus consider the one comment I've left


Port to 1.19.2 - Forge 43.3.0

Hi, first thanks for this great mod, i ported the mod to 1.19.2 version, hope this helps.
Regards,
F0x06
PS: Attached a zipped .jar for people to test/use
jeiintegration_1.19.2-9.0.0.zip

Hi is there gona be a 1.20.1 Port?😅

Hi is there gona be a 1.20.1 Port?😅

Now I'm playing on 1.19, but should be easy to port (if it's like 1.19)

Hi is there gona be a 1.20.1 Port?😅

Now I'm playing on 1.19, but should be easy to port (if it's like 1.19)

I actually hqve no Clue how it works and if it is the same

Hi is there gona be a 1.20.1 Port?😅

Now I'm playing on 1.19, but should be easy to port (if it's like 1.19)

I actually hqve no Clue how it works and if it is the same

Done ;) https://github.com/SnowShock35/jei-integration/pull/40

Hi @SnowShock35, when did you plan to release it on CurseForge :) ?
Thanks

Hi @SnowShock35, when did you plan to release it on CurseForge :) ?
Thanks

Before 2024. If you need to use it before that simply clone the mod and build it locally. 😃

Before 2024. If you need to use it before that simply clone the mod and build it locally. 😃

Hi @SnowShock35
May I ask if the 1.19.2 version will be soon on Curseforge ?
Also, thanks @F0x06 for the two ports


[WIP] api/services/content: define the content service

Bring the content service into the containerd API. This allows the
content store to be coordinated in the containerd daemon with minimal
effort. For the most part, this API follows the conventions and behavior
of the existing content store implementation with a few caveats.
Specifically, we remove the object oriented transaction mechanism in
favor of a very rich Write call.
Pains are taken to reduce race conditions around when having multiple
writers to a single piece of content. Clients should be able to race
towards getting a write lock on a reference, then wait on each other.
For the most part, this should be generically pluggable to allow
implementations of the content store to be swapped out.
We'll follow this up with an implementation to validate the model.
Signed-off-by: Stephen J Day<EMAIL_ADDRESS>

Codecov Report

Merging #520 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #520   +/-   ##
=======================================
  Coverage   37.39%   37.39%
=======================================
  Files           5        5
  Lines         706      706
=======================================
  Hits          264      264
  Misses        380      380
  Partials       62       62

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 a891727...e1f8e50. Read the comment docs.

@crosbymichael While there are few commands that need to be ported over to the GPRC API, this PR is in a solid state to merge.

LGTM


[ONNX][Relay][Op] Enable Grouped Conv1d

We just need to have the type relation support it and it seems to run on llvm target.
Need to check CUDA still.

Thanks for doing this Andrew. Any chance you could do a quick check on the performance before and after? I worry that without group conv1d schedule support in topi, this will result in a regression.

Superseded by https://github.com/apache/tvm/pull/9832


Indent error with terminal

VSCode Version: Code 1.12.1 (f6868fce3eeb16663840eb82123369dec6077a9b, 2017-05-04T21:26:50.689Z)
OS Version: Windows_NT ia32 10.0.15063

I've got this problem that always some spaces in front of every line. When I change the terminal from powershell to cmd and git-bash, there's still this problem.

Same problem with MSys2

VSCode Version: Code 1.12.1 (f6868fce3eeb16663840eb82123369dec6077a9b, 2017-05-04T21:26:50.689Z)
OS Version: Windows_NT ia32 10.0.15063

Duplicate https://github.com/Microsoft/vscode/issues/19665


Add possibility to pass extended waterline models

It's possible to add extended waterline models to dogwater ?

It currently isn't possible to do that.  Can you explain your use-case?  I'd consider a PR for the feature if it seems like a common pain-point.

@throrin19 closing this, but we can reopen if you'd like to continue the conversation.


harmonize RXNO object properties with RO

There are object properties (aka relations) reused in RXNO that are defined in BFO. As the OBO community uses the BFO2.0 classes only version these relations are not being resolved via their IRI. In order to be compliant with the OBO principles these relations should thus be replaced by RO equivalents.
The following relations need to be changed in RXNO (individual issues need to be made for each):

“has dispositions at all times” (BFO_0000162) should be changed to “has disposition” (RO_0000091)
‘has participant’ (BFO_0000057) should be changed to ‘has participant’ (RO_0000057)

its sup-properties 'has participant beginning to exist' (BFO_0000077) & 'has participant ceasing to exist' (BFO_0000078) should be changed to something suitable in the ‘has participant’ branch of RO, or a term request should be made

a possible candidate for 'has participant ceasing to exist' might be "has primary input" (RO_0004009)
a possible candidate for 'has participant beginning to exist' might be "has primary output" ([RO_0004008]
the remaining relations ('deprotects', 'protects', 'has specified product', 'has specified reactant', 'has_catalyst' and 'has_intermediate' need to be correctly subsumed then as well

(http://purl.obolibrary.org/obo/RO_0004008))
“has_part" and "part_of" should be replaced with "ro:has part" (BFO_0000051 ) & "ro:part of" (BFO_0000050)

moved to upstream repo: https://github.com/rsc-ontologies/rxno/issues/30


(docs) Minor fixes to documentation

These fixes include an incorrect documentation of how to override built-in cake tasks, and a minor fix to how the GitHub button is handled.

@AdmiringWorm your changes have been merged, thanks for your contribution 👍


Dates become disabled when toggling month

Description
Issue seems to only occur when you have a picker that is always visible, has a single day selection and starts with a default value (see code sample). Our use case is that the user may select the date, progress through the booking process then go back to change the date.
Code sample
<HotelDatePicker
    :alwaysVisible="true"
    :singleDaySelection="true"
    :startingDateValue="new Date('2021-06-14')"
/>

Expected behavior:
Picker defaults to startingDateValue and user is able to select a date from a different month.
Actual behavior:
Picker defaults to startingDateValue but when the user toggles the month (back or forward) all other dates become disabled.
Datepicker Version
4.0.0-beta.9

May check the issue in beta 10?

I will check this issue before release the next beta (14)

I have tested and replicated the problem on beta 15.

I have tested and replicated the problem on beta 15.

Fixed in v4.0.0-RC1

Fixed in v4.0.0-RC1


Add timestamps extension

Description

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Hi @pernielsentikaer, thanks for showing me the built-in timestamp features. Unfortunately there are some subtle differences between the extension in this PR and the built-in timestamp helpers:

I want the timestamps to be in milliseconds, the built-in is only in seconds.
I want the extension to insert the current timestamp directly to where the cursor is, the built-in only copies it to the clipboard
When converting a timestamp to date, I want to see the timezone in my locale, something like Thu Apr 11 2024 10:27:07 GMT-0600 (Mountain Daylight Saving Time), not the hard to read format like February 23, 56248 at 05:05:42 GMT-07:00 from the built-in
When converting a timestamp to date, I want to insert the date directly to the cursor location, the built-in only copies it to the clipboard
The built-in doesn't offer me to insert today's date in the format like 2024-04-11 Thu

There is already a lot of extensions doing date formatting in the store, maybe it would be better to enhance one of them if the feature is not already present - most of the things can be done with snippets too

You can setup a snippet with this content {date format="A"} - that should give you time in milliseconds
With the snippet it's possible to expand with a keyword like !ms
A snippet with {date format="yyyy-MM-dd E"} should give you this output


Malloc issues with adl applications (bbcbasic24)

With the 2.2.0 release it was found that *dir did not work on a freshly started system that booted straight into ADL BASIC.
Further investigation shows that this is an issue with the use of malloc - it seems that malloc will fail, and since DIR had no fallback it would just fail completely.
A fix has been made as part of #46 to allow dir to work in the event of the malloc failing, however this does not address the underlying issue.  malloc still fails, this is just being detected, and a fallback to a different non-sorted, and non-coloured, directory display is used instead.
Another piece of functionality that does not work from inside of BASIC24 is the *hotkey command, as that too uses malloc.
Curiously, leaving BASIC via *bye, doing a *dir, and then loading up BASIC again and running it resolves the issue, and after doing so the new version of *dir that shows a sorted directory with colours will work.

we have a solution to this released as part of 2.2.2
the fix was to perform a malloc(6502); during startup to grab a chunk of memory, which is then immediately freed.  including that seems to allow later calls to malloc work inside MOS even when an ADL application has been started which may have changed the stack pointer on the processor


Confusing wording

"Serverless SQL pool lets you query files in your data lake in read-only manner, while SQL pool lets you ingest data also."
Clients are confused by whether Serverless SQL pool has ingestion capability or not. It looks conflict with this page https://docs.microsoft.com/en-us/azure/synapse-analytics/sql/best-practices-sql-on-demand#general-considerations.

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

ID: 224a5d25-3715-3150-af60-0fd273ad19d9
Version Independent ID: 13584abc-bdae-c88e-73ab-27070f94359c
Content: Synapse SQL architecture - Azure Synapse Analytics
Content Source: articles/synapse-analytics/sql/overview-architecture.md
Service: synapse-analytics
Sub-service: ****
GitHub Login: @mlee3gsd
Microsoft Alias: martinle

@moatwork
Thank you for reaching out and bringing this to our notice. At this time we are reviewing the feedback and will provide an update as appropriate.

@moatwork
Thanks for the feedback! We have assigned the issue to the content author to investigate this and update the document as appropriate.

#assign:julieMSFT

Hi @moatwork, thank you for contacting Microsoft. I have updated the document with the following:
"Serverless SQL pool allows you to query your data lake files, while dedicated SQL pool allows you to query and ingest data from your data lake files." You will see this update soon. Thank you for your contribution to MicrosoftDocs.  I will go ahead with closing this issue. The @notification mechanism continues to work even after an item is closed, in case further communication is needed. We hope to hear from you again.

#reassign:julieMSFT

#please-close

#reassign:julieMSFT

This page (along with many others) needs reviewing by a native English speaker, as the various points being made are getting lost or garbled.  For example, the text "Task is distributed query execution unit, which is actually part of query user submitted" needs rewording. In general, there is a regular absence of definite/indefinite articles (i.e. "a" or "the") throughout.


Add an Article for Day 17 of 2023

This has a little errata. See https://github.com/scalacenter/scala-advent-of-code/pull/514
I'll handle it when I have a sec

Throwing out ideas, we do have this requirement of part1 and part2 not using IO to load the input so its scala.js compatible but perhaps we can add an extra section to each page that covers how to load the input from file that then calls the part1, part2 methods? it could even be tabbed for scala.js/scala native/jvm

@bishabosha Thanks! I like the change removing global vars with Using Grid. I'll use that pattern in future submissions (and day 23, which I'm writing up now)
re: not using IO: Yes! I think that's a great idea. Should we make it work with pastable input in 2024?

re: not using IO: Yes! I think that's a great idea. Should we make it work with pastable input in 2024?

That's what the solver widget is meant to be if I take the time to set them up


prevent applying a theme multiple times to the same control

I added ConditionalWeakTable that tracks the Controls already processed.
otherwise, in my quite complex MDI app, Controls and their object trees get processed multiple times
Should have no side effect.

Thanks.


 The expression syntax is the single letter 'd' followed by a positive integer value representing the number of sides on that die (the maximum value). When evaluating, the result is a random number between 1 and the largest value on the die inclusively.
Special Cases
There are a couple special case dice values.

 Moved to #5

These special cases should be implemented by implementing the IDie interface with NumericalDie, PercentileDie, and FateDie.
Evaluate
The IDie should evaluate to a RollResult that contains a list of IExpression results and the IDie that generated those results. The list would only contain a single value for the three types of dice listed in this issue, but for future items, such as #2 Dice, multiple results might be generated. The RollResult is an implementation of IEnumerableExpression and IDie.

Reopened because parsing wasn't completed


Parsing Artifacts fails with "Unable to validate information", Parsing Material fails on random items with "Failed to parse quantity"

There appears to be some problems parsing various stats.
I am running version 1.2.4, with the game in a windows 1600x900 (16:9 Aspect)
Artifact parsing fails with "Unable to validate information", this happens with random artifacts, as if there's something it doesn't recognise. A few examples below (taken from the logging folder created by the app):
Unable to validate information for artifact ID#27

Invalid artifact sub stats

Artifact ID: 27

Slot: goblet

Set: ResolutionOfSojourner

Rarity: 3

Level: 1

Main Stat: electro_dmg_

Substat 2: enerRech_ + 3.1%

Locked: False

Equipped character: YunJin

Unable to validate information for artifact ID#29

Invalid artifact sub stats

Artifact ID: 29

Slot: circlet

Set: PrayersForWisdom

Rarity: 3

Level: 0

Main Stat: atk_

Locked: False

Equipped character: KujouSara

It seems like it is not recognising one or more of the substats correctly. For example, it only picked up "Substat 2" on the first one, but totally missed both substats on the second one.

For Materials, the app seems to fail to parse some of the quantities, but they seem clear enough. Some examples below (again, taken from the logging folder in the app):
Failed to parse quantity for MistGrass

Failed to parse quantity for AgentsSacrificialKnife

Failed to parse quantity for EverflameSeed

Failed to parse quantity for BrilliantDiamondChunk

Also have the same error for some materials quantities. Running 1.2.4 with 1600x900 windowed.
Don't have any 3* or 4* artifacts so didn't scan it, but 5* seem to have no issue logged, only that it does not seem to finished them all.

Full log here
https://pastebin.com/0MqY6TFN
Examples.
Failed to parse quantity for BasaltPillar

Failed to parse quantity for GuideToDiligence

Failed to parse quantity for NarukamisValor

Failed to parse quantity for BitterPufferfish

Failed to parse quantity for NorthlanderPolearmBillet

Hi there, V1.2.5 has been released. Hopefully fixes in this version fix your problem. Please let us know if they're resolved, thanks!

Unfortunately, it did not fix the issue. Worse, the application exits abnormally during scanning of the artifacts with the following in the logging folder:
2022-05-16 20:41:01.447|INFO|UserInterface|Parsed 452 artifacts to scan
2022-05-16 20:41:05.946|ERROR|UserInterface|Unable to validate information for artifact ID#19
2022-05-16 20:41:05.946|ERROR|UserInterface|Invalid artifact sub stats
Artifact ID: 19
Slot: flower
Set: TravelingDoctor
Rarity: 3
Level: 12
Main Stat: hp
Substat 1: critDMG_ + 3.3%
Substat 3: hp_ + 3.5%
Substat 4: critRate_ + 3.5%
Locked: True

Since it seems to have crashed out, there's nothing logged. Switching on "Log all screenshots" does add all the screenshots, but in the above run, there's no artifact19 folder.
Also, I noticed the box sizes for the artifact captures seem a bit off -- are they supposed to cover the entire artifact box?

v1.2.6 still has the issue.

v1.2.7 still has the issue.

This thread has a duplicate description of #96
The issue about artifact information not being validated exists for low rarity has an equally low priority of being addressed.

Recent versions have improved material quantity scanning. If you are still having invalid artifact errors please create a new issue.


a question about dcgm policy listening for xid

If I register an XID through DCGM's policy and listen, when a certain XID (for example, 79) occurs, will the policy keep reporting that XID until it recovers, or will it only report it once?
I look forward to your reply

@BetaZYN,
It depends on how you read the XIDs. Each XID event is stored with its timestamp, and there is an API to get either the latest value in the TSDB or values since a specific timestamp. The dcgmi cli tool uses only the last value in the TSDB, so it may look like a "sticky" XID until another XID is reported. If you use the API directly, you may get all XIDs that happened within the last minute, for example.
Currently, the DCGM version can't report XID 79, 119, and 120 due to limitations in the NVML library. Our team is working to fix this.

@nikkon-dev ,
Thank you for your reply.

Which specific API are you referring to that can get xid and timestamp?
Our current use case is as follows:
// set group 2 policy condition with XID errors
dcgmi policy -g 2 --set 0,0 -x
// register group2 for policy updates
dcgmi policy -g 2 --reg
If a GPU generates an XID during listening, will this XID be repeatedly reported until a new XID appears or until this XID disappears?


Delphi 10.4 Support

Any plan to support Delphi 10.4?

it's under process :)

Great!
On Sun, Jun 7, 2020 at 15:40 Zeus64<EMAIL_ADDRESS>wrote:

it's under process :)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Zeus64/alcinoe/issues/132#issuecomment-640276589, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABBCWRX42PEKTUFLXB64KBTRVP3KHANCNFSM4NWM7ABQ
.

--
German Pablo Gentile
CEO Fundador.
PetroBox Internacional
https://www.petroboxinc.com
https://www.facebook.com/PetroBox
https://www.facebook.com/genartech/
(HQ Argentina)
(5411)    5983-9419
(Call Center - Bolivia)
(591)      3-388-8688
(591)      71320871
(Colombia)
(57)       1-5087762
(57)       2-8912486
(57)       2-8912544
(Panama)
(507)     8-365778/339634
(Peru)
(51)       1-706-2901
skype(VoIP)    germanpablogentile

Stephane, to try to help I have already remade / revised all the fixes and also made some minor adaptations to your source (just in files that I need). I spent the last 3 days testing and it's working perfectly on iOS64, Android32, Android64, Win32 and Win64.
Some observations:

I've never seen the source so changed in years, but almost no fixes have been resolved.
The NEXTGEN is not defined more, you can put this at the beginning of the file:{$IF CompilerVersion >= 34} {Delphi 10.4 Sydney}
  {$IF not defined(MSWINDOWS) and not defined(LINUX)}
    {$DEFINE NEXTGEN}
  {$ENDIF}
{$ENDIF}

The alcinoe does not support the metal, but we will not have problem with this because it is disabled by default.

alcinoe.zip

Another thing, I think the fixes are more organized the way I did.

Note: The TWebBrowser is crashing in the 10.4. The bug comes from original source, not from the fixes. Once I have time, I will investigate a possible workaround.

Note: The TWebBrowser in iOS is crashing in the 10.4. The bug comes from original source, not from the fixes. Once I have time, I will investigate a possible workaround.

I found the workaround:
https://en.delphipraxis.net/topic/2885-your-rad-studio-104-sydney-issues/?do=findComment&comment=23741

Ar you going to remove the IOS native code in favour of metal ?

Ar you going to remove the IOS native code in favour of metal ?

I have to tell you, there will be no visible performance difference for non game apps, at most 2~5% on iOS, as currently firemonkey is already rendered on the GPU with opengl. However on OSX that was not drawing on the GPU, the gain is around 50%.

I measured on OSX even more improvement.
The gain would be mostly maintenance.

Op 21 aug. 2020 om 02:28 heeft Vinícius Felipe Botelho Barbosa<EMAIL_ADDRESS>het volgende geschreven:

Ar you going to remove the IOS native code in favour of metal ?
I have to tell you, there will be no visible performance difference for non game apps, at most 2~5% on iOS, as currently firemonkey is already rendered on the GPU with opengl. However on OSX that was not drawing on the GPU, the gain is around 50%.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

how long, "it's under process" ?
Could you at least roughly orientate when to expect a release for 10.41? I wanted to test it, but I can't run it on 10.4 on my own. straight trouble ..

Please update, support 10.4.1
[DCC Error] ALGraphics.pas(577): E2010 Incompatible types: 'Boolean' and 'Integer'
CGContextSetShouldAntialias

Temporary correction for IOS in Delphi 10.4.1
source.zip

I hope this will happen as soon as possible
And the installation is easier than before
Why not a batch or executable file that publishes everything

full garbage.
or close your project or do normal support.
some kind of mockery. the author is probably waiting for a purchase offer from the embarcadero;)
I think it will wait as long as we support 10.4 :))))

I m sorry :( I understand your frustration! It's just that I m totally under the water right now and the restriction du to the COVID do not help me :( to migrate to 10.4. it's cost in general around one week and believe me right now I don't have one week :( I will do my best to do it as soon as possible, maybe with the 10.4.2. this exceptional situation again sorry :(

@firemen01  this is an open source project, you have no right to say that, if you want a service done in time, then pay. you could higher a developer to do this task for you or you do it your self.
this is the first time I see someone complaining about support on an opensource project.
that is why companies buy commercial licenses from opensource projects, they buy them for support.

Well, that came out 10.4.2, how long do you have to wait? half a year? year? ..

@firemen01 , gays this is opensource project... you can't force developer to finish your target... so if you want fast you can try android studio web WebRTC this easy to make like video conference... or you can buy from flashvconverter.com this real webrtc for delphi component

I use the Alcinoe controls that I need normally on Delphi 10.4.1 and I probrally will use on 10.4.2. It's open. Get the code and write if you need in time. Otherwise, wait the community develop for you or pay for that.

How can i pay for that

@okbach  look in the project readme file, apparently they are offering their services send them an email.

Sorry it's was a long way :( The source code is now compatible with 10.4.2


[zh-cn] Improve feature gate StatefulSetAutoDeletePVC description

sync content/en/docs/concepts/workloads/controllers/statefulset.md
en PR: https://github.com/kubernetes/website/pull/37632

/lgtm
/approve
good job


npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed

Problem
I'm trying to install react-native-maps by Airbnb in my project. Installation instructions: airbnb/react-native-maps/installation
when I run the following command in cmd: npm install react-native-maps --save
This is what I get:
`--<EMAIL_ADDRESS>npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
Ignoring this if I proceed with further installation, and run the following command: react-native link react-native-maps
This is what I get:
Scanning 561 folders for symlinks in C:\RNProjects\Mapp\node_modules (39ms)
rnpm-install info Linking react-native-maps ios dependency
rnpm-install info iOS module react-native-maps has been successfully linked
^As we can see, only iOS related maps modules get installed. On checking the project directory, I was able to validate that no android related module was installed.
Troubleshooting
Here's the list of thing I tried to resolve the issue, but none of them worked.

Tried installing npm install<EMAIL_ADDRESS>--save
Got these warnings:
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
So, in order to resolve this, I tried installing<EMAIL_ADDRESS>with: npm install<EMAIL_ADDRESS>I got the same message again:
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.

Tried updating react-native npm update react-native.
Still getting the same warning message.

Tried installing the latest version of npm: npm install npm@latest -g.
Didn't work.

Additional Information

react-native: 0.43.3
react-native-cli: 2.0.1
Platform: Android
Development Operating System: Windows 10 OS

Dont worry , If peer dependencies aren't matched this throws unmet peer dependency error . But it will work . I too had the same issue .

@PARAGJYOTI
If I Ignore this and continue then there are two major issues.

After running the following command: react-native link react-native-maps,
I get the following in cmd:
Scanning 561 folders for symlinks in C:\RNProjects\Mapp\node_modules (39ms)
rnpm-install info Linking react-native-maps ios dependency
rnpm-install info iOS module react-native-maps has been successfully linked
^As we can see, only iOS related maps modules get installed. On checking the project directory, I was able to validate that no android related module was installed.

As mentioned here: airbnb/react-native-maps/installation/android
In the first instruction itself (In your android/app/build.gradle add: ..dependencies..) it says: This step is not necessary if you ran "react-native link react-native-maps"
But when I open the following gradle file using my editor, it appears that I do have to add these lines (stating that the android-related-modules of react-native-maps were not linked). When I add these lines manually, and run project, it throws errors.

Besides gradle you have to look for MainApplication.java . If MapsPackage() is included in the package there must be some problem in the version . I just upgraded to 0.14.0 and getting some error like Configuration with name default not found. It may be issue with the version . Try installing 0.12.5

@PARAGJYOTI  did u find any solution with new version?? 0.12.5 works fine

Same problem here!

@PARAGJYOTI @iiitmahesh @pooya1361 Can someone please guide me on how to install the react-native-maps 0.12.5 version. I'm very new to the react-native. Thanks.

Don't worry guys , its totally normal to get a lots of errors during compiling with react-native run-android if you are new to react-native . I will guide you as far as I can .
There are some basic  things to be look carefully before you compile your app especially if you are beginner .
1) When you try to add a new library that's based on java into your app , they will tell you either to run react-native link some-package . What it does is simply adds some lines of codes to 3 different files ,
First to the settings.graddle file , which contains the public path to the library in your node-modules folder . It somewhat looks like this .
       ` include ':react-native-maps'
         project(':react-native-maps').projectDir = new File(rootProject.projectDir,'../node_modules/react
          native-maps/android')

          include ':react-native-fcm'
          project(':react-native-fcm').projectDir = new File(rootProject.projectDir, '../node_modules/react
          native-fcm/android') `

So , if you run react-native link react-native-maps it will automatically add the codes to settings.graddle .
Again , 2nd is to the build.graddle file which in in the  app  directory , like  yourapp/android/app/build.graddle . Its the most important file that contains all build instructions to graddle .
I am pasting my entire (almost) build.graddle  script for your better understand .
` android {
   compileSdkVersion 25
 buildToolsVersion "23.0.1"

    dexOptions {
  jumboMode = true
      }

defaultConfig {
    applicationId "com.shopfort"
    minSdkVersion 16
    targetSdkVersion 22
    versionCode 1
    versionName "1.0"
    ndk {
        abiFilters "armeabi-v7a", "x86"
    }
}
	signingConfigs {
    release {
        storeFile file(MYAPP_RELEASE_STORE_FILE)
        storePassword MYAPP_RELEASE_STORE_PASSWORD
        keyAlias MYAPP_RELEASE_KEY_ALIAS
        keyPassword MYAPP_RELEASE_KEY_PASSWORD
    }
}

splits {
    abi {
        reset()
        enable enableSeparateBuildPerCPUArchitecture
        universalApk false  // If true, also generate a universal APK
        include "armeabi-v7a", "x86"
    }
}
buildTypes {
    release {
        minifyEnabled enableProguardInReleaseBuilds
        proguardFiles getDefaultProguardFile("proguard-android.txt"), "proguard-rules.pro"
		 signingConfig signingConfigs.release
        }
   }

   }
 }
      dependencies {
     compile project(':react-native-vector-icons')
     compile project(':react-native-spinkit')
     compile project(':react-native-share')

     compile project(':react-native-linear-gradient')

      compile(project(':react-native-fcm')){
  exclude group: "com.google.firebase"
  }
  compile(project(':react-native-maps'))
   {
    exclude group: 'com.google.android.gms'
     }

  compile ("com.google.firebase:firebase-core:9.8.0"){
    force = true;
  }
compile ('com.google.firebase:firebase-messaging:9.8.0'){
   	    force = true;
}

compile ("com.google.android.gms:play-services-base:9.8.0") {
    force = true;
     }
   compile ("com.google.android.gms:play-services-maps:9.8.0") {
    force = true;
    }

   compile ('com.google.android.gms:play-services-gcm:9.8.0') {
    force = true;
    }

 compile project(':react-native-fbsdk')
compile project(':react-native-camera')
compile project(':lottie-react-native')
compile project(':gl-react-native')
compile fileTree(dir: "libs", include: ["*.jar"])
compile "com.android.support:appcompat-v7:23.0.1"
compile "com.facebook.react:react-native:+"  // From node_modules
 }
apply plugin: 'com.google.gms.google-services'`

##CompileSdk
So , Lets start with the first part CompileSdk version says with which version of sdk you want to compile the Scripts . Mine is 25 , note that its specially for react-native-maps .  Some days ago , airbnb implicitely  said to use compileSdk version 25 , but probably they removed it now from the documentation , but you rather try with 25 .
#buildTools
The buildTools builds  the package and does everything . So before installing any library make sure you are using the proper version of build tools for that .  If that's not in your sdk package , which you can look for in the android studio sdk manager and download it . Some libraries may not tell you about anything , so you should go and check the build tools the package is using in node-modules folder within the app/build.graddle file .
Another thing to mind , that for the  appcompat package com.android.support:appcompat , (second line from last) you have to use the same version as buildTools  otherwise it will throw error .. As I am using build-tools 23.0.1 , so my appcompat has also the 23.0.1 version .
#dexOptions
Sometimes  you have to add dexOptions for enabling jumbo mode when you get errors like
UNEXPECTED TOP-LEVEL EXCEPTION:
com.android.dex.DexException: Cannot merge new index 65536 into a non-jumbo instruction!
Sometimes you enable multidex , by multidex : true , and sometimes largeHeap : true for increasing memory uses .
#the signingConfigs
They are for releasing the  apk .  see https://facebook.github.io/react-native/docs/signed-apk-android.html for that.
#dependencies 👍
These dependecies are the the packages to be compiled .
every project get their path from the settings.graddle file that's been said earlier .
Now , some points to be noted , if you are using react-native-maps with some other google-services , a common problem may occour due to the version conflict of google-services , specially fcm (firebase cloud messaging) . To prevent that you have to exclude the group like
   compile(project(':react-native-maps'))
   {
    exclude group: 'com.google.android.gms'
   }

One thing to note again,  if you are using lower version  of gms than  the version used by react-native-maps , its wont gonna work . So , check it out in the node-modules/react-native-maps/android/app/build.graddle file , which version of gms it is using . If its 9.8.0 , you have to use the same or higher . 10.0.1 have some issues , so I suggest you to use 9.8.0 .
And lastly put the google-maps credentials on your menifest file .
Here are some common issues faced when you are new to react-native .

Couldnot connect to the development server . (running on real-devices) or taking to much long while fetching data .
Simply go the directory    /android/sdk/pltform-tools and run  'adb reverse tcp:8081 tcp:8081'

2.Unfortunately app has been crashed on start .
It may due to many problems . You can try to log the issue by running adb logcat > somefile.txt

Error thrown during predexDebug or dexDebug .
if its not any bug in the java code , sometimes build may fail for unexpected reason. You can try running   gradlew clean on android directory .

4.couldnot create  directory 'somedirectory'
Its specially happens with google-services debug folder . It throws error saying couldn't delete directory debug in android/src/build/generated/debug . You have to simply delete the folder manually .

At the very end during release apk sometimes a error is thrown saying couldn't complete progressEvent . so you should run 'gradlew assembleRelease --console plain'

These are some error I can remember at this moment .
Anyway , if you guys find other problem during installation I would be there .
Happy learning , happy coding .  👍

@PARAGJYOTI  Thanks for a long troubleshooting guide.
I stuck here:
* What went wrong:
A problem occurred configuring project ':app'.
> Could not resolve all dependencies for configuration ':app:_debugApk'.
   > Configuration with name 'default' not found.

\androind\app\ build.gradle :
apply plugin: "com.android.application"

import com.android.build.OutputFile

apply from: "../../node_modules/react-native/react.gradle"

def enableSeparateBuildPerCPUArchitecture = false

def enableProguardInReleaseBuilds = false

android {
    compileSdkVersion 25
    buildToolsVersion "23.0.1"

    defaultConfig {
        applicationId "com.base10"
        minSdkVersion 16
        targetSdkVersion 22
        versionCode 1
        versionName "1.0"
        ndk {
            abiFilters "armeabi-v7a", "x86"
        }
    }
    splits {
        abi {
            reset()
            enable enableSeparateBuildPerCPUArchitecture
            universalApk false  // If true, also generate a universal APK
            include "armeabi-v7a", "x86"
        }
    }
    buildTypes {
        release {
            minifyEnabled enableProguardInReleaseBuilds
            proguardFiles getDefaultProguardFile("proguard-android.txt"), "proguard-rules.pro"
        }
    }
    // applicationVariants are e.g. debug, release
    applicationVariants.all { variant ->
        variant.outputs.each { output ->
            // For each separate APK per architecture, set a unique version code as described here:
            // http://tools.android.com/tech-docs/new-build-system/user-guide/apk-splits
            def versionCodes = ["armeabi-v7a":1, "x86":2]
            def abi = output.getFilter(OutputFile.ABI)
            if (abi != null) {  // null for the universal-debug, universal-release variants
                output.versionCodeOverride =
                        versionCodes.get(abi) * 1048576 + defaultConfig.versionCode
            }
        }
    }
}

dependencies {
    compile fileTree(dir: "libs", include: ["*.jar"])
    compile "com.android.support:appcompat-v7:23.0.1"
    compile "com.facebook.react:react-native:+"  // From node_modules
  compile(project(':react-native-maps'))
   {
    exclude group: 'com.google.android.gms'
     }

}

// Run this once to be able to run the application with BUCK
// puts all compile dependencies into folder libs for BUCK to use
task copyDownloadableDepsToLibs(type: Copy) {
    from configurations.compile
    into 'libs'
}

android\ settings.gradle :
rootProject.name = 'base10'

include ':app'
include ':react-native-maps'
         project(':react-native-maps').projectDir = new File(rootProject.projectDir,'../node_modules/react-native-maps/android')

\android\app\src\main\java\com\base10\ MainApplication.java :
package com.base10;

import android.app.Application;

import com.facebook.react.ReactApplication;
import com.facebook.react.ReactNativeHost;
import com.facebook.react.ReactPackage;
import com.facebook.react.shell.MainReactPackage;
import com.facebook.soloader.SoLoader;

import java.util.Arrays;
import java.util.List;

import com.airbnb.android.react.maps.MapsPackage;

public class MainApplication extends Application implements ReactApplication {

  private final ReactNativeHost mReactNativeHost = new ReactNativeHost(this) {
    @Override
    public boolean getUseDeveloperSupport() {
      return BuildConfig.DEBUG;
    }

    @Override
    protected List<ReactPackage> getPackages() {
      return Arrays.<ReactPackage>asList(
          new MainReactPackage(),
          new MapsPackage()
      );
    }
  };

  @Override
  public ReactNativeHost getReactNativeHost() {
    return mReactNativeHost;
  }

  @Override
  public void onCreate() {
    super.onCreate();
    SoLoader.init(this, /* native exopackage */ false);
  }
}

@pooya1361  There's nothing wrong in your code bro . Its just a issue of the package react-native-maps 0.14.0 . Simply Downgrade the version to 0.12.5 .  It will work fine .

I did so and now I get this error message:
FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':react-native-maps:prepareComAndroidSupportAppcompatV72301Library'.
> Could not expand ZIP 'C:\Users\Haktiebolaget\AppData\Local\Android\sdk\extras\android\m2repository\com\android\support\appcompat-v7\23.0.1\appcompat-v7-23.0.1.aar'.```

@pooya1361  Try Twice or thrice . It should work . If not run cd android && gradlew clean . And build again . Its normal . Happened to me too.

@PARAGJYOTI
I did what you said and it built successfully but now it breaks in the second window it opens with this message:
´´´
error: bundling: UnableToResolveError: Unable to resolve module react/lib/ReactDebugCurrentFrame from c:\base10\node_modules\react-native\Libraries\Renderer\src\renderers\shared\stack\reconciler\ReactCompositeComponent.js: Module does not exist in the module map or in these directories:
c:\base10\node_modules\react-native\node_modules\react\lib
,   c:\base10\node_modules\react\lib
This might be related to https://github.com/facebook/react-native/issues/4968
To resolve try the following:

Clear watchman watches: watchman watch-del-all.
Delete the node_modules folder: rm -rf node_modules && npm install.
Reset packager cache: rm -fr $TMPDIR/react-* or npm start --reset-cache.
at p.catch.error (c:/base10/node_modules/react-native/packager/src/node-haste/DependencyGraph/ResolutionRequest.js:366:19)
Bundling index.android.js  96.2% (559/570), failed.´´´

I cleaned and rebuilt several times but it doesn't work :/

After a couple of hours struggling with the error, I deleted the whole project and created a new one and copied my files there. It finally compiled without error [Yeeay] but it shows nothing 😞
I keep trying (fighting actually) later.

@pooya1361  oops you should have notice the error properly . it happend duo to unmeet peer dependency . So your current react version doesnot have the file required by the react-native library . So , you can get that file from an older version , or may be newer version of react . If you have further problems try using react-native 0.41.2 . Its the most stable version I have worked with if you doesn't really need all the latest components like Flatlist or VirtualizedList .

@PARAGJYOTI Can you please tell me how to downgrade react-native-map version to 0.12.5 . Thankyou.

npm i<EMAIL_ADDRESS>--S

@prasang7 You can change it in package.json too

yes , if you do --S , it will automatically  get saved.

Here's what I did (chronological order).

Deleted node_module directory from the project.
npm cache clean
Modified package.json with: "react":"~15.4.1"; "react-native":"0.42.3"; "react-test-renderer":"15.4.1"
npm install all module
npm install
npm i<EMAIL_ADDRESS>--S
react-native link react-native-maps

Once I do this, it's successfully able to link the android and ios react-native-modules to the project :)
Thank you @PARAGJYOTI @pooya1361

But, even though I am successfully able to link the map modules, whenever I run react-native run-android, it throws the following error:
Execution failed for task ':react-native-maps:prepareComGoogleAndroidGmsPlayServicesMaps980Library'.
Could not expand ZIP 'C:\Android\sdk\extras\google\m2repository\com\google\android\gms\play-services-maps\9.8.0\play-services-maps-9.8.0.aar'.
I searched about the error extensively on the internet, but couldn't find a probable cause.
@PARAGJYOTI @pooya1361 Any idea what might be causing this error?
Here's the screenshot of my command prompt:

have you cleaned the build ? try
cd android && gradlew clean && cd .. && react-native run-android
actually sometimes it goes away after 2 3 trials .

Yes, It worked like magic ❤
Thank you.

@prasang7 I just cleaned gradlew and reset cache, after that I installed maps again. It works like a charm!

I has success build in ios with tips:
init project with version 0.42.3 (to fix 'npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed').
skip step react-native link react-native-maps to use Option 3: Manually in the docs.
in Xcode open libraries -> right click AirMaps.xcodeproj -> add file to AirMaps -> go to find react-native-maps into node_module -> lib -> ios -> AirMaps -> select RCTConvert+MapKit.h/m -> add (to fix 'File RCTConvert+MapKit.m not found').
npm install --save-dev babel-plugin-module-resolver (to fix .babelrc cause error).
watchman watch-del-all. And rebuild project.

I recently made a fresh example build with pods and GoogleMaps with no issues.
https://github.com/alvelig/react-native-maps-pods-example.
Try to check it out or build through the README.MD. If it does not help, let me know.

Deleted node_modules and npm install worked.

@suganthankumar savior! Deleted the folder and it works for me~!!!!

👋
When you see this type of warning, let NPM do the work simply by reseting your nodes-modules like this :
$ rm -rf node_modules package-lock.json
$ npm install

Let us know if it's working for you ...
Enjoy
🖖