How to use cosmos client directly along with spring data repository as I could not find few operations like partial updates in spring data cosmos

Query/Question
I am trying to use spring data cosmos in my project. But some operations which I need I could not find in spring data cosmos. For eg., partial updates, do only an insert (not upsert), do only an update (not upsert).
So, I was thinking of using cosmosclient of cosmos sdk directly. Is there a way to use cosmosclient along with spring data cosmos such that both use the same cosmosclient instance?
Why is this not a Bug or a feature Request?
This is not a bug/feature request. I want to use the convenience of spring data for GET queries along with flexibility cosmosclient provides for some other operations.
Setup (please complete the following information if applicable):
Library: (com.azure:azure-spring-data-cosmos:3.30.3)]

Hi, @arunim29
Thanks for reaching out.
CosmosAsyncClient is a bean. You can get the bean same way as you get the XxxRepository.
For example:
@SpringBootApplication
public class SampleApplication implements CommandLineRunner {

    private XxxRepository repository;
    private CosmosAsyncClient client;

    public SampleApplication(XxxRepository repository, CosmosAsyncClient client){
        this.repository= repository;
        this.client= client;
    }
}

@arunim29 what @chenrujun mentioned is absolutely correct. However, I would like to know your use case of Patch API in Spring and how / why do you plan to use this feature. Since we are planning on developing Patch API support in spring-data-cosmos SDK, it would really help us to know how our customers plan to use this feature.
@TheovanKraay  / @trande4884 - please take this discussion forward, thanks!

@arunim29 - looks like you are already involved in the discussion here, but this is the issue that we are tracking for support of patch operations - https://github.com/Azure/azure-sdk-for-java/issues/21433

@arunim29 what @chenrujun mentioned is absolutely correct. However, I would like to know your use case of Patch API in Spring and how / why do you plan to use this feature. Since we are planning on developing Patch API support in spring-data-cosmos SDK, it would really help us to know how our customers plan to use this feature.
@TheovanKraay / @trande4884 - please take this discussion forward, thanks!

@kushagraThapar I was on vacation. Hence, could not reply. So, my usecase is pretty straight-forward. I have a bunch of columns in my document but in one of the heavily used apis of my service, I only need to update one column. Hence, I do not want to first fetch the document and then update it. It will be a waste of resources.

@arunim29 + @chenrujun we plan to expose patch something like the below. Let us know if this is not what you expect. Feel free to make suggestions or ask questions here.
CosmosPatchOperations patchOperations = CosmosPatchOperations
    .create()
    .replace("/size", 5)
    .add("/color", "blue");

User patchedUser = userRepository.save(user,patchOperations,null);

@TheovanKraay what is the user in userRepository.save(user,patchOperations,null) ? In patched operations, we are not expected to pass any existing object. Also, what is this null param?

@TheovanKraay I had a look at ur PR for this feature and understand now that null represents CosmosPatchItemRequestOptions. This is fine.
BUT I have a concern in this method signature. Here in the userRepository.save(user,patchOperations,null) , my assumption is that you are expecting caller to create a User object with just id and pk set right? If that is the case, what if the user has added jackson annotations in this POJO which prevent non-nulls values for some fields?

@TheovanKraay I had a look at ur PR for this feature and understand now that null represents CosmosPatchItemRequestOptions. This is fine.
BUT I have a concern in this method signature. Here in the userRepository.save(user,patchOperations,null) , my assumption is that you are expecting caller to create a User object with just id and pk set right? If that is the case, what if the user has added jackson annotations in this POJO which prevent non-nulls values for some fields?
Will u just get id and pk and ignore rest of the POJO?

Yes that’s correct. Caller may pass the whole object if it’s available, or they can set only id and pk if they prefer. Either way only id and pk will be used from the object passed. Only the values defined in patchOperations are the ones that will actually be patched.

@arunim29 in light of this discussion, we're making changes to avoid the confusion you ran into. Caller will now pass id + partition key of the entity to be patched, along with it's domainType, and the intended patchOperations:
CosmosPatchOperations patchOperations = CosmosPatchOperations
    .create()
    .replace("/size", 5)
    .add("/color", "blue");

User patchedUser = userRepository.save(user.getId(), new PartitionKey(user.getLastName()), User.class, patchOperations);

See PR for patch: https://github.com/Azure/azure-sdk-for-java/pull/32630

This has been implemented, closing it.


Custom splash screen

every time flet loads, we get a loading page of flet icon or a loading wheel spinning.
Is there a way to make a custom splash screen come up in place of the flet logo?

This is exactly the issue i was about to raise!
I checked the docs, but nothing.
Please HELP on this.

you can create an assets folder then replace the elements you want in there.
https://flet.dev/docs/guides/python/packaging-desktop-app#packaging-assets

That’s for desktop apps.
I mean for web apps. (For example, those made with fly.io or replit)

https://github.com/flet-dev/examples/tree/main/python/apps/custom-loading-image
I tried to follow this example. But when I implement this in my project, it doesn't work. I mean, the flet logo still shows up at the web app's launch.

you created assets folder, and inside that icons folder, and in there the file to replace?
Only put in what you want changed

Yes. Exactly.
I named the file just as in the example. (loading-animation.png)
I even tried with the icon image provided in the example's assets/icon folder.
But nothing.

post your code and structure in a repo?

Enable logging and see what's being loaded and from where:
import logging
logging.basicConfig(level=logging.DEBUG)

It worked now.
DEBU[2022-09-20T15:40:15+02:00] EmbedAssetsFS item: icons/loading-animation.png
I just had to restart all from scratch(i was in hotreload mode). Just from noticing while closely looking at the logs that, any file added to my assets/icons directory(while still in HotReload) is never loaded/added. Is this a Bug or something normal @FeodorFitsner ?
So I can say briefly that, Restarting my app from scratch kind of solved the problem. Hope yours too @vv2006-mc !
Thanks y'all.

By the way, does the name of the icon(loading-animation.png) matters.
Will any other image(icon) with a different name get loaded? (what if there are several image files in this iconsdirectory? Any order of priority or something..?)
And also, is it possible to change the Animation at launch screen? By default, the image get bigger and fades away before our app's UI is shown...is it possible to change this Animation? If Yes, how please?

post your code and structure in a repo?

https://github.com/ndonkoHenri/ToDo-App-with-Flet
a personalised version of the example in the docs

yeah hot reload works great but usually needs a reboot if you change something on init / startup. glad you got it fixed i know how those small overlooked things can drive you nuts
nice repo btw!

is there a way to change this loading animation too? @FeodorFitsner

Ah, well, that UI is built before user app's data is loaded. I think it would be possible to customize with a custom Flet app runner which is coming soon™.

custom Flet app runner

Still in your todo?


Install correct pytorch/cudatoolkit versions on sagemaker

This change updates the sagemaker-create script to check the Cuda version and install the appropriate cudatoolkit and pytorch versions using conda.
Before this change, it took about 40 minutes to run the first learn.fit_one_cycle(4) call in lesson 1 of the 2019 deep-learning 1 course on my Sagemaker instance. After this change, it took 1m50s.
Before this change:

After this change:

If you check the installed/active Cuda version using a Terminal, you find:
sh-4.2$ nvcc --version
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2017 NVIDIA Corporation
Built on Fri_Sep__1_21:08:03_CDT_2017
Cuda compilation tools, release 9.0, V9.0.176

One thing I would ask of a reviewer - I wrote a bit of awk script to extract the active Cuda version and ask conda to use that version for cudatoolkit. I'm new to all these tools and I have no idea if that is super-brittle and there might be a better way to find and select the correct version.

I had this same issue and downgrading CUDA version in the Conda environment worked for me like it is described in this PR.

Many thanks! I don't see anything particular problematic about the awk bit, although personally I tend to think whatever awk can do, perl can do better :)


Test Hive Metastore 4.x and see if we can archive my HMS image

          Looks like this fixed the issue.

https://github.com/bitsondatadev/trino-getting-started/issues/38#issuecomment-1925347891

Any reason why this wasn't used instead of custom HMS + psql?

There were some oddities about earlier HMS images in 3.x. If the latest 4.x metastore has those worked out then I would love to get rid of my HMS Docker image.
I'll open a new Issue to remind myself to look at that later.
Originally posted by @bitsondatadev in https://github.com/bitsondatadev/trino-getting-started/issues/38#issuecomment-2025768063

The only recent thing I remember we added quickly to this was the ability to run on an M1 processor, but I'm hopeful this is resolved in the latest.

Hi @bitsondatadev - this works for me in the trino-b2 tutorial:
  hive-metastore:
    image: 'apache/hive:4.0.0'
    hostname: hive-metastore
    ports:
      - '9083:9083' # Metastore Thrift
    volumes:
      - ./conf/core-site.xml:/opt/hadoop/etc/hadoop/core-site.xml:ro
      - ./conf/metastore-site.xml:/opt/apache-hive-metastore-4.0.0-bin/conf/metastore-site.xml:ro
    environment:
      SERVICE_NAME: metastore
      METASTORE_DB_HOSTNAME: mariadb
      HIVE_AUX_JARS_PATH: /opt/hadoop/share/hadoop/tools/lib/hadoop-aws-3.3.6.jar:/opt/hadoop/share/hadoop/tools/lib/aws-java-sdk-bundle-1.12.367.jar
    depends_on:
      - mariadb
    networks:
      - trino-network

There may be more optimal ways to do this, but a few hours of trial and error led me here.

See https://github.com/bitsondatadev/trino-getting-started/pull/50

Fixed with #55


Add pop method for List trait.

Objective

The reflection List trait does not have a pop function.
Popping elements off a list is a common use case and is almost always supported by List-like types.

Solution

Add the pop() method to the List trait and add the appropriate implementations of this function.

Migration Guide

Any custom type that implements the List trait will now need to implement the pop method.

bors r+


Implement missing operator%

We need the modulo operator in xsimd for operations in xtensor.

After thinking about it and looking at other libraries, I think integer division & modulo should either be polyfills (falling back to scalar code) or removed from the library (which should be detected in xtensor, and then fall back).

done in #130


Explanation of where the value for 'code=' is coming from is missing

I have finally figured out that the value of "code" (in query strings) is the value of the "default" "function key" for the function being called.
It is found in the Azure portal by opening the function and selecting the "Function keys" menu item under the "Developer"-section.
Also: please also update the blogpost which states nothing about this.
https://developer.microsoft.com/en-us/graph/blogs/microsoft-graph-mailbag-blazor-web-assembly-and-microsoft-graph-with-azure-functions/
Apart from this, thanks for a great post :-)

Thanks @roblom for the feedback. The bullet point at the top of the post regarding Azure Function spoke to anonymous v/s function code. I made a deliberate attempt as this post dealt with 3 very big topics to point to deeper explanations in existing articles and guidance but as it was in the code as well, i will add comments there for it in my next update. Cheers.


Write a Python program to convert days to minutes

Description
Write a Python program to convert days to minutes
1 day = 24 hours = 1440 minutes
Input  : 5
Output : 7200

How to contribute

Save the solution in convert-days-to-minutes.py file
Add convert-days-to-minutes.py file in convert-days-to-minutes folder

!assign


fixing formatting in container image to be consistent with image

