Unable to write file due to EACCES: permission denied

I cannot write a file in subdirectory of a repository, but writing files under the repository is OK.
I enabled docker rootless.
Behavior of the bug
Screencast from 2023-05-04 01-10-34.webm
Environment
No changes in /etc/docker/daemon.json or ~/.config/docker/daemon.json.
Operating System: Ubuntu 23.04
Docker info
Client:
 Context:    default
 Debug Mode: false
 Plugins:
  buildx: Docker Buildx (Docker Inc.)
    Version:  v0.10.4
    Path:     /usr/libexec/docker/cli-plugins/docker-buildx
  compose: Docker Compose (Docker Inc.)
    Version:  v2.17.3
    Path:     /usr/libexec/docker/cli-plugins/docker-compose

Server:
 Containers: 0
  Running: 0
  Paused: 0
  Stopped: 0
 Images: 0
 Server Version: 23.0.5
 Storage Driver: overlay2
  Backing Filesystem: extfs
  Supports d_type: true
  Using metacopy: false
  Native Overlay Diff: false
  userxattr: true
 Logging Driver: json-file
 Cgroup Driver: systemd
 Cgroup Version: 2
 Plugins:
  Volume: local
  Network: bridge host ipvlan macvlan null overlay
  Log: awslogs fluentd gcplogs gelf journald json-file local logentries splunk syslog
 Swarm: inactive
 Runtimes: io.containerd.runc.v2 runc
 Default Runtime: runc
 Init Binary: docker-init
 containerd version: 2806fc1057397dbaeefbea0e4e17bddfbd388f38
 runc version: v1.1.5-0-gf19387a
 init version: de40ad0
 Security Options:
  seccomp
   Profile: builtin
  rootless
  cgroupns
 Kernel Version: 6.2.0-20-generic
 Operating System: Ubuntu 23.04
 OSType: linux
 Architecture: x86_64
 CPUs: [Hidden]
 Total Memory: [Hidden]
 Name: piqua-Desktop
 ID: [Hidden]
 Docker Root Dir: /home/piqua/.local/share/docker
 Debug Mode: false
 Registry: https://index.docker.io/v1/
 Experimental: false
 Insecure Registries:
  <IP_ADDRESS>/8
 Live Restore Enabled: false

WARNING: No cpuset support
WARNING: No io.weight support
WARNING: No io.weight (per device) support
WARNING: No io.max (rbps) support
WARNING: No io.max (wbps) support
WARNING: No io.max (riops) support
WARNING: No io.max (wiops) support
WARNING: bridge-nf-call-iptables is disabled
WARNING: bridge-nf-call-ip6tables is disabled

Context of the devcontainer.json:
Click to view
{
  "name": "Node.js & TypeScript",
  "image": "mcr.microsoft.com/devcontainers/typescript-node:0-20",
  "features": {
  "ghcr.io/supermatt/devcontainer-features/uid-gid-fixer:latest": {},
  "ghcr.io/devcontainers/features/node:1": {
    "nodeGypDependencies": true,
    "version": "lts"
  },
  "ghcr.io/devcontainers/features/rust:1": {
    "version": "latest",
    "profile": "default"
  },
  "ghcr.io/devcontainers-contrib/features/bash-command:1": {
    "command": "echo hi!"
  },
  "ghcr.io/devcontainers-contrib/features/npm-package:1": {
    "package": "typescript",
    "version": "latest"
  }
  }
}

Permission of each directory
Host

Container

Please use ls -la to check the permissions. (id shows the id and groups of the current user)
The folder might me owned by root in the container and your user on the local machine (rootless does the mapping).

Here you are:
Host

Container

Thanks you. Your suggestion works.

@chrmarti is there also a solution when running with a non-root container user like "remoteUser": "vscode" (where vscode has UID 1000 for example)?


Fetching examples from master causes errors during release

Describe the bug
because of
const SPEC_EXAMPLES_ZIP_URL = 'https://github.com/asyncapi/spec/archive/refs/heads/master.zip';

that is in https://github.com/asyncapi/cli/blob/master/scripts/fetch-asyncapi-example.js#L9 script, we will always have few hours during spec release where CLI will fail in development, on ci/cd. It is because it is fetching stuff from master, that was already released, like now, we released spec 2.3. But CLI is not updated with parser that supports 2.3 cause it is not yet released
so we need to use:
const SPEC_EXAMPLES_ZIP_URL = 'https://github.com/asyncapi/spec/archive/refs/tags/v2.2.0.zip';

but then it means we have to have a manual bump of CLI, unless we improve the script to somehow check what version it should fetch. Like maybe use parser before fetching, to check what is the latest version of the spec that it is using 🤔

Hey @derberg I hope this fix works :  #213

Hey @derberg
The latest release of cli (v0.13.0 ), supports parser version  1.14.0 which supports the spec version 2.3.0
If we are following this pattern.

parser
spec

1.13.x
2.2.x

1.14.x
2.3.x

Using this pattern, MAJOR.MINOR.PATCH it seems
for every  MINOR update in spec,  there is MINOR update required for parser.
And if we stay consistent with this approach, we can implement the script in that logic.

@imabp do you plan to continue working on this one? https://github.com/asyncapi/cli/pull/213


NBC/ThePlatform Extractor "HTTP Error 403: Forbidden" With Some Videos.  Unrecognized DRM?

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting that yt-dlp is broken on a supported site
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[ ] I've read about sharing account credentials and I'm willing to share it if required

Region
United States of America
Provide a description that is worded well enough to be understood
The NBC/ThePlatform extractor seems to fail on some NBC/SyFy movies with "HTTP Error 403: Forbidden" errors apparently related to invalid video URLs and no HLS formats found, so it thinks only images are available.  Maybe there has been a modest change to some movie pages for some reason and a small update to the extractor will fix it.  Otherwise, it might be that some movies are being DRM'ed and the current logic in the extractor(s) isn't identifying that situation.  My example of a failure case is from the movie "The Terminator".  I thought it might be a global change of the site, or something impacting just the SyFy movies, but things still worked with an older SyFy movie that had downloaded fine before ("Caved In: Prehistoric Terror").  For comparison, I included the output from that successful case in the second half of the verbose output below.
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Command-line config: ['-v', 'https://www.nbc.com/syfy-movies/video/the-terminator/4053509']
[debug] Encodings: locale cp1252, fs utf-8, pref cp1252, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp-nightly-builds [e59c82a74] (win_exe)
[debug] Python 3.8.10 (CPython AMD64 64bit) - Windows-10-10.0.19045-SP0 (OpenSSL 1.1.1k  25 Mar 2021)
[debug] exe versions: none
[debug] Optional libraries: Cryptodome-3.20.0, brotli-1.1.0, certifi-2024.08.30, curl_cffi-0.5.10, mutagen-1.47.0, requests-2.32.3, sqlite3-3.35.5, urllib3-2.2.3, websockets-13.1
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests, websockets, curl_cffi
[debug] Loaded 1838 extractors
[NBC] Extracting URL: https://www.nbc.com/syfy-movies/video/the-terminator/4053509
[NBC] 4053509: Downloading JSON metadata
[NBC] 4053509: Downloading JSON metadata
[debug] Loading ap-mvpd.nbcentertainment from cache
[NBC] 4053509: Retrieving Media Token
[ThePlatform] Extracting URL: http://link.theplatform.com/s/NnzsPC/media/guid/2304990266/4053509?mbr=true&manifest=m3u&switch=HLSServiceSecure&auth=%3CsignatureInfo%3EQEjscQpT5uNpzB7uTsQgpfPhrfOl5Fbm2pNOqQa1x3WX9UZ9l0ozV7BxQ3g0HE4fVGcl87ymH8g9Svegfbpki14HZ%2BDWkdIjmMQE6K59BZrM9B%2FrBhdWOSb%2FoMWZr3QX72P7vEYVvbFAgh3qnlXCPmdnYrKw8U%2B%2B9KyLxHYHccE%3D%3CsignatureInfo%3E%3CauthToken%3E%3CsessionGUID%3E90089f8249a9c90a7a87345572d92521%3C%2FsessionGUID%3E%3CrequestorID%3Enbcentertainment%3C%2FrequestorID%3E%3CresourceID%3E%3C%21%5BCDATA%5B%3Crss+version%3D%222.0%22+xmlns%3Amedia%3D%22http%3A%2F%2Fsearch.yahoo.com%2Fmrss%2F%22%3E%3Cchannel%3E%3Ctitle%3Esyfy%3C%2Ftitle%3E%3Citem%3E%3Ctitle%3EThe+Terminator%3C%2Ftitle%3E%3Cguid%3E4053509%3C%2Fguid%3E%3Cmedia%3Arating+scheme%3D%22urn%3Av-chip%22%3ETV-MA%3C%2Fmedia%3Arating%3E%3C%2Fitem%3E%3C%2Fchannel%3E%3C%2Frss%3E%5D%5D%3E%3C%2FresourceID%3E%3Cttl%3E420000%3C%2Fttl%3E%3CissueTime%3E2024-10-01+23%3A03%3A27+-0700%3C%2FissueTime%3E%3CmvpdId%3ESpectrum%3C%2FmvpdId%3E%3C%2FauthToken%3E#__youtubedl_smuggle=%7B%22force_smil_url%22%3A+true%7D
[ThePlatform] 4053509: Downloading SMIL data
[ThePlatform] 4053509: Checking video URL
[ThePlatform] 4053509: video URL is invalid, skipping: HTTP Error 403: Forbidden
[ThePlatform] 4053509: Checking for HLS formats
WARNING: [ThePlatform] No HLS formats found: HTTP Error 403: Forbidden
[ThePlatform] 4053509: Downloading JSON metadata
WARNING: Only images are available for download. use --list-formats to see them
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), channels, acodec, size, br, asr, proto, vext, aext, hasaud, source, id
[debug] Default format spec: best/bestvideo+bestaudio
ERROR: [ThePlatform] 4053509: Requested format is not available. Use --list-formats for a list of available formats
Traceback (most recent call last):
  File "yt_dlp\YoutubeDL.py", line 1626, in wrapper
  File "yt_dlp\YoutubeDL.py", line 1782, in __extract_info
  File "yt_dlp\YoutubeDL.py", line 1894, in process_ie_result
  File "yt_dlp\YoutubeDL.py", line 1841, in process_ie_result
  File "yt_dlp\YoutubeDL.py", line 2977, in process_video_result
yt_dlp.utils.ExtractorError: [ThePlatform] 4053509: Requested format is not available. Use --list-formats for a list of available formats

