Active TCP causing connections to fail

Your environment.

Version: 2.3.4
Client: any client that generates passive TCP candidates (iOS, Go, other libwebrtc clients)

What did you do?
Reproducible with all iOS devices using WebRTC M103+. This is because iOS is generating passive TCP candidates that triggers active TCP.
The issue appears to be due to how connection to active TCP candidate pair are implemented. When a new TCP candidate is discovered, it's attempting to dial to it in a blocking manner here.
Because this is blocking the Agent's runloop, it can cause other candidates to fail. This is particularly noticeable when a remote candidate is added with an address behind NAT. That connection would never resolve.
What did you expect?
the presence of Active TCP candidates do not cause other ICE candidates to fail.

@ashellunts I was able to reproduce this with the following test
// Assert that Active TCP connectivity isn't established inside
// the main thread of the Agent
func TestAgentActiveTCP_NonBlocking(t *testing.T) {
        report := test.CheckRoutines(t)
        defer report()

        lim := test.TimeOut(time.Second * 5)
        defer lim.Stop()

        cfg := &AgentConfig{
                NetworkTypes: supportedNetworkTypes(),
        }

        aAgent, err := NewAgent(cfg)
        if err != nil {
                t.Error(err)
        }

        bAgent, err := NewAgent(cfg)
        if err != nil {
                t.Error(err)
        }

        isConnected := make(chan interface{})
        err = aAgent.OnConnectionStateChange(func(c ConnectionState) {
                if c == ConnectionStateConnected {
                        close(isConnected)
                }
        })
        if err != nil {
                t.Error(err)
        }

        // Add a invalid ice-tcp candidate to each
        invalidCandidate, err := UnmarshalCandidate("1052353102 1 tcp<PHONE_NUMBER> <IP_ADDRESS> 8080 typ host tcptype passive")
        if err != nil {
                t.Fatal(err)
        }
        aAgent.AddRemoteCandidate(invalidCandidate)
        bAgent.AddRemoteCandidate(invalidCandidate)

        connect(aAgent, bAgent)

        <-isConnected
        assert.NoError(t, aAgent.Close())
        assert.NoError(t, bAgent.Close())
}

I think we need to defer calling Dial until the candidate is moved to its own thread. I tried creating a TCPFutureConn the idea was to Dial when Read or Write was called.
This didn't work because we depend on LocalAddr and RemoteAddr
package ice

import (
        "io"
        "net"
        "sync"
        "sync/atomic"
        "time"
)

type tcpFutureConn struct {
        tcpConn     net.Conn
        remoteAddr  net.Addr
        dialAddress string
        haveDialed  int32
        dialLock    sync.Mutex
}

func (t *tcpFutureConn) maybeDial() error {
        if atomic.LoadInt32(&t.haveDialed) != 1 {
                t.dialLock.Lock()
                defer t.dialLock.Unlock()

                if atomic.LoadInt32(&t.haveDialed) == 1 {
                        return nil
                }

                conn, err := net.Dial("tcp", t.dialAddress)
                if err != nil {
                        // a.log.Errorf("Failed to dial TCP address %s: %v", t.dialAddress, err)
                        return err
                }

                t.tcpConn = conn
                atomic.StoreInt32(&t.haveDialed, 1)
        }

        return nil
}

func (t *tcpFutureConn) Read(buff []byte) (n int, err error) {
        if err = t.maybeDial(); err != nil {
                return
        }

        return t.tcpConn.Read(buff)
}

func (t *tcpFutureConn) Write(buff []byte) (n int, err error) {
        if err = t.maybeDial(); err != nil {
                return
        }

        return t.tcpConn.Write(buff)
}

func (t *tcpFutureConn) Close() error {
        if atomic.LoadInt32(&t.haveDialed) != 1 {
                return nil
        }

        return t.tcpConn.Close()
}

func (t *tcpFutureConn) LocalAddr() net.Addr {
        return &net.TCPAddr{}
}

func (t *tcpFutureConn) RemoteAddr() net.Addr {
        return &net.TCPAddr{}
}

func (t *tcpFutureConn) SetDeadline(time.Time) error      { return io.EOF }
func (t *tcpFutureConn) SetReadDeadline(time.Time) error  { return io.EOF }
func (t *tcpFutureConn) SetWriteDeadline(time.Time) error { return io.EOF }

@stv0g @ashellunts Any ideas on how we could make this work?

@Sean-Der I will take a look.

@Sean-Der I assume this issue can be closed? It seems like you fixed it in db5d7ea72b110ed3fbd1843b1a484ce91add933a?

@stv0g yes fixed! Sorry forgot to resolve


When adding a dependency it resets my .android and .ios folder

Steps to Reproduce

Create a new project
add some meta data or permissions into the .android project
add a new dependency
the .android and .ios folder is reset

Expected results:
Added the dependency without removing my permissions or information
Actual results:
The dependency is added but it resets my .android and .ios folder

Logs

[√] Flutter (Channel stable, 3.7.10, on Microsoft Windows [Version 10.0.22621.1470], locale en-US)
    • Flutter version 3.7.10 on channel stable at C:\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 4b12645012 (4 days ago), 2023-04-03 17:46:48 -0700
    • Engine revision ec975089ac
    • Dart version 2.19.6
    • DevTools version 2.20.1

[√] Windows Version (Installed version of Windows is version 10 or higher)
    • Flutter extension version 3.62.0

[√] Connected device (3 available)
    • sdk gphone64 x86 64 (mobile) • emulator-5554 • android-x64    • Android 13 (API 33) (emulator)
    • Windows (desktop)            • windows       • windows-x64    • Microsoft Windows [Version
      10.0.22621.1470]
    • Edge (web)                   • edge          • web-javascript • Microsoft Edge 111.0.1661.44

[√] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 2 categories.

 The project template was module and not app.


How to display dialog with custom view

Hi friends,
I did not able to locate any wiki or doc about how to use this library to display dialog with custom view. Is there any feature available? How to use it?

Hi @ripal and sorry for a late reply. Anvil only deals with views. You can then use Anvil views inside Activities, Fragments or Dialogs as you would use any other view object.
For dialogs you can do:
myDialog.setContentView(new RenderableView() {
  public void render() {
    // your Anvil layout and data bindings here
  }
});

For Activities and Fragments the story is similar - just create Anvil views and use them as content views.


counter

Hello I know this is supposed to only count the successfuly looted chests but can't we make it count every chest please even the one that spawns swarms its easier to track the total progress?
thanks for the great effort and the great plugin

Implemented in PR: https://github.com/runelite/plugin-hub/pull/3530

thank you so much I really apprecaite all the efforts to make this accept my respect sir


ReactorKafkaBinder - Add ReceiverOptionsCustomizer, SenderOptionsCustomizer

Existing Spring Boot related PR https://github.com/spring-projects/spring-boot/pull/30567


Update verifier documentation. [phab:D270]

Created by zarko at 2015-05-21 00:08:52:
is => typed; ordering of param edges; forward reference to =

shahms accepted this revision at 2015-05-21 15:15:45

zarko closed via commit cad77e9d004f6403d32b665187e7733473a3c270 at 2015-05-21 16:43:11


Update mlp.py

list comprehensions!!!  \O/
TODO: If fprop/bprop(None) returrns a constant, then this should be moved to initialization time, right?

testing now.  I will update if this passes contribution guidelines tests.

I don't think this is even needed, as I am going to try a map + const assignment at instantiation time to see if this can be sped up.
List comprehension is inappropriate given the degradation in performance (almost half a second!)


chore(docs): linked to most recent versions in publish-your-sdk.mdx

Description
Added text to the publish-your-sdk.mdx that links to fern-api README where most up to date version table is accessible
Testing
Will look at site preview, unsure if that information will always be stored there

This change doesn't appear properly in preview, looking into it now


Feat/enforce custom provider type

Closes #24
This is still a WIP. We need additional test cases and cover scenarios where we rename the Provider type

@thiagomini I took the liberty to add a test case for where we alias Provider type as a starter to contribute. Hope you don't mind 🙂

That's awesome, @tolgap , your help is very much appreciated! Thanks for contributing to our cause 🙌


dep fails to init docker

moroz@minigrind [±:master] docker % ~/project/workspace/bin/dep init
ouchie, solve error: No versions of github.com/gogo/protobuf met constraints:
        master: Could not introduce github.com/gogo/protobuf@master, as its subpackage github.com/gogo/protobuf/types is missing. (Package is required by (root).)
        v0.3: Could not introduce<EMAIL_ADDRESS>as it is not allowed by constraint master from project github.com/docker/docker.
        v0.2: Could not introduce<EMAIL_ADDRESS>as it is not allowed by constraint master from project github.com/docker/docker.
        v0.1: Could not introduce<EMAIL_ADDRESS>as it is not allowed by constraint master from project github.com/docker/docker.
        master: Could not introduce github.com/gogo/protobuf@master, as it is not allowed by constraint master from project github.com/docker/docker.
        bigendian: Could not introduce github.com/gogo/protobuf@bigendian, as it is not allowed by constraint master from project github.com/docker/docker.
        bigskip: Could not introduce github.com/gogo/protobuf@bigskip, as it is not allowed by constraint master from project github.com/docker/docker.
        gopherjs: Could not introduce github.com/gogo/protobuf@gopherjs, as it is not allowed by constraint master from project github.com/docker/docker.
        ptypes: Could not introduce github.com/gogo/protobuf@ptypes, as it is not allowed by constraint master from project github.com/docker/docker.
        std: Could not introduce github.com/gogo/protobuf@std, as it is not allowed by constraint master from project github.com/docker/docker.
        wkt: Could not introduce github.com/gogo/protobuf@wkt, as it is not allowed by constraint master from project github.com/docker/docker.No versions of github.com/gogo/protobuf met constraints:
        master: Could not introduce github.com/gogo/protobuf@master, as its subpackage github.com/gogo/protobuf/types is missing. (Package is required by (root).)
        v0.3: Could not introduce<EMAIL_ADDRESS>as it is not allowed by constraint master from project github.com/docker/docker.
        v0.2: Could not introduce<EMAIL_ADDRESS>as it is not allowed by constraint master from project github.com/docker/docker.
        v0.1: Could not introduce<EMAIL_ADDRESS>as it is not allowed by constraint master from project github.com/docker/docker.
        master: Could not introduce github.com/gogo/protobuf@master, as it is not allowed by constraint master from project github.com/docker/docker.
        bigendian: Could not introduce github.com/gogo/protobuf@bigendian, as it is not allowed by constraint master from project github.com/docker/docker.
        bigskip: Could not introduce github.com/gogo/protobuf@bigskip, as it is not allowed by constraint master from project github.com/docker/docker.
        gopherjs: Could not introduce github.com/gogo/protobuf@gopherjs, as it is not allowed by constraint master from project github.com/docker/docker.
        ptypes: Could not introduce github.com/gogo/protobuf@ptypes, as it is not allowed by constraint master from project github.com/docker/docker.
        std: Could not introduce github.com/gogo/protobuf@std, as it is not allowed by constraint master from project github.com/docker/docker.
        wkt: Could not introduce github.com/gogo/protobuf@wkt, as it is not allowed by constraint master from project github.com/docker/docker.