Issue #, if available:
N/A
Description of changes:
This PR aims to keep how containerImage tag is generated/built consistent and insync with how image tag in the cluster service version is generated. This makes readability of both sections of code/templates/sh's/etc the same and easier in the future if a change in the formate is needed.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/lgtm


Error: Failed to call EnumProcessModules

Hello, i cant get TB to work, i deleted all old TB files and re downloaded 2.8, it either doesn't change anything ingame or give me this Error code 10.
went step by step of whitelist and admin rights.

GWToolbox++
Error: Failed to call 'EnumProcessModules' (2)
Error code: 10
Return value: 0
OK

Alas, your suggestion results in #195

Should be fix with the new launcher that can be found here: https://github.com/HasKha/GWToolboxpp/releases/tag/2.14_Release


Make 'pop' the first method that uses 'T' in 'Array'

Chatted with @mhegazy about this, but I'd like to get your thoughts on this @ahejlsberg since you sent out #436.
When elaborating errors on an Array<Foo> to an Array<Bar>, diving into a structural comparison is often dizzying for the user since the first elaboration occurs on push.
Since push uses T contravariantly, we'll end up flipping the direction which is much more difficult for users to parse. Furthermore, in cases where the covariant check would provide better error messages and error message spans (such as excess property checking spans, and picking better branches in unions), we would end up doing significantly worse.
As an example from @cliffkoh and @dzearing, the following
type IStyle = IStyleBase | IStyleArray;
interface IStyleArray extends Array<IStyle> {}
interface IStyleBase {
    foo: string;
}

const blah: IStyle = [
    [[{
        foo: 'asdf',
        jj: 1 // intentional error
    }]]
];