[debug] Command-line config: ['-v', 'https://www.nbc.com/syfy-movies/video/caved-in-prehistoric-terror/3837162']
[debug] Encodings: locale cp1252, fs utf-8, pref cp1252, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp-nightly-builds [e59c82a74] (win_exe)
[debug] Python 3.8.10 (CPython AMD64 64bit) - Windows-10-10.0.19045-SP0 (OpenSSL 1.1.1k  25 Mar 2021)
[debug] exe versions: none
[debug] Optional libraries: Cryptodome-3.20.0, brotli-1.1.0, certifi-2024.08.30, curl_cffi-0.5.10, mutagen-1.47.0, requests-2.32.3, sqlite3-3.35.5, urllib3-2.2.3, websockets-13.1
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests, websockets, curl_cffi
[debug] Loaded 1838 extractors
[NBC] Extracting URL: https://www.nbc.com/syfy-movies/video/caved-in-prehistoric-terror/3837162
[NBC] 3837162: Downloading JSON metadata
[NBC] 3837162: Downloading JSON metadata
[debug] Loading ap-mvpd.nbcentertainment from cache
[NBC] 3837162: Retrieving Media Token
[ThePlatform] Extracting URL: http://link.theplatform.com/s/NnzsPC/media/guid/2304990266/3837162?mbr=true&manifest=m3u&switch=HLSServiceSecure&auth=%3CsignatureInfo%3ESeCEn9so9wj17VWIooYz15CRCuHxXZJHatLOD81MZeFuHjXgYhMz2Zj31aeT5zFEA7m%2B9620CpMj2WIKW2vqtR41%2F3MBltWDeDovcmXoXzipioCjJk7VKVM%2BKlCl7xN4iM7hIdsymVjh2rakqGpq6YbI9TOe2%2F6rF2kH9aQV5Pc%3D%3CsignatureInfo%3E%3CauthToken%3E%3CsessionGUID%3E90089f8249a9c90a7a87345572d92521%3C%2FsessionGUID%3E%3CrequestorID%3Enbcentertainment%3C%2FrequestorID%3E%3CresourceID%3E%3C%21%5BCDATA%5B%3Crss+version%3D%222.0%22+xmlns%3Amedia%3D%22http%3A%2F%2Fsearch.yahoo.com%2Fmrss%2F%22%3E%3Cchannel%3E%3Ctitle%3Esyfy%3C%2Ftitle%3E%3Citem%3E%3Ctitle%3ECaved+In%3A+Prehistoric+Terror%3C%2Ftitle%3E%3Cguid%3E3837162%3C%2Fguid%3E%3Cmedia%3Arating+scheme%3D%22urn%3Av-chip%22%3ETV-14%3C%2Fmedia%3Arating%3E%3C%2Fitem%3E%3C%2Fchannel%3E%3C%2Frss%3E%5D%5D%3E%3C%2FresourceID%3E%3Cttl%3E420000%3C%2Fttl%3E%3CissueTime%3E2024-10-01+23%3A17%3A22+-0700%3C%2FissueTime%3E%3CmvpdId%3ESpectrum%3C%2FmvpdId%3E%3C%2FauthToken%3E#__youtubedl_smuggle=%7B%22force_smil_url%22%3A+true%7D
[ThePlatform] 3837162: Downloading SMIL data
[ThePlatform] 3837162: Downloading m3u8 information
[ThePlatform] 3837162: Downloading JSON metadata
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), channels, acodec, size, br, asr, proto, vext, aext, hasaud, source, id
[debug] Default format spec: best/bestvideo+bestaudio
[info] 3837162: Downloading 1 format(s): hls-8106
[debug] Invoking hlsnative downloader on "https://vod-lf-oneapp-prd.akamaized.net/prod/video/kdZ/ix8/3837162/1688054046254-QFQEv/7830k_1080_hls/7830k_1080_hls.m3u8"
[hlsnative] Downloading m3u8 manifest
[hlsnative] Total fragments: 883
[download] Destination: Caved In： Prehistoric Terror [3837162].mp4
[debug] File locking is not supported. Proceeding without locking
[download] 100% of    5.00GiB in 00:03:02 at 28.07MiB/s
WARNING: 3837162: Possible MPEG-TS in MP4 container or malformed AAC timestamps. Install ffmpeg to fix this automatically

it might be that some movies are being DRM'ed and the current logic in the extractor(s) isn't identifying that situation

Does the movie play in browser with DRM disabled?


Make grpc.sln compile

Based on #9331.
-- remove some unnecessary broken projects from grpc.sln
-- reintroduce missing symbols that are breaking grpc_dll project (symbols defined in grpc.def but are not defined).
CC @bogdandrutu  @a-veitch  (uncommenting code that for reasons unknown to me was commented out in #8052)

@nicolasnoble  LGTM involves changes in #9331 or you will review them separately?

Yes, that's okay :)

Known failures:
https://github.com/grpc/grpc/issues/9309


Remove unused method

PR Checklist

[x] A description of the changes is added to the description of this PR.
[ ] If there is a related issue, make sure it is linked to this PR.
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If you've added or modified a feature, documentation in docs is updated

Description of changes
Remove unused method

thank you @ebyhr


Devel

Addressing issues:
https://github.com/01org/meta-security-isafw/issues/62
https://github.com/01org/meta-security-isafw/issues/60

I can't tell how much commit 2e05dae helps, but it sounds like the right thing to do either way.

@pohly, would you mind if I merge this and change the logic of handing error code in next commit? Otherwise it would again take a day in check run and I want the second commit to reach master asap since it does improve things greatly with memory at least on my local runs.

@ereshetova yes, please merge.


Installation error

Hi, I get this message:

What can I do to override this issue?
Thank you.
Harun.

unzip the archive


is there utf8 support?

Do you plan utf8 support?

I don't have time to support it now.

In linux I managed to have utf8 running by simply linking to ncursesw. There are some issues with drawing on short views, but it may be simple to solve it with Lua 5.4

In linux I managed to have utf8 running by simply linking to ncursesw.

We also need to consider pdcurse with wchar.

There are some issues with drawing on short views, but it may be simple to solve it with Lua 5.4 or with a library such as https://github.com/starwing/luautf8

I also need to consider the compatibility with xmake, because xmake also uses ltui, https://github.com/xmake-io/xmake/tree/master/xmake/core/ui
But xmake's ltui does not need luautf8, because its runtime rewrites built-in modules such as io/string, and has added support for utf8.

I will investigate it further when I have the time

Thanks

I also need to consider the compatibility with xmake, because xmake also uses ltui, https://github.com/xmake-io/xmake/tree/master/xmake/core/ui
But xmake's ltui does not need luautf8, because its runtime rewrites built-in lua modules such as io/string, and has added support for utf8.

We shall try dealing with utf8 characters locally in ltui or should we modify the io/string lib? The latter seems more elegant to me, but may increase the cpu usage a bit for the entire project.

Thank you very much, as long as you can ensure compatibility with xmake's ui module, only modify curses/pdcurses, or replace some native io interfaces, I welcome pr.
But I don't want to add new c modules and interface calls to ltui. Because when these patches are merged into xmake, these c libraries will also be added.


Unknown encoding

in https://github.com/lk-geimfari/mimesis/blob/master/mimesis/data/en/business.json and https://github.com/lk-geimfari/mimesis/blob/master/mimesis/data/en/address.json I've found:
"A�ropostale", -> Aéropostale
"Est�e Lauder Companies", -> Estée Lauder Companies

and
"La Ca�ada Flintridge", -> La Cañada Flintridge

@pimuzzo Can you create PR with a fix, please? Thanks!

@lk-geimfari Sure!


rocksdb: update version to fix ldb print error

Signed-off-by: Xintao<EMAIL_ADDRESS>
What is changed and how it works?

Issue Number: Close #12438
What's Changed:

rocksdb: update version to fix ldb print error

Related changes

PR to update pingcap/docs/pingcap/docs-cn:
Need to cherry-pick to the release branch

Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Side effects

Performance regression

Consumes more CPU
Consumes more MEM

Breaking backward compatibility

Release note
None

Close to code freeze window, it looks like there is no new PRs to be merged in rust-rocksdb, so I create this PR to update

/merge


Tag v2.1.0

Mind pushing a new tag when you get a chance? Quite a few commits since the last one.

Done.


Refactor Dockerfile post locale errors

Basing BigchainDB docker image from python:3.6 in Dockerfile and Dockerfile-dev.
Added ENV var for websocket interface
Optimization to Dockerfile-dev to save build time during dev/test.
Pre-installing the pynacl dependeny in separate command so that devs
do not need to wait while testing. The trade-off is that devs will
need to clear (rmi) the image to start from scratch.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1410   +/-   ##
=======================================
  Coverage   98.37%   98.37%
=======================================
  Files          56       56
  Lines        2588     2588
=======================================
  Hits         2546     2546
  Misses         42       42


Don't use embedded "sub" interfaces for sql.Driver.

I'm referring to the unexposed interfaces that are embedded in Driver, such as eventStoreDriver. We should just put the methods directly in Driver so that they are visible in documentation.

This package has been refactored such that this no longer makes sense.


When quick start, package.json is in "docs" directory.

https://vuepress.vuejs.org/guide/getting-started.html#manual-installation
I read he explanation, and I commanded this in my Windows PC.
yarn create vuepress-site twtwtw-gj-site

Then, The file twtwtw-gj-site/package.json doesn't exist.
The file twtwtw-gj-site/docs/package.json exists. And, in docs directory, I commanded yarn dev and sacceeded.
It confused me. The main directory should be twtwtw-gj-site.

@billyyyyy3320 I created a pull request for this, please review.

Oh, this issue is duplicated. Not intentionally.


New website domain

Currently, it's based on lengthy repo name https://elixir-oslo.github.io/norwegian-life-sci-rdm-lookup/
Suggestions:

rdm-lookup.elixir-no.org?
life-sci-rdm.elixir-no.org?
life-sci-rdm.no

how about rdm-lookup.elixir.no ? (then we don't need a new elixir.no.org ;) )
@sveinugu could you help sorting this?

Let's sort out rdm-lookup.elixir.no

I think the best solution is to set up a reverse proxy on the elixir.no web server for this. @morj-uio should be able to sort this out. He is not working until next week, I believe. Is this urgent?
Also you should make sure that all the links are relative so that that the path part of the URL is not referred to anywhere.
GitHub pages has a solution for hosting on a remote domain, but that uses let's encrypt which uio do not support and that we are moving (very slowly) away from.

Yes this is urgent, thank you

I interpret from the fact that it took three whole days for you to answer that this is urgent, as well as not hearing anything about the urgency of this in Finse, that it is really not that urgent and can wait until @morj-uio comes to work next week (which was what I was asking about).
Please reserve the word urgent to times when thing really are urgent (i.e. within the same day or the next), and also provide a deadline (and reason for this, e.g. presentation/reporting/whatever) so that we can actually use this information for something.. We have enough stress as is!

When I used the term urgent in this context, I took into account Morten's availability and meant that it would be nice if it is done soon once Morten is back. There is no strict definition of urgent used here as I am respectful to other people's schedule including mine. I did mention it during the meeting in Finse to which Korbinian responded that it's a secondary priority and that it will be nice to have something to link to - that concluded the conversation, in my opinion. Please kindly give a benefit of doubt and be respectful to your colleagues, that could make communication a lot less stressful. You are not the only one dealing with stress.

Urgent is defined as "requiring immediate action or attention".
I don't think it is disrespectful of me to inform you about how I and I believe others understand this word (which is according to its definition), on the contrary, I think that this is a respectful thing to do. However, this is not a good place to have this conversation and I see that I started it here and that it was thoughtless of me to respond in this context. I am sorry about that!
In any case, thanks for a great F2F and wishing you a nice weekend!

Thank you, and I am sorry for any misunderstanding/inconvenience caused from my side. Hope you reach home fine and have a relaxing weekend. Take care

Grrr, I had written many words, detailing different options, then clicked a link and lost it all.. Reminder to use email when replying to issues.
Anyway, as an example, I quickly set up this proxy instead:
https://elixir.no/norwegian-life-sci-rdm-lookup
Do you really need a new domain? It is not possible to hide the repo name/prefix anyway. It will be visible in menu links and search results etc, because they are root-relative. The GitHub custom domain option probably strips the prefix, but that is not possible with an external proxy solution. Can you shorten the repo name somewhat instead?
Of course we can order rdm-lookup.elixir.no anyway and use it as a shortcut/redirect to the above, optionally.
Btw, do we know where we are wrt UiO-certified certificates for elixir.no? Is it possible to get yet?

Then it will simply be elixir.no/rdm-lookuphttp://elixir.no/rdm-lookup. I think that is good as can be and will adjust the proxy config soon.

feb. 2023 kl. 15:55 skrev Nazeeefa<EMAIL_ADDRESS>
Thanks @morj-uiohttps://github.com/morj-uio for setting up proxy.
I have shortened the repo name, hope that sort of helps.
—
Reply to this email directly, view it on GitHubhttps://github.com/elixir-oslo/rdm-lookup/issues/25#issuecomment-1437146374, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADELF4W5XHYTGOZIV6CUSZLWYOAVJANCNFSM6AAAAAAU2JNGUU.
You are receiving this because you were mentioned.Message ID: @.***>


Move to newer version of java

Could this be upgrade to a newer version of java

Hi @capkmd, please let us know which java version you are using and what is the issue you face.
We plan on updating the README to include java version


Centralize Add Plates modal dialog javascript and html

Fixes #246, #351, #466 and #458.
I request that the reviewers actually download this PR as a branch and install it, and try adding plates on each of the SEVEN different pages that are affected (see list below).
I realize this is a big pain; however, as far as I know LabControl has no front-end javascript unit testing approach, and the code changes here affect PRIMARILY the front-end javascript ... so the fact that the PR passes the back-end unit-tests provides approximately no assurances about its fitness.  I have trialed the plate addition on each of these pages in my own test environment, but I certainly don't trust myself to catch all problems.

Affected pages to test "Add Plate" functionality on:
normalization.html
compression.html
extraction.html
library_pooling.html
library_prep_shotgun.html
library_prep_16S.html
parse_quantification.html

@AmandaBirmingham This sounds really nice! If you don't mind, I will finish up a WIP first before I review this one - should be in the next day or so.

Quick note for pulling this down, the branch is in "upstream" (i.e., jdereus). So for pulling:
$ git remote -v | grep upstream
upstream	git@github.com:jdereus/labman.git (fetch)
upstream	git@github.com:jdereus/labman.git (push)
$ git checkout -b issue_468
$ git pull --rebase upstream Issue246and351

Will do!
On Fri, Apr 19, 2019 at 1:35 PM Daniel McDonald<EMAIL_ADDRESS>wrote:

Quick note for pulling this down, the branch is in "upstream" (i.e.,
jdereus). So for pulling:
$ git remote -v | grep upstream
upstream	git@github.com:jdereus/labman.git (fetch)
upstream	git@github.com:jdereus/labman.git (push)
$ git checkout -b issue_468
$ git pull --rebase upstream Issue246and351
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/jdereus/labman/pull/468#issuecomment-485007637, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AKFU7E2LOCLUIJGNR56A5QDPRIUIDANCNFSM4HHB23QQ
.

Thanks, @AmandaBirmingham! I've gone through the requested pages. Below are the comments that I wasn't sure warranted separate issues. Doing a code review now.
Major:

There is a race condition with the user. If the user is adding plates fast, the modal pops up and disappears before the user can do anything. This behavior also appears to allow for unexpected behavior: I was able to add 5 384-well plates to compress resulting in an exception (see first .gif; traceback below .gif). It is likely this issue will worsen with increased network lag (this was running locally). I think this is generalized to multiple page. I don't believe this is due to the changes within the PR but leaving within this issue for the time being as it may make sense to place a protection within the add plate interface code.

AN ERROR HAS OCCURED!
Please copy the following into an issue in our github repository

Error
Cannot compress 5 gDNA plates. Please provide 1 to 4 gDNA plates

Traceback
Traceback (most recent call last):
File "/Users/dtmcdonald/miniconda3/envs/labcontrol/lib/python3.7/site-packages/tornado/web.py", line 1697, in _execute result = method(*self.path_args, **self.path_kwargs)
File "/Users/dtmcdonald/miniconda3/envs/labcontrol/lib/python3.7/site-packages/tornado/web.py", line 3174, in wrapper return method(self, *args, **kwargs)
File "/Users/dtmcdonald/ResearchWork/software/labman/labcontrol/gui/handlers/process_handlers/gdna_compression_process.py", line 50, in post self.current_user, plates, plate_ext_id, Equipment(robot))
File "/Users/dtmcdonald/ResearchWork/software/labman/labcontrol/db/process.py", line 773, in create 'gDNA plates' % len(plates))
ValueError: Cannot compress 5 gDNA plates. Please provide 1 to 4 gDNA plates

