Pagination helpers

Using the normalized cache with lists and especially pages is cumbersome as different pages get stored in different records:
query GetRepositories($first: Int, $offset: Int) {
  repositories(first: $first, offset: $offset) {
    id
    owner
    name
  }
}

This will store the following root Record:
{
  "repositories(first: 100, offset: 0)": [
     { "owner": .. },
     { "owner": .. },
  ],
  "repositories(first: 100, offset: 100)": [
     { "owner": .. },
     { "owner": .. },
  ],
}

Because the data is spread between two Record fields, there's no easy way for a watcher to retrieve the full list.
API is yet to be defined.
Part of apollographql/apollo-kotlin#2331

Hi 👋 The apollo-kotlin-normalized-cache-incubating artifacts now support this.
Documentation is at https://apollographql.github.io/apollo-kotlin-normalized-cache-incubating/pagination.html.
Let us know what you think.

@martinbonnin I tried it right away, and it's working smoothly even with nodes! Thank you very much!

build.gradle.kts

-    implementation("com.apollographql.apollo:apollo-normalized-cache:$apolloVersion")
+    val apolloNormalizedCacheIncubating = "0.0.2"
+    implementation("com.apollographql.cache:normalized-cache-incubating:$apolloNormalizedCacheIncubating")

-         schemaFiles.from("src/main/java/com/xxx/android/storefront.graphqls")
+        schemaFiles.from("src/main/java/com/xxx/android/storefront.graphqls", "src/main/java/com/xxx/android/connection.storefront.graphqls")

connection.storefront.graphqls

+ extend type Collection @typePolicy(connectionFields: "products")
+ extend type Collection @fieldPolicy(forField: "products" paginationArgs: "after")

Collection.kt

-                                        val resp = storefront.query(afterQuery).fetchPolicy(FetchPolicy.NetworkOnly).execute()
-                                        val respData = resp.data ?: return@LaunchedEffect
-                                        val respCollection = respData.collection ?: return@LaunchedEffect
-                                        val newNodes = collection.products.nodes + respCollection.products.nodes
-                                        val newData =
-                                            respData.copy(
-                                                collection =
-                                                    respCollection.copy(
-                                                        products = respCollection.products.copy(nodes = newNodes)
-                                                    )
-                                            )
-                                        val keys = storefront.apolloStore.writeOperation(q, newData)
-                                        storefront.apolloStore.publish(keys)
+                                        storefront.query(afterQuery).fetchPolicy(FetchPolicy.NetworkOnly).execute()

collection.storefront.graphql

query StorefrontCollectionProductsPage($id: ID!, $sordKey: ProductCollectionSortKeys, $reverse: Boolean, $first: Int!, $after: String, $filters: [ProductFilter!]!, $countryCode: CountryCode!, $languageCode: LanguageCode!) @inContext(country: $countryCode, language: $languageCode) {
    collection(id: $id) {
        id
        title
        products(first: $first, sortKey: $sordKey, reverse: $reverse, after: $after, filters: $filters) {
            nodes {
                id
                ... ProductItemFragment
            }
            pageInfo {
                endCursor
                hasNextPage
            }
        }
    }
}