Provides an error on blah:
const blah: IStyle = [
      ~~~~

[ts] Type '{ foo: string; jj: number; }[][][]' is not assignable to type 'IStyle'.
       Type '{ foo: string; jj: number; }[][][]' is not assignable to type 'IStyleArray'.
         Types of property 'push' are incompatible.
           Type '(...items: { foo: string; jj: number; }[][][]) => number' is not assignable to type '(...items: IStyle[]) => number'.
             Types of parameters 'items' and 'items' are incompatible.
               Type 'IStyle' is not assignable to type '{ foo: string; jj: number; }[][]'.
                 Type 'IStyleBase' is not assignable to type '{ foo: string; jj: number; }[][]'.
                   Property 'length' is missing in type 'IStyleBase'.

Whereas with this change, we provide an error on jj (the excess property):
        jj: 1 // intentional error
        ~~

[ts] Type '{ foo: string; jj: number; }[][][]' is not assignable to type 'IStyle'.
       Type '{ foo: string; jj: number; }[][][]' is not assignable to type 'IStyleArray'.
         Types of property 'pop' are incompatible.
           Type '() => { foo: string; jj: number; }[][]' is not assignable to type '() => IStyle'.
             Type '{ foo: string; jj: number; }[][]' is not assignable to type 'IStyle'.
               Type '{ foo: string; jj: number; }[][]' is not assignable to type 'IStyleArray'.
                 Types of property 'pop' are incompatible.
                   Type '() => { foo: string; jj: number; }[]' is not assignable to type '() => IStyle'.
                     Type '{ foo: string; jj: number; }[]' is not assignable to type 'IStyle'.
                       Type '{ foo: string; jj: number; }[]' is not assignable to type 'IStyleArray'.
                         Types of property 'pop' are incompatible.
                           Type '() => { foo: string; jj: number; }' is not assignable to type '() => IStyle'.
                             Type '{ foo: string; jj: number; }' is not assignable to type 'IStyle'.
                               Object literal may only specify known properties, and 'jj' does not exist in type 'IStyle'.

This is horrifyingly longer, but it gives the right span and gets to the root cause at the end.

Really the RWC baseline changes are minimal, and aren't impacted at all. I'm just seeing reports of missing pop instead of push:
  Argument of type 'Uint8ClampedArray' is not assignable to parameter of type 'any[]'.
+   Property 'pop' is missing in type 'Uint8ClampedArray'.
-   Property 'push' is missing in type 'Uint8ClampedArray'.

Can you improve those errors for Arrays even further. I got confused with code like this:

interface Values {
    textValues: {
        values: string[]
    }
}

const items = [{ name: 'a' }, { name: 'b' }]

var values: Values = {
    textValues: items.map(i => i.name)
}

Type 'string[]' is not assignable to type '{ values: string[]; }'.
  Types of property 'values' are incompatible.
    Type '() => IterableIterator<string>' is not assignable to type 'string[]'.
      Property 'pop' is missing in type '() => IterableIterator<string>'


refactor: Added a generic FileStream (still in active development!)

References:

https://filesystem-spec.readthedocs.io/en/latest/api.html#base-classes

📚 Documentation preview 📚: https://meltano-sdk--2654.org.readthedocs.build/en/2654/

These are working now
FTP
{
    "filesystem": "ftp",
    "path": "fixtures/csv",
    "read_mode": "one_stream_per_file",
    "delimiter": "\t",
    "ftp": {
        "host": "<IP_ADDRESS>",
        "port": 21,
        "username": "my_ftp_user",
        "password": "my_ftp_password"
    }
}

SFTP
{
    "filesystem": "sftp",
    "path": "fixtures/csv",
    "read_mode": "one_stream_per_file",
    "delimiter": "\t",
    "sftp": {
        "host": "<IP_ADDRESS>",
        "port": 2022,
        "username": "my_ftp_user",
        "password": "my_ftp_password"
    }
}


no monit for nginx

We should run monit on nginx.  I know we plan to run that in a pod, but right now it crashes somewhat often, so we should monit it to keep it up.

I will be moving nginx etc to its own pod so that we wont need monit..

@ArtfulCoder what is the ETA? Should we implement this in the mean time? If the ETA is soon, please close this issue.

I'm taking this over from @ArtfulCoder ...  This will also include the version update to make websockets work for us.

Thanks Brendan!
You can use the net=host feature for this one.

Closing this as it was fixed by #6334


[AutoDiff][Sema] Correctly process differentiable function type declarations with inout or Void

Sourced from #41174. The previous pull request included some commentary that was off-topic or did not reflect an affinity for constructive criticism. On top of that, the goal changed from classifying Void as non-differentiable to resolving ambiguities between lack of a return type and lack of a differentiable return type. To make the process of reviewing easier, please use this new pull request instead.

Resolves SR-15808 and SR-15818.

This PR permanently closed by accident because I force-pushed without re-opening it.


[BUG] likely leak in device memory

I am seeing that RMM memory is increasing after two tries for one of the TPCDS queries. I expect memory to go back to some known initial quantity. Normally that's ~500MBs, but in this query:
We start at:
20/11/19 21:27:17 WARN GpuShuffleEnv: RMM alloc 536870912. Device store amnt: 0

Then after iteration 1 completes (and we have unregistered shuffle):
20/11/19 21:27:44 WARN GpuShuffleEnv: RMM alloc 912163328. Device store amnt: 0

After iteration 2 completes (also after shuffle is unregistered):
20/11/19 21:28:32 WARN GpuShuffleEnv: RMM alloc<PHONE_NUMBER>. Device store amnt: 0

Finally we OOM after this case. This is with the RAPIDS shuffle. But if a query fit on iteration 1 it should fit in the next iteration, provided we cleaned up everything that needed to be cleaned up.

At this point this has been attributed to accumulating broadcast, and broadcast derived batches (during a join). So this is a known issue somewhat.
The solution is to make the broadcast spillable, since we can't control the lifetime of the broadcast in spark. We also want to figure out how to deal with the projections from this broadcast variable, as we are seeing this data size get fairly large in comparison with the original batch that was broadcasted.

So do we close this as a duplicate of #836 ?

Yes that works for me. The slight difference is that so far the broadcast variable size << than the rest of the memory used when using the broadcast variable to produce inputs into the join. So I'll just carry this to the other issue.


Add retry logic to mass storage test

Under certain conditions a PC running DAPLink tests can clear the disk cache DAPLink is mounted on. When this occurs the host OS will re-read disk contents. Since DAPLink's disk contents never change the
drive will look like it did when it first mounted. None of the new files that were created will exist. If this cache clear occurs in the middle of a write operation in the mass storage test an IOError will be raised, causing testing to terminate abruptly.
To prevent testing from spuriously failing this patch adds retry logic to the mass storage test. This prevents these spurious failures.
Note - a disk cache clear can be triggered by allocating most of the systems memory in a program. For example on a windows 10 PC with 16GB of RAM running the following code (which allocates 15GB) in a python terminal will typically trigger a cache clear of all disks:
"a = bytearray(1024 * 1024 * 1024 * 15)"

@mbed-bot: TEST


Added line_break tag to insert \n into liquid interpolated string

Supersedes #707
Using a custom tag to insert line breaks seems like a much cleaner approach to me, no clue why it did not work yesterday.
/cc @drbig @virtadpt

Looks good!

Looks good to me too.
I wonder now that custom tags could be also user-definable, e.g. I'd do a filter so that all my IM-sinked messages get the same format with simple message: {{ message | format_im_message }} and basically remove my whole 'edit dozen agents to keep the format consistent' 'problem'. Which for IM messages is just being pedantic, but imagine the message is consumed by some webhook, then it matters technically.

@drbig That would be a bit tricky because we need to evaluate the ruby code the user enters. It might be possible to secure the evaluation with some kind of ruby sandbox or running the code in a docker container.


Local_balance amount decreasing fast in open channels

Background
Maybe this is silly I don't know, but I think there is something wrong with the commit_fee of 2 open channels of mine.
The amount of commit_fee feels huge, and it seems it is depleting my local_balance which is rapidly decreasing.
It might not be a bug, maybe I did something wrong, but I can't figure it out.
Your environment

lnd version 0.4.2-beta commit=b5a228808b0446d6758a23e24d6c686aab9936fc
Linux Celeri 4.15.0-23-generic #25-Ubuntu SMP Wed May 23 18:02:16 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
"version": 160000, "subversion": "/Satoshi:0.16.0/", "protocolversion": 70015,

Steps to reproduce
I just happened to have 2 open channels with nodes :

02a456b816697c6b8407ee2176475d74b790a96143bc30e896e8bf7a8852a8b637
02ad6fb8d693dc1e4569bcedefadf5f72a931ae027dc0f0c544b34c1c6f3b9a02b

Expected behaviour
As far as I know, the local_balance of a channel should be pretty stable if you don't use it, minus variation of commit_fee maybe ?
Actual behaviour
The available amount to spend is falling very fast :

02a456b816697c6b8407ee2176475d74b790a96143bc30e896e8bf7a8852a8b637 : I funded 20000 sat (minus 153 sat fees) on the 16th of this month, made 1 payment of 264 fees, today local balance is only 3445
02ad6fb8d693dc1e4569bcedefadf5f72a931ae027dc0f0c544b34c1c6f3b9a02b : I funded 79647 sat (minus 200 sat fees) on the 19th of this month, made no payment, today local balance is only 49963

Here are the listchannels output :

{

"channels": [
    {
        "active": true,
        "remote_pubkey": "02a456b816697c6b8407ee2176475d74b790a96143bc30e896e8bf7a8852a8b637",
        "channel_point": "f1d4726b33944bbcfbe0e9e3b18461675a1e40db21d3acce776ee364269a9bbf:0",
        "chan_id": "580769738400137216",
        "capacity": "79647",
        "local_balance": "49963",
        "remote_balance": "0",
        "commit_fee": "29684",
        "commit_weight": "600",
        "fee_per_kw": "41000",
        "unsettled_balance": "0",
        "total_satoshis_sent": "0",
        "total_satoshis_received": "0",
        "num_updates": "16",
        "pending_htlcs": [
        ],
        "csv_delay": 144,
        "private": false
    },
    {
        "active": true,
        "remote_pubkey": "02ad6fb8d693dc1e4569bcedefadf5f72a931ae027dc0f0c544b34c1c6f3b9a02b",
        "channel_point": "ac8f2f0f77698a71e4ee35de89cedcf3fa64c6ae96685dd078ea084cc1eb1fd6:0",
        "chan_id": "580280455773945856",
        "capacity": "20000",
        "local_balance": "3445",
        "remote_balance": "264",
        "commit_fee": "16555",
        "commit_weight": "600",
        "fee_per_kw": "22500",
        "unsettled_balance": "0",
        "total_satoshis_sent": "264",
        "total_satoshis_received": "0",
        "num_updates": "23",
        "pending_htlcs": [
        ],
        "csv_delay": 144,
        "private": false
    }
]

}

This is just the commitment fee being automatically adjusted. If you're on testnet, then fee estimation is a bit wonky, so you might have high-ish fees on your commitment transaction. For mainnet, the fee situation is a lot more realistic.

Ok thanks, I just wanted to be sure I did nothing wrong. Was on mainnet btw.

What exactly is the commitment fee? Can this be explained in more detail and can you adjust it manually?

@jrwagh333 It is just the fee used for the pre-signed commitment transactions. Cannot be set manually at this point.


[RFC] logging in headers + LOG_MODULE_REGISTER/DECLARE refactoring

Module handling macros has been refactored to take log level as a parameter. Under the hood static inline functions for getting local log level are created, same as in case of addresses of module structures. Solution is cleaner now:

registration LOG_MODULE_REGISTER(foo, CONFIG_FOO_LOG_LEVEL)
referencing LOG_MODULE_DECLARE(foo, CONFIG_FOO_LOG_LEVEL)

Second commit extends logger to support logging in headers in static inline functions. Solution suggested in the comment in #10184 was used which is less complicated than actual #10184 proposal.
Tricky (yet required) part is that module must be declared in every static inline function which is using the logger. It should not impact the code size since macros creates same static inline functions for fetching addresses and log level.
Example of logging in static inline function:
static inline void foo(void)
{
   LOG_MODULE_DECLARE_IN_FUNC(foo, CONFIG_FOO_LOG_LEVEL)
   LOG_INF("foo");
}

Codecov Report

Merging #10215 into master will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #10215      +/-   ##
==========================================
+ Coverage   53.03%   53.07%   +0.03%
==========================================
  Files         214      214
  Lines       26176    26176
  Branches     5642     5642
==========================================
+ Hits        13883    13892       +9
+ Misses      10024    10016       -8
+ Partials     2269     2268       -1

Impacted Files
Coverage Δ

include/logging/log_core.h
100% <ø> (ø)
:arrow_up:

subsys/logging/log_core.c
73.44% <0%> (+3.38%)
:arrow_up:

include/logging/log_backend.h
75% <0%> (+9.37%)
:arrow_up:

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 b4d8563...5bee084. Read the comment docs.

@mbolivar regarding GCC-ism. I must agree with you. Check that clang does not support it so i had to drop that. i have changed to use variables instead of functions and that seems to work and be cleaner.

Note that tests will be failing when it comes to complex macros stuff noted by checkpatch. Those cannot be fixed.

@nashif @carlescufi we need to get that in to allow logging in headers. For now things will compile like here:
https://github.com/nashif/zephyr/blob/6c2ac92e1838c006727d8c07707fa1a30153c59f/arch/arm/core/cortex_m/mpu/arm_mpu_v7_internal.h#L34
but it will not work.
There is shippable checkpatch error that cannot be fixed (macro).
@pizi-nordic would you like to take another look? I know that you initially accepted when it was RFC-labeled.

Should you enclose this in a do{}while(0); ?
-:152: ERROR:MULTISTATEMENT_MACRO_USE_DO_WHILE: Macros with multiple statements should be enclosed in a do - while loop
#152: FILE: include/logging/log.h:304:
+#define _LOG_MODULE_DATA_CREATE(_name, _level)			\
+	_LOG_MODULE_CONST_DATA_CREATE(_name, _level);		\
+	_LOG_MODULE_DYNAMIC_DATA_COND_CREATE(_name)

@carlescufi it is not used in functions but outside so i cannot use do-while.

@carlescufi it is not used in functions but outside so i cannot use do-while.

Right, I see. This is OK by me. Waiting for @nashif to see if we merge.

@jukkar i'm trying to make shippable green with change to have LOG_REGISTER_MODULE(name, level) and I am having a lot of troubles with the fact that net_core.h is actually calling LOG_REGISTER_MODULE assuming that modules will define MODULE NAME and LOG_LEVEL. Because of that there are workarounds in many files (e.g ethernet drivers) which needs to define NET_LOG_ENABLED 0 to supress that LOG_MODULE_REGISTER in net_core.h. It's a hack. Do you think net could be refactored to call LOG_MODULE_REGISTER() in C files?

I'm closing this one. Reworking whole repo was too much for that PR (already 175 files edited). I made #11035 where macro should be backward compatible .


FEAT: collection insert methods return vector id

Use case
When inserting a vector record to a collection, users need a way to get the newly inserted record vector ID. When users need to update or delete the vector records, this returned vector ID can be used as the input parameters to the methods.
Proposed solution
Modify the return type and value of the Collection.insert method.
// Before
-> Result<(), Error>

// After
-> Result<VectorID, Error>

I'd like take this

Sounds good! Looking forward to seeing another PR 😁

pr: #60

Thank you for the PR! I have reviewed and merged them 😁


LibWeb: Fix handling of find-in-page with pseudo-element content

This change makes find-in-page ignore content that’s been added to the document using CSS ::after or ::before pseudo-elements. Ignoring such pseudo-element content for find-in-page matches the behavior in Chrome and Safari (though not in Firefox).
Otherwise, without this change, find-in-page doesn’t ignore the pseudo-element content, and we instead crash in DOM::Range::common_ancestor_container after hitting an assert, due to the start container and end container for the matched range not having a common ancestor.
Note: I’ve not found any spec that normatively defines the required behavior for find-in-page with pseudo-element content. But since there are two existing implementations which ignore it — versus one which doesn’t — it seems preferable to match the “prevailing” behavior.
Fixes https://github.com/LadybirdBrowser/ladybird/issues/514

Marked as draft because I haven’t yet figured out how — for the case where there’s a match both for the pseudo-element content and the actual DOM content — to get the browser find-in-page UI to the show the right match count (and to cycle back through the matches as expected).

Thanks for looking into this!
I tested your change on the site where I first encountered the issue and it's not quite working as expected.
Site: https://en.wikipedia.org/wiki/2023_in_American_television
Search text: 2023
Video:
https://github.com/LadybirdBrowser/ladybird/assets/2817754/e96b8ab7-d8ac-4b62-84fb-179cdf61dc65
The "2023" text in the table on the right hand side is skipped (the bold text). "2023" is not part of the pseudoelement content in this case.
It seems reasonable to me to not include pseudoelement content in find-in-page results given the behavior of other engines. I think there's a more direct way to achieve this, which should also solve the problem shown above.
We should be able to exclude pseudoelement text at the point we construct the text blocks that Document::find_matching_text() uses to identify all the matching ranges.
Changing this line:
https://github.com/LadybirdBrowser/ladybird/blob/d890be6e0f7db08ab39ba546cb3421b50b687cda/Userland/Libraries/LibWeb/Layout/Viewport.cpp#L60
to if (layout_node.is_box() || layout_node.is_generated()) will cause a new text block to be created if the current layout node is a pseudoelement. It will also cause any text within this node to be ignored.

I tested your change on the site where I first encountered the issue and it's not quite working as expected. Site: en.wikipedia.org/wiki/2023_in_American_television Search text: 2023
…
The "2023" text in the table on the right hand side is skipped (the bold text). "2023" is not part of the pseudoelement content in this case.

I see, yep — tested with the build from this PR branch, and I see that’s getting skipped. (Though I’ll confess I don’t understand the code enough yet to understand why…)

It seems reasonable to me to not include pseudoelement content in find-in-page results given the behavior of other engines. I think there's a more direct way to achieve this, which should also solve the problem shown above.
We should be able to exclude pseudoelement text at the point we construct the text blocks that Document::find_matching_text() uses to identify all the matching ranges.

Aha — yeah, had wondered about that but didn’t poke further, after I (mistakenly) thought my initial change had fixed it

Changing this line:
https://github.com/LadybirdBrowser/ladybird/blob/d890be6e0f7db08ab39ba546cb3421b50b687cda/Userland/Libraries/LibWeb/Layout/Viewport.cpp#L60
to if (layout_node.is_box() || layout_node.is_generated()) will cause a new text block to be created if the current layout node is a pseudoelement. It will also cause any text within this node to be ignored.

✅ I walked back through the stack of the code that calls into that, and I see:
https://github.com/LadybirdBrowser/ladybird/blob/d890be6e0f7db08ab39ba546cb3421b50b687cda/Userland/Libraries/LibWeb/DOM/Document.cpp#L5120
…in DOM::Document::find_matching_text (some lines up from where I’d made my initial change) so I see now how this works — thanks much
I’ve updated the patch in this PR with that change


Fix bugs with LoadModule & add sqlite3 support

Simple fixes. If you have any questions let me know.

Thanks!


Yarn install error when building docker image

I build the base image:
$ docker build -t beakerx-base:latest -f docker/base/Dockerfile .

then attempt to build the beakerx image:
$ docker build -t beakerx:latest -f docker/Dockerfile .

and I get the following stacktrace:
Installing collected packages: beakerx
  Running setup.py develop for beakerx
    Running command /opt/conda/envs/beakerx/bin/python -c "import setuptools, tokenize;__file__='/home/beakerx/beakerx/setup.py';f=getattr(tokenize, 'open', open)(__file__);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, __file__, 'exec'))" develop --no-deps
    running develop
    running js
    Installing build dependencies with yarn.  This may take a while...
    > yarn install
    yarn install v1.7.0
    [1/4] Resolving packages...
    [2/4] Fetching packages...
    error An unexpected error occurred: "https://registry.yarnpkg.com/comment-json/-/comment-json-1.1.3.tgz: unable to get local issuer certificate".
    info If you think this is a bug, please open a bug report with the information provided in "/home/beakerx/js/notebook/yarn-error.log".
    info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/home/beakerx/beakerx/setup.py", line 107, in <module>
        setup(**setup_args)
      File "/opt/conda/envs/beakerx/lib/python3.6/site-packages/setuptools/__init__.py", line 131, in setup
        return distutils.core.setup(**attrs)
      File "/opt/conda/envs/beakerx/lib/python3.6/distutils/core.py", line 148, in setup
        dist.run_commands()
      File "/opt/conda/envs/beakerx/lib/python3.6/distutils/dist.py", line 955, in run_commands
        self.run_command(cmd)
      File "/opt/conda/envs/beakerx/lib/python3.6/distutils/dist.py", line 974, in run_command
        cmd_obj.run()
      File "/home/beakerx/beakerx/setupbase.py", line 404, in run
        [self.run_command(cmd) for cmd in cmds]
      File "/home/beakerx/beakerx/setupbase.py", line 404, in <listcomp>
        [self.run_command(cmd) for cmd in cmds]
      File "/opt/conda/envs/beakerx/lib/python3.6/distutils/cmd.py", line 313, in run_command
        self.distribution.run_command(command)
      File "/opt/conda/envs/beakerx/lib/python3.6/distutils/dist.py", line 974, in run_command
        cmd_obj.run()
      File "/home/beakerx/beakerx/setupbase.py", line 284, in run
        run(['yarn', 'install'], cwd=node_package)
      File "/home/beakerx/beakerx/setupbase.py", line 152, in run
        return check_call(cmd, *args, **kwargs)
      File "/opt/conda/envs/beakerx/lib/python3.6/subprocess.py", line 291, in check_call
        raise CalledProcessError(retcode, cmd)
    subprocess.CalledProcessError: Command '['yarn', 'install']' returned non-zero exit status 1.
Cleaning up...

There was no such file as /home/beakerx/js/notebook/yarn-error.log, so I wasn't able to inspect it per the error message, but there was a yarn.lock file.

 I suspended my VPN and all was well.


Fix merge conflict with branch-21.10

NOTE: This must be merged with a merge commit, not squashed!
Fixes the merge conflict identified in #3747.  Conflict was in GpuCoalesceBatches where collect and total time metrics were removed in 21.12.

build

Codecov Report

:exclamation: No coverage uploaded for pull request base (branch-21.12@db0c2d5). Click here to learn what that means.
The diff coverage is n/a.

:exclamation: Current head e811543 differs from pull request most recent head 32cf237. Consider uploading reports for the commit 32cf237 to get more accurate results

@@               Coverage Diff               @@
##             branch-21.12    #3748   +/-   ##
===============================================
  Coverage                ?   47.97%
  Complexity              ?     1742
===============================================
  Files                   ?      231
  Lines                   ?    29766
  Branches                ?     4958
===============================================
  Hits                    ?    14281
  Misses                  ?    13321
  Partials                ?     2164

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 db0c2d5...32cf237. Read the comment docs.

build

CI failed due to known issue #3742.  Merging this to restore auto-merge functionality.


Add name= to your example

First off thanks for actually packaging this into add-on form, I wasn't sure how to do it. Did you see both my solutions? It looks like you went with the simpler of the two, but in this one you do have to pass name='some-group-name' so that the radio buttons will bind into a group.
so it should be {{radio-button name='numbers' value='one' checked=selectedNumber}} instead of {{radio-button value='one' checked=selectedNumber}} in your readme

I'm closing this because it seems to be working as is. If you want I can re-open it


APIPort does not detect usage of the Serializable Attribute

When running the analyzer tool on code that uses the Serializable attribute you will get a 100% pass rate for .NET Core even though it is not supported there.  It looks like we do not check for flags that are set on types, just custom attributes.

[Serializable] is translated into a flag on the type instead of an attribute in the metadata. Looks like there's some helpful stuff on this Roslyn issue: https://github.com/dotnet/roslyn/issues/3898. We should be able to pull out a flag of "IsSerializable" and flag issues for that

Looks like a synthetic attribute should be created in Roslyn now. So we'll not worry about it for now.


Update S3 tests to use public S3 buckets

I now maintain a public bucket on Continuum's account and direct all tests towards that.

Hrm, this is still going to fail.  We need some credentials to use boto3, even if they aren't for this account.
Is there a way to use boto3 anonymously?

Ahh. Yes, there maybe elegant solution but in the past I have checked in a credentials file with fake AWS key and import it in test setup.

Ah, that sounds great.

I plan to give this a shot http://stackoverflow.com/questions/34865927/can-i-use-boto3-anonymously

Updated.  Tests pass for me without credentials.

this looks awesome! Kudos for adding s3 connection caching as well. Thanks.


Improve User Interface

There is great need to improve UI. For example, "NOT DONE" and "DONE" and placement of geekiT box.

Keeping the geekIT vertically longer, with using either a Symbol, or smaller text, might be good.
Similarly placing the Done/NotDone box on the right extreme might look nice.


Atom serial number ('100000') exceeds PDB format limit.

Hi there,
I'm trying to assemble a homopentamer with subunits of ~1700 aa (above 51000 atoms) which causes a problem due to the atom count limit of the PDB file specs of 99,999 atoms.
I'm guessing that at some point subunits in PDB chains in the pair files are joined resulting in this error.
I ran prepare_fastas.py --stage pairs with a limit of 3,900 aa as it's close to the limit I can currently run in my system.
I know I can probably avoid this by setting smaller domains but checking the pair sizes might be a good idea even before going through the modelling of the pairs just to crash at the last step.
At very least I thought of documenting this.
Cheers,
Last lines of log with the error (everything else in the log is fine):
found 10 transformations between 487 and 487
--- Finished building unified representation
--- Running combinatorial assembly algorithm, may take a while
--- Finished combinatorial assembly, writing output models
Traceback (most recent call last):
  File "/g/data1a/u71/xabi/miniconda3/envs/combfold/lib/python3.12/site-packages/Bio/PDB/PDBIO.py", line 379, in save
    s = get_atom_line(
        ^^^^^^^^^^^^^^
  File "/g/data1a/u71/xabi/miniconda3/envs/combfold/lib/python3.12/site-packages/Bio/PDB/PDBIO.py", line 188, in _get_atom_line
    raise ValueError(
ValueError: Atom serial number ('100000') exceeds PDB format limit.

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/g/data1a/u71/xabi/CombFold/scripts/run_on_pdbs.py", line 400, in <module>
    run_on_pdbs_folder(os.path.abspath(sys.argv[1]), os.path.abspath(sys.argv[2]), os.path.abspath(sys.argv[3]))
  File "/g/data1a/u71/xabi/CombFold/scripts/run_on_pdbs.py", line 380, in run_on_pdbs_folder
    assembled_files = create_complexes(clusters_path, first_result=0, last_result=max_results_number,
                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/g/data/u71/xabi/CombFold/scripts/libs/prepare_complex.py", line 142, in create_complexes
    create_transformation_pdb(assembly_path, transforms_strs[i], output_path=output_path, output_cif=output_cif)
  File "/g/data/u71/xabi/CombFold/scripts/libs/prepare_complex.py", line 108, in create_transformation_pdb
    _merge_models(output_path, tmp_pdb_path, output_path, output_cif=output_cif)
  File "/g/data/u71/xabi/CombFold/scripts/libs/prepare_complex.py", line 51, in _merge_models
    io.save(output_path)
  File "/g/data1a/u71/xabi/miniconda3/envs/combfold/lib/python3.12/site-packages/Bio/PDB/PDBIO.py", line 391, in save
    raise PDBIOException(
Bio.PDB.PDBExceptions.PDBIOException: Error when writing atom ('s_pdb', 0, 'D', (' ', 1552, ' '), ('HE1', ' '))

Hey,
As the PDB format doesn't support structures with over 99,999 atoms, so CombFold enables you to output CIF files instead of PDB.
You can look here for an explanation of how to enable CIF output:
https://github.com/dina-lab3D/CombFold/issues/4#issuecomment-1884906966
let me know if that works for you.


Allow for ViewModel navigation

In my viewmodels, I would like to kick of navigation by

injecting INavigator
and calling NaviateToAsync(string path, object argument)

As I see it, this is not implemented out of the box.
Also, I am not sure it has to be part of the Shell.
But: Maybe you can give some advice on how one should approach this.
Do you have any thoughts on this? Or is this of no value/interest to you? :-)

Already cover in the example here: https://github.com/AvaloniaInside/Shell/issues/26


test: add vpc sweepers

About this change—what it does
Add sweepers for VPC package resources

@Serpentiel waiting for this merger first https://github.com/aiven/terraform-provider-aiven/pull/1525 to avoid conflicts

@ivan-savciuc done

duplicates #1529


Adding Brazil and Mexico admin roles and perms

Brazil and Mexco now have separate roles to allow editors from those countries to add content without the ability to publish.  This PR adds the new roles to dosomething_user and updates several of the features that hold the user perms for the functionality they'll need to access.
I didn't put this into the dosomething_global feature because that won't be enabled on production for some time, and these roles need to be used immediately.

:+1:

Awesome @blisteringherb!


pydantic.validate_call has different behavior when type hint union has typing.Iterable

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
In the below code, both functions should return the same value. The only difference is where in the union the typing.Iterable type hint resides. The code works when it's the first type in the union but fails when it's the last.
My wild guess is pydantic is draining the input for validation and subsequent validators get an empty iterable because the 1st one drained it. If that's the case you need to make that idempotent for the other validators... So they have access to the previously read values as if they're reading the iterable for the first time. And if you do that, you need to make sure the actuall caleee can do this, but just once.. So maybe use the iter function to return a read-once only stream of the input.
Example Code
import pydantic
import typing

@pydantic.validate_call
def back(tags: dict[str, object]|list[dict[str, object]]|typing.Iterable[dict[str, object]]):
    return list(tags)

@pydantic.validate_call
def front(tags: typing.Iterable[dict[str, object]]|dict[str, object]|list[dict[str, object]]):
    return list(tags)

input = [
  {
    "TagKey": "tr7h6hturtyrtyr",
    "TagValue": "23424234324"
  },
  {
    "TagKey": "rtyge4fwesft",
    "TagValue": "Sep 27 2024 09:43AM EST"
  },
  {
    "TagKey": "h6gurgyreyg45g",
    "TagValue": "85654654645"
  },
  {
    "TagKey": "Name",
    "TagValue": "5yhtrytr54g646h4"
  }
]

# should return a list of dictionaries but instead returns empty list. This is not working.
print(back(iter(input)))

# should return a list of dictionaries, and returns a list of dictionaries. This is working.
print(front(iter(input)))

Python, Pydantic & OS Version
python -c "import pydantic.version; print(pydantic.version.version_info())"
             pydantic version: 2.9.1
        pydantic-core version: 2.23.3
          pydantic-core build: profile=release pgo=false
                 install path: D:\code\site-packages\pydantic
               python version: 3.12.1 (tags/v3.12.1:2305ca5, Dec  7 2023, 22:03:25) [MSC v.1937 64 bit (AMD64)]
                     platform: Windows-11-10.0.22631-SP0
             related packages: typing_extensions-4.12.2
                       commit: unknown

Here's my take on that repeatable iterator

class _IdempotentIterable(collections.abc.Iterator):
  def __init__(self, iterable: typing.Iterable):
    if iterable is None:
        raise ValueError("value cannot be null.")
    if not isinstance(iterable, typing.Iterable):
        raise ValueError("value is not an iterable.")
    self._iterable = iterable
    self._previous = []

  def __next__(self):
    next = self._iterable.__next__()
    self._previous.append(next)
    return next

  def __iter__(self):
    yield from self._previous
    while True:
      try:
        yield next(self)
      except StopIteration:
        break

def iter_idempotent(iterable: typing.Iterable) -> typing.Iterable:
    return _IdempotentIterable(iterable)

Hmm, our support for typing.Iterable is nuanced and limited. I'm not surprised that this is an issue.
Will probably be fixed when we address https://github.com/pydantic/pydantic/issues/9266

Hmm, our support for typing.Iterable is nuanced and limited. I'm not surprised that this is an issue.
Will probably be fixed when we address #9266

V3 yikes!!! I'll make sure I have integration tests when I apply he validate_call to Iterable type hints.

We'll probably add a flag in v2.10 (in a month) that supports eager validation of iterables that'll fix this for ya :).


Run test problem

hello , i try to run the test,but it still have problem,and i don't where and how to solve.All the need file i have prepared.
when i use this CUDA_VISIBLE_DEVICES="0" python tools/test_lanenet.py --
weights_path /home/pch/test/SCNN/culane_lanenet_vgg_2018-12-01-14-38-37.ckpt-10000  --image_path /home/pch/test/SCNN/test_img.txt
get  the problem 'Segmentation fault (core dumped)' and i don't know how to solve it.

@Endless-Hao, what is the version of your tensorflow, cuda and cudnn?

@cardwing  my tensorflow is 1.10.1  python is 3.6 cuda 9.0 cudnn 7

@Endless-Hao, no other information except 'Segmentation fault (core dumped)'? It is difficult to debug with such limited information. Can you have a try to run a simple tensorflow code which can be downloaded from the official site? I guess it may be caused by the improper installation of tensorflow.

I use deeplab is ok. Just now i delete your some import that not use in your code such as numpy..,the code  can run. The new problem is i change the save_dir, but there are no output .And the code run very fast.
2018-12-13 11:57:39.641641: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1097] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:0 with 8935 MB memory) -> physical GPU (device: 0, name: GeForce GTX 1080 Ti, pci bus id: 0000:01:00.0, compute capability: 6.1)
INFO:tensorflow:Restoring parameters from /home/pch/test/SCNN/culane_lanenet_vgg_2018-12-01-14-38-37.ckpt-10000
I1213 11:57:39.782603 23581 tf_logging.py:115] Restoring parameters from /home/pch/test/SCNN/culane_lanenet_vgg_2018-12-01-14-38-37.ckpt-10000
pch@pch:~/1-lane detection/3

