DoH/DoT certificate pinning

One issue with allowing users to trust a validating resolver over a secure channel is that the secure connection itself relies on WebPKI which takes away the advantages provided by DANE.
Pinning is typically not recommended by DoH providers as the public key of the certificate could change, which will eventually break clients' setup. Even though trusting a resolver to do the validation is not ideal or recommended it's still needed by some users that don't want to compile letsdane with unbound and don't have a validating resolver.
Maybe with DANE-TA(2) support (issue #10) we can allow pinning the CA of a DoH or DoT provider? The CA is less likely to change making the setup less brittle. The advantage of this is trusting a single CA used by the resolver instead of all CAs installed on the device.
Users still need to manually provide a TLSA record  for the pinning (which can be optional)

I think CA pinning is a sound idea, will a CAA record suffice for that?
so did i read correctly that will this still give users / resolvers the option of explicit individual cert pinning (e.g. for when i am managing my own personal infrastructure)?

I think CA pinning is a sound idea, will a CAA record suffice for that?

These records are not suitable for pinning. They are mostly intended for public certificate authorities and aren't normally checked by clients (CAs query and check those records before issuing certificates). CAA records include issuer name not the hash or public key of a certificate.

so did i read correctly that will this still give users / resolvers the option of explicit individual cert pinning (e.g. for when i am managing my own personal infrastructure)?

when using a DNS over HTTPS resolver, a bootstrap TLSA record could be used to authenticate the TLS connection of the DoH resolver without relying on the traditional WebPKI:
If we use a TLSA record for this, users would be able to choose DANE-TA usage 2 to pin the CA for the resolver like this (not sure about the flag name yet):
$ letsdane -r https://<IP_ADDRESS> -resolver-tlsa '2 1 1 7b4211cf94e2a37180d57b387d4556987d711c3887d9d31b56d0814a438876a3'

They can also pin an individual leaf certificate using DANE-EE usage 3:
$ letsdane -r https://<IP_ADDRESS> -resolver-tlsa '3 1 1 a1df686ec7285d0343e7afc38a4ca966b2645c6bdb415e58eb54067dc362b47a'

it's a bit weird to create a TLSA record for <IP_ADDRESS> it uses TLS but it's an IP address.
DNS Stamps
Another solution is using dns stamps (also stamp calculator) it works with DoH, DoT and even Oblivious DoH. It encodes the resolver address and the certificate hash into a single string.
The stamp used in this example is for Cloudflare DoH (certificate hash is encoded as well):
$ letsdane -r sdns://AgcAAAAAAAAABzEuMC4wLjGgENk8mGSlIfMGXMOlIlCcKvq7AVgcrZxtjon911-ep0cg63Ul-I8NlFj4GplQGb_TTLiczclX57DvMV8Q-JdjgRgSZG5zLmNsb3VkZmxhcmUuY29tCi9kbnMtcXVlcnk

the nice thing about this solution is it doesn't add any extra flags and easy to copy as a single string. There is a list of public revolvers and their DNS stamps but any resolver can advertise their own stamp to be used here.

thanks for clarifying
the stamp solution seems pretty nice. i'm wondering now why i hadn't heard of it before. is there some caveat other than it being less human readable?

the stamp solution seems pretty nice. i'm wondering now why i hadn't heard of it before. is there some caveat other than it being less human readable?

It was made by the dnscrypt devs. The spec mentions that dnscloak, nextdns, adguard, and a few others support it. I think I've seen it in the dnscloak app but not sure about the others


Fix quadratic iterations in transaction pool ready set

Consider using AtomicUSize to avoid locking here.
Future set needs similar fix it seems.

Why was this pr split? #6244
Please include the other changes here as well.

Just accidentally pushed branch there with cherry-picked commit

We still can merge these both prs. No need for the second one.

We still can merge these both prs. No need for the second one.

I will just rebase the other one once first merged, not a fan of many changes in one pr

We still can merge these both prs. No need for the second one.

I will just rebase the other one once first merged, not a fan of many changes in one pr

We are speaking about 3 log messages....

We still can merge these both prs. No need for the second one.

I will just rebase the other one once first merged, not a fan of many changes in one pr

We are speaking about 3 log messages....

And a logic change that can require to revert/cherry-pick separately

changed to different approach

The name of the pr should also be changed, because you only made a linear iteration O(1) and no quadratic iteration.

It is quadratic over added transactions, each transaction added invokes this linear iteration, so it M*N

bot merge


add usecase with pretrained embeddings

Following the great advice from @rnyak (thank you!!!) I was able to create a use case around using pretrained embeddings.
I use TensorInitializer to provide the model with pretrained embeddings for the movies.
model = mm.DCNModel(
train.schema,
    depth=2,
    deep_block=mm.MLPBlock([64, 32]),
    prediction_tasks=mm.BinaryClassificationTask(target_column),
    embedding_options=mm.EmbeddingOptions(
        embeddings_initializers={
            "movieId": mm.TensorInitializer(pretrained_movie_embs),
        }
    )
)

But while this works, I am not sure that this is what this functionality should look like (I am using preexisting pieces that probably were developed for some other purpose).
The full-blown scenario IMHO should look a little bit more like this:

you still can train embeddings from scratch for each of your categorical variables
but you can also specify any number of pretrained embedding matrices for any of your categorical features

For instance, in an extreme scenario where you have 3 different shots for each product, you should be able to train a regular embedding for that product, and then feed in each of the 3 pretrained image embeddings for each item.
Something like this
    embedding_options=mm.EmbeddingOptions(
        embedding_dims={'product_id': 128}
        embeddings_initializers={
            "product_id_shot_A": mm.TensorInitializer(pretrained_product_id_embs_A),
            "product_id_shot_B": mm.TensorInitializer(pretrained_product_id_embs_B),
            "product_id_shot_C": mm.TensorInitializer(pretrained_product_id_embs_C),
        }
    )

where the model would still be able to tie each of the embeddings to the product_id (to know which row to fetch for a given example).
This is by all means not finished, just probably another step in the conversation.
resolves #421

@radekosmulski thanks for the PR. Arent we already capable of doing this?
    embedding_options=mm.EmbeddingOptions(
        embedding_dims={'product_id': 128}
        embeddings_initializers={
            "product_id_shot_A": mm.TensorInitializer(pretrained_product_id_embs_A),
            "product_id_shot_B": mm.TensorInitializer(pretrained_product_id_embs_B),
            "product_id_shot_C": mm.TensorInitializer(pretrained_product_id_embs_C),
        }
    )

dont we have such functionality right now? if not, what's missing can you please identify? thanks.

@rnyak I am not fully sure if I am understanding the functionality right but if I right now do the following:
model = mm.DCNModel(
train.schema,
    depth=2,
    deep_block=mm.MLPBlock([64, 32]),
    prediction_tasks=mm.BinaryClassificationTask(target_column),
    embedding_options=mm.EmbeddingOptions(
        embeddings_initializers={
            "movieId": mm.TensorInitializer(pretrained_movie_embs),
        }
    )
)

whatever embeddings would be automatically created for my movieId are replaced with the embedding matrix I create via mm.TensorInitializer. The movieId ids map to the indices in the newly created embedding matrix.
If I do the following:
    embedding_options=mm.EmbeddingOptions(
        embedding_dims={'product_id': 128}
        embeddings_initializers={
            "product_id_shot_A": mm.TensorInitializer(pretrained_product_id_embs_A),
            "product_id_shot_B": mm.TensorInitializer(pretrained_product_id_embs_B),
            "product_id_shot_C": mm.TensorInitializer(pretrained_product_id_embs_C),
        }
    )

there is no product_id_shot_A index in my dataset. I would like the embedding matrix to map to movieId index. But I don't think I can do the following either:
    embedding_options=mm.EmbeddingOptions(
        embedding_dims={'product_id': 128}
        embeddings_initializers={
            "movieId": mm.TensorInitializer(pretrained_product_id_embs_A),
            "movieId": mm.TensorInitializer(pretrained_product_id_embs_B),
            "movieId": mm.TensorInitializer(pretrained_product_id_embs_C),
        }
    )
    ```

or can I? 🤔

Will try that out right now 😄

Just tried that
pretrained_movie_embs_A = np.random.random((1682, 64))
pretrained_movie_embs_B = np.random.random((1682, 64))

model = mm.DCNModel(
train.schema,
    depth=2,
    deep_block=mm.MLPBlock([64, 32]),
    prediction_tasks=mm.BinaryClassificationTask(target_column),
    embedding_options=mm.EmbeddingOptions(
        embeddings_initializers={
            "movieId": mm.TensorInitializer(pretrained_movie_embs_A),
            "movieId": mm.TensorInitializer(pretrained_movie_embs_B),
        }
    )
)

and I believe only a single embedding matrix can be associated with a movieId. We can either have the "default" embeddings for movieId, like for any other categorical variable, or we can have the pretrained embeddings that we provide, but we cannot have both or multiple pretrained embeddings.
This would be a super useful feature to have (and would allow us to train multimodal models).
I am thinking that this example might be good to go as is, it demonstrates the use of pretrained embeddings. There is also a workaround to this I believe, by creating duplicated movieId columns, but it would be really elegant if we could tie the same column ID to embeddings coming from different embedding tables
Maybe we could merge this use case and continue the conversation in NVIDIA-Merlin/Merlin#211?

Just tried that
pretrained_movie_embs_A = np.random.random((1682, 64))
pretrained_movie_embs_B = np.random.random((1682, 64))

model = mm.DCNModel(
train.schema,
    depth=2,
    deep_block=mm.MLPBlock([64, 32]),
    prediction_tasks=mm.BinaryClassificationTask(target_column),
    embedding_options=mm.EmbeddingOptions(
        embeddings_initializers={
            "movieId": mm.TensorInitializer(pretrained_movie_embs_A),
            "movieId": mm.TensorInitializer(pretrained_movie_embs_B),
        }
    )
)

and I believe only a single embedding matrix can be associated with a movieId. We can either have the "default" embeddings for movieId, like for any other categorical variable, or we can have the pretrained embeddings that we provide, but we cannot have both or multiple pretrained embeddings.
This would be a super useful feature to have (and would allow us to train multimodal models).
I am thinking that this example might be good to go as is, it demonstrates the use of pretrained embeddings. There is also a workaround to this I believe, by creating duplicated movieId columns, but it would be really elegant if we could tie the same column ID to embeddings coming from different embedding tables
Maybe we could merge this use case and continue the conversation in NVIDIA-Merlin/Merlin#211?

I think it is expected to have one embedding table per input column.  What I meant, if you have say image embeddings per movie_id, you can create an extra column in the input dataframe by copying the movieId column, and name it to something like movieId_image and then add that column name to TensorInitializer like below. Is that not working?

> pretrained_movie_embs_B = np.random.random((1682, 64))
>
> model = mm.DCNModel(
> train.schema,
>     depth=2,
>     deep_block=mm.MLPBlock([64, 32]),
>     prediction_tasks=mm.BinaryClassificationTask(target_column),
>     embedding_options=mm.EmbeddingOptions(
>         embedding_dims={'movieId': 128},
           embeddings_initializers={
>             "movieId_image": mm.TensorInitializer(pretrained_movie_embs_B),
>         }
>     )
> )
> ```

It works but I was not sure if this is the solution we want? That was my entire point.
If we are happy with this approach, of duplicating the ID column, then that's perfect. We have everything we need here 🙂
And I am thinking you might be right -- this might be the preferred way to do this, as opposed to probably significantly increasing code complexity just to address this in a more elegant fashion (via being able to tie multiple embeddings to the same ID column)

@gabrielspmoreira can you provide your input pls? thanks.

It works but I was not sure if this is the solution we want? That was my entire point.
If we are happy with this approach, of duplicating the ID column, then that's perfect. We have everything we need here 🙂
And I am thinking you might be right -- this might be the preferred way to do this, as opposed to probably significantly increasing code complexity just to address this in a more elegant fashion (via being able to tie multiple embeddings to the same ID column)

Let's loop Gabriel in and get his opinion as well.

Yes @radekosmulski . Currently there is a many-to-one mapping between categorical features and embedding table. Meaning that one embedding table can be shared by multiple categorical features in the schema.pbtxt but not the opposite.
In this example snippet for a schema.pbtxt file you have the  categ features clicked_item and previously_clicked_item sharing the same embedding table item_id.
feature {
  name: "clicked_item"
  type: INT
  int_domain {
    name: "item_id"
    max: 3370
    is_categorical: true
  }
}
feature {
  name: "previously_clicked_item"
  type: INT
  int_domain {
    name: "item_id"
    max: 3370
    is_categorical: true
  }
}

Allowing many-to-many relationship between categ features and embedding tables would add complexity to the schema, requiring changes in both NVTabular and Merlin Models.

Another possibility would be "cloning" features in the model side. For example, we could have a CloneFeatures block that clones a set of features with new names. For example:
clone_features_block = CloneFeatures(cloning_dict={"item_id": ["Item_product_image", "item_product_test"]})
new_schema = clone_features_block.get_schema_with_cloned_features(schema)

Then you could define DLRM model using the low-level API like in this example, so that you can create your own embedding_block like this, setting the clone_features_block as pre and providing the new schema which includes the cloned features.
embedding_block = mm.EmbeddingFeatures.from_schema(new_schema, pre=clone_features_block)

Here is a prototype of how such CloneFeatures block could look like
@tf.keras.utils.register_keras_serializable(package="merlin.models")
class CloneFeatures(TabularBlock):

    def __init__(
        self, cloning_dict, name=None, **kwargs
    ):
        self.cloning_dict = cloning_dict
        super().__init__(name=name, **kwargs)

    def call(self, inputs: TabularData, **kwargs) -> TabularData:
        outputs = inputs
        for feat_name in inputs:
            if feat_name in self.cloning_dict:
                new_feature_names = self.cloning_dict[feat_name]
                for new_feat_name in new_feature_names:
                    outputs[new_feat_name] = inputs[feat_name]

        return outputs

    def compute_call_output_shape(self, input_shape):
        output_shape = {k: v for k, v in input_shape.items()}

        for original_feat_name, new_feat_names in self.cloning_dict.items():
            for new_feat_name in new_feat_names:
                output_shape[new_feat_name] = output_shape[original_feat_name]

        return output_shape

    def get_config(self):
        config = super().get_config()
        config["cloning_dict"] = self.feature_names

        return config

    def get_schema_with_cloned_features(self, schema):
        new_column_schemas = []
        for column_schema in schema:
            if column_schema.name in self.cloning_dict:
                new_feature_names = self.cloning_dict[column_schema.name]
                for new_feat_name in new_feature_names:
                    new_col_schema = copy.deepcopy(column_schema)
                    new_col_schema.name = new_feat_name
                    new_column_schemas.append(new_col_schema)

        new_schema = Schema([schema] + new_column_schemas)
        return new_schema

Would that design make sense @marcromeyn @sararb ?

In such cases, you might prefer setting the pre-trained embedding to trainable=False. Currently we don't have in the API an option to freeze weights of embedding tables.
But I know that @marcromeyn is working in a refactory that will make it more flexible to set embedding table options. What are your thoughs on this @marcromeyn ?

Yes @radekosmulski and @rnyak . Currently there is a many-to-one mapping between categorical features and embedding table. Meaning that one embedding table can be shared by multiple categorical features in the schema.pbtxt but not the opposite. In this example snippet for a schema.pbtxt file you have the categ features clicked_item and previously_clicked_item sharing the same embedding table item_id.
feature {
  name: "clicked_item"
  type: INT
  int_domain {
    name: "item_id"
    max: 3370
    is_categorical: true
  }
}
feature {
  name: "previously_clicked_item"
  type: INT
  int_domain {
    name: "item_id"
    max: 3370
    is_categorical: true
  }
}

Allowing many-to-many relationship between categ features and embedding tables would add complexity to the schema, requiring changes in both NVTabular and Merlin Models.

that's a good point.

Yes @radekosmulski and @rnyak . Currently there is a many-to-one mapping between categorical features and embedding table. Meaning that one embedding table can be shared by multiple categorical features in the schema.pbtxt but not the opposite. In this example snippet for a schema.pbtxt file you have the categ features clicked_item and previously_clicked_item sharing the same embedding table item_id.
feature {
  name: "clicked_item"
  type: INT
  int_domain {
    name: "item_id"
    max: 3370
    is_categorical: true
  }
}
feature {
  name: "previously_clicked_item"
  type: INT
  int_domain {
    name: "item_id"
    max: 3370
    is_categorical: true
  }
}

Allowing many-to-many relationship between categ features and embedding tables would add complexity to the schema, requiring changes in both NVTabular and Merlin Models.

that's a good point. why we need to tag item_product_image as item_id in the schema? isnt it an item feature like brand and category, so why not only tagging it as item. May be I am missing something :)

You are right @rnyak . Only the item id feature should be tagged with item_id Tag

Seems the CI is not too happy about the label, I wonder what should we do here? I can create the examples label or would we rather align the CI with the scheme that we have there currently?

@bschifferer it is a test regarding a label on the PR that is failing:

I think it might not like the label area/examples, not sure what would be the best way to fix it (change the label to documentation or amend the test)
I also see that we have another test failing regarding code formatting, will fix this now

A really good day of meetings yesterday 🙂 I would like to thank @bschifferer and @oliverholworthy for their invaluable feedback! 🙂 Apart from working on this PR, I spent some time modifying the example for xgboost based on Oliver's comments. Seems talking to a person who implemented a particular functionality adds another dimension to what is happening in code and is super useful for me in creating examples 🙂 Might start bothering people more actively going forward for their perspective as I continue to work on these notebooks 😄 Thanks a lot for the help again!
I will push a new version of this example in a few minutes, just wanted to leave one comment before I do so.
Tried to add additional explanations in the areas identified by @bschifferer. One thing I was not able to do though that we discussed was showing how the embedding table changes after training. The blocker here was that the embedding table doesn't seem to exist before calling fit on the model (even after compile).

The embedding matrix should be accessible here:
model.layers[0].layers[0].layers[0].parallel_layers['categorical'].embedding_tables
and it indeed is after training.
BTW the fact that all this happens in fit is probably a useful consideration for designing how we can give users the ability to freeze these embedding layers.
I felt training for a single epoch and then showing the embedding table, and only then training for more epochs, is a little bit inelegant, hence opted for not including this comparison. But happy to add it if that would be the call on this one!

@radekosmulski did you create a bug ticket for The blocker here was that the embedding table doesn't seem to exist before calling fit on the model (even after compile). issue? Thanks.

@rnyak apologies, didn't realize this would be a bug, thank you for catching this! Raised one now


NIFI-12846 Fixed Region handling for AWS Assume Role Credentials with…

… VPCE Endpoint URL
Summary
NIFI-12846
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[x] Apache NiFi Jira issue created

Pull Request Tracking

[x] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[x] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[x] Pull Request based on current revision of the main branch
[x] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[ ] Build completed using mvn clean install -P contrib-check

[ ] JDK 21

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[ ] Documentation formatting appears as expected in rendered files

Merged in 1.x via 436f63c52b62a37c256a272b107cce380414df25


Add support for getting clients in a given room

Added a clients function that calls the adapter's clients function, in order to get the connected clients in a room.
Pulls requests implementing the underlying adapter functions:
Redis adapter: https://github.com/Automattic/socket.io-redis/pull/15
Memory adapter: https://github.com/Automattic/socket.io-adapter/pull/5

Was this actually merged, or just closed?

We have another issue tracking this I believe. Is this why you closed @defunctzombie  ?

I thought it was merged per the comment of "coming in next release". Sorry if that is not the case yet.

If this is ready for merge, could the OP please cleanup the commit history.

It's not though.

In which version is this feature coming?

Is it part of 1.3.x or not ? I can't see it in the release notes changes .....
I mean, the real question is: did the bug get fixed somehow via another PR ?

It's the top priority for our next release. https://github.com/Automattic/socket.io/issues/1945

Great! Thanks :)