moroz@minigrind [±:master] docker % go env
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/moroz/project/workspace"
GORACE=""
GOROOT="/home/moroz/go"
GOTOOLDIR="/home/moroz/go/pkg/tool/linux_amd64"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build254478034=/tmp/go-build -gno-record-gcc-switches"
CXX="g++"
CGO_ENABLED="1"

I've removed vendor dir before doing init :)

I have another issue for this and tried again after a merge of a fix this
morning and it worked
On Tue, Jan 24, 2017, 09:20 Alexander Morozov<EMAIL_ADDRESS>wrote:

moroz@minigrind [±:master] docker % ~/project/workspace/bin/dep init
ouchie, solve error: No versions of github.com/gogo/protobuf met constraints:
master: Could not introduce github.com/gogo/protobuf@master, as its subpackage github.com/gogo/protobuf/types is missing. (Package is required by (root).)
v0.3: Could not introduce<EMAIL_ADDRESS>as it is not allowed by constraint master from project github.com/docker/docker.
v0.2: Could not introduce<EMAIL_ADDRESS>as it is not allowed by constraint master from project github.com/docker/docker.
v0.1: Could not introduce<EMAIL_ADDRESS>as it is not allowed by constraint master from project github.com/docker/docker.
master: Could not introduce github.com/gogo/protobuf@master, as it is not allowed by constraint master from project github.com/docker/docker.
bigendian: Could not introduce github.com/gogo/protobuf@bigendian, as it is not allowed by constraint master from project github.com/docker/docker.
bigskip: Could not introduce github.com/gogo/protobuf@bigskip, as it is not allowed by constraint master from project github.com/docker/docker.
gopherjs: Could not introduce github.com/gogo/protobuf@gopherjs, as it is not allowed by constraint master from project github.com/docker/docker.
ptypes: Could not introduce github.com/gogo/protobuf@ptypes, as it is not allowed by constraint master from project github.com/docker/docker.
std: Could not introduce github.com/gogo/protobuf@std, as it is not allowed by constraint master from project github.com/docker/docker.
wkt: Could not introduce github.com/gogo/protobuf@wkt, as it is not allowed by constraint master from project github.com/docker/docker.No versions of github.com/gogo/protobuf met constraints:
master: Could not introduce github.com/gogo/protobuf@master, as its subpackage github.com/gogo/protobuf/types is missing. (Package is required by (root).)
v0.3: Could not introduce<EMAIL_ADDRESS>as it is not allowed by constraint master from project github.com/docker/docker.
v0.2: Could not introduce<EMAIL_ADDRESS>as it is not allowed by constraint master from project github.com/docker/docker.
v0.1: Could not introduce<EMAIL_ADDRESS>as it is not allowed by constraint master from project github.com/docker/docker.
master: Could not introduce github.com/gogo/protobuf@master, as it is not allowed by constraint master from project github.com/docker/docker.
bigendian: Could not introduce github.com/gogo/protobuf@bigendian, as it is not allowed by constraint master from project github.com/docker/docker.
bigskip: Could not introduce github.com/gogo/protobuf@bigskip, as it is not allowed by constraint master from project github.com/docker/docker.
gopherjs: Could not introduce github.com/gogo/protobuf@gopherjs, as it is not allowed by constraint master from project github.com/docker/docker.
ptypes: Could not introduce github.com/gogo/protobuf@ptypes, as it is not allowed by constraint master from project github.com/docker/docker.
std: Could not introduce github.com/gogo/protobuf@std, as it is not allowed by constraint master from project github.com/docker/docker.
wkt: Could not introduce github.com/gogo/protobuf@wkt, as it is not allowed by constraint master from project github.com/docker/docker.
moroz@minigrind [±:master] docker % go env
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/moroz/project/workspace"
GORACE=""
GOROOT="/home/moroz/go"
GOTOOLDIR="/home/moroz/go/pkg/tool/linux_amd64"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build254478034=/tmp/go-build -gno-record-gcc-switches"
CXX="g++"
CGO_ENABLED="1"
I've removed vendor dir before doing init :)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/golang/dep/issues/129, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABYNbHZeIw1j2N-xSErS9QkMGceLwnV6ks5rVjLogaJpZM4Lsi1G
.

https://github.com/golang/dep/issues/116

I wonder if this is hitting the rate limiting problem and just had it all locally

If you try with -v it will give more output but will try again when I get
to the office
On Tue, Jan 24, 2017, 10:02 Jess Frazelle<EMAIL_ADDRESS>wrote:

I wonder if this is hitting the rate limiting problem and just had it all
locally
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/golang/dep/issues/129#issuecomment-274884414, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABYNbEbxzPtORx-MZNFnV3HNYab9tpncks5rVjxsgaJpZM4Lsi1G
.

        master: Could not introduce github.com/gogo/protobuf@master, as its subpackage github.com/gogo/protobuf/types is missing. (Package is required by (root).)

Errors like this are vaguely familiar to me - protobuf generated code. I don't have an immediate solution, but this was the original root problem in the solve, and doesn't look like it would have been caused by API rate limiting. That package just wasn't there in master.

@sdboyer It looks like it's in master though
https://github.com/gogo/protobuf/tree/master/types

Indeed, and since Aug 26. Might be a bug, then. Curious...did @jessfraz suggestions help? If not, did you have github.com/gogo/protobuf on your GOPATH when you ran dep init? If so, what branch/rev was it at?

On second run I've encountered another error:
ouchie, solve error: No versions of github.com/docker/distribution met constraints:
        master: Could not introduce github.com/docker/distribution@master due to multiple problematic subpackages:
        Subpackage github.com/docker/distribution/digestset is missing. (Package is required by (root).)        Subpackage github.com/docker/distribution/registry/client/auth/challenge is mis
sing. (Package is required by (root).)

Both packages are exist at the master of distribution.
I wonder if there is way to specify versions first and then vendor them.
I have github.com/gogo/protobufin GOPATH indeed and it has 74b6e9deaff6ba6da1389ec97351d337f0d08b06 revision where there is no types package. On other side it's quite weird to depend on any package in GOPATH for init.

yeah ok when the errors change like that, in the past for me it means rate limiting :(

@jessfraz Cool. So, we need ratelimit in tool and better error reporting :)

I wonder if there is way to specify versions first and then vendor them.

This is one of the issues I need to write up - yes, we've discussed and would like to add some different init modes, rather than just inferring from GOPATH.

On other side it's quite weird to depend on any package in GOPATH for init.

Depends on your perspective :) In the simple/vanilla Go user case that still just uses GOPATH, it's certainly the right approach. For people who use a tool, something else is needed. Which is what those different import modes is about.
If you just remove these projects from your GOPATH, then you'll probably have better results - doing so will make dep init just operate from info available upstream.

I have github.com/gogo/protobufin GOPATH indeed and it has 74b6e9deaff6ba6da1389ec97351d337f0d08b06 revision where there is no types package.

Ahh yeah - that'd do it! It does highlight another thing I need to do in gps, though - we should make sure to visit the upstream state of master, too. So many issues to write today...

@jessfraz Cool. So, we need ratelimit in tool and better error reporting :)

yep! ratelimit over at #115; error reporting in #22 as a placeholder, and I'm writing a bigger issue about it, too.

If you just remove these projects from your GOPATH.

Heh, it's not really a cool option. I can try in empty gopath instead.

I'm closing this as dupe of #115 to reduce noise.

Heh, it's not really a cool option.

Totally fair - state mutation in GOPATH is never really an option. It's why we're in this mess! :)

I can try in empty gopath instead.

That should do it!

this is terrible advice but if you want it to eventually work for now just continually running it after like 2 times w the rate limit and waiting should work, well it did for me, but yeah sad story


Cannot browse follower's private folder from macOS Finder

Keybase GUI Version: 5.0.0-20191114182642+f73f97dac6
Hello,
I have two singed in accounts in my app, say for example "work" and "personal".
Keybase filesystem is mounted as a volume in the macOS Finder.
User "work" can see and browse /Volumes/keybase/private/work,personal from the Finder.
Switching to user "personal", /Volumes/keybase/private/work,personal is shown as an alias, and I cannot open the folder to see its contents. On the other hand I can easily access the folder from the terminal, from the desktop and mobile client apps.
macOS 10.14.6
Am I missing something?
Thanks in advance for any help you will offer.
Gabriele

If you do a keybase log send after trying to look at the folder from "personal", and post the resulting ID here, I can take a look to see if anything stands out.
When you say you can access it from the terminal, do you mean with the keybase fs commands, or do you mean directly via the /Volumes/Keybase file system?  If the latter, then it seems like it might be a Finder bug of some kind...

Thanks for the prompt reply!
The two Keybase accounts are named "nr1" and "musico" (although I wrote "personal" and "work").
From "nr1" I cannot access private/nr1,musico from the Finder.
From "musico" I just works.
From the terminal I mean:
$ ls -lha /Volumes/Keybase\ \(gabriele\)/private/nr1\,musico/
total 336577
drwx------  1 gabriele  wheel   231B Dec  1 08:51 .
dr-x------  1 gabriele  wheel     0B Dec  2 07:19 ..
-rw-------  1 gabriele  wheel   164M Dec  1 08:51 report.pdf

------------
Success! Your log ID is:

  5ad89c8429999cabdd38db1c

Here's a URL to submit new bug reports containing this ID:

  https://github.com/keybase/client/issues/new?body=[write%20something%20useful%20and%20descriptive%20here]%0A%0Amy%20log%20id:%205ad89c8429999cabdd38db1c

Thanks!
------------

Sorry, I meant the opposite:
From "musico" I cannot access private/nr1,musico from the Finder.
From "nr1" I just works.

Thanks for more info.  Sure weird.  Nothing is sticking out to me in the log -- can you post exactly what error you're seeing in the finder when you try to click through to the folder?
I'm also curious what happens when you click the "Show in Finder" icon in the upper right corner of the Keybase app when you have the folder open in there.

I prepared a small screencast, hope this helps… :-)
https://musico.keybase.pub/kebase.mp4
TIA
Gabriele

Thanks that's helpful.  But it's pretty weird behavior.  I guess maybe there is some kernel invalidation call we're missing on a user switch.
What happens if you are signed in as musico, then completely quit the Keybase app (including the file system), and then restart Keybase.  Does it work as expected in that case, until the point where you sign in as nr1 and then back into musico?

I was able to reproduce this myself.  However, I have triple-checked that we're sending all the right calls up to the kernel as far as I can tell when the user changes.  This seems to be a bug in Finder itself, where it caches the old link to a folder whose names has changed (in this case, from mr1,musico to musico,nr1) even though KBFS has invalidated it.  Unfortunately I can't think of anything else we can do to force it to update.
The good news is you can still access it from the terminal and from the app -- I guess a (sad) workaround is that if you really need Finder access, you can restart Keybase after switching users and it should work again.  Sorry for the bad experience!

