Go compiler is built using clang, but buildGoPackage function then uses gcc, which can clash

Issue description
The NixOS go package uses clang while building the compiler. When that go compiler later is used to compile go programs which make use of C bindings the go compiler will also invoke a C compiler. It seems that this C compiler needs to match the one that was used at build time. If one uses stdenv.mkDerivation or buildGoPackage one gets gcc instead, which leads to failed builds in some cases.
Steps to reproduce
I have an example Go package over here: https://github.com/javgh/nixpkgs/blob/master/pkgs/applications/altcoins/go-ethereum.nix .
For that expression I manually set CC to clang to make it work. Removing that preBuild step leads to a vanilla buildGoPackage environment with gcc and the build fails.
Tweaking CC in the package directly works for me, but it would probably make more sense to set this correctly already in buildGoPackage. I'm pretty new to NixOS, so I haven't yet attempted to do that myself.
Technical details

System: 17.09pre106916.c5badb123a (Hummingbird)
Nix version: nix-env (Nix) 1.11.9
Nixpkgs version: 17.09pre106916.c5badb123a
Sandboxing enabled: false

Yet the line https://github.com/NixOS/nixpkgs/blob/19ebff1d95dae17cff81f8e9c133d82827317ed2/pkgs/development/compilers/go/1.8.nix#L137 says $CC is cc unless on Darwin.

(I usually build go-ethereum with go get in order to use the HEAD version)

You are right, I'm not sure what made me think that clang is used, I jumped to some conclusions. Sorry for the misleading title, I'm just very confused now.
Can you successfully compile this file: https://raw.githubusercontent.com/golang/go/master/misc/cgo/errors/issue14669.go ?
When I do this within nixshell -p go gcc it fails, but when I use nixshell -p go clang it works. I guess that made me think that clang is the matching compiler. But why doesn't it work with the matching gcc compiler but does in fact work with, clang, a different compiler than used at build time?

Can you successfully compile this file: https://raw.githubusercontent.com/golang/go/master/misc/cgo/errors/issue14669.go ?

# command-line-arguments
cgo-dwarf-inference:2:8: error: enumerator value for '__cgo_enum__0' is not an integer constant
cgo-dwarf-inference:5:8: error: enumerator value for '__cgo_enum__2' is not an integer constant
cgo-dwarf-inference:8:2: error: initializer element is not constant
cgo-dwarf-inference:8:2: note: (near initialization for '__cgodebug_data[0]')
cgo-dwarf-inference:10:2: error: initializer element is not constant
cgo-dwarf-inference:10:2: note: (near initialization for '__cgodebug_data[2]')

Further check,
ldd `which go`
	linux-vdso.so.1 (0x00007fffb1770000)
	libpthread.so.0 => /nix/store/63gvnrj4z154kpyjpskl6s0hwmyx9x0w-glibc-2.25/lib/libpthread.so.0 (0x00007f236c314000)
	libc.so.6 => /nix/store/63gvnrj4z154kpyjpskl6s0hwmyx9x0w-glibc-2.25/lib/libc.so.6 (0x00007f236bf75000)
	/nix/store/63gvnrj4z154kpyjpskl6s0hwmyx9x0w-glibc-2.25/lib/ld-linux-x86-64.so.2 (0x00007f236c532000)