@cardwing  Yes there are no other information, just there sentence. My environment is ok.I use deeplab is ok. Just now i delete your some import that not use in your code such as numpy..,the code can run. The new problem is i change the save_dir, but there are no output .And the code run very fast.
2018-12-13 11:57:39.641641: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1097] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:0 with 8935 MB memory) -> physical GPU (device: 0, name: GeForce GTX 1080 Ti, pci bus id: 0000:01:00.0, compute capability: 6.1)
INFO:tensorflow:Restoring parameters from /home/pch/test/SCNN/culane_lanenet_vgg_2018-12-01-14-38-37.ckpt-10000
I1213 11:57:39.782603 23581 tf_logging.py:115] Restoring parameters from /home/pch/test/SCNN/culane_lanenet_vgg_2018-12-01-14-38-37.ckpt-10000
pch@pch:~/1-lane detection/3

@cardwing  and the code can not debug ,if you debug at some red point ,it say pydev debugger: process 23699 is connecting

@Endless-Hao, the save_dir is not used in fact (I will clean the code and remove useless codes soon). In the testing phase, it should output a sequence of numbers. Please check the image path since this line for i in range(int(len(image_list) / batch_size)): print(i) should at least output a number every episode.

@cardwing  Ok i continue to look look.Thanks


[Enhancement] Dark Mode