I can confirm that quitting and relaunching the application from the "buggy profile" will workaround the problem… :-)

My file manager, Dolphin Version 19.11.80, on openSUSE Leap 15.1, in KDE Frameworks 5.64.0, has a similar cache problem. When I log out of the client the /keybase directory remains. Forcing a refresh (F5) solves the issue for me. I suspect that a similar issue would exist if I used multiple accounts and tried to access linked directories. Possibly of help in developing a solution or workaround.


Merge upstream SQLite 3.45.1

this will be very useful

I did a merge of this, but Github squashed it, making the repository incompatible with upstream SQLite. I dropped the merge so let's re-open.


Change nav titles

@TPG-Teklada Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

Apologies, I meant to merge within my own fork.


How should quasi-duplicate JS katas concerning arrows get handled?

The recent announcement of ES6 support and therefore arrow notation lead to a bunch of new katas like Take an Arrow to the knee, Functionally. Those katas don't provide additional content compared to the one that didn't ask for arrows.
For example, the early versions of Sum with Arrows could be solved with any solution of Use reduce() to calculate the sum of the values in an array, which made it clearly a duplicate. Matt has changed the kata so that one has to provide a string instead of a function, which kind of resolves this issue, but some other katas are basically duplicates.
Also, the "arrow" notation is mostly used as a shorthand lambda notation. It cannot get translated properly into other languages for the same reason it cannot get tested in JS properly: lambdas and functions are often indistinguishable. There's (almost) no difference between sum = \a x -> a + x, sum = (+) and sum a x = a+ x in Haskell or sum = lambda x,y: x+y and def sum:… in Python.
So, how should one handle the "use-an-arrow-to-solve-this-kata!" katas?

To be honest, at least in JS one could also use the .toString() method on a function and see if "=>" was used; cheatable because you could just put it into a comment, but still better than nothing (and if you are that decent to come up with such a plan to cheat the tester, you probably don't need it at all).

To be honest, at least in JS one could also use the .toString() method on a function and see if "=>" was used;

You can't. Check the discussion in the linked kata "Sum of Arrows". The transpiler processes the source before it gets handed to node. Try it yourself:
var test_func = arr => arr.map( a => a + 1)
console.log(test_func.toString())
// console output:
// function test_func(arr) {
//     return arr.map(function (a) {
//         return a + 1;
//     });
// }

Oh, didn't know about the priority of the transpiler as I tried with other mock functions: thanks for letting me know!

Closing old issue.
I believe users are now more strict about duplicates and doesn't allow kata like this to be approved anymore.


Placeholder show times cooldown

Plugin Version: Version #3.1.1
Request:
I would like a Placeholder for my menu which show the times cooldown staying before player use /rtp again.
This placeholder will use in lore for a button in my menu because I use you're plugin with menu.
How/Why:
I think it could be great :)
An example : https://zupimages.net/viewer.php?id=21/15/jtm6.png

I would like this feature, I will implement it for the next patch ;)

Hey, so this features could be implemented soon ?

Placeholders has now officially been added in version 3.4.5
A file with supported placeholders will generate in the plugin folder!
https://www.spigotmc.org/resources/betterrtp.36081/update?update=475857


show score on Scores Page

again do we need in a one page app?...Team to discuss if this is actually a feature we should work on or leave it for now

Add to future improvement list!


Write architecture decision records

💩 How it is currently developed
Since the start of this repository, we did not write any ADR at all.
🌟 What is the optimal solution
We should document relevant architecture decisions for future inspection.
😓 Why we did that way
To move fast.
💣 What are the possible impacts
In the future, it could be unclear why we took some decisions.

Closed since we're now managing our engineering priorities on this Jira board.


Input window is terrable

Now input window may so less things. It doesn't allow many lines, hasn't eye to see password etc.
Need to fix and heal.

Maybe I can do some enhancements for this


nixos/nix-daemon: use structural settings

The nix.* options, apart from options for setting up the
daemon itself, currently provide a lot of setting mappings
for the Nix daemon configuration. The scope of the mapping yields
convience, but the line where an option is considered essential
is blurry. For instance, the extra-sandbox-paths mapping is
provided without its primary consumer, and the corresponding
sandbox-paths option is also not mapped.
This increases the maintenance burden as maintainers have to
closely follow upstream changes which in the case of Nix is made
worse by having to juggle between two state versions, the stable release
and the unstable release, which may include or deprecate features.
One conditional option publicHostKey was added to the buildMachines
submodule corresponding to the newly added base64 encoded public host
key setting exposed in the builder syntax. The build machine generation
was subsequently rewritten to use concatStringsSep to group concatenations.
This commit aims to following the standard outlined in RFC 42[1] to
implement a structural setting pattern. The Nix configuration is encoded
at its core as key-value pairs which maps nicely to attribute sets, making
it feasible to express in the Nix language itself. Some existing options are
kept such as buildMachines and registry which present a simplified interface
to managing the respective settings.
Legacy configurations are mapped to their corresponding options under the
structural settings for backwards compatibility.
Various options settings in other nixos modules and relevant tests have been
updated to use the structural setting for consistency.
The generation and validation of the configration file has been modified to
use writeTextFile instead of runCommand for clarity. Note that validation
is now mandatory as strict checking of options has been pushed down to the
derivation level due to freeformType consuming unmatched options. Furthermore,
validation can not occur when cross-compiling due to current limitations.
[1] - https://github.com/NixOS/rfcs/blob/master/rfcs/0042-config-option.md

Motivation for this change
Initial effort for https://github.com/nix-community/home-manager/issues/2324
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[x] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

It has been a while after the 21.11 branch-off.
Looking forward to its merging!

Is there something blocking this merge?
(Except from the conflicts).

I don't see any blockers and the conflict seems simple.

I have resolved the conflict and updated the publicHostKey option to be static.

As the target changelog release note is 21.11, this is to be back-ported to release-21.11. Is that right?
It seems alright as the backward-compatibility is kept with the mapping layer.

Thanks for pointing this out, I was close to merging this ;)
I don't think we should backport this since it introduces a lot of warnings for existing setups and the large size of this PR leaves room for regressions.
If the release notes are moved to 22.05, I will happily merge this (unless @roberth or anyone else intervenes)