StorefrontApolloClient.kt

        .normalizedCache(
            normalizedCacheFactory = memoryCacheFactory,
            cacheKeyGenerator = IdCacheKeyGenerator,
-            cacheResolver = IdCacheResolver,
+            metadataGenerator = ConnectionMetadataGenerator(connectionTypes = connectionTypes),
+            recordMerger = ConnectionRecordMerger

@martinbonnin The cacheResolver has been removed from normalizedCache. How can I use the id field as the key for the cache? Or will the id field automatically be used as the cache key without any configuration?

@sonatard Glad that it works well for you! Also you should be able to remove extend type Collection @fieldPolicy(forField: "products" paginationArgs: "after") as it's done automatically with connectionFields.
About the cache resolver: it's still there but the API changed a bit: resolveField now takes a ResolverContext that contains the values that were previously passed as arguments separately - it should be straightforward to migrate to the new signature but don't hesitate to tell us if this isn't clear!

@BoD

Also you should be able to remove extend type Collection @fieldPolicy(forField: "products" paginationArgs: "after") as it's done automatically with connectionFields

Thanks!! I removed it.

About the cache resolver: it's still there but the API changed a bit: resolveField now takes a ResolverContext that contains the values that were previously passed as arguments separately - it should be straightforward to migrate to the new signature but don't hesitate to tell us if this isn't clear!

I found this PR, but it seems it hasn't been released yet. Is there a planned release date?
https://github.com/apollographql/apollo-kotlin-normalized-cache-incubating/pull/20/files

Woops you are right @sonatard, sorry about the confusion, I forgot this hadn't been released yet. I just published a new release, v0.0.3, which contains the change. It should be available in a few minutes.

@BoD Thank you for your prompt response. It turned out to be a simple diff.

version 0.0.3

object IdCacheResolver : CacheResolver {
    override fun resolveField(context: ResolverContext): Any? {
        val id =
            context.field
                .argumentValue("id", context.variables)
                .getOrNull()
                ?.toString()
        if (id != null) {
            return CacheKey(id)
        }

        return DefaultCacheResolver.resolveField(context)
    }
}

            normalizedCacheFactory = memoryCacheFactory,
            cacheKeyGenerator = IdCacheKeyGenerator,
            cacheResolver = IdCacheResolver,
+            metadataGenerator = ConnectionMetadataGenerator(connectionTypes = connectionTypes),
+            recordMerger = ConnectionRecordMerger

Everything worked perfectly. Thank you for the excellent work. I am looking forward to its official release.


Move credentials from environment variables to rails credentials

Notion card
https://saeloun.notion.site/Need-move-credentials-from-environment-variables-to-rails-credentials-edd145fbb9b24aa788f1c1e34c71d256
Type of change

[x] This change requires a documentation update

Checklist:

[x] I have manually tested all workflows
[x] I have performed a self-review of my own code
[x] I have added automated tests for my code

@aniket-k-kaushik Can you resume work on this ticket after you finish the ones assigned to you currently.

@aniket-k-kaushik Can you resume work on this ticket after you finish the ones assigned to you currently.

@supriya3105 It was finished, the keys required for the credential files were only with Akhil and test are failing because, credential key aren't pushed with this PR. Rest of the things are done.

@supriya3105  I'm closing this PR as @akhilgkrishnan informed he will be taking over the rest of the work and will create a separate PR


Render bare URLs as hyperlinks

Is your feature request related to a problem? Please describe.
I'm always frustrated when I add bare URLs in my Markdown editor and see them rendered as hyperlinks, but then when fastpages compiles them they are rendered  on the blog as merely plain text with no opportunity to click on them.  For example typing "https://www.google.com" appears as a clickable link as I'm editing the blog...
(^^ btw see how GitHub automatically rendered that bare URL as a clickable link?  That's what I'm used to and that's what I'd like.)
...then I have to go back and manually either add angle-brackets around all the URLs, or add some kind of square-brackets-parentheses construction like [https://www.google.com](https://www.google.com)
Describe the solution you'd like
That any properly-formatted bare URL gets rendered as a clickable link, i.e. "blue with an underline".
Describe alternatives you've considered
As I mentioned, manually going back and reformatting the URLs.
Additional context
As mentioned, even GitHub Issues formats bare URLs as hyperlinks by default.  As do many Markdown viewers.

Seems this is a "known issue" with Jekyll.  I found multiple feature requests of this sort around the internet for various markdown-based apps.  Kramdown in particular does not support this.
I'm looking into either writing a little script to auto-add angle brackets around bare URLs, or a Jekyll plugin of some sort.

I have a solution,  thanks to "@Mandy8055" on SO.  This will add angle brackets around bare URLs and leave other links (HTML or regular Markdown) alone.  I have tested it with some rather complicated URLs and found it to be compliant.
Currently this is set up to modify the file in place:
perl -i.orig  -pe 's/(?!>)^(https?:\/\/(?:www\.)?[-a-zA-Z0-9@:%._\+~#=]{1,256}\.[a-zA-Z0-9()]{1,6}\b(?:[-a-zA-Z0-9()@:%_\+.~#?&\/\/=]*))(?!>)$/<$1>/g' markdown_file.md

where markdown_file.md is the post.
If you want to see what the output will be first without overwriting the file, you can remove the "-i.orig" in the perl command, and it will just print to stdout.
Whether this gets integrated in everyone's fastpages is, totally up to you.  The perl dependence may not be desirable.  I tried to use sed instead of perl, but I couldn't get the former to accept the regex even after trying various command-line flags.  Whereas with perl it just-worked.
It's easy enough to copy and paste the above command line that users who are having trouble with bare URLs not getting processed they way they like can simply run the above line for each post once, and problem solved.
I have yet to test this in Jupyter notebooks.

perl is arguably more powerful, but in case that's regarded as overkill, @Mandy8055 also came up with a sed version that works for all the cases I tried:
sed -i -E 's@^[^<]?(https?://(www\.)?[-a-zA-Z0-9@:%._\+~#=]{1,256}\.[a-zA-Z0-9()]{1,6}\b([-a-zA-Z0-9()@:%_\+.~#?&/=]*))[^>]?$@<\1>@gm' markdown_post.md

I'll go ahead and close this as you have found a workaround.  Thanks for documenting your approach so that other people can find this


Can't bind numpad keys as hotkeys

OS: Windows 10
Issue Descriptions: Can't bind any numpad keys (numeric or operators) to play control in the Hotkeys section of Settings.
Steps to Reproduce:

open Desktop Settings
select the Hotkeys tab
click in one of the command fields
press any numpad key, with or without mod keys and with or without numlock on eg ctrl-*, shift-enter

result: key doesn't seem to be recognized.

@JPLeBreton This is a limitation of the Electron globalShortcut API.  Basically if the accelerator is Control+1 how do we know which "1" you mean (numpad or number row) 😄 On some platforms they are mirrored so if you set a shortcut to be Control+1 and use the numpad 1 it will work


"BOOST_LOG_DYN_LINK" redefined  & BOOST_BIND_GLOBAL_PLACEHOLDERS issues resolved

Issues related to warnings from "BOOST_LOG_DYN_LINK" redefined and BOOST_BIND_GLOBAL_PLACEHOLDERS were solved.

Can one of the admins verify this patch?

The PR resolves the Redmine task 1314
The tests are passed without the targeted warnings. Merging the PR.


Support clang as a CUDA compiler

Clang is capable of CUDA compilation these days.
It would be great to add support for using it for CUDA compilation with bazel.

Yeah, there is plan. I have local modification but it doesn't make it now.

@Artem-B Considering you are one of the devs of clang cuda, I have a question about it. I now have a nearly working configuration for clang. The only problem I am facing is, for example:
__global__ void kernel() {
  // blahblah with impl
}

with nvcc, the compiled object have symbol
000000000000019a T __device_stub__Z6kernelv()
00000000000002aa T kernel()

but with clang version 15.0.0 (https://github.com/llvm/llvm-project.git 009d56da5c4ea3666c4753ce7564c8c20d7e0255)
0000000000000000 T __device_stub__kernel()

the kernel() is missing from the object, which is causing my nccl example a linker error for various __global__ functions. In that example a libnccl.so is produced first then link to various perf tests binaries. I am curious what it the root problem here.

nvcc and clang do have somewhat different behavior under the hood and you're likely dealing with more than one issue here.
First, kernels and stubs. Clang indeed no longer has kernel on the host side. https://cuda.godbolt.org/z/833ejYs1c
The main reason for that was to make it possible to distinguish the host-side stub from the actual kernel function in debugger on AMD GPUs. It does mean that CUDA objects compiled with clang will not be able to use <<<...>>> to launch kernels from objects compiled with nvcc. This is probably not a big deal, as mixing two different compilers in the same build is a bad idea to start with. I do not think that's the root cause of the problem you see.
NCCL does rely on RDC compilation (i.e. each source compiles to a GPU object file, instead of a fully linked GPU executable) and that part works very differently in clang vs nvcc. In a nutshell, object files need an extra final linking step, and a bit of extra host-side 'glue' code. NVCC does that under the hood. Clang does not, yet.
Here's how tensorflow implements cuda_rdc_library it uses to compile nccl: https://github.com/tensorflow/tensorflow/blob/ee7cf722e4ca9d02b6e62eb3d1b7506ead995422/third_party/nccl/build_defs.bzl.tpl#L261
@jhuber6 has been working on clang driver changes to make compile-to-object-and-link "just work" on the GPU side. E.g. https://github.com/llvm/llvm-project/commit/b7c8c4d8cf07d2e9e8cd157bccc8bd9e7c76415a
It's an experimental feature at the moment, but it will make things much simpler, once it's ready.

Also, I believe CMake has recently added support for clang as the CUDA compiler. It may be worth checking whether/how they handle RDC compilation there.

I think the expected way to perform RDC-mode compilation is via the CUDA_SEPARABLE_COMPILATION option. I think this is supported for Clang as well judging by this issue.

Judging by the commit that has implemented it in cmake they did use the same RDC compilation process that we've implemented in tensorflow that I've pointed to above.
You may as well just pick up tensorflow's implementation directly. @chsigg would probably be the most familiar with the details, if you have questions.

OK, problem solved, it turns out that I need compile all C code of nccl as cuda with -x cu. Otherwise, there will be linker error caused by global functions.  Also metioned here.

@Artem-B  I think this is addressed in c13ebaada58956435658b18bb91fb46a23534995
Use --@rules_cuda//cuda:compiler=clang to select compiler, See detect_clang macro for detect auto detecting, otherwise, you'll need to bring the toolchain config and registration yourself.
As you are a member of tensorflow, I am wondering if this can be mentioned or evaluated in tf. Might be good for bazel community ;)

@Artem-B I think this is addressed in https://github.com/cloudhan/rules_cuda/commit/c13ebaada58956435658b18bb91fb46a23534995

Do you mean that your build rules are ready to use?
I guess the right sequence to make it all work is to get these rules upstreamed into bazel, then port TF build to it. I'll see what I can do to bring it to the attention of the right people.

Not necessary production ready, but at least usable. It needs more users to test it out before I can say it is production ready. Because it is a build system, there are too many corner cases in it.

One thing that could serve as a motivation to adopt these changes would be to try getting Tensorflow to build using your rules, instead of the ones TF carries. It would be a pretty decent test of the real-world usability of the rules -- TF is probably the largest bazel user outside of Google and is almost certainly the largest user of clang for CUDA compilations. Having them convinced would go a long way towards convincing bazel owners that these rules should to be part of bazel.
Having a proof of concept at that level would also give TF owners rough idea how much work it would take to adopt it and whether it's worth it. One thing to keep in mind is that TF also has to work with our internal build. I don't know yet how hard it would be to switch to your rules. If it's a drop-in replacement of the cuda_library() implementation, it should be doable. NCCL and other RDC compilation users would need some more work, but it should be manageable, too.

@Artem-B  Do we have prebuilt llvm package with NVPTX backend enabled. I'd like adding a building CI. So that I can confidently close this issue finally.

LLVM/Clang releases should have NVPTX built in. E.g https://github.com/llvm/llvm-project/releases/tag/llvmorg-14.0.6
On a side note, just a FYI that there's been a lot of offloading-related changes in clang driver lately that are going to make GPU compilation much closer to C++ compilation. E.g. RDC compilation would "just work" -- clang -c a.cu -o a.o; clang -s b.cu -o b.o; clang -o app a.o b.o would do the job, with compiler and linker taking care of the GPU-side linking. It's still work in progress, but when it is ready that would help to simplify the build process quite a bit and will make things like GPU-side LTO possible with clang.

If you want to try out the new driver I would appreciate it. For compiling an application in RDC mode you can do the following.
clang++ a.cu b.cu --offload-new-driver -fgpu-rdc --offload-arch=sm_70 -c
clang++ a.o b.o --offload-link -lcudart

Right now what's missing from the new driver is support for textures / surfaces, Windows / MacOS support, and compiling in non-RDC mode. The benefits is simplified compilation, static library support, and LTO among others.

It'd be good to be able to load clang from https://github.com/grailbio/bazel-toolchain so that we can have a hermetic toolchain setup. I'll probably look into this at some point soon as we're already using that toolchain for our host builds and will be using rules_cuda soon within one of our product builds.

llvm apt clang is also built with NVPTX enabled, we can use that too.

This is partially fixed by #143. Later I will add a full integration test by adding nccl as an example. The cloudhan/nccl-example branch should be buildable with both clang and nvcc.

Is there any flags I should add besides maybe those:
build:clang --@rules_cuda//cuda:compiler='clang'
build:clang --@rules_cuda//cuda:archs=compute_61:compute_61,sm_61
build:clang --@rules_cuda//cuda:runtime=@local_cuda//:cuda_runtime_static
build:clang --@rules_cuda//cuda:copts='-stdlib=libc++'

in theory for this to work ? I'm having a weird issue: eveything compiles fine, but then on execution it just dies without any output. Maybe I'm living a bit too close to the edge using clang 17 and CUDA 12.1 ? It does say it's only partially supported...

Thanks for the extremely fast and detailed response. I just tried a few things based on your inputs, but no luck. I should add that I'm working in Ubuntu 22.04 in WSL 2 if it's relevant. The same code was running fine a few versions ago using nvcc.

Does the tool ./bin/nvptx-arch return anything? Pretty good litmus for if we're detecting the GPU and runtime correctly at all.

@hypdeb Could you please try --repo_env=CC=clang? I am not sure how it goes if you mix gcc as cc compiler and clang as cuda compiler

@hypdeb Could you please try --repo_env=CC=clang? I am not sure how it goes if you mix gcc as cc compiler and clang as cuda compiler

If we're doing -fno-gpu-rdc (default) I would expect it to work because all the CUDA specific handling is done per-TU. But it's worth a shot.

@hypdeb Could you please try --repo_env=CC=clang? I am not sure how it goes if you mix gcc as cc compiler and clang as cuda compiler

I am already using clang as my cc compiler: https://github.com/hypdeb/lawrencium/blob/1694b0f1707d2bc6d2a782a734749ae1c1379336/toolchain/cc_toolchain_config.bzl#L24

Does the tool ./bin/nvptx-arch return anything?  I should probably try re-building my whole environment from scratch, it's possible I screwed something up along the way as it was a long journey :D Thanks for the amazing rules by the way, it works great with nvcc :)

Close with #158


Fix Retries.withRetry delay

Noticed that Retries.withRetry where not using its delay parameter. And that Retries.withRetryOnCancel where calling sleep twice.

@cla-bot[bot] check

@cla-bot[bot] check


Handle conflicts within in-memory advisories

Due to  this we had to add 'is_vulnerable' in PackageRelatedVulnerability. This happened because we were unable to  handle conflicts within in-memory advisories. This is fixed and the db constraints are changed too

From a chat with @sbs2001 he does not want to merge that yet

Closed in  favour of https://github.com/nexB/vulnerablecode/pull/255


add new prioritized cancer data sources

CancerXGene	http://www.cancerrxgene.org	Genomics of Drug Sensitivity in Cancer
CancerXGene GDSC1000	http://www.cancerrxgene.org/gdsc1000/	A landscape of pharmacogenomic interaction in cancer: Resources and Results
CancerXGene GDSC1000 data summary	http://www.cancerrxgene.org/gdsc1000/GDSC1000_WebResources/Data_Summary.html	Data Summary
cBioportal	http://www.cbioportal.org	Access to large scale primary cancer genomics datasets from TCGA
CGAP: Cancer Genome Anatomy Project	http://cgap.nci.nih.gov	determine the gene expression profiles of normal, precancer, and cancer cells, leading eventually to improved detection, diagnosis, and treatment for the patient.
DoCM: Database of Curated Mutations	http://docm.genome.wustl.edu	highly curated database of known, disease-causing mutations that provides easily explorable variant lists with direct links to source citations for easy verification.
PMKB: Precision Medicine Knowledgebase	https://pmkb.weill.cornell.edu	PMKB is organized to provide information about clinical cancer variants and interpretations in a structured way, as well as allowing users to submit and edit existing entries for continued growth of the knowledgebase. All changes are reviewed by cancer pathologists.
canSAR	https://cansar.icr.ac.uk	an integrated knowledge-base that brings together multidisciplinary data across biology, chemistry, pharmacology, structural biology, cellular networks and clinical annotations, and applies machine learning approaches to provide drug-discovery useful predictions.
A CRISPR Dropout Screen Identifies Genetic Vulnerabilities and Therapeutic Targets in Acute Myeloid Leukemia	https://www.ncbi.nlm.nih.gov/pmc/articles/PMC5081405/	To identify additional therapeutic targets in AML, we optimize a genome-wide clustered regularly interspaced short palindromic repeats (CRISPR) screening platform and use it to identify genetic vulnerabilities in AML cells.
TCIA: The Cancer Imaging Archive	http://www.cancerimagingarchive.net	The image data is organized into purpose-built collections of subjects. The subjects typically have a cancer type and/or anatomical site (lung, brain, etc.) in common.

All suggestions are logged in the 'Data Source Backlog' on Google Drive and will be assessed shortly.


Adding more items that need to be ignored

Adding venv to the ignored directories since it's pretty commonly used as a virtualenv directory name

Adding Visual Studio specific files and directories to .gitignore

Thanks 👍

Thank you! This was great 😊
Sonia Rizzo
https://www.soniarizzodev.com
Il Mer 22 Lug 2020, 12:26 Fredrik Fornwall<EMAIL_ADDRESS>ha
scritto:

Thanks 👍
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/meeshkan/py-http-types/pull/37#issuecomment-662374287,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AH7G3M25RVP2ZY2TRVPR6RLR425ELANCNFSM4PEQ5CYQ
.


Rustup component list shows removed components

I have removed rustfmt-preview however, rustup component list still shows it.
✔ ~
04:19 $ rustup component list
cargo-x86_64-apple-darwin (default)
...
rustfmt-preview-x86_64-apple-darwin
✔ ~
04:19 $ rustup component remove rustfmt-preview
error: toolchain 'stable-x86_64-apple-darwin' does not contain component 'rustfmt-preview' for target 'x86_64-apple-darwin'
✘-1 ~
04:19 $ rustup toolchain list
stable-2018-05-10-x86_64-apple-darwin
stable-x86_64-apple-darwin (default)
nightly-2018-11-14-x86_64-apple-darwin
nightly-x86_64-apple-darwin
✔ ~
04:19 $ rustup component remove --toolchain nightly-x86_64-apple-darwin rustfmt-preview
error: toolchain 'nightly-x86_64-apple-darwin' does not contain component 'rustfmt-preview' for target 'x86_64-apple-darwin'
✘-1 ~
04:19 $ rustup component remove --toolchain nightly rustfmt-preview
error: toolchain 'nightly-x86_64-apple-darwin' does not contain component 'rustfmt-preview' for target 'x86_64-apple-darwin'
✘-1 ~
04:19 $ rustup component remove --toolchain nightly-2018-11-14-x86_64-apple-darwin rustfmt-preview
info: removing component 'rustfmt-preview'
✔ ~
04:20 $ rustup component list
cargo-x86_64-apple-darwin (default)
...
rustfmt-preview-x86_64-apple-darwin
✔ ~
04:20 $ cargo fmt
error: toolchain 'stable-x86_64-apple-darwin' does not have the binary `cargo-fmt`
✘-1 ~
04:20 $ cargo +nightly fmt
error: toolchain 'nightly-x86_64-apple-darwin' does not have the binary `cargo-fmt`
04:21 $ rustup --version
rustup 1.13.0 (ea9259c1b 2018-07-16)

same error

With the migration to Rust 2018 and rustfmt-preview -> rustfmt and this bug, I just ended up re-installing rustup entirely.

The component list now (since February) indicates if a component is installed or not.  I believe that with that UX improvement, this is now resolved.

It is not resolved
▶ rustup component list|grep fmt
rustfmt-x86_64-apple-darwin (installed)
▶ rustup component remove rustfmt
info: removing component 'rustfmt'
▶ rustup component list|grep fmt
rustfmt-x86_64-apple-darwin
▶ rustup --version
rustup 1.20.2 (13979c968 2019-10-16)


cloudstack api is not filtering project or account in usagerecord

ISSUE TYPE

Bug Report

COMPONENT NAME
API UsageRecord
CLOUDSTACK VERSION
4.15.2
CONFIGURATION
Cloudstack Usage enabled => true
SUMMARY
When I try to bring the record usage of a project, nothing is happening. It brings me all records from all projects
STEPS TO REPRODUCE

Open cloudmonkey and type:

list usagerecords projectid=408271ae-fa1c-44e3-b8d7-50f7dab3100c startdate=2022-01-01 enddate=2022-01-02
or
list usagerecords account=myaccount startdate=2022-01-01 enddate=2022-01-01
EXPECTED RESULTS
Retrieve only usage records from the desired project

@sureshanaparti  it seems to be a bug in <IP_ADDRESS>
accountid is not set in lines below
https://github.com/apache/cloudstack/blob/<IP_ADDRESS>/server/src/main/java/com/cloud/usage/UsageServiceImpl.java#L260-L262

@KuasarCloud when you list usage records by accountname, you should pass domainid as well.

In fact @KuasarCloud , if you don't give a domain with accountname you should get an exception/error message about it. Did you?

Hi Guys,
Yes I forgot to pass the domain in the example but when I do and the result is the same, I retrieve all usage records from all accountid/projectid.
BR
Ricardo P
From: dahn @.>
Reply-To: apache/cloudstack @.>
Date: Friday, 14 January 2022, 8:46 AM
To: apache/cloudstack @.>
Cc: oAuth @.>, Mention @.***>
Subject: Re: [apache/cloudstack] cloudstack api is not filtering project or account in usagerecord (Issue #5860)
@KuasarCloudhttps://github.com/KuasarCloud when you list usage records by accountname, you should pass domainid as well.
In fact @KuasarCloudhttps://github.com/KuasarCloud , if you don't give a domain with accountname you should get an exception/error message about it. Did you?
—
Reply to this email directly, view it on GitHubhttps://github.com/apache/cloudstack/issues/5860#issuecomment-1013131029, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQ3A57X7L4KI55QWIAQ5MRLUWASLHANCNFSM5L5ECKAA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID: @.***>

@KuasarCloud are you able to verify a PR (#5866)? It should at least solve the project part of your problem.

I think the fix would work, thanks very much, if you have any release date would be wonderful
Atte,
Ricardo Pertuz
From: dahn @.>
Reply-To: apache/cloudstack @.>
Date: Friday, 14 January 2022, 9:17 AM
To: apache/cloudstack @.>
Cc: oAuth @.>, Mention @.***>
Subject: Re: [apache/cloudstack] cloudstack api is not filtering project or account in usagerecord (Issue #5860)
@KuasarCloudhttps://github.com/KuasarCloud are you able to verify a PR (#5866https://github.com/apache/cloudstack/pull/5866)? It should at least solve the project part of your problem.
—
Reply to this email directly, view it on GitHubhttps://github.com/apache/cloudstack/issues/5860#issuecomment-1013154615, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQ3A57VEFJTDK6WDG2L3FI3UWAWANANCNFSM5L5ECKAA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID: @.***>

I think the fix would work, thanks very much, if you have any release date would be wonderful

Do you mean to say, you cannot test a PR (and have to wait for a release) @KuasarCloud ?
I will test the code before merging but it would be nice if you can help with the test effort. At least to make sure your use-case works.

Sorry I didn't understood. Just checked the  PR page and seems like working, I will do the test and come back to you
BR

From: dahn @.>
Sent: Monday, January 17, 2022 5:14:04 AM
To: apache/cloudstack @.>
Cc: oAuth @.>; Mention @.>
Subject: Re: [apache/cloudstack] cloudstack api is not filtering project or account in usagerecord (Issue #5860)
I think the fix would work, thanks very much, if you have any release date would be wonderful
Do you mean to say, you cannot test a PR (and have to wait for a release) @KuasarCloudhttps://github.com/KuasarCloud ?
I will test the code before merging but it would be nice if you can help with the test effort. At least to make sure your use-case works.
—
Reply to this email directly, view it on GitHubhttps://github.com/apache/cloudstack/issues/5860#issuecomment-1014352848, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQ3A57RMZZ6DTQHH4JKF25LUWPTWZANCNFSM5L5ECKAA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID: @.***>

Fixed in #5866


add support for max_traces_per_second option to datadog.conf

What does this PR do?
This PR does two things:

Uncomments the [trace.sampler] section in the stock datadog.conf
Adds the max_traces_per_second configuration option to datadog.conf

This allows a user to configure this value by setting the MAX_TRACES_PER_SECOND environment variable.
Motivation
We are heavy users of APM.  We require a higher max_traces_per_second than what the default provides (10).
What inspired you to submit this pull request?
The inability to configure max_traces_per_second via an environment variable.

Can someone please take a look at this?  Anything that needs to be done before it's approved/merged?


Switch must be exhaustive

ARCL project gives error after installing it.
Switch must be exhaustive
Do you want to add missing cases?

See this patch for the fix : https://github.com/ProjectDent/ARKit-CoreLocation/pull/95
The problem is also mentionned on issue #100

fixed in #95


feat: audit persistent entities using envers

Description

Closes : https://github.com/devs-from-matrix/hexagonal-spring-boot-java/issues/150
Checklist:

[x] My code follows the contribution guidelines of this project
[x] I have performed a self-review of my own code
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] My commits follow conventional commit message guidelines

thank you @anupbaranwal for your contribution :rocket:


process not defined in node_modules/util/util.js

**Before reporting, I have already ***

checked LIT Protocol developer docs and/or the lit-js-sdk.
searched for other open issues.

Describe the bug
I am getting this error- util.js:109 Uncaught (in promise) ReferenceError: process is not defined
at node_modules/util/util.js (util.js:109:1)
at __require2 (chunk-XBCBYTQI.js?v=4f83186a:19:50)
at bls-sdk.ts:248:11
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment:

Lit SDK version (including the network (ie, "serrano") if different, and type - browser or backend)
Node version
Framework (React, Vue, Angular, Svelte - just Plain HTML!, etc)
Specific tooling (Webpack, Vite, browserify, etc)

Additional info
Add any other context about the problem here.

hey @Tejaswini-Gurumoorthy can you complete To Reproduce, Expected behaviour, Screenshots and Environments so that we could better debug?


added nodedrag events to selection box

closes #3997

I think it would make sense to set the passed node to null if one uses the selection box. This way users could distinguish between a node drag and a selection drag.

👌


[WIP][IBC] Implement ICS-23 and ICS-24 - Tracking IBC store transitions in the network state

Description

Summary generated by Reviewpad on 20 Jun 23 19:03 UTC
This pull request includes multiple changes across the project:

The generation and implementation of Go structures for IBC communication protocol
Adding and renaming package import types in the CHANGELOG.md file
Changes to import statements and function parameters in generic_test.go
Testing for ICS23 proofs in the IBC store package
Enabling a private key for IBC
Adding import github.com/stretchr/testify/require in a file
New functions to handle IBC store updates and pruning along with a new import for "ibcTypes"
Addition of a new field in the IBCConfig message called private_key
Three new test functions in the persistence/trees package
Changes for importing new packages and renaming some variables in fisherman_test.go
Introduction of a new protobuf file that defines two message types related to IBC store updates and pruning
Addition of a new message content type and a call to HandleMessage method on the IBCModule module
Importing and renaming symbols in a file actor_test.go
Changes in IBC configuration
A new constant for an IbcMessage content type for messaging
A new component ICS-23 Vector Commitments in the IBC documentation
New error codes and associated error messages, as well as new functions to create those errors
Changes to the persistence/debug.go file involving the addition of the types.ClearAllIbcStoreMessagesQuery function
Modified go.sum file to add a new dependency and a temporary replacement for another dependency
A new package "sql" with functions for fetching data from the database using SQL queries
Implementation of the ProvableStore interface for the Pocket Network IBC implementation
A new file ibc.go added to the persistence package

Issue
Fixes #<issue_number>
Type of change
Please mark the relevant option(s):

[ ] New feature, functionality or library
[ ] Bug fix
[ ] Code health or cleanup
[ ] Major breaking change
[ ] Documentation
[ ] Other

List of changes

Change #1
Change #2
...

Testing

[ ] make develop_test; if any code changes were made
[ ] make test_e2e on k8s LocalNet; if any code changes were made
[ ] e2e-devnet-test passes tests on DevNet; if any code was changed
[ ] Docker Compose LocalNet; if any major functionality was changed or introduced
[ ] k8s LocalNet; if any infrastructure or configuration changes were made

Required Checklist

[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added, or updated, godoc format comments on touched members (see: tip.golang.org/doc/comment)
[ ] I have tested my changes using the available tooling
[ ] I have updated the corresponding CHANGELOG

If Applicable Checklist

[ ] I have updated the corresponding README(s); local and/or global
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added, or updated, mermaid.js diagrams in the corresponding README(s)
[ ] I have added, or updated, documentation and mermaid.js diagrams in shared/docs/* if I updated shared/*README(s)

Note: This PR is based ontop of #843 and #842 (inturn also on #756) but the main logic for the mempool interactions, ICS-23 proof generation/verification and ICS-24 stores is in place

🚨 I REWROTE HISTORY 🚨
This PR was based off of numerous other branches which have now been merged into main. Because of this I have rebased entirely off of ibc/ics23_integration and this the commit history has changed.

Can you please merge with main before we take another look at this?

@Olshansk Dunno why it did this, I merged with main earlier today o-o did again should be fine now


Update Posts “lorem-ipsum”

Automatically generated by Netlify CMS

👷 Deploy Preview for parsonscabinetry processing.

Name
Link

🔨 Latest commit
34f2039f3fe496e3cc60c716e3ff02178925c0e7

🔍 Latest deploy log
https://app.netlify.com/sites/parsonscabinetry/deploys/62b28ceb5b43d3000988d51a


[Autocomplete] Incorrect handling of multiline fields

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
Autocomplete currently not working correctly with multiline fields, producing the following error:
**MUI: Unable to find the input element. It was resolved to [object HTMLTextAreaElement] while an HTMLInputElement was expected.
Instead, Autocomplete expects an input element.
Make sure you have customized the input component correctly.**
example:
<Autocomplete
    options={options}
    fullWidth
    inputValue={input}
    isOptionEqualToValue={(option, value) => option.id === value.id}
    onInputChange={(e, value) => setInput(value)}
    renderInput={(params) => <TextField multiline={!!rows} rows={rows}  {...params} label={label} />}

Expected behavior 🤔
no error
Steps to reproduce 🕹
Steps:

Context 🔦
No response
Your environment 🌎

`npx @mui/envinfo`
  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.

Duplicate of #29961


fix(react-email): Remove obsolete appDir option from next.config.js

This pull request removes the no longer necessary appDir option from the next.config.js  (experimental) file in the react-email repository. The App Router is now stable as of Next.js 13.4, eliminating the need for this option.
changes:

Removed experimental appDir option from next.config.js.

Thanks @Mriganka5137!

Happy to help 😄


Updates for Issue 537

The PR adds the changes for fulfilling the issue 537 AC. The changes adds a function to determine either to use dask or pandas.After research i noticed that pandas dataframe fail or become slow when datasets are larger or equivalent to available RAM. Instead of using a fixed threshold, i have set the code to check if the dataset size is larger than 70% available memory then use dask else pandas.
A config variable can also be set to supersede the normal criteria for determining which dataset to use. If config is set it returns the name of dataset set in the config

Can this be added to issue #550 I'm currently working on? I'm using this new feature to create Dataset instances.

Closing this as it is being addressed in #550


It's necessary if there is an external CI/CD pipeline which is invoked per change in master. @objectiser can provide more info why it is needed.
Some open questions:

should be old SHA tags removed (per number/time)?
publish sha tags to a different repo?

https://docs.travis-ci.com/user/environment-variables/#default-environment-variables
Signed-off-by: Pavol Loffay<EMAIL_ADDRESS>

I'd like to have a separate repo for these so we don't pollute the main repos.

how do we call it jaeger-foo-stream|build|ci|sha?

Just for the record I have tried to push ~600 different tags(with different layers) on docker hub. I was still able to access the first tags.

how do we call it jaeger-foo-stream|build|ci|sha?

jaeger-foo-snapshot ?

sounds good to me, @jpkrohling  @objectiser?

snapshot sounds alright to me.

@pavolloffay snapshot is fine with me


fix: singleton check

What I did
Noticed that the bytecode we were using did not properly match when it was deployed to an alternative network
fixes: #27
How I did it
How to verify it
Checklist

[x] Passes all linting checks (pre-commit and CI jobs)
- [ ] New test cases have been added and are passing
- [ ] Documentation has been updated
[x] PR title follows Conventional Commit standard (will be automatically included in the changelog)

@solarthesis just leave an approval if it all works for you now with this PR


chore: bump sdr version

bump SDR version to get some SDR fixes
@W-111111@

Already included in 151


Trilean

What Boolean is to 2 states (or perhaps 3 if Bool?), Trilean is to 3 states (or perhaps 4 if Trilean?).

:+1:

Since this isn't directly extending an stdlib, UIKit, or Foundation class, I'm not sure this belongs in this framework. Everything in Alexandria extends previous classes, but this introduces a new one, which goes against the "library of Swift extensions" purpose of this framework.


PAYDAY 2 blackscreen for a while, then crashes

Software information
PAYDAY 2 latest version, lowest settings except for:
resolution: 1280x720
use hq weapons
frame rate limit: 60
streaming chunk: 1024
also i used 4gb patcher here, but it doesn't matter which exe i use.
System information

GPU: GTX 950M
Driver: 516.40
Wine version: -
DXVK version: 1.10.1

Apitrace file(s)
https://drive.google.com/file/d/1ERcQSTF8IW2RucWozZvoYA9xJBwz2X4J/view?usp=sharing
Log files

d3d9.log: https://drive.google.com/file/d/1WKDm8nSFCLc1L7AmdC8wwAnc--2UIayj/view?usp=sharing
d3d11.log: -
dxgi.log: -

I haven't been able to reproduce this with my amd R9 380 on windows. It sounds a bit like this issue https://github.com/doitsujin/dxvk/issues/2336 which is wasn't able to reproduce either.
Can you check if the latest master build helps you? You can find a down load of it here. https://github.com/doitsujin/dxvk/actions/runs/2601899023

I haven't been able to reproduce this with my amd R9 380 on windows. It sounds a bit like this issue #2336 which i wasn't able to reproduce either.
Can you check if the latest master build helps you? You can find a download of it here. https://github.com/doitsujin/dxvk/actions/runs/2604934947

its the same blackscreen, then crashing without any error messages. here's the log:
payday2_win32_release_d3d9.log

oh my god, sorry im new to github 🤦

Oh i just noticed according to the log above that your driver version is 398.140.0 not 516.40. then that new version i gave you won't work.
Could you try updating your driver to the latest version? 🙂

Glad to hear it. I wasn't actually convinced that would do it 😁

@JasonSlav Would you be able to test something for me?
Could you try if it also works with this build https://github.com/doitsujin/dxvk/actions/runs/2605265029
This is from the 1.10.x branch for a new point release that doesn't include all the latest master changes. I'm curios if the commit that fixed it is also included there or is unique to the master branch.


how to know properties of a wind turbine EX-ITI barge type  ?

Is your feature request related to a problem? Please describe.

Describe the solution you'd like

Describe alternatives you've considered

Additional context

Dera @jyotrimaya1993,
Your issue comment is blank, but I see your question in the title.  Are you simply asking where to find an OpenFAST model of the NREL 5-MW baseline wind turbine atop the ITI Energy barge?  If so, you can find a model in the OpenFAST r-test: https://github.com/OpenFAST/r-test/tree/main/glue-codes/openfast/5MW_ITIBarge_DLL_WTurb_WavesIrr.  Otherwise, please clarify your question.
Best regards,


PID - Business Case

Includes the background, purpose, required benefits, and proposed solutions

Proposed solutions may need expansion


Converting model to Pytorch

Hi @arashno ,
I need to convert your res152 recognition model to pytorch. I am using mmdnn for conversion, but I cannot find the output node which needs to be given to the mmdnn conversion command. I have visualized the graph using the meta file, and there is no name that looks like output, softmax or predictions.
Can you or anyone else help with the conversion?

Hi,
For the recognition only model, the output node of the ResNet-152 model is the fully connected layer at resnet.py line 53. I think the name of the output node should be: 'output:0'.
For the model with identifying, counting, and additional attributes outputs, the name of the species recognition output node should be 'output0:0'
Best,
Arash

I have tried output:0 with the following command:
mmconvert -sf tensorflow -in snapshot-55.meta -iw snapshot- 55 --dstNode output:0 -df pytorch -om serengeti_resnet_152.pth
but it gives me the error that node is not in the graph.
AssertionError: output:0 is not in graph

Actually, I am going to release a new version of this repo with new features in a few days.
I will check convertibility to PyTorch for the new version.
Please wait until I release the new version.
Thanks,
Arash

Unfortunately, I was unable to convert the Tensorflow models to PyTorch.
But, the good news is that I will release a PyTorch version in 1-2 weeks.
I finished the coding and now I am doing testing and preparing the trained models now.
Arash

Just wondering what became of this?

@arashno : Did you ever release a PyTorch version?

@gullalc : It appears that valid node names for mmconvert include
 'output0/weights/Initializer/truncated_normal/shape'
 'output0/weights/Initializer/truncated_normal/mean'
 'output0/weights/Initializer/truncated_normal/stddev'
 'output0/weights/Initializer/truncated_normal/TruncatedNormal'
 'output0/weights/Initializer/truncated_normal/mul'
 'output0/weights/Initializer/truncated_normal'
 'output0/weights'
 'output0/weights/Assign'
 'output0/weights/read'
 'output0/weights/Regularizer/l2_regularizer/L2Loss'
 'output0/weights/Regularizer/l2_regularizer'
 'output0/biases/Initializer/Const'
 'output0/biases'
 'output0/biases/Assign'
 'output0/biases/read'
 'output0/weights/Momentum/Initializer/Const'
 'output0/weights/Momentum'
 'output0/weights/Momentum/Assign'
 'output0/weights/Momentum/read'
 'output0/biases/Momentum/Initializer/Const'
 'output0/biases/Momentum'
 'output0/biases/Momentum/Assign'
 'output0/biases/Momentum/read'

Hi,
Yes, the PyTorch version is available at https://github.com/arashno/pytorch_multigpu_csv
But, unfortunately, I didn't have enough computational resources to train a full model.
If you have access to enough computational power, I am up to improving the PyTorch repository.


i18n(es): Translated middleware-no-data-or-next-called.mdx

What kind of changes does this PR include?

New or updated content
Translated content

Description

What does this PR change? Give us a brief description.
Translated middleware-no-data-or-next-called.mdx to spanish

Gracias por la revision 🤠


[PAR-356] feat: add lightbulb icon

how do I check that I did this correctly?
The svg I used has the lightbulb right side up but in the docs it is upside down? Should I make this a different icon name? Or change the docs?
https://openedx.atlassian.net/wiki/spaces/BPL/pages/2152661212/Iconography

Coverage remained the same at 87.223% when pulling fabd148a2e800567a10176fd8d83a9914a51c72f on PAR-356 into aa29f96edf30a941b73103906a455d0cf441730a on master.

You can check that you've done it correctly by navigating to this page in the doc site: https://paragon-edx.netlify.app/foundations/icons/ and confirming that the icon you've added is there.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Evaluation code of references is slightly off

There is a subtle known bug in the evaluation code of the classification references (and other references as well, but not all):
https://github.com/pytorch/vision/blob/261cbf7e939a253ea316b74bec8dbba58155ab4e/references/classification/train.py#L65-L66
It deserves some attention, because it's easy to miss and yet can impact our reported results, and those of research papers.
As the comment above describes, when computing the accuracy of the model on a validation set in a distributed setting, some images will be counted more than once if len(dataset) isn't divisible by batch_size * world_size [^bignote].
On top of that, since the test_sampler uses shuffle=True by default, the duplicated images aren't even the same across executions, which means that evaluating the same model on the same dataset can lead to different results every time.
Should we try to fix this, or should we just leave it and wait for the new lightning recipes to handle it? And as a follow-up question, is there a builtin way in lightning to mitigate this at all? (I'm not familiar with lightning, so this one may not make sense.)
[^bignote]: For example if we have 10 images and 2 workers with a batch_size of 3, we will have something like:
```
worker1: img1, img2, img3
worker2: img4, img5, img6
worker1: img7, img8, img9
worker2: img10, **img1, img2**
                  ^^^^^^^^^
 "padding": duplicated images which will affect the validation accuracy
 ```

cc @datumbox

Thanks for raising.
Here are some thoughts:

Reporting slightly biased val stats during the training process is less concerning. I agree that this is concerning when we report final results.
We could change the batch_size to 1 similar to what we do on detection. If that slows down the epochs too much we can try doing it only when --test-only is passed. This way we will be able to report correct stats for our models.
Setting shuffle=False for testing is IMO something worth doing anyway to get more determinism.
We could also try setting drop_last=True on the DataLoader during training. Not sure it improves things by much though.

I think lightning is likely to face similar issues but for now I think it's worth patching batch_size=1 when --test-only and settings shuffle=False on the validation sampler. This can be a good bootcamp task BTW. Let me know what you think.

OK, things are a bit clearer to me now.
As @fmassa's suggested (thanks!) the variance might come from the non-deterministic algorithms that are in use. I set the following:
    torch.backends.cudnn.benchmark = False
    torch.use_deterministic_algorithms(True)
    torch.backends.cudnn.deterministic = True

and now I'm getting consistent results across batch-sizes. I also patched the code to figure out how many samples are effectively processed. Here are the results for different number of processes, all of them using the default batch-size = 32:
Test:  Acc@1 69.762 Acc@5 89.076, 50000 samples processed -- world_size = 1 or 2 or 4 or 5 or 8
Test:  Acc@1 69.760 Acc@5 89.075, 50004 samples processed -- world_size = 6
Test:  Acc@1 69.761 Acc@5 89.074, 50001 samples processed -- world_size = 3 or 7

So:

my previous understanding that we really need to have  len(dataset) % (batch_size * world_size) == 0 is wrong. Sometimes, the DataLoader can reduce the batch size of the last batches so that exactly len(dataset) samples are processed. Setting world_size == 1 as @fmassa suggested above should indeed always process exactly len(dataset) samples, no matter the batch size.
There's still some visible variance in the result across world_sizes * batch_size values, but it's not as high as what the previous analysis in https://github.com/pytorch/vision/issues/4559#issuecomment-939974184 would suggest. That being said, for other dataset sizes the number of duplicated samples may be higher. I might be wrong but I think that in the worst case, we can have at least world_size - 1 duplicated samples. For small dataset sizes this might impact the result quite a bit, but this doesn't matter too much for our datasets.

Thanks both for your input!!
Considering most of the variance is captured by disabling stochastic algorithms as above I would suggest to just set these flags to True if test_only is True, and to keep https://github.com/pytorch/vision/pull/4600 in the back of our mind for the next version of the references / recipes.
I think we could also raise a warning if not exactly len(dataset) samples have been processed, to let the user know that the results might be slightly biased. This would require a small patch like this:

diff --git a/references/classification/train.py b/references/classification/train.py
index a71d337a..de520fb3 100644
--- a/references/classification/train.py
+++ b/references/classification/train.py
@@ -54,6 +54,13 @@ def evaluate(model, criterion, data_loader, device, print_freq=100, log_suffix="
     model.eval()
     metric_logger = utils.MetricLogger(delimiter="  ")
     header = f"Test: {log_suffix}"
+    def _reduce(val):
+        val = torch.tensor([val], dtype=torch.int, device="cuda")
+        torch.distributed.barrier()
+        torch.distributed.all_reduce(val)
+        return val.item()
+
+    n_samples = 0
     with torch.no_grad():
         for image, target in metric_logger.log_every(data_loader, print_freq, header):
             image = image.to(device, non_blocking=True)
@@ -68,7 +75,12 @@ def evaluate(model, criterion, data_loader, device, print_freq=100, log_suffix="
             metric_logger.update(loss=loss.item())
             metric_logger.meters["acc1"].update(acc1.item(), n=batch_size)
             metric_logger.meters["acc5"].update(acc5.item(), n=batch_size)
+            n_samples += batch_size
     # gather the stats from all processes
+
+    n_samples = _reduce(n_samples)
+    print(f"We processed {n_samples} in total")


pinsSetup() typeError

I am having an issue attempting to change the default SER, RCLK & SRCLK pins. I changed the value of the aforementioned pins within the shiftpi.py file as follows. Simple enough.
# Define pins
_SER_pin   = 24    #pin 14 on the 75HC595
_RCLK_pin  = 23    #pin 12 on the 75HC595
_SRCLK_pin = 22  #pin 11 on the 75HC595
And then I  call the pinsSetup() function in my program with the format used in the readme, as follow:
import shiftpi.shiftpi as shiftpi
.
(other code)
.
shiftpi.pinsSetup({"ser": shiftpi._SER_pin, "rclk": shiftpi._RCLK_pin, "srclk": shiftpi._SRCLK_pin}) # that's it!
After that, I receive the following output:
Traceback (most recent call last):
File "modbusTest2.py", line 23, in <module>
shiftpi.pinsSetup({"ser": shiftpi._SER_pin, "rclk": shiftpi._RCLK_pin, "srclk": shiftpi._SRCLK_pin}) # that's it!
TypeError: pinsSetup() takes exactly 0 arguments (1 given)
I'm brand new to python, so I'm having some trouble figuring this one out. If anybody could offer advice as to whether I'm approaching this the wrong way or to just let me know that this is a bug, it would be much appreciated. I can't find an example of anybody using this function(?), so I'm not sure if there is any easier way to do it that I'm just oblivious to or what. Also, not sure if it matters, but I am using Python 2.7 and running with sudo.
Any input is appreciated. Thanks!

shiftpi.pinsSetup(**{"ser": 11, "rclk": 12, "srclk": 13}) seems to work ok


no sunnybrook plugin in Deb packages 14.04

Hello
i can't add sunnybrook plugin to digits
Digits installed from Deb packages ubuntu 14.04

Hello, plug-ins are a new feature in DIGITS 5.0. Deb packages haven't been released for 5.0 yet but you can get it from source there.

Please close if your issue is solved.

thanks!


Prune some unused code + files

Pruning osme unused files and code using knip

Also I'm wondering if maybe the yarn.lock should be ignored? I'm not entirely sure how that might affect the build process


[HOLD for payment 2023-04-13] Editing Workspace Name does not navigate you back to the previous Settings page

Coming from https://github.com/Expensify/App/issues/16768, as it is a very similar issue (but in a different spot)

Action Performed:

Open a workspace
Go to General Settings
Change the room name
Click save

Expected Result:
The name should be updated and the settings modal should close.
Actual result:
The name updates but the settings modal does not close.
Workaround:
None
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number:
Reproducible in staging?:
Reproducible in production?:
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
Issue reported by:
Slack conversation:
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01f751877ab50c048c
Upwork Job ID:<PHONE_NUMBER>784540160
Last Price Increase: 2023-04-04

Triggered auto assignment to @adelekennedy (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Job added to Upwork: https://www.upwork.com/jobs/~01f751877ab50c048c

Triggered auto assignment to Contributor Plus for review of internal employee PR - @Santhosh-Sellavel (Internal)

Removing C+, as it is a very minor fix

Reviewing label has been removed, please complete the "BugZero Checklist".

The solution for this issue has been :rocket: deployed to production :rocket: in version 1.2.95-0 and is now subject to a 7-day regression period :calendar:. Here is the list of pull requests that resolve this issue:

https://github.com/Expensify/App/pull/16903

If no regressions arise, payment will be issued on 2023-04-13. :confetti_ball:
After the hold period is over and BZ checklist items are completed, please complete any of the applicable payments for this issue, and check them off once done.

[ ] External issue reporter
[ ] Contributor that fixed the issue
[ ] Contributor+ that helped on the issue and/or PR

As a reminder, here are the bonuses/penalties that should be applied for any External issue:

Merged PR within 3 business days of assignment - 50% bonus
Merged PR more than 9 business days after assignment - 50% penalty

BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

[ ] [@Gonals] The PR that introduced the bug has been identified. Link to the PR:
[ ] [@Gonals] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:
[ ] [@Gonals] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:
[ ] [@adelekennedy] Determine if we should create a regression test for this bug.
[ ] [@Gonals] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.
[ ] [@adelekennedy] Link the GH issue for creating/updating the regression test once above steps have been agreed upon:

I don't think any payment is due here (or in the related issue) @Gonals am I missing something?

I don't think any payment is due here (or in the related issue) @Gonals am I missing something?

Nope, you are 100% right!


Unrelated paths are diffed relatively

When two completely unrelated paths are diffed, we get back a relative path:
assert_eq!(
    pathdiff::diff_paths("/a/b/c", "/d/e/f"),
    Some("../../../a/b/c".into())
);

This is longer than the input path, although it is relative, but this isn't what I want for displaying a path to a user nicely.
I'd really like an option to disable this behavior — maybe we could introduce a pathdiff::Options type?
Otherwise, if this is the intended behavior, we should document it.

Yes, this is intentional. I don't think "disable this behavior" is as straightforward as it sounds; there's no clear heuristic for when a relative path should be used vs an absolute one: I can think of multiple heuristics and reasons to choose different ones and none of them are universally good. Happy to accept a PR documenting this.

Oh, I actually already added a doctest for this in #11; would you mind cutting a release so it shows up on docs.rs/pathdiff? Thanks so much!

Published 0.2.2. Thanks!


UITest Run with no Error but no screenshots generated

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description
Fastlane snapshot appears to run with no errors but no screenshots are generated.
Complete output when running fastlane, including the stack trace and command used

Sorry but I can't figure out to exactly to use __capture_output. Going by the description above I was expecting this to work.
fastlane snapshot --capture_output
but it generates an error:

invalid option: --capture_output

But here's the tail end of the log file showing that the target ran OK,

snapshot: 20ProgrammerView-Portrait
t =   399.07s Set device orientation to Unknown
t =   399.16s Tear Down
Test Case '-[aioCalculator.aioScreenshots testGenerateScreens]' passed (399.369 seconds).
Test Suite 'aioScreenshots' passed at 2018-03-21 15:07:55.208.
Executed 1 test, with 0 failures (0 unexpected) in 399.369 (399.371) seconds
Test Suite 'aioScreenshots.xctest' passed at 2018-03-21 15:07:55.210.
Executed 1 test, with 0 failures (0 unexpected) in 399.369 (399.373) seconds
Test Suite 'All tests' passed at 2018-03-21 15:07:55.211.
Executed 1 test, with 0 failures (0 unexpected) in 399.369 (399.376) seconds

[INSERT OUTPUT HERE]

Environment

✅ fastlane environment ✅
Stack

Key
Value

OS
10.12.6

Ruby
2.0.0

Bundler?
false

Git
git version 2.16.2

Installation Source
/usr/local/bin/fastlane

Host
Mac OS X 10.12.6 (16G1212)

Ruby Lib Dir
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib

OpenSSL Version
OpenSSL 0.9.8zc 19 Mar 2015

Is contained
false

Is homebrew
false

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
9.2

System Locale

Variable
Value

LANG
en_US.UTF-8
✅

LC_ALL

LANGUAGE

fastlane files:
`./Fastfile`
# This file contains the fastlane.tools configuration
# You can find the documentation at https://docs.fastlane.tools
#
# For a list of all available actions, check out
#
#     https://docs.fastlane.tools/actions
#

# Uncomment the line if you want fastlane to automatically update itself
# update_fastlane

default_platform(:ios)

platform :ios do
  desc "Generate new localized screenshots"
  lane :screenshots do
    capture_screenshots(workspace: "aioCalculator.xcworkspace", scheme: "aioScreenshots”)
    upload_to_app_store(skip_binary_upload: true, skip_metadata: true)
  end
end

`./Appfile`
app_identifier("AppaSoftware.aioCalc") # The bundle identifier of your app
<EMAIL_ADDRESS># Your Apple email address

itc_team_id "*removed*" # iTunes Connect Team ID
team_id "*removed*" # Developer Portal Team ID

# For more information about the Appfile, see:
#     https://docs.fastlane.tools/advanced/#appfile

fastlane gems

Gem
Version
Update-Status

fastlane
2.86.2
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

slack-notifier
2.3.2

atomos
0.1.2

CFPropertyList
2.3.6

claide
1.0.2

colored2
3.1.2

nanaimo
0.2.3

xcodeproj
1.5.6

rouge
1.10.1

xcpretty
0.2.4

terminal-notifier
1.7.1

terminal-table
1.4.5

plist
3.1.0

multipart-post
2.0.0

word_wrap
1.0.0

public_suffix
2.0.5

tty-screen
0.6.4

tty-cursor
0.5.0

tty-spinner
0.8.0

babosa
1.0.2

colored
1.2

highline
1.7.8

commander-fastlane
4.4.6

excon
0.45.4

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20160826

http-cookie
1.0.3

faraday-cookie_jar
0.0.6

fastimage
2.1.1

gh_inspector
1.1.3

mini_magick
4.5.1

multi_xml
0.5.5

rubyzip
1.1.7

security
0.1.3

xcpretty-travis-formatter
0.0.4

dotenv
2.1.1

bundler
1.13.6

faraday
0.9.2

faraday_middleware
0.10.0

naturally
2.1.0

simctl
1.6.2

uber
0.0.15

declarative
0.0.10

declarative-option
0.1.0

representable
3.0.4

retriable
2.1.0

addressable
2.5.2

mime-types-data
3.2016.0521

mime-types
3.1

little-plugger
1.1.4

multi_json
1.12.1

logging
2.1.0

jwt
1.5.6

memoist
0.15.0

os
0.9.6

signet
0.7.3

googleauth
0.5.1

httpclient
<IP_ADDRESS>

google-api-client
0.13.6

libxml-ruby
2.6.0

json
1.8.3

generated on: 2018-03-21

Reinstalled everything, double checked everything and appears to be working fine now.


Read data dimensions directly from files

Make R\diagnostics.R less brittle by reading object size directly from the input and report files rather than hard coding.

Commit 283e836 modifies R/diagnostics.R to read data directly from the hydradata object rather than hard coding row numbers in the text files.
Note I did not make this fix for the sample size plots which aren't working yet anyway.


HA builtin energy usage

Hi,
With HA 2021.8 they implemented energy usage tools, but I cannot link usage from my sonoff POWs devices as they dont appear on the list. In help there is tip to inform integration author about this: https://developers.home-assistant.io/docs/core/entity/sensor#long-term-statistics
Is there anything You can do here?

I can confirm the same with a S31.  I have one unit flashed with Tasmota that is reporting the power data to the Energy integration and on with Sonoff firmware that is not.
I know that in of itself is of little help but please let me know what information or set up/configs you may want me to test out.
My timezone is GMT-8

I am also looking for the same. Monitor energy usage using sonoff firmware.

https://aliexpress.ru/item/4001119010843.html
I have such a relay, it does not have a built-in calculation of the consumed energy, how can I get this data from this device? Mb there is some description of api from ewelink?

@yousaf465 @smartDev22
You need to create a sensor in HA via a template.
In your sensor.yaml create the sensor from the Sonoff device:

platform: integration
source: sensor.sonoff_10003cc37b_power
name: IT Power Manual
unit_prefix: k
unit: kWh
round: 2

That creates a kWh sensor you can either place into the UI as usual and/or the new Energy dashboard.
Devices such as these Sonoff's maybe updated via firmware updates later to show the information to HA in the way HA wants but it's unlikely

Additionally, if your device only gives the Watts or Load Power or Power as an attribute then you can use this code in sensor.yaml to make that attribute into a sensor for Watts/Power then then use the code above.

platform: template
sensors:
water_filter_watts:
device_class: power
unit_of_measurement: 'W'
friendly_name: 'Water Filter Watts'
value_template: "{{ states.switch.xiaomi_miio_switch.attributes.load_power }}"

@johnjoemorgan
I have done what You wrote:
sensor:
- platform: integration
  source: sensor.sonoff_10007c87fb_power
  name: Dishwasher
  unit_prefix: k
  unit: kWh
  round: 2

- platform: template
  sensors:
    power_dishwasher:
      friendly_name: DIshwasher
      unit_of_measurement: W
      device_class: power
      value_template: "{{ state_attr('switch.sonoff_10007c87fb', 'power') }}"

and still no statistics available in step 3 of energy dashboard.

@johnjoemorgan
I have done what You wrote:
sensor:
- platform: integration
  source: sensor.sonoff_10007c87fb_power
  name: Dishwasher
  unit_prefix: k
  unit: kWh
  round: 2

- platform: template
  sensors:
    power_dishwasher:
      friendly_name: Dishwasher
      unit_of_measurement: W
      device_class: power
      value_template: "{{ state_attr('switch.sonoff_10007c87fb', 'power') }}"

and still no statistics available in step 3 of energy dashboard.

Add:
state_class: measurement

@Gferretta it can take up to an hour for data
End of the day here fir me (UTC-8) Will compare you code asap

Added       state_class: measurement
sensor:
- platform: integration
  source: sensor.sonoff_10007c87fb_power
  name: Dishwasher
  unit_prefix: k
  unit: kWh
  round: 2

- platform: template
  sensors:
    power_dishwasher:
      friendly_name: Dishwasher
      unit_of_measurement: W
      device_class: power
      state_class: measurement
      value_template: "{{ state_attr('switch.sonoff_10007c87fb', 'power') }}"

Now in HA > Settings > Server configuration > Check configuration I have:
Invalid config for [sensor.template]: [state_class] is an invalid option for [sensor.template]. Check: sensor.template->sensors->power_pralka->state_class. (See ?, line ?).

since I have created a energy sensor before hand, all I had to do was change the entity type. https://community.home-assistant.io/t/integration-solar-inverter-huawei-2000l/132350/589?u=yousaf465
`Thanks for the tip. I was able to add Sonoff Pow2 using Alexxit’s SonoffLan control to Energy dashboard. GitHub - AlexxIT/SonoffLAN: Control Sonoff Devices with eWeLink (original) firmware over LAN and/or Cloud from Home Assistant 2
Steps are simple
configuration
Scroll down to Customizations
select your entity
select device class as “energy”
select state class as " measurement"
Select other
Attribute name “last_reset”.
8 attribute value “1970-01-01T00:00:00+00:00”
click save
Now your sensor will show up in energy dashboard.`

@guciopl Here's the first part of my sensor.yaml
'
Date Time

platform: websocket_api

platform: time_date
display_options:

'time'
'date'

platform: template
sensors:
water_filter_watts:
device_class: power
unit_of_measurement: 'W'
friendly_name: 'Water Filter Watts'
value_template: "{{ states.switch.xiaomi_miio_switch.attributes.load_power }}"

platform: integration
source: sensor.tyxxxxxxxxxxxxxde_2
name: Desk Power Manual
unit_prefix: k
unit: kWh
round: 2

platform: integration
source: sensor.sonoff_10003cc37b_power
name: IT Power Manual
unit_prefix: k
unit: kWh
'

:-) Maybe the order in which the "software" sensors appear in the file makes a difference.  My template appears before the integration??????????

I dont understand Your code, I dont see those sensors to be linked by no means.
In my code both sensors are linked to same device, in Yours we have 3 devices.
Anyway I managed to create software sensors which are visible in HA energy tab, however they are all empty, no power usage is transfered from hardware sensors to software sensors.

@guciopl  Sorry I may have over complicated things in my example.
For my Sonoff devices the only code is in sensor.yaml:

platform: integration
source: sensor.SONOFF-ENTITY_POWER
name: NEW ENTITY FOR ENERGY
unit_prefix: k
unit: kWh
round: 2

You can find your sensor.SONOFF-ENTITY_POWER from your entities list and replace the one above.
After checking your config and restarting you should see a new sensor "NEW ENTITY FOR ENERGY" or whatever you have nemaed it.
This is the entity you can add to the energy dashboard or anywhere in the HA UI.
Note it may take up to an hour for any data to show in the new entity.  So please be patient :-)
Hope this help and sorry for the confusion.

Well I don't have S31, I have POWs R2 only, but added it to sensor.yaml same as You wrote:

and no sensor in energy tab:

Well I don't have S31, I have POWs R2 only, but added it to sensor.yaml same as You wrote:

and no sensor in energy tab:

you have to change the state class and device class too along with last reset attribute value.

you have to change the state class and device class too along with last reset attribute value.
those entiry doesnt take that attributes

I moved sensors back from sensor.yaml to configuration.yaml and now they are possible to pick in energy tab but not populating anything, will wait some time

Added energy sensor for pow devices in latest master-version. Will be in next release.
Data loaded from the cloud every hour.

Is it beneficial to assign a unique id to energy sensors? I noticed when I do - it shows up in the Entities list as "Riemann sum integral" rather than just "Sensor"?


RFE: Airgapped RKE2 etcd to AWS S3 via proxy

Request description:
We are trying to backup our ETCD cluster to AWS S3 from within out network. To do so requires that action to go through a proxy.
While the Rancher-Backup chart has the ability for proxy/noproxy configuration for the deployment only, there does not appear to be a way to do that with the ETCD backup's only as they is at RKE2's level and not a specific backup deployment/job/container.
Adding the noproxy/proxy to the RKE2 level would not be an option as kubelet would then be able to talk externally and that is restricted in our clusters.
Actual behavior:
Grant kubelet access via proxy to get S3 backups placed offsite
Expected behavior:
Grant etcd backup process access to proxy to push backups offsite
Workaround:
None

@brandond Sorry if my example on how the functionality could be addressed for specifically Rancher Provisioned Systems gave you the feeling that you could pass this feature request to another team, but the issue still exists for native RKE2 servers with no Rancher Management.
This should be tackled in RKE2 first or entirely to ensure compatibility with Rancher after. The fact of the S3 process being defined directly in the RKE2 binary is an issue. It could instead be handled using the post process in RKE2 when it generates the rke2-etcd-snapshot configmap. At that time there could be an operator or something in RKE2 that monitors the snapshot and provides the S3 upload/download capabilities. This will still allow the original functionality to be available for DR purposes.
The current solution of the RKE2 binary being allowed to restore S3 snapshots and force push them to S3 is fine, but the secret management and flexibility is still not what our systems require.

Not stale. Please re-open

Its not closed...

The K3s ADR supporting this has been open for a while; we need someone from Hostbusters to take a look and sign off on it.

Not stale, pending acceptance of ADR by Rancher teams.

Not stale, waiting implementation

See above linked issues for implementation. Closing issue as completed.


Wrong ovr library path in install instructions

Building instructions
http://gearvrf.org/bin/view/GearVRfDeveloperGuide/GearVRfDevGuide200Start tell to copy Oculus Mobile SDK libraries to  GVRf/Framework/libs/. This resulted built applications failing in startup because of missing class, at least when building with Eclipse.
Copying libraries to GVRf/Framework/framework/src/main/libs/ instead made it work.

Thank you for catching this. We will have it fixed.

Ok, I updated it.  Take a look now.  Is anything else off?  Thanks!

@lassehe Can we close this now?

Thank you. Yes, the documentation looks now correct. Path Framework/framework/ feels a bit redundant though, but I guess there is some reason for that.

Agreed about the redundancy. The 'framework' is a gradle module under the project root 'Framework'. Make the root 'GVRf'? Any suggestions? Thanks.

Right, the second 'framework' is the module name. And, we have a few other items under 'GVRf' umbrella already.
How about ‘GearVRf-Frameworks’? Any other suggestions?


Create a form on the site without adding a subject to main menu

Hi,
Thanks for a great theme! I am constructing a website for my father who has written a book that he would like to sell through his site. The site is here. Now I would like to make it possible for people to order his book on his website by entering their names, emails, postal addresses, how many copies of the book they would like + the possibility to write a message. The issue is, where and how do I write the required form code? I am open to using Google forms or Netlify forms or any form as long as the form is embedded in the overall design of the site (using the same fonts, colours and styles). However, I would NOT like to include another main meny choice called 'Order book'. Rather I would like to have an 'Order book' link or buttom on the front page as well as under the 'Boken' tab. Where would I enter the form code then? I am completely new to creating forms on websites so please be gentle. Thanks in advance! /Joakim

Hi @jfrogren,
The following articles describe how to add the form to the page:
https://un-static.com/how-to/add-hugo-contact-form/
https://formkeep.com/guides/contact-form-hugo


Change the recipient parameter to "chatId" to also support group chats

solves issue #65
In the Blockly element, change the recipient parameter  in the Javascript from "user" to "chatId" in order to also support group chats.
This will NOT break and NOT change  the current behavior, but will add an extra feature.
No additional documentation needed.

As my change is in the Blockly only... is there a way to send to multiple users in the Blockly "sendTo"?
It's not in the documentation there, is it?


First draft of the doc on the import of ESLint reports into SonarQube

Documentation on how to use the ESLint and SonarQube Scanner CLIs to generate ESLint reports and load them into SonarQube.

Hello @lmarniazman, thank you for your contribution.
I put some comments that need to be resolved before merge your changes.

@lmarniazman can you add also the link to that file in master-devon4node.asciidoc, Home.asciidoc and _Sidebar.asciidoc? The right place to put that should be at the end of guides section. Thank you so much.

@dario-rodriguez Thanks a lot for the feedback. I adjusted the guide, but I am not sure if the additional parameter for SonarQube inside the Jenkinsfile is correct / needed at all. Do you have any info on that?


getJSSyntacticDiagnosticsForFile: minor cleanup

ConstKeyword is not specific to TypeScript syntax, therefore it doesn't need to be checked there.
This function doesn't abort if cancellation is requested, so it doesn't need to be wrapped in runWithCancellationToken. And even if it used the cancellation token, the checker wouldn't be in an invalid state.
This change is best viewed without whitespace changes: https://github.com/Microsoft/TypeScript/pull/29208/files?w=1

@sandersn please verify

This PR hasn't seen any activity for quite a while and is blocked on #29213, so I'm going to close it to keep the number of open PRs manageable. Feel free to open a fresh PR or continue the discussion there.


Make it easier to tell if an error is due to timeout

Right now we bury the timeout error in an io::Error with a different inner kind depending on the platform. It would be nice to be able to tell whether an error is from a timeout (so we could retry/reschedule the request) without having to know that it's captured in an io::Error variant.
What do you think about adding an is_timeout method to reqwest::Error to do this? We could either look for any captured io::ErrorKind::WouldBlock/io::ErrorKind::TimedOut and return true. Or we could add an extra variant to reqwest::error::Kind so only errors from Waited are considered timeouts.

Seems like a reasonable desire. I remember raising an issue on the Rust repo about how it would be nice to consolidate a timeout error into a single ErrorKind, but back then it was declared better to report exactly what the OS declared.
With that in mind, I'm not certain (as in I'm neutral, not negative) if we should collect these errors into a higher reqwest Kind or not. I also have a tiny worry that translating a WouldBlock into a timeout without thought could have misleading errors when using async...

Yeh, I was wondering about the async + WouldBlock case too... I think it would probably, usually, approximately be about right to treat WouldBlock in the reqwest::Error as a timeout, since in the futures machinery we communicate readiness through Async, or through the io::Error rather than through reqwest::Error. So if I get a hold of a reqwest::Error I'd assume whatever future produced it is probably dead and can't be polled again.

There's now a Error::is_timeout.


Is "position: absolute" required on  elements ?

was struggling to get rn-carousel working correctly
with the slide positions being all out of wack - until i stumbled across this issue:
is position: absolute required on <li> elements ?
or do i have a bug somewhere ?
...or am I not understanding something basic about css :-/
Nothing mentioned in the (rather scant) documentation...
tried this on the following version ranges with the same result:

rn-carousel versions 0.3.5 - 0.3.9
angularjs versions 1.3.0 - 1.3.12

Plunker here showing carousel with and without absolute positioning :
http://plnkr.co/edit/HDp7yfK75gCZUpokaEEZ
Note the gaps in the non position:absolute carousel due to the display: inline-block within an ng-repeat loop
StackOverflow question here if someone wants the points:
http://stackoverflow.com/questions/28373275

Hi
Did you try with including the rn-carousel.css ?
https://github.com/revolunet/angular-carousel/blob/master/dist/angular-carousel.css

thanks for the reply - no I didn't - since we have a bunch of custom css within the carousel we don't want changed...
Is this directive is dependent on an external css file style definition to work correctly ?

the basic positionning/layout is defined in this css and should be namespaced and overridable

BTW since it's only 1 style required to get the carousel working properly, I'm a little surprised the directive doesn't simply add an element.style of position: absolute on the li as it does with display: inline-block; - which is added manually by the directive, even though it's also included in the CSS class you list above - that would simplify things and allow users to add the CSS if they thought it was useful and/or necessary...


make install DESTDIR not honored

make DESTDIR=$HOME/temp/stage3/ install
[INSTALL] Installing public C headers
install: cannot create directory ‘/usr/local/include/grpc’: Permission denied
make: *** [install-headers_c] Error 1

DESTDIR seems ignored

Setting prefix instead of DESTDIR should do what you want.


CAMEL-15557: Stream Cache file not deleted

If you have a route with a Multicast with parallel processing and a
timeout and a sub-route in the multicast which is creating an
OutputStreamCache before the timeout and is writing to the
OutputStreamCache after the timeout then the created file is never
deleted from the file system.
[ ] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] If you're unsure, you can format the pull request title like [CAMEL-XXX] Fixes bug in camel-file component, where you replace CAMEL-XXX with the appropriate JIRA issue.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Run mvn clean install -Psourcecheck in your module with source check enabled to make sure basic checks pass and there are no checkstyle violations. A more thorough check will be performed on your pull request automatically.
Below are the contribution guidelines:
https://github.com/apache/camel/blob/master/CONTRIBUTING.md

I dont think is correct to throw an exception as the business process would work.
The current behaviour should be as today, and what can be done is to add an option on that cache strategy where you can set the spool directory, to have a new boolean option you can turn on which in this rare use-case can then cause an exception instead.


[test] Refactor the ssl transport security test

Looks great! Going to go on and approve so I'm not blocking once the tests are working

The msan passed. All the remaining tests seem to be flaky lately.


fix codeFence linebreak missing when convert to markdown

@aimane-chnaif @deetergp
Fixed Issues
$ https://github.com/Expensify/App/issues/15302
Tests

Send this markdown: \n\n# test\n\n as new message

```

# test

```

Verify that converted html is

# test

Edit this message
Verity that converted markdown is \n\n# test\n\n which is same as original one in Step 1.
Add one more line break at the beginning - \n\n\n# test\n\n and update message

```

# test

```

Verify that converted html is

# test

Edit this message
Verity that converted markdown is \n\n\n# test\n\n which is same as original one in Step 5.

QA
Same as Tests step
https://user-images.githubusercontent.com/108292595/225628295-6b7f5b5a-b1dd-4612-8d00-b4840f65084a.mov

I have read the CLA Document and I hereby sign the CLA

For safety, I pulled from latest main.
@deetergp bump for review

@PauloGasparSv you should check out the GH, @situchan gave a very excellent explanation of the regex change 👍


Fix permissions bug in UserUpdate

We use the UserUpdate class in Account::RoleOrganisationsController to update the user's role and organisation. We're not passing supported_permission_ids in the params parameter to UserUpdate#initialize because we're not interested in changing a user's permissions in these operations. Unfortunately it turns out that UserUpdate expects to have the user's existing permissions passed in if you want them to remain unchanged. Otherwise UserUpdate#filtered_user_params will reset/clear all permissions for that user.
I've chosen to fix this in UserUpdate#filtered_user_params because it seems reasonable to me that leaving supported_permission_ids out of params should be interpreted as meaning that supported permissions shouldn't be changed.
Note that this bug would've caused superadmins to lose their permissions if they changed their role or organisation, and admins to lose their permissions if they changed their organisation.
I've also updated UserUpdate to record an EventLog when a user's organisation is changed. We appear to record an event when anything else is changed by UserUpdate so it seems reasonable that we should also record an event when the organisation is changed.

Thanks, @floehopper :+1:


FileNotFoundError: [Errno 2] No such file or directory: './data/scannet/planes_9/normal_anchors.npy'

FileNotFoundError: [Errno 2] No such file or directory: './data/scannet/planes_9/normal_anchors.npy'
I was trying to run the demo with demo dataset.
python demo.py --cfg ./config/demo.yaml
But I got the above error. Since I was not trying to use scannet, this should not be a problem,right?

Oops, yes actually the normal_anchors.npy is generated in the process of generating scannet gt. If you didn't run data preparation for ScanNet, you need to download normal anchors, and change the NORMAL_ANCHOR_PATH in demo.yaml. Thanks!

Much appreciated!

Oops, yes actually the normal_anchors.npy is generated in the process of generating scannet gt. If you didn't run data preparation for ScanNet, you need to download normal anchors, and change the NORMAL_ANCHOR_PATH in demo.yaml. Thanks!

Much appreciated!


Fix/mobile menu

This PR fixes and applies the UI kit design to the mobile nav.

The share button is removed as the designs don't have it anymore.
The share modal was extracted for consistency

Ready again!

checking :eye: :eye:

lts-explore page doesn't work:

Changing selected tab under the NDC comparison page removes the highlight from the menu:

About climate watch page is always highlighted when selecting other pages from the About section:

Ready again!


Error in exporting mesh and video when applying official examples

similar as closed issue #1, but I did not find an answer.
when I try the official example exporting videos, the videos are blank
python -m lrm.inferrer --model_name openlrm-base-obj-1.0 --source_image ./assets/sample_input/owl.png --export_video
In the videos I output, it shows a white image with a faintly visible object rotating in the center, and the object is very unclear with the white background. I have tried this on both 2080ti and 3090, also used the openlrm-small-obj-1.0 model and other example images, but the results are similar.
when I try the official example exporting mesh the vertices are empty.
python -m lrm.inferrer --model_name openlrm-base-obj-1.0 --source_image ./assets/sample_input/owl.png --export_mesh
lrm/models/rendering/synthesizer.py", line 189, in forward_points
for k in outs[0].keys()
IndexError: list index out of range
I looked at the result after executing forward_planes in line 165 inferrer.py.

and result after excuting model.synthesizer in line 157 inferrer.py.，in which the images_rgb values for each frame seem to be abnormal.

How can I fix it and perform similar as official example?

Hi Mengjiao,
We haven't encountered such issue in our environment. If you are still seeing this problem, could you plz share your environment versions and other useful information in the log?

Dear authors,
Thank you for the reply and the update! I changed my cuda version to 11.8 and everything works now.
Previous environment cuda11.3, torch 2.2.0
Now environment cuda 11.8, torch 2.2.0

Dear authors,
Thank you for the reply and the update!
I changed my cuda version to 11.8 and use the updated code, everything
works perfectly now.
Previous environment cuda11.3, torch 2.2.0, on 3090
Now environment cuda 11.8, torch 2.2.0, on 3090
I guess the reason is cuda version too old, leading to some error in
forward and wrong "plane" feature value(no error message though).
Thank you for your great work and the following update again!
Best,
Mengjiao
Zexin He @.***> 于2024年3月5日周二 14:04写道：

Hi Mengjiao,
We haven't encountered such issue in our environment. If you are still
seeing this problem, could you plz share your environment versions and
other useful information in the log?
—
Reply to this email directly, view it on GitHub
https://github.com/3DTopia/OpenLRM/issues/20#issuecomment-1978026856,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUKV2TN6TUXBLGU2G4MXYHTYWVN6JAVCNFSM6AAAAABD6CVDI6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZYGAZDMOBVGY
.
You are receiving this because you authored the thread.Message ID:
@.***>


Style guide

Bug Report
Different code styles are used in the library. A style guide should be created for the developers (including us) to follow to make the formatting more consistent in Java Suite.
Expected Behavior
Consistent code styling through the library.
Observed Behavior
Code styling is inconsistent through the library.
OS & Version Information

SDL Android Version: 4.11.0

Updated the wiki.


publish: v0.2.4-alpha

Pull request
https://github.com/Azure/azure-cli-extensions/pull/6495

Merged by extension team


Issue with OpenImageIO.BASETYPE in python3

Describe the bug
Sorting BASETYPEs causes a TypeError exception in python3
To Reproduce
Steps to reproduce the behavior:

execute the python 3 interpreter
run the snippet below

import OpenImageIO as oiio; sorted([oiio.BASETYPE.UCHAR, oiio.BASETYPE.UCHAR])

You will see the error below:

  File "<stdin>", line 1, in <module>
TypeError: '<' not supported between instances of 'OpenImageIO.BASETYPE' and 'OpenImageIO.BASETYPE'

Expected behavior
No TypeError should be thrown for the given snippet
Evidence

Platform information:

OIIO branch/version: 2.2.1
OS: OSX and Linux
C++ compiler:
Any non-default build flags when you build OIIO:

I'm admittedly not a Python expert (I can get around for writing small programs, but I don't use it myself for large-scale software).
But it looks here https://docs.python.org/3/library/enum.html#comparisons like this is not expected to work:

Ordered comparisons between enumeration values are not supported.

I note also that you can sort the integer values by casting:
# works:
sorted([int(oiio.BASETYPE.UCHAR), int(oiio.BASETYPE.FLOAT)])

# generates exception:
sorted([oiio.BASETYPE.UCHAR, oiio.BASETYPE.FLOAT])

Do the experienced Python programmers out there tend to expect that our enums should allow comparison without the cast?

In general, I wouldn't really expect Python enums to support any meaningful arithmetic unless they are enums of flags.
If the values are arithmetically comparable, there's a bit of a danger in someone assuming that, because (most of) the BASETYPE enum labels describe a clearly-sized type, the comparison is based on the type's bit size rather than the enum value (e.g. they might consider it a bug that oiio.BASETYPE.INT64 < oiio.BASETYPE.HALF == True), but that can easily be considered user error.
The comparisons do currently work when using Python 2, so maybe this really comes down to a question of correctness vs. consistency? Personally, I don't feel like changing the current Python 3 behavior would be particularly valuable, and I think the reduced ambiguity is good.

Thanks, @nrusch, that confirms my intuition. I'm going to close this issue.

Cheers @lgritz and @nrusch for looking into it.


CLI should set :mkdirs option by default (#1241)

Thank you for opening a pull request and contributing to AsciidoctorJ!
Please take a bit of time giving some details about your pull request:
Kind of change

[x] Bug fix
[ ] New non-breaking feature
[ ] New breaking feature
[ ] Documentation update
[ ] Build improvement

Description
What is the goal of this pull request?
Make the CLI set the :mkdirs option.
How does it achieve that?
Similar to the implementation of Asciidoctor it sets the option :mkdirs if the input is not read from stdin.
Are there any alternative ways to implement this?
Are there any implications of this pull request? Anything a user must know?
Issue
Fixes #1241

@mojavelinux This PR should be a minimal solution for adding the mkdirs option as reported in #1241
It would generate all results directly into the target directory though, so it would at most create that target directory, but not any structure below that which would reflect the path of the documents relative to the source directory set with -R.

That's fine because this is orthogonal to supporting **/*.adoc. Most uses of the CLI are to convert a single document to a specified output file or directory. That is what is needed at a minimum in order for AsciidoctorJ to be used with the Antora PDF extension.

:tada:


Amounts do not appear in transaction histories

Turkish translations are attached, when will they be active on the website?

@ahmetsagirli
This issue was resolved


sysstat: 12.3.2 -> 12.3.3

Semi-automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/sysstat/versions.
meta.description for sysstat is: "A collection of performance monitoring tools for Linux (such as sar, iostat and pidstat)"
meta.homepage for sysstat is: "http://sebastien.godard.pagesperso-orange.fr/"
meta.changelog for sysstat is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 12.3.3 with grep in /nix/store/nsbapalx3h6qv4z40v7lzbwl8x6fy85a-sysstat-12.3.3

directory tree listing: https://gist.github.com/546b1215529df08431e84c6dcb5d020a

du listing: https://gist.github.com/7eadd3efbf563f076579bbd7fd8eb155

Rebuild report (if merged into master) (click to expand)

376 total rebuild path(s)

134 package rebuild(s)

134 x86_64-linux rebuild(s)
120 i686-linux rebuild(s)
2 x86_64-darwin rebuild(s)
120 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
bareos
ceph
ceph-client
ceph-dev
collectd
collectd-data
deepin.dde-api
deepin.dde-control-center
deepin.dde-daemon
deepin.dde-dock
deepin.dde-file-manager
deepin.dde-kwin
deepin.dde-launcher
deepin.dde-session-ui
deepin.deepin-desktop-base
deepin.deepin-desktop-schemas
deepin.deepin-wallpapers
deepin.startdde
docker-machine-kvm
docker-machine-kvm2
easysnap
gnome3.gnome-boxes
grub2
grub2_efi
grub2_full
grub2_pvgrub_image
grub2_xen
haskellPackages.libvirt-hs
haskellPackages.libzfs
i3blocks-gaps
libceph
libguestfs
libguestfs-with-appliance
libvirt
libvirt-glib
libvirt_5_9_0
libvmi
linuxPackages-libre.zfs
linuxPackages-libre.zfsStable
linuxPackages-libre.zfsUnstable
linuxPackages.zfs
linuxPackages.zfsStable
linuxPackages.zfsUnstable
linuxPackages_4_14.zfs
linuxPackages_4_14.zfsStable
linuxPackages_4_14.zfsUnstable
linuxPackages_4_19.zfs
linuxPackages_4_19.zfsStable
linuxPackages_4_19.zfsUnstable
linuxPackages_4_4.zfs

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/nsbapalx3h6qv4z40v7lzbwl8x6fy85a-sysstat-12.3.3 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A sysstat https://github.com/r-ryantm/nixpkgs/archive/8e0c72c6845ff61f401331638205daffc6ffb1bc.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/nsbapalx3h6qv4z40v7lzbwl8x6fy85a-sysstat-12.3.3
ls -la /nix/store/nsbapalx3h6qv4z40v7lzbwl8x6fy85a-sysstat-12.3.3/bin

Pre-merge build results
NixPkgs review skipped

Maintainer pings
cc @edolstra for testing.

This updates systat to the development version so we might just wait for the next stable.

12.4.0 is up on repology


Support crystal tools format

I have instaled crystal 0.26 but it does not support crystal formatting built in tool, I have it but it is not working

Hi @mrkaspa
Currently formatting is failing because some changes in crystal itself, see #80
I'm working on a quick fix, also future major version will use Scry :+1:


Support setting reserved concurrency for warmup lambdas

User story:
As a developer whose organization requires that all lambdas define a reserved concurrency amount, I want to set a reserved concurrency amount on the lambdas managed by the WarmUp plugin, so that I can keep my lambdas warm, while following my organization's rules.

Duplicates #201
If you think that it does not duplicate it, feel free to reopen with more details 🙂