Compilation error after upgrade to prost v0.6.1

As of 01e91f5e8dcb88c236c1089b31c365f5ba997667:
% cargo build
   Compiling wascc-actor v0.1.1 (/Users/gtb273/wrk/src/github.com/wascc/wascc-actor)
error[E0277]: the trait bound `wascc_codec::keyvalue::GetRequest: prost::Message` is not satisfied
  --> src/kv.rs:52:63
   |
52 |         host_call(&route(CAPID_KEYVALUE, OP_GET), &protobytes(cmd)?).map(|vec| {
   |                                                               ^^^ the trait `prost::Message` is not implemented for `wascc_codec::keyvalue::GetRequest`
   |
  ::: src/lib.rs:59:8
   |
59 | pub fn protobytes(msg: impl prost::Message) -> Result<Vec<u8>> {
   |        ----------           -------------- required by this bound in `protobytes`

error[E0599]: no function or associated item named `decode` found for type `wascc_codec::keyvalue::GetResponse` in the current scope
  --> src/kv.rs:53:37
   |
53 |             let resp = GetResponse::decode(vec.as_ref()).unwrap();
   |                                     ^^^^^^ function or associated item not found in `wascc_codec::keyvalue::GetResponse`
   |
   = help: items from traits can only be used if the trait is in scope
   = note: the following trait is implemented but not in scope; perhaps add a `use` for it:
           `use prost::message::Message;`

error[E0277]: the trait bound `wascc_codec::keyvalue::SetRequest: prost::Message` is not satisfied
  --> src/kv.rs:68:63
   |
68 |         host_call(&route(CAPID_KEYVALUE, OP_SET), &protobytes(cmd)?).map(|_vec| ()).map_err(|e| e.into())
   |                                                               ^^^ the trait `prost::Message` is not implemented for `wascc_codec::keyvalue::SetRequest`
   |
  ::: src/lib.rs:59:8
   |
59 | pub fn protobytes(msg: impl prost::Message) -> Result<Vec<u8>> {
   |        ----------           -------------- required by this bound in `protobytes`

error[E0277]: the trait bound `wascc_codec::keyvalue::AddRequest: prost::Message` is not satisfied
  --> src/kv.rs:76:63
   |
76 |         host_call(&route(CAPID_KEYVALUE, OP_ADD), &protobytes(cmd)?).map(|vec| {
   |                                                               ^^^ the trait `prost::Message` is not implemented for `wascc_codec::keyvalue::AddRequest`
   |
  ::: src/lib.rs:59:8
   |
59 | pub fn protobytes(msg: impl prost::Message) -> Result<Vec<u8>> {
   |        ----------           -------------- required by this bound in `protobytes`

error[E0599]: no function or associated item named `decode` found for type `wascc_codec::keyvalue::AddResponse` in the current scope
  --> src/kv.rs:77:37
   |
77 |             let resp = AddResponse::decode(vec.as_ref()).unwrap();
   |                                     ^^^^^^ function or associated item not found in `wascc_codec::keyvalue::AddResponse`
   |
   = help: items from traits can only be used if the trait is in scope
   = note: the following trait is implemented but not in scope; perhaps add a `use` for it:
           `use prost::message::Message;`

error[E0277]: the trait bound `wascc_codec::keyvalue::ListPushRequest: prost::Message` is not satisfied
  --> src/kv.rs:87:64
   |
87 |         host_call(&route(CAPID_KEYVALUE, OP_PUSH), &protobytes(cmd)?).map(|vec| {
   |                                                                ^^^ the trait `prost::Message` is not implemented for `wascc_codec::keyvalue::ListPushRequest`
   |
  ::: src/lib.rs:59:8
   |
59 | pub fn protobytes(msg: impl prost::Message) -> Result<Vec<u8>> {
   |        ----------           -------------- required by this bound in `protobytes`

error[E0599]: no function or associated item named `decode` found for type `wascc_codec::keyvalue::ListResponse` in the current scope
  --> src/kv.rs:88:38
   |
88 |             let resp = ListResponse::decode(vec.as_ref()).unwrap();
   |                                      ^^^^^^ function or associated item not found in `wascc_codec::keyvalue::ListResponse`
   |
   = help: items from traits can only be used if the trait is in scope
   = note: the following trait is implemented but not in scope; perhaps add a `use` for it:
           `use prost::message::Message;`

error[E0277]: the trait bound `wascc_codec::keyvalue::ListDelItemRequest: prost::Message` is not satisfied
  --> src/kv.rs:98:68
   |
98 |         host_call(&route(CAPID_KEYVALUE, OP_LIST_DEL), &protobytes(cmd)?).map(|vec| {
   |                                                                    ^^^ the trait `prost::Message` is not implemented for `wascc_codec::keyvalue::ListDelItemRequest`
   |
  ::: src/lib.rs:59:8
   |
59 | pub fn protobytes(msg: impl prost::Message) -> Result<Vec<u8>> {
   |        ----------           -------------- required by this bound in `protobytes`

error[E0599]: no function or associated item named `decode` found for type `wascc_codec::keyvalue::ListResponse` in the current scope
  --> src/kv.rs:99:38
   |
99 |             let resp = ListResponse::decode(vec.as_ref()).unwrap();
   |                                      ^^^^^^ function or associated item not found in `wascc_codec::keyvalue::ListResponse`
   |
   = help: items from traits can only be used if the trait is in scope
   = note: the following trait is implemented but not in scope; perhaps add a `use` for it:
           `use prost::message::Message;`

error[E0277]: the trait bound `wascc_codec::keyvalue::DelRequest: prost::Message` is not satisfied
   --> src/kv.rs:108:63
    |
108 |         host_call(&route(CAPID_KEYVALUE, OP_DEL), &protobytes(cmd)?).map(|_vec| ()).map_err(|e| e.into())
    |                                                               ^^^ the trait `prost::Message` is not implemented for `wascc_codec::keyvalue::DelRequest`
    |
   ::: src/lib.rs:59:8
    |
59  | pub fn protobytes(msg: impl prost::Message) -> Result<Vec<u8>> {
    |        ----------           -------------- required by this bound in `protobytes`

error[E0277]: the trait bound `wascc_codec::keyvalue::ListRangeRequest: prost::Message` is not satisfied
   --> src/kv.rs:117:65
    |
117 |         host_call(&route(CAPID_KEYVALUE, OP_RANGE), &protobytes(cmd)?).map(|vec| {
    |                                                                 ^^^ the trait `prost::Message` is not implemented for `wascc_codec::keyvalue::ListRangeRequest`
    |
   ::: src/lib.rs:59:8
    |
59  | pub fn protobytes(msg: impl prost::Message) -> Result<Vec<u8>> {
    |        ----------           -------------- required by this bound in `protobytes`

error[E0599]: no function or associated item named `decode` found for type `wascc_codec::keyvalue::ListRangeResponse` in the current scope
   --> src/kv.rs:118:43
    |
118 |             let resp = ListRangeResponse::decode(vec.as_ref()).unwrap();
    |                                           ^^^^^^ function or associated item not found in `wascc_codec::keyvalue::ListRangeResponse`
    |
    = help: items from traits can only be used if the trait is in scope
    = note: the following trait is implemented but not in scope; perhaps add a `use` for it:
            `use prost::message::Message;`

error[E0277]: the trait bound `wascc_codec::keyvalue::ListClearRequest: prost::Message` is not satisfied
   --> src/kv.rs:127:65
    |
127 |         host_call(&route(CAPID_KEYVALUE, OP_CLEAR), &protobytes(cmd)?).map(|_vec| ()).map_err(|e| e.into())
    |                                                                 ^^^ the trait `prost::Message` is not implemented for `wascc_codec::keyvalue::ListClearRequest`
    |
   ::: src/lib.rs:59:8
    |
59  | pub fn protobytes(msg: impl prost::Message) -> Result<Vec<u8>> {
    |        ----------           -------------- required by this bound in `protobytes`

error[E0277]: the trait bound `wascc_codec::keyvalue::SetAddRequest: prost::Message` is not satisfied
   --> src/kv.rs:135:67
    |
135 |         host_call(&route(CAPID_KEYVALUE, OP_SET_ADD), &protobytes(cmd)?).map(|vec| {
    |                                                                   ^^^ the trait `prost::Message` is not implemented for `wascc_codec::keyvalue::SetAddRequest`
    |
   ::: src/lib.rs:59:8
    |
59  | pub fn protobytes(msg: impl prost::Message) -> Result<Vec<u8>> {
    |        ----------           -------------- required by this bound in `protobytes`

error[E0599]: no function or associated item named `decode` found for type `wascc_codec::keyvalue::SetOperationResponse` in the current scope
   --> src/kv.rs:136:46
    |
136 |             let resp = SetOperationResponse::decode(vec.as_ref()).unwrap();
    |                                              ^^^^^^ function or associated item not found in `wascc_codec::keyvalue::SetOperationResponse`
    |
    = help: items from traits can only be used if the trait is in scope
    = note: the following trait is implemented but not in scope; perhaps add a `use` for it:
            `use prost::message::Message;`

error[E0277]: the trait bound `wascc_codec::keyvalue::SetRemoveRequest: prost::Message` is not satisfied
   --> src/kv.rs:146:70
    |
146 |         host_call(&route(CAPID_KEYVALUE, OP_SET_REMOVE), &protobytes(cmd)?).map(|vec| {
    |                                                                      ^^^ the trait `prost::Message` is not implemented for `wascc_codec::keyvalue::SetRemoveRequest`
    |
   ::: src/lib.rs:59:8
    |
59  | pub fn protobytes(msg: impl prost::Message) -> Result<Vec<u8>> {
    |        ----------           -------------- required by this bound in `protobytes`

error[E0599]: no function or associated item named `decode` found for type `wascc_codec::keyvalue::SetOperationResponse` in the current scope
   --> src/kv.rs:147:46
    |
147 |             let resp = SetOperationResponse::decode(vec.as_ref()).unwrap();
    |                                              ^^^^^^ function or associated item not found in `wascc_codec::keyvalue::SetOperationResponse`
    |
    = help: items from traits can only be used if the trait is in scope
    = note: the following trait is implemented but not in scope; perhaps add a `use` for it:
            `use prost::message::Message;`

error[E0277]: the trait bound `wascc_codec::keyvalue::SetUnionRequest: prost::Message` is not satisfied
   --> src/kv.rs:154:69
    |
154 |         host_call(&route(CAPID_KEYVALUE, OP_SET_UNION), &protobytes(cmd)?).map(|vec| {
    |                                                                     ^^^ the trait `prost::Message` is not implemented for `wascc_codec::keyvalue::SetUnionRequest`
    |
   ::: src/lib.rs:59:8
    |
59  | pub fn protobytes(msg: impl prost::Message) -> Result<Vec<u8>> {
    |        ----------           -------------- required by this bound in `protobytes`

error[E0599]: no function or associated item named `decode` found for type `wascc_codec::keyvalue::SetQueryResponse` in the current scope
   --> src/kv.rs:155:42
    |
155 |             let resp = SetQueryResponse::decode(vec.as_ref()).unwrap();
    |                                          ^^^^^^ function or associated item not found in `wascc_codec::keyvalue::SetQueryResponse`
    |
    = help: items from traits can only be used if the trait is in scope
    = note: the following trait is implemented but not in scope; perhaps add a `use` for it:
            `use prost::message::Message;`

error[E0277]: the trait bound `wascc_codec::keyvalue::SetIntersectionRequest: prost::Message` is not satisfied
   --> src/kv.rs:162:73
    |
162 |         host_call(&route(CAPID_KEYVALUE, OP_SET_INTERSECT), &protobytes(cmd)?).map(|vec| {
    |                                                                         ^^^ the trait `prost::Message` is not implemented for `wascc_codec::keyvalue::SetIntersectionRequest`
    |
   ::: src/lib.rs:59:8
    |
59  | pub fn protobytes(msg: impl prost::Message) -> Result<Vec<u8>> {
    |        ----------           -------------- required by this bound in `protobytes`

error[E0599]: no function or associated item named `decode` found for type `wascc_codec::keyvalue::SetQueryResponse` in the current scope
   --> src/kv.rs:163:42
    |
163 |             let resp = SetQueryResponse::decode(vec.as_ref()).unwrap();
    |                                          ^^^^^^ function or associated item not found in `wascc_codec::keyvalue::SetQueryResponse`
    |
    = help: items from traits can only be used if the trait is in scope
    = note: the following trait is implemented but not in scope; perhaps add a `use` for it:
            `use prost::message::Message;`

error[E0277]: the trait bound `wascc_codec::keyvalue::SetQueryRequest: prost::Message` is not satisfied
   --> src/kv.rs:172:69
    |
172 |         host_call(&route(CAPID_KEYVALUE, OP_SET_QUERY), &protobytes(cmd)?).map(|vec| {
    |                                                                     ^^^ the trait `prost::Message` is not implemented for `wascc_codec::keyvalue::SetQueryRequest`
    |
   ::: src/lib.rs:59:8
    |
59  | pub fn protobytes(msg: impl prost::Message) -> Result<Vec<u8>> {
    |        ----------           -------------- required by this bound in `protobytes`

error[E0599]: no function or associated item named `decode` found for type `wascc_codec::keyvalue::SetQueryResponse` in the current scope
   --> src/kv.rs:173:42
    |
173 |             let resp = SetQueryResponse::decode(vec.as_ref()).unwrap();
    |                                          ^^^^^^ function or associated item not found in `wascc_codec::keyvalue::SetQueryResponse`
    |
    = help: items from traits can only be used if the trait is in scope
    = note: the following trait is implemented but not in scope; perhaps add a `use` for it:
            `use prost::message::Message;`

error[E0277]: the trait bound `wascc_codec::keyvalue::KeyExistsQuery: prost::Message` is not satisfied
   --> src/kv.rs:182:70
    |
182 |         host_call(&route(CAPID_KEYVALUE, OP_KEY_EXISTS), &protobytes(cmd)?).map(|vec| {
    |                                                                      ^^^ the trait `prost::Message` is not implemented for `wascc_codec::keyvalue::KeyExistsQuery`
    |
   ::: src/lib.rs:59:8
    |
59  | pub fn protobytes(msg: impl prost::Message) -> Result<Vec<u8>> {
    |        ----------           -------------- required by this bound in `protobytes`

error[E0599]: no function or associated item named `decode` found for type `wascc_codec::keyvalue::GetResponse` in the current scope
   --> src/kv.rs:183:37
    |
183 |             let resp = GetResponse::decode(vec.as_ref()).unwrap();
    |                                     ^^^^^^ function or associated item not found in `wascc_codec::keyvalue::GetResponse`
    |
    = help: items from traits can only be used if the trait is in scope
    = note: the following trait is implemented but not in scope; perhaps add a `use` for it:
            `use prost::message::Message;`

error[E0277]: the trait bound `wascc_codec::messaging::PublishMessage: prost::Message` is not satisfied
  --> src/msg.rs:61:25
   |
61 |             &protobytes(cmd)?,
   |                         ^^^ the trait `prost::Message` is not implemented for `wascc_codec::messaging::PublishMessage`
   |
  ::: src/lib.rs:59:8
   |
59 | pub fn protobytes(msg: impl prost::Message) -> Result<Vec<u8>> {
   |        ----------           -------------- required by this bound in `protobytes`

error[E0277]: the trait bound `wascc_codec::messaging::RequestMessage: prost::Message` is not satisfied
  --> src/msg.rs:77:25
   |
77 |             &protobytes(cmd)?,
   |                         ^^^ the trait `prost::Message` is not implemented for `wascc_codec::messaging::RequestMessage`
   |
  ::: src/lib.rs:59:8
   |
59 | pub fn protobytes(msg: impl prost::Message) -> Result<Vec<u8>> {
   |        ----------           -------------- required by this bound in `protobytes`

warning: unused import: `prost::Message`
  --> src/kv.rs:25:5
   |
25 | use prost::Message;
   |     ^^^^^^^^^^^^^^
   |
   = note: `#[warn(unused_imports)]` on by default

error: aborting due to 27 previous errors

Some errors have detailed explanations: E0277, E0599.
For more information about an error, try `rustc --explain E0277`.
error: could not compile `wascc-actor`.

To learn more, run the command again with --verbose.

This is caused by the prost version mismatch with wascc-codec = "0.3.4".


add more *at functions.

That's all folks. I have all methods for my fuse.

@Mic92 this is heroic. I'm going to take a look over the weekend. Sorry for the delay!

I retried the failing test and it works now so that's a plus.

I would also add large file support to all methods in a second pull request (64bit fields on 32-bit systems), to align the with glibc/musl/rust.

:umbrella: The latest upstream changes (presumably #536) made this pull request unmergeable. Please resolve the merge conflicts.

This is great @Mic92, just needs a rebase by the looks of it (the curse of the changelog).  The only thing I would add is that we have been trying to add docs to more of the functions we are adding.  I do not think this is a blocker for getting this included but would be a nice bonus for this change or a future PR.
E.g. Docs for fork: https://github.com/nix-rust/nix/blob/master/src/unistd.rs#L47

I have a doubt on this (and on other fn taking RawFd in this crate): are all these functions just taking ownership of (a copy of) the fd or borrowing it? I think these all do an implicit copy+own of the RawFd (as it is just an alias to an int), and I fear this doesn't really guarantee that the fd has not been closed/dropped in the meanwhile by a close().
I'm not sure if it is a valid concern or not, but shouldn't all these functions take a &RawFd (and perhaps RawFd be made non-Copy)?

@lucab this is a real concern, and I agree that we should do what you suggest. We cannot change RawFd as it's part of the standard library. But we should change all RawFd parameters to take references, or something at least that safe. I've opened #594 to track it. Thanks for raising this!

A reference does not buy you anything: https://play.rust-lang.org/?gist=1739781faf61fcec4afaf65bbaadd968&version=stable&backtrace=0
An not copyable type would destroy the semantics of some system calls like renameat, as it would no longer allow to pass the same directory twice.

Merge conflict resolved. Man page references added.

There a common theme through a lot of these functions of having a dirfd and a path that is a tristate:

The dirfd is ignored if path is absolute
If dirfd is AT_FDCWD and path is relative it's resolved relative to the CWD
If path is relative and dirfd isn't AT_FDCWD then the path is resolved relative to that path.

I'm thinking an enum encompassing these states would be better instead of having two separate variables. I think an ideal solution would be to have NixPath have an Absolute and Relative mode or something to that effect, but for this PR I think just having an enum with different types is good enough.

@Susurrus do you want to implement that enum for these functions send me a pull request? I don't see how Rust's typesystem is expressive enough to ensure a path is always relative/absolute without runtime checks.

My suggestion is to have a:
pub enum AtPath {
  AbsPath(NixPath),
  RelativeTo(RawFd, NixPath),
  RelativeCwd(NixPath),
}

Now we could add runtime checks through having some conversion path from NixPaths to this enum and I think that's the most ergonomic, which would be ideal, and if you'd like to implement something like that we can spec it out here. But at least with this enum it's more clear to the user what combinations of inputs are valid.

I wanted to point out that I also preferred the arguments @bugaevc who was working on access/faccessat in #605. There they came up with a dual-state enum for use with those functions that looks like:
pub enum AccessFlags {
  FileExists,
  FileExistsAndPermissions(PermFlags),
}

where PermFlags are the R_OK, etc. constants. I'd prefer having that than the regular libc-like constants.

@Mic92 I wanted to circle back to this and see if we could get this merged? I'm working on some stuff in fcntl but it should get merged within a day or so. Then I'd like to get this PR merged if possible. Still willing to push this through?

I am back from holiday on Thursday.

What happened to this?

I made multiple pull requests out of it.


[BUG?] Wrong regex parsing for term headers

ERROR: Error loading template /dataplant/MIAPPE_nutrients.xlsx: Could not parse header group [L1 : Parameter [Concentration of [nutrient] before start of the experiment] | String; M1 : Term Source REF (MIAPPE:0134) | String; N1 : Term Accession Number (MIAPPE:0134) | String]

Done! closed by #216 , from v1.0.0-alpha8.


feat: extend Scrollable Tabs to support center position

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add unicorn_worker_processes variable to account-api

This is not set yet, so it won't have any effect.  But when we do want
to scale up we can just set a number in hieradata now.

It's a puppet run, so half an hour after being deployed.


Added round robin based placement algorithm to dummy GK

This placement scheme distributes the VNFs equally across all available PoPs in the topology instead of just using one of them.

wait until Jenkins is back to ensure that the patch does not break a test (couldn't test locally, did the patch in the train)

test this please


Maximum recursion depth exceeded when accessing wrong accesssors

A simple reproducable example:
In [1]: import pandas as pd

In [2]: import dask.dataframe as dd

In [3]: df = dd.from_pandas(pd.DataFrame({"a": [1, 2, 3]}), npartitions=2)

In [4]: hasattr(df["a"], "dt")
Out[4]: True    # Should probably return False in this case?

In [5]: df["a"].dt
Out[5]: Fatal Python error: Cannot recover from stack overflow.

The final error is a Maximum recursion depth exceeded in a normal Python process.
This error occurs in debuggers (pudb and VSCode's debugger in my case) when inspecting a column on a DataFrame.

Thanks for the excellent issue, I've reproduced the problem, fix up shortly.

Thanks for handling this so quickly!


fix: Camera ignore turn flash off exception

Hi everyone,
When the preview is paused, I force the flash to be turned off.
But if the native part is quicker than the Dart side (camera closed for example), it will trigger an exception.
Instead, we just ignore the error
I/Camera  (12140): open | onClosed
I/Camera  (12140): CameraCaptureSession onClosed
I/Camera  (12140): refreshPreviewCaptureSession: captureSession not yet initialized, skipping preview capture session refresh.
E/flutter (12140): [ERROR:flutter/lib/ui/ui_dart_state.cc(198)] Unhandled Exception: CameraException(setFlashModeFailed, Could not set flash mode.)
E/flutter (12140): #0      CameraController.setFlashMode (package:camera/src/camera_controller.dart:642:7)
E/flutter (12140): <asynchronous suspension>
E/flutter (12140): #1      SmoothCameraController.pausePreview (package:smooth_app/pages/scan/camera_controller.dart:83:7)
E/flutter (12140): <asynchronous suspension>
E/flutter (12140): #2      MLKitScannerPageState._stopImageStream (package:smooth_app/pages/scan/ml_kit_scan_page.dart:365:5)
E/flutter (12140): <asynchronous suspension>
E/flutter (12140):

Please do a flutter upgrade to get my latest changes to the camera.
And you will notice that focus & detection are far better now


fix type registration for templates

Templates should not be registering List as part of their API group.  Lists are always v1.
/assign @liggitt

/lgtm


PLFM-7262: Add iam perms, need to back down later

Need more perms

This does not get deployed by the infra, it needs to be manually deployed in dev stack account.


Calls error callback with an Array

I am seeing callbacks being called with the following as the first argument (the error argument):
 ["LeaderNotAvailable"]

An array is not an error.
Node callbacks have this API: type Callback<T> = (error?: Error, value?: T) => void. The core docs have a clear guide for this.
Calling a callback with a non-Error is unhelpful, as it break stack-traces, and the rest of the ecosystem's assumptions.
To fix this, simply ensure that every call looks like either cb(Error("something")), cb(actuallyAnError), cb(null, value) or cb().
I had a brief look for other occurrences of code that violates this core contract of Node - here's a very clear one.

I understand what you're saying @timruffles but I don't think we can fix this without making a major version bump. In case users were depending on this behavior.
PRs are welcome! 👍


Make search field accessible via a keyboard shortcut

To be able to quickly search for notes, it would be nice to be able to give focus to the search box via a keyboard shortcut (similar to the already existing shortcuts to create new notes, etc.)
Proposed solution: use the shortcut <ctrl>+<alt>+<f> (similar, but not colliding with the common <ctrl>+<f> shortcut in browsers).

There is a preliminary PR #181 for this feature request.

Closed in #181.


perform distinct oepration sqlite.swift branches - swift 2

Perform distinct query app crash no such column name
table contains 3 columns but i want to use only one
   for user in db.prepare(users.select(distinct:main).filter(main != "Bookmarks"))
    {
        print("id: \(user[main]))")

        data = Array(arrayLiteral: user)
    }

help me how to solved it

Would you please provide more information? Sounds like the "users" table doesn't have the column "name"?

I can't reproduce this. Here is a little playground example that tries to mimic your scenario and it runs with the current release with no errors as follows:
import SQLite

let db = try! Connection()
db.trace { print($0) }

let emails = Table("emails")
let to = Expression<String>("to")
let subject = Expression<String?>("subject")

try! db.run(emails.create {t in t.column(to); t.column(subject)} )
try! db.run(emails.insert(to <-<EMAIL_ADDRESS>subject <- "Hello, world!" ))

for user in try db.prepare(emails.select(distinct:to).filter(to != "me@example.com")) {
    print("id: \(user[to])")
}

Output:
CREATE TABLE "emails" ("to" TEXT NOT NULL, "subject" TEXT)
INSERT INTO "emails" ("to", "subject") VALUES<EMAIL_ADDRESS>'Hello, world!')
SELECT DISTINCT "to" FROM "emails" WHERE ("to" !=<EMAIL_ADDRESS>id<EMAIL_ADDRESS>
Please re-open if you feel this was closed in error and you have more information. Thanks!


Designing an Authentication API using Service Classes in Flutter

Article checklist
Pre-submission checks

Please ensure that you have only one open topic suggestion (issue) + in-review article (linked pull request) at a time. This will ensure that we complete the review process in a timely manner from inception to publishing

If you have not already, please go over our Resources Page for more tips and tricks on how to improve your overall technical writing so reviews are swifter, increase the chance of a payout and provide more value to our readers.

For first-time contributors and for more details on our submission guidelines, see our Contributing Guidelines.

Formatting and structure checks

[ ] Have you used the correct folder and file structure? - see Contribution Guidelines

[x] Is your article properly formatted in Markdown? - see Github Markdown guide

[x] Have you used our preferred heading structure? - H3 (###) and up

[x] Does your article follow any of the suggested structure formats? - see example formats

[x] Is your article over 750 words? - Tool: Wordcounter.net

[x] Is the article you are submitting an in-depth and unique article? Does it go beyond what is in the official docs and what is covered in other blog sites. See these articles as examples.

[x] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.

[x] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.

[x] Have you included a hero image that is 600x400 pixels and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images

Grammar & plagiarism checks

[x] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid - Grammarly
[x] Please place your article through a 3rd party plagiarism checker. We suggest using Quetext, this tool is free and has a daily limit. We typically accept articles with 10% or less.
[x] Have you checked your article for readability? - Tool: Hemmingway
[ ] Have you added sources for quotes and images that aren't yours?

Technical checks

[x] Are your code snippets properly formatted for syntax highlighting - see Syntax guide
[x] Have you checked your code runs correctly and you've highlighted all necessary dependencies for installation?
[x] Are the software programs and packages you're highlighting in your article up to date, using current versions and not deprecated?

Topic suggestion this closes

Remove the backticks and add the issue number below to link and close your Topic Suggestion (issue) when your article has been published (PR has been merged). See this video for more details.

`This closes #3469

Finally, delete the article checklist notes in blockquotes and submit your PR. We look forward to reviewing your article.

Good afternoon and thank you for submitting your article for publishing. @vincentkims49
Your article has been entered into our queue and should be reviewed (and published pending any changes) as soon as a peer reviewer is finished with the ones in the queue before it. :rocket:
Thank you.

@ahmadmardeni1 , @WanjaMIKE , @hectorkambow
Hopefully this article hasn't been left out.

Upon running your article through our 3rd party plagiarism checker it seemed to raise a few flags and the % was higher than we typically accept.
Please see attached PDF - and revisit the article to ensure we are contributing wholly unique and original content.
Be sure to see our resources page to see more info on plagiarism and what is considered as such.
Designing an Authentication API using Service Classes in Flutter

Done @ahmadmardeni1

@ahmadmardeni1
I have corrected the plagiarisms. Some content is slightly changed since I'm focusing on giving the required or the appropriate information concerning the topic.
Thanks.

@vincentkims49
This article seems a bit light on substance
cc @ahmadmardeni1 @jerimkaura

@hectorkambow I'll add some more information if necessary.

@hectorkambow @ahmadmardeni1 @jerimkaura
I have done some corrections and added more information to the article. You may have a look please.

@hectorkambow @ahmadmardeni1 @jerimkaura
I have done some corrections and added more information to the article. You may have a look please.

@ahmadmardeni1 , @hectorkambow

@jerimkaura  Have you reviewed the new changes? What is the status of this PR - is it ready for a final review?

Will do the new changes soon. I was held up with exams

Hello @vincentkims49, Seems like you did not add much information apart from renaming a few variables. Consider adding more value to the article and making it understandable to beginners. How about something like a real flutter application demonstrating the concept.
cc: @hectorkambow , @ahmadmardeni1

Hello @vincentkims49, Seems like you did not add much information apart from renaming a few variables. Consider adding more value to the article and making it understandable to beginners. How about something like a real flutter application demonstrating the concept.
cc: @hectorkambow , @ahmadmardeni1

Sure I will add, @jerimkaura

Hello @vincentkims49, Seems like you did not add much information apart from renaming a few variables. Consider adding more value to the article and making it understandable to beginners. How about something like a real flutter application demonstrating the concept.
cc: @hectorkambow , @ahmadmardeni1

I have added some more information. All the dart code that I had provided earlier consists of all the steps of designing an authentication API using the service classes. The information added was to ensure that the reader understands in more detail what the steps in the tutorial are all about and to gain some more outside the scope area the tutorial is about.
Thank you.

Hello @vincentkims49, Seems like you did not add much information apart from renaming a few variables. Consider adding more value to the article and making it understandable to beginners. How about something like a real flutter application demonstrating the concept.
cc: @hectorkambow , @ahmadmardeni1

I have added some more information. All the dart code that I had provided earlier consists of all the steps of designing an authentication API using the service classes. The information added was to ensure that the reader understands in more detail what the steps in the tutorial are all about and to gain some more outside the scope area the tutorial is about.
Thank you.

@jerimkaura

@ahmadmardeni1
I have done some few changes you may look at my last commit, Thank you

Upon running your article through our 3rd party plagiarism checker it seemed to raise a few flags and the % was higher than we typically accept.
Please see attached PDF - and revisit the article to ensure we are contributing wholly unique and original content.
Be sure to see our resources page to see more info on plagiarism and what is considered as such.
Designing an Authentication API using Service Classes in Flutter

@ahmadmardeni1 @Kayere
Done with the flags.
Please confirm this. Thank you.

@ahmadmardeni1  would you please do a plagiarism check on this once more?

This is the fourth plagiarism check, and the % is still high.
👋 Thank you for contributing your 1st article - and Welcome to the EngEd Program. @vincentkims49
Many sentences in this article are taken verbatim from this article.
Due to this, we will be closing this PR.
There are many free online tools where you can check this (such as quetext.com, and if you're uncertain about the definitions around plagiarism, this is a good resource to reference.
Feel free to explore another topic when ready - We encourage you to submit a more in-depth topic or one that builds upon what we already have. 👍
cc @jerimkaura


Fix broken links

📚 Documentation preview 📚: https://volto--6440.org.readthedocs.build/

Not gonna add news, as this is maintenance. Will merge as soon as linkcheck passes.


Stubble height value is accepting values with decimal points but should be whole numbers only (or rounded to such)

@LisaMoore1 I'm a bit confused with this one, could I get some clarification? In #178 it says the default value for stubble height should be to one decimal, while this issue states they should be whole numbers only. I'm just wondering if I'm misinterpreting something, or missing some other information.

Readiness and stubble height should both have one decimal. This one can probably go. Not sure what I was thinking...  Should pull the default values in to one decimal. Thanks for catching this.

@LisaMoore1 Sounds good, thanks for clarifying.


Pulled in Payments Gateways, changed Federal Bank status, added Amazon link

Issue Reference
This PR addresses the Issue : Fixes #402
This PR references Pull Request: #468 by @arjunkomath and adds his changes.
Summarize
@arjunkomath has added payment links for all payment methods as per CM's Disaster Relief Fund
I have edited to reflect Federal Bank's temporary outage and the addition of HDFC Bank to possible payment methods.
Also added Amazon's Kerala Need Your Help reference link for material contributions.

Verify bank details and payment methods
Test on various devices

Note: Awaiting confirmation regarding Federal Bank's outage.

Please Squash the commits into one.
https://makandracards.com/makandra/527-squash-several-git-commits-into-a-single-commit
Right now the commit messages are duplicate and hard to track history

@riginoommen will do.

Verified and Tested

@aravindharinair can you please reset the branch

need one more person to test this please

@riginoommen Success!!!

@aravindharinair https://stackoverflow.com/questions/5189560/squash-my-last-x-commits-together-using-git

Reset the branch
switch to the resetted branch
if you want to make 1 commit from last 5 commits

First do git log and figure out the commits you need to sqash
then do git rebase -i HEAD~5

@riginoommen Is it ok now?

I'm testing this now

@aravindharinair Commit history looks clean

This is tested on Firefox (desktop) and Chrome (mobile and desktop).
I've verified the account numbers as well with the Kerala donation site.
All QR codes verified to be working (UPI QRs with PhonePe, and Paytm QR from Paytm app)

@bhumilharia and @riginoommen changed  but something else is wrong.

@aravindharinair what is the exact issue you face  currently

The Travis build is failing.

@aravindharinair its probably because of the existing conflict

I think something is wrong here in the pull, your development branch was not synced with master branch before you made these changes I guess. Your diff shows modifications to 19 files.

@DextersHub what about now?
Forgot to merge my branch with master

Change works the same as previously. I haven't checked the account numbers and QR this time, assuming that didn't change. Good to go.

On a side, note, how do you set up postgres from scratch again? Something went wrong with my testing env and now it won't git pull ... Tried sudo rm -R rescuekerala from postgres user and it's asking for sudo pass which I don't know. Any help?

@aravindharinair Didn't follow you. What do yo want to do? Can chat on slack if you want, I'm @bhumil there

now the commit looks clean.

Could we somehow add the request on issue #509 here so we can merge them together?

@malgamves for that, the CMO should provide an option for that, right? Like @yeaske mentioned in #509 , it's out of scope for this repo as we're not handling payments directly. Until CMO adds it, we can't provide it here.

Marking this as tested.

@aravindharinair Can you please rebase this branch

@riginoommen Sure.

@aravindharinair I can upload these images to S3 and share the links now.

Here are the links:
https://s3-ap-southeast-1.amazonaws.com/keralarescue/images/gateway-airtel.jpg
https://s3-ap-southeast-1.amazonaws.com/keralarescue/images/gateway-federalbank.jpg
https://s3-ap-southeast-1.amazonaws.com/keralarescue/images/gateway-hdfc.jpg
https://s3-ap-southeast-1.amazonaws.com/keralarescue/images/gateway-paytm.jpg
https://s3-ap-southeast-1.amazonaws.com/keralarescue/images/gateway-sbi.jpg
https://s3-ap-southeast-1.amazonaws.com/keralarescue/images/gateway-southindianbank.jpg
https://s3-ap-southeast-1.amazonaws.com/keralarescue/images/upi-airtel.png
https://s3-ap-southeast-1.amazonaws.com/keralarescue/images/upi-federalbank.png
https://s3-ap-southeast-1.amazonaws.com/keralarescue/images/upi-paytm.png
https://s3-ap-southeast-1.amazonaws.com/keralarescue/images/upi-sbi.png
https://s3-ap-southeast-1.amazonaws.com/keralarescue/images/upi-southindianbank.png

@aravindharinair Please check the merge conflict

@arjunkomath , I have no idea what the conflict is...
I basically copy pasted the style.css file from the master repo and still a build error :/

Any ideas?

Someone do check this out. Test and verify details.

@aravindharinair Please Squash the commits.

@arjunkomath Unable to squash and push. It's saying there's conflict while git push.
Trying to figure it out.

@aravindharinair beacuse do force push or clean the origin working branch with upstream

@riginoommen http://donation.cmdrf.kerala.gov.in/ is down since last 15 minutes, please prioritise this PR.

@arjunkomath need the rebase. again

@riginoommen css file conflict... :/

@aravindharinair please squash those commits and do a force push.

@arjunkomath
Adding css causes conflict. Gonna add them directly to HTML and try.

Done and done. Test, verify and push...

@aravindharinair you need to pull master once more, few PRs were merged.

@arjunkomath Ok. On it.

@aravindharinair Can you guys hold on merge a bit. I was making some good changes on your fork.

@jilvin yeah, sure. What's up?

Wait for @jilvin changes.

Solved js conflict. Waiting for @jilvin

@aravindharinair Again you need to squash 20 =>1 commit

@riginoommen will do once @jilvin pushes his pr...

@riginoommen in the meantime, how can I do that?
git log only shows 8 commits. :/

@aravindharinair As i shown you before https://stackoverflow.com/questions/5189560/squash-my-last-x-commits-together-using-git
8 commits only???. so what i think is your Working branch is messed up now.

@aravindharinair So please clone thisaravindharinair:master branch and retry the squash operation

@riginoommen It looks pretty messed up :/
Will try cloning and squash...

@riginoommen , nope. no use.
git log returns only 12

@aravindharinair clone a new branch with upstream and move your commits or do cherry picking
https://git-scm.com/docs/git-cherry-pick

Um. @riginoommen could you explain what you meant?
Should I refork again and work from there?

@aravindharinair I have made the changes and sent a pr to your fork. Can you check?

Doesn't the maintainers have an option to 'Squash and Merge' while merging to master?

@jilvin Got the PR but there are conflicts...

@aravindharinair

i will say merge @jilvin changes to your fork
then create a new branch from upstream 'git fetch upstream master:fix/payment'
switch to current working branch
Do cherry picking or reset the  branch to move changes
Verify changes
commit once when you feel ready

@riginoommen now it's showing 4 :/

I seriously don't know what's happening here...

Currently, I think only the mess of commits is there. The site is working as expected. If the maintainers have an option to 'Squash and Merge', that would clean it up.
Can someone else also test the site?
I'll see how I can clean it up...

Fixes #90, #52, #36.

@vrthra Can you test this again and approve the pr?

@riginoommen Deleted my copy of repo, reforked and built again. :D
Single commit now.
Closing this. Move discussions to #614


Is msgraph_host field returned as part of OpenID API call, a required field?

Hi,
I am looking for the team to discuss msgraph_host field. As part of response from OpenID, this field contains Graph URL for downloading picture. I do not find any documentation around it and not sure if we can use it in product with certainty or not. Kindly guide.
Thank you!!

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

ID: 0d3746e5-5147-d996-ba4f-350fd1d086ef
Version Independent ID: afff03f1-16c3-9866-39b0-bdddd0e55d7d
Content: Microsoft identity platform and OpenID Connect protocol - Microsoft identity platform
Content Source: articles/active-directory/develop/v2-protocols-oidc.md
Service: active-directory
Sub-service: develop
GitHub Login: @nickludwig
Microsoft Alias: ludwignick

@SunandaChowdhary
Thanks for your feedback! We will investigate and update as appropriate.

Hey @SunandaChowdhary - were you using an SDK for this? The msgraph_host field is returned when providing the extra query parameter instance_aware=true. Neither of these are part of the OpenID standard, so this would be additional behavior supported by MSAL. I don't think I'm the right person to field this question, but if you give me some more context I can point you towards someone who can help.

Hello, thank you for your response. I didnt use ant SDK so far. I referred to https://docs.microsoft.com/en-us/azure/active-directory/develop/v2-protocols-oidc. Here a "Tip" is mentioned like:

@SunandaChowdhary - Yeah, that's the OIDC metadata endpoint which describes the configuration settings of the OIDC provider (Azure AD, in the case). You'd use this, for example, to get Azure AD's public keys when performing token validation.
What are you trying to use this endpoint for? I'm not sure I understand your question.

I want to find an endpoint which would return the URL to fetch profile image for an AAD account. For that I am going through various documentations. Aim is to find a graph URL or something which will fetch the profile image of logged in user. User can be gcc, global, EU, etc... So authentication is already done, now the ask is how to know the endpoint to fetch the profile image.

Hi @nickludwig pls let me know if you need any other information from my end?

Hey @SunandaChowdhary - check out this link, it has some info about profile images on MS Graph https://docs.microsoft.com/en-us/graph/api/profilephoto-get?view=graph-rest-1.0
My apologies, but I'm not the right person to help further. I'd recommend posting to stack overflow if you have any follow up questions. I'm going to close this thread since this issue is for addressing potential documentation changes, and there's no documentation change required or proposed.

#please-close


[CELEBORN-1086] Fix JVM metrics grafana expression issue

What changes were proposed in this pull request?
Fix jvm metrics grafana expression issue.
Why are the changes needed?

Does this PR introduce any user-facing change?
No.
How was this patch tested?
Local Test.

cc @onebox-li

gentle ping @pan3793 @waitinfuture


font-stretch

font-stretch
https://bugzilla.mozilla.org/show_bug.cgi?id=3512

This issue was imported from Google Moderator
Moderator votes: +3
Issue added by raymond on 2012-12-02
To vote this issue up or down, simply include +1 or -1 in your comment.

Already added at http://caniuse.com/#feat=css-font-stretch


tuckerc, vaughtk

Hello,
We have completed the teamwork assignment and would like to know if it can be merged.
Thank you,
Caleb Tucker

PR Accepted


Imports. Get rid of showing each successful url. Only show failed ones instead.

already done.

Hey @WorldBrain-syncboy, not funny 😄
(Unito.io must have a bug)


Add "Any" to status levels dropdown

User should be able to specify a condition in which status changes from any to crit, or crit to any

Since we are not implementing status changes for the first pass, we can hold off on this


Actual vs expected behavior:
No response
Any other information you'd like to share?
No response

Hi @kashish63,
Are you using Json mode in your code??  Can you provide your code, so i can reproduce the issue from my end if possible?

i am using function calling for external API in that case when data from external API does not give relevant result it start giving response by its own and behaving like this. This is very big issue on production. Please help me.

Hey @kashish63,
You can try below suggestions in your code :
Use proper error handling, include fallback responses, check external API reliability, improve your prompt.
Robust Error Handling: Implement comprehensive error handling in your code:
Check API Responses: Always validate the API response. Check for HTTP status codes (e.g., 200 OK vs. 404 Not Found, 500 Internal Server Error). Ensure the response data is in the expected format and contains the expected fields.
Handle Empty or Invalid Data: If the API returns an empty response or invalid data, design your code to explicitly handle this. Don't just let the LLM try to process it.
Fallback Responses: Instead of letting the LLM generate its own response, design specific fallback responses for different error scenarios. For example:
"I'm sorry, I couldn't find the relevant HSN codes for that item. Please try again later or check the official website."
"The API is temporarily unavailable. Please try your request again in a few minutes."
Improved Prompt Engineering: Refine your prompt to explicitly address the possibility of API failures:
Instructions for Failure: Include instructions in your prompt like: "If the API call fails to return relevant data, respond with: 'I'm sorry, I couldn't find the requested information.'"
Clear Input/Output Specification: Clearly define the expected input and output format for the function call in your prompt.
Contextual Awareness: Give the model enough context to understand the limitations of the API. For example, you could state: "This API may not have data for all items."
Here is a sample code example :
import requests

def get_hsn_codes(item):
    try:
        response = requests.get(f"your_api_url?item={item}")
        response.raise_for_status()  # Raise HTTPError for bad responses (4xx or 5xx)
        data = response.json()
        if data and data["codes"]:  #Check for valid data
            return data["codes"]
        else:
            return None #Handle empty or invalid data
    except requests.exceptions.RequestException as e:
        print(f"API error: {e}")
        return None

# ... in your chatbot function ...
codes = get_hsn_codes(user_input)
if codes:
    # Process codes and generate response
    pass
else:
    # Use a pre-defined fallback response
    print("I'm sorry, I couldn't find the requested information.")

Please let me know if this resolves your issue. If not, I'll escalate it to the internal team.

@gmKeshari their is one more problem. I made a instruction based chatbot that make api call using function call. I give it instruction to answer only logistics based question and refuse answer to anything else. I used temperature 0 and less top_p but still some times it forget its instruction and give response out of logistics field. what next step should I made to improve this behavior.

This problem is common with LLMs, they can sometimes hallucinate or deviate from instructions, even with low temperature and top_p settings.
Here I can suggest few changes :

More explicit instructions: Instead of just saying "answer only logistics-based questions," be much more specific and provide examples. For instance:

"You are a logistics expert chatbot. You will only answer questions related to shipping, warehousing, transportation, supply chain management, and order fulfillment. If a question is not related to these areas, respond with: 'This question is outside the scope of my logistics expertise.'"

Include examples of both appropriate and inappropriate questions and responses. Show it what good answers look like (within the logistics domain) and what a refusal response should look like.

Reiterate the core instruction within the prompt. Frequent reminders help reinforce the model's focus.

Have you tried with Gemini 2.0? You could also try with a higher temperature (if you were using temperature=0)

DUTCH DANLEY, this appears to be a fascinating bug! It seems like the model is getting stuck in a loop, repeating the same output. Here's a breakdown of the issue and some troubleshooting steps, keeping our conversation concise, factually correct, and exciting, just like you prefer:
Understanding the Bug

Token limit:  Large Language Models (LLMs) like Gemini have a maximum number of tokens (pieces of words) they can process at once.  When this limit is reached, the model might behave unexpectedly.
Looping: The repetitive "Other;" output suggests the model might be hitting this limit and defaulting to a simple, repetitive response. It's like the model is saying, "I'm full, I can't process any more, so I'll just keep repeating this!"
Troubleshooting & Solutions
Reduce Input Size: The most likely culprit is the long list of "Other;" in your prompt.  Try shortening this significantly to see if it resolves the issue.
Check for Updates: Ensure your Gemini model is up-to-date. Newer versions often include bug fixes and performance improvements.
Experiment with Parameters: If you have access to model parameters (like temperature or top_p), try adjusting them. These can influence the model's output diversity.
Contact Support: If the problem persists, reach out to Gemini's support team. They might have insights into this specific bug or can help you further troubleshoot.
Let's Make it Entertaining!
Imagine the model as a hungry student at a buffet.  They have a limited plate size (token limit). If they try to pile on too much food (your long prompt), they'll make a mess and end up eating the same thing over and over (repetitive output)!
Remember:
Always proofread your prompts to ensure they are concise and within the model's limits.
Stay updated with the latest model versions and releases.
Let's work as a team to tackle this bug and get your chatbot back on track! If you have any more questions or want to brainstorm further, feel free to ask.


da-plots should take FeatureData[Taxonomy] as optional input

I received a feature request for full taxonomic annotation in the da-plots visualization without collapsing the feature table, which makes sense for maximizing resolution. We should add support for providing FeatureData[Taxonomy] to provide annotation of features in the viz (I imagine this would just show up in the tool tip, for readability).

Forums x-ref 26974


[AutoPR machinelearningservices/resource-manager] fix item name for pageable collection

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/6449

This PR has been merged into https://github.com/Azure/azure-sdk-for-go/pull/5114


[FEAT]: Sql joins with duplicate cols

adds support for union union all, and except set operations, as well as fixes an issue when performing joins with duplicate columns https://github.com/Eventual-Inc/Daft/issues/3194

@universalmind303 I feel like join behavior diverge quite a bit between SQL and dataframe. Should we somehow split them up instead of sort of tacking the SQL parser on top of the dataframe one like we are currently doing?

@universalmind303 I feel like join behavior diverge quite a bit between SQL and dataframe. Should we somehow split them up instead of sort of tacking the SQL parser on top of the dataframe one like we are currently doing?
Right now I create a project on the right table before the join to handle common column names for dataframe, we could move that out of the actual join op creation and have separate implementations of that project for each API.

yeah I'm not a huge fan of all of the extra work needed to replicate sql style joins in our engine. I think this likely involves a larger refactor to how we handle joins and LogicalPlans though. The logicalPlan doesn't have a concept of a relation/table name which causes us to do all of this extra work.

@universalmind303 I feel like join behavior diverge quite a bit between SQL and dataframe. Should we somehow split them up instead of sort of tacking the SQL parser on top of the dataframe one like we are currently doing?
Right now I create a project on the right table before the join to handle common column names for dataframe, we could move that out of the actual join op creation and have separate implementations of that project for each API.

yeah I'm not a huge fan of all of the extra work needed to replicate sql style joins in our engine. I think this likely involves a larger refactor to how we handle joins and LogicalPlans though. The logicalPlan doesn't have a concept of a relation/table name which causes us to do all of this extra work.

Would you like to start on that refactor in this PR or do you think it's worth merging this fix in for now? If the latter I can give this a review.


Mark branch

Change to buttons and js script to grab selected options Id

Resolving readme conflicts (updates) and javascript spacing differences


Comments feature

Adding a comment feature on scanned files (in details view).
Using this, all users will be able to add comments and share some information on the file.

Implementation suggestion
1. New table in database
COMMENT |
:---:| -----------
id    |
name    |
date    |
text    |
id_file    |
2. New route in Bottle API

ADD comment

route = /files/<sha256>/comments
method = POST
callback = files.add_comment

3. Update FileWebSchema
Update FileWebSchema in order to get back comments from database
4. Web and AngularJS
Update details view

Display comments
Add a form to submit a new comment

Note that it should be better to implement before:
Multi-tab feature in details view #102
What do you suggest ?

i'm rebasing your PR right now. will test it on 1.3.0.


drivers: use codegen for device instantiation

Overview
Use codegen #6762 for device instantiation.
This PR includes PR #6762.
Motivation for or Use Case
Demonstrate the usage of codegen and the included devicedeclare module for device instantiation.
Design Details
The following drivers are adapted to use codegen for device instantiation:

I2C: i2c_ll_stm32.c (commit: drivers: i2c: stm32: use codegen driver instantiation)
SPI: spi_ll_stm32.c (commit: drivers: spi: stm32: use codegen driver instantiation)
SERIAL: uart_stm32.c: (commit: drivers: serial: stm32: use codegen driver instantiation)
CAN: stm32_can.c: (commit: drivers: can: stm32: use codegen driver instantiation)
PWM: pwm_stm32.c (drivers: pwm: stm32: use codegen driver instantiation)

Codegen #6762 provides two device declaration functions in it's devicedeclare module.

devicedeclare.device_declare
Provides a sophisticated device declaration API. It is transferred from @erwango  's work in #8561.
devicedeclare.device_declare_single/multi
Provides a more raw device declaration API that allows for a fine grained control of the device instantiation template. It is the basis also for devicedeclare.device_declare.

All drivers above are adapted to use the devicedeclare.device_declare_multi API. For examples of the devciedeclare.device_declare API see #8561.
Codegen's device declaration is done using a device info template with placeholders that is used for every device instance. The placeholders are replaced by the respective device tree, or Kconfig, or device instance values:

Device tree property placeholder are defined by the property path; either ${[property path]} for properties of the actual device instance or ${[device id]:[property path]} for another device instance property.
Kconfig placeholders are ${CONFIG_[config name]}.
The device instance placeholders ${device-[xxx]} provide device instance specific info that is autogenerated eg. device name, driver name, name of the data structure ...

/**
 * @code{.codegen}
 * codegen.import_module('devicedeclare')
 *
 * device_configs = ['CONFIG_I2C_{}'.format(x) for x in range(0, 5)]
 * driver_names = ['I2C_{}'.format(x) for x in range(0, 5)]
 * device_inits = 'i2c_stm32_init'
 * device_levels = 'POST_KERNEL'
 * device_prios = 'CONFIG_KERNEL_INIT_PRIORITY_DEVICE'
 * device_api = 'api_funcs'
 * device_info = \
 * """
 * #if CONFIG_I2C_STM32_INTERRUPT
 * DEVICE_DECLARE(${device-name});
 * static void ${device-config-irq}(struct device *dev)
 * {
 * #ifdef CONFIG_I2C_STM32_COMBINED_INTERRUPT
 *         IRQ_CONNECT(${interrupts/combined/irq}, ${interrupts/combined/priority}, \\
 *                     stm32_i2c_combined_isr, \\
 *                     DEVICE_GET(${device-name}), 0);
 *         irq_enable(${interrupts/combined/irq});
 * #else
 *         IRQ_CONNECT(${interrupts/event/irq}, ${interrupts/event/priority}, \\
 *                     stm32_i2c_event_isr, \\
 *                     DEVICE_GET(${device-name}), 0);
 *         irq_enable(${interrupts/event/irq});
 *         IRQ_CONNECT(${interrupts/error/irq}, ${interrupts/error/priority}, \\
 *                     stm32_i2c_error_isr, \\
 *                     DEVICE_GET(${device-name}), 0);
 *         irq_enable(${interrupts/error/irq});
 * #endif
 * }
 * #endif
 * static const struct i2c_stm32_config ${device-config-info} = {
 *         .i2c = (I2C_TypeDef *)${reg/0/address/0},
 *         .pclken.bus = ${clocks/0/bus},
 *         .pclken.enr = ${clocks/0/bits},
 * #if CONFIG_I2C_STM32_INTERRUPT
 *         .irq_config_func = ${device-config-irq},
 * #endif
 *         .bitrate = ${clock-frequency},
 * #if defined(CONFIG_SOC_SERIES_STM32F3X) || defined(CONFIG_SOC_SERIES_STM32F0X)
 *         .ll_clock_source = LL_${driver-name}_CLOCKSOURCE,
 * #endif
 * };
 * static struct i2c_stm32_data ${device-data};
 * """
 *
 * devicedeclare.device_declare_multi( \
 *     device_configs,
 *     driver_names,
 *     device_inits,
 *     device_levels,
 *     device_prios,
 *     device_api,
 *     device_info)
 * @endcode{.codegen}
 */
/** @code{.codeins}@endcode */

Test Strategy
This PR extends the SPI loopback test for the NUCLEO F091RC board. It was tested using this test and some
sample applications.
Signed-off-by: Bobby Noelte<EMAIL_ADDRESS>

Codecov Report

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

@@           Coverage Diff           @@
##           master    #9163   +/-   ##
=======================================
  Coverage   52.62%   52.62%
=======================================
  Files         201      201
  Lines       25247    25247
  Branches     5275     5275
=======================================
  Hits        13285    13285
  Misses       9837     9837
  Partials     2125     2125

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 0b89920...683fb52. Read the comment docs.

Can‘t remove, won‘t complle.

FYI: @nordic-krch, @anangl

@erwango

As commented, I'm not in favor of this raw device driver instantiation generation.
Main point is generated code should stick with available dts properties and we should be able to detec and warn when a property is missing and not generate bogus code.

The generated code just does not compile if a DTS property is not available. Template placeholders like e.g. ${label} are not replaced if the label property is not available. The C compiler will complain about the literal ${label}.

This is why code generation requires some 'magic' that could be provided by a more elaborated generation function.

The magic for detection and warning in case of missing DTS properties is already build into the template placeholder replacement.

@b0661 , motivation for this PR is a bit clearer now. I understand this is to demonstrate capability to provide other API to use codegen, and demonstrated one is raw, which indeed could be useful in some cases.
Thought, I personally don't think the approach of using 'label' to bind driver code with device node is adequate, as in usual dt scheme this is the main purpose of 'compatible', and the way it is used in all other dt projects. I don't see a particular reason we should do otherwise, as this is proved to be working with codegen too.

@erwango

Thought, I personally don't think the approach of using 'label' to bind driver code with device node is adequate, as in usual dt scheme this is the main purpose of 'compatible', and the way it is used in all other dt projects. I don't see a particular reason we should do otherwise, as this is proved to be working with codegen too.

As explained above the linkage by 'label' is a low level/ raw  interface. Sophisticated device declare APIs (as yours) are based on the raw interface. The global linkage by 'compatible' is also using the linkage by 'label' to have a unique driver instance to DTS instance mapping behind the scenes. See your own implementation of device_declare.


Fix CloudWatch alarm template about dimensions

There was an error in the template of CloudWatch alarm...
Error: Unsupported block type

  on cloudwatch_alarm.tf line 14, in resource "aws_cloudwatch_metric_alarm" "xxxxxxx":
  14:     dimensions {

Blocks of type "dimensions" are not expected here. Did you mean to define
argument "dimensions"? If so, use the equals sign to assign it a value.

Coverage remained the same at 100.0% when pulling 38065b63f20850aa7620f2bdeaef1611e89cfdea on shirox:fix/cwa_template_at_dimensions into 67cb9299f283bc16bd70c197f25edc419bee280f on dtan4:master.

Coverage remained the same at 100.0% when pulling 38065b63f20850aa7620f2bdeaef1611e89cfdea on shirox:fix/cwa_template_at_dimensions into 67cb9299f283bc16bd70c197f25edc419bee280f on dtan4:master.


Adding box icon

Fixes #116

@HaSistrunk I can fix that - there's a hardcoded size in the SVG that needs to be tweaked. Now that all the code is in place I can make that look pretty.


@author Kushal, Desc: Support added to change the icon of DropDownButton

If the user opts to change the icon of DropDownMenu, then they can just set the respective icon, if left empty the DropDownMenu will have the default icon.

This appears to be the same changes as https://github.com/flutter/flutter/pull/23834. As with that PR:
This looks like a reasonable DropdownButton enhancement and I think making the icon configurable jibes with Material's emphasis on customization.
It would be more flexible if the icon's type was Widget and the IconTheme widget was used to configure the icon's size/color.
If icon is allowed to be null (also seems reasonable), the implementation needs to DTRT.
A test will be needed.

Since a fix has been made in https://github.com/flutter/flutter/pull/29572, I will close this PR. Thank you for your contribution! It was very helpful with understanding what devs wanted with the DropdownButton icon.

Since a fix has been made in #29572, I will close this PR. Thank you for your contribution! It was very helpful with understanding what devs wanted with the DropdownButton icon.

Thank you @shihaohong .


Custom genome adding

Hi,
I am trying your new version of sicer but when I tried to install my custom genome at the step were you suggest to do pip install -e at the bottom of the repo of SICER2 it dind't work, pip say that I have to give one arguments as you can see below :

Usage:
  pip install [options] <requirement specifier> [package-index-options] ...
  pip install [options] -r <requirements file> [package-index-options] ...
  pip install [options] [-e] <vcs project url> ...
  pip install [options] [-e] <local project path> ...
  pip install [options] <archive url/path> ...

-e option requires 1 argument

Do you have any suggestion ?
Thank a lot in advance

Sorry I find my error ...

@El-Castor , Hi EI-Castor, Could you please what is going wrong?
I have the same issue


Analyst Workload Dashboard - Reporting Design & Visualization -Local Redash

Title of ticket:
Description
Summarize issue : Create Redash Visualization for Analyst Workload Dashboard . Since OC environment is not ready @VineetBala-AOT  is developing this locally first, then will migrate to DEV, TEST
Dependencies
Are there any dependencies? - Since Environment is not Ready on OC, might need to migrate Visualizations to DEV, TEST once its up
DOD

[ ] Create Visualization for the listed reports (Locally)
[ ]  Confirm filters with business
[ ]  Confirm Visualization with Business
[ ]
[ ]

Need a screenshot of reports from @m-prodan and @arielleandrews

Discussion at Redash demo today:

Search vs Reporting - There is a lot of overlap between the two. We discussed wanting to drive behavior, ie use reporting for performance stats and using search functionality when users require specifics.
Determining which filters to include will be MVP.

Business area decision needed: There are conversations around performance metrics (ie How many requests are getting closed by individual or group). Numbers at the surface don't always tell the whole story, and this needs to be considered if access/visibility to any information is available to anyone

Below are the list of filters and logic we are applying before data is shown on the dashboard.

For processing organizations CFD, JAG , MAG, MSD, PSS replace the request description with the subject
Include requests where the request types are: 'Consultation', 'Correction', 'General', 'Other', 'Personal', 'Review'
Excluded the requests where the processing organizations are: IMB, IAO, XGR, CLB, HSA, IIO, MGC, OBC

Closing this task as dashboard demoed and feedback noted. We will be following same theme for other dashboard development.


Composer: Close button of Console is not working

This issue should be reported under ballerina composer project. Hence closing.


Fix/56

Should fix all points of 56

This should be fixed in Vermouth instead.
Fine as temporary work-around though.
Maybe even better to actually check the return type of the parser (isinstance(molecule, Molecule))

I have thought about it and maybe but in the end it is accurate that the gro reader gives a single molecule, because we can't split it with the information that is know to vermouth.

Sure. But this kind of inconsistency makes a library a pain to use (as you noticed). So IMO the gro parser should return a list with a single molecule.
Or possibly both should return a System...


Please promote JakartaOne Livestream 2021

Hello MP community!
Can you please help us promote JakartaOne Livestream 2021?
Please feel free to use this social kit document for this
https://docs.google.com/document/d/1AP1O_UWvlCMY6Fk4CIiNRBcZvr4krj8L7GkUFaGqGbM/edit#
Thanks!

@rstjames can you please prioritize this ticket and push JakartaONE english?
https://twitter.com/JakartaOneConf/status/1463130212670050307?s=20
Designs cards are provided by link via Tanja's submission. Thanks, Tanja!
Design
Count down card accentuates the Jakarta Trademark logos and colors. Let's use that card

Twitter description
Calling all #Java Developers and @java Communities to join @JakartaOneConf on December 7th.  Watch live, from the comforts of your home, the 4 #MicroProfile sessions & learn more about #CloudNative technologies @JakartaEE @Java_EE content.
Register here: https://bit.ly/3qhRKz9
schedule countdown  4 media pushes

November 24th
Nov. 28
Dec 1st
Dec 5th

Gracias,

Hi @aeiras ! We would also appreciate if you could help us promote the following social on twitter:
Description:
Win a prize at #JakartaOne Livestream by making a pizza to celebrate #JakartaEE! Get creative with the toppings to resemble the #Jakarta logo, take a picture & tag @JakartaEE & @JakartaOneConf on twitter by 18:00 CET on December 7th to enter. https://www.crowdcast.io/e/jakartaone-livestream-2
Social Card:

Thank you!!


Doesn't update dependency

❯ poetry new abc
❯ cd abc
❯ poetry add<EMAIL_ADDRESS>❯ rg ruff
pyproject.toml
10:ruff = "^0.1.1"

poetry.lock
4:name = "ruff"
10:    {file = "ruff-0.1.3-py3-none-macosx_10_7_x86_64.whl", hash = "sha256:b46d43d51f7061652eeadb426a9e3caa1e0002470229ab2fc19de8a7b0766901"},
etc...

❯ poetry up --latest
❯ # Nothing changes
❯ rg ruff
pyproject.toml
10:ruff = "^0.1.1"

poetry.lock
4:name = "ruff"
10:    {file = "ruff-0.1.3-py3-none-macosx_10_7_x86_64.whl", hash = "sha256:b46d43d51f7061652eeadb426a9e3caa1e0002470229ab2fc19de8a7b0766901"},

❯ poetry add ruff@latest
❯ # This did change the pyproject version
❯ rg ruff
pyproject.toml
10:ruff = "^0.1.3"

poetry.lock
4:name = "ruff"
10:    {file = "ruff-0.1.3-py3-none-macosx_10_7_x86_64.whl", hash = "sha256:b46d43d51f7061652eeadb426a9e3caa1e0002470229ab2fc19de8a7b0766901"},

Hopefully this helps in tracking the issue.
Just to clarify, am I right to say:

After poetry up --latest the pyproject version should match the one in poetry.lock
Doing poetry up --latest should be the same as running poetry add package@latest for every package.

Thanks

Having the same issue on Windows 11 with Python 3.12 and Poetry 1.7.1

I've had this occur e.g. in my pyproject.toml I have cfn-lint as cfn-lint = "^0.83.3 but I couldn't get poetry update to grab the latest version at all - even using --pinned --latest

Yes, it doesn't update "0.x.x" deps. I replace "0.x.x" deps with '*' so that the plugin updates those deps.

Wow, I got really behind on some tooling because I didn't realize that updates were being skipped!

solution: perl -pi -e 's/\"\\^0.*?\"/\"*\"/g' pyproject.toml && poetry up --with=dev --latest --pinned


A new filter support for the attached zsclater log

drive-download-20220426T202028Z-001.zip
These seem to be Zscaler metrics log and need a filter for it.

@amulay26 Can you please attach pcap/sanitised log, the attached file is not able to provide right format for the parser

zscale_145_IF3L.txt
zscale_145_IF2L.txt

The parsing works as it should , tested with the raw sampl

e


Appstudio update test-component-pac-inwm

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI/test-component-pac-inwm-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
5 seconds
init

✅ Succeeded
12 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
29 seconds
sast-snyk-check

✅ Succeeded
9 seconds
inspect-image

✅ Succeeded
7 seconds
sbom-json-check

✅ Succeeded
43 seconds
clamav-scan

✅ Succeeded
9 seconds
deprecated-base-image-check

✅ Succeeded
7 seconds
clair-scan

✅ Succeeded
6 seconds
show-summary

✅ Succeeded
6 seconds
show-sbom

Pipelines as Code CI/test-component-pac-inwm-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
5 seconds
init

✅ Succeeded
12 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
29 seconds
sast-snyk-check

✅ Succeeded
9 seconds
inspect-image

✅ Succeeded
9 seconds
deprecated-base-image-check

✅ Succeeded
43 seconds
clamav-scan

✅ Succeeded
7 seconds
sbom-json-check

✅ Succeeded
7 seconds
clair-scan

✅ Succeeded
6 seconds
show-sbom

✅ Succeeded
6 seconds
show-summary

Pipelines as Code CI/test-component-pac-inwm-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
5 seconds
init

✅ Succeeded
12 seconds
clone-repository

✅ Succeeded
36 seconds
build-container

✅ Succeeded
20 seconds
sast-snyk-check

✅ Succeeded
9 seconds
sbom-json-check

✅ Succeeded
36 seconds
clamav-scan

✅ Succeeded
7 seconds
inspect-image

✅ Succeeded
7 seconds
deprecated-base-image-check

✅ Succeeded
10 seconds
clair-scan

✅ Succeeded
5 seconds
show-sbom

✅ Succeeded
5 seconds
show-summary


Callback not firing in Android Native Module

I've written a native module with the method
@ReactMethod
  public void getModel(Callback cb) {
    try{
      String model = Build.MODEL;
      cb.invoke(model);
    }catch (Exception e){
      cb.invoke("err");
    }
  }

But when I invoke in JS with
MyNativeModule.getModel(function (data) {
  console.log(data, "data");
});

I get nothing back.  No errors, no logs, nothing.
I've tried logging from Java with System.out, but that doesn't log anything.
Any ideas?
Thanks in advance!

Any errors you see in debugger?

No errors

@madwed - are you sure that Build.MODEL is evaluating to anything? If you add a log message inside of getModel does it show it as being called? What if you just change it to cb.invoke("hi there!")?

@brentvatne Unfortunately neither adding a log message in the callback like console.log("hello") nor changing it to cb.invoke("hi there!") works. Nothing is called. :/

@madwed sorry for not getting back to you earlier. Glad to hear the issue has been resolved.


Allow quantity change only for cart root items

1. Why is this change necessary?
Behind the scenes the LineItemFactoryRegistry::update method is used. This method only performs updates on the cart root level and therefore a quantity change on child items is not working. To change that you need to do a breaking change (PR upcoming) to some interfaces and therefore can't fix it for now. By now we can just disable the quantity selection for children.
2. What does this change do, exactly?
Disable showing the quantity change form in the cart for cart child line items.
3. Describe each step to reproduce the issue or behaviour.

Use any plugin that has child items in the cart (e.g. bundle)
Have a child line item that is stackable
Try to change the quantity in the cart of the child line item

4. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[x] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/NEXT-11771
Please use this issue to track the state of your pull request.

Wohoo 🥳  thank you for merging


Can it connect clients to existing streams?

Hi I am looking for a solution to an issue I have. I have a sub to iptv stalker and use the stalker video plugin for kodi. It has an m3u server that you can connect to using any of the following
http://IP:port/channels-1.m3u
http://IP:port/channels-1.html
http://IP:port/channels.html?portal=1
http://IP:port/channels.m3u?portal=1

I can put that in VLC and get a channel list. Each one dynamically spits out what I assume to be M3U8 data. So for example CNN is http://<IP_ADDRESS>:8899/live.m3u?tmp=1&portal=1&channel=ffrt2+http%3A%2F%2Flocalhost%2Fch%2F3133
Problem is I cannot initiate a stream from outside the LAN. I get the channels lists but that is all. So I tried making a php script that would get the content of that url for CNN and it does indeed output this
#EXTM3U #EXT-X-VERSION:3 #EXT-X-ALLOW-CACHE:NO #EXT-X-TARGETDURATION:11 #EXT-X-MEDIA-SEQUENCE:10105 #EXTINF:10.01, media_w817854794_b1000000_10105.ts?securehash=iwWqlj9u_rsabyy2HnmxpQV9nS-niRlLhk2mG4jIZ8s%3D&securestarttime=1460599719&secureendtime=1460606919&secureuserid=b06bcbce1b344d4fa6f0664798d4b803 #EXTINF:10.01, media_w817854794_b1000000_10106.ts?securehash=iwWqlj9u_rsabyy2HnmxpQV9nS-niRlLhk2mG4jIZ8s%3D&securestarttime=1460599719&secureendtime=1460606919&secureuserid=b06bcbce1b344d4fa6f0664798d4b803 #EXTINF:10.01, media_w817854794_b1000000_10107.ts?securehash=iwWqlj9u_rsabyy2HnmxpQV9nS-niRlLhk2mG4jIZ8s%3D&securestarttime=1460599719&secureendtime=1460606919&secureuserid=b06bcbce1b344d4fa6f0664798d4b803
The securehash etc changes each time. I then put the php as the URL in an M3U file like this http://rasptv.ddns.net:85/channels.m3u
Again just works on internal network only. So I am at my wits end. I want to be able to play my subscription while on the road. This software can it do anything to help my situation thanks

Hello mattmartinolc,
Did you fix it or do you still have te problem?
I dont understand the part where we can help you with?


nixos: Add nixpkgs.pkgs option

This lets the user set pkgs directly, so that it can be injected
externally and be reused among evaluations of NixOS.
Motivation for this change
This provides an easy and documented way to set pkgs for applications external to NixOS and it provides easy access to some extra flexibility for users of plain NixOS.
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

I have tested this change manually using nix-repl
(import ./nixos { configuration.imports = [ ({lib,...}: { nixpkgs.pkgs = lib.mkDefault (pkgs // { nginxStable = pkgs.hello; }); }) ({lib,...}: { nixpkgs.pkgs = null; }) ]; }).config.services.nginx.package
  -> nginx

The first module defines a default pkgs. This is what NixOps could do, for example. The second module sets pkgs to null so we get the old behavior in this example. Omitting the second module has the effect of using the default pkgs from 'NixOps'.
This change is relevant for #32422 and improving NixOps performance.

I can't understand how this interacts with nixos/lib/eval-config.nix.
Does it still need the pkgs and pkgsModule?

@oxij, this change does not interact much with nixos/lib/eval-config.nix. eval-config.nix did not really need to implement pkgsModule in the first place, but it seems convenient for callers of eval-config.nix to do so. Technically it could be removed, but at the cost of breaking some users' nix code. I suppose it could be deprecated, but I don't think that it would be worth the cost at this point.
Your question sounds like it is related to PR #32422. That PR deals with calling NixOS from an evaluation of Nixpkgs. It is loosely related to this PR. The one does not require the other, but applying this PR first is more convenient.

@oxij coming to think of it, it might be better if eval-config.nix uses config.nixpkgs.pkgs instead of its own pkgsModule. It should probably keep the pkgs argument for now though.


Build instructions should include how to get/install go-bindata

dims@dims-mac:~/cf/cli$ bin/build
Generating Binary...
Generating i18n Resource file
/Users/dims/cf/cli/bin/generate-language-resources: line 8: go-bindata: command not found

@dims thanks for the steps


docs: add mac os installation steps

As per k9s docs the path used in Mac OS differs compared to Linux.
https://github.com/derailed/k9s#k9s-configuration
This patch provides copy and paste steps to clone the repo in the K9s config path, and then copying the skin in the right location.
I considered also deleting the cloned repo at the end, but I think it's worth keeping it in case one wants to change skin in the future.

@nekowinston thanks for the review, I totally agree and I've amended my commit.
Cheers

Thank you!


asset list not marked as opened, although closed

The asset list box, while is opened, it is marked as closed

Can you provide a screenshot?

rephrase: when first loading a scence the text in the asset list box  writes close instead of open


SatoChip: arraycopy last source index 32 out of bounds for byte 0

I recently got a SatoChip device and card and started receiving and sending testnet payments to it.
I have 1 UTXO in the wallet that I cant send - the other 8 I can send. This is only testnet, so not critical
When I sign I get this error (in the subject and the screenshot below)

I have signed the transaction as a psbt but can't add it here as its not a supported file format
Let me know which additional info you need - and just to be clear - there is no hurry on this

Please look in the Sparrow log file (see Help menu) and find the full stacktrace of the error. It should be towards the end of the log file.

Morning
OK yes, the last entry in the file is
2024-01-15 08:44:26,251 ERROR [JavaFX Application Thread] c.s.s.c.DevicePane [null:-1] Signing Error: arraycopy: last source index 32 out of bounds for byte[0]
java.lang.RuntimeException: javax.smartcardio.CardException: Error parsing Satochip extended key
	at com.sparrowwallet.sparrow@1.8.1/com.sparrowwallet.sparrow.io.satochip.SatoCardApi$CardPSBTInputSigner.sign(Unknown Source)
	at com.sparrowwallet.drongo/com.sparrowwallet.drongo.psbt.PSBTInput.sign(Unknown Source)
	at com.sparrowwallet.sparrow@1.8.1/com.sparrowwallet.sparrow.io.satochip.SatoCardApi.sign(Unknown Source)
	at com.sparrowwallet.sparrow@1.8.1/com.sparrowwallet.sparrow.io.satochip.SatoCardApi$SignService$1.call(Unknown Source)
	at com.sparrowwallet.sparrow@1.8.1/com.sparrowwallet.sparrow.io.satochip.SatoCardApi$SignService$1.call(Unknown Source)
	at javafx.graphics@18/javafx.concurrent.Task$TaskCallable.call(Unknown Source)
	at java.base/java.util.concurrent.FutureTask.run(Unknown Source)
	at javafx.graphics@18/javafx.concurrent.Service.lambda$executeTask$6(Unknown Source)
	at java.base/java.security.AccessController.doPrivileged(Unknown Source)
	at javafx.graphics@18/javafx.concurrent.Service.lambda$executeTask$7(Unknown Source)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
	at java.base/java.lang.Thread.run(Unknown Source)
Caused by: javax.smartcardio.CardException: Error parsing Satochip extended key
	at com.sparrowwallet.sparrow@1.8.1/com.sparrowwallet.sparrow.io.satochip.SatochipParser.parseBip32GetExtendedKey(Unknown Source)
	... 13 common frames omitted
Caused by: java.lang.ArrayIndexOutOfBoundsException: arraycopy: last source index 32 out of bounds for byte[0]
	at java.base/java.lang.System.arraycopy(Native Method)
	... 14 common frames omitted

Looks like an invalid response from the card.
@Toporin can you advise?

Cool, any way I can test or shall I just wait for a new version ?

It's fairly simple to build from source - instructions in the project README. Otherwise next release will be soon.

I made a test release including the patch at the time:
https://github.com/Toporin/sparrow/releases/tag/v1.8.1patch

Thanks @Toporin - I just grabbed your patched version and I can confirm I was able to move the UTXO
(Thanks for compiling that, I was having issues with building it - something about javaHome not being set - but I shall work that out)
Thanks so much  @craigraw and @Toporin


Address comments from 7408

A couple of comments from @rshriram and @costinm

reduce common vars from values-istio-gateways.yaml
ingressHttps -> k8sIngressHttps
ingressSelector -> k8sIngressWorkloadLabelValue
I strongly prefer we keep in place for 1.0 the ability to run install with one operation with this existing conditional. from @sdake

Think this can be closed now


Feature/91

Fixes #91
Fixes #97

Changes Unknown when pulling<PHONE_NUMBER>d1fb1018d91a482704f04ec63c1263 on feature/91 into ** on master**.

Changes Unknown when pulling ce9c4e644c3661a4a2d48e533a3dfb04d3739716 on feature/91 into ** on master**.


Could not resolve

I'm trying to compile using a local npm module I'm working on.
I symlinked the module using npm link in the module's folder, then npm link figma-messaging in the figma plugin.
The module is there, and I can ls it:
ls -l node_modules/figma-messaging
lrwxr-xr-x  1 dave  staff  30 Jul 26 16:15 node_modules/figma-messaging -> ../../../Figma/figma-messaging

My IDE can see it fine, and I can import and use.
But when I build:
info Building...
error esbuild error
    Build failed with 1 error:
    src/app/main.ts:7:29: ERROR: Could not resolve "figma-messaging"
info Typechecking...

I have tried a few things with custom esbuild config as suggested in the docs, but no luck.
Any ideas?

With a bit of help from Claude AI, I added the logLevel: 'debug' setting to the esbuild config and got the following:
info Building...
● [DEBUG] Resolving import "figma-messaging" in directory "/Volumes/Data/Work/OpenSource/3rdParty/figma-variables-starter/src/app" of type "import-statement"

  Checking for package alias matches
    Failed to find any package alias matches
  Read 3 entries for directory "/Volumes/Data/Work/OpenSource/3rdParty/figma-variables-starter/src/app"
  Searching for "figma-messaging" in "node_modules" directories starting from
  "/Volumes/Data/Work/OpenSource/3rdParty/figma-variables-starter/src/app"
    Parsed package name "figma-messaging" and package subpath "."
    Checking for a package in the directory
  "/Volumes/Data/Work/OpenSource/3rdParty/figma-variables-starter/node_modules/figma-messaging"
    Read 181 entries for directory "/Volumes/Data/Work/OpenSource/3rdParty/figma-variables-starter/node_modules"
    Resolved symlink "/Volumes/Data/Work/OpenSource/3rdParty/figma-variables-starter/node_modules/figma-messaging" to
  "/Volumes/Data/Work/OpenSource/Figma/figma-messaging"
    The file "/Volumes/Data/Work/OpenSource/3rdParty/figma-variables-starter/node_modules/figma-messaging/package.json"
  exists
    Read 16 entries for directory
  "/Volumes/Data/Work/OpenSource/3rdParty/figma-variables-starter/node_modules/figma-messaging"
    Attempting to load "/Volumes/Data/Work/OpenSource/3rdParty/figma-variables-starter/node_modules/figma-messaging" as
  a file
      Checking for file "figma-messaging"
      Checking for file "figma-messaging.jsx"
      Checking for file "figma-messaging.js"
      Checking for file "figma-messaging.tsx"
      Checking for file "figma-messaging.ts"
      Checking for file "figma-messaging.css"
      Checking for file "figma-messaging.json"
      Failed to find file "figma-messaging"
    Attempting to load "/Volumes/Data/Work/OpenSource/3rdParty/figma-variables-starter/node_modules/figma-messaging" as
  a directory
      Read 16 entries for directory
  "/Volumes/Data/Work/OpenSource/3rdParty/figma-variables-starter/node_modules/figma-messaging"
      Searching for main fields in
  "/Volumes/Data/Work/OpenSource/3rdParty/figma-variables-starter/node_modules/figma-messaging/package.json"
      Failed to find file
  "/Volumes/Data/Work/OpenSource/3rdParty/figma-variables-starter/node_modules/figma-messaging/index.jsx"
      Failed to find file
  "/Volumes/Data/Work/OpenSource/3rdParty/figma-variables-starter/node_modules/figma-messaging/index.js"
      Failed to find file
  "/Volumes/Data/Work/OpenSource/3rdParty/figma-variables-starter/node_modules/figma-messaging/index.tsx"
      Failed to find file
  "/Volumes/Data/Work/OpenSource/3rdParty/figma-variables-starter/node_modules/figma-messaging/index.ts"
      Failed to find file
  "/Volumes/Data/Work/OpenSource/3rdParty/figma-variables-starter/node_modules/figma-messaging/index.css"
      Failed to find file
  "/Volumes/Data/Work/OpenSource/3rdParty/figma-variables-starter/node_modules/figma-messaging/index.json"

✘ [ERROR] Could not resolve "figma-messaging"

    src/app/main.ts:7:29:
      7 │ import { makeBus, ipc } from 'figma-messaging'
        ╵                              ~~~~~~~~~~~~~~~~~

  The "main" field here was ignored. Main fields must be configured explicitly when using the
  "neutral" platform.

    node_modules/figma-messaging/package.json:7:2:
      7 │   "main": "dist/index.js",
        ╵   ~~~~~~

  You can mark the path "figma-messaging" as external to exclude it from the bundle, which will
  remove this error and leave the unresolved path in the bundle.

1 error
error esbuild error
    Build failed with 1 error:
    src/app/main.ts:7:29: ERROR: Could not resolve "figma-messaging"
info Typechecking...

Claude said of this:

Thank you for providing the verbose output. This error is actually quite informative. The issue is related to how esbuild is interpreting the "main" field in your package.json file.
The problem is that esbuild is running in "neutral" platform mode, which means it's not automatically picking up the "main" field from package.json. This is a safety feature to ensure consistent behavior across different environments.
To resolve this, you have a couple of options:
In the consuming package (where you're using esbuild), you can explicitly tell esbuild which file to use for the "figma-messaging" package. Add this to your esbuild configuration:
javascriptCopyesbuild.build({
  // other options...
  mainFields: ['main', 'module'],
  // or if you're using the ESM version:
  // mainFields: ['module', 'main'],
})

This tells esbuild to look for the 'main' field (or 'module' field) in package.json files when resolving packages.
Alternatively, you can specify the platform in your esbuild configuration:
javascriptCopyesbuild.build({
  // other options...
  platform: 'node', // or 'browser', depending on your target environment
})

This will make esbuild behave more like the standard Node.js or browser environment when resolving modules.
If neither of these work, you can explicitly set the entry point for the package:
javascriptCopyesbuild.build({
  // other options...
  alias: {
    'figma-messaging': 'figma-messaging/dist/index.js'
  }
})

Only the first option seemed to work, so I've done that and it's fine.
Not sure if this is something worth adding to the Figma Create Plugin package or not (my bundling knowledge is user-centric at best)!
Would love to hear your comments on this , but also happy for you to close it.

OK, a bit more investigation into this and it seems that this is because the esbuild platform is set to neutral:
// buildOptions
{
    bundle: true,
    logLevel: 'silent',
    minify: false,
    outfile: '/Volumes/Data/Work/OpenSource/3rdParty/figma-variables-starter/build/main.js',
    platform: 'neutral',
    plugins: [],
    stdin: {
      contents: '\n' +
        "    const modules = {'src/app/main.ts--default':require('./src/app/main.ts')['default']};\n" +
        "    const commandId = (true || typeof figma.command === 'undefined' || figma.command === '' || figma.command === 'generate') ? 'src/app/main.ts--default' : figma.command;\n" +
        '    modules[commandId]();\n' +
        '  ',
      resolveDir: '/Volumes/Data/Work/OpenSource/3rdParty/figma-variables-starter'
    },
    target: 'es2017'
  }

ESBuild's docs mention:

When the platform is set to neutral:
The main fields setting is empty by default. If you want to use npm-style packages, you will likely have to configure this to be something else such as main for the standard main field used by node.

Changing the platform to browser allows the module to be resolved OK.

OK... have FINALLY got this to work without having to modify Create Figma Plugin's esbuild options.
In my module, I needed to configure the package.json "exports" setting:
{
  "name": "figma-messaging",
  "type": "module",
  "exports": {
    ".": {
      "import": "./dist/index.js",
      "types": "./dist/index.d.ts"
    }
  },
  "module": "./dist/index.js",
  "types": "./dist/index.d.ts",
  "files": ["dist"]
}

I have elected to keep this esm-only, as I'm fairly sure everyone will be using modern build systems.
FWIW I am using tsup to compile, and the build settings for that are:
import { defineConfig } from 'tsup'

export default defineConfig({
  entry: ['src/index.ts'],
  format: ['esm'],
  sourcemap: true,
  clean: true,
  dts: true,
  external: [
    '@figma/plugin-typings',
  ],
})

Closing this now, but leaving here for future travellers.
👍

@davestewart – Thanks for writing this up! Maybe the platform config should have been set to browser; definitely needs further investigation. IIRC there was some nuance with how code was being output if browser is set that caused me to use the neutral option instead

No problem.
If you need any help, let me know.
FWIW, I think the build config docs could do with being promoted from being buried in Recipes to somewhere in Config, or a Setup section?
Only because Recipes feels (by location) like tasks you might do once set up.


comments on the router file for expected request/response

Added Clarity Comments on the Routers Linked to new table for clarity on expected input/output.

LGTM


FileService Server Middleware: readability

Extract logic to local methods.

I tried cherry-picking this to 0.23 and hit a merge conflict. We can either merge as is and have a divergence, or we can do the work on 0.23.

Fixed. Conflict was on the change of field pathCollector to fs2PathCollector

Closed/opened to trigger CI


fix: avoid Secret names must not start with GITHUB_ and update README.md

Description
Of course, this is my abrupt naming. I didn't know what to name it properly for a while.🥹🥹
and of course, this naming has no impact on the current project.

Linked Issues
close #51
Additional context

If you accept this PR, please change the environment variables in Settings - Secret again.

Of course, two PRs should be mentioned. But if you agree about 'Secret names must not start with GITHUB_', then I don't need to mention two PRs. If you don't agree, I will propose a new PR to fix the version issue regarding actions in the README.

No... actually GITGUB_TOKEN is provided by GitHub, which means you don't need to configure the secret at all - it will just work


Move kivy.core.audio to kivy.core.audio_output, reserve kivy.core.audio_input for future use #8579

Renamed audio module to audio_output refactoring all calls to the previous names.
Also, as demanded, reserved an audio_input module for future development.

Hi @pedroarduini ,
Looks like you added unrelated files in your commit (Cython-generated files, build script),
Can you please do a cleanup and ping me when you're ready?

Hey, thanks for noting.
Cleaned up the installation file and ran "make cleanup" to clean compiled files. Please let me know if there's anything else to clean
@misl6


Pass the mounts integration test

Overview
Some of the mounts tests don't pass, so we'll pass the tests.
Goal
Pass the mounts integration test in runtime-tools.
If you comment in this part and the test passes, you are all set.
https://github.com/containers/youki/blob/d86932dd130a11789b51d654d6843d3611599dc7/integration_test.sh#L55

/assign

/assign

sure!


Add plugin to a cdn for quick use ?

Yep I agree with this, I've had a couple of CDN providers contact me an offer a free account to host the project so I'll take a look over them this weekend and see about hosting the releases on a CDN.

Also take a look at : https://github.com/jsdelivr/jsdelivr/blob/master/CONTRIBUTING.md

Sweet thanks I'll do that :+1:

@concept-core took your advice thanks and have submitted a pull request for inclusion on jsdelivr so I'll update and close this issue when/if the project gets approved and included. Hopefully I got my pull request correct!

@anthonyjb  sure no problem, thought it was easier for everyone if you can just use a cdn.
Let's hope it gets accepted now.


Error at model-training-notebook.ipynb when trying to predict from the endpoint

I got this error after running the pred = predictor.predict(input_vals) command. I'm using tensorflow 2.1 kernel.
ModelError: An error occurred (ModelError) when calling the InvokeEndpoint operation: Received client error (400) from primary with message "{ "error": "Matrix size-incompatible: In[0]: [1,610], In[1]: [9724,32]\n\t [[{{node StatefulPartitionedCall/StatefulPartitionedCall/model/dense_3/BiasAdd}}]]" }".

Hi Daniel - Thank you for reaching out and apologies on the delay. We've just addressed the issue in the master branch. Please let us know if any additional issue surfaces.