proves that go is compiled with gcc by default on NixOS.
Running nix-shell -p go clang wouldn't do since I have go installed deterministically from configuration.nix. I had also uncommented go from configuration.nix and rerun nix-shell -p go clang, to no avail.
note: here is an alternative path for gcc for now (that you had figured out the clang one but I can't reproduce), I have geth running just fine after a make from the source dir.

So you see the error as well for issue14669.go, but are able to successfully run make in the go-ethereum repository?
This is what I see when I add go and gcc to configuration.nix and attempt to build the master branch from go-ethereum:
git clone https://github.com/ethereum/go-ethereum.git
cd go-ethereum
make

build/env.sh go run build/ci.go install ./cmd/geth
>>> /nix/store/k114bnw0gzag4b07492dgxi2gs858xrl-go-1.8/share/go/bin/go install -ldflags -X main.gitCommit=40976ea1a009f1358bcc6f85db97efd61868dd4f -v ./cmd/geth
github.com/ethereum/go-ethereum/common/hexutil
github.com/ethereum/go-ethereum/common/math
github.com/ethereum/go-ethereum/rlp
github.com/ethereum/go-ethereum/vendor/github.com/go-stack/stack
github.com/ethereum/go-ethereum/vendor/gopkg.in/karalabe/cookiejar.v2/collections/prque
github.com/ethereum/go-ethereum/crypto/sha3
github.com/ethereum/go-ethereum/vendor/github.com/aristanetworks/goarista/monotime
github.com/ethereum/go-ethereum/vendor/github.com/rcrowley/go-metrics
github.com/ethereum/go-ethereum/crypto/secp256k1
github.com/ethereum/go-ethereum/crypto/randentropy
github.com/ethereum/go-ethereum/common/mclock
github.com/ethereum/go-ethereum/event
github.com/ethereum/go-ethereum/vendor/github.com/pborman/uuid
github.com/ethereum/go-ethereum/log
github.com/ethereum/go-ethereum/common
github.com/ethereum/go-ethereum/vendor/github.com/rjeczalik/notify
github.com/ethereum/go-ethereum/vendor/golang.org/x/crypto/pbkdf2
github.com/ethereum/go-ethereum/cmd/internal/browser
github.com/ethereum/go-ethereum/vendor/golang.org/x/crypto/scrypt
github.com/ethereum/go-ethereum/params
github.com/ethereum/go-ethereum/vendor/github.com/edsrzf/mmap-go
github.com/ethereum/go-ethereum/common/bitutil
github.com/ethereum/go-ethereum/vendor/github.com/syndtr/goleveldb/leveldb/util
github.com/ethereum/go-ethereum/vendor/github.com/syndtr/goleveldb/leveldb/comparer
github.com/ethereum/go-ethereum/vendor/github.com/syndtr/goleveldb/leveldb/storage
github.com/ethereum/go-ethereum/vendor/github.com/golang/snappy
github.com/ethereum/go-ethereum/vendor/github.com/hashicorp/golang-lru/simplelru
github.com/ethereum/go-ethereum/vendor/golang.org/x/net/context
github.com/ethereum/go-ethereum/vendor/github.com/hashicorp/golang-lru
github.com/ethereum/go-ethereum/vendor/github.com/rs/xhandler
github.com/ethereum/go-ethereum/vendor/github.com/syndtr/goleveldb/leveldb/cache
github.com/ethereum/go-ethereum/vendor/github.com/syndtr/goleveldb/leveldb/filter
github.com/ethereum/go-ethereum/vendor/golang.org/x/net/websocket
github.com/ethereum/go-ethereum/vendor/gopkg.in/fatih/set.v0
github.com/ethereum/go-ethereum/vendor/golang.org/x/crypto/ripemd160
github.com/ethereum/go-ethereum/vendor/github.com/rs/cors
github.com/ethereum/go-ethereum/vendor/github.com/syndtr/goleveldb/leveldb/opt
github.com/ethereum/go-ethereum/vendor/github.com/huin/goupnp/httpu
github.com/ethereum/go-ethereum/vendor/github.com/huin/goupnp/scpd
github.com/ethereum/go-ethereum/vendor/github.com/huin/goupnp/soap
github.com/ethereum/go-ethereum/vendor/golang.org/x/net/html/atom
github.com/ethereum/go-ethereum/vendor/github.com/syndtr/goleveldb/leveldb/errors
github.com/ethereum/go-ethereum/vendor/github.com/huin/goupnp/ssdp
github.com/ethereum/go-ethereum/vendor/github.com/syndtr/goleveldb/leveldb/iterator
github.com/ethereum/go-ethereum/vendor/github.com/syndtr/goleveldb/leveldb/journal
github.com/ethereum/go-ethereum/vendor/golang.org/x/net/html
github.com/ethereum/go-ethereum/trie
github.com/ethereum/go-ethereum/vendor/github.com/rcrowley/go-metrics/exp
github.com/ethereum/go-ethereum/rpc
github.com/ethereum/go-ethereum/vendor/golang.org/x/text/encoding/internal/identifier
github.com/ethereum/go-ethereum/vendor/golang.org/x/text/transform
github.com/ethereum/go-ethereum/vendor/golang.org/x/text/internal/utf8internal
github.com/ethereum/go-ethereum/vendor/github.com/syndtr/goleveldb/leveldb/memdb
github.com/ethereum/go-ethereum/metrics
github.com/ethereum/go-ethereum/vendor/github.com/syndtr/goleveldb/leveldb/table
github.com/ethereum/go-ethereum/vendor/golang.org/x/text/encoding
github.com/ethereum/go-ethereum/vendor/golang.org/x/text/runes
github.com/ethereum/go-ethereum/vendor/golang.org/x/text/internal/tag
github.com/ethereum/go-ethereum/vendor/golang.org/x/text/encoding/internal
github.com/ethereum/go-ethereum/vendor/golang.org/x/text/language
github.com/ethereum/go-ethereum/vendor/golang.org/x/text/encoding/charmap
github.com/ethereum/go-ethereum/vendor/golang.org/x/text/encoding/japanese
github.com/ethereum/go-ethereum/vendor/github.com/syndtr/goleveldb/leveldb
github.com/ethereum/go-ethereum/vendor/golang.org/x/text/encoding/korean
github.com/ethereum/go-ethereum/vendor/golang.org/x/text/encoding/simplifiedchinese
github.com/ethereum/go-ethereum/vendor/golang.org/x/text/encoding/traditionalchinese
github.com/ethereum/go-ethereum/vendor/golang.org/x/text/encoding/unicode
github.com/ethereum/go-ethereum/vendor/github.com/jackpal/go-nat-pmp
github.com/ethereum/go-ethereum/p2p/netutil
github.com/ethereum/go-ethereum/vendor/gopkg.in/sourcemap.v1/base64vlq
github.com/ethereum/go-ethereum/vendor/github.com/robertkrimen/otto/token
github.com/ethereum/go-ethereum/vendor/github.com/robertkrimen/otto/dbg
github.com/ethereum/go-ethereum/vendor/github.com/robertkrimen/otto/registry
github.com/ethereum/go-ethereum/vendor/gopkg.in/sourcemap.v1
github.com/ethereum/go-ethereum/vendor/github.com/karalabe/hid
github.com/ethereum/go-ethereum/log/term
github.com/ethereum/go-ethereum/vendor/github.com/mattn/go-colorable
github.com/ethereum/go-ethereum/vendor/github.com/robertkrimen/otto/file
github.com/ethereum/go-ethereum/vendor/gopkg.in/urfave/cli.v1
github.com/ethereum/go-ethereum/les/flowcontrol
github.com/ethereum/go-ethereum/vendor/github.com/robertkrimen/otto/ast
github.com/ethereum/go-ethereum/internal/jsre/deps
github.com/ethereum/go-ethereum/vendor/golang.org/x/text/encoding/htmlindex
github.com/ethereum/go-ethereum/vendor/golang.org/x/net/html/charset
github.com/ethereum/go-ethereum/vendor/github.com/mattn/go-isatty
github.com/ethereum/go-ethereum/internal/web3ext
github.com/ethereum/go-ethereum/vendor/github.com/peterh/liner
github.com/ethereum/go-ethereum/vendor/github.com/robertkrimen/otto/parser
github.com/ethereum/go-ethereum/vendor/github.com/fatih/color
github.com/ethereum/go-ethereum/vendor/github.com/huin/goupnp
github.com/ethereum/go-ethereum/vendor/golang.org/x/tools/go/ast/astutil
github.com/ethereum/go-ethereum/vendor/github.com/huin/goupnp/dcps/internetgateway1
github.com/ethereum/go-ethereum/vendor/github.com/huin/goupnp/dcps/internetgateway2
github.com/ethereum/go-ethereum/ethdb
github.com/ethereum/go-ethereum/internal/debug
github.com/ethereum/go-ethereum/vendor/golang.org/x/tools/imports
github.com/ethereum/go-ethereum/vendor/github.com/robertkrimen/otto
github.com/ethereum/go-ethereum/vendor/github.com/maruel/panicparse/stack
github.com/ethereum/go-ethereum/vendor/github.com/mattn/go-runewidth
github.com/ethereum/go-ethereum/vendor/github.com/mitchellh/go-wordwrap
github.com/ethereum/go-ethereum/vendor/github.com/naoina/go-stringutil
github.com/ethereum/go-ethereum/vendor/github.com/nsf/termbox-go
github.com/ethereum/go-ethereum/vendor/github.com/naoina/toml/ast
github.com/ethereum/go-ethereum/p2p/nat
github.com/ethereum/go-ethereum/vendor/github.com/naoina/toml
github.com/ethereum/go-ethereum/vendor/github.com/gizak/termui
# github.com/ethereum/go-ethereum/vendor/github.com/karalabe/hid
cgo-dwarf-inference:2:8: error: enumerator value for '__cgo_enum__0' is not an integer constant
cgo-dwarf-inference:10:8: error: enumerator value for '__cgo_enum__7' is not an integer constant
cgo-dwarf-inference:18:2: error: initializer element is not constant
cgo-dwarf-inference:18:2: note: (near initialization for '__cgodebug_data[0]')
cgo-dwarf-inference:25:2: error: initializer element is not constant
cgo-dwarf-inference:25:2: note: (near initialization for '__cgodebug_data[7]')
github.com/ethereum/go-ethereum/internal/jsre
github.com/ethereum/go-ethereum/console
github.com/ethereum/go-ethereum/crypto
github.com/ethereum/go-ethereum/core/types
github.com/ethereum/go-ethereum/crypto/ecies
github.com/ethereum/go-ethereum/p2p/discover
github.com/ethereum/go-ethereum/p2p/discv5
github.com/ethereum/go-ethereum/accounts/abi
github.com/ethereum/go-ethereum
github.com/ethereum/go-ethereum/core/state
github.com/ethereum/go-ethereum/core/vm
github.com/ethereum/go-ethereum/accounts
github.com/ethereum/go-ethereum/ethclient
github.com/ethereum/go-ethereum/accounts/keystore
github.com/ethereum/go-ethereum/p2p
github.com/ethereum/go-ethereum/consensus/misc
github.com/ethereum/go-ethereum/consensus
github.com/ethereum/go-ethereum/eth/downloader
github.com/ethereum/go-ethereum/consensus/ethash
github.com/ethereum/go-ethereum/consensus/clique
github.com/ethereum/go-ethereum/eth/fetcher
github.com/ethereum/go-ethereum/accounts/abi/bind
github.com/ethereum/go-ethereum/whisper/whisperv5
github.com/ethereum/go-ethereum/core
github.com/ethereum/go-ethereum/eth/filters
github.com/ethereum/go-ethereum/internal/ethapi
github.com/ethereum/go-ethereum/miner
github.com/ethereum/go-ethereum/light
github.com/ethereum/go-ethereum/eth/gasprice
util.go:44: exit status 2
exit status 1
make: *** [Makefile:15: geth] Error 1

But this series of commands work for you with go and gcc in configuration.nix?

So you see the error as well for issue14669.go, but are able to successfully run make in the go-ethereum repository?

Yes
Update: I pulled from the latest master and found the same err:
# github.com/ethereum/go-ethereum/vendor/github.com/karalabe/hid
cgo-dwarf-inference:2:8: error: enumerator value for '__cgo_enum__0' is not an integer constant
cgo-dwarf-inference:10:8: error: enumerator value for '__cgo_enum__7' is not an integer constant
cgo-dwarf-inference:18:2: error: initializer element is not constant
cgo-dwarf-inference:18:2: note: (near initialization for '__cgodebug_data[0]')
cgo-dwarf-inference:25:2: error: initializer element is not constant
cgo-dwarf-inference:25:2: note: (near initialization for '__cgodebug_data[7]')

From git-bisect, it is due to this commit, in particular the line https://github.com/ethereum/go-ethereum/commit/6ec81352560e3e4268855115eda9d78f6e27875f#diff-d9b21908b8e4e705df4660d35ceacfdfR40.

I am going to close this issue, as it has a really misleading title and wrong speculation. I have written up a - hopefully - better description of the issue here: https://github.com/NixOS/nixpkgs/issues/25959 .


Issue#38

add reset button for code editor

Still having same issues from this PR
https://github.com/testbrew/testbrew/pull/42#issuecomment-1428829071


Multiple regressions in 1.0.0

Test url:
http://www.appleinsider.com/articles/11/12/14/inside_anobit_why_apple_is_investing_in_flash_ram_technology.html

Result for both 0.6.1 and 1.0.0:
https://gist.github.com/5d1776577a95927e98b3

Regressions:

title is not found
main content is not fetched(it was working in 0.6.1)
multiple irrelevant results was resulted as body
\r\n is not converted to \n
multiple empty <p> and <div> tags
<script> tag is still present

Additional info: it was introduced in c06234eddd205c433fa6ba3771b5c644fd69420b (earlier revisions are working correctly)

Found difference: http://nodejs.org/docs/latest/api/http.html#http.request
link uses pathname key, but http requires path key. Adding on line to node/index.js:27
link.path = link.pathname;

results in correct title and content. Other methods like protocol is also unsupported - need check for auth.
I will check other regressions with path set to http://www.appleinsider.com to see if that was regressions from earlier version.

It looks that all listed problems are valid for 0.6.1 and http://www.appleinsider.com :/
So - to make summary:

http and https require path param instead of pathname and thus 1.0.0 will ignore any provided path.
all versions of readabilitySAX after 0.6.1 (can't test earlier releases - there is some strange error when I try) will let \r\n, empty tags and <script> tags in certain circumstances

I can't reproduce the error, the object returned by url.parse should be accepted by http.request (to quote the documentation: "options align with url.parse()"). What version of node are you using?
Filtering tags doesn't work if they aren't lowercase - I set the corresponding option in node/index.js, so all tags will be lowercased.

node.js version 0.4.7 - the same as Heroku(some of my projects are hosted there) Requiring node version >= 0.6 will prevent all projects from running there so please don't set this like in version 1.0.1(attached commit) This will require just mentioned small addition to support all versions of node from 0.3.1 onward.

Okay, I fixed it.


[PDI-20086] Add a kettle parameter to control whether we wait for a lock

file before starting karaf.

 Note:

Frogbot also supports Contextual Analysis, Secret Detection, IaC and SAST Vulnerabilities Scanning. This features are included as part of the JFrog Advanced Security package, which isn't enabled on your system.

🐸 JFrog Frogbot


Prepare master for next version

Prepare master for next version

@bot-gradle test and merge

I've triggered a build for you. Click here to see all failures if there's any.


Network Error

Undiscord Version
5.2.3
Browser
Firefox
Extension
Violent Monkey
Reproduction steps
Select "me" "current" "current" Delete.
Actual results
Fetching messages...
Grand total: 14326	(Messages in current page: 25	To be deleted: 0	Skipped: 25)	offset: 0
Delete delay: 1000ms, Search delay: 30000ms	Last Ping: 203ms, Average Ping: 203ms
Rate Limited: 0 times.	Total time throttled: 0h 0m 0s.
Estimated time remaining: 9h 33m 44s
There's nothing we can delete on this page, checking next page...
Skipped 25 out of 25 in this page.	(Offset was 0, ajusted to 25)
Waiting 30.00s before next page...
Fetching messages...
Grand total: 14326	(Messages in current page: 25	To be deleted: 0	Skipped: 25)	offset: 25
Delete delay: 1000ms, Search delay: 30000ms	Last Ping: 215ms, Average Ping: 204ms
Rate Limited: 0 times.	Total time throttled: 0h 0m 0s.
Estimated time remaining: 9h 34m 1s
There's nothing we can delete on this page, checking next page...
Skipped 25 out of 25 in this page.	(Offset was 25, ajusted to 50)
Waiting 30.00s before next page...
Fetching messages...
Search request threw an error:	{"fileName":"","lineNumber":0,"columnNumber":1,"message":"NetworkError when attempting to fetch resource."}
CoreException	{"fileName":"","lineNumber":0,"columnNumber":1,"message":"NetworkError when attempting to fetch resource."}
Expected results
Messages to be deleted.
Bug affects

[X] DMs
[ ] DMs (group)
[ ] Channels
[ ] Channel (NSFW)

Additional information & file uploads
The program will usually work for about a page's worth before stopping due to the error. But lately, it won't even delete one message.

so far reducing the search delay to <10 seconds has fixed the issue.

above only works for about 20 messages then I get
Ended because API returned an empty page.
[End state]	{"running":true,"delCount":27,"failCount":0,"grandTotal":14199,"offset":143,"iterations":10,"_seachResponse":{"analytics_id":"850ddf2df57d3f91063703a981ee71e9","doing_deep_historical_index":false,"total_results":14197,"channels":[{"id":"980772990415437854","type":1,"last_message_id":"1157083335810617384","flags":0,"last_pin_timestamp":"2023-08-09T03:46:50+00:00","recipients":[{"id":"565725426488705024","username":"oreoz87","global_name":"oreoz","avatar":"29f86412210be2c06fc41e186d445ea1","avatar_decoration_data":{"asset":"a_10b9f886b513b77ccdd67c8784f1a496","sku_id":"1144059132517826601"},"discriminator":"0","public_flags":64}]}],"messages":[]},"_messagesToDelete":[],"_skippedMessages":[]}

above works until about 20 messages then I get "Ended because API returned an empty page."

Yo I need some help, can you add me on Discord?
cleanestturban


 Many are skipped. I have a separate PR coming up that fixes the semantics of mustaches in attributes (and hence all of skipped tests in has-block/has-block-params) and hopefully can reduce some of code duplication that is present between the mustache/attributeMustache/SubExpression handlers in the template-compiler.

ping @wycats


Reconsider window.addEventistener() behavior

All window.addEventistener() calls are redirected to document here.
I've no idea why, but it does break valid code like:
window.addEventListener('scroll', scroll);
window.addEventListener('resize', update);

Except for this bit, your shims work really well.
:thumbsup:

Thank you! :)))
As you probably know, IE8 lacks support for window.addEventListener... but window.attachEvent isn't a seamless replacement. Specifically, its event bubbling is flawed, and certain events dispatched on the window global aren't communicated to the document.
I wasn't aware there were issues with this; could you please elaborate on how this is breaking? Perhaps provide me with a few stack-traces?

There are no stack traces, the events simply don't fire. Small code example that handles onload, onscroll and onresize:
  function ready(fn) {
    if (document.readyState == 'complete') {
      fn();
    }
    else {
      window.addEventListener('load', fn);
    }
  }

  ready(function() {
    window.addEventListener('scroll', function() {
        console.log('scroll');
    });
    window.addEventListener('resize', function() {
        console.log('resize');
    });
  });

Those events are window object events, document object doesn't fire them, hence the breakage.
Honestly, I'm not familiar with IE8 quirks. If redirection is important for other use cases, maybe you can skip it if the event is a window-only event?