Minor:

Modal is dense with text, I wonder if forcing a name truncation, or widening the modal would be beneficial?
I'm not sure I grok the search functionality (note the search terms used):

Thanks, @wasade -- this is great!  Re https://github.com/jdereus/labman/pull/468#issuecomment-485601425 , I tracked down the issue you found with adding 5 plates to the compression page and verified that it is not due to adding plates fast and exists in master, so I created a new issue for it at #477 ; fortunately it looks super easy to fix and the fix can be added easily to this PR.
With that red herring out of the way, could you say more about the race condition you observed? ("If the user is adding plates fast, the modal pops up and disappears before the user can do anything").  I have not been able to reproduce it--perhaps I am not fast enough? :)  If you could provide either instructions to reproduce or a gif of it happening (on a page other than compression bc we know that one is screwed up) it would really help me track this down.
I will look into the search functionality ...

Per https://github.com/jdereus/labman/pull/468#issuecomment-485601425 , I checked into the search functionality.  It is provided by the external DataTables javascript library; here is the description of its search API: https://datatables.net/reference/api/search() .
Essentially, it appears to be a pretty greedy search that returns results that contain within them all of the strings entered, in any order, case-insensitively, across the union of all searchable fields . Since the study name is a searchable field and since "Identification of the Microbiomes for Cannabis Soils" contains the strings fo (in "for"), so (in "Soils"), ca (in "Cannabis"), and de (in "Identification"), and since the plate name is a searchable field and contains the string la (in "plates"), the results returned for the search input "fo so ca la de" are consistent with the DataTables search rules.
From the DataTables documentation, it looks like we can tweak the search functionality in various ways, e.g. below, pulled from the link above.  Please provide feedback on whether you feel this is necessary.

The smart search ability of DataTables is performed using a regular expression and can be enabled or disabled using the third parameter of this method. If you wish to use a custom regular expression, for example to perform whole word exact matching, you would need to enable the regular expression option (second parameter) and disable the smart search option (third parameter) to ensure that the two do not conflict.

@AmandaBirmingham race condition may not be the most accurate term but felt approximately like what was observed :) I haven't had this happen on other pages, so it may have been isolated to the compression page. #473 and the gif contained highlights a possibly related issue though where the modal can be opened even though the loading is still happening which can be misleading
That makes total sense re: search. Let's drop that item.

Putting this in the PR, since I'm testing these code changes: When I try to create a new sample plate, I find that I cannot. Has anyone else seen this issue? Possible it's something wrong with my configuration; in any case, it's odd that I have no idea why:

@charles-cowart wooo, that is weird.  @wasade can you reproduce in the PR branch?  I cannot, although I will keep trying ...

@AmandaBirmingham @wasade and I couldn't reproduce it in his dev environment, implying that it's something with my local configuration; although it's not immediately obvious to the both of us what it is. Confirmed I'm on commit fca5b..... I will look at a few other things. In the meantime, I took a screenshot of the developer console:

I tested add plate functionality on all of the requested pages, and this is what I encountered:
normalization.html -> normalize
Appears to be working as intended, but I also get a js error
compression.html	-> gdna_compression
Working as intended, no js errors
extraction.html -> gdna_extraction
Appears to be working as intended, but I also get a js error (but not all the time when adding multiple plates)
library_pooling.html -> poollibraries
Working as intended, no js errors (both Amplicon and Shotgun)
library_prep_shotgun.html -> library_prep_shotgun
Appears to be working as intended, but I also get a js error
library_prep_16S.html -> library_prep_16S
Appears to be working as intended, but I also get a js error
parse_quantification.html -> parse_quantify
working as intended, no js error
I've attached animated GIFs for most of the errors I encountered. More than 3 seemed redundant:

I also noticed some steps were not available in subsequent steps after I created them, but I will dig deeper on those and report anything as a separate issue.

@charles-cowalt, please recall that 404 errors for plate names that don’t exist yet are in fact required: they are LabControl’s way of informing the front end that the plate name is available for use.  That is why they are silently hidden rather than reported.
Given this, it appears to me that the only “real” :) error shown in these screenshots is the plating page’s TypeError when the code is unable to find the constructor for the Slick.CellSelectionModel object.  I’d make a small bet that this is linked to your inability to create a sample plate (probably the error in the  upstream JavaScript prevents later code that would enable the “Create” button from being executed).
If I recall correctly, you were doing some development work aimed at determining if we could remove some of the SlickGrid components from the plating page.  Is it possible that some file or some file contents were removed or modified on your system during this investigation, and that some portion of those changes remain on your machine?  This might speak to why the plating  issue is occurring in your dev environment but mine or @wasade ‘s ...

@AmandaBirmingham Ah, of course! :D Thanks for the clarification. It's a fresh pull from GH so I don't think anything should be removed or modified, but I'll look into it shortly - thanks for the tip!

Found it! It looks like the 'slickgrid.plugins' directory didn't get installed into my '~/miniconda3/envs/labman/lib/python3.5/site-packages/labcontrol/gui/static/vendor/js' directory. Once I copied it manually from the source tree to the installed base, everything worked fine. I'll check out the install script to see if that's the cause. Thanks again!