It'd be really good if there was an option for Refined GItHub to include a dark mode.

This would be best as a separate Userstyle/Userscript and you can probably find one already

Just because we can find other ones, why wouldn't you add it to here?
If you have people with a desire to use a product, and you have that product, then people use your product. i.e. If you add this, it means that people who don't want to install Stylish can just install this.

Refined GitHub doesn't have to be everything for everyone.
Keeping a dark mode up to date for the whole site is not gonna be easy work and it will easily become out of date. It's easier to just build it as a separate extension so once the maintainer loses interest, another one pops up.
I've seen this all the time for Facebook and YouTube: No one wants to put that much work for too long.

https://github.com/StylishThemes/GitHub-Dark


[LOC] Showing "図 2. USB サム ドライブの選択" (extra asterisks appears)

https://docs.microsoft.com/ja-jp/surface/microsoft-surface-data-eraser

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

ID: 93a0e269-be9b-781a-b9b4-52b94e68618b
Version Independent ID: 725c2d4a-5397-6c67-7605-93a8cd40cf08
Content: Microsoft Surface Data Eraser (Surface)
Content Source: devices/surface/microsoft-surface-data-eraser.md
Product: w10
Technology: windows
GitHub Login: @BRecords
Microsoft Alias: jdecker

@officedocsbot assign @OfficeGlobal

@OfficeGlobal - Kindly assist with this Translation (Localization) issue. Thank you.

@officedocsbot assign @mypil

@dongmicro - Can you assist with this Translation (Localization) issue?
Thank you.

@dongmicro - I would like to follow up on this. Do you have any updates? Thanks.
cc: @AndreaBarr

@officedocsbot add label localization

@dongmicro - I would like to follow up on this. Can you please assist with this Translation (Localization) issue? Thanks.

Under investigation.

Oh. I think I can see why this is happening. This is about the image text *Figure 2.
Right above the image, there is a note message. followed by the image and the affected figure text:

Select the USB drive of your choice from the USB Thumb Drive Selection page as shown in Figure 2, and then click Start to begin the USB creation process. The drive you select will be formatted and any existing data on this drive will be lost.

[!NOTE]
If the Start button is disabled, check that your removable drive has a total capacity of at least 4 GB.

*Figure 2. USB thumb drive selection*

To me, it looks like the Github indentation, along with the asterisks used to format the text as italics, puts the image text in a code block instead of showing it as formatted text, hence making the asterisks show up on the Github page. This looks like the note above the image, along with the following indentation, makes this happen instead of showing the text as regular formatted text.
The weird thing to me is that the format issue does not show up on the docs.microsoft.com page, neither the Japanese version nor the en-us version of the page, neither in MS Edge nor in Chrome.
I admit I am a little bit confused now, seeing that I can only reproduce it using Github, not the docs.microsoft.com page.

@Maruktruk @dongmicro - Any updates on this?

Looking forward to hear from you, @Maruktruk & @dongmicro.

@officedocsbot unassign @OfficeGlobal
@officedocsbot assign @dongmicro

Similar: #3394 #3453 #3452 #3451

@officedocsbot unassign @mypil

@officedocsbot assign @e0i

@officedocsbot assign @MollyMalone

@officedocsbot unassign @MollyMalone
@officedocsbot assign @TinaMcN

@e0i : Looking back at the pages now, this specific issue seems to be resolved, both for the Japanese and Github pages:

https://docs.microsoft.com/ja-jp/surface/microsoft-surface-data-eraser#microsoft-surface-data-eraser-usb-スティックの作成方法
https://github.com/MicrosoftDocs/windows-itpro-docs/blob/master/devices/surface/microsoft-surface-data-eraser.md#how-to-create-a-microsoft-surface-data-eraser-usb-stick

May I ask for you and @hyoshioka0128 to confirm that this is the case, testing several of your browsers as well?
If this issue really is solved, would you like to collect the remaining open linked issues in a different issue page?

@officedocsbot unassign @dongmicro

@officedocsbot close
This issue appears to be fixed.

@TinaMcN Thank you. Would you confirm that associated group of issues have also been fixed?

@e0i thanks so much for doing that - much appreciated 👍


[Beam 8138] Code snippet in FileIO Java docs fixed

Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

R: @pabloem

LGTM!


Fixed errors and warnings

A few standard libs where needed here and there, and fixed a few warnings related to signedness.

@MarkOates can you merge this pull request as well ? It fixes build errors on Linux.

@MarkOates can you merge this pull request as well ? It fixes build errors on Linux.

Definitely.  Give me a couple seconds to take a look.

Also, the reason why you may not have any "algorithm" warning is because you might have it "in cache" for you builds. It happened to me quite a few times. You shall try to make a clean build.


fix: included only transfer amounts that are charged to the operator in getTransferAmountSumForAccount()

Description:
Currently, getTransferAmountSumForAccount() in both the MirrorNodeClient and SdkClient classes is used to calculate the total sum of transfer amounts for a specific account from a transaction record related to the operator account. These amounts should only reflect payments made by the operator, so they can be tracked in the Hbar Limiter's metrics. However, the current implementation also includes transfer amounts received by the operator, which inaccurately updates the remaining budget in the Hbar Limiter.
This PR fixes getTransferAmountSumForAccount() to only include transfer amounts charged to the operator by checking whether the amount attribute in the transfer records is negative.
When requesting a transaction record from the SDK or MirrorNode, the transfers array looks something like this:
[
  { account: '0.0.1022', amount: -300, is_approval: false },
  { account: '0.0.1023', amount: -600, is_approval: false },
  { account: '0.0.1024', amount: 900, is_approval: false },
  { account: '0.0.1022', amount: 900, is_approval: false },
  { account: '0.0.1023', amount: -900, is_approval: false },
]

The positive amounts indicate funds charged from the account's balance, while the negative amounts indicate transfers into the account. In this case, we only want to include amounts charged or paid by the account, meaning those with negative values. This is why we add the condition transfer.amount < 0 in the check.
Related issue(s):
Fixes #3098
Notes for reviewer:
Checklist

[ ] Documented (Code comments, README, etc.)
[ ] Tested (unit, integration, etc.)

Please clarify in the description why the fix involves filtering for less than 0 amounts.

added example in PR description https://github.com/hashgraph/hedera-json-rpc-relay/pull/3103#issue-2589374343


Fix get_vcs_repo by moving it to the Mixin

There was a timeout error on one of the search tests. I restarted the job but it should be fine.

THis is deployed and fixed the error. 👍


for some reason my age is not listed correctly

Subject: for some reason my age is not listed correctly
Description:
How can i correct the age on my profile please
Results:
Backend version: develop-0023fa73
Frontend version:
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
Page: https://couchers.org/profile
User: fay_sal (7098)

contact support


"IOError" in merge.py

I like to keep my precompiled CSS in one directory (i.e. ".../_css") and then output compiled/compressed  CSS to a production location (i.e. ".../css"). In a fresh deployment, if I've failed to create the /css tree structure, webassets fails on MemoryHunk.save(). Here is part of a sample stack trace:
File "/application/blueprints/web2/templates/web2/commons/landing.html", line 3, in block "css"
    {% assets "web2/_css/base.ccss", "web2/_css/commons/landing.ccss",
File "/.virtualenvs/site/lib/python2.7/site-packages/webassets/ext/jinja2.py", line 102, in _render_assets
    'filters': filter}).urls(env=env)
File "/.virtualenvs/site/lib/python2.7/site-packages/webassets/bundle.py", line 448, in urls
    urls.extend(bundle._urls(env, extra_filters, *args, **kwargs))
File "/.virtualenvs/site/lib/python2.7/site-packages/webassets/bundle.py", line 421, in _urls
    *args, **kwargs)
File "/.virtualenvs/site/lib/python2.7/site-packages/webassets/bundle.py", line 350, in _build
    hunk.save(env.abspath(self.output))