I'm a little too busy right now to look at this closer (will investigate later), but I vaguely recall IE8 may have used a different mechanism for monitoring page-scrolling. In the typically broken fashion of Internet Explorer, the event was somewhere else (possibly the body, or a non-standard interface).
My memory of that's become faded over the years, though, but it could be worth investigating if it leads you to an interim solution. :)
It probably won't take me long to get back to this, I just hate knowing something I've written is broken and affecting users.

Oh, don't worry about. I've simply replaced document with window in those two lines and my vanilla JS scroll plugins written for IE10+ work perfectly in IE8.
So in fact, IE8 scroll works just like any other browser, and for scroll and load events, attachEventHandler is equivalent to addEventListener.

Wait, it's actually possible IE doesn't dispatch any events when scrolling, period. I recall scrollTop not existing in Explorer prior to IE9, which might explain why you're not witnessing any response from the handlers.

IE8 does dispatch onscroll just fine, and scrollTop (aka scrollY) is included in your shims!
It just works as long as you don't redirect window.addEventHandler to document.attachEvent.

Erh... yeah, you're right. I really shouldn't think in more than 10 headspaces... sorry (scrollY might've been what I was recollecting inaccurately).
Rerouting was a solution for a different problem; it may have been getting click events to bubble. I was developing this around the time these shims were published on their own, and it's possible the accordion folds weren't opening on IE8 when I tested.
Either way, it was a kludgey solution which certainly needs revising; thanks for bringing it to my attention. :)


README: fix website url

Hello,
the README link to the website seems to be broken. I am not sure if there used to be some image.
I changed it to the direct URL.

The link works for me:

@sudo-bmitch this should not be merged. Why did you approve?

@sudo-bmitch this should not be merged. Why did you approve?

@vsoch from https://github.com/opencontainers/specs.opencontainers.org, the link on the line above the "Usage" section is broken.


Add text and example about short-circuiting

Fixes #22548

File Type
File Name
Published Url

Content
docs/csharp/language-reference/operators/member-access-operators.md
https://review.docs.microsoft.com/en-us/dotnet/csharp/language-reference/operators/member-access-operators?branch=pr-en-us-25644

Thanks @BillWagner!


Read session from redis

See #102

@FinnStutzenstein This works. But you have to set the environment variable SECRET_KEY. And the current key_salt is django.contrib.sessionsSessionStore, this has probably to be changed.
I am not sure if it is a good idea, that on validation errors the user-id 0 is used. I think it would be better, if the client got some sort of error. For example, if you set the environment variable SECRET_KEY to a wrong value, it seems to work but the autoupdate-server uses the uid 0 for all users.
I would propose, that when there is not session data in redis for a given session ID or the session can not be validated, then 403 is returned.

I would propose, that when there is not session data in redis for a given session ID or the session can not be validated, then 403 is returned.

Ok for me. Since I do not know the session management in django too well, we will see if it works. (If not, the server code must be adopted IMO since 403 is the better and clearer solution)

@FinnStutzenstein should be ready. Could you also test it in the prod build with a real session?

Connected to OpenSlides at http://server:8000

Do you still need to know the server? You do not send a WhoAmI anymore, right?

Connected to OpenSlides at http://server:8000

Do you still need to know the server? You do not send a WhoAmI anymore, right?

I do not. I removed the environment variabes, the log line and any other place it was used.
All should be ready now. Please test/review again

Looks good, dev and prod tested. Waiting for next week to merge.


how to Reset ttl ?