Not clear if this is in or not? (It's been a few months since last comment on this pull)

:) It is not. I'm guessing... soon.

Any updates on this pr?

+1

+1

getClientsInRoom: (io, roomName) ->
    ret = []
    room = io.sockets.adapter.rooms[roomName]

    return ret if !room

    for socketID of room
      ret.push(io.sockets.connected[socketID])
    ret

+1

I'm still waiting for this feature. Is there any blocker?

@samuel281 needs to be rebased

+1

io['sockets']['adapter']['sids'] is returning empty for me even though a client is clearly connected

For anyone interested, I'm currently using the following to accomplish that:
io.in( room ).clients(function( error, clients ) {
    if (error) throw error;
    for ( var i = 0, len = clients.length; i < len; i++ ) {
        io.sockets.connected[clients[i]].emit( 'something', {some: 'data'} );
    }
});

Closed by https://github.com/socketio/socket.io-adapter/pull/24 and https://github.com/socketio/socket.io-redis/pull/109. Thanks!


PR: Close master memory leak

Description of Changes

[ ] Wrote at least one-line docstrings (for any new functions)
[ ] Added unit test(s) covering the changes (if testable)

[ ] Included a screenshot or animation (if affecting the UI, see Licecap)

Issue(s) Resolved

Fixes #18855
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct:

I will skip the test for now so that the test suite works. I closed one leak but there is still one left that I can't find. It looks like the leak is from the stack somehow?

I will skip the test for now so that the test suite works

Ok, no problem. Let's hope it's not so hard for you to find the leak later.


A release was created but the version was not bumped in package.json

I was trying to pull the latest version of this library v1.1.5 but kept on getting v1.1.4
I found out in the end that I did get the right version with git but npm reported it as v1.1.4
I see that this is because the version number has not been bumped in package.json, is this something that you would do?

I suspect it was just forgotten that we need to do an npm publish after merging. @benhodgson87   is your person to ask to do that :)

Thanks


bug: frequency_penalty Parameter in model.yml Only Functions Correctly with Value 1, Produces Gibberish for other values