After this change nix.buildMachines, which I did not change at all, is apparently failing.
error: value is a string while a list was expected

       at /etc/nixos/nixpkgs/nixos/modules/services/misc/nix-daemon.nix:676:14:

          675|           (machine:
          676|             (concatStringsSep " " [
             |              ^
          677|               "${optionalString (machine.sshUser != null<EMAIL_ADDRESS>
       … while evaluating anonymous lambda

       at /etc/nixos/nixpkgs/nixos/modules/services/misc/nix-daemon.nix:675:12:

          674|         concatMapStrings
          675|           (machine:
             |            ^
          676|             (concatStringsSep " " [

       … from call site

       … while evaluating 'concatMapStrings'

Yup, this broke the evaluation for any/everyone using nix.buildMachines

I missed a bracket around the append operator. I apologize for thoroughly testing the changes enough.

We should consider not warning about the renames until 21.11 is end of life. #157667 describes why sudden renames are a problem for out-of-tree modules.

The s/warning/error/ and validation introduced in this commit is causing errors now:
building '/nix/store/d2wav40xyk3d13fpbzc3dn1d5249zp1s-etc-registry.json.drv'...
building '/nix/store/41y6qwy8v3n08navcmkgk34m9rjwqpmb-nix.conf.drv'...
error: builder for '/nix/store/41y6qwy8v3n08navcmkgk34m9rjwqpmb-nix.conf.drv' failed with exit code 1;
       last 2 log lines:
       > Validating generated nix.conf
       > error: unknown experimental feature 'ca-references'
       For full logs, run 'nix log /nix/store/41y6qwy8v3n08navcmkgk34m9rjwqpmb-nix.conf.drv'.
error: 1 dependencies of derivation '/nix/store/kydaidpymdirx9yxyfhrnwbfdk2pykj6-etc.drv' failed to build
error: 1 dependencies of derivation '/nix/store/7snhkayxbb1rpmqbhs8fgjwsmkdsmlpy-nixos-system-host-22.05.20220130.efeefb2.drv' failed to build

The only way is to disable nix.checkConfig.

The validation must catch all warnings and errors because the config checker does not distinguish between invalid syntax and invalid data. Admittedly this is not as flexible as the type checking in the module system but a necessity given the limitations of freeform modules.
In this case, it seems the ca-references experimental feature have been removed as per https://github.com/NixOS/nix/issues/5913#issuecomment-1013513176.

I don't have ca-references explicitly declared in my nix configuration, but still it's making it there somehow.
❯ fgrep ca- /etc/nix/nix.conf
extra-experimental-features = nix-command ca-references flakes

And my nix nixos configuration looks like:
{ inputs, pkgs, ...} : {
  nix = {

    gc = {
      automatic = true;
      dates = "13:37";
      options = "--delete-older-than 10d";
    };

    settings = {
      allowed-users = [ "@wheel" "root" ];
      sandbox = true;
      cores = 4;
      max-jobs = 1;
      substituters =  [
        "https://nix-community.cachix.org"
        "https://hydra.iohk.io"
        "https://nixpkgs-wayland.cachix.org"
      ];
      trusted-public-keys =  [
        "nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs="
        "hydra.iohk.io:f/Ea+s+dFdN+3Y/G+FDgSq+a5NEWhJGzdjvKNGv0/EQ="
        "nixpkgs-wayland.cachix.org-1:3lwxaILxMRkVhehr5StQprHdEo4IrE8sRho9R9HOLYA="
      ];
    };

    checkConfig = false; # workaround for https://github.com/NixOS/nixpkgs/pull/139075#issuecomment-1027772745

    nixPath = [ "nixpkgs=${inputs.nixpkgs}" ];

    package = pkgs.nix_2_6;

    extraOptions = ''
      experimental-features = nix-command flakes
      keep-outputs = true
      keep-derivations = true
    '';
  };
}

I cannot seem to reproduce this with the example configuration given in a vm.
> fgrep experimental-features /etc/nix/nix.conf
experimental-features = nix-command flakes

Could something else be writing to nix.extraOptions or nix.settings? You could test it by writing some debugging assertion or use mkRemovedOptionModule to get a trace of where it is called, but I am afraid such assertions wouldn't be able to distinguish merging of nix.extraOptions.

Found it, sorry.

The options module argument contains info about which modules contribute to config values.
nix repl '<nixpkgs>'
nix-repl> s = (pkgs.nixos { }).options.nix.settings

nix-repl> :p lib.zipListsWith lib.nameValuePair s.files s.definitions
[ { name = "/home/user/h/nixpkgs/nixos/modules/services/misc/nix-daemon.nix"; value = { substituters = [ "https://cache.nixos.org/" ]; system-features = { _type = "override"; content = [ "nixos-test" "benchmark" "big-parallel" "kvm" ]; priority = 1000; }; trusted-public-keys = [ "cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=" ]; }; } { name = "/home/user/h/nixpkgs/nixos/modules/services/misc/nix-daemon.nix"; value = { builders = null; }; } { name = "/home/user/h/nixpkgs/nixos/modules/services/misc/nix-daemon.nix"; value = { sandbox-fallback = false; }; } ]

You'll probably want to use your own config instead of pkgs.nixos { } which is an empty/default configuration except for nixpkgs.pkgs.

Could anyone have a look at #159014 please?


Punctuation update for how-to-catch-outgoing-emails-locally.md

Minor punctuation comma fixes

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Hi @talrazmob
It seems that similar changes have already been accepted earlier for this article you're editing, sorry about that. 😓
If you feel you have more to add, please feel free to open up a new PR.
Thanks again! 😊

If you have any questions, feel free to reach out through Gitter or by commenting below. 💬


Write Single Register Configuration (IDFGH-11919)

Hello!
We are running into issues trying to understand how to use this component properly, specifically in regards on how to write to a single holding register.
Our Slave Device does not currently support writing multiple registers using function code 0x10 as this functionality wasn't yet needed as up till now we used the function code 0x06 to write to a single Holding Register.
We know that we could manually write the request but this is more error prone in our opinion.
Therefore the question, is it possible to configure a member of our mb_parameter_descriptor_t[] to use Function code 0x06 instead of 0x10?

Hi,
Custom implementation is also possible. I also have the ideas on how to accomplish this but currently I am busy with some other things. Please feel free to share your ideas. I will try to review as soon as possible from my side. It is important for me to provide more convenient way for users related to this feature.
Thanks.

Hi,
I will get into working on that either later today or sometime later this week. Either way, no problem.

One important comment: I have an requirement for the project is that the external API interface for v1.0.x of esp-modbus should not be changed. This includes the format of Data dictionary. The changes of interface for v1.0.x will not be applied. This is why I did not changed this directly but added the workaround for this as an additional function. However, the changes you are working on can be implemented in esp-modbus v2.0.0-beta.

Entirely understandable, my approach would likely end up adding to the Data Dictionary and thus causing this requirement to break.
However I am of the opinion that sometimes you gotta break compatibility, document it and be fine with it.

However I am of the opinion that sometimes you gotta break compatibility, document it and be fine with it.

I have got your opinion and will try to address this when possible.

I think I actually found a way that wouldn't break compatibility but still allow commands to be configurable when wanted while the current behavior stays default.

PR is opened regarding this

The PR will not be merged due to the breaking changes. For v1.0.x I recommend to use the existing compatibility way of sending the custom requests with command 0x05 which works just fine. See the documentation and examples mentioned above.

As far as I am aware the changes shouldn't be breaking, however I would love to get more detailed information as to why you think they are.
Either way, implementing these changes should be, imo, done for the next major version.

@ DomiMartinGlogi ,
Sorry for even very late response. Unfortunately your code changes the interface and will cause the maintaining issues later. I have other idea to implement this using compatible way. I will address this when possible.


Fix for bug renaming non-coord var to same name as dim without a var

Part of #597.
Fixes #1297.
This is a fix for a tricky rename bug.
This PR includes all my pending PRs. It may be easier to review this code after some of them get merged.

I will wrap this and some other prs into an aggregation in short order! Have a great weekend.
On Jan 31, 2019, 8:00 AM -0700, Ed Hartnett<EMAIL_ADDRESS>wrote:

@WardF can we get this merged soon? I have some more rename fixes, and then a comprehensive rename fix that I think will solve many or all of the remaining rename problems...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks!


Correct translation of word"pity"

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


Fuzzer terminating before starting, with any input seed

I followed the BKM as is, with 5.5 kernel and the stretch.img downloaded from the site as well as creating one myself.
Things I tried:

Try with input seeds from seeds/ . Change the input seed from the given set to ones I created.
Load a usb device image with qemu and loaded.
Tried running with the bzImage in the repo as well as one I created myself (no special instrumentaion- only the patching recommended in this BKM)

Every trial resulted in the same outcome, show below.

~/fuzz/USBFuzz$ ./USBFuzz --seeddir seeds --kernel_image bzImage --os_image stretch.img
afl-fuzz USBFuzz-Adaption-V1.0 by<EMAIL_ADDRESS>[+] Disabling the UI because AFL_NO_UI is set.
[+] You have 96 CPU cores and 5 runnable tasks (utilization: 5%).
[+] Try parallel jobs - see docs/parallel_fuzzing.txt.
[*] Checking CPU core loadout...
[+] Found a free CPU core, binding to #1.
[*] Checking core_pattern...
[*] Setting up output directories...
[+] Output directory exists but deemed OK to reuse.
[*] Deleting old session data...
[+] Output dir cleanup successful.
[*] Scanning 'seeds'...
[+] No auto-generated dictionary tokens to reuse.
[*] Creating hard links for all input files...
[*] Validating target binary...
[*] Attempting dry run with 'id:000000,orig:usb_s04tbco'...
[*] Spinning up the fork server...

[-] Hmm, looks like the target binary terminated before we could complete a
    handshake with the injected code. Perhaps there is a horrible bug in the
    fuzzer. Poke<EMAIL_ADDRESS>for troubleshooting tips.

[-] PROGRAM ABORT : Fork server handshake failed
         Location : init_forkserver(), afl-fuzz.c:2266

I got the same problems. Have you addressed it?

I got the problem too. Have you solved it?

I got the same problems. Have you addressed it?


/schoolwork user links- some blue on hover, some not

I don't care which it is, just make it consistant

Ok I'll start work on that soon

Will now start work on this :)


API Key Permission denied error; HTTPError at 401 Client Error: PermissionDenied for url: https://westus2.api.cognitive.microsoft.com/vision/v2.0/read/core/asyncBatchAnalyze?language=unk&detectOrientation=true

Hi All,
I was hoping if you can help me. I was trying to run the DeepRule repo. I have ran it before; it was working just fine. I tried using it recently, I'm getting the stated error below.
When I give the command
python manage.py runserver 8800
the upload photo page works fine but when I upload a photo, the following error shows up.
On Website
HTTPError at
401 Client Error: PermissionDenied for url: https://westus2.api.cognitive.microsoft.com/vision/v2.0/read/core/asyncBatchAnalyze?language=unk&detectOrientation=true

Request Method:
POST

http://<IP_ADDRESS>:8800/

2.2.5

HTTPError

401 Client Error: PermissionDenied for url: https://westus2.api.cognitive.microsoft.com/vision/v2.0/read/core/asyncBatchAnalyze?language=unk&detectOrientation=true

/home/sidharth/anaconda3/envs/DeepRule/lib/python3.6/site-packages/requests/models.py in raise_for_status, line 940

/home/sidharth/anaconda3/envs/DeepRule/bin/python

3.6.5

['/home/sidharth/Documents/DeepRule2',  '/home/sidharth/anaconda3/envs/DeepRule/lib/python36.zip',  '/home/sidharth/anaconda3/envs/DeepRule/lib/python3.6',  '/home/sidharth/anaconda3/envs/DeepRule/lib/python3.6/lib-dynload',  '/home/sidharth/.local/lib/python3.6/site-packages',  '/home/sidharth/anaconda3/envs/DeepRule/lib/python3.6/site-packages',  '/home/sidharth/anaconda3/envs/DeepRule/lib/python3.6/site-packages/azureml/_project/vendor']

Tue, 4 Jan 2022 20:32:49 +0000

On Terminal
We met some errors!
Internal Server Error: /
Traceback (most recent call last):
File "/home/sidharth/anaconda3/envs/DeepRule/lib/python3.6/site-packages/django/core/handlers/exception.py", line 34, in inner
response = get_response(request)
File "/home/sidharth/anaconda3/envs/DeepRule/lib/python3.6/site-packages/django/core/handlers/base.py", line 115, in _get_response
response = self.process_exception_by_middleware(e, request)
File "/home/sidharth/anaconda3/envs/DeepRule/lib/python3.6/site-packages/django/core/handlers/base.py", line 113, in _get_response
response = wrapped_callback(request, *callback_args, **callback_kwargs)
File "/home/sidharth/Documents/DeepRule2/server_match/view.py", line 30, in get_group
plot_area, image_painted, data, chart_data = test('static/target.png', min_value_official=min_value, max_value_official=max_value)
File "/home/sidharth/Documents/DeepRule2/test_pipeline.py", line 232, in test
title2string, min_value, max_value = try_math(image_path, cls_info)
File "/home/sidharth/Documents/DeepRule2/test_pipeline.py", line 164, in try_math
word_infos = ocr_result(image_path)
File "/home/sidharth/Documents/DeepRule2/test_pipeline.py", line 130, in ocr_result
response.raise_for_status()
File "/home/sidharth/anaconda3/envs/DeepRule/lib/python3.6/site-packages/requests/models.py", line 940, in raise_for_status
raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 401 Client Error: PermissionDenied for url: https://westus2.api.cognitive.microsoft.com/vision/v2.0/read/core/asyncBatchAnalyze?language=unk&detectOrientation=true
[05/Jan/2022 15:01:43] "POST / HTTP/1.1" 500 114613
BTW it API and key are in the ocr.py file.
My best guess is, the error coming because this project is using some Microsoft API and the key written in the system expired starting of 2022. Please let me know if that is the case or if I'm missing something. I will appreciate it if you can help. Please let me know if you have any questions for me.

subscription_key = "ad143190288d40b79483aa0d5c532724"
vision_base_url = "https://westus2.api.cognitive.microsoft.com/vision/v2.0/"
ocr_url = vision_base_url + "read/core/asyncBatchAnalyze"
headers = {'Ocp-Apim-Subscription-Key': subscription_key, 'Content-Type': 'application/octet-stream'}
params = {'language': 'eng', 'detectOrientation': 'true'}
you may need to replace the Microsoft OCR key, the key in the original code may just be expired.

Is there any way around running the code without using the Microsoft OCR key?
Can we skip it?
Or by any chance can you update the code with a new key?
Thank you so much.


Update .gitignore

I applied in the python .gitignore template and added in the custom definitions. This is useful for working with a venv, so the venv is not added.

Please add these to your system gitignore rather than the project one since they're applicable to all projects that you work on.


update friend name from ark to velero

update Ark to use the brand new name Velero
also update the Velero logo

cc @kris-nova @xmudrii


Cannot correctly read cell display text | 不能正确读出某些单元格显示值

Hi,
Thank you for your great work brings us such a great library.
I've to read cell's display text instead of value, after a little research, I think I can use DataFormatter.FormatCellValue but I found some display text cannot be read correctly. I attached my test file (data.xlsx), cells have problem are A10, A12, A14.
data.xlsx
My test code is simple, just get these cell object
var cell = A10, A12, A14;
var df = = new DataFormatter(CultureInfo.InvariantCulture);
var text = df.FormatCellValue(cell);

Thank you in advance!

Try this
`                        short columnFormat = sheet.GetColumnStyle(columnIndex).DataFormat;
short cellFormat = cell.CellStyle.DataFormat;
if (columnFormat > 0 && cellFormat == 0)
{
cell.CellStyle.DataFormat = columnFormat;
}
                    string showValue = dataFormatter.FormatCellValue(cell);

`

This is not correct. You cannot a new empty DataForamtter. Instead, you should reference the existing DataFormatter binded to the cell.

Can you show me a demo code?


Cleanups once Debian:9 goes EOL

Soon (2022-06-30) Debian:9 (Stretch) goes EOL
https://endoflife.date/debian
At this point we can disable the Debian:9 builds.  We will also have no need to support CMake < 3.8, as this is the last platform that requires it.  The oldest version of GCC and Clang that we must support may get a bump too.

Fixed by #9405


Use actions/changed-files instead of jitterbit/get-changed-files

jitterbit/get-changed-files is not anymore maintained and use deprecated APIs
Switch to actions/changed-files or git diff is required.
https://github.com/jitterbit/get-changed-files/issues/65

I am looking to add the same functionality to studiorack-registry
I noticed you were using https://github.com/marketplace/actions/get-all-changed-files instead of https://github.com/marketplace/actions/changed-files
Came here to create an issue, but you beat me to it :)

I am looking to add the same functionality to studiorack-registry I noticed you were using https://github.com/marketplace/actions/get-all-changed-files instead of https://github.com/marketplace/actions/changed-files
Came here to create an issue, but you beat me to it :)

Hi Kim, thanks for the intent !
I've seen you're working on "hardening" things for studiorack registry. I'll keep an eye on changes 😉

This is what I ended up with:
https://github.com/studiorack/studiorack-registry/blob/main/.github/workflows/validate.yml#L22


Show message when constraint scores are unavailable

Moved from https://github.com/macarthur-lab/gnomad_browser/issues/37
Originally created by @mattsolo1 on Tue, 14 Mar 2017 16:06:10 GMT

When constraint scores are unavailable, display message "Constraint scores unavailable due to [reason]."

I was too quick to close this... the reason why constraint scores are unavailable is not shown.

We don't have information about why constraint was not calculated for a transcript.


cargo fmt --all

As the title says...

@wally-the-cartographer merge

Merge requested by authorized user SirVer. Merge queue now has a length of 1.

@wally-the-cartographer merge


fix panic when file/dir is deleted

Sometimes when a file/dir which is being watched is deleted a panic will occur. This appears to be because there is an assumption that if an error returns true for IsNotExist() that it should be castable to os.PathError, but this evidently is not always the case, causing a panic.

Hi @radovskyb any plans to review and merge this anytime soon? I'm hitting the same issue as @tul

@radovskyb  +1 to merge this change as is. It's really annoying for directories with dense create and delete operations.


feat!: Drop support for Node 16

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 99.677%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
245

Relevant Lines:
245

💛  - Coveralls


Add-Member : Cannot add a member with the name "Version" because a member with that name already exists

When running New-BcContainer with the -includeTestToolkit switch enabled, the script fails with the error message:
Add-Member : Cannot add a member with the name "Version" because a member with that name already exists. To overwrite
the member anyway, add the Force parameter to your command.
At C:\Users\MichaelScott\OneDrive - Caracal Lynx
Ltd\Documents\WindowsPowerShell\Modules\BcContainerHelper\2.0.14\AppHandling\Sort-AppFilesByDependencies.ps1:49
char:30
+            $_ | Add-Member -NotePropertyName 'Version' -NotePropertyValue ...
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (@{AppId=63ca2fa...0-172fef340d3f}:PSObject) [Add-Member], InvalidOpe
   rationException
    + FullyQualifiedErrorId : MemberAlreadyExists,Microsoft.PowerShell.Commands.AddMemberCommand

Run the same command without the switch enabled, runs perfectly.
Scripts used to create container and cause the issue
artifacturl = Get-BcArtifactUrl -storageAccount bcinsider -type Sandbox -country gb
New-BcContainer `
        -accept_eula `
        -containerName $containername `
        -artifactUrl $artifacturl `
        -credential $credential `
        -auth $auth `
        -assignPremiumPlan `
        -useSSL `
        -installCertificateOnHost `
        -updateHosts `
        -includeAL `
        -includeTestLibrariesOnly `
        -includeTestToolkit

Full output of scripts
BcContainerHelper is version 2.0.14
BcContainerHelper is running as administrator
Host is Microsoft Windows 11 Enterprise - Unknown/Insider build
Docker Client Version is 25.0.3
Docker Server Version is 25.0.3
WARNING: Ignoring custom format, because both --format and --quiet are set.
Fetching all docker images
Fetching all docker volumes
Using image mcr.microsoft.com/businesscentral:10.0.20348.2322
Creating Container BC25-0-DEV
Version: 25.0.16618.0-GB
Style: sandbox
Multitenant: Yes
Platform: 24.0.16518.0
Generic Tag: <IP_ADDRESS>
Container OS Version: 10.0.20348.2322 (unknown)
Host OS Version: 10.0.22631.3155 (Unknown/Insider build)
Using hyperv isolation
Using locale en-GB
Disabling the standard eventlog dump to container log every 2 seconds (use -dumpEventLog to enable)
Files in C:\ProgramData\BcContainerHelper\Extensions\BC25-0-DEV\my:
- AdditionalOutput.ps1
- MainLoop.ps1
- SetupNavUsers.ps1
- SetupVariables.ps1
- updatehosts.ps1
Creating container BC25-0-DEV from image mcr.microsoft.com/businesscentral:10.0.20348.2322
1324598ac3d83341748762b23283628ee9044f85cb7aef1e3f20afea5ee6ca29
Waiting for container BC25-0-DEV to be ready
Using artifactUrl https://bcinsider.azureedge.net/sandbox/25.0.16618.0/gb
Using installer from C:\Run\240
Installing Business Central: multitenant=True, installOnly=False, filesOnly=False, includeTestToolkit=False, includeTestLibrariesOnly=False, includeTestFrameworkOnly=False, includePerformanceToolkit=False, appArtifactPath=c:\dl\sandbox\25.0.16618.0\gb, platformArtifactPath=c:\dl\sandbox\25.0.16618.0\platform, databasePath=c:\dl\sandbox\25.0.16618.0\gb\BusinessCentral-GB.bak, licenseFilePath=c:\dl\sandbox\25.0.16618.0\gb\Cronus.bclicense, rebootContainer=True
Installing from artifacts
Starting Local SQL Server
Starting Internet Information Server
Copying Service Tier Files
c:\dl\sandbox\25.0.16618.0\platform\ServiceTier\Program Files
c:\dl\sandbox\25.0.16618.0\platform\ServiceTier\System64Folder
Copying PowerShell Scripts
c:\dl\sandbox\25.0.16618.0\platform\WindowsPowerShellScripts\Cloud\NAVAdministration
c:\dl\sandbox\25.0.16618.0\platform\WindowsPowerShellScripts\WebSearch
Copying Web Client Files
c:\dl\sandbox\25.0.16618.0\platform\WebClient\Microsoft Dynamics NAV
Copying ModernDev Files
c:\dl\sandbox\25.0.16618.0\platform
c:\dl\sandbox\25.0.16618.0\platform\ModernDev\program files\Microsoft Dynamics NAV
Copying additional files
Copying ConfigurationPackages
C:\dl\sandbox\25.0.16618.0\gb\ConfigurationPackages
Copying Test Assemblies
C:\dl\sandbox\25.0.16618.0\platform\Test Assemblies
Copying Extensions
C:\dl\sandbox\25.0.16618.0\gb\Extensions
Copying Applications
C:\dl\sandbox\25.0.16618.0\platform\Applications
Copying Applications.GB
C:\dl\sandbox\25.0.16618.0\gb\Applications.GB
Copying dependencies
Importing PowerShell Modules
Restoring CRONUS Demo Database
Setting CompatibilityLevel for tenant on localhost\SQLEXPRESS
Exporting Application to CRONUS
Removing Application from tenant
Modifying Business Central Service Tier Config File for Docker
Creating Business Central Service Tier
Installing SIP crypto provider: 'C:\Windows\System32\NavSip.dll'
Starting Business Central Service Tier
Importing license file
Copying Database on localhost\SQLEXPRESS from tenant to default
Taking database tenant offline
Copying database files
Attaching files as new Database default
Putting database tenant back online
Mounting tenant database
Mounting Database for default on server localhost\SQLEXPRESS with AllowAppDatabaseWrite = False
Sync'ing Tenant
Tenant is Operational
Stopping Business Central Service Tier
Installation took 440 seconds
Installation complete
Initializing...
Setting host.docker.internal to <IP_ADDRESS> in container hosts file (copy from host hosts file)
Setting gateway.docker.internal to <IP_ADDRESS> in container hosts file (copy from host hosts file)
Setting kubernetes.docker.internal to <IP_ADDRESS> in container hosts file (copy from host hosts file)
Setting host.containerhelper.internal to <IP_ADDRESS> in container hosts file
Starting Container
Hostname is BC25-0-DEV
PublicDnsName is BC25-0-DEV
Using NavUserPassword Authentication
Creating Self Signed Certificate
Self Signed Certificate Thumbprint 278A514A34C50890D7868A92BD2F6D5E68C77D7E
DNS identity BC25-0-DEV
Modifying Service Tier Config File with Instance Specific Settings
Starting Service Tier
CertificateThumprint 278A514A34C50890D7868A92BD2F6D5E68C77D7E
Registering event sources
Creating DotNetCore Web Server Instance
Using application pool name: BC
Using default container name: NavWebApplicationContainer
Copy files to WWW root C:\inetpub\wwwroot\BC
Create the application pool BC
Create website: NavWebApplicationContainer with SSL
Update configuration: navsettings.json
Done Configuring Web Client
Enabling Financials User Experience
Dismounting Tenant
Mounting Tenant
Mounting Database for default on server localhost\SQLEXPRESS with AllowAppDatabaseWrite = False
Sync'ing Tenant
Tenant is Operational
Creating http download site
Setting SA Password and enabling SA
Creating admin as SQL User and add to sysadmin
Creating SUPER user
Assign Premium plan for ADMIN
Container IP Address: <IP_ADDRESS>
Container Hostname  : BC25-0-DEV
Container Dns Name  : BC25-0-DEV
Web Client          : https://BC25-0-DEV/BC/?tenant=default
Dev. Server         : https://BC25-0-DEV
Dev. ServerInstance : BC
Dev. Server Tenant  : default
Setting BC25-0-DEV to <IP_ADDRESS> in host hosts file
Setting BC25-0-DEV-default to <IP_ADDRESS> in host hosts file
Setting BC25-0-DEV-default to <IP_ADDRESS> in container hosts file

Files:
http://BC25-0-DEV:8080/ALLanguage.vsix
http://BC25-0-DEV:8080/certificate.cer

Container Total Physical Memory is 8.5Gb
Container Free Physical Memory is 5.1Gb

Initialization took 191 seconds
Ready for connections!
Reading CustomSettings.config from BC25-0-DEV
Certificate with thumbprint 278A514A34C50890D7868A92BD2F6D5E68C77D7E imported successfully
Creating Desktop Shortcuts for BC25-0-DEV
Synchronizing Permissions Mock on default
App successfully synchronized
Installing Permissions Mock on default
App successfully installed
Synchronizing Test Runner on default
App successfully synchronized
Installing Test Runner on default
App successfully installed
Synchronizing Any on default
App successfully synchronized
Installing Any on default
App successfully installed
Synchronizing Library Assert on default
App successfully synchronized
Installing Library Assert on default
App successfully installed
Skipping app 'C:\Applications.GB\Microsoft_Permissions Mock_25.0.16618.0.app' as it is already installed
Synchronizing Library Variable Storage on default
App successfully synchronized
Installing Library Variable Storage on default
App successfully installed
Add-Member : Cannot add a member with the name "Version" because a member with that name already exists. To overwrite
the member anyway, add the Force parameter to your command.
At C:\Users\MichaelScott\OneDrive - Caracal Lynx
Ltd\Documents\WindowsPowerShell\Modules\BcContainerHelper\2.0.14\AppHandling\Sort-AppFilesByDependencies.ps1:49
char:30
+ ...        $_ | Add-Member -NotePropertyName 'Version' -NotePropertyValue ...
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (@{AppId=63ca2fa...0-172fef340d3f}:PSObject) [Add-Member], InvalidOpe
   rationException
    + FullyQualifiedErrorId : MemberAlreadyExists,Microsoft.PowerShell.Commands.AddMemberCommand
...

Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Version    Name                                Repository           Description
-------    ----                                ----------           -----------
6.0.6      BcContainerHelper                   PSGallery            PowerShell module, which makes it easier to wor.

does it happen all the time? Yes
did it use to work? The first time I used that option
Device - Windows 365 cloud PC

Please ignore this issue; I was running the wrong version of BCContainerHelper. Sorry.

Please ignore this issue; I was running the wrong version of BCContainerHelper. Sorry.

Thanks for notifying us


Add a missing character from zh documents

The Kubernetes Master is a collection of three processes that run on a single node in your cluster

The chinese word corresponding to the word node is 节点 , which looks like missing one character.

/check-cla

I've signed CLA.

/check-cla

/lgtm
/approve


sc apps: Dashboard for visualizing how spread-out pods are across nodes

What this PR does / why we need it: See #1345
Which issue this PR fixes #1345
Special notes for reviewer: TBD
Add a screenshot or an example to illustrate the proposed solution:

Checklist:

[x] Added relevant notes to WIP-CHANGELOG.md
[x] Proper commit message prefix on all commits
[ ] Updated the public facing documentation
Is this changeset backwards compatible for existing clusters? Applying:

[x] is completely transparent, will not impact the workload in any way.
[ ] requires running a migration script.
[ ] will create noticeable cluster degradation.
E.g. logs or metrics are not being collected or Kubernetes API server
will not be responding while upgrading.
[ ] requires draining and/or replacing nodes.
[ ] will change any APIs.
E.g. removes or changes any CK8S config options or Kubernetes APIs.
[ ] will break the cluster.
I.e. full cluster migration is required.

Chart checklist (pick exactly one):

[ ] I upgraded no Chart.
[x] I upgraded a Chart and determined that no migration steps are needed.
[ ] I upgraded a Chart and added migration steps.

Pipeline config (if applicable)
If you change some config options (e.g. add/rename variable or change the default value) you may need to update the config used by the pipeline in pipeline/config.

@davidumea

Problem is that most of the infra providers that we use at least have all nodes in the same zone.

My intent was to do add a second dashboard for zones in a second PR. Are you saying it can be skipped?

My intent was to do add a second dashboard for zones in a second PR.

Okay, I think it would be nice to have it in the same dashboard, that way the user can find all information about how a deployment is spread in one place.

Are you saying it can be skipped?

No I don't think it should be skipped :slightly_smiling_face:. I meant that maybe we want to do something extra here, possibly sort out/not show the data if there only is one zone in the environment. If more than one zone exists then it should be obvious if multiple replicas of a deployment are in the same zone.

Pushing an updated version where you can pick between node or zone skew.

The "pods per node" graph, is that stacked? I think I would prefer if that was not stacked. I think that it might then be easier to see how many pods are on each node.

The "pods per node" graph, is that stacked? I think I would prefer if that was not stacked. I think that it might then be easier to see how many pods are on each node.

Not stacked currently, so lines with the same number of pods overlap.

If they are not stacked, then something seems wrong with the last screenshot you sent. For rook-ceph it is reporting a skew of 2, but looking at the graph I think it should be 6-3=3. Am I missing something?

This is shown per deployment/statefulset, right?

Currently per namespace.
Didn't you have it working for deployment/statefulset earlier? This could still be useful, but it would be much better to see the skew per deployment/statefulset since that is what kubernetes should be balancing.

If they are not stacked, then something seems wrong with the last screenshot you sent. For rook-ceph it is reporting a skew of 2, but looking at the graph I think it should be 6-3=3. Am I missing something?

That's shown per node but the view is set to zone granularity, so it's the difference between 8 pods in zone 'a' and 6 pods in zone 'b'.

Didn't you have it working for deployment/statefulset earlier? This could still be useful, but it would be much better to see the skew per deployment/statefulset since that is what kubernetes should be balancing.

Sorta, but I had trouble keeping all the labels trough joins with variable granularity. Trying again this morning, I made some progress on that.

After talking with @viktor-f I'll try to rearrange the per-Namespace graphs to loop over ReplicaSet / StatefulSet instead

Now with per-ReplicaSet/StatefulSet calculations:

Updated first comment with new screenshot. Dropped multi-select for namespaces because it caused wrong results. Dropped the overview for similar reason.


centos : This platform is not yet supported by forever-service

My platform is
cat /etc/*release
CentOS release 5.10 (Final)

Can you paste contents of /etc/system-release file

It has not /etc/system-release file

so which release file it has in /etc/ for which you have put the content?

/etc/redhat-release

should I create system-release which copied of redhat-release

Oh ok. A quick word around would be to copy redhat-release file to /etc/system-release file and it should work. Looks like older release of CentOS never had /etc/system-release file on which forever-service is relying to check the OS.


RTC ID?

The PCF8523, DS1307 and DS3231 RTCs have the same id (0x68).

[ ] Can the I2C master tell them apart?
[ ] Shall be in different I2C buses?

There is no need to tell the DS1307/DS3231 apart date/time registers are identical, and the same code can be used to read/adjust the time.
It seems that the PCF8523/PCF8563 also share the same registers, which will solve #2.
For the time being will use a single bus and try to read/update the DS1307/DS3231 first and PCF8523/PCF8563 second


Add PHP-Fusion topic

Thank you for contributing! Please confirm this pull request meets the following requirements:

[x] I followed the contributing guidelines: https://github.com/github/explore/blob/master/CONTRIBUTING.md
[x] I have no affiliation with the project I am suggesting (as a maintainer, creator, contractor, or employee).

Which change are you proposing?

[ ] Suggesting edits to an existing topic or collection
[x] Curating a new topic or collection

Curating a new topic or collection

[x] I've formatted my changes as a new folder directory, named for the topic or collection as it appears in the URL on GitHub (e.g. https://github.com/topics/[NAME] or https://github.com/collections/[NAME])
[x] My folder contains a *.png image (if applicable) and index.md
[x] All required fields in my index.md conform to the Style Guide and API docs: https://github.com/github/explore/tree/master/docs

PHP-Fusion is a lightweight open source CMS written in PHP.

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

Hi @RobiNN1  thanks for the contribution. Unfortunately, as per our contribution guidelines, we don't allow self promotion if you are a maintainer of the project:

Unaffiliated with the project (not self-promoting as e.g. a maintainer, creator, contractor or employee)


Annotate for trim/AOT (in)compatibility

Adds annotations to propagate trim and AOT compatibility warnings to callers. This way, a trimmed/AOT-compiled app referencing Newtonsoft.Json will get warnings at the places where it calls into Newtonsoft.Json, instead of warnings that point at Newtonsoft.Json internals.
Adds a net8.0 target, because the AOT analyzer requires net7.0+.
I'm not setting up a test app because I don't want to be responsible for some ad-hoc infrastructure or script that does the validation - but I did check this change against a test app locally. I think relying on the analyzer is good enough in this case because the goal isn't to make the code trim-compatible, but to surface existing trim incompatibilities earlier in the development cycle for consumers. If we do miss some warnings it's not a big problem since the problems will still be caught when publishing an app, as they are today.
@eerhardt @agocke @michalstrehovsky @jtschuster

@sbomer I pushed some changes to improve JToken writing and ToString.
It looks like they're safe as long as there are no converters. I added new overloads without converters and used from ToString.

Looks good to me. Thanks!


feat: order models, slices, and metrics by name per default

Description
Think we should order models, slices, and metrics per default. This is also helpful to group them since similar models also typically have a similar name (e.g. GPT). Also makes it so they're next to each other in charts when all models is selected.
Now it's somewhat random which model appears first.
fix ZEN-413

Is this not in the order they are uploaded in? or reverse?
For systems I imagine reverse chronological order makes the most sense in most situations, but maybe we can add that later

The only reason you usually get a consistent sort order is that the database usually executes the query in the same way.  It's so consistent, in fact, that eventually most developers forget that you can't actually rely on this order.

So, currently we might get something like this but not necessarily. I also don't know if chronological even makes the most sense, maybe sometimes, but you often want similarly named models next to each other (e.g. first I want to see the GPT model family then the Mistral models, etc.)

SGTM! Eventually I think we'll want a more sophisticated model picker with multiple fields you can sort by, search, etc. so maybe a different problem.


Fixes in the LaTeX pretty-printer

[x] Use the RandomVariables' _op_name attribute for pretty printing
[x] Move the graph's output inside gathered
[x] Remove the numbers' trailing zeros using {0:g}

Closes #187. Once we have support information in AePPL we can also change this in the pretty printer. I left a test that I marked as xfail as a reminder.

I can reformat the numbers here although that should be done in Aesara upstream. It could do with a good clean; singledispatch could simplify the logic a lot there.

The changes can be
S_rv = at.random.invgamma(0.5, 0.5, name="S")
Y_rv = at.random.normal(0.0, at.sqrt(S_rv), name="Y")

$$
\begin{equation}
\begin{gathered}
S \sim \operatorname{invgamma}\left(0.5, 0.5\right),  \in \mathbb{R}
\
Y \sim \operatorname{N}\left(0.0, {\sqrt{S}}^{2}\right),  \in \mathbb{R}
\end{gathered}
\
Y
\end{equation}
$$

I can reformat the numbers but that change should ultimately be propagated in Aesara upstream.
The whole printing system could do with a good clean. For instance usingsingledispatch to get a text representation of Ops could simplify the logic a lot there and make the whole system more extensible than it is today. We should really only have to dispatch the formatter on RandomVariables, and maybe a few other things.

We used singledispatch in symbolic-pymc and it worked fairly well (see here).  I had planned on such a refactoring in Aesara, but never got around to it.

I don't have merging rights on AePPL btw


Statement list width & vertical bar problem

On some settings now the vertical bar does not show:

On other settings, it shows, but the function buttons of the statement list are partly hidden:

This is more a problem of the list not getting the full with of the box @Ptrhnk


Added: Disable pinch zoom on iOS

This pull request adds a new hook called useDisableIOSPinchZoomOnTouchDevices to the frontend codebase. The hook is responsible for preventing pinch zoom gestures on iOS devices. It is used in the App component to disable pinch zoom on touch devices. Additionally, the pull request includes necessary file changes to implement the hook and add the touch-action property to disable pinch zoom on iOS.
Additionally, the css-property touch-action: manipulation; is added to the body root class in order to prevent touch actions.
Resolves #1274

LGTM. Unfortunately, cannot test it.

Same, I will have to test this on my phone on staging 😅


Add support for PDB files

This is amazing. It would be great if the PDB information (when available) could also be utilized.

Thank you for the positive feedback on our work. Right now, our scope is confined to the Linux x86_64 platform. However, our team is actively developing a a more versatile llm decompiler that will support various platforms and settings. Keep watching this space for updates, and thanks again for your attention!

Hi @albertan017, is the team also planning on adding other architectures such as ARM64?

Hi @albertan017, is the team also planning on adding other architectures such as ARM64?

Yes, we're working on more supporting more architectures. Our final goal is to develop something like Decompiler-ALL: Support popular languages/platforms and settings (e.g., decompile multiple functions).


How to add plugins via npm?

None of the Matter.js plugins are included in the matter-js bundle installed via npm, and these plugins don't seem to have their own packages on npm, so how are we expected to install and use plugins with webpack/browserify + npm?

Just finishing work on the plugin boilerplate first, then plugins will have their own separate packages. Hopefully this weekend!
In the meantime, try making a local copy and then change the reference when they are released.

These plugins are now fully packaged!

matter-attractors
matter-wrap

Also check out the matter-plugin-boilerplate.


Adding developers section with data working group co-chairs only

Progress towards #278.
See discussion in issue #280.

Sorry for the noise, tried unsuccessfully to squash commits, will create a new pull request.


No ticket: Share project-specific PhpStorm config through git

Same idea as https://github.com/thebiggive/matchbot/pull/448

I don't think this is useful but not hugely important - so Noel if you have any concerns feel free to close.


Multiple migration paths

We develop a framework which we include as a module in several projects. Both the framework and the projects contain phinx migrations.
Is there a way I can set up two (or more) migration paths in phinx.yml (one for framework migrations and one for project-specific migrations), so phinx looks in both paths for migrations? If not, it would be a great feature! The documentation talks about a "migration path" (singular) which implies that only one path can be specified.
It would be important that migrations are run in the sequence of the timestamps in the migration file names (as oposed to running all framework migrations and then project-specific migrations.
Here's an example of what I'd like to do, where migrations 1-4 are run in sequence:
phinx.yml (specify two paths, e.g. separated with semicolon)
paths:
    migrations: %%PHINX_CONFIG_DIR%%/migrations;%%PHINX_CONFIG_DIR%%/vendor/our-company/our-framework/migrations

./migrations
<PHONE_NUMBER>0000_migration_1.php
<PHONE_NUMBER>0000_migration_3.php
./vendor/our-company/our-framework/migrations
<PHONE_NUMBER>0000_migration_2.php
<PHONE_NUMBER>0000_migration_4.php

Has this been solved in the meantime?

Also starring hard to this, any chance to get this @robmorgan ?

@robmorgan , what is the status of this feature?

Ping @robmorgan.
High demand. Any feedback for us whatsoever?

As I couldn't wait for this feature, I have implemented a workaround. I just copy all migration files from different paths into a single migrations directory during the deployment process. It's quite easy and works well for us.

I wrote a little helper. Script loads app config (Horus\Config) and creates a Phinx config each time it is called.
It's just my workaround but I am very happy. The creation of migrations is no pain anymore and it looks for migrations in all my bundle paths. Just take a look at the script...
#!/usr/bin/env php
<?php
$root = dirname(__DIR__);
require_once $root . '/vendor/autoload.php';

use Symfony\Component\Console\Event\ConsoleCommandEvent;
use Symfony\Component\Console\ConsoleEvents;
use Symfony\Component\EventDispatcher\EventDispatcher;
use Phinx\Console\PhinxApplication;
use Phinx\Config\Config as PConfig;
use Horus\Config;

$dispatcher = new EventDispatcher();
$dispatcher->addListener(ConsoleEvents::COMMAND, function (ConsoleCommandEvent $event) use ($root) {

    $command = $event->getCommand();
    if ($command instanceof Phinx\Console\Command\AbstractCommand) {

        $input       = $event->getInput();
        $output      = $event->getOutput();
        $environment = 'development';

        if ($input->hasOption('environment')) {
            if (null !== $input->getOption('environment')) {
                $environment = $input->getOption('environment');
            } else {
                $input->setOption('environment', $environment);
            }
        }

        $config = Config::load($root . '/app/config/config_' . $environment . '.yml');

        if (empty($config->toArray())) {
            $output->writeln(sprintf('<error>Unknown environment "%s"</error>', $environment));
            exit;
        }

        switch (get_class($command)) {
            case 'Phinx\Console\Command\Create':
            case 'Phinx\Console\Command\SeedCreate':
                $migrations = 'Resources/migrations';
                $seeds      = 'Resources/migrations/seeds';
                break;
            default:
                $migrations = $root . '/src/*/*/Resources/migrations';
                $seeds      = $root . '/src/*/*/Resources/migrations/seeds';
        }

        $config = new PConfig([
                'environments' => [
                    'default_migration_table' => 'phinxlog',
                    'default_database'        => $environment,
                    $environment              => [
                        'adapter'  => 'mysql',
                        'host'     => $config->database->host,
                        'name'     => $config->database->dbname,
                        'user'     => $config->database->username,
                        'pass'     => $config->database->password,
                        'port'     => 3306,
                        'charset'  => 'utf8'
                    ]
                ],
                'paths' => [
                    'migrations' => $migrations,
                    'seeds'      => $seeds
                ],
            ]);

        $command->setConfig($config);
    }
});