@AmandaBirmingham The slickgrid.plugins issue has been resolved w/ https://github.com/jdereus/labman/pull/484. No other issues w/this PR outside of what is currently being discussed.

@wasade Have updated pull request, please re-review when you can.

@AmandaBirmingham, a static view of the commits looks great, thank you!! Are you requesting a re-review of this PR in a live dev environment as well for the pages noted above, or are you comfortable with a static review?

@wasade Urgh.  I think I'm going to say static review .... Giving optimism a try :)

@wasade Just realized I'm not sure where we stand with this.  Assuming we are doing static review only, do you approve this PR? If so, I'd like to merge it soon; if not, would like to know what changes to make.

Sorry for the delay. I approve :)
On Thu, Apr 25, 2019, 1:49 PM Amanda Birmingham<EMAIL_ADDRESS>wrote:

@wasade https://github.com/wasade Just realized I'm not sure where we
stand with this. Assuming we are doing static review only, do you
approve this PR? If so, I'd like to merge it soon; if not, would like to
know what changes to make.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jdereus/labman/pull/468#issuecomment-486832738, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AADTZMV67C2PQTLFP26LR4LPSIKNRANCNFSM4HHB23QQ
.

Just to clarify, this PR fixes: #246, #351, #466, #458, #477.


Link "OpenXR Toolkit / VirtualDesktopXR Discord" does not work

If I choose Issue / New and click on "OpenXR Toolkit / VirtualDesktopXR Discord" the link does not work.
I would like to join the Discord server to ask some questions.

The server no longer exist.


Kiwi makes blue light filter app close

Device : Galaxy Tab SM-T550, Android 7.1.1
Kiwi version : 93.0.4577.9
App
The blue light filter app that I use keeps closing when I use the latest version of Kiwi

I get a not found error when clicking your link.

The link works for me, I tried opening it in a private tab and got the same error. I didn't notice it was discontinued since it still works, weird

In that case, the app might be crashing because it is using some feature that is discontinued and is therefore incompatible with the latest Kiwi.
I therefore suggest that you use an up to date alternative.
Thanks

Ah, the crash is on the blue light filter app, yes I tend to agree that it may not be caused by Kiwi directly but rather the app trying to bind to elements that don't exist in recent Chromium


Make sure all functions have docstrings

I noticed that the function that enables the Ivy patches does not have a docstring, for example.

Woops.


Implemented Company Prefix param

Overview/Summary
Implements the parCompanyPrefix from hubNetworking.
This PR fixes/adds/changes/removes

Adds parCompanyPrefix as a param.

Breaking Changes
none
As part of this Pull Request I have

[x] Checked for duplicate Pull Requests
[ ] Associated it with relevant ADO items
[x] Ensured my code/branch is up-to-date with the latest changes in the main branch
[ ] Performed testing and provided evidence.
[ ] Updated relevant and associated documentation.

Hey @cloudchristoph,
Thanks for the PR and you efforts here. Really appreciated 👍
For this one I'm afraid we don't want to make this change in isolation and we actually are tracking a naming feature all up internally to follow the CAF naming recommendations as much as we can, where it makes sense.
So if you are okay we will close this PR as it's not something we will do in this way.
The reason we haven't created the names in the way you suggest is to provide the flexibility to the end user via parameter inputs so they can call the resources whatever they like.
Whilst your approach also does this as users can override the default input. If a user does this, parCompanyPrefix will then be unused and this will mean an unrequited parameter and the linter may pick this up on build/deployment and fail the deployment.
So in summary we want to leave it as it today as we provide the most flexibility. And we are reviewing naming all up to align to CAF naming and we will publish a naming standard we followed. 👍
Really appreciate the PR and effort here though ❤

Hey @jtracey93,
thanks for your feedback. Totally fine with this.
I read out that with this also in the hubNetworking module this parameter will be removed in the future. Not today or tomorrow, but as soon as the naming scheme is in place, right?
If you need any help with the modules or the schema: I'm happy to help!


Refresh calendar month by month

Hi!
first of all, thank you very much for this great plugin!
For a calendar of availabilities, I need to enabled/disable dates when I press next or prev month buttons.
I was thinking about

sending data to php file with ajax
make a query to database
get the date to enable/disable
and then use the set method to disable

I don't know what I am missing but I can find a way to achieve that.
I couldn't get all the dates to disable on loading because it take too muck time.
Is it possible to do that?
Best Regards
Fanny

Hello,
nobody can help me?


cant update

I am running version 2.17. now I tried several times in the last days to update to the latest release. Cant get trough.

Hi @JoergHoffmann , to update the app:

Download update from https://defichain.com/downloads/
Open defi-app-2.1.15.dmg file
Drag and replace your current app

Hope this fixes your issue.


Permissive regex for conda env names from nb_conda_kernels

This changes the behavior of environment name extraction from the nb_conda_kernels name/kernel string. Previously couldn't match normal-looking names like my-env, but certainly not crazy ones like my-env[lastest] new changes, which is a totally valid environment name (except for post-link scripts that don't enclose paths in quotes).
also,

use strict, because always,
and var m to avoid global scope leaking

heh, for #19.

Thanks for the fix, it looks good.

so the plan is chill out until we get OSX build verification, then do a release?

OK, if we merge this one we should bump the version before merging... and after the Distribution packaging, migrate to the pattern outlined here: https://github.com/Anaconda-Server/nb_conda_kernels/pull/16

Looks like everything passed. Unless there are any objections, I'll do the bump, tag, build, release.

bump, tag, merge, build, release

bump, tag, merge, build, release

Thanks... I was about putting a comment asking for the merge :wink:

Ok, upload build is running! It might be a tick before I can come back and move them... but it might be a tick before the osx worker runs!

Jaja... yes... taking so long today...

Windows and linux builds are looking good off main.


Separate out the diffing capability into another framework?

The company I work for has been using various unidirectional data flow architectures in several production apps. One thing we have need for in that is a robust diffing tool for lists, which IGListKit provides. However, we have no need for the rest of the framework.
As the diffing capability can stand on its own, from our perspective it would be nice to be able to pull in that alone. It would additionally be easier to ramp developers up on our approach since they would not be distracted by the other features of IGListKit.
New issue checklist

[x] I have reviewed the README and documentation
[x] I have searched existing issues and this is not a duplicate

General information

IGListKit version:
iOS version(s):
CocoaPods/Carthage version:
Xcode version:
Devices/Simulators affected:
Reproducible in the demo project? (Yes/No):
Related issues:

Duplicate of #53. See also: #365.

Thanks @ianterrell !
We have a diffing subspec — does your team use CocoaPods?

Check out the installation guide for how to pull in just the diffing subspec. That'll only pull in stuff from /Common.

Thank you for the quick reply — I did not notice the subspec option originally. We are currently moving from CocoaPods to Carthage for most of our projects.


fuel-core v0.13 upgrade

$rB on state opcodes is now used as a flag for whether the storage slot was previously set
$rD on quad word state opcodes is reserved for use as accessing a range of slots (currently hardcoded to 1)

currently blocked on fuel-rs upgrade

As we were just talking about the fuel-core in CI, I remembered that https://github.com/FuelLabs/sway/blob/master/.github/workflows/ci.yml#L244 and https://github.com/FuelLabs/sway/blob/51987a96a6f0ac23f4102478290e0bc1736b2af6/.github/workflows/ci.yml#L265 should be updated.


Why was this package discontinued?

I noticed on pub.dev that it's discontinued and was curious why

Absolutely agreed. "New" package is not anyhow capable of being a substitute for this one


Update legalzoom-ip.md

👷 Deploy Preview for famous-marshmallow-cfbd61 processing.

Name
Link

🔨 Latest commit
9cb2c4808ec051878fe2b9bcfdad533bcb032dac

🔍 Latest deploy log
https://app.netlify.com/sites/famous-marshmallow-cfbd61/deploys/64a7462475ecaa00081a8748


chore: publish 0.35.0

🤔 这个 PR 的性质是？

[ ] 日常 bug 修复
[ ] 新特性提交
[ ] 文档改进
[ ] 演示代码改进
[ ] 组件样式/交互改进
[ ] CI/CD 改进
[ ] 重构
[ ] 代码风格优化
[ ] 测试用例
[ ] 分支合并
[ ] 其他

🔗 相关 Issue

💡 需求背景和解决方案

📝 更新日志

fix(组件名称): 处理问题或特性描述 ...

[ ] 本条 PR 不需要纳入 Changelog

☑ 请求合并前的自查清单
⚠ 请自检并全部勾选全部选项。⚠

[ ] 文档已补充或无须补充
[ ] 代码演示已提供或无须提供
[ ] TypeScript 定义已补充或无须补充
[ ] Changelog 已提供或无须提供

Codecov Report

Merging #869 (aba429d) into develop (8fdcb64) will decrease coverage by 0.44%.
The diff coverage is 41.23%.

@@             Coverage Diff             @@
##           develop     #869      +/-   ##
===========================================
- Coverage    60.81%   60.37%   -0.45%
===========================================
  Files          914      926      +12
  Lines        15385    16036     +651
  Branches      3467     3605     +138
===========================================
+ Hits          9357     9682     +325
- Misses        6028     6354     +326

Impacted Files
Coverage Δ

src/addon/Addon.tsx
0.00% <0.00%> (-92.31%)
:arrow_down:

src/comment/Comment.tsx
95.45% <ø> (ø)

src/date-picker/DateRangePicker.tsx
10.79% <0.00%> (-0.08%)
:arrow_down:

src/date-picker/base/Cell.tsx
20.00% <0.00%> (+13.33%)
:arrow_up:

src/date-picker/base/Footer.tsx
73.33% <0.00%> (+59.04%)
:arrow_up:

src/date-picker/base/Header.tsx
42.42% <ø> (+37.87%)
:arrow_up:

src/date-picker/defaultProps.ts
100.00% <ø> (ø)

src/date-picker/hooks/useTableData.tsx
69.23% <ø> (ø)

src/date-picker/index.ts
100.00% <ø> (ø)

src/date-picker/panel/ExtraContent.tsx
100.00% <ø> (+100.00%)
:arrow_up:

... and 117 more

Continue to review full report at Codecov.

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


How to inference custom-trained model

Hello OpenLRM Team,
Following the guidance provided in issue #24, I've managed to run python scripts/convert_hf.py --config <YOUR_EXACT_TRAINING_CONFIG> using the files from exps/checkpoints such as custom_check and model.sfet. This process successfully generated a config.json file and model.safetensors in the releases folder. However, I'm struggling to use these files for inference. Following the instructions in the readme section on inference, I keep encountering 'key not found' errors. It seems I might be setting the properties incorrectly, but I can't figure out the right approach. Could you provide additional guidance on how to proceed with inference using trained models? Any assistance would be greatly appreciated.