my config.yml ttl is 1440 ,but i need after login success , reset ttl 604800 , and logout reset ttl is 1440 , I added an event after login successfully, but it was unsuccessful
please help me ( chinese: 我默认设置的是1440，但是我想在用户登录成功后，redis的ttl变成604800，退出时，又变成1400，我在登录成功后加了一个事件，但是不成功）

use session proxy!

The issue is unclear to me and therefore being closed to clean up the issue backlog. Please open a new issue if you still have questions or requests.


TypeError: Cannot read property 'endsWith' of undefined

Bug report
webpack support node version >= 10
but in "lib/NormalModuleFactory.js" code use "endsWith"

What is the current behavior?
If the current behavior is a bug, please provide the steps to reproduce.

What is the expected behavior?

Other relevant information:
webpack version:  5.92.0
Node.js version: 12.21.0
Operating System:
Additional tools:

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/endsWith

4.0.0 (Released 2015-09-08)


Change api generation

This PR switches our api generation from typescript-jquery to typescript-fetch so that our generated enums are strings. It also makes our generated types use snake_case for naming, so that it matches what our backend expects.

the most relevant changed files are src/utils/api.ts, apiTypes.ts, auth.ts
I also changed the bat file for generation: generate-types.bat

I also updated our precommit hooks to exclude the types folder from formatting (see .pre-commit-config.yaml)
You can ignore the types folder when reviewing these files.

this PR is blocked by https://github.com/battlecode/galaxy/pull/671.

seems good, would wait for lowell though


fix: fix minor issues in settings and fluent version.

'.setup.models' should be '.settings.setup.models' from versions 25.1 and above.
solver.settings.setup.models.viscous.model.set_state("k-epsilon"), etc. were returning some values in PyConsole. This has been rectified.
FluentVersion enum's string representation has been updated as follows:
>>> str(FluentVersion.v241) -> 'Fluent version 2024 R1'

I renamed the PR to "fix: fix..." from "refactor: fix" since the two meanings are very different.


Sentry report is not including set context

Environment

Elixir version (elixir -v): Elixir 1.8.1
Erlang/OTP version (erl): Erlang/OTP 21
Sentry version (mix deps): sentry 7.0.6
Operating system: MacOS

Description
I am using sentry's Sentry.Context.set_user_context under exq processes.  I have set the context by using
def perform(args) do
    Sentry.Context.set_user_context(args)
    IO.inspect Sentry.Context.get_all()
    ......
end

, and during my function's running, i am seeing
%{
  breadcrumbs: [],
  extra: %{},
  request: %{},
  tags: %{},
  user: %{"email" =><EMAIL_ADDRESS>}

in my console.  But on my sentry issues website, I am not seeing any user tag.
Expected Behavior
user tag on sentry issues website
Actual Behavior
Not seeing user tag
Steps to Reproduce

@LiyangW Thanks for opening an issue!
My hunch is that this is because the job is running in a separate process than the one that sends the exception, and the context gets lost since the context is stored in the process dictionary.
Potential solutions are to make use of the callback described here, though it may duplicate then.
Are you using Sentry.LoggerBackend?

@mitchellhenke Thanks for answering me back!
I am using
{:ok, _} = Logger.add_backend(Sentry.LoggerBackend)
inside my App.Appplication's start function.
Originally, I thought it is because of the separate processes issue as well, so I actually moved the Sentry.Context.set_user_context to the worker functions.
Therefore the exq command
{:ok, _ack} = Exq.Enqueuer.enqueue(Exq, "file_uploading", FileUploadWorker, [args])
create a worker process by calling worker's perform function and the perform function contains  Sentry.Context.set_user_context.
In my understanding, the above procedure would make sure each process has its user_context set up, after being spawned.  Thanks for the help!

Yep, that's the right idea, but unfortunately the worker process is separate from the LoggerBackend process that receives the event.
Because of that, I'm not sure how best to handle this.  try/rescue with a manual sending of event and context would "solve" it in that the context would then be included, but is more of a hassle.
I'll be able to dig into this a bit further to see if a more complete solution is feasible.

Would it make sense to potentially explore :persistent_store which was introduced in OTP 21.2? It's a VM-wide storage accessible from all processes/modules.

Indeed, something backed with a core feature and has no dependencies would be preferable.
(linking #383, as these are related)

Any update on this issue? Not having context makes debugging much harder for us.

There were some improvements over the past 6 months or so that should resolve most of these issues.
@1player are you using exq and experiencing this problem?

No I'm not using exq, I'm just noticing my async processes sometimes fail, but the context isn't sent to Sentry. It used to, the code hasn't changed in months, so the only suspect is the Sentry client library, which has been updated since, failing to push that additional context data.

@1player Hm, that would be a significant bug if the tests are passing and you're experiencing that. Could you provide a minimal way to reproduce?

My bad, there is no regression. I'm having the same issue as others where context is set on another process than the one that actually sends the exception, and I'll have to work around that until this library is able to work across processes (using :persistent_term) or the docs clarify the best way to deal with this common problem.

Hi there, I hit the same problem with Exq, The only workaround that works for me is "double exception" with:
  def perform(args) do
    #code
  rescue
    ex ->
      Sentry.capture_exception(ex,
        stacktrace: __STACKTRACE__,
        extra: %{....},
        tags: %{worker: __MODULE__}
      )

    raise ex # <- this create another event without context, the "common flow".
  end

Any other idea?

I found a small change that may fix this in Exq: https://github.com/akira/exq/pull/436

With that pull request merged, I'm going to close this issue for now 🙂

With that pull request merged, I'm going to close this issue for now 🙂


  All doors and windows are hardwired.
In addition to security functions, I would like for my Homebridge server to know when a window/door sensor changes state.  In this way, I want to be able "turn off the AC if the door is subsequently opened for longer than 5 minutes", or "send a message if windows are open and the temperature drops too low".  I know that Qolsys can communicate with certain AC systems, but Homekit/Homebridge is the center of my automation logic, and Qolsys does not integrate with my HVAC.
Can you confirm that your plug-in enables such functionality?   Is access to the Qolsys local, on WiFi, or remote through a Qolsys site or Alarm.com?  Any other salient tips for my intended use?
Thanks!

  All doors and windows are hardwired.
In addition to security functions, I would like for my Homebridge server to know when a window/door sensor changes state.  In this way, I want to be able "turn off the AC if the door is subsequently opened for longer than 5 minutes", or "send a message if windows are open and the temperature drops too low".  I know that Qolsys can communicate with certain AC systems, but Homekit/Homebridge is the center of my automation logic, and Qolsys does not integrate with my HVAC.
Can you confirm that your plug-in enables such functionality?   Is access to the Qolsys local, on WiFi, or remote through a Qolsys site or Alarm.com?  Any other salient tips for my intended use?
Thanks!

Hi,
First of all, access to the Qolsys panel is local through the fairly limited Control4 interface.  A remote subscription is not needed. Dealer code for your panel is required to enable this local integration.
I have not been experimenting with wired sensors myself, but from my understanding you would be using a iq hardwire translator. From a plugin perspective, these wired sensors will be exposed as regular wireless sensors without additional Homebridge configuration.
Each Qolsys sensor/zone gets exposed as a Homekit sensor and regular Homekit automations may be used.
I don't think you can achieve your particular needs with regular basic Homekit automations though. The other way around could be to convert automations to shortcuts and scripting or use homebridge delay dummy switches for complex patterns.
I hope it answers your questions :)

Hi,
To clarify, the limited Control 4 interface is a public API created by Control 4 to talk to the Qolsys?  Hence the limitation?
Yes, there is a wired-wireless bridge required, to expose each wired sensor to the Qolsys panel.  My goal is HomeKit arm/disarm, knowing the sensor state, and detecting whether the state changes.  This also enables, e.g., "When backdoor opens, turn on backdoor light."
I was planning to use Homebridge delay switches to enable.  (Delays should really be native to Homekit.)

Control4 wrote a driver in 2019 allowing direct connexion to supported Qolsys panels
(Qolsys wrote a server running on port 12345 of their panels allowing interraction with that Control4 driver)
API/integration is officialy undocumented by Qolsys.
Basic functionality has been documented by people on Home Assistant forum.
Connexion is all local on your home network. All events on your panel will be pushed almost instantaneously to Homekit.
Homebridge plugin allows you to:

Arm/Disarm partitions on your Qolsys panel through the Homekit security accessory and automations.
Get notifications in Homekit when an alarm is triggered on your panel
Get an almost instantaneous Homekit sensor state change for any sensors events on your Qolsys panel and trigger associated Homekit automations.

Anything related to zwave on Qolsys panel is not supported by the integration and this homebridge plugin.
To sum it up, you can achieve your goal with this plugin !

Thank!


MediaTypeReader does not work as expected

When certain RequestMappings do not specify a produces/consumes the ApiListing level produces and consumes do not reflect that set of expected produces and consumes correctly
class SomeController {

    @RequestMapping(value ="/method1", produces={"image/jpeg"})
    public void method1(){...}

    @RequestMapping(value ="/method2")
    public void noProducesOrConsumes(){...}
}

should render
apiVersion: "1.0",
swaggerVersion: "1.2",
basePath: "/context-path",
resourcePath: "/method1",
produces: [
"*/*", //<- Needs this
"image/jpeg"
],
consumes: [
"*/*"
],
apis: [{
    "path": "/method2",
    "produces": [
        "*/*",  //<- Needs this
    ],
    "consumes": [
       "*/*",  //<- Needs this
    ]
}]

This is not in 1.0.0 :(
See: https://github.com/springdox/springdox/blob/master/swagger-springmvc/src/main/java/com/mangofactory/swagger/readers/MediaTypeReader.java
Line 60.
Do you know any workaround?


Client crashes

Getting constant client crashes:
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x40 pc=0x454bc3b]

goroutine 1180 [running]:
github.com/broderickhyman/albiondata-client/client.(*httpUploaderPow).uploadWithPow(0xc0000b32e0, 0x0, 0x0, 0x0, 0x0, 0xc0000adab0, 0x10, 0xc0000161e0, 0xed, 0xf0, ...)
        /usr/src/app/client/uploader_http_pow.go:81 +0x33b
github.com/broderickhyman/albiondata-client/client.(*httpUploaderPow).sendToIngest(0xc0000b32e0, 0xc0000161e0, 0xed, 0xf0, 0x47729e3, 0x13)
        /usr/src/app/client/uploader_http_pow.go:134 +0x111
github.com/broderickhyman/albiondata-client/client.sendMsgToUploaders(0xc0000161e0, 0xed, 0xf0, 0x47729e3, 0x13, 0xc0001b2a20, 0x1, 0x1)
        /usr/src/app/client/dispatcher.go:116 +0xae
github.com/broderickhyman/albiondata-client/client.sendMsgToPublicUploaders(0x46a51c0, 0xc0006e4aa0, 0x47729e3, 0x13, 0xc0001bf200)
        /usr/src/app/client/dispatcher.go:68 +0x1e9
github.com/broderickhyman/albiondata-client/client.operationAuctionGetOffersResponse.Process(0xc0004380b0, 0x1, 0x1, 0xc0001bf200)
        /usr/src/app/client/operation_auction_get_offers.go:58 +0x392
created by github.com/broderickhyman/albiondata-client/client.(*Router).run
        /usr/src/app/client/router.go:52 +0x2d5

Last messages from the debug log:
DEBU[2023-01-27T10:25:31+05:00] Got response to AuctionGetOffers operation...
INFO[2023-01-27T10:25:31+05:00] Sending 18 market requests to ingest
DEBU[2023-01-27T10:25:31+05:00] GETTING POW
DEBU[2023-01-27T10:25:31+05:00] EventDataType: ERROR - map[0:2196826 1:[3 -13 82 112 -24 38 0 -37 8 -89 -27 -53 -63 107 -23 55 66 34 102 102 52 65 90 -75 -85 -63 -4 -68 70 66]]
DEBU[2023-01-27T10:25:31+05:00] EventDataType: ERROR - map[0:2197741 1:[3 -29 73 107 -24 38 0 -37 8 113 -12 93 -63 112 -109 20 66 -19 -102 -103 65 65 0 -14 -116 -63 -60 88 50 66]]
DEBU[2023-01-27T10:25:31+05:00] EventDataType: ERROR - map[0:2197714 1:[3 -29 -116 75 -24 38 0 -37 8 66 -35 45 -63 -17 -63 -69 65 -89 -52 -52 56 65 -36 36 -121 -63 34 97 -102 65]]
DEBU[2023-01-27T10:25:31+05:00] EventDataType: [10]evActiveSpellEffectsUpdate - map[0:2185660 1:[695 969 311 371 514 431] 2:[100 184.92752 243.11807 243.11807<PHONE_NUMBER>7] 3:[100 146.82451 174.19029 174.19029<PHONE_NUMBER>9] 4:[66649249 67154428 67369245 67388382 59898185 67389213] 5:[1 1 3 4 1 4] 7:[75 18 36 73 -110 36 73 -110 36 73 -110 36 1] 8:[17208836] 9:[17] 10:[0 0] 252:10]
DEBU[2023-01-27T10:25:31+05:00] EventDataType: [20]evCastHits - map[0:2185660 1:2185660 2:431 3:1 4:0 252:20]
DEBU[2023-01-27T10:25:31+05:00] EventDataType: ERROR - map[0:2194099 1:[3 67 22 113 -24 38 0 -37 8 37 112 -64 -63 -83 105 47 66 26 0 0 26 65 -25 -72 -94 -63 -45 54 67 66]]
DEBU[2023-01-27T10:25:31+05:00] EventDataType: ERROR - map[0:2197587 1:[3 83 8 101 -24 38 0 -37 8 18 -39 -87 -63 -44 -1 105 63 -56 0 0 4 65 -23 -22 -81 -63 24 -49 -121 63]]
DEBU[2023-01-27T10:25:31+05:00] EventDataType: [13]evCastStart - map[0:2164282 1:67389171 4:67389971 5:2920 6:-101 7:2164282 8:92 9:13 252:13]
DEBU[2023-01-27T10:25:31+05:00] EventDataType: ERROR - map[0:2197637 1:[3 -29 -100 114 -24 38 0 -37 8 55 18 60 -62 -109 89 -15 65 -100 0 0 -80 64 -79 33 62 -62 104 81 -20 65]]
DEBU[2023-01-27T10:25:31+05:00] EventDataType: ERROR - map[0:2197584 1:[3 67 -121 115 -24 38 0 -37 8 63 -42 73 -63 62 21 68 66 -45 0 0 -80 64 -55 103 -116 -63 32 71 78 66]]
DEBU[2023-01-27T10:25:31+05:00] EventDataType: ERROR - map[0:2192861 1:[3 -45 87 119 -24 38 0 -37 8 -118 -56 -109 -63 -46 82 52 66 -98 -102 -103 21 65 -68 -82 -33 -63 76 -60 9 66]]

% uname -a
Darwin macbook.local 22.2.0 Darwin Kernel Version 22.2.0: Fri Nov 11 02:03:51 PST 2022; root:xnu-8792.61.2~4/RELEASE_ARM64_T6000 arm64

How often does this happen when you say constant?
I've seen this in rare occasions but failed to reproduce it willingly.

Also, note it's running on a M1/M2 Mac. It's possible it could be a emulation thing since the binary is not targeted for ARM.

Well it happens every few hours.
No idea what leads to the crash yet.

I suspect your internet connection to be unstable? Or can you rule that out?
https://github.com/ao-data/albiondata-client/blob/40dead766ecc2be22cb01d27ca3a12a7161336b6/client/uploader_http_pow.go#L81
This would happen if resp.Body was nil (e.g. no response from server)
The error handling can be improved here

I suspect your internet connection to be unstable? Or can you rule that out?
https://github.com/ao-data/albiondata-client/blob/40dead766ecc2be22cb01d27ca3a12a7161336b6/client/uploader_http_pow.go#L81
This would happen if resp.Body was nil (e.g. no response from server) The error handling can be improved here

Thats definitely could be the case, my internet connection is flaky

INFO[2023-02-19T17:27:56+03:00] Sending 50 market requests to ingest
ERRO[2023-02-19T17:27:56+03:00] Error in Pow Get request: Get "http://www.albion-online-data.com:4223/pow": read tcp <IP_ADDRESS>:58772-><IP_ADDRESS>:4223: read: connection reset by peer
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x40 pc=0x81b98a]
goroutine 25 [running]:
github.com/broderickhyman/albiondata-client/client.(*httpUploaderPow).uploadWithPow(0xc00000e810, 0x0, 0x0, 0x0, 0x0, 0xc00034a4f0, 0x10, 0xc000293500, 0x301b, 0x3500, ...)
/home/runner/work/albiondata-client/albiondata-client/client/uploader_http_pow.go:81 +0x34a
github.com/broderickhyman/albiondata-client/client.(*httpUploaderPow).sendToIngest(0xc00000e810, 0xc000293500, 0x301b, 0x3500, 0x93a36a, 0x13)
/home/runner/work/albiondata-client/albiondata-client/client/uploader_http_pow.go:134 +0x111
github.com/broderickhyman/albiondata-client/client.sendMsgToUploaders(0xc000293500, 0x301b, 0x3500, 0x93a36a, 0x13, 0xc00017ea30, 0x1, 0x1)
/home/runner/work/albiondata-client/albiondata-client/client/dispatcher.go:116 +0xb4
github.com/broderickhyman/albiondata-client/client.sendMsgToPublicUploaders(0x8a2360, 0xc0000e66c0, 0x93a36a, 0x13, 0xc00018f200)
/home/runner/work/albiondata-client/albiondata-client/client/dispatcher.go:68 +0x1e8
github.com/broderickhyman/albiondata-client/client.operationAuctionGetRequestsResponse.Process(0xc000179880, 0x32, 0x32, 0xc00018f200)
/home/runner/work/albiondata-client/albiondata-client/client/operation_auction_get_requests.go:44 +0x3a5
created by github.com/broderickhyman/albiondata-client/client.(*Router).run
/home/runner/work/albiondata-client/albiondata-client/client/router.go:52 +0x2a5

Agan crash
INFO[2023-02-19T17:34:20+03:00] Starting Albion Data Client, version: 0.1.23
INFO[2023-02-19T17:34:20+03:00] This is a third-party application and is in no way affiliated with Sandbox Interactive or Albion Online.
INFO[2023-02-19T17:34:20+03:00] Additional parameters can listed by calling this file with the -h parameter.
INFO[2023-02-19T17:34:20+03:00] Watching Albion
INFO[2023-02-19T17:35:52+03:00] Updating player location to @ISLAND@ec5aa555-e889-4f19-8eb4-576e81c643e5.
INFO[2023-02-19T17:35:52+03:00] Updating player ID to 4edb1b3e-0f2b-4341-b4ad-eb90dd59910d.
INFO[2023-02-19T17:35:52+03:00] Updating player to Umabaka.
INFO[2023-02-19T17:36:12+03:00] Updating player location to 4000.
INFO[2023-02-19T17:36:13+03:00] Updating player location to 4000.
INFO[2023-02-19T17:36:28+03:00] Updating player location to 4002.
INFO[2023-02-19T17:36:33+03:00] Sending 50 market requests to ingest
ERRO[2023-02-19T17:36:34+03:00] Error in Pow Get request: Get "http://www.albion-online-data.com:4223/pow": read tcp <IP_ADDRESS>:35308-><IP_ADDRESS>:4223: read: connection reset by peer
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x40 pc=0x81b98a]
goroutine 28 [running]:
github.com/broderickhyman/albiondata-client/client.(*httpUploaderPow).uploadWithPow(0xc00000e810, 0x0, 0x0, 0x0, 0x0, 0xc00033b7e0, 0x10, 0xc0003b7500, 0x312e, 0x3500, ...)
/home/runner/work/albiondata-client/albiondata-client/client/uploader_http_pow.go:81 +0x34a
github.com/broderickhyman/albiondata-client/client.(*httpUploaderPow).sendToIngest(0xc00000e810, 0xc0003b7500, 0x312e, 0x3500, 0x93a36a, 0x13)
/home/runner/work/albiondata-client/albiondata-client/client/uploader_http_pow.go:134 +0x111
github.com/broderickhyman/albiondata-client/client.sendMsgToUploaders(0xc0003b7500, 0x312e, 0x3500, 0x93a36a, 0x13, 0xc00017ea30, 0x1, 0x1)
/home/runner/work/albiondata-client/albiondata-client/client/dispatcher.go:116 +0xb4
github.com/broderickhyman/albiondata-client/client.sendMsgToPublicUploaders(0x8a2360, 0xc0003788b8, 0x93a36a, 0x13, 0xc00018f200)
/home/runner/work/albiondata-client/albiondata-client/client/dispatcher.go:68 +0x1e8
github.com/broderickhyman/albiondata-client/client.operationAuctionGetRequestsResponse.Process(0xc000179c00, 0x32, 0x32, 0xc00018f200)
/home/runner/work/albiondata-client/albiondata-client/client/operation_auction_get_requests.go:44 +0x3a5
created by github.com/broderickhyman/albiondata-client/client.(*Router).run
/home/runner/work/albiondata-client/albiondata-client/client/router.go:52 +0x2a5