$phinx = new Phinx\Console\PhinxApplication();
$phinx->setDispatcher($dispatcher);
$phinx->run();

Hi... It works!
Setup your variable paths.migrations like this:
{/path/to/my-project/db/migrations,/path/to/my-project/vendor/*/*/db/migrations}
So, multiple directories with wildcard expansion in curly brackets, delimited by comma.
I hope, it helps :)

+1

Yes the problem is the creating and to get the timestamp in the file name

👍

Look at this https://github.com/lulco/phoenix. Syntax is very similar to phinx but has this feature implemnted already.

fyi this is possible since 0.7.0 but Unfortunately it wasn't mentioned in the release notes.. ;)
https://github.com/robmorgan/phinx/blob/v0.7.0/tests/Phinx/Config/_files/valid_config.php

Confirmed. This feature was initially part of the 0.7 dev branch (https://github.com/robmorgan/phinx/pull/1022/commits/28d9afb01175704062277c5365f610a08fed21d4) It was merged as part of merging the dev branches into master.
Sorry for the lack of release notes.


Refused to set unsafe header "Referer" when you use remote debug

Hi,
I'm trying to debug the data that comes back from the module.  No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'http://localhost:8081' is therefore not allowed access.

If I disable the remote debugger, the module works fine. Another thing to note: my API Key doesn't have any restrictions.
How to use this module and debug it properly?

The issue really comes from: XMLHttpRequest cannot load https://maps.googleapis.com/maps/api/place/autocomplete/json?&input=Pa&key=AIzaSyDAXXXXXXXXXX5Dw&language=en&types=%28cities%29. No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'http://localhost:8081' is therefore not allowed access.
My API key has no restriction, so I guess the issue is I'm using Google Places API Web Service, as mentionned by the docs. However, this key can't be used on client-side (that's what I'm trying to do with the remote debugger).
Any solutions to debug JS while using this module?

Hey, sorry for the late response.
Looks like you are using the GLOBAL.originalXMLHttpRequest to debug, right? So... it is not going to work because of the CORS issue. And I think there's no need for a debug prop in the component since this is more for lib maintainers usage.
I'd recommend you to these steps to try to get more info about the request:

Comment the http debug line
Go to your local node_modules
Open the react-native-google-places-autocomplete folder
Open GooglePlacesAutocomplete.js
Log the result of the line 240 or find this line.
Profit. :)