Thank you for your quick response. Below are the errors that are currently occurring
gpuadmin@gpusystem:~/ihjune/OpenLRM$ python -m openlrm.launch infer.lrm  --infer $INFER_CONFIG model_name=$MODEL_NAME image_input=$IMAGE_INPUT export_video=$EXPORT_VIDEO export_mesh=$EXPORT_MESH
[2024-03-27 14:55:35,448] openlrm.models.modeling_lrm: [INFO] Using DINOv2 as the encoder
/home/gpuadmin/ihjune/OpenLRM/openlrm/models/encoders/dinov2/layers/swiglu_ffn.py:43: UserWarning: xFormers is available (SwiGLU)
warnings.warn("xFormers is available (SwiGLU)")
/home/gpuadmin/ihjune/OpenLRM/openlrm/models/encoders/dinov2/layers/attention.py:27: UserWarning: xFormers is available (Attention)
warnings.warn("xFormers is available (Attention)")
/home/gpuadmin/ihjune/OpenLRM/openlrm/models/encoders/dinov2/layers/block.py:39: UserWarning: xFormers is available (Block)
warnings.warn("xFormers is available (Block)")
Loading weights from local directory
0%|                                                                | 0/1 [00:00<?, ?it/s]/home/gpuadmin/ihjune/OpenLRM/openlrm/datasets/cam_utils.py:153: UserWarning: Using torch.cross without specifying the dim arg is deprecated.
Please either pass the dim explicitly or simply use torch.linalg.cross.
The default value of dim will change to agree with that of linalg.cross in a future release. (Triggered internally at ../aten/src/ATen/native/Cross.cpp:63.)
x_axis = torch.cross(up_world, z_axis)
0%|                                                                | 0/1 [00:27<?, ?it/s]
Traceback (most recent call last):
File "/home/gpuadmin/miniconda3/envs/lrm/lib/python3.10/runpy.py", line 196, in _run_module_as_main
return _run_code(code, main_globals, None,
File "/home/gpuadmin/miniconda3/envs/lrm/lib/python3.10/runpy.py", line 86, in _run_code
exec(code, run_globals)
File "/home/gpuadmin/ihjune/OpenLRM/openlrm/launch.py", line 38, in
main()
File "/home/gpuadmin/ihjune/OpenLRM/openlrm/launch.py", line 33, in main
runner.run()
File "/home/gpuadmin/ihjune/OpenLRM/openlrm/runners/infer/base_inferrer.py", line 62, in run
self.infer()
File "/home/gpuadmin/ihjune/OpenLRM/openlrm/runners/infer/lrm.py", line 284, in infer
self.infer_single(
File "/home/gpuadmin/ihjune/OpenLRM/openlrm/runners/infer/lrm.py", line 244, in infer_single
mesh = self.infer_mesh(planes, mesh_size=mesh_size, mesh_thres=mesh_thres, dump_mesh_path=dump_mesh_path)
File "/home/gpuadmin/ihjune/OpenLRM/openlrm/runners/infer/lrm.py", line 207, in infer_mesh
vtx_colors = self.model.synthesizer.forward_points(planes, vtx_tensor)['rgb'].squeeze(0).cpu().numpy()  # (0, 1)
File "/home/gpuadmin/ihjune/OpenLRM/openlrm/models/rendering/synthesizer.py", line 206, in forward_points
for k in outs[0].keys()
IndexError: list index out of range

Hi,
It looks like the same cause to another issue here https://github.com/3DTopia/OpenLRM/issues/20.
Could you plz try using a newer CUDA version such as 11.8 and keeping your torch version at least 2.1.2?
I haven't encountered such problem locally so that I'm not sure where is the root cause either. Hopefully upgrading to newer versions would help.
Plz feel free to comment if it persists.

Thank you so much for your quick and kind reply.
I updated my environment following the issue connected to CUDA 11.8 and Torch 2.2.0, but I get the same error. Even before I changed the settings, I noticed that the points were empty in the function EXPORT_VIDEO=true, EXPORT_MESH=true, INFER_CONFIG="/configs/infer-b.yaml," MODEL_NAME="zxhexexin/openlrm-mix-base-1.1," and the code worked fine in the IMAGE_INPUT="/assets/sample_input/lowl.png" settings. In the custom-trained model, the error continues to occur! This issue appears to be caused by empty points in the forward_points in the openlrm/models/rendering/synthsizer. I checked that it was filled correctly up to tube_grid, but the points were empty in forward_points. Can you provide any advice or insight on this issue?

This is a problem caused by too little training data in the custom trained model, but the threshold value is large. In the openlrm/runners/infer/lrm.py file infer_mesh function, the shape of vtx and faces was 0,3 due to the threshold! Adjusting the threshold to match the value of grid_out['sigma'] solved the problem! It's been resolved, so we'll close the issue. Thank you.

Hi @eolgaeng-i , recently I've generated checkpoint model, and tried inference on my trained model.
But the result quality of the inference ply is not very good.
Could you please check my https://github.com/3DTopia/OpenLRM/issues/41 when you have time?
It would be a huge help if you could check it when it possible.
Thank you in advnace!


chore(package): update semantic-release to version 15.13.30

Coverage remained the same at 85.96% when pulling 032457640af2954cb92a1edf316d0f2f7992b237 on greenkeeper/semantic-release-15.13.30 into 75ad680041f68f3b270585bb2b475877e8270ee4 on master.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add market complex methods

See https://vk.com/dev/market

Changes Unknown when pulling 2d72727d4fb73f30388852c52b0a3b8fcfccebee on nuklea:patch-1 into ** on kmike:master**.


Fix cache invalidation in put and pipe (#473)

Addresses #473, by ensuring invalidate cache is run when small files are created using .pipe or .put, and adds a test for each.
I had three failing tests, which might be related to me needing to set some test config? Here are the failing test names:
FAILED gcsfs/tests/test_core.py::test_request_user_project - ValueError: User-provided project 'project' does not match the google default project 'None'. Either
FAILED gcsfs/tests/test_core.py::test_request_header - TypeError: Invalid variable type: value should be str, int or float, got None of type <class 'NoneType'>
FAILED gcsfs/tests/test_credentials.py::test_googlecredentials_none - ValueError: User-provided project 'myproject' does not match the google default project 'None'. Either

Happy to tweak / add anything useful!

Please run linting on this - it is failing on whitespace. You can pre-commit install (if pre-commit itself is in your environment), if you want that automated.


Incorrect Count After Asynchronous Concurrent Updates When Using execute_after_commit

When enqueueing multiple background jobs (e.g. Sidekiq) which utilize counter_culture's execute_after_commit, the final count will be incorrect (lower than expected).
Here is an over simplification of the setup:
class Campaign < ApplicationRecord
  has_many :participants, dependent: :destroy
end

class Participant < ApplicationRecord
  belongs_to :campaign
  counter_culture :campaign, execute_after_commit: true
end

class DeleteParticipantsJob
  include Sidekiq::Worker

  def perform(campaign_id, participant_ids)
    campaign = Campaign.find(campaign_id)
    participant_ids.each do |participant_id|
      campaign.participants.find(participant_id)&.destroy
    end
  end
end

I'm running 2 Sidekiq workers, each with 10 threads. When enqueuing multiple DeleteParticipantsJobs, each with the same or overlapping set of participant_ids, the final count on the Campaign will often be lower than expected (or negative in the case where all participants are deleted).
This issue is not present when not using execute_after_commit.
I suspect that if multiple threads attempt to find and destroy the same participant that counter_culture is executing the after commit count update even if the participant was not actually destroyed by the thread in question (e.g. the record was already destroyed, no SQL DELETE was executed, yet counter_culture still executed the after_commit callback).

I believe that I have isolated this issue counter_culture's the use of after_create, after_update and before_destroy callback methods regardless of whether the execute_after_commit option is present. I believe that when the execute_after_commit option is present, counter_culture should use after_commit callback methods to ensure that the callback is not executed multiple times.
@magnusvk I have a draft fix in a fork. It's not yet ready for me to submit as a pull request, but I would love if you could check it out and provide your initial feedback (am I on to something here, or way off in the weeds?). Thanks!

Interesting! Honestly I have no idea if your fix in the fork works—but if it does, great! One could argue that’s a better way to do this anyhow, so if you can show that works and it keeps all the other tests green, then 👌🏻
My other reaction here is just—if you do things outside of the transaction it’s not going to be 100% safe. But that’s not to say we shouldn’t fix it if we can! Let me know what you find.

@magnusvk Thanks for your feedback.
The feature branch in the fork linked above does solve the issue of incorrect counts; however, when using counter_culture's support for Paranoia and Discard, the counts recorded by soft deleting & restoring can still be incorrect. This is because those respective gems do not utilize after_commit callbacks.
Given I'm also using Discard in a few places, it looks like I'll have to solve this myself with some pessimistic locking in my Sidekiq job.
While I still think that solving this in counter_culture would be the best approach, I'm not sure if a half-baked solution is better than no solution. Your thoughts on whether you want a pull request for the above?

I think it's still an improvement to use the after_commit callbacks, right? If it solves the issue unless you're using Paranoia or Discard, that'll still apply to a large chunk of users. So I'd welcome a PR for that change. I don't think we can add a test that verifies this fixes the concurrency issue, but even without that I'd be happy to merge as long as all the other tests are green.

@jbritten any follow-up here, did you want to open a PR with those changes?

@magnusvk sorry for letting this fall through the cracks. I've just open PR #325.

Closing for inactivity.


Long parameters names use single - instead of -- when displaying an error

For example, if you run dts-gen -e "" twice the error

File dts_gen_expr.d.ts already exists and -overwrite was not specified; exiting.

will show up instead of

File dts_gen_expr.d.ts already exists and --overwrite was not specified; exiting.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: Suisse00 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Thanks!


Build & publish workflows + nuget package polishing

I can't find the name of the secret in my inbox, can you help me out, @bingenito?

I can't find the name of the secret in my inbox, can you help me out, @bingenito?

NUGET_KEY


ISPN-12693 xml config cleanup and tabbed examples

https://issues.redhat.com/browse/ISPN-12693
This PR adds https://github.com/bmuschko/asciidoctorj-tabbed-code-extension for tabbed configuration and dependency examples in documentation.
Also removes several unused examples, re-organizes some configuration examples, and removes some callout.

@tristantarrant and @ryanemerson In preparation for XML and YAML config examples, should we remove all callouts from the files under documentation/src/main/asciidoc/topics/config_examples?
There are a few files with callouts "<1>" left. We can replace the callouts with commented text but I wanted to ask before going ahead. It might be the case that we want to preserve callouts for some examples, especially in the Server Guide for things like security realm configuration.

In preparation for XML and YAML config examples, should we remove all callouts from the files under documentation/src/main/asciidoc/topics/config_examples?

+1 It would be really good if users could just copy and paste without any editing.

In preparation for XML and YAML config examples, should we remove all callouts from the files under documentation/src/main/asciidoc/topics/config_examples?

+1 It would be really good if users could just copy and paste without any editing.

@ryanemerson It is possible to copy/paste examples without also copying the callouts. However the problem arises downstream where tabbed code blocks are not possible. In the downstream, you get two code blocks. One on top of the other. If you have callouts then they only render for the last code block and the doc tooling throws a fatal error.
When it comes to tabbed code blocks it's better to have comments so examples are "self-contained". However I'm unsure if we want to convert all the xml config examples to yaml. I've opened https://issues.redhat.com/browse/ISPN-12754 so we can look at removing callouts as a separate issue.

When it comes to tabbed code blocks it's better to have comments so examples are "self-contained". However I'm unsure if we want to convert all the xml config examples to yaml. I've opened https://issues.redhat.com/browse/ISPN-12754 so we can look at removing callouts as a separate issue.

We want conversion to be automated, otherwise it's going to be a nightmare

When it comes to tabbed code blocks it's better to have comments so examples are "self-contained". However I'm unsure if we want to convert all the xml config examples to yaml. I've opened https://issues.redhat.com/browse/ISPN-12754 so we can look at removing callouts as a separate issue.

We want conversion to be automated, otherwise it's going to be a nightmare

Indeed. So it's better to go with commented text so the markup is valid. Otherwise the conversion mechanism will need to handle callouts.

@oraNod Overall LGTM, but there's a couple of .xml files that need updating still:
grep -Rl "<1" documentation/src --include="*.xml"
documentation/src/main/asciidoc/topics/config_examples/server_credentials.xml
documentation/src/main/asciidoc/topics/config_examples/server_ssl_identity.xml

@ryanemerson Should be fixed now. Could you please take another look?

Thanks @oraNod


Create Figma prototype project

We want to make some wireframes of the new GUI so that a) we can make a prototype to show testers/potential customers/Ekkart, b) we have an agreed upon vision of what the project should be, c) the team can use the wireframes as a resource when working on implementing the GUI (easier to work concurrently on different components, everyone knows what the goal is).