What happens if you goto http://www.albion-online-data.com:4223/pow in a web browser?

Omg Blocked?

Sounds like it. Not by us, but by your ISP/country/etc. I forget exactly where it was, but Russia apparently blocked a bunch of Digital Ocean IP ranges. You can try a full connection VPN that reroutes ALL traffic (TCP/UDP and all ports). Someone was successful with TOR, but I don't know the details of how they got it to work.

WTF
Working with DE vpn. Dunno. Roskomnadzor block domain or ip

Are you able to goto https://www.albion-online-data.com/ successfully?

Try albiondata-client.exe -i http+pow://pow.albion-online-data.com and see if the issue goes away.

Reboot router and working, sorry

Ah yes, the old "Microsoft Fix".. reboot and see if it works. Well, glad your issue is sorted. Others have had this same issue with no resolve yet.
Going to close this since you're working properly now.


[Feature Request] Ignore naming case in comment

Naming cases are checked in comments

I think it would be of greater utility to have Harper ignore commented-out code entirely. This should be possible, but I'd like a bit more clarification on your specific issue first.

That can be good too. But people might type both descriptions and code examples in doc comments, so it would be better to ignore cases but still check on comments. And I might miss-used the word ignore, I should mean to check words by certain cases too, like myUserName and my_user_name should be both valid.


Fix a typo in KeyWrapAlgorithm.xml

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

xml/Microsoft.Azure.KeyVault.Cryptography/KeyWrapAlgorithm.xml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

FYI @shua27 - I'm going to merge this, and your fix will take effect, but it'll get overwritten by our CI build later (since the docs are generated by running MDoc on the binaries).  Never fear, though!  I asked the product team to fix it in the source, so we're all good!
Thanks so much for your contribution! 🙂


Fix order of steps in lint-fmt job

Reworking fixing an error Shon introduced in
https://github.com/ocurrent/ocaml-ci/pull/937
This commit is on live-engine but does not appear on master

The PR to fix this initially in https://github.com/ocurrent/ocaml-ci/pull/942 was into master. Do happen to know how it got removed?