Let me know if you need help!

@guilhermepontes Thanks for your quick answer. Actually, I realize I could simply use react-native log-ios / log-android to do this.

cool! :)

@guilhermepontes I working with axios and FormData, but when I using this debugging options it's crashed  in others cases works good. Do you know any others approach to debug networks? Thanks in advance :)

I am facing this same issue, but i am converting an existing react-native app to react-native-web app.
We had separate keys for both mobile app and legacy web app, i am using the web app key in the react-native-web version, still same error. If i use restricted mobile app key, error is still the same.
This is odd, any help will be appreciated. Thanks.

How about this status, bros.... :D Look my case the same with @ziaulrehman40 . I also convert from mobile app to web app. And get this error when using it :(

Any update here? Looks like this uses the server-side library. Right now this is unusable on web.... Needs to be replaced with the client library for web implementations.
See this thread:
https://github.com/googlemaps/google-maps-services-js/issues/59


Reference asset directly

Is it possible to reference an asset only by its id, not through an entry? Or is this discouraged?
I try to create a website where the start page shows an image. An editor should be able to define/change this image, so I thought it's best as a contentful asset. Anyway, I don't have an entry for the start page, so I can't reference the asset.

@eggapauli beyond referencing it's CDN url directly, we don't have this feature yet. We'll put it on our roadmap though

@kylemac Let me know if this is still an intended feature and I can add it to #55

@declandewet this one still isn't a priority. we'll check in on this' viability after our newest version


--strict mode not working, even with via additional_swiftlint_args

I just wanted to setup swiftlint via danger using this gem, but when I specify the additional_swiftlint_args: '--strict' as implemented in #59, nothing actually changes, still all warnings show up as warnings and danger doesn't fail, although I have setup both fail_on_error: true on this gem and even --fail-on-errors=true on danger itself. Even setting max_num_violations to 0 doesn't fail if there's only a warning. So basically there seems to be no way to make the CI fail on warnings as of now. 😞
Note that the --strict parameter does not convert warnings to errors, instead it just ensures the overall swiftlint command fails by returning with a non zero status code (see this comment). That's probably why --strict is doing nothing ...

I have the same problem. I wrote like the following in Dangerfile:
swiftlint.lint_files inline_mode: true, fail_on_error: true, additional_swiftlint_args: '--strict'

Has anyone found a (maybe temporary) solution of the problem in the meantime?

I have same issue.
Did anyone find the solution for this problem? Thank you.


Refactorings:

Needed to supply Useragent
using cookie container
CodeVerifier random string also using uppercase range for stronger security
demo/test using proper async/await
disposing of responses
using c# 8 syntax for cleaner code

Thanks for the PR @ramonsmits - however it isn't working for me. The HTTP request is hanging on step 1.
Are you able to successfully authenticate?

I'll check today. Pretty sure it worked and heard my car honk :-).
Thing is that I made these changes because it wasn't working for me. My account has 2FA enabled.

I've got 2FA on too. Your branch definitely wasn't working when I tested it, although Tesla seem to be changing things rapidly so it's entirely possible it worked at the time. My changes are pretty minimal so it shouldn't be too hard to merge them into your branch. Originally it seemed that each call required different headers, but it seems they are all pretty consistent so we could probably centralise the code to set up the HttpClient.

Depends on #9 which is already included in this branch.


[vsc-extension] [C4GT] Stencil VSCode extension

Create VSCode Extension for Stencil - Data Verification Feature
Description
The goal is to develop the first version of a VSCode extension for Stencil, aiming to enhance the Developer Experience (DX). The primary feature for this initial release is to verify data intended for seeding in the database against the model generated by the Prisma client. This verification will be conducted using CSV or JSON files.
Goals

VSCode Extension Setup: Establish the basic structure and setup for the VSCode extension tailored for Stencil.

Data Verification Feature: Implement the feature that allows users to verify data from CSV/JSON files against the Prisma client-generated model.

Acceptance Criteria

Extension Setup: Successfully set up the VSCode extension project with Stencil integration.

Data Verification Functionality: Implement functionality within the extension that verifies data present in CSV/JSON files against the Prisma client-generated model.

User Interface (UI): Develop a simple and user-friendly UI within VSCode to facilitate the data verification process.

Testing: Ensure the data verification feature works as intended and is capable of identifying inconsistencies between the provided data and the Prisma model.

Documentation: Provide clear documentation on how to use the VSCode extension, specifically focusing on the steps for data verification and any relevant considerations. Include your docs as a ReadME.md in your PR. The docs will be pushed to official stencil documentation after successful merging of your PR.

Checklist (To be filled by the developer)

[ ] Set up the basic structure and configuration for the Stencil VSCode extension.

[ ] Implement the data verification feature, allowing users to check data from CSV/JSON files against the Prisma client-generated model.

[ ] Design and implement a simple and user-friendly UI within VSCode for the data verification process.

[ ] Perform thorough testing to ensure the accuracy and effectiveness of the data verification feature.

[ ] Document the usage of the VSCode extension, emphasizing the steps for data verification and any important information for users.

Product Name
Stencil
Organisation Name
SamagraX
Tech Skills Needed
TypeScript
Mentors
@RyanWalker277
Complexity
Hard
Domain
Category
Miscellaneous
Sub Category
VsCode Extensions

Hello @RyanWalker277, I'm keen on taking up this issue. I would like more information, particularly regarding step 2, which involves implementing the data verification feature. This feature aims to enable users to validate data from CSV/JSON files against the Prisma client-generated model. Can you provide more guidance to help me proceed with this task?

Hey @Apurv428, so basically the VS-Code extension should be able to seed the data in the db. The user will provide a CSV including the data to be seeded. It should then verify if the CSV is according to our prisma model. If the validation succeeds, then it should seed the csv data into the database in the appropriate table.

Thanks for the explanation. So, according to my understanding
The user uploads a CSV, we validate its compatibility with our Prisma model, and if it aligns correctly, we insert it into the necessary database table.
Please correct me if I am wrong.
Do you have any recommendations on how to begin with this? Also, could you share the database format to get acquainted with it?

Yes the understanding is correct. You can refer to this to begin:
https://www.freecodecamp.org/news/making-vscode-extension/
The database format is provided along with the code in the schema.prisma

Thank you for providing the reference. If I have any questions, I'll reach out to you.

Hi @RyanWalker277
Could you please provide more details on the first point of the acceptance criteria, specifically regarding the extension setup? Are you referring to a basic setup similar to configuring a VSCode extension, or does it involve additional steps or considerations?

Hey @Apurv428, in the first iteration it should be a basic extension which provides the functionality to seed db from csv data. There would be more features and enhancements after this feature is initialised. Let me know when you are ready to create a pull request, I'll create a separate repository for the extension. You'll have to create a PR on that.

Okay, I have created a basic extension and am working on the upload functionality.
Thank you!

Hi @RyanWalker277 ,  how can I get the format of database? Is it same as the Role model?

Hey @Apurv428 ,
The database schema would be dynamic, we'll need to parse the schema.prisma file to figure that out. The Role schema is just for reference.

So, similar to Role model, there may be other models mentioned in that file?

Yes

Sure!

Hi,
Could y'll provide the following details about database seeding?
Where should we retrieve the database details from? Are they obtained in the same way from Prisma? Could you share the env variable values related to the database? Additionally, will the table names correspond to the model names?

Hey @Apurv428,
Database seeding is still something we are figuring out how to implement into stencil, we can add that as a part of this extension once it has been discussed and a seeding strategy has been agreed upon.

Okay, thank you!

@techsavvyash  I understand the problem, and I believe I have the necessary skills. Could I have the opportunity to work on this?

is it still available to work on ? Can I start work on it?
If yes, then what is the current status of extension?

@Abhi-0001 the first version of the vs code extension was very generously contributed by @Apurv428. The issue is still open to work on by adding more functionality enhancing the developer experience.

Thanks @techsavvyash, so Can you give me instructions about setting the code up in my machine (as this is my first time in vscode extension development, though I know typescript) and the requirements I would have to full fill for next version.

Refer #78

@techsavvyash by adding some more features, will I be getting the points?

yes @thekavikumar

provide immediate feedback on whether the data matches the model, reducing errors and debugging time.
@techsavvyash sir is this feature good to implement?

@techsavvyash
Is the PR and the work around this issue completed? If yes, will you pls close the issue? Thanks.

@VedantKhairnar this is a long running ticket.


Add table of endpoints + relative links

Hi @vloschiavo ,
I was looking at the documentation you've got the other night to see if there were any endpoints I could document and wondered if you'd be up for a bit of a re-structure of the readme to have a table of endpoints + links to the relevant detail behind them?
I've got an example in a branch here: https://github.com/vls29/powerwall2/tree/structure-readme#api-reference
I wanted to run it past you and find out your thoughts before I went ahead and re-structured everything only to find you weren't interested in the change :)
If you do think it's worth continuing with, I can work on it over the next few weeks and send you a pull request when the majority has been tabulated.
Regards
vls29

Yes!  I like!  Happy to merge when you're done.

Yes!  I like!  Happy to merge when you're done.


[visualstudio] Refactor script

Make the script more readable, mostly by:

using the Product class,
a little bit of renaming and documentation.

Got only expected version updates, merging.


Add parameter to specify file compression for csv files

Add file compression param while writing csv/parquet. For parquet, it compression defaults to snappy in parquet.py def write_table but in function to_csv, there is no param to specify file compression
ie. pandas.py
def to_csv(
self,
dataframe,
path,
database=None,
table=None,
partition_cols=None,
preserve_index=True,
mode="append",
procs_cpu_bound=None,
procs_io_bound=None,
)
Request is to add a new parameter for specifying the file compression eg - gzip
https://pandas.pydata.org/pandas-docs/stable/reference/api/pandas.DataFrame.to_csv.html
compression: str, default ‘infer’
Compression mode among the following possible values: {‘infer’, ‘gzip’, ‘bz2’, ‘zip’, ‘xz’, None}. If ‘infer’ and path_or_buf is path-like, then detect compression from the following extensions: ‘.gz’, ‘.bz2’, ‘.zip’ or ‘.xz’. (otherwise no compression).

Hey @nitin-kakkar, thank you for open this issue.
I've added compression options for Pandas.to_parquet and prepare the structures to do the same with Pandas.to_csv in the future.
But for now the second is blocked by a Pandas limitation.
How we aim to write compressed files directly to S3, we must wait until Pandas unlock the option of write compressed files in memory (instead of disk).


Updated to 1.6 and some more

Hi. I am not sure if you still maintain this example application. I updated for myself for a talk, so thought I'll share if you are interested. Else just close the PR :)
What I have done:

Upgrade to v1.6
Add OpenAPI and Swagger UI to the REST Endpoints
Add MicroProfile Health
Add MicroProfile Metrics
Add MicroProfile GraphQL
Added links (of above) for convenience (in the GUI)
Added tests

I am happy to maintain this app if this is something we want.

@cescoffier what's your take on this, it's your app?
If we plan to maintain it, we should:

set up GH actions
get rid of using separate branches for each supported database and use a profile

This repo is used by the openshift team if I'm not mistaken. So we should give it some love and implement the automation. I'm not sure about the automation as it can complexify a bit the deployment (you would need to pass the profile to openshift which leads to configuring the s2i). I would need the input from openshift.

@sshaaf what's your take on this?

We have been using this demo in the runtimes team as well for customers and events.
Using Openshift to delpoy: I would suggest we should keep the deployment open, for folks who might want to deploy it elsewhere. If needed we could just provide some docs on how to deploy to openshift. But demo on a local machine should just be good enough IMHO.
I also ended up working on my own fork and updating it there; it might help to maintain this since it helps all of us forks.

Hi @sshaaf, @gsmet, @cescoffier  - so we can merge this here, and then let's use the issues to add all the other things we want to do, like:

set up GH actions
get rid of using separate branches for each supported database and use a profile
Document how to demo this on OpenShift (local and remote)

etc.
w.d.y.t ?

I had a quick chat with Shaaf.
We should experiment with profiles an see how far we can go and document
the OpenShift deployment in the readme.
So, we can merge this PR and play with profiles to see how it goes.
Le jeu. 16 juil. 2020 à 19:17, Phillip Krüger<EMAIL_ADDRESS>a
écrit :

Hi @sshaaf https://github.com/sshaaf, @gsmet https://github.com/gsmet,
@cescoffier https://github.com/cescoffier - so we can merge this here,
and then let's use the issues to add all the other things we want to do,
like:

set up GH actions
get rid of using separate branches for each supported database and
use a profile
Document how to demo this on OpenShift (local and remote)

etc.
w.d.y.t ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/quarkusio/todo-demo-app/pull/10#issuecomment-659552344,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADCG7O6DZMM4BENJ36Y643R34Y2DANCNFSM4O2WRZPA
.

Cool. I can not merge. So sure who can. I'll create issues for the items as discussed. Thanks :)

Merged!

Thanks :)