File "/.virtualenvs/site/lib/python2.7/site-packages/webassets/merge.py", line 96, in save
    f = open(filename, 'wb')
IOError: [Errno 2] No such file or directory: '/application/blueprints/web2/static/css/commons/landing.css'

I'm using the Flask-Assets extension on top of webassets. At the very least, it would be great if webassets provided a configuration setting (perhaps False by default) that ensures the output directory structure will be created if it doesn't already exist.

I remember this being a conscious decision, wanting to be conservative, but I can change my mind on this one.
What I'm not a fan of though is adding too many options, so I'd prefer to pick one behaviour.
Currently, I personally tend to have an EMPTY file in these kinds of directories.

I was doing the same as you - checking a ".gitignore" into all of those
directories. However, lately I've been taking a more strategic CSS/JS
approach, culminating in a lot of subdirectories. It turns out to be a lot
to manage and have in-place in order to achieve a successful deployment with
webassets.
Your component is the central point of control on this issue. It would be
good if you provided us with a way to enhance the behavior (i.e. some kind
of pre-filter override where I can make sure the directories are created) or
a config setting to allow your component to do it for me.
It turns out this would save a fair amount of effort.
Thanks for considering,
Jonathan.
On Thu, Oct 27, 2011 at 1:40 PM, Michael Elsdrfer <
<EMAIL_ADDRESS>
I remember this being a conscious decision, wanting to be conservative, but
I can change my mind on this one.
What I'm not a fan of though is adding too many options, so I'd prefer to
pick one behaviour.
Currently, I personally tend to have an EMPTY file in these kinds of
directories.
--
Reply to this email directly or view it on GitHub:
https://github.com/miracle2k/webassets/issues/75#issuecomment-2549044

How do you feel about using os.mkdir() for this, which would create only the last leaf of the path if it doesn't exist, but fails if there is more than one hierarchy level missing. (as opposed  to makedirs(), which would create the full path).


Possible small discrepancy in description of 'observed bandwidth'?

Hi, thanks for posting so many useful details.
In docs/source/bandwidth_tor.rst under "observed bandwidth", it says "max bandwidth sustained output over any 10secs in the past day". Should this say "in the past 5 days"? In the rep_hist_bandwidth_assess function comments it says: "find the largest sums in the past NUM_SECS_BW_SUM_IS_VALID (roughly) seconds", where NUM_SECS_BW_SUM_IS_VALID is defined as (5*24*60*60) (5 days).
I may be misreading this as I'm not very familiar with the tor code, but just thought I'd mention it.

In docs/source/bandwidth_tor.rst under "observed bandwidth", it says "max bandwidth sustained output over any 10secs in the past day". Should this say "in the past 5 days"?

That sentence comes from the Tor directory protocol [0]

In the rep_hist_bandwidth_assess function comments it says: "find the largest sums in the past NUM_SECS_BW_SUM_IS_VALID (roughly) seconds", where NUM_SECS_BW_SUM_IS_VALID is defined as (5*24*60*60) (5 days).

Indeed, i've been explained that that constant was changed to 5 days [1], but the specification wasn't updated.
@teor2345 has created this ticket [2] to update the specification.

I may be misreading this as I'm not very familiar with the tor code, but just thought I'd mention it.

Thanks a lot for mentioning this.
[0] https://gitweb.torproject.org/torspec.git/tree/dir-spec.txt#n439
[1] https://github.com/torproject/tor/commit/8be50ca3ea90ac04b8a18c7d9a33f5ec3f3a6e28, https://trac.torproject.org/projects/tor/ticket/23856#comment:13
[2] https://trac.torproject.org/projects/tor/ticket/23856#comment:13


Add rules for QEMU/libvirt guests (bridged virtual networking)

Needs approval again after rebasing, one should do.

Thanks, @duritong


crash

Please make sure your issue complies with these guidelines:

[ ] Server Software must be the official PMMP w/o vanilla patching plugins (i.e. TeaSpoon)

[ ] Issue must have not been reported previously

[ ] Make sure you are on the latest version of PMMP & PiggyAuctions

[ ] Have a detailed title.

Link to Crashdump:

Extra Information

closing because there's no actual information provided
please fill in the given template


Added exports for distributions.jl

Looks like the exports from Utilities to the rest of the Turing namespace were missing after the reorganization. This just reexports all the stuff inside the distributions.jl file.

@mohamed82008 Looks like the matrix_support.jl test failed on Appveyor, but only for x64. Is this anything we need to be concerned about, or can I merge this over?


Update for language changes

r? @brson
(rust_highfive has picked a reviewer for you, use r? to override)


Training on Google Compute Engine

Hello Rayhane,
I have listened to the sample produced by this project. They are Amazing!!!, Great Work!!!
Now, I want to train English(Indian accent) on Tacotron-2. I have setup Compute Engine with configuration '4' CPUs and '8' Tesla K80 GPUs.
I have enabled the use_all_gpus=True and num_gpus=8 in hparams.py. The training seems very slow 5sec/step.  I verified the GPU usage by uisng the tool "nvidia-smi".  It shows only one GPU is being used (output below).
| NVIDIA-SMI 390.30                 Driver Version: 390.30                    |
|-------------------------------+----------------------+----------------------+
| GPU  Name        Persistence-M| Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp  Perf  Pwr:Usage/Cap|         Memory-Usage | GPU-Util  Compute M. |
|===============================+======================+======================|
|   0  Tesla K80           Off  | 00000000:00:04.0 Off |                    0 |
| N/A   40C    P0    89W / 149W |   8398MiB / 11441MiB |     62%      Default |
+-------------------------------+----------------------+----------------------+
|   1  Tesla K80           Off  | 00000000:00:05.0 Off |                    0 |
| N/A   39C    P0    68W / 149W |    112MiB / 11441MiB |      0%      Default |
+-------------------------------+----------------------+----------------------+
|   2  Tesla K80           Off  | 00000000:00:06.0 Off |                    0 |
| N/A   35C    P0    56W / 149W |    112MiB / 11441MiB |      0%      Default |
+-------------------------------+----------------------+----------------------+
|   3  Tesla K80           Off  | 00000000:00:07.0 Off |                    0 |
| N/A   40C    P0    72W / 149W |    112MiB / 11441MiB |      0%      Default |
+-------------------------------+----------------------+----------------------+
|   4  Tesla K80           Off  | 00000000:00:08.0 Off |                    0 |
| N/A   32C    P0    57W / 149W |    112MiB / 11441MiB |      0%      Default |
+-------------------------------+----------------------+----------------------+
|   5  Tesla K80           Off  | 00000000:00:09.0 Off |                    0 |
| N/A   40C    P0    68W / 149W |    112MiB / 11441MiB |      0%      Default |
+-------------------------------+----------------------+----------------------+
|   6  Tesla K80           Off  | 00000000:00:0A.0 Off |                    0 |
| N/A   35C    P0    57W / 149W |    112MiB / 11441MiB |      0%      Default |
+-------------------------------+----------------------+----------------------+
|   7  Tesla K80           Off  | 00000000:00:0B.0 Off |                    0 |
| N/A   38C    P0    71W / 149W |    112MiB / 11441MiB |      0%      Default |
+-------------------------------+----------------------+----------------------+
I am wondering if multiple GPUs are supported in this project?
Thank you,
SRHyd

Just found #43. Multi-gpu support is not present.

gcloud is a nice choice. If you want to run with multi-gpu, you can wrap model with tf.estimator.EstimatorSpec then call tf.contrib.estimator.replicate_model_fn to create model_fn for tf.estimator.Estimator, now you can use all the gpus.

Thank you @Cecilwang. I will try this.


fix textcat_goemotions textcat factory

As I was testing this out, I realized that it was using the incorrect factory for this multilabel dataset.

Thanks for the PR! This is indeed a recent change in spaCy. Merging :-)

Thanks for the PR! This is indeed a recent change in spaCy. Merging :-)


Use default GOOS and allow to specify GOOS on cmd-line

# Build for current OS
make build

# Build for Linux
GOOS=linux make build

Thanks! I'm ready to merge, but GitHub doesn't recognize your email and didn't associate commit with your account. Do you want to fix it or I can merge as is?
You can see your commit's user.email here: https://github.com/kovetskiy/mark/commit/7f310205ae56802eb90e06ed50397ef5b170aa32.patch

Hi, thanks for merge!
Now email confirmed but I was not aware of this, thanks for pointing it out!


const fn new

Problem
I want to define static or const quantities with units.

Solution
Can the new function in the Measure and MeasurePoint be const? Also, if any of the arithmetic and conversion ops can be const, that’s handy too.

Alternatives

Additional context

The arithmetic operations and the conversions cannot be trivially made const, because they use traits functions, and traits do not support const functions.
I have just pushed a commit with this comment:
Setting const functions whenever possible
The following functions have become const fn:

For type Measure: new.
For type MeasurePoint: new.
For type UnsignedDirection: to_measure_point.
For type SignedDirection: to_measure_point.
For type Measure2d: new, x, y.
For type MeasurePoint2d: new, x, y.
For type LinearMap2d: new.
For type AffineMap2d: new.
For type Measure3d: new, x, y, z.
For type MeasurePoint3d: new, x, y, z.
For type LinearMap3d: new
For type AffineMap3d: new

Thanks @carlomilanesi


Column names for various event types

The event types to be separated in three types separated by types. Following are suggested column names for each:

SNV: the columns remain the same as we have right now.
CNV: Gene | Role in Cancer | Variant | Type | Effect | Tier | Total Segmental Copies | Location
Fusions: Gene | Role in Cancer | Variant | Type | Effect | Tier | Total Fusion Copies | Location

Hide Tier 3 calls.
@kevinmhadi can elaborate on the "total fusion copies" column and any other fusion related details.

Yes, total fusion copies column should be the output from (value = min_cn) from the oncokb_fusions output.
I think the other changes were to move the exon and cytoband information
out of the column into a tooltip. Although we are linking to the tier modal
in the gene names but would a tooltip on hover be possible for fusion
events?
On Mon, Nov 25, 2024 at 11:08 AM Aditya Deshpande @.***>
wrote:

The event types to be separated in three types separated by types.
Following are suggested column names for each:

SNV: the columns remain the same as we have right now.
CNV: Gene | Role in Cancer | Variant | Type | Effect | Tier | Total
Segmental Copies | Location
Fusions: Gene | Role in Cancer | Variant | Type | Effect | Tier |
Total Fusion Copies | Location

Hide Tier 3 calls.
@kevinmhadi https://github.com/kevinmhadi can elaborate on the "total
fusion copies" column and any other fusion related details.
—
Reply to this email directly, view it on GitHub
https://github.com/mskilab-org/case-report/issues/19, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEKVY73DPI2EJGI5BF4H4I32CNDP7AVCNFSM6AAAAABSOLFHEKVHI2DSMVQWIX3LMV43ASLTON2WKOZSGY4TCMZQGA3TSMA
.
You are receiving this because you were mentioned.Message ID:
@.***>

@xanthopoulakis
Minor points of changes to the table:

Tabs:

Change "CNV" tab to "CNA" tab and add all events with type "SCNA"
For fusions, all events marked 'fusions"
Rest all SNVs