Huh... looking at the history here, I am quite confused about why I didn't merge this into master in the first place via my PR, why it was deployed an not merged etc... :/
I clearly messed up there, so thanks for the fix. IMO, this points to the need for a cleaner CD story here (and a staging env so we don't need to test on prod). But those needs are tracked already. Thank you for the fix and sorry for the trouble!


feat: allow headful run using PUPPETEER_HEADFUL=1

This is a very small change to allow the usage of PUPPETEER_HEADFUL environment variable to run "headful".

It's a good idea to open an issue first for discussion.

[x] Tests pass
[x] Appropriate changes to README are included in PR

zeropaper:main


[main] Upgrade to latest dependencies

Cron -knative-prow-robot
/cc knative-sandbox/networking-wg-leads
/assign knative-sandbox/networking-wg-leads
Produced by: knative-sandbox/knobots/actions/update-deps
Details:
bumping knative.dev/hack 0456e8b...a8ef7b6:
  > a8ef7b6 Skip presubmit tests for .github updates (# 154)
bumping knative.dev/networking 48ac02f...17e1877:
  > 17e1877 upgrade to latest dependencies (# 621)

/lgtm
/approve


Display descriptor

This is the work @DuFF14 has done on addressing #6

Quoting @DuFF14:

Display descriptor example: https://github.com/OSVR/OSVR-Core/blob/DisplayConfigs/apps/displays/HMD.json
Some questions/comments:

Any reason to keep everything wrapped in the top level "hmd" object (not included in this draft)? What other types of displays should be described in this schema?

In this draft, "resolutions" is of type "object" rather than "array". The Unity plugin is set up to handle one resolution. An HMD that supports multiple resolutions requires multiple display descriptors. Would change to "resolution" and type "object" if there will only be one per display descriptor.

Same for "eyes". It is an array in our current examples, but there is only ever one entry in the examples I've seen. centerProjX, centerProjY, rotate180 is applied to both eyes. Are there instances where these values differ per eye? If so, should this always be an array of size two? (Or an object properties that specify left/right?) If not, I would change this to type "object" instead of "array".

Added a meta field with version number. We will still need some more information about eyes. Distance to screen, lens properties?

Can I close this one out? I've put in the v1 schema based on this work, with some fixes, etc. The overall display model re-architecting probably deserves its own issue somewhere else, and the schema for a descriptor will fall out of that.

Yes, this can be closed now.


Bug: pressing 0 for DRO on any axis in Probing tab causes DispatcherException

Happens in 2.0.41 XL only (no DRO present on standard with probing selected)
Method to cause crash:
open iosender
Select Probing tab
Press 0 next to any axis.
Result: DispatcherException.

See issue #273 for resolution. I guess it is soon time to release a new build...

Ah. Sorry for the duplicate post.  I did read through the issues but I somehow missed it.  I'll mark this closed.


release-20.2: s/Active Sessions/Sessions/ and move Tx link

PR: https://github.com/cockroachdb/cockroach/pull/56652
From release notes:

Rename Active Sessions to Sessions on the Session Details page. [#56652][#56652] {% comment %}doc{% endcomment %}

Closing as this doesn't need a docs update.


openvino has no cls_pred_stride

std::vector<HeadInfo> heads_info_{
    // cls_pred|dis_pred|stride
    {"cls_pred_stride_8", "dis_pred_stride_8", 8},
    {"cls_pred_stride_16", "dis_pred_stride_16", 16},
    {"cls_pred_stride_32", "dis_pred_stride_32", 32},
};

how ro check or set in self code?

use mo.py can modify the name of onnx layer name,but not use FP16,use FP32 ,not error

just use mo_onnx.py to get openvino.xml do not include "cls*" or "dis",but onnx include them

It seems that the output names are lost after converting to openvino. I'll check it later.


Use prow-friendly label commands in cherry-pick-pull.sh

How to categorize this PR?

/area dev-productivity
/kind bug
What this PR does / why we need it:
hack/cherry-pick-pull.sh adds labels in a command format that is not accepted by prow, e.g.:
/kind/bug
/area/monitoring

This PR fixes this and switches to the following format:
/kind bug
/area monitoring

Which issue(s) this PR fixes:
Example PR: https://github.com/gardener/gardener/pull/6984

/remove-label area/monitoring
😄

/remove-area monitoring

/test pull-gardener-e2e-kind-migration


Allow arrays in parameter files and support numeric types

Arrays can be supplied to the AWS SDK for the List<> types, so StackMaster should support that.
Also allows numeric types to be passed in and converted to strings before being passed to the AWS SDK . Ref: https://github.com/envato/stack_master/pull/67 and https://github.com/envato/stack_master/issues/66

:+1:


Properly setup Travis CI on this repo

@anthrotype, @madig myself and @davelab6 got into a conversation about setting up Travis CI properly for this repo, but we discussed this in the middle of a pull request thread here:
https://github.com/googlefonts/fontbakery/pull/1900#issuecomment-393600755
Since that PR was merged and closed, I just opened now this issue to keep track on the progress of this task. It seems by @davelab6's https://github.com/googlefonts/fontbakery/pull/1900#issuecomment-395511467 that Dave may have just fixed the problem.
@madig / @anthrotype please let us know here when you feel the system is working well so that we can close the issue. Otherwise, please describe what else might still be missing in the setup.
Cheers!

Works now: https://github.com/googlefonts/fontbakery/pull/1908

thanks!


API broke again?

Seems like they changed the API again...
Getting "Es ist ein Fehler aufgetreten. Realm nicht aktiviert" as HTML response at line 52 in "/usr/local/lib/python3.9/dist-packages/vienna_smartmeter/client.py".
Any fixes yet?
I would like to get my data or else the smartmeter is completely useless for me...
(can't install an IR sensor since the meter is too far away - cabling would be too expensive, no chance for wireless)

@davidschoefberger: I've seen that you where able to fix it in the past.
Would you mind taking a look at it or do you think the Wiener Netze API is unusable for us now?
Right now I'm getting an "IndexError: list index out of range" in line 52 of vienna_smartmeter/client.py"

Forget it... it was my fault!
Seems like the package in pip is outdated. When running the client.py from this repo it works flawlessly.


"hide_menubutton: true" not working

Hi
I'm using dwain's dashboard integration and I'm able to lock an user in the dashboard but the menu button still appears and is the only thing not working. Am I missing something?
Edit: it hides in browser but not on my phone

I am experiencing the same issue. The menu icon will hide on desktop, but be visible (but disabled) on mobile.


Auto-upgrade to <IP_ADDRESS>

Base PullRequest
default branch (https://github.com/http4k/http4k-connect/tree/master)
Command results

Details:

add path
/home/runner/work/_actions/technote-space/create-pr-action/v2/node_modules/npm-check-updates/build/src/bin

./upgrade_http4k.sh <IP_ADDRESS>

Changed files

Changed file:

gradle.properties

:octocat: Repo | :memo: Issues | :department_store: Marketplace

Codecov Report

Merging #184 (b527889) into master (a9d7fd5) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #184   +/-   ##
=========================================
  Coverage     65.22%   65.22%
  Complexity      808      808
=========================================
  Files           415      415
  Lines          4860     4860
  Branches        430      430
=========================================
  Hits           3170     3170
  Misses         1443     1443
  Partials        247      247

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


🌱 Bump CAPI to v1.7.0-rc.1

as title says

/test-ubuntu-integration-main
/test-centos-e2e-integration-main


unic_langid_impl should be re-exported

pub use unic_langid_impl; please

What is your purpose? If you want LanguageIdentifier you can get it from fluent-templates directly, using fluent_templates::LanguageIdentifier.

@mondeja Thank you! :-) My silly workaround was to add unic_langid_impl as a dependency.


Update parameter quoted strings to nameof() operator.

Fixes about two dozen mismatched names.
Based on audit of VerifyThrowArgumentNull, VerifyThrowInternalNull, VerifyThrowInternalLength, VerifyThrowArgumentLength, VerifyThrowArgumentLengthIfNotNull, VerifyThrowArgumentOutOfRange
For #3320

Also some automated removal of unneeded usings (side effect of tooling)


Dependency updates and cleanup

Dependency updates & cleanup
Adding JS sourcemap for chance.min.js

Hmm not really sure why travis is timing out on that 1 test there

Hmm, just tried re-running the test here and it timed out again so it seems repeatable for some reason

Should clarify, I love this cleanup and the source map but I'm hesitant to merge this until we figure out why things are a bit wonky. I can't see any reason that any of your changes would have any impact on the test, but still would like to figure out the issue before merging.

@victorquinn No problem, test works locally just not on Travis. If I get some free time again I'll take a peek at it. Thanks!

Ok, think this should be fixed so I've merged it!

@victorquinn Cook, thanks for taking care of that!


python310Packages.pylast: 5.0.0 -> 5.1.0

Description of changes
https://github.com/pylast/pylast/releases/tag/5.1.0

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 198516 run on x86_64-linux 1

9 packages built:

beets
beets-unstable
home-assistant-component-tests.lastfm
lollypop
mopidy-scrobbler
pithos
python310Packages.pylast
python39Packages.pylast
tauon


chore(release): v4.28.0

Prepares v4.28.0 release
Marks aiven_organization_user_list beta

tests


Can't download from Tumblr - unsupported URL

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2020.05.08
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log

C:\Users\MiningRig\Downloads\yt-dl>youtube-dl -v https://retrogamingblog2.tumblr.com/post/618778238111678464
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', 'https://retrogamingblog2.tumblr.com/post/618778238111678464']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2020.05.08
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.18362
[debug] exe versions: ffmpeg 4.2.1, ffprobe 4.2.1
[debug] Proxy map: {}
[Tumblr]<PHONE_NUMBER>11678464: Downloading webpage
[generic]<PHONE_NUMBER>11678464#_=_: Requesting header
WARNING: Falling back on generic information extractor.
[generic]<PHONE_NUMBER>11678464#_=_: Downloading webpage
[generic]<PHONE_NUMBER>11678464#_=_: Extracting information
ERROR: Unsupported URL: https://retrogamingblog2.tumblr.com/post/618778238111678464#_=_
Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp4z7swgz7\build\youtube_dl\YoutubeDL.py", line 797, in extract_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp4z7swgz7\build\youtube_dl\extractor\common.py", line 530, in extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp4z7swgz7\build\youtube_dl\extractor\generic.py", line 3370, in _real_extract
youtube_dl.utils.UnsupportedError: Unsupported URL: https://retrogamingblog2.tumblr.com/post/618778238111678464#_=_

Description

Hello! I can't download anything from Tumblr. Neither photos nor videos. The error is: Unsupported URL. Could you help me please?

Are you sure this tool is suitable for tumblr downloads? It isn't mentioned in the readme...

youtube-dl does not support downloading photos.


Update to 3.1.4 and build with OpenSSL 3

Update to 3.1.4 and build with OpenSSL 3.
3.1 seems to be compatible with OpenSSL 3. 3.2 is officially supported. I looked in Fedora and they don't have any OpenSSL 3 patches even if they are using OpenSSL 3: https://src.fedoraproject.org/rpms/ruby/tree/f37.

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_82v_9kwyyb/clone/recipe/meta.yaml:41: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_82v_9kwyyb/clone/recipe/meta.yaml:44: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_82v_9kwyyb/clone/recipe/meta.yaml:47: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
INFO: /tmp/abs_82v_9kwyyb/clone/recipe/meta.yaml:86: invalid_url: https://svn.ruby-lang.org/cgi-bin/viewvc.cgi/ : URL domain redirect svn.ruby-lang.org ->  git.ruby-lang.org
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_86weys_b5z/clone/recipe/meta.yaml:41: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_86weys_b5z/clone/recipe/meta.yaml:44: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_86weys_b5z/clone/recipe/meta.yaml:47: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
INFO: /tmp/abs_86weys_b5z/clone/recipe/meta.yaml:86: invalid_url: https://svn.ruby-lang.org/cgi-bin/viewvc.cgi/ : URL domain redirect svn.ruby-lang.org ->  git.ruby-lang.org
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_6bt1z1w4t2/clone/recipe/meta.yaml:41: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_6bt1z1w4t2/clone/recipe/meta.yaml:44: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_6bt1z1w4t2/clone/recipe/meta.yaml:47: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
Errors were found


Removed need for secure random library by using window.crypto's csprng.

What benefit does this have over using secure-random?
That lib already uses window.crypto for browsers (https://github.com/jprichardson/secure-random/blob/master/lib/secure-random.js#L50) plus an equivalent for node.js compatibility. This change breaks node.js.
Please let me know your reasoning if I missed something.


Add OVN TLS certs for metadata agent

Metadata agent requires TLS certs to access the southdb. Also cleanup dup volume mounts in ovn services.

Confirmed metadata API now works in a TLS deployment:
~/install_yamls/devsetup$ ssh <IP_ADDRESS> -l cirros<EMAIL_ADDRESS>password:  $ curl http://<IP_ADDRESS>/openstack 2012-08-10 2013-04-04 2013-10-17 2015-10-15 2016-06-30 2016-10-06 2017-02-22 2018-08-27 2020-10-14 latest$

/lgtm


[Breaking] Change to serialize only dirty attributes

Closes #342.
This changes Ioki::Model::Base to only serialize those attributes, which were explicitly passed to the model.
The current behavior always serializes the whole object:
irb(main):001> Ioki::Model::Operator::Line.new(slug: 'my-line').serialize
=>
{:type=>nil,
 :id=>nil,
 :created_at=>nil,
 :updated_at=>nil,
 :name=>nil,
 :mode=>nil,
 :route_number=>nil,
 :skip_time_window_check=>nil,
 :slug=>"my-line",
 :variant=>nil,
 :line_stops=>nil}

Using omit_if_blank_on or omit_if_nil_on, we can avoid serializing certain attributes. We cannot distinguish whether we want to send an attribute with the value nil, or not send the attribute at all.
This PR solves that issue. The new behavior only serializes the explicitly passed attributes:
irb(main):001> Ioki::Model::Operator::Line.new(slug: 'my-line').serialize
=> {:slug=>"my-line"}

We achieve this by including ActiveModel::Dirty in Ioki::Model::Base and track changes whenever an attribute is set. The serialization then only serializes "dirty" (i.e. changed) attributes.
Additional Changes

Add Ioki::Model::Base.changed_attributes to return only changed attributes

Caveats

This is a breaking change, so we probably should increase our major version.

If you don't want to send an attribute at all, but you already initialized it, you have to manually clear the dirty state. Example:
irb(main):002> line = Ioki::Model::Operator::Line.new(slug: 'my-line')
=> Ioki::Model::Operator::Line:3400 @attributes={:slug=>"my-line"}
irb(main):003> line.serialize
=> {:slug=>"my-line"}

# Now I notice that I don't want to send in `slug` at all.
# I can't set it to `nil`, because then I would still send in `slug: nil`
irb(main):007> line.slug = nil
=> nil
irb(main):008> line.serialize
=> {:slug=>nil}

# The correct way is to use the `ActiveModel::Dirty` API and clear the change state:
irb(main):009> line.clear_slug_change
=> nil
irb(main):010> line.serialize
=> {}

Also I noticed, not all attributes of a model come with a clear_ABC_change method. Why is that? For example, the vehicle model has no change_external_id_change method.

Also I noticed, not all attributes of a model come with a clear_ABC_change method. Why is that? For example, the vehicle model has no change_external_id_change method.

Works on my machine :see_no_evil: Which exact model did you try?
irb(main):003> vehicle = Ioki::Model::Operator::Vehicle.new(external_id: 'test')
=> Ioki::Model::Operator::Vehicle:3440 @attributes={:external_id=>"test"}

irb(main):004> vehicle.serialize
=> {:external_id=>"test"}

irb(main):005> vehicle.clear_external_id_change
=> nil

irb(main):006> vehicle.serialize
=> {}

Also quickly checked with the method attributes and it looks as every attribute has a clear_ .. _change method:
irb(main):019> vehicle.methods.select { |m| m.to_s.include?("clear") }.map { |m| m.to_s.sub(/^clear_/, '').sub(/_change$/, '') } - vehicle.attributes.keys.map(&:to_s)
=> ["changes_information", "attribute_changes"]


how to use on module level?

Hi,
Normally I use
_logger = logging.getLogger(__name__)

on every python module so.
How can I switch to logzero? I am missing the name
thx

You would have to use logzero.setup_logger to define a custom logger, which does take an
optional name argument. From looking at the code the default name for a custom logger actually seems to be __name__.

Yeah, you can either use setup_logger if you want a custom name, or if one logger is enough you could just do from logzero import logger.
https://logzero.readthedocs.io


When using VSCodeVim, making Tab and Shift+Tab work in normal mode as well as insert mode

<Tab> and <Shift+Tab> work in insert mode, but not normal mode. Anyone know how to make them work in vim normal mode?

I couldn't get <Tab> to work with Vim too, but I remapped text-tables.gotoNextCell and text-tables.gotoPreviousCell in the keyboard shortcuts to <Shift+Tab> and <Ctrl-Tab>, respectively. In this way, I can still navigate between the cells, although with slightly different key combinations.


Module renamed to prince_cr

As discussed this will close #32

Please wait. I will silence a waning and add another commit to this PR.

Done


Future IR desiderata.

Using this issue to track bigger enhancements for IR, once we mainline it.
Improved addressing and casting.
In sway-core/src/asm_generation/from_ir.rs there are a lot of address and size calculations going on, where for all of the aggregate element accesses the offsets and sizes are attempted to be put into (usually) 12 bit immediates, so instructions like ADDI or MCPI can be used rather than the slightly more expensive (due to the extra register calculations required) ADD and MCP.
It's quite verbose and complicated and inconsistent and really should be done in IR optimisation passes.  The IRgen can assume the pessimistic register based versions and an optimisation can convert it to the immediate based versions when possible.  This would simplify the ASMgen greatly and probably do a better job.
This would probably require some integer casting ops to be added to IR, so that the types presented to ASMgen are strictly 12 bit integers when required.
Refactor structs while we're at it.
The design choice of using insert_value and extract_value for structs was taken in an effort to keep things simple, but in the end structs are not simple and it's not working very well.  Instead we should simply lay out the structs in memory, and manage the fields with load or store.  get_ptr has been upgraded now to allow casting, so it pretty much resembles LLVM GEP which is the complexity I was trying to avoid in the first place.  It's inevitable though.
Management of the data section.
We load const words from the data section with a LWDataId virtual op at the moment.  This is potentially wasteful for small values, especially if they're constant sizes which fit in 12 bits and therefore probably an imm for an op.
There's the potential to just use $zero or $one instead of the data section, especially for booleans.
Knowing what is const or not and can be put in the data section is currently decided by IRgen and could be refined with optimisation passes.
There's also an issue of initialising constants inside of conditional blocks which could cause undefined behaviour.  See #821.
Removal of call site function inlining and proper paths for symbols.
At the moment the call nodes contain the function body of the callee and not much else about the callee is provided, including its full name or Self name (though that can be derived).  This makes things harder to reason about and therefore optimise.  The IR currently regenerates the callees as IR functions as not doing so (just putting the inlined code at the caller) is actually harder since each of the args to the function have to be given unique local variable names and assigned their proper values before hand (which is what doing a call is).
Eventually we don't necessarily want to inline all functions anyway.
Miscellaneous fixes.

Resolve the aggregates code, stop pretending that structs and arrays are related.
Re-jig the unit tests.  It'd be much easier to just have the original Sway, then check it creates correct IR, and then the correct ASM from there.  Different optimisations could be turned on or off explicitly.

All of the above is also tracked in:

https://github.com/FuelLabs/sway/issues/2819
https://github.com/FuelLabs/sway/issues/2505
https://github.com/FuelLabs/sway/issues/1823
https://github.com/FuelLabs/sway/issues/1557
https://github.com/FuelLabs/sway/issues/1693


Soil Organic Carbon (SOC) Analysis

This is a soil organic carbon analysis on the carbon levels in soil within fermented forests.
pskoulgi/SOCanalysis
Todo;

[x]  Review and Decide if the Data will be Useful to the Master Project

No actual data in this repo. Just analysis scripts.


Bump stable DMD and DUB version

With DUB 1.6.0 (part of the DMD 2.077.0 release), DUB will retry
downloads from the DUB registry and alternatively use fallback mirrors.
../dmd/src/posix.mak uses 2.072.0 atm, hence the bump of the host
version.

From https://circleci.com/gh/dlang/phobos/4633:

Package libdparse not found in registry at http://code.dlang.org/: HTTP request returned status code 502 (Bad Gateway)

Well, things are still broken, but at least we're getting sensible error messages, eh?


chore(release): 1.17.0

See CHANGELOG

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-5lHf64IXfvmr
Commit ID: f57e5c94e8aaba4cdaf7f674988dbbd1e391ec1f
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-5lHf64IXfvmr
Commit ID: f57e5c94e8aaba4cdaf7f674988dbbd1e391ec1f
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

Hmm looking into superchain build failures here https://github.com/aws/jsii/pull/2416/checks?check_run_id=1691864490

Hmm looking into superchain build failures here https://github.com/aws/jsii/pull/2416/checks?check_run_id=1691864490

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-5lHf64IXfvmr
Commit ID: 540d9224b0afa17ee0f9bc1df5b96b6806c38c7f
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-5lHf64IXfvmr
Commit ID: 540d9224b0afa17ee0f9bc1df5b96b6806c38c7f
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-5lHf64IXfvmr
Commit ID: 83997b66fda812b253515bb377c13e364fa3da17
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-5lHf64IXfvmr
Commit ID: 83997b66fda812b253515bb377c13e364fa3da17
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


TVer AttributeError: 'list' object has no attribute 'isdigit'

Checklist

[X] I'm reporting a broken site
[X] I've verified that I'm running yt-dlp version 2022.05.18 (update instructions) or later (specify commit)
[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 the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
Japan
Description
#4033 Still not works with some url like: https://tver.jp/episodes/epswy2pl7x
Link can be opened in browser and downloaded with m3u8 link retrieved in console.
Verbose log
> yt-dlp --geo-bypass-country JP --all-subs https://tver.jp/episodes/epswy2pl7x -v
[debug] Command-line config: ['--geo-bypass-country', 'JP', '--all-subs', 'https://tver.jp/episodes/epswy2pl7x', '-v']
[debug] Encodings: locale cp936, fs utf-8, pref cp936, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2022.05.18 [b14d52355]
[debug] Python version 3.9.6 (CPython 64bit) - Windows-10-10.0.19043-SP0
[debug] Checking exe version: ffmpeg -bsfs
[debug] Checking exe version: ffprobe -bsfs
[debug] exe versions: ffmpeg 4.1.3, ffprobe 4.1.3
[debug] Optional libraries: Cryptodome-3.14.1, brotli-1.0.9, certifi-2018.01.18, mutagen-1.45.1, sqlite3-2.6.0, websockets-10.2
[debug] Proxy map: {}
[debug] Using fake IP <IP_ADDRESS> (JP) as X-Forwarded-For
[TVer] Creating session
[debug] [TVer] Extracting URL: https://tver.jp/episodes/epswy2pl7x
[TVer] epswy2pl7x: Downloading JSON metadata
[TVer] epswy2pl7x: Downloading JSON metadata
ERROR: 'list' object has no attribute 'isdigit'
Traceback (most recent call last):
  File "D:\Python39\lib\site-packages\yt_dlp\YoutubeDL.py", line 1420, in wrapper
    return func(self, *args, **kwargs)
  File "D:\Python39\lib\site-packages\yt_dlp\YoutubeDL.py", line 1490, in __extract_info
    ie_result = ie.extract(url)
  File "D:\Python39\lib\site-packages\yt_dlp\extractor\common.py", line 642, in extract
    ie_result = self._real_extract(url)
  File "D:\Python39\lib\site-packages\yt_dlp\extractor\tver.py", line 82, in _real_extract
    if not r_id.isdigit():
AttributeError: 'list' object has no attribute 'isdigit'

$ git diff
diff --git a/yt_dlp/extractor/tver.py b/yt_dlp/extractor/tver.py
index df12e9702..cebd027c8 100644
--- a/yt_dlp/extractor/tver.py
+++ b/yt_dlp/extractor/tver.py
@@ -76,7 +76,7 @@ def _real_extract(self, url):
                 'Referer': 'https://tver.jp/',
             })
         p_id = video_info['video']['accountID']
-        r_id = traverse_obj(video_info, ('video', ('videoRefID', 'videoID')))
+        r_id = traverse_obj(video_info, ('video', ('videoRefID', 'videoID')), get_all=False)
         if not r_id:
             raise ExtractorError('Failed to extract reference ID for Brightcove')
         if not r_id.isdigit():

@pukkandan Can I commit this directly? Or you'll mix it in the next minor changes?

In the future, if there are bugs in your own commit, feel free fix it directly with a commit message like:
[extractor/tvver] Fix bug in h6837633a4a614920b6e43ffc6b4b8590dca8c9d7


FEATURE REQUEST: support for external yaml files

It would be great if the following could be done with sveltos:
kubectl apply -f https://raw.githubusercontent.com/dragonflydb/dragonfly-operator/main/manifests/dragonfly-operator.yaml
currently what we do is we copy the yaml files and we put them in a configmap.
If we can point the policyRef to a URL that would make things so much easier.

Hi, @gianlucam76! Could you assign this issue to me?


RTD build broken because of "bad" dependencies in umbrella package

https://readthedocs.org/projects/google-cloud-python/builds/5997521/

Same here https://travis-ci.org/getsentry/sentry/jobs/277442282
In our case, this is related to google-cloud-pubsub having a looser pin than google-gox which was introduced here: https://github.com/googleapis/gax-python/commit/7c018cf3b487a19d1c2ccf59610db15a72407e2a
So when installing just pip install google-cloud-pubsub in a new virtualenv, you still end up installing grpcio==1.6.0 since the parent allows it, but the subdependency then chokes. The change in gax-python also needs to be applied here and a new version tagged so they are in sync.
In the meantime, pinning grpcio in your project should work around it as grpcio==1.4.0

@mattrobenolt Your issue is not what I'm referring to here. RTD == Read The Docs and the umbrella issue I'm referring to is just a version bound conflict for videointelligence.
Thanks for reporting though.

But it's the same root cause:

Installed /home/docs/checkouts/readthedocs.org/user_builds/google-cloud-python/envs/latest/lib/python3.5/site-packages/google_cloud_language-0.28.0-py3.5.egg
error: grpcio 1.6.0 is installed but grpcio<1.6dev,>=1.0.2 is required by {'google-gax'}

Just replace google-cloud-pubsub with google-cloud-language. They are both a problem because the each are more liberal with their dependencies than google-gox.
And both of these libraries are within the same repository here.

Ahh I see that now. Cheers!

We're really suffering from this issue as well when trying to use google-cloud-pubsub.
We use setuptools to resolve dependencies and it fails fast on grpcio 1.6.0 is installed but..., breaking all our builds.
Is there a workaround for this?

Just pin grpcio like https://github.com/getsentry/sentry/commit/8fd26a305a6844b9a6425038c65728e43916c100

Cheers @mattrobenolt

Thanks for the report!  Current builds are passing, but please feel free to reopen if they break again.

@tseaver They are not passing. I manually "wipe" our build cache for about 1/3 of our builds. There is some strange issue where pip installs an egg rather than a wheel for some packages listed in requirements.txt.

looks like an old issue since it references gax.  Closing.  Reopen or file a new issue if necessary.

The (latest build](https://readthedocs.org/projects/google-cloud/builds/) is currently passing, but has only been doing so for the past week.

I did some tweaks to it in the last few weeks.


[request] Add helper to bundle multiple views

It would be great to be able to do something like:
ArticlesController.bundle(["edit", "update", "index"], function(){
  // Actions shared by all three
});

This could just be a helper method that does exactly the same thing as calling them individually.

+1
Is this being worked on?

@Undistraction @fahdoo Before callbacks are now implemented on 5.0.0.


UI enhancements

Would it be possible to speed up the animation speed when changing screens and add a black theme for OLED screens?

Hello! Thank you for using the app and for the feedback.
Yes, I will look into this.

The animations are much improved now in the new versions. At some point in the near future I will add support for changing the theme colors.

The lastest 3.5.0 now includes android monet dynamic theming as well as light/dark mode support. I'll keep this open for now as I think the nav animations could still use some work.


Finish style on Column 3

Ryan is adding Wiki api to generate pic. Shel change RED & BLUE and style as needed.


Master secret can't be used to create a compatible mnemonic seed

I used my Trezor to create a 2 out of 3 Shamir, here are 2 of parts:

admit upstairs academic acid disease frequent medal rich domain receiver fragment extra maiden airline triumph airline hormone lips research briefing
admit upstairs academic agency crunch provide spew overall voice exotic smell umbrella dining climate activity club editor item hesitate mustang

2 I used the shamir recover to acquire the master secret, which resulted in the following master secret
0db59247dfa48d61e5809337423e5371

I used various services including https://github.com/trezor/python-mnemonic, with this code:

from mnemonic import Mnemonic
mnemo = Mnemonic("english")

data = bytes.fromhex("0db59247dfa48d61e5809337423e5371")
mnemo.to_mnemonic(data)

Which resulted in this mnemonic seed

asset prosper music satisfy emotion rail noodle bar dance ball network tiny

When restoring the Trezor with the seed list it doesn't result in the same wallet. There's a chance I'm using the master secret incorrectly, in which case I would appreciate if you could guide me.
Thanks,
Noam.

You can't convert shamir mnemonics to mnemonics and vice versa.
See https://github.com/satoshilabs/slips/blob/master/slip-0039.md#Bip39Compatibility for rationale


Update Provider.checkFileUrl to allow non-URL file types (like Arweave)

Required by https://github.com/oceanprotocol/market/pull/1732
Changes proposed in this PR:

Update Provider.checkFileUrl API to take UrlFile | Arweave objects instead of a string which assumes type url.
Update Provider test

Hello @alexcos20 @kremalicious @mihaisc @bogdanfazakas
I believe this PR is ready for your review.
CC @AnaLoznianu

@kremalicious Thank you so much for your review!  I have gone through and tried to address all of your concerns and I believe this PR is ready again for review.
CC @mihaisc @bogdanfazakas @AnaLoznianu

Thanks, @kremalicious for the quick 2nd review!  I am unable to merge this myself because I do not have write access to the repo, so I invite you or any other core team member to merge it on my behalf.


Assistance Needed with Deploying Ranger alongside Deputy and SDL Parser

Hello,
I hope this message finds you well. I'm currently working on deploying Ranger from the Open Cyber Range GitHub repository and integrating it with an existing setup of Deputy and SDL Parser, which are running smoothly.
Despite my efforts, I'm facing challenges in setting up Ranger correctly to ensure it integrates seamlessly with Deputy and SDL Parser.
Could someone please provide a detailed guide or steps to follow for deploying Ranger alongside an existing setup of Deputy and SDL Parser? It would be immensely helpful to receive an overview of the necessary configurations or dependencies required for successful integration.
Thank you very much for your assistance!
Best regards,
Muhammad Faizan Arshad

For a working solution to deploy and start exercises, a working connection to a vCenter virtualization environment needs to exist.
In essence, the exercises are deployed on vCenter via the different ranger-vmware-* containers that you need to start using docker compose.
For the VMWare handlers, I can send you example configuration files as a quick help. The example files and a reference in the README will be added to the repository in the future.

Thank you! But again, there's a heavy confusion about how do I have a vCenter virtualization environment, are there again any suggestions about how to start the ranger from scratch? Even mere steps would help a lot!


wappalyzer detects 'Plentymarkets' and apps.json has 'plentymarkets'

Notice the difference in upper and lower cases :/
These should be the exact same.

The regular expressions are case insensitive so that's expected. Is that not right?

It would be much easier to have both the detected names as well as the source-of-truth, i.e, apps.json have precisely the same technology names everywhere.
If it's done this way, then a simple equality check would do in our applications, SQL queries etc. Even though lowerCase()-ing and regex-ing would work, equality checks are always better (and more performant when used in SQL queries searching among millions of rows!)

Ah, I see what you mean. apps.json is the source of truth, where are you seeing the upper-case version?
It looks like the casing was corrected recently:
https://github.com/AliasIO/wappalyzer/pull/3149/files#diff-557d7d60c04092a680ce152f367618c0L8496

Yes, exactly.
So, I've been storing responses as is in my database. I encountered this when I updated apps.json, wappalyzer scraper. After updating, some of the rows showed "plentymarkets" while others "Plentymarkets" - which, since the name is used as primary key, it started showing up as two different things.
So, ideally, the names should be treated as immutable from the beginning, i.e, once set they should never be renamed. This would be the ideal scenario, where we take greater care to ensure that when some new tech is added to wappalyzer, we're 100% sure the name is accurate, so future fixes are avoided/minimized. When the underlying tech name itself changes, then we don't have much choice, but we can still try to keep names consistent everywhere.


No data state

This adds a no data state to help indicate that a source provider has been created, but we're still processing their data. This makes use of the new current_month_data flag now available from the sources API response.
https://issues.redhat.com/browse/COST-914
Details pages

Overview tabs

Codecov Report

Merging #1817 (cd0531c) into master (32f96ae) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1817   +/-   ##
=======================================
  Coverage   72.59%   72.59%
=======================================
  Files         325      325
  Lines        6188     6188
  Branches     1448     1448
=======================================
  Hits         4492     4492
  Misses       1678     1678
  Partials       18       18

Impacted Files
Coverage Δ

src/api/providers.ts
84.61% <ø> (ø)

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 7a3d05e...cd0531c. Read the comment docs.

Codecov Report

Merging #1817 (cd0531c) into master (32f96ae) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1817   +/-   ##
=======================================
  Coverage   72.59%   72.59%
=======================================
  Files         325      325
  Lines        6188     6188
  Branches     1448     1448
=======================================
  Hits         4492     4492
  Misses       1678     1678
  Partials       18       18

Impacted Files
Coverage Δ

src/api/providers.ts
84.61% <ø> (ø)

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 7a3d05e...cd0531c. Read the comment docs.


Error: resource already mapped in custom_all_reduce.cuh

This is a issue relared to https://github.com/vllm-project/vllm/issues/2619#issuecomment-1913490065
I have tried ray=2.9.1 with dev code in commit #2636
vllm.entrypoints.openai.api_server --model ./Mistral-7B-Instruct-v0.2-AWQ --quantization awq --dtype auto --host <IP_ADDRESS> --port 8081 --tensor-parallel-size 2
but I meet another error
Failed: Cuda error /home/my/vllm/csrc/custom_all_reduce.cuh:417 'resource already mapped' Segmentation fault (core dumped)
I am running with python=3.11, CUDA 12.1, driver 530 with 2x RTX 3090 NVLink.
When I rollback to commit #2622, the program works well. So it seems it is caused by #2192

Can you try if upgrading to driver 535 helps? I highly suspect this is a driver bug, as cudaIpcOpenMemHandle shouldn't return such an error and should returns an existing handle of the IPC address is already mapped.
If not, I can do a software fix as well.

@yippp Can you try if upgrading to driver 535 helps? I highly suspect this is a driver bug, as cudaIpcOpenMemHandle shouldn't return such an error and should return an existing handle of the IPC address is already mapped.
If not, I can do a software fix as well.

@hanzhi713 Thanks for your sugguestion, I have tried to update the driver to newest 535.154.05, unfortunalty the same error appears.
Failed: Cuda error /home/my/vllm/csrc/custom_all_reduce.cuh:417 'resource already mapped'
[W CudaIPCTypes.cpp:15] Producer process has been terminated before all shared CUDA tensors released. See Note [Sharing CUDA tensors]

@yippp
I've pushed a software fix: #2642. Please check it out.
Also, could you please check if you're using the latest version of cuda 12.1 (i.e. 12.1 update 1)? Because other maintainers and I don't have this issue on our platforms.

@yippp I've pushed a software fix: #2642. Please check it out.
Also, could you please check if you're using the latest version of cuda 12.1 (i.e. 12.1 update 1)? Because other maintainers and I don't have this issue on our platforms.

thanks for your work, I have tried to run the code with fix #2642. Unfortunalty, the same error appears. I also check the cuda version, it is 12.1 update 1. Maybe I need to further check what is the problem with my enviroment

@hanzhi713 Can we directly use cudaOpenIpcHandle (w/ dummy inputs) for checking whether the user env is suitable for the custom all reduce kernel?

@yippp Can you double check the line number when you got the error? It should be line 432 instead of 417. Just making sure you've compiled the code.
@WoosukKwon His case is weird, because the cudaIpcOpenMemHandle that errors out is the second invocation. The first invocation is in the constructor of CustomAllreduce.

@hanzhi713 I have doudle check it, I use the correct source code and definity the error is in line 417 of file csrc/custom_all_reduce.cuh, i.e. the line CUDACHECK(cudaIpcOpenMemHandle(
I tried to removal the entire enviroment and reinsall it from source, but still same error.
I notice that before the error show, the graphic memory usage increased to ~300MB, and then show the error, hope this limited info could help you.

@yippp No I mean once you use #2642, the error should be at line 432, not 417.

@hanzhi713 Thanks for your patient instruction! I have correctly build it, and the error disappear. I have checked it, currently the program could run normally.

@yippp Thanks for sharing! @hanzhi713 Thanks for the fix!


Mention the "Storybook for Server" documentation in the README

This will make it easier to contribute given that the official
Storybook documentation has no reference to it and it's hard to find.

Thanks for the contribution. We will update the Storybook docs to detail Server a little more clearly.


chore: bdd-gh-1587381005 to 0.0.1

chore: Promote bdd-gh-1587381005 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Add Action Bar Item to Playground Activities to Switch to Other Playground Activity

IOW, PictureActivity has an action bar item to launch VideoActivity and vice-versa. Put the right flags (reorder-to-front, probably) to only have one instance of each no matter how many switches are made.

Addressed in v0.3.2.


Include irreversibility of Trusted Launch

The irreversibility of Trusted Launch created VMs should be included in this section:
"Trusted launch requires the creation of new virtual machines. You can't enable trusted launch on existing virtual machines that were initially created without it."
=>Once a VM is created with Trusted Launch, Trusted Launch cannot be disabled, only the associated services (vTMP and Secure Boot).

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

ID: cedc5fc7-5576-f22f-9c9a-a2977ad93607
Version Independent ID: e7cda6ea-acd1-b56f-9d81-92d6fde31921
Content: Trusted launch for Azure VMs - Azure Virtual Machines
Content Source: articles/virtual-machines/trusted-launch.md
Service: virtual-machines
Sub-service: trusted-launch
GitHub Login: @lakmeedee
Microsoft Alias: dejv

@OrlandoAMunoz
Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.

@lakmeedee
Can you please check and add your comments on this doc update request as applicable.

Please use the following line instead:
Once a VM is created with Trusted Launch, Trusted Launch cannot be disabled. Only the associated capabilities such as vTPM and Secure Boot can be disabled individually.

@lakmeedee Thanks for responding back on this. I have created a PR for accommodating these changes. Please review and approve.

Thank you all!
Kind regards,
Orlando Muñoz
Support Engineer
Microsoft Azure Technical Support
Customer Service and Support
Office: +351 (21) 0491844
<EMAIL_ADDRESS>Working hours: Mon-Fri, 8:30 - 17:30 UTC+1
If you need to work with another Support Engineer outside of my working hours, please send email to<EMAIL_ADDRESS>with your case number, and availability.
We are always interested to hear your feedback. Please feel free to reach my manager regarding the level of service you have received, at<EMAIL_ADDRESS>[Microsoft Azure logo]
From: VikasPullagura-MSFT @.>
Sent: Tuesday, May 17, 2022 12:47
To: MicrosoftDocs/azure-docs @.>
Cc: Orlando Muñoz @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] Include irreversibility of Trusted Launch (Issue #92972)
@OrlandoAMunozhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FOrlandoAMunoz&data=05|01|orlando.munoz%40microsoft.com|5b13887e25ff41d9f49c08da37fae2f4|72f988bf86f141af91ab2d7cd011db47|1|0|637883848007945393|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=RBHPPrtcF3DbxmA5jYtLwYh2WdFT4Ixvbw8xYS8%2BG%2Bo%3D&reserved=0 The document is updated and changes will go live soon.

Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F92972%23issuecomment-1128768714&data=05|01|orlando.munoz%40microsoft.com|5b13887e25ff41d9f49c08da37fae2f4|72f988bf86f141af91ab2d7cd011db47|1|0|637883848007945393|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=YFM8%2BEzB4v68G0w1UKrNr6GBSl5YM58rWmgztouKJus%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAXZ7IC5UGPLCCR22ASLFOFDVKOBJPANCNFSM5WEMDV2A&data=05|01|orlando.munoz%40microsoft.com|5b13887e25ff41d9f49c08da37fae2f4|72f988bf86f141af91ab2d7cd011db47|1|0|637883848007945393|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=HhGyoktzQa%2F0jiXp%2BszIMXFrIMpXUN4fFwb8jcSnUhc%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>


feat: Hero design simplification

PR Type

[ ] Bugfix
[x] Feature
[ ] Refactoring

Context

[x] Closes #10787

Needs Design check

@exezbcz please review

Did your issue had any of the "$" label on it?

[x] Fill up your DOT address: Payout

Screenshot 📸

[x] My fix has changed something on UI;

forgot to mention that the notification should be on the mobile as well

otherwise looking good!

forgot to mention that the notification should be on the mobile as well
otherwise looking good!

done :)

thanks!


RenderBox was not laid out: _RenderTheatre#8b6f6 NEEDS-LAYOUT NEEDS-PAINT NEEDS-COMPOSITING-BITS-UPDATE 'package:flutter/src/rendering/box.dart': Failed assertion: line 1927 pos 12: 'hasSize'

Flutter Doctor is a successful, but newly created project - Default Generation Code,  it fails
Performing hot reload...
Syncing files to device sdk gphone64 x86 64...

======== Exception caught by rendering library =====================================================
The following assertion was thrown during performResize():
The _RenderTheatre class does not implement "computeDryLayout".

If you are not writing your own RenderBox subclass, then this is not
your fault.
In either case, please report this assertion by filing a bug on GitHub:
  https://github.com/flutter/flutter/issues/new?template=2_bug.md

Hi @pink9330, Thanks for filing the issue. In order  to investigate further we would need a minimal and complete reproducible code along with the output of flutter doctor -v .
Thanks.