Describe the bug
The frequency_penalty parameter in the model.yml file only functions correctly when its value is set to 1. The model generates gibberish responses when the value is set to 0 or 0.5.
To Reproduce
Steps to reproduce the behavior:

Go to the model.yml file.
Set the frequency_penalty parameter to 0.
Run the model and observe the responses.
Repeat steps 2 and 3 with frequency_penalty set to 0.5.
Set the frequency_penalty parameter to 1 and observe the responses.

Expected behavior
The model should produce coherent and contextually appropriate responses regardless of the frequency_penalty value, whether set to 0, 0.5, or 1.
Screenshots

Additional context
This issue affects the consistency and reliability of the model's output, impacting its usability in applications that rely on varied frequency_penalty settings for diverse response generation.

Moved to sprint 24 to handle tensorrt-llm engine

Deprecated due to TensorRT-LLM not supporting Desktop
Parent issue: https://github.com/janhq/cortex.cpp/issues/1742


Unable to connect

When calling .connect(), I'm getting a connection error (connect returns False). I've verified that my token is correct and my bot is enabled. Trying to call some APIs returns a SlackConnectionError:
File ".envs/my_bot/lib/python2.7/site-packages/slackclient/_server.py", line 92, in ping
    return self.send_to_websocket({"type": "ping"})
File ".envs/my_bot/lib/python2.7/site-packages/slackclient/_server.py", line 89, in send_to_websocket
    self.rtm_connect(reconnect=True)
File ".envs/my_bot/lib/python2.7/site-packages/slackclient/_server.py", line 47, in rtm_connect
    self.connect_slack_websocket(self.ws_url)
File ".envs/my_bot/lib/python2.7/site-packages/slackclient/_server.py", line 65, in connect_slack_websocket
    raise SlackConnectionError
SlackConnectionError

For instance, I followed the example in the README, and got this. Running .api_call('api.test') does appear to work, though.

It appears that my virtualenv was corrupted. You can close this. Thanks!

Got it!


[git-exporter] More accurate instructions

Which addon?
Git-exporter
How do i initially setup this addon? Could you add more informations?

I am facing always the same error. I have setted up a fresh github repository and it won't push the contents to it.
Here  is the full log:

[16:24:04] INFO: Start git export
[16:24:05] INFO: Pull latest
Auto packing the repository in background for optimum performance.
See "git help gc" for manual housekeeping.
warning: unable to rmdir 'config': Directory not empty
HEAD is now at f17e099 first commit
[16:24:05] INFO: Get Home Assistant config
[16:24:14] INFO: Get addon repositories
[16:24:15] INFO: Commit changes and push to remote
warning: adding embedded git repository: config
hint: You've added another git repository inside your current repository.
hint: Clones of the outer repository will not contain the contents of
hint: the embedded repository and will not know how to obtain it.
hint: If you meant to add a submodule, use:
hint:
hint: 	git submodule add  config
hint:
hint: If you added this path by mistake, you can remove it from the
hint: index with:
hint:
hint: 	git rm --cached config
hint:
hint: See "git help submodule" for more information.
Auto packing the repository in background for optimum performance.
See "git help gc" for manual housekeeping.
[master eb5c066] Home Assistant Git Exporter
2 files changed, 113 insertions(+)
create mode 100644 addons/repositories.yaml
create mode 160000 config
fatal: The current branch master has no upstream branch.
To push the current branch and set the remote as upstream, use
git push --set-upstream origin master

Did you manually checkout the git repo? Or did you let the addon do it on the first run?
Did it start as a repo on disk, or did you create it originally on GH?

So if I start with an empty repo, but don't even create the expected branch...
[15:41:59] INFO: Start git export
[15:42:00] INFO: Create local repository
[15:42:00] INFO: Clone existing repository
Cloning into '/data/repository'...
warning: You appear to have cloned an empty repository.
error: pathspec 'main' did not match any file(s) known to git

If I create a new repo, adding a LICENSE and a README.md, onto the default branch of main...
I get further, until it complains about /data/repository/config/configuration.yaml containing secrets. https://github.com/Poeschl/Hassio-Addons/pull/425 improves the output a little for this...
Move the secrets, run it again...
Pushed fine...

It might be worth making a backup of your config, delete the add-on and reinstall it to clear the "repository" it currently has...


ReferenceError with ?log=bind

 Issue by jyasskin
Thursday Apr 10, 2014 at 07:22 GMT
Originally opened as https://github.com/Polymer/polymer-dev/issues/21

c94c8723767b8f173bde0efe22d57781d42decd6 broke the ?log=bind option by removing the inB parameter to bindProperties but leaving it in the console.log expression.

Closing this issue due to age and the release of version 1 of Polymer - please feel free to re-open if this is incorrect.


[core] Improve autolinking

Currently requires links to be on newlines.

Indirectly fixed by change to Alloy.

Unfixed by unusing Alloy.

Fixed by switch to Alloy.


Performance regression: PlaintextNonPipelined

A performance regression has been detected for the following scenarios:

Scenario
Environment
Date
Old RPS
New RPS
Change
Deviation

PlaintextNonPipelined
Linux, h2c, KestrelSockets
2019-09-13 06:42:46Z
736169
695125
-5.58 %
-3.03 σ

Before versions:
Microsoft.AspNetCore.App 3.0.0-rc2.19459.5
Microsoft.NetCore.App 5.0.0-alpha1.19404.5
After versions:
Microsoft.AspNetCore.App 3.0.0-rc2.19461.7
Microsoft.NetCore.App 5.0.0-alpha1.19404.5
Commits:
Microsoft.AspNetCore.App
https://github.com/aspnet/AspNetCore/compare/e090a0ff33f6aa4721c0678358075515dabe08ab...33120b6093eb5ee9ecf603f56e80a6a7d5a25f07
Logs

More CentOS regressions.


fix(tab): make active optional

This change makes the active binding added under #5425 optional.  Plunk with updated code.

LGTM


Factory History Upgrade

Merge _history with _views, and _history with _controllers

this should be done with all views

This wont work.


0.16.0

Required for https://github.com/denoland/deno/pull/13563

Waiting for new deno_ast release

Waiting for new deno_ast release

Locked deno_ast to 0.11.0 (Same as CLI) for now.


Update 05.methods.md

Codecov Report

Merging #736 (fa876e3) into master (a405521) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #736   +/-   ##
=======================================
  Coverage   85.78%   85.78%
=======================================
  Files          22       22
  Lines        1688     1688
=======================================
  Hits         1448     1448
  Misses        240      240

Flag
Coverage Δ

unittests
85.78% <ø> (ø)

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

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 a405521...28b32d8. Read the comment docs.

Codecov Report

Merging #736 (fa876e3) into master (a405521) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #736   +/-   ##
=======================================
  Coverage   85.78%   85.78%
=======================================
  Files          22       22
  Lines        1688     1688
=======================================
  Hits         1448     1448
  Misses        240      240

Flag
Coverage Δ

unittests
85.78% <ø> (ø)

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

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 a405521...28b32d8. Read the comment docs.


packaging improvements

