Having trouble setting up command for development

I am trying to setup up command for development. When I run npm install followed by npm run dev I get the following error message shown in the npm-debug.log file.
0 info it worked if it ends with ok
1 verbose cli [ 'C:\\Program Files\\nodejs\\\\node.exe',
1 verbose cli   'C:\\Program Files\\nodejs\\node_modules\\npm\\bin\\npm-cli.js',
1 verbose cli   'run',
1 verbose cli   'dev' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose node symlink C:\Program Files\nodejs\\node.exe
5 verbose run-script [ 'predev', 'dev', 'postdev' ]
6 info predev<EMAIL_ADDRESS>7 info dev<EMAIL_ADDRESS>8 verbose unsafe-perm in lifecycle true
9 info<EMAIL_ADDRESS>Failed to exec dev script
10 error<EMAIL_ADDRESS>dev: `NODE_ENV=development ./node_modules/.bin/babel-node ./dev-env/dev.js`
10 error Exit status 1
11 error Failed at the<EMAIL_ADDRESS>dev script.
11 error This is most likely a problem with the Command package,
11 error not with npm itself.
11 error Tell the author that this fails on your system:
11 error     NODE_ENV=development ./node_modules/.bin/babel-node ./dev-env/dev.js
11 error You can get their info via:
11 error     npm owner ls Command
11 error There is likely additional logging output above.
12 error System Windows_NT 6.1.7601
13 error command "C:\\Program Files\\nodejs\\\\node.exe" "C:\\Program Files\\nodejs\\node_modules\\npm\\bin\\npm-cli.js" "run" "dev"
14 error cwd C:\Users\%username%\Documents\Github\slash-command
15 error node -v v0.10.35
16 error npm -v 1.4.28
17 error code ELIFECYCLE
18 verbose exit [ 1, true ]

It looks like you have a very old version of node and NPM. Can you try updating to node >4 and npm > 3? If that doesn't help, please reopen!

I am getting this error message shown in the npm-debug.log file after upgrading node & npm.
0 info it worked if it ends with ok
1 verbose cli [ 'C:\\Program Files\\nodejs\\node.exe',
1 verbose cli   'C:\\Program Files\\nodejs\\node_modules\\npm\\bin\\npm-cli.js',
1 verbose cli   'run',
1 verbose cli   'dev' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose run-script [ 'predev', 'dev', 'postdev' ]
5 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>6 silly lifecycle<EMAIL_ADDRESS>no script for predev, continuing
7 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>8 verbose lifecycle<EMAIL_ADDRESS>unsafe-perm in lifecycle true
9 verbose lifecycle<EMAIL_ADDRESS>PATH: C:\Program Files\nodejs\node_modules\npm\bin\node-gyp-bin;C:\Users\%username%\Documents\Github\slash-command\node_modules\.bin;C:\Program Files\nodejs;C:\ruby215\bin;C:\Windows\system32;C:\Windows;C:\Windows\System32\Wbem;C:\Program Files\Intel\WiFi\bin\;C:\Program Files\Common Files\Intel\WirelessCommon\;C:\Windows\System32\WindowsPowerShell\v1.0\;C:\Program Files (x86)\Gow\bin;C:\Program Files\nodejs\;C:\ProgramData\chocolatey\bin;C:\Windows\System32\WindowsPowerShell\v1.0\;C:\Windows\System32\WindowsPowerShell\v1.0\;C:\Windows\System32\WindowsPowerShell\v1.0\;C:\Program Files (x86)\Git\cmd;C:\PROGRA~2\Oracle\VirtualBox;C:\Users\%username%\AppData\Local\Pandoc\;C:\Python27;C:\Python33;C:\Python27\Scripts;C:\Python27\lib\site-packages\django\bin;C:\ruby193\bin;C:\ruby200\bin;C:\Users\%username%\AppData\Roaming\npm;C:\Users\%username%\bin;C:\Users\%username%\AppData\Local\atom\bin;C:\Program Files (x86)\Microsoft VS Code\bin
10 verbose lifecycle<EMAIL_ADDRESS>CWD: C:\Users\%username%\Documents\Github\slash-command
11 silly lifecycle<EMAIL_ADDRESS>Args: [ '/d /s /c',
11 silly lifecycle   'NODE_ENV=development ./node_modules/.bin/babel-node ./dev-env/dev.js' ]
12 silly lifecycle<EMAIL_ADDRESS>Returned: code: 1  signal: null
13 info lifecycle<EMAIL_ADDRESS>Failed to exec dev script
14 verbose stack Error<EMAIL_ADDRESS>dev: `NODE_ENV=development ./node_modules/.bin/babel-node ./dev-env/dev.js`
14 verbose stack Exit status 1
14 verbose stack     at EventEmitter.<anonymous> (C:\Program Files\nodejs\node_modules\npm\lib\utils\lifecycle.js:239:16)
14 verbose stack     at emitTwo (events.js:100:13)
14 verbose stack     at EventEmitter.emit (events.js:185:7)
14 verbose stack     at ChildProcess.<anonymous> (C:\Program Files\nodejs\node_modules\npm\lib\utils\spawn.js:24:14)
14 verbose stack     at emitTwo (events.js:100:13)
14 verbose stack     at ChildProcess.emit (events.js:185:7)
14 verbose stack     at maybeClose (internal/child_process.js:850:16)
14 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:215:5)
15 verbose pkgid<EMAIL_ADDRESS>16 verbose cwd C:\Users\%username%\Documents\Github\slash-command
17 error Windows_NT 6.1.7601
18 error argv "C:\\Program Files\\nodejs\\node.exe" "C:\\Program Files\\nodejs\\node_modules\\npm\\bin\\npm-cli.js" "run" "dev"
19 error node v5.10.1
20 error npm  v3.8.3
21 error code ELIFECYCLE
22 error<EMAIL_ADDRESS>dev: `NODE_ENV=development ./node_modules/.bin/babel-node ./dev-env/dev.js`
22 error Exit status 1
23 error Failed at the<EMAIL_ADDRESS>dev script 'NODE_ENV=development ./node_modules/.bin/babel-node ./dev-env/dev.js'.
23 error Make sure you have the latest version of node.js and npm installed.
23 error If you do, this is most likely a problem with the Command package,
23 error not with npm itself.
23 error Tell the author that this fails on your system:
23 error     NODE_ENV=development ./node_modules/.bin/babel-node ./dev-env/dev.js
23 error You can get information on how to open an issue for this project with:
23 error     npm bugs Command
23 error Or if that isn't available, you can get their info via:
23 error     npm owner ls Command
23 error There is likely additional logging output above.
24 verbose exit [ 1, true ]

Can you just running this?
$ ./node_modules/.bin/babel-node ./dev-env/dev.js

I'm on Windows by the way. I'm getting '.' is not recognized as an internal or external command, operable program or batch file. when I run the command.

Is there a line number attached to that? My guess is that it has to do with
relative path syntax differing between Mac and Windows.
On Mon, Apr 11, 2016 at 2:27 PM ishu3101<EMAIL_ADDRESS>wrote:

I'm on Windows by the way. I'm getting '.' is not recognized as an
internal or external command,
operable program or batch file. when I run the command.
—
You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub
https://github.com/jessepollak/command/issues/39#issuecomment-208570514

There is no line number attached to that. When I run the above command on git bash on windows. I get this error message.
Overriding 'node_modules/webpack/lib/JsonpMainTemplate.runtime.js'
Overriding 'node_modules/webpack/hot/log-apply-result.js'
fs.js:842
  return binding.mkdir(pathModule._makeLong(path),
                 ^

Error: ENOENT: no such file or directory, mkdir 'c:\Users\%username%\Documents\Github\slash-command\release\build'
    at Error (native)
    at Object.fs.mkdirSync (fs.js:842:18)
    at Manifest.prepareBuildDir (index.js:37:8)
    at Manifest.run (index.js:15:10)
    at Object.<anonymous> (dev.js:12:10)
    at Module._compile (module.js:413:34)
    at loader (c:\Users\ishu\Documents\Github\slash-command\node_modules\babel-cli\node_modules\babel-register\lib\node.js:126:5)
    at Object.require.extensions.(anonymous function) [as .js] (c:\Users\%username%\Documents\Github\slash-command\node_modules\babel-cli\node_modules\babel-register\lib\node.js:136:7)
    at Module.load (module.js:357:32)
    at Function.Module._load (module.js:314:12)

Did you ever resolve this build issue? Please reopen if you're still struggling with it — would love to help you get started.

No, still struggling with the setup. See my previous message.

@ishu3101 I just pushed a fix (above) that should resolve this issue. can you pull and try building? thanks!

What do you mean by "Add  command/build"  in your instruction for adding the development build of the extension to your browser.

When I run npm install. I am getting this message.
npm WARN optional Skipping failed optional dependency /chokidar/fsevents:
npm WARN notsup Not compatible with your operating system or architecture<EMAIL_ADDRESS>npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.

OK, I just pushed another commit. Try pulling and npm installing again.
I mean:

Navigate to chrome://extensions
Check the 'Developer Mode' box
Click 'Load unpacked extension' and load the command/build directory

I still get the same message after running npm install. I also don't see the command/build directory.


Support for importers plugins

This PR adds support for adding additional importers to the pysteps interface without modifying the pysteps source code.  Instead, the users can add custom importers to pysteps by installing external packages, called plugins, that are automatically "discovered" and integrated to pysteps.
How it works?
Let's suppose that we have a plugin (python package), named importer_abc_xyz, that contains the following importers:
importer_abc_xyz.importers.import_abc_yyy
importer_abc_xyz.importers.import_abc_zzz
When the plugin is installed, it adds entry_points to the python installation advertising the components of this plugin to other packages (in our case, pysteps).
Next, when pysteps is imported, it discovers the available entry_points that belong to the pysteps.plugins.importers group (the group name is defined in the plugin). These discovered importers are added as attributes to the io.importers module, and they are registered to the io.get_method interface.
After the external importers are registered, they become part of the pysteps interface and can be used together with the other available importers.
Plugin project template
To make the creation of the plugin easier, a cookiecutter project template, available at https://github.com/aperezhortal/cookiecutter-pysteps-plugin, can be used to quickly create a plugin project by executing:
$> cookiecutter https://github.com/aperezhortal/cookiecutter-pysteps-plugin
and follow the interactive instructions.
At the moment, this template is used only for testing purposes. However, if we decide to go forward with the idea, the documentation needs to be improved, and perhaps, and move the template to the main PySTEP Github (https://github.com/pySTEPS).
PR progress

[x] Minimal working example
[ ] Improve documentation
[ ] Add more tests (multiple plugins, tests unexpected behaviors, etc.)
[ ] Improve the plugin template documentation

Hi @aperezhortal
This looks like a great idea!
I still have to familiarize myself with the details and give it a try, but I like it already ;-)
So, if I understand correctly, to install a new custom importer, I would have to first run the cookiecutter, then implement my importer in the template, right? After this, how is the registration into pysteps done? Simply using pip install?
One more question, I assume new plugins won't be shared with the rest of the community, correct? These will essentially remain private methods.
Once again, great idea, thanks!

So, if I understand correctly, to install a new custom importer, I would have to first run the cookiecutter, then implement my importer in the template, right?

The first step is creating the plugin (a python package). To facilitate this, cookiecutter (and the cookiecutter-pysteps-plugin template) is used to create the skeleton of the plugin project. Then, the new importers need to be added to the barebone project.

After this, how is the registration into pysteps done? Simply using pip install?

When you install the package using pip install or python setup.py install, it add the entry points defined in the setup.py script to the python distribution. The installation adds the entry points a .txt file located in, for example,  <path_to_my_python_environment>/site-packages/<name_of_my_importer_plugin>-.dist-info/entry_points.txt.
Then, when pysteps is importer, it calls the pysteps.io.interface.discover_importers() function that looks for the entry points available in the python distribution and registers the ones that belong to the "pysteps.plugins.importers" group (the group name was defined in the setup.py).
The main advantage of this approach is that the plugins will keep working if the pysteps version is updated without any user intervention.

One more question, I assume new plugins won't be shared with the rest of the community, correct? These will essentially remain private methods.

This is the best part, in my opinion. The users who create the plugins have ownership and they are free to distribute or not the packages. For example, if an organization uses "in-house" tools to read files that cannot be openly distributed, they can internally distribute the plugin.
(I will include the information of these anwers in the plugins documentation)

This sounds really great, I personally think we should go ahead. What would be the next steps? How can we help?

Great!
The plugin support is already in place. The next step should be writing a tutorial to add a new plugin and write the template documentation. Help with this is more than welcome :bowtie:
I will push the first draft soon with the minimal installation instructions to use it as a starting point. We can then incrementally improve the tutorial to make it as clear and easy to follow as possible.


Fix zoom and implement test

The feature of zooming where selected on the dashboard did not function, and this pull request fixes the issue.

Manually merged into main.


Copying Files, flatten = false

Issue when tried to use flatten = false

Not sure why you're getting that, but just do:
mix.copy('./resources/assets/icons/, 'public', false);


SSE EventSource Server

First of all, thanks for creating the release with the PATCH method included (as well as the other updates for Xcode 11)!
I'm wondering what your recommended approach is for implementing a SSE event server. If it were up to me, I'd just use WebSockets, but I need to mimic a 3rd party service that uses SSE in order to achieve a compatible server for my clients (in case you're not familiar, here's an example of client-side EventSource code that I'll need to make this server work with).
I successfully implemented SSE for Vapor (surprisingly, there was nothing out there already for this...):
https://gist.github.com/JUSTINMKAUFMAN/032e3d004350d9a4dae2cd767085753a
...but put simply, the code just needs to handle the routing of a regular HTTPRequest (i.e. a get at some path) by creating a keep-alive socket with a stream that data can be written to.
I see you have nice-looking TCPSocket and DataStream implementations already, but I'm not certain if they can be used out-of-the-box to achieve this?
Thanks again for your work on this cool library!

You are very welcome!
Server side events is a bit of a tricky one. I think the easiest way to do this is by diving into this segment of code of the Server class (line 150):
extension Server: HTTPConnectionDelegate {
  /// Raised when a HTTP connection receives an incoming request.
  public func connection(_ httpConnection: HTTPConnection, handleIncomingRequest request: HTTPRequest, error: Error?) {
    workerQueue.addOperation { [weak self, weak httpConnection] in
      guard let self = self, let httpConnection = httpConnection else { return }

      // Get a response for the request
      let response = self.workerProcess(request: request, error: error)

      // Send the response or close the connection
      self.connectionsQueue.async {
        if let response = response {
          httpConnection.send(response: response, toRequest: request)
        } else {
          httpConnection.close(immediately: true)
        }
      }
    }
  }

When there is an incoming HTTP request the Server moves the request onto the workerQueue so that it can handle concurrent requests. Instead of adding the operation to the workerQueue you could wrap the request into your own class and send multiple responses over time.
I don't think it is possible to override the connection(_ httpConnection:request:error:) method, but I could add a method in between if you like, mark it as open, so that you can override it.

Thanks for the quick reply! I forked the library and got it working pretty easily.

So it turns out my code for holding on to SSE client connections and sending them event data is not working as expected.
Here's what I'm trying to get the connection (in my forked version of the repo):
override open func handleConnectionRequest(_ connection: HTTPConnection, incomingRequest request: HTTPRequest, error: Error?) {
    if request.method == .GET && request.uri.path.contains("meval") {
        sse.addStream(request: request, connection: connection)
    } else {
        super.handleConnectionRequest(connection, incomingRequest: request, error: error)
    }
}

And here's the SSEStream class:
import Cocoa
import Foundation
import Telegraph

public class SSEStream: NSObject {
    public let id: String
    private let queue: OperationQueue
    private var connection: HTTPConnection
    private var socket: TCPSocket
    private var response: HTTPResponse
    private var request: HTTPRequest

    private var defaultEvents: [UInt8] {
        return Array(
            payload(
                SSEEvent(type: .put, featureFlags: FeatureFlag.allFlags())
            ).utf8
        )
    }

    public init(request: HTTPRequest, connection: HTTPConnection) {
        self.connection = connection
        self.id = request.uri.path
        self.request = request
        self.queue = OperationQueue()

        let (socket, _) = connection.upgrade()
        self.socket = socket
        socket.read()

        response = request.response(.ok)
        response.headers.contentType = "text/event-stream"
        response.headers.transferEncoding = "chunked"
        response.headers.cacheControl = "no-cache"
        response.headers.connection = "keep-alive"
        response.headers["Access-Control-Allow-Origin"] = "*"
        response.body = Data()

        connection.send(response: response, toRequest: request)

        super.init()
    }

    public func publish(_ event: SSEEvent)  {
        queue.addOperation { [unowned self] in
            self.socket.write(data: event.data!)
        }
    }
}

It seems the client disconnects and attempts to reconnect when I call connection.send(...).
And if I don't call send, none of my socket.write(...) calls are received on the client side.
I'd be super grateful for any advice!

Nevermind, I figured it out! Here's the SSEStream code I ended up with:
import Cocoa
import Foundation
import Telegraph

public class SSEStream: NSObject {
    public let id: String
    private let queue: OperationQueue
    private let socket: TCPSocket
    private let connection: HTTPConnection
    private var request: HTTPRequest
    private var timer: Timer?

    public init(request: HTTPRequest, connection: HTTPConnection) {
        self.queue = OperationQueue()
        self.queue.maxConcurrentOperationCount = 1
        self.id = request.uri.query ?? "SSE-ID"
        self.request = request
        self.connection = connection
        self.socket = connection.upgrade().0
        super.init()
        setup()
    }

    public func publish(_ event: SSEEvent)  {
        queue.addOperation { [unowned self] in
            self.socket.write(data: event.body, timeout: 10.0)
        }
    }

}

private extension SSEStream {
    private func setup() {
        connection.open()

        request.headers.connection = "keep-alive"
        request.headers.contentType = "text/event-stream"
        request.headers.transferEncoding = "chunked"
        request.headers.cacheControl = "no-cache"
        request.headers["Access-Control-Allow-Origin"] = "*"

        pulse()
    }

    @objc private func pulse() {
        queue.addOperation { [unowned self] in
            self.socket.write(data: "\n\n".utf8Data, timeout: 7.0)
        }

        if timer == nil {
            timer = Timer(
                timeInterval: 10.0,
                target: self,
                selector: #selector(pulse),
                userInfo: nil,
                repeats: true
            )
        }
    }
}

extension Array where Element: SSEStream {
    public func publish(_ event: SSEEvent) {
        self.forEach { $0.publish(event) }
    }
}

Nice! Great to see that this works!
Do you need the Objective-C compatibility? Otherwise you can use the DispatchTimer class (see Sources/Helpers) if you like. It is also a bit more performant than the regular Timer and you can use closures with it.
I should also add Access-Control-Allow-Origin to the list of built-in headers.

Access-Control-Allow-Origin is now a built-in header

I've also added an extra send method to HTTPConnection to make it easier to write raw Data. That way you won't have to use the upgrade function to get access to the socket.

That's excellent! I'll look out for the release :)

Well actually, unless you're also adding an interface for custom handling of an incoming HTTPConnection, I'll still need to use my fork :(

Perhaps I should move some of the code in Server to a Worker class.
The Worker would have the queue (now workerQueue in Server).
By replacing the Worker instance on Server with a custom Worker it would probably become a lot easier to customize request handling.
Let me know what you think.

I’d need to see the API to give substantive feedback, but that sounds like the right idea.
An approach I was considering implementing in my fork was to make connections (optionally) routable with a special conformance in the Server class.
Essentially, the most familiar/friendly and potentially least disruptive thing would be to let users route connections based on the request.
For example, say I need to setup SSE clients anytime there’s a GET request at https://localhost/sse:
server.routeConnection(.get, “/sse”, sseClientConnectionHandler)
(And you’d need to add the corresponding Server extension method)
Thoughts?

On Jul 3, 2019, at 12:35 PM, Yvo<EMAIL_ADDRESS>wrote:
Perhaps I should move some of the code in Server to a Worker class.
The Worker would have the queue (now workerQueue in Server).
By replacing the Worker instance on Server with a custom Worker it would probably become a lot easier to customize request handling.
Let me know what you think.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Yes it would be a good idea to allow a specific handler to process requests on a specific path. This probably requires a bit of a redesign of the Server class and the routing classes.
To prevent the change from becoming too big, I've modified the Server class a bit to make it easier to override the functions. With this change you can create a class that inherits from Server and override the handleIncoming(request:connection:error:) function.

That’s perfect for now. Thanks!

On Jul 3, 2019, at 2:05 PM, Yvo<EMAIL_ADDRESS>wrote:
Yes it would be a good idea to allow a specific handler to process requests on a specific path. This probably requires a bit of a redesign of the Server class and the routing classes.
To prevent the change from becoming too big, I've modified the Server class a bit to make it easier to override the functions. With this change you can create a class that inherits from Server and override the handleIncoming(request:connection:error:) function.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


[Error] NotFoundError: Failed to execute 'removeChild' on 'Node': The node to be removed is not a child of this node.

Page
https://registry.platformio.org/libraries/hwspeedy/arduino-mqtt/compatibility
Stack
at j (https://cdn.registry.platformio.org/static/js/main.34c13448.js:2:484344)
    at R (https://cdn.registry.platformio.org/static/js/main.34c13448.js:2:487010)
    at Suspense
    at div
    at div
    at ee (https://cdn.registry.platformio.org/static/js/main.34c13448.js:2:191495)
    at div
    at main
    at ne (https://cdn.registry.platformio.org/static/js/main.34c13448.js:2:194387)
    at j (https://cdn.registry.platformio.org/static/js/main.34c13448.js:2:484344)
    at O (https://cdn.registry.platformio.org/static/js/main.34c13448.js:2:486360)
    at P (https://cdn.registry.platformio.org/static/js/main.34c13448.js:2:643931)
    at j (https://cdn.registry.platformio.org/static/js/main.34c13448.js:2:484344)
    at R (https://cdn.registry.platformio.org/static/js/main.34c13448.js:2:487010)
    at B (https://cdn.registry.platformio.org/static/js/main.34c13448.js:2:486423)
    at h (https://cdn.registry.platformio.org/static/js/main.34c13448.js:2:477493)
    at o.defineProperty.value (https://cdn.registry.platformio.org/static/js/main.34c13448.js:2:594096)
    at gl (https://cdn.registry.platformio.org/static/js/main.34c13448.js:2:441638)
    at gl (https://cdn.registry.platformio.org/static/js/main.34c13448.js:2:441638)

Press Ctrl+R / Full page reload without cache.


fix: send html as attachment rather than streamed content

when mp_streamfile is used on all platforms

:tada: This PR is included in version 4.18.11 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


CubeList.copy() should be a deepcopy()

The CubeList class should be given .copy() behaviour using deepcopy() in the same way that Cube.copy() works.
Using deepcopy() was a deliberate decision as this behaviour was considered less confusing for most Iris users. No such change was made for CubeList.copy(), so now instead there is a confusion about the behaviour difference between Cube and CubeList.

Hmmm.  To make CubeList.copy() consistent with Cube.copy(), you'd be making it inconsistent with list.copy().  So it would likely still be surprising for someone. :worried:
I guess this never came up in early Iris because, under python2, list had no copy method.

Yes it has since been explained to me that the situation is more complex than it appears. I'm still not clear on the details, and whether this deserves a dev discussion or should perhaps just be closed.

There is no real consistency to be expected in Python on these things.
But if CubeList is considered to be a container class, then it should copy the way containers do : you get a new container with the same members as content (not copies of them).
CubeList is an odd thing, really, whose actual purpose is rather unclear.
Many new uses, when first encountering it, assume that the cubes contained must have some special relationship, like all having the same grid, or matching X- and Y-winds for instance.
But in fact it says nothing in particular about its contents.  It is really just a home for some special convenience behaviours, like a special printout for a list of cubes, and merge/concatenate (which really could have been delivered as utility operations rather than cubelist methods anyway, IMHO).

a container class

Mind you, that concept has no definitive meaning.  See https://stackoverflow.com/a/11576019/2615050

I may have misunderstood - was this done at #4094?

I may have misunderstood - was this done at #4094?

@rcomer that PR only concerned the returning of the correct type. It's still a shallow copy, unless I've misunderstood Python internals, which is always possible!

There doesn't seem to be much interest in this, so will close. Can reopen if desired.


castnow youtube not working, shows "idle" and YouTube logo

This is the only reason I need castnow, castnow shows idle status for few minutes and then disconnected. On TV it shows only YouTube logo.

yeh, seeing the same issue.

Dupe of #108 presumably. #108 should be re-opened as it has more information, and this should be closed.


Replace k8s.gcr.io with registry.k8s.io

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
This PR replaces all k8s.gcr.io references with registry.k8s.io. See the linked issue for more details.
Which issue(s) this PR fixes:
xref https://github.com/kubernetes/k8s.io/issues/4738
Release note:
Replace all `k8s.gcr.io` references with `registry.k8s.io`

Coverage: 83.952%. Remained the same when pulling 0d2f3b1d56c2af69f8844b0038c9940ff9bb2429 on xmudrii:registry-k8s-io into 58fa16b5733a7b955adb1cbdeaf5a3fd551b9c70 on kubernetes-sigs:master.

@ambiknai If you already push to k8s.gcr.io, then your images will be available under registry.k8s.io automatically. You can find more details about how registry.k8s.io works in the project repo.

/lgtm


Fix exposing "host" metadata

This got added to get the cherry pick for #28198 landed internally, and so should be merged prior to any imports happening.

@drfloob

Maybe we should have a test for what metadata gets exposed outside of core?


Buttons in button groups don't get blue :focus ring

Hi,
Button groups doesn't active on click on Mozilla Firefox and Safari..
Mac os operating system

I don't know about 3.3.7 but the mobile menu in v3.3.5 does not appear to allow a focus ring around the button element. In order to meet WCAG SC 2.4.7 focus visible, I changed the background by more than 4:5 ratio of contrast with the default colour.


Refactor classes to use callbacks

Each class should have its own set of callbacks. It will be easier to handle callbacks from object level if eventually will be possible to have 2 or more instances of the same class like 2x CDC or HID.

Most classes refactored with new callback system.


It was previously unclear that OrchestrationClient couldn't be used f…

…rom within an orchestration
See: https://github.com/Azure/azure-functions-durable-extension/issues/244#issuecomment-380806431

@ElanHasson : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@cgillum or @kashimiz does this note look OK? #assign:kashimiz #assign:cgillum

Thanks @ElanHasson #sign-off


Poor performance with U8 input format in concat

When doing concat with U8 input, the performance is worse than that when input data format is FP32.
Suppose performance should be better in U8 case.

Environment
Intel MKL-DNN includes hardware-specific optimizations and may behave
differently on depending on the compiler and build environment. Include
the following information to help reproduce the issue:

CPU make and model : Intel(R) Xeon(R) Gold 6148M CPU @ 2.40GHz

OS version (uname -a): Linux wlp3 3.10.0-514.el7.x86_64 #1 SMP Tue Nov 22 16:42:41 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux

Compiler version (gcc --version) : gcc (GCC) 4.8.5 20150623 (Red Hat 4.8.5-16)

CMake version (cmake --version) : cmake version 3.5.2

CMake output log

Actual behavior
U8 concat performance is worse than that in fp32 format
Expected behavior
U8 concat has better performance than FP32 concat.

In my test, there are 4 inputs: {10, 14, 14, 192}, {10, 14, 14, 208}, {10, 14, 14, 48}, {10, 14, 14, 64} in NHWC format.

Could you please provide the benchmark you were using and performance results?

Thanks a lot for your feedback, Vpirogov.
I got following test results by calling MKL-DNN in my test code.
U8U8 Concat: 0.0319 ms
Float32 Concat: 0.0269 ms
4 input data: {10, 14, 14, 192}, {10, 14, 14, 208}, {10, 14, 14, 48}, {10, 14, 14, 64}  in NHWC layout.

Could you please also provide a benchmark that you used to measure the performance and instructions on how to reproduce you results?

Closing due to lack of activity. Feel free to reopen with a reproducer.


Generate new API release

Once #229, #354, #318 and #359 is completed, we should release another version of the API!

Release generated:
https://github.com/MobilityData/mobility-feed-api/releases/tag/v1.1.0


Add credentials for NGC to README

Summary
Add additional information to user on needed step to add required credentials to NGC containers.
Additional Notes
Following #21

Should this go under https://github.com/NVIDIA/cloudai/blob/main/USER_GUIDE.md instead?

Should this go under https://github.com/NVIDIA/cloudai/blob/main/USER_GUIDE.md instead?

I believe it belongs to README, as it is a required part of the needed step for installation. Also, README is the first document you encounter when opening the repo. Some users asked me how to do that. But also USER_GUIDE can fit, so whatever you decide. which one should it be?

@TaekyungHeo please review as well


Allow to centralize cursor after navigating hunks

First of all, great job on making such a nice plugin!
And about my idea, I think it would be nice to put the cursor in the middle of the screen after navigating through hunks, just like the standard zz command in vim. I tried to make both these remaps with no avail:
vim.keymap.set('n', '[c', ':Gitsigns prev_hunk<CR>zz')
vim.keymap.set('n', '[c', ':lua require("gitsigns").prev_hunk()<CR>zz')

Maybe the prev_hunk and next_hunk functions could accept an option like "centralize" that would act like a zz after the navigation. Them we could have a mapping like this, easily combining all the options:
local gitsigns = require('gitsigns')
-- ...
vim.keymap.set('n', '[C', function() gitsigns.prev_hunk({ wrap = false, preview = true, centralize = true}) end)

I have also tried a remap just like the first one, but with O instead of zz, and it actually did the "O" action before navigating through hunks. Maybe the navigation function is async and the zz was performed on the line before moving the cursor, so thats why it did not work. I tried some clunky :sleep 100m before the zz in the keymaps, and then it worked, but it does not work very well with the preview option set to true, not to mention that adding this sleep kinda makes it feels laggy and it was more of a test then a solution.
Thanks a bunch again for all your work! (:

Something like this could work as an okay workaround I think
vim.keymap.set("n", "[C", function()
    gitsigns.prev_hunk({...})
    vim.schedule(function()
        vim.api.nvim_feedkeys("zz", "n", false)
    end)
end)

Yeah, this actually looks pretty good. I've used here and it seems to just work.

Solution above seems good enough to me.

Hi everyone,
Proposed workaround doesn't work for me (also with newer interfaces - nav_hunk("prev")).
Are there still no plans to support something like this natively?
I'd also appreciate other workarounds (other than random sleep interval :smile:) that would force these commands to execute synchronously, if anyone knows them.

Looks like new API comes with then callback which will be executed right after the command, so if anyone still needs this, it can cleanly be achieved with:
gitsigns.nav_hunk("next", {...}, function() vim.fn.feedkeys("zz", "n") end)


[main] Upgrade to latest dependencies

Cron -knative-prow-robot
/cc we don't use Prow
/assign we don't use Prow
Produced by: knative-sandbox/knobots/actions/update-deps
Details:
bumping knative.dev/hack e28525d...815cd31:
  > 815cd31 Stop performing pre-releases (# 77)
bumping knative.dev/pkg 21eb4c1...7b5ecbc:
  > 7b5ecbc Sketch out an alternate way of injecting clients/informers (# 2210)
  > 8c88fa2 Revert "Remove the unused file (# 1935)" (# 2213)
  > 50410e0 Use consistent case for "Deprecated" comments (# 2216)
  > 35bcd16 Drop redundant pointers and decoders (# 2211)
  > bf176d5 Sink two codegen passes below loop. (# 2209)
  > 2c8a7b6 Update community files (# 2208)
  > 23b0147 Update actions (# 2206)
bumping knative.dev/eventing 0c41e6d...4b18ab8:
  > 4b18ab8 Add myself to support roster (# 5640)
  > 5c08bd3 be more explicit about the semantic of ceOverrides (# 5641)
  > c183833 Resurrect source spec (# 5637)
  > 72fae58 Add trace exporting to mt broker (# 5634)
  > 41e36ee Update community files (# 5632)
  > 6296a81 Add tracing support to eventdisplay and heartbeats (# 5631)

Codecov Report

Merging #274 (22865ec) into main (7f0354e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #274   +/-   ##
=======================================
  Coverage   80.65%   80.65%
=======================================
  Files          20       20
  Lines         765      765
=======================================
  Hits          617      617
  Misses        134      134
  Partials       14       14

Continue to review full report at Codecov.

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


Fix TNG100

The current TNG100 NPE pipeline has clear issues:

Check I/O issues and the entire data pipeline.

TNG100 had a few subhalos that had mass = 0. This caused NaNs for log(mass). Addressed in 0d05840.


Why pnpm over yarn workspaces

I understand the value of pnpm over yarn in most cases. However in an expo app to me it gets a little more complicated.
From the expo monorepo docs their example uses yarn workspaces. In the section about using other tools they list out a few rules. One of which is that all dependencies must live in a node_modules directory. pnpm by default doesn't do this.

The default settings of tools like pnpm do not follow these rules. You can change that by adding an .npmrc file with node-linker=hoisted (see docs). That config option will change the behavior to match these rules.

I know there's a node-linker=hoisted option which is what this boilerplate uses. However unless my understanding is incorrect this would mean there is no symlinking. So in our use case we want to have an apps/mobile folder with the expo app and a ui package that contains many components that will be consumed by the expo app. I think this would mean for the expo app to pull in changes made in the ui package we would need to run pnpm i every time we make a change in the ui folder. This seems like a seriously degraded developer experience.
The alternative would be using yarn workspaces without the nohoist flag as mentioned in the expo docs. I believe this would allow us to develop packages in the ui folder and automatically consume them in the app folder without having to run pnpm i
This is the first production grade mobile app monorepo project I've setup so I may be misunderstanding something of course.

Hmm after doing a bit of testing it seems that create-t3-turbo will have instant updates from the packages folder with pnpm even with the node-linker=hoisted flag on.
Whats odd to me is from the pnpm docs

hoisted - a flat node_modules without symlinks is created. Same as the node_modules created by npm or Yarn Classic. Legitimate reasons to use this setting:
Your tooling doesn't work well with symlinks. A React Native project will most probably only work if you use a hoisted node_modules.

I would think without the symlink though we wouldn't get instant updates on packages. How is this working then?

I dont know the technicals of this but the packages arent being built and put into nodemodules, they are transpiled by nextjs directly.
Afaik, internal packages like this doesnt work the same as external where you have to publish to npm and download and all that.
You'll notice you have the same excellent dx with trpc for example, where changes are registered without even saving the file

Have tried the above suggestions and default install is still very broken. Prisma does not like being in a package.

Have tried the above suggestions and default install is still very broken. Prisma does not like being in a package.

No problems having prisma in a package.
Repo broke on Theos stream yesterday and i havent reverted. Undo the custom output path from his PR and it works.

Closing for now.
pnpm works fine and repo is restored to work on initial install


Slack invite link not working

Challenge http://www.freecodecamp.com/challenges/waypoint-join-our-chat-room has an issue. Please describe how to reproduce it, and include links to screen shots if possible.
In the section called Waypoint: Join Our Chatroom on the 3rd item, there is a link to join the chatroom (http://www.freecodecamp.com/api/slack)
When I click, a box appears at the top confirming an email will be sent to my account. It's been over 20 minutes and the email has not arrived. I checked the spam folder and it wasn't there either.
I'm on an iPhone 6 running iOS 8.3 using Chrome.

This is sent to the email listed in your account. This is usually populated by oauth. It may be sending your email to another account. Can you double check their and let me know if it hasn't shown up.

Looks like we are having issues with slack itself. Thanks for reporting. We will use this issue when we make progress on this.

Sure thing. Was just about to let you know I checked all other email
accounts (including spam) and did not see an email.
Thanks for the update! :)
On Sat, Jun 20, 2015 at 1:11 PM Berkeley Martinez<EMAIL_ADDRESS>wrote:

Looks like we are having issues with slack itself. Thanks for reporting.
We will use this issue when we make progress on this.
—
Reply to this email directly or view it on GitHub
https://github.com/FreeCodeCamp/freecodecamp/issues/1014#issuecomment-113790328
.

@andreagoulet Thanks for your patience. Definitely seems to be an issue on the Slack side of things. Still investigating.

We have found the issue and are looking for ways to resolve it. Thank you all for you patience.

We have moved away from Slack due to an undocumented hard limit on numbers of users. We are now on Gitter.
Please see our blog post on the subject.
We deeply apologies for the inconvenience and hope y'all have a great experience on Gitter!
Best,
-BerkeleyTrue

As of March, 2016, I'm still having problems getting things through to my Yahoo! mail.  I've checked my spam, and the message is simply not showing up.  It works fine on my iCloud account.
This is an ongoing bug.

We have moved away from Slack due to an undocumented hard limit on numbers of users. We are now on Gitter.


torbrowser-launcher as distribution platform for all TPO bundles?

torbrowser-launcher is an excellent way to get and update the TBB. Given that TorBirdy and TIMB (Tor Instant Messaging Bundle) probably have the same distribution restrictions as TBB, I was thinking that torbrowser-launcher can be extended to handle fetching and maintenance of those packages as well?
That would depend on how much effort is needed of course.
The point of this proposal is to turn torbrowser-launcher into a common platform, allowing easy and secure distribution of all TPO Tor-centric software suites for privacy oriented OSs such as TAILS, Whonix and any other distro that has it packaged in its repos.

I like this idea. It doesn't seem like TorBirdy needs this, since Thunderbird has its own way of updating add-ons. However, I'd love to have something really similar to torbrowser-launcher for Tor Messenger, once it's stable and supports auto-updates, and has a release channel.
I suspect it make be simpler and make more sense to just fork torbrowser-launcher to make, e.g. tormessenger-launcher in the future, though. They might support separate mirrors, and the settings GUI should perhaps look different.
I'll keep this open though for discussion.

Forking torbrowser-launcher to tormessenger-launcher will probably fail. As in not being very useful. Debian has a policy against code duplication. So a fork of torbrowser-launcher would not be allowed to enter official Debian repositories as it would duplicate most code of torbrowser-launcher. @intrigeri might confirm this.
Sukhbir is aware of this issue. Talked to him at the 2015 dev meeting. He agrees for tormessenger to use a similar download folder layout as TBB. And supports this use case of keeping torlauncher in this regard similar to TBB so downloaders such as torbrowser-launcher and tb-updater can be turned into more general TPO software downloaders.

Forking torbrowser-launcher to tormessenger-launcher will probably fail. As in not being very useful. Debian has a policy against code duplication. So a fork of torbrowser-launcher would not be allowed to enter official Debian repositories as it would duplicate most code of torbrowser-launcher. @intrigeri might confirm this.
Sukhbir (@azadi) is aware of this issue. Talked to him at the 2015 dev meeting. He agrees for tormessenger to use a similar download folder layout as TBB. And supports this use case of keeping torlauncher in this regard similar to TBB so downloaders such as torbrowser-launcher and tb-updater can be turned into more general TPO software downloaders.

Sukhbir (@azadi), will there be TorBirdy bundles? (I hope not...)

Hi,
It's unlikely at this stage that we will have TorBirdy bundles. We had them in the past but with the patches for the two leaks merged upstream, I feel that there is little purpose in having a bundle.
As far as the directory structure is concerned, we are currently working on porting the secure updater to Tor Messenger, which as a side effect ensures that we will have the same directory structure. (In fact, we started by unifying the directory structure to make sure the patches apply seamlessly.)

Extending torbrowser-launcher to manage TorBirdy and TIMB distribution is an interesting idea. The feasibility will depend on the required effort. This proposal aims to create a common platform for distributing Tor-centric software, enhancing accessibility and security for privacy-focused OSs like TAILS and Whonix. Let's discuss this on the GitHub repository to explore its potential and gather community input.

TorBirdy and TIMB no longer exists. There are currently no other TPO bundels other than TBB.
Maybe it could support downloading Mullvad Browser (MB).
Whonix:

https://github.com/torproject/torbrowser-launcher/issues/239


Fix issue with podman logs on fresh containers

Fixes #433

lgtm

@rh-atomic-bot r+

@rh-atomic-bot retry

Sorry looks like I've a pending comment similar to @edsantiago.  From a user perspective, I think this error is very confusing.  They've just done 'podman create ...' successfully and now when they do the 'podman log' they're told it's not been created.  I too think we should change that to started.

Someone open a PR for this. Maybe
Container logs do not exist, since the container has not be started.


Implementation of CuPy Fusion

I Implemented the fusion class for CuPy ufuncs.

#2297 includes this PR. And #2297 was merged.
Thank you!


Version Update Change for search-documents May 21

This PR is a version update PR for the search-documents SDK release for May 21.
@xirzec Please review and approve the PR.
@ramya-rao-a FYI.....

/check-enforcer reset


macOS Ventura new settings & Linux

Any plan to have an early version to support macOS Ventura as host for the new settings?
maybe an option for the Unix VM too ( the sample code provided by Apple is light and don't include the Rosseta part ...)

Added the new boot option thing that works on Ventura. For Unix support, maybe UTM makes more sense?


[request] Dark Theme

Stylish has a really great dark theme for Allo, would be great to implement on the desktop side, https://userstyles.org/styles/146643/allo-dark-theme

Seconded. Does anyone even enjoy looking at "light" themes anymore? I look at them like the horse-and-buggy: a product of their time but something best left in the past.

Hi there!
Thanks for your patience and for the link to the dark theme. I have now included both light and dark versions in the latest release. Feel free to give them a download here:
https://github.com/kelyvin/Google-Allo-For-Desktop/releases/tag/v0.0.5
Hope this helps!

Fantastic! Thanks!


ZDF: 160 kbps AAC tracks missing

Checklist

[x] I'm reporting a site feature request
[x] I've verified that I'm running youtube-dl version 2020.09.20
[x] I've searched the bugtracker for similar site feature requests including closed ones

The ZDF Mediathek is delivering audio in AAC 160 kbps VBR, but ytdl only lists 128 kbps AAC tracks with the -F parameter. I know for sure that 160 kbps must be there because the aggregator site MediathekViewWeb is listing the shows as links with 160 kbps AAC. Compare this show in ytdl with the "Hoch" quality version of the aggregator.

I beg to differ, this is not the same as #26659.

I beg to differ, this is not the same as #26659.


Add global util for exiting dirty form

I want to merge this change because it adds exit dirty form prompt

PR intended to be tested with API branch:
Screenshots

Pull Request Checklist

[ ] This code contains UI changes
[ ] All visible strings are translated with proper context including data-formatting
[ ] Attributes [data-test-id] are added for new elements
[ ] Changes are mentioned in the changelog
[ ] The changes are tested in different browsers and in light/dark mode

Test environment config

API_URI=https://qa.staging.saleor.cloud/graphql/

Task linked: SALEOR-2990 Add global util for exiting dirty form


Dart type with import fails as 'Invalid json'

Bug description
I'm trying to have the generated output import a file so that the custom type is defined.
Specs
Artemis version: 5.1.0

build.yaml:
targets:
  $default:
    builders:
      artemis:
        options:
          scalar_mapping:
            - custom_parser_import: 'package:graphbrainz_example/coercers.dart'
              graphql_type: BigDecimal
              dart_type:
                name: Decimal
                imports:
                  - 'package:decimal/decimal.dart'

It always throws here
factory DartType.fromJson(dynamic json) {
    if (json is String) {
      return DartType(name: json);
    } else if (json is Map<String, dynamic>) {
      return _$DartTypeFromJson(json);
    } else {
      throw 'Invalid json: $json';
    }
  }

I've tried forking the repo and printing the runtimeType to check what is going on but I can't get it to print anything.

Hello @jifalops, how are you doing?
We're focusing development of artemis as prerelease versions, to avoid doing a lot of breaking changes on master.
This particular issue is already solved in the beta branch, so please, try to use a prerelease version (eg. 6.12.2-beta.1) and report back!
Feel free to re-open this issue if it persists in latest prerelease version!


Improve linter

The linter now recognizes whether he has already commented on a found error. If so, it will not comment this error again.

Failure is unrelated → merging


python3Packages.kivy: fix build on Darwin

Description of changes

Add the Accelerate framework as a buildInput; and
Fix incompatible function pointer conversion errors with clang 16.

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (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.

Result of nixpkgs-review pr 262816 run on aarch64-darwin 1

8 packages built:

nsz (python311Packages.nsz)
nsz.dist (python311Packages.nsz.dist)
python310Packages.kivy
python310Packages.kivy.dist
python310Packages.nsz
python310Packages.nsz.dist
python311Packages.kivy
python311Packages.kivy.dist


Adds Dockerfile for aarch64

Adds a Dockerfile.aarch64[1] file to use for building a Docker image for the aarch64 (or arm64) cpu.  The Dockerfile.aarch64 only change is using the official python:3.4 to aarch64/python3.4.
I am using this to run Home Assistant in Docker on my Pine64.
[1] Docker uses this naming scheme within their own repository

My problem with adding extra docker files to the repo is that by accepting it, it becomes our responsibility for maintenance, like reviewing PRs that come in on it.
I rather link to the container page that you setup on Docker hub or something similar.

@mathewpeterson does this create a build variant on Docker Hub? Curious how/if Docker Hub uses the Dockerfile..

@balloob I understand the hesitation in adding this however, I think you could communicate that the team does not support or maintain that architecture. Currently, I have to update my git repo with your changes and then copy this Dockerfile and then build it. One alternative would be to place this file in a contrib directory instead of the repo root.
@lwis I don't believe Docker Hub handles different architectures natively.. yet. So, Docker Hub simply does not recognize this file.
Thanks for taking the time to comment and keep up the awesome work!

Can you move the file to the folder virtualization/Docker/ and add a note on the top that this is not officially maintained.

@mathewpeterson For reference I do the following in my personal Alpine container;
RUN pip install --upgrade homeassistant==0.28.2
# RUN pip install git+https://github.com/home-assistant/home-assistant.git@dev

I'm not sure what you're gaining by adding a new Dockerfile here if you're not gaining any automated builds - won't that still require manual intervention?

Closing this as I realised that just adding all these images is not going to help. If you want to make an alternative image available, you're welcome to generate and publish it on the Docker registry and we can link to it.


Prepare Zexe for recursion

Updated. This is a bundle of changes to prepare Zexe for recursion. This PR consists of several independent changes:

Add the CycleEngine, which some of us may have heard of. This is to represent a cycle of pairing-friendly curves.

/// A cycle of pairing-friendly elliptic curves.
pub trait CycleEngine:
    Sized
    + 'static
    + Copy
    + Debug
    + Sync
    + Send
where
    <Self::E2 as PairingEngine>::G1Projective: MulAssign<<Self::E1 as PairingEngine>::Fq>,
    <Self::E2 as PairingEngine>::G2Projective: MulAssign<<Self::E1 as PairingEngine>::Fq>,
{
    type E1: PairingEngine;
    type E2: PairingEngine<
        Fr = <Self::E1 as PairingEngine>::Fq,
        Fq = <Self::E1 as PairingEngine>::Fr,
    >;
}

Add a prepared verifier inNIZKVerifierGadget and implement it for Groth16 and GM17.

Add a new trait ToConstraintFieldGadget and implement it for a number of group and field types.

pub trait ToConstraintFieldGadget<ConstraintF: PrimeField> {
    fn to_field_gadgets<CS: ConstraintSystem<ConstraintF>>(
        &self,
        cs: CS,
    ) -> Result<Vec<FpGadget<ConstraintF>>, SynthesisError>;
}

The test has passed. Please leave comments.

Could you explain why the change to PrimeField is necessary? Couldn't you just bound it downstream by ConstraintF: PrimeField?
Either way, how difficult do you think would it be to split that into a separate PR so that it's easier to see the main logic changes in this PR? If it's too difficult then it's fine to leave it in here
Finally, I'm not entirely convinced by the need for a separate PreparedVerifierGadget; is it not possible to simply add a new function to the existing VerifierGadget? I feel like some logic could be unified that way (because the first thing that VerifierGadget does is call vk.prepare())

Very good questions.

For the first one, I remember I encountered an issue when the constraint field might not be PrimeField but I forgot. I will try to recollect the information. If it is not needed (i.e., can be avoided by other ways), I will avoid such massive changes.
But anyway, I will separate those changes from this commit, thereby simplifying the matter.

I can merge it into VerifierGadget.

The changes have been made. I updated the PR text above.

Just woke up. Will apply the changes soon.

Suggestions applied. CI passed.
A new implementation of ToConstraintField for Boolean is made, which gets rid of the one constraint (from conditional_select).
https://github.com/scipr-lab/zexe/pull/241/files#diff-fd3cb57c15b5fca1c3c09d44fcbf134dR856

I have the feeling that prepared group elements are not going to be used unless when they are hardcoded? (i.e., alloc_constant)?

repost in case of missing:
I have the feeling that prepared group elements are not going to be used unless when they are hardcoded? (i.e., alloc_constant)

Since alloc is untrusted, alloc_input for prepared arguments is unnecessarily costly.

Thanks for the commit!

You could have a commitment to the prepared gadgets (if you have a very efficient hash function)
Either way, I've moved it into todo-s for the time being.
By they way, I've followed your suggestion in the refactor to make AllocGadget have a "main" method which takes in a enum parameter
enum AllocType {
	Const,
	Witness,
	Input,
}

This should make the duplication disappear in the refactor.

Great!


A crash in the test suite forces the whole suite to be re-run in the future

When a crash occurs during testing, it causes the next round of testing to run all tests in the project, even if only the play button for a single file is clicked to run that file only. This is very time-consuming, as my project has many tests across the project.

Hi, thanks for the report! The way I see it there's probably two likely causes: a) this is an issue with Xcode/XCTest and runtime test case generation, or b) Kiwi has a bug. Currently I'm inclined to think it's the former, but the best way to validate that would probably be to work out if other testing frameworks similar to Kiwi have the same problem.
@modocache have you seen this kind of thing on Quick?

@sharplet I should've been more clear: it's not just after a crash, it's after a clean and rebuild and other  cache invalidations.
After looking into the issue more, it appears that prior to running the test suite, Xcode doesn't see any methods for any of the test cases, I would assume because of Kiwi's unusual syntax. So when you run the tests, it is first figuring out what tests are in these actual files and running the tests. Then when you run a later test, it already knows what methods there are and so it doesn't seem to mind just testing one file. This is speculation.

You can tell this because when you click the arrow to expand the tests for a given file, when looking in the test navigator, no methods are shown until after you run the full test suite.

This really seems to me like an XCTest / Xcode test navigator issue. This isn't really something we can fix in Kiwi unfortunately. If this is still an issue for you, I'd recommend you file a radar.


Skip DevTest Lab tests

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[ ] I adhere to the Command Guidelines.

View a preview at https://prompt.ws/r/Azure/azure-cli/6153
This is an experimental preview for @microsoft users.

Please rebase with the latest on dev branch. There was a fix just merged to fix a Travis CI change.

@derekbekoe and @tjprescott ?

Pending me figuring out the right way to automate this with DevTest lab team.


Added UCSC to university list

@zdolson I am sorry, the domain ucsc.edu is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


Pods support

We're building a Flutter Plugin for BLE. On Android side we're using other NordicSemiconductor's library and on the iOS side we've wanted to use this one. However Flutter does not support SPM yet and it won't change soon https://github.com/flutter/flutter/issues/126005
Could you add CocoaPods support for this project? It will make it more generally available and allow us two integrate two of your libraries.

Now you can use library with pod 'IOS-BLE-Library', '~> 0.1.2'


added privacy policy page, rendered image in error page, fixed all buttons and added input class field of contact us page in global css

Description

Issue Ticket Number
Fixes #(issue_number)

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[ ] I have followed the contributing guidelines of this project as mentioned in CONTRIBUTING.md
[ ] I have checked to ensure there aren't other open Pull Requests for the same update/change?
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes needed to the documentation

@Iron-Thunder Fix the deployment errors


Java doc broken link fix and minor update

Description
The Java on IntelliJ doc has a broken link that this commit fixes, and a couple of minor text edits.
Changes

Fix a broken link to the Java Infer API examples
Text edits to the readme

@lanking520 @andrewfayres please take a look.
@mxnet-label-bot add [pr-awaiting-review]

Changes look good. Thanks for the link fix. CI seems to of failed and needs to be restarted.

@mxnet-label-bot add [pr-awaiting-review]

Since this was a minor fix I went ahead and included it in #13405

Resolved by @andrewfayres in #13405 - closing.


Update shelljs to 0.8.5

The dependency shelljs has a HIGH vulnerability in version 0.8.4 that is resolved in 0.8.5
This is detailed in this snyk report
See also (CVE-2022-0144)[https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2022-0144]

@tomwillis608
Thanks for your contributions!
I merged your PR and released new version which is v0.6.21.
Please try it !


Version 0.2.0, Includes bundled notebook and Jupyterlab 3 extensions

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)

Version 0.2.0 includes the javascript assets for Notebook and JupyterLab 3 (uses the new extension system which avoids the runtime build step that JupyterLab 2 required).
I've also added myself as a maintainer, is that OK?

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe:

noarch: python recipes are recommended to have a lower bound on the python version. This recommendation will become a requirement in the future.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe:

noarch: python recipes are recommended to have a lower bound on the python version. This recommendation will become a requirement in the future.

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Release v4.0.0

[x] Add a Dispatching link in the summary
[x] Add a deprecation notice for Matomo 3.x in the README.md
[x] Add #97 to the CHANGELOG.md

Thanks for the feedbacks @EPNW-Eric!
I was expecting that some changes were left to do, this is why I've made this PR a draft. I will add them for a better visibility of what's left before the release.

The release might be delayed a bit, @MeixDev made some investigations and discovered that ad blockers might break flutter web applications that are using the package because of the tracking_order_item.dart file.
Renaming seemed to fix this but investigating a bit more wouldn't hurt.

Hm that's actually very interesting... I would love to hear more about this (out of personal interest 😅).
Where those tests run with flutter release or development builds? Afaik release builds should do some minifying and type renaming automatically, so the file should not contain the word "tracking" there (what I would assume to be the problem, but again, would love to hear more about the details 😊).

It is true I only tried on Debug builds, with  the uBlock Origin extension on Firefox, for now.
If the production JS code is minified, which it indeed should be if only to reduce serve time, this would probably save us from trouble.
At the same time though, it's an annoying error to track when you're testing things (it makes the whole app crash while loading, but it doesn't log anything in the Flutter console, only in the browser's).
Some further investigations should be done, with other browsers and adblockers, and with extensions that only block tracking.

I know that it might be kind of "late" to mention this now, but since this is going to be a major release it might be a good opportunity to make the parameters of the track... methods consistent: So methods take named (both, required and not required) parameters, some take positional ones. To me, it appears to be a bit arbitrary. I could have followed the logic to make things that should not be null positional and the rest optional named, but this is not the case since we have some things that are name requried. I personally would prefere a style where all track... parameters are named ones.

@MeixDev I'm going to merge this PR and release v4.0.0 on pub.dev, even if your issue with ad blockers persist, renaming a file so it won't be blocked shouldn't be a breaking change.


Anasazi preconditioner

Will rebase on master once Anasazi is merged. All relevant changes are in the last commit.

Anasazi is merged. You can rebase.

Will rebase on master once Anasazi is merged. All relevant changes are in the last commit.

Any explanation what this pull request does at all?


Fix Validate Function

The following snippet does not return the result of validation but instead the validation function. The patch returns the actual validation result.
def validate(proto_message):
  return _validate_inner(ValidatingMessage(proto_message))

It looks like the tests aren't passing due to a type error in the Python code.

I will take a look at this next week.

The updated validator.py works fine for my use case. I have no idea, why the tests are breaking. I tried to understand the test flow but I am not quite sure how the tests work. I will close this PR.


Update JsPlugin for proper server-side support

In addition, updates JsPlugin documentation to be consistent with https://github.com/deephaven/deephaven-core/pull/4925
Fixes #15

Approved, one clarification: the intention w.r.t. allow/deny lists for files in path is to add a new, non-abstract method in the future as needed?

If needed, yes, something like that. Or regex, glob, etc.


GIF IMAGEVIEW

Can't play gif image of drawable in imageView

From 2.3.2 need to rely on sketch-gif library to play the GIF, as follows:
compile 'me.xiaopan:sketch-gif:2.4.0-beta1'

Please help me to show progress bar while get bitmap from web into
ImageView.
On 7 June 2017 at 16:11, Flyge<EMAIL_ADDRESS>wrote:

Closed #34 https://github.com/xiaopansky/sketch/issues/34.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/xiaopansky/sketch/issues/34#event-1113411174, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AVgy6zvsc4VpYb6VEHuPF4igXHdNk15Fks5sBn5igaJpZM4NhOOv
.

--
By Arunramya

I find thankyou so much.
On 12 August 2017 at 14:07, professional<EMAIL_ADDRESS>wrote:

Please help me to show progress bar while get bitmap from web into
ImageView.
On 7 June 2017 at 16:11, Flyge<EMAIL_ADDRESS>wrote:

Closed #34 https://github.com/xiaopansky/sketch/issues/34.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/xiaopansky/sketch/issues/34#event-1113411174, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AVgy6zvsc4VpYb6VEHuPF4igXHdNk15Fks5sBn5igaJpZM4NhOOv
.

--
By Arunramya

--
By Arunramya


Use BK v2 protocol from Presto connector

Motivation
We should read with BK v2 protocol to ensure all buffers are pooled and coming from direct memory.

run java8 tests


Fix the extensions XML namespace per NIST.IR.8060 page iii.

This pull request is on top of https://github.com/strongswan/swidGenerator/pull/38.


Upgrade Prometheus to v2.43.0

Issue #, if available:
N/A
Description of changes:
Upgrade Prometheus to v2.43.0
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve


Peek API not implemented for UDP streams

Using the example provided for the peek API for a UDP stream, within the preread_by_lua_block, gives the error:
Attempt to call method 'peek' (a nil value)
Is it possible to get the peek API implemented for UDP streams?

Same here, Need this api.


Markdown preview doesn't render when closing certain editor groups

Repro:
Get in this state:

Close the right editor group:

Version: 1.76.0-insider (user setup)
Commit: beb9ed3758624b918f918886bb904b0b62511c08
Date: 2023-02-08T12:47:14.528Z
Electron: 19.1.9
Chromium: 102.0.5005.194
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22621
Sandboxed: Yes

Actually seems like a generic webview issue

Fixed webview layout issue but now it seems like we also need a fix to the markdown extension

Actually not a bug. If you have two editors a.md and b.md side but side with a preview below a.md, closing b.md switches focus to the preview tab. However the preview stays for the content of b.md
This is now a duplicate of #66132.
To verify this issue:

Open two editors a.md and b.md side by side
Open a markdown preview below a.md
Close the entire b.md group and confirm that the preview still shows its contents


TypeError: existing.set is not a function

const collection = new Backbone.Collection();
const model = new Backbone.Model({
    id: 'hasOwnProperty'
});
collection.set([model]); // TypeError: existing.set is not a function

In Collection#set, it does: var existing = this.get(model); which in this case is setting existing to Object#hasOwnProperty rather than undefined because Collection#get is returning the result ofthis._byId['hasOwnProperty']
Have reproduced this bug in a codepen here.

This seems like an odd thing to fix.  This to me seems unlikely in general, but fixing it would add at least some cost to most actions across backbone where this issue is never likely to arise.  If anything the best fix to me seems to be something like adding to the docs "ids cannot be members of Array.prototype.String"

Can also cause the same bug by setting it to a different attribute and then setting idAttribute of the model to that, e.g:
  const keyedModel = Backbone.Model.extend({ idAttribute: 'key' });
  const keyedModelCollection = Backbone.Collection.extend({ model: keyedModel });
  const collection = new keyedModelCollection();
  const model = new keyedModel({ key: 'hasOwnProperty' });
  collection.set([model]); // TypeError: existing.set is not a function

Agree it's an edge case, but I think it's a valid use case if the string is a valid unique identifier in the database

As far as I'm concerned, this isn't specific to Backbone; it's on the list of things you should do nowhere in JavaScript. Of course, it's fine if you learn by experimentation that doing stuff like this breaks things, but this doesn't need documentation in Backbone, let alone a fix.
As for unique identifiers in databases, I expect those to be serial numbers or something more advanced like a UUID. Certainly not names of JavaScript's global object methods.


Early return when checking toJson invoke in List items

Hey there! I was looking at the new feature you've added for handling list responses as json and I saw that the code tries to run toJson on all items in the list even if the any of them doesn't match.
This change may be negligible in terms of performance, but maybe it could add up with bigger lists and many requests.
Before:

After:

Let me know what you think.
Thanks for the work on the package!

Returning early totally makes sense :-)
Thank you for the contribution.


Incorrect homotopy in Modelica.Fluid.Machines.BaseClasses.PartialPump

Reported by hansolsson on 10 May 2016 08:00 UTC
The equation (actually two of them):
head = homotopy((N/N_nominal)^2*flowCharacteristic(V_flow_single* N_nominal/N),
N/N_nominal*(flowCharacteristic(0)+delta_head_init*V_flow_single));

is not unit-correct; in the simplified version delta_head_init*V_flow_single has the correct unit m4/s but flowCharacteristic(0) has unit m.
I assume the idea was to use a linearization such as f(x)=f(0)+(f(x0)-f(0))*x/x0 to compute this - but the x0 part is missing.
Based on that it should be:
head = homotopy((N/N_nominal)^2*flowCharacteristic(V_flow_single* N_nominal/N),
N/N_nominal*(flowCharacteristic(0)*(N/N_nominal)+delta_head_init*V_flow_single/V_flow_single_init));

Which is unit-correct, but so is the simpler variant:
head = homotopy((N/N_nominal)^2*flowCharacteristic(V_flow_single* N_nominal/N),
N/N_nominal*(flowCharacteristic(0)+delta_head_init*V_flow_single/V_flow_single_init));

However, in practice both would cause errors since V_flow_single_init is as default 0 - which would lead to division by zero (actually zero divided by zero since delta_head_init is also zero in those cases).
Thus the actual corrections is more complex.
Note that as long as the homotopy-solver isn't activated this will not cause any problems.
The issues has been present at least since MSL 3.2.1; maybe earlier. It isn't detected by Dymola 2016 FD01 and earlier due to an unrelated issue.

Migrated-From: https://trac.modelica.org/Modelica/ticket/1995

Comment by hansolsson on 10 May 2016 08:23 UTC
Something that was indirectly stated before (but should be emphasized is that): as long as system.m_flow_start is zero (the default) the unit-incorrect part adds zero and can thus be ignored.
Correction: The first sentence of the ticket should say:
The equation (actually two of them):
head = homotopy((N/N_nominal)^2*flowCharacteristic(V_flow_single* N_nominal/N),
N/N_nominal*(flowCharacteristic(0)+delta_head_init*V_flow_single));

is not unit-correct; in the simplified version delta_head_init*V_flow_single has the incorrect unit m4/s but flowCharacteristic(0) has the correct unit m.

Modified by fcasella on 11 Nov 2016 15:33 UTC

Modified by fcasella on 11 Nov 2016 15:34 UTC

It seems this one is only partially corrected and ModelicaTest.Fluid.TestComponents.Machines.TestControlledPump  thus fails with division by zero.


fix asset startup error #81 #82

@hal-8999-alpha lemme know if the issue is resolved

@hal-8999-alpha can we merge this?


AS-path loops

It looks like BGP receivers in Batfish detect AS-path loops only by looking for their AS number in the first position of the AS-path. That would allow for loopy AS-paths such as A-X-Y-A-Z. In reality, such paths will not be allowed.

That's not what was happening, but in any case I fixed AS-path checking in commit bc8ab6298273346457108e9ba8497e4475027e70


Minor branching modification to matching policy (route branch)

Is your feature request related to a problem? Please describe.
It seems that there could be a better branching function easily implemented in the match/non-matching concept.
Right now, there is a matching function which has "include" and "exclude" criteria.  Also, there is a "policy" concept, which says what to do with messages that do not meet the matching criteria, but that policy: concept has a critical flaw - it can only jump to the routes: specified at the end of the statement, which is ALSO where all the matching results are sent after the transformations. This model essentially forces branching behaviors closer to ingestion in order to avoid this inadvertent combining of routing.
Describe the solution you'd like
It seems that instead of a "policy" statement, you could have a "route-unmatched:" statement that would give the ability to fork queries based on their meeting or not meeting the "include:" criteria.  It is implicit that anything that meets the "matching: include:" statements will then have the transformations or aggregations or other actions applied to them, and then they can proceed to a next route statement. Adding "route-unmatched:" would allow immediate branching to a different path.
(also: is "exclude:" just another way of saying that the message is unmatched?)
Example of how this path logic has to work today:
# Take in data from a dnstap stream.

  - name: dnsdist-from-outside
    dnstap:
      listen-ip: <IP_ADDRESS>
      listen-port: 59310
    routes: [ check-for-google, main-process ]

# This is a "dead-end" check - once we add the tag, we don't want to put
#  it back in our main processing flow.  We want to send it to a dnstap forwarder
#  and end the process.  Therefore, we MUST split the stream from dnsdist-from-outside
#  and duplicate the ENTIRE data set to be evaluated twice.

  - name: check-for-google
    dnsmessage:
      matching:
        include:
          dns.qname: "^.*\\.google\\.com$"
      policy: "block-unmatched"
      transforms:
        atags:
          tags: [ "TXT:Le-Goog" ]
    routes: [ send-to-dnstap-forwarder ]

  - name: send-to-dnstap-forwarder
    dnstap:
      <specify dnstap receiver etc.>

  - name: main-process
     dnsmessage:
      <stuff below here that we want to do>

Here's the example below of how it could look with a slightly different method:
We get flexibility if this is what things look like - messages are not split, and actions taken on matching messages are not necessarily inserted back into the single pathway.  This permits forking on match, but keeping most messages in a single unduplicated stream for lower processing overhead.
# Take in data from a dnstap stream.

  - name: dnsdist-from-outside
    dnstap:
      listen-ip: <IP_ADDRESS>
      listen-port: 59310
    routes: [ check-for-google ]

  - name: check-for-google
    dnsmessage:
      matching:
        include:
          dns.qname: "^.*\\.google\\.com$"
        route-unmatched: [ main-process ]    <-- this is the key line
      transforms:
        atags:
          tags: [ "TXT:Le-Goog" ]
    routes: [ send-to-dnstap-forwarder ]

  - name: send-to-dnstap-forwarder
    dnstap:
      <specify dnstap receiver etc.>

  - name: main-process
     dnsmessage:
      <stuff below here that we want to do>

Describe alternatives you've considered
It is possible to achieve the same outcomes by creating multiple route destinations, each leading to a specific conclusion. This seems quite wasteful, and means every packet has to be evaluated many times over, and also requires strict include/exclude matching sets as the contexts become deeper.

I have this type of feature in mind, but I am not yet clear on how to specify properly it in the YAML configuration.
I  would like to find a better description between routes, drop-policy' and route-unmatched`

perhaps something like that:
To apply specific routes
routing-policy:
  transformed: [ blockA ]
  discarded: [ blockB ]

To discard unmatched packet
routing-policy:
  transformed: [ blockA ]
  discarded: [  ]

And finally, the default routing for backward compatibility with any checks
routing-policy:
  default: [ blockA, blockB, .... ]

The model you describe in the last comment seems to be a good way of approaching the issue, of course assuming that there can (as usual) be multiple paths specified in the routing output for each option. Example:
routing-policy:
  accepted: [ stanzaA, stanzaC ]
  discarded: [ stanzaB, stanzaD ]

Question: Would objects that match an "exclude:" statement automatically go to the "discarded:" route path?  Are "exclude:" rules considered first, then, before "include:" rules?

Completed but need documentations

implemented in #719
The pipeline mode with be the default mode for next release.


Implement ContactList::getNextContact and ContactList::getActiveContact in Contact component

This PR adds ContactList::getNextContact and ContactList::getActiveContact functions that are useful to get the current active contact and the next active contact in a contact list

Ops :( the automerge merges it automatically. I'm going to directly fix it


CocoaPods: build standalone version of SQLite with subspec

Adresses #401.
Not sure about the "standalone" spec name – I just followed the pattern used in https://github.com/ccgus/fmdb/blob/master/FMDB.podspec.
Passes pod spec lint --allow-warnings.
The existing behavior and setup is left unchanged. I can't think of a good way to integrate the SQLite sources in a submodule, maybe this could be addressed in a separate PR.
It also needs tests – maybe a dummy project could be set up automatically, including test suite.

Nice! This looks good, but I wonder if we can somehow retain the CSQLite module name? We were trying to adhere to the SwiftPM System Module guidelines: https://github.com/apple/swift-package-manager/blob/master/Documentation/SystemModules.md#system-modules

Ah, I wondered about the weird naming, now it makes sense. I changed it so that the renaming is avoided, it also makes it clearer which module is being used.

Beautiful, thanks!


2205 about page

Summary
Cleaned up the "About" section so that there are now only 3 pages:

Overview: brief overview of the project
FAQ
Contact Us: the project contact info

Testing Guidance
Review pages for content.

Had to rebase against css/html updates due to _site dir being committed... was easier to just start fresh with a new branch


vertical layout

Thank you for your package, can you support vertical arrangement

@itMcdull What do you mean by vertical layout, can you please elaborate more

@itMcdull You may use the RotatedBox widget above the SliderController widget to achieve this like,


Filter operation is not working.

We are currently setting up the MSK Connect pipeline to transfer data from a Kafka Topic to OpenSearch using the Aiven connector as a plugin. We have successfully created a topic and produced dummy data on that topic. Additionally, we've configured an MSK connector to read and write the data, successfully writing it into OpenSearch.
However, when attempting to filter the data based on a specific field, data is not being written to OpenSearch(It is also not creating the Index in OpenSearch). Below is the configuration for the MSK Connect:
connector.class=io.aiven.kafka.connect.opensearch.OpensearchSinkConnector
type.name=kafka-connect
connection.password=***********
transforms.Filter.type=io.aiven.kafka.connect.transforms.FilterByFieldValue$Value
transforms.Filter.field.name=EventId
tasks.max=1
topics=json-topic
connection.username=sample
transforms=Filter
bootstrap.servers="**************************"
key.ignore=true
schema.ignore=true
key.converter.schemas.enable=false
value.converter.schemas.enable=false
transforms.Filter.field.value=4690
transforms.Filter.field.value.matches=true
value.converter=org.apache.kafka.connect.json.JsonConverter
connection.url=""
key.converter=org.apache.kafka.connect.storage.StringConverter
It is not working. I am not getting any errors but the filter data is not written to OpenSearch. I checked the data in topic also it contains the event id 4690.
Can you please help us.

Hi @himanshuankercloud.
It's hard to help troubleshooting this without an idea of how input messages look like. Could you share some sample input messages to see how this transformation may be applied?
You could also disable the SMT, and share how input and output messages look like. We can get some idea of what may be missing from these.
You could also look into the FilterByFieldValueTest tests to potentially reproduce the issue.
PS. I suggest to not copying real endpoints on public issues as it could lead to security issues.

Hi Jorge,
Please find the sample input for the topic and output as a message:-
You could also look into the FilterByFieldValueTest tests to potentially
reproduce the issue: can you explain how we can use it?
I have attached the python script using which we are uploading the data.
*Input:- *
{"Id":"DkiLMaqgagcCVZ4","EventId":4663,"TimeStampEvent":"2021-07-19T15:28:01.1055284+00:00","TimeStamp":"2021-07-19T15:28:01.3095677+00:00","Machine":"win7x64.mydomain.local","Properties":{"subjectUserSid":"S-1-5-21-2482720086-1306900138-933716382-500","subjectDomainName":"MYDOMAIN","objectServer":"Security","objectName":"C:\Windows\System32\rundll32.exe","accessList":"%%4421\n\t\t\t\t","processId":"0x16a0"},"EventRecordId":520668533,"ProcId":0,"ThreadId":0}
{"Id":"pz1zkEXx4JV04","EventId":4688,"TimeStampEvent":"2021-07-19T15:28:01.1055284+00:00","TimeStamp":"2021-07-19T15:28:01.3256558+00:00","Machine":"win7x64.mydomain.local","Properties":{"subjectUserSid":"S-1-5-21-2482720086-1306900138-933716382-500","subjectDomainName":"MYDOMAIN","newProcessId":"0x17b0","tokenElevationType":"%%1936","commandLine":"rundll32
637"},"EventRecordId":520668534,"ProcId":0,"ThreadId":0}
*Output:- *
[image: image.png]
On Wed, Jan 24, 2024 at 12:28 AM Jorge Esteban Quilcate Otoya <
@.***> wrote:

Hi @himanshuankercloud https://github.com/himanshuankercloud.
It's hard to help troubleshooting this without an idea of how input
messages look like. Could you share some sample input messages to see how
this transformation may be applied?
You could also disable the SMT, and share how input and output messages
look like. We can get some idea of what may be missing from these.
You could also look into the FilterByFieldValueTest tests to potentially
reproduce the issue.
PS. I suggest to not copying real endpoints on public issues as it could
lead to security issues.
—
Reply to this email directly, view it on GitHub
https://github.com/Aiven-Open/transforms-for-apache-kafka-connect/issues/126#issuecomment-1906730642,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWTROQ2IF6DDOCRUMYNY4ZDYQAB5BAVCNFSM6AAAAABB4N337GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMBWG4ZTANRUGI
.
You are receiving this because you were mentioned.Message ID:
<Aiven-Open/transforms-for-apache-kafka-connect/issues/126/1906730642@
github.com>

--
Thanks & Best Regards,
*Himanshu Shukla *
Senior Data Engineer
Ankercloud Technologies Pvt. Ltd.
91Springboard, Creaticity Mall,
Off, Airport Rd, opposite Golf Course,
Shastrinagar, Yerawada, Pune,
Maharashtra, India, 411006.
*M *+91-(8920773869)
www.ankercloud.com http://www.ankercloud.com/
@.*** @.**>
Founder & CEO : Santhosh Jayaprakash

@jeqo
Can you please share your feedback.
Its urgent.

@himanshuankercloud thanks for sharing details on how to reproduce.
Can confirm that this is an issue. Seems to be a bug on how value schema is inferred on the Connect schema library and how validation happens on the transform:

Inferred schema from input message is INT16 (integer)
But inferred schema parsed from expected value is INT8 (short)

This cause the filter test to always fail. See test https://github.com/Aiven-Open/transforms-for-apache-kafka-connect/pull/127/files
I see a couple of alternatives on how to fix:

Add a new config to set the type for expected value. In this case INT16.
If value is numeric, then validate against every type possible. In this case, instead of only testing input value as INT16, also test against INT8 and INT32.

Given this knowledge of mismatching numeric types, I wonder if you could apply some workaround with other SMTs to cast the value properly and apply the filter condition correctly, e.g. set the EventId as Key, then cast to INT16, and apply filter on the Key instead of Value. Maybe this would be enough to fix your use-case while this bug is getting fixed.


[EasyCodingStandard] Prefixed PHAR not working with PHP config format

$ ./vendor/bin/ecs check

 [ERROR] Argument 1 passed to
         _HumbugBox32bf47d4e469\Symfony\Component\DependencyInjection\Loader\ProtectedPhpFileLoader::{closure}() must be
         an instance of Symfony\Component\DependencyInjection\Loader\Configurator\ContainerConfigurator, instance of
         _HumbugBox32bf47d4e469\Symfony\Component\DependencyInjection\Loader\Configurator\ContainerConfigurator given,
         called in
         phar:///Users/nhart/dev/temp/prefixed-phar-php-config/vendor/symplify/easy-coding-standard-prefixed/ecs/vendor/
         symfony/dependency-injection/Loader/PhpFileLoader.php on line 43

Configuration:
<?php

declare(strict_types=1);

use Symfony\Component\DependencyInjection\Loader\Configurator\ContainerConfigurator;
use Symplify\EasyCodingStandard\Configuration\Option;
use Symplify\EasyCodingStandard\ValueObject\Set\SetList;

return static function (ContainerConfigurator $containerConfigurator): void {
    $parameters = $containerConfigurator->parameters();

    $parameters->set(Option::SETS, [
        SetList::SYMFONY,
    ]);
};

This is currently not detected by easy-coding-standard-prefixed test suite because of #2063, and test with php config was added during the same period.
I made a little project to reproduce this with the latest version of easy-coding-standard-prefixed really prefixed: https://github.com/wizifarm/prefixed-phar-php-config.

I think we had same issue in @rectorphp
Should be easy to replicate the fix in compiler directory

In fact, the fix is already in the compiler.
I tried to use the latest available prefixed version which is 8.1.17 (see #2063) and this version does not include the fix about ContainerConfigurator exclusion.
So everything should work as expected in future prefixed releases.
Sorry for the noise.

Thanks for verification


result of wtsexporter -i -b

Dear KnugiHK, I also tested with iPhone backup from README, and it turned out working fine, again, great project it is! I also noticed there's Media folder in Message folder, and I wonder what's the logic of (0  b  d) folders in<EMAIL_ADDRESS>folder?
also I noticed TODO 2. Reply in iPhone, may I know more detail description of this feature?

The logic of folder naming inside @s.whatsap.net is created by WhatsApp. As far as I know, every media will have a UUID as their filename. And the app will take the first two characters of the file name as their parent folders. It is probably about indexing.
In WhatsApp, there is a function for you to reply to a specific message. By using that function, both party in the conversation will know which message you are replying to and when you click that reply, you will be navigated to that message. Reply in iPhone is exactly the same thing. And the reply feature is already present in the Android version of this project.

And also, many metadata are not present in iPhone version extraction due to the messy (at least I consider it as messy) data structure of the iPhone WhatsApp database. I am not sure if I can learn their data structure in a short period, so I didn't put that into the TODO list. If anyone has any idea about the data structure in the iPhone database, feel free to contribute.

Dear KnugiHK, thank you for your sharing! and I'm also trying to figure the "messy" database about iPhone(chatStorage.sqlite) and Android(msgstore.db), It will be very helpful to share the ideas! Thanks!


Preload routes controller image

The feature was added in crc by https://github.com/code-ready/crc/pull/1793
The image digest needs to be sync between both projects.

This image is part of pod so that means if anything change on the image we need to update it snc and also on crc side, would it be possible to have that info in metadata and on crc side we just consume it from metadata instead hardcoding there? That way it would be updated on single place.

I could use a generic tag inside the VM like routes-controller:current.
Or, in crc, list all images with crictl images and run the first one with routes-controller name.
WDYT? I found that changing the json is a little bit painful each time.

I could use a generic tag inside the VM like routes-controller:current.
Or, in crc, list all images with crictl images and run the first one with routes-controller name.
WDYT? I found that changing the json is a little bit painful each time.

@guillaumerose I would go with generic tag then because using crictl images and get the image ID with route-controller looks bit hacky. Generic tag looks like a clean way to avoid some situations.

I tried with :current tag. It works but I am worry that the user might delete the image from the bundle by mistake.
If the user does something like podman rmi $(podman images -q), the pod won't be able to start anymore as the tag is only local.
I propose to keep the duplication of version for now.. If the preloading is not right when we develop, it's not a big deal, just few MB to download. We just need to ensure at release time, versions are the same.
WDYT?
(cc from https://github.com/code-ready/crc/pull/2273#issuecomment-827384565)

I tried with :current tag. It works but I am worry that the user might delete the image from the bundle by mistake.
If the user does something like podman rmi $(podman images -q), the pod won't be able to start anymore as the tag is only local.

The way I thought it to have latest or current tag https://quay.io/repository/crcont/routes-controller?tab=tags which always points to latest release of the image (I created latest one which on the quay UI show it comes from v0.0.1 tag) . The benefit which we are going to have is we no longer worry about changing tag on snc side or on crc side for each router controller release.

If we change the behavior of the router and the user deletes the image, it will fetch it again and it might not work.
I guess it is unlikely to happen but still. WDYT?

If we change the behavior of the router and the user deletes the image, it will fetch it again and it might not work.
I guess it is unlikely to happen but still. WDYT?

@guillaumerose  Yes and it is really unlikely since we are doing same with dnsmasq https://github.com/code-ready/snc/blob/master/createdisk.sh#L44 with latest and didn't get any issue from users. But we can argue that dnsmasq is not a suitable example here since we only care about dnsmasq package but here we are actually put router-controller code which developed by us. Still I am thinking to use latest is better than pointing version here because it will avoid us some of code maintainability because of this version saga on crc and snc repo :)
@code-ready/crc-team wdyt?

OK I change it to latest :+1:

/cherry-pick 4.7


[Question] Working JWT token

Hi!
I'm working with protected route by jwt in the graphql. In the integrated tests can pass the token for inbound authentication everything works great, I would like to know how can I pass the token in the graphiql playground to be able to use the queries..
Thank you very much in advance.

bom dia!
you can pass the token as a header in the custom fetcher prop which is required for the GraphiQL component. in the webpack example, you can see we set content type for example. so you can either have your own react login flow that then exposes the GraphiQL component once a valid token is retrieved, or ive also seen implementations that retrieve the value from the user session or local storage.


Classifier training after ResNet

I am not sure why you used a prediction for classifier training in train_frcnn.py ?
loss_rpn = model_rpn.train_on_batch(X, Y) #train on the original image and the segmented output
P_rpn = model_rpn.predict_on_batch(X)
Actually I am trying to follow the code line by line to have a deeper understand of how the fast RCNN works. I am confused about this part of adding the classifier after the ConvNet.

Hello,
The prediction on the Region Proposal Network (RPN) is used to know whether the location found contains an object or not (binary).
Yet, this is only used in Faster R-CNN, not Fast R-CNN (that you mentionned).
Regards,

Thanks Ahleroy for your answer. But why we are using the prediction (P_rpn) and not using the Y that is in the database to know the location found ?

You're welcome :)

Oh, I am still confused why not we use the Y that is given in the training data directly instead of predicting the P-rbn ?

The Y used in model_rpn.train_on_batch(X,Y) is the expected output whereas the P_rpn = model_rpn.predict_on_batch(X)  is the predicted output.
The first one is used for training the Region Proposal Network whereas the second one is used to process the ROI (Region Of Interest).

You mean the expected output is the ground truth for the trained data ? If so, why not using this expected output in place of the predicted output to process the ROI ? It's more accurate as it's the labeled areas of the trained data.

Indeed the expected output is used for the ROI which is then given to the CNN.
I am not sure but if we train the CNN with a ROI whose the input comes from a perfect input (expected output of the RPN), then when we will test it, if the RPN is not perfect, the CNN might have difficulties to ignore the noise added by the RPN whereas if it is trained to recognize output from the RPN, it will be more flexible.
In practice, the RPN might be good enough to feed the CNN with the expected output.
Once again, I'm not 100% sure, I would need time to test differents things. If you have the time to compare both solution please do it and tell us your results :)

I would like to compare both solutions. May you tell me a suitable machine on AWS for testing ? I am still have challenges to configure my GPU to be recognized on mac Yosemite !

Sorry but I cannot help you with that :/
Regards,

I mean a suggestion for a EC2 that can run this code to use under my account. I dont mean you provide me with a machine :)

@yhenon the code right now trains updates the weights and then predicts the ROIs for the ROI pool layer. As shown below:
model_rpn.train_on_batch(X,Y)
P_rpn = model_rpn.predict_on_batch(X)

But this seems to disturb the unified training framework slightly, given that the RPN and the Fast R-CNN classifiers and regressors are training on different weights for each image. I think the prediction should be done first meaning you predict the region proposals with the weights of that iteration and then update the weights. And when you pass the region proposals to the Fast r-cnn classifier and regressor the regions where predicted with the original weights.
Right now in the RPN you are predicting regions, updating the weights and then predicting regions again. Instead I think this should be applied:
P_rpn = model_rpn.predict_on_batch(X)
model_rpn.train_on_batch(X,Y)

@jahdiel, I get your point. But still the issue is why we use the predicted version of RPN P_rpn instead of the expected given one Y ?