Grey the tabs without any events
Counts of events found to be displayed
Search bar for genes that autocomplete gene names
Hide Tier 3 events
Columns specific for each tab as described in this ticket (above)

@asd1289 , please feel free to review

Just adding the suggestion from today's clinical meeting to include an all tab as well.

I think I am going to close this for now. We can re-open if Marcin wants something else in there. Great job all!


Read value for Voice Silence Penalty, fixing replay parsing

This fixes the following error:
System.IndexOutOfRangeException: 'Index was outside the bounds of the array'
at MPQFiles/ReplayInitData.cs, line 242: replay.ClientListByWorkingSetSlotID[workingSetSlotID.Value]

Not sure why this error is only showing up in patch 65617, since the value should be in replays since patch 61718 ( according to @barrett777 )


Argument completers lack functionality

Currently, the two argument completers here lack functionality. https://github.com/PowerShell/SecretManagement/blob/30991fd56ef43074c3ff76f303fbf35c9e46be7d/src/code/SecretManagement.cs#L79
https://github.com/PowerShell/SecretManagement/blob/30991fd56ef43074c3ff76f303fbf35c9e46be7d/src/code/SecretManagement.cs#L778
Both argument completers do not account for spaces in the name or vault parameter. For example, if I have these vaults and secrets set up
Name        Type VaultName
----        ---- ---------
Test Name String SecretStore
Test      String Cred Man

the following command does not wrap quotes around each value when you tab through the values.
Get-Secret -Name Test Name shows instead of
Get-Secret -Name 'Test Name'
Also, the -Name parameter does not take in to account the current value for the -Vault parameter. I've added quotes around the name and vault parameter values below to show the expected behavior from above.
Get-Secret -Vault 'Cred Man' -Name 'Test Name' shows first when I tab through the name parameter instead of
Get-Secret -Vault 'Cred Man' -Name Test
The expected behavior would be that when 'Cred Man' is selected as the vault, the only option I get when I tab through the name parameter is Test.

I'm not familiar enough with C# to tell but looking at #99, it looks my second scenario isn't accounted for. I don't know what this would look like in C# but below is something I put together in Powershell where $fakeBoundParameters is used as a filter. This allows for only secret names to show from a vault if the vault parameter is used first.
$commands = 'Get-Secret', 'Get-SecretInfo', 'Remove-Secret', 'Set-Secret'

Register-ArgumentCompleter -CommandName $commands -ParameterName Vault -ScriptBlock {
    param (
        $commandName, $parameterName, $wordsToComplete, $commandAst, $fakeBoundParameters
    )

    $filter = $fakeBoundParameters.Name

    (Get-SecretInfo | Where-Object VaultName -like "$wordsToComplete*" |
        Where-Object Name -like "$filter*").VaultName | Select-Object -Unique |
        ForEach-Object {
            $completionText = if ($_ -match '\s') { "'$_'" } else { $_ }

            New-Object System.Management.Automation.CompletionResult (
                $completionText, $_, 'ParameterValue', $_
            )
        }
}

Register-ArgumentCompleter -CommandName $commands -ParameterName Name -ScriptBlock {
    param (
        $commandName, $parameterName, $wordsToComplete, $commandAst, $fakeBoundParameters
    )

    $filter = $fakeBoundParameters.Vault

    (Get-SecretInfo | Where-Object Name -like "$wordsToComplete*" |
        Where-Object VaultName -like "$filter*").Name |
        ForEach-Object {
            $completionText = if ($_ -match '\s') { "'$_'" } else { $_ }

            New-Object System.Management.Automation.CompletionResult (
                $completionText, $_, 'ParameterValue', $_
            )
        }
}

Fixed in RC2.


[Kernel] Register punica ops directly

Motivation
Inspired by #9896, directly registering punica  ops  can reduce the current punica custom op  overhead when using eager mode. The improvement is shown in the table below.
ping @youkaichao  could you plz look at this PR, thanks

"It looks like the CI failure is not due to this PR @youkaichao


Is there another way to start create3？

Is there another way to start create3？

The Create 3 can only be turned on by placing it on the dock.


inputText; date formatting review

User story
As a UI designer of Auro, I want to be able to review the work, so I can give feedback and prepare for next steps.
Outline subtasks or tasks

List specific steps required to deliver this work.

[ ] review work as presented by the product designer
[ ] provide feedback
[ ] prepare for next steps of development and symbolize
[ ] Add a link to Abstract in story comments

Definition of “Done” (exit criteria)

The story is generally “done” when the user can complete the outlined task, but make sure to define what that is.

This story will be considered done when the presented work has been approved and prepared for dev work and symbolization.
reference
https://github.com/AlaskaAirlines/auro/issues/183

Resources are not available to address this task. The intent is to have an FTE designer assigned to Auro to address this and other related work items.

Resources are not available to address this task. The intent is to have an FTE designer assigned to Auro to address this and other related work items.


Fix permission in install-deps.sh

Add sudo in last 2 commands:
virtualenv --system-site-packages -p python3 env
env/bin/pip install google-assistant-sdk[auth_helpers]==0.1.0
grpc-google-cloud-speech-v1beta1==0.14.0 protobuf==3.1.0

These commands shouldn't be run as root - they create a virtual environment in the pi user's home directory.
Why do you want to run them with sudo?

I face a permission issue while running this script, eventually my
configuration is wrong
2017-05-10 9:59 GMT+02:00 Rodrigo Queiro<EMAIL_ADDRESS>
These commands shouldn't be run as root - they create a virtual
environment in the pi user's home directory.
Why do you want to run them with sudo?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/google/aiyprojects-raspbian/pull/18#issuecomment-300405784,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AO-c5mj1joK9E8EOiSq2RTo6gHlvKTlkks5r4W5rgaJpZM4NVr1N
.

Could you post the error you get, and also the output of the following commands?
cd ~/voice-recognizer-raspi  # or wherever you've cloned the repo
ls -l
id

It appears there was no issue with the permission... thanks for support


Dutch translation

Is a translation to Dutch of https://github.com/firoorg/firo-site/tree/master/_i18n welcome here via a PR or is another translation service used for this?

It is welcome!

Hey @PanderMusubi! Are you still interested in translating the website in Dutch? Please reach out if you are, would be really appreciated. We can help you out and give support :)

I have initial translation done but have difficulty translatiing:

anonymity set
trustless setup

How would you describe this in other words?

Other question is about the file format. With git diif I see a lot of ^M

When I run file on the YAML files, I see:
firo-site/_i18n$ file *
en.yml:    HTML document, ASCII text, with very long lines (843), with CRLF line terminators
id.yml:    Unicode text, UTF-8 text, with very long lines (884), with CRLF line terminators
nl.yml:    HTML document, Unicode text, UTF-8 text, with very long lines (923), with CRLF line terminators
zh-cn.yml: Unicode text, UTF-8 text, with very long lines (326)
Shall I fix those too and how? Or leave the file format as is?

How would you describe this in other words?

Anonimity set is the amount of inputs that get aggregated together to create a transaction. Higher anonimity set means that more inputs are used, so the transaction is more private.
A trustless setup is more complex to explain, you probably find more specific info online, but it's basically an initial setup where you don't have to trust the people that created the inizial set up. It's basically the guarantee that none of the people involved in the set up tempered with it in any way.
These are both very technical definitions, i suggest to take a look online for more detailed explanations

Other question is about the file format. With git diif I see a lot of ^M

Ignore it. ^M, it's about marking line endings. It's a matter of editors doing things differently.

Shall I fix those too and how? Or leave the file format as is?

Leave as it is please, we can take a look later on in case of issues. Ideally we use UTF-8 everywhere.
Are you on Matrix/Element or Telegram? Would be easier to help you there.

#386


fix thread sync bug in MD360VideoTexture

MD360VideoTexture has risk because mSurfaceTexture was touched by multi threasd without any synchronized protect.

Thank you very much, but MD360VideoTexture#release is called in main thread, and sync MD360Texture#resize MD360VideoTexture#create is the better choice, I will merge this PR and commit some change.


Composite electrode model

Description
Add a composite electrode particle model for composite electrodes with two phases. e.g. graphite/silicon for the LG M50. This model includes the basic functions, and users are encouraged to add more features by adapting it to sub-models.
This work is referred to a recent paper below
W. Ai, N. Kirkaldy, Y. Jiang, G. Offer, H. Wang, B. Wu (2022). A composite electrode model for lithium-ion battery with a silicon/graphite negative electrode. Journal of Power Sources. 527, 231142.
Fixes # (1433)
Type of change

[x] New feature (non-breaking change which adds functionality)
[ ] Optimization (back-end change that speeds up the code)
[ ] Bug fix (non-breaking change which fixes an issue)

Key checklist:

[x] No style issues: $ flake8
[x] All tests pass: $ python run-tests.py --unit
[ ] The documentation builds: $ cd docs and then $ make clean; make html

You can run all three at once, using $ python run-tests.py --quick.
Further checks:

[ ] Code is commented, particularly in hard-to-understand areas
[ ] Tests added that prove fix is effective or that feature works

Actions involved