@markusbrammer har oprettet et projekt her: https://www.figma.com/files/project/68194096/Team-project?fuid=805904396270732910


Error while buiding

[vite] Error when evaluating SSR module /opt/buildhome/repo/astro.config.mjs: failed to import "astro-compress"
11:18:19.066	|- Error: Could not load the "sharp" module using the linux-x64 runtime
11:18:19.067	Possible solutions:
11:18:19.067	- Add platform-specific dependencies:
11:18:19.067	    npm install --os=linux --cpu=x64 sharp
11:18:19.068	  or
11:18:19.068	    npm install --force @img/sharp-linux-x64
11:18:19.068	- Consult the installation documentation: https://sharp.pixelplumbing.com/install
11:18:19.068	    at Object.<anonymous> (/opt/buildhome/repo/node_modules/sharp/lib/sharp.js:85:9)
11:18:19.068	    at Module._compile (node:internal/modules/cjs/loader:1233:14)
11:18:19.068	    at Module._extensions..js (node:internal/modules/cjs/loader:1287:10)
11:18:19.068	    at Module.load (node:internal/modules/cjs/loader:1091:32)
11:18:19.069	    at Module._load (node:internal/modules/cjs/loader:938:12)
11:18:19.069	    at Module.require (node:internal/modules/cjs/loader:1115:19)
11:18:19.069	    at require (node:internal/modules/helpers:119:18)
11:18:19.069	    at Object.<anonymous> (/opt/buildhome/repo/node_modules/sharp/lib/constructor.js:10:1)
11:18:19.069	    at Module._compile (node:internal/modules/cjs/loader:1233:14)
11:18:19.069	    at Module._extensions..js (node:internal/modules/cjs/loader:1287:10)
11:18:19.069
11:18:19.069	[astro] Unable to load your Astro config
11:18:19.069
11:18:19.538	Could not load the "sharp" module using the linux-x64 runtime
11:18:19.539

Hi, @SyedMuzamilM, can you provide more information on your environment, the OS, hosting provider, stuff like that, so that I can investigate it further? In the meantime, you can try running the #no-sharp branch, which disables image compression, but removes this error. To install it just replace the package.json entry for astro-compress with:
{
    "astro-compress": "astro-community/AstroCompress#no-sharp"
}

Which Node version are you using?

There are similar issues in the sharp repository. Maybe try npm ls sharp as per https://github.com/lovell/sharp/issues/3896#issuecomment-1857516925 and see if there's a version mismatch somewhere in your code or with an outdated dependency.

Which Node version are you using?

I am using the latest node version 20.10.0

There are similar issues in the sharp repository. Maybe try npm ls sharp as per https://github.com/lovell/sharp/issues/3896#issuecomment-1857516925 and see if there's a version mismatch somewhere in your code or with an outdated dependency.

Will check the and see if this works.
The previous command worked fine which doesn't use sharp. It is now getting deployed properly.
Will check id any other dependency is using sharp so that the error could be resolved.

I see, astro 3.6.4 is running an older version of sharp than astro-compress
├─┬<EMAIL_ADDRESS>│ └──<EMAIL_ADDRESS>└─┬<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>
Can you try pinning the version of sharp for astro with the same one astro-compress uses. This is done by updating the package.json to override the dependency like so:
package.json
{
  "name": "your-project",
  "version": "1.0.0",
  "dependencies": {
    "astro": "3.6.4",
    "astro-compress": "2.2.5"
  },
  "resolutions": {
    "sharp": "0.33.0"
  }
}

or
package.json
{
  "overrides": {
    "astro": {
      "sharp": "0.33.0"
    }
  }
}