Hi ✋  Whilist using kv-asset-handler I've seen some points of improvement worth to share. Would be happy to discuss on each, making sub-issues or going with some PRs to address.
Unnecessary files published inside npm package
Checking unpkg files like tsconfig.json or .github are published which increase install size. You may use files field in package.json to whitelist only dist (or also src)
Legacy dist
I can understand that cloudflare workers support (at least partial) of ES6 so we may keep things like await syntax which may help reduce final worker size (and probably more v8 optimization by using modern syntax). Using tsc, you may use ESNext but also careful not using unsupported features like optional chaining. Or using a modern tool like siroc (/cc @danielroe) or directly esbuild. (benefit of using siroc would be to also generate .mjs output that allows final compiler to do better tree-shaking, publishing public types.
Using full version of mime
mime has a lite entry (25 kB vs 9 kB) which is containing all of standard mimes. This can significantly reduce bundle size. You may import from mime/lite

Three years have passed since the issue was first raised, and yet the mime package remains the main factor contributing to the large bundle size.
Even when with mime/lite, the size of the bundled/minified package is still substantial at 12.1 KiB.
A noteworthy alternative is the mrmime package (whose maintainer is a former Cloudflare Staff—kudos to him!) that builds upon the mime/lite. After bundling and modification, its size is reduced to 10.8 KiB.
The data sources for both the mime and mrmime packages are quite outdated and no longer actively maintained. The last update to the mime-db was a year ago, and numerous issues remain unaddressed.
Moreover, a larger code size leads to slower bootstrap times, higher Time To First Byte (TTFB), and overall poorer performance.

I may be wrong, but I am not sure there is a great appetite for a rework at CF on this one. I did pretty much the entire refactor you are describing in the PR I submitted above (replicating the functionality of mime/mime-lite to remove the dependency and make it more maintainable, ES modules etc) but it ends up being quite a substantial rewrite for questionable(?) gain.
In a sense, from our (customer) perspective, Pages has somewhat superseded kv-asset-handler (although, based on the defaults for Pages, I strongly suspect it is running kv-asset-handler or a modified version/derivative of it).
If Greg (or anyone else at CF) wants to revisit (some, or all of) the changes in my PR, or generally discuss, I'm happy to contribute as needed.


Behave better when GO_GIT_DESCRIBE_SYMBOL is unset

Necessary when working in an environment using set -u. Eg:

a Cloud Foundry buildpack
a multi-buildpack setting -u

Unfortunately, I've had to remove this feature entirely in 36476a80efb196676091c07acfe54e4443f974e3. Here's the description from that commit, for easy reference:

This feature has never worked correctly.

See https://gist.github.com/kr/f22beb85af1ca27890f3
for some analysis. The buildpack runs inside a
pre-commit hook, so the to-be-committed object isn't yet
present in the history to be queried by git commands.
This means every push will see the previous head as its
"current" commit, and the very first push will fail
entirely (since there is no history at all yet).

Further, Heroku is in the process of removing all git
artifacts from the build process. This is meant to
accomodate other non-git deployment systems and make the
build process consistent for all deployment transport
mechanisms.

Until Heroku provides an explicit means of getting some
identifying information into the build process, it seems
there is not much we can do on this front.


Fixes referencing issue in decoder module's API docs

Codecov Report

Merging #132 (8704be8) into main (05ce9aa) will increase coverage by 0.09%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #132      +/-   ##
==========================================
+ Coverage   80.64%   80.73%   +0.09%
==========================================
  Files          65       65
  Lines        2149     2149
==========================================
+ Hits         1733     1735       +2
+ Misses        416      414       -2

Impacted Files
Coverage Δ

ezflow/functional/data_augmentation/operations.py
89.09% <0.00%> (+3.63%)
:arrow_up:


typing period also inserts comma

Please complete the following information:

OS or distro: ArchCraft
neoray version: latest

Describe the bug
Hi! I'm a laptop user, my laptop has a calculator keyboard which I mostly use to type numbers, the strange thing is, when I press the period button at the bottom of this part of my keyboard, it inserts ,. instead of a single period, and it's really annoying.
Steps to reproduce
open a blank buffer or a file and press period button at the bottom of calculator keyboard.

I tried but can not reproduce this. Please open an empty buffer, press Ctrl-v in insert mode, press the key and send what it puts.

@hismailbulut
Sorry for being late, I was out of town.
It puts ,. as I said.

What is your keyboard layout? Does your numpad keys are regular or has different design? Also do you use some sort of keyboard software that translates keys?


ultimatevocalremovergui Virtual Audio Cable Driver

ultimatevocalremovergui as Virtual Audio Cable Driver
Features

Any output sound can be separated, games video microphone etc.
Sound synchronization in real time, no waiting for processing.

options:

not enabled separated
enabled remove human voice
enabled remove Musical instrument

I think this will not come shortly. The current models are optimized for separation-quality, not for speed. Also, many of the models have rather huge FFT window-sizes which will lead to inacceptable audio-delays if you ever manage to run them real-time.
If you're after a real-time tool, you should look for something spleeter-based like this here: https://github.com/james34602/SpleeterRT
HTH

I’d partially agree but also not.
With my absolutely god-awful coding skills, I’ve managed to whip up a script that:
Takes an audio input via pyaudio and listens for one second (which I found to basically be the lowest it can go before just not working) – let’s call this sample 1. Next, I send it through to the model for processing while simultaneously recording another second – sample 2. Once the second second (sample 2) finishes recording, I play the audio from the first second (sample 1) as it begins to record the third second – sample 3 – and the AI processes the second second (sample 2).

As shown in the image and demonstrated by my not terrible but awfully mediocre RTX 2060 Super, I usually am able to run the model 6 times quicker than what the real time input audio is using hdemucs_mmi. That means we have overhead and while I wasn't able to code it in because I'm a moron, you could use the last samples of audio to increase the model's context. ie:

This takes the last one second and the second before and shoves it into the model. Then when it's sent off to the output, it's cut off again to the normal 1 second.
For the sake of everyone's sanity, I won't be posting my script because it is cursed (unless by popular demand you want to cringe and instantly die and somewhat figure out a bit more of what i did) but I used
ultimatevocalremover_api which has some weird quirks that I wasn't able to figure out (such as the input only allowing for wav files, not just raw pcm or other audio data). Also it couldnt figure out how use my GPU so it only used my CPU. Fortunately I have stupidly overpowered cpu r9 5950x and it is able to run the model around 3x real time.
before i descend into madness, ill upload a crappy recording of it working (that is an overstatement, sometimes it plays two things at once and other times it plays the same sample twice, idk why just does LOL. I think it has to do with how idle my cpu is.)
https://github.com/Anjok07/ultimatevocalremovergui/assets/96999629/6861a217-8cdf-44ab-b4fa-a7d1a9e30664
Please someone who knows what they're doing, make a better version of this,
Astrid.

Scratch that fade in idea, it seems like that would add time unless wanted or unless you can code a time machine in python.
I was able to redo my whole project and it needs an exorcist, but it now is able to take in more context from before.
Issues with my code: Everything somehow works, but everything. Note that the delay shouldn't be too bad, recording time + processing time + some buffer or something = basically 2x recording time. With a length of around one second (extra context is ignored), expect about two seconds of delay from input to output. Also again, I am only using my CPU because the code is having a conniption and doesn't CUDA so processing could be quicker.
video
The song in that is Earth by K-391 if you wanted to see what the unsplit version sounds like. (I have no idea why it didn't embed this time but i dont care to figure it out)
Exorcist needed,
Astrid


Get selectedDays

Hello and thanks for your wonderful work!
I'm struggeling with an issue i'm not able to resolve for a few days now, that's why i'm asking here directly:
How can i access {{pickedDays}} directly and asign a variable in jquery to it?
http://stackoverflow.com/questions/43897324/multiple-date-picker-to-jquery-variable
Here i asked on stackoverflow, without a solution...
Something like:
var arrayDates = [];
arrayDates = {{pickedDays}};
Do you have any ideas?
Thanks in advance!

@Robstaer are you using this for Angularjs or Angular

@Robstaer sorry I don't know what you're referring to exactly but this is for Angular 2+ not jquery.... I am going to close this issue.  please restate your question if you need additional help.


Pin ceph container to latest stable vX version

Tested it and everything worked successfully again.
No huge memory usage of anything else. My laptop was all the time between 32% and 40% during the setup.
This is the latest version found with this command:
for i in {1..20}; do
  curl -s -L https://quay.io/api/v1/repository/ceph/daemon/tag?page_size=100\&page=$i | jq '."tags"[] .name';
done | awk '/stable/'

Regarding the command to find the image: That uses https://hub.docker.com/r/ceph/daemon/, which hasn't been updated since two years.  quay.io should have it's own registry api available, did that not work?

Regarding the command to find the image: That uses https://hub.docker.com/r/ceph/daemon/, which hasn't been updated since two years. quay.io should have it's own registry api available, did that not work?

Indeed, I just tried it out with quay (description updated). And the latest stable version that works is the one I used now. There's a newer version, but it complains because the file entrypoint.sh has issues.

Regarding podman and containerd/runc versions: Podman is it's own container engine and doesn't use these two like Docker.  It does seem to have some dependencies on them, but I don't see any indication that these are the cause: https://github.com/containers/podman/blob/main/go.mod

Regarding podman and containerd/runc versions: Podman is it's own container engine and doesn't use these two like Docker. It does seem to have some dependencies on them, but I don't see any indication that these are the cause: https://github.com/containers/podman/blob/main/go.mod

Yes, seems to be using an older runc version compared to yours and the same containerd. But that is definitely weird. Let's pin to this version for now, and maybe we will find out the reason later


What is the difference to https://github.com/FL550/dwd_weather?

What is the difference to https://github.com/FL550/dwd_weather?
What was the reason to build another component that seems very similar?

The reason that both exist is mainly because the other one didn’t exist yet when I started this one, so they were more or less developed in parallel. I just added it to HACS much much later, because I don’t use HACS myself. And even had it running at some for some time, before I even put it on GitHub.
That‘s why when you look at the two integrations, they are quite different in their approach. For example, this one from the beginning on focused much on real measurements which are really good where I live, while the other one only uses forecast data on purpose. However, while this one supports „only“ on the official Weather Entity, the other one provides additional sensors. There are many other differences, best you try out both and decide for yourself, which one better fits your needs.
Because of the huge differences, it would be hard to unify the two, the outcome would probably rather be a third one. 😆 Or maybe both develop step by step in a similar direction and one gets obsolete eventually 🤷♂

Thanks for the detailed explanation. I live close to a weather station, so the actual values should be quite accurate. I will give it a try for sure!

Hi, I am the developer of the other mentioned integration. As you already said, I focused on the forecast data. The reason for this was, that the "real" measured values were published with at least an hour delay and I found the forecasted values quite accurate. Has this changed in the meantime?
If yes, we should try to merge the two integrations, because I was also a bit confused what the differences between the two are.

Hi @FL550,
the quality always depended a lot on the weather station, and I believe it still does. When I started my integration a few years ago, for my closest station the measurements were actually very good (more accurate than the forecast), so it was a no-brainer for me to use them. See e.g. this one, it has fresh measurements every hour:
https://opendata.dwd.de/weather/weather_reports/poi/10739-BEOB.csv
Originally, I even used the unofficial APIs of the official Warnwetter App but switched to the Open Data Server later. My observation is, that the measurement data that is available on the Open Data Server is exactly the same as for the Warnwetter App (the paid one that contains weather, not only warnings), except that they are a bit delayed, but always significantly les than one hour, and usually still better than the forecast for the currently hour for my favorite stations.
Strangely, one station got worse, it suddenly stopped providing the "present_weather" (sunny, rainy etc.), but the other measurement values still work perfectly. But the behavior for the Warnwetter App is exactly the same.
To get the best out of their individual station, I give the user the possibility to configure it:

And of course there are still many stations without measurement data, see https://github.com/hg1337/homeassistant-dwd/blob/main/stations.md. It's also shown in the setup dialog so the user can decide to use a closer station without measurements or one with measurements, but more far away.

So the bottom line is basically, for me the measurements are very valuable and I wouldn't want to miss them, and except for a bit delay, they are exactly the same as for the official Warnwetter app. And each user can decide and configure for themselves what's better in their individual situation.

Added to Q&A.


rename contract, remove reward_per_second field

Codecov Report
Patch and project coverage have no change.

Comparison is base (ba62d06) 87.93% compared to head (430827b) 87.93%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main       #6   +/-   ##
=======================================
  Coverage   87.93%   87.93%
=======================================
  Files           4        4
  Lines         348      348
=======================================
  Hits          306      306
  Misses         42       42

Files Changed
Coverage Δ

contracts/halo-farm-factory/src/contract.rs
74.62% <ø> (ø)

contracts/halo-farm/src/contract.rs
91.04% <ø> (ø)

contracts/halo-farm/src/formulas.rs
87.50% <ø> (ø)

contracts/halo-farm/src/state.rs
100.00% <ø> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

@haitranwang @hoanm cái này tạm ổn thì merge đi xong làm pull mới cho dễ nhìn


[NETBEANS-444] Change all apple.laf.AquaLookAndFeel to com.apple.laf.AquaLookAndFeel…

…. Fixes netbeans-444.

Looks good to me. I see confirmation on the mailing list it works with Java 8 which is nice.

Great. Should we merge this?


[Qt] Give some space for the PS/IS on Send Page

Ideally Balance needs to be aligned to the right a lot more and yes for translations further spacing between "PS" and "IS" is needed.


Issue when accessing hardware h264 decoder

Software Versions

Python: 3.10
OS: Android
Kivy: v2.1.0
Kivy installation method: Buildozer

Describe the bug
I am developing android app with kivy and ffpyplayer for watching video streams.
But i'm using low performance hardware and getting lag on streams with 2500 bitrate and higher, so I decided to use hardware accelerated codecs like h264_v4l2m2m.
I add 'vcodec': 'h264_v4l2m2m' parameter of MediaPlayer in kivy.core.video.video_ffpyplayer.py but get this issue:
12-28 00:44:29.802  2065  2138 I python  : [ERROR  ] [ffpyplayer  ] [h264_v4l2m2m @ 0x7b181c6100] can't configure decoder
12-28 00:44:29.803  2065  2138 I python  : [CRITICAL] [ffpyplayer  ] Failed to open file 'http://<IP_ADDRESS>:80/cam1/stream.m3u8' or configure filtergraph
12-28 00:44:29.787  2138  2138 W Thread-6: type=1400 audit(0.0:25): avc: denied { read } for name="/" dev="tmpfs" ino=7196 scontext=u:r:untrusted_app:s0:c512,c768 tcontext=u:object_r:device:s0 tclass=dir permissive=0

I found some information about this issue and it seems that
W Thread-6: type=1400 audit(0.0:25): avc: denied { read } for name="/" dev="tmpfs" ino=7196 scontext=u:r:untrusted_app:s0:c512,c768 tcontext=u:object_r:device:s0 tclass=dir permissive=0

is a SELinux denial. So SELinux blocks ffmpeg when trying to access the hardware codec but I don't understand why this is happening.
I also post this issue here: https://github.com/matham/ffpyplayer/issues/137 because I don't know it is ffpyplayer or kivy problem.

Not an issue on Kivy side, if the codec is usable on Android, the issue should be addressed on python-for-android or ffpyplayer.


Test fail when not in iiasa network

When installing ixmp from source into an otherwise clean new anaconda (python 3.7) environment outside the iiasa environment several test failed (see picture attached). Is this the expected behavior or did I do something wrong with the isntallation?

@ClaraLuisa from a screenshot, it is not possible to see which tests have failed or the associated output/exceptions/traceback, and thus not possible to debug.
Can you please copy the whole output—from where you call pytest to what's shown here—into a text file and attach it to this issue? Please also attach the output of pip list.

Dear @khaeru khaeru, unfortunately there is too much output from the test scripts to copy it all. However I have attached the output from the last 5000 lines. Please let me know if this helps else, I'm happy to share my screen with you while the script executes.
conda prompt test report.txt
output pip list.txt

I have attached the output from the last 5000 lines.
conda prompt test report.txt

Thank you! I pulled out the relevant lines and put them in the issue description above.
Respectively:

test_init_set: In this case, the code in JDBCBackend should catch the exception and re-raise it as a RuntimeError, as show here: https://github.com/iiasa/ixmp/blob/399e1b04db32b8fb6f090dba611a900fd4c421ea/ixmp/backend/jdbc.py#L484-L489 I'm not sure why this is failing in your environment.
test_reporter_visualize:graphviz.backend.ExecutableNotFound: failed to execute ['dot', '-Tpng'], make sure the Graphviz executables are on your systems' PATH

This tests a reporting feature that requires the graphviz program. ixmp automated installation of the python-graphviz package, but you probably will need to install this program.
test_close:AssertionError: assert 'at.ac.iiasa....e connection!' == 'Database con...lready closed'
  - at.ac.iiasa.ixmp.exceptions.IxException: Error closing the database connection!
  + Database connection could not be closed or was already closed

This seems similar to (1).

At first glance, what's happening in (1) and (3) is that the exceptions returned by the Java/ixmp_source code, when converted to str(), have the class name prepended ("at.ac.iiasa.ixmp.exceptions.IxException: This Scenario cannot..." instead of "This Scenario cannot..."). ixmp does not expect this, so it fails to convert these Java exceptions into normal Python exceptions.
I've changed the title to better describe the issue. We will need to think a little further to diagnose why this happens on your system but not on Travis, AppVeyor, or other users' systems here.
The good news is that ixmp is functioning properly; the only issue is that some code might raise a different exception class from RuntimeError. If you are not using try: ... except: clauses in your code, this is probably not an issue.


update rule: react-a11y-anchors - checks should span files

These two checks should keep state across multiple files:
Links with the same HREF should have the same link text.

(http://oaa-accessibility.org/wcag20/rule/39/)

Links that point to different HREFs should have different link text.

(http://oaa-accessibility.org/wcag20/rule/41/)

☠  It's time! ☠
Per #876, this repository is no longer accepting feature pull requests. TSLint is being deprecated and we recommend you switch to https://typescript-eslint.io.
Thanks for open sourcing with us, everyone!


[#cleanup] remove unnecessary code

Codecov Report

Merging #6282 into master will increase coverage by 0.02%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #6282      +/-   ##
==========================================
+ Coverage   39.56%   39.58%   +0.02%
==========================================
  Files        3061     3061
  Lines       91401    91396       -5
  Branches    11845    11842       -3
==========================================
+ Hits        36163    36180      +17
+ Misses      52256    52233      -23
- Partials     2982     2983       +1

Impacted Files
Coverage Δ

...boss/interceptor/ContextInvocationInterceptor.java
0% <ø> (ø)
:arrow_up:

...nt/interceptor/ChannelBasicPublishInterceptor.java
0% <0%> (ø)
:arrow_up:

...p/pinpoint/rpc/stream/StreamChannelRepository.java
75% <0%> (-10%)
:arrow_down:

...cluster/zookeeper/ZookeeperClusterDataManager.java
52.9% <0%> (-1.94%)
:arrow_down:

...java/com/navercorp/pinpoint/rpc/DefaultFuture.java
72.5% <0%> (-1.67%)
:arrow_down:

...rc/main/java/com/navercorp/pinpoint/test/Item.java
70% <0%> (ø)
:arrow_up:

...point/rpc/client/DefaultPinpointClientHandler.java
72.4% <0%> (+0.32%)
:arrow_up:

...orp/pinpoint/rpc/server/DefaultPinpointServer.java
80.98% <0%> (+1.05%)
:arrow_up:

...om/navercorp/pinpoint/rpc/codec/PacketDecoder.java
59.57% <0%> (+2.12%)
:arrow_up:

.../pinpoint/rpc/client/PinpointClientHandshaker.java
65% <0%> (+5%)
:arrow_up:

... and 2 more

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 3632455...d08df24. Read the comment docs.


Accept float in days_over_precip_thresh

Pull Request Checklist:

[x] This PR addresses an already opened issue (for bug fixes / features)
[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)
[ ] HISTORY.rst has been updated

After merging to master and before closing the branch:

[ ] bumpversion (minor / major / patch) has been called on master branch
[ ] Tags have been pushed

What kind of change does this PR introduce?

Add support for simple annual percentiles.

Does this PR introduce a breaking change?
No

Other information:
TODO

[ ] Review code and naming.
[x] Apply same logic to other relevant functions

This PR fixes #79

Pull Request Test Coverage Report for Build 1043

10 of 11 (90.91%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 92.541%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

xclim/utils.py
7
8
87.5%

Totals

Change from base Build 1041:
0.02%

Covered Lines:
1340

Relevant Lines:
1448

💛  - Coveralls

Is the name resample_doy sufficiently informative ? resample_from_doy ?

@Zeitsperre Are the azure failures expected ? Is this something we really want to invest in ?

@huard Yeah, I'm thinking it might be worthwhile to move from Azure as well. The goal was simply to perform the same sort of platform checks as macOS. I learned a few weeks ago that Travis CI recently added a Windows image. I'd much rather use their infra.
I'll open an issue for replacing Azure with Travis CI for Windows builds.

@tlogan2000 Could you take a look ?

@huard The Azure builds are no longer valid. There was a misconfiguration in their .yaml and I didn't see the value in investing time into making them all work. I feel like we can wait for Travis CI to create some Windows images.
For this PR, I would simply ignore those checks. Everything is passing on Travis.

@tlogan2000 Could you take a look ?

I will take a look at this soon... sorry it slipped through a crack I think

@huard with the roll back of float/string input what new features does this PR still include?  Is it the doy_resample()?

Yes, that's mostly it. I'd call it a clean-up of how doy percentiles are handled.
Temperature indicators require a doy percentile, while precip percentiles also accept a single "annual" value.

Yes, that's mostly it. I'd call it a clean-up of how doy percentiles are handled.
Temperature indicators require a doy percentile, while precip percentiles also accept a single "annual" value.

Sounds good

Made change in docstring. Won't really work as-is however until xarray 0.13 because keep_attrs does not yet work for DataArray.quantiles.

@huard if you want to update the history and bumpversion patch master, I can pull the new master and send it to PyPI/conda-forge.

@Zeitsperre
So bumpversion does not actually change the tag anymore ? I had to commit manually the change and call git tag myself... weird.

The tagging is re-enabled on another PR since I realize it doesn't cause commits to fail (only commit = true seems to do that). I'll inform people when that is in master.
I had it turned off since it was conflicting with pre-commit. It turns out that the bumpversion python library has a few bugs with automatically adding lines (causing code formatting to fail). This might be fixed in a fork that Birdhouse uses called bump2version but I need to look into it.


can't install elm on ubuntu 19.04

ubuntu 19.04

npm install -g elm
produced following log:

0 info it worked if it ends with ok
1 verbose cli [ '/usr/bin/node', '/usr/bin/npm', 'i', 'elm' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose config Skipping project config: /home//.npmrc. (matches userconfig)
5 verbose npm-session a491879c9a532655
6 silly install loadCurrentTree
7 silly install readLocalPackageData
8 http fetch GET 304 https://registry.npmjs.org/elm 925ms (from cache)
9 silly pacote tag manifest for elm@latest fetched in 939ms
10 timing stage:loadCurrentTree Completed in 1003ms
11 silly install loadIdealTree
12 silly install cloneCurrentTreeToIdealTree
13 timing stage:loadIdealTree:cloneCurrentTree Completed in 1ms
14 silly install loadShrinkwrap
15 timing stage:loadIdealTree:loadShrinkwrap Completed in 2ms
16 silly install loadAllDepsIntoIdealTree
17 silly resolveWithNewModule<EMAIL_ADDRESS>checking installable status
18 silly removeObsoleteDep removing<EMAIL_ADDRESS>from the tree as its been replaced by a newer version or is no longer required
19 timing stage:loadIdealTree:loadAllDepsIntoIdealTree Completed in 6ms
20 timing stage:loadIdealTree Completed in 11ms
21 silly currentTree ***
21 silly currentTree └──<EMAIL_ADDRESS>22 silly idealTree ***
22 silly idealTree └──<EMAIL_ADDRESS>23 silly install generateActionsToTake
24 warn checkPermissions Missing write access to /home//node_modules/elm
25 warn checkPermissions Missing write access to /home//node_modules
26 timing stage:rollbackFailedOptional Completed in 1ms
27 timing stage:runTopLevelLifecycles Completed in 1025ms
28 silly saveTree ***
28 silly saveTree └──<EMAIL_ADDRESS>29 warn enoent ENOENT: no such file or directory, open '/home//package.json'
30 verbose enoent This is related to npm not being able to find a file.
31 warn *** No description
32 warn *** No repository field.
33 warn *** No README data
34 warn *** No license field.
35 verbose stack Error: EACCES: permission denied, access '/hom//node_modules/elm'
36 verbose cwd /home/
37 verbose Linux 5.0.0-21-generic
38 verbose argv "/usr/bin/node" "/usr/bin/npm" "i" "elm"
39 verbose node v12.7.0
40 verbose npm  v6.10.0
41 error path /home//node_modules/elm
42 error code EACCES
43 error errno -13
44 error syscall access
45 error Error: EACCES: permission denied, access '/home//node_modules/elm'
45 error  [Error: EACCES: permission denied, access '/home//node_modules/elm'] {
45 error   stack: "Error: EACCES: permission denied, access '/home//node_modules/elm'",
45 error   errno: -13,
45 error   code: 'EACCES',
45 error   syscall: 'access',
45 error   path: '/home/***/node_modules/elm'
45 error }
46 error The operation was rejected by your operating system.
46 error It is likely you do not have the permissions to access this file as the current user
46 error
46 error If you believe this might be a permissions issue, please double-check the
46 error permissions of the file and its containing directories, or try running
46 error the command again as root/Administrator (though this is not recommended).
47 verbose exit [ -13, true ]

All that the npm install route is doing is downloading a binary from https://github.com/elm/compiler/releases/tag/0.19.0, so you can get unstuck by downloading it directly.
The error message you are showing here is saying that there may be a permissions issue:
It is likely you do not have the permissions to access this file as the current user

I have seen this kind of thing happen with npm recently. I usually try adding sudo or searching online for the particular npm error. Do you think this is something specific to Elm though?

@evancz, thanks for your response...
Tried with sudo ... didn't work ... same response
Guess I should try the binary file.

I'm wondering about the path /home//node_modules/elm did you remove you username from the output? otherwise that would be weird / a problem.

This doesn't seem like a bug in Elm itself, so I recommend asking for help in the Elm slack where folks can give you feedback really quickly. This seems like something a lot of folks have experience with, and they can probably give good recommendations in that format!


⚠ AVS (kty) has degraded performance

In 4d1be09, AVS (kty) (https://avus.com.br) experienced degraded performance:

HTTP code: 200
Response time: 8456 ms

Resolved: AVS (kty) performance has improved in 03518ca.


Support migration from ChirpStack V4

Summary

Closes #58
Changes

Use ChirpStack V4 API
Unify flag names between sources

Testing

Tested with local ChirpStack v4 instance.
Regressions

Flag name changes affect all sources. TTS and ChirpStack sources were tested.
Notes for Reviewers

ChirpStack v3 and v4 protobuf APIs conflict with each other and cannot be compiled into the same binary.
See https://github.com/TheThingsNetwork/lorawan-stack-migrate/pull/75/files#r1232266208 since I'm not sure if this is handled correctly
Checklist

[x] Scope: The referenced issue is addressed, there are no unrelated changes.
[ ] Documentation: Relevant documentation is added or updated.
[ ] Changelog: Significant features, behavior changes, deprecations and fixes are added to CHANGELOG.md.

I think this is acceptable going forward indeed. I don't see another clean solution either.

@happyRip: What's the status of addressing the review comments?

replaced by https://github.com/TheThingsNetwork/lorawan-stack-migrate/pull/112


url or templateUrl support return script

I combine Framework7 with PHP7.2
now I like use componentUrl because that I can manage code and framework7 code by file
now, can possible if feature route with url or templateUrl but support return script
<script> return { 	on: { 		pageInit: function (e, page) { 			// script for this page after load url 		}, 	} } </script>
Are that possible?
becase I need that for manage code for more flexible
2 year I'm use framework7 and not have issue but this important case I found in my project

templateUrl with return script is already what componentUrl is


Public distribution group urls are unsecure compared to HockeyApp

In our company, we use public distribution urls to publish alpha versions to clients and testers and we have projects for each of them.
The public distribution urls are based on the project id, which is a random alphanumeric:
https://rink.hockeyapp.net/apps/<project_id>/app_versions/67
But in AppCenter, public distribution groups are like this:
https://install.appcenter.ms/orgs/<organization_name>/apps/<project_name>/distribution_groups/<group_name>
So, it is quite easy to, given an organization name, try to guess another project names.
Our suggestion is to have public urls based on project id, instead of project name.

@gmerinojimenez thanks for the feedback. The public download pages on HockeyApp aren't considered "secure" either, however, I see your point that at least the IDs are not guessable.

Sure, I agree. If it is public, it will be unsecure, but considering that those urls are shared between a small group of people, for us having them obscured by the id is safe enough

Referencing https://github.com/microsoft/appcenter/issues/1255

I think this issue is still valid, are there any plans to take it into account?


Document best practices/requirements for writing JAX libraries

One of the JAX best practices (enforced internally by our test harness) is to never use any JAX functionality at import time before flags are parsed.  This rule is not really stated in any public facing documentation that I could find, and, most code that ignores it works (with some potential edge cases that are very difficult to catch with tests).
For example, this pull request on Diffrax (https://github.com/patrick-kidger/diffrax/commit/23a1c199ac5e50d86a500cddd612b3b06ab45c50) passes all tests, but, by invoking JAX functionality at import time, breaks our own stricter internal tests.

cc @patrick-kidger

Actually, beyond just documentation -- how do these stricter internal tests function? I'd like to (a) detect this through the Diffrax codebase to fix all occurences, and (b) add a test to prevent a regression.

(CC @froystig as @FedericoV wasn't sure.)

JAX and many of its downstream libraries obey a rule: do not call into (most) JAX C++ code before main() is called (with a small number of exceptions, e.g., jit decorators). In particular, we do not execute jit-compiled code or call into jax.numpy functions at module initialization time. The most common place this comes up is for parameterized test harnesses.
However, this rule is not something we require users of JAX to follow in general! The rule stems from the fact that main also initializes a bunch of other C++ code in our Google-internal builds, and it is not safe to call, say, XLA, before that initialization takes place. Python code which wants to run inside Google must obey this rule. So any JAX libraries that Google folks want to consume must also obey the same rule.
Now, whether that's something you care about is entirely up to you: if you aren't using the Google-internal setup you have little reason to care about this, but if you have collaborators inside Google they will certainly appreciate it if you follow this rule!
However, it's probably a reasonable rule to follow even outside Google. JAX will initialize its backends on the first call into a jit compiled-function. If you want to for example parse flags or configure JAX before initialization, you will need to arrange that that configuration happens before JAX is initialized. By avoiding accidental initialization from say, harnesses calling jax.numpy functions you can control when initialization happens and make sure that any configuration you need happens first.

Maybe we could insert a check somewhere in the config implementation of whether an option is read ahead of flags being parsed?

That'd be great!


Test Reports and Email notifications issue

Hi,
Since the last 2 days, I'm getting discrepancies in the test tab of my release and the corresponding email that we receive.
Here is an example:
Report in test tab of release:

Report of same test in Email:

2 days ago, we were getting correct emails as well. Please look into this.
Also is there a better option for test reportion?
I explored Test Plans, but the build queue dropdown in settings is not properly paginated, and I cannot scroll down to our queues.

@manu4rhyme I am assuming working with the team the data issue is resolved. Can you please share feedback on 'better option for test reporting?'


labelImg crashes at launch when installed as an app on macOS

Hello I tried to follow the instructions in the README.rst to install labelImg as an app on macOS but I have the following error message "labelImg has encountered a fatal error, and will now terminate." when I am trying to launch it once I successfully completed the installation:

I used this manual installation:
brew install python3
pip install pipenv
pipenv --three
pipenv shell
pip install py2app
pip install PyQt5 lxml
make qt5py3
rm -rf build dist
python setup.py py2app -A
mv "dist/labelImg.app" /Applications

Moreover, I had to modify my ~/.bash_profile. to solve a localization problem when using pipenv --three. I added the following lines (source):
export LC_ALL=en_US.UTF-8
export LANG=en_US.UTF-8

I don't know if this could be the problem or not but I am also using Anaconda and therefore my python version is the following: Python 3.6.5 :: Anaconda, Inc.
Thanks a lot for your time.

OS: macOS High Sierra 10.13.15
PyQt version: PyQt5

I was able to get this running:
Execute these:
brew install python3
pip install pipenv
pipenv --three
pipenv shell
pip install py2app
pip install PyQt5 lxml
Unzip the labelImg downloaded from https://github.com/tzutalin/labelImg , and execute the following
cd labelImg-master
brew install qt qt5
brew install libxml2
brew install pyqt5
make qt5py3
rm -rf build dist
python setup.py py2app -A
mv "dist/labelImg.app" /Applications
this should fix it.

I encountered the same error message.
This happens if the python framework is not found when running py2app.
So what I recommend is like the following:

copy libpython3.6m.pylib and libpython3.6m.a to libpython3.6.pylib and libpython3.6.a
-> Those files are located on your anaconda python virtual environment, like User/user/anaconda3/envs/LabelImg/lib/ in my case.
Note, LabelImg is the name of the virtual env. So, it may be different in your case.
run setup again, like "$ python setup.py py2app"
-> You can check if it works by running "open ./dist/labelImg.app/". If it works then, move the app to /Applications folder.

This is how I fix the issue.


libertem-server doesn't load the web GUI anymore

When starting libertem-server I am greeted with an infinite loading screen (Win 11, Chrome). With 0.13.1 it doesn't happen. git bisect yielded this output:
There are only 'skip'ped commits left to test.
The first bad commit could be any of:
3ac7a743bd0812d0ef15b97a990c426b3bff1a4e
eea9a6f7f1771f22dd91a756a837eb70b0e638e4
c475a6cd32ae845cd9c2068120310000e96d139a
e024848fcce1c091a1e9000e34a8b17079d96d5b
710913abf742c75e9617484b37e25f78333d94af
dbfc0feb441e0f00667fae92c3e0cc72d2f4d6d1
212d813d019a31b720ca4c80233e18ba13e397ce
aab7e6df703a85b513c3e7d876610e15a7821f56
010d41a96729c727dd212d000e4f5841031d3a05
5d4d2de2256113b3760fabede1a316716fadaa6b
501fe673ddc021dbc1250be782141295c38d6ee1
86acfcac304b8e6bf7ca9816a662b72d0a26a991
cf8eaf01bd361b1cbcab4ce5b57fb194f607042d
We cannot bisect more!

I will look at this this morning. Windows 10 at work but hopefully enough to track down.


[Bug] Using Context with Suspense Throws a Warning

Hiya 👋,
When using the use_context hook in a full-stack setup, I occasionally encounter a warning indicating that the hook is being called twice. It seems this happens because Suspense renders the component twice.
The warning doesn’t appear consistently, but it does pop up often enough to be concerning.
Opening this bug report to track the issue.
Best

Calling use_context multiple times when running a component shouldn't cause an issue. Can you provide more details about the issue from the template including:

The version of dioxus you are using
The panic message
The steps to reproduce the issue along with the code

Calling use_context multiple times when running a component shouldn't cause an issue. Components are supposed to rerun when they need to rerender. Can you provide more details about the issue from the template including:

The version of dioxus you are using
The panic message
The steps to reproduce the issue along with the code

Oops, my bad for not including the steps to reproduce (*/ω＼) ! It was 3 a.m., and I was deep in the Dioxus rabbit hole. But, AFAIR, I was using use_context, and while poking around in the browser console, I spotted a warning about this issue on Firefox. Next time I run into it, I’ll be sure to update this issue (>ᴗ•)!


converted specs to rspec 3 style

Replace should with expect in all specs

:+1:

Thanks for the PR, but I'm going to keep #746 open instead since it's more recent, therefore more likely to be accurate against the current master.


[FEATURE] Scroll to Top button doesn't match theme

Detailed description
The current scroll to top doesn't match the theme of the website. I can modify the css of the button to match the theme of the scroll bar.
Context
It will enhance the look of the the site and enhance the UI as well.
Possible implementation
I will modify the css of the button to match the scroll bar.
Additional information
No response
Checklist

[X] I have read the Contributing Guidelines
[X] I have checked the existing issues
[X] I am willing to work on this issue (optional)
[X] I am a GSSoC'23 contributor

Hey @Anmol-Baranwal I want to work on this issue.... please assign me this

@Riyazul555
What are you planning to change? Please describe the changes you would make.

@Anmol-Baranwal I would like add hover effect on the top scroll button to look more illustrative and make its css match the theme

@Riyazul555
Could you please do something like this but in dark mode, except for the logo part, which should have a slight shadow that suits the dark theme of this codebase.

Image

If you are unable to make something like this, stick to your original idea. Others will review the changes.


Create  Untitled_app.md

📝 Description
Add a brief description.
💣 Is this a breaking change (Yes/No):

[ ] Yes
[x] No

🚧 How to migrate?
Don't need.
📝 Additional Information
lol

Dear user,
Thanks for your contribution. We found that your app is too simple and the PR cannot be accepted. you can

Create an application that can implement simple functions
Create a beautiful layout

Or if you think creating an App is complicated, you can consider creating an AI Agent. Looking forward to your updates.


I have some problems when Creating a ImageProcessor

I want to create a ImageProcessor for add some drop shadows to the image.
So first when I put the sample code of "Creating your own processor" at Cheat Sheet in to my code:
`struct WebpProcessor: ImageProcessor {
// `identifier` should be the same for processors with the same properties/functionality
// It will be used when storing and retrieving the image to/from cache.
let identifier = "com.yourdomain.webpprocessor"

// Convert input data/image to target image and return it.
func process(item: ImageProcessItem, options: KingfisherParsedOptionsInfo) -> Image? {
    switch item {
    case .image(let image):
        print("already an image")
        return image
    case .data(let data):
        return WebpFramework.createImage(from: webpData)
    }
}
}

I got the error said

Type 'WebpProcessor' does not conform to protocol 'ImageProcessor'

But the process(item:options:) function is alway there, and if click the fix button of the 'Do you want to add protocol stubs?' and it add another process(item:options:) function to the Struct.
Anybody knows the problem?

I tried the sample code and everything goes fine.
The wiki is for the latest version (Kingfisher 5), are you using a previous version? The signature was changed a bit for this method, so it is possible that the sample is not working in a Kingfisher 4 integration.


Aqara QBKG38LM can't connect

Hi all,
trying to connect Aqara QBKG38LM switch but after joining zigbee network homebridge-zigbee-nt start to restarting homebridge continously. Here is log:
[17/10/2024, 01:11:10] [ZigBee] Cluster with name 'aqaraOpple' does not exist
[Thu Oct 17 2024] [ZigBee-UI] Request: http://homebridge:9000/coordinator
[Thu Oct 17 2024] [ZigBee-UI] Request: http://homebridge:9000/index.a6fec750.js
[Thu Oct 17 2024] [ZigBee-UI] Request: http://homebridge:9000/api/coordinator
[Thu Oct 17 2024] [ZigBee-UI] Request: http://homebridge:9000/api/devices
[Thu Oct 17 2024] [ZigBee-UI] Request: http://homebridge:9000/api/devices/0x54ef441000d68c89
[Thu Oct 17 2024] [ZigBee-UI] Request: http://homebridge:9000/api/devices/0x54ef441000d68c89/checkForUpdates
zh:controller:endpoint: CommandResponse 0x54ef441000d68c89/1 genOta.imageNotify({"payloadType":0,"queryJitter":100}, {"timeout":10000,"disableResponse":false,"disableRecovery":false,"disableDefaultResponse":true,"direction":1,"srcEndpoint":null,"reservedBits":0,"manufacturerCode":null,"transactionSequenceNumber":null,"writeUndiv":false,"sendWhen":"immediate"})
zh:zstack: sendZclFrameToEndpointInternal 0x54ef441000d68c89:31756/1 (0,0,1)
zh:zstack:znp: SREQ: --> AF - dataRequest - {"dstaddr":31756,"destendpoint":1,"srcendpoint":1,"clusterid":25,"transid":2,"options":0,"radius":30,"len":5,"data":{"type":"Buffer","data":[25,3,0,0,100]}}
zh:zstack:unpi:writer: --> frame [254,15,36,1,12,124,1,1,25,0,2,0,30,5,25,3,0,0,100,36]
zh:zstack:unpi:parser: <-- [254,1,100,1,0,100]
zh:zstack:unpi:parser: --- parseNext [254,1,100,1,0,100]
zh:zstack:unpi:parser: --> parsed 1 - 3 - 4 - 1 - [0] - 100
zh:zstack:znp: SRSP: <-- AF - dataRequest - {"status":0}
zh:zstack:unpi:parser: --- parseNext []
zh:zstack:unpi:parser: <-- [254,3,68,128,0,1,2,196]
zh:zstack:unpi:parser: --- parseNext [254,3,68,128,0,1,2,196]
zh:zstack:unpi:parser: --> parsed 3 - 2 - 4 - 128 - [0,1,2] - 196
zh:zstack:znp: AREQ: <-- AF - dataConfirm - {"status":0,"endpoint":1,"transid":2}
zh:zstack:unpi:parser: --- parseNext []
zh:zstack:unpi:parser: <-- [254,32,68,129,0,0,25,0,12,124,1,1,0,68,0,250,46,107,0,0,12,1,174,1,0,95,17,8,26,23,0,0,0,12,124,29,243]
zh:zstack:unpi:parser: --- parseNext [254,32,68,129,0,0,25,0,12,124,1,1,0,68,0,250,46,107,0,0,12,1,174,1,0,95,17,8,26,23,0,0,0,12,124,29,243]
zh:zstack:unpi:parser: --> parsed 32 - 2 - 4 - 129 - [0,0,25,0,12,124,1,1,0,68,0,250,46,107,0,0,12,1,174,1,0,95,17,8,26,23,0,0,0,12,124,29] - 243
zh:zstack:znp: AREQ: <-- AF - incomingMsg - {"groupid":0,"clusterid":25,"srcaddr":31756,"srcendpoint":1,"dstendpoint":1,"wasbroadcast":0,"linkquality":68,"securityuse":0,"timestamp":7024378,"transseqnumber":0,"len":12,"data":{"type":"Buffer","data":[1,174,1,0,95,17,8,26,23,0,0,0]}}
zh:controller: Received payload: clusterID=25, address=31756, groupID=0, endpoint=1, destinationEndpoint=1, wasBroadcast=false, linkQuality=68, frame={"header":{"frameControl":{"frameType":1,"manufacturerSpecific":false,"direction":0,"disableDefaultResponse":false,"reservedBits":0},"manufacturerCode":null,"transactionSequenceNumber":174,"commandIdentifier":1},"payload":{"fieldControl":0,"manufacturerCode":4447,"imageType":6664,"fileVersion":23},"command":{"ID":1,"response":2,"parameters":[{"name":"fieldControl","type":32},{"name":"manufacturerCode","type":33},{"name":"imageType","type":33},{"name":"fileVersion","type":35}],"name":"queryNextImageRequest"}}
zh:zstack:unpi:parser: --- parseNext []
zh:controller:database: Writing database to '/var/lib/homebridge/zigBee.db'
Error: Cluster with name 'aqaraOpple' does not exist
    at getClusterDefinition (/var/lib/homebridge/node_modules/homebridge-zigbee-nt/node_modules/zigbee-herdsman/src/zspec/zcl/utils.ts:157:19)
    at Object.getCluster (/var/lib/homebridge/node_modules/homebridge-zigbee-nt/node_modules/zigbee-herdsman/src/zspec/zcl/utils.ts:277:29)
    at Endpoint.getCluster (/var/lib/homebridge/node_modules/homebridge-zigbee-nt/node_modules/zigbee-herdsman/src/controller/model/endpoint.ts:804:26)
    at Endpoint.write (/var/lib/homebridge/node_modules/homebridge-zigbee-nt/node_modules/zigbee-herdsman/src/controller/model/endpoint.ts:372:30)
    at Object.configure (/var/lib/homebridge/node_modules/homebridge-zigbee-nt/node_modules/zigbee-herdsman-converters/src/devices/xiaomi.ts:3106:41)
    at ConfigurableAccessory.<anonymous> (/var/lib/homebridge/node_modules/homebridge-zigbee-nt/src/accessories/zig-bee-accessory.ts:175:28)
    at Generator.next (<anonymous>)
    at /var/lib/homebridge/node_modules/homebridge-zigbee-nt/dist/accessories/zig-bee-accessory.js:8:71
    at new Promise (<anonymous>)
    at __awaiter (/var/lib/homebridge/node_modules/homebridge-zigbee-nt/dist/accessories/zig-bee-accessory.js:4:12)
[17/10/2024, 01:11:44] [ZigBee] Child bridge process ended

Using zstack CC2351
Any Ideas?
Thanks and best regards

Here is joining log:
Initializing ZigBee device: 0x54ef441000d68c89 - lumi.switch.b2lc04 - LUMI
[16/10/2024, 23:43:39] [ZigBee] Registering new accessory with uuid c126f6d3-922a-4253-83c8-fa1dfeec3d0d and name 0x54ef441000d68c89
[16/10/2024, 23:43:39] [ZigBee] Registered device: 0x54ef441000d68c89 LUMI lumi.switch.b2lc04
[16/10/2024, 23:43:39] [ZigBee] Mounting device 0x54ef441000d68c89...
[16/10/2024, 23:43:39] [ZigBee] Configuring device 0x54ef441000d68c89
zh:controller:database: Writing database to '/var/lib/homebridge/zigBee.db'
zh:zstack:unpi:parser: <-- [254,34,68,129,0,0,6,0,3,168,2,1,0,23,0,231,116,110,0,0,14,24,146,10,0,0,16,0,245,0,35,0,255,255,2,3,168,29,95]
zh:zstack:unpi:parser: --- parseNext [254,34,68,129,0,0,6,0,3,168,2,1,0,23,0,231,116,110,0,0,14,24,146,10,0,0,16,0,245,0,35,0,255,255,2,3,168,29,95]
zh:zstack:unpi:parser: --> parsed 34 - 2 - 4 - 129 - [0,0,6,0,3,168,2,1,0,23,0,231,116,110,0,0,14,24,146,10,0,0,16,0,245,0,35,0,255,255,2,3,168,29] - 95
zh:zstack:znp: AREQ: <-- AF - incomingMsg - {"groupid":0,"clusterid":6,"srcaddr":43011,"srcendpoint":2,"dstendpoint":1,"wasbroadcast":0,"linkquality":23,"securityuse":0,"timestamp":7238887,"transseqnumber":0,"len":14,"data":{"type":"Buffer","data":[24,146,10,0,0,16,0,245,0,35,0,255,255,2]}}
zh:controller: Received payload: clusterID=6, address=43011, groupID=0, endpoint=2, destinationEndpoint=1, wasBroadcast=false, linkQuality=23, frame={"header":{"frameControl":{"frameType":0,"manufacturerSpecific":false,"direction":1,"disableDefaultResponse":true,"reservedBits":0},"manufacturerCode":null,"transactionSequenceNumber":146,"commandIdentifier":10},"payload":[{"attrId":0,"dataType":16,"attrData":0},{"attrId":245,"dataType":35,"attrData":50331392}],"command":{"ID":10,"name":"report","parameters":[{"name":"attrId","type":33},{"name":"dataType","type":32},{"name":"attrData","type":1000}]}}
zh:zstack:unpi:parser: --- parseNext []
zh:controller:database: Writing database to '/var/lib/homebridge/zigBee.db'
Error: Cluster with name 'aqaraOpple' does not exist
    at getClusterDefinition (/var/lib/homebridge/node_modules/homebridge-zigbee-nt/node_modules/zigbee-herdsman/src/zspec/zcl/utils.ts:157:19)
    at Object.getCluster (/var/lib/homebridge/node_modules/homebridge-zigbee-nt/node_modules/zigbee-herdsman/src/zspec/zcl/utils.ts:277:29)
    at Endpoint.getCluster (/var/lib/homebridge/node_modules/homebridge-zigbee-nt/node_modules/zigbee-herdsman/src/controller/model/endpoint.ts:804:26)
    at Endpoint.write (/var/lib/homebridge/node_modules/homebridge-zigbee-nt/node_modules/zigbee-herdsman/src/controller/model/endpoint.ts:372:30)
    at Object.configure (/var/lib/homebridge/node_modules/homebridge-zigbee-nt/node_modules/zigbee-herdsman-converters/src/devices/xiaomi.ts:3091:41)
    at ConfigurableAccessory.<anonymous> (/var/lib/homebridge/node_modules/homebridge-zigbee-nt/src/accessories/zig-bee-accessory.ts:175:28)
    at Generator.next (<anonymous>)
    at /var/lib/homebridge/node_modules/homebridge-zigbee-nt/dist/accessories/zig-bee-accessory.js:8:71
    at new Promise (<anonymous>)
    at __awaiter (/var/lib/homebridge/node_modules/homebridge-zigbee-nt/dist/accessories/zig-bee-accessory.js:4:12)


fix: add module ty and typealias to symbol

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[x] N
[ ] Y

2. What is the scope of this PR (e.g. component or file name):

/kcl/sema
3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[x] Other
fix: add module ty and typealias to symbol

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[x] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[ ] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[x] Other

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2293

Relevant Lines:
2612

💛  - Coveralls


Smart story names (path, filename, describes, test title)

Issue: #AP-4015
What Changed

For the story names, we now use more than just the test title. Previously, we've been only using the Playwright/Cypress test title (a Playwright test with test('some-title') would have the story name some-title), which resulted in no hierarchy and possible overwriting since tests in different files with the same title would collide into one story name.
Now, we use the test title, then any containing describe()s as folders, then the filename as a folder (truncating the .spec.ts stuff), then any directories between the filename and the tests directory as folders.
I'm trying to keep as similar to possible as how CSF 3 does auto-titling, so I'm not replacing _ or - with spaces, or capitalizing things, or doing any modifications besides stripping off everything after the dot in the file name. I'm not doing the redundancy-removing that CSF does, however (reducing MyComponent/MyComponent.stories.ts to MyComponent), since I bet it's unlikely that Playwright or Cypress users would folder their individual test files in folders of the same name. But I could be assuming too much there.
Example
Given the following Playwright test:
// tests/ is root tests directory
// this file path is tests/some-dir/file.spec.ts

test.describe('a containing describe', () => {
  test('test title', () => {
    // actual test stuff
  })
})

the resulting story name would be some-dir / file / a containing describe / test title.
Note
This is a breaking change for folks, as the first build they run after installing this version of the package will result in all-new snapshots (since all the story names will have changed).
How to test

Check out the chromatic-e2e repo
In the Playwright and tests, add some test.describe() blocks around some test()s (describe()s around it()s for Cypress)
In a test title, add a period to the beginning of the title (like test('.someMethod() should behave'))
Run yarn build && yarn test:playwright && yarn test:cypress
Run yarn archive-storybook:playwright and yarn archive-storybook:cypress
Verify that the story names include the describe titles as folder names
Verify that the tests' filenames are folders
Verify that the test title with the period did not get truncated
Create a folder at the tests level
Move a tests file into this folder
Build, test, and archive-storybook again
Verify that this folder is included as a folder in Storybook for this test

[ ] Author QA
[ ] Reviewer QA

@vanessayuenn if you can have someone from the Storybook team look over how we create the titles (code and requirements -- see the Example section in the PR description), that'd be great. I want to make sure we doing this as similar to how CSF 3 auto-titles as possible.

Storybook 8 actually does this a bit differently, only stripping off .stories.ts. So in SB 7, foo.bar.stories.ts would become foo, but in SB8 it's foo.bar. See https://github.com/storybookjs/storybook/blob/next/MIGRATION.md#autotitle-breaking-fixes

@JReinhold thanks so much for this feedback! I realized I can just strip off any .spec.ts|js (Playwright) and .cy.ts|js (Cypress) and thus accomplish the same thing, without truncating directories or test titles.
There's a slim possibility I'd miss some of the test files if someone can have a custom file extension for test files (not sure the test frameworks would allow that, though), but it's definitely better to err on the side of not truncating enough, since we can always add more rules later.


RawFile LocalPV Project Discussion

RawFile LocalPV is a similar but more flexible, yet more complex.. derivation of the LocalPV-Hostpath Data-Engine.

RawFile is officially classified by the CNCF project rules as a sub-project within the OpenEBS project.
RawFile was conceived, designed and is maintained by a community that operates independently and in concert with the OpenEBS community.
OpenEBS and RawFile co-exist in a symbiotic project/product engineering relationship for the benefit of both projects.
RawFile is very complimentary to OpenEBS and is designed to integrate tightly with OpenEBS. RawFile provides additional value to OpenEBS, to the community, to OpenEBS users and to CNCF.
The current maintainers (at the time that we began the CNCF assisted project restructuring initiative (started on 2 Feb 2024) are:
Mehran Kholdi
Kiran Mova

@avishnu had a meeting with Mehran where he acknowledges the following:
Project needs to be more active w.r.t. contributions, he's willing to commit more time to the project.
Project needs a roadmap and he has ideas that he'd like to share with us CI needs to be stabilized

Maintainers meeting discussion and minutes:
https://github.com/openebs/community/discussions/55

We'll be tracking the rawfile-localpv readme and roadmap updates with this issue.

@semekh any update on the repo? I still see no changes there.

FTWA, what are your thoughts on moving or merging the rawfile-localpv capabilities into a unified local hostpath csi-driver, which would allow you to use it either as the current hostpath or the rawfile (block device via loopback)?
This way we could focus on managing a single repository, and using the same code language.


What value do I measure with "L1" compared to "L1-N" and "L1-L2", according to the protocol specifications?  What value do I get via "L1"?

Ciao,
It is not clear to me, looking at the specification of the protocol itself, what value is measured with the variable/value "L1"?
Shouldn't "L1-N" be used to measure the phase voltage, if the current is measured it would be clear to me, so I don't know what is measured across the phase with the value of L1 and what measured value we get only with L1, except for the marking of the first phase of a three-phase system? If there is no potential difference on L1 itself, what does the result of that value represent, in relation to voltage, current, power?
The question was about this function below:
addMeterValueInput({return (int32_t)33;}, "Energy.Active.Import.Register", "kW", "Body", "L1", 1);
I noticed this and by looking through earlier queries I was unable to find out #110, #185.
I would like to know, so if you can help me then know how to better define the values related to phase AC measurements, for better diagnostic settings?

Hi @ZvonimirRaic,
Interesting question, didn't notice this before.
Instead of "L1", you can insert any value of the following table (OCPP 1.6 ed. 2, page 89):

I never had the case where the energy was reported per phase and I don't know if OCPP backend systems process that information in a meaningful way. If you rather want to report the overall energy register (i.e. consumption over all phases), you can omit the phase field by passing nullptr.


samples for tests

Hi
Having not found success in samples captured from pluto sdr ive found some samples here from a similar project, its captured with the same sdr as this project https://github.com/RUB-SysSec/DroneSecurity/tree/public_squash/samples
so wondering if anyone tried this before.? thanks BR

You cannot use the PlutoSDR for capturing DJI DroneID signals as the PlutoSDR is not able to capture signals over about ~ 4 MSPS (in this case equivalent to ~ 4 MHz of spectrum). The only way that the PlutoSDR could demodulate these signals is if the bursts were detected and captured in the FPGA and then sent to the ARM and then over USB to your desktop/laptop. If someone wanted to it wouldn't be horrifically difficult.
Warning: Lots of rambling below. Many thoughts came to mind that I wanted to put out there for others to possibly polish and implement :)
In fact, here's a rough idea of what would be entailed with an FPGA detector that doesn't demodulate:

Instantiate a FIR filter with coefficients that are the conjugate of the first ZC sequence (this is a constant value, so nothing too tricky here)
The next part has two options: normalized or not

If normalized (which is the better option) then the normalization (power estimation mainly) functionality of [1] needs to be replicated in hardware which isn't terrible

Normalization allows a threshold that's between 0 and 1 which is agnostic to the amount of power present in the signal

If not normalized then an absolute threshold needs to be determined which will likely false trigger a fair bit and also miss lower power bursts, but it should work and is a lot easier

Use the filter and possibly normalization logic to find the start of the first ZC sequence and write a state machine to count until all samples in the burst have been received (this is a constant value) and write this out over DMA to the ARM cores

Both the normalized and non-normalized would require a separate buffer to hold onto the time domain samples while the FFT filter is running. The output of the FFT filter would just tell you when to look back by N samples into the past since the FFT takes a long time to process

Once the samples are sent over DMA, the ARM can do demodulation, or the data can be sent over USB to some other x86 system to demod via MATLAB (data rate would be low enough for this to be possible)

There will need to be a software component to this the FPGA portion is writing data out to DMA. There are examples on Xilinx's website/wiki about how to accomplish this and some examples that should work.
The hardest part of this IMO will be integrating into PlutoSDR block design in Vivado and writing the software to pull data from DMA.
The correlation filter might be a bit large for the PlutoSDR's FPGA, but switching off the dual channel and TDD logic in the AD9361 block will save a lot of resources. The correlation will have to be done with an FFT since the number of DSP48's is very limited in the 7010 (no way you're doing a time domain systolic FIR filter with 1024 taps on the Zynq-7010 without time sharing multipliers which gets really complicated).
[1] https://github.com/proto17/dji_droneid/blob/main/matlab/updated_scripts/normalized_xcorr_fast.m


bumped brooklyn version

good point @adriannieto! Updated!

@kiuby88 I'll leave that with you, feel free to upgrade brooklyn version and test yaml files accordingly. Thanks

:+1:


Adjust padding of element an element

Green box padding goes to 10px no matter what  value you set . Challenge is to set padding to match red box  which is 20px

Challenge Name

Issue Description

Browser Information

Browser Name, Version:
Operating System:
Mobile, Desktop, or Tablet:

Your Code

Screenshot

Hi @mattp991
Thanks for reporting this issue.
Here are some next steps that you should follow before we can help you with your query:

Check these troubleshooting steps first.
Please use the Chat Rooms or try looking through our forum for help with a specific challenge and any coding assistance.

IMPORTANT:
Please NOTE that this tracker is for reporting bugs and enhancements to freeCodeCamp's code base only.
Happy Coding!