A new notebook  [How to use composite particle model] (https://github.com/weilongai/PyBaMM/blob/composite-release/examples/notebooks/models/composite_particle.ipynb)
A new basic function basic_dfn_composite.py
A new set of parameters for the composite electrode materials graphite_Chen2020_composite
A new test function test_dfn_composite.py
Modifications to the lithium_ion_parameters.py

There is an error in the test function test_basic_models.py, but it is unclear what are the reasons.

It's because in basic_dfn.py the variables name still has the word "of" in it. See here.

It's because in basic_dfn.py the variables name still has the word "of" in it. See here.

Thanks, Robert. Another issue appears (see below), but it seems not to be from this branch.
FAIL: test_compare_dfns (test_models.test_full_battery_models.test_lithium_ion.test_compare_basic_models.TestCompareBasicModels)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/runner/work/PyBaMM/PyBaMM/tests/integration/test_models/test_full_battery_models/test_lithium_ion/test_compare_basic_models.py", line 31, in test_compare_dfns
    np.testing.assert_array_almost_equal(
  File "/home/runner/work/PyBaMM/PyBaMM/.tox/integration/lib/python3.8/site-packages/numpy/testing/_private/utils.py", line 1046, in assert_array_almost_equal
    assert_array_compare(compare, x, y, err_msg=err_msg, verbose=verbose,
  File "/home/runner/work/PyBaMM/PyBaMM/.tox/integration/lib/python3.8/site-packages/numpy/testing/_private/utils.py", line 844, in assert_array_compare
    raise AssertionError(msg)
AssertionError:
Arrays are not almost equal to 4 decimals

Are you planning another PR where the composite electrode model is added as a submodel?

Are you planning another PR where the composite electrode model is added as a submodel?

No, I don't have a plan for submodels now. The basic functions serve to explain the interactions between silicon and graphite. If someone would like to study more complex chemistry (I guess there are plans already), he (or she) is encouraged to convert the basic functions to submodels and then to add more features.

I'm disappointed that you've only added a basic model and not a submodel for this. It goes against the spirit of the common modeling framework, which is to be able to couple different submodels. Now this model can't be used with thermal effects, experiments, degradation, etc. I know it might not be obvious how you would add this as a submodel, but we could have helped you, and you're now asking someone to do this entirely for you with no incentive, when you could have asked earlier and someone would have helped in return for co-authorship on your paper.

I did raise an issue  #1433, but found that it would take much more time to make it submodel. There are similar papers appearing (see https://doi.org/10.1149/1945-7111/ac5481 and http://dx.doi.org/10.1149/1945-7111/ac51f3), so it was in a rush. This follows a solution-led strategy, i.e. using the simplest way (basic functions) to solve the problem. When people are studying new problems (adding more interactions), it is straight to adapt this model to meet their requirements, which are unknown at this stage.

Thanks @rtimms, I don't like the way of making lots of parameter sets (Chen2020) as well, but it will not address this issue by making this model to submodels. Dr. Mayur Bonkile (new postdoc at Imperial) will take it over and forward, for other coupled models.
I will keep this branch locally, and instead of leading to my repository, how about creating a copied branch at the pybamm main repository?

Yes, agreed that making this into submodels won't solve the issue about parameter sets, but having the model more closely integrated would justify adding an extra parameter set now while we figure out a better way to handle parameters.
I was suggesting doing something like this example where you can define your model and parameters (and specify the version of pybamm it works in). Then you can include links to your paper and people can download and run the model, but it doesn't require maintaining a separate pybamm branch. My worry with just having a branch on the main pybamm repo with this composite model in is that it will get buried away. By having a separate repo it's easier to point people to and easier to google - hopefully people searching your paper can find the repo or you could tweet or similar to point people to it when you advertise the paper.

Thanks @rtimms, I will close this pull request and quote my own repository in the issue #(1433)

Great, thanks! Let us know if you or Mayur want any help in the future integrating this work.


[Spell] Sleet Storm

The Sleet Storm spell doesn't apply any effects on enemies and doesn't make them pass any saving throws. It looks like the script just wouldn't boot.
All the required modules are totally up-to-date.

Check that you have the setting template listener on.

I sure have. For example, Spirit Guardiand from your module work fine for me.

Spirit Guardians uses a different setting.

Anyway, every single button in "general" and "spell" windows is on, including template listener.


First draft of rearrangable items

This adds a new widget Rearrangable which allows its children to be dragged and reordered. It has several problems:

It requires its closures to be Clone, and then makes judicious use of this trait. I'm not sure how else to code this to make the lifetimes happy.
You can currently only drag a widget one space at a time, then you have to grab it again. I suspect this is because of #25 causing the holding state to be reset whenever the data updates.

The main use case I have in mind for this is to build tabs that can be reordered.

I don't know if it makes much difference in performance, but when I've needed to pass closures into bindings as you have here I've first wrapped them in Rc which removes the need for the closure to be clone but still allows you to clone the Rc and move it into the build closure.


Private Registry - API does not declare privateRegistry password as required

Version: v2.2.5 (pre-existing issue)
Version: master-head (7/16/19)
What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):

Create a cluster, fill out private registry info. Do not supply a password.

Result:
The API does not stop us. If review /v3/schemas/privateRegistry you will see that password field does not have required: true declared.
The user is able to create their cluster and will never authenticate properly with registry.
You will get a generic error like:
[Failed to pull image [https://testregistry/rancher/rke-tools:v0.1.34] on host [redacted_ip]: Error checking if image [https://testregistry/rancher/rke-tools:v0.1.34] exists on host [redacted_ip]: Error response from daemon: no such image: https:/testregistry/rancher/rke-tools:v0.1.34: invalid reference format]

We may need to touch base with @westlywright on this? Not sure.
Maybe if it was required then couldn't create cluster without sending a prviateRegistry password each time. So maybe the UI needs to handle this? Not completely sure.

We can't mark the field as required in the api as the private registry might not be password protected. What we can do is - make sure password is passed when user name is passed, and vice versa.


How long should this take to run?

Not complaining - trying to plan - but approx. how long should the allan_variance computation process take for say ~3hr long data?
I have a rosbag ~3hrs (11214s) long being processed.
The message Computed X averages for period Y is progressing through 1-unit of "period" every ~3.5s real-time.
If I take "period" to mean the seconds in the rosbag it looks like my computer will take >10hrs to process... Is just how it is or am I missing something?
Obviously this is dependant on my hardware. I'm on a Ryzen 3600X based machine 16GB if that helps.

Solved!
I bothered to read the code (shame on me being lazy).

If you build with Release it's much faster.
I processed a 8h bag in 215seconds on a 3950X.


Fix build errors for angular 13

ModuleWithProviders should also be imported from '@angular/core' and not '@angular/compiler/src/core'

admin please merge this PR so solve mentioned issue - as this package is unusable without this fix

I have the same problem for Angular 14.

@gabrielbusarello  It would be really great to merge it (and also add support for ng14)

@cattingcat check this fork and let me know how it works out for you
https://github.com/wadie/material2-carousel

@gabrielbusarello could you please merge it for Angular 13

@gabrielbusarello duvido fazer o merge...

@gabrielbusarello : Please merge it .


Building freeze for Ubuntu

I'm on Ubuntu and simply running pip install cffi torch-unique leaves an empty stdout for a very long time such that I have to manually kill the pip.
The I tried to run build.sh directly from the source repo, the following building statement again halts for a long time with nothing on stdout:
nvcc -c -o $BUILD_DIR/kernel.so $SRC_DIR/kernel.cu -arch=sm_35 -Xcompiler -fPIC -shared -I$TORCH/lib/include/TH -I$TORCH/lib/include/THC -I$SRC_DIR

Eventually, I have to manually stop the kernel compilation.
My nvcc --version output:
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2016 NVIDIA Corporation
Built on Tue_Jan_10_13:22:03_CST_2017
Cuda compilation tools, release 8.0, V8.0.61

Any idea on what might be going on?

Hello,
sadly no, I don't know exactly what's going on yet. Compilation can take up to one minute on my machine (thrust include slows this process down). However, I tested compilation on macOS and Ubuntu and both worked fine, so I'm really sorry to hear that your build freezes. I will investigate it further. Can you post the --verbose output of the nvcc command?

My nvcc --verbose output:
nvcc warning : The 'compute_20', 'sm_20', and 'sm_21' architectures are deprecated, and may be removed in a future release (Use -Wno-deprecated-gpu-targets to suppress warning).
nvcc fatal   : No input files specified; use option --help for more information

Thanks for the --verbose tip! Problem got solved.
For some reason, my machine is going slow and I was being impatient. It took minutes for the pip install to finish but I'm able to import the library now.

Glad that it's now working. However, I will try to make the installation process faster.


Repeated link on 4.4

In the section, 4.4: Encrypt all sensitive information in transit, the hyperlink is repeated twice.
https://learn.microsoft.com/en-us/security/benchmark/azure/baselines/expressroute-security-baseline?toc=%2Fazure%2Fexpressroute%2FTOC.json#44-encrypt-all-sensitive-information-in-transit
Is it supposed to only have 1 hyperlink or should the second one be pointing somewhere else useful, like this:
https://learn.microsoft.com/en-us/azure/expressroute/expressroute-about-encryption

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

ID: 0a52cba1-71f6-3338-d4cb-b1a8ee03a936
Version Independent ID: bcaa9f84-b6ec-7e5c-96e3-a286d0dc9043
Content: Azure security baseline for ExpressRoute
Content Source: benchmark/azure/baselines/expressroute-security-baseline.md
Service: expressroute
GitHub Login: @msmbaldwin
Microsoft Alias: mbaldwin

@thommck Thank you for your feedback! We have assigned this issue to the author to review further and take the next course of action.

@thommck Thank you for catching the duplicate. I have submitted a request to get that updated.
https://github.com/MicrosoftDocs/security-benchmark-docs-pr/pull/476
#please-close


Repeated 500 Errors in Gitea Logs

Description
I have observed a recurring issue where multiple 500 Internal Server Error responses are being logged in Gitea. Below are some sample log entries indicating the problem:
router.log:513093:2024/07/09 11:38:42 ...eb/routing/logger.go:102:func1() [I] router: completed POST /api/internal/ssh/10/update/1559 for <IP_ADDRESS>:0, 500 Internal Server Error in 13.3ms @ private/key.go:16(private.UpdatePublicKeyInRepo)
router.log:513104:2024/07/09 11:38:42 ...eb/routing/logger.go:102:func1() [I] router: completed POST /api/internal/ssh/10/update/5689 for <IP_ADDRESS>:0, 500 Internal Server Error in 4.9ms @ private/key.go:16(private.UpdatePublicKeyInRepo)
router.log:513122:2024/07/09 11:38:42 ...eb/routing/logger.go:102:func1() [I] router: completed POST /api/internal/ssh/10/update/5749 for <IP_ADDRESS>:0, 500 Internal Server Error in 4.2ms @ private/key.go:16(private.UpdatePublicKeyInRepo)
router.log:513135:2024/07/09 11:38:42 ...eb/routing/logger.go:102:func1() [I] router: completed POST /api/internal/ssh/10/update/54 for <IP_ADDRESS>:0, 500 Internal Server Error in 5.7ms @ private/key.go:16(private.UpdatePublicKeyInRepo)
router.log:513257:2024/07/09 11:38:50 ...eb/routing/logger.go:102:func1() [I] router: completed POST /api/internal/ssh/10/update/2166 for <IP_ADDRESS>:0, 500 Internal Server Error in 5.1ms @ private/key.go:16(private.UpdatePublicKeyInRepo)
...
access.log:931321:<IP_ADDRESS> - - [09/Jul/2024:11:49:32 +0300] "POST /api/internal/ssh/10/update/1597 HTTP/1.1" 500 27 "" "GiteaServer"
access.log:931322:<IP_ADDRESS> - - [09/Jul/2024:11:49:32 +0300] "POST /api/internal/ssh/10/update/5982 HTTP/1.1" 500 27 "" "GiteaServer"
access.log:931339:<IP_ADDRESS> - - [09/Jul/2024:11:49:32 +0300] "POST /api/internal/ssh/10/update/5972 HTTP/1.1" 500 27 "" "GiteaServer"
access.log:931341:<IP_ADDRESS> - - [09/Jul/2024:11:49:32 +0300] "POST /api/internal/ssh/10/update/1215 HTTP/1.1" 500 27 "" "GiteaServer"
access.log:931372:<IP_ADDRESS> - - [09/Jul/2024:11:49:33 +0300] "POST /api/internal/ssh/10/update/1936 HTTP/1.1" 500 27 "" "GiteaServer"
access.log:931398:<IP_ADDRESS> - - [09/Jul/2024:11:49:33 +0300] "POST /api/internal/ssh/10/update/63 HTTP/1.1" 500 27 "" "GiteaServer"
access.log:931405:<IP_ADDRESS> - - [09/Jul/2024:11:49:33 +0300] "POST /api/internal/ssh/10/update/4344 HTTP/1.1" 500 27 "" "GiteaServer"
...
How can I identifying the root cause of these repeated 500 errors? How can I trace and resolve this issue effectively?
Thanx in advance
Gitea Version
1.22.1
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
No response
Screenshots
No response
Git Version
2.31.1
Operating System
CentOS Stream 9
How are you running Gitea?
Self-hosted
Database
PostgreSQL

Thanks for the report. Those logs are both just router logs, but if you can provide more verbose logs around the error we can start to debug.

router.log
gitea.log
access.logI collected logs at a certain point around one of the 500 events, hid the names of repositories and organizations