Since astro:image uses sharp as its default image service, I think that might be causing issues when using astro-compress
[https://docs.astro.build/en/guides/images/#default-image-service](Astro Default Image Service) you can check it here.

Can you try pinning the version of sharp for astro with the same one astro-compress uses? This is done by updating the package.json to override the dependency like so:
package.json
  "resolutions": {
    "sharp": "0.33.0"
  }

This fixed it for me! Thank you for your work

Interesting…  setting "resolutions": {"sharp": "0.32.6"} worked with my M1 MacBook Air, but setting it to 0.33.0 resulted in the error:
[vite] Error when evaluating SSR module /Users/kswedberg/Sites/ks/blog.karlswedberg.com/astro.config.mjs: failed to import "astro-compress"
|- Error: Could not load the "sharp" module using the darwin-arm64 runtime

Interesting… setting "resolutions": {"sharp": "0.32.6"} worked with my M1 MacBook Air, but setting it to 0.33.0 resulted in the error:
[vite] Error when evaluating SSR module /Users/kswedberg/Sites/ks/blog.karlswedberg.com/astro.config.mjs: failed to import "astro-compress"
|- Error: Could not load the "sharp" module using the darwin-arm64 runtime

Use
{
  "overrides": {
    "astro": {
      "sharp": "0.33.0"
    }
  }
}

resolutions is supported only on Yarn.

@NikolaRHristov , I'm using Yarn. Thanks for the suggestion, but neither resolutions nor overrides worked with sharp 0.33.0. Maybe I need to leave it set as "resolutions": {"sharp": "0.32.6"} — unless you have another idea I can try.

@NikolaRHristov , I'm using Yarn. Thanks for the suggestion, but neither resolutions nor overrides worked with sharp 0.33.0. Maybe I need to leave it set as "resolutions": {"sharp": "0.32.6"} — unless you have another idea I can try.

Well, the version of sharp depends on the AstroCompress and Astro versions and whichever you choose to override. I'm just suggesting overrides as a general solution since resolutions is only supported on Yarn.

I'll see if I can implement something dynamic that overrides the AstroCompress version pre-install to match Astro's sharp version.

Well, the version of sharp depends on the AstroCompress and Astro versions and whichever you choose to override

Yeah, I get that. All I'm saying is that astro build fails with the error when I "choose to override" using 0.33.0, while it works with 0.32.6. In case it helps at all, I'm using astro 3.4.0 and astro-compress 2.2.5. And here is the (truncated) stdout of yarn why sharp:
=> Found<EMAIL_ADDRESS>info Reasons this module exists
   - "astro-compress" depends on it
   - Hoisted from "astro-compress#sharp"

=> Found<EMAIL_ADDRESS>info This module exists because "astro" depends on it.

I'll see if I can implement something dynamic that overrides the AstroCompress sharp version pre-install to match Astro's sharp version.

That would be wonderful. Thanks so much!

{
  "overrides": {
    "astro-compress": {
      "sharp": "0.32.6"
    }
  }
}

Does this work?

No, "overrides": {"astro-compress": {"sharp": "0.32.6"}} didn't work for me. I'll just keep using "resolutions": {"sharp": "0.32.6"} until some later version irons out the issues. thanks again

No, "overrides": {"astro-compress": {"sharp": "0.32.6"}} didn't work for me. I'll just keep using "resolutions": {"sharp": "0.32.6"} until some later version irons out the issues. thanks again

@kswedberg I'm sorry, can I use you to debug something else? Does the resolutions work for sharp 0.33.1?

No problem, happy to help. Trying it with "resolutions": {"sharp": "0.33.1"} did not work, unfortunately. astro build produced this:
12:29:08 PM [vite] Error when evaluating SSR module /Users/kswedberg/Sites/ks/blog.karlswedberg.com/astro.config.mjs: failed to import "astro-compress"
|- Error: Could not load the "sharp" module using the darwin-arm64 runtime
Possible solutions:

... [a bunch of stuff] ...

[astro] Unable to load your Astro config
 error   Could not load the "sharp" module using the darwin-arm64 runtime

Thank you so much! We will investigate this further, for now glad that the resolutions hotfix works.


Include times in admin notification, fixes #1035

In the emailed report to admins, report the project entry
created_at time and (where appropriate) the difference
between the created_at time and the achieved_passing_at time.
This will help highlight "unusually speedy" badge achievements,
e.g., so that we can review those more closely.
Signed-off-by: David A. Wheeler<EMAIL_ADDRESS>

+1. Love that distance_of_time_in_words helper.


Hide empty legend titles

LLegend shows an empty row for an empty string, so we pass nothing for empty titles instead

Ah, didn't know about that, thanks!


Running pychromeless locally on Docker

Me again.
For those of us who are unfamiliar with Docker, can you please tell us how to test our app locally in it, or how to set up environment variables? I followed the Docker tutorial, but it's not clear.
For example I keep getting these errors:
When I run:
docker-compose run lambda
I get:
botocore.exceptions.ClientError: An error occurred (InvalidAccessKeyId) when calling the GetObject operation: The AWS Access Key Id you provided does not exist in our records.
Or when I run:
docker run --rm -v "$PWD":/var/task lambci/lambda:python3.6 lambda_function.lambda_handler
I get:
Unable to import module 'lambda_function': No module named 'selenium'
Thanks for any additional help you can spare.

Hey! I've updated the README because it was WRONG (sorry). You just need to do:
make fetch-dependencies
make docker-run
Aaand you're good to go!

Thank you for this. Recently I did:
make fetch-dependencies
make docker-build
make docker-run
And it worked. Is the docker-build step unnecessary?


[Contextual Cursor]

This issue is for the external plugin: Contextual Cursor
When you use fullscreen mode either on streched or resizable the cursor is proportionally big, too big compared to the normal cursor.

This is due to using an overlay instead of changing the cursor, to make it cross platform. Due to this, it is not possible to fix.


Put the Tokel logo in the app somewhere

"Can you make me an issue for that so I don't forget about it?"

Easy one we can add to the next release!


The new list-copier tries to set readonly indexers

Describe the bug
If a class has a get-only indexer, generated code throws exception.
To Reproduce
        public Something this[string headerName]
        {
            get
            {
                    ///some code here
            }
            ///no set!
        }

Environment:

Mapperly Version: 2.7.0
.NET 6
Target Framework: .net6.0

Some actual code to provoke this exception:
using System.Net.Http;

[Mapper]
public static partial class PropertyMapper
{
	public static partial void CopyTo(HttpRequestMessage source, HttpRequestMessage dest);
}

HttpRequestMessage is a class has some properties with get-only indexers that your library tries to clone.

Thank you for reporting this, I'll look into it.

#286 implements that no mapping is generated if the target does not implement IDictionary<,> and therefore Mapperly should not write to read only dictionaries anymore. However, while testing this fix with HttpRequestMessage I encountered another Bug which isn't that easy to fix: https://github.com/riok/mapperly/issues/286. I'll look into that one as soon as I have the time.


Pull parameter default from config

Depends on #681

Coverage increased (+0.11%) to 77.4% when pulling 4061ed95908f43960b55a2369abcf2c9f79b4987 on erikbern/configuration into 1939a06c1b205fcb20ba80223537345333f45c6a on master.

Any thoughts?

Can we merge this?

Looks useful. I was thinking about this when arriving at the default values in Range stuff (kind of suited for our arbitrary situation and maybe unsuitable for others'): can have the hardcoded default overridable by a configured default, overridable by the dynamically specified value? Great if this does that; a test with both a hardcoded default and a configured value is lacking.
(Not sure of the usefulness of config_path, wonder why it was invented. Letting user come up with accidental complexity, namely mismatch between config section names and task names - for what purpose?)

config_path is an artifact from before, now that they are pulled by default, we should probably deprecate config_path.
Let me add a couple of more tests then merge this.


Problem and suggestion about icheck

Hello, I use this great template, but icheck plugin is problematic. So I wrote pure css bootstrap checkboxes   to use in my project, later I have published it, and can suggest you to consider it for checkboxes  and radios. You can check it out at: icheck-bootstrap

and what is the question?

Thanks for the suggestion. We'll keep it in mind.


Add config for TravisCi

Codecov Report

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

@@            Coverage Diff            @@
##             master       #1   +/-   ##
=========================================
  Coverage          ?   87.93%
=========================================
  Files             ?        1
  Lines             ?      116
  Branches          ?        0
=========================================
  Hits              ?      102
  Misses            ?        8
  Partials          ?        6

Continue to review full report at Codecov.

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


Add widget id to debug mode for dash controls

Randomly failed test, not connected with changes in PR


Port tslint-microsoft-contrib's no-increment-decrement

See https://github.com/palantir/tslint/issues/1142.
I remember reading a book (JavaScript: The Good Parts? Clean Code? The Art of Unit Testing?) by one of the big influential programming folks and encountering a passage arguing in favor of this type of rule. The justification was something like these points:

It's easy to type +i or -i instead of --i
Standardizing widths of increments/decrements makes them easier to read
It's easy to forget the difference between ++i and i++

Perhaps this rule should be named increment-decrement and only ban the pre- operators, to offend fewer people? The first point is the only one of the three I feel would withstand a real debate.

It's easy to type +i or -i instead of --i

when used as an ExpressionStatement, no-unused-expression would warn you about that.
This is only an issue when assigning the result.
But even then you get an error from prefer-const because you never reassign the variable (if this is the only place you might have modified it).

Counter-example:
const items = ["apple", "banana", "cherry"];
let index = 0;

while (+index) {
    // ^
    if (items[index][0] === "b") {
        ++index;
    }
}

Fixed by #3575.


AttributeError: module 'PIL.Image' has no attribute 'fromArray'

Hi,
it looks like a great project! many thanks! Nevertheless I get the above mentioned error.
any Idea what's wrong?
Many thanks!

changing fromArray() to fromarray() fixed the error for me.


Lack of ARM64 Builds for Windows

Godot version
4.3stable
Plugin version
v1.0.6
System information
Windows 11 Arm64 (Snapdragon X Elite)
Issue description
I’ve noticed that the repository currently does not provide binaries for the ARM64 architecture on Windows.
ARM64-based devices are becoming more prevalent, especially with the new Copilot + PCs.
I tried to compile the project for this architecture, but I'm don't know the C++ environment well, and I didn't succeed.
Steps to reproduce
Add the webrtc-native to your godot project.
An error pop in the output panel:
No GDExtension library found for current OS and architecture (windows.arm64) in configuration file: res://webrtc/webrtc.gdextension
Minimal reproduction project
No response

I honestly have no experience in building form windows arm64 either.
I guess the first step would be to be able to build libdatachannel (finding the right CMake flags) and openssl (which uses its own build system).
It also seems that mingw support for windows arm64 is still experimental, so this probably needs to use MSVC for now, which is a pain for me to test.
In short, help is welcome.

I finally managed to build the project for Windows Arm64 !
I slightly modified the OpenSSL configuration and indeed used MSVC.
So, I will open a pull request.


therapy icons

would be great to have icons for various therapies / clinics (e.g. physical therapy, occupational therapy, speech therapy, vision clinic, urology clinic, etc). i'm looking to put these icons on signage around the hospital for those with literacy challenges. Let me know what you think!

Great idea. A Canadian doctor friend of mine created this list below that we're going to work on. Do you see any that are missing?

We just pushed a bunch of these live @davidpearl to healthicons.org under the subheading SPECIALTIES @eleary4


Only "secure" image filename is produced; no option to not randomize filename

This started in version 3.0. The integration only produces a randomized image filename (ie. b332f214-f0ed-402a-9167-0d2ebfe076cf.gif) and there is no option checkbox in the integration configuration to NOT produce this (ie. use the filename mail_today.gif).
The randomized filename does not work with the companion Lovelace card.

This is by design.
For the Card, please open issues in the Card repo: https://github.com/moralmunky/Home-Assistant-Mail-And-Packages-Custom-Card


Capture all tickets from February tg

Capture the whiteboard tasks from the February meeting at NCEAS, and categorize and tag them properly.

the whiteboard transcription document is here: https://docs.google.com/document/d/1QFTzxiVFtvzurkPg4EzcCojm2wK0ax_QD4-pl8I46dU/edit


Secure file storage and access

I would really like options for secure file storage and access. Currently files are available on a server publicly if someone knows their 24 character without any kind of authorisation. Realistically that is not what we call secure.

This sounds like a duplicate of https://github.com/matrix-org/matrix-doc/issues/701


[RFE] Add Version to subscribe

Is your feature request related to a problem? Please describe.
Now that some events are being deprecated and receiving new versions, TESjs needs to allow for specification of the subscription version when subscribing to an event.
Describe the solution you'd like
Add a third argument to subscribe for the subscription version
Additional context
https://dev.twitch.tv/docs/eventsub/eventsub-subscription-types/#channelfollow

This can be made non-breaking by having a default value of "1"


Intuit's Oauth2 support- Request 3- Keys for Authorize url have _

Intuit's spec/implementation https://developer.intuit.com/app/developer/qbo/docs/develop/authentication-and-authorization/openid-connect#prerequisites
uses _ each of the keys used in AuthorizeUrl call-
GET https://appcenter.intuit.com/connect/oauth2?
client_id=Q3ylJatCvnkYqVKLmkH1zWlNzNWB5CkYB36b5mws7HkKUEv9aI&
response_type=code&
scope=openid%20email&
redirect_uri=https://www.mydemoapp.com/oauth-redirect&
state=security_token%3D138r5719ru3e1%26url%3Dhttps://www.mydemoapp.com/oauth-redirect&
Using object extra is the only way of supporting this variation in names?  Also, clientId and responseType is required in IdentityModel, so can't leave those out while building the Authorize request url.
Our devs would prefer standard keys already defined in the class. Please suggest.

We support everything directly besides state. State could be passed in via the extra parameters as you mentioned.
Two thoughts here

state was designed to help keep the client state in a stateless environment - I don't see a benefit of using it in a stateful client.
if you absolutely need the state, why don't you simply provide an extension method to OidcClient that has all the parameters you need?

well - identitymodel is spec compliant. These are the official names.

You are right, but unfortunately I cannot use them due to the underlying implementation. What do you suggest then? Just like above, will be supporting CreateAuthorizeUrl or RequestTokenAsync with 'all' not required params also so that if I send only extra object values, only those are used?

yea - extension method for RequestUrl and use RequestTokenAsync - then pass in all your custom values. give that a try. Please open a new issue if there is a problem.

Thanks!


[BUG] Teach SeasonalMovingAverageModel to work with out-sample predictions

🐛 Bug Report
Currently SeasonalMovingAverageModel and MovingAverageModel don't pass some tests written in test_inference. Try also to make sure that in-sample predictions work as expected.
Expected behavior
Tests are passed.
How To Reproduce
Run tests in test_inference.
Environment
No response
Additional context
No response
Checklist

[X] Bug appears at the latest library version

I've run the tests in tests/test_models/test_inference and after thorough examination have found that both MovingAverageWindow and SeasonalMovingAverageWindow pass all the tests.
However, upon examining these models more closely, I found two behaviours I believe to be bugs.
I link Colab notebook with their reproduction and description.

It seems that we already completed this particular task, I'm sorry we forgot to close it.
About colab notebook. It seems like you are confusing classes. There are two separate classes: SeasonalMovingAverageModel and _SeasonalMovingAverageModel. The first is etna model that works with TSDataset, the second is private class that holds implementation details and works with pd.DataFrame.

It seems like this task is already done during Inference track.


Serve Model Azure Functions How-To

Serve Model Azure Functions How-To
Internal Review URL
#AB1484868

Make sure it is clear that these are alternatives
Pre-trained model.
- Use the ML.NET Sentiment Analysis tutorial to build your own model or (<-- or similar)
- Download this [pre-trained sentiment analysis machine learning model]

This is a real nit but can we indent the code samples to align with the numbered list. I think this would increase readability

This is wrong doing that on a static object because the PredictionEngine is not threadsafe.
_predictionEngine = _mlContext.Model.CreatePredictionEngine<SentimentData, SentimentPrediction>(_model);

You could probably use a static PredictionEnginePool because the pool is thread-safe.
Similar to how we use it in ASP.NET Core but without using dependency injection.
But I would really prefer to use dependency injection in the Azure Function in a similar way it is explained in this blog post:
https://blog.wille-zone.de/post/azure-functions-dependency-injection/
Then, the implementation using PredictionEnginePool would be very similar to the one recommended in ASP.NET Core WebAPI or apps.


[WIP] Begin cutover to new method sigs - Pinecone 0.1.6

Work in Progress [WIP] Please do no merge yet. Opening this for awareness and to start running it through CI.
Closes https://github.com/hwchase17/langchainjs/issues/2437
These changes cutover the Pinecone integration to the Pinecone TypeScript client version 0.1.6-dev.

Thanks! Converted it to a draft PR, please convert back and ping when ready for review.

Hi @zackproser -- do you still plan on releasing this change or will you target the new 1.0.0 release instead?

Hi @zackproser -- do you still plan on releasing this change or will you target the new 1.0.0 release instead?

Hi - I'm actually planning on targeting 1.1.1 now.

Hi @zackproser -- do you still plan on releasing this change or will you target the new 1.0.0 release instead?

Hi - I'm planning on targeting the latest v1.0.0 release now.

Make that version v1.0.1 - with the latest fixes :)


Regenerate Spanner Gapic to use new HTTP headers.

Updates #1268
cc @bjwatson @swcloud

Ah, I forgot that spanner also creates an instance of OperationsClient, and like Speech this PR depends on google-gax 0.8.1.


[RFC] luci-nginx and luci-ssl-nginx: collect files from nginx and uwsgi-cgi packages

I am new to contributing and I am not sure if this change is a good idea. I tried to collect the files that are needed to run luci on nginx from the uwsgi-cgi and the nginx package and would move it here. See my other draft PRs in packages, too:
https://github.com/openwrt/packages/pull/9736 ([RFC] package etesync and its dependencies and move luci on nginx)
uwsgi: add packages with modules and nginx: enable /etc/nginx/conf.d directory.
I would do this only if Ansuel approves it since it is his project.
See also the topic at:
https://forum.openwrt.org/t/package-etesync-and-its-dependencies/42269
I hope it is understandable what I try to do.

Sorry for the confusion by opening and closing this PR repeatedly. I wanted to change it to a draft PR.

Please add a prefix in front of your commits.  Have a look at the commit history
luci-:

Thank you :-) I will change the prefixes. Right now I am changing the PR for the packages repository …

The uwsgi-cgi-plugin would be in the uwsgi Makefile of my other commits in the packages repository. Right now I am redoing them with better commit messages, it will be in https://github.com/peter-stadler/packages/tree/uwsgi – right now I want to change username/email before the PR …

Sorry for the late answer (it took me some time to fix the CircleCI build for the etesync-server work).
I made the init files executable, changed the commit messages and included luci-app-opkg as dependency (it is in the standard luci collection, too). Thank you @feckert.
Both commits depend on the uwsgi-cgi-plugin of the modular uwsgi and that the /etc/nginx/conf.d directory is enabled.

@jow- do you think this is a good idea (regarding also the openwrt guidelines)?

Closing this as its outside of the scope of LuCI. The necessary work appears to have happened in the package repo now.


Fixes saving noise-levels in guidance_eval

Minor fix in saving noise_levels for wandb. Doesn't affect anything other than guidance_eval, which is only used in zero123 so far.
Fixes issue of saving noise level with guidance_eval image : earlier, the image was saved, read, noise level was printed on it, then the image was saved again. wandb does not read the second image, so the noise level does not appear.
Now the noise level is first printed on image, then image is saved.
BEFORE:

AFTER:

Done!

Thanks, it may be a bit confusing that the noise level is written on top of the noiseless NeRF render, but I understand why you did it that way, so for now, that's fine. ;-)


Mapster.Tool or TextTemplate with EnableNonPublicMembers

Backstory
I am looking at following a pattern that will allow keeping domain objects fully encapsulated and use different objects to represent tables (single responsibility) the setters on the domain object are currently protected.
Issue
I like the idea of using generated code and have experimented with Mapster.Tool, but found it doesn't appear to support EnableNonPublicMembers.
Question
Is there way to make generated code that will compile? The compile error is:

CS0272	The property or indexer 'User.Username' cannot be used in this context because the set accessor is inaccessible

Example Source:
    public class User
    {
        public User(string username)
        {
            Username = username;
        }

        public int UserID { get; protected set; }
        public string Username { get; protected set; }
    }

    public class DBUser
    {

        public int UserID { get; set; }
        public string Username { get; set; }
    }

    [Mapper]
    public interface IUserMapper
    {
        User Map(DBUser value);
    }

    public class TableRegister : IRegister
    {
        public void Register(TypeAdapterConfig config)
        {
            config.ForDestinationType<User>()
                .MapToConstructor(true)
                .EnableNonPublicMembers(true);
        }
    }

Example Generated:
    public partial class UserMapper : IUserMapper
    {
        public User Map(DBUser p1)
        {
            return p1 == null ? null : new User(p1.Username)
            {
                UserID = p1.UserID,
                Username = p1.Username
            };
        }
    }

Codegen doesn't support non-public property. It is language limitations not library limitations. Please try changing to internal. If not work, please map using Adapt method instead of code geneation.


DBZ-2221 Wait for keepalive thread initialization

Test failure unrelated, fixed via https://issues.redhat.com/browse/DBZ-2262.
LGTM, merging. Thx, @jpechane!


Known issue where some resources do not load in the portal is obsolete since 1802

@yudomsft : Thanks for your contribution! The author, @brenduns, has been notified to review your proposed change.

#sign-off


.

Bug type
Component
Component name
MudDataGrid
What happened?
Nullable ints work in MudNumericField, but not in their MudDataGrid equivalent.
I believe to have implemented a workaround after some quick googling that led me to this post by gzaxx.
In MudBlazor/Components/DataGrid/Cell.cs - line 98 I found the line
property.SetValue(_item, Convert.ChangeType(value, property.PropertyType));
which after replacing with
property.SetValue(_item, Workaround.ChangeType(value, property.PropertyType));
and the added class:
internal class Workaround
{
    public static object ChangeType(object value, Type conversion)
    {
        var t = conversion;

        if (t.IsGenericType && t.GetGenericTypeDefinition().Equals(typeof(Nullable<>)))
        {
            if (value == null)
            {
                return null;
            }

            t = Nullable.GetUnderlyingType(t);
        }

        return Convert.ChangeType(value, t);
    }
}

again worked as expected.
As this seems to be a deeper issue I did not file a pull request.
Some further documentation I found to be relevant:
MudBlazor discord post
Microsoft - API Documentation - Convert.ChangeType Method
Microsoft - System/Convert.cs source
Expected behavior
The behaviour was expected to be at least similar to its fully functional counterpart MudNumericField.
Reproduction link
https://try.mudblazor.com/snippet/mawGkycHUfsnsgLV
Reproduction steps

Create a new class MyClass that contains a nullable int.
Create a MudDataGrid, set its EditMode to DataGridEditMode.Cell, and assign a collection of MyClass objects to it.
Observe how the value of the nullable integer cannot be changed, it crashes instead.

Relevant log output
System.InvalidCastException: Invalid cast from 'System.Double' to 'System.Nullable`1[[System.Int32, System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e]]'.
   at System.Convert.DefaultToType(IConvertible value, Type targetType, IFormatProvider provider)
   at System.Double.System.IConvertible.ToType(Type type, IFormatProvider provider)
   at System.Convert.ChangeType(Object value, Type conversionType, IFormatProvider provider)
   at System.Convert.ChangeType(Object value, Type conversionType)
   at MudBlazor.Cell`1.<NumberValueChangedAsync>d__15[[Try.UserComponents.__Main.MyClass, Try.UserComponents, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null]].MoveNext()
   at Microsoft.AspNetCore.Components.ComponentBase.CallStateHasChangedOnAsyncCompletion(Task task)
   at MudBlazor.MudBaseInput`1.<SetValueAsync>d__159[[System.Double, System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e]].MoveNext()
   at MudBlazor.MudBaseInput`1.<SetTextAsync>d__106[[System.Double, System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e]].MoveNext()
   at Microsoft.AspNetCore.Components.ComponentBase.CallStateHasChangedOnAsyncCompletion(Task task)
   at MudBlazor.MudBaseInput`1.<SetValueAsync>d__159[[System.String, System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e]].MoveNext()
   at MudBlazor.MudInput`1.<UpdateValuePropertyAsync>d__70[[System.String, System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e]].MoveNext()
   at MudBlazor.MudBaseInput`1.<SetTextAsync>d__106[[System.String, System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e]].MoveNext()
   at MudBlazor.MudInput`1.<OnChange>d__16[[System.String, System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e]].MoveNext()
   at Microsoft.AspNetCore.Components.ComponentBase.CallStateHasChangedOnAsyncCompletion(Task task)
   at Microsoft.AspNetCore.Components.RenderTree.Renderer.GetErrorHandledTask(Task taskToHandle, ComponentState owningComponentState)

Version (bug)
5.0.9
Version (working)
No response
What browsers are you seeing the problem on?
Firefox, Chrome, Microsoft Edge
On what operating system are you experiencing the issue?
Windows, Android
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Since the NumberValueChangedAsync function seems to be relatively isolated, I can't think of any reason why your solution wouldn't work for this.  Ideally, it may be worth exploring a greater usage of nullable value types in the future.  But for the interim, you have my +1 on this.

Was this solved in the latest release?

Closing this, since the datagrid had several reworks after and the codebase significantly changed.

Good strategy lol

Good strategy lol

What kind of attitude is this, removing the title and the description?
Your bug report was created for version 5.0.9, which is no longer supported. The DataGrid underwent significant rework twice after that version. This means that your reproduction code is no longer compatible with the latest changes, making it hard to check.
The code part you linked has also changed, and if I recall correctly, it now takes nullability into account, meaning that the bug might not even exist anymore.
That's why the issue was closed. However, if it does still exist, you can create a new issue with evidence that it is still broken in the latest version. If you have a solution on how to fix it, you can create a pull request instead of waiting for someone else to fix it for you, which might never happen.


upload serving.yaml artificat to the job workflow

if for some reason goreleaser does not upload the artifact to the release we can recover from the workflow

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@46996dd). Click here to learn what that means.
The diff coverage is n/a.

:exclamation: Current head 28f655f differs from pull request most recent head f0b6b16. Consider uploading reports for the commit f0b6b16 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main      #99   +/-   ##
=======================================
  Coverage        ?   70.76%
=======================================
  Files           ?        4
  Lines           ?      195
  Branches        ?        0
=======================================
  Hits            ?      138
  Misses          ?       40
  Partials        ?       17

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.